-
Notifications
You must be signed in to change notification settings - Fork 11
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
Set up deployment pipeline for this repo's site #4
Comments
Follow up on how to do this for reals, ty @mrbobbytables
cc @npintaux |
Hey @lauralorenz - I'm keen to support sig-multicluster. I have raised a pr with some ideas on initial scaffolding for building the site via netlify 🙏🏻 |
Awesome thank you @jmhbnz !!!! |
Hey @lauralorenz with the initial scaffolding for I'm happy to raise if not, looking at the template it looks like we just need to supply two values, here is what I would propose: Repository
Project domain
|
@mrbobbytables took a very quick look and said a better template for our netlify.toml would be this one in kueue. Basically we don't need the redirects in the netlify.toml (the ones in the Gateway example are out of date as netlify actually switched from netlify.com to netlify.app, and whatever netlify might have needed in the past to detect the proper prod branch is apparently no longer needed) so we can take those out. For the actual netlify issue now that it's raised we need a github admin to help us, I'm poking around on slack some too to see if we can get in someone's TODO list any time soon :) Thank you @jmhbnz and @npintaux !! |
Thanks for the feedback @lauralorenz - I've raised #7 to tidy up those unnecessary redirects and also streamline the local environment setup for the site. Fingers crossed we can get the netlify domain setup soon! 🙏🏻 |
Ok netlify is setup (see kubernetes/org#4086), if someone can run point for the DNS request that would be great! |
Great news! Ok I have raised our dns request here to get the CNAME for the new netlify address: kubernetes/k8s.io#4991 Edit: Have raised the dns pull request to try and speed things up: kubernetes/k8s.io#5068 Edit2: Dns request completed, now just need a quick update on netlify side and will be all done :) |
Hey @lauralorenz - Site pipeline is done along with dns, all running nicely here: https://multicluster.sigs.k8s.io I think we can close this now? 🙏🏻 |
Yes all looks good and we successfully had a merge and redeploy all work too! YAY! Thank you everyone!!! /close |
@lauralorenz: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Per the README, and in the proposal site, we would deploy from local using the directions on the mkdocs website. The Gateway API website appears to use netlify instead (kubernetes-sigs/gateway-api#559) and this also appears to be how they got proper DNS resolution from
*.k8s.io
so I would like us to do that. Determine if we want to do the same; otherwise set up GitHub Actions to deploy automatically on a merge into themain
branch.The text was updated successfully, but these errors were encountered: