Skip to content
This repository has been archived by the owner on Apr 4, 2023. It is now read-only.

Use node:10.15-stretch-slim as a base image for dotnet's plugins #188

Merged
merged 2 commits into from
Apr 19, 2019

Conversation

svor
Copy link
Contributor

@svor svor commented Apr 19, 2019

Signed-off-by: Valeriy Svydenko vsvydenk@redhat.com

What does this PR do?

Use node:10.15-stretch-slim as a base image for remote .Net plugin

What issues does this PR fix or reference?

eclipse-che/che#12793

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
@svor svor requested review from benoitf and evidolob April 19, 2019 11:40
@svor svor requested a review from riuvshin as a code owner April 19, 2019 11:40
@svor svor self-assigned this Apr 19, 2019
FROM ${BUILD_ORGANIZATION}/${BUILD_PREFIX}-theia-endpoint-runtime:${BUILD_TAG} as endpoint
FROM node:10.15-stretch-slim

RUN apt-get update && apt-get install -y --no-install-recommends \
Copy link
Contributor

Choose a reason for hiding this comment

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

shoulw we merge all RUN instructions to reduce the numbers of layers ?

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

approving again :-)

@svor
Copy link
Contributor Author

svor commented Apr 19, 2019

@benoitf Thanks!

@svor svor merged commit a44b0ca into master Apr 19, 2019
@svor svor deleted the sv/dotnetStretch branch April 19, 2019 13:27
vinokurig pushed a commit that referenced this pull request Apr 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants