Bug #34902
closedUn-break Katello after Foreman settings change
Description
apparently when https://github.com/theforeman/foreman/pull/9050 is merged, this will break Katello for reasons I don't fully understand yet
we need to test Katello and figure out how to un-break it
Ondrej Ezr, 6:32 AM
hi! 🙂I've got bad news, PR that is about to be merged will break Katello _ https://github.com/theforeman/foreman/pull/9050
It happened to foreman as well and it was fixed in https://github.com/theforeman/foreman/pull/9155
so the root cause is that some weird AR subclass is initialized before some well behaved class (the well behaved first class was always Setting, but it will no longer be true)
Ondrej Ezr, 10:12 AM
so what I did was put pp caller on top of <foreman>/app/models/application_record.rb, figured that first time it gets required is from Facet, so I've postponed facet registration and all worked well again
so my educated guess would be you will need to move https://github.com/Katello/katello/blob/2a10b1eb15f0dab79630aeba832b67b8ef043767/lib/katello/plugin.rb#L291 into a [in_to_prepare](https://github.com/theforeman/foreman/blob/8e0e3f78fb69789cacc29888bc121a541134269c/app/registries/foreman/plugin.rb#L481)
Updated by Jeremy Lenz over 2 years ago
- Status changed from New to Rejected
turns out it was a foreman-tasks issue
Updated by Samir Jha over 2 years ago
- Target version changed from Katello 4.5.0 to Katello Recycle Bin
Moving to recycle bin as status is rejected.