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

fscloud module: create global 'deny' rule when more narrow scoped rules are created by the module #388

Closed
vburckhardt opened this issue Jan 18, 2024 · 1 comment · Fixed by #396
Assignees
Labels
enhancement New feature or request internal-synced

Comments

@vburckhardt
Copy link
Member

Description

2 related aspects in this ticket:

  1. Add the ability to scope a rule per region (in addition to the existing instance_id, resourcegroup, tags)
  2. When a scope is specified in a rule, get the fscloud module to also create a global 'deny' all rule for the service (using the 1.1.1.1 context). In other words, create 2 rules, a global one and a scoped one. There should be a flag to opt out of this behavior by service (or more exactly by pseudo service to take account of the 2 pseudo services management and cluster for kube)

Context:


By submitting this issue, you agree to follow our Code of Conduct

@vburckhardt vburckhardt added the enhancement New feature or request label Jan 18, 2024
@Aashiq-J Aashiq-J self-assigned this Jan 22, 2024
@Aashiq-J
Copy link
Member

@vburckhardt ,
For the 1st aspect, all the services doesn't support region attribute. For example, kms doesn't have an option of region as an attribute.
Screenshot 2024-01-23 at 4 43 09 PM

In that case, should we have a validation to check for which services support regions?
even if we dont have a validation and user provides region for a service which doesn't support region, the terraform apply doesn't fail. It creates without the region attribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request internal-synced
Projects
None yet
3 participants