Skip to content

Onboard off public-images GitLab job-token trigger - #299

Draft
AliDatadog wants to merge 1 commit into
mainfrom
ali.b/onboard-dd-pkg-publish-image
Draft

Onboard off public-images GitLab job-token trigger#299
AliDatadog wants to merge 1 commit into
mainfrom
ali.b/onboard-dd-pkg-publish-image

Conversation

@AliDatadog

Copy link
Copy Markdown
Contributor

Summary

DataDog/extendeddaemonset currently publishes public container images by directly triggering the DataDog/public-images GitLab CI pipeline via trigger: project: DataDog/public-images. We're migrating repos off that legacy job-token trigger mechanism onto dd-pkg publish-image, a CLI call against the artifact-gateway service.

dd-pkg publish-image still drives the same public-images pipeline under the hood, but routes through a policy/audit layer instead of an unauthenticated cross-project trigger. No per-repo registration or allowlist step is needed — authorization is implicit via CI job identity — so this change is purely a .gitlab-ci.yml edit.

Changes:

  • Added a shared .docker_publish_job_definition template that runs dd-pkg publish-image, translating the existing IMG_* job variables into CLI flags.
  • Replaced the trigger: project: DataDog/public-images block in publish_public_main, publish_public_tag, and publish_public_latest with extends: .docker_publish_job_definition.
  • Left stage:, rules:, needs:, and variables: on each job untouched; only the trigger mechanism changed.
  • Left the unrelated trigger: project: DataDog/images jobs (internal image promotion) as-is — out of scope for this migration.

Background: https://datadoghq.atlassian.net/wiki/x/ooQkngE
Ref: https://datadoghq.atlassian.net/browse/BARX-1951

Test plan

  • ruby -ryaml -e "YAML.load_file('.gitlab-ci.yml'); puts 'OK'" confirms the file is valid YAML.
  • Merge to main and confirm publish_public_main (and -fips variant) successfully publish images via dd-pkg publish-image.
  • Cut a tag and confirm publish_public_tag/publish_public_latest (and -fips variants) still publish correctly when run manually.

…h-image

Replaces the direct `trigger: project: DataDog/public-images` CI jobs
with `dd-pkg publish-image` calls against the artifact-gateway service.
This routes public image publishing through a policy/audit layer while
still driving the same public-images pipeline; no per-repo registration
is required since authorization is implicit via CI job identity.
See https://datadoghq.atlassian.net/wiki/x/ooQkngE

Ref: BARX-1951
@datadog-official

datadog-official Bot commented Jul 27, 2026

Copy link
Copy Markdown

Pipelines  Code Coverage

⚠️ Warnings

🚦 2 Pipeline jobs failed

pull request linter | Check Milestone   View in Datadog   GitHub Actions

pull request linter | build   View in Datadog   GitHub Actions

ℹ️ Info

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 37.19% (+0.00%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: dcd1056 | Docs | Datadog PR Page | Give us feedback!

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.01%. Comparing base (b6860a0) to head (dcd1056).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   38.01%   38.01%           
=======================================
  Files          65       65           
  Lines        3856     3856           
=======================================
  Hits         1466     1466           
  Misses       2275     2275           
  Partials      115      115           
Flag Coverage Δ
unittests 38.01% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b6860a0...dcd1056. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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