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

Downgrade Level DB to Stable Version #13671

Merged
merged 11 commits into from
Feb 29, 2024
Merged

Downgrade Level DB to Stable Version #13671

merged 11 commits into from
Feb 29, 2024

Conversation

nisdas
Copy link
Member

@nisdas nisdas commented Feb 27, 2024

What type of PR is this?

Bug Fix

What does this PR do? Why is it needed?

This downgrades our leveldb version to a stable version so that consumers of prysm do not use an unstable version of leveldb. This PR also updates our geth dependency to the deneb mainnet release.

Which issues(s) does this PR fix?

N.A

Other notes for review

@nisdas nisdas added the Ready For Review A pull request ready for code review label Feb 27, 2024
@nisdas nisdas requested a review from a team as a code owner February 27, 2024 08:58
@nisdas nisdas added the dependencies Pull requests that update a dependency file label Feb 27, 2024
@nisdas nisdas added this pull request to the merge queue Feb 29, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 29, 2024
@nisdas nisdas added this pull request to the merge queue Feb 29, 2024
Merged via the queue into develop with commit b61d177 Feb 29, 2024
17 checks passed
@nisdas nisdas deleted the downgradeLevelDB branch February 29, 2024 05:10
nalepae pushed a commit that referenced this pull request Feb 29, 2024
* downgrade level db

* fix current issues

* update geth

* Fix zstd build. The patch is now longer needed now that bazelbuild/rules_go#3411 is fixed.

* Revert "update geth"

This reverts commit 2a7c51a.

* change to hex

---------

Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com>
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants