Skip to content

feat(infra): remove second WalletConnect node operator from US - #340

Merged
xDarksome merged 1 commit into
mainfrom
feat/remove-second-wc-us-operator
Sep 10, 2026
Merged

xDarksome merged 1 commit into
mainfrom
feat/remove-second-wc-us-operator

Conversation

@xDarksome

Copy link
Copy Markdown
Member

Description

Title

How Has This Been Tested?

Not tested

Due Diligence

  • Breaking change
  • Requires a documentation update
  • Requires a e2e/integration test update

@xDarksome xDarksome self-assigned this Sep 10, 2026
@claude

claude Bot commented Sep 10, 2026 •

Copy link
Copy Markdown

Claude finished @xDarksome's task in 1m 14s —— View job


Review Complete

Reviewed infrastructure changes removing second WalletConnect node operator from US region.

Todo List

  • Read changed Terraform file
  • Analyze the diff
  • Check for orphaned SOPS configuration file
  • Check for other potential orphaned resources
  • Complete review and report findings

Issues Found

Issue 1: Orphaned SOPS Configuration File
File: infra/mainnet/sops/us.wallet-connect-2.json
Problem: This SOPS configuration file still exists but is no longer referenced after removing the wallet-connect-2 operator from us_operators. The Terraform code at line 266 references sops/us.${each.key}.json, and since wallet-connect-2 is removed from the us_operators map, this file is now orphaned.

Recommendation: Delete infra/mainnet/sops/us.wallet-connect-2.json as part of this PR to complete the cleanup.

Fix this →


Note: VPC CIDR vpc_cidr_octet = 0 (10.0.0.0/16) previously used by the removed US operator is now used by eu_operators.wallet-connect-2. This is acceptable since VPC CIDRs only need to be unique within a region, not across regions.


@xDarksome
xDarksome force-pushed the feat/remove-second-wc-us-operator branch from 080c907 to f7152b7 Compare September 10, 2026 11:25
@xDarksome
xDarksome merged commit 03a7238 into main Sep 10, 2026
12 checks passed
@xDarksome
xDarksome deleted the feat/remove-second-wc-us-operator branch September 10, 2026 11:36
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