Skip to content

chore(ci): migrate ai-platform/snippets #4100

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.

Sign up for GitHub

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

Closed
glasnt wants to merge 5 commits into from
+26 −20
Closed

chore(ci): migrate ai-platform/snippets #4100

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
9cc6ab0
chore(ci): migrate ai-platform/snippets
glasnt May 27, 2025
d6c0efd
try increase timeout
glasnt May 27, 2025
1236464
attempt to debug test failures by using a more verbose reporter
glasnt May 27, 2025
4b44369
attempt deps bump
glasnt May 27, 2025
1b2a2b8
try import change
glasnt May 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/config/nodejs-dev.jsonc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,7 @@
"recaptcha_enterprise/demosite/app", // no tests exist

// These tests are already passing in prod, so skip them in dev.
"ai-platform/snippets",
"appengine/building-an-app/build",
"appengine/building-an-app/update",
"appengine/datastore",
Expand Down
1 change: 0 additions & 1 deletion .github/config/nodejs.jsonc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,6 @@
"recaptcha_enterprise/demosite/app", // no tests exist

// TODO: fix these
"ai-platform/snippets", // PERMISSION_DENIED: Permission denied: Consumer 'projects/undefined' has been suspended.
"automl", // (untested) FAILED_PRECONDITION: Google Cloud AutoML Natural Language was retired on March 15, 2024. Please migrate to Vertex AI instead
"cloud-sql/sqlserver/tedious", // (untested) TypeError: The "config.server" property is required and must be of type string.
"compute", // GoogleError: The resource 'projects/long-door-651/zones/us-central1-a/disks/disk-from-pool-name' was not found
Expand Down
5 changes: 5 additions & 0 deletions ai-platform/snippets/ci-setup.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"secrets": {
"CAIP_PROJECT_ID": "nodejs-docs-samples-tests/nodejs-docs-samples-ai-platform-caip-project-id"
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot May 27, 2025

Choose a reason for hiding this comment

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

medium

This new ci-setup.json is a key part of resolving the permission issues for ai-platform/snippets. To ensure this configuration is effective, could you clarify the format and resolution mechanism for the secret path "nodejs-docs-samples-tests/nodejs-docs-samples-ai-platform-caip-project-id"?

Specifically:

  • How does the CI system interpret this string to fetch the secret? For instance, is nodejs-docs-samples-tests the GCP project ID where the secret is stored, and nodejs-docs-samples-ai-platform-caip-project-id the name of the secret itself?
  • Is this a standard convention within your CI, or if it's intended as a direct path, how does it align with typical Google Cloud Secret Manager path formats (e.g., projects/PROJECT_ID/secrets/SECRET_NAME/versions/latest)?

Confirming this will help ensure the CAIP_PROJECT_ID is correctly supplied to the tests. A brief note in the PR description or a comment in the file about this convention could also be beneficial for future maintainers.

}
}
Loading