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

Utilise l'infrastructure de build par container sur Travis #2672

Merged
merged 4 commits into from
May 13, 2015

Conversation

sandhose
Copy link
Contributor

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? oui
Tickets (issues) concernés #2664

Travis a actuellement deux infrastructures de build: une, qui virtualise un ubuntu complet, et l'autre qui utilise des container, à base de Docker.

Cette PR passe le build travis vers cette nouvelle infra, qui est bien plus rapide, autant à démarrer (le build démarre au bout d'une minute, contre 5min voir plus auparavant), qu'a lancer les tests.

Exemple de build avec container: https://travis-ci.org/sandhose/zds-site/builds/62295015

Le gain de temps est assez impressionnant, puisqu'on passe d'environ 30-35min à 15-20min pour le back, et de 10-12min à 5min pour le front.

Une des restrictions de cette infra est de tout lancer sans root, pour des raisons de sécurité. J'ai du donc installer texlive, qui n'est pas dans la liste des paquets autorisés par travis manuellement (d'où le scripts/install_texlive.sh, qui installe texlive avec les paquets qu'il faut dans ~/.texlive).

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.11% when pulling 0f41f14 on sandhose:travis-container into 85e62f9 on zestedesavoir:release-v15.5.1.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 81.9% when pulling 0f41f14 on sandhose:travis-container into 85e62f9 on zestedesavoir:dev.

@Eskimon
Copy link
Contributor

Eskimon commented May 13, 2015

Travis est heureux donc nous aussi ? @SpaceFox tu en penses quoi ?

@SpaceFox
Copy link
Contributor

\o/

Je ne merge pas tout de suite pour laisser le temps de la relecture par contre. Sait-on jamais :)

- sudo chmod a+r /usr/share/fonts/truetype/Merriweather/*.ttf
- sudo fc-cache -f -v
- mkdir -p ~/.fonts/truetype
- wget -P ~/.fonts/truetype https://www.dropbox.com/s/4vzkldw55iag546/Andale-Mono.ttf
Copy link
Contributor

Choose a reason for hiding this comment

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

C'est nouveau la polise Andale-Mono ?

@GerardPaligot
Copy link
Member

J'aime pas l'idée de dépendre de dropbox, il n'y aurait pas moyen d'avoir un truc plus stable ?

@Situphen
Copy link
Member

Je plussoie @GerardPaligot !

D'ailleurs, pourquoi tu supprimes l'installation de Node.JS ?

@sandhose
Copy link
Contributor Author

Pour Andale-Mono, c'est l'une des police fournie par ttf-msfonts-installer, qui ne s'installe pas correctement, puisque l'on ne peut pas accepter l'EULA de Microsoft sans root... Du coup, j'ai fait comme Merriweather, et je l'ai envoyé sur Dropbox. Dropbox n'est donc pas totalement nouveau ici...

Pour Node, c'est juste que ça fait un moment qu'il est installé par défaut dans les container travis, et que cette installation était simplement inutile

@Situphen
Copy link
Member

OK. Et pour NPM, la version disponible dans les conteneurs est à jour (ou au minimum c'est en v2.x) ?

@GerardPaligot
Copy link
Member

Du coup, j'ai fait comme Merriweather, et je l'ai envoyé sur Dropbox. Dropbox n'est donc pas totalement nouveau ici...

On fait du caca autre part alors on continue à en faire ?

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.10% when pulling 2636aa4 on sandhose:travis-container into 85e62f9 on zestedesavoir:release-v15.5.1.

@sandhose
Copy link
Contributor Author

C'est prêt à merge ! Les polices sont maintenant téléchargées via un dossier GDrive, et cette PR fixe aussi les récents problèmes de build dues à tox (#2690)

SpaceFox added a commit that referenced this pull request May 13, 2015
Utilise l'infrastructure de build par container sur Travis
@SpaceFox SpaceFox merged commit 2472280 into zestedesavoir:dev May 13, 2015
@landscape-bot
Copy link

Code Health
Repository health decreased by 0.10% when pulling 8b6ab86 on sandhose:travis-container into 85e62f9 on zestedesavoir:release-v15.5.1.

@Situphen
Copy link
Member

Le build back-end a échoué ! Ça a l'air d'être à cause des fonts.

D'ailleurs, je ne savais pas que quand le build-backend échoue mais que le build front-end réussi Travis fait comme si tous les builds avaient réussi ! Oo

@Eskimon
Copy link
Contributor

Eskimon commented May 14, 2015

D'ailleurs, je ne savais pas que quand le build-backend échoue mais que le build front-end réussi Travis fait comme si tous les builds avaient réussi ! Oo

Ah ca c'est chaud... ca faisait pas ca avant...

@Eskimon Eskimon mentioned this pull request May 14, 2015
@sandhose sandhose deleted the travis-container branch January 6, 2016 08:43
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.

None yet

7 participants