-
Notifications
You must be signed in to change notification settings - Fork 71
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
libvirt: Improve the setup instructions to run libvirt tests #1749
Conversation
thanks @ldoktor these change are good to me, as we change the repo in a new files structure, would you like rebase this PR with main branch. |
The Create the Kubernetes cluster is duplicate as the same steps are already executed under `config_libvirt`. Also improve some wording and examples here and there to ensure the workflow works well out of the box (tested on Fedora 38). Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
link those files together to simplify the life of newcomers. Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
the 120s timeout is quite short and frequently fails on my network. Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Let's run all example commands from the main "cloud-api-adaptor" folder (unless it's required, then use pushd/popd explicitely). Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Sure, changes:
|
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.
Thanks LGTM
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.
Hi @ldoktor , thanks for those updates!
And sorry for taking too long to ack...
Few things that should simplify the setup for the newcomers