search

k8s-at-home / charts

⚠️ Deprecated : Helm charts for applications you run at home
https://docs.k8s-at-home.com
Apache License 2.0
1.45k stars 623 forks source link

[unifi-poller] Update Helm chart influxdb to v5 #1667

Closed renovate[bot] closed 2 years ago

renovate[bot] commented 2 years ago

Mend Renovate

This PR contains the following updates:

Package Update Change
influxdb major 1.1.9 -> 5.3.6

Configuration

📅 Schedule: Branch creation - "every 3 months on the first day of the month" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

This PR has been generated by Mend Renovate. View repository job log here.

xorcat commented 2 years ago

@bjw-s - I think this probably needs to roll back to keep unifi poller happy:

image

This chart update pushes influxDB > v2; which Unifi Poller does not recommend, and probably requires some testing/confirmation/upstream changes to unifi-poller to ensure its actually what you want.

At the very least, this breaks the values.yaml provided for influx.

onedr0p commented 2 years ago

This is a chart that falls under this issue

https://github.com/k8s-at-home/charts/issues/1572

I would rather people use the old version if they need influxdb, because in the future we will remove it as a dep of the chart.

xorcat commented 2 years ago

Okay, fair, that's probably a good call; thanks!