summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Update vendor_passthru subscriptions releasenote"18.1.0Zuul2021-07-221-1/+1
|\
| * Update vendor_passthru subscriptions releasenoteIury Gregory Melo Ferreira2021-07-221-1/+1
| | | | | | | | | | | | This is a follow-up to I5b7fa99b0ee64ccdc0f62d9686df655082db3665 Change-Id: Ib1145f9d34ad53966c2f4b24981316d4a8763340
* | Merge "Add vendor_passthru method for subscriptions"Zuul2021-07-226-70/+559
|\ \ | |/
| * Add vendor_passthru method for subscriptionsIury Gregory Melo Ferreira2021-07-216-70/+559
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds two new vendor_passthru methods for Redfish: - create_subscription (create a sbuscription) - delete_subscription (delete a subscription) - get_all_subscriptions (get all subscriptions on the node) - get_subscription (get a single subscription) Unit Tests in test_utils split into multiple classes to avoid random failures due to cache. Tested in bifrost env using two different HW: - HPE EL8000 e910 - Dell R640 Story: #2009061 Task: #42854 Change-Id: I5b7fa99b0ee64ccdc0f62d9686df655082db3665
* | Prepare 18.1 and clean up release notesRiccardo Pittau2021-07-215-13/+29
| | | | | | | | Change-Id: I8f33eefdce7a60fa8e48225ec20a801b3d1222f0
* | Merge "Scoped RBAC Devstack Plugin support"Zuul2021-07-213-43/+117
|\ \
| * | Scoped RBAC Devstack Plugin supportJulia Kreger2021-07-153-43/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds support to the ironic devstack plugin to configure ironic to be used in a scope-enforcing mode in line with the Secure RBAC effort. This change also defines two new integration jobs *and* changes one of the existing integration. In these cases, we're testing functional crub interactions, integration with nova, and integration with ironic-inspector. As other services come online with their plugins and devstack code being able to set the appropriate scope enforcement configuration, we will be able to change the overall operating default for all of ironic's jobs and exclude the differences. This effort identified issues in ironic-tempest-plugin, tempest, devstack, and required plugin support in ironic-inspector as well, and is ultimately required to ensure we do not break the Secure RBAC. Luckilly, it all works. Change-Id: Ic40e47cb11a6b6e9915efcb12e7912861f25cae7
* | | Merge "Set glance limit for baremetal friendly images"Zuul2021-07-204-0/+16
|\ \ \
| * | | Set glance limit for baremetal friendly imagesJulia Kreger2021-07-194-0/+16
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The devstack default limit enforcement for glance defaults to 1GB, and unfortunately this is too small for many to use larger images such as centos which includes hardware firmware images for execution on baremetal where drivers need the vendor blobs in order to load/run. Sets ironic-base to 5GB, and updates examples accordingly. Depends-On: https://review.opendev.org/c/openstack/devstack/+/801309 Change-Id: I41294eb571d07a270a69e5b816cdbad530749a94
* | | Merge "Fix handling driver_info[agent_verify_ca] == False"Zuul2021-07-203-3/+33
|\ \ \
| * | | Fix handling driver_info[agent_verify_ca] == FalseDmitry Tantsur2021-06-233-3/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Because of using 'or', it will only be respected when verify_ca is also False in the configuration. Change-Id: Ie000463fa8ac2cd739a9d8f256346472fe3f4288
* | | | Merge "Burn-in: Add documentation"Zuul2021-07-192-0/+167
|\ \ \ \
| * | | | Burn-in: Add documentationArne Wiebalck2021-07-142-0/+167
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add documentation for the CPU, memory, disk, and network burn-in cleaning steps to the admin guide. Change-Id: Ide8061f96eeb303490f06ebf31d2f2e6be93d0d1
* | | | | 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
|\ \ \ \ \ \