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

Current chart version is 2 releases behind Immich #68

Closed
michalramus opened this issue Mar 2, 2024 · 5 comments
Closed

Current chart version is 2 releases behind Immich #68

michalramus opened this issue Mar 2, 2024 · 5 comments

Comments

@michalramus
Copy link

Hi, could you update version of chart to match latest Immich release?

@bo0tzz
Copy link
Member

bo0tzz commented Mar 2, 2024

Updating the chart every time there is a new Immich version is unnecessary. Updates are usually only done when needed, for example when there is a breaking change. You can set the latest version in your own values.yaml

image:
  tag: v1.97.0

@bo0tzz bo0tzz closed this as not planned Won't fix, can't repro, duplicate, stale Mar 2, 2024
@michalramus
Copy link
Author

michalramus commented Mar 2, 2024

Could you add information about this, because it is a little confusing for new Immich users(like me).

@bo0tzz
Copy link
Member

bo0tzz commented Mar 2, 2024

#69

@michalramus
Copy link
Author

Thank you:)

@maybeanerd
Copy link

Updating the chart every time there is a new Immich version is unnecessary.

How so? When using a chart, I generally expect it to keep versions up to date, especially abstracting away things like e.g. compatability with other systems (DB, redis, etc)

Currently, as this chart doesn't provide this for me, I maintain effectively a fork for my own use.

I think this repo could be set up to update versions mostly automatically, keeping manual effort low.
In my repo (https://github.com/maybeanerd/home-cluster/tree/main/charts%2Fimmich) I use renovate.
It can be configured to:

  • autobump patch versions
  • prepare PRs, but don't merge them for minor versions (as they might be breaking until immich hits stable. This is a bit of manual work, as one would need to review the patchnotes before merging)
  • same for major version releases
  • autobump the chart version according to the proposed change: minor immich version change -> minor chart version bump

Would you consider something like this?
If so, I could prepare a PR with the config for you to check out in more detail.

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

No branches or pull requests

3 participants