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

Allowed labels for cronjobs, jobs, and changelog. #343

Closed
wants to merge 1 commit into from

Conversation

Strigix
Copy link

@Strigix Strigix commented Oct 31, 2023

@Strigix Strigix requested a review from a team as a code owner October 31, 2023 16:01
@Strigix Strigix enabled auto-merge (squash) October 31, 2023 16:06
@QuentinBisson
Copy link
Contributor

Hey @Strigix have you tested it ?

@Strigix
Copy link
Author

Strigix commented Oct 31, 2023

Not yet.
To be honest, I wasn't sure how.
The newest observe bundle hasn't got this version yet. Can I just edit the version of prometheus-operator-app?
I was unsure because it was pushed to the catalog as kube-prometheus-stack-app.
What would work best do you reckon?

@QuentinBisson
Copy link
Contributor

Well i'm in a pickle. Is that for MCs or WCs as well?

@Strigix
Copy link
Author

Strigix commented Nov 1, 2023

Oh, that was quite the late answer.
I believe this is mostly for MC's.

@QuentinBisson
Copy link
Contributor

QuentinBisson commented Nov 1, 2023

Then I would suggest you add it here https://github.com/giantswarm/config/blob/main/default/apps/observability-bundle/configmap-values.yaml.template instead like https://github.com/giantswarm/config/pull/2047 but beware this was not tested, you might want to test this config

@QuentinBisson
Copy link
Contributor

Maybe we still need this here :)

@Strigix Strigix closed this Nov 2, 2023
auto-merge was automatically disabled November 2, 2023 13:54

Pull request was closed

@Strigix Strigix deleted the add-cronjobs-jobs-labels branch November 2, 2023 13:55
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