| opendevreview | Verification of a change to openstack/ironic master failed: add default conductor group capability https://review.opendev.org/c/openstack/ironic/+/855705 | 01:31 |
|---|---|---|
| opendevreview | Verification of a change to openstack/ironic master failed: Remove traces of Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885245 | 02:38 |
| opendevreview | Verification of a change to openstack/ironic master failed: Transiton to Storage resource from SimpleStorage https://review.opendev.org/c/openstack/ironic/+/892252 | 03:30 |
| opendevreview | Steve Baker proposed openstack/ironic master: grenade: convert neutron cli commands to openstack https://review.opendev.org/c/openstack/ironic/+/897513 | 04:46 |
| stevebaker[m] | An attempt to fix the grenade job ^^ | 04:48 |
| rpittau | good morning ironic! happy friday! o/ | 07:28 |
| frickler | for the doc build failures see https://bugs.launchpad.net/ironic/+bug/2026345 | 07:35 |
| rpittau | thanks frickler | 07:51 |
| rpittau | looks like blockdiag is not really maintained anymore | 07:58 |
| rpittau | going to try and remove it like they did for designate | 07:58 |
| rpittau | lol nvm we don't use blockdiag apparently | 08:06 |
| frickler | rpittau: maybe some other diag tool like https://github.com/blockdiag/seqdiag ? seems to be affected as well, cf. octavia | 08:08 |
| rpittau | yeah, I'm looking for that | 08:08 |
| rpittau | looks like it is indeed seqdiag | 08:10 |
| rpittau | well there's a bunch, let's see what we can do | 08:15 |
| rpittau | ooook let's try with a quick workaround first, then we see to convert all to graphviz | 08:23 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Pin pillow version to 9.5.0 https://review.opendev.org/c/openstack/ironic/+/897523 | 08:28 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Pin pillow version to 9.5.0 https://review.opendev.org/c/openstack/ironic/+/897523 | 08:28 |
| rpittau | mm ofc upper constraints do not allow that | 09:55 |
| *** gryf is now known as Guest2360 | 10:12 | |
| *** Guest2360 is now known as _gryf | 10:13 | |
| iurygregory | good morning Ironic | 11:47 |
| iurygregory | dtantsur, you around? I was testing my gophercloud code to get /v1/nodes/<node_id>/firmware , wondering if the output is making sense the way I've implemented it would return the [] of Firmware Components, since last version flashed we return can be null or a string from irnic API I've decided to use *string, the problem is that now I get the pointer to the value when trying to print, do you have any ideas? | 12:17 |
| iurygregory | wondering if I should just treat as string and accept omitempty in https://github.com/gophercloud/gophercloud/pull/2795/files#diff-4b8e645356462cea2c242db58b00a159810f507ea45eee9dc7ae737f0f76b363R633 | 12:20 |
| dtantsur | iurygregory, if you use just a string, I think you'll get "" on nil | 12:25 |
| dtantsur | that seems to be the common pattern (check it) | 12:25 |
| iurygregory | yeah | 12:29 |
| iurygregory | but since it comes from json I think I need to allow omitempty | 12:30 |
| iurygregory | going to test here | 12:30 |
| iurygregory | [{CreatedAt:2023-10-03 18:30:00 +0000 +0000 UpdatedAt:<nil> Component:bios InitialVersion:U30 v2.36 (07/16/2020) CurrentVersion:U30 v2.36 (07/16/2020) LastVersionFlashed:} {CreatedAt:2023-10-03 18:30:00 +0000 +0000 UpdatedAt:2023-10-03 18:45:54 +0000 +0000 Component:bmc InitialVersion:iLO 5 v2.78 CurrentVersion:iLO 5 v2.81 LastVersionFlashed:iLO 5 v2.81}] | 12:32 |
| iurygregory | ok, it works :D | 12:32 |
| iurygregory | perfect, just notice that acceptance failed .-. in previous patch | 12:36 |
| TheJulia | good morning | 13:27 |
| rpittau | good morning TheJulia :) | 13:31 |
| iurygregory | good morning TheJulia =) | 13:31 |
| iurygregory | dtantsur, by any chance you remember how to check for the full logs in gophercloud? | 13:32 |
| iurygregory | trying to figure out the problems in https://github.com/gophercloud/gophercloud/actions/runs/6431875166/job/17465610247?pr=2795 | 13:32 |
| dtantsur | iurygregory, no idea unfortunately | 14:00 |
| rpittau | bye everyone, have a great weekend! o/ | 14:43 |
| opendevreview | Julia Kreger proposed openstack/ironic master: grenade: convert neutron cli commands to openstack https://review.opendev.org/c/openstack/ironic/+/897513 | 14:55 |
| TheJulia | Would anyone be willing to take a few minutes to do a quick review on https://review.opendev.org/c/openstack/ironic/+/885087 ? | 15:04 |
| TheJulia | zigo: so it occurs to me we turned off buffering output on OpenStack CI... wonder if that is why we are not seeing it | 15:10 |
| opendevreview | Julia Kreger proposed openstack/metalsmith stable/2023.2: Set role default metalsmith_provisioning_timeout https://review.opendev.org/c/openstack/metalsmith/+/897453 | 15:13 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Generic API for attaching/detaching virtual media https://review.opendev.org/c/openstack/ironic/+/894918 | 15:29 |
| dtantsur | rpittau, I've reduced scope of ^^^ so far | 15:29 |
| dtantsur | it's enough of a problem to handle caching.. | 15:29 |
| opendevreview | Julia Kreger proposed openstack/ironic master: grenade: convert neutron cli commands to openstack https://review.opendev.org/c/openstack/ironic/+/897513 | 15:42 |
| opendevreview | Merged openstack/ironic master: [DOC] Add a little more clarity regarding parent/child nodes https://review.opendev.org/c/openstack/ironic/+/894222 | 15:46 |
| opendevreview | Iury Gregory Melo Ferreira proposed openstack/ironic stable/zed: Testing Zed CI https://review.opendev.org/c/openstack/ironic/+/897573 | 16:16 |
| TheJulia | iurygregory: I looks like it is in good shape, just waiting for metalsmith on another change | 16:55 |
| TheJulia | but its prior run was earlier in the week | 16:55 |
| TheJulia | https://review.opendev.org/c/openstack/ironic/+/897513 <-- looks like this should help fix up master branch | 17:04 |
| iurygregory | TheJulia, ack, i just because gophercloud devstack jobs seems to be having some problems from zed to victoria | 17:19 |
| TheJulia | ugh :( | 17:19 |
| iurygregory | https://github.com/gophercloud/gophercloud/actions/runs/6431875166/job/17465610247#step:3:5418 | 17:19 |
| iurygregory | grub... | 17:19 |
| iurygregory | maybe is just something in the repo it uses, going to check later, | 17:20 |
| iurygregory | going to the orthopedist now | 17:21 |
| TheJulia | I think what is happening is the package is trying to redo grub and the running environment breaks | 17:26 |
| TheJulia | is secure boot stuffs needed there? | 17:27 |
| TheJulia | woot https://review.opendev.org/c/openstack/ironic/+/897513 passes | 17:29 |
| JayF | someone (elodliles or frickler) ^ was asking about that the other day, thank you for picking it up | 17:34 |
| JayF | I think we may have been the last project with neutronclient usage | 17:35 |
| TheJulia | Thank stevebaker | 17:39 |
| JayF | of course it's stevebaker[m] :D | 17:40 |
| opendevreview | Julia Kreger proposed openstack/sushy-tools master: WIP: Support testing for HttpBootUri https://review.opendev.org/c/openstack/sushy-tools/+/896963 | 17:53 |
| opendevreview | Merged openstack/metalsmith stable/2023.2: Set role default metalsmith_provisioning_timeout https://review.opendev.org/c/openstack/metalsmith/+/897453 | 18:14 |
| opendevreview | Merged openstack/ironic master: grenade: convert neutron cli commands to openstack https://review.opendev.org/c/openstack/ironic/+/897513 | 19:05 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Reset parent_node values to uuids instead of names https://review.opendev.org/c/openstack/ironic/+/889750 | 19:13 |
| opendevreview | Verification of a change to openstack/ironic stable/2023.1 failed: redfish_address - wrap_ipv6 address https://review.opendev.org/c/openstack/ironic/+/895795 | 19:15 |
| iurygregory | interesting, our CI works fine with grub-efi-amd64-signed 1.187.6~20.04.1+2.06-2ubuntu14.4 , but the one in gophercloud doesn't .-. | 19:20 |
| opendevreview | Verification of a change to openstack/ironic master failed: Remove traces of Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885245 | 20:27 |
| opendevreview | Julia Kreger proposed openstack/sushy master: Add a boot progress indicator https://review.opendev.org/c/openstack/sushy/+/896835 | 20:34 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add additional mock tests to unit tests. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 20:34 |
| opendevreview | Yusuke Okada proposed openstack/ironic stable/xena: Align iRMC driver with Ironic's default boot_mode https://review.opendev.org/c/openstack/ironic/+/866627 | 21:48 |
| opendevreview | Yusuke Okada proposed openstack/ironic stable/wallaby: Align iRMC driver with Ironic's default boot_mode https://review.opendev.org/c/openstack/ironic/+/866628 | 21:55 |
| TheJulia | question for the weekend: if a checksum fails, should we retry the download? | 21:57 |
| opendevreview | Merged openstack/ironic stable/2023.2: Fix the HTTP code for reaching max_concurrent_deploy: 503 instead of 500 https://review.opendev.org/c/openstack/ironic/+/897169 | 21:57 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add additional mock tests to unit tests for read only devices. Change ordering to ensure mock tests work correctly. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 22:02 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add additional mock tests to unit tests for read only devices. Change ordering to ensure mock tests work correctly. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 22:04 |
| opendevreview | Merged openstack/ironic stable/2023.1: redfish_address - wrap_ipv6 address https://review.opendev.org/c/openstack/ironic/+/895795 | 22:05 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add mock tests for WSL read-only SDA. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 22:15 |
| opendevreview | Merged openstack/ironic master: Remove traces of Docker from devstack dependencies https://review.opendev.org/c/openstack/ironic/+/885245 | 22:15 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add mock tests for WSL read-only SDA. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 22:15 |
| opendevreview | Boushra Sondos Bettir proposed openstack/ironic-python-agent master: Add mock tests for WSL read-only SDA. https://review.opendev.org/c/openstack/ironic-python-agent/+/897611 | 22:16 |
| opendevreview | Merged openstack/ironic master: Transiton to Storage resource from SimpleStorage https://review.opendev.org/c/openstack/ironic/+/892252 | 22:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!