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] allow security group id as source in security group rules #54

Closed
yongzhang opened this issue Jun 24, 2019 · 11 comments
Closed

Comments

@yongzhang
Copy link

yongzhang commented Jun 24, 2019

Please support to add security group id as source in security group rules:

image

image

@likexian
Copy link
Contributor

Hi @hiscal2015
Please refer to security group resource.
https://www.terraform.io/docs/providers/tencentcloud/r/security_group.html

@yongzhang
Copy link
Author

@likexian
Sorry, which argument can be defined with a security group id as source? cidr_ip?

image

@likexian
Copy link
Contributor

security group id is returns by resource tencentcloud_security_group
https://www.terraform.io/docs/providers/tencentcloud/r/security_group.html

@yongzhang
Copy link
Author

@likexian I don't think we're on the same page, I mean, one security group id can be defined as a "source" in one security group rule, just like ip or cidr block, see my picture below:

image

@likexian
Copy link
Contributor

@yongzhang
Copy link
Author

https://www.terraform.io/docs/providers/tencentcloud/r/security_group_rule.html#security_group_id is used to define the security group id you want to add rules to, not the "来源"

@yongzhang
Copy link
Author

To be more clear, from API perspective, it should be one argument under Ingress/egress.N

image

@likexian
Copy link
Contributor

Sorry @hiscal2015
I have catch what your means, we will add support to it soon.

@yongzhang
Copy link
Author

Sorry @hiscal2015
I have catch what your means, we will add support to it soon.

Wonderful! Thanks a lot.

@likexian
Copy link
Contributor

Sorry @hiscal2015
I have catch what your means, we will add support to it soon.

Wonderful! Thanks a lot.

Sorry for the trouble.

@likexian
Copy link
Contributor

likexian commented Jun 28, 2019

@hiscal2015
This has been fixed and released, please try the latest version.

The argument name is: source_sgid, please refer to: https://www.terraform.io/docs/providers/tencentcloud/r/security_group_rule.html#source_sgid

Any problem, please open a new issue.
Thank you.

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

No branches or pull requests

2 participants