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

rescoped prometheus secrets and refactor manifest generation to use makefile #3424

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

jackjoy-gds
Copy link
Contributor

@jackjoy-gds jackjoy-gds commented Sep 29, 2023

What

rescoped prometheus secrets to prep the use of the new makefile manifest generation
refactored pipeline manifest generation to use makefile

How to review

👀
run in a dev env

🚨⚠️ Please do not merge this pull request via the GitHub UI ⚠️🚨

@jackjoy-gds jackjoy-gds force-pushed the 185716198_rescope-prometheus-secrets-for-bosh branch 2 times, most recently from 5ac6880 to e385d3b Compare September 29, 2023 12:13
@jackjoy-gds jackjoy-gds changed the title rescoped prometheus secrets rescoped prometheus secrets and refactor manifest generation to use makefile Oct 3, 2023
@jackjoy-gds jackjoy-gds force-pushed the 185716198_rescope-prometheus-secrets-for-bosh branch 7 times, most recently from 26da167 to 109a0e4 Compare October 5, 2023 16:09
Copy link
Contributor

@dark5un dark5un left a comment

Choose a reason for hiding this comment

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

👍 also verified in a dev env

@jackjoy-gds jackjoy-gds force-pushed the 185716198_rescope-prometheus-secrets-for-bosh branch from 109a0e4 to 473625f Compare November 24, 2023 13:51
@jackjoy-gds jackjoy-gds merged commit 96ac87c into main Nov 27, 2023
8 checks passed
@jackjoy-gds jackjoy-gds deleted the 185716198_rescope-prometheus-secrets-for-bosh branch November 27, 2023 08:18
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