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

fix: update helm index urls #673

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

ammmze
Copy link
Contributor

@ammmze ammmze commented Dec 4, 2023

Description

Updates urls to use new org name. Note: Docs still need updating, but this should unblock people until the docs are updated.

Issues Resolved

Unblocks users from issue #672

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Updates urls to use new org name. Note: Docs still need updating, but this should unblock people until the docs are updated.

Signed-off-by: Branden Cash <[email protected]>
@prudhvigodithi
Copy link
Member

Thanks @ammmze.

@prudhvigodithi prudhvigodithi merged commit 713e0c6 into opensearch-project:gh-pages Dec 5, 2023
2 of 3 checks passed
@ammmze ammmze deleted the patch-1 branch December 5, 2023 16:12
@ammmze
Copy link
Contributor Author

ammmze commented Dec 5, 2023

👍🏻 Thank you! I just verified it after the merge and as expected you can add the helm repo like this:

helm repo add opensearch-charts https://opensearch-project.github.io/opensearch-k8s-operator

And to verify you can actually render the helm charts I tested with the following (which then spat out all the kubernetes resources as expected...this is the part that was broken before, but now works)

helm template opensearch-operator opensearch-charts/opensearch-operator

@ammmze ammmze mentioned this pull request Dec 5, 2023
@prudhvigodithi
Copy link
Member

Got it, thanks for your help @ammmze.

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