-
Notifications
You must be signed in to change notification settings - Fork 4k
Supporting Stretch Edition Database in PowerShell #2465
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
Conversation
Hi @sw145, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution! TTYL, AZPRBOT; |
Can one of the admins verify this patch? |
@sw145, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR. |
#> | ||
function Test-CreateStretchDatabase | ||
{ | ||
Test-CreateDatabaseInternal "12.0" "Southeast Asia" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should not have hard-coded locations, as this prevents tests from running in AzureChina, AzureStack, etc.
@azuresdkci add to whitelist |
I understand we can use existing Cmdlets to retrieve location list, but then how to select which one in the location list that can satisfy every developer's need? We'll still need to write scripts or change codes in the <tests.ps1> to decide locations. Also, I just use the same way how other checked-in tests are using and i believe this is just common practice for writing tests? So do i need to really make this change to my current code? |
@sw145 This is the required pattern for tests. See an example of thsi here:https://github.com/Azure/azure-powershell/blob/dev/src/ResourceManager/KeyVault/Commands.KeyVault.Test/ScenarioTests/Common.ps1#L37 |
Hi @sw145, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
@sw145 There are merge conflicts, please update |
@markcowl May I know how can see these merge conflicts between my branch and Azure/dev? I can only see the files changes between my branch and Azure/dev through the page for pull request, but i can't find where's the conflicts I tried on my local branch the following cmd: |
@sw145 if you pull the latest from the dev branch intop your branch, you will see the merge conflicts, and can fix them with git resolve. |
@markcowl Yeah, i know i need to pull the lastest dev into mine and compare, but the thing is i don't know how to realize this using git commands. Since i'm in sw145/azure-powershell not in Azure/azure-powershell, and the dev branch is not up-to-date with Azure/dev, so what i'm planning to do is like following:
Am i doing anything wrong for the above steps? Would you like to provide an example for the right things to do with specific commands? Thanks! |
@markcowl OK, i think i figured out the best way to do the pull, i'm using the following command: git checkout dev-shicwu This generates all the conflicts and i just need to solve them manually, however, what takes me extremely long time is I tried to do the following: Also I think there must be some documentations specificly for Azure to tell us what to do when merge conflicts happens so that won't waste too much time even for Github newers since everyone will come into merge conflicts from time to time. |
@sw145 The remotes you show are specific to the clone of the repo on your machine, these you set up at some point with |
@markcowl sure, thanks for sharing! So is there anything i need to do right now for this PR to be checked-in? |
on demand run here: http://azuresdkci.cloudapp.net/view/1-AzurePowerShell/job/powershell-demand/927/ |
LGTM once the on-demand build passes |
@markcowl Hi, I check the on-demand build link you gave me, looks like it failed because of the wrong name for my repository, it believe we miss "s" in the name typed in the cmd. Thanks for your help! |
This commit aims to supporting Stretch Edition Database in PowerShell