Closed mdbooth closed 3 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 | a3c6d591183fb755fb27cfcb0fe0c939463b6adc |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/6606e14fd08a6500089cc8c9 |
Deploy Preview | https://deploy-preview-1961--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.
This needs a docs update, and also an update to the machine printcolumns.
Should be good to go now.
/test pull-cluster-api-provider-openstack-e2e-full-test
/test pull-cluster-api-provider-openstack-e2e-full-test
/test pull-cluster-api-provider-openstack-e2e-full-test
/lgtm
New changes are detected. LGTM label has been removed.
Okay, I think I found the issue.
@mdbooth: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-cluster-api-provider-openstack-e2e-test | a3c6d591183fb755fb27cfcb0fe0c939463b6adc | link | true | /test pull-cluster-api-provider-openstack-e2e-test |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
PR needs rebase.
I've decided to replace this with https://github.com/kubernetes-sigs/cluster-api-provider-openstack/pull/1988, which has a smaller scope.
This is a status field and should not be included in the spec. ProviderID is the same, but cannot be removed as it is required by Cluster API.
While we're changing it, we change the default printcolumns for OpenStackMachine to report server ID instead of provider ID. We don't need to report provider ID here because we already know everything is an openstack server. New output example:
/hold