Skip to content

perf(ci-rust): n'interroge apt que si un paquet manque vraiment - #197

Merged
BryanFRD merged 1 commit into
mainfrom
perf/skip-apt-when-present
Aug 4, 2026
Merged

perf(ci-rust): n'interroge apt que si un paquet manque vraiment#197
BryanFRD merged 1 commit into
mainfrom
perf/skip-apt-when-present

Conversation

@BryanFRD

@BryanFRD BryanFRD commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Observé sur FerrFleet-Cloud aujourd'hui, mais le défaut est ici et concerne tout dépôt Rust qui passe apt-packages — FerrVault et consorts envoient les trois mêmes.

Fetched 32.0 MB in 28min 58s (18.4 kB/s)
build-essential is already the newest version (12.10ubuntu1).
pkg-config is already the newest version (1.8.1-2build1).
libssl-dev is already the newest version (3.0.13-0ubuntu3.12).
0 upgraded, 0 newly installed, 0 to remove

29 minutes pour installer zéro paquet. L'image des runners self-hosted porte ces paquets ; l'étape ne faisait que télécharger des listes pour conclure qu'elle n'avait rien à faire.

Le correctif

dpkg -s d'abord, sortie immédiate si tout est présent, et sinon on n'installe que ce qui manque réellement. Appliqué aux quatre occurrences de l'étape.

C'est exactement le motif que suit déjà l'étape Install postgresql-client juste à côté (command -v createdb puis sortie) — donc l'intention existait, elle n'avait simplement pas été portée ici.

Au passage, les trois occurrences qui interpolaient ${{ inputs.apt-packages }} directement dans le run passent par env:, comme le faisait déjà la quatrième.

Le débit, séparément

18,4 ko/s reste anormal. Mesuré depuis le nœud : 53 Mo/s vers github.com, 637 ko/s vers archive.ubuntu.com. Le pod a donc obtenu 35× moins que son hôte sur la même destination — un problème d'egress des pods que cette PR ne résout pas, mais dont elle supprime l'exposition sur ce chemin.

Portée

Les dépôts épinglent ce workflow par SHA ; le changement ne se propagera qu'au rythme de Renovate, dépôt par dépôt.

L'étape téléchargeait 32 Mo de listes pour n'installer aucun paquet quand l'image du runner les portait déjà; un run observé y a passé 29 minutes.
Copilot AI lite review requested due to automatic review settings August 4, 2026 17:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@BryanFRD
BryanFRD merged commit ab3cfaa into main Aug 4, 2026
9 checks passed
@BryanFRD
BryanFRD deleted the perf/skip-apt-when-present branch August 4, 2026 17:58
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.

2 participants