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

task-8 #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

task-8 #1

wants to merge 1 commit into from

Conversation

ivanp-epam
Copy link
Owner

@ivanp-epam ivanp-epam commented Jun 17, 2021

HTTPS version - https://6ux44zy8w5.execute-api.eu-west-1.amazonaws.com/dev
HTTP version - http://ivanp-epam-cart-api-demo.eu-west-1.elasticbeanstalk.com/
FE PR - ivanp-epam/nodejs-aws-fe#6
SITE: https://dvcrugb1m68oa.cloudfront.net/

Evaluation:

  • 1 - Dockerfile is prepared, image is building. Image size is minimised to be less than 500 MB.
  • 2 - Dockerfile is optimized. Files that change more often and commands that depend on them should be included later, files and commands that change less should be at the top.
  • 3 - Folders are added to .dockerignore, with explanations. At least 2 big directories should be excluded from build context. Elastic Beanstalk application is initialized.
  • 4 - Environment is created and the app is deployed to the AWS cloud. You must provide a link to your GitHub repo with Cart API service or PR with created Dockerfile and related configurations.
  • 5 - FE application is updated with Cart API endpoint. You must provide a PR with updates in your FE repository and OPTIONALLY link to deployed front-end app which makes proper API calls to your Cart service.
  • OPTIONAL: Minimize docker image size to about 100 MB.

Docker size:

Docker image shrank to 129Mb. I think it's its limit for the official node-14 container image since it's the smallest (alpine) size already 115Mb.

Screenshot 2021-06-17 at 16 42 51
Screenshot 2021-06-17 at 16 41 48

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.

1 participant