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

[telegraf] Add init container in telegraf chart #479

Closed
wants to merge 1 commit into from
Closed

[telegraf] Add init container in telegraf chart #479

wants to merge 1 commit into from

Conversation

sebastienrospars
Copy link

@sebastienrospars sebastienrospars commented Jun 8, 2022

Hello,

This pull request is made in order to add optional init container in telegraf chart.

My goal is to use it to create an influx database with specified retention policy instead of create a dedicated job to do this.

Test performed
Deployed in kubernetes cluster with default values, and with simple init container (echo "OK")

  • CHANGELOG.md updated
  • Rebased/mergable
  • Tests pass
  • Sign CLA (if not already signed)

Thanks by advance

@sebastienrospars sebastienrospars changed the title Add init container in telegraf chart [telegraf] Add init container in telegraf chart Jun 8, 2022
@powersj
Copy link
Contributor

powersj commented Nov 14, 2022

@wojciechka is there a way to kick of tests? Any objection to this?

@zekena2
Copy link

zekena2 commented Nov 17, 2022

I made a PR that implements the same functionalitiy (and more) please check it as well it can add multiple init containers.

@sebastienrospars
Copy link
Author

I closing my pull request, the work provided by @zekena2 is more exhaustive in this PR #523.

Thanks by advance :)

@sebastienrospars sebastienrospars deleted the feat/add_init_container_in_telegraf_chart branch December 8, 2022 13:01
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.

None yet

3 participants