Skip to content

{RDBMS} Fixed bugs and updated local context test#16437

Merged
evelyn-ys merged 49 commits intoAzure:devfrom
DaeunYim:daeunyim/s181
Jan 14, 2021
Merged

{RDBMS} Fixed bugs and updated local context test#16437
evelyn-ys merged 49 commits intoAzure:devfrom
DaeunYim:daeunyim/s181

Conversation

@DaeunYim
Copy link
Copy Markdown
Contributor

@DaeunYim DaeunYim commented Jan 7, 2021

Description

  • Bug fix: not invoking error when command fails. Currently only log error message which will not give 'return 1' error when running a script with azure cli commands.
  • Bug fix: az mysql/postgres flexible-server delete command don't output error when server name is not fed
  • Test update: local context test update to properly validate the feature

Testing Guide

azdev test FlexibleServerLocalContextScenarioTest


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

DaeunYim and others added 30 commits September 18, 2020 16:40
ServerNameAvailability API to validate server names
Connect, test fixes, parameter and description updates
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Jan 7, 2021

RDBMS

@yonzhan yonzhan requested review from jsntcy and zhoxing-ms January 7, 2021 04:57
@yonzhan yonzhan added this to the S181 milestone Jan 7, 2021
@DaeunYim
Copy link
Copy Markdown
Contributor Author

CI pass fixed. Requesting review for this PR

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Jan 11, 2021

@evelyn-ys will help you with this.

@DaeunYim
Copy link
Copy Markdown
Contributor Author

@evelyn-ys Can this be merged for this train? :)

@evelyn-ys evelyn-ys changed the title [RDBMS] Fixed bugs and updated local context test {RDBMS} Fixed bugs and updated local context test Jan 14, 2021
@evelyn-ys evelyn-ys merged commit 27f5250 into Azure:dev Jan 14, 2021
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