refactor: rename values key with compat #1357

Open
viceice wants to merge 2 commits from refactor/rename-settings into main
Owner

There is a compat level to still support the gitea key, so no breaking change.
closes #80
depends on #1356 (should be merged after)

There is a compat level to still support the `gitea` key, so no breaking change. closes #80 depends on #1356 (should be merged after)
viceice added this to the v14 milestone 2025-08-21 13:07:05 +00:00
refactor: rename values key with compat
All checks were successful
build / lint-node (pull_request) Successful in 21s
build / lint-helm (pull_request) Successful in 28s
build / e2e (v1.28.15+k3s1) (pull_request) Successful in 2m56s
build / e2e (v1.32.7+k3s1) (pull_request) Successful in 3m9s
build / e2e (v1.33.3+k3s1) (pull_request) Successful in 3m16s
build / test-success (pull_request) Successful in 1s
build / publish (pull_request) Has been skipped
7f1a0e8010
chore: missing renames
All checks were successful
build / lint-node (pull_request) Successful in 20s
build / lint-helm (pull_request) Successful in 23s
build / e2e (v1.28.15+k3s1) (pull_request) Successful in 3m16s
build / e2e (v1.32.7+k3s1) (pull_request) Successful in 3m39s
build / e2e (v1.33.3+k3s1) (pull_request) Successful in 3m38s
build / test-success (pull_request) Successful in 1s
build / publish (pull_request) Has been skipped
0f3cc6bd57
Member

Did you test this on existing installations? In my first attempt this wasn't "so easy" unfortunately, but maybe I also overlooked something.

The main point is that you need to ensure important resources are not getting recreated (and by that in the worst case deleted if the reclaim policy is Deleted) due to the renaming. Others are immutable after creation, which is another distinct problem.

In my first tests I had issues with

  • renaming ingress
  • ensuring PVC isn't recreated but rather taken as existingClaim

One can of course account for all of these manually but this needs detailed migration instructions and initially I was trying to find a way to make this a seamless change instead.

Did you test this on existing installations? In my first attempt this wasn't "so easy" unfortunately, but maybe I also overlooked something. The main point is that you need to ensure important resources are not getting recreated (and by that in the worst case deleted if the reclaim policy is `Deleted`) due to the renaming. Others are immutable after creation, which is another distinct problem. In my first tests I had issues with - renaming ingress - ensuring PVC isn't recreated but rather taken as `existingClaim` One can of course account for all of these manually but this needs detailed migration instructions and initially I was trying to find a way to make this a seamless change instead.
viceice modified the milestone from v14 to v16 2025-08-22 07:19:41 +00:00
Author
Owner

ok, we'll shift this to v15. i like to get v14 out today

ok, we'll shift this to v15. i like to get v14 out today
viceice modified the milestone from v16 to v18 2026-01-15 18:38:16 +00:00
All checks were successful
build / lint-node (pull_request) Successful in 20s
Required
Details
build / lint-helm (pull_request) Successful in 23s
Required
Details
build / e2e (v1.28.15+k3s1) (pull_request) Successful in 3m16s
Required
Details
build / e2e (v1.32.7+k3s1) (pull_request) Successful in 3m39s
Required
Details
build / e2e (v1.33.3+k3s1) (pull_request) Successful in 3m38s
Required
Details
build / test-success (pull_request) Successful in 1s
Required
Details
build / publish (pull_request) Has been skipped
This pull request has changes conflicting with the target branch.
  • README.md
  • templates/_helpers.tpl
  • templates/gitea/deployment.yaml
  • templates/gitea/gpg-secret.yaml
  • templates/gitea/init.yaml
  • values.yaml
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin refactor/rename-settings:refactor/rename-settings
git switch refactor/rename-settings
Sign in to join this conversation.
No reviewers
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
forgejo-helm/forgejo-helm!1357
No description provided.