Cmdlets for Cross server gt #1471
Conversation
Conflicts: src/ResourceManager/Sql/Commands.Sql.Test/packages.config src/ResourceManager/Sql/Commands.Sql/Commands.Sql.csproj src/ResourceManager/Sql/Commands.Sql/Properties/Resources.resx src/ResourceManager/Sql/Commands.Sql/packages.config
The server communication link tests will be re-enabled as soon as the feature is fully deployed to the region the tests use.
|
Can one of the admins verify this patch? |
|
Hi @adumitr, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
|
Is it at all possible to get this merged today? Also, after, please send me some details on how to get the changes published so they become publicly accessible (my alias is adumitr) |
|
@azuresdkci add to whitelist |
|
After another build fix and retrying a few builds to get flaky tests to pass, finally got a successful build. Can we complete this merge? Thanks |
|
what is the status with this pull request? It says checks have been pending for way too long, plus there had been a prior successful build that passed all checks. What does it take at this point to complete this? |
|
@azuresdkci retest this please |
| { | ||
| [Cmdlet(VerbsCommon.Get, "AzureRmSqlServerCommunicationLink", | ||
| ConfirmImpact = ConfirmImpact.None)] | ||
| public class GetAzureSqlServerCommunicationLink : AzureSqlServerCommunicationLinkCmdletBase |
There was a problem hiding this comment.
@adumitr What is the OutputType for this cmdlet? Please add the attribute for OutputType with the cmdlet OutputType
|
Is anyone still around to merge this? |
|
@adumitr please fix merge conflicts |
|
Resolved conflicts. Does the merge work now? |
Cmdlets for Cross server gt
No description provided.