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

error handling for local testnet REST query #3739

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

etan-status
Copy link
Contributor

When querying ALTAIR_FORK_EPOCH fails, the launch_local_testnet
script got stuck because comparing CURRENT_FORK_EPOCH against it
would keep failing with [: : integer expression expected.
Querying ALTAIR_FORK_EPOCH is now retried until a number is returned.

When querying `ALTAIR_FORK_EPOCH` fails, the `launch_local_testnet`
script got stuck because comparing `CURRENT_FORK_EPOCH` against it
would keep failing with `[: : integer expression expected`.
Querying `ALTAIR_FORK_EPOCH` is now retried until a number is returned.
@tersec tersec merged commit 21a1f7e into unstable Jun 14, 2022
@tersec tersec deleted the dev/etan/testnet-numbercheck branch June 14, 2022 08:38
etan-status added a commit that referenced this pull request Jun 14, 2022
…/dag-statevars

* commit '6bf7d6468036c3f010c6ce08727d63331ada5e0d':
  update CI timeouts (#3751)
  Fix /eth/v1/node/syncing (#3720)
  use block ID vs full block in LC data caching (#3741)
  cleanup LC data helpers (#3746)
  fix LC data import for Altair fork period (#3744)
  fix CI archive step (#3752)
  Support displaying the version number in the status bar; Implements #2959 (#3747)
  ci: install cmake on windows in daily job
  don't use slow-to-compress bzip2 with larger Jenkins artifacts
  rename light client config parameters (#3740)
  error handling for local testnet REST query (#3739)
  keep fcU consistent with actual DAG (#3748)
  remove unused `getExistingForkedBlock` overload (#3742)
  Version 22.5.2
  change proposer boost to 40%
  bump merge-testnets to get new ropsten TTD (#3668)
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