forked from open-telemetry/opentelemetry-collector-contrib
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Merge main to prod #3834
Merged
Merge main to prod #3834
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The same string can be found in metadata.Type Signed-off-by: Alex Boten <aboten@lightstep.com>
) Updating golangci-lint raises the following warnings: ``` integration_test.go:385:2: if-return: redundant if ...; err != nil check, just return error instead. (revive) if err := recordsWaitAndCheck(func() (recordsCheckable, as.Error) { return c.Query(queryPolicy, geoStm1) }); err != nil { return err } integration_test.go:145:2: empty-block: this block is empty, you can remove it (revive) for range chk.Results() { } scraper.go:112:36: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) func (r *aerospikeReceiver) scrape(ctx context.Context) (pmetric.Metrics, error) { ^ config.go:54:72: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint) return multierr.Append(allErrs, fmt.Errorf("%w: %s", errBadEndpoint, err)) ^ config.go:63:71: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint) allErrs = multierr.Append(allErrs, fmt.Errorf("%w: %s", errBadPort, err)) ^ config.go:84:78: non-wrapping format verb for fmt.Errorf. Use `%w` to format errors (errorlint) allErrs = multierr.Append(allErrs, fmt.Errorf("%w: %s", errFailedTLSLoad, err)) ``` Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
…n the `seconds` unit is provided. (open-telemetry#22001) Fix initialization of the default histogram buckets when the seconds unit is provided.
…y#22856) Updating golangci-lint raises the following warnings: ``` factory.go:26:2: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) ctx context.Context, ^ logs.go:133:51: unused-parameter: parameter 'host' seems to be unused, consider removing or renaming it as _ (revive) func (l *logsReceiver) Start(ctx context.Context, host component.Host) error { ^ logs.go:140:33: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) func (l *logsReceiver) Shutdown(ctx context.Context) error { ``` Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
…elemetry#22857) Updating golangci-lint raises the following warnings: ``` receiver.go:42:53: unused-parameter: parameter 'host' seems to be unused, consider removing or renaming it as _ (revive) func (m *metricReceiver) Start(ctx context.Context, host component.Host) error { ^ receiver.go:49:35: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) func (m *metricReceiver) Shutdown(ctx context.Context) error { ``` Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
Updating golangci-lint raises many unused-parameter errors, this is addressing some of those. Note in some cases, the context passed into start methods wasn't used, I fixed this. Linked issue: open-telemetry#20424 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
Linked issue: open-telemetry#20424 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
Linked to: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
…ons (open-telemetry#22763) * Replace StandardTypeGetter with type-specific instances * changelog
Created Metadata file and added stability + type changes. Link to tracking Issue: open-telemetry#21213
This PR updates the component to generate the status table using mdatagen. Linked issue: open-telemetry#21213 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
…ry#22942) dependabot updates Tue May 30 15:26:51 UTC 2023 Bump cloud.google.com/go/pubsub from 1.30.0 to 1.31.0 in /exporter/googlecloudpubsubexporter Bump cloud.google.com/go/pubsub from 1.30.0 to 1.31.0 in /receiver/googlecloudpubsubreceiver Bump github.com/Azure/azure-amqp-common-go/v4 from 4.1.0 to 4.2.0 in /receiver/azureeventhubreceiver Bump github.com/Azure/azure-event-hubs-go/v3 from 3.5.0 to 3.6.0 in /receiver/azureblobreceiver Bump github.com/Azure/azure-event-hubs-go/v3 from 3.5.0 to 3.6.0 in /receiver/azureeventhubreceiver Bump github.com/ClickHouse/clickhouse-go/v2 from 2.9.3 to 2.10.0 in /exporter/clickhouseexporter Bump github.com/DataDog/agent-payload/v5 from 5.0.84 to 5.0.85 in /exporter/datadogexporter Bump github.com/DataDog/opentelemetry-mapping-go/pkg/otlp/attributes from 0.2.1 to 0.2.3 in /exporter/datadogexporter Bump github.com/DataDog/opentelemetry-mapping-go/pkg/quantile from 0.2.1 to 0.2.3 in /exporter/datadogexporter Bump github.com/SAP/go-hdb from 1.2.6 to 1.3.4 in /receiver/saphanareceiver Bump github.com/SAP/go-hdb from 1.2.6 to 1.3.4 in /receiver/sqlqueryreceiver Bump github.com/aliyun/aliyun-log-go-sdk from 0.1.45 to 0.1.46 in /exporter/alibabacloudlogserviceexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /exporter/awscloudwatchlogsexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /exporter/awsemfexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /exporter/awsxrayexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /exporter/datadogexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /exporter/kafkaexporter Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /extension/observer/ecsobserver Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/awsutil Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/cwlogs Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/k8s Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/proxy Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/xray Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/aws/xray/testdata/sampleapp Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /internal/metadataproviders Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /processor/resourcedetectionprocessor Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /receiver/awscontainerinsightreceiver Bump github.com/aws/aws-sdk-go from 1.44.270 to 1.44.271 in /receiver/awsxrayreceiver Bump github.com/docker/docker from 24.0.1+incompatible to 24.0.2+incompatible in /extension/observer/dockerobserver Bump github.com/docker/docker from 24.0.1+incompatible to 24.0.2+incompatible in /internal/docker Bump github.com/docker/docker from 24.0.1+incompatible to 24.0.2+incompatible in /internal/metadataproviders Bump github.com/docker/docker from 24.0.1+incompatible to 24.0.2+incompatible in /processor/k8sattributesprocessor Bump github.com/docker/docker from 24.0.1+incompatible to 24.0.2+incompatible in /receiver/dockerstatsreceiver Bump github.com/snowflakedb/gosnowflake from 1.6.18 to 1.6.21 in /receiver/sqlqueryreceiver Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common from 1.0.664 to 1.0.666 in /exporter/tencentcloudlogserviceexporter Bump go.mongodb.org/atlas from 0.26.0 to 0.28.0 in /receiver/mongodbatlasreceiver Bump go.opentelemetry.io/collector/cmd/builder from 0.77.0 to 0.78.2 in /internal/tools Bump go.opentelemetry.io/otel from 1.15.1 to 1.16.0 in /internal/aws/metrics Bump go.opentelemetry.io/otel from 1.15.1 to 1.16.0 in /receiver/simpleprometheusreceiver/examples/federation/prom-counter Bump go.opentelemetry.io/otel/exporters/prometheus from 0.38.1 to 0.39.0 in /receiver/simpleprometheusreceiver/examples/federation/prom-counter Bump go.opentelemetry.io/otel/metric from 0.38.1 to 1.16.0 in /receiver/simpleprometheusreceiver/examples/federation/prom-counter Bump go.opentelemetry.io/otel/sdk/metric from 0.38.1 to 0.39.0 in /receiver/simpleprometheusreceiver/examples/federation/prom-counter
Created Metadata file and added stability + type changes. Link to tracking Issue: open-telemetry#21213
Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
Signed-off-by: Alex Boten <aboten@lightstep.com>
…elemetry#22831) Enable SHA-256 as hashing algorithm by default for attributesprocessor hashing action
* [pkg/stanza]: return raw Windows events as strings * [pkg/stanza] remove unnecessary windows build tags
This is waiting on a permission issue to be resolved. Disabling for now to reduce the noise. Note: I've also updated the repository to point to the correct one, for the future where the tidy job is re-enabled. Linked issue: open-telemetry#22953 Signed-off-by: Alex Boten <aboten@lightstep.com>
Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com>
…-telemetry#22086) Currently the only way to configure the OracleDB receiver is to manually define a custom datasource string. This string must have a specific format with specific pieces of information. It's easier for a user to identify information in smaller sizes, and leave the formatting of the connection string to the receiver itself. This change allows the user to define their connection information in whichever format they prefer.
…lemetry#22848) * [receiver/splunk_hec] Fix reusing the same splunkhecreiver between logs and metrics --------- Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
…2e test (open-telemetry#22845) Fix the k8sattribute processor e2e test bugs, here are two enhancements start the otlp receiver after all telemetrygen pods are running change the telemetrygen job to keep running for a long time Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>
…emetry#22799) Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com>
…nd fix breaking changes (open-telemetry#22952) Upgrade opentelemetry-mapping-go version and fix breaking changes Link to tracking Issue: Fixes open-telemetry#22917
Fixes open-telemetry#22958 --------- Signed-off-by: Alex Boten <aboten@lightstep.com>
…-telemetry#22751) * [chore][receiver/jmx] Fix flaky integration test
…ration test (open-telemetry#23003) * [chore][receiver/mysql] Migrate away from timing based setup in integration test
…rom metric processing (open-telemetry#22752) [chore][receiver/zookeeper] Refactor to separate connection/request logic from metric processing
…2968) * [pkg/ottl] Added ottl functions for hashing strings * [pkg/ottl] Return nil on errors * Documentation and test changes for the hash functions Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com> --------- Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
…ct (open-telemetry#22833) Verify that `retry_on_failure.enabled` config option makes the receiver correctly re-send all the logs temporarily rejected by the next consumer
On failure to read job metrics, this type can be nil causing a panic.
* [bugfix][jmxreceiver] Cancel rogue subprocess
…try#23148) The previous links are broken. I've added the latest, so that reference docs can generate properly.
Adds running the lifecycles in parallel and named subtests to improve visibility into time spent --------- Co-authored-by: Alex Boten <aboten@lightstep.com>
…pen-telemetry#23137) translator.go was copied into the DataDog/opentelemetry-mapping-go repo in a previous PR, which is now available in the v0.3.0 release of that repo. Update the dependency version and replace imports of translator.go with the opentelemetry-mapping-go equivalent. --------- Co-authored-by: Yang Song <songy23@users.noreply.github.com>
--------- Signed-off-by: Alex Boten <aboten@lightstep.com>
The following commands were run to prepare this release: - make chlog-update VERSION=v0.79.0 - sed -i.bak s/0.78.0/0.79.0/g versions.yaml - make multimod-prerelease - make multimod-sync
Signed-off-by: Alex Boten <aboten@lightstep.com>
Implemented the component. It works by standing up a server with a route to accept post requests from the webhook its configured to receive from. These requests are then converted into otel logs and shipped. Can handle gzip encoded payloads. Link to tracking Issue: 18101
* Fix cri-o log format time layout See related discussion and PR in open-telemetry/opentelemetry-helm-charts#718 * Create fix-cri-o-log-format-time-layout.yaml Add changelog file * Update fix-cri-o-log-format-time-layout.yaml Remove external issue as it seems it does not validate * Update fix-cri-o-log-format-time-layout.yaml Add issue number
…rentPid (open-telemetry#22920) * change method to get parent Pid in function parentPid * refactor - remove if * fix after review - remove assignment * fix tests * add chlog file * update chlog file
…y#23207) Bump github.com/DataDog/datadog-agent/pkg/trace from 0.46.0-rc.1 to 0.46.0-rc.2 in /exporter/datadogexporter Bump github.com/GoogleCloudPlatform/opentelemetry-operations-go/detectors/gcp from 1.13.1 to 1.15.0 in /processor/resourcedetectionprocessor Bump github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector from 0.37.1 to 0.39.0 in /exporter/googlecloudexporter Bump github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector from 0.37.1 to 0.39.0 in /exporter/googlemanagedprometheusexporter Bump github.com/GoogleCloudPlatform/opentelemetry-operations-go/exporter/collector/googlemanagedprometheus from 0.37.1 to 0.39.0 in /exporter/googlemanagedprometheusexporter Bump github.com/SAP/go-hdb from 1.2.6 to 1.3.6 in /receiver/sqlqueryreceiver Bump github.com/SAP/go-hdb from 1.3.4 to 1.3.6 in /receiver/saphanareceiver Bump github.com/aliyun/aliyun-log-go-sdk from 0.1.46 to 0.1.49 in /exporter/alibabacloudlogserviceexporter Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /exporter/awscloudwatchlogsexporter Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /exporter/awsemfexporter Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /exporter/awsxrayexporter Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /exporter/kafkaexporter Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /extension/observer/ecsobserver Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/awsutil Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/cwlogs Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/k8s Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/proxy Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/xray Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/aws/xray/testdata/sampleapp Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /internal/metadataproviders Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /processor/resourcedetectionprocessor Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /receiver/awscontainerinsightreceiver Bump github.com/aws/aws-sdk-go from 1.44.274 to 1.44.277 in /receiver/awsxrayreceiver Bump github.com/golangci/golangci-lint from 1.52.2 to 1.53.2 in /internal/tools Bump github.com/grpc-ecosystem/grpc-gateway/v2 from 2.15.2 to 2.16.0 in /receiver/opencensusreceiver Bump github.com/hashicorp/consul/api from 1.20.0 to 1.21.0 in /internal/metadataproviders Bump github.com/hashicorp/consul/api from 1.20.0 to 1.21.0 in /processor/resourcedetectionprocessor Bump github.com/influxdata/influxdb-observability/common from 0.3.14 to 0.4.2 in /exporter/influxdbexporter Bump github.com/influxdata/influxdb-observability/common from 0.3.14 to 0.4.2 in /receiver/influxdbreceiver Bump github.com/influxdata/influxdb-observability/influx2otel from 0.3.14 to 0.4.2 in /receiver/influxdbreceiver Bump github.com/influxdata/influxdb-observability/otel2influx from 0.3.14 to 0.4.2 in /exporter/influxdbexporter Bump github.com/mattn/go-sqlite3 from 1.14.16 to 1.14.17 in /extension/storage Bump github.com/shirou/gopsutil/v3 from 3.23.4 to 3.23.5 in /receiver/jmxreceiver Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /extension/observer Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /internal/aws/awsutil Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /internal/aws/k8s Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /internal/aws/metrics Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /internal/docker Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /internal/metadataproviders Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /pkg/experimentalmetricmetadata Bump github.com/stretchr/testify from 1.8.3 to 1.8.4 in /pkg/winperfcounters Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common from 1.0.670 to 1.0.672 in /exporter/tencentcloudlogserviceexporter Bump go.opentelemetry.io/build-tools/checkdoc from 0.7.0 to 0.8.0 in /internal/tools Bump go.opentelemetry.io/build-tools/chloggen from 0.7.0 to 0.8.0 in /internal/tools Bump go.opentelemetry.io/build-tools/issuegenerator from 0.7.0 to 0.8.0 in /internal/tools Bump go.opentelemetry.io/build-tools/multimod from 0.7.0 to 0.8.0 in /internal/tools --------- Signed-off-by: Alex Boten <aboten@lightstep.com> Co-authored-by: Alex Boten <aboten@lightstep.com>
…#22992) Fix add client context and use other http settings defined in config. I've used ToServer() and ToListener(), which is similiar to otlp http receiver. They add a bunch of common logic, such as handling client context.
…try#23209) - Describe transformations currently applied in the Prometheus receiver - Fix `crypto.dogecoin.value_dollars`-> `crypto_dogecoin_value_dollars` typo
…y#23224) dependabot updates Wed Jun 7 17:40:37 UTC 2023 Bump github.com/DataDog/datadog-agent/pkg/trace from 0.46.0-rc.2 to 0.47.0-devel in /exporter/datadogexporter Bump github.com/influxdata/influxdb-observability/common from 0.4.2 to 0.4.3 in /exporter/influxdbexporter Bump github.com/influxdata/influxdb-observability/common from 0.4.2 to 0.4.3 in /receiver/influxdbreceiver Bump github.com/influxdata/influxdb-observability/influx2otel from 0.4.2 to 0.4.3 in /receiver/influxdbreceiver Bump github.com/influxdata/influxdb-observability/otel2influx from 0.4.2 to 0.4.3 in /exporter/influxdbexporter Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common from 1.0.672 to 1.0.673 in /exporter/tencentcloudlogserviceexporter
songy23
requested review from
mx-psi,
gbbr,
dineshg13 and
liustanley
as code owners
June 7, 2023 21:21
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Link to tracking Issue:
Testing:
Documentation: