Skip to content

[WIP]: Docker#10

Open
luca-c-xcv wants to merge 8 commits intoTheG-r-itters:mainfrom
luca-c-xcv:docker
Open

[WIP]: Docker#10
luca-c-xcv wants to merge 8 commits intoTheG-r-itters:mainfrom
luca-c-xcv:docker

Conversation

@luca-c-xcv
Copy link
Member

Describe your changes

Creation of all docker files for building and deploying the containers needed to run the system

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • CHANGELOG.md has been updated should there be relevant changes in this PR.

add Logger as submodule of gitAssistant
@luca-c-xcv luca-c-xcv added the enhancement New feature or request label Apr 28, 2024
@luca-c-xcv luca-c-xcv requested a review from feed3r April 28, 2024 17:56
Created the contribution file to be used to explain the guidelines for
completing a proper 'pull request'.
Inside is a comment with a link to a github help page useful for
understanding how to build the file.
created filesystem structure with README files inside the directories to keep
the strucutre consistent
the python cache directories and the log files added to the new
gitignore file
added a docker folder where to put all the docker files (Dockerfile and
docker-compose) to create images for all the necessary services.
created a draft docker compose in which a set of networks was defined for
internal and external communications and some services under development
removed the filesystem structure because it was redesigned
fixed some issue in the docker compose file and removed the
docker/logger directory and files inside
Copy link
Contributor

@feed3r feed3r left a comment

Choose a reason for hiding this comment

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

This feature should be adapted to the presence of Communications submodule, which creates the Kafka infrastructure

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants