Skip to content
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

Merged
merged 2 commits into from
Nov 13, 2024
Merged

helm chart for hypershift #698

merged 2 commits into from
Nov 13, 2024

Conversation

geoberle
Copy link
Collaborator

@geoberle geoberle commented Oct 9, 2024

What this PR does

use native hypershift install functionality within helm chart
the 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

geoberle added a commit to geoberle/hypershift that referenced this pull request Oct 13, 2024
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>
geoberle added a commit to geoberle/hypershift that referenced this pull request Oct 17, 2024
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>
geoberle added a commit to geoberle/hypershift that referenced this pull request Oct 18, 2024
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>
geoberle added a commit to geoberle/hypershift that referenced this pull request Oct 24, 2024
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>
geoberle added a commit to geoberle/hypershift that referenced this pull request Oct 25, 2024
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>
Copy link

github-actions bot commented Nov 1, 2024

Please rebase pull request.

Copy link
Collaborator

@tony-schndr tony-schndr left a 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.

Copy link

github-actions bot commented Nov 7, 2024

Please rebase pull request.

@tony-schndr
Copy link
Collaborator

When deploying this the servicemonitor is created under the wrong group, should be azmonitoring.coreos.com

kubectl get servicemonitor.monitoring.coreos.com -A
NAMESPACE                                                       NAME                                  AGE
hypershift                                                      operator                              18h

@geoberle
Copy link
Collaborator Author

geoberle commented Nov 8, 2024

When deploying this the servicemonitor is created under the wrong group, should be azmonitoring.coreos.com

kubectl get servicemonitor.monitoring.coreos.com -A
NAMESPACE                                                       NAME                                  AGE
hypershift                                                      operator                              18h

we can't avoid that hypershift installs the monitoring.coreos.com CRs for the time being, but i've added an azmonitoring.coreos.com flavor of the operator ServiceMonitor so we can scrape with managed prometheus.

tony-schndr
tony-schndr previously approved these changes Nov 12, 2024
@geoberle geoberle merged commit 3bf5906 into main Nov 13, 2024
20 checks passed
@geoberle geoberle deleted the hypershift-helm branch November 13, 2024 14:42
@geoberle geoberle restored the hypershift-helm branch November 14, 2024 13:14
@geoberle geoberle deleted the hypershift-helm branch November 28, 2024 14:29
katherinelc321 pushed a commit to katherinelc321/ARO-HCP that referenced this pull request Jan 30, 2025
leverage helm chart for hypershift install/upgrade

Signed-off-by: Gerd Oberlechner <goberlec@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants