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

test/e2e/livirt: allow prop file overwrite cluster name #1174

Conversation

bookinabox
Copy link
Contributor

Creates a cluster_name property that can be added to the libvirt properties file and corresponding LibvirtProvisioner struct.

Before createCluster executes kcli_cluster.sh, it adds the cluster name, network, and storage pool to the environment.

Fixes: #1163

@wainersm
Copy link
Member

@bookinabox thanks!

There is at least one fix though. When I ran on my machine it created correctly the "peer-pods-e2e" cluster but then later tried to connect to "peer-pods" (see the logs below). You will need to change this line: https://github.com/confidential-containers/cloud-api-adaptor/blob/main/libvirt/kcli_cluster.sh#L55

peer-pods-e2e-ctlplane-0 deployed on local
Deploying workers
Deploying Vms...
peer-pods-e2e-worker-0 deployed on local
Kubernetes cluster peer-pods-e2e deployed!!!
INFO export KUBECONFIG=$HOME/.kcli/clusters/peer-pods-e2e/auth/kubeconfig
INFO export PATH=$PWD:$PATH
Adding app ingress
Wait at least one worker be Ready
W0714 15:43:53.282825  281510 loader.go:221] Config not found: /home/wmoschet/.kcli/clusters/peer-pods/auth/kubeconfig
The connection to the server localhost:8080 was refused - did you specify the right host or port?

@bookinabox bookinabox force-pushed the libvirt-e2e-test-cluster-name-overwrite branch from bb614e0 to 9156272 Compare July 15, 2023 00:39
@wainersm
Copy link
Member

@bookinabox could you please resolve the conflict and resubmit?

Creates a cluster_name property that can be added to the libvirt
properties file and corresponding LibvirtProvisioner struct.

Before createCluster executes kcli_cluster.sh, it adds the cluster name,
network, and storage pool to the environment.

Fixes: confidential-containers#1163

Signed-off-by: Derek Lee <derlee@redhat.com>
@bookinabox bookinabox force-pushed the libvirt-e2e-test-cluster-name-overwrite branch from 9156272 to 959be76 Compare July 17, 2023 15:18
Copy link
Member

@wainersm wainersm left a comment

Choose a reason for hiding this comment

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

Hi @bookinabox , I just tested this PR and it looks good. thanks!

Copy link
Contributor

@snir911 snir911 left a comment

Choose a reason for hiding this comment

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

LGTM., thanks

@wainersm wainersm merged commit 2d35e53 into confidential-containers:main Jul 26, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test/e2e/libvirt: allow to overwrite the cluster name
3 participants