-
Notifications
You must be signed in to change notification settings - Fork 120
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
Add kamaji deployment on eks #680
Add kamaji deployment on eks #680
Conversation
✅ Deploy Preview for kamaji-documentation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There were some typos corrected with Grammarly, besides some minor improvements and clarification on Kamaji: let me know your thoughts!
thank you for your review, everything is fixed now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you can browse from the preview link from Netlify, the kamaji-aws-deployment.md
file must be added here:
Lines 63 to 73 in abfc65a
- 'Guides': | |
- guides/index.md | |
- guides/kamaji-azure-deployment.md | |
- guides/alternative-datastore.md | |
- guides/kamaji-gitops-flux.md | |
- guides/upgrade.md | |
- guides/datastore-migration.md | |
- guides/backup-and-restore.md | |
- guides/certs-lifecycle.md | |
- guides/cluster-api.md | |
- guides/console.md |
We could place after the Azure one.
981e806
to
1329a60
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the hard work in getting this done, and welcome as a Kamaji contributor: it's an important milestone to keep the project open and well-documented for new comers! 🚀
closes #69