| *** ijw has joined #openstack-ironic | 00:04 | |
| *** ijw has quit IRC | 00:04 | |
| *** ijw has joined #openstack-ironic | 00:04 | |
| *** namnh has quit IRC | 00:05 | |
| *** rpioso is now known as rpioso|afk | 00:05 | |
| *** yangyapeng has quit IRC | 00:12 | |
| *** rwsu has joined #openstack-ironic | 00:13 | |
| *** yangyapeng has joined #openstack-ironic | 00:13 | |
| *** zshi__ has joined #openstack-ironic | 00:15 | |
| *** yangyapeng has quit IRC | 00:17 | |
| *** fragatina has quit IRC | 00:31 | |
| *** fragatina has joined #openstack-ironic | 00:31 | |
| *** fragatina has quit IRC | 00:32 | |
| *** zshi__ has quit IRC | 00:32 | |
| *** fragatina has joined #openstack-ironic | 00:32 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 00:35 | |
| *** rloo has quit IRC | 00:38 | |
| *** hw_wutianwei has joined #openstack-ironic | 00:39 | |
| *** zshi__ has joined #openstack-ironic | 00:39 | |
| *** Nisha_Agarwal has quit IRC | 00:41 | |
| *** zshi__ has quit IRC | 00:51 | |
| *** zshi__ has joined #openstack-ironic | 00:53 | |
| *** charcol has joined #openstack-ironic | 01:01 | |
| *** phuongnh has joined #openstack-ironic | 01:05 | |
| *** fragatina has quit IRC | 01:05 | |
| *** fragatina has joined #openstack-ironic | 01:06 | |
| *** fragatina has quit IRC | 01:10 | |
| *** yangyapeng has joined #openstack-ironic | 01:12 | |
| *** yangyapeng has quit IRC | 01:12 | |
| *** yangyapeng has joined #openstack-ironic | 01:13 | |
| *** zshi__ has quit IRC | 01:18 | |
| *** syqian has joined #openstack-ironic | 01:30 | |
| *** namnh has joined #openstack-ironic | 01:31 | |
| *** rama_y has quit IRC | 01:49 | |
| *** zhangjl has joined #openstack-ironic | 01:53 | |
| *** zhangfei has joined #openstack-ironic | 01:58 | |
| *** crushil has quit IRC | 01:58 | |
| *** yangyape_ has joined #openstack-ironic | 02:07 | |
| *** yangyapeng has quit IRC | 02:10 | |
| *** zshi has quit IRC | 02:14 | |
| *** zshi has joined #openstack-ironic | 02:16 | |
| *** kzaitsev_pi has quit IRC | 02:39 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 02:39 | |
| *** kzaitsev_pi has joined #openstack-ironic | 02:46 | |
| *** fragatina has joined #openstack-ironic | 02:52 | |
| *** ijw has quit IRC | 02:55 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 02:57 | |
| *** Nisha_Agarwal has quit IRC | 03:00 | |
| *** AlexeyAbashkin has quit IRC | 03:02 | |
| *** yangyape_ has quit IRC | 03:08 | |
| *** yangyapeng has joined #openstack-ironic | 03:09 | |
| *** yangyape_ has joined #openstack-ironic | 03:10 | |
| *** ijw has joined #openstack-ironic | 03:12 | |
| *** fragatina has quit IRC | 03:14 | |
| *** fragatina has joined #openstack-ironic | 03:15 | |
| *** yangyapeng has quit IRC | 03:16 | |
| *** ijw has quit IRC | 03:19 | |
| *** fragatina has quit IRC | 03:19 | |
| *** vsaienk0 has joined #openstack-ironic | 03:27 | |
| *** fragatina has joined #openstack-ironic | 03:33 | |
| *** vsaienk0 has quit IRC | 03:37 | |
| *** mat128 has quit IRC | 03:39 | |
| *** ijw has joined #openstack-ironic | 03:48 | |
| *** alex_xu has quit IRC | 03:52 | |
| *** mat128 has joined #openstack-ironic | 03:52 | |
| *** rbudden has quit IRC | 03:53 | |
| *** alex_xu has joined #openstack-ironic | 03:54 | |
| *** mat128 has quit IRC | 03:57 | |
| *** zhangjl has left #openstack-ironic | 04:02 | |
| *** aparnav has joined #openstack-ironic | 04:07 | |
| *** syqian has quit IRC | 04:10 | |
| *** syqian has joined #openstack-ironic | 04:10 | |
| *** syqian has quit IRC | 04:14 | |
| *** udesale has joined #openstack-ironic | 04:22 | |
| *** yangyape_ has quit IRC | 04:24 | |
| *** yangyapeng has joined #openstack-ironic | 04:25 | |
| *** jaianshu has joined #openstack-ironic | 04:28 | |
| *** ijw has quit IRC | 04:31 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:32 | |
| *** adisky_ has quit IRC | 04:35 | |
| *** mat128 has joined #openstack-ironic | 04:54 | |
| *** hshiina has joined #openstack-ironic | 04:56 | |
| *** udesale has quit IRC | 05:12 | |
| *** udesale has joined #openstack-ironic | 05:13 | |
| *** udesale has quit IRC | 05:13 | |
| *** ChubYann has quit IRC | 05:13 | |
| *** udesale has joined #openstack-ironic | 05:17 | |
| *** dmellado has quit IRC | 05:21 | |
| *** dmellado has joined #openstack-ironic | 05:22 | |
| *** zzzeek has quit IRC | 05:24 | |
| *** zzzeek has joined #openstack-ironic | 05:26 | |
| *** jtomasek has quit IRC | 05:27 | |
| *** dmellado has quit IRC | 05:37 | |
| *** Nisha_Agarwal has quit IRC | 05:40 | |
| *** zhangjl has joined #openstack-ironic | 05:40 | |
| *** dmellado has joined #openstack-ironic | 05:40 | |
| *** yolanda has joined #openstack-ironic | 05:44 | |
| *** zshi__ has joined #openstack-ironic | 05:46 | |
| *** hoonetorg has quit IRC | 06:02 | |
| *** zhangfei has quit IRC | 06:09 | |
| *** hoonetorg has joined #openstack-ironic | 06:16 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:16 | |
| *** spectr has quit IRC | 06:20 | |
| *** spectr has joined #openstack-ironic | 06:21 | |
| *** spectr has quit IRC | 06:22 | |
| *** zhangfei has joined #openstack-ironic | 06:22 | |
| *** mjura has joined #openstack-ironic | 06:23 | |
| *** spectr has joined #openstack-ironic | 06:24 | |
| *** stendulker has joined #openstack-ironic | 06:26 | |
| *** yolanda has quit IRC | 06:40 | |
| *** vsaienk0 has joined #openstack-ironic | 06:45 | |
| *** zhangfei has quit IRC | 06:46 | |
| *** Goneri has joined #openstack-ironic | 06:46 | |
| *** karimb has joined #openstack-ironic | 06:47 | |
| *** pcaruana has joined #openstack-ironic | 07:02 | |
| *** zhangfei has joined #openstack-ironic | 07:03 | |
| *** tesseract has joined #openstack-ironic | 07:11 | |
| *** yangyape_ has joined #openstack-ironic | 07:14 | |
| *** yangyapeng has quit IRC | 07:16 | |
| *** ccamacho has joined #openstack-ironic | 07:25 | |
| *** zhangfei has quit IRC | 07:25 | |
| *** charcol has quit IRC | 07:26 | |
| *** yangyape_ has quit IRC | 07:27 | |
| *** yangyapeng has joined #openstack-ironic | 07:27 | |
| *** racedo has joined #openstack-ironic | 07:28 | |
| *** yangyape_ has joined #openstack-ironic | 07:30 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:32 | |
| *** yangyapeng has quit IRC | 07:32 | |
| *** zhangfei has joined #openstack-ironic | 07:38 | |
| *** rbartal has joined #openstack-ironic | 07:43 | |
| *** yolanda has joined #openstack-ironic | 07:46 | |
| *** spectr has quit IRC | 07:48 | |
| *** adisky_ has joined #openstack-ironic | 07:48 | |
| *** spectr has joined #openstack-ironic | 07:48 | |
| openstackgerrit | fengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data https://review.openstack.org/514552 | 07:49 |
|---|---|---|
| openstackgerrit | fengzhr proposed openstack/ironic-inspector master: Use database to store introspection_data https://review.openstack.org/514552 | 07:51 |
| *** Nisha_Agarwal has quit IRC | 08:00 | |
| *** marios has joined #openstack-ironic | 08:03 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent master: Fix to return 'root_uuid' as part of command status https://review.openstack.org/500719 | 08:03 |
| *** karthiks has quit IRC | 08:04 | |
| *** rbartal has quit IRC | 08:10 | |
| *** dtantsur|afk is now known as dtantsur | 08:12 | |
| dtantsur | morning ironic | 08:12 |
| *** rbartal has joined #openstack-ironic | 08:12 | |
| *** Goneri has quit IRC | 08:16 | |
| *** karthiks has joined #openstack-ironic | 08:20 | |
| *** ralonsoh has joined #openstack-ironic | 08:20 | |
| *** namnh has quit IRC | 08:21 | |
| *** pmannidi has quit IRC | 08:31 | |
| *** lucas-afk is now known as lucasagomes | 08:32 | |
| *** priteau has joined #openstack-ironic | 08:33 | |
| stendulker | Good morning dtantsur :) | 08:34 |
| *** rbartal has quit IRC | 08:34 | |
| *** derekh has joined #openstack-ironic | 08:37 | |
| dtantsur | o/ | 08:42 |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:43 | |
| dtantsur | aarefiev: hey! mind checking https://review.openstack.org/507559 and https://review.openstack.org/#/c/514462/ please? I'd like to release ironic-inspector soon(ish) | 08:43 |
| patchbot | patch 507559 - ironic-inspector - Generate version_id upon add_node | 08:43 |
| patchbot | patch 514462 - ironic-inspector - refer to 'openstack baremetal' CLI in docs | 08:43 |
| *** jtomasek has joined #openstack-ironic | 08:44 | |
| * aarefiev is looking | 08:46 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Improve the catching of calls to 'execute' related functions https://review.openstack.org/466100 | 08:47 |
| *** hwoarang has quit IRC | 08:48 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/514256 | 08:48 |
| *** litao__ has joined #openstack-ironic | 08:49 | |
| *** amarao has joined #openstack-ironic | 08:49 | |
| *** jtomasek has quit IRC | 08:50 | |
| *** hwoarang has joined #openstack-ironic | 08:51 | |
| *** jtomasek has joined #openstack-ironic | 08:51 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Clean up release notes before a release https://review.openstack.org/514566 | 08:55 |
| dtantsur | aarefiev: also this one please ^^^ | 08:55 |
| dtantsur | sambetts|afk: too | 08:55 |
| *** yangyapeng has joined #openstack-ironic | 08:57 | |
| *** Goneri has joined #openstack-ironic | 08:58 | |
| Nisha_Agarwal | dtantsur, we are moving zuulv3 jobs in tree, so if we want to add any new job(i.e. for python35) we can directly do in our project, correct? | 08:58 |
| dtantsur | Nisha_Agarwal: yep | 08:58 |
| Nisha_Agarwal | dtantsur, currently it is in legacy folder....the new one shouldnt be in legacy folder IMO | 08:58 |
| dtantsur | Nisha_Agarwal: ideally, yes | 08:59 |
| vdrok | good morning ironic, dtantsur Nisha_Agarwal stendulker | 08:59 |
| dtantsur | o/ | 08:59 |
| Nisha_Agarwal | dtantsur, Thanks. I will abandon my infra patch where i added a gate for py35 for inspector and base the new patch in inspector on pas-ha's patch | 09:00 |
| Nisha_Agarwal | vdrok, good morning | 09:00 |
| stendulker | Good morning vdrok :) | 09:00 |
| Nisha_Agarwal | :) | 09:00 |
| dtantsur | yep, thanks! | 09:00 |
| *** yangyape_ has quit IRC | 09:01 | |
| vdrok | dtantsur: so how do you think, do we need to backport that zullv3 stuff into newton? | 09:01 |
| dtantsur | vdrok: nope. I've submitted the last release, and the branch will got EOL right after it's out | 09:01 |
| *** jaganathan has joined #openstack-ironic | 09:01 | |
| dtantsur | no more changes should merge to stable/newton | 09:01 |
| vdrok | okie | 09:02 |
| openstackgerrit | zenghui.shi proposed openstack/ironic-specs master: Hardware interface for BIOS configuration https://review.openstack.org/496481 | 09:02 |
| *** jaianshu_ has joined #openstack-ironic | 09:03 | |
| *** karthiks has quit IRC | 09:04 | |
| *** jaianshu has quit IRC | 09:07 | |
| *** milan has joined #openstack-ironic | 09:07 | |
| milan | morning Ironic! :) | 09:08 |
| milan | #pixiesay -mflexing work...work...... | 09:09 |
| PixieBoots | ᕙʕ⇀ᴥ⇀ʔᕗ: work...work...... | 09:09 |
| *** zshi__ has quit IRC | 09:09 | |
| *** hshiina has quit IRC | 09:10 | |
| dtantsur | yo milan, please also check https://review.openstack.org/514566 | 09:10 |
| patchbot | patch 514566 - ironic-inspector - Clean up release notes before a release | 09:10 |
| milan | dtantsur, morning! sure | 09:11 |
| * milan checks | 09:11 | |
| *** ankkumar has joined #openstack-ironic | 09:13 | |
| *** karthiks has joined #openstack-ironic | 09:20 | |
| *** gmonteiro has quit IRC | 09:22 | |
| *** gmonteiro has joined #openstack-ironic | 09:22 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Clean up release notes before a release https://review.openstack.org/514566 | 09:23 |
| vdrok | good morning milan , finally a proper nick :) | 09:23 |
| *** makowals has quit IRC | 09:23 | |
| dtantsur | aarefiev, milan: updated ^^^ | 09:23 |
| milan | morning vdrok! :D | 09:24 |
| milan | yeah there's a clash | 09:24 |
| *** makowals has joined #openstack-ironic | 09:24 | |
| milan | so you never know to which milan you're talking | 09:24 |
| vdrok | milan: is milano one free? | 09:25 |
| *** milan is now known as milano | 09:25 | |
| milano | lol | 09:25 |
| milano | seems free | 09:25 |
| vdrok | ++ | 09:25 |
| vdrok | register while it is! | 09:25 |
| milano | rloo gonna ask about it definitely :D | 09:26 |
| vdrok | you can trade it for milan afterwards, when everyone will want such a cool nicj | 09:26 |
| vdrok | *nick | 09:26 |
| *** zzzeek has quit IRC | 09:30 | |
| *** Nisha_Agarwal has quit IRC | 09:30 | |
| *** athomas has quit IRC | 09:32 | |
| *** zzzeek has joined #openstack-ironic | 09:32 | |
| milano | lol | 09:33 |
| vdrok | dtantsur: hrm, seems like we are missing py27 job on ironic-specs | 09:33 |
| vdrok | we currently have only docs-build running | 09:33 |
| dtantsur | wut | 09:33 |
| milano | dtantsur, you gonna address the nit from #1? cause I'm cool even w/o +2 ;) | 09:33 |
| milano | * w/o ; +2 | 09:34 |
| *** hw_wutianwei has quit IRC | 09:34 | |
| dtantsur | milano: I addressed Anton's nits, should I address something else? | 09:34 |
| milano | I raced with aarefiev ;) | 09:34 |
| milano | morning aarefiev :D | 09:34 |
| aarefiev | morning milano :) | 09:34 |
| milano | dtantsur, jsut one thing: the wording sounded a bit too directive but | 09:35 |
| milano | #pixiesay -mmeh | 09:35 |
| PixieBoots | ¯\_(ツ)_/¯ | 09:35 |
| dtantsur | I actually kind of prefer this, to avoid people thinking "should I? or must I? or can I?" | 09:35 |
| milano | it do just! | 09:36 |
| *** yangyape_ has joined #openstack-ironic | 09:40 | |
| *** sambetts|afk is now known as sambetts | 09:43 | |
| *** yangyapeng has quit IRC | 09:44 | |
| sambetts | exit | 09:58 |
| dtantsur | sambetts: NO EXIT | 09:59 |
| *** yangyapeng has joined #openstack-ironic | 10:02 | |
| milano | morning sambetts! :D | 10:02 |
| milano | exit before enter?! | 10:03 |
| *** yangyapeng has quit IRC | 10:03 | |
| *** yangyapeng has joined #openstack-ironic | 10:03 | |
| *** yangyape_ has quit IRC | 10:06 | |
| *** yangyapeng has quit IRC | 10:08 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic-python-agent master: Add default configuration files to data_files https://review.openstack.org/509052 | 10:09 |
| *** spectr has quit IRC | 10:10 | |
| *** spectr has joined #openstack-ironic | 10:10 | |
| sambetts | dtantsur, milano o/ | 10:24 |
| dtantsur | o/ | 10:24 |
| vdrok | good morning sambetts | 10:28 |
| sambetts | hey vdrok | 10:29 |
| stendulker | Good morning sambetts, milano :) | 10:29 |
| milano | morning stendulker! :) | 10:29 |
| *** racedo has quit IRC | 10:37 | |
| *** e0ne has joined #openstack-ironic | 10:40 | |
| *** racedo has joined #openstack-ironic | 10:40 | |
| *** racedo has quit IRC | 10:41 | |
| *** jaianshu__ has joined #openstack-ironic | 10:41 | |
| *** jaianshu_ has quit IRC | 10:45 | |
| *** rbartal has joined #openstack-ironic | 10:46 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Generate version_id upon add_node https://review.openstack.org/507559 | 10:46 |
| openstackgerrit | Merged openstack/ironic-inspector master: refer to 'openstack baremetal' CLI in docs https://review.openstack.org/514462 | 10:46 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent master: Move legacy jobs into tree https://review.openstack.org/513380 | 10:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent master: Publish tiny-instacen-uec image https://review.openstack.org/514384 | 10:49 |
| *** udesale has quit IRC | 10:53 | |
| *** dtantsur is now known as dtantsur|brb | 10:53 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-lib master: Move legacy ironic-lib jobs in-tree https://review.openstack.org/514601 | 10:55 |
| *** yolanda has quit IRC | 10:55 | |
| *** rbartal has quit IRC | 10:58 | |
| *** zhangfei has quit IRC | 10:59 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-lib master: Move legacy ironic-lib jobs in-tree https://review.openstack.org/514601 | 11:01 |
| *** vsaienk0 has quit IRC | 11:01 | |
| *** parlos has quit IRC | 11:03 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic master: Add additional capabilities discovery for iRMC driver https://review.openstack.org/417347 | 11:06 |
| openstackgerrit | Merged openstack/python-ironic-inspector-client master: Remove deprecated ironic_inspector_client.client https://review.openstack.org/513735 | 11:09 |
| nicodemos | good morning, ironic'ers. | 11:13 |
| *** fellypefca has joined #openstack-ironic | 11:14 | |
| *** lucasagomes is now known as lucas-hungry | 11:14 | |
| *** stendulker_ has joined #openstack-ironic | 11:14 | |
| vdrok | dtantsur|brb: regarding the specs repo py27 job, this might be the reason https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/zuul.d/jobs.yaml#L237 | 11:17 |
| vdrok | morning nicodemos | 11:17 |
| *** stendulker has quit IRC | 11:18 | |
| nicodemos | vdrok o/ | 11:18 |
| *** stendulker_ has quit IRC | 11:20 | |
| *** phuongnh has quit IRC | 11:21 | |
| *** racedo has joined #openstack-ironic | 11:24 | |
| *** amarao1 has joined #openstack-ironic | 11:24 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: [DNM] Test py27 job https://review.openstack.org/514611 | 11:28 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: [DNM] Test py27 job https://review.openstack.org/514611 | 11:29 |
| *** vsaienk0 has joined #openstack-ironic | 11:31 | |
| *** yangyapeng has joined #openstack-ironic | 11:32 | |
| *** yangyapeng has quit IRC | 11:33 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Unittest node_info is added with a version_id https://review.openstack.org/514613 | 11:33 |
| *** yangyapeng has joined #openstack-ironic | 11:33 | |
| milano | aarefiev, ^ the follow-up | 11:34 |
| *** fragatina has quit IRC | 11:35 | |
| *** racedo has quit IRC | 11:36 | |
| *** fragatina has joined #openstack-ironic | 11:36 | |
| *** zshi__ has joined #openstack-ironic | 11:37 | |
| fellypefca | Good morning Ironic! :D | 11:39 |
| *** milano is now known as milan0 | 11:40 | |
| milan0 | morning fellypefca! :) | 11:40 |
| fellypefca | milan0 o/ | 11:41 |
| *** adisky_ has quit IRC | 11:43 | |
| *** udesale has joined #openstack-ironic | 11:46 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic-specs master: Nova-compatible console log for Ironic https://review.openstack.org/453627 | 11:50 |
| openstackgerrit | Sam Betts proposed openstack/ironic stable/pike: [DNM] Testing Cisco Third Party CI https://review.openstack.org/514619 | 11:53 |
| *** mrch has joined #openstack-ironic | 11:53 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-lib master: Move legacy ironic-lib jobs in-tree https://review.openstack.org/514601 | 11:55 |
| *** milan0 is now known as milan | 11:58 | |
| *** litao__ has quit IRC | 11:58 | |
| vdrok | morning fellypefca | 12:00 |
| fellypefca | hey vdrok :) | 12:00 |
| *** efried has joined #openstack-ironic | 12:02 | |
| *** appu103 has joined #openstack-ironic | 12:04 | |
| *** trown|outtypewww has quit IRC | 12:04 | |
| *** trown has joined #openstack-ironic | 12:05 | |
| *** aparnav has quit IRC | 12:05 | |
| *** jlabarre has joined #openstack-ironic | 12:05 | |
| *** hoonetorg has quit IRC | 12:06 | |
| *** trown|brb has joined #openstack-ironic | 12:07 | |
| *** trown|brb has quit IRC | 12:07 | |
| *** gcb has joined #openstack-ironic | 12:07 | |
| *** hoonetorg has joined #openstack-ironic | 12:08 | |
| *** appu103 has quit IRC | 12:09 | |
| *** jaganathan has quit IRC | 12:12 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Add custom py27 job for zuulv3 https://review.openstack.org/514626 | 12:14 |
| *** lucas-hungry is now known as lucasagomes | 12:15 | |
| *** dprince has joined #openstack-ironic | 12:16 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironic-inspector-client master: Remove deprecated setting ipmi creds feature https://review.openstack.org/513768 | 12:16 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironic-inspector-client master: Remove deprecated setting ipmi creds feature https://review.openstack.org/513768 | 12:20 |
| *** bfournie has quit IRC | 12:22 | |
| *** bfournie has joined #openstack-ironic | 12:22 | |
| vdrok | dtantsur|brb: I think we'll need https://review.openstack.org/514626 for ironic-specs | 12:23 |
| patchbot | patch 514626 - ironic-specs - Add custom py27 job for zuulv3 | 12:23 |
| efried | Howdy folks. Not sure if this is the right place or if I should be asking in -nova, but: I've been led to understand that ironic can't see the flavor while it's deploying an instance? | 12:25 |
| *** bfournie has quit IRC | 12:26 | |
| sambetts | efried: ironic can't see the flavor as it appears to nova, but certain parts of the flavor are passed to ironic in the instance_info field, like the root_gb | 12:27 |
| efried | sambetts Passed from where to where? Like from ironic's virt driver to... whatever's really doing the deploy? | 12:27 |
| sambetts | ironic's virt driver does an ironic node update and populates an ironic node's instance_info field with a few fields pulled from the flavor https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L59 | 12:29 |
| efried | sambetts Nyaha, okay. And extra_specs isn't included presumably because it's totally freeform, and ironic needs the format nailed down so it knows how to interpret it on the other side? | 12:32 |
| *** milan is now known as milan_ | 12:33 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic-specs master: Hardware interface for BIOS configuration https://review.openstack.org/496481 | 12:33 |
| sambetts | efried: we use some things in extraspecs, if they allign with a certain format https://github.com/openstack/nova/blob/master/nova/virt/ironic/patcher.py#L98 | 12:34 |
| sambetts | efried: so it allows us to specify boot mode net or local etc | 12:34 |
| sambetts | on the flavor | 12:34 |
| efried | sambetts Sweet, thanks for the help. | 12:34 |
| efried | sambetts FYI, the debate here is johnthetubaguy's spec on using placement traits. | 12:35 |
| *** milan_ is now known as milan | 12:35 | |
| sambetts | efried: yeah, I saw the thread, although I've been a little distracted by CI work to keep on top of it | 12:36 |
| efried | sambetts At the moment it looks like he's trying to overload traits as a way to pass information about settings to ironic. The placement folks are pretty likely to balk at that. | 12:36 |
| *** ankkumar has quit IRC | 12:36 | |
| sambetts | efried: yeah, I saw Jay's comments | 12:36 |
| johnthetubaguy | so this is what we discussed at the PTG | 12:36 |
| efried | sambetts But if we could add a clause to that get_deploy_patch you pointed to, the tunables could be passed through the flavor. | 12:37 |
| sambetts | so traits are purly for sechduling onto a node that has the capability and then extra specs is used to turn it on? | 12:37 |
| efried | sambetts Yes | 12:38 |
| sambetts | I think that fits similar to what we have now with things like boot mode | 12:38 |
| sambetts | /boot_option | 12:38 |
| johnthetubaguy | hmm... but how do we say which hosts can do a thing? | 12:38 |
| *** dtantsur|brb is now known as dtantsur | 12:38 | |
| sambetts | the trait specifys what a host can do | 12:38 |
| sambetts | by existance | 12:38 |
| efried | johnthetubaguy Your other spec. You can mark up nodes with whatever traits represent what they *can* do. | 12:39 |
| sambetts | if a host has X trait it means its possible for that host to do that | 12:39 |
| sambetts | but doesn't say if it is going to do that thing | 12:39 |
| efried | johnthetubaguy Then the flavor has two things: 1) the required trait - that's what gets you scheduled to the node; and 2) a separate setting that says you want that feature switched on. | 12:39 |
| dtantsur | efried: "use traits" was what we heard from nova folks on the PTG, sooo... | 12:39 |
| dtantsur | I'm open to any ideas, as long as we're not stuck is "no you can't do anything" situation we've been in for last 2 years :) | 12:40 |
| johnthetubaguy | efried: hmm, I guess I don't understand why we need that distinction | 12:40 |
| johnthetubaguy | efried: sounds like extra confusing flags for operators to me | 12:40 |
| efried | johnthetubaguy: alex_xu described it well | 12:40 |
| sambetts | e.g. a node would have two traits, BOOT_OPTION_LOCAL and BOOT_OPTION_NET and then the flavor would have REQUIRES TRAIT BOOT_OPTION_NET and extra_spec you would have boot_option=net | 12:40 |
| dtantsur | efried: I agree with johnthetubaguy, the distinction does not make practical sense | 12:40 |
| johnthetubaguy | then I hate the CUSTOM_ thing as it is, because that isn't practical either :( | 12:41 |
| efried | sambetts ++ | 12:41 |
| dtantsur | sambetts: that will probably not work, as traits are matched before ironic will process extra_specs | 12:41 |
| *** rloo has joined #openstack-ironic | 12:41 | |
| dtantsur | edleafe: ^^^ | 12:41 |
| sambetts | dtantsur: schduling doesn't care about extra_sepcs | 12:41 |
| sambetts | thats the point | 12:41 |
| efried | dtantsur That's why you use the extra setting. | 12:41 |
| johnthetubaguy | I am guess I am missing a conversation somewhere :S | 12:41 |
| dtantsur | efried: how do I mean BOOT_OPTION_NET, if it's not set yet | 12:41 |
| dtantsur | s/mean/match/ ugh | 12:42 |
| * milan relocating | 12:42 | |
| sambetts | the node is capable of both, so you can scheudle onto it | 12:42 |
| *** milan has quit IRC | 12:42 | |
| * alex_xu try to figure out what happened | 12:42 | |
| dtantsur | sambetts: and if you miss extra_specs, your scheduling is wrong? | 12:42 |
| sambetts | nope | 12:42 |
| dtantsur | well, that's what you wrote :) | 12:42 |
| sambetts | its not wrong | 12:42 |
| efried | dtantsur By marking your node with BOOT_OPTION_NET and BOOT_OPTION_WHATEVER_ELSE, you're telling placement that the node is *capable* of booting in those ways. | 12:42 |
| dtantsur | "capable of UEFI" != "uses UEFI" | 12:42 |
| efried | dtantsur Then your flavor says "I need a node that is capable of BOOT_OPTION_NET". | 12:43 |
| sambetts | no, but at scheduling you only care about capable of UEFI | 12:43 |
| dtantsur | I mean, what's the benefit for operators here? you're talking from purity point of view | 12:43 |
| sambetts | not whether it uses it | 12:43 |
| dtantsur | but that will hurt operators who will have to understand one more magical action on flavors | 12:43 |
| efried | dtantsur Then your extra spec says "boot from net, with these IP parameters" | 12:43 |
| efried | dtantsur How would you send the IP params using traits? | 12:44 |
| dtantsur | let's stick with UEFI or boot mode example please | 12:44 |
| johnthetubaguy | ... so I think I get the point from alex_xu | 12:44 |
| dtantsur | we're not doing IP params this way | 12:44 |
| johnthetubaguy | like efried just said its about passing parameters | 12:44 |
| johnthetubaguy | the point of all this was ironic stores and validates the parameters in advance | 12:44 |
| sambetts | dtantsur: given your example with UEFI, a some nodes would have two traits, BOOT_MODE_BIOS and BOOT_MODE_UEFI, and some would have one trait, BOOT_MODE_UEFI | 12:45 |
| sambetts | that defines what they are cabable of doing | 12:45 |
| dtantsur | right. why isn't it enough for us? | 12:45 |
| sambetts | in the flavor I define, REQUIRES TRAIT BOOT_MODE_UEFI | 12:45 |
| dtantsur | okay, okay, this is correct | 12:45 |
| sambetts | that schdules my flavor onto a node that has UEFI | 12:45 |
| sambetts | then in my extra specs I say boot_mode=UEFI | 12:46 |
| dtantsur | define "has" please | 12:46 |
| efried | For a binary option like that, dtantsur, it might be possible to overload a trait that way. But the same won't be true for all possible configurables. | 12:46 |
| dtantsur | okay, here you lost me. what use extra specs for redundant information | 12:46 |
| sambetts | its not redundent | 12:46 |
| dtantsur | efried: previously the nova team said firm "no" to passing non-binary information, essentially | 12:46 |
| sambetts | the sechudling can land the instance on a node that has the capabilit to do both bios and uefi | 12:46 |
| dtantsur | sambetts: it is, we have the request to boot in UEFI | 12:46 |
| efried | dtantsur Via placement, that's true. | 12:46 |
| sambetts | so the extra_spec is there to define which we want | 12:47 |
| dtantsur | sambetts: again, why have the *2nd* place? | 12:47 |
| johnthetubaguy | so I think there are a few things going on all at once here | 12:47 |
| dtantsur | you already have the trait | 12:47 |
| efried | The UEFI example is a wrinkly one because it *is* a binary option. | 12:47 |
| dtantsur | efried: as I mentioned above, we only talk about binary options, because all other options were discussed and rejected previously | 12:47 |
| johnthetubaguy | so we could combine things here, I am just not sure how it helps | 12:48 |
| johnthetubaguy | could we have a quick (ish) hangout, text is failing me here? | 12:48 |
| johnthetubaguy | like a 10 min hangout | 12:48 |
| efried | If you wanted, you could have the virt driver's get_deploy_patch (that sambetts pointed at earlier) examine the flavor for specific required traits and use that to send the information down to ironic. | 12:48 |
| dtantsur | johnthetubaguy: we already have one being planned. do you want a preliminary one? :) | 12:49 |
| efried | dtantsur is setting one of those up, right? | 12:49 |
| johnthetubaguy | efried: thats what it currently does | 12:49 |
| johnthetubaguy | dtantsur: yeah, preliminary one | 12:49 |
| rloo | ++, you're already discussing, so you might as well do it in a more effective way, at least for johnthetubaguy | 12:49 |
| johnthetubaguy | as in rather than typing here, lets jump into a hangout for 10 mins? | 12:49 |
| rloo | hi and good morning all :) | 12:49 |
| dtantsur | morning rloo | 12:49 |
| * efried waves at rloo | 12:49 | |
| dtantsur | efried, johnthetubaguy, okay, let's have an early pre-call call :) | 12:50 |
| johnthetubaguy | https://hangouts.google.com/call/xJfU84R-XzQrY8zLjZtlAAEI | 12:50 |
| dtantsur | lemme find my chromium... | 12:50 |
| rloo | just don't forget to summarize :) /me stays to hold the fort... | 12:50 |
| sambetts | johnthetubaguy: can't hear you | 12:51 |
| johnthetubaguy | OK | 12:51 |
| *** yolanda has joined #openstack-ironic | 12:56 | |
| *** rbartal has joined #openstack-ironic | 13:00 | |
| *** fragatina has quit IRC | 13:03 | |
| *** fragatina has joined #openstack-ironic | 13:03 | |
| *** jaianshu__ has quit IRC | 13:05 | |
| *** racedo has joined #openstack-ironic | 13:07 | |
| johnthetubaguy | efried: this patch: https://review.openstack.org/#/c/508116/ | 13:08 |
| patchbot | patch 508116 - nova - WIP: Send traits to ironic on server boot | 13:08 |
| *** rbartal has quit IRC | 13:09 | |
| *** jcoufal has joined #openstack-ironic | 13:11 | |
| *** athomas has joined #openstack-ironic | 13:12 | |
| *** cdearborn_ has joined #openstack-ironic | 13:12 | |
| *** rbudden has joined #openstack-ironic | 13:12 | |
| *** milan has joined #openstack-ironic | 13:13 | |
| *** cdearborn__ has joined #openstack-ironic | 13:13 | |
| *** bfournie has joined #openstack-ironic | 13:14 | |
| *** mjturek has joined #openstack-ironic | 13:15 | |
| dtantsur | vdrok: re https://review.openstack.org/#/c/514626/, is it possible to somehow change the parent job? | 13:16 |
| patchbot | patch 514626 - ironic-specs - Add custom py27 job for zuulv3 | 13:16 |
| dtantsur | I got surprised for a second to see two jobs :) | 13:16 |
| *** cdearborn_ has quit IRC | 13:16 | |
| vdrok | We can remove the standard py27 from project config | 13:17 |
| vdrok | dtantsur: after me merge that one. Standard was run only because the change was to yaml | 13:17 |
| dtantsur | k | 13:18 |
| dtantsur | rloo: ugh.. --os-baremetal-api-version=1 is a mess of two bugs, I think >_< | 13:20 |
| rloo | dtantsur: quite possibly. do you know what it *should* do? | 13:20 |
| rloo | dtantsur: we should separate it out into its own bug then? | 13:20 |
| openstackgerrit | Merged openstack/ironic master: Add a timeout for powering on/off a node on HPE OneView Driver https://review.openstack.org/510963 | 13:20 |
| dtantsur | rloo: yes, and fix it first, I guess.. | 13:21 |
| rloo | dtantsur: i don't think we need to fix that for the imminent client release? | 13:21 |
| *** baoli has joined #openstack-ironic | 13:21 | |
| dtantsur | rloo: well, I want to understand what version we end up with, when we fix it | 13:21 |
| dtantsur | like, if we fix it in the next release, we may end up with 1 == minimum | 13:22 |
| dtantsur | and then we'll have to do another round of deprecations and breaking changes | 13:22 |
| rloo | dtantsur: oh. i figure if we 'fix' it, we should just make it do what it should do | 13:22 |
| rloo | dtantsur: i mean, did it ever work? | 13:22 |
| dtantsur | I'm not sure. it was supposed to work | 13:22 |
| dtantsur | lemme stare at the code for some time | 13:23 |
| *** rbartal has joined #openstack-ironic | 13:24 | |
| *** milan has left #openstack-ironic | 13:25 | |
| *** _milan_ has joined #openstack-ironic | 13:25 | |
| *** _milan_ has quit IRC | 13:25 | |
| *** milan has joined #openstack-ironic | 13:26 | |
| *** milan has left #openstack-ironic | 13:29 | |
| dtantsur | aha! | 13:29 |
| *** parlos has joined #openstack-ironic | 13:31 | |
| openstackgerrit | John Garbutt proposed openstack/ironic-specs master: Config Template traits https://review.openstack.org/504952 | 13:31 |
| dtantsur | rloo: this is the bug: https://bugs.launchpad.net/python-ironicclient/+bug/1726870 | 13:32 |
| openstack | Launchpad bug 1726870 in python-ironicclient "Fallback to root endpoint (when negotiating an API version) does not work" [Medium,Triaged] - Assigned to Dmitry Tantsur (divius) | 13:32 |
| dtantsur | it's quite trivial to fix fwiw | 13:33 |
| rloo | dtantsur: ok, so what's your plan? fix it to do 1.9?, then change it to do 1.<negotiated or I forgot what> ? | 13:33 |
| rloo | dtantsur: if it never worked, we don't need to fix it to do 1.9. | 13:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: [WIP] Pass missing arguments to session in SessionClient._make_session_request https://review.openstack.org/514673 | 13:36 |
| dtantsur | rloo: this is the fix, it's not related to OSC code ^^^ | 13:36 |
| dtantsur | (lacks unit tests and reno) | 13:36 |
| rloo | dtantsur: ok, if you say so, that's code that I'm not familiar with :) | 13:37 |
| dtantsur | rloo: with that in place, making --os-baremetal-api-version=1 work is 2 lines of code :) | 13:39 |
| rloo | dtantsur: what does 'work' mean? the way we want it to work? | 13:39 |
| *** racedo has quit IRC | 13:39 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 13:41 |
| rloo | dtantsur: 'work' == using the latest compatible minor version? | 13:41 |
| nicodemos | dtantsur: ^^^ :) | 13:42 |
| rloo | dtantsur: anyway, just keep piling the patches to be reviewed :) | 13:42 |
| *** racedo has joined #openstack-ironic | 13:43 | |
| TheJulia | Good morning | 13:43 |
| *** rbartal has quit IRC | 13:44 | |
| rloo | Good morning TheJulia, hope you're feeling a lot better today! | 13:44 |
| nicodemos | morning, TheJulia. | 13:45 |
| *** cdearborn__ is now known as cdearborn | 13:45 | |
| dtantsur | rloo: yes | 13:45 |
| dtantsur | morning TheJulia | 13:45 |
| *** MattMan has quit IRC | 13:46 | |
| *** MattMan has joined #openstack-ironic | 13:46 | |
| *** AlexeyAbashkin has quit IRC | 13:48 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Pass missing arguments to session in SessionClient._make_session_request https://review.openstack.org/514673 | 13:48 |
| TheJulia | rloo: some ways better, but I've not gotten my day started. | 13:50 |
| *** AlexeyAbashkin has joined #openstack-ironic | 13:50 | |
| *** markguz_ has joined #openstack-ironic | 13:50 | |
| TheJulia | The cats got me up.... | 13:50 |
| rloo | TheJulia: those pesky cats! Mine almost pushed me out of my bed last night. | 13:51 |
| dtantsur | heh | 13:56 |
| *** milan has joined #openstack-ironic | 14:00 | |
| * TheJulia starts the espresso machine | 14:04 | |
| TheJulia | I actually feel a little better today which is \o/ | 14:04 |
| milan | TheJulia, morning! :) | 14:05 |
| openstackgerrit | Merged openstack/ironic master: Update validating node information docs https://review.openstack.org/513231 | 14:06 |
| rloo | TheJulia: ) | 14:11 |
| rloo | hi milan | 14:11 |
| milan | morning rloo! :) | 14:11 |
| dtantsur | TheJulia: \o/ | 14:15 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Set the default API version of OSC CLI to "latest" https://review.openstack.org/512989 | 14:15 |
| dtantsur | rloo: lemme know what you think about this release note ^^^ | 14:15 |
| rloo | dtantsur: what do you mean by the Bare Metal API v2 stuff? | 14:18 |
| dtantsur | rloo: sorry, what exactly is not clear? | 14:18 |
| rloo | dtantsur: is that something we're going to do soon? I'm not sure why you want to mention it. | 14:18 |
| dtantsur | rloo: how else would you explain the difference between 1 and latest? | 14:18 |
| rloo | dtantsur: we're v1 now. | 14:18 |
| rloo | dtantsur: when are we going to v2? didn't think we had talked about it. | 14:19 |
| dtantsur | rloo: we're not, and this is pretty clear from the release note | 14:19 |
| rloo | dtantsur: then why mention 'However, if we ever introduce...' | 14:19 |
| dtantsur | rloo: how else would you explain the difference between 1 and latest? | 14:19 |
| rloo | dtantsur: I think the first sentence is sufficient. | 14:20 |
| rloo | dtantsur: i think the rest of it causes confusion. | 14:21 |
| dtantsur | rloo: the first sentence does not explain the difference :) | 14:21 |
| dtantsur | it says they're the same | 14:21 |
| rloo | it sez 'currently'. | 14:21 |
| rloo | dtantsur: ok, let me suggest something... | 14:21 |
| dtantsur | well, maybe it's fine. yes please! | 14:21 |
| *** udesale has quit IRC | 14:21 | |
| *** zshi__ has quit IRC | 14:22 | |
| dtantsur | anyone else wants to vote on the doodle re ironic-nova traits discussion? I'd like to wrap it up before I leave for a day today | 14:26 |
| rloo | dtantsur: just wondering when you were thinking of an ironic release, cuz this bumps the api version, wondering if we'd want a corresponding change to client: https://review.openstack.org/#/c/501761/ | 14:30 |
| patchbot | patch 501761 - ironic - Add ability to provide configdrive when rebuilding | 14:30 |
| dtantsur | rloo: we don't have any link between ironicclient and ironic releases, except for release series. so I'm fine doing it later, as long as it goes in Queens. | 14:31 |
| dtantsur | do we even have a patch up for that? | 14:31 |
| rloo | dtantsur: i don't think so :) | 14:33 |
| dtantsur | heh, then we don't have an option of quickly merging it :) | 14:33 |
| rloo | dtantsur: ok, so i'll put it on my list to review unless it merges before then (i can only hope). | 14:33 |
| TheJulia | heh | 14:33 |
| rloo | dtantsur: it should be very easy client patch. | 14:33 |
| dtantsur | yeah | 14:33 |
| rloo | dtantsur: but we can't put it in a released client before we release ironic. i think. | 14:34 |
| dtantsur | we never had such rule, I htink | 14:34 |
| dtantsur | anyway, that's not going to happen, apparently :) | 14:35 |
| *** hamzy has quit IRC | 14:35 | |
| rloo | dtantsur: no such rule, but to release a client that supports a server that hasn't been released, would seem odd. i think. | 14:35 |
| dtantsur | yep | 14:35 |
| dtantsur | rloo: hah, you changed the leading time slot :) | 14:37 |
| rloo | dtantsur: darn. let me look. | 14:38 |
| *** racedo has quit IRC | 14:38 | |
| dtantsur | rloo: it's fine, we had a tie between Thu and Mon | 14:38 |
| rloo | dtantsur: ah, looks ok still. i want to make sure the key players can attend (vs number of folks) | 14:39 |
| dtantsur | yeah, next Mon works for everyone, except for Shiv, I think | 14:40 |
| rloo | dtantsur: i'm ok if you all meet on Thurs too. Am wondering if sooner might be better. | 14:40 |
| dtantsur | johnthetubaguy, efried, opinions on ^^^? | 14:40 |
| rloo | dtantsur: well, the Thur timeslot didn't work for shiv either. | 14:40 |
| dtantsur | yep | 14:40 |
| dtantsur | we had a preliminary discussion, so we don't have to hurry IMO | 14:41 |
| rloo | dtantsur: ok, whatever you decide | 14:41 |
| rloo | but goes to show, the power of 1 vote!!! | 14:41 |
| dtantsur | :) | 14:41 |
| *** Nisha_Agarwal has joined #openstack-ironic | 14:43 | |
| efried | dtantsur I'm flexible | 14:43 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Set the default API version of OSC CLI to "latest" https://review.openstack.org/512989 | 14:43 |
| dtantsur | rloo: with your suggestion (thanks!) ^^^ | 14:43 |
| rloo | dtantsur: yw, that's what reviewers are for! | 14:44 |
| dtantsur | TIL! | 14:46 |
| dtantsur | :D | 14:46 |
| *** parlos has quit IRC | 14:46 | |
| dtantsur | could anyone please review https://review.openstack.org/#/c/514566/ as well? I'd like an inspector release | 14:49 |
| patchbot | patch 514566 - ironic-inspector - Clean up release notes before a release | 14:49 |
| *** AlexeyAbashkin has quit IRC | 14:51 | |
| dtantsur | sambetts: ^^^? | 14:52 |
| *** racedo has joined #openstack-ironic | 14:53 | |
| *** Nisha_Agarwal has quit IRC | 14:53 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 14:59 | |
| *** spectr has quit IRC | 15:00 | |
| *** ccamacho has quit IRC | 15:02 | |
| *** mjura has quit IRC | 15:02 | |
| *** AlexeyAbashkin has quit IRC | 15:10 | |
| jlvillal | Good morning Ironic :) | 15:16 |
| NobodyCam | Good morning Ironic folks | 15:17 |
| NobodyCam | hey hey jlvillal Good morning :) | 15:17 |
| vdrok | good morning rloo TheJulia jlvillal and NobodyCam | 15:18 |
| dtantsur | morning jlvillal, NobodyCam | 15:18 |
| jlvillal | :) | 15:18 |
| NobodyCam | morning vdrok and dtantsur :) | 15:18 |
| rloo | morning vdrok, NobodyCam, jlvillal! | 15:18 |
| NobodyCam | morning rloo :) | 15:18 |
| gmonteiro | good morning everyone o/ | 15:18 |
| NobodyCam | good morning gmonteiro :) | 15:18 |
| vdrok | sambetts: I think we can not do that in our case http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul.d/projects.yaml#n14683 | 15:19 |
| vdrok | as we need to override, not add new fields | 15:19 |
| vdrok | and AJaeger said that the lists are merged | 15:19 |
| vdrok | like if in base job there are keys a and b and in child - only b, the resulting dict still contains both a and b | 15:20 |
| vdrok | good morning gmonteiro | 15:20 |
| vdrok | rloo: FYI ^^ | 15:20 |
| vdrok | although I think the best way would be indeed to add that check as part of docs-build job | 15:21 |
| *** hamzy has joined #openstack-ironic | 15:21 | |
| rloo | vdrok: ++ better to clean up our stuff, vs mucking with the zuul jobs stuff. i think it'd be easier to maintain/understand in the long run. consistency etc :) | 15:22 |
| rloo | vdrok: so either add to docs build, or better? to add a separate doc8 thing. it is somewhat similar to our pep8 thing. | 15:23 |
| *** ccamacho has joined #openstack-ironic | 15:23 | |
| vdrok | rloo: yeah. I'll take a look at it today and come up with something | 15:23 |
| rloo | vdrok: thx. i don't know how a new thing would xlate to being run in gate though, but i'm sure you (will) know. it is all magic to me :) | 15:24 |
| vdrok | rloo: I (hopefully) will :) | 15:24 |
| *** athomas has quit IRC | 15:25 | |
| TheJulia | good morning NobodyCam vdrok | 15:25 |
| NobodyCam | good morning TheJulia :) | 15:25 |
| TheJulia | dtantsur: inspector reno update reviewed and on its way | 15:27 |
| dtantsur | thankd! | 15:27 |
| *** nicodemos has quit IRC | 15:27 | |
| TheJulia | woot! my local dev VM is finally working again | 15:29 |
| *** rpioso|afk is now known as rpioso | 15:30 | |
| *** Goneri has quit IRC | 15:34 | |
| johnthetubaguy | dtantsur: would be great to have rloo in the discussion, particular given all the review cycles spent brushing up the spec. | 15:37 |
| dtantsur | yep, so probably next Monday | 15:37 |
| rloo | johnthetubaguy: i think that first spec is ok regardless of what is discussed, but maybe i'm wrong, i still have to look at latest revision :) | 15:38 |
| johnthetubaguy | rloo: that's my hope, I tried to put the non-controversial stuff in that one, although the CLI interface might count as controversial now :) | 15:38 |
| rloo | johnthetubaguy: ha ha | 15:39 |
| openstackgerrit | Merged openstack/ironic-inspector master: Clean up release notes before a release https://review.openstack.org/514566 | 15:39 |
| *** ijw has joined #openstack-ironic | 15:39 | |
| *** e0ne has quit IRC | 15:43 | |
| *** marios has quit IRC | 15:48 | |
| *** parlos has joined #openstack-ironic | 15:50 | |
| *** ccamacho has quit IRC | 15:51 | |
| *** yolanda has quit IRC | 15:54 | |
| *** exodusftw has quit IRC | 15:59 | |
| TheJulia | dtantsur: yeouch that is going to be an early call for me :( | 16:02 |
| TheJulia | dtantsur: on a plus side, I think we are on the same page presently | 16:02 |
| dtantsur | TheJulia: sorry.. well, it's 4pm here, the overlap is not huge | 16:03 |
| dtantsur | hmmm, lemme double-check the daylight actually | 16:03 |
| TheJulia | my calendar says 5 AM local... but that doesn't sound right.... | 16:03 |
| dtantsur | 2pm UTC cannot be 5 am for you... | 16:04 |
| dtantsur | but it seems that I should have said 3pm UTC, as we're back on Winter time that Monday | 16:05 |
| dtantsur | ugh | 16:05 |
| dtantsur | TheJulia: it's 11am eastern | 16:08 |
| TheJulia | I'm in Mountain right now | 16:08 |
| dtantsur | then probably 9am? | 16:08 |
| TheJulia | But yeah, that means my calendar is all messed up | 16:08 |
| *** exodusftw has joined #openstack-ironic | 16:08 | |
| TheJulia | yeah, should be 9am | 16:08 |
| dtantsur | ok :) | 16:10 |
| dtantsur | see you tomorrow, have to run home (in the local office today) | 16:10 |
| *** dtantsur is now known as dtantsur|afk | 16:10 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Unittest node_info is added with a version_id https://review.openstack.org/514613 | 16:20 |
| *** markguz__ has joined #openstack-ironic | 16:20 | |
| *** lucasagomes is now known as lucas-afk | 16:22 | |
| *** tesseract has quit IRC | 16:22 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 16:22 | |
| *** markguz_ has quit IRC | 16:23 | |
| *** fragatina has quit IRC | 16:25 | |
| *** pcaruana has quit IRC | 16:25 | |
| *** AlexeyAbashkin has quit IRC | 16:26 | |
| *** rama_y has joined #openstack-ironic | 16:32 | |
| *** amarao has quit IRC | 16:34 | |
| *** amarao1 has quit IRC | 16:35 | |
| *** parlos has quit IRC | 16:36 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:37 | |
| *** vsaienk0 has quit IRC | 16:39 | |
| *** vsaienk0 has joined #openstack-ironic | 16:43 | |
| *** racedo has quit IRC | 16:44 | |
| *** racedo has joined #openstack-ironic | 16:45 | |
| *** racedo has quit IRC | 16:45 | |
| *** nicodemos has joined #openstack-ironic | 16:48 | |
| *** trown is now known as trown|lunch | 16:50 | |
| *** derekh has quit IRC | 16:52 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 16:55 | |
| *** jtomasek has quit IRC | 16:56 | |
| *** milan has quit IRC | 16:57 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Run spec validation in pep8 job instead of py27 https://review.openstack.org/514626 | 16:57 |
| *** AlexeyAbashkin has quit IRC | 16:59 | |
| *** baoli has quit IRC | 17:00 | |
| *** sambetts is now known as sambetts|afk | 17:00 | |
| sambetts|afk | Night all | 17:00 |
| sambetts|afk | o/ | 17:00 |
| *** mrch has quit IRC | 17:01 | |
| vdrok | good night sambetts|afk | 17:01 |
| *** baoli has joined #openstack-ironic | 17:01 | |
| *** noam__ has joined #openstack-ironic | 17:07 | |
| *** vsaienk0 has quit IRC | 17:09 | |
| *** vsaienk0 has joined #openstack-ironic | 17:11 | |
| *** fragatina has joined #openstack-ironic | 17:11 | |
| TheJulia | Oh nice https://www.openstack.org/summit/sydney-2017/summit-schedule/events/20488/baremetal-scheduling was accepted | 17:13 |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Soft power operations for OneView hardware type https://review.openstack.org/510685 | 17:14 |
| *** vsaienk0 has quit IRC | 17:21 | |
| *** vsaienk0 has joined #openstack-ironic | 17:22 | |
| *** sasha2 has left #openstack-ironic | 17:22 | |
| fellypefca | TheJulia nice! :D | 17:24 |
| TheJulia | Seems my weird speaker shadow profile has re-appeared or something *shrugs* | 17:25 |
| *** ralonsoh has quit IRC | 17:32 | |
| vdrok | rloo: the main problem is, how to run unittests when rst changes. the default openstack-tox-py27 won't allow that | 17:36 |
| vdrok | so we either have to define a custom job, or to put the unittest run under some other env, not py27 | 17:36 |
| vdrok | I can move testr command to openstack-tox-linters | 17:37 |
| vdrok | and leave doc8 in the pep8 env | 17:37 |
| TheJulia | my $0.02 is that is reasonable | 17:37 |
| vdrok | linters job description says the following https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/zuul.d/jobs.yaml#L289 | 17:37 |
| vdrok | kinda fits? | 17:38 |
| * TheJulia will look in one minute | 17:44 | |
| TheJulia | okay, I can look now | 17:44 |
| TheJulia | slides != fun | 17:44 |
| TheJulia | yeah, it is linting... | 17:45 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Run spec validation in linters job instead of py27 https://review.openstack.org/514626 | 17:45 |
| * TheJulia wonders if we can just change py27 to 35 | 17:46 | |
| TheJulia | heh | 17:47 |
| TheJulia | I just read the commit message | 17:47 |
| *** trown|lunch is now known as trown | 17:50 | |
| *** dyasny__ has quit IRC | 17:50 | |
| *** dyasny__ has joined #openstack-ironic | 17:51 | |
| vdrok | rloo: TheJulia AJaeger says that running two jobs for that would be a waste of resources. switching back to pep8 only | 17:55 |
| rloo | vdrok: sigh. ok, sorry about making you do more work. | 17:56 |
| vdrok | rloo: although I wonder what fits better in our case, pep8 or linters? | 17:57 |
| vdrok | it's just a matter of a name | 17:57 |
| rloo | vdrok: pep8 is specific. so linters makes more sense. | 17:58 |
| vdrok | yup, ++ | 17:59 |
| rloo | pep8 == style guide for python code. and we don't have python code :) | 17:59 |
| rloo | unless we want to pep8 our unit test code :D | 17:59 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Run spec validation in linters job instead of py27 https://review.openstack.org/514626 | 18:01 |
| *** baoli has quit IRC | 18:02 | |
| *** baoli has joined #openstack-ironic | 18:03 | |
| *** baoli has quit IRC | 18:05 | |
| *** ijw has quit IRC | 18:08 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 18:12 |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Add validations for OneView ML2 driver https://review.openstack.org/508946 | 18:12 |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Get a new OneView client when needed https://review.openstack.org/510644 | 18:12 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 18:15 |
| *** vsaienk0 has quit IRC | 18:16 | |
| *** Nisha_Agarwal has quit IRC | 18:17 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 18:18 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Publish tiny-instance-uec image https://review.openstack.org/514384 | 18:18 |
| *** vsaienk0 has joined #openstack-ironic | 18:20 | |
| vdrok | rloo: seqdiag does not seem to support loops | 18:26 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add validations for OneView ML2 driver https://review.openstack.org/508946 | 18:26 |
| rloo | vdrok: bugger. | 18:26 |
| rloo | vdrok: can we draw an arrow/maybe diff from others, from ramdisk to conductor then, with 'deploy done' or whatever it is | 18:26 |
| rloo | vdrok: let me think/see that diagram again, sec... | 18:27 |
| vdrok | sure, no worries | 18:27 |
| TheJulia | hamzy: you around? | 18:27 |
| hamzy | yes | 18:28 |
| hamzy | did I break something? :) | 18:28 |
| *** baoli has joined #openstack-ironic | 18:29 | |
| rloo | vdrok: ok, so 'is deploy done' is asked with every heartbeat. so i think it might be ok to do 'Still working... until...' and then <--- ...deploy is done ? wdyt? | 18:29 |
| vdrok | rloo: or I can put ... in that block in the middle, and then add an entry node->conductor saying that deploy is done :) | 18:29 |
| rloo | vdrok: yeah, that :) | 18:30 |
| TheJulia | hamzy: anything worth mentioning regarding molteniron? | 18:30 |
| *** ChubYann has joined #openstack-ironic | 18:30 | |
| TheJulia | w/r/t project update | 18:30 |
| hamzy | currently stable... no new functionality planed | 18:31 |
| TheJulia | ok | 18:32 |
| *** baoli has quit IRC | 18:33 | |
| *** baoli has joined #openstack-ironic | 18:33 | |
| *** baoli has quit IRC | 18:33 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revise deploy process documentation https://review.openstack.org/458867 | 18:37 |
| *** priteau has quit IRC | 18:42 | |
| *** fellypefca has quit IRC | 18:43 | |
| *** baoli has joined #openstack-ironic | 18:43 | |
| TheJulia | sambetts|afk: w/r/t networking-baremetal, anything to bring up during the project update? | 18:53 |
| *** vsaienk0 has quit IRC | 18:55 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:55 | |
| vdrok | good night everyone! | 18:57 |
| vdrok | TheJulia: nothing new there afaik | 18:58 |
| TheJulia | vdrok: thanks! | 18:58 |
| TheJulia | vdrok: goodnight! | 18:58 |
| *** AlexeyAbashkin has quit IRC | 19:00 | |
| *** e0ne has joined #openstack-ironic | 19:03 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Migrate python-oneviewclient validations to Ironic OneView drivers https://review.openstack.org/468428 | 19:03 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add validations for OneView ML2 driver https://review.openstack.org/508946 | 19:04 |
| *** vsaienk0 has joined #openstack-ironic | 19:06 | |
| *** ijw_ has joined #openstack-ironic | 19:09 | |
| *** ijw_ has quit IRC | 19:09 | |
| *** ijw_ has joined #openstack-ironic | 19:09 | |
| openstackgerrit | Stenio Araujo proposed openstack/ironic master: Get a new OneView client when needed https://review.openstack.org/510644 | 19:13 |
| *** vsaienk0 has quit IRC | 19:16 | |
| *** vsaienk0 has joined #openstack-ironic | 19:20 | |
| *** ijw_ has quit IRC | 19:29 | |
| *** ijw has joined #openstack-ironic | 19:29 | |
| *** vsaienk0 has quit IRC | 19:30 | |
| *** vsaienk0 has joined #openstack-ironic | 19:36 | |
| *** ijw has quit IRC | 19:46 | |
| TheJulia | Okay, one slide deck... mostly done. | 19:53 |
| *** e0ne has quit IRC | 20:06 | |
| TheJulia | Anyone have any thoughts on things that would be good to convey during project on boarding. Such as things that you wish you knew when you first got started? | 20:06 |
| *** priteau has joined #openstack-ironic | 20:26 | |
| *** priteau has quit IRC | 20:27 | |
| *** jcoufal has quit IRC | 20:29 | |
| *** baoli has quit IRC | 20:32 | |
| *** dprince has quit IRC | 20:45 | |
| *** baoli has joined #openstack-ironic | 20:47 | |
| *** markguz__ has quit IRC | 21:09 | |
| *** vsaienk0 has quit IRC | 21:09 | |
| *** vsaienk0 has joined #openstack-ironic | 21:11 | |
| *** ijw has joined #openstack-ironic | 21:20 | |
| *** ijw has quit IRC | 21:25 | |
| * TheJulia ponders running the onboarding session as an interactive q&a | 21:34 | |
| *** vsaienk0 has quit IRC | 21:35 | |
| *** dyasny__ has quit IRC | 21:38 | |
| rloo | TheJulia: that makes sense | 21:38 |
| rloo | TheJulia: other thought -- what do the other projects do in their onboarding sessions? | 21:39 |
| *** gcb has quit IRC | 21:48 | |
| TheJulia | Seems to be a mix, some seem to do arch overviews, some do a bit of a dive into use and then field questions. | 21:51 |
| TheJulia | There was a post to the ml about 19 hours ago raising the question of what people might expect. Maybe this is a "do on the plane" presentation ;) | 21:52 |
| *** AlexeyAbashkin has joined #openstack-ironic | 21:56 | |
| *** AlexeyAbashkin has quit IRC | 22:01 | |
| rloo | TheJulia: yup. i think if we have docs that cover most/all of it, you can just pull them up :) (but, do we have docs...) | 22:04 |
| rloo | TheJulia: wrt docs, you might want to look at https://review.openstack.org/#/c/458867/ | 22:05 |
| patchbot | patch 458867 - ironic - Revise deploy process documentation | 22:05 |
| * rloo wonders if 'onboarding' == ironic 101... | 22:05 | |
| TheJulia | kind of yeah | 22:06 |
| TheJulia | I was thinking anything that requires digging low level means we need docs | 22:06 |
| rloo | TheJulia: as a 'user' or as a new developer/contributor? | 22:06 |
| TheJulia | developer/contributor | 22:06 |
| rloo | TheJulia: there's more than 1 hour (how long is this) of info here: https://docs.openstack.org/ironic/latest/contributor/index.html | 22:09 |
| rloo | TheJulia: and if they ask questions that aren't covered ^^, we should consider adding that info | 22:10 |
| TheJulia | maybe introduction and then "click" "click" "click" and answer questions | 22:11 |
| *** rama_y has quit IRC | 22:12 | |
| rloo | TheJulia: sounds good to me! but maybe I'm the wrong person to ask :) | 22:12 |
| rloo | jlvillal: would you like to +A this? https://review.openstack.org/#/c/513380/ | 22:12 |
| patchbot | patch 513380 - ironic-python-agent - Move legacy jobs into tree | 22:12 |
| *** bfournie has quit IRC | 22:12 | |
| rloo | jlvillal: although the topic is odd/wrong | 22:12 |
| jlvillal | rloo: I wouldn't mind one more person looking at it :) | 22:12 |
| rloo | jlvillal: darn! | 22:13 |
| rloo | jlvillal: i'll take a quick look then... | 22:13 |
| jlvillal | Thanks! | 22:13 |
| rloo | jlvillal: don't think i can do it quickly. | 22:17 |
| jlvillal | rloo: Yeah, it is kind of a monster sized patch... | 22:17 |
| jlvillal | rloo: I'm thinking someone can look tomorrow from Europe | 22:18 |
| rloo | jlvillal: yeah, makes my eyes go cross-eyed. | 22:18 |
| jlvillal | rloo: For the playbook files, you just want to make sure they are the same as in the something-zuul project. I forget the exact name | 22:19 |
| *** baoli has quit IRC | 22:19 | |
| rloo | jlvillal: i feel like someone could have written something to automate them (the moves to the projects). but anyway... | 22:20 |
| jlvillal | openstack-zuul-jobs | 22:20 |
| jlvillal | rloo: I think they did. I kind of wish they had published their scripts, so I could review the script... | 22:20 |
| rloo | jlvillal: it seems to me that one (set) of scripts should mostly work for all projects, not just ironic ones. | 22:21 |
| rloo | jlvillal: anyway, just a thought and probably too late... | 22:21 |
| jlvillal | rloo: Yep. Maybe they have some and I don't know about it. | 22:21 |
| *** ijw has joined #openstack-ironic | 22:22 | |
| *** cdearborn has quit IRC | 22:22 | |
| openstackgerrit | Merged openstack/ironic-specs master: Run spec validation in linters job instead of py27 https://review.openstack.org/514626 | 22:24 |
| *** ijw has quit IRC | 22:28 | |
| *** rama_y has joined #openstack-ironic | 22:39 | |
| *** Sukhdev has joined #openstack-ironic | 22:41 | |
| *** ijw has joined #openstack-ironic | 22:57 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Use db transactions when error-finish is called https://review.openstack.org/510928 | 22:57 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Make _abort set error and finished_at in fms_event https://review.openstack.org/514854 | 22:57 |
| *** charcol has joined #openstack-ironic | 23:02 | |
| *** ijw has quit IRC | 23:06 | |
| *** rama_y has quit IRC | 23:06 | |
| *** ijw has joined #openstack-ironic | 23:07 | |
| *** ijw has quit IRC | 23:18 | |
| *** ijw has joined #openstack-ironic | 23:19 | |
| *** mjturek has quit IRC | 23:27 | |
| *** weshay|ruck is now known as weshay|PTO | 23:38 | |
| *** bfournie has joined #openstack-ironic | 23:54 | |
| *** rbudden has quit IRC | 23:55 | |
| *** bfournie has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!