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: update manifests with new prom image and delete flag #947

Merged
merged 1 commit into from
May 24, 2024

Conversation

bwplotka
Copy link
Collaborator

Wanted clean state here and what I can import into latest manifest release.

Copy link
Member

@TheSpiritXIII TheSpiritXIII left a comment

Choose a reason for hiding this comment

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

Looks good. Just see my comment about the extra labels being added by helm!

manifests/operator.yaml Outdated Show resolved Hide resolved
manifests/operator.yaml Outdated Show resolved Hide resolved
@gmp-bot gmp-bot force-pushed the release-please--branches--main branch 3 times, most recently from 8884df5 to adc7e38 Compare April 29, 2024 11:52
@TheSpiritXIII
Copy link
Member

Small nit: can we create a proper release/0.12 branch and push into that instead of release-please? I see release-please rebasing the branch and updating this PR. I'm not sure if it would keep changes such as these. Thanks!

@bwplotka bwplotka changed the base branch from release-please--branches--main to main April 29, 2024 13:04
@bwplotka bwplotka force-pushed the 0.12-go branch 2 times, most recently from a870d71 to 2b3151f Compare April 29, 2024 13:59
@bwplotka
Copy link
Collaborator Author

Have to park it until we have Prometheus image on prod.

@bwplotka bwplotka marked this pull request as draft April 29, 2024 14:00
@bwplotka bwplotka marked this pull request as ready for review May 23, 2024 10:51
@bwplotka
Copy link
Collaborator Author

bwplotka commented May 23, 2024

Updated, using SHA for now, as we can't copy tags.

Signed-off-by: bwplotka <bwplotka@google.com>
@TheSpiritXIII TheSpiritXIII merged commit 6556940 into main May 24, 2024
27 checks passed
@TheSpiritXIII TheSpiritXIII deleted the 0.12-go branch May 24, 2024 15:24
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.

None yet

3 participants