Closed mdbooth closed 6 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 | de0dffa5ff0fa07a82b81769e342f8d8a953ce91 |
Latest deploy log | https://app.netlify.com/sites/kubernetes-sigs-cluster-api-openstack/deploys/660bd3d5546e2a0008bad24e |
Deploy Preview | https://deploy-preview-1989--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.
/test pull-cluster-api-provider-openstack-e2e-full-test
/test pull-cluster-api-provider-openstack-e2e-test
PR needs rebase.
/hold cancel
In case it's important later: I manually removed the needs-rebase
label. Not sure why the bot didn't do that itself, but it doesn't currently need a rebase.
/lgtm
Panics in restore functions are a particular problem because
previous
is unvalidated user-provided input. It should have been generated by a previous conversion, but we have no way to enforce that. We don't need to preserve sensible behaviour given invalid input, but we should not crash.This PR implements a generic function for fuzzing restore functions, adds v1alpha6 and v1alpha7 tests which calls the fuzzer against all restore functions, and fixes all the discovered panics.
/hold