Skip to content

Experiment and ControllerRevision decoupling - #3371

Draft
levan-m wants to merge 9 commits into
mainfrom
levan-m/controllerrer-decoupling
Draft

Experiment and ControllerRevision decoupling#3371
levan-m wants to merge 9 commits into
mainfrom
levan-m/controllerrer-decoupling

Conversation

@levan-m

@levan-m levan-m commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

CONTP-1742

Commits
e099c93 phase 3 followup: bind baseline to patched resourceVersion; restore transient retry
219ae31 phase 3: freshness preflight, optimistic-lock patch, reason codes
396adf1 phase 8: abort in-flight experiments missing checkpoint fields
782f881 phase 7: delete experiment-specific revision machinery
d2dfce7 phase 6: manual-change detection via ExpectedSpecHash
9d5fddc phase 5: rollback by name + GC pin
b7e9cb6 phase 3a: daemon writes rollback-target-revision annotation from Status.CurrentRevision
dba72cc phase 4: processStartSignal wiring (RollbackTargetRevision + ExpectedSpecHash; missing-checkpoint abort)
ec9da1d phase 1, 2: API scaffolding for decoupling; reconcile barrier for revisions

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

  • Agent: vX.Y.Z
  • Cluster Agent: vX.Y.Z

Describe your test plan

Write there any instructions and details you may have to test your PR.

Checklist

  • PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
  • PR has a milestone or the qa/skip-qa label
  • All commits are signed (see: signing commits)

@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Aug 18, 2026

Copy link
Copy Markdown

Pipelines

Unblock PR with BitsAI

⚠️ Warnings

🚦 4 Pipeline jobs failed

DataDog/datadog-operator | build — 🔧 Needs a code fix, caused by this PR

View in Datadog · View in GitLab

validation | build — 🔧 Needs a code fix, caused by this PR

View in Datadog · View in GitHub Actions

2 lint errors: for loop can be modernized using range over int in pkg/fleet/daemon_operations.go:144 and pkg/fleet/experiment.go:155.

pull request linter | Check Milestone

View in Datadog · View in GitHub Actions

Missing milestone or qa/skip-qa label during the checks.

View all 4 failed jobs.

📋 Copy prompt for your agent
CI on my pull request is failing. Help me find and fix the root cause of each failing job below — they were flagged as caused by changes in this PR, so focus on the diff. For each job, explain the failure and propose a fix.

Branch: levan-m/controllerrer-decoupling

validation | build
Commit: e099c93a83193ccae13bd8fd69f3f17c133c9bfb
Error (code / quality):
2 lint errors: for loop can be modernized using range over int in pkg/fleet/daemon_operations.go:144 and pkg/fleet/experiment.go:155.
CI job: https://github.com/DataDog/datadog-operator/actions/runs/32193690813/job/95893247958

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: e099c93 | Docs | View more details | Give us feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant