-
Notifications
You must be signed in to change notification settings - Fork 240
namespace provisioning using UI
#3113
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
Conversation
| @@ -1,4 +1,4 @@ | |||
| # Namespace provisioning example via kpt CLI | |||
| # Namespace provisioning using kpt CLI | |||
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.
Do we want to align the naming between CLI and UI (UI is marked as "example" while CLI is not)
| - [The Rationale behind kpt](guides/rationale.md) | ||
| - [Namespace provisioning example](guides/namespace-provisioning-example.md) | ||
| - [Namespace provisioning CLI](guides/namespace-provisioning-cli.md) | ||
| - [Namespace provisioning UI](guides/namespace-provisioning-ui.md) |
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.
I feel like removing the "example" could be confusing here:1. it seems to be the prerequisite to use Porch. 2. it's placed right after "Rationale" and before "Porch", which seems to be the second thing users should be consider (even before installing porch?)
|
|
||
| In this guide, we will use the package orchestration UI to: | ||
|
|
||
| - Register blueprint and deployment repositories |
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.
Suggest addling the link to tell what "blueprint", "deployment" and "repository" means here. It seems to be the term related to "repository".
|
|
||
| - Register blueprint and deployment repositories | ||
| - Create a `kpt` package from scratch in blueprint repo | ||
| - Create a deployable instance of the `kpt` package |
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.
do we have a doc to talk about "variant constructor"? It seems confusing especially for existing kpt users.
|
Approved since the comments are not blockers. |
|
Thanks Yuwen for the feedback. I have taken notes and will address them in the follow up PRs. |
This adds
namespace provisioning using UIguide.