search

forseti-security / resource-policy-evaluation-library

Apache License 2.0
7 stars 10 forks source link

Add terminal_values to waiters for cloudsql instances #49

Closed jceresini closed 4 years ago

jceresini commented 4 years ago

This addresses some scalability issues we ran into.

  1. We ran into Cloud SQL instances in the FAILED state, which just cause rpe-lib to wait the max wait time when we know they'll never reach a running state
  2. Calling getAncestry on every resource quickly hits cloudresourcemanager API quotas. So retrieveing the organization is removed from the resources' to_dict() call. It's still available using the resource.organization property