Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Améliore la récupération de l'artifact conda #2211

Merged
merged 8 commits into from
Dec 5, 2023
Merged

Conversation

benoit-cty
Copy link
Contributor

@benoit-cty benoit-cty commented Nov 28, 2023

  • Changement mineur.
  • Périodes concernées : non applicable.
  • Zones impactées : .github/workflows/workflow.yml.
  • Détails :
    • Améliore la récupération d'artifacts en intégration continue pour le build de librairie conda
    • Permet de trouver le paquet même si le merge de la PR est fait sans avoir réalisé de commit après son ouverture.

Cette PR répond à une problématique sur la publication de librairies conda : lorsque tous les évènements de commits sont réalisés avant l'événement d'ouverture de PR, la librairie n'était pas trouvée. Avec cette PR la librairie conda est bien retrouvée.


Ces changements (effacez les lignes ne correspondant pas à votre cas) :

  • Modifient des éléments non fonctionnels de ce dépôt.

Quelques conseils à prendre en compte :

@sandcha sandcha self-requested a review November 30, 2023 14:53
@sandcha
Copy link
Contributor

sandcha commented Dec 5, 2023

La version initiale de cette PR répond à une problématique sur la publication de librairies conda : lorsque tous les évènements de commits sont réalisés avant l'événement d'ouverture de PR, la librairie n'était pas trouvée. Avec cette PR la librairie conda est bien retrouvée.
La CI est plus robuste mais cette PR en allonge le temps sur master puisque le build ne se base plus sur l'artefact de la branche de travail mais est effectué sur master.

Les problèmes d'artefacts identifiés sont :

  • Refait le job build en intégration continue déclenché lors du merge de pull request sur la branchemaster pour éviter les problèmes avec les artefacts produits pour conda.
    • les artefacts disparaissent après un certain délais (30 jours)
    • les artifacts sont produits et nommés par commit et sont complexes à récupérer d'une branche à l'autre

Mais après vérification, le sujet de cette PR semble comparable à ce qui a été investigué sur openfisca/openfisca-france-data#230 où la solution est a priori indiquée par ce commentaire. On bascule donc vers cette solution.

@benoit-cty benoit-cty changed the title Remove use of artifacts Améliore la récupération de l'artifact conda Dec 5, 2023
Copy link
Contributor

@sandcha sandcha left a comment

Choose a reason for hiding this comment

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

Merci @benoit-cty pour ce fix de CI !

.github/workflows/workflow.yml Outdated Show resolved Hide resolved
@benjello
Copy link
Member

benjello commented Dec 5, 2023

Merci @benoit-cty et @sandcha !

@benoit-cty benoit-cty merged commit 06f5c3a into master Dec 5, 2023
38 checks passed
@benoit-cty benoit-cty deleted the fix_conda_build branch December 5, 2023 12:01
@benoit-cty benoit-cty mentioned this pull request Dec 21, 2023
7 tasks
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