这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@fengtality
Copy link
Contributor

Before submitting this PR, please make sure:

  • Your code builds clean without any errors or warnings
  • You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:

@fengtality fengtality requested a review from nikspz July 25, 2023 22:36
@nikspz
Copy link
Contributor

nikspz commented Jul 26, 2023

  • Test performed:
    • Cloned and installed feature branch + latest development
    • gateway connect balancer ethereum mainnet: ok
    • balance command showing correct balance for balancer
    • connected balancer ethereum mainnet and arbitrum, polygon successfully
    • amm_arb using balancer_arbitrum_one
      • Failed to start without approval
      • approved tokens in the Client
      • failed to start amm_arb, got Price quiery failed

Steps:

  1. clone and install feature branch + latest development
  2. connect connectors and create/start amm_arb strategy using balancer_ethereum_arbitrum_one or balancer_polygon_mainnet
  3. review logs

logs_amm_arbitrum_blance.log
logs_amm_polygon_balancer.log
logs_gateway_app2023-07-26.log
gw175.zip

image

- same error for amm_arb using balancer_`polygon`_mainnet

image
image

@rapcmia
Copy link
Contributor

rapcmia commented Jul 27, 2023

  • failed to start amm_arb, got Price quiery failed

Ran the following tokens {ARB,USDC.e,WETH,USDC,DAI} and approved successfully but still getting the same error on CURL when checking prices.

image

Gateway logs:

Error: fetchPools(): Error: Issue with pool onchain data: Error: invalid BigNumber value (argument="value", value=undefined, code=INVALID_ARGUMENT, version=bignumber/5.7.0)
2023-07-27 08:46:43 | error | 	Price query failed: fetchData: Couldn't update vault data

@nikspz nikspz mentioned this pull request Nov 14, 2023
@nikspz
Copy link
Contributor

nikspz commented Jan 31, 2024

Closing this PR since bounty (#102) was re-assigned

@nikspz nikspz closed this Jan 31, 2024
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.

4 participants