Commit Graph

13018 Commits

Author SHA1 Message Date
Zuul 384c692f59 Merge "Fix iscsi url generate method for ipxe" 2024-04-29 22:57:31 +00:00
Zuul d49a1b907c Merge "ci: re-enable grenade test job" 2024-04-29 22:52:48 +00:00
Zuul b6b8ee07ce Merge "Replace glanceclient usage with openstacksdk" 2024-04-29 15:26:56 +00:00
Zuul 3a95a38b65 Merge "Redfish: fix error formatting when mounting vmedia" 2024-04-29 14:41:34 +00:00
Dmitry Tantsur f43587effd
Redfish: fix error formatting when mounting vmedia
Also add missing error into the message.

Change-Id: I9610add40afbb7beb30d375a3a455434f5446cc7
2024-04-29 14:11:06 +02:00
frankming dcf059af9e Fix iscsi url generate method for ipxe
Boot from volume feature has a ipxe template render step, and it need
to generate iscsi urls for booting the volume. However, it not works. In
the function, lun field should be hexadecimal instead of decimal,
according to SAN URIs description at https://ipxe.org/sanuri. So we
need to fix it.

Closes-Bug: #2055355
Change-Id: I080ca42c9ba05f2a4e0752312b79a32bef825752
Signed-off-by: frankming <chen27508959@outlook.com>
2024-04-29 17:06:34 +08:00
Zuul 08209c48ab Merge "docs: update ilo docs regarding status -> use redfish" 2024-04-29 08:30:06 +00:00
Steve Baker 978afbd5a1 Replace glanceclient usage with openstacksdk
Closes-Bug: #2042495
Change-Id: Ic8421bd937a3a1ab6c3b86c259cd929810c0532e
2024-04-29 12:30:33 +12:00
Zuul 6437a2e794 Merge "Validate url options at config input layer" 2024-04-27 06:52:59 +00:00
Julia Kreger 508b1ccab3 ci: re-enable grenade test job
Change-Id: I1d2af3918dcc7cf3248edb460898044e35812b52
2024-04-26 16:53:04 -07:00
Julia Kreger bccea706c8 docs: update ilo docs regarding status -> use redfish
Change-Id: I67f275556186887ae6f555379c9f4c24cde48638
2024-04-26 16:41:38 -07:00
Zuul 770a758b7d Merge "Trivial: use canonical name of microversion-parse" 2024-04-26 23:19:45 +00:00
Zuul cd489983de Merge "Ensure [molds] password is not logged" 2024-04-26 15:19:20 +00:00
Julia Kreger 6c1e002c0d doc: Add extra context around release mapping
Adds some inline text to help contributors who
have not had to deal with release mappings before.

Change-Id: I064e5c80a11005df97e38523a7d703e803fa4a41
2024-04-25 06:49:29 -07:00
Dmitry Tantsur f7fd7961c0
Trivial: use canonical name of microversion-parse
Change-Id: I8b457b531cc181bbb43e8e21bcd29929ce5725f7
2024-04-24 09:13:47 +02:00
Riccardo Pittau c1f3daf7b0 Fix attach/detach vmedia redfish implementation
We need to map with virtual media devices and not boot
devices only.

Change-Id: I88b56ae26d9f1d8642ed6ffc5c055f8d56f6939a
2024-04-23 08:55:18 +02:00
Takashi Kajinami 623949bfae Ensure [molds] password is not logged
The [molds] password option accepts a secret value apparently. So its
value should not appear in debug logs.

Change-Id: If8a54c1d4f74516f1c24f7286e76955b2e424f5c
2024-04-22 06:47:44 +00:00
Takashi Kajinami 9cead9457c Validate url options at config input layer
oslo.config provides the URIOpt class which enforces valid URI(URL)
format. Use this built-in feature to detect any malformed values for
better feedback.

Change-Id: I0d846f78f8132a2d63266b7b3331ec7118cea1b4
2024-04-22 15:27:21 +09:00
Zuul b4773ad8d0 Merge "docs: update redfish docs to detail swift url issues" 2024-04-21 12:16:13 +00:00
Zuul 1cefbbcc75 Merge "redfish: change default virtual media storage to local storage" 2024-04-21 12:16:10 +00:00
Zuul 1904d3c1bf Merge "Docs: Remove outdated RBAC content" 2024-04-19 22:10:14 +00:00
Julia Kreger ae2b5f5f1f redfish: change default virtual media storage to local storage
This may be the most overridden default of Ironic, which means
we need to change the default value.

The default for ``[redfish]use_swift`` was historically ``true``,
however we've generally found that BMCs are particularlly sensitive
to extra characters in the URL as the characters may signify a
dynamically generated file, which would be problematic as virtual
media webservers also generally require range retrieval support.

This change makes the default ``false`` which should lead to one
less override for operators being necessary in practical operation.

Change-Id: Iad57b3c6423bced0e3cb6fb4e31aad6d805f26fa
2024-04-19 21:43:24 +00:00
Julia Kreger 126041be95 docs: update redfish docs to detail swift url issues
Change-Id: I45f7b647475c81e836f6d3ee036b33aee383fe6c
2024-04-19 14:42:43 -07:00
Zuul 3c5b4cb349 Merge "Follow-up: Use ``microversion-parse`` to parse version headers in API requests" 2024-04-19 15:52:31 +00:00
Zuul 27dbc24c85 Merge "Fix device_type in attach/detach vmedia for Redfish" 2024-04-19 13:59:40 +00:00
Zuul 11309e7aea Merge "Remove redundant priority override code from service steps" 2024-04-18 21:59:40 +00:00
Zuul 7969baf58e Merge "Correct logging call in _heartbeat_service_wait method" 2024-04-18 21:59:37 +00:00
Zuul a7b2908e2c Merge "CI: Unpack cirros image partition jobs" 2024-04-18 19:20:12 +00:00
cid 20543fc115 Follow-up: Use ``microversion-parse`` to parse version headers in API requests
Adds a release note, unit test, and documentation update as a follow-up to the
`microversion-parse change <https://review.opendev.org/c/openstack/ironic/+/913793>`_

Change-Id: I535af988125a511e4f54c9d81acd47c327413774
2024-04-18 19:53:34 +01:00
Zuul f494d2760a Merge "Small Doc Fix: Moved user export for devstack guide." 2024-04-18 18:12:22 +00:00
Riccardo Pittau 68fcc6b0d2 Fix device_type in attach/detach vmedia for Redfish
We need to translate the generic device_type to sushy device_type

Change-Id: Id09e6f2b989513b41980c7b89c9d76edecc320db
2024-04-18 18:39:12 +02:00
Zuul 5fabff87d1 Merge "Use .coveragerc file" 2024-04-18 07:59:33 +00:00
Jacob Anders 92b61e280c Remove redundant priority override code from service steps
Currently, service steps are only supported as a user-requested action.
This change removes references to overriding priorities for service
steps as these overrides will have no effect and may cause confusion.

Change-Id: I35a8b59b17fdff3161df835903acec529e732c4f
2024-04-18 17:54:37 +10:00
Jacob Anders e04741ac3f Correct logging call in _heartbeat_service_wait method
Currently, _heartbeat_service_wait() is causing an error due to a
missing parameter in the call. This change resolves this issue by
removing the reference to the missing parameter.

Change-Id: I03faa67953daf282ae1b576a2a949c94a2efa973
2024-04-18 07:47:35 +00:00
Julia Kreger fb00fa5037 Docs: Remove outdated RBAC content
Change-Id: I45f4417c3075349ac3dd36413034cff1895b1eff
2024-04-17 12:27:51 -07:00
Zuul c85d9c8326 Merge "Load machine type from command-line" 2024-04-17 17:31:18 +00:00
Zuul 7df06977d3 Merge "Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED" 2024-04-17 10:34:59 +00:00
Jacob Anders 619e1ac80c Add states.SERVICING and SERVICEWAIT to _FASTTRACK_HEARTBEAT_ALLOWED
Currently, service steps may fail to start in scenarios dependent on IPA
fasttrack. This change attempts to resolve this by incorporating
servicing states in the fast track allowed states whitelist while also
making _FASTTRACK_HEARTBEAT_ALLOWED a superset of _HEARTBEAT_ALLOWED
instead of duplicating values in the two constants.

Change-Id: I47984469c1432e7fc7b4f1494b9f6c551c34672f
2024-04-17 14:04:16 +10:00
Sharpz7 9c237240ba Small Doc Fix: Moved user export for devstack guide.
A small bugfix - you now need to ensure that the user env var is set before running any openstack commands (At least the ones that devstack uses)

Change-Id: I4afad7ea588cf6505a7b1186c749d13827b24290
2024-04-16 22:48:09 +00:00
Zuul d68490c6fe Merge "Use the new microversion-parse library to parse the version headers" 2024-04-15 23:38:05 +00:00
Zuul ddca532f52 Merge "Fix the confusion around service_reboot/servicing_reboot" 2024-04-15 23:38:00 +00:00
Zuul 3123e82a14 Merge "Fix servicing clean-up" 2024-04-15 23:37:57 +00:00
Zuul 36251de6f1 Merge "Add note regarding metal3 ci job in CI config for stable runs" 2024-04-15 23:37:55 +00:00
cid 211e007472 Load machine type from command-line
Change-Id: Ib2bdac78e0eb4043e2a2f86244ea168641571817
2024-04-16 00:30:44 +01:00
Alexon Oliveira 668dd24108 Remove deprecation warning by setting schema
Closes-Bug: #2061160

Change-Id: Ie5af73dd1b8af29734d1cf34b070e2a2bbc09949
Signed-off-by: Alexon Oliveira <alolivei@redhat.com>
2024-04-15 14:54:17 -03:00
Zuul ffd0e5c95d Merge "db: Replace use of Engine.execute() method" 2024-04-15 16:11:18 +00:00
Zuul 044ca92902 Merge "Make codespell voting" 2024-04-15 10:04:14 +00:00
Zuul a329135fad Merge "Fix spelling reported by codespell" 2024-04-15 10:01:50 +00:00
Takashi Kajinami 7408a93dcd db: Replace use of Engine.execute() method
Resolve the following RemovedIn20Warning warning:

  The Engine.execute() method is considered legacy as of the 1.x
  series of SQLAlchemy and will be removed in 2.0.

Closes-Bug: #2061345
Co-Authored-By: Stephen Finucane <stephenfin@redhat.com>
Change-Id: Ib0519af8a15ca02e351f8d739d52f4e658f7615a
2024-04-15 17:27:33 +09:00
Winicius Silva 774890dd7f Make codespell voting
Change-Id: Ia3e0d347d0ca80f5f632a7ad889bea17391da136
2024-04-12 14:41:56 -03:00