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

feat: Ignore specific errors for given pods #39

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nikup
Copy link

@nikup nikup commented Nov 13, 2023

Add support for ignoring a specific restart error for a specific type of pod.
Example: We have a gke-metrics-agent that sometimes can't start because of Failed to run the service: failed to start extensions: listen tcp 127.0.0.1:8203: bind: address already in use. With the current implementation we can ignore that by adding

ignoredErrorsForPodNamePrefixes = "{ "gke-metrics-agent": [ "address already in use" ]}"

@nikup nikup requested a review from a team as a code owner November 13, 2023 12:12
@ventsislav-georgiev
Copy link

@able8 any chance of getting this in?

helpers.go Outdated Show resolved Hide resolved
helpers.go Outdated Show resolved Hide resolved
@able8
Copy link
Contributor

able8 commented Dec 6, 2023

Hi @nikup @ventsislav-georgiev, thank you for contributing. I left 2 comments.
Also, the Contributor License Agreement (CLA) is required to merge PR.
Refer to https://github.com/airwallex/k8s-pod-restart-info-collector/blob/master/CONTRIBUTION.md

@nikup nikup force-pushed the feat-support-ignored-error-for-pod-prefix branch from 47ce53e to 76c605e Compare December 29, 2023 13:24
@nikup
Copy link
Author

nikup commented Oct 3, 2024

@able8 Comments were addressed long time ago, but I just send a CLA via email - if you can take a look and merge this before it turns a year old :D

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.

3 participants