| rpittau | good morning ironic! o/ | 08:17 |
|---|---|---|
| rpittau | dtantsur: left a couple of comments in https://review.opendev.org/c/openstack/ironic/+/827882 | 08:39 |
| dtantsur | rpittau: responded to one, will update the typo | 09:05 |
| dtantsur | morning folks | 09:05 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/827882 | 09:06 |
| rpittau | good morning dtantsur :) | 09:06 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: set the tempest plugin's partition_netboot option https://review.opendev.org/c/openstack/ironic/+/827481 | 09:06 |
| dtantsur | rpittau: updated ^^^ | 09:06 |
| rpittau | dtantsur: did you miss the typo in devstack/lib/ironic ? | 09:08 |
| dtantsur | was it there too? | 09:08 |
| rpittau | yeah | 09:08 |
| rpittau | for the same variable | 09:08 |
| dtantsur | damn. on it. | 09:08 |
| rpittau | :) | 09:08 |
| dtantsur | rpittau: actually, wdyt if I squash the two patches together? | 09:09 |
| dtantsur | I mean, https://review.opendev.org/c/openstack/ironic/+/827481 | 09:09 |
| rpittau | mmm yeah, 2nd one is not that big | 09:09 |
| dtantsur | and it's quite related, especially the logic around "bios" | 09:09 |
| rpittau | yep, makes sense | 09:09 |
| dtantsur | the problem of adaptive auto-completer: once I typed CIRROR_ARCH once, it started suggesting it everywhere :) | 09:10 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/827882 | 09:13 |
| dtantsur | rpittau: should be it ^^^ | 09:13 |
| rpittau | yep, lgtm | 09:16 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Apply the correct image reference when booting an instance https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/827340 | 09:17 |
| dtantsur | rpittau: this is the 2nd part of the puzzle ^^ | 09:17 |
| dtantsur | although we may need to backport the devstack patch all the way to all supported branches first... | 09:19 |
| rpittau | yeah tempest change would break CI | 09:22 |
| dtantsur | rpittau: I'll put a WIP on it, but could you review nonetheless? | 09:23 |
| rpittau | sure | 09:23 |
| dtantsur | I wonder how many branches we still support. and how many of them still have at least somewhat working CI. | 09:24 |
| dtantsur | I intend to fix stuff up to victoria, and say that everything else much cap the tempest plugin | 09:24 |
| rpittau | up to victoria, I think, I should have the list somewhere | 09:24 |
| dtantsur | maybe we can cap it in victoria already? | 09:24 |
| rpittau | we could, yeah | 09:25 |
| rpittau | this is what I have for ironic https://paste.openstack.org/show/bLqXWgyiL0G7F9XX074t/ | 09:26 |
| dtantsur | judging by https://review.opendev.org/c/openstack/ironic/+/827227, Victoria is still operational on Ironic | 09:26 |
| dtantsur | yeah, looks valid | 09:29 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix release mapping (19.1 -> 20.0) https://review.opendev.org/c/openstack/ironic/+/829308 | 10:04 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 10:45 |
| opendevreview | Verification of a change to openstack/ironic master failed: Fix release mapping (19.1 -> 20.0) https://review.opendev.org/c/openstack/ironic/+/829308 | 11:12 |
| opendevreview | Merged openstack/bifrost master: Stop using sub-sections of driver_info https://review.opendev.org/c/openstack/bifrost/+/826307 | 11:13 |
| iurygregory | good morning Ironic | 11:38 |
| MahnoorAsghar | good morning! | 11:45 |
| opendevreview | Merged openstack/ironic stable/victoria: Avoid non-Stream CentOS, remove Grenade job https://review.opendev.org/c/openstack/ironic/+/827227 | 11:51 |
| opendevreview | Merged openstack/ironic-python-agent master: Move prepare_boot_partitions_for_softraid to raid_utils https://review.opendev.org/c/openstack/ironic-python-agent/+/826814 | 12:46 |
| opendevreview | Merged openstack/ironic stable/wallaby: Avoid non-Stream CentOS https://review.opendev.org/c/openstack/ironic/+/827226 | 13:15 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Create random character devices so systemd does not crash https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/829391 | 13:33 |
| TheJulia | rpittau: kept it simple since the vfs layers have some weird behavior | 13:39 |
| TheJulia | i.e. is there any sort of file | 13:39 |
| rpittau | TheJulia: good morning! :) | 13:40 |
| rpittau | that works! | 13:40 |
| rpittau | you have a blank space at the end of L33 if you want to fix it now | 13:40 |
| TheJulia | please, in a meeting | 13:48 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Create random character devices so systemd does not crash https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/829391 | 13:57 |
| TheJulia | fixed | 13:58 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Continue decomposing the "How to" documentation https://review.opendev.org/c/openstack/bifrost/+/829245 | 14:07 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 14:07 |
| dtantsur | morning iurygregory, MahnoorAsghar, TheJulia | 14:09 |
| dtantsur | iurygregory: if you have some time today: https://review.opendev.org/c/openstack/ironic/+/827882 | 14:09 |
| opendevreview | Merged openstack/ironic master: Fix release mapping (19.1 -> 20.0) https://review.opendev.org/c/openstack/ironic/+/829308 | 14:15 |
| MahnoorAsghar | o/ | 14:19 |
| *** pmannidi is now known as pmannidi|brb | 14:20 | |
| *** pmannidi|brb is now known as pmannidi|AFK | 14:20 | |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 14:40 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fetch uuid if it is not present in the inventory https://review.opendev.org/c/openstack/bifrost/+/829554 | 14:54 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 14:59 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fetch uuid if it is not present in the inventory https://review.opendev.org/c/openstack/bifrost/+/829554 | 15:02 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:02 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:19 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Fetch uuid if it is not present in the inventory https://review.opendev.org/c/openstack/bifrost/+/829554 | 15:24 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:24 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:29 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Add `./bifrost-cli deploy` and refactor bifrost-deploy-nodes-dynamic https://review.opendev.org/c/openstack/bifrost/+/829517 | 15:36 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Dynamic inventory: do not set network_interface https://review.opendev.org/c/openstack/bifrost/+/829562 | 15:36 |
| dtantsur | bugs, bugs everywhere! | 15:37 |
| TheJulia | all the bugs! | 15:44 |
| dtantsur | okay, this works at least partly, I think I'll continue tomorrow | 15:51 |
| dtantsur | o/ | 15:51 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix prepare ramdisk for 'wait' states https://review.opendev.org/c/openstack/ironic/+/823311 | 16:53 |
| opendevreview | Merged openstack/ironic master: CI: use a custom cirros partition image instead of the default https://review.opendev.org/c/openstack/ironic/+/827882 | 16:54 |
| rpittau | good night! o/ | 16:58 |
| opendevreview | Julia Kreger proposed openstack/ironic-python-agent-builder master: Create random character devices so systemd does not crash https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/829391 | 17:20 |
| opendevreview | Mahnoor Asghar proposed openstack/ironic master: [WIP] Create API documentation from docstrings https://review.opendev.org/c/openstack/ironic/+/827200 | 17:37 |
| TheJulia | arne_wiebalck: I just posted https://review.opendev.org/c/openstack/nova/+/829613, which should make the world a happier place | 22:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!