Paths

Table of Contentst

Differential D4734

[Services] Allow for docker paths and system packages
ClosedPublic
Actions

Authored by • jon on Aug 4 2022, 6:22 AM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

atul
• abosh
• max
• karol

Commits

rCOMM1ce887c5239d: [Services] Allow for docker paths and system packages

Summary

This allows for later diffs to not fail CI

This handles the existing and later workflows.

This work will be reverted later in https://phab.comm.dev/D4735

Depends on D4685

Test Plan

Blob, backup, and tunnelbroker CI gates pass

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jon created this revision.Aug 4 2022, 6:22 AM

Herald added subscribers: • adrian, tomek, ashoat. · View Herald TranscriptAug 4 2022, 6:22 AM

• jon added a child revision: D4671: [Services] Update tunnelbroker Dockerfile & cmake.Aug 4 2022, 6:23 AM

• jon mentioned this in D4735: [Services] Remove build_services compatibility logic.Aug 4 2022, 6:29 AM

Harbormaster completed remote builds in B11102: Diff 15293.Aug 4 2022, 6:38 AM

• jon requested review of this revision.Aug 4 2022, 6:38 AM

This is undone in https://phab.comm.dev/D4735

• jon edited the summary of this revision. (Show Details)Aug 4 2022, 6:57 AM

• abosh requested changes to this revision.Aug 4 2022, 5:29 PM

• abosh added inline comments.

services/lib/docker/build_service.sh
11 ↗	(On Diff #15293)	Should these be escaped to avoid confusion with the regex? I know the `-r` flag isn't set but `sed` can be weird. The same question applies to line 27.

This revision now requires changes to proceed.Aug 4 2022, 5:29 PM

ashoat added inline comments.Aug 5 2022, 4:04 PM

services/lib/docker/build_service.sh
8 ↗	(On Diff #15293)	It's weird that we call this entire `if` block a Folly hack where it seems that much of it isn't really related to the Folly hack

ashoat added inline comments.Aug 5 2022, 4:06 PM

services/lib/docker/build_service.sh
8 ↗	(On Diff #15293)	(Ignore this, it's undone in D4735 anyways)

Rebase

Harbormaster completed remote builds in B11165: Diff 15369.Aug 5 2022, 6:47 PM

• jon marked 2 inline comments as done.Aug 6 2022, 12:06 AM

• jon added inline comments.

services/lib/docker/build_service.sh
11 ↗	(On Diff #15293)	I would say no, this is pretty canonical GNU sed usage

• abosh accepted this revision.Aug 8 2022, 2:50 PM

This revision is now accepted and ready to land.Aug 8 2022, 2:50 PM

Closed by commit rCOMM1ce887c5239d: [Services] Allow for docker paths and system packages. · Explain WhyAug 8 2022, 6:34 PM

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMM1ce887c5239d: [Services] Allow for docker paths and system packages.

• jon mentioned this in rCOMM8d3670c56c1a: [Services] Remove build_services compatibility logic.Aug 8 2022, 8:45 PM

Revision Contents
Changeset List

Path

Size

services/

lib/

docker/

build_service.sh

35 lines

Diff 15435

services/lib/docker/build_service.sh

Loading...