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.

Sign up for GitHub

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

Jump to bottom

Ignore dynaconf stderr and install gettext #174

Merged

jctanner merged 2 commits into pulp:main from jctanner:SILENCE_DYNACONF_STDERR

Oct 10, 2024

Contributor

jctanner commented Oct 10, 2024 •

edited

Loading

These two issues are causing all galaxy oci-env based CI to fail polling checks.

One of our dependencies is emitting error messages to stderr when dynaconf runs, and the oci script is capturing that within the values it it trying to get. This shows up when the stack tries to get the API_ROOT from dynaconf and includes a bunch of errors in the result, then the poll fails because it's adding a bunch of junk to the url.
When the stack comes up, it tries to build translations which are dependant on gettext

jctanner added 2 commits

October 10, 2024 10:29


          Ignore dynaconf stderr when getting values for settings.

4d28285

No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>


          Translations require gettext.

483ad07

No-Issue

Signed-off-by: James Tanner <tanner.jc@gmail.com>

jctanner requested review from gerrod3, rochacbruno and mdellweg

October 10, 2024 14:57

gerrod3 approved these changes

View reviewed changes

jctanner merged commit 1034c81 into pulp:main

2 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet