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

Dockerize whole application #184

Merged
merged 27 commits into from
Oct 20, 2021
Merged

Dockerize whole application #184

merged 27 commits into from
Oct 20, 2021

Conversation

norkator
Copy link
Owner

@norkator norkator commented Oct 15, 2021

Goal

  • Dockerize Python apps.
  • Dockerize API and Frontend.
  • Try to create "network" config so that Frontend does no longer need manual ip configuration of server.
  • Creating new tags->releases will trigger pipeline to build up to date Docker images.

Issues

  • I have very bad internet connection, which is making Dockerfile development/testing annoying.

Outcome

  • Ended up mounting file system so no need to copy always contents of source inside container.
  • docker-compose allows mounting file system which then makes possible mounting source files.

Will merge this and take it into testing + update docs based on results.

@norkator norkator self-assigned this Oct 15, 2021
@norkator norkator added the enhancement New feature or request label Oct 15, 2021
Ignoring user specific one to avoid committing own secrets into version control
@norkator norkator requested a review from SKR301 October 18, 2021 17:17
SKR301
SKR301 previously approved these changes Oct 19, 2021
@SKR301 SKR301 marked this pull request as ready for review October 19, 2021 16:10
Add volume mount in a way that container can access output folder
@norkator norkator added the docker Docker related label Oct 20, 2021
@norkator norkator merged commit 1ee5537 into master Oct 20, 2021
@norkator norkator deleted the dockerize branch October 20, 2021 18:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docker Docker related enhancement New feature or request
Projects
None yet
2 participants