Skip to content

Commit

Permalink
Merge branch 'main' into 34188
Browse files Browse the repository at this point in the history
  • Loading branch information
hgaol authored Jan 25, 2025
2 parents 2213613 + 9841fe1 commit 4513bba
Show file tree
Hide file tree
Showing 652 changed files with 17,742 additions and 17,969 deletions.
27 changes: 27 additions & 0 deletions .chloggen/clickhouse_examples.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: enhancement

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: exporter/clickhouse

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Adding missing examples of DDL created by the clickhouse exporter

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [35903]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
27 changes: 27 additions & 0 deletions .chloggen/codeboten_add-support-for-headers.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: enhancement

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: opampsupervisor

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: add support for headers configuration for reporting own telemetry

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [37353]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
35 changes: 35 additions & 0 deletions .chloggen/datadogconnector-add-stats-obfuscation.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: enhancement

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: datadogconnector

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Support obfuscating sql queries in APM stats

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [37457]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext: |
Ensure that feature flags "enable_receive_resource_spans_v2" and "enable_operation_and_resource_name_logic_v2"
are also enabled on both Datadog Exporter and Datadog Connector so that span attributes are properly
mapped to span type and span resource in Datadog APM; otherwise spans and apm stats may not be
obfuscated and attributes on stats payloads may not match traces.
See https://docs.datadoghq.com/opentelemetry/schema_semantics/semantic_mapping/?tab=datadogexporter#mapping-opentelemetry-database-system-type-to-datadog-span-type
NOTE: Long/complex SQL queries may cause a performance impact on APM Stats calculation in Datadog Connector.
Consider implementing sampling in your pipeline prior to sending traces to Datadog Connector if you experience this.
# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: [user]
27 changes: 27 additions & 0 deletions .chloggen/receiver-otlpjson-token-attr.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: bug_fix

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: otlpjsonfilereceiver

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Include file attributes and append it to the log record

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [36641]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
27 changes: 27 additions & 0 deletions .chloggen/solace-receiver-validation-improvements.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: breaking

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: solacereceiver

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Update validation step to allow only one auth method.

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [36386]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext:

# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
30 changes: 30 additions & 0 deletions .chloggen/warn_on_dropped_dp.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
# Use this changelog template to create an entry for release notes.

# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
change_type: bug_fix

# The name of the component, or a single word describing the area of concern, (e.g. filelogreceiver)
component: signalfxexporter

# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
note: Warn on dropping metric data points when they have more than allowed dimension count

# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
issues: [37484]

# (Optional) One or more lines of additional information to render under the primary note.
# These lines will be padded with 2 spaces and then inserted directly into the document.
# Use pipe (|) for multiline entries.
subtext: |
The SignalFx exporter drops metric data points if they have more than 36 dimensions.
Currently, the exporter logs at debug level when this occurs.
With this change, the exporter will log at the warning level.
# If your change doesn't affect end users or the exported elements of any package,
# you should instead start your pull request title with [chore] or use the "Skip Changelog" label.
# Optional: The change log or logs in which this entry should be included.
# e.g. '[user]' or '[user, api]'
# Include 'user' if the change is relevant to end users.
# Include 'api' if there is a change to a library API.
# Default: '[user]'
change_logs: []
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ receiver/huaweicloudcesreceiver/ @open-telemetry
receiver/iisreceiver/ @open-telemetry/collector-contrib-approvers @Mrod1598 @pjanotti
receiver/influxdbreceiver/ @open-telemetry/collector-contrib-approvers @jacobmarble
receiver/jaegerreceiver/ @open-telemetry/collector-contrib-approvers @yurishkuro
receiver/jmxreceiver/ @open-telemetry/collector-contrib-approvers @atoulme
receiver/jmxreceiver/ @open-telemetry/collector-contrib-approvers @atoulme @rogercoll
receiver/journaldreceiver/ @open-telemetry/collector-contrib-approvers @sumo-drosiek @djaglowski
receiver/k8sclusterreceiver/ @open-telemetry/collector-contrib-approvers @dmitryax @TylerHelmuth @povilasv @ChrsMark
receiver/k8seventsreceiver/ @open-telemetry/collector-contrib-approvers @dmitryax @TylerHelmuth @ChrsMark
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/check-links.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -45,3 +45,4 @@ jobs:
uses: lycheeverse/lychee-action@f796c8b7d468feb9b8c0a46da3fac0af6874d374
with:
args: "--verbose --no-progress ${{needs.changedfiles.outputs.md_files}} ${{needs.changedfiles.outputs.yaml_files}} --config .github/lychee.toml"
failIfEmpty: false
4 changes: 2 additions & 2 deletions cmd/githubgen/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ go 1.22.0

require (
github.com/google/go-github/v68 v68.0.0
go.opentelemetry.io/collector/confmap v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/confmap/provider/fileprovider v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/confmap v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/confmap/provider/fileprovider v1.24.1-0.20250123125445-24f88da7b583
gopkg.in/yaml.v3 v3.0.1
)

Expand Down
8 changes: 4 additions & 4 deletions cmd/githubgen/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

12 changes: 6 additions & 6 deletions cmd/opampsupervisor/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ require (
github.com/knadh/koanf/v2 v2.1.2
github.com/open-telemetry/opamp-go v0.18.0
github.com/stretchr/testify v1.10.0
go.opentelemetry.io/collector/config/configopaque v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/config/configtls v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/confmap v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/confmap/provider/envprovider v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/confmap/provider/fileprovider v1.24.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/semconv v0.118.1-0.20250121185328-fbefb22cc2b3
go.opentelemetry.io/collector/config/configopaque v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/config/configtls v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/confmap v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/confmap/provider/envprovider v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/confmap/provider/fileprovider v1.24.1-0.20250123125445-24f88da7b583
go.opentelemetry.io/collector/semconv v0.118.1-0.20250123125445-24f88da7b583
go.uber.org/goleak v1.3.0
go.uber.org/zap v1.27.0
golang.org/x/sys v0.29.0
Expand Down
24 changes: 12 additions & 12 deletions cmd/opampsupervisor/go.sum

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

19 changes: 8 additions & 11 deletions cmd/opampsupervisor/supervisor/supervisor.go
Original file line number Diff line number Diff line change
Expand Up @@ -869,19 +869,16 @@ func (s *Supervisor) setupOwnMetrics(_ context.Context, settings *protobufs.Tele
} else {
s.logger.Debug("Enabling own metrics pipeline in the config")

port, err := s.findRandomPort()
if err != nil {
s.logger.Error("Could not setup own metrics", zap.Error(err))
return
data := map[string]any{
"MetricsEndpoint": settings.DestinationEndpoint,
"MetricsHeaders": []protobufs.Header{},
}

err = s.ownTelemetryTemplate.Execute(
&cfg,
map[string]any{
"PrometheusPort": port,
"MetricsEndpoint": settings.DestinationEndpoint,
},
)
if settings.Headers != nil {
data["MetricsHeaders"] = settings.Headers.Headers
}

err := s.ownTelemetryTemplate.Execute(&cfg, data)
if err != nil {
s.logger.Error("Could not setup own metrics", zap.Error(err))
return
Expand Down
21 changes: 18 additions & 3 deletions cmd/opampsupervisor/supervisor/supervisor_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,13 @@ func Test_onMessage(t *testing.T) {
},
},
OwnMetricsConnSettings: &protobufs.TelemetryConnectionSettings{
DestinationEndpoint: "http://localhost:4318",
DestinationEndpoint: "http://127.0.0.1:4318",
Headers: &protobufs.Headers{
Headers: []*protobufs.Header{
{Key: "testkey", Value: "testval"},
{Key: "testkey2", Value: "testval2"},
},
},
},
})

Expand Down Expand Up @@ -1125,7 +1131,13 @@ func TestSupervisor_setupOwnMetrics(t *testing.T) {
require.NoError(t, err)

configChanged := s.setupOwnMetrics(context.Background(), &protobufs.TelemetryConnectionSettings{
DestinationEndpoint: "localhost",
DestinationEndpoint: "http://127.0.0.1:4318",
Headers: &protobufs.Headers{
Headers: []*protobufs.Header{
{Key: "testkey", Value: "testval"},
{Key: "testkey2", Value: "testval2"},
},
},
})

expectedOwnMetricsSection := `
Expand All @@ -1137,7 +1149,10 @@ service:
exporter:
otlp:
protocol: http/protobuf
endpoint: localhost
endpoint: http://127.0.0.1:4318
headers:
"testkey": "testval"
"testkey2": "testval2"
`

assert.True(t, configChanged)
Expand Down
6 changes: 6 additions & 0 deletions cmd/opampsupervisor/supervisor/templates/owntelemetry.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,9 @@ service:
otlp:
protocol: http/protobuf
endpoint: {{.MetricsEndpoint}}
{{- if .MetricsHeaders}}
headers:
{{- range $k := .MetricsHeaders}}
"{{$k.Key}}": "{{$k.Value}}"
{{- end}}
{{- end}}
Loading

0 comments on commit 4513bba

Please sign in to comment.