Project

General

Profile

Actions
Copy link

Bug #13772

closed

The orchestration code uses SQL logger

Added by Ivan Necas over 8 years ago. Updated about 6 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
 #1

Updated by Lukas Zapletal over 8 years ago

  • Related to Bug #14980: Unused_ip AJAX request fails: undefined method `from' for #<IPAM::Dhcp..> added
Actions
Copy link
 #2

Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Lukas Zapletal
  • Pull request https://github.com/theforeman/foreman/pull/3729 added
Actions
Copy link
 #3

Updated by Daniel Lobato Garcia about 8 years ago

  • Target version set to 1.6.2
Actions
Copy link
 #4

Updated by Daniel Lobato Garcia about 8 years ago

  • Target version changed from 1.6.2 to 1.5.2
Actions
Copy link
 #5

Updated by Lukas Zapletal over 7 years ago

  • Related to Refactor #19002: Use ApplicationRecord instead of ActiveRecord::Base added
Actions
Copy link
 #6

Updated by Lukas Zapletal over 7 years ago

  • Related to Refactor #19003: Use ApplicationRecord instead of ActiveRecord::Base added
Actions
Copy link
 #7

Updated by Lukas Zapletal over 7 years ago

Actions
Copy link
 #8

Updated by Lukas Zapletal over 7 years ago

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

Actions
Copy link
 #9

Updated by Dominic Cleal over 7 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
 #10

Updated by Lukas Zapletal over 7 years ago

Actions
Copy link
 #11

Updated by Lukas Zapletal over 7 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 over 7 years ago

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

Updated by The Foreman Bot over 7 years ago

  • Pull request https://github.com/theforeman/foreman/pull/4454 added
Actions
Copy link
 #14

Updated by Ivan Necas over 7 years ago

  • Related to Bug #19395: Failure when migrating database with foreman_remote_executino from scratch added
Actions
Copy link
 #15

Updated by Marek Hulán over 7 years ago

  • Translation missing: en.field_release set to 240
Actions
Copy link

Also available in: Atom PDF