Skip to content

Conversation

@kukislav
Copy link
Contributor

@kukislav kukislav commented Dec 10, 2020

Description

  • Add option to create and update managed instance with maintenance configuration (exposing FMW feature)

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

@ghost ghost added the customer-reported Issues that are reported by GitHub users external to the Azure organization. label Dec 10, 2020
@ghost
Copy link

ghost commented Dec 10, 2020

Thank you for your contribution kukislav! We will review the pull request and get back to you soon.

@ghost
Copy link

ghost commented Dec 10, 2020

CLA assistant check
All CLA requirements met.

@yonzhan yonzhan requested a review from zhoxing-ms December 10, 2020 15:35
@yonzhan yonzhan added this to the S180 milestone Dec 10, 2020
@yonzhan
Copy link
Collaborator

yonzhan commented Dec 10, 2020

SQL

@kukislav
Copy link
Contributor Author

I checked the linter pipeline that fails and fixed all issues associated with my command. What should I do about others like this one?
image

@kukislav kukislav marked this pull request as draft December 10, 2020 17:54
@kukislav
Copy link
Contributor Author

This is not customer reported as bot labelled it, this is a new feature.

@evelyn-ys
Copy link
Member

I checked the linter pipeline that fails and fixed all issues associated with my command. What should I do about others like this one?
image

This is fixed in #16297. Please merge the latest 'dev' branch into your branch to pass CI

@yonzhan yonzhan modified the milestones: S180, S181 Dec 26, 2020
@kukislav kukislav marked this pull request as ready for review January 6, 2021 10:26
@kukislav
Copy link
Contributor Author

kukislav commented Jan 6, 2021

@evelyn-ys I have updated code and fixed everything. Please review.

@evelyn-ys evelyn-ys changed the title [SQL] Add support for maintenance configuration in managed instance operations [SQL] az sql mi create/update: Add support for maintenance configuration in managed instance operations Jan 7, 2021
@yonzhan yonzhan removed this from the S181 milestone Jan 15, 2021
@yonzhan yonzhan added this to the S182 milestone Jan 15, 2021
Copy link
Member

@evelyn-ys evelyn-ys left a comment

Choose a reason for hiding this comment

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

Please resolve conflicts before merging

@evelyn-ys evelyn-ys merged commit b3171b0 into Azure:dev Jan 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants