-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
provider/azurerm: Add example of Spark and Cassrandra on CentOS #15123
provider/azurerm: Add example of Spark and Cassrandra on CentOS #15123
Conversation
…t script, added mac specific deployment for local testing
…hmagnitude/terraform into topic-101-vm-from-user-image
* initial commit - 101-vm-from-user-image * added tfvars and info for README * added CI config; added sane defaults for variables; updated deployment script, added mac specific deployment for local testing * prep for PR
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.
let 1 comment inline about the .travis file - also, please can you remove the image of the graph? github is not good at diffing those files going forward
Thanks
Paul
.travis.yml
Outdated
@@ -41,4 +41,4 @@ notifications: | |||
matrix: | |||
fast_finish: true | |||
allow_failures: | |||
- go: tip |
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.
please can you revert the line endings change on this file?
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.
I believe this is changed back to original, @stack72 - let me know if it's still different
added return line at 45
@stack72 I added the return line on 45 of the .travis.yml and removed the crazy graph. Let me know how it looks! |
Thanks for update this - this LGTM Paul |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
This example provisions and deploys the following resources into an Azure subscription:
The preferred method of running the scripts is the VM Extension, but there is currently an issue submitted for this that is preventing me from using that resource. https://github.com/Azure/custom-script-extension-linux/issues/95
CI process: