Reduced BigQuery Read Privilege

[osandoval011]

Fix #2366

bigquery.metadataViewer is least privilege necessary to facilitate Forseti actions that were previously permitted by the more permissive bigquer.dataViewer

[googlebot]

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again. If the bot doesn't comment, it means it doesn't think anything has changed.

[EricBeach]

@osandoval011 - re: CLA, you need to submit the commit with the email you have registered internally. This is documented at go/cla#googlers

From bbbe906 Mon Sep 17 00:00:00 2001
From: Oscar Sandoval
oscarsandoval@oscarsandoval-macbookpro.roam.corp.google.com
Date: Wed, 12 Dec 2018 15:38:46 -0800
Subject: [PATCH] changed forseti's assigned bigquery role from
bigquery.dataViewer to bigquery.metadataViewer

[osandoval011]

Ran create inventory with cai and api with each of bigquery.metadataviewer and bigquery.dataviewer. All 4 produced 54 rows with identical data. I believe this is ready for merge.

[googlebot]

CLAs look good, thanks!

[angelsungoogle]

I still see some errors in the travis run?

[osandoval011]

Tables.getData and Tables.Export were only permissions taken off by this change as can be seen here

[blueandgold]

Thanks for this PR, and adding the screenshot to verify the permission changes.

[codecov]

Codecov Report

Merging #2395 into dev will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #2395      +/-   ##
==========================================
- Coverage   88.57%   88.57%   -0.01%     
==========================================
  Files         180      180              
  Lines       14120    14117       -3     
==========================================
- Hits        12507    12504       -3     
  Misses       1613     1613

Impacted Files	Coverage Δ
...loud/forseti/services/inventory/base/cloudasset.py	100% <0%> (ø)	⬆️