fix: replace 5 bare except clauses with except Exception

bin/docker-to-squash.py

@@ -1735,7 +1735,7 @@ def check_dependencies():
     try:
         command = ["setfattr", "--version"]
         shell_command(command, False, False, True)
-    except:
+    except Exception:
         logging.error("Could not find setfattr . Make sure it is installed and present on the PATH")
         return 1
 

bin/storm.py

@@ -230,7 +230,7 @@ def resolve_dependencies(artifacts, artifact_repositories, maven_local_repos_dir
     try:
         out_dict = json.loads(output)
         return out_dict
-    except:
+    except Exception:
         raise RuntimeError(f"dependency handler returns non-json response: sysout<{output}>", )
 
 
@@ -1126,7 +1126,7 @@ def set_log_level(args):
             if ":" in new_value:
                 _, timeout = new_value.split(":")
                 int(timeout)
-        except:
+        except Exception:
             raise argparse.ArgumentTypeError("Should be in the form[logger name]=[log level][:optional timeout]")
     exec_storm_class(
         "org.apache.storm.command.SetLogLevel",
@@ -1152,7 +1152,7 @@ def rebalance(args):
             new_value = int(new_value)
             if new_value < 0:
                 raise argparse.ArgumentTypeError("Executor count should be > 0")
-        except:
+        except Exception:
             raise argparse.ArgumentTypeError("Should be in the form component_name=new_executor_count")
     exec_storm_class(
         "org.apache.storm.command.Rebalance",

dev-tools/gitact/print-errors-from-test-reports.py

@@ -38,7 +38,7 @@ def print_error_reports_from_report_file(file_path):
     tree = ElementTree()
     try:
         tree.parse(file_path)
-    except:
+    except Exception:
         print("-" * 50)
         print("Error parsing %s"%file_path)
         f = open(file_path, "r")