]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
qa/tasks/cephfs: annotate variable without comment
authorKefu Chai <tchaikov@gmail.com>
Thu, 24 Nov 2022 05:46:27 +0000 (13:46 +0800)
committerIlya Dryomov <idryomov@gmail.com>
Fri, 25 Nov 2022 12:01:01 +0000 (13:01 +0100)
to silence the warning like
```
flake8 run-test: commands[0] | flake8 --select=F,E9 --exclude=venv,.tox
./tasks/cephfs/test_full.py:6:5: F401 'typing.Optional' imported but unused
```

since we've migrated to python3.6 and up, there is no need to be
compatible with Python 2 anymore.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
(cherry picked from commit 2413da8c854bc46e79ca8d6e84f789f99747c203)

qa/tasks/cephfs/test_full.py

index 962a3d036d1fce6658a9fc402a3290d97b2531e2..2b3a7d5f95c748360489da4b09d877a70cf57d64 100644 (file)
@@ -2,11 +2,7 @@ import json
 import logging
 import os
 from textwrap import dedent
-try:
-    from typing import Optional
-except:
-    # make it work for python2
-    pass
+from typing import Optional
 from teuthology.exceptions import CommandFailedError
 from tasks.cephfs.fuse_mount import FuseMount
 from tasks.cephfs.cephfs_test_case import CephFSTestCase
@@ -22,7 +18,7 @@ class FullnessTestCase(CephFSTestCase):
     data_only = False
 
     # Subclasses define how many bytes should be written to achieve fullness
-    pool_capacity = None  # type: Optional[int]
+    pool_capacity: Optional[int] = None
     fill_mb = None
 
     def is_full(self):