Migrate API key to environment secrets - #12126
Conversation
There was a problem hiding this comment.
More details
The change only scopes the Trivy job to protected-main-env while preserving its existing Datadog secret expression and leaving CodeQL unchanged. YAML parsing, trigger coverage, environment scope, and secret-resolution scenarios passed; environment protection rules could not be independently read because the repository API returned 403.
📊 Validated against 4 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit 14a5f53 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
|
🎯 Code Coverage (details) 🔗 Commit SHA: 14a5f53 | Docs | Datadog PR Page | Give us feedback! |
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
|
/merge |
|
View all feedbacks in Devflow UI.
PR already in the queue with status in_progress |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
|
In fact, you can add the PR to MQ using DDCI UI… But you got no feedback. So I guess it was in queue but stuck… |
|
Hm I didn't know you could add a PR to MQ directly from DDCI UI... Maybe that somehow caused a double add to the MQ and the error from above "PR already in the queue with status in_progress" despite there being nothing in the MQ. |
|
Yes, I did comment "/merge" to try to get the status and it failed as already in queue. Let's wait for Today and check later if it moved 🤷 |
464ce1a
into
master
What Does This Do
This PR moves the DD API key to a protected environment.
Motivation
SDLC recommendations
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: APMLP-1687