better name

Author: marioaxonops

cli/README.md

@@ -148,7 +148,7 @@ Manages **Scheduled Repair** in AxonOps.
   This parameter accepts a string value.
 * `--paxosonly` Run paxos repair only. Default is false.
 * `--skippaxos` Skip paxos repair. Default is false.
-* `--disabled` Disable Scheduled Repair. This option needs to be paired with a tags value to identify which scheduled
+* `--delete` Delete Scheduled Repair. This option needs to be paired with a tags value to identify which scheduled
   repair job to disable.
 
 #### Examples:
@@ -266,8 +266,8 @@ Run a scheduled repair skipping paxos repair with a cron expression:
 $ pipenv run python axonops.py scheduledrepair --scheduleexpr '0 0 * * 0' --paxosonly
 ```
 
-Disable a scheduled repair job with specific tags:
+Delete a scheduled repair job with specific tags:
 
 ```shell
-$ pipenv run python axonops.py scheduledrepair --disabled --tags 'Weekly repair'
+$ pipenv run python axonops.py scheduledrepair --delete --tags 'Weekly repair'
 ```

cli/axonopscli/application.py

@@ -120,8 +120,8 @@ def run(self, argv: Sequence):
                                             help='Comma-separated list of datacenters to repair, if not specified all datacenters are included')
         scheduledrepair_parser.add_argument('--tags', type=str, required=False,
                                             help='Tag for the repair job', default="")
-        scheduledrepair_parser.add_argument('--disabled', action='store_true',
-                                            help='Disable the scheduled repair instead of enabling it')
+        scheduledrepair_parser.add_argument('--delete', action='store_true',
+                                            help='Delete the scheduled repair instead of enabling it')
 
         paxos_group = scheduledrepair_parser.add_mutually_exclusive_group()
         paxos_group.add_argument('--paxosonly', action='store_true', default=False,
@@ -136,8 +136,8 @@ def run(self, argv: Sequence):
             parser.error("--tables requires --keyspace")
 
         # ensure --disabled is only used together with tags
-        if getattr(parsed_result, "disabled", None) and not getattr(parsed_result, "tags", None):
-            parser.error("--disabled requires --tags")
+        if getattr(parsed_result, "delete", None) and not getattr(parsed_result, "tags", None):
+            parser.error("--delete requires --tags")
 
         # ensure --excludedtables is only used together with --keyspace
         if getattr(parsed_result, "excludedtables", None) and not getattr(parsed_result, "keyspace", None):

cli/axonopscli/components/scheduled_repair.py

@@ -31,7 +31,7 @@ def remove_old_repairs_from_axonops(self):
                 if self.args.v:
                     print("No response received when checking for existing scheduled repair")
                 return
-            elif 'ScheduledRepairs' in response:
+            elif 'ScheduledRepairs' in response and response['ScheduledRepairs']:
                 for repair in response['ScheduledRepairs']:
                     if self.args.v:
                         print(f"Checking scheduled repair: {repair['ID']}")
@@ -40,7 +40,7 @@ def remove_old_repairs_from_axonops(self):
                         print(repair['Params'])
 
             else:
-                print("Invalid outcome when checking for existing scheduled repair")
+                print("Repair tag not found, this will be threaded as a new scheduled repair")
         else:
             if self.args.v:
                 print("No tag provided, this will be threaded as a new scheduled repair")
@@ -100,9 +100,9 @@ def set_repair(self):
         if self.args.v:
             print("POST", self.full_add_repair_url, self.repair_data)
 
-        if self.args.disabled:
+        if self.args.delete:
             if self.args.v:
-                print("This scheduled repair is disabled, not sending to AxonOps")
+                print("This scheduled repair is delete, not sending to AxonOps")
             return
 
         self.axonops.do_request(