Skip to content

Conversation

@jennchenn
Copy link
Member

What this PR does / why we need it:

Update RBAC to allow watch for karpenter resources when cluster autoscaling is enabled

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.]

  • All commits are signed (see: signing commits)
  • Chart Version semver bump label has been added (use <chartName>/minor-version, <chartName>/patch-version, or <chartName>/no-version-bump)
  • For datadog or datadog-operator chart or value changes, update the test baselines (run: make update-test-baselines)

GitHub CI takes care of the below, but are still required:

  • Documentation has been updated with helm-docs (run: .github/helm-docs.sh)
  • CHANGELOG.md has been updated
  • Variables are documented in the README.md

@jennchenn jennchenn requested a review from a team as a code owner January 20, 2026 20:20
@jennchenn jennchenn added the datadog/patch-version Patch version bump for datadog chart label Jan 20, 2026
@github-actions github-actions bot added the chart/datadog This issue or pull request is related to the datadog chart label Jan 20, 2026
- update changelog version for datadog to 3.161.2
@jennchenn
Copy link
Member Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link

gh-worker-devflow-routing-ef8351 bot commented Jan 21, 2026

View all feedbacks in Devflow UI.

2026-01-21 16:36:46 UTC ℹ️ Start processing command /merge


2026-01-21 16:37:26 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 50m (p90).


2026-01-21 17:25:38 UTC 🚨 MergeQueue: This merge request is in error

mergequeue build completed successfully, but the github api returned an error while merging the pr.
It's probably because:

  • target branch of PR is restricted to only allow up-to-date branches, but the pr is now outdated
Details

Error: PUT https://api.github.com/repos/DataDog/helm-charts/pulls/2310/merge: 405 Required status check "pr-validated" is expected. [] (Request ID: 9776:2CF49B:26D707A:AC2B421:69710C08)

FullStacktrace:
activity error (type: github.GithubService_MergePullRequest, scheduledEventID: 52, startedEventID: 53, identity: 1@github-worker-cb4895b55-jnm4k@): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/2310/merge: 405 Required status check "pr-validated" is expected. [] (Request ID: 9776:2CF49B:26D707A:AC2B421:69710C08) (type: GitFailure, retryable: false): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/2310/merge: 405 Required status check "pr-validated" is expected. [] (type: ErrorResponse, retryable: true)

@jennchenn
Copy link
Member Author

/merge

@gh-worker-devflow-routing-ef8351
Copy link

gh-worker-devflow-routing-ef8351 bot commented Jan 21, 2026

View all feedbacks in Devflow UI.

2026-01-21 18:16:10 UTC ℹ️ Start processing command /merge


2026-01-21 18:16:28 UTC ℹ️ MergeQueue: waiting for PR to be ready

This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
It will be added to the queue as soon as checks pass and/or get approvals. View in MergeQueue UI.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2026-01-21 19:03:38 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 50m (p90).


2026-01-21 19:43:52 UTC ℹ️ MergeQueue: This merge request was merged

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 datadog/patch-version Patch version bump for datadog chart mergequeue-status: done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants