Skip to content

Reimplementing the exisiting Automation cmdlets and adding new cmdlets#146

Merged
markcowl merged 102 commits intoAzure:devfrom
elvg:dev
Feb 10, 2015
Merged

Reimplementing the exisiting Automation cmdlets and adding new cmdlets#146
markcowl merged 102 commits intoAzure:devfrom
elvg:dev

Conversation

@elvg
Copy link
Copy Markdown
Contributor

@elvg elvg commented Feb 6, 2015

We are updating our existing cmdlets because of our new Automation MAML (not released yet) and adding new cmdlets for other Automation Resources likes certificates, variables, modules, etc.

Note that our PR hasn't been merged yet, that's why we haven't changed the MAML package version in our project. You can see the PR for azure-sdk-for-net from here: Azure/azure-sdk-for-net#948

Please Do Not merge until we change the MAML package version.

Thanks.

elvg and others added 30 commits December 10, 2014 17:44
@elvg
get-azureautomationschedule implementation and unit tests
a90585d
@safeermohammed
Runbook cmdlet get
73a750a
@safeermohammed
swapping byall and byfrom
35edea9
@kiranisaac
2 cmdlets for variables
acc3cda
@safeermohammed
Saving Runbook cmdlets
85c7220
@elvg
schedule automation cmdlets, unit tests and infrastructure for automa…
836e859 
…tion cmdlets
@kiranisaac
job stream cmdlet
6b951a3
@kiranisaac
merge with elvg
c442201
@mohanishpenta
cmdlets + tests
919256b
@kiranisaac
piping
4da0efc
@kiranisaac
operation excpetion from mpenta
68dbb3b
@kiranisaac
rename the functions and add basic unit test files
0f3ea14
@kiranisaac
update code to use list operation
b29bf88
@kiranisaac
Merge pull request #1 from kiranisaac/master
372d3dd 
Merge all changes from mpenta and kirab
@safeermohammed
Saving runbook cmdlet changes
e4e7ad5
@safeermohammed
Merge branch 'dev' of https://github.com/elvg/azure-powershell
fd7437f 
Conflicts:
	src/ServiceManagement/Automation/Commands.Automation.Test/Commands.Automation.Test.csproj
	src/ServiceManagement/Automation/Commands.Automation/Commands.Automation.csproj
	src/ServiceManagement/Automation/Commands.Automation/Common/AutomationClient.cs
	src/ServiceManagement/Automation/Commands.Automation/Common/AutomationCmdletParameterSet.cs
	src/ServiceManagement/Automation/Commands.Automation/Common/Constants.cs
	src/ServiceManagement/Automation/Commands.Automation/Common/IAutomationClient.cs
	src/ServiceManagement/Automation/Commands.Automation/Properties/Resources.Designer.cs
	src/ServiceManagement/Automation/Commands.Automation/Properties/Resources.resx
@elvg
schedule cmdlets code is refactored
f4392ef
@elvg
Merging with Safeer
0ad4028
@safeermohammed
Start runbook implementation + Unit test
0d5ea48
@safeermohammed
check for new properties
3570e5e
@safeermohammed
Merge branch 'dev' of https://github.com/elvg/azure-powershell
cc59a41
@elvg
AUtomation PS cmdlets: JobSchedules
3726c7e
@kiranisaac
updated job and joboutput to use id instead of name
2b550ec
@mohanishpenta
added unit testsfor Job cmdlets, code review comments for modules
fa06234
@elvg
updating job schedule cmdlets
a2e2633
@elvg
fixing conflicts
49d22b8
@kiranisaac
merge with mpenta
db080d8
@mohanishpenta
fixed and added unit tests for jobs
dc51793
@safeermohammed
Accounts implementation
ba3eab7
@kiranisaac
merged with mpenta and elvg - the unit test project has been disabled…
0078dbf 
… to due compile issues.. need to pull from mpenta
@azurecla
Copy link
Copy Markdown

azurecla commented Feb 6, 2015

Hi @elvg, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (elvg). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@markcowl
Copy link
Copy Markdown
Member

markcowl commented Feb 7, 2015

@elvg Please fix test failures

@markcowl
Copy link
Copy Markdown
Member

markcowl commented Feb 9, 2015

@elvg please fix build issue - looks like you need to have the correct version of automation cmdlets?

@elvg
Copy link
Copy Markdown
Contributor Author

elvg commented Feb 9, 2015

@markcowl since SDK hasn't merged yet, we don't have our automation SDK released with that version. We are still waiting it to be merged in azure-sdk-for-net.

@markcowl
Copy link
Copy Markdown
Member

markcowl commented Feb 9, 2015

@elvg, the azure-sdk-for-net PR is merged

ogail
ogail reviewed Feb 9, 2015
View reviewed changes
src/Common/Commands.ScenarioTest/Resources/Automation/AutomationTests.ps1
Copy link
Copy Markdown
Contributor

@ogail ogail Feb 9, 2015

Choose a reason for hiding this comment

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

nit: it's little bit awkward to have account name with a real person. Maybe automatedAccount is better?`

Copy link
Copy Markdown
Contributor Author

@elvg elvg Feb 9, 2015

Choose a reason for hiding this comment

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

thanks for catching this but we would like to change it in next release.

markcowl added a commit that referenced this pull request Feb 10, 2015
@markcowl
Merge pull request #146 from elvg/dev
60df675 
Reimplementing the exisiting Automation cmdlets and adding new cmdlets
@markcowl markcowl merged commit 60df675 into Azure:dev Feb 10, 2015
@elvg
Copy link
Copy Markdown
Contributor Author

elvg commented Feb 10, 2015

Thanks!

ogail pushed a commit that referenced this pull request Aug 27, 2015
Merge pull request #146 from MabOneSdk/dev1-anudeeb
02dcf16 
Scrubbing help text. Signed off by SureshT.
AzureRT referenced this pull request in AzureRT/azure-powershell Oct 16, 2015
@huangpf
Merge pull request #146 from AzureRT/dev
19cba6e 
Merge branch 'dev' of https://github.com/Azure/azure-powershell into dev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@elvg @azurecla @markcowl @ogail @safeermohammed @kiranisaac @mohanishpenta