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

fix(installer): Use gcr.io by default instead of public.ecr.aws #25255

Merged

Conversation

BaptisteFoy
Copy link
Contributor

What does this PR do?

Uses gcr.io by default as the AWS registry is rate limited

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

@BaptisteFoy BaptisteFoy added changelog/no-changelog qa/done Skip QA week as QA was done before merge and regressions are covered by tests team/fleet-automation labels Apr 30, 2024
@BaptisteFoy BaptisteFoy marked this pull request as ready for review April 30, 2024 08:53
@BaptisteFoy BaptisteFoy requested a review from a team as a code owner April 30, 2024 08:53
@BaptisteFoy BaptisteFoy merged commit f3cddee into 7.54.0-installer-0.0.x Apr 30, 2024
129 of 146 checks passed
@BaptisteFoy BaptisteFoy deleted the baptiste.foy/FA/installer-gcr-default branch April 30, 2024 08:57
arbll added a commit that referenced this pull request May 11, 2024
* fix(installer): Use gcr.io by default instead of public.ecr.aws (#25255)

* (fleet) fix minor issues in the installer bootstrap (#25267)

Two issues discovered during the rollout:
- `/tmp` mounted as `noexec` => this PR moves the bootstraped installer to a tmp under `/opt/datadog-installer`
- some customers are running the install script in a loop, we need to avoid re-bootstraping everytime (and later re-installing)

* (fleet) use rootTmpDir for both MkdirTemp in the bootstrap

* (fleet) use actual package size to check if we have enough disk space

This PR uses the changes in DataDog/datadog-packages#11 to properly compute the required disk space to install a package. Previously we were requiring 3GiB for any package.

* purge

* update test

* pass test

* properly remove all with go

* remove noisy warn

* (fleet) fix a warn log in telemetry

```
2024/05/03 08:04:57 Datadog Tracer v1.61.0 WARN: ddtrace/tracer: deprecated config WithServiceName should not be used with `WithService` or `DD_SERVICE`; integration service name will not be set.
```

* (fleet) default to US1 for the telemetry website when DD_SITE isn't set

* fix missing bootstrap

* quick fix

---------

Co-authored-by: Baptiste Foy <baptiste.foy@datadoghq.com>
Co-authored-by: Raphael Gavache <raphael.gavache@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog qa/done Skip QA week as QA was done before merge and regressions are covered by tests team/fleet-automation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants