Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IoT] add iot device update command and resolve #1372 #1486

Merged
merged 2 commits into from
Dec 1, 2016

Conversation

xscript
Copy link
Contributor

@xscript xscript commented Dec 1, 2016

  1. Add iot device update command.
  2. Update device management REST API version to 2016-11-14.
  3. Support --ids parameter to resolve [IoT] Support --ids parameter where applicable. #1372 .
  4. Add more examples to command help documents.

Help message for new command is shown as below.

>az iot device update -h

Command
    az iot device update: Update metadata of a device in an IoT Hub.

Arguments
    --device-id -d [Required]: Device Id.
    --hub-name     [Required]: IoT Hub name.
    --resource-group -g      : Name of resource group.

Generic Update Arguments
    --add                    : Add an object to a list of objects by specifying a path and key value
                               pairs.  Example: --add property.listProperty <key=value, string or
                               JSON string>.
    --remove                 : Remove a property or an element from a list.  Example: --remove
                               property.list <indexToRemove> OR --remove propertyToRemove.
    --set                    : Update an object by specifying a property path and value to set.
                               Example: --set property1.property2=<value>.

Examples
    Disable a device.
        az iot device update --hub-name my-iot-hub --device-id my-device --set status=disabled

@mention-bot
Copy link

@xscript, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tjprescott and @derekbekoe to be potential reviewers.

Copy link
Member

@tjprescott tjprescott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I take it the recent PR fixed the issue you had with the default name in the IDs parameter. This LGTM.

@tjprescott tjprescott merged commit a317d18 into Azure:master Dec 1, 2016
@xscript xscript deleted the kevinzha branch December 2, 2016 01:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants