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 to Vector's HPA #336

Merged
merged 3 commits into from
Nov 14, 2023

Conversation

namm2
Copy link
Contributor

@namm2 namm2 commented Oct 13, 2023

Allowing to add annotations to the Vector's HPA, this is needed when someone want to use kube-metrics-adapter

Copy link
Contributor

@dsmith3197 dsmith3197 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. Can you also update charts/vector/README.md (see lint failure)?

@dsmith3197
Copy link
Contributor

Hi @namm2, could you merge main? One of the CI jobs is broken on this branch but fixed on main. Thank you!

@dsmith3197 dsmith3197 enabled auto-merge (squash) November 14, 2023 13:00
@dsmith3197 dsmith3197 merged commit 589599a into vectordotdev:develop Nov 14, 2023
8 of 9 checks passed
@namm2 namm2 deleted the charts-hpa-annotations branch November 15, 2023 03:05
@jszwedko
Copy link
Member

I think this PR should have included:

  • A CHANGELOG entry
  • A bump to the chart version

@jszwedko
Copy link
Member

Ah, I see CI didn't flag these lacks because the PR title was not using the semantic PR format (enhancement(vector): ...). We should be enforcing that. I opened #344 to track.

jszwedko pushed a commit that referenced this pull request Nov 16, 2023
…notations on Vector's HPA (#336)

* Add annotations to Vector's HPA

* Update README.md

Signed-off-by: Jesse Szwedko <[email protected]>
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.

3 participants