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

[chore] Add replaces for new modules in core #37043

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

dmitryax
Copy link
Member

@dmitryax dmitryax commented Jan 6, 2025

To unblock collector core dependency update. See https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/12639934611/job/35222671837?pr=37037

Add the replace statements to all modules that use deprecated modules by running the following commands:

make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/extension/xextension=go.opentelemetry.io/collector/extension/[email protected]"
make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/scraper/scraperhelper=go.opentelemetry.io/collector/scraper/[email protected]"

@dmitryax
Copy link
Member Author

dmitryax commented Jan 6, 2025

They can be removed after running updating the core dependency

@dmitryax dmitryax force-pushed the overrides-for-new-modules branch from 6c568e4 to 8d0584c Compare January 6, 2025 23:48
@dmitryax dmitryax force-pushed the overrides-for-new-modules branch 2 times, most recently from 7b36c61 to 3119dc0 Compare January 7, 2025 00:05
@dmitryax dmitryax force-pushed the overrides-for-new-modules branch from 3119dc0 to 09796c2 Compare January 7, 2025 00:34
@dmitryax dmitryax force-pushed the overrides-for-new-modules branch from 09796c2 to 71d10d9 Compare January 7, 2025 01:01
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we usually address these by updating the packages in core to using v0.0.0 instead (whatever is in main)

@dmitryax
Copy link
Member Author

dmitryax commented Jan 7, 2025

Don't we usually address these by updating the packages in core to using v0.0.0 instead (whatever is in main)

Can be done that way as well. Not a big difference. Just seems like more back and forths

Run the following commands:
```
make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/extension/xextension=go.opentelemetry.io/collector/extension/[email protected]"
make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/scraper/scraperhelper=go.opentelemetry.io/collector/scraper/[email protected]"
```
@dmitryax dmitryax force-pushed the overrides-for-new-modules branch from 71d10d9 to 9c25be1 Compare January 7, 2025 01:51
@dmitryax dmitryax merged commit c35ebf0 into open-telemetry:main Jan 7, 2025
161 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 7, 2025
@dmitryax dmitryax deleted the overrides-for-new-modules branch January 8, 2025 19:21
AkhigbeEromo pushed a commit to sematext/opentelemetry-collector-contrib that referenced this pull request Jan 13, 2025
To unblock collector core dependency update. See
https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/12639934611/job/35222671837?pr=37037

Add the replace statements to all modules that use deprecated modules by
running the following commands:
```
make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/extension/xextension=go.opentelemetry.io/collector/extension/[email protected]"
make for-all CMD="go mod edit -replace go.opentelemetry.io/collector/scraper/scraperhelper=go.opentelemetry.io/collector/scraper/[email protected]"
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
connector/datadog exporter/alertmanager exporter/alibabacloudlogservice Alibaba components exporter/awscloudwatchlogs awscloudwatchlogs exporter exporter/awsemf awsemf exporter exporter/awskinesis exporter/awss3 exporter/awsxray exporter/azuredataexplorer exporter/azuremonitor exporter/carbon exporter/cassandra exporter/clickhouse exporter/coralogix exporter/datadog Datadog components exporter/dataset exporter/doris exporter/elasticsearch exporter/file exporter/googlecloud exporter/googlecloudpubsub exporter/googlemanagedprometheus Google Managed Prometheus exporter exporter/honeycombmarker exporter/influxdb exporter/kafka exporter/kinetica exporter/loadbalancing exporter/logicmonitor exporter/logzio exporter/loki Loki Exporter exporter/mezmo exporter/opencensus exporter/opensearch exporter/otelarrow exporter/prometheus exporter/prometheusremotewrite exporter/pulsar exporter/rabbitmq exporter/sapm exporter/sentry exporter/signalfx exporter/splunkhec exporter/sumologic exporter/syslog exporter/tencentcloudlogservice exporter/zipkin extension/storage internal/otelarrow internal/splunk pkg/datadog pkg/stanza processor/logstransform Logs Transform processor processor/routing Routing processor receiver/azureeventhub receiver/filelog receiver/googlecloudpubsub receiver/jmx JMX Receiver receiver/journald receiver/kafka receiver/mongodbatlas receiver/namedpipe receiver/otlpjsonfile receiver/prometheus Prometheus receiver receiver/signalfx receiver/splunkhec receiver/sqlquery SQL query receiver receiver/syslog receiver/tcplog receiver/udplog receiver/windowseventlog testbed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants