Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Change docker base image to Ubuntu-20.04 #4516

Merged
merged 5 commits into from
Oct 26, 2020

Conversation

cdonke
Copy link
Contributor

@cdonke cdonke commented Oct 25, 2020

Description

Changed the base image to ubuntu, instead of node with Debian.

Instead of running a Node+Debian image and installing the .NET Core, I used a .NET Core image and installed the NodeJS package.

The base image size is 400Mb smaller.
image

Rebasing from MAIN branch, since branching from ORCHESTRATOR branch was returning code errors.

Task Item

#minor

Tests

Validated the build and push using Az DevOps pipeline:
image

@coveralls
Copy link

coveralls commented Oct 25, 2020

Coverage Status

Coverage remained the same at 55.065% when pulling 374f792 on cdonke:feat/ubuntu-based into 83c0804 on microsoft:main.

@boydc2014 boydc2014 merged commit 54d6f7a into microsoft:main Oct 26, 2020
@cdonke cdonke deleted the feat/ubuntu-based branch October 26, 2020 13:43
@cwhitten cwhitten mentioned this pull request Nov 13, 2020
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* Ignoring .git folder for docker build

* changing base image to ubuntu

* adding verbose to health check on github actions

* fix varaible name

Co-authored-by: Dong Lei <donglei@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants