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

add annotations + podAnnotations #212

Merged
merged 5 commits into from
Aug 20, 2024
Merged

add annotations + podAnnotations #212

merged 5 commits into from
Aug 20, 2024

Conversation

b17k0
Copy link
Contributor

@b17k0 b17k0 commented Jul 22, 2024

PR for #170
I also prefer to scrape via the prometheus.io/scrape: 'true' annotation

@b17k0
Copy link
Contributor Author

b17k0 commented Jul 22, 2024

@nabokihms

Copy link
Member

@nabokihms nabokihms left a comment

Choose a reason for hiding this comment

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

Thanks, @0xbilko. This is a great improvement. Can you bump the chart version? (it is required to automatically release a new version on artifacthub)

@b17k0
Copy link
Contributor Author

b17k0 commented Aug 3, 2024

@nabokihms, done

@nabokihms
Copy link
Member

nabokihms commented Aug 5, 2024

Doc is not up to date. Could you execute the make docs command as well?

@b17k0
Copy link
Contributor Author

b17k0 commented Aug 6, 2024

@nabokihms, done

b17k0 and others added 4 commits August 8, 2024 23:25
Signed-off-by: beliys <[email protected]>
Signed-off-by: bko <[email protected]>
Signed-off-by: Oleksander B <[email protected]>
Signed-off-by: bko <[email protected]>
Signed-off-by: bko <[email protected]>
Signed-off-by: bko <[email protected]>
Signed-off-by: Maksim Nabokikh <[email protected]>
@nabokihms nabokihms merged commit 67fa1c9 into deckhouse:master Aug 20, 2024
8 checks passed
@nabokihms
Copy link
Member

@0xbilko thanks!

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