| *** zzzeek has joined #openstack-ironic | 00:01 | |
| *** k_mouza has joined #openstack-ironic | 00:24 | |
| *** k_mouza has quit IRC | 00:28 | |
| *** zzzeek has quit IRC | 00:38 | |
| *** zzzeek has joined #openstack-ironic | 00:42 | |
| *** k_mouza has joined #openstack-ironic | 00:44 | |
| *** k_mouza has quit IRC | 00:49 | |
| *** gyee has quit IRC | 00:50 | |
| *** k_mouza has joined #openstack-ironic | 00:54 | |
| *** k_mouza has quit IRC | 00:58 | |
| *** k_mouza has joined #openstack-ironic | 01:06 | |
| *** k_mouza has quit IRC | 01:11 | |
| *** k_mouza has joined #openstack-ironic | 01:20 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/ironic master: Fix handling OctetString for pysnmp https://review.opendev.org/750401 | 01:22 |
|---|---|---|
| *** k_mouza has quit IRC | 01:24 | |
| *** k_mouza has joined #openstack-ironic | 01:33 | |
| *** k_mouza has quit IRC | 01:38 | |
| *** eandersson has quit IRC | 01:44 | |
| *** eandersson has joined #openstack-ironic | 01:45 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert allocations endpoint to plain JSON https://review.opendev.org/743679 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert bios endpoint to plain JSON https://review.opendev.org/745475 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert conductors endpoint to plain JSON https://review.opendev.org/745779 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert deploy_templates endpoint to plain JSON https://review.opendev.org/745780 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ports endpoint to plain JSON https://review.opendev.org/750120 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ramdisk endpoint to plain JSON https://review.opendev.org/750857 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume endpoint to plain JSON https://review.opendev.org/750858 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON https://review.opendev.org/750859 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON https://review.opendev.org/750860 | 02:13 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Expunge the internal version of WSME https://review.opendev.org/750861 | 02:13 |
| *** zzzeek has quit IRC | 02:14 | |
| stevebaker_ | oh my word, that is the first pass DONE | 02:14 |
| *** stevebaker_ is now known as stevebaker | 02:15 | |
| *** chenhaw has quit IRC | 02:15 | |
| *** zzzeek has joined #openstack-ironic | 02:15 | |
| *** chenhaw has joined #openstack-ironic | 02:15 | |
| stevebaker | it still has CI failures, and I'd like to do a bunch of common function refactoring, but the main slog is done | 02:16 |
| stevebaker | the LOC stats on that last change: +15, -3392 | 02:16 |
| janders | good work stevebaker! | 02:19 |
| *** rh-jelabarre has quit IRC | 02:44 | |
| *** zzzeek has quit IRC | 02:45 | |
| *** mkrai has joined #openstack-ironic | 02:46 | |
| *** zzzeek has joined #openstack-ironic | 02:48 | |
| *** rcernin has quit IRC | 02:50 | |
| *** rcernin has joined #openstack-ironic | 02:59 | |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert allocations endpoint to plain JSON https://review.opendev.org/743679 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert bios endpoint to plain JSON https://review.opendev.org/745475 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert chassis endpoint to plain JSON https://review.opendev.org/745476 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert conductors endpoint to plain JSON https://review.opendev.org/745779 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert deploy_templates endpoint to plain JSON https://review.opendev.org/745780 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert drivers endpoint to plain JSON https://review.opendev.org/741865 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert event endpoint to plain JSON https://review.opendev.org/746623 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert nodes endpoint to plain JSON https://review.opendev.org/747820 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ports endpoint to plain JSON https://review.opendev.org/750120 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert ramdisk endpoint to plain JSON https://review.opendev.org/750857 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume endpoint to plain JSON https://review.opendev.org/750858 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/connectors endpoint to plain JSON https://review.opendev.org/750859 | 03:02 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert volume/targets endpoint to plain JSON https://review.opendev.org/750860 | 03:03 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Expunge the internal version of WSME https://review.opendev.org/750861 | 03:03 |
| openstackgerrit | Steve Baker proposed openstack/ironic master: Convert portgroups endpoint to plain JSON https://review.opendev.org/750864 | 03:03 |
| *** mkrai has quit IRC | 03:14 | |
| *** mkrai has joined #openstack-ironic | 03:14 | |
| *** k_mouza has joined #openstack-ironic | 03:16 | |
| *** stevebaker has quit IRC | 03:19 | |
| *** stevebaker has joined #openstack-ironic | 03:19 | |
| *** k_mouza has quit IRC | 03:20 | |
| *** k_mouza has joined #openstack-ironic | 03:25 | |
| *** k_mouza has quit IRC | 03:30 | |
| *** k_mouza has joined #openstack-ironic | 03:35 | |
| *** k_mouza has quit IRC | 03:39 | |
| *** Qianbiao has joined #openstack-ironic | 04:28 | |
| *** k_mouza has joined #openstack-ironic | 04:33 | |
| janders | dtantsur|afk I have a design question regarding https://storyboard.openstack.org/#!/story/2008038: do we want the get_mac_address to provide a) all MAC addresses b) just the MACs of the 'enabled' ports or c) make it an optional parameter so that it can work in either way? system.ethernet_interfaces.summary seems to return a dict of MAC:state pairs | 04:37 |
| janders | so thinking if it's worthwhile going through the state information. E.g. system.ethernet_interfaces.summary= {'98:03:9B:A3:E4:E4': None, '98:03:9B:A3:E4:E5': None, 'E4:43:4B:56:2C:02': None, 'E4:43:4B:56:2C:03': None, 'E4:43:4B:56:2C:01': 'enabled', 'E4:43:4B:56:2C:00': 'enabled'}. My guess is a) (all) as we're trying to identify the node but I | 04:37 |
| janders | thought I will double-check as I am not fully across the context. Let's chat further in your business hours. | 04:37 |
| *** k_mouza has quit IRC | 04:37 | |
| janders | heading off for some time, back in EMEA hours :) | 04:38 |
| *** mkrai has quit IRC | 04:53 | |
| *** mkrai has joined #openstack-ironic | 04:54 | |
| *** k_mouza has joined #openstack-ironic | 05:07 | |
| *** k_mouza has quit IRC | 05:12 | |
| *** k_mouza has joined #openstack-ironic | 05:14 | |
| *** jtomasek has joined #openstack-ironic | 05:17 | |
| *** k_mouza has quit IRC | 05:19 | |
| *** tzumainn has quit IRC | 05:20 | |
| *** jtomasek has quit IRC | 05:22 | |
| *** k_mouza has joined #openstack-ironic | 05:43 | |
| *** k_mouza has quit IRC | 05:48 | |
| *** zzzeek has quit IRC | 05:57 | |
| *** k_mouza has joined #openstack-ironic | 05:57 | |
| *** zzzeek has joined #openstack-ironic | 06:01 | |
| *** k_mouza has quit IRC | 06:01 | |
| *** k_mouza has joined #openstack-ironic | 06:08 | |
| *** ricolin has quit IRC | 06:10 | |
| *** k_mouza has quit IRC | 06:12 | |
| *** k_mouza has joined #openstack-ironic | 06:21 | |
| Qianbiao | janders I think inspector will load all mac address. | 06:24 |
| Qianbiao | because you will never know which port it will be used later in os. | 06:25 |
| *** k_mouza has quit IRC | 06:25 | |
| *** k_mouza has joined #openstack-ironic | 06:35 | |
| *** k_mouza has quit IRC | 06:39 | |
| yoctozepto | morning ironic | 06:47 |
| *** vmud213 has joined #openstack-ironic | 06:47 | |
| yoctozepto | I see bifrost is hit by http://lists.openstack.org/pipermail/openstack-discuss/2020-September/017155.html too | 06:48 |
| *** k_mouza has joined #openstack-ironic | 06:49 | |
| *** mkrai has quit IRC | 06:53 | |
| *** mkrai_ has joined #openstack-ironic | 06:53 | |
| *** k_mouza has quit IRC | 06:54 | |
| *** jtomasek has joined #openstack-ironic | 06:59 | |
| *** k_mouza has joined #openstack-ironic | 07:06 | |
| *** k_mouza has quit IRC | 07:10 | |
| *** mkrai_ has quit IRC | 07:12 | |
| *** priteau has joined #openstack-ironic | 07:16 | |
| iurygregory | good morning Ironic! | 07:21 |
| iurygregory | yoctozepto, I don't see bifrost on the link o.o | 07:23 |
| iurygregory | but yeah we probably need to switch the jobs from bionic to focal to test | 07:24 |
| iurygregory | if it's not done yet, going to check | 07:24 |
| iurygregory | ok, we probably just need to make the job voting for focal =) | 07:27 |
| yoctozepto | iurygregory: I meant lower-constraints are failing already :-) | 07:28 |
| iurygregory | yoctozepto, oh | 07:28 |
| iurygregory | yay | 07:28 |
| yoctozepto | iurygregory: yeah, it's the worst cycle so far if you ask us from kolla | 07:30 |
| yoctozepto | iurygregory: for some reason centos8 turned out to be less of pita than what was happening all this cycle ;/ | 07:31 |
| iurygregory | ouch =( | 07:31 |
| iurygregory | I will push a patch trying to fix l-c for bifrost | 07:31 |
| openstackgerrit | paresh sao proposed openstack/ironic master: Follow-up patch for One Button Secure Erase clean step https://review.opendev.org/750624 | 07:32 |
| iurygregory | if bifrost is broken I have the feeling all other projects are... =X | 07:35 |
| yoctozepto | iurygregory: many indeed | 07:37 |
| iurygregory | I'm wondering what happen for projects we did the release already.. | 07:38 |
| iurygregory | yay \o/ | 07:38 |
| Qianbiao | iurygregory Morning. | 07:40 |
| iurygregory | hey Qianbiao o/ | 07:40 |
| Qianbiao | hi iurygregory, is this right file size after diskimage-build http://paste.openstack.org/show/797688/ | 07:40 |
| Qianbiao | why iniramfs is so small | 07:40 |
| yoctozepto | iurygregory: well, depends on the kind of breakage mostly | 07:40 |
| yoctozepto | iurygregory: if it's lower-constraints, then it's probably only CI-relevant | 07:40 |
| iurygregory | yoctozepto, well I saw that cinder patch has lc requirements change | 07:41 |
| iurygregory | https://review.opendev.org/#/c/743080/7 | 07:41 |
| patchbot | patch 743080 - cinder - [goal] Fixing lower constraints for Ubuntu Focal - 7 patch sets | 07:41 |
| iurygregory | Qianbiao, humm seems small to me .-. | 07:41 |
| Qianbiao | =( | 07:42 |
| yoctozepto | iurygregory: everyone wants green CI, cinder too :-) | 07:44 |
| iurygregory | yoctozepto, ofc =) | 07:44 |
| *** dougsz has joined #openstack-ironic | 07:46 | |
| *** priteau has quit IRC | 07:47 | |
| *** zzzeek has quit IRC | 07:48 | |
| *** k_mouza has joined #openstack-ironic | 07:48 | |
| *** zzzeek has joined #openstack-ironic | 07:50 | |
| *** k_mouza has quit IRC | 07:53 | |
| *** priteau has joined #openstack-ironic | 07:53 | |
| iurygregory | yay l-c is broken in Ironic also \o/ | 07:55 |
| iurygregory | l-c party today | 07:55 |
| *** k_mouza has joined #openstack-ironic | 07:57 | |
| *** k_mouza has quit IRC | 08:01 | |
| *** lourot has joined #openstack-ironic | 08:04 | |
| *** mkrai_ has joined #openstack-ironic | 08:08 | |
| *** lucasagomes has joined #openstack-ironic | 08:11 | |
| *** k_mouza has joined #openstack-ironic | 08:19 | |
| openstackgerrit | Radosław Piliszek proposed openstack/bifrost master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750903 | 08:22 |
| yoctozepto | iurygregory: I started popping l-c bottles ^ | 08:22 |
| yoctozepto | fixed kollas already | 08:22 |
| iurygregory | yoctozepto, cool tks, I was on meeting so I'm going to test ironic now =) | 08:23 |
| *** k_mouza has quit IRC | 08:23 | |
| *** k_mouza has joined #openstack-ironic | 08:30 | |
| *** rcernin has quit IRC | 08:35 | |
| *** jtomasek has quit IRC | 08:45 | |
| openstackgerrit | Radosław Piliszek proposed openstack/bifrost master: [CI] Check with Kolla Ansible https://review.opendev.org/750637 | 08:47 |
| *** k_mouza has quit IRC | 08:47 | |
| openstackgerrit | Radosław Piliszek proposed openstack/bifrost master: test-bifrost: use existing playbooks https://review.opendev.org/750656 | 08:47 |
| *** k_mouza has joined #openstack-ironic | 08:48 | |
| Qianbiao | hello, did someone deploy a node with "ramdisk" deploy interface successed? | 08:48 |
| Qianbiao | what base distro image works. I try ubuntu16 and ubuntu18 all ends with: "ALERT! /dev/ram0 does not exist. Dropping to a shell!" | 08:48 |
| *** derekh has joined #openstack-ironic | 08:52 | |
| *** abdysn has joined #openstack-ironic | 08:59 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Fix l-c for Ubuntu Focal https://review.opendev.org/750908 | 09:05 |
| *** ricolin has joined #openstack-ironic | 09:09 | |
| *** dtantsur|afk is now known as dtantsur | 09:14 | |
| dtantsur | janders: I think just enabled NICs should be fine | 09:17 |
| dtantsur | I think we also filter what we add during inspection | 09:17 |
| openstackgerrit | zhufl proposed openstack/ironic master: Fix invalid assert_has_calls https://review.opendev.org/750910 | 09:18 |
| janders | dtantsur thank you | 09:21 |
| janders | speaking of inspection... I noticed that OOB/redfish inspection does create ports (so I could tap into this) but I think it's better to query the system via Redfish directly - more versatile. Would you agree? | 09:23 |
| *** Qianbiao has quit IRC | 09:26 | |
| *** Qianbiao has joined #openstack-ironic | 09:26 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:28 | |
| *** vmud213 has quit IRC | 09:30 | |
| openstackgerrit | Verification of a change to openstack/ironic-python-agent failed: Refactor API version negotiation code https://review.opendev.org/750097 | 09:31 |
| *** vmud213 has joined #openstack-ironic | 09:31 | |
| dtantsur | janders: yep. you can move the required code around | 09:32 |
| *** mkrai_ has quit IRC | 09:33 | |
| *** mkrai__ has joined #openstack-ironic | 09:33 | |
| iurygregory | I'm puzzled that grpcio versions aren't working locally for l-c .-. | 09:48 |
| *** k_mouza_ has joined #openstack-ironic | 09:49 | |
| *** k_mouza has quit IRC | 09:51 | |
| openstackgerrit | Merged openstack/bifrost master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750903 | 09:55 |
| dtantsur | oh, IPA affected as well.. | 09:55 |
| openstackgerrit | Jacob Anders proposed openstack/ironic master: Use OOB inspection to fetch MACs for IB inspection https://review.opendev.org/750943 | 10:00 |
| openstackgerrit | wu.shiming proposed openstack/bifrost master: Remove Babel requirement https://review.opendev.org/750944 | 10:01 |
| janders | https://review.opendev.org/750943 is very early days (hence workflow-1) but when you have time I'd appreciate feedback on whether I'm on the right track | 10:01 |
| patchbot | patch 750943 - ironic - Use OOB inspection to fetch MACs for IB inspection - 1 patch set | 10:01 |
| janders | will grab some dinner now | 10:02 |
| * iurygregory goes for lunch will fight l-c after | 10:05 | |
| openstackgerrit | Ameya Raut proposed openstack/ironic stable/train: DRAC: Fix RAID create_config clean step https://review.opendev.org/750605 | 10:19 |
| *** Lucas_Gray has joined #openstack-ironic | 10:32 | |
| *** rcernin has joined #openstack-ironic | 10:39 | |
| yoctozepto | hmm, it seems ironic inspector client is happy about receiving 404 result from api | 10:51 |
| yoctozepto | it seems to exit with return code of 0 ;d | 10:51 |
| yoctozepto | http://paste.openstack.org/show/797705/ | 10:52 |
| dtantsur | ehmmm? weird | 10:54 |
| *** k_mouza_ has quit IRC | 10:57 | |
| dtantsur | yoctozepto: I cannot trivially reproduce it locally. how do you hit it? | 10:59 |
| *** Nisha_Agarwal has quit IRC | 11:06 | |
| yoctozepto | dtantsur: it's just broken wsgi so it gives away 404 but seemingly the return code for the client is 0 | 11:11 |
| yoctozepto | dtantsur: it even tries two urls: / and the path to rules | 11:12 |
| yoctozepto | still happy about it and prints it back | 11:12 |
| yoctozepto | ;d | 11:12 |
| dtantsur | I tried setting inspector URL to google.com and get an HTML back, but $? is 1 | 11:12 |
| yoctozepto | dtantsur: yeah, I'm confused too, will research it further; I thought you might just know the answer :-) (never bad to try) | 11:13 |
| yoctozepto | it's all client parts installed from pypi using master constraints | 11:13 |
| yoctozepto | upper-constraints* | 11:13 |
| dtantsur | same for me locally | 11:16 |
| *** k_mouza has joined #openstack-ironic | 11:24 | |
| Qianbiao | Hello Ironic. | 11:29 |
| Qianbiao | Hi, i try to build image for ramdisk deploy interface. http://paste.openstack.org/show/797706/ | 11:30 |
| Qianbiao | Has tried several distrute, all end with: "ALERT! /dev/ram0 does not exist. Dropping to a shell!" | 11:30 |
| yoctozepto | dtantsur: I can't reproduce locally either, tried various ideas, no idea what CI is happy about; will debug as we try to harden our ironic testing | 11:31 |
| Qianbiao | Is it relate to the built images? Any other option should be present. | 11:31 |
| *** dougsz has quit IRC | 11:36 | |
| dtantsur | Qianbiao: I'm not sure, but maybe try using the regular IPA image, at least to test it? | 11:39 |
| Qianbiao | sure will try dtantsur | 11:40 |
| Qianbiao | btw, i find diskimage-builder provide a ramdisk-image-create, maybe this is for ramdisk deploy? | 11:41 |
| Qianbiao | dtantsur ^^ | 11:41 |
| dtantsur | Qianbiao: nope, it's an ancient thing, don't use it | 11:42 |
| Qianbiao | ok | 11:42 |
| dtantsur | I mean, it may work, but nobody remembers how it works any more :) | 11:42 |
| dtantsur | all bets are off | 11:42 |
| Qianbiao | great :=), old weapon | 11:42 |
| *** thiagop has joined #openstack-ironic | 11:42 | |
| * iurygregory is wondering if tripleO patch merged the change to the new element hehe | 11:43 | |
| openstackgerrit | Merged openstack/sushy stable/rocky: Do not log passwords and auth tokens when using SessionService https://review.opendev.org/748149 | 11:48 |
| Qianbiao | why ir-deploy-ibmc.kernel ir-deploy-ibmc.initramfs is generated, but no ir-deploy-ibmc.qcow2 | 11:48 |
| iurygregory | thats a good question O.o | 11:50 |
| dtantsur | the ramdisk does not have a real image for it | 11:50 |
| dtantsur | everything goes into the initramfs | 11:51 |
| Qianbiao | dtantsur glance requires one | 11:51 |
| Qianbiao | when create the image | 11:51 |
| Qianbiao | iurygregory i know why, ironic devstack create it in a temp file, and drop it :( | 11:51 |
| Qianbiao | dtantsur do u mean i can simple update anything, it wont be used. | 11:52 |
| dtantsur | unfortunately, I don't remember how exactly the ramdisk deploy works, and https://docs.openstack.org/ironic/latest/admin/interfaces/deploy.html#ramdisk-deploy does not help much | 11:52 |
| Qianbiao | yeah, i check the spec too, it does not talk about how to generate an image | 11:53 |
| dtantsur | are you using Nova? | 11:54 |
| Qianbiao | yes | 11:54 |
| Qianbiao | devstack | 11:54 |
| Qianbiao | cirros will be good? | 11:54 |
| Qianbiao | cirros has an uec image contains both ramdisk and kernel | 11:54 |
| Qianbiao | will try now | 11:55 |
| dtantsur | I hope so. Ideally, you should wait for TheJulia to wake up and ask her. | 11:55 |
| dtantsur | (and then we should document it better, because I'm confused myself) | 11:55 |
| Qianbiao | :) ok | 11:56 |
| openstackgerrit | Verification of a change to openstack/ironic failed: clean up mac address with the pxe configuration files https://review.opendev.org/750545 | 11:57 |
| *** rh-jelabarre has joined #openstack-ironic | 11:58 | |
| Qianbiao | dtantsur no lucky, it seems my test bm is not compatible with cirros, Kernel panic. | 12:01 |
| Qianbiao | dtantsur when using ironic deploy ramdisk(centos7 based), it can boot to login ui interface. | 12:12 |
| janders | dtantsur iurygregory when you have time, can you please have a look at https://review.opendev.org/#/c/750943/ and let me know if I'm going in the right direction - and also give me a pointer on where do I need to call this function from? This would help me hit the ground running when I get back to this tomorrow morning. Thanks! :) | 12:13 |
| patchbot | patch 750943 - ironic - Use OOB inspection to fetch MACs for IB inspection - 1 patch set | 12:13 |
| Qianbiao | I will try use dib build with centos7 | 12:13 |
| iurygregory | janders, I will try (probably not today) | 12:15 |
| Qianbiao | jander i think you update a wrong file. | 12:16 |
| Qianbiao | redfish inspector seems already query mac using redfish API. | 12:16 |
| iurygregory | ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts. | 12:21 |
| iurygregory | LOL | 12:21 |
| Qianbiao | jander sorry, it seems i misunderstand the RFE. | 12:21 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 12:38 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 12:46 |
| *** vmud213 has quit IRC | 12:52 | |
| *** vmud213 has joined #openstack-ironic | 12:56 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Adds ilo-uefi-https boot interface to ilo5 https://review.opendev.org/745501 | 12:56 |
| *** derekh has quit IRC | 12:58 | |
| *** rloo has joined #openstack-ironic | 13:09 | |
| iurygregory | the new pip is a pain (but does the right thing...) | 13:11 |
| iurygregory | https://pip.pypa.io/en/latest/user_guide/#changes-to-the-pip-dependency-resolver-in-20-2-2020 | 13:11 |
| *** priteau has quit IRC | 13:11 | |
| *** priteau has joined #openstack-ironic | 13:16 | |
| yoctozepto | dtantsur: it seems the error results in rc=-1 in gate as well but seemingly bash's errexit gets ignored; ah, the world is ending when bash fails | 13:26 |
| yoctozepto | iurygregory: LOL, I just noticed your notice | 13:26 |
| *** Goneri has joined #openstack-ironic | 13:26 | |
| iurygregory | yoctozepto, hehe | 13:26 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-python-agent master: Fix l-c https://review.opendev.org/750991 | 13:27 |
| iurygregory | my fix to ironic broke even more the l-c job lol | 13:27 |
| TheJulia | good morning everyone | 13:29 |
| iurygregory | good morning TheJulia =) | 13:30 |
| iurygregory | but I wouldn't look at the gate today | 13:30 |
| iurygregory | =) | 13:30 |
| *** tzumainn has joined #openstack-ironic | 13:34 | |
| TheJulia | dtantsur: fyi on my metalsmith changes, I think they are all good at this point. https://review.opendev.org/#/c/748315/ passes CI but it _feels_ like it is purely an ironic bug and I need to generate some artifacts and compare the code path and logs because it is extremely similar to a bug report I've got downstream as well. | 13:36 |
| patchbot | patch 748315 - metalsmith - Replace the Centos7 image job with a Centos8 UEFI ... - 19 patch sets | 13:36 |
| *** belmoreira has quit IRC | 13:36 | |
| TheJulia | well, https://review.opendev.org/#/c/748315/ doesn't pass CI yet, the job fails at where it looks like we have a legit bug | 13:36 |
| patchbot | patch 748315 - metalsmith - Replace the Centos7 image job with a Centos8 UEFI ... - 19 patch sets | 13:36 |
| * TheJulia drinks more coffee | 13:37 | |
| bfournie | dtantsur, rpioso: I'm seeing an issue with ironic setting a BIOS config value using clean steps to an idrac, the request to change the value is accepted but it doesn't stick so the subsequent verify fails. It looks like the ApplyTime must be set to "OnReset", for example that is what this script does https://github.com/dell/iDRAC-Redfish-Scripting/blob/master/Redfish%20Python/GetSetBiosAttributesREDFISH.py#L164 and | 13:38 |
| bfournie | works fine changing the value - http://paste.openstack.org/show/797718/ | 13:38 |
| bfournie | dtantsur, rpioso: should we be setting ApplyTime in sushy? | 13:38 |
| dtantsur | bfournie: I think there is a patch (from ajya?) for that | 13:39 |
| * bfournie looks | 13:39 | |
| ajya | bfournie: https://review.opendev.org/#/c/749240/ | 13:39 |
| patchbot | patch 749240 - ironic - Add Redfish BIOS interface to idrac HW type - 3 patch sets | 13:39 |
| rpioso | bfournie: That ^^^ depends on sushy change https://review.opendev.org/#/c/750020/ | 13:40 |
| patchbot | patch 750020 - sushy - Add BIOS update apply time and maintenance window (MERGED) - 2 patch sets | 13:40 |
| bfournie | ajya, rpioso: awesome! thank you | 13:41 |
| rpioso | bfournie: yw :-) | 13:42 |
| iurygregory | wut ipa l-c job is green \o/ 750991 https://zuul.openstack.org/status | 13:44 |
| dtantsur | iurygregory: is 1.0.1 really the minimum required? | 13:51 |
| dtantsur | I don't mind, but it's relatively recent.. | 13:51 |
| TheJulia | did the global requirement recently change? | 13:55 |
| iurygregory | dtantsur, maybe not the minimum ( I just gave a try with the value from requirements to see) | 13:56 |
| iurygregory | TheJulia, the problem is called ubuntu focal with python3.8... | 13:56 |
| TheJulia | I was worried that would bite us someplace | 13:57 |
| TheJulia | *sigh* | 13:57 |
| iurygregory | the l-c job now runs on focal and a few dependencies doesn't work very well | 13:57 |
| TheJulia | lovely :( | 13:57 |
| iurygregory | and pip sometimes complains with: ERROR: After October 2020 you may experience errors when installing or updating packages. This is because pip will change the way that it resolves dependency conflicts | 13:57 |
| TheJulia | yeah, I'm not looking forward to that pip release | 13:58 |
| iurygregory | yup hehe | 13:59 |
| iurygregory | but we will probably need to start looking =( https://pip.pypa.io/en/latest/user_guide/#changes-to-the-pip-dependency-resolver-in-20-2-2020 | 13:59 |
| TheJulia | nice guide! | 14:01 |
| Qianbiao | Good morning, the TheJulia. | 14:06 |
| iurygregory | I need to see a flat now (be back in a few) | 14:07 |
| Qianbiao | TheJulia, I got a new issue with ramdisk deploy interface. When OS start, it stuck with "Warning: /dev/ram0 does not exist". | 14:07 |
| Qianbiao | I have try build image base on centos7,8, ubuntu16,18. all the same. | 14:08 |
| Qianbiao | You may know the reason? | 14:09 |
| yoctozepto | iurygregory: my fav part: "Since this work will not change user-visible behavior described in the pip documentation, this change is not covered by the Deprecation Policy." | 14:10 |
| yoctozepto | smart | 14:10 |
| Qianbiao | console output: http://paste.openstack.org/show/797720/ | 14:10 |
| *** thiagop has quit IRC | 14:15 | |
| TheJulia | Qianbiao: interesting... | 14:22 |
| TheJulia | Qianbiao: if you edit the ipxe config manually and change it to root=ram0, does it work? | 14:22 |
| Qianbiao | will try. | 14:23 |
| Qianbiao | but i can not ls /dev/ram0 from anywhere | 14:23 |
| TheJulia | also, is that the kernel, or the initrd outputting that warning? | 14:23 |
| Qianbiao | is this a problem | 14:23 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add 'agent_token' to heartbeat request https://review.opendev.org/751000 | 14:23 |
| Qianbiao | initrd i think. | 14:23 |
| *** jtomasek has joined #openstack-ironic | 14:23 | |
| *** lmcgann_ has joined #openstack-ironic | 14:24 | |
| TheJulia | so yeah, that is for the kernel, it is likely the base os kernels looking for something, anything | 14:24 |
| TheJulia | because if your getting a shell, the interface worked, the host booted, it is all up to the initrd and it's contents at that point | 14:25 |
| TheJulia | Keep in mind, the ramdisk interface was intended for the scientific computing community which has long used things like disk-less machines for their compute farms. | 14:25 |
| Qianbiao | TheJulia Yes, we are providing nodes for PCSS | 14:26 |
| Qianbiao | For scientific computing | 14:27 |
| TheJulia | cool | 14:32 |
| TheJulia | yeah, so likely you just need a crafted ramdisk. Have you tried making one with diskimage-builder? | 14:33 |
| openstackgerrit | Mudit proposed openstack/ironic master: Implement reporting GPU devices in iDRAC driver https://review.opendev.org/751006 | 14:34 |
| Qianbiao | TheJulia Yes, I have create image based on cent7,8, ubuntu xenial, bionic. | 14:39 |
| Qianbiao | But no luck. btw, the disks on bm is not removed, does it matter? | 14:39 |
| dtantsur | TheJulia: I think we have a documentation gap in this area | 14:39 |
| dtantsur | Qianbiao: the presence of disks should not matter | 14:40 |
| TheJulia | dtantsur: yeah, I think it is "how to make a ramdisk" though | 14:40 |
| dtantsur | yep | 14:40 |
| TheJulia | Qianbiao: so instead of dib, have you taken one of the filesystems it generates and tried converting it over to a cpio archive which is then compressed using gzip? | 14:40 |
| dtantsur | don't we have a CI job for that? or does it only use an ISO? | 14:41 |
| TheJulia | dtantsur: ci for it explicitly got held up, it is tested via the iso path at this point | 14:41 |
| TheJulia | dtantsur: we had community feedback that it worked very early on, so I didn't stress about it much | 14:42 |
| dtantsur | once/if we figure this out, let's document it? | 14:42 |
| Qianbiao | TheJulia nope, do not know how to indeed :( | 14:42 |
| dtantsur | I guess the ISO approach won't work with Nova? Qianbiao, do you use Nova? | 14:42 |
| Qianbiao | dtantsur yes, i am using nova | 14:43 |
| dtantsur | right, hence glance, hence you need one image | 14:43 |
| dtantsur | TheJulia: how do we make it work with nova? by providing an empty image with kernel/ramdisk attached? | 14:43 |
| TheJulia | dtantsur: yeah, exactly that | 14:43 |
| Qianbiao | TheJulia dtantsur my image looks like http://paste.openstack.org/show/797725/ | 14:44 |
| TheJulia | hmm ibm's how to make a ramdisk is not exactly what Qianbiao needs | 14:44 |
| * dtantsur thinks | 14:44 | |
| dtantsur | I don't think we have an easy-to-use stock procedure for that, do we? | 14:44 |
| dtantsur | the first step would be to figure our what to pass for root if root=/dev/ram0 does not work | 14:46 |
| dtantsur | Qianbiao: could you update http://paste.openstack.org/show/797720/ with the complete output? | 14:46 |
| TheJulia | dtantsur: no, we don't and we wanted to avoid having a how to create a ramdisk doc | 14:47 |
| Qianbiao | dtantsur sure | 14:47 |
| TheJulia | https://access.redhat.com/solutions/24029 <-- "gzip format - Repack / Recompress" | 14:48 |
| TheJulia | so tl;dr is you mount the filesystem, and then make a cpio archive from it that has been gzip compressed | 14:48 |
| TheJulia | when the kernel boots, it will load that new archive you've created (which will be larger, and will need to fit into memory) | 14:49 |
| TheJulia | the initrd dib creates in most cases is just a shim with kernel modules to take the system from Kernel boot to pivot to the running operating system's root partition. | 14:49 |
| TheJulia | dtantsur: at his stage, root=/dev/ram0 is really just for the kernel, If the initrd he is loading is kicking that out it is because it doesn't grok it and thinks that the user is wanting to pivot with that ramdisk. | 14:50 |
| dtantsur | that's what we have to do, no? | 14:50 |
| Qianbiao | dtantsur paste.openstack can not paste so long. any other file share? | 14:50 |
| TheJulia | no, you use a single ramdisk, you don't try to pivot ramdisks | 14:51 |
| dtantsur | Qianbiao: try https://pastebin.com/? | 14:51 |
| TheJulia | the kernel knows how to handle stacked ramdisks anyway | 14:51 |
| dtantsur | TheJulia: well, the kernel needs a root IIRC | 14:51 |
| TheJulia | yes, and the initrd provides the first root | 14:51 |
| Qianbiao | it seems "ram0" is wrong too TheJulia. "mount: mounting ram0 on /root failed: No such device" | 14:51 |
| dtantsur | mmm, no, something via root= | 14:51 |
| TheJulia | yeah, that should also work | 14:52 |
| TheJulia | some kernels _really_ want a root on the command line :( | 14:52 |
| dtantsur | actually, what if.. | 14:52 |
| dtantsur | Qianbiao: what if you edit the ipxe template to remove root=/dev/ram0 completely? | 14:52 |
| TheJulia | that is definitely worth a try | 14:53 |
| dtantsur | (or pxe template or whatever you're using) | 14:53 |
| Qianbiao | TheJulia dtantsur for root=ram0 https://pastebin.com/EuUAgBsG | 14:53 |
| Qianbiao | ok will try now | 14:53 |
| TheJulia | Qianbiao: ^^^ was truncated | 14:54 |
| Qianbiao | hmm. too long i think.. | 14:54 |
| dtantsur | yeah.. | 14:55 |
| Qianbiao | push too a github repo? | 14:55 |
| dtantsur | Qianbiao: also, where is literal "ram0" coming from? our templates seem to have /dev/ram0.. | 14:55 |
| dtantsur | have you stripped /dev/ yourself? | 14:55 |
| dtantsur | https://gist.github.com/ maybe? | 14:55 |
| Qianbiao | <dtantsur> yes, TheJulia suggest that | 14:55 |
| dtantsur | ah, okay | 14:55 |
| Qianbiao | I updated the template | 14:55 |
| dtantsur | let's try removing root=.. completely now | 14:56 |
| Qianbiao | ok will do now | 14:56 |
| Qianbiao | gist does not work too. same position truncated | 14:56 |
| Qianbiao | maybe copy board of window 10 truncated it. | 14:57 |
| TheJulia | yeah, that could be | 14:57 |
| Qianbiao | https://gist.github.com/IamFive/28a7580e077badf49f65c3e2eacc2bc1 | 14:58 |
| Qianbiao | for root=ramo | 14:58 |
| Qianbiao | will try remove it now | 14:58 |
| * iurygregory is back | 15:00 | |
| iurygregory | yoctozepto, for sure that's the best part! hehe | 15:00 |
| TheJulia | yup, your in the limited shim ramdisk for booting a machinef rom disk. | 15:01 |
| Qianbiao | TheJulia you find the issue? | 15:02 |
| Qianbiao | I do not understand "limited shim ramdisk" means. | 15:02 |
| TheJulia | Qianbiao: yes, see the issue | 15:03 |
| TheJulia | The ramdisk that is being booted really has no concept of anything beyond trying to boot to disks. That is why you get prompted with an emergency shell | 15:03 |
| yoctozepto | iurygregory: aye; as if the primary function of pip was not documented :-) | 15:04 |
| iurygregory | yeah | 15:04 |
| Qianbiao | Hmm. do u mean the boot process still try to boot to a disk? | 15:05 |
| Qianbiao | What should i do to fix it. | 15:05 |
| TheJulia | I'm testing to see if the old diskimage-builder ramdisk-image-create command still works, if that doesn't work I'll write something up for you in the documentation since the problem is the file being submitted as the ramdisk | 15:06 |
| Qianbiao | TheJulia thanks. | 15:06 |
| Qianbiao | I have guessed maybe it's caused by the ramdisk. | 15:07 |
| Qianbiao | It displays "Attached SCSI disk", i was wondering why it still try do this :) | 15:07 |
| TheJulia | hmm, seems like my local centos8 mirror has a package issue, retrying 7 | 15:08 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Followups for TLS support https://review.opendev.org/750818 | 15:09 |
| dtantsur | iurygregory: any insight into why the update ironic l-c job failed? | 15:09 |
| iurygregory | dtantsur, seems like the version I put for psycopg2 caused problems | 15:10 |
| iurygregory | I'm trying to reproduce locally | 15:11 |
| iurygregory | because when I tested it worked fine all tests | 15:11 |
| Qianbiao | <dtantsur> update: remove root=/dev/ram0 has no luck too. | 15:13 |
| yoctozepto | dtantsur: welcome to the world of bash quirks: http://paste.openstack.org/show/797730/ - guess the return value of the script | 15:15 |
| yoctozepto | :-) | 15:15 |
| * dtantsur sighs | 15:15 | |
| dtantsur | I won't even try, bash behavior is mostly dark magic for me | 15:16 |
| dtantsur | nor do I understand why it works this way.. | 15:17 |
| dtantsur | unrelated, but apparently https://review.opendev.org/#/c/744657/ will not just solve itself | 15:18 |
| patchbot | patch 744657 - ironic - [WIP] Correctly handle default_boot_mode - 2 patch sets | 15:18 |
| yoctozepto | dtantsur: errexit stops being respected in code under the "if" clause, just like that | 15:19 |
| yoctozepto | so we had this in somewhere in function call stack | 15:19 |
| dtantsur | computers were a mistake | 15:19 |
| yoctozepto | and voila, "your errors do not matter to me" said bash | 15:19 |
| dtantsur | well, TIL | 15:19 |
| yoctozepto | indeed, hence sharing | 15:20 |
| openstackgerrit | Merged openstack/ironic master: Add 'agent_token' to heartbeat request https://review.opendev.org/751000 | 15:21 |
| *** mkrai__ has quit IRC | 15:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Handle default_boot_mode during cleaning/inspection with PXE/iPXE https://review.opendev.org/744657 | 15:33 |
| dtantsur | TheJulia: I'm impressed nobody has reported ^^^ until today | 15:33 |
| Qianbiao | is the story board broken? i can not create new story. | 15:35 |
| TheJulia | ohhh fun | 15:35 |
| TheJulia | Qianbiao: works for me, try clearing your browser cache | 15:35 |
| Qianbiao | ok | 15:35 |
| *** Qianbiao has quit IRC | 15:37 | |
| *** Qianbiao has joined #openstack-ironic | 15:38 | |
| Qianbiao | still cant. someone may help create a story: http://paste.openstack.org/show/797731/ | 15:39 |
| Qianbiao | there are js error. "Error: Input data should be a String" | 15:40 |
| *** k_mouza has quit IRC | 15:41 | |
| dtantsur | it rings some bells.. | 15:41 |
| TheJulia | did you completely fill out the project name or did you let it auto-complete it? | 15:41 |
| dtantsur | Qianbiao: how do you input project? try explicitly selecting it from the auto-completion list. | 15:41 |
| dtantsur | right :) | 15:42 |
| Qianbiao | TheJulia dtantsur when i type anything, the auto complete drop list is not shown | 15:42 |
| dtantsur | even if you start "openstack/ir.." and wait? | 15:43 |
| Qianbiao | oh.. it works now. | 15:43 |
| Qianbiao | TheJulia I create a story to track the ramdisk deploy issue, https://storyboard.openstack.org/#!/story/2007554 | 15:45 |
| Qianbiao | very thank you, big help. Have stuck on this issue for a long time. | 15:46 |
| iurygregory | ok now things will be easier I assume (got a VM with focal to test things) | 15:49 |
| *** rcernin has quit IRC | 15:50 | |
| *** Lucas_Gray has quit IRC | 15:52 | |
| TheJulia | Qianbiao: I almost have instructions for you | 15:52 |
| Qianbiao | TheJulia really! I can try it now. | 15:53 |
| TheJulia | yup, almosst there, waiting for the last command to finish so I can give you my history output | 15:53 |
| *** k_mouza has joined #openstack-ironic | 15:54 | |
| TheJulia | seems I need a desktop upgrade :\ | 15:54 |
| * Qianbiao waiting... | 15:54 | |
| TheJulia | it is compressing a ramdisk together, so will be a couple minutes | 15:54 |
| *** rloo has quit IRC | 15:57 | |
| Qianbiao | Trying to build a lineless desktop. | 15:57 |
| *** rloo has joined #openstack-ironic | 15:57 | |
| TheJulia | Qianbiao: the link you pasted was for a different story | 15:58 |
| Qianbiao | aha checking | 15:58 |
| Qianbiao | sorry, https://storyboard.openstack.org/#!/story/2008126 | 15:59 |
| *** lucasagomes has quit IRC | 16:02 | |
| *** k_mouza has quit IRC | 16:04 | |
| TheJulia | Qianbiao: http://paste.openstack.org/show/797733/ _should_ work | 16:04 |
| TheJulia | but I'm also rusty on ramdisks, it has been a long time since I built one manually | 16:04 |
| iurygregory | ++ for the name of TheJulia's machine | 16:05 |
| Qianbiao | TheJulia ty, will try. | 16:05 |
| TheJulia | iurygregory: it wants to fly! | 16:06 |
| TheJulia | for ages, my desktop was always called agamemnon | 16:06 |
| iurygregory | =O | 16:06 |
| iurygregory | mine is called blacklotus | 16:06 |
| TheJulia | my file/web server was always called "earthling" | 16:07 |
| erbarr | i was just noticing the name too | 16:07 |
| *** vmud213 has quit IRC | 16:08 | |
| * dtantsur is a boring person with his 'dtantsur-laptop' | 16:08 | |
| *** diurnalist has joined #openstack-ironic | 16:09 | |
| iurygregory | ouch | 16:09 |
| * iurygregory gives a drink to dtantsur | 16:09 | |
| dtantsur | no drinks today, planning on bouldering tomorrow morning :0 | 16:09 |
| TheJulia | now speaking of making disk images, I need to go craft a disk image because I think there is a legitimate bug we're hitting | 16:09 |
| dtantsur | I mean, :) | 16:09 |
| TheJulia | dtantsur: rock wall or actual boulders? | 16:11 |
| Qianbiao | TheJulia may the distro be others? like centos | 16:13 |
| dtantsur | TheJulia: I don't know the difference between the English terms, so I'll just show it: https://duckduckgo.com/?q=monkeyspot+duesseldorf&t=ffab&iax=images&ia=images | 16:13 |
| dtantsur | (all photos are outdated, they're rebuilding the place constantly) | 16:14 |
| Qianbiao | dtantsur nice | 16:14 |
| TheJulia | climbing wall/rock wall :) | 16:14 |
| TheJulia | at least that is how I know people that would refer to it here int he states | 16:15 |
| *** k_mouza has joined #openstack-ironic | 16:15 | |
| dtantsur | I got confused by the "rock" bit, it's not real rock :) | 16:16 |
| dtantsur | we go to another place that has an emulation of a real rock wall | 16:16 |
| dtantsur | with cracks and other features you can hold onto | 16:17 |
| TheJulia | yeah | 16:19 |
| TheJulia | nice | 16:19 |
| TheJulia | bbiam, I need to reboot my graphics drivers are giving me artifacts for my window manager :( | 16:20 |
| *** afasano has joined #openstack-ironic | 16:21 | |
| dtantsur | and I'll leave for a day and will be out tomorrow. see you on Monday! | 16:22 |
| *** dtantsur is now known as dtantsur|afk | 16:22 | |
| Qianbiao | dtantsur see you, have a nice weekend | 16:24 |
| *** diurnalist has quit IRC | 16:24 | |
| *** diurnalist has joined #openstack-ironic | 16:33 | |
| iurygregory | funny the VM with ubuntu focal desktop doesn't open the terminal lol | 16:35 |
| *** abdysn has quit IRC | 16:38 | |
| Qianbiao | <TheJulia> Hi, has a panic. ---[ end Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(0,0) ]--- | 16:44 |
| *** afasano has quit IRC | 16:45 | |
| TheJulia | Qianbiao: what is the kernel command line? | 16:45 |
| Qianbiao | it seems to try mount fs | 16:45 |
| TheJulia | actually, the entire kernel output would be good | 16:45 |
| Qianbiao | moment try to download from bmc | 16:46 |
| *** k_mouza has quit IRC | 16:49 | |
| *** diurnalist has quit IRC | 16:51 | |
| *** cp- has quit IRC | 16:52 | |
| Qianbiao | TheJulia https://github.com/IamFive/gist/blob/master/debian-minimal-ramdisk.log | 16:53 |
| Qianbiao | it seems there a some very special chars breaks copy paste in the log | 16:54 |
| Qianbiao | From line#707 every time breanks there. | 16:54 |
| TheJulia | try changing root to root=initrd and also try removing it altogether | 16:55 |
| Qianbiao | Ok TheJulia | 16:56 |
| Qianbiao | I will test it tomorrow, and feedback it to you when you are online tomorrow. | 16:56 |
| TheJulia | okay, have a good evening! | 16:57 |
| Qianbiao | Need to go, it's 1:00 AM here | 16:57 |
| Qianbiao | ok, ty. | 16:57 |
| *** cp- has joined #openstack-ironic | 16:57 | |
| *** dking has joined #openstack-ironic | 16:58 | |
| *** Qianbiao has quit IRC | 16:58 | |
| *** gyee has joined #openstack-ironic | 17:03 | |
| *** k_mouza has joined #openstack-ironic | 17:04 | |
| *** rloo has quit IRC | 17:13 | |
| *** rloo has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Yogesh Ramachandra proposed openstack/ironic master: [WIP]Fix idrac-wsman bios factory-reset setting https://review.opendev.org/748696 | 17:14 |
| *** tonyb has quit IRC | 17:15 | |
| *** k_mouza has quit IRC | 17:37 | |
| *** Lucas_Gray has joined #openstack-ironic | 17:42 | |
| *** thiagop has joined #openstack-ironic | 17:52 | |
| *** thiagop has quit IRC | 17:54 | |
| *** thiagop has joined #openstack-ironic | 17:54 | |
| *** tonyb has joined #openstack-ironic | 17:57 | |
| *** thiagop has quit IRC | 18:04 | |
| *** thiagop has joined #openstack-ironic | 18:05 | |
| *** gmann is now known as gmann_afk | 18:11 | |
| rpioso | bfournie, dtantsur|afk, TheJulia: On the ironic WhiteBoard, we changed the idrac vendor priority to https://review.opendev.org/#/c/749240/. Reviews are most welcome :) | 18:20 |
| patchbot | patch 749240 - ironic - Add Redfish BIOS interface to idrac HW type - 3 patch sets | 18:20 |
| TheJulia | ack | 18:21 |
| rpioso | TheJulia: Note that it has a dependency on a recently merged sushy change. Will sushy need to be released b4 it can land? | 18:22 |
| bfournie | rpioso: just recently updated, along with the sushy change, this fixes the issue I was seeing earlier \o/ | 18:22 |
| rpioso | bfournie: Awesome! Thank you so much for taking it for a spin against iDRAC 8. 13th Generation (13G), right? | 18:24 |
| TheJulia | rpioso: do you have the link to that sushy change? | 18:25 |
| bfournie | rpioso: yes, 430 running 2.75.75.75, just tested redfish though. this fix couldn't have come at a better time | 18:25 |
| rpioso | TheJulia: https://review.opendev.org/#/c/750020/ | 18:25 |
| patchbot | patch 750020 - sushy - Add BIOS update apply time and maintenance window (MERGED) - 2 patch sets | 18:25 |
| TheJulia | rpioso: so it should be in the latest sushy | 18:26 |
| rpioso | TheJulia: Sweet! I must have missed the release notice. | 18:26 |
| * rpioso sees sushy 3.4.0 picked it up :) | 18:29 | |
| rpioso | bfournie, TheJulia: Should we plan to backport it to stable branches? | 18:29 |
| rpioso | At least the sushy change above ^^^ | 18:30 |
| rpioso | Edit: That would be both. | 18:31 |
| TheJulia | uhh... why? | 18:36 |
| bfournie | rpioso: not a requirement for us to backport for now | 18:37 |
| TheJulia | and a backport wouldn't be approved, it is a feature backport | 18:37 |
| TheJulia | well, it is a feature | 18:37 |
| rpioso | TheJulia: bfournie referred to it as a fix. | 18:38 |
| rpioso | It makes the redfish driver BIOS work against iDRAC BMC. | 18:38 |
| TheJulia | https://review.opendev.org/#/c/750020/2/releasenotes/notes/add-apply-time-support-to-bios-315ebad429dcab3d.yaml | 18:38 |
| patchbot | patch 750020 - sushy - Add BIOS update apply time and maintenance window (MERGED) - 2 patch sets | 18:38 |
| openstackgerrit | Aija Jaunteva proposed openstack/ironic master: Redfish driver firmware update https://review.opendev.org/749619 | 18:39 |
| rpioso | TheJulia: Yep. It's subjective. Could be considered a fix, too. | 18:39 |
| bfournie | rpioso: TheJulia: yeah it did fix the bios config setting through redfish to idrac | 18:39 |
| TheJulia | It has also been semi-common knowledge that the base redfish interface was incompatible with idrac bmcs before now. So I don't see how functionality/feature enablement now is a bugfix when prior context was incompatibility. | 18:41 |
| TheJulia | well, incomatible with idrac bmcs with regards to the bios interface | 18:41 |
| rpioso | bfournie, TheJulia: I'm good either way, whichever the community prefers. I believe it could be considered either. | 18:44 |
| TheJulia | The case can absolutely be made | 18:47 |
| openstackgerrit | Julia Kreger proposed openstack/metalsmith master: Replace the Centos7 image job with a Centos8 UEFI based job https://review.opendev.org/748315 | 18:55 |
| *** priteau has quit IRC | 19:16 | |
| *** openstackgerrit has quit IRC | 20:36 | |
| *** thiagop has quit IRC | 20:55 | |
| *** outbrito has joined #openstack-ironic | 20:55 | |
| *** openstackgerrit has joined #openstack-ironic | 21:03 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add dosfsutils for EFI to tinyipa https://review.opendev.org/751103 | 21:03 |
| TheJulia | DOH! | 21:07 |
| *** jtomasek has quit IRC | 21:11 | |
| *** outbrito has quit IRC | 21:11 | |
| clarkb | we literally just ran into that with diskimagebuilder for nodepool too | 21:12 |
| TheJulia | \o/ | 21:15 |
| JayF | TheJulia: is that a little... excessive for a commit message? It violates a lot of the rules that used to be enforced by openstack-flake? | 21:22 |
| JayF | the change is simple and LGTM, but I can hear someone's git-parsing-script weeping in the distance in anticipation of the merge | 21:23 |
| TheJulia | Is it wrong that I want to laugh evilly? | 21:26 |
| JayF | Is it wrong that I remember like, forever ago, when we'd have CI that'd validate those commit messages, and it got dropped because human reviewers promised to check it :P | 21:27 |
| * JayF is willing to be the bad guy this time and stand up for the crappy-git-scraping-robots of the world | 21:27 | |
| clarkb | JayF: whats wrong with it? | 21:30 |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent-builder master: Add dosfsutils for EFI to tinyipa https://review.opendev.org/751103 | 21:30 |
| JayF | https://wiki.openstack.org/wiki/GitCommitMessages | 21:30 |
| JayF | The first line should be limited to 50 characters and should not end with a period. | 21:30 |
| JayF | Subsequent lines should be wrapped at 72 characters. | 21:30 |
| TheJulia | I'm fairly sure a lot of the git scraping scripts that HP had are long gone | 21:30 |
| clarkb | JayF: its quoting things directly though | 21:31 |
| clarkb | the prose should be wrapped though | 21:31 |
| JayF | I think it's possible to communicate the issue, and how it's fixed, without pasting a full traceback. | 21:31 |
| JayF | and honestly, I'm lawfully aligned... if it's written down as a rule, it seems like we should follow it | 21:32 |
| JayF | if the written down rule is wrong, we should fix it | 21:32 |
| clarkb | yes I think the old written rules were wrong :) | 21:32 |
| clarkb | we managed to get rid of the hacking rule for it iirc | 21:32 |
| clarkb | (or at least its off by default) | 21:32 |
| * TheJulia suggests we act like Wil E. Coyote and make some additions to the foundation of wiki.openstack.org | 21:33 | |
| JayF | IIRC that hacking rule was removed for technical reasons with a promise we'd nforce it in human review | 21:33 |
| JayF | but that was long enough ago that my memory is beyond unreliable | 21:33 |
| clarkb | JayF: ya we disabled it on all infra projects because it was silly | 21:33 |
| clarkb | long before any changes happened to the tool itself so not sure what the ultimate reasoning was | 21:33 |
| TheJulia | when did it become 2:30 pm *blink* *blink* | 21:34 |
| clarkb | fwiw I think quoting tracebacks in commit messages is often useful as it neatly captures the underlying issue assocaited with a commit without relying on secondary tools like a bug tracker | 21:34 |
| TheJulia | in this case it is also not a bug really.... tinycore is explicitly for testing | 21:34 |
| TheJulia | err, tinyipa | 21:34 |
| JayF | I don't disagree that it *can* be a valid approach to code/context management. It's just not the one OpenStack uses :) | 21:34 |
| TheJulia | so our own silly fault we didn't already find it :( | 21:34 |
| *** lmcgann_ has quit IRC | 21:35 | |
| JayF | TheJulia in here fixing problems. Me? I'm paintin' a shed :P | 21:35 |
| JayF | it's a quite lovely shade of chartruese though, if you ask me | 21:36 |
| TheJulia | we secretly replaced his paint with de-cafe. Lets see if he notices! | 21:36 |
| TheJulia | 8) | 21:36 |
| TheJulia | decaf I guess | 21:36 |
| TheJulia | decaffinated | 21:36 |
| TheJulia | https://twitter.com/ashinclouds/status/1304172364633055233 | 21:38 |
| *** Goneri has quit IRC | 21:49 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent master: Change integration test for Metalsmith to use Centos8/UEFI https://review.opendev.org/749772 | 21:52 |
| *** zzzeek has quit IRC | 22:26 | |
| *** zzzeek has joined #openstack-ironic | 22:27 | |
| *** Lucas_Gray has quit IRC | 22:35 | |
| janders | good morning Ironic o/ | 22:41 |
| stevebaker | janders: hi | 22:42 |
| *** gmann_afk is now known as gmann | 22:49 | |
| *** rloo has quit IRC | 22:53 | |
| *** rcernin has joined #openstack-ironic | 22:58 | |
| rpioso | TheJulia: We could split the pending driver changes into two -- idrac and redfish. That would support back porting redfish, should we decide to consider it a bug. | 22:58 |
| rpioso | TheJulia: When will the FFE window open? | 22:58 |
| *** rcernin has quit IRC | 22:59 | |
| *** rcernin has joined #openstack-ironic | 22:59 | |
| *** zzzeek has quit IRC | 23:24 | |
| *** zzzeek has joined #openstack-ironic | 23:29 | |
| *** zzzeek has quit IRC | 23:58 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!