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

Add Discovery Engine configuration to search-admin #762

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

csutter
Copy link
Contributor

@csutter csutter commented Jul 4, 2024

We are about to start some work that hooks Search Admin up to Discovery Engine (aka Google Vertex AI Search), so we need to:

  • Mount the gcloud tool config directory into the container (which allows the Google Cloud gem(s) to pick up on locally configured credentials)
  • Set appropriate environment variables for the engine and serving config that the app should use locally (integration as we no longer have a separate dev environment)

These changes are analogous to the ones made in #695 and #712 for search-api-v2.

We are about to start some work that hooks Search Admin up to Discovery
Engine (aka Google Vertex AI Search), so we need to:
- Mount the `gcloud` tool config directory into the container (which
  allows the Google Cloud gem(s) to pick up on locally configured
  credentials)
- Set appropriate environment variables for the engine and serving
  config that the app should use locally (`integration` as we no longer
  have a separate dev environment)

These changes are analogous to the ones made in #695 and #712 for
`search-api-v2`.
@csutter csutter merged commit d1d6271 into main Jul 5, 2024
2 checks passed
@csutter csutter deleted the de-serving-configs branch July 5, 2024 09:24
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