Skip to content

Dockerfile: also build old registry on armhf, and small optimization#40362

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:dockerfile_registry_optimized
Jan 10, 2020
Merged

Dockerfile: also build old registry on armhf, and small optimization#40362
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:dockerfile_registry_optimized

Conversation

@thaJeztah
Copy link
Member

extracted from #40094

Dockerfile: also build old registry on armhf

Building the old version of the registry is not supported
on arm64, but building on armhf is possible.

Before using a multi-arch Dockerfile, the registry was
built on armhf as well, but this got lost in the transition
in commit f1701a7.

This commit re-enables building the old registry on armhf.

Dockerfile: use tmpfs for building registry, go-swagger

Simplifies the build-stages, and doesn't require the temporary directory to be removed afterwards

Building the old version of the registry is not supported
on arm64, but building on armhf is possible.

Before using a multi-arch Dockerfile, the registry was
built on armhf as well, but this got lost in the transition
in commit f1701a7.

This commit re-enables building the old registry on armhf.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Jan 10, 2020
Copy link
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thaJeztah thaJeztah merged commit 4d63209 into moby:master Jan 10, 2020
@thaJeztah thaJeztah deleted the dockerfile_registry_optimized branch January 10, 2020 17:42
@thaJeztah thaJeztah added this to the 20.03.0 milestone Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants