~chad.smith/curtin/docs-proposed

  • Committer: Scott Moser
  • Date: 2017-08-03 16:46:08 UTC
  • mfrom: (508.2.20 trunk.tgt-cleanup)
  • Revision ID: smoser@ubuntu.com-20170803164608-4de4u4wlrm34ggbc
tools/jenkins-runner: improve tgtd cleanup logic

The previous tgtd cleanup logic was hard to follow at best. It was buggy
in that it would always kill the TGT_PID process even if it didn't start it.
both cleanup lines ended with:
  <condition> || [ -n "${TGT_PID}" ] && kill -9 ${TGT_PID}
which will always run the 'kill' if TGT_PID is non-empty.

Also:
 * clean up the output of tools/jenkins-runner by redirecting
   the find-tgt.d to a file so you don't see its debug messages.
 * use modulus to get a port in the right range.
 * improve the picking of a port and then checking to see if it was used.
   I believe a race condition where tgt wasn't listening on the port when
   netstat would run lead to false positives for failures where a
   sleep would have sufficed.
 * remove the tgt socket file(s) if it was created.
Filename Latest Rev Last Changed Committer Comment Size
..
bin 7 10 years ago Scott Moser initial stub multi-command Diff
curtin 7 10 years ago Scott Moser initial stub multi-command Diff
debian 9 10 years ago Scott Moser add initial packaging Diff
doc 2 11 years ago Scott Moser very rough initial doc using sphynx. Diff
examples 45 10 years ago Scott Moser add examples dir Diff
helpers 25 10 years ago Scott Moser block-meta simple now partitions with 'partition' Diff
tests 8 10 years ago Scott Moser make test (with 1 test) Diff
tools 8 10 years ago Scott Moser make test (with 1 test) Diff
.bzrignore 368 8 years ago Ryan Harper Enable python coverage for unittests make check 33 bytes Diff Download File
LICENSE 6 10 years ago Scott Moser add LICENSE file 33.7 KB Diff Download File
Makefile 488.1.9 7 years ago Chad Smith separate doc-deps into new target. Collapse depend 1.3 KB Diff Download File
README 1 11 years ago Scott Moser initial README 163 bytes Diff Download File
requirements.txt 380.4.1 8 years ago Scott Moser fix dependencies This fixes up dependencies to be 16 bytes Diff Download File
setup.py 387.6.4 8 years ago Ryan Harper Move curtin VERSION into module; use in setup.py, 775 bytes Diff Download File
test-requirements.txt 368 8 years ago Ryan Harper Enable python coverage for unittests make check 28 bytes Diff Download File
tox.ini 464 7 years ago Scott Moser tox: do not bother with pep8 from trusty in trusty 2.8 KB Diff Download File