Skip to content

Commit 427c30b

Browse files
authored
Merge pull request ceph#60101 from rhcs-dashboard/fix-68376-main
mgr/dashboard: fix gateways section error:”404 - Not Found RGW Daemon not found: None” Reviewed-by: Afreen Misbah <[email protected]>
2 parents d432597 + 8dd9e9d commit 427c30b

File tree

1 file changed

+16
-15
lines changed

1 file changed

+16
-15
lines changed

src/pybind/mgr/dashboard/services/rgw_client.py

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -288,21 +288,22 @@ def instance(userid: Optional[str] = None,
288288

289289
daemon_keys = RgwClient._daemons.keys()
290290
if not daemon_name:
291-
if len(daemon_keys) > 1:
292-
try:
293-
multiiste = RgwMultisite()
294-
default_zonegroup = multiiste.get_all_zonegroups_info()['default_zonegroup']
295-
296-
# Iterate through _daemons.values() to find the daemon with the
297-
# matching zonegroup_id
298-
for daemon in RgwClient._daemons.values():
299-
if daemon.zonegroup_id == default_zonegroup:
300-
daemon_name = daemon.name
301-
break
302-
except Exception: # pylint: disable=broad-except
303-
daemon_name = next(iter(daemon_keys))
304-
else:
305-
# Handle the case where there is only one or no key in _daemons
291+
try:
292+
if len(daemon_keys) > 1:
293+
default_zonegroup = (
294+
RgwMultisite()
295+
.get_all_zonegroups_info()['default_zonegroup']
296+
)
297+
if default_zonegroup:
298+
daemon_name = next(
299+
(daemon.name
300+
for daemon in RgwClient._daemons.values()
301+
if daemon.zonegroup_id == default_zonegroup),
302+
None
303+
)
304+
daemon_name = daemon_name or next(iter(daemon_keys))
305+
except Exception as e: # pylint: disable=broad-except
306+
logger.exception('Failed to determine default RGW daemon: %s', str(e))
306307
daemon_name = next(iter(daemon_keys))
307308

308309
# Discard all cached instances if any rgw setting has changed

0 commit comments

Comments
 (0)