]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
cephadm: still try to open ports in firewall on redeploy/reconfig
authorAdam King <adking@redhat.com>
Thu, 13 Apr 2023 17:05:11 +0000 (13:05 -0400)
committerAdam King <adking@redhat.com>
Thu, 1 Jun 2023 18:05:09 +0000 (14:05 -0400)
commitfdecd66f1306d3bf60780dbd44c9cb8e63b3892a
treef55a41e83fcb9874c8b3a8cb4a83f0eba7c2a39b
parentbdca2dcefa7a759b82a8b75d8bd5313e8b474d2f
cephadm: still try to open ports in firewall on redeploy/reconfig

Prior to this patch we were discarding the provided
ports on reconfig and redeploy in order to not fail
thinking there was a port conflict with the instance
of the daemon we were about to reconfig/redeploy. However,
it's still desirable for us to make sure the firewall ports
are open when we do a reconfig/redpeloy, so this refactors
the port handling approach to have it do that but
still avoid checking for port conflicts. It also include
an update of the type signature of deploy_daemon
to the py3 style. That wasn't needed for the change
but since I was added an arugment there I thought we might
as well do it now.

Signed-off-by: Adam King <adking@redhat.com>
src/cephadm/cephadm.py
src/cephadm/tests/test_cephadm.py