Project

General

Profile

Actions

Bug #44645

closed

cephfs-shell: Fix flake8 errors (E302, E502, E128, F821, W605, E128 and E122)

Added by Varsha Rao about 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Code Hygiene
Target version:
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
cephfs-shell
Labels (FS):
task(easy)
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

[root@localhost build]# flake8 --ignore=W503 --max-line-length=100 ../src/tools/cephfs/cephfs-shell
../src/tools/cephfs/cephfs-shell:67:1: E302 expected 2 blank lines, found 1
../src/tools/cephfs/cephfs-shell:114:78: E502 the backslash is redundant between brackets
../src/tools/cephfs/cephfs-shell:121:1: E302 expected 2 blank lines, found 1
../src/tools/cephfs/cephfs-shell:137:1: E302 expected 2 blank lines, found 1
../src/tools/cephfs/cephfs-shell:1223:25: E128 continuation line under-indented for visual indent
../src/tools/cephfs/cephfs-shell:1357:30: F821 undefined name 'e'
../src/tools/cephfs/cephfs-shell:1402:57: W605 invalid escape sequence '\A'
../src/tools/cephfs/cephfs-shell:1404:25: E128 continuation line under-indented for visual indent
../src/tools/cephfs/cephfs-shell:1405:25: E128 continuation line under-indented for visual indent
../src/tools/cephfs/cephfs-shell:1406:25: E128 continuation line under-indented for visual indent
../src/tools/cephfs/cephfs-shell:1407:25: E122 continuation line missing indentation or outdented
Actions #1

Updated by Varsha Rao about 4 years ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 34254
Actions #2

Updated by Varsha Rao about 4 years ago

  • Assignee set to Varsha Rao
  • Pull request ID changed from 34254 to 34546
Actions #3

Updated by Greg Farnum about 4 years ago

  • Status changed from Fix Under Review to Resolved
Actions #4

Updated by Varsha Rao almost 4 years ago

  • Status changed from Resolved to Pending Backport
  • Backport set to octopus
Actions #6

Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to Resolved
  • Backport deleted (octopus)
Actions

Also available in: Atom PDF