[OVN] Fix Router Availability Zones for segmented networks
This patch changes the get_candidates_for_scheduling() method to also
consider all gateway chassis as potential candidates (limited by
Availability Zones) in case physnet parameter is empty (as for the
segmented networks case).
This patch is a simpler/backportable fix for the segmented networks +
Router AZs use case. In the future we should consider refactoring the
code responsible for scheduling the gateway router ports, a more detailed
explanation of what is happening/needed can be found at LP #1939144.
Change-Id: I8dc5336c6e2acd0b0a2cad0e80eee91280b9f945
Closes-Bug: #1939144
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
(cherry picked from commit 8ac9e2fe6d
)
This commit is contained in:
parent
7ab4b5f246
commit
adabfc8674
@ -1293,7 +1293,7 @@ class OVNClient(object):
|
||||
cms_bmaps.append(chassis)
|
||||
else:
|
||||
bmaps.append(chassis)
|
||||
candidates = cms_bmaps or bmaps
|
||||
candidates = cms_bmaps or bmaps or cms
|
||||
|
||||
# Filter for availability zones
|
||||
if availability_zone_hints:
|
||||
@ -1307,7 +1307,7 @@ class OVNClient(object):
|
||||
if not cms_bmaps:
|
||||
LOG.debug("No eligible chassis with external connectivity"
|
||||
" through ovn-cms-options for %s", physnet)
|
||||
LOG.debug("Chassis candidates with external connectivity: %s",
|
||||
LOG.debug("Chassis candidates for scheduling gateway router ports: %s",
|
||||
candidates)
|
||||
return candidates
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user