Refactoring/gcp/stackdriverlogging

[Remi05]

Migrated GCP's Stackdriver logging service to the new architecture. Stackdriver logging is one of the simpler services for GCP and serves as a great proof of concept for the refactoring of the architecture for this provider.

[codecov-io]

Codecov Report

Merging #229 into develop will increase coverage by 0.25%.
The diff coverage is 50.53%.

@@             Coverage Diff             @@
##           develop     #229      +/-   ##
===========================================
+ Coverage    34.43%   34.69%   +0.25%     
===========================================
  Files          159      166       +7     
  Lines         5549     5626      +77     
===========================================
+ Hits          1911     1952      +41     
- Misses        3638     3674      +36

Impacted Files	Coverage Δ
...viders/gcp/resources/stackdriverlogging/service.py	100% <100%> (ø)
...roviders/gcp/resources/stackdriverlogging/sinks.py	35.29% <35.29%> (ø)
ScoutSuite/providers/gcp/resources/resources.py	44.44% <44.44%> (ø)
ScoutSuite/providers/gcp/facade/utils.py	46.15% <46.15%> (ø)
ScoutSuite/providers/gcp/resources/projects.py	46.66% <46.66%> (ø)
ScoutSuite/providers/gcp/configs/services.py	50% <50%> (ø)	⬆️
ScoutSuite/providers/gcp/facade/gcp.py	50% <50%> (ø)
ScoutSuite/providers/gcp/facade/base.py	58.33% <58.33%> (ø)
...utSuite/providers/gcp/facade/stackdriverlogging.py	66.66% <66.66%> (ø)
... and 7 more

---

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 61e7c2c...31e109a. Read the comment docs.

[x4v13r64]

👏 👏 👏

[Aboisier]

Good start!

[x4v13r64]

The three Xs next to Show all were carried over from the AWS Regions left menu. They can be removed if it was not a design choice.

I think that might be a bug (carried over from AWS)? Should be displaying the # of items.

[misg]

Looks good to me! 🚀

[Aboisier]

LGTM!

[Aboisier]

Actually, L(almost)GTM, because of the parallelization stuff.

[Remi05]

The three Xs next to Show all were carried over from the AWS Regions left menu. They can be removed if it was not a design choice.

I think that might be a bug (carried over from AWS)? Should be displaying the # of items.

I fixed it along with the AWS regions left menu!

[Aboisier]

Just the one thing about the comment, but looks good otherwise.

[Aboisier]

no wait actually

[Aboisier]

ok sure all good