From: Andrew Grimberg <agrimberg@linuxfoundation.org>
Date: Fri, 26 Aug 2022 20:15:40 +0000 (-0700)
Subject: Refactor: Cleanup MyPy errors
X-Git-Tag: v0.37.0~3
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F47%2F70547%2F1;p=releng%2Flftools.git

Refactor: Cleanup MyPy errors

While preparing to start enforcing MyPy validation by way of pre-commit
several redeclarations of click command / functions were found. After
examining the errors from MyPy these were determined to be faulty in how
they were being implemented. They were not caught before as no linter
was in place that could easily catch it.

Issue: RELENG-4388
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>
Change-Id: I35c2502cb08f5502fb30e3d14ac083c30209635c
---

diff --git a/lftools/openstack/cmd.py b/lftools/openstack/cmd.py
index 659222bd..b21f5437 100644
--- a/lftools/openstack/cmd.py
+++ b/lftools/openstack/cmd.py
@@ -56,7 +56,7 @@ def image(ctx):
     ),
 )
 @click.pass_context
-def cleanup(ctx, days, hide_public, ci_managed, clouds):
+def image_cleanup(ctx, days, hide_public, ci_managed, clouds):
     """Cleanup old images."""
     os_image.cleanup(ctx.obj["os_cloud"], ci_managed=ci_managed, days=days, hide_public=hide_public, clouds=clouds)
 
@@ -68,7 +68,7 @@ def cleanup(ctx, days, hide_public, ci_managed, clouds):
 @click.option("--days", type=int, default=0, help="Find images older than or equal to days.")
 @click.option("--hide-public", type=bool, default=False, help="Ignore public images.")
 @click.pass_context
-def list(ctx, days, hide_public, ci_managed):
+def image_list(ctx, days, hide_public, ci_managed):
     """List cloud images."""
     os_image.list(ctx.obj["os_cloud"], ci_managed=ci_managed, days=days, hide_public=hide_public)
 
@@ -103,8 +103,8 @@ def upload(ctx, image, name, disk_format):
     os_image.upload(ctx.obj["os_cloud"], image, name, disk_format)
 
 
-image.add_command(cleanup)
-image.add_command(list)
+image.add_command(image_cleanup, "cleanup")
+image.add_command(image_list, "list")
 image.add_command(share)
 image.add_command(upload)
 
@@ -133,10 +133,10 @@ def server(ctx):
     pass
 
 
-@click.command()
+@click.command(name="cleanup")
 @click.option("--days", type=int, default=0, help="Find servers older than or equal to days.")
 @click.pass_context
-def cleanup(ctx, days):
+def server_cleanup(ctx, days):
     """Cleanup old servers."""
     os_server.cleanup(ctx.obj["os_cloud"], days=days)
 
@@ -144,7 +144,7 @@ def cleanup(ctx, days):
 @click.command()
 @click.option("--days", type=int, default=0, help="Find servers older than or equal to days.")
 @click.pass_context
-def list(ctx, days):
+def server_list(ctx, days):
     """List cloud servers."""
     os_server.list(ctx.obj["os_cloud"], days=days)
 
@@ -153,14 +153,14 @@ def list(ctx, days):
 @click.argument("server")
 @click.option("--minutes", type=int, default=0, help="Delete server if older than x minutes.")
 @click.pass_context
-def remove(ctx, server, minutes):
+def server_remove(ctx, server, minutes):
     """Remove servers."""
     os_server.remove(ctx.obj["os_cloud"], server_name=server, minutes=minutes)
 
 
-server.add_command(cleanup)
-server.add_command(list)
-server.add_command(remove)
+server.add_command(server_cleanup, "cleanup")
+server.add_command(server_list, "list")
+server.add_command(server_remove, "remove")
 
 
 @openstack.group()
@@ -229,7 +229,7 @@ def volume(ctx):
 @click.command()
 @click.option("--days", type=int, default=0, help="Find volumes older than or equal to days.")
 @click.pass_context
-def cleanup(ctx, days):
+def volume_cleanup(ctx, days):
     """Cleanup old volumes."""
     os_volume.cleanup(ctx.obj["os_cloud"], days=days)
 
@@ -237,7 +237,7 @@ def cleanup(ctx, days):
 @click.command()
 @click.option("--days", type=int, default=0, help="Find volumes older than or equal to days.")
 @click.pass_context
-def list(ctx, days):
+def volume_list(ctx, days):
     """List cloud volumes."""
     os_volume.list(ctx.obj["os_cloud"], days=days)
 
@@ -246,11 +246,11 @@ def list(ctx, days):
 @click.argument("volume_id")
 @click.option("--minutes", type=int, default=0, help="Delete volumes if older than x minutes.")
 @click.pass_context
-def remove(ctx, volume_id, minutes):
+def volume_remove(ctx, volume_id, minutes):
     """Remove volumes."""
     os_volume.remove(ctx.obj["os_cloud"], volume_id=volume_id, minutes=minutes)
 
 
-volume.add_command(cleanup)
-volume.add_command(list)
-volume.add_command(remove)
+volume.add_command(volume_cleanup, "cleanup")
+volume.add_command(volume_list, "list")
+volume.add_command(volume_remove, "remove")
diff --git a/releasenotes/notes/mypy_refactor-03dea7aafa58ead4.yaml b/releasenotes/notes/mypy_refactor-03dea7aafa58ead4.yaml
new file mode 100644
index 00000000..2c138a09
--- /dev/null
+++ b/releasenotes/notes/mypy_refactor-03dea7aafa58ead4.yaml
@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Code refactored to pass MyPy validation. MyPy highlighted a few issues with
+    functional definitions that were overwriting each other in the command
+    subsystem but was not because of how it operates they weren't discovered.