| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
The pep8 jobs aren't running in gate at the moment.
Fix up the pep8 issues and remove the tox filter that stopped it
from running.
Also ignore E129 (visually indented line with same indent as next
logical line) as we don't follow it.
Change-Id: I394708ba96797bbc6fcd951e6436a104be0a3746
|
|
|
|
|
|
|
|
| |
As per the governance resolution https://review.openstack.org/119875,
projects should have a docs environment in their tox.ini testing
interface. Rename the doc environment to docs.
Change-Id: I5908033d835d35a0650b77b01ff727d81eb79eb0
|
|
|
|
|
|
|
|
|
|
| |
According to https://docs.python.org/3/howto/pyporting.html the
syntax changed in Python 3.x. The new syntax is usable with
Python >= 2.6 and should be preferred to be compatible with Python3.
Enabled hacking check H231.
Change-Id: I7ce80163cc328c0368583d1dc76e9e0b0f7cd6dc
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The Sphinx documentation now make use of the `program-output` plugin
which would invoke commands to generate inline documentation. Ex:
program-output:: zuul --help
program-output:: zuul enqueue --help
We want the resulting output to correspond to Zuul source code, not the
command which is currently installed on the host running the doc. On my
setup sphinx would die out because it cant find the command 'zuul'.
The new tox environement 'doc' runs sphinx in a virtual env which will
have the proper zuul command.
The generated doc is not written under the /.tox directory but to
/doc/build/html for convenience.
Example usage:
tox -edoc && open doc/build/html/index.html
Change-Id: Ib0170f94bb2c09eb60e555a32e101e2e0959b18e
|
|
|
|
| |
Change-Id: I64c29c3c8be66c24c4d9f26767e96200dc83dae4
|
|
|
|
|
|
|
| |
Remove pyflakes env - it's covered by flake8. Use custom install_command
to avoid unwanted pre-release software. Skip the unneeded sdist step.
Change-Id: I2c86dabeb78da9f1a9ee7a8636bc5d58b2439d20
|
|
|
|
|
|
|
|
|
|
|
| |
By default, tox passes --pre option to pip install commands so that
prerelease packages, which perhaps are not suitable for test,
would be installed, causing unforseeable errors. Like:
http://logs.openstack.org/33/47233/1/check/gate-zuul-docs/7b794af/console.html
whose installed Sphinx package is a beta version 2 causing the error in the log.
fungi's proposal https://review.openstack.org/#/c/47239/1 also applies here.
Change-Id: I9108b534bb469211434a4abf22b25c983aa444ba
|
|
|
|
|
| |
Change-Id: I2e8c9c1dab31c939d73a29b0c76d9454458515ef
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: I9ac6e301d361b7bac8561a1f468c40be23e1ddcc
Reviewed-on: https://review.openstack.org/36281
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Needed to move some directory creation around to be contained within the
testcase, but with parallel testing, we shave 48 seconds off the run
time and go from around 60s to around 11. We're also now compatible with
future subunit-based fast-fail semantics when we grow them.
Change-Id: I6c7148c29d1edb5d9469a8c2afe4b31b2b340009
Reviewed-on: https://review.openstack.org/33352
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
| |
Change-Id: I0385ac9f8336a8c2e45b0fa6ed062b6b23a182aa
Reviewed-on: https://review.openstack.org/33351
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: Id0e447a7e66562d6ad0298619e36f752ae31adb0
Reviewed-on: https://review.openstack.org/33349
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Doug Hellmann <doug.hellmann@dreamhost.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
| |
Remove the Jenkins launcher and add a new Gearman launcher (designed
to be compatible with Jenkins) in its place.
See the documentation for how to set up the Gearman Plugin for
Jenkins.
Change-Id: Ie7224396271d7375f4ea42eebb57f883bc291738
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
From the pep8 spec:
The closing brace/bracket/parenthesis on multi-line constructs
may either line up under the last item of the list, as in:
...
or it may be lined up under the first character of the line that
starts the multi-line construct, as in:
...
Bizzarly, the pep8 tool ignores the first option.
Change-Id: Ic0e8f8ad964881276d213ffd50deb75a537c250b
Reviewed-on: https://review.openstack.org/28609
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously Zuul had to merge the entire change tree for each change
in the DependentPipeline. Remember previous merges to save time and
only merge changes against their project and branch.
Co-Authored-By: James E. Blair <jeblair@openstack.org>
Change-Id: I8105ac3aa4e4697e90e8d7fac1e54c5614f79daf
Reviewed-on: https://review.openstack.org/22504
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
| |
Change-Id: I7e3bd8eed585133ee217a1e462b9833df494be21
Reviewed-on: https://review.openstack.org/19718
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a build is reported as lost, it has no start_time, so the
code that calculated the run-time for statsd threw an exception.
Correct that by only calculating the run time for jobs that have
really completed.
Add debug lines to onBuild* that would have helped diagnose this.
Keep track of the throwaway threads that are used to report LOST
builds so that the test suite can more correctly know when the
system is settled.
Add a test case for a single non-existent post job.
Add STATSD_* env variables in tox to make sure that the statsd
code paths are executed by tests.
Change-Id: I339443cbad7b0858d70e9180d63dfb701b8eaee9
Reviewed-on: https://review.openstack.org/18792
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
| |
If we put the code for zuul-server into the zuul module, then autodoc
will pick it up for documentation.
Change-Id: Idf2db82f706243c6e69984170b4783f1eece2237
Reviewed-on: https://review.openstack.org/16850
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed all pep8 errors.
E125 is ignored because of false alarms.
Change-Id: I4da60409e0095c0896230cd01bda548ed2e3f741
Reviewed-on: https://review.openstack.org/13740
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Updated pep8 version requirement to 1.3.3
Fixed E127 errors
All ignores are to be removed in the next sequence of patches
Change-Id: Ia9e922b8873686a0f905f2548cc43d534ee1c912
Reviewed-on: https://review.openstack.org/13642
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Zhongyue Luo <zhongyue.nah@intel.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add support for testing, and some initial functional tests.
Some threads in Zuul now have proper shutdown methods, since
that's needed for testing. The threads that are faked in the
testing infrastructure still lack shutdown methods.
Fix some bugs uncovered by tests:
Reverse the order of commits that are given to Jenkins (the
commits that preceded the one being tested were listed in
revers order, this corrects that).
Return 0 in reportChange if the change has already been
reported. The old return value of True was misinterpreted
as an error.
Change-Id: I4979d0efd8581061a2b856b892d4ecdb75284a1b
Reviewed-on: https://review.openstack.org/10572
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
|
|
|
|
| |
Change-Id: I7ba57af0991007361e515222db7678914d747d53
|
|
|
|
|
|
|
|
| |
Use __repr__ for more objects so they look better in log messages.
Fix a problem with erroneous information on the "depends on" log
message. Fixes bug #1011908.
Change-Id: I43adc5fd0943d99dae28276b7a72009ce9f9a91c
|
|
|
|
| |
Change-Id: I8197ec2e52596fa4136f8af9aa93ea06e56d4d0d
|
|
|
|
|
|
|
|
|
|
|
|
| |
It searches for outstanding builds that haven't reported back
or otherwise can't be found by Jenkins, and marks them complete
with a result of "LOST". This should help to avoid deadlocks
where Zuul waits forever to hear back from Jenkins after missing
a notification.
Add pyflakes to tox.ini.
Change-Id: I26d3fbf375e82e224448ec3337f9cc97deeccd56
|
|
Add tox.ini with pep8 section.
Fix pep8 errors.
Change-Id: I6c171104359b16bcb130fdd56697d0663d8ec562
|