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

Add runtime environment to FailedVerification tombstones #31991

Merged
merged 4 commits into from
Jun 6, 2023

Conversation

pgarg66
Copy link
Contributor

@pgarg66 pgarg66 commented Jun 6, 2023

Problem

The programs that failed verification could succeed after a certain runtime feature has been activated. Currently, cache does not store the runtime environment that was used when the program failed to verify/compile. It's needed to check if the re-verification should be triggered for the failed program.

Summary of Changes

Retain environment for FailedVerification tombstones.

Fixes #

@Lichtso
Copy link
Contributor

Lichtso commented Jun 6, 2023

In LoadedPrograms::prune_feature_set_transition() FailedVerification should also be added next to the Unloaded case.

@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Merging #31991 (e6628bc) into master (8a47086) will increase coverage by 0.0%.
The diff coverage is 90.9%.

❗ Current head e6628bc differs from pull request most recent head e5ba8a6. Consider uploading reports for the commit e5ba8a6 to get more accurate results

@@           Coverage Diff           @@
##           master   #31991   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         762      762           
  Lines      207641   207660   +19     
=======================================
+ Hits       170074   170100   +26     
+ Misses      37567    37560    -7     

@pgarg66 pgarg66 force-pushed the failed-verify-env branch from 58675b8 to e6628bc Compare June 6, 2023 19:09
@pgarg66 pgarg66 marked this pull request as ready for review June 6, 2023 19:10
@pgarg66 pgarg66 requested a review from Lichtso June 6, 2023 19:10
@pgarg66 pgarg66 added v1.16 PRs that should be backported to v1.16 automerge Merge this Pull Request automatically once CI passes labels Jun 6, 2023
@mergify mergify bot merged commit c86e160 into solana-labs:master Jun 6, 2023
mergify bot pushed a commit that referenced this pull request Jun 6, 2023
* Add runtime environment to FailedVerification tombstones

* modify default variant

* prune update

* add DelayVisibility in prune_feature_set_transition

(cherry picked from commit c86e160)
@pgarg66 pgarg66 deleted the failed-verify-env branch June 6, 2023 22:26
mergify bot added a commit that referenced this pull request Jun 6, 2023
…port of #31991) (#32001)

Add runtime environment to FailedVerification tombstones (#31991)

* Add runtime environment to FailedVerification tombstones

* modify default variant

* prune update

* add DelayVisibility in prune_feature_set_transition

(cherry picked from commit c86e160)

Co-authored-by: Pankaj Garg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes v1.16 PRs that should be backported to v1.16
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants