Skip to content

Optimising docker image size.#2992

Closed
JaredTan95 wants to merge 2 commits intomasterfrom
docker-image
Closed

Optimising docker image size.#2992
JaredTan95 wants to merge 2 commits intomasterfrom
docker-image

Conversation

@JaredTan95
Copy link
Member

Please answer these questions before submitting pull request

  • Why submit this pull request?
  • Bug fix
  • New feature provided
  • Improve performance

In my opinion, I think the final image is too big.
If there are no special requirements for the docker image, we can optimize it in some way.

Before:

skywalking/ui                               latest      733MB
skywalking/oap                              latest     834MB

After:

skywalking/ui                               latest     83.4MB
skywalking/oap                              latest    184MB

@JaredTan95 JaredTan95 requested review from hanahmily and wu-sheng July 3, 2019 06:31
@JaredTan95 JaredTan95 added the docker Docker build scripts or images related label Jul 3, 2019
@JaredTan95 JaredTan95 added this to the 6.3.0 milestone Jul 3, 2019
@JaredTan95 JaredTan95 requested a review from kezhenxu94 July 3, 2019 06:34
@wu-sheng
Copy link
Member

wu-sheng commented Jul 3, 2019

What is the difference between two basic images?

@wu-sheng
Copy link
Member

wu-sheng commented Jul 3, 2019

@JaredTan95 I think you should consider adding an e2e stage(parallel) to check the image we built in dev stage.

@JaredTan95
Copy link
Member Author

JaredTan95 commented Jul 3, 2019

Closed, temporarily. Other things need to be done.

@JaredTan95 JaredTan95 closed this Jul 3, 2019
@hanahmily
Copy link
Contributor

hanahmily commented Jul 3, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Docker build scripts or images related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants