Skip to content

Adiciona comandos no Makefile e configura limites de memória em serviços Docker#836

Merged
robertatakenaka merged 6 commits intomainfrom
add-makefile-cmd
Mar 5, 2026
Merged

Adiciona comandos no Makefile e configura limites de memória em serviços Docker#836
robertatakenaka merged 6 commits intomainfrom
add-makefile-cmd

Conversation

@patymori
Copy link
Copy Markdown
Collaborator

O que esse PR faz?

Melhorias para a operação da aplicação em produção:

  • Adiciona comandos ao Makefile:
    • up_scale: executa o comando docker compose up e escala Celery Worker para o número de conteineres informado
    • top: executa o comando docker compose top
    • stats: executa o comando docker compose stats
  • Altera o production.yml:
    • Remove versão fixada dos serviços Django. Assim, é necessário usar o comando Makefile (recomendado) ou alterar o compose
    • Adiciona configurações ao Celery Worker para limitar a memória usada do servidor para cada conteiner e remove o nome para possibilitar escalar o serviço para mais de um conteiner

Onde a revisão poderia começar?

  • Makefile: linha 50 - comando up_scale
  • production.yml: linha 51 - retirado nome do conteiner e inclusão de limites de uso de memória do servidor

Como este poderia ser testado manualmente?

  • Execute o comando make top compose=production.yml e verifique se é exibido o resultado do comando docker compose -f production.yml top
  • Execute o comando make stats compose=production.yml e verifique se é exibido o resultado do comando docker compose -f production.yml stats
  • Verifique se MEM USAGE / LIMIT está seguindo a configuração para o Celery Worker
  • Execute o comando make up_scale compose=production.yml numworkers=2 e verifique se o número de conteineres do Celery Workers aumentou para 2

Algum 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.yml foi 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

.

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
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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, top e stats no Makefile.
  • Remove container_name do celeryworker no production.yml para permitir escala.
  • Troca a tag default da imagem por SCMS_WEBAPP_VERSION e adiciona deploy.resources.limits.memory no 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).

Comment thread production.yml
Comment thread production.yml
Comment thread production.yml
Comment thread Makefile Outdated
Comment on lines +50 to +51
up_scale: ## Start app using $(compose) and scaling worker up to $(numworkers)
$(DOCKER_COMPOSE) -f $(compose) up -d --scale celeryworker=$(numworkers)
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
@robertatakenaka
Copy link
Copy Markdown
Member

@patymori verificar os comentários

@patymori
Copy link
Copy Markdown
Collaborator Author

@robertatakenaka Feitos ajustes aos comentários

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread Makefile Outdated
Comment thread production.yml
- Add param `--compatibility` to legacy docker-compose execution
- Change rule clean_celery_logs to use docker compose
@patymori
Copy link
Copy Markdown
Collaborator Author

@robertatakenaka pode revisar as últimas alterações?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

Comment thread Makefile
Comment thread Makefile
Comment thread Makefile
@patymori
Copy link
Copy Markdown
Collaborator Author

@robertatakenaka está OK assim para seguir com o merge deste PR.

@robertatakenaka robertatakenaka merged commit df78470 into main Mar 5, 2026
6 of 7 checks passed
@patymori patymori deleted the add-makefile-cmd branch March 5, 2026 19:11
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