Add table gcp_project_organization_policy. closes #186

[Priyanka585464]

Integration test logs

Logs

SETUP: tests/gcp_projects_organization_policy []

PRETEST: tests/gcp_projects_organization_policy

TEST: tests/gcp_projects_organization_policy
Running terraform
data.google_client_config.current: Refreshing state...
data.null_data_source.resource: Refreshing state...
google_project_organization_policy.named_test_resource: Creating...
google_project_organization_policy.named_test_resource: Creation complete after 3s [id=pikachu-aaa:serviceuser.services]

Warning: Deprecated Resource

The null_data_source was historically used to construct intermediate values to
re-use elsewhere in configuration, the same can now be achieved using locals


Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

Outputs:

project_aka = gcp://cloudresourcemanager.googleapis.com/projects/pikachu-aaa
project_id = pikachu-aaa
resource_id = serviceuser.services
resource_name = turbottest73472
resource_title = constraints/serviceuser.services

Running SQL query: test-get-query.sql
[
  {
    "akas": [
      "gcp://cloudresourcemanager.googleapis.com/projects/pikachu-aaa"
    ],
    "project": "pikachu-aaa",
    "title": "constraints/serviceuser.services"
  }
]
✔ PASSED

Running SQL query: test-list-query.sql
[
  {
    "akas": [
      "gcp://cloudresourcemanager.googleapis.com/projects/pikachu-aaa"
    ],
    "location": "global",
    "project": "pikachu-aaa",
    "title": "constraints/serviceuser.services"
  }
]
✔ PASSED

Running SQL query: test-notfound-query.sql
null
✔ PASSED

Running SQL query: test-turbot-query.sql
[
  {
    "akas": [
      "gcp://cloudresourcemanager.googleapis.com/projects/pikachu-aaa"
    ],
    "title": "constraints/serviceuser.services"
  }
]
✔ PASSED

POSTTEST: tests/gcp_projects_organization_policy

TEARDOWN: tests/gcp_projects_organization_policy

SUMMARY:

1/1 passed.

Example query results

Results

### Basic info

```sql
select
  *
from
  gcp_projects_organization_policy;

+----------------------------------------+---------------------+---------+--------------+-----------------------+----------------+-----------------+----------------------------------------------------+---
| id                                     | update_time         | version | etag         | list_policy           | boolean_policy | restore_default | title                                              | ak
+----------------------------------------+---------------------+---------+--------------+-----------------------+----------------+-----------------+----------------------------------------------------+---
| run.allowedBinaryAuthorizationPolicies | 2021-05-13 06:25:19 | 0       | BwXCMDD86pE= | {"allValues":"ALLOW"} | <null>         | <null>          | constraints/run.allowedBinaryAuthorizationPolicies | ["
+----------------------------------------+---------------------+---------+--------------+-----------------------+----------------+-----------------+----------------------------------------------------+---

Check policy's previously updated time by server

select
  id,
  version,
  update_time
from
  gcp_projects_organization_policy,

+----------------------------------------+---------+---------------------+
| id                                     | version | update_time         |
+----------------------------------------+---------+---------------------+
| run.allowedBinaryAuthorizationPolicies | 0       | 2021-05-13 06:25:19 |
+----------------------------------------+---------+---------------------+

Check the policy values given to constraint.

select
  id,
  version,
  list_policy ->> 'allValues' as policy_value
from
  gcp_projects_organization_policy;

+----------------------------------------+---------+--------------+
| id                                     | version | policy_value |
+----------------------------------------+---------+--------------+
| run.allowedBinaryAuthorizationPolicies | 0       | ALLOW        |
+----------------------------------------+---------+--------------+

[Subhajit97]

Can we query specific columns, instead of *?

[Subhajit97]

Suggested change

	func tableGcpProjectsOrganizationPolicy(ctx context.Context) *plugin.Table {
	func tableGcpProjectOrganizationPolicy(ctx context.Context) *plugin.Table {

[Subhajit97]

Suggested change

plugin.Logger(ctx).Trace("listGcpProjectOrganizationPolicies", "GCP_PROJECT: ", project)

[Subhajit97]

@Priyanka585464 Is this title required?
You are not using this title value anywhere.
Currently the title is using constraint value
https://github.com/turbot/steampipe-plugin-gcp/pull/204/files#diff-90f64bc506bbef72770fec8f8ca390942b03e818c0b89357a21e3517f02b28b8R70

[Subhajit97]

Suggested change

	default = "pikachu-aaa"
	default = "niteowl-aaa"

[Subhajit97]

@Priyanka585464 There are few more changes required. Please see comments, thanks!

[Subhajit97]

LGTM

[rajlearner17]

LGTM