Skip to content

Conversation

@v-nestan
Copy link
Contributor

Description
This change adds support for managed instance tagging on create and update.
That is achieved by adding --tags parameter to 'az sql mi create' and 'az sql mi update'.

Once managed instance is created, tags can be

  1. updated by executing
    az sql vm update --set tags.tagName=tagValue
  2. removed by executing
    az sql vm update --remove tags.tagName, or
    az sql vm update --tags ""
  3. overridden by executing
    az sql vm update --tags tagName1=tagValue1 tagName2=tagValue2

Repair item links
https://msdata.visualstudio.com/Database%20Systems/_workitems/edit/709400
https://msdata.visualstudio.com/Database%20Systems/_workitems/edit/709401

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@yonzhan yonzhan requested review from qianwens and zhoxing-ms May 13, 2020 14:17
@yonzhan yonzhan added this to the S169 - For Build milestone May 13, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented May 13, 2020

add to S169

@v-nestan v-nestan changed the title Adding --tags parameter to 'az sql mi create' and 'az sql mi update' [SQL] az sql mi create, az sql mi update: Add --tags parameter to support resource tagging May 13, 2020
@yonzhan yonzhan modified the milestones: S169 - For Build, S170 May 16, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented May 16, 2020

add to S170

@yonzhan yonzhan modified the milestones: S170, S171 May 31, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented May 31, 2020

add to S171

@Juliehzl
Copy link
Contributor

Juliehzl commented Jun 2, 2020

any update?

@Juliehzl
Copy link
Contributor

Juliehzl commented Jun 2, 2020

can you take a look at test failures?

@yonzhan yonzhan modified the milestones: S171, S172 Jun 20, 2020
@Juliehzl Juliehzl merged commit 578b564 into Azure:dev Jul 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants