Merge pull request #218 from yrvine/nlp-edits

Fixed permissions issues in NLP package
GoogleCloudPlatform · Jan 15, 2025 · abdb9fa · abdb9fa
2 parents 057abd4 + 5b33c17
commit abdb9fa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/nlp-sentiment-analysis/README.md b/nlp-sentiment-analysis/README.md
@@ -53,7 +53,7 @@ sh prereq.sh
 
 3. Run the Cloud Build Job
 ```
-gcloud builds submit . --config cloudbuild.yaml
+gcloud builds submit . --config build/cloudbuild.yaml
 ```
 
 ## Testing the architecture

diff --git a/nlp-sentiment-analysis/prereq.sh b/nlp-sentiment-analysis/prereq.sh
@@ -60,7 +60,7 @@ gcloud services enable cloudbuild.googleapis.com \
 
 echo "Granting Cloud Build's Service Account IAM roles to deploy the resources..."
 PROJECT_NUMBER=$(gcloud projects describe $PROJECT_ID --format='value(projectNumber)')
-MEMBER=serviceAccount:$PROJECT_NUMBER@cloudbuild.gserviceaccount.com
+MEMBER=serviceAccount:$PROJECT_NUMBER-compute@developer.gserviceaccount.com
 add_iam_member $MEMBER roles/editor
 add_iam_member $MEMBER roles/iam.securityAdmin
 add_iam_member $MEMBER roles/documentai.admin