-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Merge preview into AzureRM.Storage.Location #6459
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
Merge preview into AzureRM.Storage.Location #6459
Conversation
Example contained an additional parameter "Type" that is no longer valid.
Fix issue with example
Merge Network updates
…o DLL cleanup again.
…netcore resources were being optimized away.
…mes from functions in common
Versioning for 6.2.0
|
@maddieclayton There are a bunch of vmss and vm test failures in the on demand job... can you look at them? |
|
Both on demand and sign builds failed. |
|
I see the new signed job still fail:https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-sign/797/
|
|
new sign job here: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-sign/800/ @blueww Sorry for the delay, I have run into more merge issues with this PR than with all of my time here combined. Currently it seems like the sign job is failing because the Storage team and the Stack team both made significant changes to the publishmodules.ps1 script that caused issues with the signing process. Hopefully this fixes it. I'll take a look in the morning if it is still failing. |
|
@maddieclayton , @cormacpayne I have validated the “AzureRM.Storage.5.0.3-preview.nupkg” in https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-sign/800/artifact/src/Package/ All test Pass, only the version is not correct. I have send a commit to update the version to 5.0.4-preview. If the build pass, we only need to do a sanity check for this package, and then can release the signed package. |
|
The signed build has passed with my commit to upgrade version to “AzureRm. Storage.5.0.4-preview”. I have sanity check the AzureRM.Storage.5.0.4-preview.nupkg, and all works well. We can release this AzureRM.Storage.5.0.4-preview.nupkg, like the before release. Please help to update me when this is released. I need to do a sanity check after release. And we can also merge this PR now, as all build pass. |
Description
Checklist
CONTRIBUTING.mdplatyPSmodule