* wormhole-attester: port the v1.3.0 healthcheck hotfix
This PR fixes a missing assignment to HealthCheckState's enable
flag. This would cause the attester to ignore all settings (including
defaults) and keep the enable flag set to false because we would never
bother setting it from attestation_cfg that came at runtime.
The bug is fixed and the healthcheck behavior is hardened. Currently,
the lazy_static block will explicitly initialize the global with
default values from attestation_cfg (enable is true, window size is
100). This prevents a similar human error from disabling the
healthcheck. Secondly, main.rs overwrites the default values using the
HealthCheckState::new() constructor, instead of individual struct
member assignments. This ensures that all necessary values or their
defaults are passed as the desired healthcheck state.
* p2w_autoattest.py: Get debug output from first attestation
* wormhole-attester: fix ()-less function bug, harden usize convertion
This change fixes a bug that would cause a gigantic allocation to
happen, after the attestation_cfg::default_healthcheck_window_size
function is taken without being called and converted to a very large
usize value. This would result in a big allocation which would crash
the attester. At the core it's a misunderstanding about "Fn() as
usize" conversion which unfortunately is safe Rust.
* Trigger build
* wormhole-attester: Fix a misuse of a mutex guard
This commit changes the `let hc = <lock the mutex>` into a direct
assignment to the locked mutex guard. Rust mutices are typically
scope-based which makes using them in variables more error prone.
* Dockerfile.client: Remove unused ADD wormhole-attester.
This speeds up the build of the container, and prevents useless
triggers from the attester codebase
* p2w_autoattest.py: simplify log filtering to just info
* Repo structure refactor
This change introduces major codebase layout changes in three areas:
* Tilt devnet lives in tilt-devnet/ - all k8s and docker files,
scripts, local testing private keys are moved to this directory.
* pyth2wormhole becomes pyth-wormhole-attester and is relocated to
wormhole-attester/ - This long-needed rename will hopefully eradicate
most of the confusing naming around the attester. The Rust client binary
becomes pyth-wormhole-attester-client.
* Target-chain code lives in target-chain/ - This leaves just the
attester, third_party and pythnet-specific things at top-level.
Other opportunistic changes:
* Fixed rebuild trigger for Dockerfile.wasm
These items are a fraction of the necessary structure changes in the repo.
* cosmwasm: fix contract after faulty merge
* .github: remove unused dependabot config
* Fix path references in .github and .pre-commit-config.yml
* .github: Rename attester references to pyth-wormhole-attester
* .pre-commit-config.yaml: fix paths and run all commit hooks
* p2w-relay: Fix faulty merge resolution in favor of origin/main
* Dockerfile.pyth_relay: Fix Ethereum path reference
* Dockerfile.solana: Trip early cache with arbitrary change
* Dockerfile.pyth_relay: typo
* p2w-relay: fix evm build in npm script
* Dockerfile.solana: Retry invalidating cache again
* near -> target-chains/near
* wormhole-attester: bump on/off-chain major versions due to rename
Attester packages were renamed. This possibly breaks most dependees.
* Dockerfile.solana: Improve decoy-crate to have a real lib target
* .github/[...]/pyth-cosmwasm-contract.yml: typo
* rust-toolchain: Bump rust to stable 1.63
* rust-toolchain: use christmas nightly
* empty commit to trigger build
* attester-image-push.yaml: keep xc-attest image name intact
* multisig-wh-message-builder: remove accidental revert
* add binary query string to /latest_price_feeds
* fix precommit
* fix both rest and ws tests
* bump package version
* fix tests to use the latest schema
* Remove .openzeppelin and its docs
This directory is no longer useful as we are not using truffle for
upgrades anymore.
* Update Deploying.md doc according to the feedbacks
* Add *mnemonic* to gitignore to avoid mistakes
* wormhole attester: Add a healthcheck on the metrics port
* pyth2wormhole healthcheck: apply review advice
- Move metrics/healthcheck counter updates to be next to each other
- Change "0-sized window disables healthcheck" into an explicit config value
- move healthcheck updates past the atomic counter updates
* solana-devnet: Deploy a copy of the Mesh multisig program
* solana/keys: Add keys for multisig testing
* *.py: Remove airdrop, use devnet_setup.sh keypair instead
* Dockerfile.p2w-attest: Improve caching
- Move cheap Python script additions after the expensive rust build
- Cache Cargo's package cache to shorten the "updating crates.io registry" build steps
* Add a multisig Tilt resource, k8s yaml and runtime script
This contains most of the work on Tilt/testing harness side.
* multisig-wh-message-builder: Add init-vault subcommand
This new subcommand enables the runtime Python script to create a
multisig vault on the fly on the mock Solana devnet.
* multisig-wh-message-builder: findProgramAddress -> getMsPDA
* mutlisig-wh-message-builder: apply review advice
- Use default mesh program address
- remove unused program.json
- remove redundant null checks
- hardcode vault address (it is deterministic against the constant
mesh program pubkey)
- Start depending on solana-devnet in Tilt
- Add carol to vault and set threshold to 2
* Add new networks RPCs
* Update governance sdk to support Wormhole receiver
* Fix receiver deployment script bugs
* Add new network configurations
* Add a comment about merging wh and receiver chains
* Bump contract version
* Some refactoring to add types with JSDoc
* Use better RPCs for some networks
* Remove unneeded migration files
* Add initial syncPythState script that does upgrade
* Update truffle-config for new gas values
* Refactor to make this change easier
* stop mutating AttestationConfig
* get the product names also
* ok
* refactor
* cleanup
* more cleanup
* more cleanup
* comment
* i think this works
* fix stuff
* clippy
* more cleanup
* main
* main
* fix formatting
* blah
* test
* cleanup
* fix python
* config
* fix test
* grr
* grr
* comments
Co-authored-by: Jayant Krishnamurthy <jkrishnamurthy@jumptrading.com>
- Remove nodejs default metrics. We don't use them.
- Remove response time metric.
- Remove freshness metric and add gap metric for attestationTime and publishTime.
They are similar; however, freshness was measured upon user request but gap is
measured upon receiving the next update.
- Change receivedVaa to actually represent distinct vaa received. Prior
to this, the older vaas, or vaas with same attestation time were
not counted in this metric. This will also improve the performance.
- Refactors the code a little. `vaaBytes` type was not string and was
Buffer. It is fixed now.
Stanisław Drozd
Ali Behjati
Reisen
Daniel Chew
Jayant Krishnamurthy
guibescos
Stanisław Drozd
Thomaz Leite
Tom Pointon