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
Unknown Object (File)
Tue, Apr 1, 8:20 AM
Unknown Object (File)
Wed, Mar 19, 7:30 PM
Unknown Object (File)
Wed, Mar 19, 7:30 PM
Unknown Object (File)
Wed, Mar 19, 7:30 PM
Unknown Object (File)
Sun, Mar 16, 7:16 AM
Unknown Object (File)
Sun, Mar 16, 7:16 AM
Unknown Object (File)
Sat, Mar 15, 9:12 PM
Unknown Object (File)
Thu, Mar 13, 5:56 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 15930

View Options

services/backup/Dockerfile

Show All 21 Lines
a { a {
text-decoration: none; text-decoration: none;
color: #2a5db0; color: #2a5db0;
} }
img, img,
iframe { iframe {
display: block; display: block;
} }
input[type='text'], input[type='text'],
input[type='password'], input[type='password'],
textarea { textarea {
atulUnsubmitted
Not Done
Inline Actions

Moving the global input styling and moving it to the input component.

Then why are we keeping styles like border and border-radius?

Don't these styles also affect the <textarea> tag? Are you sure we want to be removing these styles for that as well? Or should we break this into two separate selectors (one for input and another for textarea)?

atul: > Moving the global input styling and moving it to the input component. Then why are we…
benschacAuthorUnsubmitted
Done
Inline Actions

Yeah, I should break these styles out a bit more. I don't think they really do anything. But you make a good point that this will also effect textarea elements.

benschac: Yeah, I should break these styles out a bit more. I don't think they really do anything. But…
-webkit-appearance: none;
-moz-appearance: none;
-webkit-border-radius: 0;
border: 1px solid #dddddd; border: 1px solid #dddddd;
border-radius: 1px; border-radius: 1px;
font-family: var(--font-stack); font-family: var(--font-stack);
} }
:global(#react-root) { :global(#react-root) {
display: flex; display: flex;
flex-direction: column; flex-direction: column;
height: 100%; height: 100%;
▲ Show 20 Lines • Show All 243 Lines • Show Last 20 Lines
View Options

services/blob/Dockerfile

Show All 21 Lines
a { a {
text-decoration: none; text-decoration: none;
color: #2a5db0; color: #2a5db0;
} }
img, img,
iframe { iframe {
display: block; display: block;
} }
input[type='text'], input[type='text'],
input[type='password'], input[type='password'],
textarea { textarea {
atulUnsubmitted
Not Done
Inline Actions

Moving the global input styling and moving it to the input component.

Then why are we keeping styles like border and border-radius?

Don't these styles also affect the <textarea> tag? Are you sure we want to be removing these styles for that as well? Or should we break this into two separate selectors (one for input and another for textarea)?

atul: > Moving the global input styling and moving it to the input component. Then why are we…
benschacAuthorUnsubmitted
Done
Inline Actions

Yeah, I should break these styles out a bit more. I don't think they really do anything. But you make a good point that this will also effect textarea elements.

benschac: Yeah, I should break these styles out a bit more. I don't think they really do anything. But…
-webkit-appearance: none;
-moz-appearance: none;
-webkit-border-radius: 0;
border: 1px solid #dddddd; border: 1px solid #dddddd;
border-radius: 1px; border-radius: 1px;
font-family: var(--font-stack); font-family: var(--font-stack);
} }
:global(#react-root) { :global(#react-root) {
display: flex; display: flex;
flex-direction: column; flex-direction: column;
height: 100%; height: 100%;
▲ Show 20 Lines • Show All 243 Lines • Show Last 20 Lines
View Options

services/scripts/build_base_image.sh

Show All 21 Lines
a { a {
text-decoration: none; text-decoration: none;
color: #2a5db0; color: #2a5db0;
} }
img, img,
iframe { iframe {
display: block; display: block;
} }
input[type='text'], input[type='text'],
input[type='password'], input[type='password'],
textarea { textarea {
atulUnsubmitted
Not Done
Inline Actions

Moving the global input styling and moving it to the input component.

Then why are we keeping styles like border and border-radius?

Don't these styles also affect the <textarea> tag? Are you sure we want to be removing these styles for that as well? Or should we break this into two separate selectors (one for input and another for textarea)?

atul: > Moving the global input styling and moving it to the input component. Then why are we…
benschacAuthorUnsubmitted
Done
Inline Actions

Yeah, I should break these styles out a bit more. I don't think they really do anything. But you make a good point that this will also effect textarea elements.

benschac: Yeah, I should break these styles out a bit more. I don't think they really do anything. But…
-webkit-appearance: none;
-moz-appearance: none;
-webkit-border-radius: 0;
border: 1px solid #dddddd; border: 1px solid #dddddd;
border-radius: 1px; border-radius: 1px;
font-family: var(--font-stack); font-family: var(--font-stack);
} }
:global(#react-root) { :global(#react-root) {
display: flex; display: flex;
flex-direction: column; flex-direction: column;
height: 100%; height: 100%;
▲ Show 20 Lines • Show All 243 Lines • Show Last 20 Lines
View Options

services/tunnelbroker/Dockerfile

Show All 21 Lines
a { a {
text-decoration: none; text-decoration: none;
color: #2a5db0; color: #2a5db0;
} }
img, img,
iframe { iframe {
display: block; display: block;
} }
input[type='text'], input[type='text'],
input[type='password'], input[type='password'],
textarea { textarea {
atulUnsubmitted
Not Done
Inline Actions

Moving the global input styling and moving it to the input component.

Then why are we keeping styles like border and border-radius?

Don't these styles also affect the <textarea> tag? Are you sure we want to be removing these styles for that as well? Or should we break this into two separate selectors (one for input and another for textarea)?

atul: > Moving the global input styling and moving it to the input component. Then why are we…
benschacAuthorUnsubmitted
Done
Inline Actions

Yeah, I should break these styles out a bit more. I don't think they really do anything. But you make a good point that this will also effect textarea elements.

benschac: Yeah, I should break these styles out a bit more. I don't think they really do anything. But…
-webkit-appearance: none;
-moz-appearance: none;
-webkit-border-radius: 0;
border: 1px solid #dddddd; border: 1px solid #dddddd;
border-radius: 1px; border-radius: 1px;
font-family: var(--font-stack); font-family: var(--font-stack);
} }
:global(#react-root) { :global(#react-root) {
display: flex; display: flex;
flex-direction: column; flex-direction: column;
height: 100%; height: 100%;
▲ Show 20 Lines • Show All 243 Lines • Show Last 20 Lines