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

Refactoring/gcp/cloudresourcemanager #231

Merged
merged 28 commits into from
Apr 4, 2019

Conversation

Remi05
Copy link
Contributor

@Remi05 Remi05 commented Mar 7, 2019

Migrated GCP's Cloud Resource Manager service to the new architecture.

Note that gcp/resources/projects.py and gcp/facade/base.py are the same as in the Stackdriver logging PR and the IAM PR; they are simply shared by all services and will appear in all GCP refactoring PRs until one of the services is merged.

@Remi05 Remi05 added this to the Iteration #4 milestone Mar 7, 2019
@Remi05 Remi05 self-assigned this Mar 7, 2019
@Remi05 Remi05 requested review from x4v13r64, Aboisier and misg March 7, 2019 04:55
@codecov-io
Copy link

codecov-io commented Mar 7, 2019

Codecov Report

Merging #231 into develop will increase coverage by 0.12%.
The diff coverage is 43.75%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #231      +/-   ##
===========================================
+ Coverage    35.17%   35.29%   +0.12%     
===========================================
  Files          174      176       +2     
  Lines         5715     5734      +19     
===========================================
+ Hits          2010     2024      +14     
- Misses        3705     3710       +5
Impacted Files Coverage Δ
...ders/gcp/resources/cloudresourcemanager/service.py 100% <100%> (ø)
...ers/gcp/resources/cloudresourcemanager/bindings.py 30% <30%> (ø)
ScoutSuite/providers/gcp/configs/services.py 50% <50%> (ø) ⬆️
ScoutSuite/providers/gcp/facade/gcp.py 50% <50%> (ø) ⬆️
...Suite/providers/gcp/facade/cloudresourcemanager.py 55.55% <55.55%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7eec287...c4787d0. Read the comment docs.

@x4v13r64
Copy link
Collaborator

x4v13r64 commented Mar 7, 2019

👏 👏 👏

Copy link
Contributor

@Aboisier Aboisier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same same.

@Aboisier Aboisier mentioned this pull request Mar 21, 2019
53 tasks
@Remi05 Remi05 changed the base branch from refactoring/resource-configs to develop April 2, 2019 02:20
@Remi05 Remi05 requested review from misg and Aboisier April 2, 2019 02:38
@Remi05
Copy link
Contributor Author

Remi05 commented Apr 2, 2019

Added left menu for projects and fixed UI.

Before :

cloudresourcemanager_before

After :

cloudresourcemanager_after

@Remi05 Remi05 changed the base branch from develop to refactoring/gcp/stackdriverlogging April 2, 2019 03:08
Copy link
Contributor

@misg misg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Remi05 Remi05 changed the base branch from refactoring/gcp/stackdriverlogging to develop April 4, 2019 01:51
@Remi05 Remi05 merged commit 20866cd into develop Apr 4, 2019
@Remi05 Remi05 deleted the refactoring/gcp/cloudresourcemanager branch April 4, 2019 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants