Skip to content

[incident-52391] Split dd-sts policies to only request an app key when needed#7941

Draft
KSerrania wants to merge 1 commit intomasterfrom
kserrania/incident-52391-split-dd-sts-policy
Draft

[incident-52391] Split dd-sts policies to only request an app key when needed#7941
KSerrania wants to merge 1 commit intomasterfrom
kserrania/incident-52391-split-dd-sts-policy

Conversation

@KSerrania
Copy link
Copy Markdown
Contributor

What does this PR do?

Follow-up of #7924. Only request an app key when needed (i.e. only in project.yaml). See policy definition in https://github.com/DataDog/dd-source/pull/404122.

Motivation

Fix CI.

Additional Notes

n/a

@KSerrania KSerrania requested review from a team as code owners April 7, 2026 15:37
@KSerrania KSerrania requested review from BridgeAR and jchrostek-dd and removed request for a team April 7, 2026 15:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Overall package size

Self size: 5.48 MB
Deduped: 6.33 MB
No deduping: 6.33 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.0.0 | 81.15 kB | 815.98 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 bot commented Apr 7, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 49.84% (-15.33%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 043a5f7 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

BridgeAR
BridgeAR previously approved these changes Apr 7, 2026
@BridgeAR BridgeAR dismissed their stale review April 7, 2026 16:03

The CI is still failing across the board

@BridgeAR BridgeAR marked this pull request as draft April 7, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants