Skip to content

all: adapt to confighttp.ServerConfig change#45308

Merged
mx-psi merged 3 commits intoopen-telemetry:mainfrom
axw:confighttp-addrconfig
Jan 15, 2026
Merged

all: adapt to confighttp.ServerConfig change#45308
mx-psi merged 3 commits intoopen-telemetry:mainfrom
axw:confighttp-addrconfig

Conversation

@axw
Copy link
Copy Markdown
Contributor

@axw axw commented Jan 9, 2026

Description

open-telemetry/opentelemetry-collector#14248 introduces a breaking change to confighttp.ServerConfig, replacing the Endpoint field with NetAddr (confignet.AddrConfig).

Link to tracking issue

Relates to open-telemetry/opentelemetry-collector#14248

Testing

make gotest

Documentation

None at this stage. Code owners can update docs if/when they want to declare support.

Update to github.com/open-telemetry/opentelemetry-collector@0755152e21d3d228af112d6984ec365e950f4e85
@axw axw force-pushed the confighttp-addrconfig branch from 74023d0 to 02eed61 Compare January 15, 2026 03:04
open-telemetry/opentelemetry-collector#14248
introduces a breaking change to confighttp.ServerConfig, replacing
the Endpoint field with NetAddr (confignet.AddrConfig).
@axw axw force-pushed the confighttp-addrconfig branch from 02eed61 to 5e45a2f Compare January 15, 2026 03:27
@axw
Copy link
Copy Markdown
Contributor Author

axw commented Jan 15, 2026

The lint failure is unrelated to this PR, will be fixed by #45425

@dmitryax
Copy link
Copy Markdown
Member

Please rebase

@axw
Copy link
Copy Markdown
Contributor Author

axw commented Jan 15, 2026

Merged main, all green!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants