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

[CONTP-519] Support K8s admin events #1639

Merged
merged 12 commits into from
Dec 20, 2024
Merged

[CONTP-519] Support K8s admin events #1639

merged 12 commits into from
Dec 20, 2024

Conversation

gabedos
Copy link
Contributor

@gabedos gabedos commented Dec 17, 2024

What this PR does / why we need it:

Adds support for the new feature for change tracking in k8s called "Kubernetes Admission Events".

Special notes for your reviewer:

Applicable on Agent v.7.62.0+

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)

@gabedos gabedos requested a review from a team as a code owner December 17, 2024 18:28
@github-actions github-actions bot added the chart/datadog This issue or pull request is related to the datadog chart label Dec 17, 2024
@gabedos
Copy link
Contributor Author

gabedos commented Dec 18, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Dec 18, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-12-18 14:33:15 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in main is 37m.


2024-12-18 15:10:16 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/1639/merge: 405 Required status check "pr-validated" is expected. []

FullStacktrace:
activity error (type: github.GithubService_MergePullRequest, scheduledEventID: 41, startedEventID: 42, identity: 1@github-worker-69dd48f4bb-p2lx4@): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/1639/merge: 405 Required status check "pr-validated" is expected. [] (type: GitFailure, retryable: false): PUT https://api.github.com/repos/DataDog/helm-charts/pulls/1639/merge: 405 Required status check "pr-validated" is expected. [] (type: ErrorResponse, retryable: true)

@gabedos gabedos enabled auto-merge (squash) December 20, 2024 15:07
@gabedos gabedos merged commit 406d5f0 into main Dec 20, 2024
16 of 17 checks passed
@gabedos gabedos deleted the gabedos/k8s-admin-events branch December 20, 2024 19:19
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: error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants