| *** devlaps has joined #openstack-ironic | 00:06 | |
| *** ijw has quit IRC | 00:06 | |
| *** sambetts|afk has quit IRC | 00:08 | |
| devlaps | hi folks.. i'm new to ironic and bifrost and would appreciate any pointers. I've managed to install bifrost and get it generating disk images. Now I'm looking to spin up a VM (I'm in test mode). I'd like to use this node profile: https://gist.github.com/delapsley/b3d40e5ac99f93d3742320737200cfee. Can someone explain what the various IP addresses correspond to? | 00:09 |
|---|---|---|
| *** sambetts_ has joined #openstack-ironic | 00:09 | |
| *** penick_ has quit IRC | 00:16 | |
| *** ijw has joined #openstack-ironic | 00:20 | |
| *** ijw has quit IRC | 00:25 | |
| *** adu has joined #openstack-ironic | 00:27 | |
| *** adu has quit IRC | 00:40 | |
| *** ijw has joined #openstack-ironic | 00:42 | |
| *** ijw has quit IRC | 00:47 | |
| *** hoangcx has joined #openstack-ironic | 00:55 | |
| *** mtanino has quit IRC | 00:59 | |
| *** rama_y has quit IRC | 01:05 | |
| *** rpioso has quit IRC | 01:10 | |
| *** rajinir has quit IRC | 01:16 | |
| *** baoli has joined #openstack-ironic | 01:18 | |
| *** baoli has quit IRC | 01:19 | |
| *** baoli has joined #openstack-ironic | 01:19 | |
| *** ijw has joined #openstack-ironic | 01:24 | |
| *** Syed has quit IRC | 01:26 | |
| *** ijw has quit IRC | 01:31 | |
| *** rloo has quit IRC | 01:40 | |
| *** romcheg has quit IRC | 01:40 | |
| *** rbudden has joined #openstack-ironic | 01:44 | |
| *** SerenaFeng has joined #openstack-ironic | 01:45 | |
| *** gcb has joined #openstack-ironic | 01:50 | |
| *** Goneri has quit IRC | 01:52 | |
| *** ccamacho has quit IRC | 01:57 | |
| *** ccamacho has joined #openstack-ironic | 02:02 | |
| *** devlaps has quit IRC | 02:07 | |
| *** ijw has joined #openstack-ironic | 02:08 | |
| *** rbudden has quit IRC | 02:08 | |
| *** ijw has quit IRC | 02:13 | |
| *** ijw has joined #openstack-ironic | 02:18 | |
| *** ijw has quit IRC | 02:18 | |
| *** ijw has joined #openstack-ironic | 02:29 | |
| *** tuanluong has joined #openstack-ironic | 02:31 | |
| *** ijw has quit IRC | 02:33 | |
| *** yufei has joined #openstack-ironic | 02:39 | |
| *** baoli has quit IRC | 02:41 | |
| *** yuanying has quit IRC | 02:46 | |
| *** yuanying has joined #openstack-ironic | 02:55 | |
| *** pmannidi has quit IRC | 03:09 | |
| *** links has joined #openstack-ironic | 03:28 | |
| *** SerenaFeng has quit IRC | 03:33 | |
| *** nicodemos has quit IRC | 03:43 | |
| *** nicodemos has joined #openstack-ironic | 03:45 | |
| *** zhangjl has joined #openstack-ironic | 03:48 | |
| *** yuanying has quit IRC | 03:49 | |
| *** Nisha_brb has joined #openstack-ironic | 03:51 | |
| *** bks has joined #openstack-ironic | 03:52 | |
| *** zhangjl1 has joined #openstack-ironic | 03:53 | |
| *** zhangjl has quit IRC | 03:55 | |
| *** rbudden has joined #openstack-ironic | 04:16 | |
| *** yuanying has joined #openstack-ironic | 04:18 | |
| *** misty-cloud has joined #openstack-ironic | 04:27 | |
| *** Nisha_brb is now known as Nisha_ | 04:31 | |
| *** baoli has joined #openstack-ironic | 04:32 | |
| *** adu has joined #openstack-ironic | 04:34 | |
| *** hoangcx has quit IRC | 04:35 | |
| *** hoangcx has joined #openstack-ironic | 04:35 | |
| *** rbudden has quit IRC | 04:36 | |
| *** pmannidi has joined #openstack-ironic | 04:39 | |
| *** misty-cloud has quit IRC | 04:51 | |
| *** misty-cloud has joined #openstack-ironic | 04:51 | |
| *** aparnav has joined #openstack-ironic | 04:52 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 04:54 |
| *** adu has quit IRC | 05:06 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool management driver for Inject NMI https://review.openstack.org/348721 | 05:06 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update the existing APIs due to adding get_supported_power_states https://review.openstack.org/382194 | 05:17 |
| *** vikrant has joined #openstack-ironic | 05:26 | |
| *** nicodemos has quit IRC | 05:49 | |
| *** nicodemos has joined #openstack-ironic | 05:52 | |
| *** gcb has quit IRC | 05:54 | |
| *** SerenaFeng has joined #openstack-ironic | 05:54 | |
| *** nicodemos has quit IRC | 06:00 | |
| *** nicodemos has joined #openstack-ironic | 06:01 | |
| *** jaosorior has joined #openstack-ironic | 06:06 | |
| *** gcb has joined #openstack-ironic | 06:07 | |
| *** nicodemos has quit IRC | 06:27 | |
| *** nicodemos has joined #openstack-ironic | 06:27 | |
| *** ChubYann has quit IRC | 06:35 | |
| *** ijw has joined #openstack-ironic | 06:37 | |
| *** ijw has quit IRC | 06:41 | |
| *** tuanluong has quit IRC | 06:42 | |
| *** user__ has joined #openstack-ironic | 06:43 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 06:46 |
| *** SerenaFeng has quit IRC | 06:58 | |
| *** SerenaFeng has joined #openstack-ironic | 06:58 | |
| *** fxpester has joined #openstack-ironic | 06:59 | |
| *** mjura has joined #openstack-ironic | 07:04 | |
| *** fxpester|2 has joined #openstack-ironic | 07:04 | |
| *** fxpester has quit IRC | 07:05 | |
| *** fxpester has joined #openstack-ironic | 07:07 | |
| *** jaosorior has quit IRC | 07:08 | |
| *** jaosorior has joined #openstack-ironic | 07:08 | |
| *** fxpester|3 has joined #openstack-ironic | 07:10 | |
| *** fxpester|2 has quit IRC | 07:10 | |
| *** mjura has quit IRC | 07:11 | |
| *** mjura has joined #openstack-ironic | 07:11 | |
| *** fxpester|2 has joined #openstack-ironic | 07:12 | |
| *** fxpester has quit IRC | 07:12 | |
| *** fxpester has joined #openstack-ironic | 07:14 | |
| *** fxpester|3 has quit IRC | 07:15 | |
| *** charlledaniel has quit IRC | 07:15 | |
| *** e0ne has joined #openstack-ironic | 07:16 | |
| *** fxpester|2 has quit IRC | 07:17 | |
| *** fxpester|2 has joined #openstack-ironic | 07:17 | |
| *** charlledaniel has joined #openstack-ironic | 07:18 | |
| *** fxpester has quit IRC | 07:19 | |
| *** rbartal has joined #openstack-ironic | 07:20 | |
| *** fxpester has joined #openstack-ironic | 07:20 | |
| *** hoonetorg has quit IRC | 07:21 | |
| *** hoonetorg has joined #openstack-ironic | 07:21 | |
| *** fxpester|2 has quit IRC | 07:22 | |
| *** parlos has joined #openstack-ironic | 07:23 | |
| *** hoonetorg has quit IRC | 07:24 | |
| *** hoonetorg has joined #openstack-ironic | 07:24 | |
| *** fxpester|2 has joined #openstack-ironic | 07:27 | |
| *** moshele has joined #openstack-ironic | 07:28 | |
| *** rcernin has joined #openstack-ironic | 07:28 | |
| *** fxpester has quit IRC | 07:29 | |
| *** fxpester has joined #openstack-ironic | 07:30 | |
| *** SerenaFeng has quit IRC | 07:31 | |
| *** fxpester|3 has joined #openstack-ironic | 07:33 | |
| *** fxpester|2 has quit IRC | 07:33 | |
| *** fxpester|2 has joined #openstack-ironic | 07:35 | |
| *** fxpester has quit IRC | 07:35 | |
| *** SerenaFeng has joined #openstack-ironic | 07:35 | |
| *** fxpester|3 has quit IRC | 07:38 | |
| *** fxpester has joined #openstack-ironic | 07:38 | |
| *** fxpester|3 has joined #openstack-ironic | 07:40 | |
| *** fxpester|2 has quit IRC | 07:40 | |
| *** fxpester has quit IRC | 07:43 | |
| *** pmannidi has quit IRC | 07:43 | |
| *** fxpester has joined #openstack-ironic | 07:43 | |
| *** fxpester|2 has joined #openstack-ironic | 07:45 | |
| *** fxpester|3 has quit IRC | 07:46 | |
| *** fxpester has quit IRC | 07:47 | |
| *** fxpester has joined #openstack-ironic | 07:50 | |
| *** gcb has quit IRC | 07:51 | |
| *** fxpester|2 has quit IRC | 07:52 | |
| *** fxpester|2 has joined #openstack-ironic | 07:53 | |
| *** gcb has joined #openstack-ironic | 07:53 | |
| *** fxpester has quit IRC | 07:55 | |
| *** fxpester has joined #openstack-ironic | 07:55 | |
| vdrok | good morning ironic! | 07:56 |
| *** fxpester|3 has joined #openstack-ironic | 07:57 | |
| *** fxpester|2 has quit IRC | 07:58 | |
| *** fxpester|2 has joined #openstack-ironic | 08:00 | |
| *** fxpester has quit IRC | 08:00 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode stuff https://review.openstack.org/368173 | 08:02 |
| *** fxpester|3 has quit IRC | 08:02 | |
| *** fxpester has joined #openstack-ironic | 08:03 | |
| *** fxpester|3 has joined #openstack-ironic | 08:05 | |
| *** fxpester|2 has quit IRC | 08:05 | |
| *** links has quit IRC | 08:06 | |
| *** fxpester|2 has joined #openstack-ironic | 08:07 | |
| *** fxpester has quit IRC | 08:08 | |
| *** Nisha_ has quit IRC | 08:08 | |
| *** amotoki has joined #openstack-ironic | 08:08 | |
| *** amotoki has quit IRC | 08:08 | |
| *** fxpester|3 has quit IRC | 08:10 | |
| *** fxpester has joined #openstack-ironic | 08:11 | |
| *** fxpester|3 has joined #openstack-ironic | 08:13 | |
| *** fxpester|2 has quit IRC | 08:14 | |
| *** fxpester has quit IRC | 08:16 | |
| *** fxpester has joined #openstack-ironic | 08:16 | |
| *** fxpester|2 has joined #openstack-ironic | 08:18 | |
| *** links has joined #openstack-ironic | 08:19 | |
| *** fxpester|3 has quit IRC | 08:19 | |
| *** ralonsoh has joined #openstack-ironic | 08:19 | |
| *** vsaienko has joined #openstack-ironic | 08:20 | |
| *** fxpester has quit IRC | 08:21 | |
| *** fxpester has joined #openstack-ironic | 08:22 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/396355 | 08:23 |
| *** fxpester|3 has joined #openstack-ironic | 08:25 | |
| *** fxpester|2 has quit IRC | 08:25 | |
| *** fxpester has quit IRC | 08:27 | |
| *** fxpester|2 has joined #openstack-ironic | 08:28 | |
| *** vsaienko has quit IRC | 08:29 | |
| *** makowals_ has joined #openstack-ironic | 08:30 | |
| *** fxpester|3 has quit IRC | 08:30 | |
| *** makowals_ has quit IRC | 08:31 | |
| *** makowals has quit IRC | 08:33 | |
| *** fxpester has joined #openstack-ironic | 08:34 | |
| *** vsaienko has joined #openstack-ironic | 08:34 | |
| *** e0ne has quit IRC | 08:35 | |
| *** fxpester|2 has quit IRC | 08:36 | |
| *** fxpester|2 has joined #openstack-ironic | 08:37 | |
| *** fxpester|3 has joined #openstack-ironic | 08:39 | |
| *** fxpester has quit IRC | 08:40 | |
| *** fxpester|2 has quit IRC | 08:41 | |
| *** vsaienko has quit IRC | 08:42 | |
| *** fxpester has joined #openstack-ironic | 08:42 | |
| *** Nisha_ has joined #openstack-ironic | 08:42 | |
| *** fxpester|2 has joined #openstack-ironic | 08:44 | |
| *** makowals has joined #openstack-ironic | 08:44 | |
| *** fxpester|3 has quit IRC | 08:45 | |
| *** fxpester has quit IRC | 08:46 | |
| *** fxpester has joined #openstack-ironic | 08:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support names for {cleaning,provisioning}_network https://review.openstack.org/357983 | 08:47 |
| *** fxpester|3 has joined #openstack-ironic | 08:49 | |
| *** fxpester|2 has quit IRC | 08:49 | |
| *** fxpester|2 has joined #openstack-ironic | 08:51 | |
| openstackgerrit | Daniel Mellado proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 08:51 |
| *** fxpester has quit IRC | 08:52 | |
| *** fxpester|3 has quit IRC | 08:54 | |
| *** fxpester has joined #openstack-ironic | 08:54 | |
| *** fxpester|2 has quit IRC | 08:57 | |
| *** fxpester|2 has joined #openstack-ironic | 08:57 | |
| *** fxpester|3 has joined #openstack-ironic | 08:59 | |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-ironic | 09:00 | |
| *** fxpester has quit IRC | 09:00 | |
| *** fxpester|2 has quit IRC | 09:02 | |
| *** fxpester has joined #openstack-ironic | 09:02 | |
| *** fxpester|3 has quit IRC | 09:05 | |
| *** fxpester|2 has joined #openstack-ironic | 09:05 | |
| *** fxpester|3 has joined #openstack-ironic | 09:07 | |
| openstackgerrit | Merged openstack/bifrost: Bifrost keystone follow-up https://review.openstack.org/404370 | 09:08 |
| *** fxpester has quit IRC | 09:08 | |
| openstackgerrit | Merged openstack/bifrost: Test timeout extension for inspection https://review.openstack.org/375050 | 09:09 |
| *** alineb has joined #openstack-ironic | 09:10 | |
| openstackgerrit | Merged openstack/ironic: Fix docs error about OOB RAID support https://review.openstack.org/404248 | 09:10 |
| *** fxpester|2 has quit IRC | 09:10 | |
| pas-ha | morning Ironic :) | 09:12 |
| *** fxpester has joined #openstack-ironic | 09:13 | |
| *** jaosorior is now known as jaosorior_lunch | 09:13 | |
| pas-ha | vsaienk0: how troublesome would it be to switch the multinode job away from SSH driver? AFAIK we do still want to stop using it, right? | 09:15 |
| *** fxpester|2 has joined #openstack-ironic | 09:15 | |
| *** fxpester|3 has quit IRC | 09:16 | |
| *** SerenaFeng has quit IRC | 09:16 | |
| vsaienk0 | pas-ha: definitely we should switch to agent_ipmitool and xenial, I don't think it should be hard | 09:16 |
| *** SerenaFeng has joined #openstack-ironic | 09:17 | |
| *** fxpester has quit IRC | 09:17 | |
| *** fxpester has joined #openstack-ironic | 09:18 | |
| *** fxpester|2 has quit IRC | 09:20 | |
| *** fxpester|2 has joined #openstack-ironic | 09:21 | |
| *** SerenaFeng has quit IRC | 09:22 | |
| *** fxpester|3 has joined #openstack-ironic | 09:23 | |
| vdrok | morning pas-ha and vsaienk0 | 09:24 |
| *** zhangjl1 has quit IRC | 09:24 | |
| *** fxpester has quit IRC | 09:24 | |
| *** SerenaFeng has joined #openstack-ironic | 09:24 | |
| *** fxpester|2 has quit IRC | 09:25 | |
| *** fxpester has joined #openstack-ironic | 09:27 | |
| *** fxpester|2 has joined #openstack-ironic | 09:29 | |
| *** fxpester|3 has quit IRC | 09:30 | |
| *** baoli_ has joined #openstack-ironic | 09:31 | |
| *** fxpester has quit IRC | 09:32 | |
| *** lucas-afk is now known as lucasagomes | 09:33 | |
| lucasagomes | morning all | 09:33 |
| *** baoli has quit IRC | 09:33 | |
| *** fxpester has joined #openstack-ironic | 09:33 | |
| *** fxpester|3 has joined #openstack-ironic | 09:36 | |
| *** fxpester|2 has quit IRC | 09:36 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Do not source old/localrc twise in grenade https://review.openstack.org/404678 | 09:37 |
| vsaienk0 | morning lucasagomes, dtantsur, sambetts, vdrok, aarefiev, milan, mgould and all Ironic'ers! | 09:37 |
| mgould | morning lucasagomes pas-ha vsaienk0 vdrok aarefiev sambetts_ and all Ironicers! | 09:37 |
| * mgould is sitting next to milan's desk and he's not here yet :-) | 09:37 | |
| pas-ha | morning mgould lucasagomes | 09:37 |
| aarefiev | morning all! | 09:38 |
| vdrok | morning lucasagomes mgould and aarefiev | 09:38 |
| *** fxpester has quit IRC | 09:38 | |
| *** SerenaFeng has quit IRC | 09:39 | |
| *** fxpester has joined #openstack-ironic | 09:40 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Make all IronicExceptions RPC-serializable https://review.openstack.org/396355 | 09:41 |
| lucasagomes | o/ | 09:41 |
| *** fxpester|2 has joined #openstack-ironic | 09:43 | |
| *** fxpester|3 has quit IRC | 09:43 | |
| *** fxpester|3 has joined #openstack-ironic | 09:45 | |
| *** fxpester has quit IRC | 09:46 | |
| *** fxpester|2 has quit IRC | 09:48 | |
| *** fxpester has joined #openstack-ironic | 09:48 | |
| *** fxpester|2 has joined #openstack-ironic | 09:50 | |
| *** fxpester|3 has quit IRC | 09:51 | |
| *** fxpester|3 has joined #openstack-ironic | 09:53 | |
| *** fxpester has quit IRC | 09:53 | |
| *** fxpester has joined #openstack-ironic | 09:55 | |
| *** fxpester|2 has quit IRC | 09:56 | |
| *** fxpester|2 has joined #openstack-ironic | 09:57 | |
| *** fxpester|3 has quit IRC | 09:58 | |
| *** Romanenko_K has joined #openstack-ironic | 09:59 | |
| *** fxpester|3 has joined #openstack-ironic | 09:59 | |
| *** fxpester has quit IRC | 10:00 | |
| *** abalfour has quit IRC | 10:01 | |
| *** abalfour has joined #openstack-ironic | 10:01 | |
| *** fxpester|2 has quit IRC | 10:02 | |
| *** zackf has quit IRC | 10:04 | |
| *** fxpester has joined #openstack-ironic | 10:04 | |
| openstackgerrit | Dao Cong Tien proposed openstack/ironic: Add iRMC drivers to provide socat console https://review.openstack.org/404702 | 10:05 |
| *** links has quit IRC | 10:05 | |
| *** fxpester|3 has quit IRC | 10:07 | |
| *** fxpester|2 has joined #openstack-ironic | 10:07 | |
| *** fxpester has quit IRC | 10:09 | |
| *** fxpester has joined #openstack-ironic | 10:09 | |
| *** fxpester|2 has quit IRC | 10:12 | |
| *** milan has joined #openstack-ironic | 10:13 | |
| *** fxpester|2 has joined #openstack-ironic | 10:14 | |
| *** yee3799 has joined #openstack-ironic | 10:15 | |
| *** hoonetorg has quit IRC | 10:15 | |
| *** baoli has joined #openstack-ironic | 10:16 | |
| *** fxpester|3 has joined #openstack-ironic | 10:16 | |
| *** fxpester has quit IRC | 10:16 | |
| *** xhku_ has joined #openstack-ironic | 10:17 | |
| *** milan has quit IRC | 10:18 | |
| *** e0ne has joined #openstack-ironic | 10:18 | |
| *** fxpester|2 has quit IRC | 10:19 | |
| *** xhku has quit IRC | 10:19 | |
| *** logan- has quit IRC | 10:19 | |
| *** cppforlife_ has quit IRC | 10:19 | |
| *** anteaya has quit IRC | 10:19 | |
| *** baoli_ has quit IRC | 10:19 | |
| *** betherly has quit IRC | 10:19 | |
| *** yee379 has quit IRC | 10:19 | |
| *** rook has quit IRC | 10:19 | |
| *** natorious has quit IRC | 10:19 | |
| *** jaosorior_lunch has quit IRC | 10:19 | |
| *** sambetts_ has quit IRC | 10:19 | |
| *** bnemec has quit IRC | 10:19 | |
| *** JayF has quit IRC | 10:19 | |
| *** rbartal has quit IRC | 10:19 | |
| *** jschwarz has quit IRC | 10:19 | |
| *** skazi has quit IRC | 10:19 | |
| *** dhellmann has quit IRC | 10:19 | |
| *** morgabra has quit IRC | 10:19 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: POC: ETAG supporting to enhance API evolution https://review.openstack.org/392213 | 10:19 |
| *** hshiina has joined #openstack-ironic | 10:20 | |
| *** JayF has joined #openstack-ironic | 10:20 | |
| *** logan- has joined #openstack-ironic | 10:20 | |
| *** bnemec has joined #openstack-ironic | 10:21 | |
| *** e0ne has quit IRC | 10:21 | |
| *** e0ne has joined #openstack-ironic | 10:21 | |
| *** bcornec2 has joined #openstack-ironic | 10:21 | |
| *** links has joined #openstack-ironic | 10:22 | |
| *** alineb- has joined #openstack-ironic | 10:22 | |
| *** cppforlife_ has joined #openstack-ironic | 10:22 | |
| *** alineb has quit IRC | 10:23 | |
| *** alineb- is now known as alineb | 10:23 | |
| *** dhellmann has joined #openstack-ironic | 10:23 | |
| *** hoonetorg has joined #openstack-ironic | 10:24 | |
| *** clsacramento_ has joined #openstack-ironic | 10:24 | |
| *** natorious has joined #openstack-ironic | 10:25 | |
| *** clsacramento has quit IRC | 10:25 | |
| *** bcornec has quit IRC | 10:25 | |
| *** Daviey has quit IRC | 10:25 | |
| *** alex_xu has quit IRC | 10:25 | |
| *** boris-42 has quit IRC | 10:25 | |
| *** makowals has quit IRC | 10:25 | |
| *** sambetts_ has joined #openstack-ironic | 10:25 | |
| *** hoangcx has quit IRC | 10:25 | |
| *** makowals has joined #openstack-ironic | 10:26 | |
| *** rook has joined #openstack-ironic | 10:26 | |
| *** sambetts_ is now known as sambetts | 10:26 | |
| *** jschwarz has joined #openstack-ironic | 10:26 | |
| sambetts | Morning all | 10:26 |
| *** rook is now known as Guest68279 | 10:26 | |
| *** alex_xu has joined #openstack-ironic | 10:27 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost: Test OPNFV CI verify job. Please ignore. https://review.openstack.org/367608 | 10:27 |
| sambetts | :( I lost all my scroll back because ZNC derped, did anyone message me for anything this morning? | 10:27 |
| *** boris-42 has joined #openstack-ironic | 10:28 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: POC: Full-featured objects with ETAGs supporting https://review.openstack.org/400335 | 10:28 |
| *** morgabra has joined #openstack-ironic | 10:29 | |
| *** Nisha_ has quit IRC | 10:30 | |
| *** rbartal has joined #openstack-ironic | 10:30 | |
| *** skazi has joined #openstack-ironic | 10:30 | |
| *** fxpester has joined #openstack-ironic | 10:30 | |
| *** anteaya has joined #openstack-ironic | 10:31 | |
| *** jaosorior_lunch has joined #openstack-ironic | 10:31 | |
| *** priteau has joined #openstack-ironic | 10:33 | |
| *** fxpester|3 has quit IRC | 10:33 | |
| *** pcaruana has joined #openstack-ironic | 10:35 | |
| *** fxpester|2 has joined #openstack-ironic | 10:36 | |
| *** e0ne has quit IRC | 10:36 | |
| *** derekh has joined #openstack-ironic | 10:36 | |
| *** Daviey has joined #openstack-ironic | 10:36 | |
| *** betherly has joined #openstack-ironic | 10:36 | |
| *** betherly has quit IRC | 10:36 | |
| *** betherly has joined #openstack-ironic | 10:36 | |
| *** e0ne has joined #openstack-ironic | 10:36 | |
| *** vsaienko has joined #openstack-ironic | 10:37 | |
| *** fxpester has quit IRC | 10:38 | |
| *** fxpester has joined #openstack-ironic | 10:41 | |
| *** jaosorior_lunch is now known as jaosorior | 10:41 | |
| *** fxpester|3 has joined #openstack-ironic | 10:43 | |
| vdrok | good morning sambetts | 10:43 |
| *** fxpester|2 has quit IRC | 10:43 | |
| openstackgerrit | Merged openstack/ironic-specs: Change node create notifications https://review.openstack.org/404224 | 10:44 |
| *** fxpester has quit IRC | 10:46 | |
| vdrok | sambetts: vsaienk0 wanted to discuss, which order of patches do you think makes the most sense - portgroup config fields, standalone ports supported, vif attach/detach. I wanted to rebase them all in one chain, to make testing easier | 10:47 |
| vdrok | in theory, they can be merged completely separately | 10:47 |
| *** yufei has quit IRC | 10:48 | |
| *** fxpester has joined #openstack-ironic | 10:49 | |
| lucasagomes | sambetts, vdrok morning | 10:50 |
| *** vsaienko has quit IRC | 10:51 | |
| *** fxpester|3 has quit IRC | 10:52 | |
| *** fxpester|2 has joined #openstack-ironic | 10:55 | |
| *** fxpester has quit IRC | 10:58 | |
| *** fxpester has joined #openstack-ironic | 10:58 | |
| *** fxpester|2 has quit IRC | 11:01 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 11:01 |
| sambetts | o/ lucasagomes | 11:01 |
| *** fxpester|2 has joined #openstack-ironic | 11:01 | |
| *** fxpester has quit IRC | 11:04 | |
| *** fxpester has joined #openstack-ironic | 11:06 | |
| vsaienk0 | folks patch to unblock grenade https://review.openstack.org/#/c/404678/ | 11:06 |
| patchbot | patch 404678 - ironic - Do not source old/localrc twise in grenade | 11:06 |
| sambetts | hey vdrok, currently attach/detach does use any of the portgroups stuff (yet) so I'm not sure what rebasing it gains you? If I had to put it in any order I would do, vif attach/detach -> portgroup config fields, standalone ports supported -> vif attach/detach portgroup support | 11:06 |
| *** links has quit IRC | 11:07 | |
| *** snozhka|afk has quit IRC | 11:07 | |
| vsaienk0 | sambetts: I think we can land standalone_ports supported first, it already has +2 from rloo | 11:08 |
| *** links has joined #openstack-ironic | 11:08 | |
| sambetts | vsaienk0: the portgroups stuff IMO is currently completely idenpendent | 11:08 |
| vdrok | sambetts: oh, yeah, right. maybe we don't need rebasing for now | 11:08 |
| *** fxpester|2 has quit IRC | 11:08 | |
| *** vsaienko has joined #openstack-ironic | 11:09 | |
| sambetts | vdrok, vsaienk0: I suggest we tackle the attach/detach stuff and portgroup enhancements in parallel, and then see what needs rebasing when attach/detach portgroups support patch lands? | 11:09 |
| sambetts | s/patch lands/is up for review | 11:11 |
| vdrok | sambetts: that makes sense for me for now | 11:12 |
| *** fxpester|2 has joined #openstack-ironic | 11:13 | |
| vsaienk0 | sambetts: develop/review them in parallel is bets option for now | 11:13 |
| *** fxpester has quit IRC | 11:16 | |
| *** fxpester has joined #openstack-ironic | 11:16 | |
| *** misty-cloud has quit IRC | 11:16 | |
| *** Nisha_ has joined #openstack-ironic | 11:17 | |
| *** fxpester|3 has joined #openstack-ironic | 11:18 | |
| *** fxpester|2 has quit IRC | 11:18 | |
| sambetts | ++ | 11:20 |
| *** fxpester has quit IRC | 11:21 | |
| *** fxpester has joined #openstack-ironic | 11:22 | |
| *** fxpester|2 has joined #openstack-ironic | 11:24 | |
| *** fxpester|3 has quit IRC | 11:25 | |
| *** fxpester has quit IRC | 11:27 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV https://review.openstack.org/397126 | 11:27 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Show team and repo badges on README https://review.openstack.org/402495 | 11:29 |
| *** fxpester has joined #openstack-ironic | 11:30 | |
| *** nicodemos has quit IRC | 11:31 | |
| *** fxpester|3 has joined #openstack-ironic | 11:33 | |
| *** fxpester|2 has quit IRC | 11:33 | |
| *** priteau has quit IRC | 11:33 | |
| *** priteau has joined #openstack-ironic | 11:34 | |
| *** fxpester has quit IRC | 11:35 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Change (started|finished)_at column type https://review.openstack.org/397636 | 11:36 |
| *** fxpester has joined #openstack-ironic | 11:37 | |
| *** priteau has quit IRC | 11:38 | |
| *** fxpester|3 has quit IRC | 11:38 | |
| *** ijw has joined #openstack-ironic | 11:39 | |
| *** fxpester|2 has joined #openstack-ironic | 11:45 | |
| *** fxpester|3 has joined #openstack-ironic | 11:47 | |
| *** bks has quit IRC | 11:47 | |
| *** fxpester has quit IRC | 11:47 | |
| *** Nisha_ has quit IRC | 11:49 | |
| *** fxpester|2 has quit IRC | 11:49 | |
| *** vsaienko has quit IRC | 11:50 | |
| *** fxpester|3 has quit IRC | 11:51 | |
| *** priteau has joined #openstack-ironic | 11:52 | |
| *** ijw has quit IRC | 11:52 | |
| *** misty-cloud has joined #openstack-ironic | 11:53 | |
| *** ijw has joined #openstack-ironic | 12:04 | |
| *** SerenaFeng has joined #openstack-ironic | 12:07 | |
| *** jtomasek has joined #openstack-ironic | 12:07 | |
| *** ijw has quit IRC | 12:09 | |
| *** ijw has joined #openstack-ironic | 12:15 | |
| *** parlos has quit IRC | 12:16 | |
| *** slagle has joined #openstack-ironic | 12:17 | |
| *** misty-cloud has quit IRC | 12:19 | |
| *** jtomasek has quit IRC | 12:19 | |
| vdrok | sambetts: lucasagomes could you take a look at https://review.openstack.org/404678 ? that's fixing grenade which is busted right now | 12:20 |
| patchbot | patch 404678 - ironic - Do not source old/localrc twise in grenade | 12:20 |
| *** baoli_ has joined #openstack-ironic | 12:20 | |
| lucasagomes | vdrok, totally ++ | 12:20 |
| vdrok | thanks! | 12:20 |
| lucasagomes | vdrok, done! | 12:21 |
| vdrok | :) | 12:21 |
| *** vsaienko has joined #openstack-ironic | 12:21 | |
| *** baoli has quit IRC | 12:22 | |
| *** ijw has quit IRC | 12:25 | |
| sambetts | tbh that seems like a bug with the devstack functions that enables plugins e.g. it should be able to be called multiple times but only actually do it once | 12:25 |
| sambetts | also I'm surprised that is affecting the existing run because it should be being called in a subshell | 12:26 |
| sambetts | but :/ | 12:26 |
| *** vsaienko has quit IRC | 12:28 | |
| *** makowals_ has joined #openstack-ironic | 12:31 | |
| *** zackf has joined #openstack-ironic | 12:31 | |
| *** links has quit IRC | 12:34 | |
| *** makowals has quit IRC | 12:34 | |
| *** jtomasek has joined #openstack-ironic | 12:35 | |
| *** jrist has joined #openstack-ironic | 12:37 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/404436 | 12:39 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Turn off tempest's multitenant network tests https://review.openstack.org/397767 | 12:39 |
| *** parlos has joined #openstack-ironic | 12:40 | |
| *** bfournie has quit IRC | 12:40 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: Testing 355586 https://review.openstack.org/397775 | 12:42 |
| jroll | morning y'all | 12:42 |
| jroll | no dtantsur|afk today? :( | 12:42 |
| jroll | who else is familiar with inspector CI config? could use a quick review on https://review.openstack.org/#/c/404422/ | 12:43 |
| patchbot | patch 404422 - openstack-infra/project-config - Load ironic tempest plugin for ironic-inspector | 12:43 |
| *** makowals has joined #openstack-ironic | 12:43 | |
| *** jkilpatr has joined #openstack-ironic | 12:44 | |
| vdrok | morning jroll | 12:44 |
| *** makowals_ has quit IRC | 12:45 | |
| *** rbartal has quit IRC | 12:50 | |
| lucasagomes | jrist, morning | 12:51 |
| lucasagomes | mila<tab> ... #failed | 12:51 |
| sambetts | aarefiev might be able to help there | 12:51 |
| jroll | two tab fails! | 12:51 |
| jroll | :P | 12:51 |
| jroll | morning lucasagomes | 12:51 |
| sambetts | o/ jroll | 12:52 |
| jroll | hey sambetts ;) | 12:52 |
| jroll | s/;/:/ | 12:52 |
| lucasagomes | jroll, hah yeah... pinged him in the internal rh channel | 12:52 |
| lucasagomes | not sure he's there either tho | 12:52 |
| jroll | heh | 12:52 |
| lucasagomes | (I mean, he's online, but might be afk) | 12:53 |
| *** yufei has joined #openstack-ironic | 12:54 | |
| *** lucasagomes is now known as lucas-hungry | 12:54 | |
| *** aparnav has quit IRC | 12:55 | |
| *** milan has joined #openstack-ironic | 12:55 | |
| milan | morning Ironic! :) | 12:56 |
| jroll | \o milan | 12:56 |
| milan | hey jroll! :) I was told you had a patch for the Inspector | 12:57 |
| * milan 's IRC client was a bit stubborn | 12:57 | |
| vdrok | morning milan :) | 12:58 |
| jroll | milan: ah yeah, a project-config change, want to make sure it won't break inspector, https://review.openstack.org/#/c/404422/ | 12:58 |
| patchbot | patch 404422 - openstack-infra/project-config - Load ironic tempest plugin for ironic-inspector | 12:58 |
| *** gcb has quit IRC | 12:58 | |
| milan | jroll, thx, /me looks | 12:58 |
| milan | morning vdrok! :) | 12:58 |
| sambetts | jroll: the ironic plugin is already loaded for inspector | 12:59 |
| jroll | sambetts: tempest plugin? where? | 12:59 |
| dmellado | sambetts: I was about to say that ;) | 12:59 |
| dmellado | ironic-inspector-tempest-plugin | 12:59 |
| sambetts | jroll: in the devstack-virtual-ironic | 12:59 |
| sambetts | builder | 12:59 |
| dmellado | imports some stuff from the ironic one | 12:59 |
| dmellado | so it should be around | 12:59 |
| *** SerenaFeng has quit IRC | 13:00 | |
| jroll | I don't see that builder in the ironic-inspector config, hrm | 13:00 |
| sambetts | jroll: the ironic-inspector-enabled builder uses the ironic-extra-vars builder which is used to optionally exend the base ironic-virtual-devstack builder | 13:01 |
| *** athomas has joined #openstack-ironic | 13:01 | |
| sambetts | jroll: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L907 | 13:02 |
| jroll | I'm still not seeing it :/ | 13:02 |
| jroll | OH | 13:03 |
| jroll | idk how I missed this | 13:03 |
| jroll | thanks sambetts | 13:03 |
| * jroll abandons patch | 13:03 | |
| *** jcoufal has joined #openstack-ironic | 13:03 | |
| sambetts | jroll: np :) tbh I wonder if we need to update some of these jobs: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L386 | 13:04 |
| sambetts | jroll: which don't use the builders | 13:04 |
| milan | thank sambetts! | 13:04 |
| jroll | sambetts: yeah those are the ones I was hrm'ing about | 13:04 |
| jroll | however | 13:04 |
| *** trown|outtypewww is now known as trown | 13:05 | |
| jroll | doesn't seem like they use tempest | 13:05 |
| jroll | that one specifically, at least | 13:05 |
| sambetts | hmm yeah, we didn't have tempest originally, we had our own bash script for black box testing, we're switching to tempest though so maybe we'll clean those things up when we're done | 13:06 |
| *** athomas has quit IRC | 13:06 | |
| jroll | right | 13:10 |
| jroll | my point is that I'm worried about the tempest config here and those jobs don't use it :P | 13:10 |
| * milan btw in a downstream mtg together with Dmitry and Miles for maybe 1 more hour | 13:12 | |
| *** jkilpatr has quit IRC | 13:12 | |
| *** jtomasek has quit IRC | 13:12 | |
| jroll | milan: I can only assume by "meeting" you mean "pub time" | 13:13 |
| *** bfournie has joined #openstack-ironic | 13:14 | |
| sambetts | haha ;) | 13:16 |
| *** athomas has joined #openstack-ironic | 13:19 | |
| *** jkilpatr has joined #openstack-ironic | 13:19 | |
| TheJulia | good morning | 13:22 |
| *** jheroux has joined #openstack-ironic | 13:23 | |
| *** karthiks has quit IRC | 13:29 | |
| *** dtantsur|afk is now known as dtantsur | 13:29 | |
| dtantsur | morning/afternoon folks | 13:29 |
| dtantsur | jroll, I'm semi-here, we have local meetings in Brno | 13:30 |
| openstackgerrit | Merged openstack/ironic: Do not source old/localrc twise in grenade https://review.openstack.org/404678 | 13:32 |
| TheJulia | ugh, does our grenade job hate us or something? | 13:33 |
| vdrok | morning TheJulia and dtantsur | 13:34 |
| *** links has joined #openstack-ironic | 13:36 | |
| *** links has quit IRC | 13:36 | |
| dtantsur | huh, do we check MAC address uniqueness when creating ports? | 13:37 |
| TheJulia | I think we do | 13:40 |
| sambetts | dtantsur: I believe we do | 13:41 |
| TheJulia | yeah, the address field in the db is a unique constraint | 13:42 |
| dtantsur | strange.. I have a customer ending up with failed introspection due to duplicate MACs | 13:42 |
| TheJulia | Has anyone seen http://logs.openstack.org/20/285220/43/check/gate-grenade-dsvm-ironic/892560d/logs/grenade.sh.txt.gz#_2016-11-29_18_11_11_811 this multiple times? I feel like I have recently | 13:43 |
| *** jcoufal_ has joined #openstack-ironic | 13:44 | |
| sambetts | TheJulia: I think this might be solving that https://review.openstack.org/397767 | 13:44 |
| patchbot | patch 397767 - ironic - Turn off tempest's multitenant network tests | 13:44 |
| TheJulia | dtantsur: dupliate as in duplicated on their infrastucture? | 13:45 |
| *** karthiks has joined #openstack-ironic | 13:45 | |
| sambetts | TheJulia: actually maybe not, I think that might be an issue with our grenade tear down and the new secgroups feature | 13:45 |
| sambetts | vsaienk0: ^^ I think secgroups and grenade might be sad | 13:46 |
| dtantsur | TheJulia, at least duplicate from ironic-inspector point of view.. dunno how they achieved that | 13:46 |
| *** jtomasek has joined #openstack-ironic | 13:47 | |
| TheJulia | dtantsur: :( | 13:47 |
| *** jcoufal has quit IRC | 13:47 | |
| *** hw_wutianwei has joined #openstack-ironic | 13:47 | |
| *** baoli has joined #openstack-ironic | 13:48 | |
| *** baoli_ has quit IRC | 13:50 | |
| TheJulia | http://logs.openstack.org/67/397767/2/check/gate-grenade-dsvm-ironic/7ef033d/logs/grenade.sh.txt.gz#_2016-11-30_13_39_01_425 | 13:51 |
| *** lucas-hungry is now known as lucasagomes | 13:51 | |
| jroll | sambetts: we haven't enabled sec groups for provisioning network in CI, have we? | 13:51 |
| jroll | (so there hasn't been a change there) | 13:51 |
| sambetts | oh :/ then it likely not our problem then | 13:51 |
| sambetts | but needs more digging | 13:51 |
| sambetts | I guess | 13:51 |
| jroll | TheJulia: we noticed the first one yesterday but looked like a one-off... I think there's a patch in the gate queue for the second | 13:52 |
| jroll | or even merged \o/ https://review.openstack.org/#/c/404678/ | 13:52 |
| patchbot | patch 404678 - ironic - Do not source old/localrc twise in grenade (MERGED) | 13:52 |
| TheJulia | woot | 13:52 |
| TheJulia | jroll: sambetts: I think I've seen the first one a few times even before we started work on security group support, so I suspect it is just a race in that scenario that tempest is attempting to execute. | 13:53 |
| jroll | yeah, I'd agree | 13:53 |
| openstackgerrit | Vadim Hmyrov proposed openstack/ironic: Block some port updates https://review.openstack.org/347360 | 13:53 |
| *** baoli_ has joined #openstack-ironic | 13:54 | |
| *** baoli has quit IRC | 13:56 | |
| sambetts | jroll: This bug was marked as an Opinion but then due to a patch being pushed against it, it has been moved to in-progress, I'm not sure how we indicate to people that this is still an Opinion https://bugs.launchpad.net/ironic/+bug/1606492 | 13:57 |
| openstack | Launchpad bug 1606492 in Ironic "Conductor allows to update port address in all node states" [Medium,In progress] - Assigned to Vadim Hmyrov (vhmyrov) | 13:57 |
| jroll | sambetts: we set it back to opinion | 13:57 |
| jroll | optional procedural -2 on the patch until we have a discussion | 13:58 |
| *** srobert has joined #openstack-ironic | 13:58 | |
| *** parlos has quit IRC | 14:00 | |
| *** karthiks has quit IRC | 14:02 | |
| sambetts | jroll: done | 14:04 |
| jroll | :) | 14:04 |
| sambetts | first -2 is scary hehe | 14:05 |
| * aarefiev scroll up | 14:06 | |
| aarefiev | ok, seems no open questions | 14:09 |
| *** glonlas has joined #openstack-ironic | 14:09 | |
| jroll | aarefiev: :D | 14:10 |
| *** rloo has joined #openstack-ironic | 14:11 | |
| jrist | morning lucasagomes :) | 14:12 |
| jrist | haha | 14:12 |
| lucasagomes | jrist, hahah morning | 14:12 |
| *** lindycoder has joined #openstack-ironic | 14:13 | |
| *** Goneri has joined #openstack-ironic | 14:14 | |
| vdrok | wooo, another bug in portgroups api https://bugs.launchpad.net/ironic/+bug/1646120 | 14:15 |
| openstack | Launchpad bug 1646120 in Ironic "Passing portgroup_uuid in POST /ports silently ignored in old api versions" [Undecided,New] - Assigned to Vladyslav Drok (vdrok) | 14:15 |
| *** karthiks has joined #openstack-ironic | 14:19 | |
| *** ohamada has joined #openstack-ironic | 14:20 | |
| TheJulia | joy | 14:21 |
| *** vsaienko has joined #openstack-ironic | 14:21 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add portgroup configuration fields https://review.openstack.org/398417 | 14:22 |
| *** [1]cdearborn has joined #openstack-ironic | 14:24 | |
| sambetts | vdrok: isn't that what is supposed to happen? Don't we ignore any field we don't recognise or do we through an unrecognised parameter exception? | 14:24 |
| *** shuquan has joined #openstack-ironic | 14:24 | |
| vdrok | sambetts: for all the other fields that are set as usual wsme attributes I think we throw an exception | 14:25 |
| vdrok | that ^^ happens because we have a custom setter | 14:25 |
| sambetts | vdrok: I mean if I pass through, "a_fake_field": "some_data" doesn't that get ignored? | 14:25 |
| sambetts | vdrok: or does it throw "Unrecognised field a_fake_field" ? | 14:26 |
| vdrok | sambetts: I think not, wsme will be trying to set all the attributes passed in the body | 14:26 |
| milan | jroll, that what happens in the evening indeed :D | 14:27 |
| milan | *beer | 14:27 |
| sambetts | vdrok: you think not to which? | 14:27 |
| milan | morning TheJulia! :) | 14:27 |
| *** vsaienko has quit IRC | 14:28 | |
| vdrok | sambetts: I think if passed an unrecognised attr post throws an exception | 14:28 |
| vdrok | lemme check | 14:28 |
| vdrok | sambetts: Unknown attribute for argument port: abc | 14:29 |
| vdrok | POST -d '{"abc": "cde", "address": "aa:bb:cc:dd:ee:ff", "node_uuid": "node-0"}' | 14:30 |
| *** srobert has quit IRC | 14:30 | |
| vdrok | for the usual attributes, we properly raise just two lines above https://github.com/openstack/ironic/blob/dd57ed5a2d2c42abc72ac13ce7eb491c27b7468a/ironic/api/controllers/v1/port.py#L504-L506 | 14:30 |
| TheJulia | good morning milan | 14:31 |
| * milan humbly suggests a midcycle in Brno ;) | 14:32 | |
| rloo | hi and good morning TheJulia, vdrok, milan, sambetts | 14:32 |
| milan | * POC of that going on very well so far ;) | 14:32 |
| *** vikrant has quit IRC | 14:33 | |
| milan | morning rloo! :) | 14:33 |
| rloo | anyone know why grenade is not cooperating? | 14:33 |
| rloo | milan: what's the weather like in Brno? I vote for virtual midcycle! | 14:33 |
| vdrok | mornin rloo, yes, fixed | 14:33 |
| rloo | vdrok: ok thx! | 14:33 |
| *** gcb has joined #openstack-ironic | 14:33 | |
| rloo | vdrok: but it was broken, right? | 14:33 |
| milan | rloo, ~0 centigrade, cloudy, occasional snow, but that doesn't matter in a cosy Pub ;) | 14:34 |
| sambetts | vdrok: ah ok so we should do the same for portgroup_uuid | 14:34 |
| milan | vdrok, \o/ | 14:34 |
| vdrok | rloo: yup, jlvillal 's patch merged yesterday, throwing an error if plugin enabled twice, and it was somehow the case in our plugin :) | 14:35 |
| rloo | milan: i might as well stay home, it is 12C today, but going down to 3 this week. | 14:35 |
| rloo | vdrok: ok, i'll update the whiteboard/etherpad | 14:35 |
| vdrok | speaking of weather, two days in a row of sunny and snowy in the evening weather in kharkiv, ~ -5C :) | 14:35 |
| rloo | vdrok: this fixed it, right? https://review.openstack.org/404678 | 14:35 |
| patchbot | patch 404678 - ironic - Do not source old/localrc twise in grenade (MERGED) | 14:35 |
| vdrok | rloo: yup | 14:35 |
| TheJulia | good morning rloo | 14:36 |
| rloo | TheJulia: hi, thx for rechecking those patches | 14:37 |
| TheJulia | milan: a cosy pub sounds perfect, it is cloudy, windy, and maybe -3C here right now. | 14:37 |
| milan | rloo, I see. Would be a sad midcycle w/o you in person though. | 14:37 |
| rloo | milan: when you're in a cozy pub, you won't miss anyone as long as there is someone there :D | 14:38 |
| milan | TheJulia, yeah, mgould dtantsur and I have been "midcycling" since Monday already :D | 14:38 |
| TheJulia | rloo: no problem on the rechecks | 14:38 |
| milan | rloo, it's about _the_ company ;) | 14:38 |
| rloo | milan: let us know if you've solved all of ironic's problems! | 14:38 |
| TheJulia | Can we have a midcycle someplace with cozy fireplaces? | 14:38 |
| vdrok | milan: will you propose a diagram of beer pubs as the outcome of the midcycle? :) | 14:39 |
| milan | rloo, sure! at leas all the insufficient beer problems :P | 14:39 |
| TheJulia | vdrok: or perhaps a pub crawl plan that minimizes exposure to cold? :) | 14:39 |
| milan | vdrok, good idea! I bet someone already did that, though :) | 14:40 |
| vdrok | milan: TheJulia yah, state machine, 'pause', 'drink', 'continue' | 14:40 |
| * rloo is reminded of that street/pub crawl in austin, and shudders | 14:40 | |
| milan | vdrok, ack :) | 14:41 |
| vdrok | has anyone seen "No module named gdbm" on tox runs? started seeing it just today, tox -r does not help too | 14:42 |
| milan | rloo, can be exhausting after couple of days, I agree, but I still find it worth it ;) ...the crawl... isn't that a swimming style too? | 14:42 |
| rloo | milan: yup, swimming style and also what babies love to do :) | 14:43 |
| *** mtanino has joined #openstack-ironic | 14:43 | |
| milan | rloo, got it thanks! :D | 14:44 |
| TheJulia | vdrok: I started seeing that last night, but no ideas | 14:44 |
| vdrok | TheJulia: my only idea is - I did run tox -epy35 today, to run it, I deleted .testrepository/times.dbm | 14:45 |
| vdrok | and now I get this on tox -epy27 | 14:45 |
| vdrok | tox -epy35 runs ok | 14:46 |
| vdrok | maybe I have to remove it both when switching from 27 to 35 and from 35 to 27 | 14:46 |
| TheJulia | vdrok: exactly the same for me | 14:46 |
| TheJulia | vdrok: except I completely nuked .testrepository | 14:48 |
| vdrok | TheJulia: yup, just removing that times.dbm helps | 14:48 |
| vdrok | which is a degradation - previously you had to remove it only when running 35 after 27, now both ways :) | 14:49 |
| rloo | vdrok: question about https://review.openstack.org/#/c/396355/. was the serialization the only problem, and/or would the msg string also have been a problem? (i'm guessing both) | 14:49 |
| patchbot | patch 396355 - ironic - Make all IronicExceptions RPC-serializable | 14:49 |
| vdrok | rloo: nope, only serialization. a class raising error in __str__ to make it non-string-convertible is just a hypothetical thing, never seen this in ironic | 14:50 |
| vdrok | tho technically possible, yes | 14:50 |
| rloo | vdrok: in that case, why remove kwargs[k] for nonserializable values? | 14:51 |
| *** krtaylor has joined #openstack-ironic | 14:51 | |
| vdrok | but that patch should take care of that too, as such things are removed from formatter kwargs too | 14:51 |
| vdrok | rloo: ^^ :) | 14:51 |
| *** devlaps has joined #openstack-ironic | 14:51 | |
| rloo | vdrok: there is already a try/except for it, starting at L110: https://review.openstack.org/#/c/396355/10..12/ironic/common/exception.py | 14:52 |
| patchbot | patch 396355 - ironic - Make all IronicExceptions RPC-serializable | 14:52 |
| rloo | vdrok: so we should just leave it alone? | 14:52 |
| vdrok | rloo: nope, see L116 :) | 14:53 |
| vdrok | it will fail there once again | 14:53 |
| rloo | vdrok: sigh. | 14:53 |
| vdrok | :D | 14:53 |
| rloo | vdrok: i don't like how this code is not easy to understand | 14:54 |
| vdrok | rloo: and, as I understand, I'll have to propose the same thing to ironic-lib | 14:54 |
| *** chlong has joined #openstack-ironic | 14:55 | |
| rloo | vdrok: i think there had been a discussion at some point about using the IronicException from ironic-lib. | 14:55 |
| vdrok | yup | 14:55 |
| *** ohamada has quit IRC | 14:56 | |
| *** ohamada has joined #openstack-ironic | 14:56 | |
| *** vsaienko has joined #openstack-ironic | 14:56 | |
| *** jaosorior has quit IRC | 14:57 | |
| *** betherly has quit IRC | 14:59 | |
| *** jaosorior has joined #openstack-ironic | 14:59 | |
| dtantsur | rloo, morning! left some responses in https://review.openstack.org/#/c/401311/, please check before I jump into fixing smaller things. | 14:59 |
| patchbot | patch 401311 - ironic - Move interface validation from API to conductor side | 14:59 |
| jlvillal | vdrok, What did I break? :( | 15:00 |
| rloo | hi dtantsur! will look in a few secs | 15:00 |
| *** rbartal has joined #openstack-ironic | 15:00 | |
| *** anush has joined #openstack-ironic | 15:00 | |
| vdrok | morning jlvillal , we did enable_plugin twice in ironic grenade :) | 15:00 |
| vdrok | fixed now | 15:00 |
| jlvillal | vdrok, Oh. I am curious how it worked before. In my case I had seen that and it caused failures. | 15:01 |
| jlvillal | vdrok, Why I wrote the patch because it wasn't obvious why it failed. | 15:01 |
| dtantsur | vsaienk0, mind checking https://review.openstack.org/#/c/357983/ if/when you have a minute? | 15:01 |
| patchbot | patch 357983 - ironic - Support names for {cleaning,provisioning}_network | 15:01 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 15:02 |
| *** glonlas_ has joined #openstack-ironic | 15:02 | |
| vdrok | jlvillal: some black magic there :) | 15:03 |
| vdrok | no idea why it did work | 15:03 |
| jlvillal | vdrok, vsaienk0 Thanks for fixing it. | 15:03 |
| * vdrok is just announcing fixes, vsaienk0 does them :) | 15:04 | |
| *** vsaienko has quit IRC | 15:05 | |
| jlvillal | heh | 15:05 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Grenade: test https://review.openstack.org/400690 | 15:05 |
| *** glonlas has quit IRC | 15:05 | |
| *** rbudden has joined #openstack-ironic | 15:06 | |
| TheJulia | rloo: looks like one of the rechecks failed, but the VM just didn't lot or possibly even power up. Can't be sure looking at the logs :( | 15:08 |
| jroll | xek: rloo: is someone already working on https://bugs.launchpad.net/ironic/+bug/1585141 ? | 15:09 |
| openstack | Launchpad bug 1585141 in Ironic "[RFE] Add new ironic-dbsync command online-data-migration" [Wishlist,Confirmed] - Assigned to Grzegorz Grasza (xek) | 15:09 |
| rloo | TheJulia: sigh. I was wondering yesterday, if there was something going on with the gate. guess it could be the code too... | 15:09 |
| jroll | xek: rloo: we need that done like ASAP if we want to get rolling upgrades happening this cycle | 15:09 |
| jroll | :/ | 15:09 |
| *** anush has quit IRC | 15:10 | |
| rloo | jroll: thx for the reminder. why asap as opposed to any time in ocata cycle? | 15:10 |
| *** Nisha_ has joined #openstack-ironic | 15:11 | |
| jroll | rloo: because the ocata cycle is pretty short, and that is called out as a dependency :) | 15:11 |
| rloo | jroll: got it. there was another question that came up. the changes to grenade to test it. if need be, can they be backported to stable/newton. jlvillal, did i get that right? | 15:12 |
| jlvillal | rloo, I need to ask sdague about that. | 15:12 |
| TheJulia | rloo: no heartbeat, no console log :( Going to dig through the rest of the logs on the db_id change. | 15:12 |
| jlvillal | rloo, I think if they are changes in ironic, we shouldn't have a problem backporting them. | 15:13 |
| jlvillal | Well it is my hope | 15:13 |
| jroll | jlvillal: rloo: I don't think anyone would block backports, if they're needed to get this done | 15:13 |
| openstackgerrit | Merged openstack/ironic: Rely on portgroup standalone_ports_supported https://review.openstack.org/360647 | 15:14 |
| jlvillal | Woot!, nice merge :) | 15:14 |
| *** yufei__ has joined #openstack-ironic | 15:14 | |
| mariojv | \o/ | 15:18 |
| TheJulia | rloo: yeah, no console log, node was powered up, bridge ports for VM went into forwarding state. Guessing maybe qemu fell down... | 15:18 |
| *** glonlas_ has quit IRC | 15:19 | |
| rloo | TheJulia: thx for looking. We should keep an eye on that, might be happening more often than we think/want :-( | 15:19 |
| *** dprince has joined #openstack-ironic | 15:21 | |
| jroll | if there's a unique log (or two) we can get, elastic-recheck is perfect for that | 15:21 |
| rloo | dtantsur: i commented on 401311, but i realized that i haven't reviewed the test changes yet. | 15:23 |
| *** anush has joined #openstack-ironic | 15:23 | |
| TheJulia | rloo: http://logs.openstack.org/73/403773/3/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/1ecd1ac/logs/libvirt/qemu/node-0.txt.gz boom | 15:23 |
| jroll | oh my | 15:24 |
| rloo | TheJulia: kind of beautiful in its own way :-( | 15:25 |
| dtantsur | rloo, I see. wanna do it now or should I submit whatever fixes I have? | 15:25 |
| rloo | dtantsur: i hate reviewing tests. ok, give me 30 min. i might be faster. have to brace myself... | 15:25 |
| dtantsur | k | 15:25 |
| TheJulia | "KVM: entry failed, hardware error 0x0" perhaps? | 15:25 |
| *** glonlas has joined #openstack-ironic | 15:26 | |
| jroll | TheJulia: elastic-search operates by files, though, so 1) need to make sure qemu logs are indexed, and 2) need to see if we can wildcard that node-0 bit | 15:26 |
| jroll | :/ | 15:26 |
| TheJulia | I _thought_ all logs that get saved get indexed, but I've truthfully not mucked with upstream elastic-recheck, only what we had downstream at hpe. | 15:27 |
| *** srobert has joined #openstack-ironic | 15:27 | |
| jroll | yeah, not sure, could ask -qa folks | 15:27 |
| dtantsur | anybody else wants to skim through https://review.openstack.org/#/c/401311/ before I create another patchset? | 15:28 |
| patchbot | patch 401311 - ironic - Move interface validation from API to conductor side | 15:28 |
| * TheJulia goes and finds some breakfast | 15:28 | |
| jlvillal | JayF, jroll, lucasagomes dtantsur: Since my last project-config change broke ironic-inspector. | 15:28 |
| jlvillal | Be nice if you could look at: https://review.openstack.org/404479 | 15:28 |
| patchbot | patch 404479 - openstack-infra/project-config - Disable devstack plugin for builder 'ironic-grenade' | 15:28 |
| *** rbartal has quit IRC | 15:28 | |
| TheJulia | jlvillal: good morning! | 15:29 |
| jlvillal | TheJulia, Good morning :) Go find some breakfast! | 15:29 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc portgroup comands https://review.openstack.org/385427 | 15:29 |
| * jlvillal needs to do the same. Eggs and bacon is the likely option. | 15:29 | |
| mariojv | boot_index has to be positive for a volume target, right? https://review.openstack.org/#/c/285218/42/ironic/db/sqlalchemy/alembic/versions/1a59178ebdf6_add_volume_targets_table.py | 15:38 |
| patchbot | patch 285218 - ironic - Add volume_targets table to database | 15:38 |
| mariojv | non-negative rather | 15:38 |
| mariojv | 0, 1, 2... | 15:38 |
| openstackgerrit | Daniel Mellado proposed openstack/ironic: Skip tempest tests if ironic isn't available https://review.openstack.org/379256 | 15:39 |
| *** rbartal has joined #openstack-ironic | 15:39 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/pyghmi: IpmiServer to allow using IP version 4 https://review.openstack.org/404837 | 15:40 |
| *** gcb has quit IRC | 15:40 | |
| lindycoder | Good morning everyone, xhku and i are working on the integration of VirtualPDU to have tests against the SNMP Power Driver, our goal is to have a gate for an snmp driver : https://review.openstack.org/#/c/404830/ and then introduce the devstack support for it, is this the right way to proceed? | 15:41 |
| patchbot | patch 404830 - openstack-infra/project-config - Introduce an experimental gate for the snmp power ... | 15:41 |
| *** rpioso has joined #openstack-ironic | 15:42 | |
| mariojv | i wonder if there's a reason we don't just use cascading deletes with certain foreign key relationships in ironic. compatibility maybe? | 15:43 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Forbid passing portgroup_uuid to POST /ports in old API https://review.openstack.org/404839 | 15:43 |
| *** Nisha_ has quit IRC | 15:44 | |
| *** moshele has quit IRC | 15:45 | |
| *** Nisha_ has joined #openstack-ironic | 15:45 | |
| *** vgadiraj has joined #openstack-ironic | 15:45 | |
| *** Sukhdev has joined #openstack-ironic | 15:46 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Forbid passing portgroup_uuid to POST /ports in old API https://review.openstack.org/404839 | 15:47 |
| TheJulia | mariojv: non-negative, 0 is expected to always be the boot volume from nova based on their data structure | 15:47 |
| *** yufei__ has quit IRC | 15:48 | |
| *** yufei has quit IRC | 15:48 | |
| mariojv | ack, thanks TheJulia | 15:48 |
| TheJulia | mariojv: I suspect so. Cascading deletes can be done with postgres and mysql, but bdb I'm not sure about... not that anyone should use bdb in prod | 15:48 |
| lucasagomes | jlvillal, will take a look | 15:49 |
| lucasagomes | mat128, if you have some time: https://review.openstack.org/404837 | 15:51 |
| patchbot | patch 404837 - pyghmi - IpmiServer to allow using IP version 4 | 15:51 |
| *** yufei has joined #openstack-ironic | 15:51 | |
| *** Syed_ has joined #openstack-ironic | 15:51 | |
| vdrok | leaving early today, good night everyone! | 15:52 |
| rloo | byt vdrok | 15:52 |
| rloo | dtantsur: finished reviewing 401311. a few minor things. | 15:53 |
| TheJulia | jroll: so yeah, looks like the list of files actually load into ES is short. I've asked -qa to see if they can point us in the right direction. An argument could be made to not index it as we have console control character jibberish which won't index. At least we could search by file name and then get ES to scan the content from there. :\ | 15:55 |
| *** yufei has quit IRC | 15:56 | |
| jroll | TheJulia: ouch, thanks | 15:56 |
| dtantsur | rloo, thanks! responded | 15:56 |
| rloo | jroll, jlvillal: maybe you can answer lindycoder ^^ (about 15 min ago) question about testing | 15:56 |
| jroll | lindycoder: you'll get people telling you it's a job, not a gate, but yeah that's the way to go | 15:58 |
| jroll | sorry, missed that, trying to focus on a spec | 15:58 |
| *** shuquan has quit IRC | 15:58 | |
| *** jaosorior has quit IRC | 15:58 | |
| *** Sukhdev_ has joined #openstack-ironic | 15:58 | |
| *** hshiina has quit IRC | 15:59 | |
| lindycoder | jroll, I'm not sure what is the difference between a job and a gate, ultimately we hope this becomes a permanent check or a gate, is it about the name of the job? | 15:59 |
| jroll | lindycoder: the 'gate' is the set of jobs that run before merge, the 'check queue' is the set of jobs that run when you upload a patch, a 'job' is an individual test. this is just part of your commit message, the code looks fine | 16:00 |
| lindycoder | jroll, Ah i see thanks, maybe i'll wait to see if someone requires a change | 16:01 |
| jroll | lindycoder: yeah, that's fine :) | 16:02 |
| jroll | I +1'd the patch fwiw | 16:02 |
| *** rcernin has quit IRC | 16:02 | |
| lindycoder | Yeah i saw thank you! | 16:02 |
| *** chlong has quit IRC | 16:03 | |
| *** hw_wutianwei has quit IRC | 16:03 | |
| jroll | rloo: okay, I got the rolling upgrades spec reviewed, I think I agree with having the versioning column there | 16:06 |
| jroll | comments have my reasoning | 16:06 |
| rloo | jroll: ok | 16:06 |
| rloo | jroll: it does seem more flexible | 16:06 |
| jroll | rloo: yeah, and I think it's the right path to get to zero-downtime upgrades later | 16:07 |
| rloo | jroll: ah, i hadn't even thought about zero downtime! | 16:07 |
| jroll | this might actually give us zero downtime immediately, if we figure out how to test that :) | 16:08 |
| *** xhku_ has quit IRC | 16:08 | |
| *** Sukhdev_ has quit IRC | 16:09 | |
| sambetts | jroll: DoS like attack fired at ironic while also performing an upgrade? and see if we drop any requests? | 16:09 |
| rloo | jroll: that would be sweet | 16:09 |
| jroll | sambetts: heh, basically, probably also want to have an instance build ongoing, etc | 16:09 |
| *** xhku has joined #openstack-ironic | 16:09 | |
| JayF | sambetts: that's essentially what we used to do downstream; start our QA tests, start deploying, and the QA tests had to still pass | 16:09 |
| sambetts | heh cool! | 16:10 |
| rloo | jlvillal: wrt rolling upgrades, let us know what you think wrt the tests. that is the only -1 now: https://review.openstack.org/#/c/299245/ | 16:11 |
| patchbot | patch 299245 - ironic-specs - Support rolling upgrades | 16:11 |
| *** glonlas has quit IRC | 16:11 | |
| *** mjura has quit IRC | 16:16 | |
| *** jjohnson2 has joined #openstack-ironic | 16:16 | |
| *** mjura has joined #openstack-ironic | 16:16 | |
| jjohnson2 | lucasagomes, hey, hope my comment was easy, should make it quick and easy change to get what you want | 16:16 |
| TheJulia | rloo: jroll: possible culprit for that failure is kvm nested virt issues. Since our devstack script now tries to use nested virt if it appears available. Guess it really is just something we will need to look through some of the failures for | 16:18 |
| lucasagomes | jjohnson2, hi there! Yes, thanks for the review btw | 16:18 |
| *** chlong has joined #openstack-ironic | 16:18 | |
| lucasagomes | I will make the changes soon-ish :-) | 16:18 |
| * lucasagomes is in a quick call atm | 16:18 | |
| jroll | TheJulia: orly, interesting | 16:18 |
| jroll | TheJulia: the provider might be an interesting data point for those as well | 16:19 |
| TheJulia | jroll: in this case, osic | 16:19 |
| jroll | TheJulia: funny story, I was just getting excited in a downstream channel about using nested kvm in that cloud... | 16:19 |
| jjohnson2 | lucasagomes, yeah, didn't know how familiar with socket programming you were, so put in the getaddrinfo just in case (should have been that way by all rights to begin with) | 16:19 |
| rloo | TheJulia: hmm. wonder if that is worth noting in our etherpad. | 16:19 |
| *** rajinir has joined #openstack-ironic | 16:19 | |
| jjohnson2 | I must have been lazy | 16:20 |
| cloudnull | hahaha. TheJulia jroll and I were just talking about that | 16:20 |
| JayF | nice catch TheJulia | 16:20 |
| cloudnull | if there are issues in the osic that need fixing or adjusting let me know . | 16:20 |
| NobodyCam | Good Morning Ironic'ers | 16:20 |
| jroll | cloudnull: :D | 16:20 |
| TheJulia | rloo: I think so, I'll not it as sporatic failures, and where to look. | 16:21 |
| jroll | cloudnull: TheJulia: one posted earlier was ovh fwiw | 16:21 |
| TheJulia | err, note | 16:21 |
| jroll | so it isn't "just" osic | 16:21 |
| cloudnull | kk. | 16:21 |
| jroll | time for errands and lunch, bbl | 16:21 |
| rloo | TheJulia: thx! | 16:21 |
| TheJulia | err, maybe it was another one that I was looking at that was osic, or I got them mixed up. :( | 16:21 |
| cloudnull | well if something in cloud1 is causing general pain let me know. happy to take a look | 16:22 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Fix takeover for AgentDeploy and PXEBoot https://review.openstack.org/404858 | 16:22 |
| *** vsaienko has joined #openstack-ironic | 16:23 | |
| JayF | dtantsur: I know you had dashbaords and such for triaging bugs, if you want to impart any of that info/process my way I'd appreciate it. Otherwise I'm just going to do searches in launchpad and go that way | 16:24 |
| sambetts | JayF: this is dtantsur's dashboard http://ironic-divius.rhcloud.com/ but I think I lost the link to the github page that had the code for it | 16:29 |
| rloo | JayF: I think dtantsur has links in the whiteboard | 16:29 |
| TheJulia | cloudnull: will do, I've made a note that hopefully people will read since the files in particular are not indexed in ES :( | 16:30 |
| rloo | JayF: http://ironic-divius.rhcloud.com/ | 16:30 |
| rloo | JayF: and thanks! :) | 16:30 |
| * JayF puts it beside his review dashboard bookmark | 16:32 | |
| JayF | thanks :D | 16:32 |
| *** vsaienko has quit IRC | 16:32 | |
| *** glonlas has joined #openstack-ironic | 16:34 | |
| jlvillal | Oh: https://techcrunch.com/2016/11/30/suse-buys-hpes-openstack-and-cloud-foundry-assets/ | 16:35 |
| * TheJulia just sighs | 16:37 | |
| lucasagomes | jjohnson2, I would still need to figure out if the address being passed in the "server" parameter is IPv4 right ? Cause, otherwise I get a: http://paste.openstack.org/show/590980/ | 16:38 |
| rloo | jlvillal: very interesting... | 16:41 |
| JayF | For new power drivers, we almost always require a spec, right? I'd imagine especially so now that we requrie CI? | 16:41 |
| rloo | JayF: that's a good question. I don't know. even if we require CI, I don't see why that'd need a spec? but maybe we do require a spec. i suppose it depends on whether the RFE has enough detail in it... | 16:43 |
| *** jkilpatr has quit IRC | 16:43 | |
| rloo | JayF: the spec (process) is a bit of a checklist, to make sure we didn't forget anything. | 16:44 |
| JayF | Let me ask it this way: does anyone object to me marking an RFE for a new power driver as needs-spec if the existing description is anemic | 16:44 |
| JayF | rloo: that's kinda how I view it too, not asking for more information, just making sure all we need is provided | 16:44 |
| *** jcoufal_ has quit IRC | 16:47 | |
| JayF | ..annnnd it already had a spec, not attached to the bug | 16:47 |
| JayF | that makes the question a lot easier :) | 16:47 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 16:47 |
| openstackgerrit | Merged openstack/ironic: Add RPCs to support volume connector operation https://review.openstack.org/214585 | 16:50 |
| * TheJulia dances a little | 16:51 | |
| *** trown is now known as trown|lunch | 16:52 | |
| *** user__ has quit IRC | 16:53 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/pyghmi: IpmiServer to allow using IP version 4 https://review.openstack.org/404837 | 16:53 |
| lucasagomes | jjohnson2, ^ | 16:53 |
| * dtantsur does not feel well, so he is going for some vodka | 16:58 | |
| sambetts | \o/ | 16:58 |
| *** rama_y has joined #openstack-ironic | 16:59 | |
| * rloo wonders about that midcycle in brno | 16:59 | |
| *** jkilpatr has joined #openstack-ironic | 16:59 | |
| *** ccamacho is now known as ccamacho|afk | 16:59 | |
| dtantsur | JayF, right, I'm using http://ironic-divius.rhcloud.com/ (patches are welcome) | 17:00 |
| *** e0ne has quit IRC | 17:03 | |
| *** mjura has quit IRC | 17:03 | |
| JayF | dtantsur: should I be setting status to Triaged or Confirmed after I'm done w/'em? | 17:04 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/pyghmi: Improve .gitignore https://review.openstack.org/404884 | 17:04 |
| dtantsur | JayF, Triaged | 17:05 |
| dtantsur | or WONTFIX ;) | 17:05 |
| JayF | lol | 17:05 |
| JayF | and bugs that I mark Incomplete, leave importance unset, right? | 17:05 |
| dtantsur | right | 17:06 |
| JayF | How do I set bugs as impacting stable; such as https://bugs.launchpad.net/ironic/+bug/1566136 | 17:06 |
| openstack | Launchpad bug 1566136 in ironic (Ubuntu) "Mitaka ironic-conductor error on stop: Cannot switch to MAINLOOP from MAINLOOP" [Undecided,New] | 17:06 |
| dtantsur | JayF, there is something like "nominate for branch" | 17:07 |
| dtantsur | * "target to series" | 17:07 |
| JayF | that bug is pretty bananas though | 17:08 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands https://review.openstack.org/385427 | 17:12 |
| JayF | dtantsur: egad, I had no idea how awful some of these bugs were :( You should have gotten help doing all this ages ago | 17:15 |
| sambetts | JayF, dtantsur: we should organise a bug smash again soon | 17:15 |
| dtantsur | would be awesome :) | 17:19 |
| JayF | I'm holding a private bug smash right now | 17:20 |
| JayF | I volunteered to help | 17:20 |
| JayF | and have no meetings today | 17:20 |
| JayF | and nothing higher priority... | 17:20 |
| JayF | so I'm smashing until my brain starts to liquify from too much bug triaging. lol | 17:21 |
| *** chlong has quit IRC | 17:21 | |
| TheJulia | rloo: CI has finally shined a favorable light on https://review.openstack.org/#/c/403773/ | 17:21 |
| patchbot | patch 403773 - ironic - Follow-up for volume connector db_id | 17:21 |
| sambetts | hahaha | 17:22 |
| rloo | TheJulia: thx for pinging me, crossing fingers... | 17:22 |
| *** pcaruana has quit IRC | 17:23 | |
| sambetts | ah fuuu... having to deal with merge conflicts regarding API versions is a pain in the ass, e.g. both the patch that merged and my patch are using the same API version now so I have to go and fix allll the places | 17:23 |
| *** Romanenko_K has quit IRC | 17:23 | |
| rloo | sambetts: OH. but it shouldn't be in too many places that you have to change? | 17:24 |
| JayF | I think it'd be less-bad if we didn't embed the API version into the constant referencing it :P | 17:25 |
| * dtantsur rebases his patch as well | 17:25 | |
| sambetts | rloo: I hope not, it just in places like where the RPC api has to match the conductor manager version | 17:25 |
| sambetts | its like eww | 17:26 |
| sambetts | :-P | 17:26 |
| rloo | sambetts: OH | 17:26 |
| sambetts | have to update my version notes and everything | 17:26 |
| TheJulia | That... is a pain :( | 17:26 |
| rloo | sambetts: we could try to be better at knowing what feature is using which version... if we think they are close to landing... | 17:27 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Move interface validation from API to conductor side https://review.openstack.org/401311 | 17:27 |
| dtantsur | rloo, rebased and hopefully fixed all the comments ^^^ | 17:27 |
| rloo | on the bright side, it means we have a lot of stuff/features happenin' :) | 17:27 |
| sambetts | :D yeah | 17:27 |
| rloo | thx dtantsur. will look in a few minutes. | 17:28 |
| *** jcoufal has joined #openstack-ironic | 17:28 | |
| TheJulia | rloo: part of the conundrum is then we would have to stack patches to avoid merge conflicts because we would hit the same line. :( | 17:30 |
| rloo | TheJulia: oh yeah. forget it. we'll have to suffer with the merge conflicts. | 17:31 |
| rloo | TheJulia: i guess we could just all review/land one feature at at time ;) | 17:31 |
| TheJulia | Yeah, necessary evil with the way we review and update revisions too | 17:31 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Support defining and loading hardware types https://review.openstack.org/336626 | 17:32 |
| sambetts | just realised how horrible this merge conflcit is too :( I split my big patch into 3 patches, and there is code thats been added to a conductor part which i remove in the second commit to use code I added in my first commit | 17:33 |
| sambetts | so this commit is conflciting with my second commit but I need to add additional code to my first one | 17:34 |
| *** chlong has joined #openstack-ironic | 17:34 | |
| TheJulia | The fun of care and feeding of long chains of patches | 17:35 |
| sambetts | aye | 17:35 |
| dtantsur | yep | 17:35 |
| dtantsur | even my two give me enough headache, as I also sometimes have to move code between them | 17:36 |
| *** penick has joined #openstack-ironic | 17:36 | |
| sambetts | sometime I really do wonder if we should do feature branches for long patch chains which are then squashed to be merged into master | 17:36 |
| dtantsur | maybe.. I suspect they have their downsides | 17:37 |
| dtantsur | ok, I'm calling it a day, see you | 17:37 |
| *** dtantsur is now known as dtantsur|afk | 17:37 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/pyghmi: IpmiServer to allow using IP version 4 https://review.openstack.org/404837 | 17:38 |
| mariojv | rloo: wrt https://review.openstack.org/#/c/285218/ - i meant non-negative - boot index can indeed be 0 | 17:44 |
| patchbot | patch 285218 - ironic - Add volume_targets table to database | 17:44 |
| *** milan has quit IRC | 17:44 | |
| mariojv | still staying +1 though, can be updated later after spec is done | 17:44 |
| rloo | mariojv: ok. i am wondering where to make that check/constraint. | 17:44 |
| mariojv | i wonder if nova / cinder enforce that also | 17:44 |
| mariojv | that might be a good place to check | 17:45 |
| rloo | mariojv: it seems to me that if we want it at the db level, we should do it now. but i'm fine if it isn't at the db level. | 17:45 |
| mariojv | i guess it'd be in nova, not cinder | 17:45 |
| mariojv | same | 17:45 |
| rloo | ok, then if we remember, we can make sure it is checked somewhere else. | 17:45 |
| mariojv | interestingly enough, there are tests in nova with boot index -1 https://github.com/openstack/nova/blob/4f91ed3a547965ed96a22520edcfb783e7936e95/nova/tests/unit/test_block_device.py#L42 | 17:46 |
| rloo | mariojv: i wonder if that is an indication that it isn't specified | 17:46 |
| rloo | mariojv: which makes me think we shouldn't constrain it at the db level, just in case a neg value might come in useful | 17:47 |
| mariojv | https://github.com/openstack/nova/blob/4f91ed3a547965ed96a22520edcfb783e7936e95/doc/source/block_device_mapping.rst#block-device-mapping-v2 | 17:47 |
| mariojv | "setting a negative value or None indicates that the device should not be used for booting" | 17:47 |
| *** derekh has quit IRC | 17:47 | |
| lucasagomes | folks, calling it a day | 17:48 |
| mariojv | given that this is just target info and could theoretically be used for non-BFV work, i think it's OK to allow negative values, then | 17:48 |
| lucasagomes | have a great evening all! | 17:48 |
| mariojv | night lucasagomes | 17:48 |
| *** lucasagomes is now known as lucas-afk | 17:48 | |
| rloo | bye lucasagomes | 17:48 |
| lucas-afk | o/ | 17:48 |
| mariojv | rloo: i'll add a comment about what negative values indicate to that patch then | 17:48 |
| rloo | mariojv: thx! | 17:48 |
| mariojv | np | 17:49 |
| jlvillal | vdrok: Are you around? Trying to figure out some multi-tenant stuff. | 17:49 |
| *** aNuposic has joined #openstack-ironic | 17:51 | |
| rloo | jlvillal: vdrok left early today. about 2 hrs ago i think. | 18:03 |
| jlvillal | rloo: Ah thanks. | 18:03 |
| *** vgadiraj has quit IRC | 18:03 | |
| sambetts | jlvillal: is it a question I might be able to answer? | 18:03 |
| jlvillal | sambetts: Thanks. I'm trying to enable grenade with multi-tenant. | 18:03 |
| jlvillal | sambetts: It is running the tempest "smoke" test and failing. | 18:04 |
| jlvillal | sambetts: http://logs.openstack.org/68/389268/5/experimental/gate-grenade-dsvm-ironic-multitenant-nv/b1a7d3e/logs/grenade.sh.txt.gz | 18:04 |
| jlvillal | sambetts: We don't run that test in our multi-tenant job. | 18:04 |
| *** ralonsoh has quit IRC | 18:04 | |
| jlvillal | sambetts: My theory is that the tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops doesn't know about multi-tenant. Well I'm sure about that. | 18:05 |
| jlvillal | sambetts: But the ping part of the test works. It is the SSH that fails. So I'm a bit confused. | 18:05 |
| *** penick has quit IRC | 18:05 | |
| sambetts | jlvillal: hmm weird looking at the most recent VM log it appears its node-3 is trying to boot the ramdisk so maybe something is going weird in regards to whats booting on the node | 18:10 |
| mariojv | rloo: hey - when you were talking to aNuposic about backward compatibility with using node name with port creation, did you just mean making sure it's hidden behind a microversion? | 18:10 |
| mariojv | rloo: so that versions prior to allowing node name won't allow it? | 18:10 |
| sambetts | jlvillal: not sure which node it is trying to ssh into though | 18:10 |
| rloo | mariojv: i don't recall. give me a few secs to switch context. was this wrt a bug? | 18:10 |
| jlvillal | sambetts: Yeah. Me either. Trying to figure out the network situation. | 18:11 |
| aNuposic | rloo: this is the review link https://review.openstack.org/#/c/193439/ | 18:11 |
| patchbot | patch 193439 - ironic - Add port creation passing the name of the node ins... (ABANDONED) | 18:11 |
| *** Sukhdev has quit IRC | 18:11 | |
| *** penick has joined #openstack-ironic | 18:11 | |
| mariojv | rloo: not sure whether it'd be considered a feature or a bug | 18:11 |
| rloo | mariojv, aNuposic: I should have just taken over and fixed it at the time. sigh. More than a year later... | 18:12 |
| rloo | mariojv, aNuposic: I think you have to make sure that whatever changes you make, will still work with older API versions. So yes, you need to prevent older versions from using/knowing about node name. but 2. you have to make sure you don't break the code; it has to still accept node uuid or whatever. A common mistake is someone renaming the eg 'node_uuid' argument to 'node', which breaks API requests that have 'node_uuid'. Something like that, | 18:14 |
| rloo | i don't recall the details and if you make me look, i'll take over and fix it myself. | 18:14 |
| rloo | mariojv, aNuposic: if your testing is thorough, you'll know what I mean if a test fails | 18:15 |
| mariojv | rloo: got it, thanks | 18:16 |
| mariojv | rloo: no need to take a look yourself | 18:16 |
| mariojv | until it's ready for review again :) | 18:16 |
| *** soliosg has joined #openstack-ironic | 18:17 | |
| *** jkilpatr has quit IRC | 18:21 | |
| *** jjohnson2 has quit IRC | 18:24 | |
| *** jkilpatr has joined #openstack-ironic | 18:24 | |
| *** e0ne has joined #openstack-ironic | 18:27 | |
| *** Guest68279 is now known as rook | 18:27 | |
| aNuposic | rloo: Got it! I am working on that one. Thanks | 18:29 |
| rloo | aNuposic: :) | 18:30 |
| openstackgerrit | Merged openstack/ironic: Follow-up for volume connector db_id https://review.openstack.org/403773 | 18:32 |
| sambetts | jlvillal: IMO the simulation we are currently using is going to be a nightmare to maintain for grenade, /me plugs his devstack patch | 18:33 |
| *** vsaienko has joined #openstack-ironic | 18:33 | |
| jlvillal | sambetts: Is it passing Jenkins yet? :) | 18:33 |
| JayF | sambetts: I'd love at some point to get a walkthrough on how your patch works; I've never fully understood how all the networking pieces interact for devstack+ironic | 18:33 |
| sambetts | jlvillal: yup on normal jobs, however it fails the jobs which require changes in devstack-gate to allign with it | 18:34 |
| sambetts | JayF: Sure, I've tried to add some documentation as part of my patch too to help out with that situation too | 18:34 |
| jlvillal | sambetts: Are you proposing devstack-gate changes too? Maybe depends-on those in your patch... | 18:34 |
| JayF | I'll take another look at that patch then | 18:34 |
| aNuposic | JayF: Regarding https://bugs.launchpad.net/ironic/+bug/1642719 I was thinking to have unit tests. | 18:35 |
| openstack | Launchpad bug 1642719 in Ironic "[RFE] RADOSGW authentication mechanism for storing config drive in Ceph" [Wishlist,New] - Assigned to Anup (anup-d-navare) | 18:35 |
| JayF | aNuposic: you should put the comments in the bug so they persist :) but to be clear, I was asking more about functional/tempest tests | 18:35 |
| JayF | aNuposic: for instance, power drivers have unit tests, but we still are kicking out power drivers that don't have actual CI running on them | 18:36 |
| sambetts | jlvillal: I need to work out what is required, its the grenade, inspector and multitenant jobs which are failing right now, grenade and multitenant are failing because of the amount of hacking the existing simulation they do | 18:36 |
| sambetts | jlvillal: inspector I need to investigate | 18:36 |
| jlvillal | sambetts: Okay. I don't envy you... | 18:36 |
| sambetts | jlvillal: my patch does support multinode devstack out of the box now without requiring devstack-gate to set anything up for us | 18:37 |
| *** soliosg has quit IRC | 18:37 | |
| sambetts | jlvillal: but again we've added hacky stuff into devstack-gate which with my patch we don't need but it breaks my patch right now | 18:38 |
| jlvillal | sambetts: Cool. Though all the multi-node patches have landed and I think it is working now. | 18:38 |
| *** soliosg has joined #openstack-ironic | 18:38 | |
| *** soliosg has left #openstack-ironic | 18:38 | |
| aNuposic | JayF: Oh okay, I just had a thought to create a unit test, but let me see of how functional/tempest tests will work with that feature. Yes i will comment on that directly :) | 18:38 |
| *** trown|lunch is now known as trown | 18:39 | |
| *** mjturek has quit IRC | 18:41 | |
| sambetts | jlvillal: although theory is I should be able to get the multinode without multiteannt working I think | 18:41 |
| sambetts | jlvillal: I need to experiment | 18:41 |
| jlvillal | sambetts: The multinode experimental job is multinode with multitenant. Thus why I am trying to get grenade plus multitenant. | 18:42 |
| *** yufei_ has quit IRC | 18:42 | |
| jlvillal | sambetts: As end goal is grenade + multi-node (with multi-tenant) | 18:42 |
| sambetts | jlvillal: do we not have a multinode not multiteant job? | 18:42 |
| mariojv | JayF: do you know about what openstack policy would be like for using ceph with a tempest / functional test? | 18:42 |
| jlvillal | sambetts: Nope | 18:42 |
| mariojv | i haven't looked at all, but i doubt that we do that for the ceph image support | 18:42 |
| mariojv | aNuposic: ^ | 18:43 |
| sambetts | jlvillal: oh... oh well with my patch I've been testing that and it works out of the box | 18:43 |
| JayF | mariojv: we don't; but literally that's just a string change vs the swift temp url bits | 18:43 |
| jlvillal | sambetts: It was decided to do them together. | 18:43 |
| mariojv | JayF: aNuposic: it should be explicitly called out if it's decided that there shouldn't be tempest tests, though | 18:43 |
| JayF | mariojv: I have NFI what the policy is, but aiui there's a little history there | 18:43 |
| JayF | mariojv: yeah, I'm not saying there has to be (nor the inverse, yet), I'm saying it should be explicitly addressed one way or another | 18:43 |
| mariojv | ++ | 18:43 |
| mariojv | aNuposic: is that clear ^ ? | 18:44 |
| aNuposic | mariojv: yes some what. So right now it can run without CI, correct? | 18:45 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface REST APIs https://review.openstack.org/404240 | 18:46 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface RPC APIs https://review.openstack.org/404239 | 18:46 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Virtual Network Interface Driver APIs https://review.openstack.org/327046 | 18:46 |
| mariojv | aNuposic: it should just be explicitly stated in the RFE whether you plan to add tempest and functional tests | 18:47 |
| aNuposic | mariojv: Okay. Got it! | 18:48 |
| mariojv | :D | 18:48 |
| *** soliosg has joined #openstack-ironic | 18:49 | |
| *** yufei_ has joined #openstack-ironic | 18:51 | |
| *** soliosg has quit IRC | 18:51 | |
| *** soliosg has joined #openstack-ironic | 18:51 | |
| *** mjturek has joined #openstack-ironic | 18:54 | |
| rloo | jlvillal: wrt testing for rolling upgrades: https://review.openstack.org/#/c/299245/19/specs/approved/support-rolling-upgrade.rst | 18:57 |
| patchbot | patch 299245 - ironic-specs - Support rolling upgrades | 18:57 |
| jlvillal | rloo: Yes | 18:57 |
| rloo | jlvillal: i was wondering. instead of specifying api/conductor on primary node, and on subnode | 18:57 |
| *** penick has quit IRC | 18:57 | |
| rloo | jlvillal: would it be better to word it wrt the actual tests we want to see | 18:57 |
| rloo | jlvillal: old api + new conductor, new api + new conductor | 18:57 |
| jlvillal | rloo: I think it would be good to state the order things should be done. | 18:58 |
| jlvillal | rloo: And I believe it should be conductor first and then API. | 18:58 |
| rloo | jlvillal: or maybe: 1. old api+oldconductor -> oldapi+newconductor. 2. newapi+newconductorpinned -> newapi+newconductor unpinned. or whatever. | 18:59 |
| jlvillal | rloo: So old API on both, 1 new conductor + 1 old conductor. | 18:59 |
| rloo | jlvillal: i've already forgotten what we are testing. | 18:59 |
| rloo | jlvillal: and how/which nodes are running which services can be worked out later. | 18:59 |
| jlvillal | rloo: which nodes are running which isn't that important. I think. At least not in the spec. | 19:00 |
| rloo | jlvillal: cuz the question (I think) seems to be that it isn't possible to run two api services and something that i am not clear on | 19:00 |
| jlvillal | rloo: But what versions we are running is probably important. | 19:00 |
| jlvillal | rloo: I think it is possible to run two API services. | 19:00 |
| rloo | jlvillal: exactly. which versions. that's what i mean by oldapi+oldconductor, etc. i didn't specify which nodes they were running on. | 19:01 |
| jlvillal | rloo: But the problem is forcing requests to go to one over the other | 19:01 |
| *** penick has joined #openstack-ironic | 19:01 | |
| jlvillal | rloo: Which I think we should be able to do. We just say connect to API server at this URL and don't use haproxy. I would think that would be possible... | 19:02 |
| rloo | jlvillal: so is it necessary to have two api services running at the same time, or can we do the testing in such a way for the coverage we want, with only one API service running. | 19:02 |
| jlvillal | rloo: I'm not sure. I kind of feel we want to simulate how a rolling upgrade would work. | 19:02 |
| *** vsaienko has quit IRC | 19:02 | |
| rloo | jlvillal: so my point is, wrt the spec, why spin our wheels about whether it is possible to run two API services and direct requests to which service. that's a technical issue. | 19:02 |
| *** Nisha_ has quit IRC | 19:02 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: IpmiServer to allow using IP version 4 https://review.openstack.org/404837 | 19:03 |
| *** Nisha_ has joined #openstack-ironic | 19:03 | |
| jlvillal | rloo: So an operator has multiple API services running. They want to upgrade them one at a time. We want to make sure that running old API and new API doesn't break anything. | 19:03 |
| rloo | jlvillal: right. but you don't need them both running at the same time, to test. i dont' think. unless there is some corner case. | 19:04 |
| jlvillal | rloo: Right, but I think we should have two API services running. How we test it is a technical thing. | 19:04 |
| jlvillal | rloo: They both access the same database... | 19:04 |
| *** vsaienko has joined #openstack-ironic | 19:04 | |
| rloo | jlvillal: ? sure, but you can replicate the same database for both scenarios. | 19:04 |
| jlvillal | rloo: I still think we want two of each service running. I'm fine with saying that. | 19:05 |
| jlvillal | rloo: And that conductor should be upgraded before API | 19:05 |
| jlvillal | rloo: And that 1 conductor at a time should be upgraded. | 19:06 |
| rloo | jlvillal: the spec itself says that the conductor should be upgraded before api. | 19:06 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation https://review.openstack.org/392959 | 19:06 |
| jlvillal | rloo: As far as API, I'm okay for leaving it as a TBD for how to upgrade the API services. | 19:06 |
| rloo | jlvillal: don't know how much detail you want spelled out in the test section but please mention it all. i don't want to go back and forth on just that one part. i would like to land it soon so please comment. | 19:07 |
| jlvillal | rloo: Okay. I don't mind redundancy in saying it twice. But fine not. | 19:07 |
| jlvillal | rloo: I did just comment. | 19:07 |
| jlvillal | About 15 minutes ago. | 19:07 |
| jlvillal | rloo: Just that I think it should state two of each service. One on each node. | 19:08 |
| *** vsaienko has quit IRC | 19:08 | |
| rloo | jlvillal: that's what it used to say (2 services per node) until michal said it wasn't possible to test that with existing infrastructure. | 19:09 |
| jlvillal | rloo: We do the impossible every day ;) | 19:09 |
| * sambetts -> home | 19:10 | |
| rloo | jlvillal: we do? | 19:10 |
| sambetts | night all | 19:10 |
| jlvillal | rloo: I'm not sure I believe that it isn't possible. | 19:10 |
| jlvillal | sambetts: Good night! | 19:10 |
| sambetts | o/ | 19:10 |
| rloo | jlvillal: honestly, i don't care. i know we will do the right thing, and we'll update the spec if we have to if we encounter issues wrt the testing. i just want to get it landed so we can move on. | 19:10 |
| *** sambetts is now known as sambetts|afk | 19:10 | |
| rloo | jlvillal: oh, i am not doubting that. i just don't know that i do the impossible every day :) | 19:10 |
| rloo | bye sambetts|afk! | 19:11 |
| jlvillal | heh | 19:11 |
| jlvillal | rloo: Well running two of each service is definitely possible. Now testing upgrading only one API and being sure on the tests is less certain. | 19:11 |
| jlvillal | rloo: But I guess I am okay with landing it as it is worded now. | 19:12 |
| rloo | jlvillal: WHAT, after our discussion? You're ok with what it sez? | 19:12 |
| jlvillal | rloo: Since I already have my ideas on the goal for doing the testing. | 19:12 |
| jlvillal | rloo: Well you wanted to get it landed and stated we could change it later ;) | 19:13 |
| rloo | jlvillal: that works for me too. saves me trying to add another comment there. thx! | 19:13 |
| jlvillal | rloo: I would prefer the line to be updated. | 19:13 |
| rloo | jlvillal: let's go with your preference then. would you mind suggesting the wording? | 19:13 |
| jlvillal | rloo: But it isn't critical. Seems like we are okay with updating specs as we learn more. Which is a good thing. | 19:13 |
| jlvillal | rloo: Okay I will write a sentence up | 19:14 |
| rloo | jlvillal: yeah, we've updated the portgroups stuff in the spec several times, the two notifications ones, ... :) | 19:14 |
| rloo | thx jlvillal | 19:14 |
| *** soliosg has quit IRC | 19:14 | |
| jlvillal | rloo: * The primary node and sub node will each run both the ironic-api and ironic-conductor services. | 19:17 |
| jlvillal | how does that sound? | 19:17 |
| *** snecklifter has joined #openstack-ironic | 19:17 | |
| openstackgerrit | Merged openstack/pyghmi: Improve .gitignore https://review.openstack.org/404884 | 19:17 |
| rloo | jlvillal: fine to me. | 19:17 |
| rloo | jlvillal: am wondering if we want to mention the issue/unknown wrt running a deterministic test with two API services | 19:18 |
| JayF | two apis is going to be a lot of headache to implement | 19:19 |
| JayF | going to have to reconfigure clients to point to one or the other as needed and/or setup an lb | 19:19 |
| rloo | jlvillal: some sort of caveat, if there is an issue... we will re-evaluate the test plan...? | 19:19 |
| rloo | JayF: yeah. but jlvillal wants to try to do the impossible :) | 19:19 |
| JayF | any objection to my marking this RFE as approved? https://bugs.launchpad.net/ironic-python-agent/+bug/1590935 | 19:20 |
| openstack | Launchpad bug 1590935 in ironic-python-agent "RFE: DIB image should be officially supported in IPA" [Wishlist,Triaged] | 19:20 |
| JayF | I filed it so I probably shouldn't approve it :D | 19:20 |
| rloo | JayF: if you have some time to spare, i think the rolling upgrades spec is pretty close. would be good to get your eyes on it, in case we missed something: https://review.openstack.org/299245 | 19:21 |
| patchbot | patch 299245 - ironic-specs - Support rolling upgrades | 19:21 |
| jlvillal | JayF: Yeah, it probably won't be easy. It is a goal, but if we decide too difficult then okay. | 19:22 |
| JayF | I'll look at it, I doubt I'll have much feedback though ;) | 19:22 |
| rloo | JayF: looking at that rfe. i'm not sure we should approve w/o a bigger audience/discuss. | 19:22 |
| jlvillal | JayF: Also not really stating in the spec the gory details of everything. Just upgrade conductor and then upgrade API. | 19:22 |
| JayF | rloo: OK, we've talked about it quite a bit in IRC but folks working on it aren't in OpenStack anymore :( | 19:23 |
| jlvillal | Actually we only really say upgrade conductor. | 19:23 |
| rloo | JayF: there's a lot to wade through in that rfe. let me read it carefully. | 19:23 |
| *** Sukhdev has joined #openstack-ironic | 19:24 | |
| rloo | JayF: guess I'm being lazy. I don't see it clearly indicated in that rfe, the answer to the three questions you posed. | 19:24 |
| rloo | JayF: perhaps someone could update the description? | 19:25 |
| JayF | That's valid. It's probably OK sitting unapproved until someone has time to keep working on it and answer questions | 19:25 |
| *** vsaienko has joined #openstack-ironic | 19:25 | |
| JayF | the dependency list on that is really long, I know stephane was working on gettin DIB to support the ironic-agent element being in our tree | 19:26 |
| JayF | but that all got clogged up and now she's not working on Ironic anymore :( | 19:26 |
| rloo | JayF: I almost want to suggest adding a spec for it, but if someone could update the description to clearly indicate what will be done, i would be fine evaluating the rfe based on that. | 19:26 |
| *** SpamapS has quit IRC | 19:26 | |
| JayF | aight | 19:27 |
| *** linuxstuff has joined #openstack-ironic | 19:27 | |
| rloo | JayF: at least there are comments in that rfe :) | 19:27 |
| linuxstuff | Any ideas on how I can troubleshoot this issue Couldn't determine the UUID of the root partition or the disk identifier after deploying node ce3d11a0-2c09-4fc4-a6a5-91002625355e | 19:28 |
| *** vgadiraj has joined #openstack-ironic | 19:30 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers https://review.openstack.org/372428 | 19:34 |
| *** vsaienko has quit IRC | 19:35 | |
| *** betherly has joined #openstack-ironic | 19:36 | |
| *** vsaienko has joined #openstack-ironic | 19:37 | |
| *** vsaienko has quit IRC | 19:40 | |
| *** SpamapS has joined #openstack-ironic | 19:41 | |
| *** mjturek has quit IRC | 19:51 | |
| *** Nisha_ has quit IRC | 19:58 | |
| *** ijw has joined #openstack-ironic | 20:12 | |
| *** ijw has quit IRC | 20:17 | |
| *** ijw has joined #openstack-ironic | 20:20 | |
| *** srobert_ has joined #openstack-ironic | 20:25 | |
| *** soliosg has joined #openstack-ironic | 20:25 | |
| *** srobert_ has quit IRC | 20:28 | |
| *** jcoufal has quit IRC | 20:28 | |
| *** srobert has quit IRC | 20:29 | |
| *** ijw has quit IRC | 20:30 | |
| *** ijw has joined #openstack-ironic | 20:32 | |
| *** ohamada has quit IRC | 20:32 | |
| *** devlaps has quit IRC | 20:36 | |
| *** rcernin has joined #openstack-ironic | 20:37 | |
| *** jjohnson2 has joined #openstack-ironic | 20:39 | |
| jroll | jlvillal: rloo: left another comment on testing, I think we're all complicating things more than needed :) https://review.openstack.org/#/c/299245/19 | 20:43 |
| patchbot | patch 299245 - ironic-specs - Support rolling upgrades | 20:43 |
| rloo | thx jroll. will comment ... :) | 20:45 |
| jroll | :) | 20:46 |
| mrda | Morning Ironic | 20:50 |
| JayF | o/ | 20:50 |
| rloo | hi mrda | 20:53 |
| mrda | o/ | 20:54 |
| *** priteau has quit IRC | 21:03 | |
| *** dprince has quit IRC | 21:03 | |
| *** priteau has joined #openstack-ironic | 21:04 | |
| jroll | \o | 21:04 |
| *** ChubYann has joined #openstack-ironic | 21:04 | |
| *** anush has quit IRC | 21:06 | |
| *** MattMan has quit IRC | 21:07 | |
| *** srobert has joined #openstack-ironic | 21:07 | |
| *** MattMan has joined #openstack-ironic | 21:07 | |
| *** mjturek has joined #openstack-ironic | 21:08 | |
| *** priteau has quit IRC | 21:08 | |
| *** srobert has quit IRC | 21:13 | |
| *** srobert has joined #openstack-ironic | 21:13 | |
| *** ijw has quit IRC | 21:14 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: Testing 355586 https://review.openstack.org/397775 | 21:15 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/404436 | 21:15 |
| *** e0ne has quit IRC | 21:22 | |
| *** vsaienko has joined #openstack-ironic | 21:22 | |
| *** e0ne has joined #openstack-ironic | 21:23 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands https://review.openstack.org/385427 | 21:23 |
| *** e0ne has quit IRC | 21:24 | |
| *** e0ne has joined #openstack-ironic | 21:24 | |
| *** e0ne has quit IRC | 21:24 | |
| *** e0ne has joined #openstack-ironic | 21:25 | |
| jroll | Sukhdev: this needs a rebase when you have time https://review.openstack.org/#/c/393962/ | 21:25 |
| patchbot | patch 393962 - ironic - Documentation for Security Groups for baremetal se... | 21:25 |
| *** e0ne has quit IRC | 21:25 | |
| *** e0ne has joined #openstack-ironic | 21:25 | |
| jlvillal | jroll: Comment looks good on the rolling upgrade spec. Thanks. | 21:26 |
| *** e0ne has quit IRC | 21:26 | |
| jroll | jlvillal: awesome, np | 21:26 |
| *** e0ne has joined #openstack-ironic | 21:26 | |
| *** e0ne has quit IRC | 21:27 | |
| *** e0ne has joined #openstack-ironic | 21:27 | |
| *** e0ne has quit IRC | 21:27 | |
| *** e0ne has joined #openstack-ironic | 21:28 | |
| *** e0ne has quit IRC | 21:28 | |
| *** e0ne has joined #openstack-ironic | 21:29 | |
| *** e0ne has quit IRC | 21:29 | |
| *** devlaps has joined #openstack-ironic | 21:34 | |
| *** vsaienko has quit IRC | 21:35 | |
| *** vsaienko has joined #openstack-ironic | 21:35 | |
| *** jkilpatr has quit IRC | 21:38 | |
| *** ijw has joined #openstack-ironic | 21:39 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Default protocol prefix for ironic api address for IPA https://review.openstack.org/404975 | 21:45 |
| *** vsaienko has quit IRC | 21:46 | |
| *** vsaienko has joined #openstack-ironic | 21:47 | |
| *** vsaienko has quit IRC | 21:47 | |
| *** jtomasek has quit IRC | 21:51 | |
| *** somebody_else has joined #openstack-ironic | 21:52 | |
| *** [1]rpioso has joined #openstack-ironic | 21:56 | |
| *** cdearborn has joined #openstack-ironic | 21:59 | |
| *** rpioso has quit IRC | 21:59 | |
| *** [1]rpioso has quit IRC | 21:59 | |
| *** somebody_else is now known as romcheg | 22:00 | |
| *** jrist has quit IRC | 22:01 | |
| *** [1]cdearborn has quit IRC | 22:01 | |
| *** jrist has joined #openstack-ironic | 22:02 | |
| *** romcheg has quit IRC | 22:02 | |
| *** ijw has quit IRC | 22:03 | |
| *** romcheg has joined #openstack-ironic | 22:03 | |
| *** jrist has quit IRC | 22:07 | |
| *** ijw has joined #openstack-ironic | 22:08 | |
| *** ijw has quit IRC | 22:08 | |
| *** ijw has joined #openstack-ironic | 22:08 | |
| *** trown is now known as trown|outtypewww | 22:10 | |
| *** bfournie has quit IRC | 22:10 | |
| *** bfournie has joined #openstack-ironic | 22:11 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add testcases for osc port group comands https://review.openstack.org/385427 | 22:11 |
| *** bfournie has quit IRC | 22:11 | |
| *** ijw has quit IRC | 22:12 | |
| *** ijw has joined #openstack-ironic | 22:12 | |
| *** penick has quit IRC | 22:12 | |
| *** ijw has quit IRC | 22:15 | |
| *** ijw has joined #openstack-ironic | 22:16 | |
| *** ijw has quit IRC | 22:16 | |
| *** bfournie has joined #openstack-ironic | 22:16 | |
| *** ijw has joined #openstack-ironic | 22:16 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix setting custom IRONIC_VM_NETWORK_BRIDGE https://review.openstack.org/404987 | 22:17 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Remove unnecessary option from plugin settings https://review.openstack.org/404988 | 22:17 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Fix broken xenial job https://review.openstack.org/404989 | 22:17 |
| *** lindycoder has quit IRC | 22:18 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/389268 | 22:19 |
| JayF | jlvillal: I didn't know our newton jobs ran on xenial | 22:19 |
| jlvillal | JayF: Well the multitenant + grenade job does... | 22:19 |
| JayF | jlvillal: I'm mainly confused why that would be impacted, but not our normal grenade job | 22:20 |
| jlvillal | JayF: Because of the multi-tenancy, I think. | 22:20 |
| jlvillal | JayF: Anyway I am doing a job-run to find out. And the patches are Workflow -1 now | 22:21 |
| JayF | I just saw 'em pop up and was curious | 22:21 |
| JayF | you might want to be more clear in the commit messages as to what these are trying to fix if/when you want them merged | 22:21 |
| jlvillal | JayF: Well they are backports from master. So I left the message the same. | 22:22 |
| jlvillal | JayF: I'm not sure if I should change the message or not when doing a backport??? | 22:22 |
| jlvillal | I'm hopeful this fixes the issue we are seeing. Will find out in about an hour :) | 22:22 |
| JayF | Hmm. I would personally prefer it be more clear, but I'll look up the policy and see what's allowed | 22:22 |
| *** Goneri has quit IRC | 22:25 | |
| JayF | jlvillal: the policy is pretty clear that unless there's extenuating circumstances, the /code/ must be identical. I don't think it causes any harm to add additional comments to the commit message. | 22:25 |
| jlvillal | JayF: Do you know the webpage for searching all of the openstack code? | 22:25 |
| JayF | jroll: ^ would you agree, re: stable patches? if adding a couple of lines to the commit helps understand why it's needed better, that's positive and not negative/ | 22:25 |
| jlvillal | JayF: Okay. If it fixes the issue I will add more to the commit message. | 22:25 |
| JayF | jlvillal: Uh, I typically find the repository on github + search it, I haven't looked through *all* the code before | 22:26 |
| jlvillal | JayF: http://codesearch.openstack.org/ :) | 22:26 |
| *** glonlas_ has joined #openstack-ironic | 22:27 | |
| *** ijw has quit IRC | 22:30 | |
| *** glonlas has quit IRC | 22:30 | |
| openstackgerrit | Annie Lezil proposed openstack/ironic-inspector-specs: Collect NIC name given by BIOS https://review.openstack.org/404372 | 22:31 |
| *** glonlas_ has quit IRC | 22:31 | |
| jroll | JayF: I agree unless the policy says otherwise | 22:32 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: WIP/DNM: Trying to test grenade multi-tenant experimental job https://review.openstack.org/389268 | 22:33 |
| JayF | jroll: policy says nothing about commit message, just cautions strongly against having the code be different from master | 22:33 |
| jroll | then sure, verbosity never hurts | 22:33 |
| JayF | cool | 22:33 |
| *** ijw has joined #openstack-ironic | 22:33 | |
| *** ijw has quit IRC | 22:35 | |
| *** ijw has joined #openstack-ironic | 22:38 | |
| jlvillal | vgadiraj: ^^^ I did the backports and running a test with them now. Hopefully the 1450 MTU size will help. | 22:40 |
| *** ijw has quit IRC | 22:40 | |
| *** jheroux has quit IRC | 22:40 | |
| *** mnaser has quit IRC | 22:41 | |
| vgadiraj | jvillal: *fingers crossed* | 22:41 |
| jlvillal | vgadiraj: I was thinking. Maybe we should propose an experimental grenade xenial job? | 22:42 |
| openstackgerrit | Félix Bouliane proposed openstack/ironic: Add virtualpdu to ironic devstack plugin https://review.openstack.org/388154 | 22:42 |
| *** jrist has joined #openstack-ironic | 22:42 | |
| *** ijw has joined #openstack-ironic | 22:45 | |
| *** ijw has quit IRC | 22:48 | |
| *** cdearborn has quit IRC | 22:48 | |
| *** mnaser has joined #openstack-ironic | 22:50 | |
| *** ijw has joined #openstack-ironic | 22:51 | |
| *** [1]rpioso has joined #openstack-ironic | 22:53 | |
| *** rbudden has quit IRC | 22:53 | |
| *** ijw has quit IRC | 22:54 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: DNM: Testing 355586 https://review.openstack.org/397775 | 22:57 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Move CONF.service_available.ironic to our plugin https://review.openstack.org/404436 | 22:57 |
| *** athomas has quit IRC | 22:58 | |
| jroll | night y'all | 22:59 |
| openstackgerrit | Merged openstack/ironic: Install syslinux package only for Wheezy / Trusty https://review.openstack.org/264213 | 22:59 |
| JayF | o/ | 22:59 |
| openstackgerrit | Merged openstack/python-ironicclient: List required arguments in '--help' message in Ironic Client. https://review.openstack.org/382999 | 23:00 |
| *** priteau has joined #openstack-ironic | 23:04 | |
| *** priteau has quit IRC | 23:09 | |
| *** Sukhdev has quit IRC | 23:12 | |
| vgadiraj | jlvillal: might be a good idea | 23:19 |
| jlvillal | vgadiraj: I'm looking into it. Trying to figure out how to do it. | 23:20 |
| *** chlong has quit IRC | 23:22 | |
| *** pmannidi has joined #openstack-ironic | 23:25 | |
| *** clsacramento_ has quit IRC | 23:26 | |
| *** rcernin has quit IRC | 23:26 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic: Default protocol prefix for ironic api address for IPA https://review.openstack.org/404975 | 23:27 |
| *** clsacramento has joined #openstack-ironic | 23:28 | |
| *** jcoufal has joined #openstack-ironic | 23:29 | |
| *** ijw has joined #openstack-ironic | 23:36 | |
| *** penick has joined #openstack-ironic | 23:38 | |
| *** Goneri has joined #openstack-ironic | 23:40 | |
| *** ijw has quit IRC | 23:41 | |
| *** penick_ has joined #openstack-ironic | 23:42 | |
| *** penick has quit IRC | 23:43 | |
| *** penick_ is now known as penick | 23:43 | |
| *** jkilpatr has joined #openstack-ironic | 23:46 | |
| *** ijw has joined #openstack-ironic | 23:46 | |
| jlvillal | Did we do anything in regards to the goal: remove-incubated-oslo-code | 23:49 |
| * jlvillal is catching up on openstack-dev email reading | 23:49 | |
| jlvillal | Ah, I see it says it is already done for us. Never mind. | 23:50 |
| *** linuxstuff has quit IRC | 23:50 | |
| *** ijw has quit IRC | 23:51 | |
| *** vgadiraj has quit IRC | 23:55 | |
| JayF | jlvillal: we have nothing to do | 23:55 |
| JayF | jlvillal: yeah, exactly ;D | 23:55 |
| jlvillal | JayF: heh | 23:55 |
| jlvillal | JayF: On the IPA patch for default protocol. Would it be bad if it just failed instead of adding a default protocol? | 23:56 |
| JayF | jlvillal: I was thinking about that, and I'm not sure I'd dislike that | 23:56 |
| JayF | joanna: ^ fyi talking about your patch | 23:56 |
| JayF | jlvillal: like, I appreciate trying to make it work, but it already fails, and making it fail earlier would resolve the bug without making any assumptions | 23:56 |
| jlvillal | Yeah. I am kind of leaning towards liking failure more than sticking in a default protocol. | 23:57 |
| JayF | If you wanna make that comment, I'll update my vote and agree with you | 23:59 |
| JayF | I pondered that one for a while but when I'm 50/50 on something I usually let the implementer's choice win :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!