| openstackgerrit | Merged openstack/ironic-python-agent: Load IPMI modules before chroot in CoreOS image https://review.openstack.org/399987 | 00:04 |
|---|---|---|
| *** Guest92303 has joined #openstack-ironic | 00:12 | |
| *** Sukhdev has quit IRC | 00:16 | |
| *** Guest92303 has quit IRC | 00:17 | |
| *** rloo has quit IRC | 00:23 | |
| *** bfournie has quit IRC | 00:23 | |
| *** shobhar_ has joined #openstack-ironic | 00:40 | |
| *** tuanluong has joined #openstack-ironic | 00:43 | |
| *** ijw has quit IRC | 00:45 | |
| *** soliosg has quit IRC | 00:45 | |
| *** anush has quit IRC | 00:52 | |
| *** rama_y has joined #openstack-ironic | 00:54 | |
| *** yolanda has joined #openstack-ironic | 00:56 | |
| *** ijw has joined #openstack-ironic | 01:00 | |
| *** SerenaFeng has joined #openstack-ironic | 01:03 | |
| *** [1]rpioso has quit IRC | 01:07 | |
| *** penick has joined #openstack-ironic | 01:14 | |
| *** penick_ has joined #openstack-ironic | 01:15 | |
| *** penick has quit IRC | 01:19 | |
| *** penick_ is now known as penick | 01:19 | |
| *** Alagar has joined #openstack-ironic | 01:31 | |
| *** bfournie has joined #openstack-ironic | 01:32 | |
| *** shobhar_ has quit IRC | 01:38 | |
| *** anush has joined #openstack-ironic | 01:41 | |
| *** mtanino has joined #openstack-ironic | 01:43 | |
| *** gcb has joined #openstack-ironic | 01:51 | |
| *** yufei has joined #openstack-ironic | 01:52 | |
| openstackgerrit | octopuszhang proposed openstack/ironic-python-agent: Skip API releated work if no api url configured https://review.openstack.org/399857 | 01:55 |
| *** syed__ has quit IRC | 01:56 | |
| *** Goneri has quit IRC | 02:00 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Python 2 and 3 Compatibility https://review.openstack.org/400577 | 02:00 |
| *** snozhka|afk has quit IRC | 02:01 | |
| *** zhangjl has joined #openstack-ironic | 02:08 | |
| *** anush has quit IRC | 02:09 | |
| *** Goneri has joined #openstack-ironic | 02:10 | |
| *** anush has joined #openstack-ironic | 02:13 | |
| *** syed__ has joined #openstack-ironic | 02:17 | |
| *** cdearborn has joined #openstack-ironic | 02:25 | |
| *** SerenaFeng has quit IRC | 02:27 | |
| *** [2]cdearborn has joined #openstack-ironic | 02:30 | |
| *** shuquan has joined #openstack-ironic | 02:34 | |
| *** [4]cdearborn has joined #openstack-ironic | 02:35 | |
| *** anush has quit IRC | 02:39 | |
| *** [1]cdearborn has quit IRC | 02:39 | |
| *** shuquan_ has joined #openstack-ironic | 02:39 | |
| *** [1]cdearborn has joined #openstack-ironic | 02:39 | |
| *** shuquan has quit IRC | 02:40 | |
| *** SerenaFeng has joined #openstack-ironic | 02:43 | |
| *** cdearborn has quit IRC | 02:44 | |
| *** aNuposic has quit IRC | 02:44 | |
| *** cdearborn has joined #openstack-ironic | 02:48 | |
| *** [2]cdearborn has quit IRC | 02:50 | |
| *** [3]cdearborn has joined #openstack-ironic | 02:51 | |
| *** Alagar has quit IRC | 02:52 | |
| *** shuquan_ has quit IRC | 02:52 | |
| *** lindycoder has joined #openstack-ironic | 02:53 | |
| *** Alagar has joined #openstack-ironic | 02:53 | |
| *** fragatina has quit IRC | 02:53 | |
| *** shuquan has joined #openstack-ironic | 02:54 | |
| *** [4]cdearborn has quit IRC | 02:54 | |
| *** Alagar has quit IRC | 02:55 | |
| *** yufei has quit IRC | 02:57 | |
| *** yufei has joined #openstack-ironic | 02:58 | |
| *** Alagar has joined #openstack-ironic | 02:58 | |
| *** shuquan_ has joined #openstack-ironic | 02:59 | |
| *** shuquan has quit IRC | 02:59 | |
| *** shuquan_ has quit IRC | 03:04 | |
| *** shuquan has joined #openstack-ironic | 03:04 | |
| *** [1]cdearborn has quit IRC | 03:04 | |
| *** lindycoder has quit IRC | 03:05 | |
| *** cdearborn has quit IRC | 03:06 | |
| *** [3]cdearborn has quit IRC | 03:09 | |
| *** anush has joined #openstack-ironic | 03:24 | |
| *** marios has quit IRC | 03:26 | |
| *** marios has joined #openstack-ironic | 03:26 | |
| *** links has joined #openstack-ironic | 03:42 | |
| *** SerenaFeng has quit IRC | 03:47 | |
| *** yufei has quit IRC | 03:51 | |
| *** fragatina has joined #openstack-ironic | 03:51 | |
| *** fragatina has quit IRC | 03:52 | |
| *** fragatina has joined #openstack-ironic | 03:52 | |
| *** yufei has joined #openstack-ironic | 03:53 | |
| *** shuquan has quit IRC | 03:53 | |
| *** ijw has quit IRC | 03:56 | |
| *** shuquan has joined #openstack-ironic | 04:01 | |
| *** hw_wutianwei has joined #openstack-ironic | 04:02 | |
| *** Nisha has joined #openstack-ironic | 04:07 | |
| *** Nisha is now known as Guest11581 | 04:07 | |
| *** Guest11581 is now known as Nisha1 | 04:08 | |
| *** Nisha1 is now known as Nisha_ | 04:08 | |
| *** shuquan_ has joined #openstack-ironic | 04:09 | |
| *** shuquan has quit IRC | 04:10 | |
| *** Goneri has quit IRC | 04:11 | |
| *** rama_y has quit IRC | 04:16 | |
| *** aNuposic has joined #openstack-ironic | 04:16 | |
| *** anush has quit IRC | 04:18 | |
| *** yufei has quit IRC | 04:19 | |
| *** Alagar has quit IRC | 04:24 | |
| *** ijw has joined #openstack-ironic | 04:24 | |
| *** shuquan_ has quit IRC | 04:24 | |
| *** ijw has quit IRC | 04:29 | |
| *** Alagar has joined #openstack-ironic | 04:44 | |
| *** moshele has joined #openstack-ironic | 04:47 | |
| *** homeski has joined #openstack-ironic | 05:08 | |
| *** shuquan has joined #openstack-ironic | 05:09 | |
| *** moshele has quit IRC | 05:12 | |
| *** shuquan has quit IRC | 05:16 | |
| *** Alagar has quit IRC | 05:16 | |
| *** shuquan has joined #openstack-ironic | 05:18 | |
| *** Alagar has joined #openstack-ironic | 05:20 | |
| *** shuquan_ has joined #openstack-ironic | 05:29 | |
| *** vikrant has joined #openstack-ironic | 05:30 | |
| *** shuquan has quit IRC | 05:30 | |
| *** Nisha_ has quit IRC | 05:32 | |
| *** SerenaFeng has joined #openstack-ironic | 05:39 | |
| *** Nisha_ has joined #openstack-ironic | 05:44 | |
| *** Alagar has quit IRC | 05:52 | |
| *** Nisha_ has quit IRC | 05:53 | |
| *** moshele has joined #openstack-ironic | 05:56 | |
| *** Nisha_ has joined #openstack-ironic | 06:05 | |
| *** adu has joined #openstack-ironic | 06:06 | |
| *** snozhka has joined #openstack-ironic | 06:11 | |
| *** Alagar has joined #openstack-ironic | 06:12 | |
| *** yufei has joined #openstack-ironic | 06:13 | |
| *** shuquan_ has quit IRC | 06:13 | |
| *** shuquan has joined #openstack-ironic | 06:14 | |
| *** rbartal has joined #openstack-ironic | 06:16 | |
| *** Nisha_ is now known as Nisha | 06:39 | |
| *** Nisha is now known as Guest86010 | 06:40 | |
| *** karthiks has quit IRC | 06:42 | |
| *** shobhar has joined #openstack-ironic | 06:44 | |
| *** ChubYann has quit IRC | 06:47 | |
| *** karthiks has joined #openstack-ironic | 06:56 | |
| *** shuquan_ has joined #openstack-ironic | 07:00 | |
| *** shuquan has quit IRC | 07:01 | |
| *** mjura has joined #openstack-ironic | 07:03 | |
| *** mjura has quit IRC | 07:08 | |
| *** mjura has joined #openstack-ironic | 07:08 | |
| pas-ha | morning Ironic :) | 07:09 |
| *** adu has quit IRC | 07:11 | |
| *** tesseract has joined #openstack-ironic | 07:12 | |
| *** Alagar has quit IRC | 07:12 | |
| *** Alagar has joined #openstack-ironic | 07:12 | |
| *** tesseract is now known as Guest59528 | 07:13 | |
| *** shuquan_ has quit IRC | 07:16 | |
| *** ccamacho has joined #openstack-ironic | 07:17 | |
| *** SerenaFeng has quit IRC | 07:22 | |
| *** e0ne has joined #openstack-ironic | 07:24 | |
| *** SerenaFeng has joined #openstack-ironic | 07:25 | |
| *** shobhar has quit IRC | 07:26 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Use kvm for ironic VMs when possible https://review.openstack.org/400941 | 07:36 |
| *** snozhka has quit IRC | 07:42 | |
| *** e0ne has quit IRC | 07:45 | |
| *** snozhka has joined #openstack-ironic | 07:55 | |
| *** chihhsin has quit IRC | 08:00 | |
| *** syed__ has quit IRC | 08:06 | |
| *** ohamada has joined #openstack-ironic | 08:06 | |
| *** SerenaFeng has quit IRC | 08:14 | |
| *** ralonsoh has joined #openstack-ironic | 08:20 | |
| *** dtantsur|afk is now known as dtantsur | 08:27 | |
| *** e0ne has joined #openstack-ironic | 08:28 | |
| dtantsur | morning Ironic | 08:29 |
| *** jaosorior has joined #openstack-ironic | 08:29 | |
| *** jpich has joined #openstack-ironic | 08:30 | |
| yufei | morning, dtantsur | 08:31 |
| *** SerenaFeng has joined #openstack-ironic | 08:34 | |
| *** e0ne has quit IRC | 08:34 | |
| zhangjl | git | 08:35 |
| zhangjl | sorry | 08:36 |
| yufei | hi, dtantsur, according to your suggestion, I push a new patch, please help review when you have time https://review.openstack.org/#/c/399857/ | 08:36 |
| patchbot | patch 399857 - ironic-python-agent - Skip API releated work if no api url configured | 08:36 |
| dtantsur | sure | 08:37 |
| yufei | :D thank you | 08:37 |
| vdrok | good morning ironic, pas-ha dtantsur and yufei | 08:42 |
| pas-ha | morning vdrok dtantsur yufei | 08:43 |
| *** Alagar has quit IRC | 08:48 | |
| *** makowals has quit IRC | 08:49 | |
| *** makowals_ has joined #openstack-ironic | 08:49 | |
| *** homeski has quit IRC | 08:49 | |
| *** priteau has joined #openstack-ironic | 08:56 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:02 | |
| *** ohamada_ has joined #openstack-ironic | 09:05 | |
| *** ohamada__ has joined #openstack-ironic | 09:06 | |
| *** ohamada_ has quit IRC | 09:06 | |
| *** ohamada__ has quit IRC | 09:06 | |
| *** ohamada__ has joined #openstack-ironic | 09:06 | |
| *** ohamada has quit IRC | 09:07 | |
| *** moshele has quit IRC | 09:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/401141 | 09:15 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Configure and use SSL-related requests options https://review.openstack.org/398992 | 09:15 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-python-agent: Use oslo-config-generator for sample config https://review.openstack.org/398989 | 09:15 |
| *** fragatina has quit IRC | 09:20 | |
| *** fragatina has joined #openstack-ironic | 09:20 | |
| *** Romanenko_K has joined #openstack-ironic | 09:25 | |
| *** lucas-afk is now known as lucasagomes | 09:31 | |
| lucasagomes | morning all | 09:31 |
| *** SerenaFeng has quit IRC | 09:34 | |
| dtantsur | hey lucasagomes, vdrok, pas-ha | 09:35 |
| pas-ha | morning lucasgomes | 09:36 |
| vdrok | morning lucasagomes | 09:36 |
| *** aNuposic has quit IRC | 09:37 | |
| lucasagomes | o/ | 09:39 |
| *** strigazi_AFK is now known as strigazi | 09:39 | |
| *** derekh has joined #openstack-ironic | 09:41 | |
| *** Guest86010 has quit IRC | 09:46 | |
| *** ccamacho is now known as ccamacho|afk | 09:47 | |
| tuanluong | morning ironic | 09:48 |
| tuanluong | mornig lucasagomes, dtantsur, pas-ha, vdrok, ccamacho|afk | 09:49 |
| vdrok | morning tuanluong | 09:49 |
| *** abalfour has quit IRC | 09:51 | |
| *** abalfour has joined #openstack-ironic | 09:52 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic: mask private keys for the ssh power driver. https://review.openstack.org/401160 | 09:55 |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: playbooks: bifrost-ironic-install: Stop and disable epmd on SUSE https://review.openstack.org/401164 | 09:58 |
| *** dtantsur is now known as dtantsur|bbl | 10:00 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: playbooks: bifrost-ironic-install: Stop and disable epmd on SUSE https://review.openstack.org/401164 | 10:01 |
| *** zhangjl has quit IRC | 10:11 | |
| *** sambetts|afk is now known as sambetts | 10:24 | |
| sambetts | Morning all | 10:24 |
| vdrok | good morning sambetts | 10:31 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 10:31 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 10:34 |
| *** mgould|afk is now known as mgould | 10:39 | |
| mgould | morning lucasagomes dtantsur|bbl pas-ha vdrok sambetts tuanluong | 10:40 |
| *** tuanluong has quit IRC | 10:40 | |
| lucasagomes | o/ | 10:40 |
| vdrok | mgould: morning! | 10:40 |
| *** yolanda has quit IRC | 10:41 | |
| *** yufei has quit IRC | 10:41 | |
| ccamacho|afk | o/ | 10:50 |
| *** ccamacho|afk is now known as ccamacho | 10:50 | |
| *** athomas has quit IRC | 10:52 | |
| *** athomas has joined #openstack-ironic | 11:00 | |
| *** priteau has quit IRC | 11:02 | |
| *** vikrant is now known as vikrant|T | 11:07 | |
| *** Michael-ZTE has joined #openstack-ironic | 11:10 | |
| *** Michael-ZTE has quit IRC | 11:11 | |
| *** yolanda has joined #openstack-ironic | 11:11 | |
| *** igordcard has quit IRC | 11:14 | |
| *** igordcard has joined #openstack-ironic | 11:15 | |
| *** priteau has joined #openstack-ironic | 11:21 | |
| TheJulia | good morning | 11:23 |
| charlledaniel | morning all | 11:24 |
| lucasagomes | TheJulia, good morning | 11:31 |
| charlledaniel | morning TheJulia, mgould, sambetts | 11:32 |
| *** ccamacho is now known as ccamacho|lunch | 11:34 | |
| *** milan has joined #openstack-ironic | 11:36 | |
| *** dtantsur|bbl is now known as dtantsur | 11:36 | |
| dtantsur | morning mgould, TheJulia, charlledaniel | 11:39 |
| dtantsur | and sambetts | 11:39 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 11:41 |
| *** alexpilotti has joined #openstack-ironic | 11:43 | |
| *** alexpilo_ has quit IRC | 11:43 | |
| *** priteau has quit IRC | 11:50 | |
| *** e0ne has joined #openstack-ironic | 11:50 | |
| *** priteau has joined #openstack-ironic | 11:50 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Don't recreate db during grenade upgrade https://review.openstack.org/401218 | 11:50 |
| sambetts | vsaienk0: have you had a chance to look at my network refactor patch recently? I've added multinode support to it | 11:51 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Correct DB Interface migration test https://review.openstack.org/401221 | 11:54 |
| TheJulia | dtantsur: ^^^ | 11:54 |
| *** priteau has quit IRC | 11:55 | |
| *** priteau has joined #openstack-ironic | 11:55 | |
| *** yolanda has quit IRC | 11:55 | |
| *** yolanda has joined #openstack-ironic | 11:55 | |
| *** openstackgerrit has quit IRC | 12:03 | |
| *** openstackgerrit has joined #openstack-ironic | 12:03 | |
| *** jcoufal has joined #openstack-ironic | 12:09 | |
| *** leduardo has quit IRC | 12:12 | |
| *** alexpilotti has quit IRC | 12:12 | |
| *** alexpilotti has joined #openstack-ironic | 12:12 | |
| vdrok | morning TheJulia charlledaniel | 12:13 |
| vdrok | sambetts: he's on paternity leave for a week or 2 | 12:14 |
| sambetts | vdrok: ah, congrats to him! | 12:15 |
| pas-ha | we have small bug in g-r for stable/newton re oslo.utils verison | 12:16 |
| pas-ha | g-r stable/newton lists oslo.utils >= 3.16 while ironic as stable/newton uses oslo_utils.netutils.is_valid_mac that only appeared in 3.17 | 12:17 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Drac: Deprecate drac_host property https://review.openstack.org/401234 | 12:18 |
| *** vsaienko has joined #openstack-ironic | 12:21 | |
| *** lucasagomes is now known as lucas-hungry | 12:21 | |
| *** vsaienko has quit IRC | 12:25 | |
| *** hw_wutianwei has quit IRC | 12:25 | |
| *** hw_wutianwei has joined #openstack-ironic | 12:26 | |
| pas-ha | disregard my last posts | 12:31 |
| *** aNuposic has joined #openstack-ironic | 12:33 | |
| *** hw_wutianwei has quit IRC | 12:46 | |
| *** links has quit IRC | 12:54 | |
| *** fellypefca has quit IRC | 12:54 | |
| *** dprince has joined #openstack-ironic | 12:56 | |
| mrtenio | Good morning Ironic | 12:57 |
| milan | morning Ironic! :) | 13:02 |
| ricardoas | morning everyone... | 13:02 |
| milan | morning aarefiev, dtantsur, mrtenio, pas-ha, ricardo as, sambetts, TheJulia, vdrok and everyone! :) | 13:02 |
| pas-ha | morning milan | 13:04 |
| *** xavierr has quit IRC | 13:05 | |
| *** yufei has joined #openstack-ironic | 13:06 | |
| dtantsur | milan, the involved patch I was talking about today: https://review.openstack.org/336626 | 13:06 |
| patchbot | patch 336626 - ironic - Support defining and loading hardware types | 13:06 |
| *** sborkows has joined #openstack-ironic | 13:06 | |
| *** priteau has quit IRC | 13:06 | |
| milan | dtantsur, ack | 13:07 |
| milan | dtantsur, the smoke test thing: https://review.openstack.org/#/c/400410/ | 13:07 |
| patchbot | patch 400410 - ironic-inspector - [WIP] Tempest config update | 13:07 |
| milan | I need to undo yesterday's experiment | 13:07 |
| milan | check revision #1; that made the smoke test run | 13:08 |
| aarefiev | morning milan | 13:08 |
| *** ccamacho|lunch is now known as ccamacho | 13:08 | |
| vdrok | morning mrtenio and milan | 13:08 |
| *** anush has joined #openstack-ironic | 13:09 | |
| milan | aarefiev, I wonder should we land my hack to make smoke run? | 13:09 |
| *** mtanino has quit IRC | 13:09 | |
| milan | *revision 1 of it | 13:09 |
| dtantsur | I wonder if we should call ironic_configure_tempest somewhere | 13:09 |
| * milan unsure | 13:09 | |
| aarefiev | milan, dtantsur: imo grenade should call tempest config for plugins | 13:10 |
| aarefiev | something like that https://review.openstack.org/#/c/400850/1 | 13:10 |
| patchbot | patch 400850 - openstack-dev/grenade - Run stack/test-config phase for plugin configuration | 13:10 |
| milan | I'm open to suggestions her | 13:11 |
| aarefiev | milan: I'm not sure how it works, I mean your patch | 13:11 |
| milan | *here | 13:11 |
| dtantsur | oh, ironic_configure_tempest does get run | 13:11 |
| dtantsur | http://logs.openstack.org/10/400410/2/check/gate-grenade-dsvm-ironic-inspector/b326e21/logs/grenade.sh.txt.gz#_2016-11-22_15_02_14_872 | 13:11 |
| milan | aarefiev, my patch is sort of a brute-force thing that is called all the time from plugin.sh in tempest config stage | 13:13 |
| dtantsur | but it does not work \o/ | 13:13 |
| dtantsur | because new tempest config does not exist yet at all | 13:14 |
| dtantsur | and iniset does not care that the input file is not provided | 13:14 |
| aarefiev | it works only for first run | 13:14 |
| dtantsur | right | 13:14 |
| dtantsur | I wonder why ironic grenade works | 13:14 |
| aarefiev | because grenade runs tempest upgrade after plugin | 13:14 |
| * dtantsur checks ironic grenade | 13:15 | |
| aarefiev | dtantsur: because it doesn't use ironic manager | 13:15 |
| aarefiev | only nova basic test | 13:15 |
| * dtantsur rage quits | 13:15 | |
| dtantsur | I need to save this conversation for the next time somebody asks me why I hate grenade | 13:16 |
| aarefiev | heh | 13:16 |
| aarefiev | so tempest setup in grenade plugins doesn't work | 13:17 |
| dtantsur | yep | 13:17 |
| aarefiev | at least for second run | 13:17 |
| dtantsur | and it seems like milan | 13:17 |
| dtantsur | milan's fix did not work either: http://logs.openstack.org/10/400410/1/check/gate-grenade-dsvm-ironic-inspector/05dcffe/console.html#_2016-11-22_11_31_45_189386 | 13:17 |
| aarefiev | yep | 13:17 |
| aarefiev | imo there is should be some api and grenade calls plugins for tempest setup | 13:18 |
| aarefiev | or some workaround like in my patch | 13:18 |
| milan | oOps | 13:18 |
| dtantsur | jroll or whoever: why do we need this https://github.com/openstack/ironic/blob/master/ironic_tempest_plugin/tests/scenario/baremetal_manager.py#L61 | 13:19 |
| *** Guest59528 has quit IRC | 13:19 | |
| dtantsur | is it for network tests? | 13:19 |
| *** priteau has joined #openstack-ironic | 13:20 | |
| milan | is it just me or the test actually ran in http://logs.openstack.org/10/400410/2/check/gate-grenade-dsvm-ironic-inspector/b326e21/logs/grenade.sh.txt.gz#_2016-11-22_14_53_21_317 ? | 13:22 |
| dtantsur | milan, yeah, it runs on old part | 13:23 |
| TheJulia | dtantsur: that doesn't seem right.... :\ | 13:23 |
| milan | ouch | 13:23 |
| dtantsur | but for new part grenade recreates tempest.conf after we modify it | 13:23 |
| *** lucas-hungry is now known as lucasagomes | 13:23 | |
| *** anush has quit IRC | 13:26 | |
| openstackgerrit | Merged openstack/ironic: Add support for Security Groups for baremetal servers https://review.openstack.org/361451 | 13:28 |
| *** yolanda has quit IRC | 13:29 | |
| aarefiev | dtantsur: looks like instance port update test will fail w/o this option | 13:30 |
| *** priteau has quit IRC | 13:30 | |
| dtantsur | I see | 13:30 |
| dtantsur | aarefiev, still, we don't care for our smoke test: we don't run instances there | 13:30 |
| dtantsur | so maybe we should override this check | 13:30 |
| milan | I can update my patch, let's try? | 13:31 |
| milan | aarefiev, went? | 13:31 |
| milan | *wdyt? | 13:31 |
| aarefiev | yeah we could try | 13:31 |
| milan | ack | 13:32 |
| * milan hacks ~10min | 13:32 | |
| jroll | dtantsur: I picked up thiago's patch, which had that, and I guess I forgot to wip it because it merged before I expected to | 13:32 |
| jroll | dtantsur: I agree, not sure that should be there | 13:33 |
| jroll | does inspector inherit that class? | 13:33 |
| aarefiev | yes | 13:33 |
| jroll | ah | 13:33 |
| aarefiev | but still configuration issue is here, and could break us in future | 13:33 |
| nicodemos | good morning, all | 13:33 |
| jroll | aarefiev: well, I think we can drop that code | 13:34 |
| jroll | it isn't useful in upstream CI, that is for sure | 13:34 |
| dtantsur | then let's drop it, I guess | 13:34 |
| jroll | indeed | 13:35 |
| aarefiev | yeah, we could try, but no guarantees that some other options will not break us | 13:35 |
| jroll | (I'm not really here today, but around for a minute, can push a patch) | 13:35 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Drop bad skip check in tempest plugin https://review.openstack.org/401262 | 13:36 |
| jroll | dtantsur: ^ | 13:36 |
| aarefiev | btw, I see a lot of tempest configuration in devstack, related to ironic | 13:37 |
| dtantsur | thanks jroll, let's see if it passed CI | 13:37 |
| jroll | indeed | 13:37 |
| milan | aarefiev, dtantsur jroll I'll then depend my patch on jroll's to try it in Inspector | 13:41 |
| * milan wipes the manager override&changes depends-on | 13:42 | |
| mrtenio | vdrok, could you rereview my patch? Thank you https://review.openstack.org/#/c/377073/ | 13:42 |
| patchbot | patch 377073 - ironic - Adds another validation step when using dynamic al... | 13:42 |
| *** baoli has joined #openstack-ironic | 13:43 | |
| *** baoli has quit IRC | 13:43 | |
| *** priteau has joined #openstack-ironic | 13:43 | |
| *** baoli has joined #openstack-ironic | 13:43 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Drac: Deprecate drac_host property https://review.openstack.org/401234 | 13:44 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: [WIP] Tempest config update https://review.openstack.org/400410 | 13:44 |
| vdrok | morning nicodemos and jroll | 13:48 |
| vdrok | mrtenio: will try today | 13:48 |
| sambetts | vdrok: We need to decide what the behaviour of refresh is then, I guess the two options are, 1. it completely resets the object to what the current DB state is or 2. it resets all the fields to the DB state but also keeps any unsaved changes you have | 13:50 |
| vdrok | sambetts: correct. right now what the docstring describes looks more like 1 | 13:51 |
| *** derekh is now known as derekh_afk | 13:51 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 13:52 |
| milan | aarefiev, sambetts, mind a review https://review.openstack.org/#/c/344921/ ? got +2 from dtantsur | 13:53 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 13:53 |
| karthiks | When is the last date for inspector spec to get approved for ocata ? | 13:53 |
| lucasagomes | TheJulia, do you remember off the top of your head if we had any change pending in Ironic for attaching cinder volumes in the running instances ? I remember we talked about it in the mitaka (?) summit | 13:54 |
| lucasagomes | TheJulia, AFAIR, we concluded that there's no work needed in Ironic, but the images should include the os-brick agent. Is that correct ? | 13:55 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 13:55 |
| TheJulia | lucasagomes: That is correct, and ultimately the images or hosts needed sufficient information to lookup the volume from cinder and get connection details because they have to perform the attachment operation. | 13:55 |
| dtantsur | milan, aarefiev, a similar problem with this patch: https://review.openstack.org/379256 | 13:56 |
| patchbot | patch 379256 - ironic - Skip tempest tests if ironic isn't available | 13:56 |
| dtantsur | dmellado, FYI ^^^ | 13:56 |
| sambetts | vdrok: hmm, it doesn't seem to mention what it should do with existing changes :/ | 13:56 |
| lucasagomes | TheJulia, gotcha! Thanks a lot | 13:57 |
| vdrok | sambetts: let's ask others then :) | 13:57 |
| sambetts | vdrok: also I think that we don't need to worry about out-of-tree things inheriting the ironic base object, IMO thats not part of our API | 13:57 |
| TheJulia | lucasagomes: tl;dr it is out of our control. If it was cinder controlling FC we could make it happen, kind of at some point, but the running instance would still need to know to rescan for new targets. | 13:57 |
| vdrok | dtantsur: lucasagomes TheJulia quick question - what would you expect node.refresh() call to do - reset the object to the db state completely, or keep the unsaved fields and update everything else? | 13:58 |
| sambetts | jroll: ^ | 13:58 |
| TheJulia | select * from nodes where id = node.id ? | 13:58 |
| dtantsur | vdrok, I would expect it to reset the state completely, similar to SQL ^^^ | 13:59 |
| vdrok | for context - that's from the discussion here https://review.openstack.org/354744 | 13:59 |
| patchbot | patch 354744 - ironic - Fix object save after refresh failure | 13:59 |
| lucasagomes | vdrok, I would expect it to drop the unsaved fields yeah | 14:00 |
| sambetts | cool :) | 14:00 |
| sambetts | seems like everyone is in agreement | 14:00 |
| vdrok | thanks all! | 14:00 |
| vdrok | sambetts: as for objects inheriting it, yeah, maybe I'm starting to be too cautious with it :) | 14:01 |
| *** rloo has joined #openstack-ironic | 14:01 | |
| vdrok | sambetts: and the nova example was the thing that made me change the implementation. if we think we're ok with doing it for every object, that should be ok too | 14:02 |
| *** anush has joined #openstack-ironic | 14:02 | |
| sambetts | vdrok: yeah we've failed to be good at documenting what is and isn't the stable API for out of tree things, that was a goal we listed a couple of summits back but it never got done as far as I know | 14:02 |
| sambetts | vdrok: yeah, looking at nova/objects/base.py they don't have a base class refresh implementation, I think in ours we do because we expect it to be pretty much the same for all objects | 14:03 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 14:05 |
| *** srobert has joined #openstack-ironic | 14:06 | |
| dtantsur | OSC bikeshedding question: is "portgroup" an actual English word or should we split it as "port group"? | 14:07 |
| *** glonlas has joined #openstack-ironic | 14:08 | |
| mgould | it's not an actual English word, AIUI | 14:09 |
| *** glonlas has quit IRC | 14:09 | |
| TheJulia | not, so... split I guess? | 14:09 |
| * mgould wonders if he can still get into the OED online | 14:10 | |
| sambetts | I guess the question is is portgroup a noun to refer to an entity or do we want to refer to portgroups as groups of ports | 14:10 |
| rloo | dtantsur: so we need to distinguish 'port group' when writing English, vs 'Portgroup' the class name. | 14:10 |
| mgould | dtantsur: it's sufficiently uncommon that I want to check the definitive English dictionary before giving a definite "no" :-) | 14:10 |
| rloo | dtantsur: but yeah, in English, it is 'port group'. | 14:10 |
| *** ijw has joined #openstack-ironic | 14:10 | |
| dtantsur | rloo, yep, I'm reviewing the OSC commands proposed where we tend to use better English than usual :) | 14:11 |
| rloo | dtantsur: having said that, if portgroup is used often enough, it becomes acceptable English. | 14:11 |
| sambetts | but a port group can exist without ports in it, so then it isn't a group of poorts | 14:11 |
| dtantsur | any group can have no members... so dunno | 14:11 |
| *** glonlas has joined #openstack-ironic | 14:12 | |
| rloo | dtantsur: i think in OSC, that we've combined terms into one 'thing'. but let me see. is it all the portgroup-related osc commands? | 14:12 |
| dtantsur | rloo, https://review.openstack.org/362130 | 14:12 |
| patchbot | patch 362130 - python-ironicclient - Add portgroup support to osc plugin | 14:12 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 14:13 |
| rloo | vdrok: ^^ did you/we decide not to make address mandatory? | 14:13 |
| mgould | the OED has never heard of "portgroup", and suggests "podagrous" as an alternative | 14:13 |
| mgould | "Of or relating to gout; suffering from gout; gouty" | 14:14 |
| rloo | ++ for podagrous | 14:14 |
| sambetts | if we had the port and group parts separate then I'd expect the commands to be different, e.g. instead of openstack baremetal portgroup create we would have openstack baremetal port group <uuid> <uuid> | 14:14 |
| vdrok | morning rloo | 14:14 |
| dtantsur | mgould, I like the suggested variant | 14:14 |
| mgould | dtantsur: I'm +0.5 on splitting; I think it's slightly better, but it doesn't make much difference | 14:14 |
| *** pcaruana has quit IRC | 14:14 | |
| sambetts | IMO group is an an action not a thing, but portgroup is a thing | 14:15 |
| *** ijw has quit IRC | 14:15 | |
| vdrok | rloo: I'm kind of OK with it, don't remember vsaienk0 disagreeing with that | 14:15 |
| sambetts | e.g. I group ports together, but I can create a portgroup | 14:15 |
| mgould | rloo: one for your list of meeting-report adjectives? :-) | 14:15 |
| *** [3]cdearborn has joined #openstack-ironic | 14:15 | |
| rloo | mgould: perhaps. not sure when I'll do P, I've got another theme going. | 14:16 |
| dtantsur | sambetts, well, if you have "port group create" then "port group" is a thing and "create" is an action | 14:16 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Drac: Deprecate drac_host property https://review.openstack.org/401234 | 14:16 |
| dtantsur | compare with e.g. "openstack workflow execution create" | 14:16 |
| mgould | sambetts: I'm not convinced. "friend group", "work group", "point group", etc all seem OK to me | 14:17 |
| sambetts | so then a group is a subresource of a port | 14:17 |
| sambetts | not its own object | 14:17 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 14:17 |
| rloo | dtantsur: one technical thing, if we already have openstack port ..., i don't know that the infrastructure will support openstack port group ... | 14:18 |
| dtantsur | rloo, it will | 14:18 |
| sambetts | rloo: it'd be openstack baremetal port group right? | 14:18 |
| dtantsur | rloo, see again my example with "openstack workflow" | 14:18 |
| dtantsur | there is "openstack workflow create" and "openstack workflow execution create" | 14:18 |
| * dtantsur started looking into mistral recently | 14:18 | |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 14:18 |
| sambetts | dtantsur: does a workflow exection require a workflow id so some kind to create it ? | 14:19 |
| rloo | dtantsur: oh yes, i see also 'hypervisor' and 'hypervisor stats' | 14:19 |
| vdrok | rloo: sambetts the only thing that worries me is that a user created a portgroup, and it won't have an address up until the nova boot request issued, then it will be written in the db and won't change? | 14:19 |
| dtantsur | sambetts, I think so | 14:19 |
| sambetts | dtantsur: then openstack baremetal port group create would requrie a port id to create | 14:20 |
| sambetts | which it doesn't today | 14:20 |
| rloo | ok, i see osc 'floating ip xx', 'ip floating ..' 'ip fixed ...'. we need an osc czar. | 14:20 |
| rloo | dtantsur: osc keypair ... | 14:21 |
| *** yolanda has joined #openstack-ironic | 14:21 | |
| rloo | dtantsur: i think it is 'key pair' in English | 14:21 |
| *** vikrant|T has quit IRC | 14:21 | |
| dtantsur | sambetts, yeah, but "floating ip" is a thing.. anyway, I'm not going to -1 based on that only. | 14:21 |
| dtantsur | I just wanted to start this discussion | 14:21 |
| sambetts | yeah :/ sigh... I guess this is why bikesheeding haha | 14:22 |
| dtantsur | e.g. as a user I would have hard time understanding why I write "floating ip" or "baremetal introspection" but "keypair" and "portgroup" :/ | 14:22 |
| rloo | on the other hand, i see 'openstack security group ...' | 14:22 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 14:22 |
| rloo | and openstack server group ... | 14:23 |
| sambetts | argh... even in the thing thats supposed to be consistant there is no constancy | 14:23 |
| sambetts | consistantcy XD | 14:23 |
| vdrok | lol | 14:23 |
| rloo | so is 'portgroup' a term that is used or is that something we created? | 14:24 |
| openstackgerrit | Merged openstack/ironic: Add user and project domains to ironic context https://review.openstack.org/295371 | 14:24 |
| sambetts | it appears to be a vswitch term | 14:24 |
| openstackgerrit | Merged openstack/ironic: Correct DB Interface migration test https://review.openstack.org/401221 | 14:25 |
| sambetts | different switch vendors use different words for the same thing | 14:25 |
| *** lindycoder has joined #openstack-ironic | 14:25 | |
| rloo | sambetts: oh. so kleenex instead of tissues? :) | 14:25 |
| sambetts | Cisco usings portchannel, arista uses LAG and MLAG, Linux uses bond | 14:25 |
| sambetts | rloo: right | 14:26 |
| rloo | we should have used bond i think. | 14:26 |
| sambetts | Yeah, I've wondered the same thing before but I assumed we invented port group because it kind of describes what it is without ties to implemention, but :/ | 14:28 |
| openstackgerrit | Vadim Hmyrov proposed openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 14:28 |
| dtantsur | if it's not an established term, then I'm still inclined to use "port group" | 14:28 |
| sambetts | doing a quick google of portgroup, I only see it as one word once, 2 words in multiple places | 14:29 |
| rloo | hmm, take a look at this paper: https://people.netfilter.org/pablo/netdev0.1/papers/MLAG-on-Linux.pdf | 14:30 |
| rloo | they used 'bond' | 14:30 |
| rloo | is it too late to rename our portgroup to bond?! | 14:30 |
| TheJulia | lol | 14:30 |
| TheJulia | Any relation to a James Bond? | 14:30 |
| rloo | otoh, we could use bond in OSC. but that would confuse *us* | 14:30 |
| sambetts | yeah it would detact the CLI from the API | 14:31 |
| sambetts | detach* | 14:31 |
| rloo | so assuming we stay with portgroup, after the short time thinking about it, i think dmitry is right, it should be 'openstack baremetal port group <operation>' | 14:31 |
| * dtantsur feels he's just opened a huge can of worms | 14:31 | |
| *** marios_ has joined #openstack-ironic | 14:32 | |
| rloo | that is consistent with use of 'port group' in the internet world, and eg in that paper. and consistent with existing openstack X group commands | 14:32 |
| TheJulia | dtantsur: Perhaps it is beer o'clock then? | 14:32 |
| rloo | sambetts: what do you think? | 14:32 |
| *** tesseract has joined #openstack-ironic | 14:32 | |
| dtantsur | haha, true TheJulia | 14:32 |
| rloo | worms == fishing | 14:32 |
| *** marios_ has quit IRC | 14:33 | |
| dtantsur | fish o'clock is usually early morning | 14:33 |
| sambetts | yeah I think I'm being convinced that it port group is better | 14:33 |
| *** tesseract is now known as Guest48653 | 14:33 | |
| * mgould had fish for lunch | 14:33 | |
| dtantsur | please comment on the patch with your opinion | 14:33 |
| rloo | thx dtantsur for raising it. i had just assumed 'portgroup' was a thing. | 14:33 |
| dtantsur | np :) | 14:33 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors table to save connector information https://review.openstack.org/200983 | 14:33 |
| rloo | dtantsur: we had updated the spec to include the osc cmds. | 14:33 |
| dtantsur | :( for some reason it hasn't come to my mind previously | 14:34 |
| openstackgerrit | octopuszhang proposed openstack/ironic-specs: pxe-rsd driver for ironic https://review.openstack.org/401185 | 14:35 |
| rloo | dtantsur: you may not have seen the update. | 14:35 |
| TheJulia | rloo: vdrok ^^^ Addressed review feedback on the initial volume connector rev. I'll rebase the next revisions on top of it a little later and perform updates in the same vain as the reviews | 14:35 |
| rloo | TheJulia: thx. yeah, i wouldn't bother rebasing until the first one is approved or almost approved. | 14:35 |
| TheJulia | I'll do it anyway :) | 14:36 |
| sambetts | dtantsur: if we do them as a subcommand how would you represent listing ports by portgroup id? openstack baremetal port list --portgroup-uuid= or openstack baremetal port list --group= ?? | 14:36 |
| rloo | TheJulia: ha ha. it is up to you, it doesn't matter to me :) | 14:36 |
| TheJulia | rloo: btw, I shuffled the test order around for you :) | 14:37 |
| dtantsur | sambetts, good question, I was thinking about --port-group, but I guess --group is saner in this case | 14:37 |
| rloo | TheJulia: thx! | 14:37 |
| TheJulia | oh, I forgot a thing | 14:38 |
| * TheJulia shrugs, not a big deal, just a docstring | 14:38 | |
| rloo | TheJulia: fix it! :) | 14:38 |
| TheJulia | rloo: soon, moar coffee first | 14:39 |
| *** Guest48653 has quit IRC | 14:39 | |
| *** tesseract- has joined #openstack-ironic | 14:39 | |
| * dtantsur has just realized that the defaults calculation logic in the driver composition reform requires moving node's creation to conductors | 14:42 | |
| sambetts | dtantsur: because of the config settings? | 14:43 |
| *** aNuposic has quit IRC | 14:43 | |
| dtantsur | yep | 14:43 |
| TheJulia | dtantsur: I pretty much anticipated that, or that we would have to keep the config setting the same cross conductors and then percolate outward to the api what drivers are available on the conductors | 14:44 |
| sambetts | dtantsur: I thought we decided that the configs on the API side had to match the conductors/ | 14:44 |
| sambetts | the same as how all the conductors had to match because of the hashring stuff | 14:45 |
| dtantsur | only interfaces, but not enabled hardware types | 14:45 |
| dtantsur | conductors can safely have non-overlapping set of hardware types enabled | 14:45 |
| milan | sambetts, second +2 on https://review.openstack.org/#/c/344921/ are you planning reviewing it again? | 14:45 |
| patchbot | patch 344921 - ironic-inspector - Add API for listing all introspection statuses | 14:45 |
| TheJulia | dtantsur: if that is the case, then the hash ring needs to be changed | 14:46 |
| dtantsur | TheJulia, why? I mean, this is the case with regular drivers right now | 14:46 |
| rloo | vdrok: wrt your comment about LOG.exception: https://review.openstack.org/#/c/361451/26/ironic/common/neutron.py | 14:46 |
| patchbot | patch 361451 - ironic - Add support for Security Groups for baremetal servers (MERGED) | 14:46 |
| dtantsur | we only require enabled interfaces to be consistent (and only per hardware type) | 14:46 |
| TheJulia | dtantsur: failover, if you have pools of different drivers... | 14:46 |
| TheJulia | well, different hardware types | 14:47 |
| sambetts | how does the API check enabled drivers right now, and isn't it the same for hardware type? | 14:47 |
| rloo | vdrok: it does make a diff, exception will log a trace too | 14:47 |
| dtantsur | sambetts, it requests an RPC topic for the driver; it's the same for hardware types | 14:47 |
| dtantsur | sambetts, but when we start to calculate defaults, we need to know which interfaces are enabled for a given hardware type | 14:47 |
| vdrok | rloo: correct, but there is no trace there | 14:47 |
| *** derekh_afk is now known as derekh | 14:47 | |
| vdrok | so it just won't be printed | 14:48 |
| rloo | vdrok: oh. right. but still misleading. do you know if someone is going to submit a followon patch? | 14:48 |
| vdrok | rloo: nope | 14:48 |
| sambetts | dtantsur: if we know the hardware_type is enabled on a conductor because we can check, and we know which interfaces are enabled, why can't we work it out on the API side? | 14:48 |
| rloo | vdrok: ok, i'll put it on my list of things to do. thx. | 14:49 |
| dtantsur | sambetts, "we know which interfaces are enabled" we don't (not on API side at least) | 14:49 |
| vdrok | rloo: thank you :) | 14:49 |
| sambetts | dtantsur: we do because all the conductors and apis are configured for the same enabled_interfaces | 14:49 |
| *** tesseract- has quit IRC | 14:49 | |
| dtantsur | sambetts, nope, this is the problem | 14:50 |
| dtantsur | example | 14:50 |
| sambetts | we decided that was a requirement for driver composition | 14:50 |
| sambetts | Im pretty sure its in the spec | 14:50 |
| dtantsur | sambetts, conductor1 is configured with enabled_hardware_types=ilo and enabled_power_interface=ilo-power; conductor2 is configured with enabled_hardware_types=ipmi and enabled_power_interfaces=ipmitool | 14:50 |
| dtantsur | this is not prohibited by the spec | 14:50 |
| dtantsur | what is prohibited by the spec is the following: | 14:51 |
| dtantsur | conductor1 is configured with enabled_hardware_types=ilo and enabled_power_interface=ilo-power; conductor2 is configured with enabled_hardware_types=ilo and enabled_power_interfaces=ilo-power | 14:51 |
| sambetts | dtantsur: all conductors must have the same interfaces enabled is what we said before even if they have different hardware types enabled | 14:51 |
| dtantsur | ugh, I messed it up | 14:51 |
| sambetts | dtantsur: otherwise we need to include the interfaces in the hashring | 14:51 |
| dtantsur | "A warning will be issued on conductor start up, if it detects that other conductors have a different set of interfaces for the same enabled hardware type" | 14:52 |
| dtantsur | so if conductors have non-overlapping sets of hardware types, this ^^^ is not violated | 14:52 |
| dtantsur | what is not allowed is: conductor1 is configured with enabled_hardware_types=ilo and enabled_power_interface=ilo-power; conductor2 is configured with enabled_hardware_types=ilo and enabled_power_interfaces=ipmitool because ilo hw type supports both these power methods | 14:53 |
| *** jaosorior has quit IRC | 14:53 | |
| sambetts | dtantsur: the rest of that sentence includes Create a new table conductor_hardware_interfaces to hold the relationship between conductors, hardware types and available interfaces. | 14:53 |
| sambetts | dtantsur: so can't we use that table to lookup the avaible interfaces for that hardawre_type | 14:53 |
| *** jaosorior has joined #openstack-ironic | 14:54 | |
| sambetts | i think this problem is the other reason we have that table | 14:54 |
| dtantsur | sambetts, we can. I thought we want to reduce API service's access to the database though, no? | 14:54 |
| dtantsur | also the database will not be ordered (as it's designed now), so we can't use it for defaults calculation | 14:55 |
| sambetts | dtantsur: the DB doesn't need to be ordered | 14:55 |
| *** jkilpatr_ has joined #openstack-ironic | 14:55 | |
| sambetts | dtantsur: the ordering is on the hardwaretype object | 14:55 |
| dtantsur | sambetts, which is not accessible on API side, yes :) | 14:56 |
| sambetts | yes is it | 14:56 |
| sambetts | its in the code the API just needs to import <module where hardwaretype is> | 14:56 |
| dtantsur | well, it happens to be. right now we don't require drivers to be loadable from API though | 14:56 |
| *** jkilpatr has quit IRC | 14:56 | |
| *** Goneri has joined #openstack-ironic | 14:56 | |
| dtantsur | and in theory if people make ironic-conductor separate and use out-of-tree drivers - it won't be possible | 14:56 |
| * dtantsur has opened the second can of worms | 14:57 | |
| sambetts | hehe | 14:57 |
| sambetts | dtantsur: if thats the case then we'll have to do it in the conductor because we have no way to reference the hardware type if its not installed | 14:58 |
| dtantsur | yep | 14:58 |
| *** glonlas_ has joined #openstack-ironic | 14:59 | |
| vdrok | sambetts: TheJulia so going back to the portgroup addresses, in case of neutron, we can set it to whatever neutron generated, but in case of standalone ironic, should we generate it somehow? or just use whatever port address inside of this portgroup? | 14:59 |
| *** shuquan has joined #openstack-ironic | 15:01 | |
| sambetts | vdrok: in standalone Ironic its perfectly possible to have portgroup with no mac address | 15:01 |
| sambetts | vdrok: then the linux implemation takes over | 15:01 |
| sambetts | vdrok: I say its down to the NetworkInterface to decide | 15:01 |
| sambetts | vdrok: and right no standalone has a no-op | 15:01 |
| sambetts | right now * | 15:01 |
| sambetts | but in the futre there might be a Networkinterface for standalone that hooks into some SDN controller, and then it might pick a mac address from what that wants | 15:02 |
| *** glonlas has quit IRC | 15:02 | |
| vdrok | sambetts: hm, ok, that makes sense | 15:03 |
| *** cdearborn has joined #openstack-ironic | 15:04 | |
| sambetts | dtantsur: only alternative is to use another table to store calculated interfaces for each hardware_type | 15:05 |
| dtantsur | ugh | 15:05 |
| sambetts | yeah not great | 15:05 |
| sambetts | dtantsur: or maybe have an RPC api to fetch the calculated interfaces from a conductor? | 15:06 |
| *** yufei has quit IRC | 15:06 | |
| TheJulia | vdrok: I think there is no absolute purity that the host will use the mac address neutron signs in that case | 15:06 |
| dtantsur | sambetts, mind reviewing https://review.openstack.org/#/c/401262/ please? judging by https://review.openstack.org/#/c/400410/ it fixes our grenade | 15:06 |
| patchbot | patch 401262 - ironic - Drop bad skip check in tempest plugin | 15:06 |
| patchbot | patch 400410 - ironic-inspector - [WIP] Tempest config update | 15:06 |
| *** [3]cdearborn has quit IRC | 15:08 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 15:08 |
| vdrok | TheJulia: what exactly do you mean by 'no absolute purity'? (sometimes I'm not able to parse complex constructs :D) | 15:09 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Block some port updates https://review.openstack.org/347360 | 15:09 |
| *** jaosorior has quit IRC | 15:09 | |
| *** jaosorior has joined #openstack-ironic | 15:10 | |
| TheJulia | vdrok: It is entirely operating system dependent, so the guest OS may choose to ignore, or a user may change the settings. The only way for it to know about that mac address is via metadata or config drive data too :\ | 15:10 |
| NobodyCam | Good Morning Ironic'ers | 15:10 |
| sambetts | dtantsur: do we still need to set the flag in the ironic devstack plugin if we're not checking it now? | 15:10 |
| TheJulia | Good morning NobodyCam | 15:10 |
| NobodyCam | good Morning TheJulia :) happy day before :turkey: day | 15:10 |
| vdrok | TheJulia: yup, exactly. in case of neutron/nova we'll be putting what neutron has generated in the configdrive, in case of standalone, operator will have to do it himself. | 15:11 |
| vdrok | morning NobodyCam ! | 15:11 |
| dtantsur | sambetts, hmm, maybe not | 15:12 |
| dtantsur | morning NobodyCam | 15:12 |
| NobodyCam | Good morning dtantsur , vdrok :) | 15:13 |
| sambetts | o/ NobodyCam | 15:13 |
| NobodyCam | morning sambetts | 15:13 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors table to save connector information https://review.openstack.org/200983 | 15:14 |
| NobodyCam | what we have done where I am is modified ironic to automatically create and attach a config drive in standalone mode | 15:14 |
| vdrok | dtantsur: I seem to have missed this whole thing with changing baremetal.driver_enabled to this port_admin_state_change, without this check, if the ironic plugin will be enabled but ironic service will be disabled for some reason, this test will start and fail, right? | 15:15 |
| dtantsur | I suspect so | 15:15 |
| TheJulia | vdrok: s/himself/theirself/ :) | 15:16 |
| vdrok | TheJulia: yup :) | 15:16 |
| TheJulia | NobodyCam: I see someone has been using slack :) | 15:17 |
| NobodyCam | lol yeppers lol :p | 15:17 |
| TheJulia | NobodyCam: Regardless, happy day before thanksgiving :) | 15:17 |
| NobodyCam | :) | 15:17 |
| TheJulia | rloo: I went back and added an example data structure for the info to be passed in for connector creation. It is brief, but hopefully sufficient context along with a massive reduction in connector_id variables | 15:18 |
| vdrok | dtantsur: so, there also is https://review.openstack.org/#/c/400032/ and https://review.openstack.org/379256 and I'm not sure how they are supposed to live together :) | 15:18 |
| patchbot | patch 400032 - ironic - Keep CONF.baremetal.driver_enabled for Tempest plugin | 15:18 |
| patchbot | patch 379256 - ironic - Skip tempest tests if ironic isn't available | 15:19 |
| rloo | Thx TheJulia. starting to look at it now. | 15:19 |
| *** rama_y has joined #openstack-ironic | 15:20 | |
| *** yufei has joined #openstack-ironic | 15:20 | |
| dtantsur | vdrok, me neither :) I think I -1ed one of them already | 15:21 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Move interface validation from API to conductor side https://review.openstack.org/401311 | 15:21 |
| vdrok | dtantsur: hmm, ok, will try to get some context on what's going on there | 15:22 |
| *** dtrainor has quit IRC | 15:22 | |
| *** shuquan has quit IRC | 15:24 | |
| *** glonlas_ has quit IRC | 15:24 | |
| *** glonlas has joined #openstack-ironic | 15:24 | |
| *** shuquan_ has joined #openstack-ironic | 15:25 | |
| *** chlong has joined #openstack-ironic | 15:25 | |
| *** zackf has quit IRC | 15:28 | |
| *** Guest86010 has joined #openstack-ironic | 15:31 | |
| *** srobert has quit IRC | 15:33 | |
| *** chlong has quit IRC | 15:34 | |
| *** shuquan_ has quit IRC | 15:35 | |
| *** anush has quit IRC | 15:37 | |
| *** yufei has quit IRC | 15:42 | |
| *** chlong has joined #openstack-ironic | 15:49 | |
| *** yufei has joined #openstack-ironic | 15:52 | |
| *** sukhdev has joined #openstack-ironic | 15:54 | |
| *** syed__ has joined #openstack-ironic | 15:54 | |
| *** makowals has joined #openstack-ironic | 15:59 | |
| *** Sukhdev_ has joined #openstack-ironic | 16:01 | |
| *** makowals_ has quit IRC | 16:02 | |
| *** yufei has left #openstack-ironic | 16:03 | |
| *** strigazi is now known as strigazi_AFK | 16:04 | |
| *** mjura has quit IRC | 16:07 | |
| vdrok | dtantsur: milan aarefiev it seems to me that this is the proper fix - https://review.openstack.org/400687, not just dropping the parameter? | 16:08 |
| patchbot | patch 400687 - ironic - Grenade: configure tempest with Ironic plugin | 16:08 |
| milan | vdrok, I'll check ;) | 16:09 |
| aarefiev | vdrok: it doesn't work | 16:10 |
| milan | vdrok, we found in the log it didn't work :-/ | 16:10 |
| milan | vdrok, /me pastes | 16:11 |
| vdrok | aarefiev: are you still at work? that way it would be faster :) | 16:11 |
| aarefiev | yeah :) | 16:11 |
| milan | vdrok, http://logs.openstack.org/10/400410/2/check/gate-grenade-dsvm-ironic-inspector/b326e21/ < a WIP that I depended on aarefiev 's patch to check whether it works | 16:12 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for CNA network card https://review.openstack.org/391836 | 16:14 |
| openstackgerrit | Szymon Borkowski proposed openstack/ironic-python-agent: Add a new Hardware Manager for CNA network card https://review.openstack.org/391836 | 16:16 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add warning message for baremetal node set/unset commands https://review.openstack.org/347856 | 16:17 |
| *** sborkows has quit IRC | 16:18 | |
| *** rpioso has joined #openstack-ironic | 16:20 | |
| *** zackf has joined #openstack-ironic | 16:21 | |
| *** makowals has quit IRC | 16:23 | |
| *** Sukhdev_ has quit IRC | 16:24 | |
| *** cdearborn has quit IRC | 16:26 | |
| openstackgerrit | Derek Higgins proposed openstack/ironic: mask private keys for the ssh power driver. https://review.openstack.org/401160 | 16:28 |
| *** fellypefca has joined #openstack-ironic | 16:29 | |
| *** daemontool has joined #openstack-ironic | 16:32 | |
| vdrok | milan: yup, I think I get what's going on now, thanks | 16:33 |
| openstackgerrit | Merged openstack/ironic-inspector: Add API for listing all introspection statuses https://review.openstack.org/344921 | 16:33 |
| *** dprince has quit IRC | 16:33 | |
| *** Guest86010 is now known as Nisha_ | 16:37 | |
| *** Nisha_ is now known as Nisha | 16:37 | |
| *** Nisha is now known as Nisha_ | 16:37 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 16:39 |
| *** rama_y has quit IRC | 16:40 | |
| *** ccamacho has quit IRC | 16:41 | |
| *** cdearborn has joined #openstack-ironic | 16:42 | |
| *** mtanino has joined #openstack-ironic | 16:52 | |
| fellypefca | sambetts Do you know if the communication of services between Neutron and Ironic is different when we have in one environment the Devstack and in another one the Openstack installed on Containers LCX? | 16:53 |
| sambetts | fellypefca: ironic only talks to Neutron using the neutron REST API via the python-neutronclient library | 16:55 |
| fellypefca | sambetts We are using Openstack installed on Containers and trying to create ports or networks using Neutron CLI. But we received this error message on CLI: "504 Gateway Time-out The server didn't respond in time." and there is no error in the Neutron Logs | 16:57 |
| *** dprince has joined #openstack-ironic | 16:58 | |
| mmedvede | pas-ha: around? I think we got our third-party CI back. Which patch where you testing yesterday? | 16:58 |
| fellypefca | sambetts When we have a devstack environment and try to do the same thing, it works. | 16:58 |
| *** ccamacho has joined #openstack-ironic | 16:59 | |
| *** zackf has quit IRC | 17:00 | |
| sambetts | Gateway timeouts means that its not even making it to Neutron likely some proxy is getting in the way | 17:01 |
| *** zackf has joined #openstack-ironic | 17:02 | |
| *** athomas has quit IRC | 17:04 | |
| *** makowals has joined #openstack-ironic | 17:06 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 17:06 |
| *** cdearborn has quit IRC | 17:11 | |
| *** rbartal has quit IRC | 17:12 | |
| *** ohamada__ has quit IRC | 17:15 | |
| *** jaosorior has quit IRC | 17:15 | |
| fellypefca | sambetts ok, thanks! | 17:16 |
| *** soliosg has joined #openstack-ironic | 17:20 | |
| *** e0ne has quit IRC | 17:20 | |
| *** rama_y has joined #openstack-ironic | 17:21 | |
| fellypefca | sambetts Do you know, on Neutron, any conf file where i increase this timeout? | 17:24 |
| sambetts | fellypefca: from what I can see its not a problem with a timeout value on either neutronclient or neutron, it seems like your request is going to a http/https proxy of some kind that can not access the IP address and therefore is timing out because the server is inaccesible | 17:26 |
| *** hogepodge has joined #openstack-ironic | 17:28 | |
| fellypefca | sambetts ok, I will check that. Thanks again! | 17:29 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs: Add bonding configuration fields to portgroups https://review.openstack.org/396610 | 17:29 |
| * sambetts -> home | 17:29 | |
| vdrok | good night sambetts | 17:30 |
| sambetts | night all | 17:30 |
| sambetts | o/ vdrok | 17:30 |
| *** Nisha_ has quit IRC | 17:30 | |
| *** sambetts is now known as sambetts|afk | 17:30 | |
| mgould | o/ sambetts|afk | 17:32 |
| lucasagomes | sambetts|afk night | 17:32 |
| * lucasagomes is heading too | 17:33 | |
| lucasagomes | have a great evening all | 17:33 |
| *** lucasagomes is now known as lucas-hungry | 17:33 | |
| *** lucas-hungry is now known as lucas-afk | 17:33 | |
| mgould | o/ lucas-afk | 17:33 |
| *** hogepodge has quit IRC | 17:33 | |
| *** glonlas has quit IRC | 17:41 | |
| *** hogepodge has joined #openstack-ironic | 17:43 | |
| *** rbrud has joined #openstack-ironic | 17:43 | |
| vdrok | time to leave for me too | 17:45 |
| vdrok | good night! | 17:45 |
| milan | night lucas-afk sambetts|afk vdrok! :) | 17:46 |
| *** daemontool has quit IRC | 17:46 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Minor changes to neutron security groups code https://review.openstack.org/401364 | 17:49 |
| *** chlong has quit IRC | 17:50 | |
| *** cdearborn has joined #openstack-ironic | 17:50 | |
| *** fellypefca has quit IRC | 17:50 | |
| * milan relocates | 17:51 | |
| *** derekh has quit IRC | 17:52 | |
| *** milan has quit IRC | 17:56 | |
| *** glonlas has joined #openstack-ironic | 17:57 | |
| *** priteau has quit IRC | 17:58 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Move interface validation from API to conductor side https://review.openstack.org/401311 | 17:58 |
| *** ralonsoh has quit IRC | 18:00 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Move interface validation from API to conductor side https://review.openstack.org/401311 | 18:02 |
| *** priteau has joined #openstack-ironic | 18:11 | |
| mrtenio | Good night lucas-afk vdrok sambetts|afk . And for the ones that celebrate it this week, Have a Happy Thanksgiving! | 18:12 |
| openstackgerrit | Merged openstack/ironic: Drop bad skip check in tempest plugin https://review.openstack.org/401262 | 18:17 |
| dtantsur | o/ | 18:22 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Move interface validation from API to conductor side https://review.openstack.org/401311 | 18:23 |
| *** dtantsur is now known as dtantsur|afk | 18:24 | |
| *** ChubYann has joined #openstack-ironic | 18:39 | |
| * mgould -> home; good night! | 18:42 | |
| *** mgould is now known as mgould|afk | 18:43 | |
| mgould|afk | and Happy Thanksgiving to all USians :-) | 18:43 |
| *** soliosg has quit IRC | 18:54 | |
| *** sukhdev has quit IRC | 19:01 | |
| *** SAulo-Aislan has joined #openstack-ironic | 19:01 | |
| *** moshele has joined #openstack-ironic | 19:12 | |
| SAulo-Aislan | Hi | 19:13 |
| TheJulia | Good morning | 19:13 |
| *** aNuposic has joined #openstack-ironic | 19:13 | |
| *** soliosg has joined #openstack-ironic | 19:20 | |
| *** fragatina has quit IRC | 19:22 | |
| *** makowals has quit IRC | 19:29 | |
| *** betherly_ has joined #openstack-ironic | 19:50 | |
| *** fragatina has joined #openstack-ironic | 19:56 | |
| *** jpich has quit IRC | 20:03 | |
| *** ijw has joined #openstack-ironic | 20:11 | |
| *** soliosg has quit IRC | 20:11 | |
| *** soliosg has joined #openstack-ironic | 20:14 | |
| *** e0ne has joined #openstack-ironic | 20:22 | |
| *** priteau has quit IRC | 20:24 | |
| *** fragatina has quit IRC | 20:30 | |
| *** e0ne has quit IRC | 20:34 | |
| *** Sukhdev has joined #openstack-ironic | 20:37 | |
| *** betherly_ is now known as betherly | 20:41 | |
| *** anush has joined #openstack-ironic | 20:42 | |
| *** dprince has quit IRC | 20:46 | |
| *** baoli has quit IRC | 20:51 | |
| *** baoli has joined #openstack-ironic | 20:51 | |
| *** moshele has quit IRC | 20:54 | |
| *** SAulo-Aislan has quit IRC | 20:56 | |
| *** baoli has quit IRC | 20:56 | |
| *** fragatina has joined #openstack-ironic | 21:01 | |
| *** anush has quit IRC | 21:05 | |
| *** MattMan has quit IRC | 21:05 | |
| *** MattMan has joined #openstack-ironic | 21:05 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors table to save connector information https://review.openstack.org/200983 | 21:06 |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add volume_connectors table to save connector information https://review.openstack.org/200983 | 21:07 |
| -openstackstatus- NOTICE: Due to a problem with the cinder volume backing the log server, jobs are failing with POST_FAILURE. Please avoid issuing 'recheck' commands until the issue is resolved. | 21:08 | |
| *** ChanServ changes topic to "Due to a problem with the cinder volume backing the log server, jobs are failing with POST_FAILURE. Please avoid issuing 'recheck' commands until the issue is resolved." | 21:08 | |
| TheJulia | heh | 21:08 |
| *** anush has joined #openstack-ironic | 21:11 | |
| anish | just in time, thanks TheJulia | 21:18 |
| *** jkilpatr_ has quit IRC | 21:22 | |
| *** ijw has quit IRC | 21:34 | |
| *** ijw has joined #openstack-ironic | 21:37 | |
| *** jkilpatr_ has joined #openstack-ironic | 21:38 | |
| *** priteau has joined #openstack-ironic | 21:41 | |
| *** yolanda has quit IRC | 21:50 | |
| *** jcoufal has quit IRC | 21:55 | |
| *** anush has quit IRC | 21:59 | |
| *** lindycoder has quit IRC | 21:59 | |
| *** e0ne has joined #openstack-ironic | 22:09 | |
| *** ccamacho has quit IRC | 22:11 | |
| *** slagle has quit IRC | 22:18 | |
| *** chlong has joined #openstack-ironic | 22:19 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add VolumeConnector object https://review.openstack.org/214584 | 22:25 |
| *** chlong has quit IRC | 22:30 | |
| *** e0ne has quit IRC | 22:39 | |
| *** rbrud has quit IRC | 22:52 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 22:52 | |
| -openstackstatus- NOTICE: The affected filesystems on the log server are repaired. Please leave 'recheck' comments on any changes which failed with POST_FAILURE. | 22:52 | |
| *** rbrud has joined #openstack-ironic | 22:52 | |
| *** glonlas_ has joined #openstack-ironic | 22:54 | |
| *** glonlas has quit IRC | 22:57 | |
| *** glonlas_ has quit IRC | 22:58 | |
| *** mtanino has quit IRC | 23:12 | |
| *** anush has joined #openstack-ironic | 23:20 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add VolumeConnector object https://review.openstack.org/214584 | 23:22 |
| *** priteau has quit IRC | 23:29 | |
| *** cdearborn has quit IRC | 23:31 | |
| *** anush has quit IRC | 23:37 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add VolumeConnector object https://review.openstack.org/214584 | 23:50 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!