Page MenuHomePhabricator
Differential D4858

[services] Fix docker base image
ClosedPublic
Actions

Authored by • karol on Aug 17 2022, 3:53 AM.
Tags
None
Referenced Files
F3266802: D4858.id15701.diff
Sat, Nov 16, 2:18 AM
Unknown Object (File)
Sun, Nov 10, 6:58 AM
Unknown Object (File)
Sun, Oct 27, 8:10 PM
Unknown Object (File)
Sun, Oct 27, 8:10 PM
Unknown Object (File)
Sun, Oct 27, 8:10 PM
Unknown Object (File)
Sun, Oct 27, 8:09 PM
Unknown Object (File)
Oct 8 2024, 1:12 AM
Unknown Object (File)
Oct 8 2024, 1:12 AM
View All Files
Subscribers
• abosh
• adrian
ashoat
atul
tomek

Details

Reviewers
• jon
tomek
ashoat
Commits
rCOMMb1525900e127: [services] Fix docker base image
Summary

The current base image has probably not been built using this script which uses buildx and that caused it not to be compatible with the M1s.

This diff fixes that, I rebuilt the base image using the script and pushed it to the hub.

Test Plan

Clean docker, build one of the services (e.g. blob) on m1, and see that there's no amd64 label.

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D4858_2
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• karol created this revision.Aug 17 2022, 3:53 AM
Herald added subscribers: • abosh, atul, • adrian and 2 others. · View Herald TranscriptAug 17 2022, 3:53 AM
• karol edited the summary of this revision. (Show Details)Aug 17 2022, 3:58 AM
• karol edited the test plan for this revision. (Show Details)
• karol added reviewers: • jon, tomek, ashoat.
• karol added a child revision: D4859: [services] Add curl to the base image.
Harbormaster completed remote builds in B11412: Diff 15701.Aug 17 2022, 4:02 AM
• karol requested review of this revision.Aug 17 2022, 4:02 AM
• max mentioned this in D4860: [services] Tunnelbroker - Move install corrosion script to lib.Aug 17 2022, 4:10 AM
• jon added a comment.Aug 17 2022, 6:52 AM

Do you mind updating the base image document? https://www.notion.so/commapp/Docker-base-image-publish-914aa735683444c9a98c5c9794f8d83a

Otherwise diff LGTM

• jon requested changes to this revision.Aug 17 2022, 6:52 AM
This revision now requires changes to proceed.Aug 17 2022, 6:52 AM
• karol requested review of this revision.Aug 18 2022, 4:11 AM
• karol added 1 blocking reviewer(s): tomek.

Ok, done

ashoat accepted this revision.Aug 18 2022, 6:46 AM
• jon accepted this revision.Aug 18 2022, 7:42 AM

Added comment about bumping of version tag, or else the push would fail

tomek accepted this revision.Aug 18 2022, 10:36 AM
This revision is now accepted and ready to land.Aug 18 2022, 10:36 AM
• karol updated this revision to Diff 15927.Aug 24 2022, 7:12 AM

fix shellcheck

Harbormaster completed remote builds in B11585: Diff 15927.Aug 24 2022, 7:23 AM
Closed by commit rCOMMb1525900e127: [services] Fix docker base image (authored by • karol). · Explain WhyAug 24 2022, 8:13 AM
This revision was automatically updated to reflect the committed changes.
• karol added a commit: rCOMMb1525900e127: [services] Fix docker base image.

Revision Contents
Changeset List

PathSize
services/
backup/
2 lines
blob/
2 lines
scripts/
6 lines
tunnelbroker/
2 lines

Diff 15927

View Options

services/backup/Dockerfile

Loading...
View Options

services/blob/Dockerfile

Loading...
View Options

services/scripts/build_base_image.sh

Loading...
View Options

services/tunnelbroker/Dockerfile

Loading...