| rpittau | good morning ironic! o/ | 07:27 |
|---|---|---|
| rpittau | 2 very quick reviews if anyone has a moment https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848826 https://review.opendev.org/c/openstack/bifrost/+/846193 thanks! | 07:34 |
| opendevreview | Merged openstack/tenks master: Use yoga upper constraints for openstack-tox-py36 https://review.opendev.org/c/openstack/tenks/+/848731 | 10:07 |
| janders | hey rpittau o/ | 10:42 |
| janders | arne_wiebalck I think your work is completely complaint with standards, but unfortuntely OEMs divert from them to a worrying degree (which may sanction further workarounds) | 10:43 |
| rpittau | hey janders :) | 10:47 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/xena: Add smartmontools package to DIB image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848608 | 11:09 |
| opendevreview | Verification of a change to openstack/ironic-python-agent-builder stable/yoga failed: Add smartmontools package to DIB image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848607 | 11:31 |
| Kirill_ | Hi everyone. Does someone know about vnc for HPE | 11:33 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/wallaby: Add smartmontools package to DIB image https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848609 | 11:34 |
| iurygregory | good morning Ironic | 11:43 |
| opendevreview | Swapnil Machikar proposed openstack/ironic-tempest-plugin master: Add iDRAC Redfish sync boot mode test https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/848709 | 11:51 |
| iurygregory | rpittau, +W on both | 11:56 |
| rpittau | thanks iurygregory :) | 11:59 |
| dtantsur | Kirill_: I don't think anyone does | 12:02 |
| opendevreview | Verification of a change to openstack/bifrost master failed: Remove old unused CentOS Stream 8 based jobs https://review.opendev.org/c/openstack/bifrost/+/846193 | 12:46 |
| * TheJulia attempts to wake up | 13:06 | |
| dtantsur | TheJulia: futile | 13:06 |
| TheJulia | as in resistance is futile? | 13:07 |
| rpittau | I guess this case is "resistance to sleep" | 13:07 |
| TheJulia | I suspect my multimeters cannot measure this resistance | 13:08 |
| iurygregory | I agree | 13:19 |
| TheJulia | :) | 13:21 |
| dtantsur | :D | 13:32 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Remove CentOS Stream 8 leftovers https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/848826 | 13:39 |
| TheJulia | rloo: fwiw, I'd also appreciate a review on https://review.opendev.org/c/openstack/ironic/+/835769 when you have a little time. I *really* wish anaconda and the related tooling had better documentation. :( | 14:01 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 14:21 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy-tools master: Replace the netboot job with a local-boot one https://review.opendev.org/c/openstack/sushy-tools/+/848964 | 14:25 |
| dtantsur | reviews appreciated ^^^ is blocking the netboot removal | 14:28 |
| opendevreview | Dmitry Tantsur proposed openstack/sushy master: Replace the netboot job with a local-boot one https://review.opendev.org/c/openstack/sushy/+/848965 | 14:29 |
| dtantsur | same ^^ | 14:29 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-lib master: Replace the netboot job with a local-boot one https://review.opendev.org/c/openstack/ironic-lib/+/848966 | 14:31 |
| rloo | TheJulia: done. And yeah, better documentation, sorry about that... (as I learn more about anaconda, etc.) Thx for improving! | 14:35 |
| TheJulia | rloo: oh no worries! not your or anyone's at your employer's fault. | 14:35 |
| TheJulia | rloo: the issue is anaconda's docs are... lacking. | 14:35 |
| rloo | yup :-( | 14:36 |
| TheJulia | dracut's anaconda invocation is undocumented | 14:36 |
| TheJulia | like... nothing to be found anywhere, you have to read the code to see what it is doing | 14:36 |
| TheJulia | or how to configure/influence it | 14:36 |
| TheJulia | Queue twitching :) | 14:36 |
| rloo | yup. so glad you are becoming an expert on it... | 14:36 |
| TheJulia | oh no, not the expert | 14:37 |
| TheJulia | nope nope nope | 14:37 |
| TheJulia | All the nopes ;) | 14:37 |
| rloo | ha ha | 14:37 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-python-agent master: Drop support for instance netboot https://review.opendev.org/c/openstack/ironic-python-agent/+/848969 | 14:38 |
| opendevreview | Merged openstack/bifrost master: Remove old unused CentOS Stream 8 based jobs https://review.opendev.org/c/openstack/bifrost/+/846193 | 14:46 |
| opendevreview | Merged openstack/tenks master: Remove Python 2 leftovers https://review.opendev.org/c/openstack/tenks/+/848698 | 14:53 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.6: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848866 | 15:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.3: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848867 | 15:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/xena: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848868 | 15:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent bugfix/8.1: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848869 | 15:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/wallaby: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848990 | 15:04 |
| opendevreview | Riccardo Pittau proposed openstack/ironic-python-agent stable/victoria: Gather details about bond interfaces if present https://review.opendev.org/c/openstack/ironic-python-agent/+/848991 | 15:05 |
| TheJulia | rloo: do you remember what liveimg_url is? | 15:15 |
| TheJulia | in anaconda/kickstart context | 15:16 |
| JayF | it's the URL to the image | 15:16 |
| TheJulia | for stage2? | 15:16 |
| JayF | which I think is actually a tarball that is extracted for the base system | 15:16 |
| JayF | I am like, 90% sure | 15:16 |
| JayF | but this knowledge is dusty and purple-colored | 15:17 |
| TheJulia | purple is acceptable :) | 15:17 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 15:17 |
| TheJulia | dusty expected as well | 15:17 |
| JayF | TheJulia: I just wrote the longest email of my life, it feels like, fwiw, you should go have a read | 15:17 |
| TheJulia | oh my... | 15:18 |
| TheJulia | dooooooom? | 15:18 |
| rloo | https://pykickstart.readthedocs.io/en/latest/kickstart-docs.html#liveimg if that helps | 15:18 |
| JayF | that is exactly what I thought it was, awesome \o/ | 15:19 |
| JayF | but a docs link is 100x more helpful | 15:19 |
| JayF | o/ rloo | 15:19 |
| rloo | i got that link from ... https://specs.openstack.org/openstack/ironic-specs/specs/18.1/anaconda-deploy-interface.html#os-image-format | 15:19 |
| TheJulia | ahh, okay | 15:19 |
| rloo | thinking we might want to incorporate a lot of stuff from that spec, into the anaconda deploy interface doc... | 15:19 |
| rloo | the assumption (I guess) being that whoever uses the anaconda deploy interface, is intimate with anaconda/kickstart... | 15:20 |
| TheJulia | yeah | 15:22 |
| TheJulia | so, okay | 15:22 |
| TheJulia | hmmm | 15:22 |
| TheJulia | yeah, we need a generalized doc update too | 15:22 |
| TheJulia | i think | 15:22 |
| TheJulia | so, I'm kind of also writing out an overall flow/interaction since this is so confusing to a newcomer or even someone who has hacked on the code :) | 15:45 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Add audit middleware options to ironic.conf https://review.opendev.org/c/openstack/ironic/+/848978 | 15:47 |
| opendevreview | Takashi Kajinami proposed openstack/ironic master: Add audit middleware options to ironic.conf https://review.opendev.org/c/openstack/ironic/+/848978 | 15:51 |
| * dtantsur wonders if he needs to repair redeploy-dynamic or just remove it | 15:55 | |
| TheJulia | I thought you were going to eradicate it? | 15:56 |
| dtantsur | I then decided to give it a chance.. but then it's not compatible with TLS | 15:56 |
| dtantsur | On the other hand, you have a good point: we need an example on how to write playbooks for bifrost | 15:57 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 16:04 |
| TheJulia | ahh, yeah | 16:07 |
| rpittau | good night! o/ | 16:08 |
| TheJulia | goodnight | 16:09 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 16:35 |
| dtantsur | soooo broken | 16:59 |
| TheJulia | :( | 16:59 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 17:03 |
| opendevreview | Dmitry Tantsur proposed openstack/bifrost master: [WIP] Test redeploy-dynamic when cirros/tinyipa are used https://review.opendev.org/c/openstack/bifrost/+/848963 | 17:05 |
| opendevreview | Merged openstack/ironic master: Move logging out of skip_automated_cleaning https://review.opendev.org/c/openstack/ironic/+/848875 | 17:33 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Make anaconda non-image deploys sane https://review.opendev.org/c/openstack/ironic/+/834709 | 18:40 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Do not require stage2 for anaconda with standalone https://review.opendev.org/c/openstack/ironic/+/835769 | 18:40 |
| TheJulia | rloo: ^^ revised both patches. I spotted an issue that I think you were starting to hit up against the second one but it didn't click until I tried to run the unit tests that there was an issue cascading further in. Fixed that. | 18:41 |
| rloo | TheJulia: thx for the update! will try to look later today or tomorrow. | 18:42 |
| TheJulia | Thanks | 18:43 |
| iurygregory | TheJulia, do you remember if we found a workaround for https://storyboard.openstack.org/#!/story/2008763 ? I think we did something upstream for it... | 22:09 |
| TheJulia | iurygregory: yeah, i wrote a patch to dedup it | 22:23 |
| TheJulia | iurygregory: give me a few and I'll hunt it down | 22:24 |
| iurygregory | https://review.opendev.org/q/Ib61a7100a059e79a8b0901fd8f46b9bc41d657dc | 22:24 |
| iurygregory | this one? | 22:24 |
| TheJulia | yes | 22:24 |
| TheJulia | that is by label | 22:24 |
| iurygregory | ack | 22:25 |
| TheJulia | and there were bugs before that in the 8.x universe of stuffs that did do weird labeling | 22:25 |
| TheJulia | so it won't get those neessarilly | 22:25 |
| TheJulia | but it should clean up the primary/label being deployed | 22:25 |
| TheJulia | the inherent need is to inject the new record before rebooting | 22:25 |
| iurygregory | yeah, I'm going to check if we have downstream before looking at the problem they reported | 22:26 |
| TheJulia | and do it after deleting | 22:26 |
| TheJulia | I think | 22:26 |
| iurygregory | seems like they have many RHEL options (which doesn't make sense...) | 22:27 |
| TheJulia | so the failure on one time boot ops may ultimately be both a firmware and an interaction bug | 22:48 |
| TheJulia | *or* the table is full. | 22:48 |
| TheJulia | We've seen that some HBA's vendor's cared in UEFI mode, it would add every single device as bootable | 22:48 |
| TheJulia | via bios boot | 22:48 |
| TheJulia | needless to say, that was clearly a firmware bug | 22:48 |
| * TheJulia thinks it is time to call it a night | 22:57 | |
| opendevreview | Steve Baker proposed openstack/metalsmith master: Do a precise node lookup for unprovision https://review.opendev.org/c/openstack/metalsmith/+/849052 | 23:19 |
| *** rcastillo|rover is now known as rcastillo | 23:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!