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

Fix a few typos on otel agent in values.yaml #1490

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Conversation

songy23
Copy link
Member

@songy23 songy23 commented Aug 14, 2024

What this PR does / why we need it:

Fix typos: trace-agent -> otel-agent

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

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md
  • For Datadog Operator chart or value changes update the test baselines (run: make update-test-baselines)

@songy23 songy23 requested review from dineshg13 and krlv August 14, 2024 20:48
@songy23 songy23 requested a review from a team as a code owner August 14, 2024 20:48
@github-actions github-actions bot added the chart/datadog This issue or pull request is related to the datadog chart label Aug 14, 2024
Copy link
Collaborator

@clamoriniere clamoriniere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @songy23

thanks for the typo finding and opening this pull request.
the comment changes are good, however you need to update the chart version and follow the github pull request checklist to trigger a new release of the chart with the typo fixes.

@songy23 songy23 marked this pull request as draft August 21, 2024 20:25
@songy23 songy23 marked this pull request as ready for review August 22, 2024 20:57
@songy23
Copy link
Member Author

songy23 commented Aug 22, 2024

Thank you @clamoriniere , I have updated the PR with version bump and docs

Copy link
Contributor

@krlv krlv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 Thanks @songy23
Please rebase with the latest main branch (the latest chart version is 3.70.7 as of now)

@songy23
Copy link
Member Author

songy23 commented Sep 10, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Sep 10, 2024

❌ MergeQueue

You are not allowed to use the merge queue towards main.

If you need support, contact us on Slack #devflow with those details!

@songy23
Copy link
Member Author

songy23 commented Sep 10, 2024

Looks like I don't have access to merge, @krlv @clamoriniere could you help add this PR to merge queue?

@krlv krlv force-pushed the songy23-patch-1 branch from ccc37cd to 7163653 Compare October 7, 2024 23:25
@krlv krlv requested a review from clamoriniere October 7, 2024 23:25
@clamoriniere
Copy link
Collaborator

/merge

@dd-devflow
Copy link

dd-devflow bot commented Oct 8, 2024

🚂 MergeQueue: pull request added to the queue

The median merge time in main is 35m.

Use /merge -c to cancel this operation!

@dd-mergequeue dd-mergequeue bot merged commit a17cc4b into main Oct 8, 2024
21 checks passed
@dd-mergequeue dd-mergequeue bot deleted the songy23-patch-1 branch October 8, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart/datadog This issue or pull request is related to the datadog chart mergequeue-status: done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants