Skip to content

Use Ubuntu 26.04 GBI for internal build#3038

Merged
tbavelier merged 6 commits into
mainfrom
tbavelier/use-datadog-operator-tmpl-v3
May 26, 2026
Merged

Use Ubuntu 26.04 GBI for internal build#3038
tbavelier merged 6 commits into
mainfrom
tbavelier/use-datadog-operator-tmpl-v3

Conversation

@tbavelier

@tbavelier tbavelier commented May 22, 2026

Copy link
Copy Markdown
Member

What does this PR do?

  • Uses tmpl-v3 for internal builds, based on Ubuntu 26.04 GBI (with FIPS handled via Jinja2 conditional in a single template)
  • Fixes FIPS staging image tag missing -fips suffix
  • Makes e2e tests manual (not auto-triggered) when PUSH_IMAGES_TO_STAGING is set, to avoid blocking/waiting staging test pipelines

Motivation

  • Vulns (Ubuntu 22.04 → 26.04 GBI upgrade)

Additional Notes

Depends on https://github.com/DataDog/images/pull/9871, now merged to master.

@tbavelier tbavelier added dependencies Pull requests that update a dependency file qa/skip-qa labels May 22, 2026
@datadog-prod-us1-4

datadog-prod-us1-4 Bot commented May 22, 2026

Copy link
Copy Markdown

Pipelines

Fix all issues with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

validation | build   View in Datadog   GitHub Actions

🔄 Retry job. This looks flaky and may succeed on retry. Failed to download archive from GitHub: An action could not be found at the URI 'https://codeload.github.com/actions/setup-go/tar.gz/4b73464bb391d4059bd26b0524d20df3927bd417'.

Useful? React with 👍 / 👎

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

@codecov-commenter

codecov-commenter commented May 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 42.24%. Comparing base (b2a60a9) to head (9032fb6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3038   +/-   ##
=======================================
  Coverage   42.24%   42.24%           
=======================================
  Files         337      337           
  Lines       28951    28951           
=======================================
  Hits        12230    12230           
  Misses      15916    15916           
  Partials      805      805           
Flag Coverage Δ
unittests 42.24% <ø> (ø)

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


Continue to review full report in Codecov by Sentry.

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

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

tbavelier and others added 5 commits May 22, 2026 16:53
trigger_custom_operator_image_fips_staging was publishing to the same
RELEASE_TAG/BUILD_TAG as the non-FIPS staging job, causing one to
overwrite the other.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <[email protected]>
When testing custom builds pushed to staging, e2e tests should not
start automatically. Changing allow_failure to when:manual so they
remain triggerable but don't consume CI resources by default.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <[email protected]>
@tbavelier
tbavelier marked this pull request as ready for review May 26, 2026 13:06
@tbavelier
tbavelier requested a review from a team May 26, 2026 13:06
@tbavelier

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@tbavelier tbavelier added this to the v1.27.0 milestone May 26, 2026
Comment thread .gitlab-ci.yml
NIGHTLY_BUILD_CONDUCTOR_NAME: "operator-nightly-build"
PROJECTNAME: "datadog-operator"
PROJECTNAME_CHECK: "datadog-operator-check"
IMAGE_VERSION: "tmpl-v3"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice to parametrize

@tbavelier
tbavelier merged commit 7a0a7a8 into main May 26, 2026
33 of 51 checks passed
@tbavelier
tbavelier deleted the tbavelier/use-datadog-operator-tmpl-v3 branch May 26, 2026 13:17
tbavelier added a commit that referenced this pull request May 27, 2026
* [datadog-operator] Use Ubuntu 26.04 GBI

* Use single tmpl image version with unification

* Fix FIPS staging image tag missing -fips suffix

trigger_custom_operator_image_fips_staging was publishing to the same
RELEASE_TAG/BUILD_TAG as the non-FIPS staging job, causing one to
overwrite the other.



* Make e2e tests manual when PUSH_IMAGES_TO_STAGING is set

When testing custom builds pushed to staging, e2e tests should not
start automatically. Changing allow_failure to when:manual so they
remain triggerable but don't consume CI resources by default.



* Point images triggers to master now that images PR is merged



---------


(cherry picked from commit 7a0a7a8)

Co-authored-by: Timothée Bavelier <[email protected]>
Co-authored-by: Claude Sonnet 4.6 (1M context) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants