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

gcp: Update for direct access-logging #40

Merged
merged 1 commit into from
Aug 15, 2024
Merged

Conversation

nbrahms
Copy link
Contributor

@nbrahms nbrahms commented Jul 30, 2024

On 2024 July 30, P0 updated GCP access-logging ingest to transition from a Pub/Sub-based ingest architecture, to a direct write from the organization's logging sink.

Update the P0 TF provider to match this new model.

On 2024 July 30, P0 updated GCP access-logging ingest to transition from
a Pub/Sub-based ingest architecture, to a direct write from the organization's
logging sink.

Update the P0 TF provider to match this new model.
@nbrahms nbrahms requested review from GGonryun and komal-dhull July 30, 2024 21:51
@@ -6,7 +6,7 @@ description: |-
An installation of P0, on a single Google Cloud project, for access-log collection,
which enhances IAM assessment.
To use this resource, you must also:
install the p0_gcp_iam_assessment resource,create a Pub/Sub topic,create a logging sink, publishing to this topic,grant your logging service account permissions to publish to this Pub/Sub topic, andgrant P0 the ability to subscribe to this Pub/Sub topic.
install the p0_gcp_iam_assessment resource, andgrant P0 the ability to create logging sinks in your project.
Copy link
Contributor

@GGonryun GGonryun Jul 31, 2024

Choose a reason for hiding this comment

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

Suggested change
install the p0_gcp_iam_assessment resource, andgrant P0 the ability to create logging sinks in your project.
install the p0_gcp_iam_assessment resource, and grant P0 the ability to create logging sinks in your project.

Oh wait, looks like this was autogenerated so it's probably not possible to fix

@nbrahms nbrahms merged commit 61988b5 into main Aug 15, 2024
14 checks passed
@nbrahms nbrahms deleted the nathan/update-access-logs branch August 15, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants