Adiciona comandos no Makefile e configura limites de memória em serviços Docker#836
Adiciona comandos no Makefile e configura limites de memória em serviços Docker#836robertatakenaka merged 6 commits intomainfrom
Conversation
Add the following commands to Makefile: - up_scale: up Docker compose services and scale up celeryworker - top: top command to compose services - stats: stats command to compose services
- Remove Django fixed version - Add Celery Worker setting to limit the use of memory
There was a problem hiding this comment.
Pull request overview
Este PR melhora a operação em produção adicionando atalhos no Makefile para facilitar monitoramento/escala via Docker Compose e ajustando o production.yml para permitir scale do Celery Worker e tentar impor limite de memória.
Changes:
- Adiciona targets
up_scale,topestatsnoMakefile. - Remove
container_namedoceleryworkernoproduction.ymlpara permitir escala. - Troca a tag default da imagem por
SCMS_WEBAPP_VERSIONe adicionadeploy.resources.limits.memoryno Celery Worker.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
production.yml |
Remove tag default e container_name do celeryworker; adiciona limite de memória via deploy.resources. |
Makefile |
Adiciona targets para scale/monitoramento (up_scale, top, stats). |
| up_scale: ## Start app using $(compose) and scaling worker up to $(numworkers) | ||
| $(DOCKER_COMPOSE) -f $(compose) up -d --scale celeryworker=$(numworkers) |
There was a problem hiding this comment.
up_scale relies on $(numworkers) but there is no default/validation; if it’s omitted the command becomes --scale celeryworker= and fails. Add a default (e.g., numworkers ?= 1) or guard the target with an explicit error when numworkers is empty/non-numeric.
|
@patymori verificar os comentários |
|
@robertatakenaka Feitos ajustes aos comentários |
- Add param `--compatibility` to legacy docker-compose execution - Change rule clean_celery_logs to use docker compose
|
@robertatakenaka pode revisar as últimas alterações? |
|
@robertatakenaka está OK assim para seguir com o merge deste PR. |
O que esse PR faz?
Melhorias para a operação da aplicação em produção:
up_scale: executa o comandodocker compose upe escala Celery Worker para o número de conteineres informadotop: executa o comandodocker compose topstats: executa o comandodocker compose statsproduction.yml:Onde a revisão poderia começar?
Makefile: linha 50 - comando up_scaleproduction.yml: linha 51 - retirado nome do conteiner e inclusão de limites de uso de memória do servidorComo este poderia ser testado manualmente?
make top compose=production.ymle verifique se é exibido o resultado do comandodocker compose -f production.yml topmake stats compose=production.ymle verifique se é exibido o resultado do comandodocker compose -f production.yml statsmake up_scale compose=production.yml numworkers=2e verifique se o número de conteineres do Celery Workers aumentou para 2Algum cenário de contexto que queira dar?
Os novos comandos do Makefile foram inseridos para ajudar no monitoramento dos conteineres e para executar mais de um celery worker e, quando possível, processar mais de uma tarefa de forma assíncrona.
A versão do
production.ymlfoi retirada porque em algumas coleções não foi usado o comando do Makefile. Isso resultou na execução de uma versão antiga da aplicação. Para evitar, será necessário corrigir manualmente o compose, forçando para que se use a versão adequada.Screenshots
n/a
Quais são tickets relevantes?
n/a
Referências
.