| kubajj | Good morning Ironic! o/ | 06:37 |
|---|---|---|
| rpittau | good morning ironic! o/ | 07:04 |
| dtantsur | TheJulia: one thing I've noticed: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/925981 accidentally includes https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/923591 and I wonder if it failed in the same way (no CI logs any more) | 12:44 |
| dtantsur | as in: the PXE booting test is actually not working, it only passes because of the wrong boot interface :( | 12:45 |
| opendevreview | cid proposed openstack/ironic master: Replace reference to ari/aki https://review.opendev.org/c/openstack/ironic/+/927202 | 12:46 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Temporarily disable PXE booting test in the standalone job https://review.opendev.org/c/openstack/ironic/+/927265 | 12:48 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: A simple standalone test for in-band inspection https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/925981 | 12:49 |
| dtantsur | Checking my assumptions in this chain ^^ | 12:49 |
| TheJulia | Umm... hmmm | 13:14 |
| TheJulia | Looking at your recent change, thank you for loosening some of the interface setting constraints/logic by the way, I'm almost wondering if we're getting the same plugin we clone out of devstack | 13:14 |
| rpittau | bye everyone, see you in 3 weeks! o/ | 15:05 |
| * TheJulia needs a 2-3 week vacation | 15:16 | |
| TheJulia | my corgi has "end of call detection" enabled apparently | 16:44 |
| TheJulia | he immediately ran to the door and back to look at me. | 16:44 |
| iurygregory | ++ to a 2-3 week vacation =) | 16:54 |
| * JayF is likely going to do just that around EOY/December | 16:57 | |
| samcat116 | Hi folks, if I'm seeing the following error using the baremetal cli with a configdrive, does it mean these tools are missing from the ironic_conductor container or are the on the IPA side: | 17:23 |
| samcat116 | Error generating the configdrive. Make sure the "genisoimage", "mkisofs" or "xorrisofs" tool is installed. Error: [Errno 2] No such file or directory: 'xorrisofs' | 17:23 |
| samcat116 | Not sure if the configdrive image is put together conductor side or on the node ramdisk side | 17:24 |
| JayF | I think the answer is actually both | 17:29 |
| JayF | but I can check real quick | 17:29 |
| samcat116 | https://bugs.launchpad.net/kolla/+bug/2077908 | 17:30 |
| JayF | nope, just ironic | 17:30 |
| JayF | you need it on conductors | 17:30 |
| samcat116 | This bug has the full error log, it was an error during the write step | 17:30 |
| samcat116 | Ok so then thats a valid bug for the Kolla folks I guess | 17:30 |
| JayF | hmm maybe it needs it in ipa | 17:30 |
| JayF | gimme a sec to check again after seeing the log | 17:30 |
| samcat116 | Funny you won't hit that it seems if you go through the Nova side, as the nova_compute_ironic container has those utilities on it | 17:31 |
| JayF | yep | 17:31 |
| JayF | we support both you send us data, we build the drive | 17:31 |
| JayF | and you send us the image, we write it | 17:31 |
| cardoe | TheJulia: they all definitely learn that one. I tried to teach mine to ring a bell by the door when she needed to go out. Worked great at first. Got her other buttons for things. But the bell remains as a human do something trigger. | 17:32 |
| TheJulia | cardoe: doh! | 17:33 |
| JayF | samcat116: you need it on conductors, and I can't find the code that uses it; but we *install it* in IPA ramdisks by default: https://github.com/openstack/ironic-python-agent-builder/blob/master/dib/ironic-python-agent-ramdisk/package-installs.yaml#L15 | 17:34 |
| TheJulia | Depends on the invocation, specifically I think that three different option is actually the client machine | 17:36 |
| TheJulia | I *think* | 17:36 |
| samcat116 | Whats weird is that this was working for us and now its not, so trying to figure that out as we definitely didn't uninstall or install anything on the container side | 17:36 |
| cardoe | So JayF I don't think that's in the IPA image. That's FOR building the IPA image. | 17:36 |
| JayF | cardoe: that's inaccurate, the package-installs.yaml file in an element is packages to be installed inside the ramdisk | 17:37 |
| cardoe | But like you and Julia said, Ironic supports building the config drive either in the conductor OR client side. | 17:37 |
| cardoe | ah yeah it's inside... that's weird. | 17:39 |
| TheJulia | I don't know why it would be inside of the ramdisk | 17:39 |
| TheJulia | I'd have to dig at the ipa code to grok why | 17:39 |
| TheJulia | oh yeah, that would be coming from the agent | 17:41 |
| TheJulia | samcat116: what IPA images are being used? | 17:41 |
| samcat116 | https://tarballs.opendev.org/openstack/ironic-python-agent/dib/files/ | 17:42 |
| samcat116 | The ones from here | 17:42 |
| cardoe | Either way, the baremetal cli with a config-drive parameter isn't gonna use the IPA for that part. | 17:43 |
| cardoe | It's gonna either build it client side or conductor side. | 17:43 |
| cardoe | I bet I know what the IPA has it JayF. If you build the config drive client side, it won't have the correct metadata for the server ID / name. So I bet the IPA opens it up injects it and repacks it. | 17:44 |
| cardoe | That's purely jump to conclusions. | 17:44 |
| JayF | let me put it this way | 17:44 |
| JayF | grep ipa codebase for `iso` doesn't show anything interesting | 17:45 |
| JayF | similarly in ironic-lib | 17:45 |
| JayF | so unsure where it's used, but it's installed | 17:46 |
| TheJulia | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/baremetal/configdrive.py | 17:50 |
| JayF | :-O | 17:54 |
| JayF | I had no idea | 17:54 |
| JayF | samcat116: ^ | 17:54 |
| TheJulia | ironic/conductor/utils.py:from openstack.baremetal import configdrive as os_configdrive | 17:56 |
| TheJulia | it is the conductor which is lacking it | 17:56 |
| JayF | okay this is super freakin' cool | 17:56 |
| JayF | I had no idea that code was in openstacksdk | 17:56 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add inspection rules https://review.opendev.org/c/openstack/ironic/+/918303 | 18:10 |
| opendevreview | cid proposed openstack/ironic master: [WIP] Add inspection rules https://review.opendev.org/c/openstack/ironic/+/918303 | 18:14 |
| iurygregory | it took some time, but I was able to find the release where ilo added the update_firmware clean step | 18:16 |
| iurygregory | \o/ | 18:16 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Disable MD5 support https://review.opendev.org/c/openstack/ironic/+/927314 | 19:01 |
| cid | o/ | 19:47 |
| TheJulia | Please pair https://review.opendev.org/c/openstack/ironic/+/927314 with "Voltaire - When You're Evil" | 20:02 |
| opendevreview | Merged openstack/ironic master: Replace reference to ari/aki https://review.opendev.org/c/openstack/ironic/+/927202 | 23:11 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!