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.

Sign up for GitHub

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

Jump to bottom

Add podAnnotations to Helm templates for injecting user-specified pod. #1589

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add podAnnotations to Helm templates for injecting user-specified pod. #1589

wants to merge 1 commit into from
+53 −0

Conversation

oniku-2929
Copy link

What this PR does / Why we need it:
#1588

Which issue(s) this PR fixes:

Closes #1588

Special notes for your reviewer:
values.yaml example.

swaggerui:
  podAnnotations:
    test.sh/component: swaggerui
query:
  podAnnotations:
    test.sh/component: query
frontend:
  podAnnotations:
    test.sh/component: frontend
backend:
  podAnnotations:
    test.sh/component: backend
synchronizer:
  podAnnotations:
    test.sh/component: synchronizer

open-match-customize:
  enabled: true
  evaluator:
    enabled: true
    podAnnotations:
      test.sh/component: evaluator
  function:
    enabled: true
    podAnnotations:
      test.sh/component: function

open-match-scale:
  enabled: true
  scaleFrontend:
    podAnnotations:
      test.sh/component: scaleFrontend
  scaleBackend:
    podAnnotations:
      test.sh/component: scaleBackend

@oniku-2929 oniku-2929 mentioned this pull request Aug 21, 2023
Open
@joeholley joeholley added this to the Not Currently Planned milestone Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aLekSer aLekSer Awaiting requested review from aLekSer aLekSer is a code owner

@HazWard HazWard Awaiting requested review from HazWard HazWard is a code owner

@calebatwd calebatwd Awaiting requested review from calebatwd calebatwd is a code owner

@sawagh sawagh Awaiting requested review from sawagh sawagh is a code owner

@amg84 amg84 Awaiting requested review from amg84 amg84 is a code owner

@scosgrave scosgrave Awaiting requested review from scosgrave scosgrave is a code owner

@markmandel markmandel Awaiting requested review from markmandel markmandel is a code owner

@joeholley joeholley Awaiting requested review from joeholley joeholley is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area/productionization area/refinement enhancement New feature or request
Projects
None yet
Milestone
Not Currently Planned
Development

Successfully merging this pull request may close these issues.

Add pod annotations to Helm templates for injecting user-specified pod annotations.
2 participants
@oniku-2929 @joeholley