summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Pin version of virtualbmc in devstack pluginmitaka-eol5.1.3stable/mitakaPavlo Shchelokovskyy2017-03-241-1/+4
| | | | | | | | | | | | | | | Virtualbmc is absent from global requirements and upper constraints, but it depends on pyghmi which is in those requirements. With new releases of virtualbmc it might lead to newer version of virtualbmc installed together with old, unsupported by it version of pyghmi, breaking vbmc functionality. Until virtualbmc is introduced into g-r and u-c, let's pin the version of installed virtualbmc to the latest one that existed at the time when the stable branch was created (0.0.4 for stable/mitaka). Change-Id: Ic964f0dda043f927547962dd8fd78cb11b4f7064
* Prepare for using standard python testsAndreas Jaeger2017-02-032-0/+64
| | | | | | | | | | | | | | | | | | | Add simple script to setup mysql and postgresql databases, this script can be run by users during testing and will be run by CI systems for specific setup before running unit tests. This is exactly what is currently done by OpenStack CI in project-config. This allows to change in project-config the python-db jobs to python-jobs since python-jobs will call this script initially. See also http://lists.openstack.org/pipermail/openstack-dev/2016-November/107784.html Update devref for this. Needed-By: I58c1d37d620ca058a4c2127e329d8d21b391965f Change-Id: Id21d3481f7b4929c40c7351e6321d9ba5ad078b8 (cherry picked from commit dfc272c43da5ae6c0294197b2f39eb0f647c814c)
* Allow to set min,max API microversion in tempestVasyl Saienko2017-01-121-0/+13
| | | | | | | | | | | | | | | | | | | This patch introduce new devstack variables to set min and max Ironic API version to allow skip tempest tests. TEMPEST_BAREMETAL_MIN_MICROVERSION TEMPEST_BAREMETAL_MAX_MICROVERSION Set TEMPEST_BAREMETAL_MAX_MICROVERSION to 1.16 as it is latest API version for mitaka release. Conflicts: devstack/lib/ironic doc/source/dev/releasing.rst Related-Bug: #1655888 Change-Id: Id6d36280f622549a2877aab99c11e4008af67fe7 (cherry picked from commit 76b91709ae24490721f0d60f19eea5ea5ec71434)
* Move CONF.service_available.ironic to our pluginJim Rollenhagen2017-01-066-2/+15
| | | | | | | | | | | | | | | | | | | | This is unwanted in tempest's plugin, bring it here instead. Also add skip checks for our tests to skip the tests if ironic is not enabled. This allows the plugin to be installed in a tempest environment, without breaking if ironic isn't present. Last, drop 'baremetal' from the test.services decorators used, as it is no longer in tempest's service list, and we use the skip check mentioned above instead. Conflicts: ironic_tempest_plugin/tests/scenario/baremetal_manager.py ironic_tempest_plugin/tests/scenario/test_baremetal_multitenancy.py Change-Id: I0b7e32dfad2ed63f9dd4d7cad130da39bc869a8a Depends-On: Idede322190d57491d4f7e18090aa8c29b5b87353 (cherry picked from commit 8ad0a424b3e3963b9fdda0e1b64d8a065ef978d2)
* Turn off tempest's multitenant network testsJim Rollenhagen2016-12-211-0/+4
| | | | | | | | | This disables the multitenant networking tests that come from tempest's tree, but not the ones in ironic's tempest plugin. Change-Id: I62cc4ae3e9fe7dc63ec4373535fe6ae07084a381 (cherry picked from commit 0da98dee34f2b79fe092661ffc311717db739304) (cherry picked from commit 8d6561103013e91ee7238145f8b9c2e39523febc)
* Bring configurations from tempest to ironic_tempest_pluginThiago Paiva2016-11-155-36/+40
| | | | | | | | | To remove our tests from tempest first we need to bring the configurations that we need to the tempest plugin. Closes-Bug: #1614516 Change-Id: Iad6e16c28cf2a35cc6bf040f57224d32c085b7bc (cherry picked from commit ab6f73f7a06a9c9e497762e1dc917a9626cccd26)
* Merge "Fix updating port MAC address for active nodes" into stable/mitakaJenkins2016-10-115-5/+148
|\
| * Fix updating port MAC address for active nodesDmitry Tantsur2016-10-115-5/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current update fails with HTTP 500 when trying to update a bound Neutron port. Remove the binding before update and restore it during update. For nodes in "active" state (or with instance_uuid assigned) only allow such update in maintenance mode. This is not a breaking change, as previously it did not work at all. The cherry-pick has a small modification: changed neutron.get_client to _build_client as get_client appeared in Newton. Change-Id: I356fc0eb3702eb16b7f8e675ea416473c65af7e3 Closes-Bug: #1611744 (cherry picked from commit f4ff926ab161297d2281394689ea25a7ef10cbb1)
* | Update stable/mitaka driver requirement for iRMCNaohiro Tamura2016-10-113-2/+6
|/ | | | | | | | This patch updates python-scciclient required version number for iRMC driver to 0.3.1 which contains the bug fix '#1561852' and maintenance updates. Change-Id: I368c4ea4e07baabc3354b1b1cbd6eb304d5aa68f
* Make tox respect upper-constraints.txtJim Rollenhagen2016-10-111-3/+1
| | | | | | | | | | | | | | | | | | This will force pip install to use the upper-constraints.txt specified version of pip modules. When you don't do this, you are out on the bleeding edge and become unstable everytime some python library in the wide world changes in a way that you don't expect. Post jobs are not yet compatible with the upper-constraints file, so override the install_command there to skip upper-constraints. Blatantly cribbed from the equivalent trove patch: https://review.openstack.org/#/c/298850 Change-Id: I73aa8cf37adc36db2c56a8f29ea165775a83dc7f Closes-Bug: #1563038 (cherry picked from commit e27e7ed4a9452eb69f6c1af434adb5620706c227) (cherry picked from commit 6e775b82f4a9b82fdf72f82239d0e8d53add6ba2)
* Merge "Fix: Duplicated driver causes conductor to fail" into stable/mitakaJenkins2016-10-073-4/+30
|\
| * Fix: Duplicated driver causes conductor to failLucas Alvares Gomes2016-09-063-4/+30
| | | | | | | | | | | | | | | | | | | | This patch is fixing a problem which causes the conductor to get confused and error out on startup in case there's a duplicated entry in the enabled_drivers configuration option. Closes-Bug: #1561564 Change-Id: Ib72bcde9b32d3c0b4b237068aa53146f58ea10a2 (cherry picked from commit 420fe0c5abdf904d84ce40e7126ba13f39b15736)
* | Merge "Increase timeout for migration-related tests" into stable/mitakaJenkins2016-10-061-0/+12
|\ \
| * | Increase timeout for migration-related testsVladyslav Drok2016-09-231-0/+12
| | | | | | | | | | | | | | | | | | Closes-Bug: #1626612 Change-Id: I5836733d916cd04990efa7cf402cca4cfd8b7e24 (cherry picked from commit 8813fc54cd689ec1ea2384e3a9acdb0f4d7d8e8b)
* | | Updated from global requirementsOpenStack Proposal Bot2016-10-031-1/+1
|/ / | | | | | | Change-Id: I206d2c4d0bd6cbec63cf44b2fff743d9cff8db75
* | Merge "Add validation of 'ilo_deploy_iso' in deploy.validate()" into ↵Jenkins2016-09-223-0/+157
|\ \ | |/ |/| | | stable/mitaka
| * Add validation of 'ilo_deploy_iso' in deploy.validate()Shivanand Tendulker2016-06-273-0/+157
| | | | | | | | | | | | | | | | | | | | | | | | | | iLO virtual media based drivers use ISO image supplied as 'ilo_deploy_iso' in node's 'driver_info' to boot node during provisioning process. This parameter was not getting validated during deploy.validate() call. This change fixes the issue. Change-Id: I8fc4a89621e09281349f88c32ed77d24aa11355b Closes-Bug: #1592335 (cherry picked from commit 2eab1ee09ac525e086b59c43c2581b9087b5a257)
* | Merge "IPMINative: Check the boot mode when setting the boot device" into ↵Jenkins2016-08-293-4/+43
|\ \ | | | | | | | | | stable/mitaka
| * | IPMINative: Check the boot mode when setting the boot deviceLucas Alvares Gomes2016-08-113-4/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is making the ipminative drive to take in consideration the boot mode (UEFI or BIOS) of the node when changing its boot device order. Otherwise, nodes in UEFI will switch to legacy BIOS as part of the set_boot_device() call. Closes-Bug: #1612287 Change-Id: I6a7f522ae9c7d58f216a7d5dfedf067da0a0fc32 (cherry picked from commit bcfab9f8ba6c1fb8f31c6b1f861620ac98d638b0)
* | | IPMITool: Check the boot mode when setting the boot deviceLucas Alvares Gomes2016-08-113-4/+77
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch is fixing a problem where the ipmitool option "efiboot" wasn't passed as part of the command to set the boot device for UEFI nodes causing them to switch back to legacy BIOS. A FIXME note was left in the code. It seems that ipmitool has a problem setting the efiboot + persistent options at the same time (see bug #1611306). It seems multiple options ought to be specified together, and that works, except for efiboot. Therefore this patch is using raw bytes to change the boot device when persistent and uefi are specified, that way it will work for newer and older versions of the ipmitool utility. Closes-Bug: #1611306 Change-Id: I11456fb035fe41fe46f1ff04e4d56c9f85b9e19d (cherry picked from commit 6419cc2118e9d8c613156f6d4aeae5db46aa385d)
* | Merge "Updated from global requirements" into stable/mitakaJenkins2016-06-231-1/+1
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-06-101-1/+1
| | | | | | | | | | | | Change-Id: I1eff79cc7e3638740bd5573e7afe93ae10146dfd
* | | Merge "remove neutron stuff from devstack deb packages" into stable/mitakaJenkins2016-06-221-2/+0
|\ \ \ | |_|/ |/| |
| * | remove neutron stuff from devstack deb packagesdparalen2016-06-071-2/+0
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It seems we are enforcing certain dependencies to be always installed on behalf of other projects. The issue I'm facing is we unconditionally install openvswitch-datapath-dkms which isn't necessary for Ubuntu kernels >= 3.13. Even worse, this package isn't available on Ubuntu 16.4 LTS thus breaking devstack setup.sh on that platform. This patch removes Neutron dependencies as those are handled by devstack[1] and devstack neutron plugin[2] [1] https://github.com/openstack-dev/devstack/blob/master/files/debs/openvswitch#L3 [2] https://github.com/openstack-dev/devstack/blob/master/lib/neutron_plugins/ovs_base#L58 Closes-bug: #1580101 Change-Id: Iff0cf13499d9cb05f241f23759d5faec9a9c4a98 (cherry picked from commit 033dbb2d104c02569ceb87d91e09b9d7d68cf9d9)
* | Mask password on agent lookup according to policy5.1.2Devananda van der Veen2016-06-214-6/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We currently mask passwords in driver_info for all API responses, except for agent lookups. This is because driver_vendor_passthru just expects a dict to return as JSON in the response, and doesn't modify it at all. Change lookup to follow the defined policy when returning the node object in the response, the same way we do for other API responses. Co-authored-by: Jim Rollenhagen <jim@jimrollenhagen.com> Change-Id: Ib19d2f86d3527333e905fdbf1f09fc3dc8b8c5a7 Closes-Bug: #1572796 (cherry picked from commit 426a306fb580762e97ada04e1253dedd9b64d410)
* | Change port used for Ironic static http to 3928Jay Faulkner2016-06-161-1/+1
|/ | | | | | | | | | | | | | | | Due to recent infrastructure changes, infra began using port 8088. This caused a conflict with our apache config, causing services to not start. Infra is changing this port (Ie67bbba02dbf61a481f66001de3e0dede9448316) but it may take up to a week to take effect. Instead of having our gate broken for a week, I'm changing the default port. 3928 was not chosen at random; it's listed as the netboot-pxe service port in /etc/services. Change-Id: Ifdcf3c82fd9ee64c1548e47dba4c78c2347959e0 Co-Authored-By: John L. Villalovos <john.l.villalovos@intel.com> Closes-bug: 1590139 (cherry picked from commit 2898b622f7072995b0c778d2a762adba5afe75aa)
* Merge "tempest: start using get_configured_admin_credentials" into stable/mitaka5.1.1Jenkins2016-05-311-1/+1
|\
| * tempest: start using get_configured_admin_credentialsEmilien Macchi2016-05-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | Tempest removed get_configured_credentials in [1]. This patch aims to update the way we get admin credentials in Tempest client code. [1] https://github.com/openstack/tempest/commit/bc0a7a6d8022c7240f5f1f80c0c93b43e9037ad4 Change-Id: Ic97c011f8beec8df4253949285794be21b10f7cf (cherry picked from commit b8f227e62e7aac87266cae53093ccb3e68bc6d32)
* | Stop unit-testing processutils internalsDmitry Tantsur2016-05-311-54/+0
|/ | | | | | Change-Id: Ia0c469669417f7de7b35ccf517750b9ecaebbed0 Closes-Bug: #1583535 (cherry picked from commit 4f463617ac9274679185993f058741bd231d9dfc)
* Update compute_driver in documentationvsaienko2016-05-181-1/+1
| | | | | | | | | | | | Nova doesn't support long driver names for compute_driver. Replace long driver name 'nova.virt.ironic.IronicDriver' with short one 'ironic.IronicDriver'. Reference: https://review.openstack.org/309504/ Change-Id: I00713abd3c230a42380593ff39c238b448bc0732 (cherry picked from commit 682a0c2c983d99f1c305ce17db5b812ffe356fe1)
* Merge "Fix usage of rest_client expected_success() in tests" into stable/mitakaJenkins2016-05-171-6/+6
|\
| * Fix usage of rest_client expected_success() in testsghanshyam2016-05-171-6/+6
| | | | | | | | | | | | | | | | | | | | rest_client expected_success() method expect read_code must be int and states the same in doc string . Tempest is converting that to error instead of false pass. Details: I3f4c58bdbb172805514831103927d3464d65d7f3 Change-Id: Ic4b0af5701df99621e3abb693644c4824c92dc4c (cherry picked from commit e8dda1db0ef72da1d98bf238a6da986729a91e41)
* | Merge "Updated from global requirements" into stable/mitakaJenkins2016-05-161-1/+1
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-05-151-1/+1
| | | | | | | | | | | | Change-Id: Ie3313ff329cb7580c1be0ee21226618943dbc833
* | | Merge "Enable download of tinyipa prebuilt image" into stable/mitakaJenkins2016-05-161-2/+14
|\ \ \
| * | | Enable download of tinyipa prebuilt imageSam Betts2016-05-111-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now there is a post job to build tinyipa and publish it to tarballs.openstack.org, we can start to use it in the ironic devstack script to prevent the need to build the image every time. This change is cherry picked from 2 commits because the original change had a follow up patch to fix a mistake in the tinyIPA file name. Change-Id: I641a5afe8318ec0848395b2b70e056c448f3dd00 (cherry picked from commit c65af51e5b4a5ebd2824250e13ec95ca1aadfe42) (cherry picked from commit b0c09844e19904bf6b8a5b78a47bdc8df0377d8e)
* | | | Merge "use stable/mitaka upper-constraints" into stable/mitakaJenkins2016-05-161-1/+1
|\ \ \ \
| * | | | use stable/mitaka upper-constraintsMark McDonagh2016-05-041-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | Move from master upper-constraints to stable/mitaka upper-constraints Change-Id: I4d15f6a3ed0ee54550ab94d9fae781795aaa5268
* | | | pull mitaka release notes from the mitaka branchDoug Hellmann2016-05-131-1/+1
| |/ / |/| | | | | | | | | | | Change-Id: I038d4b6df7ae2d7f6e3b75baf514a50d13343fc7 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | | Update tempest compute flavor_ref/flavor_ref_altvsaienko2016-05-102-0/+11
|/ / | | | | | | | | | | | | | | | | | | | | | | After Ironic is moved to devstack plugin the tempest is always executed before ironic devstack plugin. Unfortunatly there is no quick way to change order. This change introduce new function ironic_configure_tempest that is called from Ironic's devstack plugin. It updated flavor_ref and flavor_ref_alt compute tempest options. Change-Id: I85a77dd2193bac0ea4bd4fc1985c82d9f598b06b (cherry picked from commit ba15202d99b2c26ba3eaf319f1e3d20edb697483)
* | Merge "Updated from global requirements" into stable/mitakaJenkins2016-05-021-1/+1
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-04-291-1/+1
| |/ | | | | | | Change-Id: I7eac6538df72d4df71095ac200439304be8363e7
* | Merge "Force iRMC vmedia boot from remotely connected CD/DVD" into stable/mitakaJenkins2016-05-013-78/+153
|\ \ | |/ |/|
| * Force iRMC vmedia boot from remotely connected CD/DVDNaohiro Tamura2016-04-133-78/+153
| | | | | | | | | | | | | | | | | | | | This patch forces iRMC vmedia boot from remotely connected (redirected) CD/DVD using 'ipmitool raw' command instead of 'ipmitool chassis bootdev cdrom'. Closes-Bug: #1561852 Change-Id: I606e4d3a630ddc4eed071773afcb5274ee64a439 (cherry picked from commit 03fe93abfe5779f27c0eee5495f189e0104a5184)
* | Updated from global requirementsOpenStack Proposal Bot2016-04-181-1/+1
|/ | | | Change-Id: I544c652a4dd8dc6e12f9f4b6794aa6d520562ccd
* Normalize MAC OctetString to fix InvalidMAC exceptionNaohiro Tamura2016-04-073-4/+8
| | | | | | | | | This patch adds a step of MAC OctetString normalization to fix InvalidMAC exception in iRMC inspection module. Closes-Bug: #1564190 Change-Id: Ic2a5aaf2beadf06d69864ac7ea0d7a9aadebdc83 (cherry picked from commit df06a848f12d2d4712fc2fc59976d0ddc44dbddc)
* Allow fetching IPA ramdisk with branch nameJim Rollenhagen2016-04-011-2/+5
| | | | | | | | | | | | | | | | This adds branch name to the default URLs for IPA images, ensuring that devstack runs will use the correct branch of IPA for tests. It introduces a variable IPA_DOWNLOAD_BRANCH to accomplish this, which defaults to master. This default will be set to the branch name on stable branches, e.g. stable/mitaka. It is modified to match our URL scheme for agent images, e.g. stable-mitaka. This cherry-pick is modified to set the default to stable/mitaka. Change-Id: I3ec6dafb9522b5fe79e0c6681c2578d2a723380a Closes-Bug: #1561756 (cherry picked from commit 8e15d6a1388f49026c4d35ba87219a49ccc3c134)
* Update .gitreview for stable/mitakaThierry Carrez2016-03-231-0/+1
| | | | Change-Id: I03f82e1053f38f6d5f6ec12d21aaa7922a744cd0
* Documentation update for partition image support5.1.0Shivanand Tendulker2016-03-232-12/+9
| | | | | | | | | Removed note that indicated agent drivers do not support deploying partition images. Updated iLO documentation indicating support for partition images for agent_ilo driver. Change-Id: I2349604344328222db5b981fa9d75523d21a5186
* Merge "Devstack: add check of chassis creating"Jenkins2016-03-231-0/+1
|\
View Lorry Controller Status