ci(magnum): deprecate heat and use magnum-cluster-api driver#3765
ci(magnum): deprecate heat and use magnum-cluster-api driver#3765winiciusallan wants to merge 1 commit into
Conversation
| if [ -z "$_MAGNUM_IMAGE_ID" ]; then | ||
| _MAGNUM_IMAGE_ID=$(openstack image list --format value -c Name -c ID | grep -i atomic | cut -d ' ' -f 1) | ||
| fi | ||
| _MAGNUM_IMAGE_ID=$(openstack image list --format value -c Name -c ID | grep ubuntu | cut -d ' ' -f 1) |
There was a problem hiding this comment.
Wondering whether we want to be able to choose the Kubernetes version to test against. We could add it to this filter instead of just grep'ing for "ubuntu".
This works because devstack does not create any ubuntu image, so there is no conflict.
But the K8s version tells more about the underlying driver support, and CAPI compatibility
There was a problem hiding this comment.
That's a fair question. I think there's no need to do it here and we can leave that to future work.
|
coverage: 64.008%. remained the same — winiciusallan:change-magnum-ci into gophercloud:main |
|
The error on master branch has an upstream bug report on Launchpad. It looks like a consequence of migrating to use the openstacksdk. |
As a follow-up, I've proposed a patch for this issue. Let's see if we can get it merged. |
|
The error on Epoxy is because this release has been using the I filed an issue in vexxhost/magnum-cluster-api vexxhost/magnum-cluster-api#1045 |
As Magnum moved towards removing the Heat driver to use out-of-tree drivers instead[1], we cannot rely on it anymore in our CI. This pull request changes the
functional-containerinfrajob to use thevexxhost/magnum-cluster-api[2] plugin that's also being used upstream[3].The Vexxhost's driver spins up a Kubernetes cluster using Kind, installs all the dependencies, such as: kubectl, clusterctl, and CAPI, with OpenStack as the infrastructure provider (so we also have ORC here 🤩 ).
Let's see how it goes.
[1] https://review.opendev.org/c/openstack/magnum/+/985348
[2] https://github.com/vexxhost/magnum-cluster-api
[3] https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/987846