New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #11294: Prevent top level constant warning from Logging. #5370
Conversation
As with most intermittent failures, this is a hard one to recreate and I am speculating the fix as this didn't happen prior to logging and if oyu look at the error output the warning appears before the failure. |
It also happens in passing tests, e.g. http://ci.theforeman.org/job/test_katello_core/14564/database=postgresql,ruby=1.9.3/console, and happened much earlier in one of the failing tests linked in the ticket (http://ci.theforeman.org/job/test_katello_core/14563/database=postgresql,ruby=1.9.3/console). |
I don't see the referenced errors in the logs for this test, but there's other references to Foreman::Logging, even in this file. Should we make them all explicitly at the top level? To avoid this happening again in some other new test. |
Wondering if I should make this change under the guise of another issue targeted specifically at this, and over time see if the issue remains and revisit. I cannot seem to recreate it locally, and I have theories but no solid solutions yet of how to potentially make the fixture loading more robust. |
Is the Logging warning causing the fixture loading error? Or is it something else? Just had a failure with it: |
That's what I am getting at - I am not sure the cause, but I'd like to at least rule this out (and its a worth while fix as this warning is annoying) |
Seems fine to me, I was just saying what about other instances of Foreman::Logging? I do see in my failure it's only the line you changed triggering it, but might as well be consistent with all the references to it. |
The warning "warning: toplevel constant Logging referenced by Katello::Foreman::Logging" appears many times throughout test runs.
I created a new issue targeted strictly at fixing the Logging warnings and updates this PR to point to that for now. I'd like to get this fix in and then monitor to see if the intermittent failure continues on Jenkins. |
APJ |
Fixes #11294: Prevent top level constant warning from Logging.
The warning "warning: toplevel constant Logging referenced by
Katello::Foreman::Logging" appears many times throughout test runs.
Further it may be the cause of intermittent failures on Jenkins that
seem to result from the customized fixture setup being unset with
respect to referencing the Foreman fixtures.