Move the deploy workflow's routing and sentinel steps into tested scripts

Aberta
  • Algo errado?
#1,003 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 2 dias

Um pull request relacionado já foi integrado.

  • #1004 de @anth-volk — integrado

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
1/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
github-actions, python
Domínio
ci-cd, testing

Direção de pesquisa

Start with .github/workflows/simulation-deploy.reusable.yml and the seven named steps, then inspect rules/baselines/ci-multiline-run.txt and the tests that read shell from the workflow YAML. Move each block into a tested script under .github/scripts/, update the tests to run those scripts, and remove the seven baseline entries. Done when all seven steps keep identical behavior and their tests pass.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Phase 1 step 1.7 of the rules work, follow-up to #763: the seven multi-line run: blocks that #743 and #744 added to .github/workflows/simulation-deploy.reusable.yml move into tested scripts under .github/scripts/, each called from a single-command step, with identical behaviour.

The seven steps:

  • update_routing: Plan the routing change, Record the unfinished routing publication, Publish the planned routing
  • sentinel_expected: Check the sentinel names this deployment's bundle
  • integration: Check the deployed stack reproduces the sentinel numbers
  • promote_entrypoint: Verify the stable service reproduces the sentinel numbers
  • finalize_routing: Finalize the routing publication

#763 left them in rules/baselines/ci-multiline-run.txt (rule sim.ci.multiline-run-is-script) because their tests read the steps' shell from the workflow YAML. Those tests are rewritten to run the scripts, and the seven baseline entries are removed. The six blocks in simulation-rollback.yml follow in a separate pull request.

Linguagem predominante
Python
Estrelas
1
Forks
1
Merge médio
2d 48min
PRs com merge (30d)
87

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de PolicyEngine/policyengine-sim-api

Todas as issues de PolicyEngine/policyengine-sim-api

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.