Conversation
* compute.forwardingRule * compute.instanceGroup * compute.instanceGroupManager * compute.router
Codecov Report
@@ Coverage Diff @@
## dev #2235 +/- ##
==========================================
+ Coverage 88.85% 88.87% +0.01%
==========================================
Files 175 175
Lines 13348 13443 +95
==========================================
+ Hits 11860 11947 +87
- Misses 1488 1496 +8
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, great to see more resources added!
@@ -348,6 +350,24 @@ def iter_compute_firewalls(self, project_number): | |||
for rule in resources: | |||
yield _fixup_resource_keys(rule, cai_to_gcp_key_map) | |||
|
|||
def iter_compute_forwardingrules(self, project_number): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added to doc
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
after double check I realized this already exists, so deleted
@@ -498,6 +550,25 @@ def iter_compute_networks(self, project_number): | |||
for network in resources: | |||
yield _fixup_resource_keys(network, cai_to_gcp_key_map) | |||
|
|||
def iter_compute_routers(self, project_number): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added to doc
@@ -932,6 +1003,43 @@ def fetch_iam_serviceaccount_iam_policy(self, name, unique_id): | |||
# Service accounts with no IAM policy return an empty dict. | |||
return {} | |||
|
|||
def iter_iam_organization_roles(self, org_id): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for role in resources: | ||
yield role | ||
|
||
def iter_iam_project_roles(self, project_id, project_number): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Fetch IAM custom roles using Cloud Asset data. * Fetch Pub/Sub Topics and IAM Policies from Cloud Asset data. * Fetch additional Compute Engine resources from Cloud Asset data. * compute.forwardingRule * compute.instanceGroup * compute.instanceGroupManager * compute.router * Fix flake8 errors.
Thanks for opening a Pull Request!
Here's a handy checklist to ensure your PR goes smoothly.
pylint --rcfile=pylintrc
passes.These guidelines and more can be found in our contributing guidelines.