Skip to content

Dynamic control#2418

Merged
trask merged 9 commits intoopen-telemetry:mainfrom
jackshirazi:dynamic-control
Nov 21, 2025
Merged

Dynamic control#2418
trask merged 9 commits intoopen-telemetry:mainfrom
jackshirazi:dynamic-control

Conversation

@jackshirazi
Copy link
Copy Markdown
Contributor

Description:

Addition of dynamic-control module

Existing Issue(s):

#2416

Testing:

N/A

Documentation:

in the README

Outstanding items:

Future additions are listed in the issue

@jackshirazi jackshirazi requested a review from a team as a code owner November 5, 2025 15:38
@jackshirazi jackshirazi self-assigned this Nov 5, 2025
Copy link
Copy Markdown
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

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

Thanks for keeping this small and piecing it out.

include(":inferred-spans")
include(":opamp-client")
include(":gcp-auth-extension")
include(":dynamic-control")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe remote-control, to be a little more specific?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The aim of the project is to allow changes to features, and where those changes are triggered from is secondary. For example one of the proposed examples uses a local file that is re-read every N seconds to update values. So I'd say dynamic control is more accurate

Co-authored-by: jason plumb <75337021+breedx-splk@users.noreply.github.com>
@trask trask merged commit 9b46465 into open-telemetry:main Nov 21, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants