[chore] [receiver/filelog] Add test to verify receiver consume contract #22833
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
dmitryax
force-pushed
the
try-contract-checker
branch
3 times, most recently
from
659368c
to
0f33fd2
Compare
dmitryax
changed the title
Verify that `retry_on_failure.enabled` config option makes the receiver correctly re-send all the logs temporarily rejected by the next consumer
dmitryax
force-pushed
the
try-contract-checker
branch
from
0f33fd2
to
b58d792
Compare
djaglowski
approved these changes
Jun 5, 2023
evan-bradley
approved these changes
Jun 5, 2023
|
@tigrannajaryan they do fail, yes |
songy23
added a commit
to DataDog/opentelemetry-collector-contrib
that referenced
this pull request
Jun 8, 2023
* [chore] remove typeStr (open-telemetry#22854) The same string can be found in metadata.Type Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore][receiver/aerospike] addressing lint issues (open-telemetry#22855) 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> * [chore] add dbstorage to cmd/otelcontribcol (open-telemetry#22837) * [spanmetrics] Fix initialization of the default histogram buckets when the `seconds` unit is provided. (open-telemetry#22001) Fix initialization of the default histogram buckets when the seconds unit is provided. * [chore][receiver/awscloudwatch] addressing lint issues (open-telemetry#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> * [chore][receiver/awscloudwatchmetrics] addressing lint issues (open-telemetry#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> * [chore] addressing lint issues (open-telemetry#22862) 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> * [chore] remove superfluous else (open-telemetry#22918) Linked issue: open-telemetry#20424 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] remove redundant ifs (open-telemetry#22916) Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] lint issues: redefined built-in funcs (open-telemetry#22915) Linked to: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com> * [pkg/ottl] Replace StandardTypeGetter with type-specific implementations (open-telemetry#22763) * Replace StandardTypeGetter with type-specific instances * changelog * [chore][receiver/wavefront] wavefront metadata (open-telemetry#21669) Created Metadata file and added stability + type changes. Link to tracking Issue: open-telemetry#21213 * [exporter/f5cloud] use generated status header (open-telemetry#22091) 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> * [chore] dependabot updates Tue May 30 15:26:55 UTC 2023 (open-telemetry#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 * [chore][receiver/hostmetrics] create metadata (open-telemetry#21827) Created Metadata file and added stability + type changes. Link to tracking Issue: open-telemetry#21213 * [chore] more unused params (open-telemetry#22922) Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] remove unreachable code (open-telemetry#22946) Signed-off-by: Alex Boten <aboten@lightstep.com> * [processor/attributes] enable SHA-256 feature gate by default (open-telemetry#22831) Enable SHA-256 as hashing algorithm by default for attributesprocessor hashing action * [pkg/stanza] return raw Windows events as strings (open-telemetry#22863) * [pkg/stanza]: return raw Windows events as strings * [pkg/stanza] remove unnecessary windows build tags * [chore] disable renovatebot for gomod (open-telemetry#22955) 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> * [chore] more unused params (open-telemetry#22954) Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com> * [receiver/oracledb] Add a simpler alternative for configuration (open-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. * [receiver/splunk_hec] Fix reusing the shared splunkhecreiver (open-telemetry#22848) * [receiver/splunk_hec] Fix reusing the same splunkhecreiver between logs and metrics --------- Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com> * [processor/k8sattribute] fix occasionaly missing attributes bugs of e2e 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> * Update helm/kind-action action to v1.7.0 (open-telemetry#22960) * [receiver/splunkhec] support different splitting strategies (open-telemetry#22799) Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com> * Update otel/opentelemetry-collector-contrib Docker tag to v0.78.0 (open-telemetry#22964) * Update dockerfile deps (open-telemetry#22959) * [exporter/datadogexporter] Upgrade opentelemetry-mapping-go version and fix breaking changes (open-telemetry#22952) Upgrade opentelemetry-mapping-go version and fix breaking changes Link to tracking Issue: Fixes open-telemetry#22917 * [chore] downgrade dep (open-telemetry#22963) Fixes open-telemetry#22958 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] add error check to test (open-telemetry#22947) --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [receiver/vcenter] Add vm mem util (open-telemetry#20918) * [receiver/solacereceiver]: Updates to Solace Receiver for Solace PubSub+ Event Broker 10.4.1 (open-telemetry#22809) * Added new unmarshaller for egress spans SOL-88023 * Refactored unmarshaller separating out receive and egress, added egress SOL-86715 * Set the send span name to <endpoint> send SOL-88033 * Updates for latest architecture changes Use send.outcome instead of send_operation.outcome Use (anonymous) in send span name when given an anonymous topic endpoint or queue SOL-88025 * Don't error log each trace SOL-92755 * Increment span count based on number of received spans, not span messages SOL-92890 * Fixed reference error causing failing unit test * Fixed failing unit tests after merge * Added changelog * Fixed make lint errors New copyright too * [chore] Skip non-arm compatible integration tests (open-telemetry#22802) Nearly all integration tests in this repo are now compatible with arm processors. There are two remaining tests that are not, and these do not appear to have a clear path forward. Skipping these tests allows those of us with arm processors to locally run make integration-tests-with-cover. * [exporter/awsxray] Support new X-Ray specific service name attributes as Segment name (open-telemetry#22835) XRayExporter will be extended to support the new AWS specific service name attributes changed in open-telemetry/opentelemetry-java-contrib#802 for X-Ray trace segment name. The new Span attribute aws.local.service will be set as Server kind segment name, and aws.remote.service attribute will set as Client kind subsegment name if the attribute exists. * [all] Remove more unused params (open-telemetry#22971) Linked issue: open-telemetry#20424 Signed-off-by: Alex Boten <aboten@lightstep.com> * [exporter/splunk_hec] Apply compression if it's enabled in the config (open-telemetry#22969) The compression used to be enabled only if the payload size was greater than 1.5KB which significantly complicated the logic and made it hard to test. This change makes the compression unconditionally applied to the payload if it's enabled in the config. The benchmarking shows improvements in the throughput and CPU usage for large payloads and expected degradation for small payloads which is acceptable given that it's not a common case. * [processor/k8sattribute] add fatsheep9146 as codeowner (open-telemetry#22973) Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com> * [receiver/*] Fix lint issues (open-telemetry#22978) Fix lint issues in github.com/golangci/golangci-lint 1.52.2 * Update attribute processor's readme (open-telemetry#20855) * Update readme * Update test config * Remove "resource_attributes" from readme. Add "attributes" and "libraries" as invalid metrics filters. * [receiver/purefa] Some endpoints from purefareceiver were having trouble to scraped. (open-telemetry#21100) * [receiver/purefa] Some endpoints were not being scraped and this new code fixes * [receiver/purefa] Refactor a commom label present on endpoints * [receiver/purefa] re-adding the righ value with semantic conventions for the labels * [receiver/purefa] Mock a test http server to expose metrics and use the exporter to scrape * [receiver/purefa] Add the check to the return of the mock response * [receiver/purefa] Resolve conflict on test file * [correction/readme] Resolving conflict with lint errors * Revert "[receiver/purefa] Mock a test http server to expose metrics and use the exporter to scrape" This reverts commit cb8d199. * [receiver/purefa] Remove duplicated host label from volumes and hosts * [receiver/purefa] Remove duplicated host label from volumes and hosts w/ lint correction * [chore] more linting fixes (open-telemetry#22979) * [chore] more linting fixes mostly changes are unused parameters. a couple of them are false positives on empty for loops that i added a nolint to. there's also a redundant if and a couple of non-wrapping format verb issues fixed --------- Signed-off-by: Alex Boten <aboten@lightstep.com> Co-authored-by: Yang Song <songy23@users.noreply.github.com> * [chore]Port postgresql receiver integration test to new framework (open-telemetry#22967) * [chore][scraperinttest] Add support for NetworkRequest (open-telemetry#22790) * [chore][scraperinttest] Add support for NetworkRequest and migrate flink and kafka * [exporter/splunk_hec] Fix usage of `max_event_size` option (open-telemetry#22984) Make sure the `max_event_size` option is used to drop events larger than the provided value instead of using it for batch size --------- Co-authored-by: Antoine Toulme <antoine@toulme.name> * Unwrap lumberjack.logger with buffered writer (open-telemetry#22972) * Unwrap lumberjack.logger with buffered writer - change the buildFileWriter factory function - adapt the TestBuildFileWriter in factory_test * Set issue to int in changelog yaml * [chore][scraperinttest] Capture and logs errors from scrapers (open-telemetry#22981) * [receiver/mysql]: add mysql.uptime metric (open-telemetry#14747) feat(mysqlreceiver): add mysql.uptime metric Signed-off-by: Dominik Rosiek <drosiek@sumologic.com> * [chore] [exporter/splunk_hec] Remove redundant bufState struct (open-telemetry#22986) Most of the stuff was moved out from this struct, so it's not needed anymore. * [connector/*] generate status from metadata.yaml (open-telemetry#22800) This PR adds the ability to generate status from metadata.yaml for connectors. The format for connectors differs in that stability is per connection rather than per signal as it is for other components. In addition to the code to generate status, this uses the count connector as a demonstration of how this will work in the real world. * [chore] update linter version (open-telemetry#22983) Fixes open-telemetry#20424 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] remove use of deprecated wait.Poll (open-telemetry#23001) This has been replaced by `PollUntilContextTimeout`. Fixes open-telemetry#22719 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] Sort attributes prior to writing golden metrics files (open-telemetry#22718) * [k8sattributesprocessor] docs: update rbac information (open-telemetry#23013) * Propose @gbbr and @Frapschen as triagers (open-telemetry#22993) * Add the AWS distribution to the list of distributions (open-telemetry#22989) Add the AWS Distro for OpenTelemetry Collector (ADOT) to the list of distributions * [chore] [exporter/splunk_hec] Add max_event_size config option description to README (open-telemetry#22999) The max_event_size config option was added to the splunkhec exporter, but it is not documented in the README. This change adds README documentation for this option. --------- Co-authored-by: Antoine Toulme <antoine@toulme.name> * [connector/spanmetrics] generate status with mdatagen (open-telemetry#23017) This PR updates the spanmetricsconnector to use status generated by mdatagen. There was a discrepancy between the readme and the code as to the stability level for this connector. * [chore] Tag postgresql integration test with build tag (open-telemetry#23011) * Update Kusto Go SDK version , fixes issue open-telemetry#22771 (open-telemetry#22933) The underlying SDK has an update to fix an issue where the network connections are flaky. To get the metadata to connect to the service , a HTTP metadata call is issued. If this call fails , it is not retried and the init is interrupted till restart. The underlying SDK fixes it by reset of the lock that was causing this behavior * [receiver/dockerstats] adding renamed cpu_utilization metric (open-telemetry#22823) This change starts the process for deprecating the metric container.cpu.percent in favor of container.cpu.utilization to adhere to semantic convention. At this step introducing the new metric name disabled by default, leaving the old metric enabled * [chore] enable coverage for unit tests (open-telemetry#22966) This re-enables unit test coverage for the repository. Coverage will not immediately include integration tests. Fixes open-telemetry#8394 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] Upgrade prometheus/common and update tls config validation (open-telemetry#23025) The TLS validation that we have was added in the prometheus library in the unmarshaling step making our validation not utilized. This change removes the duplicating validation and updates the tests. * [receiver/httpcheck] remove default endpoint (open-telemetry#23018) Fixes open-telemetry#22995 This removes the default endpoint http://localhost:80 and makes the endpoint setting required, as described in the README. I also used the function confighttp.NewDefaultHTTPClientSettings() instead of the struct literal confighttp.HTTPClientSettings{}, as it is recommended. * [connector/servicegraphconnector] generate status with mdatagen (open-telemetry#23012) In order to use the generated metadata I changed servicegraphprocessor.NewConnectorFactory to servicegraphprocessor.NewConnectorFactoryFunc which takes a Type and TracesToMetricStability and returns a function that returns a connector.Factory when called. This lead to some additional changes in the servicegraphprocessor tests and I introduced a factory test for the connector. * [mdatagen] better formatting for connector READMEs (open-telemetry#23036) [mdatagen] better formatting for connector readmes * [chore] Update the collector core dependency to the latest commit (open-telemetry#23023) * [receiver/postgresql] Fix race condition when capturing errors (open-telemetry#23033) * [receiver/httpcheck] Update default collection interval (open-telemetry#23037) The default collection interval was documented in the receiver's README as 60s, but in the code it was being set to 10s by default. This change updates the code to set the default interval to 60s. --------- Co-authored-by: Yang Song <songy23@users.noreply.github.com> * [chore] Updates to scrape duration (open-telemetry#23030) - This is to prepare for the future release of core that includes scraping from collector start. - Fixes accidental change to apache receiver's default collection interval. * [chore][exporter/clickhouse] align success path logging to other exporters (open-telemetry#23031) * [processor/servicegraph] consume traces even metric count is equal to 0 (open-telemetry#23029) Signed-off-by: 涂浩 <haotu@alauda.io> * [chore] dependabot updates Fri Jun 2 18:12:25 UTC 2023 (open-telemetry#23044) dependabot updates Fri Jun 2 18:12:22 UTC 2023 Bump github.com/aws/aws-sdk-go from 1.44.271 to 1.44.274 in /extension/observer/ecsobserver Bump github.com/aws/aws-sdk-go from 1.44.271 to 1.44.274 in /receiver/awscontainerinsightreceiver Bump github.com/shirou/gopsutil/v3 from 3.23.4 to 3.23.5 in /receiver/awscontainerinsightreceiver Bump github.com/tencentcloud/tencentcloud-sdk-go/tencentcloud/common from 1.0.666 to 1.0.670 in /exporter/tencentcloudlogserviceexporter Bump google.golang.org/api from 0.124.0 to 0.125.0 in /receiver/googlecloudspannerreceiver * issue-21715 fetching metric values with composite(all) dimension keys (open-telemetry#21716) Fixes open-telemetry#21715 * [chore] Remove several unnecessary custom dockerfiles (open-telemetry#23046) * Remove custom postgresql dockerfile * Remove custom mysql and postgres dockerfiles in sqlquery receiver * Remove custom rabbitmq dockerfile * Remove custom apache dockerfile * [chore] add metadata status checks on the distribution (open-telemetry#21275) Add a test checking that the status metadata of a component matches their presence in the otelcontribcol test suite. * [chore][jmxreceiver] Update flaky JMX receiver integration test (open-telemetry#22751) * [chore][receiver/jmx] Fix flaky integration test * [chore][receiver/mysql] Migrate away from timing based setup in integration test (open-telemetry#23003) * [chore][receiver/mysql] Migrate away from timing based setup in integration test * [chore] Document limitation in zookeeper receiver (open-telemetry#23062) * [chore][receiver/zookeeper] Refactor to separate connection/request from metric processing (open-telemetry#22752) [chore][receiver/zookeeper] Refactor to separate connection/request logic from metric processing * [pkg/stanza/fileconsumer] Fix issue where empty files caused errors (open-telemetry#22945) * [pkg/ottl] Added ottl functions for hashing strings (open-telemetry#22968) * [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> * [chore] [receiver/filelog] Add test to verify receiver consume contract (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 * [chore] Fix failing test case for flinkmetrics (open-telemetry#23143) On failure to read job metrics, this type can be nil causing a panic. * [bugfix][jmxreceiver] Cancel rogue subprocess (open-telemetry#23051) * [bugfix][jmxreceiver] Cancel rogue subprocess * [chore] [receiver/k8scluster] Fix links in containers.go (open-telemetry#23148) The previous links are broken. I've added the latest, so that reference docs can generate properly. * [chore] Improving life cycle tests reporting (open-telemetry#23145) Adds running the lifecycles in parallel and named subtests to improve visibility into time spent --------- Co-authored-by: Alex Boten <aboten@lightstep.com> * [exporter/datadog] Use v0.3.0 of opentelemetry-mapping-go for logs (open-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> * [chore] update core to 0.79.0 (open-telemetry#23180) --------- Signed-off-by: Alex Boten <aboten@lightstep.com> * [chore] Prepare release 0.79.0 (open-telemetry#23185) 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 * [chore] add Sumo distro to AWS S3 exporter and File Stats receiver (open-telemetry#23153) Reference: https://github.com/SumoLogic/sumologic-otel-collector/blob/v0.78.0-sumo-0/otelcolbuilder/.otelcol-builder.yaml * [chore] fix test which was missed (open-telemetry#23188) Signed-off-by: Alex Boten <aboten@lightstep.com> * Webhookevent client (open-telemetry#20751) 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 * [chore] Migrate jmx receiver integration test (open-telemetry#23066) * Fix cri-o log format time layout (open-telemetry#20533) * 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 * [receiver/hostmetrics] Change method to get parent Pid in function parentPid (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 * [chore] dependabot updates Tue Jun 6 23:06:32 UTC 2023 (open-telemetry#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> * Update otel/opentelemetry-collector-contrib Docker tag to v0.79.0 (open-telemetry#23206) * [receiver/datadog] add client info to datadogreceiver (open-telemetry#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. * [chore] [pkg/translator/prometheus] Update documentation (open-telemetry#23209) - Describe transformations currently applied in the Prometheus receiver - Fix `crypto.dogecoin.value_dollars`-> `crypto_dogecoin_value_dollars` typo * [chore] dependabot updates Wed Jun 7 17:40:39 UTC 2023 (open-telemetry#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 --------- Signed-off-by: Alex Boten <aboten@lightstep.com> Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com> Signed-off-by: Dominik Rosiek <drosiek@sumologic.com> Signed-off-by: 涂浩 <haotu@alauda.io> Co-authored-by: Alex Boten <aboten@lightstep.com> Co-authored-by: Antoine Toulme <antoine@lunar-ocean.com> Co-authored-by: Ruslan Kovalov <ruslan.kovalov@grafana.com> Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com> Co-authored-by: Bomin Rahmani <38274348+BominRahmani@users.noreply.github.com> Co-authored-by: OpenTelemetry Bot <107717825+opentelemetrybot@users.noreply.github.com> Co-authored-by: Mikołaj Świątek <mswiatek@sumologic.com> Co-authored-by: Curtis Robert <92119472+crobert-1@users.noreply.github.com> Co-authored-by: Dmitrii Anoshin <anoshindx@gmail.com> Co-authored-by: Ziqi Zhao <zhaoziqi9146@gmail.com> Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com> Co-authored-by: Mend Renovate <bot@renovateapp.com> Co-authored-by: padraic cunningham <pcunningham80@gmail.com> Co-authored-by: Michael Cardy <contact@mcardy.com> Co-authored-by: Daniel Jaglowski <jaglows3@gmail.com> Co-authored-by: Min Xia <xiami@amazon.com> Co-authored-by: Paulin Todev <paulin.todev@gmail.com> Co-authored-by: Diego Amaral <diego-ii@live.com> Co-authored-by: Antoine Toulme <antoine@toulme.name> Co-authored-by: Peter Zandbergen <peter.zandbergen@myhops.com> Co-authored-by: Dominik Rosiek <58699848+sumo-drosiek@users.noreply.github.com> Co-authored-by: Matthew Wear <matthew.wear@gmail.com> Co-authored-by: Povilas Versockas <povilas.versockas@coralogix.com> Co-authored-by: Pablo Baeyens <pbaeyens31+github@gmail.com> Co-authored-by: Dewald de Jager <DewaldDeJager@users.noreply.github.com> Co-authored-by: Ramachandran A G <106139410+ag-ramachandran@users.noreply.github.com> Co-authored-by: Carlos Castro <ccastro@newrelic.com> Co-authored-by: Andrzej Stencel <astencel@sumologic.com> Co-authored-by: Sean Marciniak <30928402+MovieStoreGuy@users.noreply.github.com> Co-authored-by: Ben Blackmore <bripkens@gmail.com> Co-authored-by: Hao Tu <haotu@alauda.io> Co-authored-by: Raji Muthupandian <rajim17@users.noreply.github.com> Co-authored-by: mrsillydog <ian.adams@observiq.com> Co-authored-by: Raj Nishtala <113392743+rnishtala-sumo@users.noreply.github.com> Co-authored-by: Fabrizio Ferri-Benedetti <fferribenedetti@splunk.com> Co-authored-by: Ibraheem Aboulnaga <13734402+IbraheemA@users.noreply.github.com> Co-authored-by: shalper2 <99686388+shalper2@users.noreply.github.com> Co-authored-by: Jonas Kello <jonaskello@users.noreply.github.com> Co-authored-by: JadziaMataj <72666663+JadziaMataj@users.noreply.github.com>
Caleb-Hurshman
pushed a commit
to observIQ/opentelemetry-collector-contrib
that referenced
this pull request
Jul 6, 2023
…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
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.
Verify that
retry_on_failure.enabledconfig option makes the receiver correctly re-send all the logs temporarily rejected by the next consumer