-
Notifications
You must be signed in to change notification settings - Fork 194
feat(charts/capsule): added extra manifests in values file #1653
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
bfff3a4
to
e139d83
Compare
I've proceeded to the modifications that failed the previous run of GitHub actions workflows:
EDIT: added workflow runs links |
e139d83
to
092ba7c
Compare
Please run |
092ba7c
to
e6ffa1b
Compare
I've ran |
Signed-off-by: Llyth <[email protected]>
e6ffa1b
to
edcdfed
Compare
Installing chart with values file "charts/capsule/ci/extra-values.yaml"...
>>> helm install capsule-jpnz6c2ois charts/capsule --namespace capsule-system --wait --values charts/capsule/ci/extra-values.yaml --timeout 600s
I1006 10:03:51.006593 26391 warnings.go:110] "Warning: unknown field \"spec.defaultRegistry\""
Error: INSTALLATION FAILED: 1 error occurred:
* Internal error occurred: failed calling webhook "tenants.projectcapsule.dev": failed to call webhook: Post "https://capsule-jpnz6c2ois-webhook-service.capsule-system.svc:443/tenants/mutating?timeout=30s": dial tcp 10.96.211.216:443: connect: connection refused I've proceeded with modifications to the CI test with some random configMap instead and removed the example with a dummy Tenant to not confuse people and let them do as they wish (configMap, secret, other kubernetes object or crd). In my case I do use it to add tenant, although it's not working in install because of webhook not being ready in time on default. Reran the followings successfully:
|
Whoops theres an error on main |
It's a common issue with webhook admission controllers, resource is created but not ready. An example here with cert-manager whenever you try to add a cluster issuer with the install (issue: cert-manager/cert-manager#2908). |
Yeah got that, talking about the warning :D |
This will give the possibility to add extra kubernetes resources/objects such as Tenant, KEDA ScaledObject, configMaps.
Things that are not part of the chart itself but part of/related to Capsule.