summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add PS4 for better logfile information of devstack runs" into ↵6.2.2Jenkins2016-11-153-0/+9
|\ | | | | | | stable/newton
| * Add PS4 for better logfile information of devstack runsJohn L. Villalovos2016-11-113-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The scripts tools/ironic/scripts/cleanup-node.sh tools/ironic/scripts/create-node.sh tools/ironic/scripts/setup-network.sh are executed by sudo and the PS4 value disappears during sudo. Add a PS4 value so that we get better logs during their execution. PS4 is used to prefix the tracing output when using: 'set -x' or 'set -o xtrace' From the bash manpage: PS4 The value of this parameter is expanded as with PS1 and the value is printed before each command bash displays during an execution trace. The first character of PS4 is replicated multiple times, as necessary, to indicate multiple levels of indirection. The default is ``+ ''. Change-Id: If06f079bed16d9c84569594aa64259bf06f91a1e (cherry picked from commit a78097bf3fcbea81fc00cab10a76c709d2d3a1c2)
* | 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)
* | Avoid name errors in oneview periodicsVladyslav Drok2016-11-103-72/+181
|/ | | | | | | | | | | | | | | | | | | | | | | Solves a bug that can brake the periodic task execution in oneview drivers. The problem occurs when the method to check if a node is in use by oneview raise an exception. When that happens, the periodic task will break its execution, leaving the node with a wrong provision state and wrong maintenance mode too. The expected behavior when this occurs is skip processing the current node, iterate over the next node and process it. This patch changes the periodic task execution flow, ignoring the exception and processing the next node. In the next periodic task execution, the not processed node can be processed and updated accordingly. Co-Authored-By: Xavier <marcusrafael@lsd.ufcg.edu.br> Change-Id: I4b4b350d82d9e6863e8a4f334d2f9e7f7c192c6e Closes-Bug: 1629051 (cherry picked from commit 53521b6fd8bf01d8137b0a7cdb6504b3faf0b9ed)
* [install-guide] describe service clients authPavlo Shchelokovskyy2016-11-091-18/+68
| | | | | | | | | | | | | | | | | | Currently install guide incorrectly advises operators to configure access for service users used by ironic-conductor in [keystone_authtoken] section. Such way of configuration was deprecated in Newton release, and using it will result in a warning log message. This patch changes the advised way of configuring service users to an appropriate one using config sections corresponding to each service client to be used, as well as proper options for keystoneauth plugins and sessions instead of legacy options. Change-Id: I96967682f18988d9e32f2ecfa7792aa0e653db3f Closes-Bug: #1628842 (cherry picked from commit 492e67a64d058265683d695a07e47d5c75856cac)
* IPMI command should depend on console typeAndrey Shestakov2016-11-043-10/+93
| | | | | | | | | | This change implements _get_impi_cmd method for IPMI consoles. Depends on type of console, ipmi command should contain different arguments. Change-Id: I55e712a1a91aed3d533f636e519f0bae1f9be2d4 Closes-bug: #1611285 (cherry picked from commit 0acbf378bb543a3428856d80036d4de9c5d133a7)
* API: lookup() ignore malformed MAC addressesLucas Alvares Gomes2016-11-035-45/+54
| | | | | | | | | | | | | | | | | | This patch is adding code to ignore malformed MAC addresses which are called as part of the lookup() mechanism. Prior to this patch, if a MAC address wasn't in the expected format (six octets) the deployment of the node would fail because the ramdisk wouldn't be able to lookup which node it was in the Ironic database. One way to trigger this problem was to deploy a node with an Infiniband Card which the MAC address (or GID) contains 20 octets, that would result in a deployment failure even when that NIC wasn't used by Ironic at all (not enrolled as a port). The ListOfMacAdresses type was also deleted as part of this patch because it wasn't used anywhere else. Change-Id: I614fe63236985438d2f354d17a15d17649e72912 Closes-Bug: #1633585 (cherry picked from commit e0fd53d1830a76e8eb99be899b8302769b4fa649)
* Merge "Remove "dhcp" command from the iPXE template" into stable/newtonJenkins2016-11-025-8/+4
|\
| * Remove "dhcp" command from the iPXE templateDmitry Tantsur2016-11-025-8/+4
| | | | | | | | | | | | | | | | | | | | At this point we already have DHCP on the right NIC to be able to download this iPXE script. The "dhcp" may actually break the boot, as it runs DHCP on the first available interface, not the PXE booting one. Change-Id: I97dc897da49ceb8e03476ddacd5bc92dd40aa4b2 Closes-Bug: #1635191 (cherry picked from commit 42b2c60ce2aeb91a9fda213cda87a22f87bb0161)
* | Merge "Add a missing error check in ipmitool driver's reboot" into stable/newtonJenkins2016-11-023-3/+29
|\ \
| * | Add a missing error check in ipmitool driver's rebootNaohiro Tamura2016-11-023-3/+29
| |/ | | | | | | | | | | | | | | | | This patch adds a missing error check into ipmitool power driver's reboot so that the reboot can fail properly if power off failed. Change-Id: Icd061fe51555be3200b154c5e43e0f082864c93f Closes-bug: #1633992 (cherry picked from commit c073fb714df0477a625e2e4eee89edfee4a03576)
* | DRAC get_bios_config() passthru causes exceptionRichard Pioso2016-11-023-10/+12
|/ | | | | | | | | | | | | | | | | | | | | | The DRAC driver's (pxe_drac) get_bios_config() vendor passthru method raises an AttributeError exception. It no longer returns the current BIOS configuration. This is a regression from stable/mitaka. Triage found that get_bios_config() mistakenly treats the value returned by python-dracclient's list_bios_settings() as containing named tuples. When it calls the namedtuple _asdict() method, an AttributeError exception is raised. Revert get_bios_config()'s handling of the return value to use __dict__. Remove the comment that is not consistent with the implementation. Also revert the unit test case for a successful call to use mock to create the return value. Use mock.NonCallableMock with an empty specification (spec) to catch this type of bug in the future. Change-Id: I94afaa72a1ef25efc1b622e29e3a92a5d27f1892 Closes-Bug: #1637671 (cherry picked from commit 6d6cf18f3171f42af1567c65240a7a6a759377c2)
* Merge "Keep numbering of list in Install Guide" into stable/newtonJenkins2016-10-111-1/+1
|\
| * Keep numbering of list in Install GuideHironori Shiina2016-10-061-1/+1
| | | | | | | | | | | | | | | | This patch adds an indent to an internal link to keep numbering of an enumerated list in iPXE setup section. Change-Id: If64f85cff138ed70da074c03a1f54e86e8265058 (cherry picked from commit 602790a966a8954319050665ecd10a8fb69ba054)
* | Merge "[install-guide] describe pxe.ipxe_swift_tempurl" into stable/newtonJenkins2016-10-111-26/+74
|\ \ | |/
| * [install-guide] describe pxe.ipxe_swift_tempurlPavlo Shchelokovskyy2016-10-051-26/+74
| | | | | | | | | | | | | | | | Adds description and gotchas for this config option (new in Newton). Change-Id: I667ecee9a18f467b39b0d4e0ce2c8d654048aee8 Related-Bug: #1526404 (cherry picked from commit a48f317c3892512e887cd6bd8e7e778b8c643816)
* | Merge "Increase timeout for migration-related tests" into stable/newtonJenkins2016-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)
* | Merge "Change the logic of selecting image for tests" into stable/newton6.2.1Jenkins2016-09-301-0/+17
|\ \
| * | Change the logic of selecting image for testsVladyslav Drok2016-09-221-0/+17
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Right now devstack sets DEFAULT_IMAGE_NAME depending on the driver in stackrc. UUID of the image with this name is then written to tempest config during tempest configuration stage and is used to determine which image will be used during nova boot. As ironic python agent now supports both whole disk and partition images, we should change this logic, and the user should be able to request which image should be used in tests. This commit adds IRONIC_TEMPEST_WHOLE_DISK_IMAGE and IRONIC_IMAGE_NAME variables to provide this possibility. If IRONIC_TEMPEST_WHOLE_DISK_IMAGE is not set, we try to infer the image type by the size of the ephemeral partition requested, if it is 0, we consider the image to be whole disk. Change-Id: Ic718a6b37c09ae564ecc3913cf31234316f1da63 (cherry picked from commit e434225f9bec6f078f5514c82c9406131122a49e)
* | Merge "Update devstack IPA_DOWNLOAD_BRANCH to stable/newton" into stable/newtonJenkins2016-09-301-1/+1
|\ \
| * | Update devstack IPA_DOWNLOAD_BRANCH to stable/newtonJim Rollenhagen2016-09-221-1/+1
| |/ | | | | | | | | | | | | This ensures Newton CI uses ramdisks built from the Newton branch of IPA, rather than master. Change-Id: I75ac04402ecf5629760d0155dff01924acc0ffc5
* | Point documentation links to newton equivalentsMathieu Mitchell2016-09-2815-47/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following commands were used to produce the commit: find doc/source/ -name '*.rst' -exec gsed -i 's#/draft#/newton#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#docs.openstack.org/developer/ironic/#docs.openstack.org/developer/ironic/newton/#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#docs.openstack.org/developer/ironic-python-agent/#docs.openstack.org/developer/ironic-python-agent/newton/#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#pxe.vmlinuz#pxe-stable-newton.vmlinuz#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#oem.cpio#oem-stable-newton.cpio#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#<\(https://git.openstack.org/cgit.*\)>#<\1?stable%2Fnewton>#g' {} \; find install-guide/source/ -name '*.rst' -exec gsed -i 's#\([^<]\)\(https://git.openstack.org/cgit.*\)#\1\2?stable%2Fnewton#g' {} \; Additionally, a few links were manually updated. Change-Id: I2c4115f1038d106fc5e64b30c792f50e67fe1adf Closes-bug: #1612278
* | Fix absolute links to install-guide.rst in developer docsMathieu Mitchell2016-09-273-4/+4
| | | | | | | | | | | | | | | | | | | | | | Currently, the developer docs contains direct links to the install-guide. This commit fixes the links by pointing them directly to the new URL. Change-Id: Id8ba6b30c75cde9cfad01c4e4a6022683c829498 Partial-bug: #1612278 (cherry picked from commit 653a4ef65f6b8810dff538214f6c5592c8d329fa)
* | Update developer's guide "Installation Guide" linkMathieu Mitchell2016-09-271-1/+1
| | | | | | | | | | | | | | | | Point to the new Bare Metal service installation guide URL. Change-Id: I862628cbb73643fd87019e69897bda266cc39999 Partial-bug: #1612278 (cherry picked from commit fa7b21abca8e34346f7d1d8a150094818b8a476e)
* | Add link to new guide in old install guideMathieu Mitchell2016-09-272-62/+61
| | | | | | | | | | | | | | | | | | Additionally, reworded redirect instructions on the old page and removed the "work in progress" tag from the new installation guide. Change-Id: I0c5e4d06637627ab19c0c8a63e0056c9691a2cac Closes-bug: #1612278 (cherry picked from commit 53251b851bb4cdea2a2027f1e80ac56a062a510c)
* | [install-guide] Import "Setup the drivers for the Bare Metal service"Mathieu Mitchell2016-09-273-508/+520
| | | | | | | | | | | | Change-Id: Iaa16036547898f10ec8d3cda77ceae75fcfe27d8 Partial-bug: #1612278 (cherry picked from commit b9b8ce7a8cdc4d094720914e2912be1c38d58870)
* | [install-guide] Import "Trusted boot with partition image"Mathieu Mitchell2016-09-273-65/+76
| | | | | | | | | | | | | | | | Import Trusted boot under Advanced section. Change-Id: I33907dee5d6af49b8851761dc7a5d7b4bb3081c6 Partial-bug: #1612278 (cherry picked from commit bf926789c15d92544b90a0a348c91a9a0790297e)
* | [install-guide] Import "Building or downloading a deploy ramdisk image"Mathieu Mitchell2016-09-274-96/+104
| | | | | | | | | | | | Change-Id: I11cd67602b04c23cfe966cd2c2176ba01807c672 Partial-bug: #1612278 (cherry picked from commit f4a6dc72d579b0c450c84661db7eea9ec6bbffa1)
* | [install-guide] Import "Appending kernel parameters to boot instances"Mathieu Mitchell2016-09-273-59/+74
| | | | | | | | | | | | | | | | Import appending kernel parameters to advanced section. Change-Id: Id688a06ddcdcdfd7dae98e48381f92894825bff8 Partial-bug: #1612278 (cherry picked from commit 240ee8ffc9370e60a194908898738df7077d10ac)
* | [install-guide] Import configdriveMathieu Mitchell2016-09-273-109/+120
| | | | | | | | | | | | Change-Id: I2e93bfe565b5ef696832ca33dce1b00370147862 Partial-bug: #1612278 (cherry picked from commit 279dd758a1730e692d2b1ea603b775761a4ca7fb)
* | [install-guide] Import HTTPS, standalone and root device hintsMathieu Mitchell2016-09-277-264/+332
| | | | | | | | | | | | | | | | Import root device hints under "Advanced" section. Change-Id: I5a3002620c73568698d2ebe1af202001a8f074e1 Partial-bug: #1612278 (cherry picked from commit 6394c279ca298afc5887ac884df35bfa36c3dd9c)
* | [install-guide] Import "Enrollment" and "Troubleshooting" sectionsMathieu Mitchell2016-09-276-523/+535
| | | | | | | | | | | | Change-Id: Ie8b570d2a54c38a9d04976a0430fe0d963db57b0 Partial-bug: #1612278 (cherry picked from commit 0445213bbbaca44346e4d1160134ae1ac57bd049)
* | [install-guide] Import "Local boot with partition images"Mathieu Mitchell2016-09-274-49/+70
| | | | | | | | | | | | | | | | | | Create a new "Advanced features" section and import local boot under it. Change-Id: Icb24adf37a2874f6fd6fb446243dbe894e5cb63f Partial-bug: #1612278 (cherry picked from commit 123b5456da01af3244d352c9a49b1d7f6eda6372)
* | [install-guide] Import "Flavor creation"Mathieu Mitchell2016-09-273-26/+44
| | | | | | | | | | | | | | | | | | | | Import "Flavor creation" under a new "Create Compute flavors for use with the Bare Metal service" block inside of the "Integration with other OpenStack services" section. Change-Id: Ic37f43a34d577f99d9a1f08300a87c553c1b026f Partial-bug: #1612278 (cherry picked from commit a99968389148e0c71ddb59b7a9ccaa964c4ecb51)
* | [install-guide] Import "Image requirements"Mathieu Mitchell2016-09-273-94/+121
| | | | | | | | | | | | | | | | | | | | Import Image requirements under a new "Creating and adding images to the Image service" block in the "Configure integration with other OpenStack components" section. Change-Id: I503f8c60c7579e7ae5daf30032eb0b047ad40531 Partial-bug: #1612278 (cherry picked from commit 21e7fc3a689d0060e960162cc3b0e2f44f5a6c91)
* | [install-guide] Import "integration with other OpenStack components"Mathieu Mitchell2016-09-277-212/+279
| | | | | | | | | | | | | | | | | | | | | | | | | | The following sections of the legacy install guide were imported: * Configure Compute to use the Bare Metal service * Configure Networking to communicate with the bare metal server * Configure Tenant Networks * Configure the Bare Metal service for cleaning Change-Id: I5cf832401aba9499e9eec9bd50adcbedd356cb00 Partial-bug: #1612278 (cherry picked from commit fb027ec92410ee4f6055e6652c87e061a24a49d3)
* | [install-guide] Import Install and configure sectionsMathieu Mitchell2016-09-2716-431/+535
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following sections of the legacy install guide were imported: * Install and configure prerequisites * Install the Bare Metal service * Configure the Bare Metal service Only changes made to the documentation are the following: * Formatting changes (code-block, proper lists, etc.) * Structure changes (installation and configuration are now under a single section) * Heading titles formatting [0] [0] http://docs.openstack.org/contributor-guide/rst-conv/titles.html Change-Id: I0f1f5dacd4ed31ccfb4ac38c1ff2ecfc94e99ca4 Partial-bug: #1612278 (cherry picked from commit e26c71bcf6d25e5c73c6aeb0f9893e57d49f2abf)
* | [install-guide] Import "Bare Metal service overview"Mathieu Mitchell2016-09-277-68/+83
|/ | | | | | | | | | | | Import service overview in the new install-guide and provide a link in the old guide that points to it. Additionally, Swift was added to the list of interactions and Cinder was removed since it is not yet available. Change-Id: Ica6a9aae2c35f293228ffb75b2cf82478eb568a5 Partial-bug: #1612278 (cherry picked from commit 7131037cf7665680adee143a0f886e2c7ddd0327)
* Update UPPER_CONSTRAINTS_FILE for stable/newtonDavanum Srinivas2016-09-221-1/+1
| | | | Change-Id: I1d63f9b4e1af08b7b233c88125679a19b9200dd6
* Update .gitreview for stable/newtonDavanum Srinivas2016-09-221-0/+1
| | | | Change-Id: I970eba7690a56b95e6cad1ef26f4c09b74be374e
* Merge "Clean up release notes for 6.2.0"6.2.0stable/newtonJenkins2016-09-2116-45/+39
|\
| * Clean up release notes for 6.2.0Jim Rollenhagen2016-09-2116-45/+39
| | | | | | | | | | | | | | | | | | This is cleaned up as of commit 2351d624aed7aae9d58ce37676c0181687be0e8a, and should be sufficient for the final release. If something else with a release note lands, we can make any changes there in a later patch. Change-Id: Ia1ef442f3cc0d64c8a7c6fbb5aaf3f0160f214de
* | Merge "DRAC: list unfinished jobs"Jenkins2016-09-214-0/+71
|\ \
| * | DRAC: list unfinished jobsImre Farkas2016-09-214-0/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | After set_bios_config is invoked, there is no way to check the progress of the remote job on the DRAC card. list_unfinished_jobs on the vendor-passthru interface provides a way to check its status. Change-Id: I963504dbbec36e44312ccbf4455c45d6ec60908b Closes-Bug: #1621515
* | | Merge "Add a note about security groups in install guide"Jenkins2016-09-211-1/+15
|\ \ \
| * | | Add a note about security groups in install guideJim Rollenhagen2016-09-201-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a note that network security must be disabled, or certain ports must be allowed, for provisioning and cleaning networks. Closes-Bug: #1622727 Change-Id: I8415591d31209f8e3fbd9a4dcce30bd64bf8b24b
* | | | Merge "Sync ironic-lib.filters from ironic-lib"Jenkins2016-09-211-0/+2
|\ \ \ \
| * | | | Sync ironic-lib.filters from ironic-libDmitry Tantsur2016-09-211-0/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | For now we have to carry both in both trees, and they have gone out of sync. This changes updates the copy here with two new commands. Change-Id: Ife131cf825518012a3b04d2c7d713beeb4f8f770
* | | | Fix broken unit tests for get_ilo_objectDmitry Tantsur2016-09-211-3/+5
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First, the tested function signature was wrong. We didn't catch it in gate, as we mock proliantutils, but it does break e.g. Debian package build. Second, the arguments override was not actually working. We didn't catch it in gate, because the new values were the same as the defaults. Closes-Bug: #1626089 Change-Id: I2e4899e368b0b882dcd59bf33fdca98f47e5b405
View Lorry Controller Status