chore(portfolio-deploy): make mainnet deployment more robust#12680
Open
dckc wants to merge 5 commits into
Open
chore(portfolio-deploy): make mainnet deployment more robust#12680dckc wants to merge 5 commits into
dckc wants to merge 5 commits into
Conversation
dckc
commented
May 22, 2026
Comment on lines
+52
to
+53
| // KLUDGE! avoid flaky load-balanced setup | ||
| rpcAddrMainGood = 'https://rpc.agoric-main-eu1.ccvalidators.com:443', |
Member
Author
There was a problem hiding this comment.
@Muneeb147 and I agreed that /network-config should take care of providing a working RPC node.
Contributor
There was a problem hiding this comment.
Yes
Once SRE calls-out that main.agoric.net/network-config is providing stable rpcAddr, until then it's fine to use ccvalidators (as it's the same upstream which we use)
Member
Author
There was a problem hiding this comment.
I'm inclined to take this KLUDGE out before landing this PR.
If I see any more RPC trouble, I'll count on SRE to fix it by adjusting /network-config.
Contributor
Contributor
There was a problem hiding this comment.
But do have retries while hitting this rpc? Incase of any intermittent failure response...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Description / Upgrade Considerations
Security / Scaling / Documentation Considerations
n/a
Testing Considerations
https://github.com/Agoric/agoric-sdk/releases/tag/ymax-v0.3.2605-beta1 is an integration test
along with several runs up to https://github.com/Agoric/agoric-sdk/actions/runs/26269136326/job/77318699362