Skip to content

Update 24-metrics-apiservice using RFC 1035 labels.yaml#288

Open
sheperdsonbrown wants to merge 1 commit intokedacore:mainfrom
sheperdsonbrown:patch-1
Open

Update 24-metrics-apiservice using RFC 1035 labels.yaml#288
sheperdsonbrown wants to merge 1 commit intokedacore:mainfrom
sheperdsonbrown:patch-1

Conversation

@sheperdsonbrown
Copy link
Copy Markdown

@sheperdsonbrown sheperdsonbrown commented Jul 20, 2022

This is against the RFC 1035 Label Names Kubernetes specification to have dot notation.

https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#rfc-1035-label-names

Signed-off-by: sheperdsonbrown sheperdsonbrown@users.noreply.github.com

Provide a description of what has been changed

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)
  • README is updated with new configuration values (if applicable)

Fixes #

This is against the RFC 1035 Label Names Kubernetes specification to have dot notation. 

Signed-off-by: sheperdsonbrown <sheperdsonbrown@users.noreply.github.com>
@sheperdsonbrown sheperdsonbrown requested a review from a team as a code owner July 20, 2022 23:09
@tomkerkhove
Copy link
Copy Markdown
Member

Any thoughts on this PR @zroubalik @JorTurFer ?

@JorTurFer
Copy link
Copy Markdown
Member

I'd keep it as it is, why do you need this change?

@tomkerkhove
Copy link
Copy Markdown
Member

tomkerkhove commented Jan 6, 2023

rfc1035 - I don't have an opinion.

@zroubalik ?

wozniakjan pushed a commit to wozniakjan/keda-charts that referenced this pull request Nov 6, 2025
…ssary for post/pre install hooks (kedacore#288)

Forgot to add these annotations to let helm delete the resources necessary for post/pre install hooks.

Signed-off-by: Jirka Kremser <jiri.kremser@gmail.com>
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