Skip to content

Commit

Permalink
cephadm: expected 1 blank line before a nested definition (E306)
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Fritch <[email protected]>
  • Loading branch information
mgfritch committed Mar 3, 2021
1 parent cf9e974 commit 6c8ea9f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 3 additions & 0 deletions src/cephadm/cephadm
Original file line number Diff line number Diff line change
Expand Up @@ -982,6 +982,7 @@ def port_in_use(ctx, port_num):
# type: (CephadmContext, int) -> bool
"""Detect whether a port is in use on the local machine - IPv4 and IPv6"""
logger.info('Verifying port %d ...' % port_num)

def _port_in_use(af: socket.AddressFamily, address: str) -> bool:
try:
s = socket.socket(af, socket.SOCK_STREAM)
Expand Down Expand Up @@ -3931,8 +3932,10 @@ def command_bootstrap(ctx):
out = cli(['mgr', 'dump'])
j = json.loads(out)
epoch = j['epoch']

# wait for mgr to have it
logger.info('Waiting for the mgr to restart...')

def mgr_has_latest_epoch():
# type: () -> bool
try:
Expand Down
1 change: 0 additions & 1 deletion src/cephadm/tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ skipsdist=true
[flake8]
max-line-length = 100
ignore =
E306,
E402,
E501,
E703,
Expand Down

0 comments on commit 6c8ea9f

Please sign in to comment.