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

moved logs connected to unknown undeployable to Devel #631

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

justonedev1
Copy link
Collaborator

I talked with Vasco and it is possible to release and deploy before HI if we do just log changes. So I went over logs from OCTRL-777 logs and moved these to Devel level so crew will not get overwhelmed when this error (which is covered by retry) is triggered.

For the record: this PR is not meant to be as a whole solution for OCTRL-945, I am still working on more concise logging.

@justonedev1 justonedev1 requested review from teo and knopers8 October 29, 2024 08:53
@knopers8
Copy link
Collaborator

If we move those to DEVEL, are we left with some high-level error for an operator?

teo
teo previously approved these changes Oct 29, 2024
@justonedev1
Copy link
Collaborator Author

justonedev1 commented Oct 30, 2024

I added more logs to Devel, and one general error that shift crew is going to see only when whole retry mechanism fails.

@justonedev1 justonedev1 requested a review from teo October 30, 2024 13:12
@justonedev1 justonedev1 merged commit 18adda2 into master Oct 30, 2024
2 checks passed
@justonedev1 justonedev1 deleted the error_moving branch October 30, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants