-
Notifications
You must be signed in to change notification settings - Fork 67
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
Stop using pilot
in foo.pilot.2i2c.cloud
for new hubs
#989
Comments
Could I get some feedback from the @2i2c-org/tech-team on this one? I don't think that making this change will be too difficult and the longer we keep it as |
Another option is to just move them to .2i2c.cloud. |
@yuvipanda I'm +1 on that as well! I defer to the preference/judgment of others here, just think that we should move away from the word |
No pushback from me about removing |
Notes from sprint team meeting@sgibson91 summarized that the remaining action points may be of a technical nature of updating domains pointing to hubs. @yuvipanda suggests that we could choose to not update these and just switch to ensure new hubs get modern domains. The worry is that redirects could be troublesome to get right. Conclusion:
@sgibson91 mentions that we would need to update callback urls as registered in the GitHub OAuth clients we have reigstered manually. This is relevant IF we change a domain name. Sarah mentions that this is not relevant for Auth0 OAuth clients, as they are automatically updated by the deployer script. |
+1 from me on not renaming existing hubs unless they ask for it, and just start new hubs without |
@choldgraf is there any remaining action points besides this for this issue then? |
pilot.2i2c.cloud
to hub.2i2c.cloud
pilot
in foo.pilot.2i2c.cloud
for new hubs
@consideRatio I think it would be "documenting the new practice of not using |
@choldgraf can I assign you to fix that? I note that you have done a lot of work recently to migrate docs across the 2i2c org |
@consideRatio sure thing 👍 Ah one question I thought up for redirects. What about:
or other hubs that are 2i2c-specific? Is that worth re-naming? I don't feel strongly but wanted to note it |
@yuvipanda suggests that we can move grafana/staging and it will be fine. For demo we would need to come up with a practical solution on how to redirect. Also, what is a priority? This isn't seen as a priority among us currently, but it can change if we get input from users about it - for example about users concerned about this is a pilot project etc. |
In my opinion I don't think this is a huge priority, but the main thing I am worrying about is the feedback I got from some communities that the word "pilot" conveys "unreliable and unstable" |
To signal that this is reasonably stable infrastructure, we're removing 'pilot' from a few domain names. I've setup a wildcard domain *.2i2c.cloud to point to the pilot-hubs cluster's nginx-ingress service IP, so merging this would just switch out these 3 hubs to get rid of the .pilot part of their domain. This does mean our 'primary' cluster becomes a bit more important, so we should make it a little more resilient. See 2i2c-org#1105 and 2i2c-org#1102 Ref 2i2c-org#989
#1129 moves a few of these |
@choldgraf demo.2i2c.cloud works now! Can you tell the rocky mountain lab folks to use the new URL? |
done! think we can close one? |
I'm going to assume the answer is yes! |
Description of problem and opportunity to address it
Context to understand the problem
Currently, the hubs on our shared cluster follow the pattern
<hubname>.pilot.2i2c.cloud
.However, we are past our pilot phase, and calling it a
pilot
may cause organizations to make assumptions about the quality or reliability of our hubs. Moreover,pilot
isn't particularly descriptive, it's more of an implementation details from 2i2c's perspective rather than something that a hub's user cares about.Proposed solution
We should instead use the pattern
<foo>.hub.2i2c.cloud
. This will be:Implementation guide and constraints
*.2i2c.cloud
redirects to*.hub.2i2c.cloud
alpha.2i2c.cloud
as well). But thus far I don't think we're doing this.github.com/2i2c-org/pilot
tohttps://github.com/2i2c-org/user-guide
or something like this.Updates and ongoing work
No response
The text was updated successfully, but these errors were encountered: