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

DNR: v1.16 rebase #320

Closed
wants to merge 17 commits into from
Closed

DNR: v1.16 rebase #320

wants to merge 17 commits into from

Conversation

buffalu
Copy link
Contributor

@buffalu buffalu commented Jul 29, 2023

No description provided.

@buffalu buffalu force-pushed the v1.16 branch 2 times, most recently from db1672f to d2978ac Compare August 1, 2023 16:44
mergify bot and others added 15 commits August 1, 2023 12:53
… of #31975) (#32675)

Add support for optional metric values via Option<_> (#31975)

(cherry picked from commit 6371240)

Co-authored-by: Jeff Biseda <[email protected]>
…ns to metrics (backport of #32504) (#32674)

Report new received crds signatures and their respective origins to metrics (#32504)

* screwed up old branch and syncing with upstream branch

* add fixed size ring buff instead of variable sized vecdeque for reporting signatures

* modify difficulty to take in n 0 bits and check if signature ending ends in n 0 bits

* update to only push every 18 trailing zero bits. and clean up

* report origin with signature. and set trailing 0s to 8 for local testing

* change back to 18 trailing zeros and rm unused imports

* run cargo rmt

* run ./scripts/cargo-for-all-lock-files.sh tree

* allow integer arithmetic for bit comparison

* rm unused lifetime

* rm duplicate entry?

* re implement ring buf

* put ringbuf in sorted order

* ringbuf in cargo.toml now in sorted order

* rm ring buf, refactor, fix trailing zero bug

* fix bug in trailing zeros. was comparing wrong ones

* fix needless range loop bug

* fix bug

* change trailing zero checking to build in methods and only report first 8 bytes of signature and origin pubkey

* report full origin string and first 8 bytes of signature

* set SIGNATURE_SAMPLE_TRAILING_ZEROS back to 18

* forgot to run cargo tree

* avoid panic and change working

* rm bs58

* pass in Option<String> into datapoint_info

* shorten metric names

(cherry picked from commit 80f7082)

Co-authored-by: Greg Cusack <[email protected]>
…ounts (backport of #32514) (#32669)

* removes feature-gate code for updating rewards from cached accounts (#32514)

(cherry picked from commit 952d886)

# Conflicts:
#	runtime/src/bank.rs

* resolves mergify merge conflicts

---------

Co-authored-by: behzad nouri <[email protected]>
…ader" error to `remove_bpf_loader_incorrect_program_id` (backport of #32695) (#32700)

Fix - Adds missing "Executable account not owned by the BPF loader" error to `remove_bpf_loader_incorrect_program_id` (#32695)

Adds missing "Executable account not owned by the BPF loader" error.

(cherry picked from commit b5a80a7)

Co-authored-by: Alexander Meißner <[email protected]>
…RBPF` list (backport of #32703) (#32705)

Fix - Adds missing feature pubkeys to the `FEATURES_AFFECTING_RBPF` list (#32703)

Adds missing feature pubkeys to the FEATURES_AFFECTING_RBPF list.

(cherry picked from commit 33c33f5)

Co-authored-by: Alexander Meißner <[email protected]>
…of #32511) (#32697)

* sdk: Limited Borsh 0.9 support (Pubkey and helpers) (#32511)

* sdk: Implement Borsh 0.9 traits on Pubkey

* Alphabetize cargo.toml

* Add backwards-compatible borsh file

* Add borsh0_10.rs for more clarity

* Deprecate `borsh` utils, use borsh0_10 everywhere

* Mark borsh 0.9 helpers as deprecated

* Add macros for deriving helper impls

* Add borsh 0.9 tests

* Refactor tests into macro

(cherry picked from commit 8c14886)

# Conflicts:
#	sdk/src/lib.rs

* Fix merge conflict on re-exports

---------

Co-authored-by: Jon Cinque <[email protected]>
…f #32701) (#32713)

demotes unknown_epoch_staked_nodes metric to debug (#32701)

(cherry picked from commit b0a4669)

Co-authored-by: behzad nouri <[email protected]>
…#32678) (#32720)

rolls out merkle shreds to ~50% of mainnet slots (#32678)

(cherry picked from commit 2078208)

Co-authored-by: behzad nouri <[email protected]>
… (#32756)

improve prioritization fee cache accuracy (#32692)

* improve prioritization cache accuracy

(cherry picked from commit ef6af30)

Co-authored-by: Tao Zhu <[email protected]>
…port of #32708) (#32772)

report received message signatures only on PUSH requests (#32708)

* we only want to report received message signatures on PUSH requests, not PULL requests

* woops accidently had it has LocalMessage not PushMessage

* switch from match to if let statement

* convert if let to matches macro

(cherry picked from commit 8495257)

Co-authored-by: Greg Cusack <[email protected]>
…kport of #32723) (#32749)

* stake: deprecate on chain warmup/cooldown rate and config (#32723)

* stake: deprecate on chain warmup/cooldown rate and config

* Pr feedback: Deprecate since 1.16.7

Co-authored-by: Jon Cinque <[email protected]>

---------

Co-authored-by: Jon Cinque <[email protected]>
(cherry picked from commit fa35066)

# Conflicts:
#	programs/stake/src/stake_instruction.rs
#	programs/stake/src/stake_state.rs
#	runtime/src/bank.rs
#	runtime/src/stakes.rs
#	sdk/src/feature_set.rs

* fix conflicts

* Fix bad merge

---------

Co-authored-by: Ashwin Sekar <[email protected]>
…efault (backport of #32768) (#32777)

stake: Set the Delegation's warmup / cooldown to the correct default (#32768)

(cherry picked from commit 31b6b64)

Co-authored-by: Jon Cinque <[email protected]>
Add from field for message tracking (#32725)

* we only want to report received message signatures on PUSH requests, not PULL requests

* woops accidently had it has LocalMessage not PushMessage

* switch from match to if let statement

* convert if let to matches macro

* add in from field in PushMessage for message tracking

* update with cargo fmt

* remove display for gossip route and add lifetime param to pubkey reference in gossiproute enum

* forgot to run fmt

Co-authored-by: Greg Cusack <[email protected]>
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.

2 participants