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

QOL: Extending zarf connect usability #575

Closed
wants to merge 3 commits into from
Closed

Conversation

YrrepNoj
Copy link
Member

Zarf Connect Default Stability

Add logic so default zarf connect commands will auto-detect when the standard ports are already being used and use a random open one instead of failing the command.

Using a port other than 45003 since that port was already used on my host machine

Screen Shot 2022-06-29 at 3 46 13 PM

Zarf Connect Usability Improvement

Add a utility function so users can figure out which zarf connect commands are available from packages that have already been installed. Super useful for people who are not familiar with the packages that has already been deployed to a cluster.

List all connect options found within the cluster

Screen Shot 2022-06-29 at 3 43 06 PM

Filter for connect options that contains part of an optional provided argument filter

Screen Shot 2022-06-29 at 3 44 17 PM

@YrrepNoj YrrepNoj requested a review from jeff-mccoy June 29, 2022 19:49
@jeff-mccoy
Copy link
Member

We should probably look at these together 😆 b350f4e

@jeff-mccoy
Copy link
Member

...or at least make sure that the change isn't already in #511

@YrrepNoj
Copy link
Member Author

YrrepNoj commented Jul 5, 2022

Closing as this was also implemented in #586

@YrrepNoj YrrepNoj closed this Jul 5, 2022
@YrrepNoj YrrepNoj deleted the qol/port-auto-choosing branch July 28, 2022 15:06
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.

None yet

2 participants