Skip to content

[Backport 7.67.x] Update to an available version of maven for windows fips Docker builds #37808

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

Merged
merged 1 commit into from
Jun 10, 2025

Conversation

agent-platform-auto-pr[bot]
Copy link
Contributor

Backport cbd7198 from #37747.


What does this PR do?

Updates maven to 3.9.10 on the script that installs bouncycastle FIPS for Windows Docker images.

Motivation

Fix permanent failures on docker_build_fips_agent7_windows2022_core_jmx (such as https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/973648272)

Describe how you validated your changes

<!--
Validate your changes before merge, ensuring that:

  • Your PR is tested by static / unit / integrations / e2e tests
  • Your PR description details which e2e tests cover your changes, if any
  • The PR description contains details of how you validated your changes. If you validated changes manually and not through automated tests, add context on why automated tests did not fit your changes validation.

If you want additional validation by a second person, you can ask reviewers to do it. Describe how to set up an environment for manual tests in the PR description. Manual validation is expected to happen on every commit before merge.

Any manual validation step should then map to an automated test. Manual validation should not substitute automation, minus exceptions not supported by test tooling yet.
-->

Possible Drawbacks / Trade-offs

Additional Notes

<!--

  • Anything else we should know when reviewing?
  • Include benchmarking information here whenever possible.
  • Include info about alternatives that were considered and why the proposed
    version was chosen.
    -->

@agent-platform-auto-pr agent-platform-auto-pr bot requested a review from a team as a code owner June 10, 2025 10:00
@agent-platform-auto-pr agent-platform-auto-pr bot added changelog/no-changelog qa/no-code-change No code change in Agent code requiring validation backport bot team/agent-delivery short review PR is simple enough to be reviewed quickly labels Jun 10, 2025
@agent-platform-auto-pr
Copy link
Contributor Author

Uncompressed package size comparison

Comparison with ancestor 72b16c13c417cb262f767cf14beb0d9c338f8e9d

Diff per package
package diff status size ancestor threshold
datadog-agent-x86_64-rpm 0.43MB ⚠️ 784.46MB 784.03MB 0.50MB
datadog-agent-x86_64-suse 0.43MB ⚠️ 784.46MB 784.03MB 0.50MB
datadog-agent-amd64-deb 0.43MB ⚠️ 775.51MB 775.08MB 0.50MB
datadog-agent-aarch64-rpm 0.41MB ⚠️ 773.74MB 773.33MB 0.50MB
datadog-agent-arm64-deb 0.41MB ⚠️ 764.81MB 764.39MB 0.50MB
datadog-heroku-agent-amd64-deb 0.02MB ⚠️ 378.73MB 378.71MB 0.50MB
datadog-dogstatsd-x86_64-rpm 0.01MB ⚠️ 31.30MB 31.30MB 0.50MB
datadog-dogstatsd-x86_64-suse 0.01MB ⚠️ 31.30MB 31.30MB 0.50MB
datadog-dogstatsd-amd64-deb 0.01MB ⚠️ 31.22MB 31.21MB 0.50MB
datadog-dogstatsd-arm64-deb 0.00MB 30.04MB 30.04MB 0.50MB
datadog-iot-agent-x86_64-rpm 0.00MB 61.14MB 61.14MB 0.50MB
datadog-iot-agent-x86_64-suse 0.00MB 61.14MB 61.14MB 0.50MB
datadog-iot-agent-aarch64-rpm 0.00MB 58.39MB 58.39MB 0.50MB
datadog-iot-agent-amd64-deb 0.00MB 61.06MB 61.06MB 0.50MB
datadog-iot-agent-arm64-deb 0.00MB 58.31MB 58.31MB 0.50MB

Decision

⚠️ Warning

@chouquette chouquette merged commit f4dd783 into 7.67.x Jun 10, 2025
330 of 335 checks passed
@chouquette chouquette deleted the backport-37747-to-7.67.x branch June 10, 2025 11:21
@github-actions github-actions bot added this to the 7.67.0 milestone Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport bot changelog/no-changelog qa/no-code-change No code change in Agent code requiring validation short review PR is simple enough to be reviewed quickly team/agent-delivery
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants