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

[tempo-mixin] Additional labels YAML indentation #541

Merged
merged 1 commit into from Sep 26, 2023
Merged

[tempo-mixin] Additional labels YAML indentation #541

merged 1 commit into from Sep 26, 2023

Conversation

nlamirault
Copy link
Collaborator

@nlamirault nlamirault commented Sep 26, 2023 •
edited

What this PR does / why we need it:

Invalid YAML indentation when we set the additionalLabels entry.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

Checklist

  • Title of the PR starts with chart name (e.g. [portefaix-kyverno])
  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • Chart Version bumped
  • ChangeLog.md has beed updated
  • Variables are documented in the README.md

@nlamirault
Fix: Additional labels for Tempo mixin
d5427c6 
Signed-off-by: Nicolas Lamirault <nicolas.lamirault@gmail.com>
@nlamirault nlamirault added status/review_needed The issue or PR needs to be reviewed kind/bug Categorizes issue or PR as related to a bug priority/low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix lifecycle/active Indicates that an issue or PR is actively being worked on by a contributor. area/helm Helm labels Sep 26, 2023
@nlamirault nlamirault self-assigned this Sep 26, 2023
@github-actions github-actions bot added the size/xs Size XS label Sep 26, 2023
@nlamirault nlamirault merged commit 18f89f2 into master Sep 26, 2023
12 of 13 checks passed
@nlamirault nlamirault deleted the fix/tempo-mixin branch September 26, 2023 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nlamirault nlamirault

Labels
area/helm Helm kind/bug Categorizes issue or PR as related to a bug lifecycle/active Indicates that an issue or PR is actively being worked on by a contributor. priority/low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix size/xs Size XS status/review_needed The issue or PR needs to be reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@nlamirault