Project

General

Profile

Development #40071

supprimer (ou pas) jenkins.sh

Added by Nicolas Roche about 1 month ago. Updated 17 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
21 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

A priori ça devrait être possible, et ça permettrait de simplifier.

0001-tests-remove-jenkins.sh-40071.patch View (15.4 KB) Nicolas Roche, 21 Feb 2020 05:26 PM

0001-tests-remove-jenkins.sh-40071.patch View (14.4 KB) Nicolas Roche, 24 Feb 2020 12:57 PM


Related issues

Related to Hobo - Development #40123: tests: avoir des venv distincts pour les diffrents environnement de tests Solution déployée 24 Feb 2020
Related to Hobo - Development #40257: Tourner en python3 En cours 28 Feb 2020

Associated revisions

Revision 0d821053 (diff)
Added by Nicolas Roche about 1 month ago

tests: remove jenkins.sh (#40071)

History

#1 Updated by Nicolas Roche about 1 month ago

J'ai ouvert ce ticket pour dans l'espoir de décoincer #40012,
mais en fait je me retrouve avec le même genre d'erreur :

py2-hobo-coverage runtests: commands[3] | mv htmlcov htmlcov-py2-hobo-coverage
  /var/lib/jenkins/workspace/_wip_wip_40071-remove-jenkins.sh$ /bin/mv htmlcov htmlcov-py2-hobo-coverage 
/bin/mv: impossible de déplacer 'htmlcov' vers 'htmlcov-py2-hobo-coverage/htmlcov': Le dossier n'est pas vide
ERROR: InvocationError: '/bin/mv htmlcov htmlcov-py2-hobo-coverage'

#2 Updated by Nicolas Roche about 1 month ago

  • Assignee set to Nicolas Roche

Pour info, j'ai débloqué #40012 :

py2-coverage-hobo create: /tmp/tox-jenkins/hobo/wip/40012-run-python3-tests/py2-coverage-hobo
ERROR: cowardly refusing to delete `envdir` (it does not look like a virtualenv): /tmp/tox-jenkins/hobo/wip/40012-run-python3-tests/py2-coverage-hobo

En modifiant temporairement le jenkins.sh :
<<<
venv/bin/tox -rv
---
venv/bin/tox -v
>>>

#3 Updated by Nicolas Roche about 1 month ago

Le patch fait un petit peu plus en essayant de se rapprocher de ce qui est fait pour les autres briques (combo et passerelle par exemple).
(dommage sans ça j'avais presque un patch rouge)
En modifiant jenkins.sh j'étais embêté pour remplacer les rm :

rm -f *coverage.xml
rm -f *test_results.xml
rm -rf htmlcov htmlcov-*

Thomas m'a suggéré de voir si l'on pouvait pas spécifier directement ces répertoires, ce qui est effectivement possible via py.test :
$ py.test --help
  --cov-report=type     ...
                        html and xml may be followed by ":DEST" where DEST
                        specifies the output location. ...

#4 Updated by Nicolas Roche about 1 month ago

(en supprimant le superflux lié au fait de se rapprocher de ce qui est fait pour les autres briques)

#5 Updated by Thomas Noël about 1 month ago

Je m'interroge sur le toxworkdir, ne faut-il pas y ajouter un {env:BRANCH_NAME:} final ?

#6 Updated by Nicolas Roche about 1 month ago

  • Related to Development #40123: tests: avoir des venv distincts pour les diffrents environnement de tests added

#7 Updated by Emmanuel Cazenave about 1 month ago

  • Status changed from Solution proposée to Solution validée

Thomas Noël a écrit :

Je m'interroge sur le toxworkdir, ne faut-il pas y ajouter un {env:BRANCH_NAME:} final ?

Ce serait mieux mais pas de rapport particulier avec le fait de se débarrasser du jenkins.sh, go.

#8 Updated by Emmanuel Cazenave about 1 month ago

Emmanuel Cazenave a écrit :

Ce serait mieux mais pas de rapport particulier avec le fait de se débarrasser du jenkins.sh, go.

Et Nico me dit qu'il était déjà en train de rajouter ça dans un des tickets tests/py3.

#9 Updated by Nicolas Roche about 1 month ago

#10 Updated by Nicolas Roche about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 0d82105309b93587475721d74c258fa3fd99c641
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Fri Feb 21 11:22:43 2020 +0100

    tests: remove jenkins.sh (#40071)

#11 Updated by Frédéric Péters 17 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF