Add docs for az lock update.

Azure · Apr 5, 2017 · 09aca4b · 09aca4b
1 parent fa855de
commit 09aca4b
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 5 deletions.
diff --git a/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/_help.py b/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/_help.py
@@ -10,6 +10,18 @@
     type: group
     short-summary: Manage Azure locks.
 """
+helps['lock update'] = """
+    type: command
+    short-summary: Update the properties of a lock.
+    parameters:
+        - name: --notes
+          type: string
+          short-summary: 'Notes about this lock'
+    examples:
+        - name: Update a subscription level lock with new notes
+          text: >
+            az lock update --name lockName --resource-group group --notes newNotesHere
+    """
 helps['policy'] = """
     type: group
     short-summary: Manage resource policies.

diff --git a/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/_params.py b/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/_params.py
@@ -81,7 +81,7 @@
 register_cli_argument('tag', 'tag_value', options_list=('--value',))
 
 register_cli_argument('lock', 'name', options_list=('--name', '-n'))
-register_cli_argument('lock create', 'level', options_list=('--lock-type', '-t'), required=True, **enum_choice_list(LockLevel))
+register_cli_argument('lock', 'level', options_list=('--lock-type', '-t'), **enum_choice_list(LockLevel))
 register_cli_argument('lock', 'parent_resource_path', resource_parent_type)
 register_cli_argument('lock', 'resource_provider_namespace', resource_namespace_type)
 register_cli_argument('lock', 'resource_type', arg_type=resource_type_type,

diff --git a/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/custom.py b/src/command_modules/azure-cli-resource/azure/cli/command_modules/resource/custom.py
@@ -647,15 +647,14 @@ def _update_lock_parameters(parameters, level, notes, lock_id, lock_type):
     if lock_type is not None:
         parameters.type = lock_type
 
-def update_lock(name, resource_group_name=None,
-                level=None, notes=None, lock_id=None, lock_type=None):
+def update_lock(name, resource_group_name=None, level=None, notes=None):
     lock_client = _resource_lock_client_factory()
     if resource_group_name is None:
         params = lock_client.management_locks.get(name)
-        _update_lock_parameters(params, level, notes, lock_id, lock_type)
+        _update_lock_parameters(params, level, notes, None, None)
         return lock_client.management_locks.create_or_update_at_subscription_level(name, params)
     params = lock_client.management_locks.get_at_resource_group_level(resource_group_name, name)
-    _update_lock_parameters(params, level, notes, lock_id, lock_type)
+    _update_lock_parameters(params, level, notes, None, None)
     return lock_client.management_locks.create_or_update_at_resource_group_level(
         resource_group_name, name, params)