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

Don't create a new cluster in Assisted Installer if it already exists #289

Open
2 tasks done
gojeaqui opened this issue Nov 17, 2023 · 5 comments
Open
2 tasks done
Labels
enhancement New feature or request

Comments

@gojeaqui
Copy link
Contributor

gojeaqui commented Nov 17, 2023

Feature description

It is a common problem that not all prerequisites are fulfilled in clients and so the deploy_cluster.yml playbook fails at some step.

It would be nice if the playbook checked the Assisted Installer API to look for clusters with the same name as the one being installed from the inventory and to check the status.

Perhaps there could be a prompt asking the user if they want to erase the cluster and start again or continue the installation where it failed (I'm getting timeout failures because the image download speed is slow).

Since I didn't know much about Assisted Installer or Crucible at one point I had 20 clusters created in Assisted Installer because I had to execute the deploy_cluster.yml playbook many times.

Thanks

Required statements

  • I acknowledge that Red Hat does not provide commercial support for the content of this repository.
  • I acknowledge that any assistance is offered purely on a best-effort basis, as resource permits.
@gojeaqui gojeaqui added the enhancement New feature or request label Nov 17, 2023
@nocturnalastro
Copy link
Collaborator

The issue here is the effort it requires to check if all the variables are the same so that we can re-use this id is quite large. And some variables are required at creation.

We could perhaps make a helper playbook to go an tidy up any orphaned entries.

@gojeaqui
Copy link
Contributor Author

Yes, but what about warning the user?

Checking if there exists a cluster with the same name and prompting the user to abort everything or to delete the cluster and start over?

I'm asking because when I started with all this I wasn't aware of how everything worked and ended up with 20 clusters in Assisted Installer until I discovered the UI.

@nocturnalastro
Copy link
Collaborator

nocturnalastro commented Nov 24, 2023

The process shouldn't be interrupted if possible.

I'll have a think on how we can provide the info to the user without breaking the flow.

Thanks for the feedback :)

@gojeaqui
Copy link
Contributor Author

No, thank you.

The playbooks are very useful.

But there should be a way to tell the user that the Assisted Installer UI is accesible, perhaps in the documentation is enough.

And also to warn about the creation of multiple clusters.

Thanks!

@gojeaqui
Copy link
Contributor Author

The process shouldn't be interrupted if possible.

@nocturnalastro sorry to bother you with this, but I think this is an important thing enough to interrrupt the process so the users checks what is going on.

Just my 2 cents, best regards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants