WIP : Feature nxbt 2619 update busy slaves #40
Conversation
| for slave in ${availableSlaves}; do | ||
| slave=\${slave/[/} && slave=\${slave/]/} && slave=\${slave/,/} | ||
| echo "\$slave" | ||
| ssh jenkins@qa-ovh0"\${i}".nuxeo.com "bash -s -- \${slave}" < . ../common/check.sh |
There was a problem hiding this comment.
|| true or set +e rather than removing the whole error-on-exit behavior
There was a problem hiding this comment.
you can remove the remote ssh with a local call to
docker -H qa-ovh01.nuxeo.com:4243 ps -f name=itslave01 --format '{{.ID}}'
or
docker -H tcp://swarm-qa.nuxeo.org:4000 info
There was a problem hiding this comment.
(alternate way could have been to retrieve the SH output in a Groovy variable for later treatment)
| availableSlaves = [] | ||
| results = readFile ('result.txt'.trim()); | ||
| result = results.readLines(); | ||
| for (validatedSlave in result) { |
There was a problem hiding this comment.
results = readFile('result.txt'.trim()).readLines();
for (validatedSlave in results) {
| result = results.readLines(); | ||
| for (validatedSlave in result) { | ||
| for (slave in staticSlaves) { | ||
| if (validatedSlave == slave.getDisplayName()) { |
There was a problem hiding this comment.
validatedSlave => slaveToUpdate for instance
| // trigger this job again if we have set slaves offline | ||
| if (onlineBusySlaves.size() > 0) { | ||
| build job: 'update_static_slaves', propagate: false, quietPeriod: 3600 | ||
| } |
| @@ -61,8 +105,10 @@ def update_static_slaves(boolean doConfirm=false) { | |||
| } | |||
| } | |||
| } | |||
There was a problem hiding this comment.
here, the updated slaves need to be put online back
the start Shell recreates the slave, so it is online by default, the default one being actually deleted.
jcarsique
force-pushed
the
feature-NXBT-2619-update-busy-slaves
branch
from
329b349
to
2ad9c8c
Compare
alexistimic
force-pushed
the
feature-NXBT-2619-update-busy-slaves
branch
2 times, most recently
from
5945404
to
c30bf37
Compare
jcarsique
force-pushed
the
feature-NXBT-2619-update-busy-slaves
branch
from
341bdcf
to
c74cf56
Compare
jcarsique
force-pushed
the
feature-NXBT-2619-update-busy-slaves
branch
from
c464e5b
to
4ea9723
Compare
WIP DO NOT MERGE