Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CONTINT-3761] Use GBI for the internal eds-check image #175

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

L3n41c
Copy link
Member

@L3n41c L3n41c commented Apr 2, 2024

What does this PR do?

Migrate the internal extendeddaemonset-check image to GBI.

Motivation

Security folks want all containers running on our internal infra to be based on a GBI golden base image.

Additional Notes

See DataDog/images#5091.
The idea is to use GBI for the internal image only and to keep UBI as today for the public one.
As the binary is a standalone statically-linked file, it shouldn’t make any difference.

Describe your test plan

Update the version of extendeddaemonset-check in k8s-datadog-agent-ops and check that deployments are still working fine.

@L3n41c L3n41c requested review from a team as code owners April 2, 2024 13:13
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, documentation, tooling

@L3n41c L3n41c added this to the v0.9 milestone Apr 2, 2024
@L3n41c L3n41c added qa/skip-qa tooling Issue linked to the repository tooling labels Apr 2, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request contains a valid label.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request contains a valid label.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.10%. Comparing base (fcf8307) to head (2ed4beb).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #175   +/-   ##
=======================================
  Coverage   63.10%   63.10%           
=======================================
  Files          41       41           
  Lines        3098     3098           
=======================================
  Hits         1955     1955           
  Misses       1023     1023           
  Partials      120      120           
Flag Coverage Δ
unittests 63.10% <ø> (ø)

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 fcf8307...2ed4beb. Read the comment docs.

@clamoriniere clamoriniere merged commit af9390e into main Apr 15, 2024
28 of 29 checks passed
@clamoriniere clamoriniere deleted the lenaic/CONTINT-3761_eds-check_gbi branch April 15, 2024 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qa/skip-qa tooling Issue linked to the repository tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants