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

DM-48589: Set up preliminary DP1 Butler in idfdev #4194

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

dhirving
Copy link
Collaborator

@dhirving dhirving commented Feb 12, 2025

Set up Butler server at idfdev to host a preliminary version of Data Preview 1, using Google Cloud SQL and Google Cloud Storage.

Unlike Data Preview 0.2, end users will not be given credentials to directly access the Butler database and artifacts. Previously the credentials for Butler server had been shared with Nublado, but we now have separate secrets for the Butler server. We will continue to share secrets with Nublado on the USDF environments indefinitely, so sharing is configurable per-environment.

Similarly the Data Preview 0.2 Butler configuration file was a public resource, but for Data Preview 1 it is internal to Butler server. This required creating a separating configmap to separate the public files from private files.

@dhirving dhirving force-pushed the tickets/DM-48589 branch 2 times, most recently from c23bec6 to 632c335 Compare February 13, 2025 23:03
These have never been used.
DP1 secrets will not be made available to end users in nublado, so Butler server needs its own copy of the secrets.
Create a Butler configuration YAML file that points at the DP1 repository.  Unlike DP02 this is not a public configuration file, so the configmap was split into two, with public and private portions.
Fix an issue where a newer version of Kubernetes no longer allows regex characters in "Exact" path type matches.  This includes period ".", so it was refusing to deploy the anonymous Butler ingress which refers to paths with file extensions.
@dhirving dhirving marked this pull request as ready for review February 14, 2025 23:48
@dhirving dhirving requested a review from athornton February 14, 2025 23:55
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.

1 participant