| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Node sharding support https://review.opendev.org/c/openstack/python-ironicclient/+/887533 | 00:14 |
|---|---|---|
| NobodyCam | so I think I was able to adopt a nova deployed Baremetal instance from ussuri to antelope | 00:19 |
| NobodyCam | with only one sneaky DB call to update driver and interfaces | 00:20 |
| TheJulia | Umm we have api calls for that | 00:21 |
| TheJulia | But okay | 00:21 |
| TheJulia | Cool | 00:21 |
| NobodyCam | api calls for the driver / interface update? | 00:22 |
| JayF | yes | 00:25 |
| NobodyCam | humm I may need to take a look at that.. | 00:27 |
| NobodyCam | I've done it all with the cli but changing driver / interfaces on a deployed node errors with only in a like of states | 00:27 |
| NobodyCam | s/like/list/ | 00:27 |
| JayF | ah, doing it while the node remains active, I see | 00:28 |
| NobodyCam | if I can do it with api calls that might be nicer | 00:28 |
| JayF | I don't have any direct knowledge, but I have a hunch maintenance mode might be the trick | 00:28 |
| JayF | if not, we'd probably need to consider it because it's reasonable that people'd wanna do that during an upgrade process | 00:28 |
| NobodyCam | I will try that.. THANK YOU for the pointer | 00:29 |
| rpittau | good morning ironic! o/ | 05:50 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix the HTTP code of the BadRequest exception https://review.opendev.org/c/openstack/ironic/+/887271 | 09:45 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix the HTTP code of the BadRequest exception https://review.opendev.org/c/openstack/ironic/+/887271 | 12:16 |
| iurygregory | good morning Ironic | 13:03 |
| * iurygregory is back after fighting with broken upgrade from F35 -> F37 (thank you NodeJS....) | 13:04 | |
| dtantsur | wow | 13:19 |
| iurygregory | yeah.. | 13:20 |
| iurygregory | just me or ironic-standalone and ironic-standalone-redfish have been failing recently? | 13:26 |
| rpittau | they are failing more than usual, yeah | 13:48 |
| iurygregory | BaremetalRamdiskBootIsoVMedia.test_ramdisk_boot and BaremetalDriverAnsibleWholedisk.test_ip_access_to_server seems to be the ones having problems, I'm checking logs to see | 13:49 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add the initial skeleton of the agent inspect interface https://review.opendev.org/c/openstack/ironic/+/877814 | 13:57 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add the initial skeleton of the agent inspect interface https://review.opendev.org/c/openstack/ironic/+/877814 | 14:09 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Add the initial skeleton of the agent inspect interface https://review.opendev.org/c/openstack/ironic/+/877814 | 14:12 |
| dtantsur | huh, have we broken bifrost in the meantime? I see that Ironic generated HTTP files result in system_u:object_r:tftpdir_t:s0 selinux tag | 14:35 |
| iurygregory | .-. | 14:35 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Revert "Fix TFTP read access for enforcing SELinux" https://review.opendev.org/c/openstack/bifrost/+/887613 | 14:38 |
| dtantsur | This is my first reaction ^^, but I think I can invent something more intelligent (unfortunately, involving selinux modules) | 14:50 |
| iurygregory | for some reason it shows as Merge Conflict lol | 14:58 |
| iurygregory | This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. | 14:58 |
| dtantsur | yeah because master has diverged. does not matter. I'll play with selinux modules. | 14:58 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Allow nginx to access images again on systems with SELinux https://review.opendev.org/c/openstack/bifrost/+/887613 | 15:01 |
| dtantsur | iurygregory: maybe this ^^, will test now | 15:01 |
| iurygregory | ack | 15:02 |
| rpittau | mmm wasn't that alreadyu fixed at some point? I may be confusing it with something else though.... | 15:07 |
| dtantsur | It got regressed because of TFTP fixes | 15:12 |
| rpittau | ah right! | 15:12 |
| rpittau | my brain is skipping a cycle | 15:13 |
| dtantsur | okay, this seems to work and can be reviewed: https://review.opendev.org/c/openstack/ironic/+/877814?usp=dashboard | 15:16 |
| opendevreview | Baptiste Jonglez proposed openstack/networking-generic-switch master: Do not make actual device changes in bind_port() https://review.opendev.org/c/openstack/networking-generic-switch/+/847592 | 15:23 |
| rpittau | good night! o/ | 15:31 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: WIP: Add inspection (processing) hooks https://review.opendev.org/c/openstack/ironic/+/887554 | 17:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Very basic in-band inspection with the "agent" interface https://review.opendev.org/c/openstack/ironic/+/885450 | 17:18 |
| dtantsur | "curl: (37) Couldn't open file /opt/cache/files/cirros-0.5.1-x86_64-disk.img" | 17:24 |
| dtantsur | damn.. I thought I saw a patch for that? | 17:24 |
| dtantsur | priteau: you have a patch for ^^^, but it fails upgrade jobs.. I suspect we need to walk down from the oldest versions | 17:25 |
| dtantsur | priteau: I think you can safely remove upgrade jobs from all EM versions if that helps. | 17:26 |
| iurygregory | I seem to remember a patch for this | 17:26 |
| dtantsur | https://review.opendev.org/c/openstack/bifrost/+/885874/ and its W and V counterparts | 17:28 |
| iurygregory | interesting | 17:30 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/wallaby: CI: Update cached cirros image to 0.5.3 https://review.opendev.org/c/openstack/bifrost/+/885876 | 17:31 |
| dtantsur | iurygregory: I think ^^ may pass | 17:31 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost stable/xena: CI: Update cached cirros image to 0.5.3 https://review.opendev.org/c/openstack/bifrost/+/887639 | 17:32 |
| dtantsur | also removing upgrade jobs here so should pass as well ^^ | 17:32 |
| iurygregory | ack | 17:33 |
| dtantsur | once merged, these will let us fix Yoga and on | 17:33 |
| dtantsur | I'm not sure if Yoga should have upgrade jobs (Xena is EM), but I'll think about it at some better point | 17:33 |
| iurygregory | I will keep an eye on them today | 17:33 |
| dtantsur | good night folks o/ | 17:33 |
| dtantsur | thanks iurygregory | 17:33 |
| iurygregory | good night dtantsur o/ | 17:33 |
| JayF | that upgrade job is failing /on master/ for that cached cirros job change :( | 17:36 |
| JayF | I'm not working today but thanks for looking at this, if you ping me directly I can review and approve patches for CI fixes if I'm around | 17:36 |
| opendevreview | Jay Faulkner proposed openstack/python-ironicclient master: Node sharding support https://review.opendev.org/c/openstack/python-ironicclient/+/887533 | 17:39 |
| opendevreview | Merged openstack/bifrost stable/xena: CI: Update cached cirros image to 0.5.3 https://review.opendev.org/c/openstack/bifrost/+/887639 | 19:34 |
| opendevreview | Merged openstack/bifrost stable/wallaby: CI: Update cached cirros image to 0.5.3 https://review.opendev.org/c/openstack/bifrost/+/885876 | 19:34 |
| iurygregory | now we can think about fixing yoga zed 2023.1 (antelope) master | 20:43 |
| opendevreview | Merged openstack/python-ironicclient master: Node sharding support https://review.opendev.org/c/openstack/python-ironicclient/+/887533 | 23:27 |
| iurygregory | congrats JayF ^^ | 23:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!