Project

General

Custom queries

Profile

Actions
Copy link

Bug #13772

closed

The orchestration code uses SQL logger

Added by Ivan Necas about 9 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Lukas Zapletal
Category:
Orchestration
Target version:
1.16.0
Difficulty:
Triaged:
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/3729, https://github.com/theforeman/foreman/pull/4454
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

It should have it's own logger. Ideally with using the hierarchy system,
that the Logging library provides, https://github.com/TwP/logging/blob/master/examples/hierarchies.rb
so that one could extract logging for the whole orchestration, or a part of it.

Related issues 4 (0 open — 4 closed)

Related to Foreman - Bug #14980: Unused_ip AJAX request fails: undefined method `from' for #<IPAM::Dhcp..>ClosedLukas Zapletal05/10/2016Actions
Related to Katello - Refactor #19002: Use ApplicationRecord instead of ActiveRecord::BaseClosedLukas Zapletal03/23/2017Actions
Related to Discovery - Refactor #19003: Use ApplicationRecord instead of ActiveRecord::BaseClosedLukas Zapletal03/23/2017Actions
Related to Foreman Remote Execution - Bug #19395: Failure when migrating database with foreman_remote_executino from scratchClosedIvan Necas04/26/2017Actions
Actions
Copy link
 #8

Updated by Lukas Zapletal about 8 years ago

The implementation uses ApplicationRecord - new feature of Rails 5. Associated tracking ticket.

Actions
Copy link
 #9

Updated by Dominic Cleal about 8 years ago

Lukas Zapletal wrote:

The implementation uses ApplicationRecord - new feature of Rails 5. Associated tracking ticket.

An abstract base class is nothing to do with Rails 5!

Actions
Copy link
 #11

Updated by Lukas Zapletal about 8 years ago

Well it's now recommended by Rails 5, but if this makes you to type bangs, then I will unlink it. No biggie.

Actions
Copy link
 #12

Updated by Lukas Zapletal about 8 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions
Copy link

Also available in: Atom PDF