Skip to content

Conversation

@cmacknz
Copy link
Member

@cmacknz cmacknz commented Mar 10, 2025

This is a branch from #6912 that is testing the fix from the dependency conflict coming from elastic/beats#43108.

It builds locally, making sure CI is happy with it.

@mergify
Copy link
Contributor

mergify bot commented Mar 10, 2025

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b cmacknz/test-update-otel-v0.120.0 upstream/cmacknz/test-update-otel-v0.120.0
git merge upstream/main
git push upstream cmacknz/test-update-otel-v0.120.0

@mergify
Copy link
Contributor

mergify bot commented Mar 10, 2025

This pull request does not have a backport label. Could you fix it @cmacknz? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@cmacknz cmacknz changed the title Cmacknz/test update otel v0.120.0 Test Update OTel to v0.120.0 with Beats fix PR Mar 10, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 10, 2025

💔 Build Failed

Failed CI Steps

History

cc @cmacknz

@cmacknz
Copy link
Member Author

cmacknz commented Mar 10, 2025

panic: proto: duplicate enum registered: io.prometheus.client.MetricType
 
goroutine 1 [running]:
github.com/gogo/protobuf/proto.RegisterEnum(...)
  /opt/buildkite-agent/go/pkg/mod/github.com/gogo/[email protected]/proto/properties.go:520
github.com/prometheus/prometheus/prompb/io/prometheus/client.init.0()
  /opt/buildkite-agent/go/pkg/mod/github.com/prometheus/[email protected]/prompb/io/prometheus/client/metrics.pb.go:965 +0x359

Looks like the upstream and replaced prometheus packages have a conflict in an init function trying register something global.

@cmacknz cmacknz changed the title Test Update OTel to v0.120.0 with Beats fix PR Test updating OTel to v0.120.0 with Beats prometheus fix PR Mar 10, 2025
@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

@cmacknz cmacknz closed this Mar 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants