Skip to content

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Sep 5, 2025

What does this PR do?

Enable some automated dependency management for EDOT SDK docker images in the elastic-agent repository using updatecli. The automation tracks latest releases from EDOT SDK repositories and creates pull requests to update image versions in Helm chart configurations.

Why is it important?

Previously, EDOT SDK image versions in elastic-agent had to be updated manually, as shown in #7327. This manual process was time-consuming and could lead to outdated dependencies.

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test

Disruptive User Impact

How to test this PR locally

$ GITHUB_TOKEN=$(gh auth token) \
   GITHUB_ACTOR=v1v \
   updatecli diff --config .ci/updatecli/updatecli-bump-edot.yml --values .ci/updatecli/values.d/scm.yml

and

--- a/.ci/updatecli/values.d/scm.yml
+++ b/.ci/updatecli/values.d/scm.yml
@@ -1,8 +1,8 @@
 scm:
   enabled: true
-  owner: elastic
+  owner: v1v
   repository: elastic-agent
-  branch: main
+  branch: test/updatecli-branch
   commitusingapi: true
   # begin updatecli-compose policy values
   user: 'github-actions[bot]'

produced

SUMMARY:

⚠ Bump golang-version to latest version:
	Source:
		✔ [elastic-otel-dotnet] Get latest Elastic OTEL .NET release
		✔ [elastic-otel-java] Get latest Elastic OTEL Java release
		✔ [elastic-otel-node] Get latest Elastic OTEL Node.js release
		✔ [elastic-otel-python] Get latest Elastic OTEL Python release
		✔ [otel-go-instrumentation] Get latest OpenTelemetry Go Instrumentation release
	Target:
		⚠ [update-dotnet-managed-otlp] Update Elastic OTEL .NET image in values.yaml
		⚠ [update-go-main] Update OpenTelemetry Go Instrumentation image in values.yaml
		⚠ [update-java-managed-otlp] Update Elastic OTEL Java image in values.yaml
		⚠ [update-nodejs-managed-otlp] Update Elastic OTEL Node.js image in values.yaml
		⚠ [update-python-managed-otlp] Update Elastic OTEL Python image in managed_otlp/values.yaml


Run Summary
===========
Pipeline(s) run:
  * Changed:	1
  * Failed:	0
  * Skipped:	0
  * Succeeded:	0
  * Total:	1

then if I applied the changes

$ GITHUB_TOKEN=$(gh auth token) \
   GITHUB_ACTOR=v1v \
   updatecli apply --config .ci/updatecli/updatecli-bump-edot.yml --values .ci/updatecli/values.d/scm.yml

SUMMARY:

⚠ Bump golang-version to latest version:
	Source:
		✔ [elastic-otel-dotnet] Get latest Elastic OTEL .NET release
		✔ [elastic-otel-java] Get latest Elastic OTEL Java release
		✔ [elastic-otel-node] Get latest Elastic OTEL Node.js release
		✔ [elastic-otel-python] Get latest Elastic OTEL Python release
		✔ [otel-go-instrumentation] Get latest OpenTelemetry Go Instrumentation release
	Target:
		⚠ [update-dotnet-managed-otlp] Update Elastic OTEL .NET image in values.yaml
		⚠ [update-go-main] Update OpenTelemetry Go Instrumentation image in values.yaml
		⚠ [update-java-managed-otlp] Update Elastic OTEL Java image in values.yaml
		⚠ [update-nodejs-managed-otlp] Update Elastic OTEL Node.js image in values.yaml
		⚠ [update-python-managed-otlp] Update Elastic OTEL Python image in managed_otlp/values.yaml


Run Summary
===========
Pipeline(s) run:
  * Changed:	1
  * Failed:	0
  * Skipped:	0
  * Succeeded:	0
  * Total:	1

One action to follow up:
  * https://github.com/v1v/elastic-agent/pull/11

Caveats

It removes the inline comments:

image

Shall we remove them? Or move them one line above so they are not removed by the updatecli automation?

Related issues

Questions to ask yourself

  • How are we going to support this in production?
  • How are we going to measure its adoption?
  • How are we going to debug this?
  • What are the metrics I should take care of?
  • ...

This is an automatic backport of pull request #9686 done by [Mergify](https://mergify.com).

@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@elasticmachine
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

cc @v1v

@v1v v1v merged commit ddf0e2c into 8.19 Sep 5, 2025
21 of 24 checks passed
@v1v v1v deleted the mergify/bp/8.19/pr-9686 branch September 5, 2025 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants