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

Lk86/support ubuntu20.04 in Rosetta staged dockerfile #9911

Merged
merged 6 commits into from
Dec 15, 2021

Conversation

lk86
Copy link
Contributor

@lk86 lk86 commented Dec 13, 2021

This also includes prior work to refactor the toolchain stage out of the normal rosetta pipeline, which avoids porting the python dependencies for the integration tests and various other tools for ubuntu 20. Debian-only for those should suffice as our tests and tools in debian can still be used to deploy + test ubuntu images.

The version of rocksdb we depend on (via ocaml-rocksdb) does not build in the latest g++ without disabling some deprecation warnings, so we had to silence those, and the final ubuntu 20 dependencies are different but largely the build process still works well in ubuntu 20 and the node seems to perform comparably in rosetta testing.

We should consider upgrading to an ocaml-rocksdb version that supports rocksdb 6 but that's a problem for another ticket.

@lk86 lk86 requested a review from stevenplatt December 13, 2021 22:30
@lk86 lk86 self-assigned this Dec 13, 2021
@lk86 lk86 requested review from a team as code owners December 13, 2021 22:30
@lk86 lk86 changed the title Lk86/support ubuntu20 Lk86/support ubuntu20.04 in Rosetta staged dockerfile Dec 13, 2021
@lk86 lk86 added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Dec 13, 2021
Copy link
Member

@bkase bkase left a comment

Choose a reason for hiding this comment

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

Looks good to me. Does the container build and result in something that runs with Rosetta?

@lk86 lk86 merged commit 4c76a13 into rosetta-1.2.1-safe Dec 15, 2021
@lk86 lk86 deleted the lk86/support-ubuntu20 branch December 15, 2021 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants