Impl/include params days to go back #96
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
O que esse PR faz?
Este PR foca em aprimorar a flexibilidade e clareza das tarefas relacionadas ao processamento de logs e relatórios. Ele introduz o parâmetro
days_to_go_backem três tarefas assíncronas principais:task_parse_logs: responsável por processar os arquivos de log.task_check_missing_logs_for_date_range: verifica logs ausentes em um determinado período.task_log_files_count_status_report: gera relatórios de contagem de arquivos de log.Com a inclusão desse parâmetro, agora é possível definir o período de tempo para essas tarefas especificando um número de dias a partir da data atual, simplificando a configuração. Além disso, a descrição do método que obtém os intervalos de datas (
get_date_range_str) foi melhorada para maior clareza.Por fim, a tarefa
task_compute_metricsfoi renomeada paratask_index_documentspara refletir melhor sua função de indexação de métricas, e a versão da aplicação foi atualizada para 1.10.0.Por onde a revisão poderia começar?
A revisão pode começar pelo arquivo
core/utils/date_utils.pypara verificar a descrição aprimorada do métodoget_date_range_str. Em seguida, os arquivoslog_manager/tasks.pyemetrics/tasks.pydevem ser analisados para observar a inclusão e o uso do novo parâmetrodays_to_go_backnas respectivas tarefas, bem como a mudança de nome da tarefatask_compute_metricsparatask_index_documentsemmetrics/tasks.py. O arquivoVERSIONtambém pode ser verificado para confirmar a atualização da versão.Como este poderia ser testado manualmente?
Para testar manualmente as alterações, siga os passos abaixo:
Acesse o shell do Django no seu ambiente de desenvolvimento:
docker compose -f local.yml exec django bash python manage.py shellExecute as tarefas assíncronas com o novo parâmetro
days_to_go_back:Verifique os logs dos containers Celery:
Monitore os logs do
celery_worker(docker compose logs celery_worker) para confirmar que as tarefas foram enfileiradas e executadas com sucesso, utilizando o parâmetrodays_to_go_backconforme esperado.Algum cenário de contexto que queira dar?
Anteriormente, ao executar tarefas relacionadas a logs e métricas, era necessário sempre fornecer as datas de início (
from_date) e fim (until_date) de forma explícita. Isso tornava a execução menos flexível para cenários onde se desejava simplesmente processar dados dos últimos N dias. A inclusão do parâmetrodays_to_go_backoferece uma maneira mais intuitiva e rápida de definir esses períodos, especialmente para automação ou execuções rotineiras que sempre olham para um histórico recente.A renomeação da tarefa
task_compute_metricsparatask_index_documentsvisa alinhar o nome da função com sua operação real, que é a indexação de documentos no Elasticsearch, e não apenas o cálculo de métricas.Screenshots
Não aplicável, pois as alterações são puramente de backend e não afetam a interface gráfica.
Quais são tickets relevantes?
Nenhum ticket específico está associado a este PR.
Referências
N/A