[keyserver] Remove Python2 requirement from Docker
ClosedPublic
Actions

Authored by atul on May 23 2022, 1:17 PM.

Details

Reviewers

tomek
ashoat
• jacek

Commits

rCOMM9bf733596f64: [keyserver] Remove Python2 requirement from Docker

Summary

After bumping the sqlite3 dependency, Python2 is no longer needed when building SQLite from source.

The dependency on Python2, as well as any steps to have python resolve to Python2 can now be removed.

Linear issue: https://linear.app/comm/issue/ENG-1157/remove-python2-requirement-from-docker

Depends on D4103

Test Plan

arc patch D4103
yarn cleaninstall
cd keyserver && docker-compose up --build
Ensure that the yarn cleaninstall step "within" the container succeeds.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.May 23 2022, 1:17 PM

Herald added subscribers: • abosh, • adrian. · View Herald TranscriptMay 23 2022, 1:17 PM

atul edited the summary of this revision. (Show Details)May 23 2022, 1:17 PM

Harbormaster completed remote builds in B9398: Diff 13018.May 23 2022, 1:22 PM

atul requested review of this revision.May 23 2022, 1:22 PM

ashoat accepted this revision.May 23 2022, 2:16 PM

This revision is now accepted and ready to land.May 23 2022, 2:16 PM

rebase before landing

Harbormaster completed remote builds in B9433: Diff 13066.May 24 2022, 6:17 AM

Closed by commit rCOMM9bf733596f64: [keyserver] Remove Python2 requirement from Docker (authored by atul). · Explain WhyMay 24 2022, 6:18 AM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM9bf733596f64: [keyserver] Remove Python2 requirement from Docker.

Revision Contents
Changeset List

Path

Size

keyserver/

Dockerfile

3 lines

Diff 13067

View Options

[keyserver] Remove Python2 requirement from DockerClosedPublicActions