osism / testbed

With this testbed, it is possible to run a full OSISM installation, the baseline of the Sovereign Cloud Stack, on an existing OpenStack environment such as Cleura or REGIO.cloud.
https://osism.tech/docs/guides/other-guides/testbed
Apache License 2.0
61 stars 26 forks source link

Avoid deleting attached FIPs, better port filtering. #2419

Closed garloff closed 2 months ago

garloff commented 2 months ago

Avoid deleting attached FIPs, better port filtering.

The filter in the search_floating_ip() statement does not do what it is supposed to. Instead skip if there is a fixed_ip_address.

For port cleanup, provide better logic:

The filtering can be used to ensure we only delete IP addresses from IP ranges in subnets that we have created. This improves coexistence with other infrastructure in the smae project. (If you can, avoid coexistence, though.)

This is a port from https://github.com/SovereignCloudStack/standards/pull/712

This should fix https://github.com/osism/issues/issues/960

garloff commented 2 months ago

Note that while I have tested the version in the standards repo extensively, this one has only seen light testing. Currently we also don't use the IPADDR="ippref1,ippref2,..." filter yet when calling the script -- something that we might want to add. If we don't, we still have better protection for