Skip to content

Upgrade ci#443

Merged
amrosu merged 3 commits intomasterfrom
upgrade_ci
Nov 14, 2025
Merged

Upgrade ci#443
amrosu merged 3 commits intomasterfrom
upgrade_ci

Conversation

@amrosu
Copy link
Collaborator

@amrosu amrosu commented Nov 14, 2025

Les tests ci ne fonctionnaient plus à cause notamment de la version de python qui n'était plus trouvée dans les dépôts car elle contenait le numéro du correctif qui est remplacé dans les dépôts quand un nouveau apparaît.

Les modifications apportées pour que les tests ci refonctionnent :

  • mis à jour la version python en gardant seulement le numéro de version majeure et mineure pour plus de stabilité et souplesse
  • mis à jour les options de config gdal dans pip qui étaient devenues obsolètes et qui ne fonctionnaient plus avec les nouvelles versions de pip > 23.3

J'ai aussi figé la version d'ubuntu pour le docker ci pour mieux gérer les versions des paquets et ne pas avoir des surprises avec les versions imposées par "latest".

Avec les nouvelles versions de python, gdal etc. les tests passent, les scripts python de création/update cache sont validées, mais je vois qu'ils ont des messages avec des exceptions qui ne les empêchent pas d'arriver au bout du calcul et de générer des caches valides, ce qui fait que les tests passent. Comme cela n'est pas lié au ci, je propose de les gérer plus tard dans une autre PR.

@amrosu amrosu requested a review from ftoromanoff November 14, 2025 09:45
@amrosu amrosu self-assigned this Nov 14, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 19360662386

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.289%

Totals Coverage Status
Change from base Build 16142636101: 0.0%
Covered Lines: 5917
Relevant Lines: 6168

💛 - Coveralls

@ftoromanoff
Copy link
Collaborator

En figeant les version utilisées ca évite effectivement les désagréments de 'breaking change' et autres. Mais ca veut aussi dire qu'il faudra potentiellement être attentif au moment ou il y a des changements important pour faire l'upgrade de façon manuel si nécessaire.

@amrosu amrosu merged commit aa88e56 into master Nov 14, 2025
4 checks passed
@amrosu amrosu deleted the upgrade_ci branch November 14, 2025 10:57
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