Skip to content

add MPC_HYDRATION_RPC_WS_URL and MPC_HYDRATION_SIGNER_URI to tf#676

Open
ppca wants to merge 1 commit intodevelopfrom
xiangyi/add_hydration_env_var
Open

add MPC_HYDRATION_RPC_WS_URL and MPC_HYDRATION_SIGNER_URI to tf#676
ppca wants to merge 1 commit intodevelopfrom
xiangyi/add_hydration_env_var

Conversation

@ppca
Copy link
Copy Markdown
Contributor

@ppca ppca commented Mar 3, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the infra/partner-mainnet Terraform configuration to pass Hydration connection/configuration values into the deployed container via new env vars sourced from GCP Secret Manager.

Changes:

  • Extend node_configs schema to include Hydration RPC WS URL and signer URI secret IDs.
  • Add example tfvars values for the new Hydration secrets.
  • Add Secret Manager lookups and wire MPC_HYDRATION_RPC_WS_URL / MPC_HYDRATION_SIGNER_URI into the container environment.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
infra/partner-mainnet/variables.tf Adds Hydration-related secret ID fields to the node_configs object schema.
infra/partner-mainnet/terraform-mainnet-example.tfvars Updates the sample configuration to include the new Hydration secret IDs.
infra/partner-mainnet/resources.tf Adds Secret Manager data sources for the Hydration secrets.
infra/partner-mainnet/main.tf Exposes Hydration secrets to the container as MPC_HYDRATION_* env vars.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@auto-mausx
Copy link
Copy Markdown
Collaborator

Before this goes to main, I want to add a release to https://github.com/sig-net/mpc-infrastructure, but we can merge this into develop.

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.

3 participants