-
Notifications
You must be signed in to change notification settings - Fork 200
chore: update gitea chart to 12.x.y
#3859
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for zarf-docs canceled.
|
ef8094c
to
06dc9c6
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
Hey @a1994sc , would you mind rebasing this when you have a moment? I'd like to see where this lands for review currently. |
Oh rebase, not merge, yea I can update that later today |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E2E without cluster workflows are failing with a hardcoded gitea image
I'd be fine with updating this line again unless you see a more dynamic option available.
ok, I think I saw that recent too, so I can update that. |
12.0.0
12.1.2
reviewing the delta of updates between the indicated application version as well as the helm chart versions. No action required by the PR. Appreciative of catching the update on this. |
Not that is a very exhaustive change, but I have been rolling the latest version of the helm chart/docker image internally without too much issue. However I understand my needs aren't going to fit for the whole zarf user base |
Coming back to this (finally) - appreciate the patience. Gitea as an init package component should likely get more attention than we give it to-date. @a1994sc would you mind updating this PR with a merge commit or rebase (however you prefer). I'd love to get this over the line soon. Recognizing gitea has released more updates since. I am good with keeping this update as-is or updating the artifacts to the latest releases. |
@brandtkeller I think it would be "better" to update the artifacts to the latest, so I will update that in the next little bit. |
6e3362d
to
ca0efb7
Compare
- the upstream chart replaces `redis` with `valkey` Signed-off-by: Allen Conlon <software@conlon.dev>
Signed-off-by: Allen Conlon <software@conlon.dev>
Signed-off-by: Allen Conlon <software@conlon.dev>
ca0efb7
to
cea604f
Compare
12.1.2
12.x.y
@brandtkeller Updated |
Description
Updates
gitea
chart to 12.0.0:redis
withvalkey
, so I disabled thevalkey
cluster.gitea_image
to useghcr.io
endpoint to avoid rate-limitingregistry_image_*
to use ghcr.io endpointsRelated Issue
Fixes #N/A
Checklist before merging