]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
cephadm: Delete the unnecessary error line in open_ports 39633/head
authordonggyu_park <donggyu_park@tmax.co.kr>
Mon, 22 Feb 2021 07:52:50 +0000 (16:52 +0900)
committerdonggyu_park <donggyu_park@tmax.co.kr>
Thu, 25 Feb 2021 02:32:57 +0000 (11:32 +0900)
In #39020, d9fbd7e is cherry picked from 70722a2. there is no bug in 70722a2,
but there is a bug in d9fbd7e. It seems that the unnecessary error line was added during cherry picking.
So error only occurs in octopus branch.
This commit directly fixes issue in octopus branch instead of cherry picking
since cherry picking from 70722a2 has already been applied to octopus branch.

This commit deletes the unnecessary error line added in d9fbd7e.
In d9fbd7e, the parameter verbose_on_failure was removed in call.
However, the unnecessary line that uses verbose_on_failure was
added in open_ports and so error occurs.

Fixes: https://tracker.ceph.com/issues/49467
Signed-off-by: Donggyu Park <donggyu_park@tmax.co.kr>
src/cephadm/cephadm

index 8de809d75bf126dd0aef5a65040906b088a0fd6a..4e9f5ff6fe3a86ae3cbe00b48a0c760f2660b67a 100755 (executable)
@@ -2381,7 +2381,6 @@ class Firewalld(object):
             else:
                 logger.debug('firewalld port %s is enabled in current zone' % tcp_port)
 
-            out, err, ret = call([self.cmd, '--permanent', '--query-port', tcp_port], verbose_on_failure=False)
     def apply_rules(self):
         # type: () -> None
         if not self.available: