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.

Sign up for GitHub

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

Jump to bottom

Changed ECR -> registry.ddbuild.io and updated build images #220

Merged
merged 2 commits into from
Nov 14, 2024
Merged

Changed ECR -> registry.ddbuild.io and updated build images #220

merged 2 commits into from
Nov 14, 2024

Conversation

CelianR
Copy link
Contributor

@CelianR CelianR commented Nov 13, 2024

What does this PR do?

Buildimages are now pushed to registry.ddbuild.io, replaced ECR by this (DataDog/datadog-agent-buildimages#708).

Description of the Change

Alternate Designs

Possible Drawbacks

Verification Process

Additional Notes

Release Notes

Review checklist (to be filled by reviewers)

  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@CelianR CelianR self-assigned this Nov 13, 2024
@CelianR CelianR marked this pull request as ready for review November 13, 2024 13:16
@CelianR CelianR requested a review from a team as a code owner November 13, 2024 13:16
@CelianR
Copy link
Contributor Author

CelianR commented Nov 14, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 14, 2024 •
edited
Loading

Devflow running: /merge

View all feedbacks in Devflow UI.

2024-11-14 08:28:46 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in master is 0s.

2024-11-14 08:28:49 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in master is 0s.

2024-11-14 10:28:55 UTC ❌ MergeQueue: The build pipeline has timeout

The merge request has been interrupted because the build took longer than expected. The current limit for the base branch 'master' is 120 minutes.

@CelianR
Copy link
Contributor Author

CelianR commented Nov 14, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 14, 2024 •
edited
Loading

Devflow running: /merge

View all feedbacks in Devflow UI.

2024-11-14 12:11:34 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in master is 0s.

2024-11-14 12:11:37 UTC ℹ️ MergeQueue: pull request added to the queue

The median merge time in master is 0s.

2024-11-14 12:22:12 UTC ⚠️ MergeQueue: This merge request build was cancelled

This merge request build was cancelled

@CelianR
Copy link
Contributor Author

CelianR commented Nov 14, 2024

/remove

@dd-devflow
Copy link

dd-devflow bot commented Nov 14, 2024

Devflow running: /remove

View all feedbacks in Devflow UI.

2024-11-14 12:22:10 UTC ℹ️ Devflow: /remove

@CelianR CelianR merged commit 4abb65b into master Nov 14, 2024
2 of 3 checks passed
@CelianR CelianR deleted the celian/use-buildimages-registry-acix-322 branch November 14, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahulkaukuntla rahulkaukuntla rahulkaukuntla approved these changes

Assignees

@CelianR CelianR

Labels
mergequeue-status: removed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CelianR @rahulkaukuntla