summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
8 daysProperly handle TaskManagerStopped exceptionHEADmasterPaul Belanger
When we lose a task manager, we won't be able to create an instances. Rather then continue to look until retries limit is reached, we raise an exception early. In the case of below, the retry limit is very high and results in logs being spammed with the following: 2019-02-12 16:41:15,628 ERROR nodepool.NodeLauncher-0001616109: Request 200-0000443406: Launch attempt 39047/999999999 failed for node 0001616109: Traceback (most recent call last): File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/nodepool/driver/openstack/handler.py", line 241, in launch self._launchNode() File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/nodepool/driver/openstack/handler.py", line 142, in _launchNode instance_properties=self.label.instance_properties) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/nodepool/driver/openstack/provider.py", line 340, in createServer return self._client.create_server(wait=False, **create_args) File "<decorator-gen-32>", line 2, in create_server File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/openstack/cloud/_utils.py", line 377, in func_wrapper return func(*args, **kwargs) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/openstack/cloud/openstackcloud.py", line 7020, in create_server self.compute.post(endpoint, json=server_json)) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/keystoneauth1/adapter.py", line 357, in post return self.request(url, 'POST', **kwargs) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/openstack/_adapter.py", line 154, in request **kwargs) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/openstack/task_manager.py", line 219, in submit_function return self.submit_task(task) File "/opt/rh/rh-python35/root/usr/lib/python3.5/site-packages/openstack/task_manager.py", line 185, in submit_task name=self.name)) openstack.exceptions.TaskManagerStopped: TaskManager rdo-cloud-tripleo is no longer running Change-Id: I5f907d19ec1e637defe90eb944f4e5bd759e8a74 Signed-off-by: Paul Belanger <pabelanger@redhat.com> Notes (review): Code-Review+2: Clark Boylan <cboylan@sapwetik.org> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+1: Fabien Boucher <fboucher@redhat.com> Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Workflow+1: David Shrewsbury <dshrewsb@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 19 Feb 2019 15:26:03 +0000 Reviewed-on: https://review.openstack.org/636393 Project: openstack-infra/nodepool Branch: refs/heads/master
9 daysMerge "Rename aws flavor-name to instance-type"Zuul
12 daysbindep: add sudoMohammed Naser
sudo is needed by diskimage-builder so it would be good to add it so that the Docker image has that included by default. Change-Id: Ic23fa24fb343fb2b93d6055de1d4c27879b5dcac Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: Ian Wienand <iwienand@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 11 Feb 2019 03:54:42 +0000 Reviewed-on: https://review.openstack.org/635876 Project: openstack-infra/nodepool Branch: refs/heads/master
14 daysRename aws flavor-name to instance-typeMonty Taylor
The AWS term is instance-type, not flavor-name. Rename this while we don't have a huge userbase. While we're in there, rename a variable from image_name to image_id since we use image_id everywhere else. Change-Id: I1f7f16d2873982626d2434cf5ca1f6280adf739c Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Workflow+1: David Shrewsbury <dshrewsb@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 11 Feb 2019 18:28:48 +0000 Reviewed-on: https://review.openstack.org/635153 Project: openstack-infra/nodepool Branch: refs/heads/master
14 daysAdd documentation for AWS/EC2 driverClint Byrum
Documentation, a thing which illuminates, the cloud, and nodepool. Change-Id: I98ec01d188a3ee18175fe3ebc97d89668ec81a20 Notes (review): Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Code-Review+1: Clint 'SpamapS' Byrum <clint@fewbar.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: Tobias Henkel <tobias.henkel@bmw.de> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 06 Feb 2019 17:48:05 +0000 Reviewed-on: https://review.openstack.org/634769 Project: openstack-infra/nodepool Branch: refs/heads/master
14 daysMerge "Remove unused fields from AWS driver"Zuul
14 daysMerge "Revert "Revert "Add a timeout for the image build"""Zuul
2019-02-04Remove unused fields from AWS driverClint Byrum
These fields were cargo-culted from another provider, but they are not used in the actual AWS driver, and thus should invalidate the configuration if specified. Change-Id: Ic12b47eea1ab4b49e8c6746749f0a6d4fb322435 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 06 Feb 2019 13:19:57 +0000 Reviewed-on: https://review.openstack.org/634768 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-02-04Merge "EC2: Do not tag an instance until it is running"Zuul
2019-02-04Merge "Improve connection timeout log message"Zuul
2019-02-04EC2: Do not tag an instance until it is runningClint Byrum
Sometimes an instance stays as a reservation for a long time, sometimes it's only a reservation for a split second. Before this change, the tag would usually work, because the reservation turned to an instance fast enough. But many times we've observed the instance conversion taking 5 - 10 seconds, and this is long enough that the tag operation fails, and then nodepool loses track of the instance entirely. Change-Id: Iaa5c230976625d8a5c5afd7970691e235f0f77f7 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: Tobias Henkel <tobias.henkel@bmw.de> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 04 Feb 2019 19:23:09 +0000 Reviewed-on: https://review.openstack.org/634740 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-02-04Merge "Amazon EC2 driver"Zuul
2019-01-31Revert "Revert "Add a timeout for the image build""David Shrewsbury
This reverts commit ccf40a462a4805c7162bc34434cfe718cd96c448. The previous version would not work properly when daemonized because there was no stdout. This version maintains stdout and uses select/poll with non-blocking stdout to capture the output to a log file. Depends-On: https://review.openstack.org/634266 Change-Id: I7f0617b91e071294fe6051d14475ead1d7df56b7 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 06 Feb 2019 13:16:06 +0000 Reviewed-on: https://review.openstack.org/633792 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-28Amazon EC2 driverTristan Cacqueray
This change adds an experimental AWS driver. It lacks some of the deeper features of the openstack driver, such as quota management and image building, but is highly functional for running tests on a static AMI. Note that the test base had to be refactored to allow fixtures to be customized in a more flexible way. Change-Id: I313f9da435dfeb35591e37ad0bec921c8b5bc2b5 Co-Authored-By: Tristan Cacqueray <tdecacqu@redhat.com> Co-Authored-By: David Moreau-Simard <dmsimard@redhat.com> Co-AUthored-By: Clint Byrum <clint@fewbar.com> Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Workflow+1: David Shrewsbury <dshrewsb@redhat.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 04 Feb 2019 16:13:05 +0000 Reviewed-on: https://review.openstack.org/535558 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-27Merge "Stop building an explicit nodepool-base image"Zuul
2019-01-27Merge "Build images with Dockerfile instead of pbrx"Zuul
2019-01-25Merge "Revert "Add a timeout for the image build""Zuul
2019-01-25Revert "Add a timeout for the image build"David Shrewsbury
This reverts commit 7225354ec04f0466d95b6d68e51002723d6529c9. The disk-image-create command does not appear to be starting. Change-Id: I81abe25a253a385cae08a57561129a678546f18f Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 25 Jan 2019 22:37:34 +0000 Reviewed-on: https://review.openstack.org/633252 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-24Merge "Revert "Revert "Cleanup down ports"""Zuul
2019-01-24Merge "Add a timeout for the image build"Zuul
2019-01-24Stop building an explicit nodepool-base imageMonty Taylor
We have a utility image that we use for running the nodepool command that doens't have any additional software installed. Although it does set a COMMAND of /usr/local/bin/nodepool, it could still be useful as a general base image for other people if they wanted such a thing. Change-Id: I894e3d2dbe3cd2017f27ccc5e6fe298e9c9abd03 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sun, 27 Jan 2019 09:33:05 +0000 Reviewed-on: https://review.openstack.org/632188 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-24Build images with Dockerfile instead of pbrxJames E. Blair
Use the opendevorg/python-builder image to build nodepool images with a Dockerfile and multi-stage builds. bindep wasn't installing gcc for dpkg. Remove the platform restrictions for it. Change-Id: I0282b75ffad3d0ae1b589381010a3d4273fceb07 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sun, 27 Jan 2019 09:33:00 +0000 Reviewed-on: https://review.openstack.org/632187 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-23Add a timeout for the image buildDavid Shrewsbury
A builder thread can wedge if the build process wedges. Add a timeout to the subprocess. Since it was the call to readline() that would block, we change the process to have DIB write directly to the log. This allows us to set a timeout in the Popen.wait() call. And we kill the dib subprocess, as well. The timeout value can be controlled in the diskimage configuration and defaults to 8 hours. Change-Id: I188e8a74dc39b55a4b50ade5c1a96832fea76a7d Notes (review): Code-Review+2: James E. Blair <corvus@inaugust.com> Code-Review+2: Ian Wienand <iwienand@redhat.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 24 Jan 2019 16:15:32 +0000 Reviewed-on: https://review.openstack.org/629923 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-23Merge "Support userdata for instances in openstack"Zuul
2019-01-23Improve connection timeout log messageTobias Henkel
The log message doesn't contain the target ip address which can be crucial. Change-Id: Iff674a56267f416114b6bfd6203f3ac76bb5d569 Notes (review): Code-Review+2: David Shrewsbury <dshrewsb@redhat.com> Code-Review+2: Clint 'SpamapS' Byrum <clint@fewbar.com> Workflow+1: Clint 'SpamapS' Byrum <clint@fewbar.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 04 Feb 2019 18:35:22 +0000 Reviewed-on: https://review.openstack.org/632704 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-22Merge "Add dogpile.cache master to the -src tests"Zuul
2019-01-22Support userdata for instances in openstackSagi Shnaidman
Use "userdata" from Nova API to pass cloud-init config to nova instances in openstack. Change-Id: I1c6a1cbc5377d268901210631a376ca26f4887d8 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+1: Jesse Pretorius (odyssey4me) <jesse.pretorius@rackspace.co.uk> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 23 Jan 2019 14:36:19 +0000 Reviewed-on: https://review.openstack.org/630649 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-18Merge "Fix OpenShift driver typo"Zuul
2019-01-18Revert "Revert "Cleanup down ports""Ian Wienand
This reverts commit 7e1b8a7261d31208d853ecfa255da648ba54b656. openstacksdk >=0.19.0 fixes the filtering problems leading to all ports being deleted. However openstacksdk <0.21.0 has problems with dogpile.cache so use 0.21.0 as a minimum. Change-Id: Id642d074cbb645ced5342dda4a1c89987c91a8fc Notes (review): Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 24 Jan 2019 16:15:33 +0000 Reviewed-on: https://review.openstack.org/614370 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-17Merge "Implement an OpenShift resource provider"Zuul
2019-01-17Fix OpenShift driver typoTristan Cacqueray
This is a follow-up to fix typo introduced by: https://review.openstack.org/570667 Change-Id: I7178860f2b7a47cf58e96ee1d2ea32ceeeab5a18 Notes (review): Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: Tobias Henkel <tobias.henkel@bmw.de> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 18 Jan 2019 18:43:08 +0000 Reviewed-on: https://review.openstack.org/631391 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-15Add dogpile.cache master to the -src testsIan Wienand
This installs dogpile.cache from upstream master for the -src job. We recently saw breakage with openstacksdk and new versions of dogpile.cache, this would help catch this in the future as both will be installed from master. Depends-On: https://review.openstack.org/625467 Depends-On: https://review.openstack.org/626075 Change-Id: I7ac90804504ff37b72d089cf0cd2c7c48858da19 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+1: Yongjun Bai <baiyj666@gmail.com> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Workflow+1: David Shrewsbury <shrewsbury.dave@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 22 Jan 2019 21:15:25 +0000 Reviewed-on: https://review.openstack.org/625457 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-14Document default values of runtime argumentsTobias Henkel
Argparse can output the default values of the runtime arguments. Enable it so we have them documented. Change-Id: I4fd7f21546615bd249485707521d8222fab10962 Notes (review): Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Clint 'SpamapS' Byrum <clint@fewbar.com> Workflow+1: Clint 'SpamapS' Byrum <clint@fewbar.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 16 Jan 2019 02:04:55 +0000 Reviewed-on: https://review.openstack.org/630679 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-10Implement an OpenShift resource providerTristan Cacqueray
This change implements an OpenShift resource provider. The driver currently supports project request and pod request to enable both containers as machine and native containers workflow. Depends-On: https://review.openstack.org/608610 Change-Id: Id3770f2b22b80c2e3666b9ae5e1b2fc8092ed67c Notes (review): Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: David Shrewsbury <shrewsbury.dave@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 17 Jan 2019 15:39:06 +0000 Reviewed-on: https://review.openstack.org/570667 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-10print-zk: convert zookeeper_servers to a listTristan Cacqueray
The print-zk script needs to convert the dictionary iterator to a proper list. Change-Id: I786799ff881c361071fa06c78541c617fc7461a9 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Workflow+1: David Shrewsbury <shrewsbury.dave@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 10 Jan 2019 15:00:43 +0000 Reviewed-on: https://review.openstack.org/629754 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-09Merge "Extract common config parsing for ProviderConfig"Zuul
2019-01-09Merge "Extract out common config parsing for ConfigPool"Zuul
2019-01-09Default host_keys to empty list in static driverTobias Henkel
Everywhere in nodepool we default the host_keys to an empty list and zuul relies on this behavior. However the static driver sets this to None for non-ssh connections. Change this to return an empty list to match the behavior of the other drivers. Change-Id: I3f14803c35a2039d2c53f4939857a531ce900097 Notes (review): Code-Review+2: Jens Harbott (frickler) <j.harbott@x-ion.de> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Workflow+1: David Shrewsbury <shrewsbury.dave@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 09 Jan 2019 14:34:12 +0000 Reviewed-on: https://review.openstack.org/629478 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-09Merge "Add Fedora 29 testing"Zuul
2019-01-07Extract common config parsing for ProviderConfigDavid Shrewsbury
Adds a ProviderConfig class method that can be called to get the config schema for the common config options in a Provider. Drivers are modified to call this method. Change-Id: Ib67256dddc06d13eb7683226edaa8c8c10a73326 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 09 Jan 2019 21:41:32 +0000 Reviewed-on: https://review.openstack.org/625094 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-03Extract out common config parsing for ConfigPoolDavid Shrewsbury
Our driver code is in a less-than-ideal situation where each driver is responsible for parsing config options that are common to all drivers. This change begins to correct that, starting with ConfigPool. It changes the driver API in the following ways: 1) Forces objects derived from ConfigPool to implement a load() method that should call super's method, then handle loading driver specific options from the config. 2) Adds a ConfigPool class method that can be called to get the config schema for the common config options leaving drivers to have to only define the schema for their own config options. Other base config objects will be modeled after this pattern in later changes. Change-Id: I41620590c355cacd2c4fbe6916acfe80f20e3216 Notes (review): Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: James E. Blair <corvus@inaugust.com> Workflow+1: James E. Blair <corvus@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 09 Jan 2019 18:14:30 +0000 Reviewed-on: https://review.openstack.org/621642 Project: openstack-infra/nodepool Branch: refs/heads/master
2019-01-02Merge "Ignore removed provider in _cleanupLeakedInstances"3.4.0Zuul
2018-12-20Merge "Add unmanaged vm to nodepool functional tests"Zuul
2018-12-20Merge "Trim devstack services used in testing"Zuul
2018-12-19Add unmanaged vm to nodepool functional testsTobias Henkel
In order to increase test coverage we should add an unmanaged vm to the functional tests. Change-Id: I3908c74ea6e581d61180e1b238e4d0e869751cf3 Depends-On: https://review.openstack.org/625923 Notes (review): Code-Review+2: Monty Taylor <mordred@inaugust.com> Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Workflow+1: David Shrewsbury <shrewsbury.dave@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 20 Dec 2018 15:32:37 +0000 Reviewed-on: https://review.openstack.org/626357 Project: openstack-infra/nodepool Branch: refs/heads/master
2018-12-19Trim devstack services used in testingClark Boylan
Nodepool doesn't need metadata service access (at least not as currently tested) and doesn't need novnc console access. Remove these services as they shouldn't be needed. Note that etcd was also considered for removal as only cinder uses it today aiui. However, openstack services may start depending on it at any time. We don't disable it to avoid breaking devstack in the potential future. Change-Id: I48eb81becf5ffb0f79340bdce117de15d9c94b12 Notes (review): Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 20 Dec 2018 14:49:38 +0000 Reviewed-on: https://review.openstack.org/626044 Project: openstack-infra/nodepool Branch: refs/heads/master
2018-12-18Run devstack zookeeper on tmpfsClark Boylan
We've seen connection losses to zookeeper during functional testing on some cloud regions. Move zookeepers data dir to tmpfs to avoid io to the disk costs. Change-Id: If45d5cd7a2b63dbe7f20e7ef528b65ba41af7117 Notes (review): Code-Review+2: Ian Wienand <iwienand@redhat.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: Tobias Henkel <tobias.henkel@bmw.de> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 19 Dec 2018 05:46:29 +0000 Reviewed-on: https://review.openstack.org/626038 Project: openstack-infra/nodepool Branch: refs/heads/master
2018-12-17Switch devstack jobs to XenialIan Wienand
Change I8749ed24d5f451d29f767ebb2761abd743b7d306 modified the devstack based jobs to run on Bionic. Unfortunately we're not quite ready for that; one issue is that our devstack dependencies require zypper for opensuse which is not on bionic [1]. diskimage-builder excludes zypper on bionic in bindep, but we don't have a mechanism to use that (yet [2]). For now, switch them back to Xenial to retain the status quo. We can then take a more controlled approach to work on modernising them. [1] https://bugs.launchpad.net/ubuntu/+source/zypper/+bug/1808230 [2] https://review.openstack.org/624852 Depends-On: https://review.openstack.org/625596 Change-Id: I56646e49264dd844f5818a84e04965863542f572 Notes (review): Code-Review+2: Jens Harbott (frickler) <j.harbott@x-ion.de> Code-Review+2: Monty Taylor <mordred@inaugust.com> Workflow+1: Monty Taylor <mordred@inaugust.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 17 Dec 2018 17:44:13 +0000 Reviewed-on: https://review.openstack.org/624855 Project: openstack-infra/nodepool Branch: refs/heads/master
2018-12-11Merge "Fix race in test_handler_poll_session_expired"Zuul
2018-12-06Include host_id for openstack providerPaul Belanger
When we create a server using the openstack provider, in certain conditions it is helpful to collect the host_id value. As an example, Zuul will pass through this information into a job inventory file which will allow an operator to better profile where jobs run with in a cloud. This can be helpful trying to debug random jobs timing out within a cloud. Change-Id: If29397e67a470462561f24c746375b8291ac43ab Signed-off-by: Paul Belanger <pabelanger@redhat.com> Notes (review): Code-Review+2: David Shrewsbury <shrewsbury.dave@gmail.com> Code-Review+2: Tobias Henkel <tobias.henkel@bmw.de> Workflow+1: Tobias Henkel <tobias.henkel@bmw.de> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 07 Dec 2018 14:10:48 +0000 Reviewed-on: https://review.openstack.org/623107 Project: openstack-infra/nodepool Branch: refs/heads/master