| arne_wiebalck | Good morning, Ironic! | 07:29 |
|---|---|---|
| arne_wiebalck | stevebaker[m]: I agree, not sure this is the issue I see. | 07:29 |
| arne_wiebalck | stevebaker[m]: or better: the issue which breaks image building for me | 07:29 |
| arne_wiebalck | stevebaker[m]: problem is there is no real error in the logs ... the process stops at some point and starts cleanup | 07:30 |
| arne_wiebalck | stevebaker[m]: and the lsblk issue is one thing I found so far | 07:31 |
| stevebaker[m] | I've proposed a revert revert, any chance you could test it on what broke? | 07:31 |
| arne_wiebalck | stevebaker[m]: sure | 07:31 |
| rpittau | good morning ironic! o/ | 07:59 |
| rpittau | congrats arne_wiebalck :) | 08:12 |
| arne_wiebalck | good morning and thanks rpittau :) | 08:12 |
| opendevreview | Merged openstack/ironic bugfix/20.0: Update .gitreview for bugfix/20.0 https://review.opendev.org/c/openstack/ironic/+/830837 | 08:49 |
| arne_wiebalck | stevebaker[m]: seems to work now :) | 09:12 |
| opendevreview | Riccardo Pittau proposed openstack/ironic master: Use pycdlib to extract deploy iso https://review.opendev.org/c/openstack/ironic/+/819121 | 09:12 |
| dtantsur | <insert an adjective> morning, ironic | 09:31 |
| rpittau | hey dtantsur :/ | 09:39 |
| dtantsur | hmm, is the inspector CI broken now? | 09:43 |
| rpittau | I was wondering the same | 09:43 |
| MahnoorAsghar | o/ | 09:55 |
| iurygregory | morning Ironic o/ | 11:16 |
| rpittau | hey iurygregory :) | 11:32 |
| iurygregory | o/ | 11:57 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Added element to enable RDO deps repo to install sysbench https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/825313 | 12:10 |
| ameya49 | Hi ironic | 12:29 |
| iurygregory | arne_wiebalck, congratulations =) | 12:33 |
| iurygregory | Hello ameya49 | 12:33 |
| ameya49 | Was going through introspection_basic test case in ironic-tempest-plugin where I found we are verifying node flavor | 12:35 |
| ameya49 | any particular reason doing it? | 12:37 |
| ameya49 | https://github.com/openstack/ironic-tempest-plugin/blob/fd43dcc3b607603978c51ca605dc70e1b784d782/ironic_tempest_plugin/tests/scenario/test_introspection_basic.py#L104 | 12:37 |
| ameya49 | Hi iurygregory : Hope you are doing well :) | 12:39 |
| iurygregory | I'm doing good | 12:42 |
| iurygregory | The reason behind this is to be able to verify the data collected from introspection | 12:42 |
| ameya49 | Yes, we are doing against introspected data, but as discuss earlier it wont be equal to actual values wrt hardware | 12:47 |
| iurygregory | yeah, we got an agreement that we could change the tests so it would work | 12:48 |
| iurygregory | I can push a patch with the fixes | 12:49 |
| ameya49 | iurygregory : Im working on the fixes and will push the patch soon :) Thanks | 12:49 |
| iurygregory | ameya49, awesome tks! | 12:50 |
| ameya49 | question regarding flavor was, is there need to verify flavor values? | 12:51 |
| iurygregory | we do this to be sure it would match things as expected (the flavor says 4 vcpus, the introspection data has 4 vcpus ..) since the tests were only used in the upstream CI this wouldn't be a problem (because we define the virtual baremetal) | 12:53 |
| ameya49 | ok | 12:54 |
| ameya49 | Thanks iurygregory! | 12:55 |
| iurygregory | yw =) | 12:55 |
| * iurygregory going to reboot the laptop brb | 12:59 | |
| TheJulia | good morning | 13:19 |
| arne_wiebalck | TheJulia: good morning | 13:22 |
| arne_wiebalck | iurygregory: thanks! | 13:22 |
| arne_wiebalck | TheJulia: the latest diskimage-builder change seems to be ok :) | 13:22 |
| TheJulia | arne_wiebalck: ack, thanks! | 13:23 |
| dtantsur | morning TheJulia | 13:28 |
| iurygregory | morning TheJulia | 13:29 |
| rpittau | not sure I'm happy to see that is always a different job failing in the inspector CI | 13:55 |
| opendevreview | Merged openstack/ironic-inspector bugfix/10.10: Update .gitreview for bugfix/10.10 https://review.opendev.org/c/openstack/ironic-inspector/+/829127 | 14:13 |
| * dtantsur walk, brb | 14:14 | |
| zigo | Thanks to whoever switched ironic-inspector-client to stestr and a standard way to run tests, much appreciated over here (ie: downstream Debian...). | 14:48 |
| zigo | The more a package is doing things "normally" the easier it gets for me. | 14:50 |
| rpittau | bye everyone, have a great weekend! o/ | 15:35 |
| opendevreview | Merged openstack/ironic-inspector master: Support listening on a Unix socket https://review.opendev.org/c/openstack/ironic-inspector/+/827124 | 16:44 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 16:45 |
| NobodyCam | Good Morning and OFC... | 16:46 |
| NobodyCam | TGIF | 16:47 |
| NobodyCam | happy Friday Ironic'ers | 16:47 |
| arne_wiebalck | hey NobodyCam o/ | 16:49 |
| NobodyCam | hey hey arne_wiebalck o/ happy Friday | 16:49 |
| NobodyCam | crazy days | 16:49 |
| arne_wiebalck | yeah | 16:50 |
| dtantsur | see you on Monday folks. take care o/ | 17:44 |
| iurygregory | morning NobodyCam o/ | 18:20 |
| iurygregory | congratulations arne_wiebalck o// | 18:20 |
| iurygregory | new TC member :D | 18:20 |
| arne_wiebalck | iurygregory: thanks :) | 18:22 |
| JayF | congrats arne_wiebalck | 18:28 |
| arne_wiebalck | thanks JayF | 18:30 |
| arne_wiebalck | bye everyone, see you next week o/ | 19:44 |
| TheJulia | wheeeeeeee | 20:37 |
| TheJulia | fun bugs are fun | 20:37 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent master: Create fstab entry with appropriate label https://review.opendev.org/c/openstack/ironic-python-agent/+/831029 | 20:45 |
| TheJulia | hjensas: ^^^ | 20:45 |
| JayF | I did that when installing gentoo last time; used a UUID and addressed it as a PARTITION_UUID | 20:47 |
| JayF | it's extremely confusing | 20:47 |
| TheJulia | exceptionally | 20:48 |
| TheJulia | and we've got logs where running an lsblk after creating everything doesn't populate a uuid, but run it like 10 seconds later when we go to do bootloader setup, it is there | 20:49 |
| TheJulia | which *screams* io latency with buffer cached data... or update locked data in the kernel due to io cache | 20:49 |
| TheJulia | like, screams so hardcore | 20:49 |
| JayF | just run a few dozen `sync;` commands, I'm sure it'll fix it /s | 20:50 |
| TheJulia | lol, that is only data to be written ;) | 20:50 |
| TheJulia | I'm fairly sure there are some syncs in there, but regardless, best to just look it up... | 20:50 |
| JayF | Oh, I'm sure you fixed it in a good way, I'm just making jokes | 20:51 |
| TheJulia | i know | 20:52 |
| TheJulia | it just ate my brain earlier | 20:52 |
| JayF | it took me embarassingly long to find the problem on my single computer, I can't imagine how confusing it was through the IPA lens | 20:52 |
| TheJulia | we had walk all the way back into ironic lib code to put the pieces together :( | 20:53 |
| opendevreview | Merged openstack/ironic master: Update python-dracclient version https://review.opendev.org/c/openstack/ironic/+/830348 | 20:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!