gardener / gardener-extension-provider-openstack

Gardener extension controller for the OpenStack cloud provider (https://openstack.org).
https://gardener.cloud
Apache License 2.0
25 stars 81 forks source link

Include absent router in infra-dependency errs #794

Closed AndreasBurger closed 2 months ago

AndreasBurger commented 2 months ago

How to categorize this PR? /area disaster-recovery /kind enhancement /platform openstack

What this PR does / why we need it: With this PR, the router being absent will be considered an ERR_INFRA_DEPENDENCIES error, enabling force deletion on the shoot.

Special notes for your reviewer: I've done some tests on local shoots and this seems to only occur for shoots created using Terraform. For shoots created via our flow, deletion was able to be completed successfully even when the router was removed.

Release note:

When a missing router is being encountered during reconciliation/deletion it will now be classified as `ERR_INFRA_DEPENDENCIES`, enabling force deletion of the shoot via annotation.