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

bug: solo context connect prompts for cluster after already supplied #951

Closed
Tracked by #588
jeromy-cannon opened this issue Dec 9, 2024 · 1 comment · Fixed by #993
Closed
Tracked by #588

bug: solo context connect prompts for cluster after already supplied #951

jeromy-cannon opened this issue Dec 9, 2024 · 1 comment · Fixed by #993
Assignees
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs Internal Requirement Multiple Cluster Support Issues which are only required to enable Solo support for multi-cluster deployments. P1 High priority issue. Required to be completed in the assigned milestone. released

Comments

@jeromy-cannon
Copy link
Contributor

jeromy-cannon commented Dec 9, 2024

  • don't prompt user for cluster names if they are supplied with --cluster-name flag
❯ kind delete cluster -n solo

Deleting cluster "solo" ...
Deleted nodes: ["solo-control-plane"]
❯ kind create cluster -n solo

Creating cluster "solo" ...
 ✓ Ensuring node image (kindest/node:v1.27.3) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹️
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-solo"
You can now use your cluster with:

kubectl cluster-info --context kind-solo

Have a nice day! 👋
❯ rm -Rf ~/.solo
solo init
solo context connect -n jeromy-solo --context kind-solo --cluster-name kind-solo


******************************* Solo *********************************************
Version			: 0.99.0
Kubernetes Context	: kind-solo
Kubernetes Cluster	: kind-solo
**********************************************************************************
(node:19191) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
✔ Setup home directory and cache
✔ Check dependencies [1s]
  ✔ Check dependency: helm [OS: darwin, Release: 23.6.0, Arch: arm64] [1s]
✔ Setup chart manager [1s]
✔ Copy templates in '/Users/user/.solo/cache'


***************************************************************************************
Note: solo stores various artifacts (config, logs, keys etc.) in its home directory: /Users/user/.solo
If a full reset is needed, delete the directory or relevant sub-directories before running 'solo init'.
***************************************************************************************

******************************* Solo *********************************************
Version			: 0.99.0
Kubernetes Context	: kind-solo
Kubernetes Cluster	: kind-solo
Kubernetes Namespace	: jeromy-solo
**********************************************************************************
(node:19196) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
✔ Initialize
⠇ Prompt local configuration
◼ Update local configuration

? Enter the Solo deployment cluster names (comma separated):  ›
@jeromy-cannon jeromy-cannon added Bug A error that causes the feature to behave differently than what was expected based on design docs P0 An issue impacting production environments or impacting multiple releases or multiple individuals. P1 High priority issue. Required to be completed in the assigned milestone. Internal Requirement Multiple Cluster Support Issues which are only required to enable Solo support for multi-cluster deployments. and removed P0 An issue impacting production environments or impacting multiple releases or multiple individuals. labels Dec 9, 2024
@swirlds-automation
Copy link
Contributor

🎉 This issue has been resolved in version 0.32.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs Internal Requirement Multiple Cluster Support Issues which are only required to enable Solo support for multi-cluster deployments. P1 High priority issue. Required to be completed in the assigned milestone. released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants