Skip to content

fix(kube): replace dead keep-test Electrum endpoint#4166

Open
mswilkison wants to merge 1 commit into
mainfrom
fix/keep-test-maintainer-electrum
Open

fix(kube): replace dead keep-test Electrum endpoint#4166
mswilkison wants to merge 1 commit into
mainfrom
fix/keep-test-maintainer-electrum

Conversation

@mswilkison

@mswilkison mswilkison commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the unreachable keep-test Electrum WSS endpoint used by the TBTC v2 minter and guardian
  • align the deployment with the BOAR testnet endpoint already used by the embedded configuration and integration tests
  • preserve the existing ConfigMap and StatefulSet wiring

The legacy endpoint timed out during connection testing, while the replacement completed an Electrum WSS connection successfully.

Validation

  • git diff --check
  • repository-wide search confirms the legacy hostname is gone
  • kubectl kustomize infrastructure/kube/keep-test/tbtc-v2-maintainer, using temporary placeholder files for the gitignored password secrets
  • targeted Electrum WSS integration connection test

Fixes #4163

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The keep-test tbtc-v2-maintainer Kubernetes configuration updates its generated Electrum WSS URL to a new Electrum testnet endpoint.

Changes

Electrum endpoint update

Layer / File(s) Summary
Update Electrum WSS configuration
infrastructure/kube/keep-test/tbtc-v2-maintainer/kustomization.yaml
The electrumx-url-wss ConfigMap literal now points to wss://electrum.testnet.boar.network:443/QxbJgaSLUHqrgAa9BW7bDpnGPxrlhnCa.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Meets [#4163] by updating electrumx-url-wss to a healthy WSS endpoint while keeping the ConfigMap/StatefulSet wiring intact.
Out of Scope Changes check ✅ Passed Only the requested endpoint replacement is changed; no unrelated config or code was added.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main change: replacing the broken keep-test Electrum endpoint in kube config.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/keep-test-maintainer-electrum

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

keep-test tbtc-v2-maintainer points minter/guardian at dead Electrum WSS endpoint

1 participant