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

AzureCLI integration #427

Merged
merged 13 commits into from
Aug 8, 2019
Merged

AzureCLI integration #427

merged 13 commits into from
Aug 8, 2019

Conversation

seriva
Copy link
Collaborator

@seriva seriva commented Aug 7, 2019

  • Added login to Azure-CLI and selection of subscription.
  • Added creation of service principle.
  • Added creation of the resource group.
  • Added possibility to run without service principal

erzetpe
erzetpe previously approved these changes Aug 7, 2019
lukurde
lukurde previously approved these changes Aug 7, 2019
@seriva seriva dismissed stale reviews from lukurde and erzetpe via 9816f51 August 7, 2019 14:27
lukurde
lukurde previously approved these changes Aug 8, 2019
vnet_address_pool: 10.1.0.0/20
use_public_ips: False # When not using public IPs you have to provide connectivity via private IPs (VPN)
use_service_principle: False
Copy link
Contributor

Choose a reason for hiding this comment

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

should be probably use_service_principal

Copy link
Contributor

Choose a reason for hiding this comment

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

do we also need this for AWS?

@seriva seriva merged commit ef96749 into hitachienergy:develop Aug 8, 2019
@seriva seriva deleted the feature/azure-resource-group branch August 9, 2019 10:11
to-bar pushed a commit to to-bar/epiphany that referenced this pull request May 6, 2020
* - Added login to Azure-CLI and selection of subscription.
- Added creation of service principle.
- Added creation of the resource group.

* - Added posibility to run without service principal

* - Changed text outputs

* - Fixed typo.
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.

3 participants