Differential D3295

[docker] Run `rm -rf /var/lib/apt/lists/*` after `apt-get install`
ClosedPublic
Actions

Authored by atul on Feb 27 2022, 2:46 PM.

Tags

None

Referenced Files

	F5894972: D3295.diff
	Thu, Apr 17, 10:37 AM

	Unknown Object (File)
	Tue, Apr 8, 5:53 PM

	Unknown Object (File)
	Tue, Apr 8, 5:49 PM

	Unknown Object (File)
	Tue, Apr 8, 8:10 AM

	Unknown Object (File)
	Tue, Apr 8, 6:26 AM

	Unknown Object (File)
	Tue, Apr 8, 6:00 AM

	Unknown Object (File)
	Wed, Apr 2, 12:05 AM

	Unknown Object (File)
	Tue, Apr 1, 5:30 AM

Subscribers

Details

Reviewers

Commits

rCOMMcd1400550f0d: [docker] Run `rm -rf /var/lib/apt/lists/*` after `apt-get install`

Summary

Addresses feedback from @jimpo: https://phabricator.ashoat.com/D3212#88571

Test Plan

CI

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Feb 27 2022, 2:46 PM

Herald added subscribers: • karol, • adrian, tomek, ashoat. · View Herald TranscriptFeb 27 2022, 2:46 PM

Harbormaster completed remote builds in B7054: Diff 9943.Feb 27 2022, 3:28 PM

atul requested review of this revision.Feb 27 2022, 3:28 PM

• jim requested changes to this revision.Feb 28 2022, 6:55 AM

• jim added inline comments.

services/blob/docker-base/Dockerfile
5 ↗	(On Diff #9943)	This now needs an apt-get update before it

This revision now requires changes to proceed.Feb 28 2022, 6:55 AM

address feedback

atul marked an inline comment as done.Feb 28 2022, 7:34 AM

Harbormaster completed remote builds in B7062: Diff 9951.Feb 28 2022, 7:39 AM

• jim accepted this revision.Feb 28 2022, 9:48 AM

This revision is now accepted and ready to land.Feb 28 2022, 9:48 AM

Herald added a subscriber: • benschac. · View Herald TranscriptFeb 28 2022, 9:48 AM

rebase before landing

Harbormaster completed remote builds in B7068: Diff 9958.Feb 28 2022, 10:03 AM

Closed by commit rCOMMcd1400550f0d: [docker] Run `rm -rf /var/lib/apt/lists/*` after `apt-get install` (authored by atul). · Explain WhyFeb 28 2022, 10:06 AM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMcd1400550f0d: [docker] Run `rm -rf /var/lib/apt/lists/*` after `apt-get install`.

Revision Contents
Changeset List

Path

Size

services/

backup/

docker-base/

3 lines

base-image/

3 lines

blob/

docker-base/

2 lines

tunnelbroker/

docker-base/

3 lines

Diff 9959

services/backup/docker-base/Dockerfile

	FROM commapp/services-base:1.0			FROM commapp/services-base:1.0

	ENV SHELL=/bin/bash			ENV SHELL=/bin/bash

	COPY services/backup/docker-base/contents /transferred			COPY services/backup/docker-base/contents /transferred

	RUN apt-get update && apt-get install -y \			RUN apt-get update && apt-get install -y \
	libcurl4-openssl-dev \			libcurl4-openssl-dev \
	libssl-dev \			libssl-dev \
	uuid-dev \			uuid-dev \
	zlib1g-dev			zlib1g-dev \
				&& rm -rf /var/lib/apt/lists/*

	RUN cd /			RUN cd /
	RUN /transferred/install_aws_sdk.sh			RUN /transferred/install_aws_sdk.sh

	CMD /bin/bash			CMD /bin/bash

services/base-image/Dockerfile

	Show All 9 Lines		RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y \
	cmake \			cmake \
	git \			git \
	build-essential \			build-essential \
	autoconf \			autoconf \
	libtool \			libtool \
	pkg-config \			pkg-config \
	libboost-all-dev \			libboost-all-dev \
	libfmt-dev \			libfmt-dev \
	libgtest-dev			libgtest-dev \
				&& rm -rf /var/lib/apt/lists/*

	RUN /transferred/install_grpc.sh			RUN /transferred/install_grpc.sh

	CMD /bin/bash			CMD /bin/bash

services/blob/docker-base/Dockerfile

Loading...

services/tunnelbroker/docker-base/Dockerfile

Loading...