summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add a section to redfish doc for BIOS registry fields"Zuul2021-07-192-14/+33
|\
| * Add a section to redfish doc for BIOS registry fieldsBob Fournier2021-07-132-14/+33
| | | | | | | | | | | | | | Move the descriptions of the BIOS registry fields in the API to a new section in the Redfish documentation. Change-Id: Ifedb4526113bb84ccc598e13f5767b71a60f5532
* | Merge "Fix Redfish RAID interface_type physical disk hint"Zuul2021-07-146-3/+168
|\ \
| * | Fix Redfish RAID interface_type physical disk hintAija Jauntēva2021-07-026-3/+168
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Map between sushy's protocol and RAID config schema's interface_type constants. Not supporting `scsi` from RAID schema as Redfish does not support it in Protocol property. Change-Id: Ic042f0a87ff0723f313adb3888c24c5963624182
* | | Merge "Sanity check object version changes"Zuul2021-07-131-3/+43
|\ \ \
| * | | Sanity check object version changesDmitry Tantsur2021-07-131-3/+43
| | | | | | | | | | | | | | | | | | | | | | | | Make sure there are no gaps in release and object versions. Change-Id: Ib177a10ee5cc208494a331b0d080ad40c4a09aae
* | | | Merge "Fix iPXE docs: snponly is not always available"Zuul2021-07-131-2/+5
|\ \ \ \
| * | | | Fix iPXE docs: snponly is not always availableDmitry Tantsur2021-07-131-2/+5
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Actually, it's only available on RHEL/CentOS 8, but I hope the other distributions will catch up. Change-Id: I53314b8f16fd7b965c58370e33ab83501e7cb067
* | | | Update requirementsRiccardo Pittau2021-07-132-8/+8
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update minimum required versions of python packages. The updates is based on crosscheck requirements from required packages: oslo-log 4.3.0 depends on oslo.context>=2.20.0 oslo-config 6.8.0 depends on rfc3986>=1.2.0 oslo-config 6.8.0 depends on requests>=2.18.0 oslo-log 4.3.0 depends on pbr>=3.1.1 oslo-log 4.3.0 depends on oslo.serialization>=2.25.0 oslo-policy 3.7.0 depends on oslo.context>=2.22.0 Also update minimum versions of ddt and stestr to avoid compatibility issues with recent Python versions (3.8 and higher). Change-Id: I34b558f4e7b93618130af1ba2250f6ab8575d4d6
* | | Merge "Bring boot_iso/deploy_iso handling in iLO closer to Redfish"Zuul2021-07-135-92/+106
|\ \ \
| * | | Bring boot_iso/deploy_iso handling in iLO closer to RedfishDmitry Tantsur2021-07-075-92/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The iLO version is, as far as I remember, the oldest implementation of virtual media in Ironic. Since then a few cool feature have been developed in this area, but they are limited to the code using image_utils (e.g. Redfish). The iLO boot interface only uses image_utils when an ISO needs to be built, but not when one is provided. This patch changes that. There are a few visible side-effects to that: 1) file:/// images are now supported 2) ramdisk_image_download_source is now supported 3) the default caching behavior changes to caching http:// links (previously they were passed to the BMC directly). 4) glance images are supported with backends other than swift Story: #2008987 Task: #42638 Change-Id: I23c21188776c511eddcdaf66a222ce64876678e2
* | | | Merge "Clean up images when ejecting an ISO with Redfish"Zuul2021-07-134-44/+40
|\ \ \ \
| * | | | Clean up images when ejecting an ISO with RedfishDmitry Tantsur2021-07-074-44/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no point in keeping the disconnected ISO files around. Change-Id: I371b0f6ab74de7aa26ecc4a65e825303e3d94085
* | | | | Merge "Add ipxe ramdisk kernel append param test"Zuul2021-07-121-0/+54
|\ \ \ \ \
| * | | | | Add ipxe ramdisk kernel append param testJulia Kreger2021-07-091-0/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While investigating a bug report, I noticed we didn't explicitly test passing arguments in with the ipxe interface and a ramdisk, and given the issue reported was precisely that, I wrote a test. Change-Id: I801b71d381a69e7b8678b9e60ff36dda0ae3f7fb
* | | | | | Merge "Update iDRAC virtual media boot known issue"Zuul2021-07-121-9/+9
|\ \ \ \ \ \
| * | | | | | Update iDRAC virtual media boot known issueAija Jauntēva2021-07-081-9/+9
| |/ / / / / | | | | | | | | | | | | | | | | | | Change-Id: Ia34fce52f2b717428c7502ed7cb0ed1a7df66bc6
* | | | | | Merge "Allow reboot to hard disk following iso ramdisk deploy."Zuul2021-07-124-1/+79
|\ \ \ \ \ \
| * | | | | | Allow reboot to hard disk following iso ramdisk deploy.Derek Higgins2021-07-104-1/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Re-uses the force_persistent_boot_device driver_info option which can be set to ``Never`` in order to set the virtual media to one time boot. Story: 2009042 Task: 42805 Change-Id: Ida7adb3c02db6279ef934797614eaa7be1f35daa
* | | | | | | Merge "Add missing BIOSSettings version mapping for older releases"Zuul2021-07-112-0/+18
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Add missing BIOSSettings version mapping for older releasesDmitry Tantsur2021-07-072-0/+18
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise the upgrade fails because of old versions. Change-Id: Ibd591ab6e17e26db0b2b4dde103fe3458e1536f5
* | | | | | Add `boot_mode` and `secure_boot` to node object and expose in apiCenne2021-07-0822-17/+530
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * add fields to Node object * expose them at endpoint `/v1/nodes/{node_ident}/states` * update states on powersync / entering managed state. * tests * update api endpoint info in api-ref Story: 2008567 Task: 41709 Change-Id: Iddd1421a6fa37d69da56658a2fefa5bc8cfd15e4
* | | | | Merge "Fix typos in API sanitization change notes"Zuul2021-07-072-6/+5
|\ \ \ \ \
| * | | | | Fix typos in API sanitization change notesJulia Kreger2021-07-072-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change https://review.opendev.org/c/openstack/ironic/+/794880 included a few minor typos and required a clarification of a point which. This change just makes those minor text changes. Change-Id: I883d4ca89ba984c29b53b531af98f2f0be39edbf
* | | | | | Merge "Use selectinload for all list queries"Zuul2021-07-072-10/+107
|\ \ \ \ \ \
| * | | | | | Use selectinload for all list queriesJulia Kreger2021-07-072-10/+107
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since node objects *can* be large, and due to the nature of joined record dedeuplication, we should avoid them for list operations to speed the operation to completion by *instead* allowing the client to execute three queries and reconcile versus dedeuplicate the single node. This *should* result in generally faster list interaction, and preserves the behavior for nodes at this time in order to minimize a drastic increase of SQL queries for individual nodes. Change-Id: Iac457e483068f613ded2aeff60cf6d9fc64a7dac
* | | | | | Merge "Allow node_sanitize function to be provided overrides"Zuul2021-07-073-40/+101
|\ \ \ \ \ \ | |/ / / / / | | | / / / | |_|/ / / |/| | | |
| * | | | Allow node_sanitize function to be provided overridesJulia Kreger2021-07-063-40/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The biggest amount of API overhead is the node sanitization process, at least at this point in time. We have streamlined the database interaction to ensure specific field selection lists are as orderly as possible, but the node sanitization code re-executes some methods over and over which do not require variable data from the underlying node. These are blanket settings "is the user allowed to see x, or y". Which means we can call node_sanitize pre-seeding these arguments and execute the calls once, instead of a thousand times to have the same exact result. Story: 2008885 Task: 42433 Change-Id: I342e7900cac388cb4749480684418a5a15ac60eb
* | | | | Merge "Add note regarding configuration drives to tuning docs"Zuul2021-07-071-0/+6
|\ \ \ \ \
| * | | | | Add note regarding configuration drives to tuning docsJulia Kreger2021-07-061-0/+6
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Configuraiton drives get stored in the instance_info field which is of a text type, which in theory is not a big deal, however large configuration drives are returned over the database client and when entire node objects are generated by the client, which has an obvious bandwidth and record transfer latency overhead. Change-Id: I7f2b6ab9c3adf88202f213a42af7553cf9e8276d
* | | | | Merge "Avoid double file removal in create_boot_iso"Zuul2021-07-072-5/+8
|\ \ \ \ \
| * | | | | Avoid double file removal in create_boot_isoDmitry Tantsur2021-07-062-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The images.fetch call removes the provided destination file on error. The create_boot_iso function calls it with a temporary file, which results in another attempt to remove it. This attempt fails and masks the actual exception (e.g. validation failure). I doubt this problem is exposed to end users, but fixing it anyway. Change-Id: I77ef1cabbcea9808771613fa37b27978fd0ce770
* | | | | | Merge "Support "swift" for ramdisk_image_download_source"Zuul2021-07-074-7/+48
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Support "swift" for ramdisk_image_download_sourceDmitry Tantsur2021-07-064-7/+48
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ilo-virtual-media boot interface supports exposing Glance images via a temporary URL. This change brings this support to Redfish and others. Change-Id: Ie72bb5e62c29efa5eea1710e74f505329d28d726
* | | | | Merge "Add support for configdrive in anaconda interface"Zuul2021-07-065-0/+311
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Add support for configdrive in anaconda interfaceArun S A G2021-07-045-0/+311
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Config drive is processed by the deploy interface and written to the disk during post install of the deployment. The config drive is written to /var/lib/cloud/seed/config_drive on the disk. cloud-init should be able to process read config drive contents and take actions on firstboot. Change-Id: Id5a949e777e242d2f20966ec045ac319c61636ba
* | | | | Merge "Defer checking image size until instance info is built"Zuul2021-07-062-43/+63
|\ \ \ \ \
| * | | | | Defer checking image size until instance info is builtSteve Baker2021-07-062-43/+63
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The image size check is skipped if the image is raw and will be streamed directly to disk. However metalsmith does not expose instance_info so it is not possible to set image_disk_format=raw. image_disk_format is detected and set during build_instance_info_for_deploy but this is run *after* the call to check_image_size. This results in large raw whole-disk images always failing check_image_size when deployed by metalsmith (or any time image_disk_format isn't explicitly set). This change avoids that by moving the check_image_size call out of validate to after the call to build_instance_info_for_deploy. Blueprint: whole-disk-default Change-Id: I5da43dc244c1a4fa828a00c02e3aa5b5f1b71e14
* | | | | Merge "Nicer error message when a deploy step fails"Zuul2021-07-062-8/+6
|\ \ \ \ \
| * | | | | Nicer error message when a deploy step failsDmitry Tantsur2021-06-222-8/+6
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | No need to output the whole step structure, especially if it has arguments. Also no need to repeat "deploy". Change-Id: I06275fe894ee24638dca9bb8c78844ff3ca3d29b
* | | | | Merge "Fix oslo policy DeprecatedRule warnings"Zuul2021-07-062-391/+350
|\ \ \ \ \
| * | | | | Fix oslo policy DeprecatedRule warningsGhanshyam Mann2021-07-052-391/+350
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since 3.7.0, oslo policy started the DeprecationWarning[1] if deprecated_reason and deprecated_since param are not passed in DeprecatedRule or they are passed in RuleDefault object. These warnings are logged for every test which increase the log size and sometime can full the log buffer and fail the job. This fixes the 1431 warnings - https://zuul.opendev.org/t/openstack/build/cace355f7c0d4cbcb1df7856a03194d7/log/job-output.txt#1029 [1] https://github.com/openstack/oslo.policy/blob/3.7.0/oslo_policy/policy.py#L1538 Change-Id: I98d20aea80d547e1f301c150564f732619c9e744
* | | | | | Merge "API to pass fields to node object list"Zuul2021-07-065-61/+120
|\ \ \ \ \ \ | | |_|_|/ / | |/| | | |
| * | | | | API to pass fields to node object listJulia Kreger2021-06-255-61/+120
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change modifies the nodes _get_nodes_collection method to consider and pass in an explicit lisst of requested fields into the node list method, while also including the required fields for things like ownership/policy checking. And slightly modifies node_convert_with_links method to simplify it while enabling field validity to be checked, and specific requisite field lists provided in based upon that value. And also optionally builds the traits list as they are nolonger *always* populated on all objects with fully populated objects as only partially hydrated objects are provided back when specific fields are requested. Story: 2008885 Task: 42572 Change-Id: Ided419263d84184cab902944b6c518f98618c9d2
* | | | | | Merge "Redfish: Skip non-RAID controllers for RAID"Zuul2021-07-053-0/+30
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Redfish: Skip non-RAID controllers for RAIDAija Jauntēva2021-06-283-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Avoid processing non-RAID controllers as they cannot be used for RAID operations to save some time and requests and avoid possible errors later if non-RAID disks are picked up for configuration. Related to: I369bdbb17064baf34f90e864d0ece600529de509 Change-Id: I728f03dc829fb76cad9804c8f8df20810307ce2c
* | | | | | Merge "Add reno and reset legacy policy deprecation expectation"Zuul2021-07-052-1/+20
|\ \ \ \ \ \
| * | | | | | Add reno and reset legacy policy deprecation expectationJulia Kreger2021-07-022-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a reno to support https://review.opendev.org/c/openstack/ironic/+/799120 and ultimately removed the "anticipated to be removed in the Xena development cycle" warnings on legacy policy support as the effort is taking longer than the cross-team effort anticipated. Notes added to the code highlight that we need to update the warnings at that time to ensure we appropriately set expectations as to when the legacy policies will be removed. Change-Id: Ie23788abaa61742cdea208fb0cf026bdecaab525
* | | | | | | Merge "[doc] Update section on ESP consistency"Zuul2021-07-051-1/+4
|\ \ \ \ \ \ \
| * | | | | | | [doc] Update section on ESP consistencyArne Wiebalck2021-07-021-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Starting with Victoria, the ESPs are mirrored. Reflect this in the docs. Change-Id: Ic2c1117ea1e8c7fd118229eb0c09c6095f8195f5