-
Notifications
You must be signed in to change notification settings - Fork 72
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
helm chart for hypershift #698
Conversation
aefa276
to
4808b03
Compare
Introduce a install helm command to generate a helm chart with some templatization options similar to the install render --template command. the PR also introduces a way to template the registry overrides option of the operator required by Azure/ARO-HCP#698 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
Introduce a install helm command to generate a helm chart with some templatization options similar to the install render --template command. the PR also introduces a way to template the registry overrides option of the operator required by Azure/ARO-HCP#698 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
Introduce a install helm command to generate a helm chart with some templatization options similar to the install render --template command. the PR also introduces a way to template the registry overrides option of the operator required by Azure/ARO-HCP#698 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
Introduce a install helm command to generate a helm chart with some templatization options similar to the install render --template command. the PR also introduces a way to template the registry overrides option of the operator required by Azure/ARO-HCP#698 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
Introduce a install helm command to generate a helm chart with some templatization options similar to the install render --template command. the PR also introduces a way to template the registry overrides option of the operator required by Azure/ARO-HCP#698 Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
2485c24
to
9711e47
Compare
Please rebase pull request. |
9711e47
to
e2adb4a
Compare
8d9b958
to
d02dcfc
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.
overall LGTM, just a few nits/questions.
hypershiftoperator/deploy/helm/charts/external-dns/templates/deployment-external-dns.yaml
Outdated
Show resolved
Hide resolved
Please rebase pull request. |
When deploying this the servicemonitor is created under the wrong group, should be
|
94e3591
to
3cc2462
Compare
3cc2462
to
95d26c5
Compare
we can't avoid that hypershift installs the |
Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
95d26c5
to
1d7194a
Compare
leverage helm chart for hypershift install/upgrade Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
What this PR does
use native
hypershift install
functionality within helm chartthe external DNS part is still part of this helm chart. will be removed during P3.
Jira: https://issues.redhat.com/browse/ARO-11084
Link to demo recording:
Special notes for your reviewer