remove the explicit 'required=False' (use default) from --config under update and --run-status under list parameters

Author: cegraybl

src/acrcssc/azext_acrcssc/_params.py

@@ -26,13 +26,13 @@ def load_arguments(self: AzCommandsLoader, _):
         c.argument("dryrun", options_list=["--dry-run"], help="Use this flag to see the qualifying repositories and tags that would be affected by the workflow. Default value: false. 'config' parameter is mandatory to provide with dry-run", arg_type=get_three_state_flag())
 
     with self.argument_context("acr supply-chain workflow update") as c:
-        c.argument("config", help="Configuration file path containing the json schema for the list of repositories and tags to filter within the registry. Schema example:{\"repositories\":[{\"repository\":\"alpine\",\"tags\":[\"tag1\",\"tag2\"],\"enabled\":true},{\"repository\":\"python\",\"tags\":[\"*\"],\"enabled\":false}], \"version\": \"v1\", \"tag-convention\": \"floating\"}. \"tag-convention\" is an optional property, values can be \"incremental\" (the default behavior, will increase the patch version of the tag, for example \"{repository}:{original-tag}-1\", \"{repository}:{original-tag}-2\", etc), or \"floating\" (will reuse the tag \"{repository}:{original-tag}-patched\" for patching)", required=False)
+        c.argument("config", help="Configuration file path containing the json schema for the list of repositories and tags to filter within the registry. Schema example:{\"repositories\":[{\"repository\":\"alpine\",\"tags\":[\"tag1\",\"tag2\"],\"enabled\":true},{\"repository\":\"python\",\"tags\":[\"*\"],\"enabled\":false}], \"version\": \"v1\", \"tag-convention\": \"floating\"}. \"tag-convention\" is an optional property, values can be \"incremental\" (the default behavior, will increase the patch version of the tag, for example \"{repository}:{original-tag}-1\", \"{repository}:{original-tag}-2\", etc), or \"floating\" (will reuse the tag \"{repository}:{original-tag}-patched\" for patching)")
         c.argument("schedule", help=f"schedule to run the scan and patching task. E.g. `<n>d` where n is the number of days between each run. Max value is {CONTINUOUSPATCH_SCHEDULE_MAX_DAYS}d.")
         c.argument("run_immediately", help="Set this flag to trigger the immediate run of the selected workflow task. Default value: false.", arg_type=get_three_state_flag())
         c.argument("dryrun", options_list=["--dry-run"], help="Use this flag to see the qualifying repositories and tags that would be affected by the workflow. Default value: false. 'config' parameter is mandatory to provide with dry-run", arg_type=get_three_state_flag())
 
     with self.argument_context("acr supply-chain workflow list") as c:
-        c.argument("status", arg_type=get_enum_type(WorkflowTaskState), options_list=["--run-status"], help="Status to filter the supply-chain workflow image status.", required=False)
+        c.argument("status", arg_type=get_enum_type(WorkflowTaskState), options_list=["--run-status"], help="Status to filter the supply-chain workflow image status.")
 
     with self.argument_context("acr supply-chain workflow delete") as c:
         c.argument("yes", options_list=["--yes", "-y"], help="Proceed with the deletion without user confirmation")

src/acrcssc/azext_acrcssc/cssc.py

@@ -83,8 +83,8 @@ def update_acrcssc(cmd,
                    resource_group_name,
                    registry_name,
                    workflow_type,
-                   config,
-                   schedule,
+                   config=None,
+                   schedule=None,
                    dryrun=False,
                    run_immediately=False):
     '''Update a continuous patch task in the registry.'''
@@ -141,7 +141,7 @@ def cancel_runs(cmd,
     cancel_continuous_patch_runs(cmd, resource_group_name, registry_name)
 
 
-def list_scan_status(cmd, registry_name, resource_group_name, status, workflow_type):
+def list_scan_status(cmd, registry_name, resource_group_name, workflow_type, status=None):
     '''track in continuous patch in the registry.'''
     logger.debug('Entering track_scan_status with parameters:%s %s %s', resource_group_name, registry_name, workflow_type)
 

src/acrcssc/azext_acrcssc/helper/_taskoperations.py

@@ -310,7 +310,7 @@ def _cancel_task_runs(acr_task_run_client, registry_name, resource_group_name, r
             logger.error(f"Failed to cancel task {task.name} from registry {registry_name}: {exception}")
 
 
-def track_scan_progress(cmd, resource_group_name, registry, status):
+def track_scan_progress(cmd, resource_group_name, registry, status=None):
     logger.debug("Entering track_scan_progress")
 
     cssc_tasks_exists, _ = check_continuous_task_exists(cmd, registry)