| rpittau | good morning ironic! o/ | 06:32 |
|---|---|---|
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Update to match latest development cycle https://review.opendev.org/c/openstack/virtualpdu/+/922372 | 08:45 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Update to match latest development cycle https://review.opendev.org/c/openstack/virtualpdu/+/922372 | 08:47 |
| shajizad | Good morning ironic. Any update on my pull requests? :> | 09:53 |
| opendevreview | Mark Goddard proposed openstack/networking-generic-switch master: Cumulus NVUE support https://review.opendev.org/c/openstack/networking-generic-switch/+/907551 | 10:59 |
| masghar | Hi shajizad! I have taken a look at your PRs | 12:03 |
| shajizad | Thank you for the feedback | 12:18 |
| shajizad | I think the `exceptions.raise_from_response(response, error_message=msg)` line will not always raise an exception. If the response is okay it will just return None at the end. | 12:18 |
| masghar | You appear to be right | 12:57 |
| opendevreview | Riccardo Pittau proposed openstack/virtualpdu master: Update to match latest development cycle https://review.opendev.org/c/openstack/virtualpdu/+/922372 | 13:36 |
| dtantsur | shajizad, masghar, this is correct, raise_for_response only raises if error code >= 400 | 13:58 |
| shajizad | I changed the spelling mistake in the vmedia one | 14:06 |
| opendevreview | Jay Faulkner proposed openstack/ironic master: Enable HTTP network boot for Redfish hardware https://review.opendev.org/c/openstack/ironic/+/922160 | 15:42 |
| opendevreview | Merged openstack/ironic master: Decompose the Redfish documentation https://review.opendev.org/c/openstack/ironic/+/921844 | 15:48 |
| rpittau | good night! o/ | 16:05 |
| JayF | o/ | 16:06 |
| opendevreview | Merged openstack/ironic master: Documentation: migration guide from inspector https://review.opendev.org/c/openstack/ironic/+/922089 | 16:20 |
| opendevreview | Merged openstack/ironic master: Fix and document redfish metrics https://review.opendev.org/c/openstack/ironic/+/921951 | 17:15 |
| *** jph5 is now known as jph | 17:46 | |
| cid | o/ | 19:40 |
| TheJulia | g'night | 19:40 |
| *** dmellado075 is now known as dmellado07 | 19:44 | |
| opendevreview | Jacob Anders proposed openstack/ironic master: [WIP] add Targets to firmware.update on multi system BMCs https://review.opendev.org/c/openstack/ironic/+/922438 | 23:24 |
| opendevreview | Jacob Anders proposed openstack/sushy-tools master: [WIP] Add support for BIOS update emulation https://review.opendev.org/c/openstack/sushy-tools/+/909500 | 23:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!