*** ijw has quit IRC | 00:01 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Make local boot explicit on software raid tests https://review.opendev.org/679258 | 00:16 |
---|---|---|
* TheJulia creates some circular patches to see if things go *boom* | 00:18 | |
TheJulia | maybe circular | 00:18 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Provide a more useful error message https://review.opendev.org/679343 | 00:26 |
*** rloo has quit IRC | 00:48 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Make local boot explicit on software raid tests https://review.opendev.org/679258 | 00:51 |
*** ricolin has joined #openstack-ironic | 02:16 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic-inspector master: Update non-standalone job to use uwsgi https://review.opendev.org/675724 | 02:35 |
*** dsneddon has quit IRC | 03:44 | |
*** mkrai has joined #openstack-ironic | 03:52 | |
*** mkrai has quit IRC | 03:55 | |
*** mkrai has joined #openstack-ironic | 03:58 | |
*** mkrai has quit IRC | 04:09 | |
*** mkrai has joined #openstack-ironic | 04:11 | |
*** dsneddon has joined #openstack-ironic | 04:18 | |
*** jhesketh has quit IRC | 04:20 | |
*** mkrai has quit IRC | 04:22 | |
*** mkrai has joined #openstack-ironic | 04:23 | |
*** rachit7 has joined #openstack-ironic | 05:13 | |
*** hamzy has quit IRC | 05:16 | |
*** e0ne has joined #openstack-ironic | 05:19 | |
*** e0ne has quit IRC | 05:20 | |
*** pradip_k has joined #openstack-ironic | 05:40 | |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 05:45 |
*** igordc has quit IRC | 05:46 | |
*** e0ne has joined #openstack-ironic | 05:51 | |
*** jtomasek has joined #openstack-ironic | 05:54 | |
*** jtomasek has quit IRC | 05:54 | |
*** jtomasek has joined #openstack-ironic | 05:55 | |
*** dsneddon has quit IRC | 06:06 | |
*** e0ne has quit IRC | 06:24 | |
arne_wiebalck | Good morning, ironic! | 06:26 |
rachit7 | Good morning arne_wiebalck | 06:34 |
arne_wiebalck | rachit7: o/ | 06:34 |
*** dsneddon has joined #openstack-ironic | 06:38 | |
*** e0ne has joined #openstack-ironic | 06:39 | |
*** dsneddon has quit IRC | 06:43 | |
*** rcernin has quit IRC | 06:48 | |
*** dsneddon has joined #openstack-ironic | 07:00 | |
*** dsneddon has quit IRC | 07:05 | |
*** trident has quit IRC | 07:10 | |
openstackgerrit | paresh sao proposed openstack/ironic master: Fixed problem with UEFI iSCSI boot for nic adapters https://review.opendev.org/674731 | 07:12 |
*** trident has joined #openstack-ironic | 07:18 | |
*** trident has quit IRC | 07:22 | |
*** trident has joined #openstack-ironic | 07:30 | |
*** dsneddon has joined #openstack-ironic | 07:36 | |
*** jhesketh has joined #openstack-ironic | 07:38 | |
*** dtantsur|afk is now known as dtantsur | 07:46 | |
*** pradip_k has quit IRC | 07:50 | |
openstackgerrit | Nisha Brahmankar proposed openstack/ironic-python-agent master: Adds CI job that uses DIB build images https://review.opendev.org/676239 | 07:57 |
*** lucasagomes has joined #openstack-ironic | 08:06 | |
*** dougsz has joined #openstack-ironic | 08:07 | |
dtantsur | morning folks | 08:15 |
dtantsur | can someone please check the outstanding IPA-builder patches? https://review.opendev.org/#/q/project:openstack/ironic-python-agent-builder+status:open | 08:15 |
dtantsur | I'd really love to finish this work within the Train timeframe | 08:16 |
*** stendulker has joined #openstack-ironic | 08:24 | |
*** andrein has joined #openstack-ironic | 08:24 | |
*** stendulker has quit IRC | 08:32 | |
*** alexmcleod has joined #openstack-ironic | 08:33 | |
*** derekh has joined #openstack-ironic | 08:35 | |
*** e0ne has quit IRC | 08:40 | |
*** e0ne has joined #openstack-ironic | 08:40 | |
openstackgerrit | Merged openstack/virtualbmc master: Tiny fix for documentation RST convention https://review.opendev.org/679187 | 08:41 |
*** e0ne has quit IRC | 08:46 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Do not reuse sqlite connection https://review.opendev.org/678797 | 08:49 |
*** dsneddon has quit IRC | 08:57 | |
*** dougsz has quit IRC | 09:00 | |
kaifeng | dtantsur: just curious, why the qemu is involved in building tinyipa? | 09:01 |
*** e0ne has joined #openstack-ironic | 09:03 | |
dtantsur | kaifeng: we need qemu-img from it | 09:03 |
dtantsur | it's used by IPA | 09:03 |
kaifeng | dtantsur: oh, I thought we use the tool from distributions | 09:07 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 09:08 |
dtantsur | kaifeng: you mean, from instance images? no, only grub is used from them (when used). | 09:09 |
dtantsur | and unfortunately, tinycorelinux has ancient (2.*?) qemu | 09:10 |
*** hamzy has joined #openstack-ironic | 09:10 | |
openstackgerrit | Shuo Liu proposed openstack/ironic master: Add ironic-specs link to readme.rst https://review.opendev.org/679403 | 09:20 |
kaifeng | dtantsur: got it, so we need the qemu-img in the ipa for writing images | 09:22 |
dtantsur | yeah | 09:22 |
kaifeng | I don't remember how it costs last time I build it (maybe around ~20 minutes), qemu seems to contain quite amount of code. | 09:23 |
kaifeng | I just wonder it seems we maybe only use raw image streaming for the tinyipa, maybe this qemu building doesn't need at all. | 09:24 |
dtantsur | we need to cover the conversion code in the CI | 09:27 |
dtantsur | it's worth investigating if we can strip down the qemu installation though | 09:28 |
*** dsneddon has joined #openstack-ironic | 09:30 | |
kaifeng | I think we could test the conversion in the dib job, apparently it has more memory available :) | 09:31 |
dtantsur | if we make it voting - sure | 09:33 |
*** dsneddon has quit IRC | 09:35 | |
kaifeng | dtantsur: I suspect we never covered the conversion workflow, I didn't find a job has stream_raw_images set to false.. | 09:37 |
dtantsur | kaifeng: even if stream_raw_images is true, we only stream if glance has raw images | 09:38 |
dtantsur | OR if we cache images on the conductor | 09:38 |
dtantsur | i.e. if image_download_source=http (not swift) | 09:39 |
*** mkrai has quit IRC | 09:39 | |
*** mkrai has joined #openstack-ironic | 09:39 | |
kaifeng | it seems I need to pick up my lost memory, have an impression on the agent+http task, hmm.. | 09:40 |
*** dsneddon has joined #openstack-ironic | 09:42 | |
*** dsneddon has quit IRC | 09:47 | |
*** mkrai has quit IRC | 09:49 | |
kaifeng | goodbye ironicers o/ | 09:50 |
dtantsur | bye kaifeng | 09:51 |
arne_wiebalck | kaifeng: o/ | 09:51 |
openstackgerrit | Jan Horstmann proposed openstack/ironic master: Fix typo in handling of exception FailedToGetIPAddressOnPort https://review.opendev.org/679414 | 09:53 |
*** e0ne has quit IRC | 10:01 | |
*** e0ne has joined #openstack-ironic | 10:10 | |
*** jtwill98 has quit IRC | 10:16 | |
*** dsneddon has joined #openstack-ironic | 10:21 | |
*** trident has quit IRC | 10:23 | |
*** mrda has quit IRC | 10:23 | |
*** bnemec has quit IRC | 10:23 | |
*** baha has quit IRC | 10:23 | |
*** irclogbot_1 has quit IRC | 10:24 | |
*** dtruong has quit IRC | 10:24 | |
*** chihhsin has quit IRC | 10:24 | |
*** SpamapS has quit IRC | 10:24 | |
*** yoctozepto has quit IRC | 10:24 | |
*** eandersson has quit IRC | 10:24 | |
*** lifeless has quit IRC | 10:24 | |
*** ociuhandu has joined #openstack-ironic | 10:24 | |
*** trident has joined #openstack-ironic | 10:24 | |
*** dtruong has joined #openstack-ironic | 10:27 | |
*** yoctozepto has joined #openstack-ironic | 10:27 | |
*** chihhsin has joined #openstack-ironic | 10:27 | |
*** irclogbot_0 has joined #openstack-ironic | 10:28 | |
*** bnemec has joined #openstack-ironic | 10:28 | |
*** lifeless has joined #openstack-ironic | 10:29 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Run a standalone job in the gate https://review.opendev.org/675939 | 10:30 |
*** mkrai has joined #openstack-ironic | 10:38 | |
*** SpamapS has joined #openstack-ironic | 10:41 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Correct the relative path calculation https://review.opendev.org/679033 | 10:41 |
*** ociuhandu has quit IRC | 10:42 | |
*** ociuhandu has joined #openstack-ironic | 11:02 | |
*** ociuhandu has quit IRC | 11:14 | |
*** dsneddon has quit IRC | 11:28 | |
*** dsneddon has joined #openstack-ironic | 11:28 | |
*** dsneddon has quit IRC | 11:33 | |
*** ricolin has quit IRC | 11:35 | |
*** rh-jelabarre has joined #openstack-ironic | 11:39 | |
*** tesseract has joined #openstack-ironic | 11:42 | |
*** jawad_axd has joined #openstack-ironic | 11:48 | |
*** ociuhandu has joined #openstack-ironic | 11:50 | |
*** dsneddon has joined #openstack-ironic | 11:53 | |
*** ociuhandu has quit IRC | 11:57 | |
*** dsneddon has quit IRC | 12:02 | |
*** yolanda has quit IRC | 12:12 | |
*** dsneddon has joined #openstack-ironic | 12:14 | |
*** dsneddon has quit IRC | 12:20 | |
*** ociuhandu has joined #openstack-ironic | 12:27 | |
*** derekh has quit IRC | 12:27 | |
TheJulia | Good morning everyone | 12:30 |
*** ociuhandu has quit IRC | 12:33 | |
*** sthussey has joined #openstack-ironic | 12:34 | |
*** yolanda has joined #openstack-ironic | 12:41 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable https://review.opendev.org/679333 | 12:50 |
*** dsneddon has joined #openstack-ironic | 12:54 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Attempt tgtd session detach https://review.opendev.org/679332 | 12:54 |
*** derekh has joined #openstack-ironic | 12:59 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add software raid release note to ironic https://review.opendev.org/679075 | 13:01 |
*** jawad_axd has quit IRC | 13:02 | |
TheJulia | dtantsur: I think https://review.opendev.org/#/c/679258 is going to need a updated IPA binary posted with the other fix to hopefully let the cirros images bypass trying to install a bootloader. Realistically I can't see us putting 900 MB transfers in place for raid testing and the insistance on grub bootloader on whole disk images is really a horrible bug | 13:05 |
patchbot | patch 679258 - ironic-tempest-plugin - Make local boot explicit on software raid tests - 3 patch sets | 13:05 |
*** e0ne has quit IRC | 13:05 | |
*** edleafe has joined #openstack-ironic | 13:10 | |
*** edleafe has left #openstack-ironic | 13:13 | |
*** e0ne has joined #openstack-ironic | 13:14 | |
TheJulia | If any core reviewers could take a look at https://review.opendev.org/#/c/676889 please | 13:16 |
patchbot | patch 676889 - sushy - Change OEM extensions architecture - 4 patch sets | 13:16 |
dtantsur | TheJulia: *shrugs* metalsmith CI is not less reliable than ironic CI, to be honest | 13:19 |
dtantsur | and whole disk images themselves don't rely on grup. I guess it's a side effect of booting from md volumes | 13:20 |
dtantsur | also morning TheJulia | 13:20 |
TheJulia | from a size, I'm thinking cost in terms of cpu/disk resources/ actual cost for that data to transverse wires. :( | 13:21 |
TheJulia | size standpoint | 13:21 |
TheJulia | I dunno, just rather make IPA a little bit smarter about its operation | 13:21 |
*** mkrai has quit IRC | 13:22 | |
dtantsur | TheJulia: also, why do we need https://review.opendev.org/#/c/679257/ if the CI currently passes? | 13:22 |
patchbot | patch 679257 - ironic - Disable software raid CI testing - 1 patch set | 13:22 |
dtantsur | the story reads wrong to me, and doesn't justify regression in testing of the key feature | 13:23 |
TheJulia | dtantsur: the test is actually broken, the default is netboot, which is why the get_boot_option fix addresses, but somewhere has to give | 13:23 |
dtantsur | TheJulia: the test DOES cover software RAID | 13:23 |
* dtantsur puts -2 until this discussion is over | 13:23 | |
TheJulia | IPA either needs to be smarter for it to not explode, or we need to turn off software raid testing, land fixes to ensure that we can properly handle ipa command install_boot_loader command | 13:24 |
TheJulia | right now, your tests don't cause that to be fired, so they are not actually testing it as designed | 13:24 |
dtantsur | TheJulia: they are testing building software RAID. if you want to remove that, let's revert all software RAID patches. | 13:25 |
TheJulia | because of our existing netboot default and that logic has not been added to state that only whole disk images are built | 13:25 |
openstackgerrit | Ilya Etingof proposed openstack/ironic-specs master: Update L3 based deployment spec https://review.opendev.org/672780 | 13:25 |
TheJulia | your going to the extreme though, I'm hoping we can fix the test and IPA, I've got all the patches up to actually do that | 13:25 |
dtantsur | Me? :) | 13:25 |
TheJulia | I don't want to remove it, but there is an underlying bug in ironic that we need to fix if we're ever going to get away from the default_boot_option always being netboot | 13:26 |
dtantsur | Sure, let's fix the bug | 13:26 |
TheJulia | well, patches are up for that... | 13:26 |
dtantsur | I don't see a reason to reduce software RAID coverage to 0% now that we've discovered it's "just" 80% instead of 100% | 13:26 |
dtantsur | (roughly speaking) | 13:26 |
TheJulia | dtantsur: I was going to immediatley put a revert up, but this again, something needs to move even temporarily to fix it. | 13:27 |
dtantsur | This is why I'm asking whether CI is broken by this | 13:27 |
dtantsur | If it's not, we don't need a temporary fix. So, is it? | 13:28 |
TheJulia | it was only temorarily to allow other patches to continue to move forward | 13:28 |
TheJulia | and ultimately, the last patch I posted yesterday to change IPA negates the need | 13:28 |
dtantsur | great, let's abandon it and fix a thing at a time | 13:28 |
dtantsur | reduce the CI coverage to let a few things in sounds a bit too nuclear for non-critical fixes | 13:29 |
TheJulia | so I'll rebase the patch after it, it is goin gto fail until IPA patch has merged and is in the images. | 13:29 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 13:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix get_boot_option logic https://review.opendev.org/679132 | 13:29 |
TheJulia | and done | 13:29 |
dtantsur | thx! | 13:30 |
TheJulia | no problem | 13:30 |
TheJulia | just maybe add more logic to IPA beyond what I've done in the patch to do additional checks. :) | 13:30 |
dtantsur | something inside me is uneasy about https://review.opendev.org/#/c/679333/ but I cannot express it | 13:30 |
patchbot | patch 679333 - ironic-python-agent - Don't try to put a bootloader in place when bootable - 2 patch sets | 13:30 |
TheJulia | I know what it is, it is only reliant upon the boot flag | 13:31 |
dtantsur | please watch out for the metalsmith job failures since it's the only job that tests partitions + local boot | 13:31 |
dtantsur | I suspect it shouldn't have hit that code path in the previous run. If it does, this approach may be wrong. | 13:31 |
TheJulia | finally always gets called though, so it was stupid on my part to not put it before the try | 13:32 |
dtantsur | TheJulia: something I cannot quite get is why on Earth we're trying grub-install for whole disk images | 13:32 |
TheJulia | dtantsur: uefi nvram update | 13:32 |
dtantsur | TheJulia: right, but hitting "return" means that the bootloader wouldn't be installed | 13:32 |
dtantsur | and the bootloader has to be installed in that jobg | 13:32 |
dtantsur | ah, uefi. happiness. | 13:32 |
TheJulia | dtantsur: yes... | 13:32 |
TheJulia | uefi | 13:33 |
TheJulia | I think the refactoring ?Nitesh? started on is actually heading in the right-ish direction | 13:33 |
dtantsur | TheJulia: could you add a debug logging to that patch? something like "Skipping installing bootloader on %s because it's marked as bootable"? | 13:33 |
TheJulia | but as a separate command seems... wrong workflow wise :\ | 13:33 |
dtantsur | ".. because partition %s is marked as bootable." | 13:33 |
TheJulia | ++ | 13:33 |
TheJulia | sure | 13:33 |
dtantsur | mm, yeah, the code path was hit for a partition image. | 13:35 |
* dtantsur dives into ramdisk logs | 13:35 | |
dtantsur | TheJulia: yep, we end up with a boot partition: http://paste.openstack.org/show/767639/ | 13:36 |
dtantsur | I'm afraid this approach won't fly. Maybe we should skip grub installation only for whole disk images? | 13:36 |
TheJulia | to do that we would need to make a breaking change and pass another argument along | 13:38 |
TheJulia | hmm | 13:38 |
TheJulia | so we set bootable flag | 13:38 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable https://review.opendev.org/679333 | 13:38 |
dtantsur | yep. or it's somehow on the image | 13:38 |
* dtantsur is not sure where these flags are stored | 13:38 | |
TheJulia | so... in that case, ^^^ + actually looking at the mbr pointer | 13:38 |
TheJulia | I can hack on that after meetings today | 13:38 |
dtantsur | TheJulia: yes, we do https://github.com/openstack/ironic-lib/blob/master/ironic_lib/disk_utils.py#L320-L325 | 13:39 |
TheJulia | dtantsur: flags are in the partition table, the mbr record needs to be decoded to see if it is present and if there is a decoder | 13:39 |
TheJulia | yeah, I just pulled the same file and saw that, so I need to revise that patch a little more to make it smarter | 13:39 |
dtantsur | we may need the check on whether the image is whole disk one | 13:40 |
* TheJulia wishes file would return "x86 boot sector" when reading a device | 13:40 | |
TheJulia | we would need to make lots of changes to do that including passing a new argument into IPA | 13:40 |
TheJulia | or the install bootloader command | 13:40 |
dtantsur | indeed | 13:40 |
TheJulia | I think that, and solely trusting it for the smaller and theoretically diminishing case of mbr booting may not be worth that headache. | 13:41 |
TheJulia | because, no matter what, we likely ought to check if things actually look valid enough to boot | 13:41 |
dtantsur | so what's the suggestion? | 13:41 |
TheJulia | make the last patch I just updated smarter | 13:42 |
TheJulia | only the boot loader installation adds the mbr record | 13:42 |
TheJulia | so if it is already present, it should be there with the pointer and should at least boot | 13:42 |
TheJulia | or throw some awesome error | 13:42 |
dtantsur | oh, so check if the bootloader is already installed? might work | 13:43 |
TheJulia | yeah | 13:43 |
dtantsur | ack, cool | 13:45 |
dtantsur | this is even more robust, I guess | 13:45 |
dtantsur | although I still don't fully understand on what code path we hit that in our CI | 13:45 |
dtantsur | we don't test RAID + UEFI, do we? | 13:45 |
TheJulia | dtantsur: no, it is presently mbr only. I can't think of his name has patches up to start working towards enabling that | 13:45 |
dtantsur | then isn't the issue that we even try to install bootloader for a whole disk image? | 13:46 |
TheJulia | we hit that deep in the prepare_instance code path in the deploy interfaces | 13:46 |
dtantsur | I think for long time we have been saying that whole disk images default to local boot, and the boot loader has to be installed | 13:46 |
dtantsur | if we try grub-install, we break windows support at least | 13:46 |
TheJulia | Technically yes, except we need to ensure the nvram is up to date because some vendors decided not to look for defaults with their hardware | 13:46 |
dtantsur | nvram is not needed without uefi, no? | 13:47 |
TheJulia | yeah, we likely should do some exaustive checking | 13:47 |
dtantsur | and does it mean that we don't support windows with uefi? | 13:47 |
TheJulia | present, not required by spec, required by some vendors | 13:47 |
TheJulia | dtantsur: technically right now we wouldn't be able to with the way the code is written | 13:47 |
dtantsur | ugh | 13:47 |
TheJulia | now, if we make it smarter, then yes, we should be able to just fall through and windows should deploy | 13:48 |
* dtantsur wants a CI job with windows | 13:48 | |
TheJulia | this is where I was starting to think about ?Nitesh?'s uefibootmgr patches is heading in the right direction | 13:48 |
TheJulia | me too | 13:48 |
*** EmilienM is now known as EvilienM | 13:49 | |
*** Goneri has joined #openstack-ironic | 13:49 | |
dtantsur | and, back to the "deprecate partition images" discussion, we need to reduce the supported matrix | 13:50 |
TheJulia | ++ | 13:53 |
dtantsur | TheJulia: are you fine with me doing the actual deprecation instead of (or rather, replacing) https://review.opendev.org/#/c/677558/ ? | 13:53 |
patchbot | patch 677558 - ironic-lib - Add todo note for is_block_device - 2 patch sets | 13:53 |
TheJulia | dtantsur: go right ahead | 13:54 |
dtantsur | okay, will get some tea and jump on it | 13:54 |
TheJulia | dtantsur: I have administrative things to do later today, and I'm likely not going to have time to next week. | 13:54 |
*** dabadaba has quit IRC | 13:58 | |
*** dabadaba has joined #openstack-ironic | 13:59 | |
*** dsneddon has quit IRC | 14:00 | |
TheJulia | yay, looks like no first meeting \o/ | 14:00 |
*** ociuhandu has joined #openstack-ironic | 14:01 | |
*** ociuhandu has quit IRC | 14:02 | |
openstackgerrit | Merged openstack/sushy-tools master: Do not reuse sqlite connection https://review.opendev.org/678797 | 14:07 |
TheJulia | well, found an easy way | 14:12 |
*** dougsz has joined #openstack-ironic | 14:15 | |
*** dougsz has quit IRC | 14:17 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix non-functional driver configuration on Py2 https://review.opendev.org/679456 | 14:17 |
etingof | if rpioso could pass this ^ to jtwill98, that might be helpful.. | 14:18 |
*** baha has joined #openstack-ironic | 14:18 | |
*** dsneddon has joined #openstack-ironic | 14:18 | |
*** rachit7 has quit IRC | 14:20 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable https://review.opendev.org/679333 | 14:21 |
TheJulia | dtantsur: ^^^ | 14:21 |
*** dsneddon has quit IRC | 14:23 | |
dtantsur | TheJulia: makes sense, let's see what the CI says | 14:24 |
*** dsneddon has joined #openstack-ironic | 14:24 | |
TheJulia | yup, that will be the true test | 14:26 |
*** dsneddon has quit IRC | 14:29 | |
dtantsur | ugh, iscsi_verify_attempts is also used by ironic | 14:29 |
*** dsneddon has joined #openstack-ironic | 14:30 | |
*** ociuhandu has joined #openstack-ironic | 14:34 | |
*** e0ne has quit IRC | 14:35 | |
*** mkrai has joined #openstack-ironic | 14:39 | |
*** ociuhandu has quit IRC | 14:39 | |
*** e0ne has joined #openstack-ironic | 14:39 | |
rpioso | etingof: Done | 14:40 |
rpioso | etingof: Would you like his contact info? | 14:41 |
etingof | rpioso, I just recalled I have his e-mail! | 14:42 |
etingof | rpioso does not like to IRC proxy it seems ;) | 14:42 |
* rpioso doesn't mind, but is lossy | 14:43 | |
rpioso | etingof: I suggested he hangout with us here :-) | 14:44 |
etingof | did we share him off? | 14:44 |
etingof | *scare | 14:44 |
rpioso | Texans aren't so easily scared. | 14:45 |
etingof | sure, Chuck Norris can divide by zero | 14:47 |
*** trandles has joined #openstack-ironic | 14:47 | |
TheJulia | well, I thought one could now divide by zero in theoretical terms | 14:48 |
TheJulia | o/ trandles | 14:48 |
TheJulia | trandles: something occured to me last night, you said you were trying to use the iscsi deploy interface, except if memory serves you don't have local storage so I think you'd want the ramdisk interface | 14:49 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Prepare for deprecation of iscsi_verify_attempts in ironic-lib https://review.opendev.org/679463 | 14:49 |
dtantsur | TheJulia: step one ^^^ | 14:49 |
TheJulia | hopefully only a two step process... and not like sw raid :( | 14:49 |
dtantsur | should be only two :) | 14:52 |
dtantsur | yeah, RAID is a fun task.. but a great achievement as well | 14:52 |
TheJulia | yeah | 14:52 |
TheJulia | it is an awesome gain for ironic | 14:53 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-lib master: Deprecate [disk_utils]iscsi_verify_attempts https://review.opendev.org/677558 | 14:56 |
dtantsur | TheJulia: step 2 ^^ | 14:56 |
TheJulia | changing default boot_option would also be epic for this cycle | 14:57 |
* TheJulia hopes | 14:57 | |
dtantsur | EPIC | 14:58 |
TheJulia | CI failure rates should also go down... at least some | 14:59 |
dtantsur | do we have a reason to not deprecate netboot? BFV? | 15:00 |
*** ijw has joined #openstack-ironic | 15:05 | |
rpioso | etingof: lol | 15:06 |
*** ociuhandu has joined #openstack-ironic | 15:07 | |
*** ijw has quit IRC | 15:11 | |
*** ijw has joined #openstack-ironic | 15:11 | |
*** ijw has quit IRC | 15:13 | |
*** ijw has joined #openstack-ironic | 15:14 | |
*** eandersson has joined #openstack-ironic | 15:30 | |
*** gyee has joined #openstack-ironic | 15:33 | |
*** mkowalski has joined #openstack-ironic | 15:43 | |
*** mkowalski has quit IRC | 15:46 | |
NobodyCam | Good Morning Ironic'er and ... | 15:47 |
NobodyCam | ofc... | 15:47 |
NobodyCam | TGIF! | 15:47 |
*** alexmcleod has quit IRC | 15:50 | |
*** tesseract has quit IRC | 15:52 | |
*** jtwill98 has joined #openstack-ironic | 15:57 | |
dtantsur | \o/ | 15:58 |
dtantsur | morning NobodyCam | 15:58 |
*** lucasagomes has quit IRC | 15:59 | |
NobodyCam | Morning dtantsur :) | 16:03 |
*** e0ne has quit IRC | 16:04 | |
*** e0ne has joined #openstack-ironic | 16:09 | |
TheJulia | dtantsur: bfv and ramdisk interface.. | 16:12 |
dtantsur | TheJulia: I'm wondering if we should leave netboot as a special case for these two | 16:12 |
dtantsur | .. which probably won't reduce the amount of code too much :( | 16:12 |
*** dsneddon has quit IRC | 16:14 | |
TheJulia | Not really, partition will ease a bit if there is nobody screaming for need | 16:14 |
*** ijw_ has joined #openstack-ironic | 16:15 | |
*** ijw_ has quit IRC | 16:15 | |
*** ijw_ has joined #openstack-ironic | 16:16 | |
*** ijw_ has quit IRC | 16:17 | |
*** dsneddon has joined #openstack-ironic | 16:17 | |
*** ijw_ has joined #openstack-ironic | 16:17 | |
*** ijw has quit IRC | 16:18 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add coreutils to tinybuild's IPA image https://review.opendev.org/679478 | 16:29 |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable https://review.opendev.org/679333 | 16:30 |
*** e0ne has quit IRC | 16:30 | |
dtantsur | have a great weekend | 16:36 |
*** dtantsur is now known as dtantsur|afk | 16:36 | |
*** igordc has joined #openstack-ironic | 16:39 | |
*** dsneddon has quit IRC | 16:40 | |
*** rachit7 has joined #openstack-ironic | 16:46 | |
arne_wiebalck | Have a good week-end everyone o/ | 16:46 |
rachit7 | (y) | 16:46 |
*** rachit7 has quit IRC | 16:51 | |
TheJulia | o/ | 16:53 |
*** bnemec is now known as beekneemech | 16:53 | |
*** derekh has quit IRC | 17:00 | |
*** andrein has quit IRC | 17:01 | |
*** mkrai has quit IRC | 17:08 | |
*** dsneddon has joined #openstack-ironic | 17:09 | |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: tinyipa: update qemu and tgt dependencies https://review.opendev.org/678415 | 17:15 |
*** ricolin has joined #openstack-ironic | 17:28 | |
*** rmcall has quit IRC | 17:36 | |
*** andrein has joined #openstack-ironic | 17:45 | |
*** ociuhandu has quit IRC | 17:51 | |
*** ociuhandu has joined #openstack-ironic | 17:57 | |
*** ricolin has quit IRC | 18:00 | |
*** ociuhandu has quit IRC | 18:02 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add file to tinybuild's IPA image https://review.opendev.org/679478 | 18:02 |
*** jtwill98 has quit IRC | 18:04 | |
*** jtomasek has quit IRC | 18:07 | |
*** jtomasek has joined #openstack-ironic | 18:09 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Don't try to put a bootloader in place when bootable https://review.opendev.org/679333 | 18:17 |
*** dsneddon has quit IRC | 18:18 | |
trandles | TheJulia, yes that's correct about iscsi deploy but right now I'm configuring ironic to provision my cloud servers. Then I'll move on to the diskless HPC compute nodes. | 18:33 |
trandles | Ironic: One Service To Provision Them All | 18:33 |
TheJulia | trandles: ahh! | 18:45 |
TheJulia | trandles: Excellent | 18:45 |
* TheJulia ponders stickers.... | 18:45 | |
*** dsneddon has joined #openstack-ironic | 18:50 | |
*** dsneddon has quit IRC | 18:55 | |
*** andrein has quit IRC | 18:58 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic master: add table of available cleaning steps to documentation https://review.opendev.org/679499 | 19:25 |
dhellmann | TheJulia, JayF: ^ based on our conversation earlier this week | 19:26 |
*** dsneddon has joined #openstack-ironic | 19:29 | |
*** dsneddon has quit IRC | 19:35 | |
*** mbeierl has quit IRC | 19:37 | |
*** andrein has joined #openstack-ironic | 19:41 | |
*** Goneri has quit IRC | 19:45 | |
*** andrein has quit IRC | 19:55 | |
*** andrein has joined #openstack-ironic | 19:56 | |
donnyd | does anyone know if cern is using magnum and ironic together? | 20:05 |
donnyd | wow i should have googled that before i asked | 20:06 |
donnyd | LOL | 20:06 |
donnyd | Also has anyone gotten idrac and serial console to work | 20:06 |
*** dsneddon has joined #openstack-ironic | 20:09 | |
*** dsneddon has quit IRC | 20:14 | |
*** Goneri has joined #openstack-ironic | 20:23 | |
*** ociuhandu has joined #openstack-ironic | 20:25 | |
*** ociuhandu has quit IRC | 20:30 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add file to tinybuild's IPA image https://review.opendev.org/679478 | 20:34 |
TheJulia | donnyd: i don't think the idrac hardware type has the ability to load an ipmi socat console | 20:35 |
donnyd | it has a serial over lan console | 20:35 |
TheJulia | I mean their hardware type, give me a moment | 20:37 |
donnyd | was hoping someone had done it before with a dell server | 20:37 |
donnyd | kk | 20:37 |
TheJulia | https://opendev.org/openstack/ironic/src/branch/master/ironic/drivers/drac.py#L33 doesn't list a console interface, it is just not a part of the driver in ironic, but if you use ipmi, you should be able to use the ipmi hardware type/driver | 20:40 |
*** dsneddon has joined #openstack-ironic | 20:47 | |
*** rmcall has joined #openstack-ironic | 20:50 | |
*** dsneddon has quit IRC | 20:52 | |
donnyd | Thanks TheJulia | 20:59 |
donnyd | Is there someone using not the ipmi driver? | 20:59 |
donnyd | I use ipmi for everything | 20:59 |
donnyd | mainly because I have a mishmash of HW vendors in the rack | 21:00 |
*** dsneddon has joined #openstack-ironic | 21:19 | |
*** dsneddon has quit IRC | 21:24 | |
*** hwoarang has quit IRC | 21:25 | |
*** hwoarang has joined #openstack-ironic | 21:28 | |
*** ijw_ has quit IRC | 21:50 | |
*** dsneddon has joined #openstack-ironic | 21:52 | |
*** trandles has left #openstack-ironic | 21:55 | |
*** dsneddon has quit IRC | 21:57 | |
TheJulia | donnyd: oh, well, that is a decent reason to. So what sort errors are you seeing? I take it you have the console_interface set in ironic? | 22:07 |
*** Goneri has quit IRC | 22:08 | |
donnyd | Interestingly enough I get no errors, on the ironic side the console starts and socat is listening | 22:09 |
donnyd | but when i pull up the console in horizon I just get a blank screen. And yes I did configure nova-compute and nova-serialproxy | 22:09 |
donnyd | serialproxy is listening, but nothing seems to happen | 22:10 |
donnyd | maybe better to debug another day | 22:10 |
donnyd | Hope everyone has a great weekend :) | 22:10 |
*** Goneri has joined #openstack-ironic | 22:13 | |
TheJulia | donnyd: if you telnet to the port setup for the socat console, do you end up connected to the the console? | 22:15 |
donnyd | Oo that is a good idea | 22:16 |
donnyd | I will give that a whirl | 22:16 |
TheJulia | okY | 22:16 |
TheJulia | err, okay | 22:16 |
donnyd | okY is pretty good too | 22:17 |
*** jtomasek has quit IRC | 22:19 | |
*** dsneddon has joined #openstack-ironic | 22:25 | |
*** dsneddon has quit IRC | 22:31 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add file to tinybuild's IPA image https://review.opendev.org/679478 | 22:43 |
*** vesper has quit IRC | 22:46 | |
*** vesper11 has joined #openstack-ironic | 22:47 | |
*** rh-jelabarre has quit IRC | 22:53 | |
*** dsneddon has joined #openstack-ironic | 22:56 | |
*** yaawang has quit IRC | 23:00 | |
*** yaawang has joined #openstack-ironic | 23:01 | |
*** dsneddon has quit IRC | 23:01 | |
*** jhesketh has quit IRC | 23:18 | |
*** dsneddon has joined #openstack-ironic | 23:31 | |
*** dsneddon has quit IRC | 23:35 | |
*** gyee has quit IRC | 23:37 | |
*** Goneri has quit IRC | 23:50 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!