Skip to content

[K9VULN-11560] Start deprecation of bundled datadog-ci#815

Merged
jasonforal merged 2 commits intomainfrom
jason.foral/K9VULN-11560
Feb 25, 2026
Merged

[K9VULN-11560] Start deprecation of bundled datadog-ci#815
jasonforal merged 2 commits intomainfrom
jason.foral/K9VULN-11560

Conversation

@jasonforal
Copy link
Collaborator

@jasonforal jasonforal commented Feb 13, 2026

What problem are you trying to solve?

The Docker container currently bundles datadog-ci inside it. Due to the different release cycles between this repo and datadog-ci, this generally leads to the bundled version quickly becoming "stale".

I'd like to remove datadog-ci from this container altogether to prevent issues going forward.

What is your solution?

Introduce a passthrough shim that logs a warning message for eventual deprecation/removal.

(And tangentially related is github-action.sh, which is no longer invoked by v3 of the action. Thus, I added a nudge to upgrade, as well as a quick dev comment)

Testing

You may test this locally:

datadog-ci

docker build -t ddsa . && docker run --rm --entrypoint datadog-ci ddsa --version
================================================================================
WARNING:
The datadog-ci binary in this container will be removed in a future release.
Please install datadog-ci separately in your environment if you need it.

If using datadog-static-analyzer-github-action, consider upgrading to v3.
================================================================================
4.4.0

datadog-static-analyzer

docker build -t ddsa . && docker run --rm --entrypoint datadog-static-analyzer ddsa -v
Version: 0.7.5, revision: development

Alternatives considered

What the reviewer should know

@jasonforal jasonforal requested a review from a team as a code owner February 13, 2026 18:12
@jasonforal jasonforal changed the title [K9VULN-11560] Start depreciation of bundled datadog-ci [K9VULN-11560] Start deprecation of bundled datadog-ci Feb 13, 2026
@datadog-official
Copy link

🎯 Code Coverage (details)
Patch Coverage: -1.00%
Overall Coverage: 84.87% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 981f4e9 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@jasonforal jasonforal merged commit b947012 into main Feb 25, 2026
85 checks passed
@jasonforal jasonforal deleted the jason.foral/K9VULN-11560 branch February 25, 2026 16:31
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