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

[influxdb2] Bump to 2.7.1 #536

Merged
merged 3 commits into from
Dec 4, 2023
Merged

[influxdb2] Bump to 2.7.1 #536

merged 3 commits into from
Dec 4, 2023

Conversation

centromere
Copy link
Contributor

No description provided.

@vd-rd vd-rd mentioned this pull request Apr 5, 2023
Open
@max-os-lg
Copy link

What's going on here? Can we merge?

@centromere centromere changed the title [influxdb2] Bump to 2.6.1 [influxdb2] Bump to 2.7.1 May 2, 2023
@orgads
Copy link

orgads commented May 14, 2023

@powersj ping?

Philluxx
Philluxx approved these changes Jun 6, 2023
View reviewed changes
Copy link

@Philluxx Philluxx left a comment

Choose a reason for hiding this comment

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

Looks good

@koooge koooge mentioned this pull request Jun 9, 2023
Closed
@Philluxx
Copy link

Can someone please do another review to keep this going? Thanks a lot.

@jelmokri
Copy link

up ?

@centromere
Copy link
Contributor Author

Hi @Philluxx. Is there anything required on my end to get this PR moved through the process?

mladBlum
mladBlum approved these changes Nov 8, 2023
View reviewed changes
Copy link

@mladBlum mladBlum left a comment

Choose a reason for hiding this comment

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

Changes are fine!

@orgads
Copy link

orgads commented Nov 12, 2023

Well? :)

@mladBlum
Copy link

mladBlum commented Nov 13, 2023 •
edited
Loading

Well? :)

I did my best, sadly there is a reviewer with elevated rights needed.
To the reviewer: I am running the current chart since a few months with the influxdb image version overridden to use 2.7.x and it works fine.

@centromere
Copy link
Contributor Author

v2.7.4 is released. If I update this PR, I believe it will require re-approval.

@MarkhamLee
Copy link

Just to add my $0.02

  • I updated the image version in the values.yaml file as well and haven't encountered any issues yet.
  • Until this is fixed anyone who installs the helm chart and then tries to connect to the latest Grafana will encounter an issue where the connection "works", but Grafana is unable to detect any measurements, even if you have data in InfluxDB. I.e. this is rapidly becoming a critical issue..

jeffreyssmith2nd
jeffreyssmith2nd reviewed Dec 4, 2023
View reviewed changes
charts/influxdb2/Chart.yaml Outdated Show resolved Hide resolved
charts/influxdb2/values.yaml Outdated Show resolved Hide resolved
@jeffreyssmith2nd jeffreyssmith2nd merged commit 563857e into influxdata:master Dec 4, 2023
2 checks passed
@centromere centromere deleted the influxdb2-2.6.1 branch December 4, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Philluxx Philluxx Philluxx approved these changes

@jeffreyssmith2nd jeffreyssmith2nd jeffreyssmith2nd approved these changes

@mladBlum mladBlum mladBlum approved these changes

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

Successfully merging this pull request may close these issues.
8 participants
@centromere @max-os-lg @orgads @Philluxx @jelmokri @mladBlum @MarkhamLee @jeffreyssmith2nd