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

Runtime: change reconcile and parse errors to warns #3691

Merged
merged 6 commits into from
Dec 15, 2023

Conversation

begelundmuller
Copy link
Contributor

@begelundmuller begelundmuller commented Dec 13, 2023

This PR changes error logs that can occur during normal usage to warning level. Examples of such errors include reconcile errors and revoked Github access.

By emitting these logs at warning level instead of error level, we reserve the error level for internal errors that are worth alerting on.

@begelundmuller begelundmuller self-assigned this Dec 14, 2023
Base automatically changed from begelundmuller/refactor-conn-cache-open-close-background to main December 15, 2023 11:29
Copy link
Member

@k-anshul k-anshul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just confirming if these msgs will be available to the user in local mode. Else need to use logger.Named for some messages like Reconcile failed and Parser error ?

@begelundmuller
Copy link
Contributor Author

Just confirming if these msgs will be available to the user in local mode. Else need to use logger.Named for some messages like Reconcile failed and Parser error ?

Confirmed – these are still shown in the console

@begelundmuller begelundmuller merged commit 50b4e2b into main Dec 15, 2023
4 checks passed
@begelundmuller begelundmuller deleted the begelundmuller/change-expected-errs-to-warns branch December 15, 2023 11:47
mindspank pushed a commit that referenced this pull request Dec 18, 2023
* Runtime: Refactor conn cache to contain and detect hanging opens/closes

* Extract connection cache to pkg + use a singleflight

* Add tests

* Runtime: change reconcile and parse errors to warns
esevastyanov pushed a commit that referenced this pull request Dec 19, 2023
* Runtime: Refactor conn cache to contain and detect hanging opens/closes

* Extract connection cache to pkg + use a singleflight

* Add tests

* Runtime: change reconcile and parse errors to warns

(cherry picked from commit 50b4e2b)
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