StackStorm/st2

Remove deprecated "timer" section in st2.conf

Aberta

#4.425 aberto em 5 de nov. de 2018

Ver no GitHubGuia do colaborador
 (0 comentário) (1 reação) (0 responsável)Python (740 forks)batch import
complexity:easyenhancementgood first issuerefactor

Métricas do repositório

Stars
 (5.794 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

In v2.9.0 we deprecated timer section in st2.conf in favor of new timersengine section.

We need to remove old and deprecated timer section in st2.conf.

The current approach is also confusing - timer section has been deprecated and timerengine is the preferred approach yet the how code is structured, "timer" section actually has precedence because of the way how our code is structured.

TODO

  • Remove from config.py
  • Update affected code
  • Update upgrade notes

Guia do colaborador

Direção de pesquisa
O problema requer a remoção da seção obsoleta `timer` de `st2.conf` e a atualização de `config.py` para não mais referenciá la. A lista TODO é clara: remover de config.py, atualizar o código afetado e atualizar as notas de atualização. Não há comentários ou PRs vinculados, portanto o contribuidor deve examinar a base de código em busca de todas as referências à seção `timer` e garantir uma remoção limpa. As notas de atualização devem refletir a remoção. Esta é uma alteração localizada sem dependências.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitPython
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60