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

Ouverte
  • Quelque chose ne va pas ?
#1,003 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

Une pull request liée a déjà été fusionnée.

  • #1004 par @anth-volk — fusionnée

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
1/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
github-actions, python
Domaine
ci-cd, testing

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Python
Étoiles
1
Forks
1
Merge moyen
1 j 23 h
PR mergées (30 j)
89

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de PolicyEngine/policyengine-sim-api

Toutes les issues de PolicyEngine/policyengine-sim-api

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.