Bump pyupgrade target to 3.10+
... according to the versions currently supported. Change-Id: I688606852b0d9cc61c53f94f5bad7424f5882917 Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
This commit is contained in:
@@ -27,4 +27,4 @@ repos:
|
|||||||
rev: v3.20.0
|
rev: v3.20.0
|
||||||
hooks:
|
hooks:
|
||||||
- id: pyupgrade
|
- id: pyupgrade
|
||||||
args: [--py3-only]
|
args: [--py310-plus]
|
||||||
|
|||||||
@@ -33,7 +33,7 @@ class ClientException(Exception):
|
|||||||
formatted_string = "{} (HTTP {})".format(
|
formatted_string = "{} (HTTP {})".format(
|
||||||
self.message, self.http_status)
|
self.message, self.http_status)
|
||||||
if self.request_id:
|
if self.request_id:
|
||||||
formatted_string += " (Request-ID: {})".format(self.request_id)
|
formatted_string += f" (Request-ID: {self.request_id})"
|
||||||
|
|
||||||
return formatted_string
|
return formatted_string
|
||||||
|
|
||||||
|
|||||||
@@ -60,13 +60,13 @@ class AodhClientTest(base.ClientTestBase):
|
|||||||
self.assertEqual("another-name",
|
self.assertEqual("another-name",
|
||||||
self.details_multiple(result)[0]['name'])
|
self.details_multiple(result)[0]['name'])
|
||||||
|
|
||||||
params = "update --name {} {}".format(name, alarm_id)
|
params = f"update --name {name} {alarm_id}"
|
||||||
result = self.aodh('alarm', params=params)
|
result = self.aodh('alarm', params=params)
|
||||||
self.assertEqual(name,
|
self.assertEqual(name,
|
||||||
self.details_multiple(result)[0]['name'])
|
self.details_multiple(result)[0]['name'])
|
||||||
|
|
||||||
# Check update with no change is allowed
|
# Check update with no change is allowed
|
||||||
params = "update --name {} {}".format(name, name)
|
params = f"update --name {name} {name}"
|
||||||
result = self.aodh('alarm', params=params)
|
result = self.aodh('alarm', params=params)
|
||||||
self.assertEqual(name,
|
self.assertEqual(name,
|
||||||
self.details_multiple(result)[0]['name'])
|
self.details_multiple(result)[0]['name'])
|
||||||
|
|||||||
@@ -103,14 +103,14 @@ def format_string_list(objs, field):
|
|||||||
|
|
||||||
def format_dict_list(objs, field):
|
def format_dict_list(objs, field):
|
||||||
objs[field] = "\n".join(
|
objs[field] = "\n".join(
|
||||||
"- " + ", ".join("{}: {}".format(k, v)
|
"- " + ", ".join(f"{k}: {v}"
|
||||||
for k, v in elem.items())
|
for k, v in elem.items())
|
||||||
for elem in objs[field])
|
for elem in objs[field])
|
||||||
|
|
||||||
|
|
||||||
def format_move_dict_to_root(obj, field):
|
def format_move_dict_to_root(obj, field):
|
||||||
for attr in obj[field]:
|
for attr in obj[field]:
|
||||||
obj["{}/{}".format(field, attr)] = obj[field][attr]
|
obj[f"{field}/{attr}"] = obj[field][attr]
|
||||||
del obj[field]
|
del obj[field]
|
||||||
|
|
||||||
|
|
||||||
@@ -142,7 +142,7 @@ def dict_from_parsed_args(parsed_args, attrs):
|
|||||||
|
|
||||||
|
|
||||||
def dict_to_querystring(objs):
|
def dict_to_querystring(objs):
|
||||||
return "&".join(["{}={}".format(k, v)
|
return "&".join([f"{k}={v}"
|
||||||
for k, v in objs.items()
|
for k, v in objs.items()
|
||||||
if v is not None])
|
if v is not None])
|
||||||
|
|
||||||
|
|||||||
@@ -26,7 +26,7 @@ class AlarmManager(base.Manager):
|
|||||||
def _filtersdict_to_url(filters):
|
def _filtersdict_to_url(filters):
|
||||||
urls = []
|
urls = []
|
||||||
for k, v in sorted(filters.items()):
|
for k, v in sorted(filters.items()):
|
||||||
url = "q.field={}&q.op=eq&q.value={}".format(k, v)
|
url = f"q.field={k}&q.op=eq&q.value={v}"
|
||||||
urls.append(url)
|
urls.append(url)
|
||||||
return '&'.join(urls)
|
return '&'.join(urls)
|
||||||
|
|
||||||
|
|||||||
@@ -37,7 +37,7 @@ class AlarmHistoryManager(base.Manager):
|
|||||||
pagination = utils.get_pagination_options(limit, marker, sorts)
|
pagination = utils.get_pagination_options(limit, marker, sorts)
|
||||||
url = self.url % alarm_id
|
url = self.url % alarm_id
|
||||||
if pagination:
|
if pagination:
|
||||||
url = "{}?{}".format(url, pagination)
|
url = f"{url}?{pagination}"
|
||||||
return self._get(url).json()
|
return self._get(url).json()
|
||||||
|
|
||||||
def search(self, query=None):
|
def search(self, query=None):
|
||||||
|
|||||||
@@ -21,7 +21,7 @@ class QuotasManager(base.Manager):
|
|||||||
def list(self, project=None):
|
def list(self, project=None):
|
||||||
url = self.base_url
|
url = self.base_url
|
||||||
if project:
|
if project:
|
||||||
url = "{}?project_id={}".format(url, project)
|
url = f"{url}?project_id={project}"
|
||||||
|
|
||||||
return self._get(url).json()
|
return self._get(url).json()
|
||||||
|
|
||||||
|
|||||||
@@ -25,7 +25,7 @@ def gen_ref(ver, title, names):
|
|||||||
refdir = os.path.join(BASE_DIR, "ref")
|
refdir = os.path.join(BASE_DIR, "ref")
|
||||||
pkg = "aodhclient"
|
pkg = "aodhclient"
|
||||||
if ver:
|
if ver:
|
||||||
pkg = "{}.{}".format(pkg, ver)
|
pkg = f"{pkg}.{ver}"
|
||||||
refdir = os.path.join(refdir, ver)
|
refdir = os.path.join(refdir, ver)
|
||||||
if not os.path.exists(refdir):
|
if not os.path.exists(refdir):
|
||||||
os.makedirs(refdir)
|
os.makedirs(refdir)
|
||||||
|
|||||||
Reference in New Issue
Block a user