Project

General

Profile

Bug #34902

Updated by Jeremy Lenz almost 2 years ago

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) 

Back