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

test: multiple validators to detect divergence #10876

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

turadg
Copy link
Member

@turadg turadg commented Jan 23, 2025

closes: #10896

Description

Fast USDC testing relies heavily on multichain-testing. This bumps numValidators from 1 to 2 so the test suite can also detect divergence between validators.

Security Considerations

none

Scaling Considerations

none

Documentation Considerations

comments

Testing Considerations

per se

Upgrade Considerations

none

@turadg turadg requested review from mhofman and dckc January 23, 2025 00:09
Copy link

cloudflare-workers-and-pages bot commented Jan 23, 2025

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: a9ad1f2
Status: ✅  Deploy successful!
Preview URL: https://2debae81.agoric-sdk.pages.dev
Branch Preview URL: https://ta-multichain-multivalidator.agoric-sdk.pages.dev

View logs

@turadg turadg added the force:integration Force integration tests to run on PR label Jan 23, 2025
@turadg turadg force-pushed the ta/multichain-multivalidator branch from b7c8627 to c536a48 Compare January 23, 2025 00:32
@turadg
Copy link
Member Author

turadg commented Jan 23, 2025

It's going to take more than bumping the integer:

release "starship-devnet" uninstalled
NAME                         READY   STATUS        RESTARTS   AGE
hermes-agoric-cosmoshub-0    2/2     Terminating   0          30m
hermes-agoric-osmosis-0      2/2     Terminating   0          30m
hermes-osmosis-cosmoshub-0   2/2     Terminating   0          30m
Script dir: /home/runner/work/_actions/cosmology-tech/starship-action/0.3.0
args:  --timeout 30m --wait --debug --namespace agoric-multichain-testing --create-namespace --set-file chains[0].scripts.updateConfig.data=/home/runner/work/agoric-sdk/agoric-sdk/scripts/update-config.sh
install.go:192: [debug] Original chart version: "0.2.20"
install.go:209: [debug] CHART PATH: /home/runner/.cache/helm/repository/devnet-0.2.20.tgz

Error: INSTALLATION FAILED: failed parsing --set-file data: open /home/runner/work/agoric-sdk/agoric-sdk/scripts/update-config.sh: no such file or directory
helm.go:84: [debug] open /home/runner/work/agoric-sdk/agoric-sdk/scripts/update-config.sh: no such file or directory
failed parsing --set-file data
helm.sh/helm/v3/pkg/cli/values.(*Options).MergeValues

Copy link
Member

@dckc dckc left a comment

Choose a reason for hiding this comment

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

getting this off my queue for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ability to test validator divergence in multichain-testing
2 participants