-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add support for customizing image and extension chart repoURL across templates #493
Conversation
c7480ab
to
afcb387
Compare
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.
Numerous strange (looks like copy-paste/rebase) errors.
No updates in values.schema.json
, please add they're needed for validation
Naming of the values should be reconsidered
templates/cluster/aws-standalone-cp/templates/k0scontrolplane.yaml
Outdated
Show resolved
Hide resolved
templates/cluster/aws-hosted-cp/templates/k0smotroncontrolplane.yaml
Outdated
Show resolved
Hide resolved
templates/cluster/azure-hosted-cp/templates/k0smotroncontrolplane.yaml
Outdated
Show resolved
Hide resolved
templates/cluster/azure-hosted-cp/templates/k0smotroncontrolplane.yaml
Outdated
Show resolved
Hide resolved
templates/cluster/azure-standalone-cp/templates/k0scontrolplane.yaml
Outdated
Show resolved
Hide resolved
templates/cluster/vsphere-standalone-cp/templates/k0scontrolplane.yaml
Outdated
Show resolved
Hide resolved
afcb387
to
76dc00e
Compare
76dc00e
to
80086e5
Compare
4d44f9e
to
b8f8cb3
Compare
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.
We should be careful with possible empty values. Helm is dumb
b8f8cb3
to
01ed0f4
Compare
01ed0f4
to
f37636c
Compare
In order to support airgap, users will need to be able to specify custom extensions.imageRepository values and extensions.chartRepository URLs to be able to download extension images and charts on 'ManagedClusters'. These additions across the helm templates allow them to do so. Signed-off-by: Kyle Squizzato <[email protected]>
Signed-off-by: Kyle Squizzato <[email protected]>
Signed-off-by: Kyle Squizzato <[email protected]>
f37636c
to
355dd83
Compare
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.
LGTM
Closes: #425