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.

Sign up for GitHub

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

Jump to bottom

Fail the sync job when charts fail to sync and Alert #289

Open
sudermanjr opened this issue May 1, 2020 · 1 comment
Open

Fail the sync job when charts fail to sync and Alert #289

sudermanjr opened this issue May 1, 2020 · 1 comment
Assignees
@sudermanjr
Labels
automation Pertains to the CI/CD process surrounding this repo, not a particular chart. enhancement Adding additional functionality or improvements good first issue Good for newcomers pinned Prevents stalebot from removing priority: should Should be addressed shortly

Comments

@sudermanjr
Copy link
Member

When the sync job fails, the job passes. This less than good.

We should also setup an alert when master fails to sync
@stale
Copy link

stale bot commented Jul 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Marked as stale by stalebot label Jul 3, 2020
@sudermanjr sudermanjr added pinned Prevents stalebot from removing and removed stale Marked as stale by stalebot labels Jul 6, 2020
@sudermanjr sudermanjr added the good first issue Good for newcomers label Oct 6, 2020
@sudermanjr sudermanjr added the automation Pertains to the CI/CD process surrounding this repo, not a particular chart. label Oct 13, 2020
@sudermanjr sudermanjr added this to the Q4 2020 milestone Oct 13, 2020
@sudermanjr sudermanjr added the priority: should Should be addressed shortly label Oct 13, 2020
@sudermanjr sudermanjr self-assigned this Jan 13, 2021
@sudermanjr sudermanjr modified the milestones: Q4 2020, Q1 2021 Jan 13, 2021
@sudermanjr sudermanjr removed this from the Q2 2021 milestone Oct 27, 2021
@sudermanjr sudermanjr mentioned this issue Nov 29, 2021
Closed
@sudermanjr sudermanjr changed the title Fail the sync job when charts fail to sync Fail the sync job when charts fail to sync and Alert Dec 9, 2021
@sudermanjr sudermanjr added the enhancement Adding additional functionality or improvements label May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sudermanjr sudermanjr

Labels
automation Pertains to the CI/CD process surrounding this repo, not a particular chart. enhancement Adding additional functionality or improvements good first issue Good for newcomers pinned Prevents stalebot from removing priority: should Should be addressed shortly
Projects
Fairwinds Open Source
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sudermanjr