Fix/pr 3147#3201
Conversation
….io (#3197) Add dd-octo-sts trust policy for release-dashboard-api on us1.ddbuild.io Co-Authored-By: Claude Sonnet 4.6 <[email protected]> Co-authored-by: antonio.mejia <[email protected]>
) * ADMS: vuln minor upgrades — 6 packages (minor: 2 · patch: 4) Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com> * chore: regenerate lockfiles after rebase Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com> * fix: update LICENSE-3rdparty.csv for moby/spdystream v0.5.1 sub-package The upgrade from github.com/moby/spdystream v0.5.0 to v0.5.1 introduced a new sub-package github.com/moby/spdystream/spdy with a BSD-3-Clause license. Add the missing entry to keep LICENSE-3rdparty.csv up to date. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 4.6 <[email protected]> Co-authored-by: levan-m <[email protected]> Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com> * fix: sync api/go.mod with workspace after dependency upgrades Run 'go work sync' to keep api/go.mod consistent with the transitive dependency updates from the main module (golang.org/x/* version bumps pulled in by the moby/spdystream and opentelemetry upgrades). Environment: Datadog workspace Co-Authored-By: Claude Sonnet 4.6 <[email protected]> Co-authored-by: levan-m <[email protected]> Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com> * fix: add missing go.mod hash entries to api/go.sum 'go mod tidy' adds .mod file hash entries that 'go work sync' omits. Required to satisfy the check-golang-version CI validation. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 4.6 <[email protected]> Co-authored-by: levan-m <[email protected]> Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com> --------- Co-authored-by: dd-octo-sts-26fcfa[bot] <266798054+dd-octo-sts-26fcfa[bot]@users.noreply.github.com>
…ion (#3187) * ci(issue-triage): add reusable workflow from DataDog/issue-triage-action Add issue triage workflow that delegates to the shared reusable action at DataDog/issue-triage-action v1.0.0. On each new issue the workflow runs Claude-based triage, applies a team label, and sends a Slack notification. * Use the good commit
…3196) Add DatadogInstrumentation check containerName chore: update config sample Co-authored-by: mathew.estafanous <[email protected]>
* sync operator release workflow * simplify diff detection
Add DatadogInstrumentation logs config Co-authored-by: mathew.estafanous <[email protected]>
# Conflicts: # test/e2e/go.mod # test/e2e/go.sum
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1f78edd17c
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| // Logs configures Datadog Agent log collection for the target workload. | ||
| // +optional | ||
| // +listType=atomic | ||
| Logs []DatadogInstrumentationLogConfig `json:"logs,omitempty"` |
There was a problem hiding this comment.
Regenerate the OLM bundle example for DDI logs
Adding the new top-level spec.config.logs shape updates the CRD and config/samples, but the checked-in OLM CSV still advertises the old DatadogInstrumentation alm-examples payload with checks[].containerImage and nested checks[].logs (bundle/manifests/datadog-operator.clusterserviceversion.yaml lines 78 and 96). In OperatorHub/OLM installs, users copy or apply that bundled example, so with this CRD the log config is no longer represented correctly and the old fields are pruned. Please regenerate/update the bundle CSV along with the API/sample change.
Useful? React with 👍 / 👎.
654183a
into
engraver-auto-version-upgrade/minorpatch/go/e2e/0-1781562948
What does this PR do?
A brief description of the change being made with this pull request.
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?
Describe your test plan
Write there any instructions and details you may have to test your PR.
Checklist
bug,enhancement,refactoring,documentation,tooling, and/ordependenciesqa/skip-qalabel