Skip to content
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

tests: increase wfe log level #7982

Merged
merged 1 commit into from
Jan 27, 2025
Merged

tests: increase wfe log level #7982

merged 1 commit into from
Jan 27, 2025

Conversation

jsha
Copy link
Contributor

@jsha jsha commented Jan 27, 2025

We've been seeing some flaky integration tests where issuance fails. The integration test only has access to the generic user-facing error. The real error is available as InternalError in the WFE logs, but we need a higher log level to see it.

We've been seeing some flaky integration tests where issuance fails.
The integration test only has access to the generic user-facing
error. The real error is available as `InternalError` in the WFE
logs, but we need a higher log level to see it.
@jsha jsha requested a review from a team as a code owner January 27, 2025 19:13
@jsha jsha requested a review from aarongable January 27, 2025 19:13
Copy link
Contributor

@jsha, this PR appears to contain configuration and/or SQL schema changes. Please ensure that a corresponding deployment ticket has been filed with the new values.

1 similar comment
Copy link
Contributor

@jsha, this PR appears to contain configuration and/or SQL schema changes. Please ensure that a corresponding deployment ticket has been filed with the new values.

@jsha
Copy link
Contributor Author

jsha commented Jan 27, 2025

Hi bot! The config changes are only relevant to test. Thanks!

@aarongable aarongable merged commit 55b8cbe into main Jan 27, 2025
14 checks passed
@aarongable aarongable deleted the wfe-log-level-increase branch January 27, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants