Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Bump RpcClient node versions#21612

Merged
mergify[bot] merged 2 commits intosolana-labs:masterfrom
CriesofCarrots:bump-rpc-client-versions
Dec 4, 2021
Merged

Bump RpcClient node versions#21612
mergify[bot] merged 2 commits intosolana-labs:masterfrom
CriesofCarrots:bump-rpc-client-versions

Conversation

@CriesofCarrots
Copy link
Copy Markdown
Contributor

@CriesofCarrots CriesofCarrots commented Dec 4, 2021

Problem

solana airdrop 1 with a master or beta client is currently incompatible with devnet/mainnet-beta. This is because it is attempting to call the non-existent getLatestBlockhash RPC endpoint because the cluster satisfies the get_node_version conditional due to v1.8 jumping the normal release process.

Summary of Changes

Bump checked version to be in sync for master/v1.9
Do same for getHighestSnapshotSlot. I think this is correct. Please confirm, @brooksprumo 🙏

cc @jackcmay

Fixes #21611

Copy link
Copy Markdown
Contributor

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, thanks for fixing this! Looks good to me.

@CriesofCarrots CriesofCarrots added the automerge Merge this Pull Request automatically once CI passes label Dec 4, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 4, 2021

Codecov Report

Merging #21612 (b8aab47) into master (9c6b95e) will decrease coverage by 0.0%.
The diff coverage is 66.6%.

@@            Coverage Diff            @@
##           master   #21612     +/-   ##
=========================================
- Coverage    81.7%    81.6%   -0.1%     
=========================================
  Files         510      510             
  Lines      142502   142502             
=========================================
- Hits       116434   116393     -41     
- Misses      26068    26109     +41     

@mergify mergify Bot merged commit 3e5a5a8 into solana-labs:master Dec 4, 2021
mergify Bot pushed a commit that referenced this pull request Dec 4, 2021
* Bump blockhash/fee api check versions

* Bump snapshot api check version

(cherry picked from commit 3e5a5a8)
mergify Bot added a commit that referenced this pull request Dec 5, 2021
* Bump blockhash/fee api check versions

* Bump snapshot api check version

(cherry picked from commit 3e5a5a8)

Co-authored-by: Tyera Eulberg <teulberg@gmail.com>
jbiseda pushed a commit to jbiseda/solana that referenced this pull request Dec 6, 2021
* Bump blockhash/fee api check versions

* Bump snapshot api check version
@brooksprumo brooksprumo mentioned this pull request Jan 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to airdrop false sols

3 participants