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

Feature Request: Application Security Groups #369

Closed
tomasquith opened this issue Sep 29, 2017 · 11 comments · Fixed by #905
Closed

Feature Request: Application Security Groups #369

tomasquith opened this issue Sep 29, 2017 · 11 comments · Fixed by #905

Comments

@tomasquith
Copy link

Hi, Following the announcements at Ignite it would be nice to explore the Application Security Groups functionality which is now in preview via terraform. It seems to make security groups more like AWS and infinitely more friendly than they currently are.

https://docs.microsoft.com/en-us/azure/virtual-network/create-network-security-group-preview

@tombuildsstuff
Copy link
Contributor

👋 hey @tomasquith

So I took a look into this last week and started building out support for Application Security Groups in this branch - however given Application Security Groups are still in an invite-only Preview phase, we're not able to support them at this time.

That said, once Application Security Groups enter either Public Preview / GA we'll take another look at supporting them in Terraform, as it's definitely something we want to support. As such I'm going to put this Feature Request on the back burner for the moment - and we'll pick it up again as soon as we can :)

Thanks!

@alexwo
Copy link

alexwo commented Nov 27, 2017

Hi @tombuildsstuff ,
well done !:) we are really passionate to see this feature included in terraform 👍

Microsoft has included this as part of Public Preview already!
https://azure.microsoft.com/en-us/updates/public-preview-features-for-nsgs/

Can you pick it again? :)

thanks,
Alex

@ghost
Copy link

ghost commented Dec 5, 2017

Hello @tomasquith , @tombuildsstuff !
Could you review again if possible continue development of ASG support in terraform provider?
The ASG is under public preview which does not require an invitation anymore.

Thanks,

@dcherniv
Copy link

dcherniv commented Dec 19, 2017

@tombuildsstuff I tested your branch and it seems to work in creating asgs. However, am i correct in assuming that even though your branch implements asgs, we cannot use them because as per https://docs.microsoft.com/en-us/azure/virtual-network/create-network-security-group-preview --source-asgs --destination-asgs are not yet supported in security_rule block?
If so can we bump this up. I would really love to see this feature implemented.

@bingosummer
Copy link

@tombuildsstuff Any updates for ASG?

@nirmalparekh1984
Copy link

@tombuildsstuff are there any dates you can share with you for Application Security Group implementation through terraform. we are currently managing it through a null resource and the process is become tedious specially with changes.
Are there any alternative solution on offer to implement this?

@tombuildsstuff
Copy link
Contributor

tombuildsstuff commented Feb 26, 2018

hey folks!

I've kicked off a test run against this branch and those tests are now working as expected. There's still several things needed to get this merged:

  • Rebasing the PR / upgrading to SDKv12
  • Documenting that this feature is Public Preview only at the moment
  • Adding additional tests and use-cases for this PR

(in addition to anything else which comes up in review)

So that we can launch this feature in a fully supported state - we'd be particularly interested to hear how you're using Application Security Groups at present (for instance, if you're currently using them via the azurerm_template_deployment resource - it'd be great to see a sanitized template to ensure we cover your use case). Regarding a rough timeframe - I'd hope that we can pick this up in the next couple of weeks :)

Thanks!

@tombuildsstuff tombuildsstuff self-assigned this Feb 26, 2018
@tombuildsstuff tombuildsstuff changed the title Feature Req - Application Security Groups Feature Request: Application Security Groups Feb 26, 2018
@tombuildsstuff tombuildsstuff modified the milestones: 1.1.4, 1.1.3 Feb 28, 2018
@tombuildsstuff
Copy link
Contributor

tombuildsstuff commented Feb 28, 2018

👋 hey folks!

So I had some extra time during a flight this morning and rebased this branch/added the missing documentation - as such I've opened PR #905 which adds support for provisioning Application Security Groups with Terraform :)

Thanks!

@tombuildsstuff
Copy link
Contributor

👋 hey folks!

Just to let you know that support for Application Security Groups has just been released in v1.2.0 of the AzureRM Provider - full details of what's included are available here: https://github.com/terraform-providers/terraform-provider-azurerm/blob/v1.2.0/CHANGELOG.md#120-march-02-2018

Thanks!

@Ragsboss
Copy link

I was wondering if it makes sense to abstract security constructs like security groups and have specific providers like aws azure etc. This would help with multi cloud use cases and avoid cloud vendor lockin. Any thoughts?

@ghost
Copy link

ghost commented Mar 5, 2019

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants