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

Optimizada sincronización con CPython en CI #1186

Merged
merged 2 commits into from
Aug 2, 2021

Conversation

mondeja
Copy link
Contributor

@mondeja mondeja commented Dec 21, 2020

Este pull optimiza la sincronización con CPython en el CI, ahorrando como mínimo 1 minuto. El problema es que la configuración actual realiza un clone completo (incluído historial GIT) de python/cpython, pero esto no es necesario y será cada vez más costoso a medida que más commits sean añadidos a CPython. Añadiendo --depth=1 a git submodule update y eliminando el comando redundante git submodule sync, obtenemos el mismo resultado que actualmente en menos tiempo.

@mondeja mondeja marked this pull request as draft December 21, 2020 22:09
@mondeja mondeja force-pushed the speedup-cpython-sync branch 3 times, most recently from e210388 to 76bd613 Compare December 21, 2020 22:15
@mondeja mondeja marked this pull request as ready for review December 21, 2020 22:16
Copy link
Contributor

@eamanu eamanu left a comment

Choose a reason for hiding this comment

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

Me parece bien. Podrías cerrar y volver abrir el pr a ver si corre el CI? A nosotros nos interesa el proyecto no? No la historia git cierto? Se podría pensar en hacer algún wget para traer el proyecto?

@mondeja mondeja closed this Dec 24, 2020
@mondeja mondeja reopened this Dec 24, 2020
@mondeja
Copy link
Contributor Author

mondeja commented Dec 24, 2020

Se podría pensar en hacer algún wget para traer el proyecto?

Probé a hacer un simple clone pero fallaba en el paso de construir la documentación con algunas directivas doctest, aunque no tengo ni idea de por qué. Si te interesa puedes ver las pruebas que hice acá. En teoría debería ser posible, pero al estar configurado como submódulo en el archivo .gitmodules, no creo que sea buena idea.

@cmaureir cmaureir added the enhancement New feature or request label Mar 20, 2021
@cmaureir cmaureir changed the base branch from 3.8 to 3.9 August 2, 2021 18:35
@cmaureir cmaureir merged commit cfbdafb into python:3.9 Aug 2, 2021
cmaureir added a commit that referenced this pull request Aug 4, 2021
Co-authored-by: Cristián Maureira-Fredes <Cristian.Maureira-Fredes@qt.io>
@mondeja mondeja deleted the speedup-cpython-sync branch August 7, 2021 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants