Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

helm: add podLabels parameter #963

Merged
merged 1 commit into from Feb 9, 2021
Merged

helm: add podLabels parameter #963

merged 1 commit into from Feb 9, 2021

Conversation

t3mi
Copy link
Contributor

@t3mi t3mi commented Feb 7, 2021

Reason for Change:

Adds configuration for additional pod labels, like CostCenter=IT

Requirements

  • squashed commits
  • included documentation
  • added unit tests and e2e tests (if applicable). See test standard for more details.
  • ran make precommit

Issue Fixed:

Please answer the following questions with yes/no:

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

  • yes
  • no

Notes for Reviewers:

@chewong
Copy link
Contributor

chewong commented Feb 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-io
Copy link

Codecov Report

Merging #963 (17f6ab0) into master (36a8568) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #963      +/-   ##
==========================================
- Coverage   36.25%   36.04%   -0.21%     
==========================================
  Files          27       27              
  Lines        2913     2913              
==========================================
- Hits         1056     1050       -6     
- Misses       1778     1782       +4     
- Partials       79       81       +2     

@t3mi t3mi changed the title add podLabels parameter helm: add podLabels parameter Feb 9, 2021
@aramase aramase requested a review from chewong February 9, 2021 17:45
@chewong chewong merged commit 966b9e2 into Azure:master Feb 9, 2021
statbit pushed a commit to adobe-platform/aad-pod-identity that referenced this pull request Sep 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants