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

pcli cannot deserialize AppParameters from testnet-preview #3716

Closed
turbocrime opened this issue Jan 31, 2024 · 6 comments
Closed

pcli cannot deserialize AppParameters from testnet-preview #3716

turbocrime opened this issue Jan 31, 2024 · 6 comments

Comments

@turbocrime
Copy link
Contributor

turbocrime commented Jan 31, 2024

Describe the bug
pcli cannot deserialize AppParameters from testnet-preview

To Reproduce
Steps to reproduce the behavior:

  1. git checkout main
  2. PENUMBRA_NODE_PD_URL="https://grpc.testnet-preview.penumbra.zone/" cargo run --release --bin pcli v list-tx-hashes
  3. see error

Expected behavior
pcli should be able to interact with grpc

Screenshots

Screenshot 2024-01-31 at 15 46 56

Additional context
web clients are also unable to deserialize penumbra-zone/web#443

Screenshot 2024-01-31 at 15 15 31
@github-project-automation github-project-automation bot moved this to Future in Testnets Jan 31, 2024
@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra Jan 31, 2024
@conorsch
Copy link
Contributor

conorsch commented Feb 1, 2024

FYI preview deploys were stuck today:

preview-deploys-stuck

As of #3717 that should be resolved, and we'll correctly be serving latest main again. However, at a glance I expect the problem to persist, because the intervening PRs don't seem to touch the logic in question. Will test after latest deploy finishes and confirm.

@conorsch
Copy link
Contributor

conorsch commented Feb 1, 2024

Indeed, still a problem:

❯ cargo run --release --bin pcli -- --home ~/.local/share/pcli-preview view sync
    Finished release [optimized] target(s) in 0.26s
     Running `target/release/pcli --home /home/conor/.local/share/pcli-preview view sync`
Error: status: Internal, message: "protocol error: received message with invalid compression flag: 52 (valid flags are 0 and 1) while receiving response with status: 404 Not Found", details: [], metadata: MetadataMap { headers: {"content-type": "text/plain; charset=utf-8", "x-content-type-options": "nosniff", "content-length": "19", "date": "Thu, 01 Feb 2024 00:14:20 GMT"} }

❯ git rev-parse HEAD
66b13e98643b8af8285238c62657ded81e8824f1

We can rule out these PRs as the cause of the problem:

because they hadn't been deployed at the time this issue was reported.

@turbocrime
Copy link
Contributor Author

it seems to be working for me, now.

@erwanor
Copy link
Member

erwanor commented Feb 1, 2024

status: 404 Not Found - I think your window manager setup made you so fast you frontran the deployment

@conorsch
Copy link
Contributor

conorsch commented Feb 1, 2024

For me too:

❯ cargo run --release --bin pcli -- --home ~/.local/share/pcli-preview view balance
    Finished release [optimized] target(s) in 0.27s
     Running `target/release/pcli --home /home/conor/.local/share/pcli-preview view balance`
Scanning blocks from last sync height 72 to latest height 72
[0s] ██████████████████████████████████████████████████       0/0       0/s ETA: 0s
 Account  Amount          
 # 0      20000gm         
 # 0      500000test_usd  
 # 0      100cube         
 # 0      10000pizza      
 # 0      1000000penumbra 
 # 0      20000gn         

I think I hadn't waited long enough for the public endpoints to update. I'll think more about how we can catch this more proactively: we should have pcli view sync pass post-deploy at the least.

@turbocrime
Copy link
Contributor Author

thanks!

@github-project-automation github-project-automation bot moved this from 🗄️ Backlog to Done in Penumbra Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Status: Future
Development

No branches or pull requests

3 participants