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

fix: suppress error about unable to obtain node #12020

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

mikutas
Copy link
Contributor

@mikutas mikutas commented Oct 17, 2023

Relates to #11451

Motivation

After upgrading to v3.4.11, the error log is often reported from monitoring.

level=error msg="was unable to obtain node for " namespace=default workflow=hello-world-jq82w

Modifications

When boundaryID is empty, we don't try to get node and also don't raise any error.

Verification

@mikutas mikutas force-pushed the suppress-invalid-error-log branch 2 times, most recently from bb9455f to efdac5b Compare October 18, 2023 04:21
@mikutas mikutas marked this pull request as ready for review October 18, 2023 04:54
Signed-off-by: mikutas <23391543+mikutas@users.noreply.github.com>
@terrytangyuan terrytangyuan enabled auto-merge (squash) October 19, 2023 13:54
@mikutas
Copy link
Contributor Author

mikutas commented Oct 19, 2023

@terrytangyuan Thank you for reviewing! Please retry required Docs/docs job because it seems to be a temporary failure ((sorry I don't have the permission to retry)). 🙏
image

@terrytangyuan terrytangyuan merged commit e5d8c53 into argoproj:master Oct 19, 2023
27 of 28 checks passed
terrytangyuan pushed a commit that referenced this pull request Oct 19, 2023
Signed-off-by: mikutas <23391543+mikutas@users.noreply.github.com>
@agilgur5 agilgur5 added the area/controller Controller issues, panics label Oct 19, 2023
@mikutas mikutas deleted the suppress-invalid-error-log branch October 19, 2023 23:21
terrytangyuan pushed a commit that referenced this pull request Nov 3, 2023
Signed-off-by: mikutas <23391543+mikutas@users.noreply.github.com>
dpadhiar pushed a commit to dpadhiar/argo-workflows that referenced this pull request May 9, 2024
Signed-off-by: mikutas <23391543+mikutas@users.noreply.github.com>
Signed-off-by: Dillen Padhiar <dillen_padhiar@intuit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/controller Controller issues, panics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants