search

sapcc / manila

Shared filesystem management project for OpenStack.
http://openstack.org
Apache License 2.0
0 stars 2 forks source link

Add NeutronClientException in retry_params #172

Closed kpawar-sap closed 4 months ago

kpawar-sap commented 4 months ago

Previously observed functions which needs to retry on keystone authentication are now updated to retry on neutron client exception as well.

Change-Id: Iee79fc622b09f90f89d7924f1e9d85c7cbabbfba

Carthaca commented 4 months ago

I'm not sure if we would need that same logic https://github.com/sapcc/manila/pull/172/files#diff-b353b61bc657e6d1177e4d34206515ee6aff72ecaf04485828976dc126921101R193-R204 to check if the port creation did work although we got an error at https://github.com/sapcc/manila/pull/172/files#diff-b353b61bc657e6d1177e4d34206515ee6aff72ecaf04485828976dc126921101R190, too

But we can add that, once we see orphans due to that (the error pattern would be something like: 3 ports created, but only 2 known to manila and cleaned up)