| *** fragatina has joined #openstack-ironic | 00:00 | |
| *** fragatina has quit IRC | 00:00 | |
| *** fragatina has joined #openstack-ironic | 00:01 | |
| *** xiaoyang has joined #openstack-ironic | 00:12 | |
| *** baoli has quit IRC | 00:16 | |
| *** yuanying_ has joined #openstack-ironic | 00:19 | |
| *** yuanying has quit IRC | 00:19 | |
| *** yuanying_ has quit IRC | 00:19 | |
| *** yuanying has joined #openstack-ironic | 00:19 | |
| *** yuanying_ has joined #openstack-ironic | 00:20 | |
| *** yuanying has quit IRC | 00:20 | |
| *** yuanying has joined #openstack-ironic | 00:21 | |
| *** yuanying_ has quit IRC | 00:21 | |
| *** yuanying_ has joined #openstack-ironic | 00:21 | |
| *** yuanying has quit IRC | 00:21 | |
| *** yuanying_ has quit IRC | 00:21 | |
| *** yuanying has joined #openstack-ironic | 00:22 | |
| *** yuanying has quit IRC | 00:22 | |
| *** yuanying has joined #openstack-ironic | 00:22 | |
| *** SerenaFeng has joined #openstack-ironic | 00:50 | |
| *** hoangcx has joined #openstack-ironic | 00:55 | |
| *** rbudden has joined #openstack-ironic | 00:57 | |
| *** phuongnh has joined #openstack-ironic | 00:59 | |
| openstackgerrit | Bob Fournier proposed openstack/ironic-inspector: Add plugin to process basic management LLDP TLVs https://review.openstack.org/406496 | 01:12 |
|---|---|---|
| *** zhangjl has joined #openstack-ironic | 01:21 | |
| *** rama_y has joined #openstack-ironic | 01:27 | |
| *** zackf has joined #openstack-ironic | 01:37 | |
| *** baoli has joined #openstack-ironic | 01:45 | |
| *** aNuposic has joined #openstack-ironic | 01:46 | |
| *** baoli has quit IRC | 01:50 | |
| *** yufei has joined #openstack-ironic | 01:53 | |
| *** rbudden has quit IRC | 01:55 | |
| *** yufei has quit IRC | 01:58 | |
| *** yufei has joined #openstack-ironic | 02:00 | |
| *** rbudden has joined #openstack-ironic | 02:04 | |
| *** Goneri has joined #openstack-ironic | 02:10 | |
| *** SerenaFeng has quit IRC | 02:13 | |
| *** SerenaFeng has joined #openstack-ironic | 02:17 | |
| *** aNuposic has quit IRC | 02:23 | |
| *** rbudden has quit IRC | 02:35 | |
| *** lujinluo has joined #openstack-ironic | 02:39 | |
| *** fragatina has quit IRC | 02:52 | |
| *** yuanying has quit IRC | 02:53 | |
| *** phuongnh has quit IRC | 02:53 | |
| *** fragatina has joined #openstack-ironic | 02:53 | |
| *** baoli has joined #openstack-ironic | 02:55 | |
| *** shuquan_ has joined #openstack-ironic | 02:56 | |
| *** yuanying has joined #openstack-ironic | 03:00 | |
| *** baoli has quit IRC | 03:09 | |
| *** Goneri has quit IRC | 03:10 | |
| *** baoli has joined #openstack-ironic | 03:13 | |
| *** zackf has quit IRC | 03:17 | |
| *** baoli has quit IRC | 03:27 | |
| *** baoli has joined #openstack-ironic | 03:27 | |
| *** zackf has joined #openstack-ironic | 03:32 | |
| *** baoli has quit IRC | 03:32 | |
| *** SerenaFeng has quit IRC | 03:33 | |
| *** rbudden has joined #openstack-ironic | 03:34 | |
| *** yufei__ has joined #openstack-ironic | 03:34 | |
| *** vikrant has joined #openstack-ironic | 03:35 | |
| *** yufei has quit IRC | 03:35 | |
| *** yufei__ is now known as yufei | 03:35 | |
| *** misty-cloud has joined #openstack-ironic | 03:43 | |
| *** misty-cloud- has joined #openstack-ironic | 03:46 | |
| *** misty-cloud- has quit IRC | 03:46 | |
| *** misty-cloud- has joined #openstack-ironic | 03:47 | |
| *** yuanying has quit IRC | 03:48 | |
| *** misty-cloud has quit IRC | 03:49 | |
| *** fragatin_ has joined #openstack-ironic | 04:04 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic-specs: Use uuidutils instead of uuid.uuid4(). https://review.openstack.org/406692 | 04:04 |
| *** fragatina has quit IRC | 04:05 | |
| *** SerenaFeng has joined #openstack-ironic | 04:09 | |
| *** SerenaFeng has quit IRC | 04:10 | |
| openstackgerrit | Sukhdev Kapur proposed openstack/ironic: Documentation for Security Groups for baremetal servers https://review.openstack.org/393962 | 04:10 |
| *** zhangjl1 has joined #openstack-ironic | 04:21 | |
| *** zhangjl has quit IRC | 04:22 | |
| *** baoli has joined #openstack-ironic | 04:28 | |
| *** baoli has quit IRC | 04:33 | |
| openstackgerrit | Aparna proposed openstack/proliantutils: Update hpssacli to ssacli https://review.openstack.org/396509 | 04:39 |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:45 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Support to enable/disable iscsi boot option in iLO https://review.openstack.org/406706 | 04:52 |
| *** yuanying has joined #openstack-ironic | 04:54 | |
| *** rbudden has quit IRC | 04:55 | |
| *** krtaylor has quit IRC | 05:11 | |
| *** alexpilo_ has joined #openstack-ironic | 05:22 | |
| *** alexpilotti has quit IRC | 05:24 | |
| *** baoli has joined #openstack-ironic | 05:29 | |
| *** baoli has quit IRC | 05:34 | |
| *** SerenaFeng has joined #openstack-ironic | 05:40 | |
| *** aaa801 has quit IRC | 05:43 | |
| *** mgagne has quit IRC | 05:53 | |
| *** mgagne has joined #openstack-ironic | 05:55 | |
| *** mgagne is now known as Guest51737 | 05:55 | |
| *** rbudden has joined #openstack-ironic | 06:06 | |
| *** shuquan_ has quit IRC | 06:06 | |
| *** shuquan has joined #openstack-ironic | 06:08 | |
| *** shuquan has quit IRC | 06:13 | |
| *** shuquan has joined #openstack-ironic | 06:15 | |
| *** karts has joined #openstack-ironic | 06:19 | |
| *** rama_y has quit IRC | 06:19 | |
| *** rbudden has quit IRC | 06:24 | |
| *** ohamada has joined #openstack-ironic | 06:26 | |
| *** baoli has joined #openstack-ironic | 06:29 | |
| *** baoli has quit IRC | 06:34 | |
| *** zackf has quit IRC | 06:35 | |
| *** tuanluong has joined #openstack-ironic | 06:35 | |
| *** zackf has joined #openstack-ironic | 06:36 | |
| *** rcernin has quit IRC | 06:39 | |
| *** zackf has quit IRC | 06:41 | |
| xiaoyang | hi, jroll. Does ironic support the boot bare metal from volume? | 06:46 |
| *** penick_ has joined #openstack-ironic | 06:46 | |
| xiaoyang | ping jroll | 06:47 |
| *** zzzeek_ has joined #openstack-ironic | 06:47 | |
| *** another_larsks has joined #openstack-ironic | 06:50 | |
| *** mjura has joined #openstack-ironic | 06:52 | |
| *** zzzeek has quit IRC | 06:52 | |
| *** penick has quit IRC | 06:52 | |
| *** penick_ is now known as penick | 06:52 | |
| *** yuanying_ has joined #openstack-ironic | 06:52 | |
| *** dhellmann_ has joined #openstack-ironic | 06:53 | |
| *** larsks has quit IRC | 06:53 | |
| *** dhellmann has quit IRC | 06:53 | |
| *** Marx314 has quit IRC | 06:53 | |
| *** vinsh has joined #openstack-ironic | 06:53 | |
| *** yuanying has quit IRC | 06:55 | |
| *** vinsh_ has quit IRC | 06:56 | |
| *** dhellmann_ is now known as dhellmann | 06:56 | |
| *** mjura has quit IRC | 06:58 | |
| *** mjura has joined #openstack-ironic | 06:58 | |
| *** e0ne has joined #openstack-ironic | 07:02 | |
| *** shuquan has quit IRC | 07:06 | |
| *** zackf has joined #openstack-ironic | 07:08 | |
| *** ChubYann has quit IRC | 07:09 | |
| *** zackf has quit IRC | 07:13 | |
| *** ccamacho has joined #openstack-ironic | 07:14 | |
| *** hw_wutianwei has joined #openstack-ironic | 07:14 | |
| *** shuquan_ has joined #openstack-ironic | 07:14 | |
| *** hw_wutianwei has quit IRC | 07:15 | |
| *** hw_wutianwei has joined #openstack-ironic | 07:15 | |
| *** moshele has joined #openstack-ironic | 07:15 | |
| *** SerenaFeng has quit IRC | 07:18 | |
| *** fragatin_ has quit IRC | 07:18 | |
| *** rcernin has joined #openstack-ironic | 07:28 | |
| *** rcernin has quit IRC | 07:29 | |
| *** baoli has joined #openstack-ironic | 07:30 | |
| *** vikrant is now known as vikrant|lunch | 07:32 | |
| *** rcernin has joined #openstack-ironic | 07:32 | |
| *** e0ne has quit IRC | 07:34 | |
| *** baoli has quit IRC | 07:46 | |
| *** rbartal has joined #openstack-ironic | 07:51 | |
| *** d0ugal has joined #openstack-ironic | 07:51 | |
| *** rcernin has quit IRC | 07:52 | |
| *** jtomasek has joined #openstack-ironic | 07:52 | |
| *** rcernin has joined #openstack-ironic | 07:59 | |
| *** SerenaFeng has joined #openstack-ironic | 08:00 | |
| *** rcernin has quit IRC | 08:01 | |
| *** openstackgerrit has quit IRC | 08:03 | |
| *** rcernin has joined #openstack-ironic | 08:03 | |
| pas-ha | morning Ironic :) | 08:05 |
| pas-ha | xiaoyang: I think he is sleeping yet ;) mid-US TZ | 08:05 |
| vdrok | good morning ironic and pas-ha ! | 08:06 |
| pas-ha | an no, boot from volume is not yet supported, although support is planned | 08:06 |
| pas-ha | morning vdrok | 08:06 |
| *** ricardoas has quit IRC | 08:07 | |
| *** ricardoas has joined #openstack-ironic | 08:08 | |
| xiaoyang | thx pas-ha, morning :P | 08:09 |
| xiaoyang | pas-ha: Is there a design document on it ? | 08:11 |
| *** hoonetorg has quit IRC | 08:12 | |
| pas-ha | xiaoyang: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/boot-from-volume-reference-drivers.html | 08:13 |
| xiaoyang | Thank you very much :) | 08:16 |
| *** openstackgerrit has joined #openstack-ironic | 08:18 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector-specs: Collect Bios, Memory and Processor Info from Dmidecode https://review.openstack.org/406790 | 08:18 |
| *** yuanying_ has quit IRC | 08:19 | |
| *** yufei has quit IRC | 08:20 | |
| *** zackf has joined #openstack-ironic | 08:21 | |
| *** yufei has joined #openstack-ironic | 08:21 | |
| tuanluong | morning Ironic Team | 08:22 |
| tuanluong | morning pas-ha, vdrok xiaoyang | 08:22 |
| *** ricardoas has quit IRC | 08:23 | |
| *** ricardoas has joined #openstack-ironic | 08:25 | |
| *** zzzeek_ has quit IRC | 08:25 | |
| *** zackf has quit IRC | 08:25 | |
| *** hoonetorg has joined #openstack-ironic | 08:28 | |
| *** afazekas has quit IRC | 08:29 | |
| *** alexpilo_ has quit IRC | 08:32 | |
| *** zzzeek has joined #openstack-ironic | 08:33 | |
| *** alexpilotti has joined #openstack-ironic | 08:33 | |
| *** hamzy has quit IRC | 08:33 | |
| *** afazekas has joined #openstack-ironic | 08:34 | |
| *** hamzy has joined #openstack-ironic | 08:34 | |
| *** alexpilotti has quit IRC | 08:35 | |
| *** hoonetorg has quit IRC | 08:36 | |
| *** jistr is now known as jistr|mtgs | 08:36 | |
| *** alexpilotti has joined #openstack-ironic | 08:38 | |
| vdrok | morning tuanluong ! | 08:40 |
| tuanluong | hello vdrob | 08:41 |
| tuanluong | Do you know cloudbase-init? | 08:41 |
| vdrok | tuanluong: you mean cloud-init? | 08:42 |
| *** baoli has joined #openstack-ironic | 08:43 | |
| *** alexpilotti has quit IRC | 08:43 | |
| *** amoralej|off is now known as amoralej | 08:43 | |
| tuanluong | yes, vdrok cloud-init using for linux | 08:43 |
| tuanluong | and cloudbase-init using for window | 08:43 |
| vdrok | tuanluong: ah, gotcha. never used cloudbase-init :) | 08:44 |
| tuanluong | vdrok, does cloud-init support config window? | 08:45 |
| vdrok | tuanluong: I doubt that https://cloudinit.readthedocs.io/en/latest/topics/availability.html | 08:47 |
| *** baoli has quit IRC | 08:47 | |
| *** ralonsoh has joined #openstack-ironic | 08:48 | |
| tuanluong | thanks vdrok | 08:48 |
| tuanluong | Do you know any information to demonstrate cloud-init in Baremetal? | 08:49 |
| *** hoonetorg has joined #openstack-ironic | 08:49 | |
| *** slagle has quit IRC | 08:50 | |
| *** lhinds has quit IRC | 08:50 | |
| *** moshele has quit IRC | 08:50 | |
| *** moshele has joined #openstack-ironic | 08:51 | |
| *** slagle has joined #openstack-ironic | 08:51 | |
| *** moshele has quit IRC | 08:54 | |
| vdrok | tuanluong: hrm, you just do "nova boot ... --config-drive True", it gets written to disk with "config-2" label and then read by cloud init. but it's generated by nova completely. if you want to put there some custom stuff, you can create it manually in iso9660 format, gzip it, convert to base 64 and pass it along via "ironic node-set-provision-state active" | 08:54 |
| vdrok | (you'll have to create neutron ports manually too in this case) | 08:54 |
| *** lhinds has joined #openstack-ironic | 08:55 | |
| vdrok | tuanluong: to generate it manually, you can just copy-paste the needed bits of generation logic from nova | 08:55 |
| vdrok | tuanluong: most of generation logic is here - https://github.com/openstack/nova/blob/5158ca7dcffd583dc8dd7c29b0a9a595d7de7dcf/nova/virt/configdrive.py and you can see the directory structure/needed files in cloud-init docs | 08:59 |
| vdrok | and it's better to use iso9660 as vfat is usually much bigger and sometimes can not fit in instance_info in our database | 08:59 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** lujinluo has quit IRC | 09:00 | |
| *** gcb has joined #openstack-ironic | 09:01 | |
| *** lujinluo has joined #openstack-ironic | 09:01 | |
| *** magic has joined #openstack-ironic | 09:01 | |
| *** galyna has quit IRC | 09:02 | |
| tuanluong | thank vdrok, I look complex. I will try it first. I also find some relate information on the internet here https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/5/html/End_User_Guide/config-drive.html | 09:03 |
| *** galyna has joined #openstack-ironic | 09:03 | |
| *** xiaoyang has quit IRC | 09:05 | |
| *** xiaoyang has joined #openstack-ironic | 09:06 | |
| *** moshele has joined #openstack-ironic | 09:06 | |
| *** aparnav has joined #openstack-ironic | 09:06 | |
| *** magic has quit IRC | 09:06 | |
| *** tuanluong has quit IRC | 09:07 | |
| *** tuanluong has joined #openstack-ironic | 09:07 | |
| *** yuanying has joined #openstack-ironic | 09:08 | |
| *** e0ne has joined #openstack-ironic | 09:11 | |
| *** lujinluo has quit IRC | 09:15 | |
| *** hoonetorg has quit IRC | 09:16 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 09:18 |
| *** athomas has joined #openstack-ironic | 09:19 | |
| *** lucas-afk is now known as lucasagomes | 09:21 | |
| lucasagomes | morning all | 09:21 |
| vdrok | mornin lucasagomes | 09:21 |
| *** baoli has joined #openstack-ironic | 09:28 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add ironic resources CRUD notifications https://review.openstack.org/356541 | 09:29 |
| *** Romanenko_K has joined #openstack-ironic | 09:29 | |
| moshele | aarefiev: morning can you review https://review.openstack.org/#/c/264257/ | 09:32 |
| patchbot | patch 264257 - ironic-inspector - Adding InfiniBand Support | 09:32 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node maintenance notifications https://review.openstack.org/396239 | 09:33 |
| *** dtantsur|afk is now known as dtantsur | 09:34 | |
| dtantsur | Morning Ironic | 09:34 |
| *** baoli has quit IRC | 09:34 | |
| vdrok | morning dtantsur | 09:35 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node console notifications https://review.openstack.org/397812 | 09:37 |
| *** fragatina has joined #openstack-ironic | 09:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 09:39 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/406850 | 09:39 |
| *** misty-cloud has joined #openstack-ironic | 09:41 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Serve boot configs from Ironic API https://review.openstack.org/369438 | 09:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add RPC API to get the boot config https://review.openstack.org/394399 | 09:41 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 09:42 |
| lucasagomes | dtantsur, morning | 09:42 |
| dtantsur | o/ | 09:44 |
| *** misty-cloud- has quit IRC | 09:45 | |
| *** SerenaFeng has quit IRC | 09:51 | |
| *** pmannidi has quit IRC | 09:59 | |
| openstackgerrit | Merged openstack/ironic: Remove agent vendor passthru from most drivers https://review.openstack.org/397844 | 10:01 |
| *** abalfour has quit IRC | 10:07 | |
| *** abalfour has joined #openstack-ironic | 10:07 | |
| *** priteau has joined #openstack-ironic | 10:08 | |
| *** sambetts|pto is now known as sambetts | 10:08 | |
| sambetts | Morning all | 10:10 |
| *** derekh has joined #openstack-ironic | 10:12 | |
| vsaienk0 | morning sambetts, datantsur, lucasagomes, aarefiev, vdrok, milan, mgould and all Ironic'ers! | 10:12 |
| sambetts | hey vsaienk0 | 10:12 |
| dtantsur | morning sambetts, vsaienk0, mgould|afk | 10:13 |
| *** ccamacho has quit IRC | 10:13 | |
| *** SerenaFeng has joined #openstack-ironic | 10:13 | |
| moshele | dtantsur: morning, can you review https://review.openstack.org/#/c/264257/ | 10:14 |
| patchbot | patch 264257 - ironic-inspector - Adding InfiniBand Support | 10:14 |
| dtantsur | morning, I will try to get to it, but we have many more pressing priorities now | 10:15 |
| dtantsur | moshele, well, I'm close to -2 the patch because of https://review.openstack.org/#/c/264257/36/doc/source/usage.rst to be honest.. | 10:15 |
| patchbot | patch 264257 - ironic-inspector - Adding InfiniBand Support | 10:15 |
| dtantsur | this cannot be called "supports infiniband", this should be called "can be hacked around to more or less work with inifiniband" | 10:16 |
| dtantsur | you probably don't want me to review it :) though I remember milan had similar concerns | 10:16 |
| dtantsur | sambetts, are you ok with the very approach taken in ^^^? | 10:17 |
| sambetts | hmm, can't we blacklist on the EthOverInfiniband MAC?? | 10:17 |
| sambetts | moshele: ^ | 10:18 |
| *** fragatina has quit IRC | 10:20 | |
| moshele | sambetts:it was that easy becasue the mac is not match the "Infiniband mac", let me talk to our infiniband architect with this part | 10:23 |
| moshele | sambetts: it was not easy because the mac is not match the "Infiniband mac", let me talk to our infiniband architect with this part | 10:24 |
| sergek | morning Ironicers! | 10:25 |
| lucasagomes | sergek, sambetts moshele vsaienk0 morning | 10:26 |
| *** hoangcx has quit IRC | 10:26 | |
| moshele | lucasagomes: morning | 10:27 |
| *** slagle has quit IRC | 10:28 | |
| *** zackf has joined #openstack-ironic | 10:29 | |
| *** slagle has joined #openstack-ironic | 10:29 | |
| vdrok | morning sambetts vsaienk0 and sergek | 10:30 |
| *** ccamacho has joined #openstack-ironic | 10:30 | |
| *** baoli has joined #openstack-ironic | 10:31 | |
| *** tuanluong has quit IRC | 10:32 | |
| *** marios has quit IRC | 10:33 | |
| *** tuanluong has joined #openstack-ironic | 10:33 | |
| *** zackf has quit IRC | 10:33 | |
| *** marios has joined #openstack-ironic | 10:34 | |
| *** baoli has quit IRC | 10:35 | |
| *** SerenaFeng has quit IRC | 10:36 | |
| *** sborkows has joined #openstack-ironic | 10:37 | |
| sborkows | morning, all | 10:38 |
| *** marios has quit IRC | 10:38 | |
| *** alexpilotti has joined #openstack-ironic | 10:40 | |
| *** marios has joined #openstack-ironic | 10:40 | |
| *** zhangjl1 has quit IRC | 10:41 | |
| openstackgerrit | Merged openstack/ironic: Turn off tempest's multitenant network tests https://review.openstack.org/397767 | 10:41 |
| sborkows | JayF, I have a question about your review on my patch https://review.openstack.org/#/c/391836 : What do you mean with 'hook' word? | 10:44 |
| patchbot | patch 391836 - ironic-python-agent - Add a new Hardware Manager for CNA network card | 10:44 |
| *** alexpilotti has quit IRC | 10:44 | |
| *** shuquan_ has quit IRC | 10:44 | |
| *** yufei has quit IRC | 10:47 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 10:48 |
| *** fxpester has joined #openstack-ironic | 10:57 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 10:58 |
| *** tesseract has joined #openstack-ironic | 11:05 | |
| *** tesseract is now known as Guest17259 | 11:05 | |
| vdrok | morning sborkows | 11:09 |
| openstackgerrit | Merged openstack/ironic: Fix import method to follow community guideline https://review.openstack.org/405730 | 11:09 |
| *** daemontool has joined #openstack-ironic | 11:11 | |
| openstackgerrit | Merged openstack/ironic: Auto enable the deploy driver https://review.openstack.org/406381 | 11:20 |
| *** zackf has joined #openstack-ironic | 11:21 | |
| *** Guest17259 has quit IRC | 11:21 | |
| *** mgould|afk is now known as mgould | 11:23 | |
| mgould | morning Ironic! | 11:23 |
| *** tesseract- has joined #openstack-ironic | 11:24 | |
| *** tuanluong has quit IRC | 11:24 | |
| openstackgerrit | paresh sao proposed openstack/proliantutils: Support to disable iscsi boot option in iLO https://review.openstack.org/406706 | 11:25 |
| *** zackf has quit IRC | 11:25 | |
| openstackgerrit | Grzegorz Grasza (xek) proposed openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 11:29 |
| xek | morning! | 11:30 |
| mgould | morning xek | 11:30 |
| vdrok | good morning mgould and xek | 11:31 |
| *** baoli has joined #openstack-ironic | 11:31 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 11:36 |
| *** baoli has quit IRC | 11:36 | |
| vsaienk0 | dtantsur, lucasagomes: where do you think we should keep driver devstack spefic things like vbmc and vpdu? It is possible to keep them in emulators repo like regular devstack plugin and enable in ironic when needed. It will allow do not overload ironic repo and do changes without involving ironic community. wdyt? There is an example of vpdu https://review.openstack.org/#/c/388154/ | 11:40 |
| patchbot | patch 388154 - ironic - Add virtualpdu to ironic devstack plugin | 11:40 |
| *** alexpilotti has joined #openstack-ironic | 11:41 | |
| dtantsur | hmm, I haven't really put much thought in it | 11:42 |
| dtantsur | I don't quite like how we enroll nodes at all.. but maybe we should think about proper refactoring later. I'm not sure I want to delay un-deprecating snmp | 11:42 |
| *** Daviey has quit IRC | 11:43 | |
| *** Daviey has joined #openstack-ironic | 11:43 | |
| *** alexpilotti has quit IRC | 11:46 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 11:46 |
| lucasagomes | vsaienk0, hmm I'm not familiar with the emulators repo but, it does sounds like we should do it | 11:47 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc: Show real error when it fails to create a new VBMC https://review.openstack.org/406903 | 11:49 |
| lucasagomes | tiny ^ | 11:49 |
| dtantsur | +2 | 11:50 |
| *** jkilpatr has quit IRC | 11:54 | |
| mgould | +1 | 11:55 |
| mgould | -6 chars, the best kind of patch :-) | 11:56 |
| dtantsur | yep :) | 11:57 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: 'updated_at' field value after node is updated https://review.openstack.org/371605 | 11:58 |
| vsaienk0 | lucasagomes: emulartos repo is virtualbmc or virtualpdu for example: https://github.com/openstack/virtualbmc | 12:00 |
| mgould | dtantsur: -6 chars *and adds functionality* | 12:04 |
| lucasagomes | vsaienk0, gotcha what you mean now. Yeah it sounds more organized to keep the devstack installation/configuration of vbmc/vpdu in their own repo | 12:06 |
| *** yee379 has joined #openstack-ironic | 12:10 | |
| *** yee3799 has quit IRC | 12:11 | |
| *** derekh has quit IRC | 12:11 | |
| *** jkilpatr has joined #openstack-ironic | 12:15 | |
| *** derekh has joined #openstack-ironic | 12:16 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/405812 | 12:17 |
| openstackgerrit | Merged openstack/virtualbmc: Show real error when it fails to create a new VBMC https://review.openstack.org/406903 | 12:17 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc: Fix Python3 test errors https://review.openstack.org/406913 | 12:23 |
| *** jpich has joined #openstack-ironic | 12:24 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: Allow to attach/deatach vif to portgroup https://review.openstack.org/406915 | 12:24 |
| *** Nisha_Agarwal has quit IRC | 12:30 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/virtualbmc: Add support for a "global" configuration file https://review.openstack.org/406916 | 12:31 |
| charlledaniel | morning all | 12:32 |
| *** baoli has joined #openstack-ironic | 12:32 | |
| *** e0ne has quit IRC | 12:33 | |
| *** lucasagomes is now known as lucas-hungry | 12:33 | |
| *** vikrant|lunch has quit IRC | 12:33 | |
| *** e0ne has joined #openstack-ironic | 12:34 | |
| *** gcb has quit IRC | 12:34 | |
| *** baoli has quit IRC | 12:37 | |
| *** alexpilotti has joined #openstack-ironic | 12:42 | |
| *** sborkows has quit IRC | 12:43 | |
| nicodemos | good morning, ironic | 12:46 |
| *** alexpilotti has quit IRC | 12:47 | |
| *** misty-cloud has quit IRC | 12:48 | |
| *** hw_wutianwei has quit IRC | 12:50 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 12:59 |
| *** srobert has joined #openstack-ironic | 12:59 | |
| *** srobert has quit IRC | 13:00 | |
| *** srobert has joined #openstack-ironic | 13:00 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: [WIP] ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 13:03 |
| *** trown|outtypewww is now known as trown | 13:09 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: 'updated_at' field value after node is updated https://review.openstack.org/371605 | 13:09 |
| *** jcoufal has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 13:12 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Catch unknown exceptions in validate driver ifaces https://review.openstack.org/406157 | 13:14 |
| *** mjura has quit IRC | 13:24 | |
| *** rbudden has joined #openstack-ironic | 13:26 | |
| *** amoralej is now known as amoralej|lunch | 13:26 | |
| *** tesseract- has quit IRC | 13:27 | |
| *** aparnav has quit IRC | 13:29 | |
| *** baoli has joined #openstack-ironic | 13:33 | |
| *** clenimar has joined #openstack-ironic | 13:35 | |
| *** baoli has quit IRC | 13:38 | |
| *** jheroux has joined #openstack-ironic | 13:40 | |
| *** shuquan has joined #openstack-ironic | 13:41 | |
| *** shuquan has quit IRC | 13:46 | |
| *** lucas-hungry is now known as lucasagomes | 13:47 | |
| *** amoralej|lunch is now known as amoralej | 14:04 | |
| *** shuquan has joined #openstack-ironic | 14:04 | |
| vdrok | morning charlledaniel and nicodemos | 14:06 |
| *** [1]cdearborn has joined #openstack-ironic | 14:10 | |
| *** glonlas has joined #openstack-ironic | 14:10 | |
| *** rloo has joined #openstack-ironic | 14:13 | |
| *** mjura has joined #openstack-ironic | 14:15 | |
| *** ppiela has joined #openstack-ironic | 14:18 | |
| *** fragatina has joined #openstack-ironic | 14:21 | |
| *** mjura has quit IRC | 14:22 | |
| *** zackf has joined #openstack-ironic | 14:28 | |
| *** srobert has quit IRC | 14:33 | |
| *** milan has joined #openstack-ironic | 14:33 | |
| milan | morning Ironic! :) | 14:34 |
| vdrok | a bit late morning milan ! | 14:34 |
| milan | vdrok, a bit indeed! :) | 14:34 |
| *** krtaylor has joined #openstack-ironic | 14:35 | |
| * milan had couple of errands to resolve | 14:35 | |
| *** mjura has joined #openstack-ironic | 14:35 | |
| jroll | morning y'all :) | 14:38 |
| jroll | question for the CI experts - is it expected that we don't run tempest on the new side of grenade? | 14:39 |
| dtantsur | morning jroll. we run tempest on both sides. | 14:40 |
| milan | morning jroll! | 14:40 |
| jroll | dtantsur: I'm not seeing it here: http://logs.openstack.org/12/405812/8/check/gate-grenade-dsvm-ironic/dfffde6/logs/grenade.sh.txt.gz | 14:41 |
| jroll | (this is a requirements bot change that already merged | 14:41 |
| dtantsur | jroll, it's in console logs | 14:41 |
| jroll | aha | 14:41 |
| jroll | strange | 14:42 |
| dtantsur | jroll, http://logs.openstack.org/12/405812/8/check/gate-grenade-dsvm-ironic/dfffde6/console.html#_2016-12-05_11_08_37_003564 | 14:42 |
| jroll | yep I see it now | 14:42 |
| jroll | thank you | 14:42 |
| dtantsur | obvious grenade is very obvious \o/ | 14:42 |
| jroll | :P | 14:42 |
| milan | yeah, why rotate the screen log to a file w/ a different name ;) | 14:42 |
| * dtantsur -> short break | 14:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 14:43 |
| milan | morning aarefiev, sambetts, guys, latest states patch revision waiting for your valuable input here: https://review.openstack.org/#/c/348943/33 | 14:47 |
| patchbot | patch 348943 - ironic-inspector - Introducing node introspection state management | 14:47 |
| jroll | aarefiev: yep, you were right about tempest config :( | 14:47 |
| milan | thanks to aarefiev and mgould for recent reviews! | 14:47 |
| *** watanabe_isao has joined #openstack-ironic | 14:49 | |
| milan | ofc mgould, your review is valuable too :D /me just read last two lines together, sorry O:-) | 14:51 |
| *** watanabe_isao has quit IRC | 14:52 | |
| *** snecklifter has joined #openstack-ironic | 14:58 | |
| *** glonlas_ has joined #openstack-ironic | 14:58 | |
| vdrok | morning jroll | 14:58 |
| jroll | \o | 14:58 |
| vdrok | jroll: that's because we tell grenade to do it this way. there is some parameter, like run tempest on target. we have it false iirc and instead just run smoke after grenade finishes. that's why it's in console :) | 14:59 |
| jroll | vdrok: yeah, makes sense after dmitry explained, thanks :) | 15:00 |
| *** glonlas has quit IRC | 15:02 | |
| *** glonlas_ has quit IRC | 15:02 | |
| *** glonlas has joined #openstack-ironic | 15:02 | |
| *** glonlas has quit IRC | 15:03 | |
| *** glonlas has joined #openstack-ironic | 15:03 | |
| *** Goneri has joined #openstack-ironic | 15:05 | |
| *** Syed__ has joined #openstack-ironic | 15:07 | |
| *** daemontool has quit IRC | 15:08 | |
| *** fragatina has quit IRC | 15:08 | |
| *** fragatina has joined #openstack-ironic | 15:08 | |
| *** baoli has joined #openstack-ironic | 15:09 | |
| *** srobert has joined #openstack-ironic | 15:10 | |
| *** gcb has joined #openstack-ironic | 15:11 | |
| TheJulia | good morning everyone | 15:12 |
| nicodemos | morning TheJulia | 15:13 |
| vdrok | TheJulia: good morning | 15:13 |
| *** shuquan has quit IRC | 15:15 | |
| *** srobert has quit IRC | 15:16 | |
| openstackgerrit | Miles Gould proposed openstack/ironic: Add release names & numbers to API version history https://review.openstack.org/404211 | 15:18 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Fix typo in local inpectorclient install https://review.openstack.org/407029 | 15:21 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Allow installing Ansible from PyPI https://review.openstack.org/406321 | 15:22 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Ensure git_root folder exists https://review.openstack.org/407031 | 15:22 |
| milan | morning nicodemos, TheJulia! :) | 15:22 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: DRY when installing components https://review.openstack.org/407034 | 15:25 |
| *** hshiina has joined #openstack-ironic | 15:25 | |
| *** another_larsks is now known as larsks | 15:26 | |
| * TheJulia blinks | 15:28 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: devstack: Set [compute]/fixed_network_name https://review.openstack.org/406376 | 15:33 |
| openstackgerrit | Miles Gould proposed openstack/ironic: Add release names & numbers to API version history https://review.openstack.org/404211 | 15:35 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Fix typo in local inspectorclient install https://review.openstack.org/407029 | 15:36 |
| *** fragatina has quit IRC | 15:38 | |
| jlvillal | vsaienk0: My patch ^^^ I think you would be interested. | 15:40 |
| jlvillal | vsaienk0: Also: https://review.openstack.org/406214 Not sure what you think about breaking the multi-node job? Basically that is a patch to have the multi-node job run tempest "smoke" | 15:41 |
| patchbot | patch 406214 - openstack-infra/project-config - ironic: Have multinode job not use DEVSTACK_GATE_T... | 15:41 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic: 'updated_at' field value after node is updated https://review.openstack.org/371605 | 15:41 |
| *** fragatina has joined #openstack-ironic | 15:42 | |
| vsaienk0 | jlvillal: I'm fully support this, basically I'm on it https://review.openstack.org/#/c/369021/ just want to make sure smoke tests are passed before switching it in project-config | 15:43 |
| patchbot | patch 369021 - ironic - WIP: Ironic multitest initial commit | 15:43 |
| TheJulia | milan: thank you :) | 15:43 |
| jlvillal | vsaienk0: Oh cool. I guess we are working parallel :) | 15:43 |
| vsaienk0 | jlvillal correct link to the patch https://review.openstack.org/#/c/368173/ | 15:43 |
| patchbot | patch 368173 - ironic - Testing multinode stuff | 15:43 |
| milan | TheJulia, I'm wondering.... maybe not me? :) | 15:45 |
| *** Sukhdev has joined #openstack-ironic | 15:47 | |
| NobodyCam | Good Morning Ironic'ers | 15:48 |
| vsaienk0 | morning NobodyCam | 15:49 |
| NobodyCam | Morning vsaienk0 :) | 15:50 |
| jlvillal | Good morning NobodyCam milan vsaienk0 TheJulia vdrok dtantsur jroll rloo lucasagomes mariojv JayF and anyone else I missed. | 15:50 |
| lucasagomes | jlvillal, hi there | 15:50 |
| vdrok | morning NobodyCam and jlvillal | 15:50 |
| jroll | morning, everyone I haven't said morning too yet :) | 15:50 |
| milan | morning jlvillal NobodyCam! :) | 15:51 |
| NobodyCam | morning jlvillal lucasagomes vdrok jroll milan :) | 15:51 |
| lucasagomes | o/ | 15:51 |
| TheJulia | milan: err, mgould, I've not had enough coffee this morning | 15:52 |
| jlvillal | :) | 15:52 |
| milan | TheJulia, never mind, same here :D | 15:52 |
| jroll | http://poorlydrawnlines.com/comic/mondays/ | 15:52 |
| milan | :D | 15:53 |
| *** glonlas_ has joined #openstack-ironic | 15:53 | |
| *** harlowja_at_home has joined #openstack-ironic | 15:54 | |
| *** srobert has joined #openstack-ironic | 15:55 | |
| *** glonlas_ has quit IRC | 15:56 | |
| *** glonlas_ has joined #openstack-ironic | 15:56 | |
| *** glonlas has quit IRC | 15:57 | |
| rloo | hello and good morning jlvillal, NobodyCam, vdrok, lucasagomes, jroll, milan, TheJulia, vsaienk0 and everyone else! | 15:57 |
| lucasagomes | o/ | 15:57 |
| NobodyCam | morning rloo :) | 15:58 |
| milan | morning rloo! :) | 15:58 |
| NobodyCam | morning TheJulia :) | 15:58 |
| TheJulia | o/ | 15:59 |
| NobodyCam | :>) | 15:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 15:59 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Configure tempest to use multitenancy tests https://review.openstack.org/407047 | 15:59 |
| *** ohamada has quit IRC | 16:00 | |
| *** lindycoder has joined #openstack-ironic | 16:01 | |
| vsaienk0 | good morning rloo! | 16:01 |
| vsaienk0 | rloo, jroll: I'm wondered if we should add portgroup support to both ironicclient and osc plugin. In Neutron when they added trunks support it was implemented only in OSC since neutronclient is going to be deprecated. | 16:02 |
| dtantsur | morning jlvillal, TheJulia | 16:02 |
| rloo | vsaienk0: we are going to add portgroup support, aren't we? Are you asking to add it soon? | 16:03 |
| jroll | vsaienk0: we haven't made a plan to deprecate ironic CLI yet, so I would vote both | 16:03 |
| rloo | vsaienk0, jroll: oh, i misunderstood. yes, until we deprecate ironic CLI, we need to support both. | 16:04 |
| vsaienk0 | jroll, that is fear, if ironicclient is not deprecated we should add to both. | 16:04 |
| *** fxpester has quit IRC | 16:05 | |
| mariojv | hi jlvillal and everyone | 16:05 |
| *** rcernin has quit IRC | 16:05 | |
| *** srobert has quit IRC | 16:05 | |
| jcook | mariojv JayF for the updates to https://review.openstack.org/#/c/171878/ to I make them to them to the patchset here or as a new patchset? | 16:06 |
| patchbot | patch 171878 - ironic-specs - Implement rescue mode (MERGED) | 16:06 |
| *** mjura has quit IRC | 16:06 | |
| *** srobert has joined #openstack-ironic | 16:06 | |
| mariojv | jcook: new patchset | 16:06 |
| jcook | ack | 16:06 |
| mariojv | jcook: same file though | 16:06 |
| milan | morning mariojv! :) | 16:06 |
| mariojv | hi milan | 16:07 |
| jroll | jcook: mariojv: word nit, new patch. a patchset is an update to the same review :) | 16:07 |
| mariojv | ++ | 16:07 |
| jcook | isn't a patchset a set of patches ? :D | 16:08 |
| *** jjohnson2 has quit IRC | 16:08 | |
| jcook | ack, thanks | 16:09 |
| xhku | Good morning everyone ! I would need some review on the change testing the snmp driver, Thanks in advance ! https://review.openstack.org/#/c/388154/ | 16:09 |
| patchbot | patch 388154 - ironic - Add virtualpdu to ironic devstack plugin | 16:09 |
| *** penick has quit IRC | 16:10 | |
| xhku | ^ The experimental check gate-tempest-dsvm-ironic-ipa-partition-pxe_snmp-tinyipa-ubuntu-xenial-nv was added to show the driver works | 16:11 |
| vdrok | morning rloo mariojv and xhku ! | 16:11 |
| xhku | o/ | 16:12 |
| mariojv | hi vdrok | 16:12 |
| *** penick has joined #openstack-ironic | 16:12 | |
| *** rpioso has joined #openstack-ironic | 16:13 | |
| *** jpich has quit IRC | 16:14 | |
| *** fragatina has quit IRC | 16:14 | |
| *** athomas has quit IRC | 16:16 | |
| *** priteau has quit IRC | 16:16 | |
| *** moshele has quit IRC | 16:16 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:17 | |
| *** soliosg has joined #openstack-ironic | 16:19 | |
| *** penick has quit IRC | 16:20 | |
| mariojv | i'm +1 on rolling upgrades spec - here's a link if anyone wants to take a look before the meeting: https://review.openstack.org/#/c/299245 | 16:20 |
| patchbot | patch 299245 - ironic-specs - Support rolling upgrades | 16:20 |
| *** hshiina has quit IRC | 16:24 | |
| *** causten has joined #openstack-ironic | 16:26 | |
| *** bdeschenes has joined #openstack-ironic | 16:26 | |
| *** dtantsur is now known as dtantsur|brb | 16:29 | |
| *** rama_y has joined #openstack-ironic | 16:29 | |
| *** yuanying has quit IRC | 16:29 | |
| mariojv | this also has +2s from TheJulia and vdrok, as well as my +1, if someone would like to +W: https://review.openstack.org/#/c/396239/ | 16:29 |
| patchbot | patch 396239 - ironic - Add node maintenance notifications | 16:29 |
| vdrok | mariojv: we need to +W CRUD notifications first :) | 16:30 |
| mariojv | oops, did not notice that. thanks | 16:30 |
| mariojv | i'll re-review, it had a +1 from me a few patch sets ago | 16:31 |
| *** rbartal has quit IRC | 16:33 | |
| *** alex_xu has quit IRC | 16:35 | |
| *** bdeschenes has quit IRC | 16:35 | |
| *** vgadiraj has joined #openstack-ironic | 16:36 | |
| mariojv | CRUD now has vdrok's +2 and my +1: https://review.openstack.org/#/c/356541/ | 16:40 |
| patchbot | patch 356541 - ironic - Add ironic resources CRUD notifications | 16:40 |
| *** harlowja_at_home has quit IRC | 16:42 | |
| *** galyna_ has joined #openstack-ironic | 16:43 | |
| *** gcb has quit IRC | 16:43 | |
| *** soliosg has quit IRC | 16:44 | |
| *** soliosg has joined #openstack-ironic | 16:44 | |
| rloo | jroll: fyi, i just updated the meeting agenda | 16:45 |
| jroll | rloo: cool, thanks | 16:46 |
| sambetts | TheJulia: qq about bifrost, I'm seeing an issue where the ironic-inspecotr service file is missing, any idea why this might be happening? inspector is installed and configured correctly as far as I can see | 16:51 |
| *** causten has quit IRC | 16:51 | |
| TheJulia | sambetts: define service file in your context | 16:51 |
| sambetts | ironic-inspector.service | 16:52 |
| *** causten has joined #openstack-ironic | 16:52 | |
| TheJulia | as in systemd service file? | 16:52 |
| sambetts | yeah, so its trying to start inspector and failing | 16:52 |
| TheJulia | That is interesting, What distro are you using? | 16:53 |
| sambetts | Ubuntu | 16:53 |
| TheJulia | even more interesting... | 16:53 |
| TheJulia | what was your install arguments or defaults that you changed for your install? | 16:53 |
| jcook | mariojv JayF does the node go from ACTIVE to RESCUING in Nova driver, Ironic API, Ironic conductor, earlier in Nova? | 16:55 |
| mariojv | i don't think the nova code is written yet jcook | 16:56 |
| jcook | right | 16:56 |
| sambetts | TheJulia: bradjones, might be able to give more info, he's just pull the info up | 16:56 |
| jcook | I'm asking where it will happen. Like ACTIVE -> Cleaning | 16:56 |
| mariojv | under the "standard rescue process" section, nova goes to RESCUE before ironic api call is made | 16:56 |
| *** causten has quit IRC | 16:56 | |
| jcook | mariojv do you mean RESCUING? | 16:56 |
| mariojv | so nova RESCUE -> ironic RESCUING -> RESCUEWAIT -> RESCUING -> RESCUE | 16:56 |
| mariojv | spec says RESCUE | 16:57 |
| mariojv | i think nova might go RESCUE -> RESCUED but i'm not sure | 16:57 |
| bradjones | TheJulia, sambetts: everything default the only thing we changed was the networking_interface | 16:57 |
| jcook | mariojv that's the target state | 16:57 |
| jcook | RESCUE | 16:57 |
| mariojv | cool | 16:57 |
| jcook | it goes into a transitional state RESCUIGN | 16:57 |
| TheJulia | bradjones: when was the install performed? | 16:58 |
| jcook | I'm curious when that happens (the state change from ACTIVE -> RESCUING) | 16:58 |
| *** soliosg has quit IRC | 16:58 | |
| mariojv | rescuing for nova should probably be documented here http://docs.openstack.org/developer/nova/vmstates.html | 16:58 |
| bradjones | TheJulia: about 45mins ago | 16:58 |
| *** aNuposic has joined #openstack-ironic | 16:58 | |
| mariojv | jcook: in nova, that should happen before ironic gets the api call | 16:58 |
| TheJulia | bradjones: okay, looking | 16:58 |
| bradjones | TheJulia: thanks | 16:58 |
| jcook | mariojv so outside the driver? | 16:58 |
| mariojv | jcook: hmm, good question | 16:59 |
| mariojv | jcook: maybe whatever calls the driver handles it already | 16:59 |
| jcook | Maybe in shared code for all | 16:59 |
| mariojv | jcook: i haven't read the shared code though | 16:59 |
| mariojv | jcook: maybe looking at another driver's rescue implementation would shed some light on that | 16:59 |
| sambetts | TheJulia: we're trying to set things up with a real testbed, is there anything we need to take into account when using real hardware vs a simulated env, I noticed the default net interface was the libvirt bridge | 16:59 |
| jcook | yeah | 16:59 |
| mariojv | jcook: we can chat more after ironic meeting if you want | 17:00 |
| jcook | mariojv sure | 17:00 |
| mariojv | there is one today, right? | 17:00 |
| *** galyna_ has quit IRC | 17:00 | |
| vgadiraj | what channel is the ironic meeting in? openstack-meeting-3? | 17:00 |
| *** galyna_ has joined #openstack-ironic | 17:01 | |
| vgadiraj | nvm, it is :) | 17:01 |
| *** rajinir has joined #openstack-ironic | 17:01 | |
| jlvillal | Hi galyna :) I'll say it here instead of the meeting channel. | 17:01 |
| TheJulia | bradjones: interesting, could you humor me and try reinstalling with -e enable_inspector=true on the ansible-playbook command line? | 17:01 |
| galyna_ | :) sorry if I wrote sth extra there | 17:02 |
| *** Romanenko_K has quit IRC | 17:03 | |
| *** soliosg has joined #openstack-ironic | 17:04 | |
| *** galyna_ has left #openstack-ironic | 17:04 | |
| JayF | jcook: Nova calls Ironic's API, set provision state rescue (nova state -> rescusing). nova polls ironic waiting for a state of REScUE (nova state -> RESCUED) | 17:05 |
| bradjones | TheJulia: sure trying that now | 17:05 |
| *** Romanenko_K has joined #openstack-ironic | 17:06 | |
| *** d0ugal has quit IRC | 17:07 | |
| *** causten has joined #openstack-ironic | 17:07 | |
| *** soliosg has quit IRC | 17:07 | |
| *** soliosg has joined #openstack-ironic | 17:08 | |
| TheJulia | bradjones: We landed a change to the default, I'm wondering if something is up since bootstrapping is where the service file is placed, installation is separate from that. | 17:08 |
| bradjones | TheJulia: nope, same thing "ironic-inspector.service could not be found" | 17:10 |
| TheJulia | and definitely not present in the appropriate systemd location? | 17:11 |
| *** e0ne has quit IRC | 17:12 | |
| *** dtantsur|brb is now known as dtantsur | 17:12 | |
| *** causten has quit IRC | 17:14 | |
| bradjones | TheJulia: no I can't find any systemd files for ironic-inspector at all | 17:14 |
| *** aparnav has joined #openstack-ironic | 17:16 | |
| TheJulia | :( | 17:21 |
| bradjones | TheJulia: do you have an example of how the working service file looks, I can try installing the ironic inspector one manually then see if it creates the others ok | 17:22 |
| *** Guest51737 is now known as mgagne | 17:26 | |
| *** mgagne has quit IRC | 17:27 | |
| *** mgagne has joined #openstack-ironic | 17:27 | |
| TheJulia | bradjones: firing up my VM now | 17:27 |
| lucasagomes | bradjones, http://paste.openstack.org/show/591455/ < a working example | 17:28 |
| lucasagomes | TheJulia, ^ | 17:28 |
| TheJulia | lucasagomes: thank you | 17:28 |
| * TheJulia goes back to the meeting | 17:28 | |
| bradjones | Thanks :) | 17:28 |
| *** rloo has quit IRC | 17:29 | |
| *** rloo has joined #openstack-ironic | 17:29 | |
| *** glonlas_ has quit IRC | 17:31 | |
| *** Romanenko_K has quit IRC | 17:33 | |
| *** baoli has quit IRC | 17:33 | |
| *** baoli has joined #openstack-ironic | 17:34 | |
| *** fragatina has joined #openstack-ironic | 17:36 | |
| *** yuanying has joined #openstack-ironic | 17:38 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix keystone config template https://review.openstack.org/407105 | 17:40 |
| bradjones | TheJulia: so manually installing the ironic-inspector service works and now it failes on the ironic-conductor service, so I guess there is a missing call somewhere to run the bootstrap steps where it installs the services | 17:42 |
| TheJulia | how did you run the install process? | 17:44 |
| bradjones | what for manually getting the ironic-inspector service working? | 17:45 |
| *** fragatina has quit IRC | 17:45 | |
| bradjones | for running the playbooks I did 'ansible-playbook -K -vvvv -e enable_inspector=true -i inventory/localhost install.yaml' | 17:47 |
| *** shuquan has joined #openstack-ironic | 17:47 | |
| *** penick has joined #openstack-ironic | 17:48 | |
| TheJulia | hmmmmmm | 17:48 |
| milan | aarefiev, dtantsur, I've closed: https://bugs.launchpad.net/ironic-inspector/+bug/1646430 | 17:49 |
| openstack | Launchpad bug 1646430 in Ironic Inspector "Plugin attempted to be enabled twice: ironic" [Critical,Fix released] | 17:49 |
| milan | feel free to change status if inappropriate | 17:49 |
| *** harlowja has joined #openstack-ironic | 17:50 | |
| dtantsur | I think it's fixed. thanks | 17:50 |
| *** aparnav has quit IRC | 17:51 | |
| *** shuquan has quit IRC | 17:52 | |
| *** Guest61168 is now known as rook | 17:57 | |
| lucasagomes | calling it a day, have a great evening all! | 17:57 |
| *** jkilpatr has quit IRC | 17:58 | |
| *** lucasagomes is now known as lucas-afk | 17:58 | |
| *** derekh has quit IRC | 17:59 | |
| dtantsur | +2 on the vpdu patch | 17:59 |
| jlvillal | lucas-afk: Good night | 17:59 |
| NobodyCam | night lucas-afk | 17:59 |
| *** alexpilotti has joined #openstack-ironic | 18:00 | |
| TheJulia | bradjones: running it now on a cleaned machine to try and reproduce, something seems really off since defaults indicate that it should install | 18:00 |
| TheJulia | the files | 18:00 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Provide default to uwsgi_install_prefix https://review.openstack.org/407117 | 18:01 |
| * vdrok leaves too | 18:01 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Do not use bare variables in with_items https://review.openstack.org/407118 | 18:01 |
| vdrok | good night everyone! | 18:01 |
| *** ralonsoh has quit IRC | 18:01 | |
| NobodyCam | night vdrok | 18:01 |
| jlvillal | Good night vdrok | 18:01 |
| rloo | night vdrok, lucas-afk | 18:02 |
| mariojv | night vdrok, lucas-afk | 18:02 |
| *** vinsh has quit IRC | 18:02 | |
| dtantsur | good night everyone, /me goes as well | 18:02 |
| NobodyCam | night dtantsur | 18:02 |
| *** vinsh has joined #openstack-ironic | 18:02 | |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| rloo | bye dtantsur|afk | 18:03 |
| *** rama_y has quit IRC | 18:03 | |
| bradjones | TheJulia: ok thanks | 18:04 |
| TheJulia | bradjones: xenial, or other? | 18:05 |
| *** vinsh_ has joined #openstack-ironic | 18:06 | |
| TheJulia | bradjones: looks like it recreated the files for me on xenial :\ | 18:06 |
| bradjones | TheJulia: yeah I'm on xenial :( | 18:07 |
| *** yolanda has quit IRC | 18:07 | |
| *** ijw has joined #openstack-ironic | 18:07 | |
| bradjones | TheJulia: I guess if it's working for you i'll try reimaging this server maybe something strange is happening with this install :) | 18:09 |
| *** vinsh has quit IRC | 18:10 | |
| *** trown is now known as trown|lunch | 18:10 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost: Fix git_root default in README https://review.openstack.org/407122 | 18:12 |
| *** glonlas has joined #openstack-ironic | 18:12 | |
| *** glonlas has quit IRC | 18:12 | |
| *** glonlas has joined #openstack-ironic | 18:13 | |
| *** fragatina has joined #openstack-ironic | 18:15 | |
| TheJulia | bradjones: Yeah, when you attempt to re-install, if you could capture the output from the install playbook run, that way we can kind of dig through and see exactly what it did | 18:15 |
| * jroll lunches | 18:16 | |
| TheJulia | speaking of lunch! That sounds like a good idea... that and I'm missing a main ingredent for making dinner :( | 18:17 |
| *** e0ne has joined #openstack-ironic | 18:20 | |
| * milan gone, night Ironic! :) | 18:21 | |
| *** milan has quit IRC | 18:21 | |
| *** jcoufal_ has joined #openstack-ironic | 18:23 | |
| *** jcoufal has quit IRC | 18:25 | |
| *** Sukhdev has quit IRC | 18:28 | |
| *** vgadiraj has quit IRC | 18:36 | |
| *** amoralej is now known as amoralej|off | 18:37 | |
| NobodyCam | night milan | 18:38 |
| *** Nisha_Agarwal has quit IRC | 18:40 | |
| *** jjohnson2 has joined #openstack-ironic | 18:42 | |
| *** soliosg has quit IRC | 18:42 | |
| *** soliosg has joined #openstack-ironic | 18:43 | |
| *** rama_y has joined #openstack-ironic | 18:47 | |
| *** fragatina has quit IRC | 18:52 | |
| *** fragatina has joined #openstack-ironic | 18:52 | |
| openstackgerrit | Martin Roy proposed openstack/ironic: Remove deprecation of snmp drivers https://review.openstack.org/405639 | 18:54 |
| *** d0ugal has joined #openstack-ironic | 18:54 | |
| *** openstackstatus has joined #openstack-ironic | 18:54 | |
| *** ChanServ sets mode: +v openstackstatus | 18:54 | |
| *** awiddersheim has quit IRC | 18:58 | |
| *** awiddersheim has joined #openstack-ironic | 18:58 | |
| *** jcoufal_ has quit IRC | 19:04 | |
| *** sambetts is now known as sambetts|afk | 19:05 | |
| *** jcoufal has joined #openstack-ironic | 19:05 | |
| *** srobert_ has joined #openstack-ironic | 19:19 | |
| *** baoli has quit IRC | 19:19 | |
| *** ijw_ has joined #openstack-ironic | 19:19 | |
| *** baoli has joined #openstack-ironic | 19:19 | |
| *** BadCub_ has joined #openstack-ironic | 19:20 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 19:23 |
| *** bapalm_ has joined #openstack-ironic | 19:26 | |
| *** ijw has quit IRC | 19:28 | |
| *** srobert has quit IRC | 19:28 | |
| *** Goneri has quit IRC | 19:28 | |
| *** clenimar has quit IRC | 19:28 | |
| *** karts has quit IRC | 19:28 | |
| *** rm_work has quit IRC | 19:28 | |
| *** nicodemos has quit IRC | 19:28 | |
| *** bapalm has quit IRC | 19:28 | |
| *** gabriel-bezerra has quit IRC | 19:28 | |
| *** mjturek has quit IRC | 19:28 | |
| *** BadCub has quit IRC | 19:28 | |
| *** jroll has quit IRC | 19:28 | |
| *** hecliunyx has quit IRC | 19:28 | |
| *** BadCub_ is now known as BadCub | 19:28 | |
| *** rm_work has joined #openstack-ironic | 19:30 | |
| *** rm_work has quit IRC | 19:30 | |
| *** rm_work has joined #openstack-ironic | 19:30 | |
| *** hecliunyx has joined #openstack-ironic | 19:31 | |
| *** yolanda has joined #openstack-ironic | 19:32 | |
| *** clenimar has joined #openstack-ironic | 19:34 | |
| *** fragatina has quit IRC | 19:35 | |
| *** Goneri has joined #openstack-ironic | 19:35 | |
| *** gabriel-bezerra has joined #openstack-ironic | 19:35 | |
| *** jroll has joined #openstack-ironic | 19:35 | |
| *** karts has joined #openstack-ironic | 19:35 | |
| *** mjturek has joined #openstack-ironic | 19:36 | |
| *** ChubYann has joined #openstack-ironic | 19:40 | |
| *** fragatina has joined #openstack-ironic | 19:42 | |
| *** penick has quit IRC | 19:42 | |
| *** aweeks_ has joined #openstack-ironic | 19:46 | |
| *** penick has joined #openstack-ironic | 19:46 | |
| *** aweeks_ is now known as aweeks | 19:46 | |
| TheJulia | bradjones: any luck? | 19:47 |
| *** trown|lunch is now known as trown | 19:47 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix keystone config templates https://review.openstack.org/407105 | 19:50 |
| jcook | JayF are you saying that Nova and Ironic have different states? | 20:00 |
| JayF | jcook: the Ironic state machine and the Nova state machine do not interact | 20:01 |
| JayF | jcook: beyond nova using the ironic state machine to put the nodes into proper states | 20:01 |
| JayF | jcook: I'm not 100% sure what's confusing you? | 20:01 |
| jcook | JayF so the states Nova reports are Nova states and might not match Ironic states? | 20:01 |
| JayF | yes, absolutely | 20:01 |
| JayF | the nova state machine and the ironic state machine are completely separate | 20:02 |
| jcook | JayF my original and outstanding question is exactly when does a node enter Ironic RESCUING | 20:02 |
| *** mtanino has joined #openstack-ironic | 20:02 | |
| JayF | jcook: after someone (including the nova driver) issues an api call equivalent of `ironic node-set-provision-state UUID rescue` | 20:03 |
| *** fragatina has quit IRC | 20:03 | |
| JayF | Question: Shoudl we have a bug (https://bugs.launchpad.net/ironic-inspector/+bug/1605574) that lists an issue only when used with unmerged patches? It seems like this should be closed as invalid? Any other opinions? | 20:03 |
| openstack | Launchpad bug 1605574 in Ironic Inspector "inspector fails in validate_interfaces" [Undecided,Confirmed] - Assigned to Zhenguo Niu (niu-zglinux) | 20:03 |
| jroll | from 1 to 3 +2s on the rolling upgrade spec while I'm commenting on it \o/ | 20:04 |
| jroll | JayF: I'd agree, but didn't verify those are unmerged | 20:04 |
| jroll | rloo: do you want to look at rolling upgrade spec before I land it? | 20:05 |
| JayF | jroll: some of them have -2s on them | 20:05 |
| JayF | lol. | 20:05 |
| jroll | JayF: ha. | 20:05 |
| jcook | JayF so step 5 line 67: https://review.openstack.org/#/c/171878/22/specs/approved/implement-rescue-mode.rst ACTIVE -> RESCUING in Ironic? | 20:05 |
| patchbot | patch 171878 - ironic-specs - Implement rescue mode (MERGED) | 20:05 |
| *** fragatina has joined #openstack-ironic | 20:05 | |
| JayF | 5. Ironic API receives set_provision_state call, and performs do_node_rescue | 20:05 |
| JayF | RPC call. | 20:05 |
| jcook | Ironic API receives set_provision_state call (RESCUE -> RESCUING), and performs do_node_rescue RPC call. | 20:06 |
| jcook | erm | 20:06 |
| JayF | the do_node_rescue RPC call should likely be what does the task.process_event(rescue) | 20:06 |
| jcook | sorry s/RESCUE/ACTIVE | 20:06 |
| jcook | ah ok | 20:06 |
| JayF | which would change the state from ACTIVE -> RESCU[ING|WAIT] | 20:06 |
| jcook | so Ironic API receives set_provision_state call, and performs do_node_rescue (ACTIVE -> RESCUING) | 20:06 |
| jroll | +1 | 20:06 |
| JayF | yep | 20:06 |
| jcook | cool thanks | 20:07 |
| JayF | jroll: have you seen https://bugs.launchpad.net/ironic/+bug/1566136 yet? | 20:08 |
| openstack | Launchpad bug 1566136 in ironic (Ubuntu) "Mitaka ironic-conductor error on stop: Cannot switch to MAINLOOP from MAINLOOP" [Undecided,New] | 20:08 |
| jroll | I saw the title, nothing more | 20:08 |
| jroll | JayF: oh hey it isn't just us http://lists.openstack.org/pipermail/openstack-dev/2016-February/086978.html | 20:09 |
| jroll | or wasn't | 20:09 |
| JayF | oh perfect | 20:09 |
| jroll | JayF: also https://bugzilla.redhat.com/show_bug.cgi?id=1288528 | 20:11 |
| openstack | bugzilla.redhat.com bug 1288528 in python-oslo-service "services can´t stop due to broken oslo service" [Urgent,Closed: errata] - Assigned to vstinner | 20:11 |
| JayF | jroll: linked that in there, asked for version of oslo.service | 20:11 |
| JayF | jroll: and marked incomplete | 20:11 |
| jroll | I wonder if the ubuntu package was using the wrong version yeah that | 20:11 |
| jroll | :) | 20:11 |
| jroll | thanks | 20:11 |
| JayF | I saw that rh bug, but not the openstack ml post | 20:11 |
| JayF | the openstack ml post is much more helpful :) | 20:11 |
| jroll | the bugzilla has The race condition occurs when two SIGTERM are received shortly. | 20:11 |
| *** ijw_ has quit IRC | 20:12 | |
| JayF | lucas-afk: https://bugs.launchpad.net/ironic/+bug/1646477 you are assigned to this, can you set status/importance based on your work in the morning? | 20:13 |
| openstack | Launchpad bug 1646477 in Ironic "Callback false timeouts" [Undecided,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 20:13 |
| jroll | rloo: landing it | 20:14 |
| jroll | wheeeeeeeee | 20:14 |
| openstackgerrit | Merged openstack/bifrost: Do not use bare variables in with_items https://review.openstack.org/407118 | 20:14 |
| *** jcoufal has quit IRC | 20:16 | |
| *** srobert_ has quit IRC | 20:16 | |
| stevemar | rloo: "dumbfounded" ? :) | 20:16 |
| openstackgerrit | Merged openstack/ironic-specs: Support rolling upgrades https://review.openstack.org/299245 | 20:17 |
| JayF | aarefiev: dtantsur|afk: https://bugs.launchpad.net/ironic-inspector/+bug/1616057 might be smoething you all would be interested in looking at... seems crazy/impossible | 20:19 |
| openstack | Launchpad bug 1616057 in Ironic Inspector ""Unknown column 'nodes.finished_at' in 'where clause'"" [Undecided,Incomplete] - Assigned to Waldemar Znoinski (wznoinsk) | 20:19 |
| JayF | jroll: curious, is https://bugs.launchpad.net/ironic/+bug/1310131 an Ironic bug at all? The nova-half of it was expired | 20:23 |
| openstack | Launchpad bug 1310131 in OpenStack Compute (nova) "Some non-supported actions in Ironic nova driver do not return errors to the user" [Undecided,Expired] | 20:23 |
| JayF | jroll: I just don't think there's any action Ironic can take on that bug | 20:23 |
| jroll | JayF: I agree, I believe we're waiting on the nova feature (that I forget the name of) that makes that easy | 20:24 |
| *** fragatina has quit IRC | 20:24 | |
| JayF | jroll: so going to mark it invalid against ironic, since all the action to take is in nvoa | 20:24 |
| jroll | "capabilities discovery" | 20:24 |
| *** fragatina has joined #openstack-ironic | 20:24 | |
| jroll | sure | 20:24 |
| *** shuquan_ has joined #openstack-ironic | 20:25 | |
| *** fragatina has quit IRC | 20:25 | |
| * JayF going to get all the bugs finished today | 20:25 | |
| JayF | s/finished/triaged/ | 20:25 |
| *** jtomasek has quit IRC | 20:26 | |
| *** vgadiraj has joined #openstack-ironic | 20:28 | |
| *** fragatina has joined #openstack-ironic | 20:29 | |
| *** fragatina has quit IRC | 20:29 | |
| *** shuquan_ has quit IRC | 20:29 | |
| *** moshele has joined #openstack-ironic | 20:30 | |
| jcook | JayF stupid question. Is the agent always running or is it an agent ramdisk that only runs when host OS or other ramdisk is not? | 20:31 |
| JayF | jcook: the agent ramdisk only runs when the agent ramdisk is told to run by ironic. It isn't inside instnace at all. | 20:32 |
| JayF | jcook: in fact, rescue will be the first time we ever even have a user login to a ramdisk that used to run an agent | 20:32 |
| *** d0ugal has quit IRC | 20:32 | |
| JayF | Ironic runs in devstack on keystone v3 now, right? Is there any reason I can't mark https://bugs.launchpad.net/ironic/+bug/1494776 as Fix Released? | 20:33 |
| openstack | Launchpad bug 1494776 in Ironic "Ironic API fails when keystone /v2.0 pipeline is disabled" [High,Confirmed] - Assigned to Pavlo Shchelokovskyy (pshchelo) | 20:33 |
| jroll | JayF: I believe that's correct, devstack is v3 by default, but I'm not sure that implies v2.0 is disabled | 20:36 |
| JayF | looking at the linked code, it links to the place we now set keystone v3 | 20:37 |
| JayF | so I'm going to comment and mark it resolved | 20:37 |
| *** fragatina has joined #openstack-ironic | 20:41 | |
| *** vinsh_ has quit IRC | 20:44 | |
| *** vinsh has joined #openstack-ironic | 20:44 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 20:45 |
| JayF | mgould: https://bugs.launchpad.net/ironic/+bug/1590408 you OK with me marking this as resolved now? | 20:45 |
| openstack | Launchpad bug 1590408 in Ironic "Node cleaning fails with 'Failed to tear down from cleaning for node UUID'" [High,Confirmed] - Assigned to Miles Gould (mgould) | 20:45 |
| *** penick has quit IRC | 20:45 | |
| *** fragatina has quit IRC | 20:46 | |
| *** jkilpatr has joined #openstack-ironic | 20:48 | |
| *** penick has joined #openstack-ironic | 20:48 | |
| *** vinsh_ has joined #openstack-ironic | 20:49 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 20:49 |
| *** soliosg has quit IRC | 20:49 | |
| *** soliosg has joined #openstack-ironic | 20:49 | |
| *** srobert has joined #openstack-ironic | 20:52 | |
| *** vinsh has quit IRC | 20:53 | |
| *** ijw has joined #openstack-ironic | 20:57 | |
| *** penick has quit IRC | 20:58 | |
| *** moshele has quit IRC | 21:02 | |
| *** ijw has quit IRC | 21:02 | |
| *** vinsh_ has quit IRC | 21:02 | |
| *** penick has joined #openstack-ironic | 21:04 | |
| openstackgerrit | Merged openstack/bifrost: Ensure git_root folder exists https://review.openstack.org/407031 | 21:07 |
| jcook | JayF so what does the Agent do with the rescue password and the network config info for the Rescue ramdisk? | 21:07 |
| openstackgerrit | Merged openstack/bifrost: DRY when installing components https://review.openstack.org/407034 | 21:07 |
| *** MattMan has quit IRC | 21:08 | |
| *** MattMan has joined #openstack-ironic | 21:08 | |
| JayF | jcook: it will have to run a program (such as glean) to configure local ramdisk networking to match "tenant" networking config and set root password | 21:10 |
| jcook | JayF how does that work / where does the config info live? | 21:11 |
| JayF | jcook: I believe the original plan was for the agent to extract the configdrive from node.instance_info and use that | 21:11 |
| jcook | is there a config drive on disk somewhere? | 21:11 |
| JayF | jcook: basically slurp the configdrive network informatino out of node.instance_info it gets on lookup | 21:11 |
| mariojv | JayF: so we should just pass the configdrive during prepare_rescue? | 21:11 |
| JayF | mariojv: well, we pass a node, right? | 21:12 |
| mariojv | as well as rescue_password, of course, since the pass is different from whatever's in the configdrive | 21:12 |
| JayF | mariojv: IDK if we need to explicitly pass a configdrive | 21:12 |
| JayF | all this info is in the node | 21:12 |
| *** aNuposic has quit IRC | 21:12 | |
| JayF | so sorta a question of what should be gotten from the node | 21:12 |
| JayF | and what should be passed | 21:12 |
| JayF | I guess for security reasons, we should pass it all in | 21:12 |
| jcook | JayF mariojv I'm looking at this fwiw: https://etherpad.openstack.org/p/rescue-plan | 21:13 |
| JayF | since configdrive and rescue_password should be censored from unauth'd lookup endpoint | 21:13 |
| mariojv | JayF: is the whole node passed by default, or you mean that should be part of the call to the agent? | 21:13 |
| mariojv | i'm looking at the ironic command code and don't see it passing the full node | 21:13 |
| JayF | mariojv: the agent has a node from lookup | 21:13 |
| JayF | mariojv: but it's censored, so I don't think it'll have instance info and/or rescue password | 21:13 |
| mariojv | oh, i didn't know that got cached | 21:13 |
| mariojv | k | 21:14 |
| JayF | jcook: 2.2.4.2.4 is wrong, I think | 21:14 |
| jcook | according to that ^ the node is obtained from API and then prepare_rescue passes in network info|config drive and rescue password | 21:14 |
| JayF | jcook: I think we're going to have to have it passed in via prepare_rescue | 21:14 |
| mariojv | JayF: yeah - so i'll have prepare_rescue pass configdrive and rescue pass | 21:14 |
| jcook | JayF I'm looking at 2.1.1 | 21:14 |
| JayF | jcook: in which case I don't fully understand | 21:15 |
| jcook | jcook don't fully understand 2.1.1 or something I said? | 21:15 |
| mariojv | JayF: also - we decided that there's a separate rescue ramdisk, right? | 21:15 |
| JayF | mariojv: no we didn't | 21:16 |
| jcook | JayF ^ | 21:16 |
| mariojv | oh | 21:16 |
| jcook | talking to myself lol | 21:16 |
| JayF | jcook: don't understand what your questino is w/r/t 2.1.1 | 21:16 |
| mariojv | i thought we did since there are references to "rescue ramdisk" everywhere | 21:16 |
| mariojv | that makes my job easier :D | 21:16 |
| JayF | mariojv: "rescue ramdisk" | 21:16 |
| JayF | == IPA with a rescue extension | 21:16 |
| JayF | which we'll ship in every IPA :P | 21:16 |
| mariojv | ok - i thought we might want to have a separate one b/c vendor tooling | 21:16 |
| JayF | but for doc purposes, many deployers will want to use a different ramdisk | 21:17 |
| jcook | I'm trying to grok it in terms of what info it receives, in what order, what it does with it, and it finishes talking to conductor before it reboots into rescue ramdisk | 21:17 |
| JayF | we want a deployer to be able to use a different ramdisk | 21:17 |
| JayF | but in our gate it'll be the same | 21:17 |
| mariojv | ok | 21:17 |
| jcook | so one question, what does it do with node in 2.1.1.1 | 21:17 |
| mariojv | that'll require that all the code be set up to prep a separate ramdisk, then | 21:17 |
| mariojv | which is fine - that's what i was planning to do | 21:17 |
| JayF | jcook: maybe me/you/mario should jump into a vidyo? | 21:17 |
| jcook | Is that just so it knows what node it is? | 21:17 |
| jcook | sure | 21:17 |
| jlvillal | jroll: Why is the one line deleted? https://review.openstack.org/#/c/404436/4/devstack/lib/ironic | 21:21 |
| patchbot | patch 404436 - ironic - Move CONF.service_available.ironic to our plugin | 21:21 |
| jlvillal | I didn't see mention of it in the commit message. Wanted to make sure it was intentional | 21:22 |
| jroll | jlvillal: is intentional, yes, it isn't needed | 21:22 |
| jlvillal | jroll: Thanks | 21:22 |
| *** Sukhdev has joined #openstack-ironic | 21:23 | |
| *** srobert has quit IRC | 21:24 | |
| *** srobert has joined #openstack-ironic | 21:25 | |
| *** rama_y has quit IRC | 21:26 | |
| jlvillal | jroll: The patch looks good to me. I already see two +2. Any known reason it hasn't been +A yet? | 21:32 |
| jroll | jlvillal: possibly aarefiev's comment (which is accurate and we need to fix *somewhere*), but I'm not sure | 21:36 |
| jroll | jlvillal: see https://review.openstack.org/#/c/397775/5 <- grenade is failing, but I'm not sure why, and not entirely sure that's related to anton's comment | 21:37 |
| patchbot | patch 397775 - ironic - DNM: Testing 355586 | 21:37 |
| jroll | er, sorry, where'd anton's comment go | 21:37 |
| jroll | I think it's fine to land, going to mark tempest thing wip until I can sort out the grenade thing | 21:38 |
| jroll | oh I can't wip, sigh | 21:38 |
| jroll | jlvillal: I guess don't land it :) | 21:38 |
| jlvillal | jroll: Oh okay. Do you want to WIP the patch then? Which patch can't you WIP? | 21:39 |
| jroll | I just did wip it, was hoping to wip the tempest patch but I'm not the owner | 21:39 |
| jlvillal | Thanks | 21:40 |
| *** Goneri has quit IRC | 21:40 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix keystone config templates https://review.openstack.org/407105 | 21:42 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone to git contents download https://review.openstack.org/407215 | 21:42 |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 21:53 |
| *** baoli has quit IRC | 21:53 | |
| *** trown is now known as trown|outtypewww | 21:55 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: multi-TENANT with tempest "smoke" test helper https://review.openstack.org/406368 | 21:58 |
| *** rbudden has quit IRC | 21:58 | |
| *** awiddersheim has quit IRC | 22:00 | |
| *** awiddersheim has joined #openstack-ironic | 22:04 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Test Tempest change https://review.openstack.org/407220 | 22:06 |
| *** ijw has joined #openstack-ironic | 22:07 | |
| *** srobert has quit IRC | 22:10 | |
| *** fragatina has joined #openstack-ironic | 22:14 | |
| rloo | jroll: thx for the +A on rolling upgrades. Yay! Am not feeling well so good that you didn't wait for me to reply! | 22:15 |
| jroll | rloo: :) | 22:16 |
| jroll | rloo: go rest and feel better! | 22:16 |
| rloo | stevemar: i'm dumbfounded as to what i picked dumbfounded. well, i'm not sure why i picked that but I did :) | 22:16 |
| stevemar | rloo: :) | 22:17 |
| rloo | jroll: i did rest and feel like crap still. hopefully i'll be better tomorrow. | 22:17 |
| rloo | ciao til tomorrow folks. | 22:17 |
| *** rloo has left #openstack-ironic | 22:18 | |
| *** fragatina has quit IRC | 22:23 | |
| mariojv | sambetts|afk: hey, let me know if you have a bit of time tomorrow to chat about networking | 22:24 |
| mariojv | sambetts|afk: essentially, for rescue mode, "unconfigure_tenant_networks" is too destructive and actually destroys connections between node ports and tenant networks rather than just removing the config | 22:25 |
| mariojv | sambetts|afk: i'm thinking about renaming that "destroy_tenant_networks" and having a new, less destructive "unconfigure_tenant_networks" | 22:25 |
| mariojv | i'd like to see how that relates to network attach/detach | 22:26 |
| mariojv | the reason rescue needs something like that is because we have to 1. remove tenant network from node 2. put rescue network on node 3. remove rescue network from node 4. re-add tenant networks on node | 22:26 |
| jroll | mariojv: fwiw, seems like y'all have some similar goals https://review.openstack.org/#/c/327046/27/ironic/drivers/modules/network/flat.py | 22:27 |
| patchbot | patch 327046 - ironic - Add Virtual Network Interface Driver APIs | 22:27 |
| * mariojv looks | 22:27 | |
| jroll | maybe not intentional | 22:27 |
| mariojv | ah, yeah, unconfigure -> pass now | 22:27 |
| mariojv | jroll: i'd think unintentional because unconfigure_tenant_networks behaves the same way in the neutron driver | 22:28 |
| jroll | yeah | 22:28 |
| mariojv | in that patch set | 22:28 |
| mariojv | but i'll let sambetts|afk speak to that | 22:28 |
| jroll | indeed | 22:28 |
| *** vgadiraj has quit IRC | 22:28 | |
| *** fragatina has joined #openstack-ironic | 22:29 | |
| mariojv | one interesting thing i noticed about the neutron driver unconfigure_tenant_networks is that it just removes the port from neutron - it doesn't remove the vif id from the ironic port | 22:29 |
| mariojv | so maybe that's less destructive than we were talking about? | 22:29 |
| jroll | deleting the port is pretty destructive imo :P | 22:30 |
| mariojv | oh | 22:30 |
| mariojv | remove_neutron_ports | 22:30 |
| mariojv | not remove_neutron_ports_from_network | 22:30 |
| mariojv | oops | 22:30 |
| jroll | but yeah, not sure why that doesn't remove the id from the node | 22:30 |
| JayF | unrecoverable system error!?! your ports are very... deleted!!!!! | 22:30 |
| jroll | probably handled later | 22:30 |
| mariojv | yeah | 22:31 |
| mariojv | and configure_tenant_networks just updates the existing ports. it doesn't create them in neutron | 22:31 |
| mariojv | presumably nova does that | 22:31 |
| jroll | yep | 22:32 |
| *** penick has quit IRC | 22:33 | |
| mariojv | sambetts|afk: another option that was discussed was adding a destroy=True kwarg instead of different methods, to be less disruptive to 3rd party drivers | 22:34 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic: [WIP] Agent driver rescue implementation https://review.openstack.org/400437 | 22:38 |
| *** rcernin has joined #openstack-ironic | 22:40 | |
| *** garthb has joined #openstack-ironic | 22:45 | |
| *** garthb_ has joined #openstack-ironic | 22:45 | |
| *** garthb_ has quit IRC | 22:45 | |
| *** garthb has quit IRC | 22:45 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 22:48 |
| *** penick has joined #openstack-ironic | 22:50 | |
| *** jheroux has quit IRC | 22:51 | |
| *** lindycoder has quit IRC | 22:54 | |
| *** e0ne has quit IRC | 22:54 | |
| *** e0ne has joined #openstack-ironic | 22:56 | |
| *** glonlas has quit IRC | 22:56 | |
| *** e0ne has quit IRC | 22:56 | |
| *** e0ne has joined #openstack-ironic | 22:56 | |
| *** e0ne has quit IRC | 22:57 | |
| *** e0ne has joined #openstack-ironic | 22:57 | |
| *** e0ne has quit IRC | 22:58 | |
| *** e0ne has joined #openstack-ironic | 22:58 | |
| *** e0ne has quit IRC | 22:58 | |
| *** e0ne has joined #openstack-ironic | 22:59 | |
| *** e0ne has quit IRC | 22:59 | |
| *** e0ne has joined #openstack-ironic | 23:00 | |
| *** e0ne has quit IRC | 23:00 | |
| *** e0ne has joined #openstack-ironic | 23:00 | |
| *** e0ne has quit IRC | 23:01 | |
| *** rbudden has joined #openstack-ironic | 23:02 | |
| *** aNuposic has joined #openstack-ironic | 23:03 | |
| *** fragatin_ has joined #openstack-ironic | 23:05 | |
| *** fragatina has quit IRC | 23:08 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Default keystone to uuid tokens https://review.openstack.org/407231 | 23:13 |
| *** penick has quit IRC | 23:21 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic-specs: Update rescue spec https://review.openstack.org/407236 | 23:30 |
| *** Sukhdev has quit IRC | 23:31 | |
| openstackgerrit | Jesse J. Cook proposed openstack/ironic-specs: Update rescue spec https://review.openstack.org/407236 | 23:32 |
| jcook | mariojv JayF ^ | 23:33 |
| JayF | Awesome, I'll review it before EOD | 23:33 |
| JayF | ty! | 23:33 |
| jcook | thanks :) | 23:34 |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 23:34 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add keystone to git contents download https://review.openstack.org/407215 | 23:35 |
| *** penick has joined #openstack-ironic | 23:36 | |
| *** rama_y has joined #openstack-ironic | 23:44 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Adding staging driver support https://review.openstack.org/406401 | 23:45 |
| *** pmannidi has joined #openstack-ironic | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!