Skip to content

[CONTP-1780] Add untaint controller config to datadog operator - #2773

Merged
adel121 merged 11 commits into
mainfrom
CONTP-1612/untaint-controller-config
Jul 10, 2026
Merged

[CONTP-1780] Add untaint controller config to datadog operator#2773
adel121 merged 11 commits into
mainfrom
CONTP-1612/untaint-controller-config

Conversation

@adel121

@adel121 adel121 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

Adds untaintController.enabled to allow enabling untaint controller, in addition to other configuration options documented in readme.md.

Special notes for your reviewer:

After merging this PR, we should bump operator chart version in datadog chart dependency in this PR and merge it.

Checklist

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

  • All commits are signed and show as "Verified" on GitHub (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)
  • For datadog chart changes, received ✅ from a member of your team

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

@github-actions github-actions Bot added the chart/datadog-operator This issue or pull request is related to the datadog-operator chart label Jul 6, 2026
@datadog-official

This comment has been minimized.

@adel121
adel121 force-pushed the CONTP-1612/untaint-controller-config branch from e940674 to b132d34 Compare July 9, 2026 11:35
@github-actions github-actions Bot added the chart/datadog This issue or pull request is related to the datadog chart label Jul 9, 2026
@adel121
adel121 force-pushed the CONTP-1612/untaint-controller-config branch from c4081f2 to 0ce8262 Compare July 9, 2026 11:53
@adel121 adel121 changed the title add untaint controller config to datadog operator [CONTP-1780] Add untaint controller config to datadog operator Jul 9, 2026
@adel121 adel121 added datadog-operator/minor-version Minor version bump for datadog-operator chart and removed chart/datadog This issue or pull request is related to the datadog chart labels Jul 9, 2026
@adel121
adel121 marked this pull request as ready for review July 9, 2026 12:39
@adel121
adel121 requested a review from a team as a code owner July 9, 2026 12:39
@adel121

adel121 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

/merge

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351 Bot commented Jul 9, 2026

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-07-09 13:19:38 UTC ℹ️ Start processing command /merge


2026-07-09 13:19:44 UTC ℹ️ MergeQueue: pull request added to the queue

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


2026-07-09 13:26:54 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/2773/merge: 405 Required status check "pr-validated" is expected. [] (Request ID: CCCF:128753:18DF89B:5714B9A:6A4FA193)

FullStacktrace:
activity error (type: github.GithubService_MergePullRequest, scheduledEventID: 50, startedEventID: 51, identity: 1@github-worker-65778c9c86-7j2z7@): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/2773/merge: 405 Required status check "pr-validated" is expected. [] (Request ID: CCCF:128753:18DF89B:5714B9A:6A4FA193) (type: GitFailure, retryable: false): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/2773/merge: 405 Required status check "pr-validated" is expected. [] (type: ErrorResponse, retryable: true)

@adel121
adel121 force-pushed the CONTP-1612/untaint-controller-config branch from 4a45476 to 1e603bb Compare July 9, 2026 13:43
@adel121
adel121 requested review from a team as code owners July 9, 2026 13:43
@adel121
adel121 requested review from ddogDiegoGonzalez and removed request for a team July 9, 2026 13:43
@adel121
adel121 removed the request for review from ddogDiegoGonzalez July 9, 2026 14:38
Comment thread charts/datadog-operator/CHANGELOG.md Outdated
adel121 and others added 5 commits July 10, 2026 11:54
Co-authored-by: levan-m <116471169+levan-m@users.noreply.github.com>
- bump version for datadog-operator to 2.25.0 (minor-version)
- update changelog version for datadog-operator to 2.25.0
- update readme for datadog-operator
- bump version for datadog-operator to 2.25.0 (minor-version)
- update readme for datadog-operator
@adel121 adel121 added datadog-operator/patch-version Patch version bump for datadog-operator chart and removed datadog-operator/minor-version Minor version bump for datadog-operator chart labels Jul 10, 2026
dd-octo-sts Bot added 2 commits July 10, 2026 12:58
- bump version for datadog-operator to 2.25.0-dev.2 (patch-version)
- update changelog version for datadog-operator to 2.25.0-dev.2
- update readme for datadog-operator
- bump version for datadog-operator to 2.25.0-dev.2 (patch-version)
- update changelog version for datadog-operator to 2.25.0-dev.2
- update readme for datadog-operator
@adel121
adel121 merged commit 91445b8 into main Jul 10, 2026
43 checks passed
@adel121
adel121 deleted the CONTP-1612/untaint-controller-config branch July 10, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chart/datadog-operator This issue or pull request is related to the datadog-operator chart datadog-operator/patch-version Patch version bump for datadog-operator chart mergequeue-status: error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants