mirror of
https://github.com/cozystack/cozystack.git
synced 2026-11-09 14:02:56 +09:00
fix(platform): add cert-manager dependency to webhook-cert consumers (#2726)
## What this PR does Adds `cert-manager` as a `dependsOn` entry to the five PackageSources whose charts apply `cert-manager.io/v1` `Certificate` CRs and need the webhook ready before their first reconcile. Without this, the helm install renders Certificate manifests, the cert-manager webhook isn't admitting requests yet (cold cluster), the apply fails, and helm-controller retries until the webhook eventually comes up — burning ~2-3 minutes per affected package on cold installs. The five PackageSources: `capi-operator`, `cozystack-engine`, `ingress-nginx`, `kamaji`, and `rabbitmq-operator` (the platform `cozy-system` set with webhook-cert consumers; not the apps layer). In `cozystack-engine`, the dependency is added to both the `default` and `oidc` variants — the `oidc` variant installs the same `cozystack-api` and `lineage-controller-webhook` components that render Certificate resources, so it had the same race. ## Origin First commit lifted unchanged from #2619. Second commit (`oidc` variant) added in response to review feedback. ### Release note ```release-note NONE ``` <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **Chores** * Updated package source dependencies across multiple components to ensure proper installation sequencing. <!-- review_stack_entry_start --> [](https://app.coderabbit.ai/change-stack/cozystack/cozystack/pull/2726?utm_source=github_walkthrough&utm_medium=github&utm_campaign=change_stack) <!-- review_stack_entry_end --> <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
5 files changed
+6
No files matched your search
@@ -13,6 +13,7 @@ spec:
|
||||
- name: default
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.cert-manager
|
||||
components:
|
||||
- name: capi-operator
|
||||
path: system/capi-operator
|
||||
|
||||
@@ -13,6 +13,7 @@ spec:
|
||||
- name: default
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.cert-manager
|
||||
libraries:
|
||||
- name: cozy-lib
|
||||
path: library/cozy-lib
|
||||
@@ -57,6 +58,7 @@ spec:
|
||||
- name: oidc
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.cert-manager
|
||||
- cozystack.keycloak
|
||||
- cozystack.keycloak-operator
|
||||
libraries:
|
||||
|
||||
@@ -14,6 +14,7 @@ spec:
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.prometheus-operator-crds
|
||||
- cozystack.cert-manager
|
||||
components:
|
||||
- name: ingress-nginx
|
||||
path: system/ingress-nginx
|
||||
|
||||
@@ -13,6 +13,7 @@ spec:
|
||||
- name: default
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.cert-manager
|
||||
components:
|
||||
- name: kamaji
|
||||
path: system/kamaji
|
||||
|
||||
@@ -13,6 +13,7 @@ spec:
|
||||
- name: default
|
||||
dependsOn:
|
||||
- cozystack.networking
|
||||
- cozystack.cert-manager
|
||||
components:
|
||||
- name: rabbitmq-operator
|
||||
path: system/rabbitmq-operator
|
||||
|
||||
Reference in new issue
Block a user