Skip to content

Dev: Azure Automation additional cmdlet bugfixes #1714

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

Merged
merged 35 commits into from
Jan 28, 2016
Merged

Dev: Azure Automation additional cmdlet bugfixes #1714

merged 35 commits into from
Jan 28, 2016

Conversation

safeermohammed
Copy link
Contributor

Cmdlet bugfixes
taking latest Automation SDK
Scenario tests

balukambala and others added 29 commits December 1, 2015 14:12
Adding a subscription ID property to the AutomationAccount object
Add Subscription ID property
Disabling passing Parameters value from pipeline per Bug 1731159
…bject and assigning the value from the SDK model.
Added DSC delete cmdlets
Scenario tests, DSC reportId/NodeId fixes
@hovsepm
Copy link
Contributor

hovsepm commented Jan 26, 2016

@safeermohammed remove restore.config file from the PR.

<package id="Microsoft.Azure.Management.Automation" version="0.50.2-prerelease" targetFramework="net45" />
<package id="Microsoft.Azure.Management.Resources" version="2.18.11-preview" targetFramework="net45" />
<package id="Microsoft.Azure.Management.Automation" version="2.0.0" targetFramework="net45" />
<package id="Microsoft.Azure.Management.Resources" version="2.18.7-preview" targetFramework="net45" />
Copy link
Contributor

Choose a reason for hiding this comment

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

do NOT downgrade any version. BTW, we are going to bump everyone to Resource 2.18.13.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

looks like incorrectly updated during merge within team, Fixed to be back to 2.18.11.
Sure 2.18.13 should be fine for us

@hovsepm
Copy link
Contributor

hovsepm commented Jan 27, 2016

@safeermohammed
Copy link
Contributor Author

Made exactly how it is in ASM scenario tests now.

hovsepm pushed a commit that referenced this pull request Jan 28, 2016
Dev: Azure Automation additional cmdlet bugfixes
@hovsepm hovsepm merged commit 80f8c52 into Azure:dev Jan 28, 2016
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