Skip to content
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

Azure Automation #125

Closed

Conversation

FISHMANPET
Copy link

Adds support for Azure Automation deployment type, fixes #124, I'll add a docs page for this as well but wanted to get a PR created to (hopefully) run a build test

@FISHMANPET FISHMANPET marked this pull request as ready for review May 27, 2019 22:27
@FISHMANPET
Copy link
Author

This should be ready to go. Give it a single script and it'll deploy it to the specified Azure Automation account. It will work for my needs, and hopefully it's general enough that it will work for anyone else that wants to deploy a runbook as well.

One thing that I was a little confused about when writing the definition, in what scenario would the $Deployment passed into the script be a set of multiple deployments? The existing scripts all have a line like foreach ($deploy in $Deployment) (as does mine) but I can't find any examples of a deployment that would have multiple deployments that would need the foreach loop.

@FISHMANPET
Copy link
Author

You know, I've tried to use this and it's actually not very useful. The problem is that there are 5 types of runbooks, but two of the can be PS1 files. For my usecase I can assume they're "Powershell" runbooks but I can't know that someone else hasn't written a bunch of "Powershell Workflow" runbooks instead. So it'd be nice to be able to deploy a whole directory, but you need some way to specify what type each runbook is.

For my own use I was able to take what I learned writing this to deploy my code but I won't be able to use this because I can't figure out a way to generalize it enough for PSDeploy but have the options to make it specific enough for actual use.

@RamblingCookieMonster
Copy link
Owner

Ahh too bad! I'll close this, if someone has use of it, will be sure to point them this way - cheers!

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.

New Deployment Type: Azure Automation runbook
2 participants