rshamunov commented on code in PR #26513:
URL: https://github.com/apache/beam/pull/26513#discussion_r1196258644


##########
playground/terraform/infrastructure/cloudbuild-manual-setup/README.md:
##########
@@ -20,97 +20,131 @@
 This directory organizes Infrastructure-as-Code to provision dependent 
resources and set up Cloud Build for Beam Playground.
 Cloud Build triggers created by terraform scripts from this directory automate 
steps described in 
[readme](https://github.com/apache/beam/blob/master/playground/terraform/README.md).
 
-## Requirements:
+# Requirements:
 
-- [GCP 
project](https://cloud.google.com/resource-manager/docs/creating-managing-projects)
-- [GCP User 
account](https://cloud.google.com/appengine/docs/standard/access-control?tab=python)
 _(Note: You will find the instruction "How to create User account" for your 
new project)_<br>
+1. [GCP 
project](https://cloud.google.com/resource-manager/docs/creating-managing-projects)
+
+2. [GCP User 
account](https://cloud.google.com/appengine/docs/standard/access-control?tab=python)
 _(Note: You will find the instruction "How to create User account" for your 
new project)_<br>
   Ensure that the account has at least the following [IAM 
roles](https://cloud.google.com/iam/docs/understanding-roles):
     - Service Account Admin
     - Storage Admin
     - Service Usage Admin
     - Cloud Build Editor
     - Security Admin
     - Service Account User
-- [gcloud CLI](https://cloud.google.com/sdk/docs/install-sdk)
-- An existing GCP Bucket to save Terraform state - `state-bucket`
-- DNS name for your Playground deployment instance
-- [Terraform](https://www.terraform.io/)
-- [Apache Beam GitHub](https://github.com/apache/beam) repository cloned 
locally
+    - Secret Manager Admin
 
-## 1. Set up the Google Cloud Build for your GCP project
+3. [Google Cloud Storage 
buckets](https://cloud.google.com/storage/docs/creating-buckets)for:
+- Terraform state for Cloud Build triggers: \<triggers-state-bucket\>
+- Cloud Build private logs: \<private-logs-bucket\>
+- Cloud Build public logs: \<public-logs-bucket\>. Don't enforce public access 
prevention on this bucket.
 
-The `playground/terraform/infrastructure/cloudbuild-manual-setup/01.setup` 
provisions dependencies required to set up Cloud Build for Playground:
-- Required API services
-- Cloud Build service account
-- IAM roles for Cloud Build service account
+4. DNS name for your Playground deployment instance
 
-#### To execute the module:
-
-**Note:**  Please see [Cloud Build 
locations](https://cloud.google.com/build/docs/locations) for the list of all 
supported locations.
+5. OS with installed software listed below:
+- [Java](https://adoptopenjdk.net/)

Review Comment:
   They required for deployment but not for trigger creation. Removed excessive 
requirements.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to