Skip to content

Commit

Permalink
Merge pull request #49 from memes/fix/44_document_permission_diffs
Browse files Browse the repository at this point in the history
fix: Document the differences with F5 YAML
  • Loading branch information
memes authored Sep 5, 2023
2 parents 55a68b7 + 069cc8b commit 1c5564d
Showing 1 changed file with 21 additions and 0 deletions.
21 changes: 21 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,27 @@ and [Azure](https://github.com/memes/terraform-azure-f5-distributed-cloud-role).
> require assistance please join our
> [Slack GCP channel](https://f5cloudsolutions.slack.com/messages/gcp) and ask!
## Difference with F5 published role

F5 publishes a [YAML](https://gitlab.com/volterra.io/cloud-credential-templates/-/blob/master/gcp/f5xc_gcp_vpc_role.yaml)
declaration that encapsulates F5 recommended permissions in a fixed role; this
Terraform module includes *additional permissions* that seem to be needed.

### Added permissions for project-scoped role

* `compute.addresses.createInternal`
* `compute.addresses.deleteInternal`
* `compute.addresses.list`
* `compute.addresses.useInternal`

### Added permissions for organization-scoped role

* `compute.addresses.createInternal`
* `compute.addresses.deleteInternal`
* `compute.addresses.list`
* `compute.addresses.useInternal`
* `resourcemanager.projects.list`

## Examples

### Create the custom role at the project, and assign to an existing service account
Expand Down

0 comments on commit 1c5564d

Please sign in to comment.