Skip to content
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

Explore/Consider Dynamic Configuration re-upload #1007

Closed
PettitWesley opened this issue May 21, 2020 · 3 comments
Closed

Explore/Consider Dynamic Configuration re-upload #1007

PettitWesley opened this issue May 21, 2020 · 3 comments
Labels
enhancement New feature or request help wanted Good issue for contributors to OpenTelemetry Service to pick up Stale
Milestone

Comments

@PettitWesley
Copy link

As mentioned in the Collector SIG today, I think it'd be useful to be able to dynamically re-upload and change configuration with zero downtime. This should be discussed/explored- is stopping and restarting the collector to change the config acceptable? How much data will be lost?

This is a very long term thing; I don't think it is a high priority right now. Opening this issue mostly just so that if we someday start seriously talking about it, I can claim that I first brought up the idea 😁

@bogdandrutu bogdandrutu added this to the Backlog milestone Aug 4, 2020
@bogdandrutu bogdandrutu added the help wanted Good issue for contributors to OpenTelemetry Service to pick up label Aug 4, 2020
@andrewhsu andrewhsu added enhancement New feature or request and removed feature request labels Jan 6, 2021
@shilicqupt
Copy link

shilicqupt commented Apr 28, 2021

type Component interface {
	Start(ctx context.Context, host Host) error

	Shutdown(ctx context.Context) error

       // may be we can add reload func to support
        Reload(ctx context.Context) error
}

@appdashishmehta
Copy link

Definitely a novel idea, I just wonder so far it was not given much thought. We are hitting such requirement as well for our use case of enabling/disabling exporter(s) dynamically without re-building the pipeline or data loss.

@buptubuntu
Copy link

Reload
type Component interface {
	Start(ctx context.Context, host Host) error

	Shutdown(ctx context.Context) error

       // may be we can add reload func to support
        Reload(ctx context.Context,cfg interface{}) error
}

I think cfg param is required for reload func

@github-actions github-actions bot added the Stale label May 29, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jun 29, 2023
Troels51 pushed a commit to Troels51/opentelemetry-collector that referenced this issue Jul 5, 2024
swiatekm pushed a commit to swiatekm/opentelemetry-collector that referenced this issue Oct 9, 2024
* add mountedConfigMaps support

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

* add mountedConfigMaps support

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

* add mountedConfigMaps support

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

* add mountedConfigMaps support

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>

---------

Signed-off-by: Pierre Tessier <pierre@pierretessier.com>
Co-authored-by: Jared Tan <jian.tan@daocloud.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Good issue for contributors to OpenTelemetry Service to pick up Stale
Projects
None yet
Development

No branches or pull requests

6 participants