Closed mdbooth closed 7 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mdbooth
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Name | Link |
---|---|
Latest commit | 4d3e79c8cc23dc4d8bab0f77bec8b44e45e5319e |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/65e5b433a1a5a60008618fbc |
Deploy Preview | https://deploy-preview-1919--kubernetes-sigs-cluster-api-openstack.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
/cc @JoelSpeed whose CPMS code I cribbed from heavily.
/lgtm /hold
add a hold in case anyone want to see this again
Interesting that it didn't re-run the tests 🤔
/test all
/test pull-cluster-api-provider-openstack-test
This series adds a framework for testing API validations via calls to a running apiserver using envtest. It initially adds only a couple of example tests.
Firstly we update ginkgo to have the same version in tools and the main module, because otherwise it emits a warning.
The we fix an actual bug discovered by the example test:
managedSecurityGroups
was missing anomitempty
, which caused creation of a bare cluster object to fail. We will likely want to visit the minimum set of required parameters, but this was an obvious bug.Running the tests requires a set of assets to have been downloaded, which the
make test
target conveniently does for us. However, it wasn't possible to restrictmake test
to only the tests we are interested in, so we give it an optionalTEST_PATHS
parameters.Finally we add a new test suite with 2 trivial tests, enough to demonstrate that it is working correctly.