summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
5 daysMerge "CI: Add chart build gate"HEADmasterZuul
6 daysFix: flake8 tests failing due to invalid types comparisonRoman Gorshunov
Change-Id: I7266c0740652da40858a5a34cec73452ed0e10e6 Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 11 Feb 2019 17:43:32 +0000 Reviewed-on: https://review.openstack.org/636175 Project: openstack/airship-drydock Branch: refs/heads/master
2019-01-28CI: Add chart build gateDrew Walters
This commit introduces a chart build gate that triggers when changes are made to charts in the charts/ directory. Change-Id: I8c2c428302f3297109f37213b93c088321b13204 Notes (review): Code-Review+2: Bryan Strassner <strassner.bryan@gmail.com> Code-Review+1: Vladyslav Drok <vdrok@mirantis.com> Code-Review+2: Roman Gorshunov <roman.gorshunov@att.com> Workflow+1: Roman Gorshunov <roman.gorshunov@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 11 Feb 2019 18:24:40 +0000 Reviewed-on: https://review.openstack.org/633563 Project: openstack/airship-drydock Branch: refs/heads/master
2019-01-08Added line no to drydock message format for troubeshootingpd2839
Change-Id: I88c8cd6a56771cf6e97cfa874dc63963e247a890 Notes (review): Code-Review+1: Evgeniy L <eli@mirantis.com> Code-Review+1: Dustin Specker <dustin.specker@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+2: Pete Birley <petebirley@gmail.com> Workflow+1: Matt McEuen <matt.mceuen@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 08 Jan 2019 21:35:01 +0000 Reviewed-on: https://review.openstack.org/626367 Project: openstack/airship-drydock Branch: refs/heads/master
2019-01-04Remove the repetition words in readme.mdzhouxinyong
Change-Id: I79c024e432292aad0a0cbc1218699909d5af9432 Notes (review): Code-Review+1: Evgeniy L <eli@mirantis.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 04 Jan 2019 18:22:07 +0000 Reviewed-on: https://review.openstack.org/628418 Project: openstack/airship-drydock Branch: refs/heads/master
2018-12-26Merge "Validate mountpoint in a HostProfile"Zuul
2018-12-21Merge "Add Redfish as OOB driver"Zuul
2018-12-17Workaround MAAS race conditionScott Hussey
- If two threads concurrently update the power parameters of VMs on the same libvirt host, it seems to leave MAAS with broken state. Add locking in Drydock so that Drydock does not do this concurrently. Change-Id: I85575f4ba48152b4dc79c646871f33b69f845ab9 Notes (review): Code-Review+2: Kaspars Skels <kaspars.skels@gmail.com> Code-Review+2: Craig Anderson <craig.anderson@att.com> Workflow+1: Craig Anderson <craig.anderson@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 19 Dec 2018 20:54:08 +0000 Reviewed-on: https://review.openstack.org/625674 Project: openstack/airship-drydock Branch: refs/heads/master
2018-12-08Add Redfish as OOB driverHemanth Nakkina
This patch implements Refish as new OOB driver for Drydock. All the existing Drydock Orchestrator actions are implemented. Change-Id: I31d653fb41189a18c34cfafb0f490ca4f4d661b5 Notes (review): Code-Review+2: Roman Gorshunov <roman.gorshunov@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 21 Dec 2018 14:36:22 +0000 Reviewed-on: https://review.openstack.org/584786 Project: openstack/airship-drydock Branch: refs/heads/master
2018-12-04Merge "(fix) Use endpoint for MAAS URL"Zuul
2018-11-27(fix) Use endpoint for MAAS URLScott Hussey
- Instead of forcing a user to provide the full URL for the MAAS API, instead use the endpoints pattern and render the URI via HTK templates. - Add secret name to chart to support HTK ingress - Install libyaml to take advantage of faster parsing by pyyaml - Add exception logging when node compiling fails. - Add caching of parsed design to gain efficiency - Add TLS certificate secret for use by the ingress document Change-Id: I5a2dbc415483c336d38d67edcebdfc5812f7bb0c Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Mark Burnett <mark.m.burnett@gmail.com> Workflow+1: Mark Burnett <mark.m.burnett@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 04 Dec 2018 14:25:20 +0000 Reviewed-on: https://review.openstack.org/616709 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-14omit the twice occured words in orchestrator.pymelissaml
Change-Id: I50fb7b96cb0ee6bee670fd3b85007e99ef0aabc9 Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 14 Nov 2018 19:43:34 +0000 Reviewed-on: https://review.openstack.org/617920 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-13delete the duplicate words in networkLink3.yamlzhouxinyong
Change-Id: I21315ec670cc6f4d68e6dfef41e9676541cde47a Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 14:41:17 +0000 Reviewed-on: https://review.openstack.org/617408 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-13remove the repetition words in fullsite_networks.yamlzhouxinyong
Change-Id: I2abb2bed6dc418bcf23c7a0c1dbaed027763ca37 Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 14:41:17 +0000 Reviewed-on: https://review.openstack.org/617407 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-13remove the repetition words in multidoc.yamlzhouxinyong
Change-Id: Ida718c82ca57eb7b995019fc7aa7676201d74a04 Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 14:41:16 +0000 Reviewed-on: https://review.openstack.org/617406 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-13omit the twice occured words in fullsite.yamlzhouxinyong
Change-Id: I9c67d03f6745d30c035c8ab618ad956333042cef Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 14:41:16 +0000 Reviewed-on: https://review.openstack.org/617405 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-13delete the duplicate words in hostprof.yamlzhouxinyong
Change-Id: Iec982af9735116ae159330e462f8267499ee6df6 Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 13 Nov 2018 14:41:15 +0000 Reviewed-on: https://review.openstack.org/617404 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-08Merge "Fix: adding back the possibility to add arbitrary labels"Zuul
2018-11-08Merge "[Bug fix] Remove "clean" from "chart" target"Zuul
2018-11-08Fix: adding back the possibility to add arbitrary labelsRoman Gorshunov
Arbitrary labels could be added as `make` parameter `LABEL=`, which is used in att-comdev/cicd Jenkins pipelines. Sample parameter: 'LABEL=org.label-schema.vcs-url=${GERRIT_CHANGE_URL} \ --label org.label-schema.base-image=${base_sha256}' Sample usage: See Jenkinsfile files under images/ directory in att-comdev/cicd repo. In addition to that, if `COMMIT` variable is undefined when invoking `make`, we use result of `git rev-parse HEAD` command, which should output latest git commit ID. [0] https://github.com/att-comdev/cicd Change-Id: I9193ca03cb3970e3a3e1cb64ff643a4d24543636 Notes (review): Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 08 Nov 2018 20:10:03 +0000 Reviewed-on: https://review.openstack.org/613025 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-07Merge "(fix) Support node filter by rack name"Zuul
2018-11-05Fix: pep8 airship-drydock-omni-test fixRoman Gorshunov
Fixes for pep8 test. Change-Id: Id2b7a187ffe56a47a184314d1a19507a78f7d88a Notes (review): Code-Review+1: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 06 Nov 2018 19:33:13 +0000 Reviewed-on: https://review.openstack.org/615584 Project: openstack/airship-drydock Branch: refs/heads/master
2018-11-02[Bug fix] Remove "clean" from "chart" targetDimitrios Markou
In Makefile there is a "clean" target as prerequisite to "chart" target. That causes problems because it removes the "build" directory that was created at the begining of the file and the helm_install.sh script cannot write to the "build" directory in subsequent step [0]. [0]: https://github.com/openstack/airship-drydock/blob/master/tools/helm_install.sh#L30 Change-Id: Ic608630c38ff3ad7caf3a4d42e0b318d0d9d4d7e Signed-off-by: Dimitrios Markou <dm844v@att.com> Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Pete Birley <petebirley@gmail.com> Workflow+1: Pete Birley <petebirley@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Thu, 08 Nov 2018 18:24:55 +0000 Reviewed-on: https://review.openstack.org/615292 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-23Merge "Support multiple rack controllers"Zuul
2018-10-18Trivial typo fix in fullsite.xml commentskovaleff
Change-Id: Ia6a1a9a33a2a29f5d8715f505a502e3730c8d4f6 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 19 Oct 2018 05:08:49 +0000 Reviewed-on: https://review.openstack.org/611733 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-18Support multiple rack controllersScott Hussey
- Update the maasdriver to support two concurrent rack controllers when configuring networking. - Identify a baremetal node from both the MAAS node and rack controller collections - Relax validation to only require at least one healthy rack controller Change-Id: I04beb56a8212b65061840021b13b412fbb37ae81 Notes (review): Code-Review+2: Mark Burnett <mark.m.burnett@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+1: Serge Kovaleff <sk607s@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 23 Oct 2018 18:52:25 +0000 Reviewed-on: https://review.openstack.org/607086 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-17Validate mountpoint in a HostProfileDimitrios Markou
This patch fixes the issue that a reverted patch was causing [0]. In this patch we add a check for partitions which are not having mountpoints and propably was causing the problem in airship-seaworthy. [0] https://review.openstack.org/#/c/610594/ Story: #2002951 Change-Id: Ic3055ebdcf37461f52391cf564c29f8283637684 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com> Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Code-Review+2: Pete Birley <petebirley@gmail.com> Workflow+1: Pete Birley <petebirley@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 26 Dec 2018 15:28:40 +0000 Reviewed-on: https://review.openstack.org/611332 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-16(fix) Support node filter by rack nameScott Hussey
- Add unit tests for node filtering based on rack name, node tag and node label. - Code updates to get all unit tests passing. Change-Id: I79e7a8b35c5635364c37e22153b26687a684bfed Notes (review): Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 07 Nov 2018 09:28:15 +0000 Reviewed-on: https://review.openstack.org/611102 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-15Merge "Revert "Validate mountpoints in a HostProfile""Zuul
2018-10-15Revert "Validate mountpoints in a HostProfile"Scott Hussey
This reverts commit 1466835626f5cf7a2426404a4058789640fd3ad9. Change-Id: I9d9df1c5b8c9d905836d3d3c111e507f71bef0ce Notes (review): Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Code-Review+2: Kaspars Skels <kaspars.skels@gmail.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 15 Oct 2018 17:16:27 +0000 Reviewed-on: https://review.openstack.org/610594 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-13Merge "Fix typo in doc"Zuul
2018-10-12Fix typo in Site Topology docskovaleff
Change-Id: If62e95908e9141a23244bf11a606cdca078c9e10 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sat, 13 Oct 2018 03:17:29 +0000 Reviewed-on: https://review.openstack.org/610181 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-12Fix typo in docskovaleff
Change-Id: Icd9f860405de26d2199efca52b7f7cb67beee3e2 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Sat, 13 Oct 2018 03:24:16 +0000 Reviewed-on: https://review.openstack.org/610176 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-11Validate mountpoints in a HostProfileDimitrios Markou
Prevent the partitions and logical volumes to use the same mount point on the same baremetal node Story: #2002951 Change-Id: Ie936b5b20859d511e4cb858202c4ed50378bb72d Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com> Notes (review): Code-Review+1: Roman Gorshunov <roman.gorshunov@att.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 12 Oct 2018 19:31:41 +0000 Reviewed-on: https://review.openstack.org/607603 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-08Fix invalid comments and class names in unit testsDimitrios Markou
Change-Id: I68cb5c2b4b8e59d36d7d13c06da5bf27cbff5674 Signed-off-by: Dimitrios Markou <mardim@intracom-telecom.com> Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Felipe Monteiro <felipe.monteiro@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 08 Oct 2018 14:39:16 +0000 Reviewed-on: https://review.openstack.org/608635 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-05Merge "(fix) Fix two integration tests"Zuul
2018-10-05Merge "(fix) Register error for missing disk"Zuul
2018-10-05Merge "fix: Drydock exceptions documentation incorrectly rendering"Zuul
2018-10-05(fix) Register error for missing diskScott Hussey
- When a block device isn't found on a node the failure isn't registered as such Change-Id: Iaeab6d2e797e1d0c6089e26da9abcd1c60f2c5c5 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Workflow+1: Bryan Strassner <bryan.strassner@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 05 Oct 2018 21:12:39 +0000 Reviewed-on: https://review.openstack.org/608340 Project: openstack/airship-drydock Branch: refs/heads/master
2018-10-04Merge "Add release uuid to pods and rc objects (drydock)"Zuul
2018-10-03Merge "Fix: git commit id labels on images"Zuul
2018-10-02Merge "Catchup YAPF formatting"Zuul
2018-10-01Merge "Fix: various documentation and URL fixes"Zuul
2018-09-26Catchup YAPF formattingScott Hussey
Change-Id: Ic54f77b4b0bdd9199fbc10dfdfc43d3af8f0bfd1 Notes (review): Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 02 Oct 2018 20:52:13 +0000 Reviewed-on: https://review.openstack.org/605431 Project: openstack/airship-drydock Branch: refs/heads/master
2018-09-26(fix) Fix missing parameter for node pkg listScott Hussey
- When compiling a list of packages to be installed on deploying nodes the function call to render all bootactions was missing a parameter introduced in [0] to secure the bootaction. - Add unit testing for this flow Change-Id: I9f59819d6b79290a9f72db2a5a4c3422f2c79739 Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Workflow+1: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+1: Serge Kovaleff <sk607s@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Wed, 26 Sep 2018 21:12:24 +0000 Reviewed-on: https://review.openstack.org/605246 Project: openstack/airship-drydock Branch: refs/heads/master
2018-09-26Fix: various documentation and URL fixesRoman Gorshunov
1) UCP -> Airship 2) readthedocs.org -> readthedocs.io (there is redirect) 3) http -> https 4) attcomdev -> airshipit (repo on quay.io) 5) att-comdev -> openstack/airship-* (repo on github/openstack git) 6) many URLs have been verified and adjusted to be current 7) no need for 'en/latest/' path in URL of the RTD 8) added more info to some setup.cfg and setup.py files 9) ucp-integration docs are now in airship-in-a-bottle 10) various other minor fixes Change-Id: I64fb508a3ff72258d82c795193e918f2945deb29 Notes (review): Code-Review+2: Pete Birley <petebirley@gmail.com> Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Workflow+1: Aaron Sheffield <ajs@sheffieldfamily.net> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Mon, 01 Oct 2018 19:18:16 +0000 Reviewed-on: https://review.openstack.org/604428 Project: openstack/airship-drydock Branch: refs/heads/master
2018-09-25(fix) Fix two integration testsScott Hussey
- Two integration tests broke with the addition of the Kubernetes driver test. Fix them. Change-Id: I461be4dac08272b634c3921cde223df60cc40f21 Notes (review): Code-Review+2: Aaron Sheffield <ajs@sheffieldfamily.net> Code-Review+2: Felipe Monteiro <felipe.monteiro@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Fri, 05 Oct 2018 21:31:27 +0000 Reviewed-on: https://review.openstack.org/605297 Project: openstack/airship-drydock Branch: refs/heads/master
2018-09-25Merge "Roll pyghmi back to 1.0.18"Zuul
2018-09-25Roll pyghmi back to 1.0.18Scott Hussey
The attempted rollback to 1.0.44 still showed instability in controlling baremetal. This rolls all the way back to the version used prior to the upgrade. Change-Id: If8cb8580043575eb68510ca00a2cb19e881ae76e Notes (review): Code-Review+2: Matt McEuen <matt.mceuen@att.com> Code-Review+2: Mark Burnett <mark.m.burnett@gmail.com> Workflow+1: Mark Burnett <mark.m.burnett@gmail.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 25 Sep 2018 21:08:30 +0000 Reviewed-on: https://review.openstack.org/605106 Project: openstack/airship-drydock Branch: refs/heads/master
2018-09-25Update node-labels through Kubernetes ProvisionerSoumitra Khuntia
Blueprint: https://airshipit.readthedocs.io/projects/specs/en/latest/specs/approved/k8s_update_node_labels_workflow.html This commit adds: 1. A new task action ''relabel_nodes'' added to update nodes labels 2.A new Kubernetes driver added for Kubernetes cluster interaction through Promenade. Change-Id: I37c2d7bfda4966d907556036bc2b343df451994c Notes (review): Code-Review+1: Dan Crank <dan.no@att.com> Code-Review+2: Bryan Strassner <bryan.strassner@gmail.com> Code-Review+1: Ahmad Mahmoudi <am495p@att.com> Code-Review+2: Scott Hussey <sthussey@att.com> Workflow+1: Scott Hussey <sthussey@att.com> Verified+2: Zuul Submitted-by: Zuul Submitted-at: Tue, 25 Sep 2018 20:55:53 +0000 Reviewed-on: https://review.openstack.org/593609 Project: openstack/airship-drydock Branch: refs/heads/master