Skip to content
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

imply edge quickstart --already-initialized if --home #1659

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

qrkourier
Copy link
Member

…me ZITI_HOME is set and ZITI_HOME/db exists on subsequent runs

@qrkourier qrkourier requested a review from dovholuknf December 20, 2023 18:16
@qrkourier qrkourier requested review from a team as code owners December 20, 2023 18:16
@qrkourier qrkourier requested a review from dovholuknf December 20, 2023 19:29
@qrkourier qrkourier requested a review from dovholuknf December 21, 2023 17:42
@@ -65,7 +65,7 @@ func NewQuickStartCmd(out io.Writer, errOut io.Writer, context context.Context)
cmd := &cobra.Command{
Use: "quickstart",
Short: "runs a Controller and Router in quickstart mode",
Long: `runs a Controller and Router in quickstart mode. By default, this will create a totally ephemeral network, only valid while running.`,
Long: "runs a Controller and Router in quickstart mode with a temporary Ziti home directory; suitable for testing and development",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is the changelog updated for this already? if not, probably worthwhile to update the changelog too

@qrkourier qrkourier requested a review from dovholuknf January 8, 2024 16:02
…me ZITI_HOME is set and ZITI_HOME/db exists on subsequent runs
@qrkourier qrkourier force-pushed the quickstart-imply-init-if-home branch from ebf5609 to 3b308df Compare January 9, 2024 23:11
@qrkourier qrkourier merged commit 1598205 into release-next Jan 10, 2024
18 checks passed
@qrkourier qrkourier deleted the quickstart-imply-init-if-home branch January 10, 2024 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants