Fixed More Linting Errors

Author: Lucifergene

old_docker_images_auto_delete/main.py

@@ -2,8 +2,10 @@
 """
     Rules:
     1. All images with 'latest' tag would not be touched.
-    2. For a particular repository, the tag with the highest number would be preserved.
-    3. A provision is made to add exception images which would be never stopped.
+    2. For a particular repository, the tag with
+        the highest number would be preserved.
+    3. A provision is made to add exception images
+         which would be never stopped.
 """
 
 import subprocess as sp
@@ -20,36 +22,31 @@ def __init__(self):
         self.img_list = []
         self.hash_list = []
 
-
     def get_all(self):
         """
         Storing All the Docker Image Details Found on the System to a File
         """
-        file = open("temp.txt","r+" , encoding='utf-8')
+        file = open("temp.txt", "r+", encoding='utf-8')
         file.truncate(0)
-        (sp.run("sudo docker image list --format '{{.Repository}}~{{.Tag}}' >> temp.txt",
-                 shell=True , capture_output=True, check=True))
+        sp.run("sudo docker image list --format '{{.Repository}}~{{.Tag}}' >> temp.txt",shell=True , capture_output=True, check=True) # noqa
         file.close()
 
-
-
-    def is_excluded(self , tag):
+    def is_excluded(self, tag):
         """
         Add other exceptions here
         """
-        #Excluding all the images with Alpine, Buster, Slim & Latest Tags
+        # Excluding all the images with Alpine, Buster, Slim & Latest Tags
         reg = r"alpine|buster|slim|latest"
         flag = re.search(reg, tag)
         if flag is not None:
             return 0
         return 1
 
-
     def load_all(self):
         """
         Loading data from the File to the Python program
         """
-        file = open("temp.txt", "r" , encoding='utf-8')
+        file = open("temp.txt", "r", encoding='utf-8')
 
         for line in file:
             line = line.rstrip("\n")
@@ -59,23 +56,21 @@ def load_all(self):
                 regex = r"^(((\d+\.)?(\d+\.)?(\*|\d+)))(\-(dev|stage|prod))*$"
                 match = re.search(regex, image[1])
 
-                img_dict = { 'Repository':image[0] , 'Tag': match.group(2) }
+                img_dict = {'Repository': image[0], 'Tag': match.group(2)}
                 self.img_list.append(img_dict)
         file.close()
 
-
     def man_data(self):
         """
         Manipulating Data to perform the reqd operation
         """
         key = operator.itemgetter('Repository')
         b_key = [{'Repository': x, 'Tag': [d['Tag'] for d in y]}
-        for x, y in itertools.groupby(sorted(self.img_list, key=key), key=key)]
+        for x, y in itertools.groupby(sorted(self.img_list, key=key), key=key)] # noqa
 
         self.img_list.clear()
         self.img_list = b_key.copy()
 
-
     def sort_tag(self):
         """
         Sorting Tags according to the Version Numbers
@@ -94,8 +89,8 @@ def sort_tag(self):
             for new, i in enumerate(img['Tag']):
                 final_list.append(template_string.format(i))
 
-            for i in range(0 , len(img['Tag'])):
-                hash_map = { 'TagsManipulated': final_list[i] , 'TagsOriginal': temp[i] }
+            for i in range(0, len(img['Tag'])):
+                hash_map = {'TagsManipulated': final_list[i], 'TagsOriginal': temp[i]}  # noqa
                 self.hash_list.append(hash_map)
 
             final_list.sort()
@@ -104,10 +99,9 @@ def sort_tag(self):
             img['Tag'].extend(final_list[:-1])
 
         print(self.hash_list)
-        print (self.img_list)
+        print(self.img_list)
 
-
-    def hash_function(self , tag):
+    def hash_function(self, tag):
         """
         Hash Function for Error Detection
         """
@@ -119,7 +113,6 @@ def hash_function(self , tag):
                 temp = 'Error in Manipulation'
         return temp
 
-
     def remove_image(self):
         """
         Running the Docker RMI Command to Delete the Older Versions
@@ -129,9 +122,8 @@ def remove_image(self):
                 for tag in img['Tag']:
                     val = self.hash_function(tag)
                     image_url = img['Repository'] + ":" + val
-                    print ("Deleting Image : " + image_url )
-                    (sp.run("sudo docker rmi " + image_url,
-                    shell=True,capture_output=True , check=True))
+                    print("Deleting Image : " + image_url)
+                    sp.run("sudo docker rmi " + image_url,shell=True,capture_output=True , check=True) # noqa
 
 
 # Main Function