{"push_id":778726539,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"74c75c7af40a718a891cbe5f5adcf7582c709f28","before":"00e97867ddde17b7647e811fb01e946ed3071d01","commits":[{"sha":"d4ced59ee99063cee244dfe3e81af98c97acd9e8","author":{"email":"f131bf03abf9995c2a0d18ad727a0a4241b4f24e@redhat.com","name":"Greg Blomquist"},"message":"[Namespacing] Ensure that workers require runners\n\nIf workers do not `require_dependency` their runners, the\n`const_defined?(:Runner, false)` call in `corresponding_runner` will fail to\nfind the appropriate runner class when starting up the worker.\n\nThis results in finding the \"legacy\" worker runner.  For instace, with the\nlegacy MiqEventCatcherOpenstack, the corresponding runner used to be\n'workers/event_catcher_openstack'.  However, after the namespacing of OpenStack\nCloudManager, the worker is\nManageIQ::Providers::Openstack::CloudManager::EventCatcher and the corresponding\nrunner is ManageIQ::Providers::Openstack::CloudManager::EventCatcher::Runner.\n\nWhen `corresponding_runner` fails to find the appropriately namespaced runner\nclass, it throws errors like:\n\n> (LoadError) cannot load such file -- workers/event_catcher_openstack on adding\n> openstack provider\n\n(Upstream only)\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1253134","distinct":true,"url":"https://api.github.com/repos/ManageIQ/manageiq/commits/d4ced59ee99063cee244dfe3e81af98c97acd9e8"},{"sha":"25c6dd53564a0ea528dc0df04ebdd31c47758bbb","author":{"email":"f131bf03abf9995c2a0d18ad727a0a4241b4f24e@redhat.com","name":"Greg Blomquist"},"message":"[Namespacing] Fix namespaced EventCatcherMixin\n\nThe EventCatcherMixin for Openstack was located at:\n\n`manageiq/providers/openstack/event_catcher_mixin.rb`\n\nBut, the namespaced classname was:\n\n`ManageIQ::Providers::Openstack::CloudManager::EventCatcherMixin`\n\nI'm guessing that this may have been changed after the OpenStack Infra Manager\nwas namespaced.\n\n(Upstream only)\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1253134","distinct":true,"url":"https://api.github.com/repos/ManageIQ/manageiq/commits/25c6dd53564a0ea528dc0df04ebdd31c47758bbb"},{"sha":"74c75c7af40a718a891cbe5f5adcf7582c709f28","author":{"email":"3a27c03342c611f9be074564116603701f744c9e@gmail.com","name":"Oleg Barenboim"},"message":"Merge pull request #4052 from blomquisg/bz1253134-missing-required-dependency\n\n[Provider Namespacing] Ensure that workers require runners","distinct":true,"url":"https://api.github.com/repos/ManageIQ/manageiq/commits/74c75c7af40a718a891cbe5f5adcf7582c709f28"}]}