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

Support k8s 1.25+ #132

Merged
merged 1 commit into from
Apr 21, 2023
Merged

Support k8s 1.25+ #132

merged 1 commit into from
Apr 21, 2023

Conversation

khvn26
Copy link
Member

@khvn26 khvn26 commented Apr 18, 2023

  • Lock influxdb2 to latest version to account for upstream k8s 1.25+ compatibility

Thanks for submitting a PR! Please check the boxes below:

  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have bumped the version number in /charts/flagsmith/Chart.yaml in the section version or I'm merging to a
    release branch

Changes

The influxdb2 dependency is pinned to a newer version. This allows the chart to get successfully deployed on Kubernetes versions 1.25+.

How did you test this code?

  1. Installed the chart with Helm 3.11.3 against Kubernetes 1.26.
  2. Registered a new Flagsmith organization and verified that the metrics were sent correctly to the provisioned InfluxDB instance.

* Lock influxdb2 to latest version to account for upstream k8s 1.25+ compatibility
@khvn26
Copy link
Member Author

khvn26 commented Apr 20, 2023

I suggest we merge this before #133

@khvn26 khvn26 merged commit 00b0a63 into main Apr 21, 2023
@khvn26 khvn26 deleted the fix/k8s-1.25 branch April 21, 2023 12:31
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.

2 participants