Skip to content

Commit

Permalink
Merge pull request #53110 from adk3798/wip-61545-reef
Browse files Browse the repository at this point in the history
reef: cephadm: Adding support to configure public_network cfg section

Reviewed-by: John Mulligan <jmulligan@redhat.com>
  • Loading branch information
adk3798 committed Aug 31, 2023
2 parents 7ee6f7a + a93d2a4 commit 703c63c
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/cephadm/cephadm.py
Original file line number Diff line number Diff line change
Expand Up @@ -5649,8 +5649,10 @@ def finish_bootstrap_config(
cli(['config', 'set', 'global', 'container_image', f'{ctx.image}'])

if mon_network:
logger.info(f'Setting mon public_network to {mon_network}')
cli(['config', 'set', 'mon', 'public_network', mon_network])
cp = read_config(ctx.config)
cfg_section = 'global' if cp.has_option('global', 'public_network') else 'mon'
logger.info(f'Setting public_network to {mon_network} in {cfg_section} config section')
cli(['config', 'set', cfg_section, 'public_network', mon_network])

if cluster_network:
logger.info(f'Setting cluster_network to {cluster_network}')
Expand Down

0 comments on commit 703c63c

Please sign in to comment.