delta/openstack/zuul.git - opendev.org: zuul/zuul.git

	Commit message (Collapse)	Author	Age	Files	Lines
*	Merge "Stop running tox-cover job" into feature/zuulv3feature/zuulv3	Zuul	2018-01-18	1	-5/+0
\|\
\| *	Stop running tox-cover job	Monty Taylor	2018-01-16	1	-5/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	tox-cover frequently fails and is non-voting so is also frequently ignored. Stop running the job until such a time as someone cares enough to figure out how to make it stable enough that we might make it voting. Change-Id: Icfffb8060c80a4cbaa834987654f754bc35fbea0
* \|	Merge "Link to zuul-base-jobs docs from User's Guide" into feature/zuulv3	Zuul	2018-01-18	1	-2/+5
\|\ \
\| * \|	Link to zuul-base-jobs docs from User's Guide	Jeremy Stanley	2018-01-08	1	-2/+5
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	In the User's Guide section dealing with Git Repositories, link to the zuul-base-jobs documentation to solve half of the TODO comment there. Change-Id: Ic333278f1f00184f033ab01ab4568e53b2e40fea
* \| \|	Merge "Register term_handler for all zuul apps" into feature/zuulv3	Zuul	2018-01-17	6	-58/+17
\|\ \ \
\| * \| \|	Register term_handler for all zuul apps	Tobias Henkel	2017-12-27	6	-58/+17
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Almost all zuul apps use the method term_handler for SIGINT and SIGTERM. Defining this centrally in ZuulDaemonApp makes this much simpler and without repitition. Change-Id: I68f8d1bf52b0e16340818d2bcc44cd9fc5868ca7
* \| \| \|	Merge "Make ZuulDaemonApp an abstract base class" into feature/zuulv3	Zuul	2018-01-17	1	-1/+9
\|\ \ \ \ \| \|/ / /
\| * \| \|	Make ZuulDaemonApp an abstract base class	Tobias Henkel	2017-12-27	1	-1/+9
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We use ZuulDaemonApp like an abstract base class with run() as an abstract method so make that explicit. This creates the groundwork for later refactorings like centralized signal handling. Change-Id: I20f14274df27ab181711b2ca2b80251fa5b09938
* \| \| \|	Document connection information for components	James E. Blair	2018-01-17	1	-2/+57
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add some information about how the different Zuul components communicate with each other. Change-Id: I637a949e089c8df75203c6e585d6aeb933c2cb98
* \| \| \|	Stabilize git driver tests	James E. Blair	2018-01-17	3	-16/+33
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	These tests relied on sleeps which can cause races when running the full test suite in parallel. Instead, wait for the events we know will happen to happen. Also remove the dependency on yarl now that aiohttp has made a release which works with yarl 1.0 (however, it does not work with <1.0 which is why this needs to be combined with this change to fix tests). Change-Id: Ib1c626cdd3f083dd1d23a3c6547bd7163b66567e
* \| \| \|	Merge "Centrally register stack dump handler" into feature/zuulv3	Zuul	2018-01-17	5	-7/+4
\|\ \ \ \ \| \|/ / /
\| * \| \|	Centrally register stack dump handler	Tobias Henkel	2017-12-27	5	-7/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	We want the stack dump handler to be present in all zuul apps so this can be registered in a central place. Change-Id: I0c4a97d6ee983aa4d57928682dfb6eeffd050197
* \| \| \|	Merge "Fix dependency cycle false positive" into feature/zuulv3	Zuul	2018-01-17	4	-4/+67
\|\ \ \ \
\| * \| \| \|	Fix dependency cycle false positive	James E. Blair	2018-01-17	4	-4/+67
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This corrects a false-positive in the dependency cycle detection, but only for the new URL-style depends-on headers. It does not do so for the legacy gerrit headers. We used a single history list to store all the changes we enqueued ahead of a given change, but this meant that if there was more than one path to a change, we would see it in the history on the second traversal. Instead, when traversing the tree, use copies of the history list at each stage so that it can be rewound when going back up the tree. The second path to a change will not trip the cycle detection, and will proceed on to the point where it notices the change is already in the queue and return harmlessly. Also, check whether the exact change is in the history, not just the number, since numbers are no longer unique with multiple sources. Also, fix a bug in the test_crd_cycle test which was causing the test to always pass since the changes were never enqueued due to missing approval requirements. Change-Id: I3241f90a1d7469d433cfa176e719322203d4d089 Story: 2001427 Task: 6133
* \| \| \| \|	Merge "Remove unused method term_handler" into feature/zuulv3	Zuul	2018-01-17	1	-4/+0
\|\ \ \ \ \ \| \| \|/ / / \| \|/\| \| \|
\| * \| \| \|	Remove unused method term_handler	Tobias Henkel	2017-12-27	1	-4/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This method seems to be superseeded by exit_handler and grep tells me it is unused. Change-Id: I5a4dc126acbbe1ac2f99153bc7757c4f6e46fc8c
* \| \| \| \|	Merge "Enable direct use of github driver in debug tool" into feature/zuulv3	Zuul	2018-01-17	1	-38/+54
\|\ \ \ \ \
\| * \| \| \| \|	Enable direct use of github driver in debug tool	Tobias Henkel	2017-12-22	1	-38/+54
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The github debugging tool proved to be quite useful for prototyping new stuff using the github api. In the process I ended up copying half of the GithubConnection into it. However for trying out new stuff it would be great to implement and try out the features/fixes directly in the Github driver. And here it is! The script now starts up a standalone version of the Github driver which can be used to query and debug isolated stuff against Github. Change-Id: Ifdad1e69dae009011847869d51ff24000c44adb8
* \| \| \| \| \|	Merge "Add --strip option to encrypt_secret.py" into feature/zuulv3	Zuul	2018-01-17	1	-0/+4
\|\ \ \ \ \ \
\| * \| \| \| \| \|	Add --strip option to encrypt_secret.py	Clint Byrum	2017-12-30	1	-0/+4
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Sometimes you're storing a password in a file by editting it, but you want it to be raw without newlines/whitespace/etc. This lets you do that easily. Change-Id: Idc961b89a5ec3fb639e70a321b4ea587cf743b9d
* \| \| \| \| \| \|	Merge "Handle sigterm in nodaemon mode" into feature/zuulv3	Zuul	2018-01-16	3	-9/+12
\|\ \ \ \ \ \ \ \| \| \|_\|/ / / / \| \|/\| \| \| \| \|
\| * \| \| \| \| \|	Handle sigterm in nodaemon mode	Tobias Henkel	2017-12-18	3	-9/+12
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When running zuul within a container it normally runs in nodaemon mode as pid 1. Currently in this mode zuul just ignores SIGTERM which is used normally to stop containers. Thus when running within OpenShift it waits for a timeout until it gets killed forcefully. Fix this by handling SIGINT and SIGTERM equally. Change-Id: I24bd8c953e734fdb9545714126d77cbcdc161bbd
* \| \| \| \| \| \|	Merge "Remove updateChange history from github driver" into feature/zuulv3	Zuul	2018-01-16	1	-16/+3
\|\ \ \ \ \ \ \ \| \| \|_\|_\|/ / / \| \|/\| \| \| \| \|
\| * \| \| \| \| \|	Remove updateChange history from github driver	James E. Blair	2018-01-16	1	-16/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This is not necessary because updateChange in this driver is no longer recursive. Change-Id: If0322a5938b17000e9ef4106fa4b0ffcbfa80c89
* \| \| \| \| \| \|	Merge "Documentation changes for cross-source dependencies" into feature/zuulv3	Zuul	2018-01-16	2	-57/+35
\|\ \ \ \ \ \ \ \| \|/ / / / / /
\| * \| \| \| \| \|	Documentation changes for cross-source dependencies	James E. Blair	2018-01-16	2	-57/+35
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: Idb6530df5c6e647acd201509d2f0d9d86803db14 Story: 2001334 Task: 5885
* \| \| \| \| \| \|	Merge "Add support for protected jobs" into feature/zuulv3	Zuul	2018-01-16	11	-1/+175
\|\ \ \ \ \ \ \
\| * \| \| \| \| \| \|	Add support for protected jobs	Tobias Henkel	2017-12-15	11	-1/+175
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	For some use cases protected jobs can be useful. Protected jobs can only be inherited by jobs defined in the same project. This adds support for these protected jobs. Change-Id: I62a8ecbbfa9eec54ab599bb34148976dcabfd40a
* \| \| \| \| \| \| \|	Merge "Disambiguate with Netflix and Javascript zuul" into feature/zuulv3	Zuul	2018-01-16	2	-0/+16
\|\ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \|	Disambiguate with Netflix and Javascript zuul	Clint Byrum	2018-01-04	2	-0/+16
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	There are two other projects named zuul. Let's make sure we disambiguate so users aren't confused. Change-Id: I6c459d062970e2abcbb890d626297595d979d324
* \| \| \| \| \| \| \| \|	Merge "Use hotlink instead log url in github job report" into feature/zuulv3	Zuul	2018-01-16	3	-7/+26
\|\ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \|	Use hotlink instead log url in github job report	liusheng	2018-01-08	3	-7/+26
\| \| \|_\|_\|_\|_\|_\|/ / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This change make the job name in the job report comments of github driver as a hotlink to instead of using a log url directly. This can make the report comments more brief. Change-Id: I19fb8ffbc153230b7f8eedfcd5ac15ec81a66c72
* \| \| \| \| \| \| \| \|	Merge "Really change patchset column to string" into feature/zuulv3	Zuul	2018-01-16	3	-5/+34
\|\ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Really change patchset column to string	Tobias Henkel	2018-01-10	3	-5/+34
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The previous change I26ff56159c2710af1515955d27bf4e9ebfcf76e9 had a small bug which rendered it basically a noop. Further it missed some important parts and test changes. Change-Id: I6cfb8a6af05e589140c2f7c8b1d7228f6d4d8fcb
* \| \| \| \| \| \| \| \|	Merge "Add cross-source tests" into feature/zuulv3	Zuul	2018-01-16	20	-5/+1198
\|\ \ \ \ \ \ \ \ \ \| \| \|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add cross-source tests	James E. Blair	2018-01-16	20	-5/+1198
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: Iaf31211d12a2c8ce3b4a2860e079748f7e705aba Story: 2001334 Task: 5885
* \| \| \| \| \| \| \| \|	Merge "Support cross-source dependencies" into feature/zuulv3	Zuul	2018-01-16	22	-129/+433
\|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / /
\| * \| \| \| \| \| \| \|	Support cross-source dependencies	James E. Blair	2018-01-16	22	-129/+433
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Additional tests and docs in later patches. Change-Id: I3b86a1e3dd507fa5e584680fb6c86d35f9ff3e23 Story: 2001334 Task: 5885
* \| \| \| \| \| \| \| \|	Merge "Add skipped CRD tests" into feature/zuulv3	Zuul	2018-01-16	1	-0/+633
\|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / /
\| * \| \| \| \| \| \| \|	Add skipped CRD tests	James E. Blair	2018-01-10	1	-0/+633
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Change-Id: I145617342b424397ae19cf92335b357e413559ba
* \| \| \| \| \| \| \| \|	Temporarily pin yarl while aiohttp is broken	James E. Blair	2018-01-15	1	-0/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Aiohttp had an open requirement specification on yarl which has released a 1.0 that is backwards incompatible. Pin to <1.0 until https://github.com/aio-libs/aiohttp/issues/2662 is fixed. Change-Id: I4e750900501ed92bdbb616f5664f7e8ab7fa99c3
* \| \| \| \| \| \| \| \|	Merge "Remove need to start executor as root" into feature/zuulv3	Zuul	2018-01-10	6	-42/+13
\|\ \ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \| \|	Remove need to start executor as root	David Shrewsbury	2018-01-10	6	-42/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Now that we have a finger gateway, we no longer need to start the executor as root so that the finger streamer on the executor can bind to port 79 (default port for the finger streamer is changed from 79 to 7900). Remove that requirement. Change-Id: I6df685044c4ce81fd263043adba832609da100af
* \| \| \| \| \| \| \| \| \|	Merge "Move CRD tests to test_gerrit_legacy_crd" into feature/zuulv3	Zuul	2018-01-10	2	-605/+629
\|\ \ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / / \|/\| / / / / / / / / \| \|/ / / / / / / /
\| * \| \| \| \| \| \| \|	Move CRD tests to test_gerrit_legacy_crd	James E. Blair	2018-01-09	2	-605/+629
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This new class holds the unit tests which verify the soon-to-be "legacy" form of cross-repo-dependencies which use the gerrit change-id in the commit message footer. The new form will use the URL instead. We will maintain both forms for some time, so keep these tests around. In a later change, this file will be duplicated to exercise the same behavior with the new URL-based form. Change-Id: I63a7398640bf82963fadf19acdcd2208c185da0b
* \| \| \| \| \| \| \| \|	Merge "Add specific setup inventory" into feature/zuulv3	Zuul	2018-01-10	4	-0/+59
\|\ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / \|/\| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \|	Add specific setup inventory	Ricardo Carrillo Cruz	2017-12-28	4	-0/+59
\| \| \|_\|_\|_\|_\|/ / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	By default, Zuul uses runAnsibleSetup on all inventory nodes prior to running job playbooks. This translates to doing an 'ansible -m setup' against all nodes, but this won't work on nodes where Python is not available, like network nodes. This change adds a specific setup_inventory.yaml file, which will not contain nodes where setup module cannot work. Change-Id: Ieb02a19036854b8d9089bcd4cc9dd0b46e3ce2fc
* \| \| \| \| \| \| \|	Merge "Fix error handling for pidfile" into feature/zuulv3	Zuul	2018-01-10	1	-1/+2
\|\ \ \ \ \ \ \ \
\| * \| \| \| \| \| \| \|	Fix error handling for pidfile	Paul Belanger	2018-01-09	1	-1/+2
\| \| \|/ / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Zuul will now try to open the pidfile and raise an exception if there is a permission / issue with the pidfile path. Change-Id: I3276ffe2de0eeb99a4eb11ba7c0e12a62f1d9886 Signed-off-by: Paul Belanger <pabelanger@redhat.com>
* \| \| \| \| \| \| \|	Add alembic.ini	Tobias Henkel	2018-01-09	1	-0/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	When adding a new migration 'alembic revision' required an alembic.ini with at least the script location set. Change-Id: I4704f48229a000b8283802f74d302b53ac5cd080