-
Notifications
You must be signed in to change notification settings - Fork 87
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
Use JFrog repository with hazelcast.com domain #173
Comments
If we can provide healthy redirection mech from s3 to JFRog, having a official-looking URL would be good. Otherwise, we need to do so much documentation/README update :) We moved our chart repo from github to s3 to get metrics about their traffic so @alparslanavci can community metrics tool get some traffic info from JFrog? |
JFrog Artifactory has a Helm repository option, but IMO it will be better if we can keep s3 and do a re-direction with something like @hasancelik wdyt? |
will the helm client follow the redirect? |
@hasancelik will know it better, but if we migrate to Artifactory, it has already a re-direction at |
Is it a redirection (HTTP 301 or 302) or just a CNAME record in DNS? The former requires a cooperation from clients. The later is transparent (done in DNS resolver) |
It is the second (CNAME record in DNS) for |
ok, that should work with all clients. one just have to be carefully with certificates. |
@alparslanavci I have checked the official doc, adding CNAME record in DNS is enough. As @jerrinot mentioned, old URL will continue to work for helm clients. |
instructions says this
helm repo add hazelcast https://hazelcast-charts.s3.amazonaws.com/
we should explore how to have a repository on a nicer and more official-looking URL. something like
https://helm.hazelcast.com
orhttps://helm.repository.hazelcast.com
.@alparslanavci is working on Bintray migration so perhaps the new repo could be used for Helm chart hosting too?
Obviosly we would have to deprecate the old repo and make it to send redirects to the new one.
The text was updated successfully, but these errors were encountered: