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.

Sign up for GitHub

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

Jump to bottom

Bump telegraf-operator to 1.1.6 (official 1.1.5 + helm2 support) #1008

Merged
merged 1 commit into from
Oct 16, 2020
Merged

Bump telegraf-operator to 1.1.6 (official 1.1.5 + helm2 support) #1008

merged 1 commit into from
Oct 16, 2020

Conversation

sumo-drosiek
Copy link
Contributor

@sumo-drosiek sumo-drosiek commented Oct 15, 2020 •
edited
Loading

Description

Bump telegraf-operator to 1.1.6 (official 1.1.5 + helm2 support)

Testing performed
  • ci/build.sh
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

@sumo-drosiek sumo-drosiek reopened this Oct 15, 2020
@pmalek-sumo
Copy link
Contributor

@sumo-drosiek Do you mind adding a commit(s) link(s) which added helm 2 support in our fork (https://sumologic.github.io/influxdata-helm-charts)?

@sumo-drosiek
Copy link
Contributor Author

I can, but this is not the purpose of this update:
SumoLogic/influxdata-helm-charts@c05747e#diff-82963fc54504fded6413192013dfb203b11593c4cf92725dfabbd2319d951afa

@pmalek-sumo
Copy link
Contributor

I can, but this is not the purpose of this update:
SumoLogic/influxdata-helm-charts@c05747e#diff-82963fc54504fded6413192013dfb203b11593c4cf92725dfabbd2319d951afa

Bump telegraf-operator to 1.1.6 (official 1.1.5 + helm2 support)

🤔

@sumo-drosiek
Copy link
Contributor Author

We can do it long way:

I want to bump the version in requirements, because our fork of telegraf-operator helm chart (v1.1.6) will consists of official telegraf-operator helm chart (v1.1.5) with helm2 support. Telegraf-operator helm chart v1.1.5 is going to have telegraf-operator in version v.1.1.1 which fixes one of the issue we hit during testing our solution: influxdata/telegraf-operator#40

@sumo-drosiek sumo-drosiek changed the title Bump telegraf-operator to 1.1.5 (official 1.1.4 + helm2 support) Bump telegraf-operator to 1.1.6 (official 1.1.5 + helm2 support) Oct 15, 2020
perk-sumo
perk-sumo approved these changes Oct 15, 2020
View reviewed changes
Copy link
Contributor

@perk-sumo perk-sumo left a comment

Choose a reason for hiding this comment

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

👍

@perk-sumo
Copy link
Contributor

Please backport it to the release-v1.3 as well.

@sumo-drosiek sumo-drosiek force-pushed the drosiek-bump-telegraf-operator branch from 808d756 to b1ac781 Compare October 16, 2020 07:58
@sumo-drosiek sumo-drosiek merged commit cd23f3d into master Oct 16, 2020
@sumo-drosiek sumo-drosiek deleted the drosiek-bump-telegraf-operator branch October 16, 2020 08:12
@sumo-drosiek sumo-drosiek mentioned this pull request Oct 16, 2020
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pmalek-sumo pmalek-sumo pmalek-sumo approved these changes

@perk-sumo perk-sumo perk-sumo approved these changes

@frankreno frankreno Awaiting requested review from frankreno

@andrzej-stencel andrzej-stencel Awaiting requested review from andrzej-stencel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sumo-drosiek @pmalek-sumo @perk-sumo