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

Add support for storing configuration in configmap #123

Open
anilarora opened this issue Oct 2, 2023 · 1 comment
Open

Add support for storing configuration in configmap #123

anilarora opened this issue Oct 2, 2023 · 1 comment

Comments

@anilarora
Copy link

anilarora commented Oct 2, 2023

Today, the operator stores the configuration of the telegraf sidecar in a secret. Our environments are restricted on who can create secrets, so it is preferable to use a configmap instead. And as our configuration never has a credential stored in it, this would be safe for us to use. Would like a feature enhancement to enable storing configuration in a configmap instead.

Relevant URLs

Would work well with #122

What products and version are you using?

Telegraf Operator 1.3.8

@eli-cook
Copy link

eli-cook commented Jan 4, 2024

This is interesting to me, I am actually working on a fork that allows you to define a configmap location in an annotation telegraf.influxdata.com/env-configmap: <configmap-name>

As a single annotation, and the configmap would have values corresponding to the currently supported set of configuration annotations today. Something like this

configmap := &corev1.ConfigMap{ ObjectMeta: metav1.ObjectMeta{ Name: "my-config", Namespace: "mynamespace", }, Data: map[string]string{ "telegraf.influxdata.com/port": "6060", "telegraf.influxdata.com/path": "/metrics", }, }
Is this what you were describing?

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

No branches or pull requests

2 participants