| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Fix various issues in the anaconda deploy interface https://review.opendev.org/c/openstack/ironic/+/815871 | 00:18 |
|---|---|---|
| stevebaker[m] | janders: hey \o | 00:19 |
| *** pmannidi|AFK is now known as pmannidi | 00:23 | |
| *** sshnaidm is now known as sshnaidm|off | 03:06 | |
| *** pmannidi is now known as pmannidi|Lunch | 04:14 | |
| opendevreview | Merged openstack/python-ironicclient master: Made `baremetal <command> --help` display command specific help. https://review.opendev.org/c/openstack/python-ironicclient/+/816098 | 04:25 |
| *** pmannidi|Lunch is now known as pmannidi | 05:00 | |
| hgy_ | Hi, Could I ask if I can use linuxbridge in network when I deploy ironic | 06:58 |
| hgy_ | The ironic document default use ovs,and not refer to linuxbridge | 06:59 |
| hgy_ | but the neutron guide default uses linuxebridge now | 06:59 |
| hgy_ | I would appreciate any kind of help | 07:00 |
| arne_wiebalck | Good morning, Ironic! | 07:05 |
| hgy_ | Good morning, arne_wiebalck | 07:19 |
| arne_wiebalck | Hey hgy_ o/ | 07:19 |
| arne_wiebalck | hgy_: I saw your question, but I'd be a little out of my depth to answer them, probably better to wait for others to come online. | 07:21 |
| hgy_ | arne_wiebalck: Thank you very much, I will wait others. Have a good day | 07:22 |
| arne_wiebalck | hgy_: The reason is that in our deployment networking for bare metal is handled completely different than networking for VMs. So, we do run Neutron, but we do not use it for bare metal, for instance. | 07:24 |
| hgy_ | arne_wiebalck: emm~~~ sorry,I can't compeletely understand it "we do not use it for bare metal, for instance." | 07:32 |
| arne_wiebalck | hgy_: Sorry, we do not use Neutron for bare metal. | 07:33 |
| hgy_ | arne_wiebalck:oh! thank you, you woke me up | 07:34 |
| hgy_ | arne_wiebalck: So, we just use the dhcp of neutron network | 07:35 |
| arne_wiebalck | hgy_: When we started with Ironic, we had a fleet already and also systems which manage the network, so we integrated Ironic to use them. | 07:35 |
| arne_wiebalck | hgy_: right | 07:36 |
| hgy_ | arne_wiebalck: Oh, you make me clear it | 07:37 |
| hgy_ | love you | 07:37 |
| hgy_ | arne_wiebalck: Sorry for my many question. If we just integrated Ironic to use them, | 07:39 |
| hgy_ | arne_wiebalck: whether it is no effect that we use linuxbridge or ovs in neutron | 07:40 |
| hgy_ | arne_wiebalck: if you are busy, you can don't answer me. You have help me so much | 07:40 |
| rpittau | good morning ironic! o/ | 08:00 |
| arne_wiebalck | hgy_: I think if you can, you should go the more standard way, i.e. use Neutron for the networking (rather than bypassing it). We plan to have at look at this as well at some point. | 08:02 |
| arne_wiebalck | hey rpittau o/ | 08:02 |
| hgy_ | arne_wiebalck： Ok, Thack you | 08:04 |
| hgy_ | arne_wiebalck: wishes to you | 08:04 |
| iurygregory | good morning Ironic o/ happy friday =) | 08:05 |
| rpittau | hey arne_wiebalck :) | 08:05 |
| rpittau | hey iurygregory, happy Friday! | 08:06 |
| iurygregory | rpittau, o/ | 08:06 |
| arne_wiebalck | hgy_: thanks, to you, too :) | 08:07 |
| arne_wiebalck | hey iurygregory o/ | 08:07 |
| iurygregory | arne_wiebalck, o/ | 08:08 |
| hgy_ | happy Friday | 08:14 |
| rpittau | hey hgy_ happy friday :) | 08:14 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Call execute from ironic-lib in hardware.py https://review.opendev.org/c/openstack/ironic-python-agent/+/816791 | 08:28 |
| rpittau | forgot to re-reun pep8.... | 08:43 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent master: Call execute from ironic-lib in hardware.py https://review.opendev.org/c/openstack/ironic-python-agent/+/816791 | 08:47 |
| opendevreview | Aija Jauntēva proposed x/sushy-oem-idrac master: Add destructive field flag to export configuration https://review.opendev.org/c/x/sushy-oem-idrac/+/816815 | 10:28 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Update idrac-redfish export configuration step https://review.opendev.org/c/openstack/ironic/+/816816 | 10:33 |
| *** dviroel|out is now known as dviroel|rover | 10:38 | |
| dtantsur | hey ironic, happy Monday | 10:41 |
| dtantsur | arne_wiebalck: have you tried reordering calls, e.g. re-reading partitions before udev settle? | 10:42 |
| arne_wiebalck | dtantsur: yes | 10:42 |
| iurygregory | dtantsur, go back to sleep! | 10:42 |
| iurygregory | happy Monday?! | 10:42 |
| arne_wiebalck | dtantsur: this did not fix the original issue | 10:42 |
| arne_wiebalck | dtantsur: I cannot reproduce the issue on a random VM (by just calling blockdev) | 10:43 |
| arne_wiebalck | dtantsur: I keep looking, it seems the part UUIDs really disappear, also from the tree in /dev/disk/... | 10:43 |
| arne_wiebalck | dtantsur: but they are back when the instance boots | 10:43 |
| arne_wiebalck | dtantsur: I have not tried reordering to see if this brings the UUIDs back | 10:44 |
| arne_wiebalck | dtantsur: I am creating a new image atm, to give me more debugging handles | 10:44 |
| arne_wiebalck | dtantsur: I will let you know what I find | 10:45 |
| dtantsur | iurygregory: ugh, yeah, this is Friday feeling like Monday | 10:51 |
| dtantsur | arne_wiebalck: the UUIDs are back even when we're collecting ramdisk logs, so there is a really brief period | 10:52 |
| arne_wiebalck | dtantsur: aha, yeah, this is what I mentioned yesterday already | 10:53 |
| opendevreview | Merged openstack/ironic-python-agent stable/xena: Always include the oslo_log log file in ramdisk logs https://review.opendev.org/c/openstack/ironic-python-agent/+/816664 | 10:53 |
| arne_wiebalck | dtantsur: any idea if there is sth we could trigger to wait for them to re-appear? | 10:53 |
| arne_wiebalck | like udev settle or similar | 10:53 |
| arne_wiebalck | sth synchronous | 10:53 |
| arne_wiebalck | dtantsur: ha, yes | 10:55 |
| arne_wiebalck | dtantsur: I could not reproduce on the VM since I am too slow :) | 10:55 |
| iurygregory | dtantsur, oh god (please no!) =) | 10:55 |
| arne_wiebalck | "blockdev --rereadpt /dev/vdb && lsblk -o NAME,UUID,PARTUUID" shows no partition UUIDs | 10:56 |
| arne_wiebalck | but a "blockdev --rereadpt /dev/vdb" then "lsblk -o NAME,UUID,PARTUUID" does | 10:56 |
| dtantsur | WUT | 10:57 |
| dtantsur | hmmm | 10:57 |
| dtantsur | no, seriously, WUT | 10:57 |
| dtantsur | I highly suspect now that putting it before udev settle will fix the things | 10:57 |
| rpittau | should we put udevadm settle before partx ? | 11:00 |
| arne_wiebalck | "blockdev --rereadpt /dev/vdb && udevadm settle && lsblk -o NAME,UUID,PARTUUID" works fine | 11:00 |
| arne_wiebalck | but it does not fix the original issue | 11:00 |
| arne_wiebalck | https://usercontent.irccloud-cdn.com/file/psF0cDDs/imagetesting.png | 11:00 |
| arne_wiebalck | blockdev after udev should read udev after blockdev | 11:01 |
| arne_wiebalck | these are the tests I ran | 11:01 |
| arne_wiebalck | the green blocks are errors | 11:01 |
| arne_wiebalck | default image means unpatched | 11:01 |
| arne_wiebalck | blockdev image means the proposed patch | 11:01 |
| arne_wiebalck | pls don't ask me why :) | 11:02 |
| dtantsur | jesus | 11:02 |
| dtantsur | this is really Monday | 11:02 |
| arne_wiebalck | yep :) | 11:03 |
| dtantsur | what if we do udev settle twice: before and after? :) | 11:03 |
| rpittau | heh was going to say that! :D | 11:03 |
| arne_wiebalck | I did not dare to say that :-D | 11:03 |
| rpittau | lol | 11:03 |
| dtantsur | well, let's at least try it as an experiment? | 11:03 |
| arne_wiebalck | ok, how about I try this | 11:03 |
| dtantsur | although I suspect sleep(5) can work just as well.... | 11:03 |
| dtantsur | rpittau: JFYI we have a lot of packages in lower-constraints that are not compatible with python 3.10 | 11:04 |
| arne_wiebalck | yes: we do know if it is the udev or just the additional time | 11:04 |
| rpittau | dtantsur: yeah, I'm aware..... | 11:04 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable Redfish and iPXE by default https://review.opendev.org/c/openstack/ironic/+/816721 | 11:04 |
| dtantsur | and adding sushy in ^^^ revealed that we have an incompatible version of stevedore | 11:04 |
| arne_wiebalck | "blockdev --rereadpt /dev/vdb && ls && lsblk -o NAME,UUID,PARTUUID" does not work | 11:04 |
| * arne_wiebalck is building a new image | 11:07 | |
| dtantsur | all I read so far says that partx should be enough btw | 11:08 |
| dtantsur | oh, but we seem to be using it wrong >_< | 11:09 |
| dtantsur | from the man page it seems that `partx -u` should be used with a partition, not a disk | 11:09 |
| dtantsur | sigh | 11:09 |
| dtantsur | SIGH | 11:09 |
| dtantsur | it seems like we should be using `partx -a` instead | 11:10 |
| arne_wiebalck | `partx -a` seems to be the correct way, I agree | 11:11 |
| arne_wiebalck | `partx -a /dev/vdb` does not work on my VM | 11:13 |
| dtantsur | \o/ | 11:14 |
| arne_wiebalck | :-D | 11:14 |
| rpittau | from the source code it seems -u can be used with a device to update all the partitions on that device https://github.com/util-linux/util-linux/blob/master/disk-utils/partx.c#L453 | 11:14 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable Redfish by default https://review.opendev.org/c/openstack/ironic/+/816721 | 11:15 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable iPXE boot interface by default https://review.opendev.org/c/openstack/ironic/+/816824 | 11:17 |
| arne_wiebalck | I will try `partx -a` none the less | 11:19 |
| *** jssfr is now known as foorl | 11:33 | |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Enable Redfish by default https://review.opendev.org/c/openstack/ironic/+/816721 | 12:51 |
| * TheJulia doesn’t want to get out of bed | 12:56 | |
| dtantsur | TheJulia: don't, it's a huge mistake | 12:58 |
| TheJulia | So Admril Ackbar is screaming “it’s a trap”? | 12:59 |
| dtantsur | absolutely | 13:00 |
| TheJulia | I eventually have to drive out to my father’s place this morning | 13:00 |
| janders | see you on Monday Ironic o/ | 13:09 |
| janders | have a great weekend everyone | 13:09 |
| dtantsur | you too | 13:11 |
| rpittau | thanks janders :) | 13:12 |
| rpittau | dtantsur: for python 3.10 to work 100% we need to wait next release of eventlet, I guess :/ | 13:15 |
| dtantsur | le sigh | 13:16 |
| TheJulia | dtantsur: fyi https://review.opendev.org/c/openstack/ironic-python-agent/+/816723 came back as "yes, works perfectly now" on Lenovo sr650s | 13:32 |
| TheJulia | with the first patch as well | 13:32 |
| TheJulia | success() | 13:32 |
| dtantsur | nice! | 13:33 |
| dtantsur | TheJulia: if you want to scream in the darkness, janders has filed https://storyboard.openstack.org/#!/story/2009663 | 13:34 |
| rpittau | "standards" | 13:35 |
| dtantsur | right | 13:35 |
| dtantsur | it may not be obvious from the story, but UsbCd is a non-standard value | 13:35 |
| rpittau | no, just saw now, it's clearly mentioned | 13:36 |
| TheJulia | dtantsur: lolsob | 13:37 |
| * TheJulia needs to stop IRC'ing from bed and go head out to the ranch | 13:37 | |
| dtantsur | the worst thing, the change won't be backportable (we need to talk with janders about this) | 13:39 |
| dtantsur | maybe we need to use a low level HTTP query in ironic for the time being | 13:40 |
| dtantsur | (then update sushy and fix it) | 13:40 |
| TheJulia | ugh | 13:40 |
| dtantsur | sushy won't accept a non-standard value even before my enum patches | 13:40 |
| dtantsur | and with enum patches it will become even harder | 13:40 |
| TheJulia | I don't entirely grok the enum stuff your doing right now | 13:42 |
| TheJulia | I feel like I missed a memo, but I've had tons going on | 13:42 |
| * TheJulia goes on the great sock hunt fearing the sock gnomes have stolen them all | 13:43 | |
| dtantsur | replacing string constants with python enums | 13:50 |
| dtantsur | and added a script to generate them from DMTF schemas | 13:50 |
| TheJulia | I figured you were replacing them, just not on the driver why... aside from you know sanity | 13:50 |
| dtantsur | sanity, first and foremost :) | 13:51 |
| * dtantsur tries to get a booster vaccination, brb | 13:58 | |
| TheJulia | good luck | 13:59 |
| TheJulia | next month, I hope | 13:59 |
| rpittau | bye everyone, have a great weekend! o/ | 14:46 |
| opendevreview | Merged openstack/ironic-python-agent master: Fix UEFI record regex https://review.opendev.org/c/openstack/ironic-python-agent/+/816723 | 14:59 |
| * dtantsur back | 15:48 | |
| arne_wiebalck | dtantsur: I have Rally test the `partx -a` image now and will leave it like this over the weekend, maybe switching to the old `partx -u` along the way to see that this still breaks things. I will see how things look like on Monday. | 16:09 |
| opendevreview | Verification of a change to openstack/ironic master failed: [Trivial] Clarify conditions under which power recovery is attempted https://review.opendev.org/c/openstack/ironic/+/816733 | 16:09 |
| dtantsur | arne_wiebalck++ | 16:09 |
| dtantsur | TheJulia: should we backport the UEFI regex patch? | 16:16 |
| TheJulia | dtantsur: both, but I can do it next week | 16:32 |
| arne_wiebalck | bye everyone, see you next week o/ | 16:35 |
| opendevreview | Verification of a change to openstack/ironic master failed: [Trivial] Clarify conditions under which power recovery is attempted https://review.opendev.org/c/openstack/ironic/+/816733 | 16:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Move manage_uefi from the image extension to a public location https://review.opendev.org/c/openstack/ironic-python-agent/+/815651 | 17:02 |
| dtantsur | TheJulia: could you at some point check that I'm applying your changes correctly in ^^^? | 17:02 |
| TheJulia | dtantsur: tag me on it and I’lol try looking in the next day or so | 17:04 |
| dtantsur | cool, thanks | 17:04 |
| dtantsur | have a great weekend y'all! | 17:51 |
| opendevreview | Merged openstack/ironic master: Fix idrac-wsman deploy with existing non-BIOS jobs https://review.opendev.org/c/openstack/ironic/+/816521 | 18:26 |
| *** yoctozepto8 is now known as yoctozepto | 21:28 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
