Skip to content

ci: deploy released version of envoy contrib to reliability env#3761

Merged
dd-mergequeue[bot] merged 1 commit into
mainfrom
flavien/envoy/rel-env
Jul 24, 2025
Merged

ci: deploy released version of envoy contrib to reliability env#3761
dd-mergequeue[bot] merged 1 commit into
mainfrom
flavien/envoy/rel-env

Conversation

@e-n-0

@e-n-0 e-n-0 commented Jul 15, 2025

Copy link
Copy Markdown
Member

What does this PR do?

This PR adds the auto deployment of the Envoy docker image after its release.
The implementation of the auto deploy was tricky because the reliability environment are jobs on Gitlab and the release process of the External Processor docker image is done via Github.

To overcome this problem, a Gitlab job was implemented to wait for the GitHub release workflow of the image.
#3795 Added the github token provider authorization for Action read to the gitlab workflow, in order to not be rate limited by github while waiting for the release action to finish.

The version of the release is extracted from the tag like it's done in the github action.

This PR is linked to the Envoy implementation PR in rel-env https://github.com/DataDog/datadog-reliability-env/pull/1118.

Motivation

QA.

Reviewer's Checklist

  • Changed code has unit tests for its functionality at or near 100% coverage.
  • System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • There is a benchmark for any new code, or changes to existing code.
  • If this interacts with the agent in a new way, a system test has been added.
  • New code is free of linting errors. You can check this by running golangci-lint run locally.
  • Add an appropriate team label so this PR gets put in the right place for the release notes.
  • Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

Unsure? Have a question? Request a review!

@github-actions github-actions Bot added the apm:ecosystem contrib/* related feature requests or bugs label Jul 15, 2025
@e-n-0
e-n-0 changed the base branch from main to flavien/envoy/env-vars July 15, 2025 19:20
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 3 times, most recently from a7a1f08 to 16d0804 Compare July 15, 2025 19:28
@pr-commenter

pr-commenter Bot commented Jul 15, 2025

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2025-07-23 12:34:13

Comparing candidate commit 72aec1f in PR branch flavien/envoy/rel-env with baseline commit 5bdbb59 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 24 metrics, 0 unstable metrics.

@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 13 times, most recently from 5d19598 to 3b4aa39 Compare July 15, 2025 21:03
@e-n-0
e-n-0 force-pushed the flavien/envoy/env-vars branch from 6e906e2 to 954ae77 Compare July 15, 2025 21:04
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 2 times, most recently from 4bc9bd9 to beb94a0 Compare July 15, 2025 21:18
@e-n-0
e-n-0 force-pushed the flavien/envoy/env-vars branch from 34cef86 to bfb5cb4 Compare July 16, 2025 14:29
Base automatically changed from flavien/envoy/env-vars to main July 17, 2025 09:42
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 6 times, most recently from 01b5ce7 to fe96dd0 Compare July 18, 2025 13:19
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 3 times, most recently from 94af0a7 to 11ff887 Compare July 21, 2025 13:04
@e-n-0 e-n-0 mentioned this pull request Jul 21, 2025
7 tasks
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch 6 times, most recently from ea12c22 to e45de48 Compare July 21, 2025 14:23
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Jul 23, 2025

Copy link
Copy Markdown

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 72aec1f | Was this helpful? Give us feedback!

@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch from 16831cf to f2d0a23 Compare July 23, 2025 11:39
@e-n-0
e-n-0 force-pushed the flavien/envoy/rel-env branch from f2d0a23 to 72aec1f Compare July 23, 2025 12:17
@e-n-0
e-n-0 marked this pull request as ready for review July 23, 2025 12:17
@e-n-0
e-n-0 requested review from a team as code owners July 23, 2025 12:17
@e-n-0
e-n-0 requested a review from eliottness July 23, 2025 12:18
@e-n-0

e-n-0 commented Jul 24, 2025

Copy link
Copy Markdown
Member Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Jul 24, 2025

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2025-07-24 09:47:42 UTC ℹ️ Start processing command /merge


2025-07-24 09:47:52 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2025-07-24 11:18:11 UTC ℹ️ MergeQueue: merge request added to the queue

The expected merge time in main is approximately 13m (p90).


2025-07-24 11:32:32 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue
dd-mergequeue Bot merged commit afc85f1 into main Jul 24, 2025
229 of 234 checks passed
@dd-mergequeue
dd-mergequeue Bot deleted the flavien/envoy/rel-env branch July 24, 2025 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apm:ecosystem contrib/* related feature requests or bugs mergequeue-status: done

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants