Backport Release Manifest changes #61

Merged

ipetrov117 merged 1 commits from release-manifest-backport into 3.2 2024-12-19 12:54:57 +01:00

Conversation 0 Commits 1 Files Changed 1 +4 -1

ipetrov117 commented 2024-12-19 12:30:21 +01:00

Owner

Copy Link

Backports #60 to 3.2.

Backports [#60](https://src.opensuse.org/suse-edge/Factory/pulls/60) to 3.2.

ipetrov117 added 1 commit 2024-12-19 12:30:21 +01:00

Release manifest updates in relation to corner case use-cases (#60) ... b3c6ef5811

Changes:

- Rancher version convention was changed from `v2.10.0` to `2.10.0` to better map to the actual version in the upstream helm chart repo which is without the `v` prefix.

- Rancher's `postDelete` hook has been disabled - done to ensure that we will not hit a corner case where:

   1. The Rancher helm chart upgrade fails, because of a core component not yet being ready
   2. The `helm-controller` schedules a `helm uninstall` which deletes the Rancher Helm release and triggers the `postDelete` hook.
   3. The problematic core component is up and running, so `helm-controller` schedules a `helm install` with the new version.
   4. Due to insufficient resources, or network connection (or other unforeseen problems), the `postDelete` hook is still running and it wrongly removes the new Rancher installation resulting in a missing rancher from the cluster after an upgrade.

The `postDelete` hook ensures that no accidental delete of the Rancher application will happen during an upgrade over a machine with fewer resources.

Reviewed-on: #60
Reviewed-by: Denislav Prodanov <dprodanov@noreply.src.opensuse.org>
Reviewed-by: Nicolas Belouin <nbelouin@noreply.src.opensuse.org>
Reviewed-by: Atanas Dinov <atanasdinov@noreply.src.opensuse.org>
Co-authored-by: Ivo Petrov <ivo.petrov@suse.com>
Co-committed-by: Ivo Petrov <ivo.petrov@suse.com>
(cherry picked from commit fde506f9ef)
Signed-off-by: Ivo Petrov <ivo.petrov@suse.com>

ipetrov117 requested review from atanasdinov 2024-12-19 12:30:26 +01:00

ipetrov117 requested review from dprodanov 2024-12-19 12:30:26 +01:00

atanasdinov approved these changes 2024-12-19 12:30:42 +01:00

ipetrov117 merged commit 81c1b981c2 into 3.2 2024-12-19 12:54:57 +01:00

ipetrov117 deleted branch release-manifest-backport 2024-12-19 12:54:57 +01:00

ipetrov117 referenced this issue from a commit 2024-12-19 12:54:58 +01:00

Backport Release Manifest changes (#61)

Sign in to join this conversation.

Reviewers

No reviewers

dprodanov

atanasdinov

Labels

Clear labels

No items

No Label

Milestone

Clear milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

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: suse-edge/Factory#61

No description provided.