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

Configure Balancer V2 subgraph for Goerli #391

Merged
merged 1 commit into from
Jul 27, 2022
Merged

Configure Balancer V2 subgraph for Goerli #391

merged 1 commit into from
Jul 27, 2022

Conversation

nlordell
Copy link
Contributor

@nlordell nlordell commented Jul 27, 2022

This PR configures our Subgraph initialization to know about the Görli subgraph.

This is the last piece required for Balancer V2 to work on Görli.

Test Plan

Make sure we can query Balancer V2 liquidity on Görli:

% cargo run -p solver -- \
  --baseline-sources BalancerV2 \
  --solver-account 0xA70B6e99FCF41bF78d626a95B8C095b87D4A4285 \ 
  --transaction-strategy DryRun \
  --orderbook-url https://barn.api.cow.fi/goerli/                            
...
2022-07-27T13:53:32.736Z  INFO shared::gas_price_estimation: estimator Web3, networkid 5
2022-07-27T13:53:32.856Z  INFO solver: using baseline sources baseline_sources=[BalancerV2]
2022-07-27T13:53:34.019Z DEBUG shared::sources::balancer_v2::pool_init: initialized registered pools block=7299817 pools=100
...
2022-07-27T13:53:37.348Z DEBUG solver::driver: single run finished ok
...

@nlordell nlordell requested a review from a team as a code owner July 27, 2022 13:55
@nlordell nlordell enabled auto-merge (squash) July 27, 2022 13:56
@nlordell nlordell disabled auto-merge July 27, 2022 13:56
@nlordell nlordell enabled auto-merge (squash) July 27, 2022 13:56
@nlordell nlordell merged commit b183252 into main Jul 27, 2022
@nlordell nlordell deleted the bal-goerli branch July 27, 2022 14:20
@github-actions github-actions bot locked and limited conversation to collaborators Jul 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants