| *** sachin has quit IRC | 00:04 | |
| *** zhangjn has quit IRC | 00:05 | |
| *** veena has joined #openstack-nova | 00:05 | |
| *** agireud has quit IRC | 00:08 | |
| *** sachin has joined #openstack-nova | 00:09 | |
| *** r-daneel has joined #openstack-nova | 00:09 | |
| *** achanda has joined #openstack-nova | 00:11 | |
| *** armax has quit IRC | 00:13 | |
| *** mtanino_ has quit IRC | 00:13 | |
| *** tsekiyama has quit IRC | 00:14 | |
| *** armax has joined #openstack-nova | 00:17 | |
| *** bnemec has quit IRC | 00:18 | |
| *** achanda has quit IRC | 00:19 | |
| *** dave-mcc_ has joined #openstack-nova | 00:24 | |
| *** achanda has joined #openstack-nova | 00:25 | |
| *** agireud has joined #openstack-nova | 00:26 | |
| *** tpeoples has joined #openstack-nova | 00:27 | |
| *** dave-mccowan has quit IRC | 00:27 | |
| *** garyk has joined #openstack-nova | 00:28 | |
| *** xyang1 has quit IRC | 00:31 | |
| *** tpeoples has quit IRC | 00:36 | |
| *** agireud has quit IRC | 00:40 | |
| *** asd112z has joined #openstack-nova | 00:41 | |
| *** salv-orlando has quit IRC | 00:43 | |
| *** kylek3h has quit IRC | 00:46 | |
| *** sachin has quit IRC | 00:46 | |
| *** bnemec has joined #openstack-nova | 00:47 | |
| *** sachin has joined #openstack-nova | 00:48 | |
| *** zhangjn has joined #openstack-nova | 00:49 | |
| *** sachin has quit IRC | 00:49 | |
| *** sachin has joined #openstack-nova | 00:50 | |
| *** sachin_ has joined #openstack-nova | 00:50 | |
| *** sachin has quit IRC | 00:50 | |
| *** zhangjn has quit IRC | 00:51 | |
| *** Marga_ has joined #openstack-nova | 00:51 | |
| *** sachin_ has quit IRC | 00:52 | |
| *** sachin has joined #openstack-nova | 00:52 | |
| *** sachin has quit IRC | 00:53 | |
| *** armax has quit IRC | 00:53 | |
| *** zhangjn has joined #openstack-nova | 00:53 | |
| *** sachin has joined #openstack-nova | 00:53 | |
| *** zhangjn has quit IRC | 00:54 | |
| *** shoutm has joined #openstack-nova | 00:54 | |
| *** Marga_ has quit IRC | 00:54 | |
| *** Marga_ has joined #openstack-nova | 00:55 | |
| gmann | alex_xu: hi | 00:55 |
|---|---|---|
| *** zhangjn has joined #openstack-nova | 00:55 | |
| alex_xu | gmann: hi | 00:55 |
| *** andreykurilin__ has quit IRC | 00:55 | |
| gmann | alex_xu: https://review.openstack.org/#/c/214085/ need rebase on master as other tests merge patches has been merged | 00:56 |
| gmann | alex_xu: I know what all specific changes needed to remove | 00:56 |
| gmann | alex_xu: can i rebase https://review.openstack.org/#/c/215436/ on master and then paste.ini one? | 00:56 |
| alex_xu | gmann: sure, thanks for rebase | 00:57 |
| gmann | https://review.openstack.org/#/c/215436/ should not loose johnthetubaguy +2 :) | 00:57 |
| gmann | alex_xu: cool. i will do right now | 00:57 |
| *** shoutm_ has quit IRC | 00:57 | |
| alex_xu | gmann: cool | 00:57 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Strip the extra properties out when using legacy v2 compatible middleware https://review.openstack.org/215436 | 00:58 |
| *** Marga_ has quit IRC | 01:00 | |
| *** Marga_ has joined #openstack-nova | 01:00 | |
| *** skylerberg has quit IRC | 01:02 | |
| *** meghal has joined #openstack-nova | 01:03 | |
| *** veena has quit IRC | 01:04 | |
| *** zhenguo has joined #openstack-nova | 01:05 | |
| *** meghal has quit IRC | 01:05 | |
| *** yangyp has joined #openstack-nova | 01:06 | |
| *** lykinsbd_ has quit IRC | 01:09 | |
| *** thorst has quit IRC | 01:09 | |
| *** shoutm has quit IRC | 01:10 | |
| *** angdraug has quit IRC | 01:10 | |
| *** shoutm has joined #openstack-nova | 01:14 | |
| *** meghal has joined #openstack-nova | 01:15 | |
| *** SourabhP has quit IRC | 01:17 | |
| *** meghal has quit IRC | 01:17 | |
| *** asettle has joined #openstack-nova | 01:18 | |
| *** unicell has quit IRC | 01:19 | |
| *** vilobhmm1 has quit IRC | 01:23 | |
| *** jcoufal has joined #openstack-nova | 01:25 | |
| *** jcoufal has quit IRC | 01:29 | |
| *** ssurana has quit IRC | 01:30 | |
| *** dannywilson has quit IRC | 01:31 | |
| *** dannywilson has joined #openstack-nova | 01:32 | |
| *** r-daneel has quit IRC | 01:35 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: add support for muti_statuses_query from migration https://review.openstack.org/169631 | 01:35 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: DB:support for migration objects multi-statuses https://review.openstack.org/169604 | 01:35 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: resize-revert can break anti-affinity policy https://review.openstack.org/160110 | 01:35 |
| *** dannywilson has quit IRC | 01:36 | |
| *** dannywil_ has joined #openstack-nova | 01:36 | |
| *** dannywil_ has quit IRC | 01:41 | |
| *** bnemec has quit IRC | 01:41 | |
| *** dims has joined #openstack-nova | 01:43 | |
| *** tpeoples has joined #openstack-nova | 01:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Make query to quota usage table order preserved https://review.openstack.org/213584 | 01:50 |
| *** sachin has quit IRC | 01:50 | |
| *** tpeoples has quit IRC | 01:52 | |
| *** armax has joined #openstack-nova | 01:58 | |
| *** shuquan has joined #openstack-nova | 01:59 | |
| *** ndipanov has quit IRC | 02:00 | |
| *** haomaiwa_ has joined #openstack-nova | 02:03 | |
| *** jlk has left #openstack-nova | 02:04 | |
| *** haomaiwa_ has quit IRC | 02:09 | |
| *** mdrabe has joined #openstack-nova | 02:09 | |
| *** haomaiwang has joined #openstack-nova | 02:10 | |
| *** lyanchih has joined #openstack-nova | 02:16 | |
| *** summer has joined #openstack-nova | 02:17 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: api: tidy up paste.ini https://review.openstack.org/214085 | 02:17 |
| *** summer has quit IRC | 02:18 | |
| *** summerWendy has joined #openstack-nova | 02:18 | |
| gmann | alex_xu: johnthetubaguy : ^^ merged paste.ini one | 02:18 |
| alex_xu | gmann: thanks | 02:19 |
| gmann | alex_xu: np | 02:19 |
| *** shuquan has quit IRC | 02:20 | |
| *** Apoorva has quit IRC | 02:21 | |
| *** Marga_ has quit IRC | 02:28 | |
| *** skylerberg has joined #openstack-nova | 02:29 | |
| *** shuquan has joined #openstack-nova | 02:29 | |
| *** shuquan has quit IRC | 02:29 | |
| openstackgerrit | Sean McCully proposed openstack/nova: Creates disk.fallocate_test https://review.openstack.org/188249 | 02:30 |
| openstackgerrit | huan proposed openstack/nova: Do not rely on XenAPI to guess device names https://review.openstack.org/213112 | 02:32 |
| *** shuquan has joined #openstack-nova | 02:33 | |
| *** yamahata has quit IRC | 02:34 | |
| *** zzzeek has joined #openstack-nova | 02:34 | |
| *** shuquan_ has joined #openstack-nova | 02:40 | |
| *** shuquan has quit IRC | 02:40 | |
| *** shuquan_ is now known as shuquan | 02:40 | |
| *** mriedem has quit IRC | 02:43 | |
| *** salv-orlando has joined #openstack-nova | 02:44 | |
| *** sinisak has quit IRC | 02:46 | |
| *** zzzeek has quit IRC | 02:47 | |
| *** salv-orlando has quit IRC | 02:48 | |
| *** asd112z has quit IRC | 02:50 | |
| *** hakimo_ has joined #openstack-nova | 02:52 | |
| *** takedakn has joined #openstack-nova | 02:53 | |
| *** dave-mcc_ has quit IRC | 02:54 | |
| *** hakimo has quit IRC | 02:55 | |
| *** dims has quit IRC | 02:56 | |
| *** summerWendy has quit IRC | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| openstackgerrit | Huang Zhiteng proposed openstack/nova: libvirt: Force iSCSI disconnect after timeout https://review.openstack.org/167815 | 03:01 |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** kylek3h has joined #openstack-nova | 03:10 | |
| *** kylek3h has quit IRC | 03:15 | |
| *** kylek3h has joined #openstack-nova | 03:15 | |
| *** kylek3h has quit IRC | 03:20 | |
| *** shuquan has quit IRC | 03:21 | |
| *** liusheng has joined #openstack-nova | 03:22 | |
| *** shuquan has joined #openstack-nova | 03:32 | |
| *** EinstCrazy has joined #openstack-nova | 03:32 | |
| *** deepthi has joined #openstack-nova | 03:42 | |
| *** jgriffith has quit IRC | 03:43 | |
| *** jgriffith has joined #openstack-nova | 03:44 | |
| *** jgriffith is now known as Guest54162 | 03:45 | |
| *** alex_xu has quit IRC | 03:45 | |
| *** takedakn has quit IRC | 03:45 | |
| *** alex_xu has joined #openstack-nova | 03:46 | |
| openstackgerrit | Michael Still proposed openstack/nova: libvirt: take account of disks in migration data size https://review.openstack.org/206632 | 03:57 |
| *** chaochin has quit IRC | 03:58 | |
| *** chaochin has joined #openstack-nova | 03:58 | |
| *** irenab has joined #openstack-nova | 04:00 | |
| *** rajesht has joined #openstack-nova | 04:00 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-nova | 04:01 | |
| *** jamielennox is now known as jamielennox|away | 04:04 | |
| *** jamielennox|away is now known as jamielennox | 04:05 | |
| *** Nikolay_1t has quit IRC | 04:06 | |
| *** achanda has quit IRC | 04:08 | |
| *** agireud has joined #openstack-nova | 04:11 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: api: tidy up paste.ini https://review.openstack.org/214085 | 04:15 |
| *** shyu has quit IRC | 04:16 | |
| *** Sree has joined #openstack-nova | 04:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/216968 | 04:17 |
| *** ajayaa has joined #openstack-nova | 04:20 | |
| *** shyu has joined #openstack-nova | 04:20 | |
| *** shuquan has quit IRC | 04:23 | |
| *** garyk has quit IRC | 04:26 | |
| *** mdrabe has quit IRC | 04:30 | |
| *** rushil has joined #openstack-nova | 04:32 | |
| *** rushil_ has joined #openstack-nova | 04:34 | |
| *** agireud has quit IRC | 04:34 | |
| *** rushil has quit IRC | 04:37 | |
| *** summer has joined #openstack-nova | 04:38 | |
| *** links has joined #openstack-nova | 04:38 | |
| *** asettle has quit IRC | 04:44 | |
| *** achanda has joined #openstack-nova | 04:44 | |
| *** changbl has quit IRC | 04:46 | |
| *** unicell has joined #openstack-nova | 04:46 | |
| *** rushil_ has quit IRC | 04:50 | |
| *** unicell has quit IRC | 04:53 | |
| *** unicell has joined #openstack-nova | 04:53 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** armax has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-nova | 05:01 | |
| *** ajayaa has quit IRC | 05:01 | |
| *** SourabhP has joined #openstack-nova | 05:01 | |
| *** salv-orlando has joined #openstack-nova | 05:02 | |
| *** nkrinner has joined #openstack-nova | 05:02 | |
| *** SourabhP_ has joined #openstack-nova | 05:03 | |
| *** lyanchih has quit IRC | 05:05 | |
| *** armax has joined #openstack-nova | 05:05 | |
| *** skylerberg has quit IRC | 05:05 | |
| *** irenab has quit IRC | 05:05 | |
| *** garyk has joined #openstack-nova | 05:06 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Remove API_V3_CORE_EXTENSIONS from v2.1 API https://review.openstack.org/217518 | 05:06 |
| *** SourabhP has quit IRC | 05:06 | |
| *** SourabhP_ is now known as SourabhP | 05:06 | |
| *** salv-orlando has quit IRC | 05:07 | |
| *** irenab has joined #openstack-nova | 05:07 | |
| openstackgerrit | Jay Pipes proposed openstack/python-novaclient: Add mechanism to vm list to return all resources https://review.openstack.org/217101 | 05:08 |
| *** tojuvone has joined #openstack-nova | 05:09 | |
| *** openstackgerrit has quit IRC | 05:16 | |
| *** openstackgerrit has joined #openstack-nova | 05:17 | |
| *** ankit_ag has joined #openstack-nova | 05:23 | |
| *** albertom has quit IRC | 05:29 | |
| *** pkoniszewski has quit IRC | 05:30 | |
| *** pkoniszewski has joined #openstack-nova | 05:30 | |
| *** albertom has joined #openstack-nova | 05:31 | |
| *** ildikov has quit IRC | 05:34 | |
| *** shyu has joined #openstack-nova | 05:43 | |
| *** armax has quit IRC | 05:43 | |
| *** rdopiera has joined #openstack-nova | 05:44 | |
| *** rushiagr_away is now known as rushiagr | 05:45 | |
| *** vilobhmm1 has joined #openstack-nova | 05:45 | |
| *** dims has joined #openstack-nova | 05:46 | |
| *** ajayaa has joined #openstack-nova | 05:49 | |
| *** agireud has joined #openstack-nova | 05:49 | |
| *** EinstCrazy has quit IRC | 05:50 | |
| *** Marga_ has joined #openstack-nova | 05:51 | |
| *** Marga_ has quit IRC | 05:51 | |
| *** Marga_ has joined #openstack-nova | 05:51 | |
| *** geekinutah has quit IRC | 05:52 | |
| *** dims has quit IRC | 05:52 | |
| *** EinstCrazy has joined #openstack-nova | 05:53 | |
| *** agireud has quit IRC | 05:54 | |
| *** abhishekk has joined #openstack-nova | 06:00 | |
| *** lyanchih has joined #openstack-nova | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** 5EXABZ8QP has joined #openstack-nova | 06:01 | |
| *** Sree has quit IRC | 06:02 | |
| *** sahid has joined #openstack-nova | 06:04 | |
| *** Sree has joined #openstack-nova | 06:06 | |
| *** Murali has joined #openstack-nova | 06:06 | |
| *** afazekas_ has joined #openstack-nova | 06:06 | |
| *** jichen has joined #openstack-nova | 06:19 | |
| *** fawadkhaliq has joined #openstack-nova | 06:21 | |
| *** deepakcs has joined #openstack-nova | 06:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/212367 | 06:24 |
| *** geekinutah has joined #openstack-nova | 06:28 | |
| openstackgerrit | Li Yuanyuan proposed openstack/nova: Fix exception message mistake https://review.openstack.org/215509 | 06:29 |
| *** Nikolay_St has joined #openstack-nova | 06:32 | |
| *** albertom has quit IRC | 06:34 | |
| *** pkoniszewski has quit IRC | 06:35 | |
| *** humble__ has joined #openstack-nova | 06:35 | |
| *** mpaolino has joined #openstack-nova | 06:35 | |
| *** eliqiao has quit IRC | 06:35 | |
| *** fesp has joined #openstack-nova | 06:35 | |
| *** wanghua has joined #openstack-nova | 06:37 | |
| *** fesp has quit IRC | 06:37 | |
| *** alexschm has joined #openstack-nova | 06:38 | |
| openstackgerrit | Radomir Dopieralski proposed openstack/nova: Make the create_instance call accept security group ids as well as names https://review.openstack.org/217292 | 06:38 |
| *** humble_ has quit IRC | 06:38 | |
| openstackgerrit | Radomir Dopieralski proposed openstack/nova: Make the create_instance call accept security group ids as well as names https://review.openstack.org/217292 | 06:39 |
| *** humble__ has quit IRC | 06:40 | |
| *** albertom has joined #openstack-nova | 06:40 | |
| *** vilobhmm1 has quit IRC | 06:45 | |
| *** vilobhmm1 has joined #openstack-nova | 06:45 | |
| *** vilobhmm1 has quit IRC | 06:46 | |
| *** salv-orlando has joined #openstack-nova | 06:47 | |
| *** pkoniszewski has joined #openstack-nova | 06:50 | |
| openstackgerrit | JuPing proposed openstack/nova: Fix description for "Inapt spelling of a word" https://review.openstack.org/217547 | 06:51 |
| *** salv-orlando has quit IRC | 06:52 | |
| *** eliqiao has joined #openstack-nova | 06:53 | |
| *** rushiagr is now known as rushiagr_away | 06:53 | |
| *** kaisers has quit IRC | 06:54 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Remove API_V3_CORE_EXTENSIONS from v2.1 API https://review.openstack.org/217518 | 06:56 |
| *** vilobhmm1 has joined #openstack-nova | 06:59 | |
| *** ildikov has joined #openstack-nova | 06:59 | |
| *** markus_z has joined #openstack-nova | 07:00 | |
| *** 5EXABZ8QP has quit IRC | 07:01 | |
| *** moshele has joined #openstack-nova | 07:01 | |
| *** browne has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:02 | |
| *** shyu has quit IRC | 07:03 | |
| *** shyu has joined #openstack-nova | 07:03 | |
| *** salv-orlando has joined #openstack-nova | 07:06 | |
| *** achanda has quit IRC | 07:06 | |
| *** drankis has joined #openstack-nova | 07:06 | |
| *** markus_z has quit IRC | 07:07 | |
| *** stefan_amann has joined #openstack-nova | 07:08 | |
| *** tobasco_ is now known as tobasco | 07:14 | |
| *** Sree has quit IRC | 07:19 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: add in folder support on VC https://review.openstack.org/165060 | 07:30 |
| openstackgerrit | garyk proposed openstack/nova: VMware: improve instance names on VC https://review.openstack.org/166608 | 07:31 |
| *** dims has joined #openstack-nova | 07:35 | |
| *** vilobhmm1 has quit IRC | 07:36 | |
| *** ircuser-1 has joined #openstack-nova | 07:36 | |
| *** eliqiao has quit IRC | 07:37 | |
| *** pkoniszewski has quit IRC | 07:38 | |
| *** mpaolino has quit IRC | 07:38 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: add in vif resource limitations https://review.openstack.org/180202 | 07:39 |
| *** eliqiao has joined #openstack-nova | 07:39 | |
| *** pkoniszewski has joined #openstack-nova | 07:39 | |
| *** dims has quit IRC | 07:40 | |
| *** vivekd has joined #openstack-nova | 07:41 | |
| *** yuntongjin has joined #openstack-nova | 07:44 | |
| *** matrohon has joined #openstack-nova | 07:44 | |
| *** lalitd has quit IRC | 07:47 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 07:48 |
| openstackgerrit | Jianghua Wang proposed openstack/nova: XenAPI: Expose labels for root and ephemeral disks https://review.openstack.org/213094 | 07:49 |
| *** yassine has joined #openstack-nova | 07:49 | |
| *** achanda has joined #openstack-nova | 07:51 | |
| *** yuntongjin1 has joined #openstack-nova | 07:51 | |
| *** yuntongjin has quit IRC | 07:52 | |
| *** yamahata has joined #openstack-nova | 07:53 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Expose VIF net-id attribute in os-virtual-interfaces https://review.openstack.org/217572 | 07:54 |
| *** SourabhP has quit IRC | 07:54 | |
| *** IanGovett has joined #openstack-nova | 07:56 | |
| *** shoutm_ has joined #openstack-nova | 07:57 | |
| *** doude has joined #openstack-nova | 07:59 | |
| *** doude has quit IRC | 07:59 | |
| *** doude has joined #openstack-nova | 07:59 | |
| *** ildikov has quit IRC | 07:59 | |
| *** shoutm has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** lpetrut has joined #openstack-nova | 08:02 | |
| *** lpetrut1 has joined #openstack-nova | 08:02 | |
| *** lpetrut has left #openstack-nova | 08:03 | |
| *** lpetrut1 has quit IRC | 08:03 | |
| *** lpetrut1 has joined #openstack-nova | 08:03 | |
| openstackgerrit | Eli Qiao proposed openstack/nova: Scheduler: Move notifier.info to scheduler manager https://review.openstack.org/217574 | 08:04 |
| openstackgerrit | garyk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object https://review.openstack.org/188321 | 08:05 |
| openstackgerrit | garyk proposed openstack/nova: VMware: support limits via image meta data https://review.openstack.org/180942 | 08:06 |
| openstackgerrit | Eli Qiao proposed openstack/nova: Scheduler: Move notifier.info to scheduler manager https://review.openstack.org/211887 | 08:06 |
| openstackgerrit | Alex Xu proposed openstack/nova: Add a note about the 400 response not requiring a microversion https://review.openstack.org/217577 | 08:08 |
| *** takedakn has joined #openstack-nova | 08:08 | |
| *** rajesht_ has joined #openstack-nova | 08:09 | |
| *** karimb has joined #openstack-nova | 08:09 | |
| *** abhishekkk has joined #openstack-nova | 08:10 | |
| *** boris-42 has quit IRC | 08:10 | |
| *** kaisers has joined #openstack-nova | 08:11 | |
| *** kiran-r has joined #openstack-nova | 08:11 | |
| *** ildikov has joined #openstack-nova | 08:12 | |
| *** kaisers has quit IRC | 08:12 | |
| *** abhishekk has quit IRC | 08:12 | |
| *** rajesht has quit IRC | 08:13 | |
| *** summer has quit IRC | 08:13 | |
| *** kaisers has joined #openstack-nova | 08:14 | |
| *** jistr has joined #openstack-nova | 08:15 | |
| *** irenab has quit IRC | 08:16 | |
| *** hparekh has quit IRC | 08:17 | |
| *** summer has joined #openstack-nova | 08:17 | |
| *** irenab has joined #openstack-nova | 08:19 | |
| *** lucasagomes has joined #openstack-nova | 08:22 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add mechanism to vm list to return all resources https://review.openstack.org/217101 | 08:25 |
| *** danpb has joined #openstack-nova | 08:27 | |
| *** karimb has quit IRC | 08:29 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch libvirt race condition (do not merge) https://review.openstack.org/191732 | 08:30 |
| *** ndipanov has joined #openstack-nova | 08:30 | |
| *** shoutm_ has quit IRC | 08:32 | |
| *** longsube has joined #openstack-nova | 08:35 | |
| longsube | Hi | 08:36 |
| longsube | Do you have issue in live migrating VM in KILO Version | 08:36 |
| *** derekh has joined #openstack-nova | 08:40 | |
| *** sudipto has joined #openstack-nova | 08:42 | |
| openstackgerrit | garyk proposed openstack/nova: Compute: be consistent with logs about NotImplemented methods https://review.openstack.org/188392 | 08:42 |
| garyk | johnthetubaguy: ^ addresed your comments | 08:42 |
| *** yuntongjin has joined #openstack-nova | 08:45 | |
| *** yuntongjin1 has quit IRC | 08:46 | |
| *** garyk has quit IRC | 08:47 | |
| *** hparekh has joined #openstack-nova | 08:48 | |
| *** achanda has quit IRC | 08:50 | |
| *** irenab has quit IRC | 08:50 | |
| *** haomaiwang has quit IRC | 08:51 | |
| *** irenab has joined #openstack-nova | 08:51 | |
| *** haomaiwang has joined #openstack-nova | 08:51 | |
| *** jistr has quit IRC | 08:54 | |
| *** jistr has joined #openstack-nova | 08:56 | |
| *** apuimedo_ has joined #openstack-nova | 08:58 | |
| *** alex_klimov has joined #openstack-nova | 08:58 | |
| *** garyk has joined #openstack-nova | 08:59 | |
| *** wuhg has joined #openstack-nova | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** jistr has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:02 | |
| *** lyanchih has quit IRC | 09:03 | |
| *** kylek3h has joined #openstack-nova | 09:03 | |
| *** jistr has joined #openstack-nova | 09:05 | |
| *** Sree has joined #openstack-nova | 09:07 | |
| *** apuimedo_ has quit IRC | 09:11 | |
| *** lyanchih has joined #openstack-nova | 09:12 | |
| johnthetubaguy | garyk: sweet, thanks for the update, LGTM | 09:17 |
| johnthetubaguy | longsube: there are certainly some known issues in that area, but I would ask on #openstack for more info | 09:18 |
| *** andreykurilin__ has joined #openstack-nova | 09:20 | |
| *** Sree has quit IRC | 09:26 | |
| *** kiran-r has quit IRC | 09:27 | |
| *** irenab has quit IRC | 09:29 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the api v2 extension configuration https://review.openstack.org/214601 | 09:30 |
| bauzas | longsube: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/migration-fix-resource-tracking.html | 09:32 |
| bauzas | longsube: is one of the areas that we know are buggy | 09:33 |
| *** Sree has joined #openstack-nova | 09:34 | |
| *** irenab has joined #openstack-nova | 09:34 | |
| ndipanov | bauzas, that's only an issue if you have pci devices (which you can't really live migrate) or NUMA stuff | 09:39 |
| alex_xu | johnthetubaguy: do we need upgradeimpact flag for https://review.openstack.org/#/c/214592/6 ? | 09:39 |
| ndipanov | bauzas, livemig not working for someone is unlikely to be caused by that | 09:39 |
| ndipanov | or rather | 09:40 |
| ndipanov | there are far more likely things that could have gone wrong | 09:40 |
| alex_xu | johnthetubaguy: as renamed the options, operator need update their config when upgrade the code | 09:40 |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the concept of extensions in v2.1 https://review.openstack.org/214592 | 09:40 |
| johnthetubaguy | alex_xu: yeah, we totally do, added that ^ | 09:40 |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the api v2 extension configuration https://review.openstack.org/214601 | 09:40 |
| openstackgerrit | Hiroyuki Eguchi proposed openstack/nova: Cannot rebuild a instance booted from volume https://review.openstack.org/201458 | 09:40 |
| alex_xu | johnthetubaguy: yea :), saw you just updated | 09:40 |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the api v2 extension configuration https://review.openstack.org/214601 | 09:41 |
| johnthetubaguy | alex_xu: good spotted, added that into both | 09:41 |
| alex_xu | johnthetubaguy: thanks | 09:42 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 09:42 |
| *** Sree_ has joined #openstack-nova | 09:43 | |
| *** Sree_ is now known as Guest25899 | 09:43 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova: Remove unused sample files from /doc dir https://review.openstack.org/217606 | 09:45 |
| johnthetubaguy | alex_xu: how are we looking with the API stuff, do we have that etherpad all up to date now? | 09:46 |
| *** yamahata has quit IRC | 09:46 | |
| gmann | alex_xu: johnthetubaguy : ^^ quick one, missed on previous patch | 09:46 |
| *** Sree has quit IRC | 09:46 | |
| *** shuquan has joined #openstack-nova | 09:47 | |
| alex_xu | johnthetubaguy: emm...let me check | 09:48 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Add mechanism to vm list to return all resources https://review.openstack.org/217101 | 09:48 |
| alex_xu | I should summary all the key stuff of API before release | 09:48 |
| openstackgerrit | garyk proposed openstack/nova: VMware: create method for getting datacenter from datastore https://review.openstack.org/205819 | 09:49 |
| johnthetubaguy | alex_xu: that would be very cool, I have a project update summit session, it would be good to roll the info into that too | 09:49 |
| johnthetubaguy | alex_xu: looking at your comment in here: https://review.openstack.org/#/c/214085/22/etc/nova/api-paste.ini,cm | 09:49 |
| alex_xu | johnthetubaguy: yea | 09:49 |
| johnthetubaguy | alex_xu: not sure I understand your question | 09:49 |
| johnthetubaguy | alex_xu: I think liberty we make v2.1 the default API, by default no use of v2, at least that was the idea I think | 09:50 |
| *** dims has joined #openstack-nova | 09:50 | |
| *** ildikov has quit IRC | 09:51 | |
| alex_xu | johnthetubaguy: "and is suggested that you use it as the default." looks like suggestion user use v2.1 compat mode as default. | 09:51 |
| alex_xu | johnthetubaguy: as you said, we suggest use v2.1 as default | 09:51 |
| alex_xu | johnthetubaguy: maybe we should say: In liberty, v2.1 is as the default. if you want to use v2 API, we suggest you use v2.1 compat mode. | 09:52 |
| johnthetubaguy | alex_xu: hmm, but you should never use v2.1 for /v2, thats the only way you are "allowed" to use v2.1 right? | 09:52 |
| johnthetubaguy | alex_xu: no, we must have /v2 and /v2.1 | 09:53 |
| *** oomichi has quit IRC | 09:53 | |
| johnthetubaguy | alex_xu: otherwise we break the backwards compatibility point | 09:53 |
| alex_xu | johnthetubaguy: ah, I see now | 09:53 |
| rajesht_ | hi cores, | 09:53 |
| alex_xu | johnthetubaguy: that word means suggest use v2.1 compat mode as default for /v2 | 09:54 |
| johnthetubaguy | alex_xu: now I am guessing my comment doesn't explain that well, so we should really do something better in there I guess | 09:54 |
| rajesht_ | have a look at it https://review.openstack.org/#/c/170118/ | 09:54 |
| alex_xu | johnthetubaguy: I think I understand now, just I miss understand the English words | 09:55 |
| johnthetubaguy | alex_xu: no worries, I am not sure my english works were that good to start with! | 09:55 |
| alex_xu | johnthetubaguy: :) | 09:56 |
| *** yuntongjin has quit IRC | 09:56 | |
| *** shuquan has quit IRC | 09:58 | |
| *** VikasC has joined #openstack-nova | 09:58 | |
| *** hparekh2 has joined #openstack-nova | 10:00 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api: use v2.1 only in api-paste.ini https://review.openstack.org/214085 | 10:00 |
| *** hparekh has quit IRC | 10:00 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api: use v2.1 only in api-paste.ini https://review.openstack.org/214085 | 10:00 |
| *** haomaiwang has quit IRC | 10:01 | |
| bauzas | ndipanov: sorry was in a call, but yeah, agreed | 10:02 |
| *** shuquan has joined #openstack-nova | 10:02 | |
| *** summer has quit IRC | 10:02 | |
| *** haomaiwang has joined #openstack-nova | 10:02 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Strip the extra properties out when using legacy v2 compatible middleware https://review.openstack.org/215436 | 10:02 |
| johnthetubaguy | gmann: I just spotted this change, seems to need a rebase: https://review.openstack.org/#/c/202431/ | 10:04 |
| *** lyanchih has quit IRC | 10:04 | |
| *** ildikov has joined #openstack-nova | 10:05 | |
| *** jistr has quit IRC | 10:06 | |
| bauzas | johnthetubaguy: not sure if you saw https://wiki.openstack.org/wiki/ReleaseNotes/Liberty#Upgrade_Notes but could you also take a look at the allocation ratios branch ? sorry to hassle you, I guess you're already pretty busy these days :/ | 10:07 |
| bauzas | johnthetubaguy: I won't particularly address that as a top feature, but I feel that as a quickwin for Nova | 10:08 |
| johnthetubaguy | bauzas: I did have a look recently, I guess there have been updates | 10:08 |
| bauzas | johnthetubaguy: it had, because dansmith had an excellent point | 10:08 |
| johnthetubaguy | bauzas: cool, will take a peak | 10:08 |
| johnthetubaguy | gmann: alex_xu: just looking at the list of patches on the etherpad, looks like we are missing links to some of the ones edleafe posted, I don't have the links, any others? https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 10:09 |
| bauzas | oh, Jenkins is unhappy because of a gate problem, will recheck the last one - but the change is safe | 10:09 |
| bauzas | johnthetubaguy: thanks, I usually hate pinging people, just wanted to keep you informed | 10:09 |
| bauzas | johnthetubaguy: if that doesn't get merged because of other stuff, fair enough, it's just a matter of opportunities and bandwidth | 10:10 |
| *** kylek3h has quit IRC | 10:10 | |
| *** zhenguo has quit IRC | 10:10 | |
| *** kylek3h has joined #openstack-nova | 10:11 | |
| johnthetubaguy | bauzas: yeah, I am spending today going up and down the etherpad, and you are in there, so thats fine | 10:11 |
| johnthetubaguy | bauzas: yeah, looks like the ones I +2ed have merged now, hadn't quite got my head around this end of the chain yet, looking now | 10:12 |
| johnthetubaguy | well, after I get a glass of water | 10:12 |
| *** shz has joined #openstack-nova | 10:13 | |
| *** EinstCrazy has quit IRC | 10:13 | |
| *** yangyp has quit IRC | 10:13 | |
| *** maurosr has quit IRC | 10:14 | |
| *** zhangjn has quit IRC | 10:15 | |
| *** rotbart has joined #openstack-nova | 10:15 | |
| *** kylek3h has quit IRC | 10:15 | |
| *** shihanzhang has quit IRC | 10:16 | |
| *** rotbeard has quit IRC | 10:16 | |
| *** maurosr has joined #openstack-nova | 10:17 | |
| *** xyhuang has joined #openstack-nova | 10:17 | |
| *** jistr has joined #openstack-nova | 10:18 | |
| openstackgerrit | Huang Zhiteng proposed openstack/nova: libvirt: Force iSCSI disconnect after timeout https://review.openstack.org/167815 | 10:21 |
| johnthetubaguy | bauzas: this doesn't seem quite right here: https://review.openstack.org/#/c/216696/3/nova/scheduler/filters/core_filter.py,cm | 10:23 |
| johnthetubaguy | bauzas: is there not a way we can use the config? or maybe warn if there is a miss-match or something like that? | 10:24 |
| *** Marga_ has quit IRC | 10:24 | |
| *** hparekh2 has quit IRC | 10:27 | |
| *** hparekh has joined #openstack-nova | 10:27 | |
| johnthetubaguy | bauzas: just looking for a middle ground, this does feel like it violates our upgrade rules right now, of no need to change your config during your upgrade | 10:27 |
| *** gszasz has joined #openstack-nova | 10:27 | |
| *** ociuhandu has quit IRC | 10:30 | |
| ndipanov | johnthetubaguy, fwiw - I don't like that approach at all | 10:31 |
| ndipanov | it's too tricky to get it right when upgrading | 10:31 |
| johnthetubaguy | ndipanov: reporting the conf from the compute node, or just the compatibility bit? | 10:31 |
| ndipanov | so imho we need a new conf | 10:32 |
| ndipanov | otherwise you have to make sure to roll out the conf with the upgrade or it will do the wrong thing | 10:32 |
| johnthetubaguy | ndipanov: hmm, I hadn't thought of doing that, but that might be cleaner in some ways | 10:32 |
| ndipanov | but I may be overthinking it | 10:32 |
| johnthetubaguy | ndipanov: so we do claim to support upgrades with zero configuration changes | 10:33 |
| johnthetubaguy | ndipanov: this sure violates that | 10:33 |
| johnthetubaguy | ndipanov: unless you happen to stupidly put the same nova.conf everywhere, not sure that would even work | 10:33 |
| ndipanov | johnthetubaguy, if you were messing with those 2 confs than yes | 10:33 |
| ndipanov | you have to roll the conf update with the code update | 10:34 |
| johnthetubaguy | ndipanov: trying to think of a way to do it, without deprecating one config and adding another, something tells me the compute nodes reporting None would work | 10:34 |
| johnthetubaguy | ndipanov: we claim you don't need a conf update when you upgrade, as long as you fix up all the warnings after the upgrade, you should be good for next time (in theory) | 10:35 |
| ndipanov | johnthetubaguy, yes but how would they know that the conf they see is not the one set by the policy so as to send None | 10:35 |
| johnthetubaguy | ndipanov: would need to have a better default I think | 10:35 |
| ndipanov | :) | 10:36 |
| ndipanov | let me pop my -1 back on there | 10:38 |
| *** Kennan has quit IRC | 10:38 | |
| *** chaochin has quit IRC | 10:39 | |
| *** park_ has quit IRC | 10:40 | |
| *** kiran-r has joined #openstack-nova | 10:40 | |
| *** takedakn1 has joined #openstack-nova | 10:41 | |
| *** takedakn has quit IRC | 10:42 | |
| *** shuquan has quit IRC | 10:43 | |
| ndipanov | bauzas, johnthetubaguy https://review.openstack.org/#/c/216362/ | 10:44 |
| johnthetubaguy | I just added some ideas in how I think this could work | 10:44 |
| ndipanov | johnthetubaguy, can we change the default just like that | 10:45 |
| *** ociuhandu has joined #openstack-nova | 10:46 | |
| ndipanov | we need to make sure that we are handling NONE everywhere | 10:46 |
| ndipanov | at least | 10:46 |
| johnthetubaguy | ndipanov: yeah, I think we can, assuming that we make the default the same in the code after the fact | 10:48 |
| ndipanov | right | 10:48 |
| johnthetubaguy | ndipanov: it seems the best option at least, without adding a new config | 10:48 |
| *** vivekd has quit IRC | 10:48 | |
| ndipanov | yeah if we can make it None | 10:48 |
| ndipanov | then it should work | 10:48 |
| ndipanov | provided we have compat code in the scheduler | 10:48 |
| johnthetubaguy | ndipanov: the annoying bit is the scheduler still needs to know the default, but we can fix that up next release when we drop that compatibility stuff | 10:49 |
| *** Kevin_Zheng_ is now known as Kevin_Zheng | 10:49 | |
| ndipanov | agreed | 10:49 |
| johnthetubaguy | you have to upgrade the control plane first, so this should work, I think | 10:49 |
| ndipanov | yes | 10:49 |
| johnthetubaguy | I don't like us using the value in the conductor, like the current code does | 10:49 |
| ndipanov | so I guess we agreed that there is real value in moving this to compute nodes | 10:50 |
| ndipanov | I think that having a global knob is also kinda useful | 10:50 |
| ndipanov | but I don't really care too much | 10:50 |
| johnthetubaguy | so I think it should love in the resource tracker, I am cool with that | 10:50 |
| johnthetubaguy | its a per compute node type thing, anyways | 10:50 |
| ndipanov | johnthetubaguy, yes in theory | 10:51 |
| ndipanov | but a lot of use cases will not need that granularity | 10:51 |
| ndipanov | so it's kinda overhead | 10:51 |
| ndipanov | conceptually yes - I agree | 10:51 |
| johnthetubaguy | its broken having nova config inside the scheduler though, thats the big deal here | 10:51 |
| johnthetubaguy | now this is something reported by the compute nodes, it sits much more comfortably | 10:52 |
| *** chaochin has joined #openstack-nova | 10:52 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding https://review.openstack.org/217567 | 10:52 |
| ndipanov | well my point is that having a global policy set by the scheduler is not completely dumb | 10:52 |
| ndipanov | maybe it is | 10:52 |
| johnthetubaguy | but when we separate out the scheduler, it needs to push it to the compute nodes, and that feels a bit odd really | 10:53 |
| ndipanov | yeah | 10:53 |
| ndipanov | still feels lika a global know would be useful too - sane defaults and all | 10:53 |
| johnthetubaguy | now only setting the config in the scheduler will avoid a lot of screw ups, so its not without merit | 10:53 |
| ndipanov | knob not know | 10:54 |
| *** sudipto has quit IRC | 10:54 | |
| johnthetubaguy | yeah, its just when we get nova-scheduler out of tree, it really doesn't want to be in there, I think thats the main drive here | 10:54 |
| johnthetubaguy | (turns out you get a few other things for free like the granularity, but agreed they are an upside and a downside!) | 10:55 |
| *** sudipto has joined #openstack-nova | 10:55 | |
| *** Kennan has joined #openstack-nova | 10:55 | |
| ndipanov | well a split out scheduler could have a global thing too,,, | 10:55 |
| ndipanov | even if not set by the config | 10:55 |
| *** park_ has joined #openstack-nova | 10:56 | |
| *** kiran_c has joined #openstack-nova | 10:56 | |
| *** vivekd has joined #openstack-nova | 10:57 | |
| *** kiran_c has left #openstack-nova | 10:59 | |
| openstackgerrit | Marian Horban proposed openstack/nova: Rsync compression removed https://review.openstack.org/209957 | 10:59 |
| *** haomaiwang has quit IRC | 11:01 | |
| johnthetubaguy | ndipanov: it could, its just the resource tracker should really own that allocation ratio, so it gets complicated | 11:01 |
| *** haomaiwa_ has joined #openstack-nova | 11:02 | |
| *** aix has quit IRC | 11:02 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: prevent volume deletion if volume in instance folder https://review.openstack.org/193103 | 11:02 |
| *** tojuvone has quit IRC | 11:04 | |
| *** mwagner_lap has quit IRC | 11:06 | |
| * johnthetubaguy goes for some food | 11:06 | |
| openstackgerrit | garyk proposed openstack/nova: Cells: remove redundant check if cells are enabled https://review.openstack.org/198537 | 11:06 |
| *** lyanchih has joined #openstack-nova | 11:08 | |
| openstackgerrit | garyk proposed openstack/nova: Cells: remove redundant check if cells are enabled https://review.openstack.org/198537 | 11:08 |
| *** longlq has joined #openstack-nova | 11:10 | |
| openstackgerrit | garyk proposed openstack/nova: Flavors: use dict comprehension for get_all_flavors https://review.openstack.org/198702 | 11:10 |
| *** haomaiwa_ has quit IRC | 11:11 | |
| *** smatzek has joined #openstack-nova | 11:11 | |
| *** Guest25899 has quit IRC | 11:12 | |
| *** kylek3h has joined #openstack-nova | 11:12 | |
| *** yuntongjin has joined #openstack-nova | 11:12 | |
| *** longsube has quit IRC | 11:14 | |
| *** wanghua has quit IRC | 11:15 | |
| *** kylek3h has quit IRC | 11:17 | |
| *** irenab has quit IRC | 11:18 | |
| *** shyu has quit IRC | 11:20 | |
| *** alex_xu has quit IRC | 11:20 | |
| openstackgerrit | garyk proposed openstack/nova: Virt: add context to attach and detach interface https://review.openstack.org/209362 | 11:21 |
| *** yuntongjin1 has joined #openstack-nova | 11:22 | |
| *** alex_xu has joined #openstack-nova | 11:23 | |
| *** yuntongjin has quit IRC | 11:24 | |
| *** thorst has joined #openstack-nova | 11:27 | |
| *** thorst1 has joined #openstack-nova | 11:29 | |
| *** dave-mccowan has joined #openstack-nova | 11:30 | |
| *** irenab has joined #openstack-nova | 11:30 | |
| *** thorst has quit IRC | 11:32 | |
| *** longlq has quit IRC | 11:32 | |
| *** aix has joined #openstack-nova | 11:33 | |
| *** dprince has joined #openstack-nova | 11:34 | |
| *** garyk has quit IRC | 11:35 | |
| *** lyanchih has quit IRC | 11:40 | |
| *** yuntongjin1 has quit IRC | 11:42 | |
| *** VikasC has quit IRC | 11:44 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova-specs: Add soft policies for server-group feature https://review.openstack.org/212504 | 11:45 |
| *** lucasagomes is now known as lucas-hungry | 11:48 | |
| *** wanghua has joined #openstack-nova | 11:50 | |
| *** vivekd has quit IRC | 11:52 | |
| *** haomaiwang has joined #openstack-nova | 11:53 | |
| *** deepakcs has quit IRC | 11:54 | |
| *** deepthi has quit IRC | 11:56 | |
| *** kiran-r has quit IRC | 11:58 | |
| *** mmedvede has quit IRC | 11:58 | |
| ftersin | sdague: hi. could you please look again to https://review.openstack.org/#/c/173204/ | 11:59 |
| ftersin | i've put a comment there to you and mriedem | 12:00 |
| *** mmedvede has joined #openstack-nova | 12:00 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** wanghua has quit IRC | 12:02 | |
| *** haomaiwang has joined #openstack-nova | 12:02 | |
| openstackgerrit | Noel Nelson Dsouza proposed openstack/nova: Delete flavor should raise error, if it is inuse https://review.openstack.org/217674 | 12:03 |
| *** rfolco has joined #openstack-nova | 12:04 | |
| bauzas | johnthetubaguy: heya, was out for lunch | 12:05 |
| sdague | ftersin: ok, I think I understand now, yeh the defaults with no user input was what I was most concerned about. It looks like the patch needs rebasing anyway. | 12:06 |
| bauzas | ndipanov: sorry, missed your pings too | 12:07 |
| bauzas | johnthetubaguy: ndipanov: so I actually discussed with dansmith about all of that and we agreed upon having config flags being provided correctly in https://wiki.openstack.org/wiki/ReleaseNotes/Liberty#Upgrade_Notes | 12:07 |
| bauzas | johnthetubaguy: ndipanov: the idea is to say that anytime, the ComputeNode object will provide the allocation ratios | 12:08 |
| sdague | johnthetubaguy: so, sorry I wasn't around much yesterday, I was mostly trying to sleep off the bug I picked up at LinuxCon. On the testing for the API front here is what I'm currently thinking: | 12:08 |
| sdague | the default jobs get switched to nova v2.1 | 12:09 |
| bauzas | johnthetubaguy: ndipanov: but it will be actually be done either by the RT (if the compute is Liberty), or by the scheduler's nova.conf as it's already | 12:09 |
| sdague | then we build another job for the 2 v2.0 stacks. v2.0 old code and v2.0 on v2.1 | 12:09 |
| sdague | which we run as a new functional job on nova, and just execute tempest twice with 2 different service catalog entries | 12:10 |
| ftersin | sdague: thanks. at least now the rebasing makes sense :) | 12:10 |
| *** raildo-afk is now known as raildo | 12:10 | |
| johnthetubaguy | bauzas: I am just back from lunch myself | 12:11 |
| bauzas | johnthetubaguy: okay, I actually missed all your convo with ndipanov, my bad :/ | 12:12 |
| bauzas | johnthetubaguy: I understand your problem, what if the operator had provided a change to nova.conf in the scheduler but not in the computes, correct? | 12:12 |
| johnthetubaguy | bauzas: no worries, was hoping to catch you | 12:12 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 12:13 |
| johnthetubaguy | bauzas: so my thinking is simple, we currently say you don't need to change your nova.conf when you upgrade, that patch set seems to violate that rule, when I think about it | 12:13 |
| bauzas | johnthetubaguy: yes, but we also make the assumption that all nova.conf are identical, right? | 12:13 |
| johnthetubaguy | bauzas: I added a proposal for a slightly alternative approach | 12:13 |
| johnthetubaguy | bauzas: I don't think we can make that assumption | 12:13 |
| bauzas | johnthetubaguy: well, I discussed that with dansmith last week f2f | 12:13 |
| *** dims has quit IRC | 12:14 | |
| *** kylek3h has joined #openstack-nova | 12:14 | |
| bauzas | johnthetubaguy: and dansmith made a good point in https://review.openstack.org/#/c/215471/4/nova/objects/compute_node.py,cm | 12:14 |
| *** rushil has joined #openstack-nova | 12:14 | |
| johnthetubaguy | bauzas: agreed, just not sure I agree with the outcome of that chat, willing to be persuaded, what don't you like about my proposed direction? | 12:14 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make detach_device to return async object https://review.openstack.org/217680 | 12:15 |
| johnthetubaguy | bauzas: yeah, agreed with his point, I just don't like the trade off | 12:15 |
| bauzas | johnthetubaguy: I don't really like to change the default values for the ratios and thus having a nullable field for the reason that dansmith explained in https://review.openstack.org/#/c/215471/4/nova/objects/compute_node.py,cm | 12:15 |
| *** thangp has joined #openstack-nova | 12:15 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance rescue mode support https://review.openstack.org/217679 | 12:15 |
| bauzas | johnthetubaguy: I just wonder if having a flag for saying 'I want to have allocation ratios per compute' is not worth it | 12:15 |
| johnthetubaguy | I know its more complicated, but I don't want to violate our upgrade rules | 12:16 |
| bauzas | johnthetubaguy: where in that case, there would be an if branch for all the filters | 12:16 |
| johnthetubaguy | bauzas: so I don't think thats the point here, the point is its not something a separate scheduler project should own, its something the resource tracker should be enforcing | 12:16 |
| johnthetubaguy | bauzas: yep, there would be an if in all the filters, be we get to drop it during the next release | 12:17 |
| johnthetubaguy | s/be we/but we/ | 12:17 |
| bauzas | johnthetubaguy: I see that BP uncorrelated for the separate sched IMO | 12:17 |
| ndipanov | bauzas, imho - even if we decide to go with this, you still want to avoid having to do code and config rollout at the same tiem | 12:17 |
| ndipanov | that's what we are mostly complaining about | 12:18 |
| bauzas | ndipanov: well, before upgrading, you have to follow release notes nope ? | 12:18 |
| *** kylek3h has quit IRC | 12:18 | |
| *** rushil_ has joined #openstack-nova | 12:18 | |
| bauzas | ndipanov: so you would change the compute opt values before upgrading | 12:18 |
| johnthetubaguy | bauzas: yes, but we promise you can upgrade without changing your config | 12:18 |
| *** jcoufal has joined #openstack-nova | 12:19 | |
| johnthetubaguy | bauzas: this breaks that promise (unless you happen to have the same nova.conf everywhere) | 12:19 |
| ndipanov | bauzas, I would not do anything - I am just pointing out an issue with the current proposed implementation | 12:19 |
| ndipanov | johnthetubaguy, it breaks the promise period since it is dependant on the config | 12:19 |
| bauzas | johnthetubaguy: when is the nova meeting ? | 12:20 |
| bauzas | johnthetubaguy: I'd love to provide those changes as stale reviews ;) | 12:20 |
| bauzas | johnthetubaguy: I see, 9pm UTC | 12:20 |
| *** ociuhandu has quit IRC | 12:20 | |
| johnthetubaguy | bauzas: yep | 12:20 |
| bauzas | johnthetubaguy: ok, considering your proposal | 12:20 |
| ndipanov | why is nova meeting relevant here? | 12:20 |
| bauzas | ndipanov: because I think we're discussing about a promise we made | 12:21 |
| johnthetubaguy | bauzas: to be fair, its a promise we made but never wrote down anywhere, I should check... | 12:21 |
| ndipanov | regardless it's still a goofy way to do it imho | 12:21 |
| *** rushil has quit IRC | 12:22 | |
| bauzas | johnthetubaguy: so, given that we change the rules | 12:22 |
| bauzas | johnthetubaguy: saying that RT values would be set to None | 12:22 |
| bauzas | johnthetubaguy: that would impact the current situation because now, the scheduler filters are importing those opts | 12:22 |
| *** haigang has joined #openstack-nova | 12:22 | |
| *** lbeliveau has joined #openstack-nova | 12:22 | |
| ndipanov | the difference between for example the flavor stuff dansmith did that had a realease not and this is - there is a clear benefit to the flavor thing - no potentially huge db downtime | 12:22 |
| *** ajayaa has quit IRC | 12:23 | |
| ndipanov | this is just "we mess your stuff up LOL" | 12:23 |
| johnthetubaguy | ndipanov: but the flavor stuff worked seamlessly across the release boundy? | 12:23 |
| johnthetubaguy | boundry | 12:23 |
| ndipanov | well you do need to run a script eventually | 12:23 |
| ndipanov | right | 12:23 |
| ndipanov | the nova manage thing | 12:23 |
| johnthetubaguy | ndipanov: agreed, after the full upgrade | 12:23 |
| bauzas | yup | 12:23 |
| ndipanov | so that's kind of the relnote | 12:23 |
| johnthetubaguy | just like you must change your config after the upgrade before trying the next one | 12:24 |
| ndipanov | this is more disruptive for less (no?) gain to the user | 12:24 |
| ndipanov | agreed | 12:24 |
| johnthetubaguy | this is worse because you need to change *before* the upgrade, not at some point after the upgrade when its good for you | 12:24 |
| ndipanov | right | 12:24 |
| ndipanov | my point exactly | 12:24 |
| *** mtanino has joined #openstack-nova | 12:24 | |
| johnthetubaguy | thinking I should clarify this in here, we clearly don't all have the same understanding of the rules right now: http://docs.openstack.org/developer/nova/project_scope.html#upgrades | 12:25 |
| johnthetubaguy | (I think it was some talk of tags for upgrade rules that reminded me about the configuration thing) | 12:25 |
| *** vladikr has joined #openstack-nova | 12:25 | |
| *** aysyd has joined #openstack-nova | 12:26 | |
| bauzas | okay, I need to consider another approach because changing the default of the ratios couldn't work | 12:26 |
| johnthetubaguy | bauzas: why? | 12:26 |
| ndipanov | why? | 12:26 |
| bauzas | johnthetubaguy: because the filters are importing those opts | 12:26 |
| *** ociuhandu has joined #openstack-nova | 12:26 | |
| *** haigang has quit IRC | 12:26 | |
| bauzas | johnthetubaguy: so the core filter would be having the allocation ratio defaulted to None | 12:26 |
| ndipanov | so make filter not import them and hard code them instead | 12:26 |
| johnthetubaguy | bauzas: but you upgrade the control plane first, thats fine right? | 12:26 |
| bauzas | johnthetubaguy: yeah, that's the usual approach | 12:27 |
| ndipanov | scheduler gets upgraded first | 12:27 |
| johnthetubaguy | yeah, as mentioned in my comments, the scheduler filters will have to use some util that defaults them to the existing default for one release only | 12:27 |
| johnthetubaguy | bauzas: are we missing something big here :S | 12:27 |
| *** kiran_c has joined #openstack-nova | 12:28 | |
| bauzas | johnthetubaguy: well, https://review.openstack.org/#/c/215278/ is merged | 12:28 |
| johnthetubaguy | bauzas: we can revert it if we want, although I don't think we need to | 12:28 |
| bauzas | johnthetubaguy: agreed | 12:28 |
| *** dims has joined #openstack-nova | 12:29 | |
| johnthetubaguy | bauzas: I want the config moved, just the default says in the scheduler for one release, so we can detect when the config gets moved | 12:29 |
| bauzas | johnthetubaguy: rereading your proposal | 12:29 |
| johnthetubaguy | its a bit messy for us, but that goes away after one cycle (all except the object needing a major bump to remove the null-able bit) | 12:29 |
| *** mmedvede has quit IRC | 12:29 | |
| *** annegentle has joined #openstack-nova | 12:31 | |
| *** openstackgerrit has quit IRC | 12:31 | |
| bauzas | johnthetubaguy: just trying to understand correctly | 12:32 |
| *** mmedvede has joined #openstack-nova | 12:32 | |
| *** openstackgerrit has joined #openstack-nova | 12:32 | |
| bauzas | johnthetubaguy: we need to have a signal by the operator saying "okay, read the opt values in RT, right?" | 12:32 |
| *** edmondsw has joined #openstack-nova | 12:32 | |
| bauzas | johnthetubaguy: which is in your proposal to have nullable ratios by default, and set them by the operator | 12:33 |
| *** burgerk has joined #openstack-nova | 12:33 | |
| bauzas | johnthetubaguy: but that proposal implies that the scheduler will get those nullable ratios by default, correct? | 12:34 |
| bauzas | johnthetubaguy: that's what I dislike | 12:34 |
| johnthetubaguy | bauzas: the signal is the operator setting that config value right? | 12:34 |
| johnthetubaguy | bauzas: if the config is set, we use it, I am fine with that | 12:35 |
| bauzas | johnthetubaguy: yup, but that means that if they haven't set that yet in controller's nova.conf, it will default to None for the filters | 12:35 |
| bauzas | johnthetubaguy: and I guess most of the deployments haven't set that | 12:35 |
| *** Marga_ has joined #openstack-nova | 12:35 | |
| johnthetubaguy | bauzas: no, the filters will see the computes value is not None, and just use that | 12:35 |
| *** andreykurilin__ has quit IRC | 12:35 | |
| *** jcoufal has quit IRC | 12:35 | |
| bauzas | johnthetubaguy: sorry, I don't get that :/ | 12:36 |
| bauzas | johnthetubaguy: so, the filter is getting a nullable ratio | 12:36 |
| johnthetubaguy | bauzas: shall we do a quick hangout? | 12:36 |
| bauzas | johnthetubaguy: probably | 12:36 |
| johnthetubaguy | bauzas: I can't do this very well in text | 12:36 |
| bauzas | sure | 12:36 |
| johnthetubaguy | bauzas: let me start one and I will post the link | 12:37 |
| *** lucas-hungry is now known as lucasagomes | 12:37 | |
| bauzas | ack | 12:37 |
| johnthetubaguy | bauzas: ndipanov: https://plus.google.com/hangouts/_/eq7jgaupopsdih3vfq5dvvtu4qb | 12:37 |
| ndipanov | johnthetubaguy, ugh in a shared office space today :/ | 12:38 |
| johnthetubaguy | ndipanov: ah, doh | 12:38 |
| johnthetubaguy | ndipanov: will try fill you in after if that works? | 12:38 |
| ndipanov | sure - let me know if you come up with something more clever than changing the default | 12:39 |
| openstackgerrit | Noel Nelson Dsouza proposed openstack/nova: Delete flavor should raise error, if it is inuse https://review.openstack.org/217674 | 12:40 |
| *** wuhg has quit IRC | 12:41 | |
| johnthetubaguy | ndipanov: +1 | 12:42 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: libvirt: Race condition leads to instance in error https://review.openstack.org/185549 | 12:46 |
| *** abhishekkk has quit IRC | 12:48 | |
| *** jcoufal has joined #openstack-nova | 12:49 | |
| *** salv-orlando has quit IRC | 12:53 | |
| openstackgerrit | jichenjc proposed openstack/nova: Remove 'Retry-After' in server create and resize https://review.openstack.org/214691 | 12:55 |
| bauzas | ndipanov: so johnthetubaguy will summarize our convo in a new vote | 12:55 |
| bauzas | ndipanov: we basically agreed on a new approac | 12:56 |
| bauzas | approach | 12:56 |
| *** Marga_ has quit IRC | 12:56 | |
| edleafe | alex_xu: around? Questions about your comments on https://review.openstack.org/#/c/212851 | 12:56 |
| alex_xu | gmann: are you still aournd, looks like nothing for this https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:merge_sample_tests,n,z , right? | 12:57 |
| ndipanov | bauzas, nice | 12:57 |
| alex_xu | edleafe: yea | 12:57 |
| johnthetubaguy | ndipanov: slight change is we can hide some of the mess in the object, which is nice | 12:57 |
| edleafe | alex_xu: I was simply replacing the /v3 with /v2. Where did the need for adding a tenant_id come from? | 12:57 |
| alex_xu | edleafe: yea, we should, there isn't 'tenant_id' in the url for v3. But that added back for v2.1 | 12:58 |
| *** rushil_ has quit IRC | 12:58 | |
| alex_xu | edleafe: although most of them just fake url, but avoid confuse people, we should add tenant_id | 12:58 |
| edleafe | alex_xu: ugh, yeah | 12:59 |
| edleafe | alex_xu: so add it after the /v2 ? | 12:59 |
| *** baoli has joined #openstack-nova | 12:59 | |
| alex_xu | edleafe: yea, but some of them not need, like the image_href | 13:00 |
| edleafe | alex_xu: ok, let me work on it... | 13:00 |
| alex_xu | edleafe: thanks | 13:00 |
| *** rfolco has quit IRC | 13:01 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** yuntongjin has joined #openstack-nova | 13:01 | |
| alex_xu | johnthetubaguy: sorry, update late https://etherpad.openstack.org/p/liberty-nova-priorities-tracking, I saw you just update, looks like we have everything here for liberty | 13:01 |
| johnthetubaguy | alex_xu: I think this is the last patch from gmann https://review.openstack.org/#/c/217606/ | 13:02 |
| johnthetubaguy | alex_xu: do we have the patches from edleafe in there now? | 13:02 |
| *** haomaiwang has joined #openstack-nova | 13:02 | |
| alex_xu | johnthetubaguy: yea, I think so, it is last one | 13:03 |
| alex_xu | johnthetubaguy: the left patch in https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:merge_sample_tests,n,z , I think it isn't the scope of L | 13:03 |
| johnthetubaguy | edleafe: alex_xu: should we not change /v3 to /v2.1 ? | 13:03 |
| johnthetubaguy | alex_xu: OK, should we just abandon that one? | 13:03 |
| johnthetubaguy | (for now) | 13:03 |
| alex_xu | johnthetubaguy: we should change to /v2. As before we said, we only have '/v2' endpoint, then we change them all to '/v2'. And v2.1 compat with v2, so I think we just make all the test consistent use '/v2' for now | 13:04 |
| edleafe | johnthetubaguy: just catching up after a day on the road. | 13:04 |
| edleafe | johnthetubaguy: which patch are you referring to? | 13:04 |
| alex_xu | johnthetubaguy: we can just remove it from the priority etherpad | 13:04 |
| johnthetubaguy | alex_xu: thats not what we have in api-paste.ini, thats my issue, but anyways, I would rather have /v2 than /v3, so I am a bit loose | 13:06 |
| johnthetubaguy | alex_xu: OK | 13:06 |
| *** dustins has joined #openstack-nova | 13:06 | |
| *** changbl has joined #openstack-nova | 13:06 | |
| *** I has joined #openstack-nova | 13:06 | |
| alex_xu | johnthetubaguy: can we continue remove v3 stuff in RC? | 13:07 |
| *** I is now known as Guest62749 | 13:07 | |
| *** Marga_ has joined #openstack-nova | 13:07 | |
| johnthetubaguy | alex_xu: tests, yes | 13:07 |
| alex_xu | johnthetubaguy: ok, got it | 13:07 |
| johnthetubaguy | alex_xu: my main worry is getting the paste-api changes and config changes in | 13:07 |
| alex_xu | johnthetubaguy: yea | 13:07 |
| *** rushil has joined #openstack-nova | 13:08 | |
| bauzas | alaski: when you're around, I just remember something that we haven't discussed about primitiving RequestSpec | 13:09 |
| bauzas | alaski: basically, if doing that, then the payload would change for notifications | 13:09 |
| bauzas | alaski: I'm not saying it's not okay, just something that we need to care | 13:10 |
| *** diogogmt has joined #openstack-nova | 13:10 | |
| bauzas | alaski: eg. http://paste.openstack.org/show/429655/ | 13:10 |
| sudipto | dansmith, There are around, 3 dict compat reviews - we could close when you have sometime... | 13:10 |
| sudipto | dansmith, https://review.openstack.org/#/c/201282/ https://review.openstack.org/#/c/216371/ https://review.openstack.org/#/c/216822/ | 13:11 |
| *** mtanino has quit IRC | 13:11 | |
| *** ankit_ag has quit IRC | 13:12 | |
| *** kiran_c has quit IRC | 13:12 | |
| *** mdrabe has joined #openstack-nova | 13:12 | |
| alaski | bauzas: all request specs should have been built by scheduler/utils.py:build_request_spec, the object should generate somethat that matches that | 13:12 |
| alaski | gah, *something | 13:12 |
| bauzas | alaski: yeah, I agree with that, just saying something about how clean our notifications are :) | 13:13 |
| bauzas | alaski: but that's fine, working on the stuff | 13:13 |
| *** neelabh_ has joined #openstack-nova | 13:15 | |
| alaski | yeah, notifications are a terribly freeform thing right now | 13:15 |
| neelabh_ | Guys Please tell me how to access nova.servers.list() through only token.... | 13:15 |
| neelabh_ | alaski: hi | 13:15 |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the concept of extensions in v2.1 https://review.openstack.org/214592 | 13:16 |
| *** hartsock1 has joined #openstack-nova | 13:16 | |
| alaski | neelabh_: if it's about your question I would point you at http://docs.openstack.org/developer/python-novaclient/api.html#usage | 13:16 |
| bauzas | neelabh_: http://docs.openstack.org/developer/python-novaclient/api.html#usage | 13:17 |
| bauzas | ergh | 13:17 |
| *** hartsock1 has quit IRC | 13:17 | |
| neelabh_ | alaski: I already read this doc, but they are not using any token but username and password, please see https://ask.openstack.org/en/question/80901/how-to-get-list-of-all-servers-through-token-based-authetication/ | 13:18 |
| neelabh_ | bauzas: thanks for responding, I already go through this doc, they are not using token, I want to use only token to access server-list not password... | 13:19 |
| alaski | neelabh_: I have no experience with the bindings in that way, as I almost always use the CLI, but you may want to look at http://docs.openstack.org/developer/python-keystoneclient/api/keystoneclient.auth.html#keystoneclient.auth.token_endpoint.Token | 13:20 |
| alaski | neelabh_: also, this sort of question is better suited for #openstack | 13:20 |
| johnthetubaguy | sdague: alex_xu: generally whooping, this is nice and green now, great to see that: https://review.openstack.org/#/c/214085/ | 13:20 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: virt: Allow multiple volume attachments https://review.openstack.org/193135 | 13:20 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: comp: Allow multiple volume attachments https://review.openstack.org/193134 | 13:20 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: obj: Allow multiple volume attachments https://review.openstack.org/193133 | 13:20 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: db: Allow multiple volume attachments https://review.openstack.org/153033 | 13:20 |
| neelabh_ | alaski: I was thinking this channel for nova... | 13:20 |
| *** kylek3h has joined #openstack-nova | 13:21 | |
| neelabh_ | alaski: thanks.. | 13:21 |
| alaski | neelabh_: this is the channel for nova development. usage questions are better handled in #openstack | 13:21 |
| *** zzzeek has joined #openstack-nova | 13:22 | |
| *** salv-orlando has joined #openstack-nova | 13:23 | |
| *** zhenguo has joined #openstack-nova | 13:25 | |
| *** afazekas_ has quit IRC | 13:26 | |
| *** rlrossit has joined #openstack-nova | 13:28 | |
| alex_xu | johnthetubaguy: yea | 13:29 |
| alex_xu | johnthetubaguy: but I'm thinking alaski's comment on https://review.openstack.org/#/c/215436 | 13:30 |
| openstackgerrit | John Garbutt proposed openstack/nova: api: deprecate the api v2 extension configuration https://review.openstack.org/214601 | 13:30 |
| *** xyang1 has joined #openstack-nova | 13:31 | |
| ndipanov | is stable/icehouse eoled or? | 13:32 |
| ndipanov | johnthetubaguy, ^ you may know | 13:32 |
| *** mriedem has joined #openstack-nova | 13:32 | |
| sdague | johnthetubaguy: nice | 13:32 |
| dansmith | ndipanov: johnthetubaguy I'm confused about why there is a config impact | 13:33 |
| ndipanov | dansmith, if someone changed the defautls | 13:33 |
| johnthetubaguy | alex_xu: yeah alaski has a good and valid concern, just not sure the way around it | 13:33 |
| ndipanov | then they would need to roll those defaults out to new compute nodes | 13:33 |
| ndipanov | er those changed values | 13:34 |
| ndipanov | at the same time when they roll out the code | 13:34 |
| ndipanov | or the new compute nodes would start using the defaults | 13:34 |
| dansmith | ndipanov: so previously we made call that people using openstack are using config management and that we don't need to distinguish between config on computes and config on, say, the scheduler | 13:34 |
| sdague | johnthetubaguy: were you planning on adding those log warnings to that patch, or as a follow on? | 13:34 |
| dansmith | johnthetubaguy: remember in mountainview? | 13:34 |
| johnthetubaguy | dansmith: I had totally forgot that, sorry | 13:35 |
| dansmith | johnthetubaguy: we were talking about moving something from one to the other, I asked if that was a problem and mikal said "dude, config management, no" | 13:35 |
| ndipanov | dansmith, that seems like a weird assumption to make | 13:35 |
| dansmith | ndipanov: well, I had the same concern as you and was the minority opinion in the room, | 13:35 |
| johnthetubaguy | dansmith: we need to start writing these down, I have a bad memory | 13:35 |
| dansmith | so I've come to assume that to be the case | 13:35 |
| dansmith | my point is, we need to either have it one way or the other | 13:36 |
| dansmith | last time it was "config is pervasive" | 13:36 |
| johnthetubaguy | sdague: good point, I dropped that bit, do you have time to add that as a follow on patch for me, me owning patches is not going well right now. | 13:36 |
| alaski | to offer a data point, we have separate configs for each type of service but we roll configs out with code each time | 13:36 |
| *** claudiub has joined #openstack-nova | 13:37 | |
| *** atuvenie has joined #openstack-nova | 13:37 | |
| dansmith | alaski: the question is, do you have things like filter configs in your compute nodes? | 13:37 |
| *** ctrath has joined #openstack-nova | 13:37 | |
| dansmith | because the argument in mountainview was that the compute nodes could make decisions about what to send to the scheduler based on what filters were configured | 13:37 |
| johnthetubaguy | dansmith: we totally need to write this down and tell people our decision | 13:37 |
| alaski | dansmith: we do not | 13:37 |
| dansmith | alaski: okay, well, then that's a useful data point | 13:38 |
| johnthetubaguy | dansmith: so stuff like the DB config that you don't want on your compute nodes does violate that that rule, thinking about it (although thats not the strongest form of security) | 13:38 |
| alaski | dansmith: we would certainly add them if needed. but that seems a weird thing for them to need to know about | 13:38 |
| dansmith | johnthetubaguy: either way, that's a little different, IMHO | 13:38 |
| dansmith | johnthetubaguy: since the db_url should really be different on every host | 13:39 |
| sdague | johnthetubaguy: I'll see what I can do. My current status is 6 guys banging on my roof doing a replacement | 13:39 |
| dansmith | alaski: okay | 13:39 |
| *** Guest62749 has quit IRC | 13:39 | |
| johnthetubaguy | dansmith: true | 13:39 |
| dansmith | alaski: well, I'm just echoing what we discussed there.. I forget, were you in mountainview? | 13:39 |
| ndipanov | that assumption breaks in a number of other cases I think | 13:39 |
| johnthetubaguy | sdague: ouch, that sucks | 13:39 |
| sdague | well, it will be good once I have a new roof | 13:39 |
| alaski | dansmith: I was, but I don't clearly remember this discussion | 13:39 |
| dansmith | okay | 13:40 |
| *** Murali has quit IRC | 13:40 | |
| ndipanov | so back to my unrelated question - stable/icehouse... eoled? | 13:40 |
| alex_xu | johnthetubaguy: sdague, you are talk about "add log messages when deprecated v3 and v2 code gets used"? | 13:40 |
| alaski | but I tend to agree with mikals statement. with config management I really don't care what config is needed where, as long as I know beforehand what needs to be where | 13:40 |
| alex_xu | if yes, I can help on | 13:40 |
| dansmith | johnthetubaguy: so your suggestion is to make the param nullable and use the scheduler side if it's null on the compute side? | 13:40 |
| sdague | alex_xu: yes, that | 13:40 |
| johnthetubaguy | ndipanov: yeah, I was looking up the wiki page: https://wiki.openstack.org/wiki/StableBranch#Support_phases | 13:40 |
| alex_xu | sdague: that isn't hard, if you are busy, I can do that | 13:40 |
| mriedem | ndipanov: icehouse has been dead for a while | 13:41 |
| johnthetubaguy | dansmith: yeah, its a bit nasty, but seems to work out, and we can clean it up next time | 13:41 |
| sdague | alex_xu: that would be great if you could, thank you | 13:41 |
| alex_xu | sdague: np, let me do that | 13:41 |
| ndipanov | tnx mriedem johnthetubaguy | 13:41 |
| sdague | alex_xu: mostly to warn people to update their paste.ini if they are using the old pipelines | 13:41 |
| dansmith | johnthetubaguy: so, we can't clean it up next time I don't think | 13:41 |
| sdague | alex_xu: great, thank you | 13:41 |
| alex_xu | sdague: ok, got it | 13:41 |
| dansmith | johnthetubaguy: we can't go from nullable to non-nullable across a single release, right? | 13:41 |
| johnthetubaguy | mriedem is a stable czar, he knows the deal there, I just tend to look herE: https://github.com/openstack/nova/branches | 13:41 |
| dansmith | johnthetubaguy: because then L configs (with null) won't work in M where they're non-nullable | 13:42 |
| dansmith | so this is a year to correct | 13:42 |
| johnthetubaguy | dansmith: I was figuring we fix that with compat logic over the next two releases, I must be missing a piece | 13:43 |
| bauzas | dansmith: johnthetubaguy: by discussing with john, a middleground was found where we could define a ratio equal to 0.0 | 13:44 |
| dansmith | johnthetubaguy: you're talking about making the config nullable, right? | 13:44 |
| johnthetubaguy | dansmith: like L sends null/or0.0 which gets mapped to 16 in the object, then the following release, the new nodes start sending 16, old ones send 0.0, then we drop it all the following release | 13:44 |
| bauzas | dansmith: johnthetubaguy: that wouldn't be needing a nullable field | 13:44 |
| dansmith | ah, 0.0 | 13:44 |
| johnthetubaguy | dansmith: yeah, just using the default default in oslo.config really, or 0.0 if we have to | 13:44 |
| bauzas | dansmith: yeah, I was pretty against a nullable field, and so we discussed how to properly default it | 13:44 |
| dansmith | bauzas: and still, we do the 0.0 detection in the object and not in every filter, right? | 13:45 |
| bauzas | dansmith: that, we agreed with johnthetubaguy | 13:45 |
| bauzas | dansmith: to keep the object facade | 13:45 |
| bauzas | dansmith: I had 3 concerns | 13:45 |
| johnthetubaguy | dansmith: yeah, I got confused with the object code, I thought it was running in nova-conductor, but its not | 13:45 |
| bauzas | 1/ the default value for RTs | 13:45 |
| dansmith | johnthetubaguy: well, I don't think it really matters where it's running, but anyway | 13:45 |
| bauzas | 2/ the hardcoded value in the object in case it goes to the default RT value | 13:45 |
| dansmith | sounds like this is a solution, so that's fine | 13:45 |
| bauzas | 3/ keeping the compat in the object | 13:46 |
| bauzas | so I'm about to spin-off a new change, not really a big stuff | 13:46 |
| johnthetubaguy | dansmith: its the need to read the config on the scheduler node bit, I think we can actually do that in the object, apparently | 13:46 |
| *** mwagner_lap has joined #openstack-nova | 13:46 | |
| dansmith | right | 13:46 |
| *** diogogmt has quit IRC | 13:47 | |
| johnthetubaguy | I got that mixed up in my head with the first set of comments, certainly hides the mess putting it in there | 13:47 |
| bauzas | okay, hold your fire, writing a new set, should be done in 20 mins | 13:49 |
| bauzas | johnthetubaguy: dansmith: so you could comment on | 13:49 |
| *** yuntongjin has quit IRC | 13:49 | |
| *** ferest has joined #openstack-nova | 13:50 | |
| *** yuntongjin has joined #openstack-nova | 13:51 | |
| *** mtanino has joined #openstack-nova | 13:52 | |
| *** mjturek1 has quit IRC | 13:53 | |
| *** mjturek1 has joined #openstack-nova | 13:54 | |
| *** ferest has quit IRC | 13:54 | |
| *** tpeoples has joined #openstack-nova | 13:56 | |
| *** Piet has quit IRC | 13:56 | |
| *** rods has left #openstack-nova | 13:58 | |
| *** yuntongjin1 has joined #openstack-nova | 13:58 | |
| *** exploreshaifali has joined #openstack-nova | 13:59 | |
| *** yuntongjin has quit IRC | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** 7GHAA12OT has joined #openstack-nova | 14:02 | |
| *** Kennan has quit IRC | 14:02 | |
| *** Kennan has joined #openstack-nova | 14:03 | |
| *** lyanchih has joined #openstack-nova | 14:03 | |
| *** jwcroppe has joined #openstack-nova | 14:03 | |
| *** fawadkhaliq has quit IRC | 14:04 | |
| *** fawadkhaliq has joined #openstack-nova | 14:05 | |
| *** rfolco has joined #openstack-nova | 14:06 | |
| *** krtaylor has quit IRC | 14:07 | |
| *** fawadkhaliq has quit IRC | 14:08 | |
| *** lyanchih has quit IRC | 14:09 | |
| *** ildikov has quit IRC | 14:10 | |
| *** eglynn_ has quit IRC | 14:11 | |
| *** doude has quit IRC | 14:12 | |
| *** yuntongjin1 has quit IRC | 14:15 | |
| openstackgerrit | John Garbutt proposed openstack/nova: api: allow any scheduler hints https://review.openstack.org/217727 | 14:16 |
| johnthetubaguy | sdague: alex_xu: I have put this idea up, to see if that helps with the case alaski is worried about, what do you think? | 14:17 |
| johnthetubaguy | sdague: alex_xu: alaski: https://review.openstack.org/#/c/217727/ | 14:17 |
| * alex_xu is checking | 14:17 | |
| *** shoutm has joined #openstack-nova | 14:18 | |
| alex_xu | johnthetubaguy: emm... it is change the v2.1 api contract also, maybe we should doing this only for v2 | 14:19 |
| jwcroppe | dansmith: morning... regarding https://review.openstack.org/#/c/216923/ - feel free to let me know if you're cool with this, it seems to align with your comment in terms of general direction | 14:19 |
| alex_xu | johnthetubaguy: if sdague alaski like this, I can help for find a way tune that tomorrow | 14:20 |
| dansmith | jwcroppe: yep, wanna make sure jay is cool with that, when he gets on | 14:20 |
| dansmith | should be shortly | 14:20 |
| jwcroppe | dansmith: cool. thanks | 14:20 |
| *** diogogmt has joined #openstack-nova | 14:21 | |
| jwcroppe | dansmith: it will be good to get that bugger squashed | 14:21 |
| dansmith | yep | 14:21 |
| *** romainh has joined #openstack-nova | 14:21 | |
| *** eharney has joined #openstack-nova | 14:23 | |
| *** yamahata has joined #openstack-nova | 14:23 | |
| *** yamahata has quit IRC | 14:23 | |
| *** yamahata has joined #openstack-nova | 14:24 | |
| *** IanGovett has quit IRC | 14:24 | |
| *** diogogmt has quit IRC | 14:25 | |
| *** IanGovett has joined #openstack-nova | 14:25 | |
| sdague | alex_xu: so I think we agreed at this point that scheduler filters would be porous, i.e. allow extension. Because a bunch of folks do that. | 14:27 |
| sdague | we can work on locking that down in the future | 14:28 |
| alex_xu | sdague: emm...I also agree we allow extend schedule hints | 14:29 |
| *** angdraug has joined #openstack-nova | 14:30 | |
| alex_xu | sdague: just thinking that we shouldn't limit that in the beginning of v2.1, but we did that | 14:30 |
| *** achanda has joined #openstack-nova | 14:31 | |
| *** browne has joined #openstack-nova | 14:31 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add cpu_allocation_ratio and ram_allocation_ratio to ComputeNode https://review.openstack.org/215471 | 14:33 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update HostManager and filters to use ComputeNode ratios https://review.openstack.org/216696 | 14:33 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update ComputeNode values with allocation ratios in the RT https://review.openstack.org/216362 | 14:33 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix the default allocation ratios to 0.0 https://review.openstack.org/217737 | 14:33 |
| bauzas | ndipanov: johnthetubaguy: dansmith: ^ sorry for pinging you guys, but here is the latest news | 14:33 |
| alex_xu | mriedem: sdague johnthetubaguy , extend 403 case https://review.openstack.org/#/c/217577/ | 14:33 |
| bauzas | ndipanov: johnthetubaguy: dansmith: since it's a pretty sensitive change, I'd love to get feedback soon if you could | 14:34 |
| johnthetubaguy | alex_xu: funny, just reading that, about to add a +2 | 14:34 |
| alex_xu | johnthetubaguy: thanks :) | 14:34 |
| *** krtaylor has joined #openstack-nova | 14:34 | |
| *** agireud has joined #openstack-nova | 14:35 | |
| *** ijuwang has joined #openstack-nova | 14:35 | |
| *** yuntongjin has joined #openstack-nova | 14:36 | |
| johnthetubaguy | alex_xu: I have two slight nits on the text, but not going to block on them, but let me know if you update it | 14:36 |
| *** yuntongjin has quit IRC | 14:36 | |
| *** romainh has left #openstack-nova | 14:36 | |
| mriedem | alex_xu: i'm sure jichen will appreciate that :) | 14:36 |
| alex_xu | johnthetubaguy: thanks, update soon | 14:37 |
| *** links has quit IRC | 14:37 | |
| alex_xu | mriedem: yea :) | 14:37 |
| mriedem | +2 but want sdague to review it | 14:37 |
| *** Nikolay_St has quit IRC | 14:37 | |
| bauzas | johnthetubaguy: ndipanov: dansmith: holy shit, I badly covered the case where the compute is new and the default config is there, just fixing that | 14:37 |
| ndipanov | bauzas, so that was my point right | 14:38 |
| sdague | mriedem: +A | 14:38 |
| *** salv-orlando has quit IRC | 14:38 | |
| bauzas | ndipanov: yeah, fixing that now | 14:38 |
| bauzas | ndipanov: I mean, my test was wrong and not covering that, but my if branch is correct, unless I missed something | 14:39 |
| *** achanda has quit IRC | 14:40 | |
| alex_xu | sdague: thanks, should I fix the words now? | 14:40 |
| sdague | alex_xu: if you like | 14:41 |
| sdague | alex_xu: the gate is so full, you might as well | 14:41 |
| openstackgerrit | Alex Xu proposed openstack/nova: Add a note about the 400 response not requiring a microversion https://review.openstack.org/217577 | 14:41 |
| alex_xu | sdague: done ^ :) | 14:41 |
| bauzas | oh man, value is 0.0 | 14:41 |
| bauzas | 'value is 0.0' | 14:41 |
| bauzas | bauzas: you have to copy 100 times "I shall not copy/paste dumbly" | 14:42 |
| *** agireud has quit IRC | 14:42 | |
| * bauzas hides | 14:42 | |
| johnthetubaguy | bauzas: be sure to cut and paste your lines | 14:42 |
| * alex_xu go to bed now. will continue thinking about scheduling-hint in the sleep.... | 14:42 | |
| bauzas | johnthetubaguy: nah, just quickly copied 'if value is None' and changed it to 'if value is 0.0' which is just silly | 14:43 |
| bauzas | johnthetubaguy: in the last change I quickly wrote | 14:44 |
| bauzas | and since my coverage was unsufficient... | 14:44 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add cpu_allocation_ratio and ram_allocation_ratio to ComputeNode https://review.openstack.org/215471 | 14:45 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update HostManager and filters to use ComputeNode ratios https://review.openstack.org/216696 | 14:45 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Update ComputeNode values with allocation ratios in the RT https://review.openstack.org/216362 | 14:45 |
| bauzas | dansmith: johnthetubaguy: ndipanov: ^that is much better to review | 14:45 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Fix precedence of image bdms over image mappings https://review.openstack.org/213624 | 14:47 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Change order of bdms in merged bdm list https://review.openstack.org/217743 | 14:47 |
| *** yamahata has quit IRC | 14:48 | |
| *** dustins has quit IRC | 14:48 | |
| *** yamahata has joined #openstack-nova | 14:49 | |
| *** dims has quit IRC | 14:51 | |
| *** dims has joined #openstack-nova | 14:51 | |
| *** agireud has joined #openstack-nova | 14:52 | |
| *** ildikov has joined #openstack-nova | 14:53 | |
| *** afazekas_ has joined #openstack-nova | 14:54 | |
| *** jaypipes has joined #openstack-nova | 14:55 | |
| *** armax has joined #openstack-nova | 14:55 | |
| *** ijuwang has quit IRC | 14:55 | |
| *** davideagnello has quit IRC | 14:56 | |
| *** tpeoples has quit IRC | 14:56 | |
| *** dustins has joined #openstack-nova | 14:57 | |
| jaypipes | morning all | 14:58 |
| *** tonytan4ever has joined #openstack-nova | 14:58 | |
| *** drankis has quit IRC | 14:59 | |
| dansmith | jaypipes: wanna revisit this? https://review.openstack.org/#/c/216923/4 | 14:59 |
| dansmith | jaypipes: that's my preference for solving it, which I think you were okay with, but you're still -1 so wanted to check | 14:59 |
| bauzas | yeah, I think it's a seamless approach | 15:00 |
| *** 7GHAA12OT has quit IRC | 15:01 | |
| *** doude has joined #openstack-nova | 15:02 | |
| *** doude has quit IRC | 15:02 | |
| *** doude has joined #openstack-nova | 15:02 | |
| *** haomaiwa_ has joined #openstack-nova | 15:02 | |
| *** krtaylor has quit IRC | 15:02 | |
| *** eglynn has joined #openstack-nova | 15:02 | |
| *** hemnafk is now known as hemna | 15:03 | |
| *** nkrinner has quit IRC | 15:03 | |
| *** xiaohui has quit IRC | 15:03 | |
| *** xiaohui has joined #openstack-nova | 15:04 | |
| *** tpeoples has joined #openstack-nova | 15:06 | |
| *** fawadkhaliq has joined #openstack-nova | 15:06 | |
| *** agireud has quit IRC | 15:06 | |
| andreykurilin | sdague: hi! Is there a way to launch each functional test(in novaclient) in separate tenant? | 15:06 |
| sdague | andreykurilin: hmmmm I don't think so | 15:06 |
| *** mdrabe has quit IRC | 15:08 | |
| jaypipes | dansmith: looking.. | 15:08 |
| andreykurilin | sdague: :( it's hard to test list command(there is a limit in 10 vms for 4 parallel tests) | 15:08 |
| jaypipes | andreykurilin: there is that ability in tempest (I know because I added it...) perhaps it's possible to use tempest for tests of that kind of functionality that you need to run? | 15:09 |
| jaypipes | bauzas: "criticity" <-- interesting. :) | 15:10 |
| *** fawadkhaliq has quit IRC | 15:10 | |
| sdague | jaypipes: yeh, well was hoping to get this on the novaclient side instead. | 15:10 |
| andreykurilin | jaypipes: no, I don't want to use tempest for this task:) I just want to add several functional tests in novaclient, which should check new cli stuff | 15:10 |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Handle neutron exception on bad floating ip create request https://review.openstack.org/210684 | 15:10 |
| *** jmccrory has quit IRC | 15:10 | |
| bauzas | jaypipes: heh | 15:10 |
| mtreinish | dims, mriedem: ^^^ sorry for the slow turnaround | 15:10 |
| sdague | andreykurilin: so, honestly, I'd just make the novaclient functional tests run in serial for now. There aren't that many of them. | 15:11 |
| jwcroppe | jaypipes: we love new words :) | 15:11 |
| mnestratov | mriedem: hi, could you please take a look at https://review.openstack.org/#/c/184153/ | 15:11 |
| *** jmccrory has joined #openstack-nova | 15:12 | |
| bauzas | jaypipes: okay, hope it's not critical :p | 15:12 |
| jaypipes | dansmith, bauzas, jwcroppe: +Wallaby'd it. | 15:12 |
| dansmith | jaypipes: Wcool | 15:12 |
| bauzas | Well | 15:12 |
| bauzas | jaypipes: so you jumped on your chair? | 15:12 |
| jaypipes | andreykurilin, sdague: sure... understood. perhaps andreykurilin, you could take a look at how tempest accomplishes the creation of isolated tenants and port that functionality to python-novaclient's functional tests. just a suggestion, nothing more. | 15:13 |
| *** burgerk has quit IRC | 15:13 | |
| jaypipes | bauzas: I have mad hops, yo. | 15:13 |
| *** davideagnello has joined #openstack-nova | 15:14 | |
| jwcroppe | jaypipes: thanks | 15:14 |
| bauzas | jaypipes: that reminds me that the allocation-ratios slightly diverged from what we discussed initially, your thoughts would be appreciated :) | 15:15 |
| bauzas | jaypipes: huge UpgradeImpact stuff (/teaser) | 15:15 |
| *** thedodd has joined #openstack-nova | 15:15 | |
| *** garyk has joined #openstack-nova | 15:16 | |
| jwcroppe | dansmith, bauzas: thanks for the quick reviews on that | 15:17 |
| andreykurilin | jaypipes: since I want to merge new feature before new release of novaclient, I'll try to work on isolation of tests starting next week:) | 15:17 |
| bauzas | jwcroppe: np, tell me anytime when you consider something with huge criticity | 15:17 |
| jwcroppe | bauzas: will do | 15:18 |
| *** Guest54162 is now known as jgriffith | 15:18 | |
| *** ccarmack has left #openstack-nova | 15:18 | |
| *** jgriffith is now known as Guest28159 | 15:19 | |
| *** yangyp has joined #openstack-nova | 15:21 | |
| *** yangyp has quit IRC | 15:21 | |
| *** exploreshaifali has quit IRC | 15:22 | |
| *** links has joined #openstack-nova | 15:22 | |
| sdague | andreykurilin: yeh, so the run time is short enough now, I'd just make it serial. And do the isolation post release. | 15:25 |
| *** afazekas_ has quit IRC | 15:27 | |
| andreykurilin | sdague: is it already in master or may I do it? | 15:27 |
| *** mdrabe has joined #openstack-nova | 15:27 | |
| *** dustins has quit IRC | 15:28 | |
| jaypipes | andreykurilin: sure, no worries :) | 15:28 |
| mtreinish | andreykurilin: fwiw, it's been on the list to migrate the tenant isolation mechanisms from tempest to tempest-lib, although no progress has been made on that so far (except for basically dead patches renaming the providers before the migration) | 15:29 |
| mtreinish | whenever that's done you should likely be able to reuse it for the client functional suite | 15:29 |
| sdague | andreykurilin: it's not yet in master, it should just be a single line change to the tox.ini | 15:29 |
| andreykurilin | sdague: ok, I'll try to do it now | 15:29 |
| sdague | andreykurilin: if you do it, I'll +2 it pretty quickly | 15:29 |
| andreykurilin | mtreinish: thanks for info | 15:30 |
| sdague | mtreinish might have the magic add for that somewhere pretty easily I forget if you can pass --concurrency=1 through the python setup.py testr command | 15:31 |
| *** kumar has joined #openstack-nova | 15:32 | |
| mtreinish | sdague: I normally use: python setup.py test --no-parallel | 15:34 |
| mtreinish | but --concurrency=1 will do it too | 15:34 |
| *** toure1 has joined #openstack-nova | 15:34 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Launch functional tests serially https://review.openstack.org/217768 | 15:34 |
| *** toure1 has quit IRC | 15:35 | |
| *** afazekas_ has joined #openstack-nova | 15:35 | |
| *** toure has quit IRC | 15:35 | |
| *** toure has joined #openstack-nova | 15:35 | |
| mtreinish | andreykurilin: ^^^ the concurrency flag has to be passed into testr args | 15:36 |
| mtreinish | because it's a testr arg | 15:36 |
| *** tsekiyama has joined #openstack-nova | 15:36 | |
| *** atuvenie has quit IRC | 15:37 | |
| andreykurilin | mtreinish sdague: after reading run-tests.sh file from tempest, I thought that tests are launched serially by default(when no option is determined) - https://github.com/openstack/tempest/blob/master/run_tests.sh#L93-L95 | 15:37 |
| *** changbl has quit IRC | 15:37 | |
| sdague | andreykurilin: these are triggered via tox -e functional | 15:38 |
| sdague | which will be parallel by default | 15:38 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Launch functional tests serially https://review.openstack.org/217768 | 15:39 |
| andreykurilin | mtreinish: should it be like this^ | 15:39 |
| andreykurilin | sdague: got it | 15:39 |
| *** toure has quit IRC | 15:39 | |
| *** SourabhP has joined #openstack-nova | 15:40 | |
| *** skylerberg has joined #openstack-nova | 15:40 | |
| *** ccesario has quit IRC | 15:41 | |
| mtreinish | andreykurilin: that will make it parallel if you want to filter what you're running | 15:41 |
| mtreinish | but yes it'll make the gate run serially | 15:41 |
| bauzas | edleafe: oh man, why have you provided the reservation_id for the logs in the nova.filters ? | 15:42 |
| mtreinish | andreykurilin: also by switching to the pbr command you lose the subunit-trace output formatting | 15:42 |
| bauzas | edleafe: that stuff is not really done | 15:42 |
| *** toure has joined #openstack-nova | 15:42 | |
| *** bnemec has joined #openstack-nova | 15:42 | |
| bauzas | edleafe: and I don't want it to be passed to the Scheduler honestly - I haven't planned to pass it in the Spec object | 15:43 |
| *** shoutm has quit IRC | 15:43 | |
| *** jdurgin has quit IRC | 15:43 | |
| bauzas | edleafe: here https://github.com/openstack/nova/blob/master/nova/filters.py#L109 | 15:43 |
| *** Marga_ has quit IRC | 15:43 | |
| *** changbl has joined #openstack-nova | 15:44 | |
| *** jdurgin has joined #openstack-nova | 15:44 | |
| *** toure has quit IRC | 15:44 | |
| *** Nikolay_St has joined #openstack-nova | 15:44 | |
| *** toure has joined #openstack-nova | 15:44 | |
| *** toure has quit IRC | 15:46 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: Launch functional tests serially https://review.openstack.org/217768 | 15:46 |
| *** toure has joined #openstack-nova | 15:46 | |
| andreykurilin | mtreinish: ^ | 15:47 |
| *** vivekd has joined #openstack-nova | 15:47 | |
| jaypipes | bauzas: FYI, on your question to ndipanov about why not set move_type to 'resize'... | 15:47 |
| bauzas | jaypipes: yup ? | 15:47 |
| bauzas | jaypipes: that's for my knowledge | 15:47 |
| jaypipes | bauzas: because resize is not actually moving anything... i.e. it's not a migration. | 15:47 |
| bauzas | mmm | 15:47 |
| ndipanov | jaypipes, well it can be | 15:47 |
| bauzas | really ? I thought it can be moved | 15:47 |
| ndipanov | it's really confusing | 15:47 |
| bauzas | resize can be providing a new dest host IIRC | 15:48 |
| *** andreykurilin__ has joined #openstack-nova | 15:48 | |
| * jaypipes considers a resize to a different host as being a cold migration... | 15:48 | |
| ndipanov | we are exceptionally bad at naming things | 15:48 |
| *** Apoorva has joined #openstack-nova | 15:48 | |
| ndipanov | but it's called a resize | 15:48 |
| mriedem | mnestratov: looking | 15:48 |
| bauzas | cold migration is not changing flavors | 15:48 |
| bauzas | hence migration != resize AFAICU | 15:48 |
| *** lykinsbd has joined #openstack-nova | 15:49 | |
| bauzas | jaypipes: but all of that is pretty done the same way in the conductor IIRC | 15:49 |
| *** Marga_ has joined #openstack-nova | 15:49 | |
| mriedem | mnestratov: sed -i -e 's/MIN_LIBVIRT_PARALLELS_VERSION = (1, 2, 12)/MIN_LIBVIRT_PARALLELS_VERSION = (1, 2, 8)/' nova/virt/libvirt/driver.py ? | 15:49 |
| *** rushil has quit IRC | 15:50 | |
| *** SourabhP has quit IRC | 15:50 | |
| *** afazekas_ has quit IRC | 15:50 | |
| *** sahid has quit IRC | 15:50 | |
| *** changbl has quit IRC | 15:51 | |
| mriedem | mnestratov: so i see you're running this ci with libvirt-1.2.8-16.vz7_1.3.13.x86_64 | 15:51 |
| *** SourabhP has joined #openstack-nova | 15:51 | |
| mnestratov | mriedem: it does look weird but it's temporary till we switch to later version | 15:51 |
| mnestratov | yes | 15:51 |
| bauzas | jaypipes: I have a question for you since edleafe is not responding but why did we approve https://github.com/openstack/nova/blob/master/nova/filters.py#L109 ? | 15:51 |
| bauzas | jaypipes: reservation_id is purely an EC2 hack | 15:52 |
| *** SourabhP_ has joined #openstack-nova | 15:52 | |
| mriedem | mnestratov: so if you can run parallels with libvirt 1.2.8, why isn't that in the driver code in nova? | 15:52 |
| bauzas | jaypipes: that I don't want to see in the scheduler | 15:52 |
| *** alexschm has quit IRC | 15:52 | |
| *** lykinsbd has quit IRC | 15:52 | |
| bauzas | jaypipes: instead, we have something called context.request_id :) | 15:52 |
| mnestratov | we backported all upstream libvirt patches to libvirt 1.2.8 | 15:52 |
| mriedem | mnestratov: just for your CI? | 15:53 |
| *** lykinsbd has joined #openstack-nova | 15:53 | |
| mriedem | bauzas: we still support the ec2 api | 15:53 |
| mriedem | and reservation ID is in that API and in other parts of the code | 15:53 |
| bauzas | mriedem: agreed, but that has nothing to deal with the scheduler | 15:53 |
| jaypipes | bauzas: two different things. | 15:53 |
| *** longlq has joined #openstack-nova | 15:53 | |
| bauzas | mriedem: the scheduler is a dead-end and no filters are using it | 15:53 |
| mnestratov | in upstream libvirt minimal stuff for openstack came at 1.2.12 | 15:53 |
| mriedem | mnestratov: ok, so no distro would actually work with this and libvirt 1.2.8 | 15:54 |
| bauzas | jaypipes: I know the difference, but still | 15:54 |
| *** longlq has quit IRC | 15:54 | |
| *** Guest28159 has quit IRC | 15:54 | |
| bauzas | jaypipes: I mean, we are generating an id and persisting it | 15:54 |
| mnestratov | mriedem: actually we want to rebase to something more fresh | 15:55 |
| *** moshele has quit IRC | 15:55 | |
| bauzas | jaypipes: but that's not really something we have a contract on, except for the EC2 API | 15:55 |
| mriedem | mnestratov: ah yeah i see the vz7 in the package name libvirt-1.2.8-16.vz7_1.3.13.x86_64 | 15:55 |
| mriedem | so that makes sense | 15:55 |
| mriedem | just threw me off a bit | 15:55 |
| *** jgriffit1 has joined #openstack-nova | 15:55 | |
| *** SourabhP has quit IRC | 15:56 | |
| *** SourabhP_ is now known as SourabhP | 15:56 | |
| mriedem | mnestratov: i don't want to -1 for this, but can you add a follow up patch that updates the hypervisor support matrix for this? http://docs.openstack.org/developer/nova/support-matrix.html | 15:57 |
| mriedem | http://docs.openstack.org/developer/nova/support-matrix.html#operation_snapshot | 15:57 |
| bauzas | alaski: thoughts on that ? https://github.com/openstack/nova/blob/master/nova/filters.py#L109 | 15:57 |
| bauzas | alaski: I honestly don't want it shipped in the ReqSpec object | 15:58 |
| mnestratov | mriedem: sure | 15:58 |
| *** agireud_ has joined #openstack-nova | 15:59 | |
| *** agireud_ has quit IRC | 15:59 | |
| alaski | bauzas: is it used anywhere? | 16:00 |
| bauzas | alaski: in the scheduler, no | 16:00 |
| alaski | okay | 16:00 |
| *** irenab has quit IRC | 16:00 | |
| bauzas | I would be pleased to drop it off | 16:00 |
| alaski | I don't see a reason for that to go to the scheduler | 16:00 |
| *** VW_ has joined #openstack-nova | 16:00 | |
| bauzas | alaski: I do think the same | 16:00 |
| *** haomaiwa_ has quit IRC | 16:01 | |
| bauzas | we already have the instance id | 16:01 |
| bauzas | and that's a hack | 16:01 |
| *** haomaiwa_ has joined #openstack-nova | 16:02 | |
| alaski | yeah, I'm working towards making that private | 16:02 |
| *** artom has joined #openstack-nova | 16:04 | |
| jaypipes | ndipanov: plan to push a rev for https://review.openstack.org/#/c/214713/ ? :) | 16:05 |
| ndipanov | jaypipes, yes hopefully in the next 45 min | 16:05 |
| ndipanov | I am working on some patches further up that branch atm | 16:05 |
| *** haomaiwa_ has quit IRC | 16:06 | |
| *** gszasz has quit IRC | 16:07 | |
| *** afazekas_ has joined #openstack-nova | 16:08 | |
| edleafe | bauzas: just catching up - in two meetings right now | 16:08 |
| bauzas | edleafe: I'm working on a change to remove the res_d | 16:09 |
| edleafe | bauzas: that's just logging for the ops | 16:09 |
| bauzas | res_id from the payload | 16:09 |
| *** SourabhP has quit IRC | 16:09 | |
| *** tjones has joined #openstack-nova | 16:09 | |
| *** tjones has left #openstack-nova | 16:09 | |
| bauzas | edleafe: yeah, got that but you know that we're working on providing a Spec object for the scheduler, right ? | 16:09 |
| edleafe | bauzas: sure, and we can update it then | 16:10 |
| bauzas | edleafe: which means that we make a contract, and we can't just pick an item from the current filt_props dict | 16:10 |
| bauzas | edleafe: I'm strongly opposed to adding reservation_id as a filed | 16:10 |
| bauzas | field | 16:10 |
| *** ccesario has joined #openstack-nova | 16:10 | |
| edleafe | bauzas: ops need a way to tie log entries to specific requests that failed | 16:10 |
| *** changbl has joined #openstack-nova | 16:10 | |
| edleafe | bauzas: what do you suggest> | 16:10 |
| edleafe | ? | 16:10 |
| bauzas | edleafe: using the instance uuid | 16:11 |
| bauzas | edleafe: and if 2 instances done at the same time, then dig in the logs and find the req-id | 16:11 |
| edleafe | bauzas: ok, that's already there | 16:11 |
| bauzas | edleafe: or look at instance-actions | 16:11 |
| bauzas | edleafe: I'm not against your change, it helps | 16:12 |
| edleafe | bauzas: so if I remove the reservation_id, will that suffice? | 16:12 |
| bauzas | edleafe: but it just adds one element to a contract | 16:12 |
| edleafe | from the logs | 16:12 |
| bauzas | edleafe: I'm just writing that | 16:12 |
| mriedem | rlrossit: https://bugs.launchpad.net/nova/+bug/1489531 | 16:12 |
| openstack | Launchpad bug 1489531 in OpenStack Compute (nova) "There should be a nova.objects.fields.DiskFormatEnum for the ImageMeta.disk_format field" [Wishlist,Confirmed] | 16:12 |
| *** smatzek has quit IRC | 16:13 | |
| *** smatzek has joined #openstack-nova | 16:13 | |
| rlrossit | thanks mriedem. Will start that in a bit | 16:13 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: Do not specify default security group to Neutron by default https://review.openstack.org/173204 | 16:13 |
| *** jichen has quit IRC | 16:13 | |
| *** fawadkhaliq has joined #openstack-nova | 16:13 | |
| *** jgriffit1 has quit IRC | 16:14 | |
| *** jgriffit1 has joined #openstack-nova | 16:15 | |
| *** smatzek_ has joined #openstack-nova | 16:15 | |
| *** smatzek has quit IRC | 16:15 | |
| *** kiran_c has joined #openstack-nova | 16:17 | |
| *** unicell has quit IRC | 16:20 | |
| *** lpetrut1 has quit IRC | 16:20 | |
| *** smatzek_ has quit IRC | 16:22 | |
| mriedem | mnestratov: so close, but -1 | 16:22 |
| mriedem | there is a false positive unit test, and looks like some other missing unit test coverage for the changes to libvirt utils | 16:22 |
| *** alex_klimov has quit IRC | 16:22 | |
| *** burt has joined #openstack-nova | 16:24 | |
| *** irenab has joined #openstack-nova | 16:24 | |
| *** jistr has quit IRC | 16:24 | |
| *** dustins has joined #openstack-nova | 16:24 | |
| *** exploreshaifali has joined #openstack-nova | 16:26 | |
| *** eharney has quit IRC | 16:26 | |
| mriedem | can anyone explain to me how libvirt + lxc or libvirt + parallels is different from the nova-docker driver? | 16:27 |
| *** claudiub|2 has joined #openstack-nova | 16:27 | |
| *** marzif has joined #openstack-nova | 16:27 | |
| *** sudipto has quit IRC | 16:27 | |
| *** afazekas_ has quit IRC | 16:27 | |
| mriedem | it would be an interesting post mortem in our devref as to why we included one set in tree but not the other - barring the whole mess of no community support for nova-docker, or is that the only reason it's not back in tree? | 16:28 |
| *** dannywilson has joined #openstack-nova | 16:29 | |
| *** matrohon has quit IRC | 16:29 | |
| *** dannywilson has quit IRC | 16:29 | |
| *** claudiub has quit IRC | 16:29 | |
| *** dannywilson has joined #openstack-nova | 16:29 | |
| *** salv-orlando has joined #openstack-nova | 16:29 | |
| dansmith | mriedem: quite different in mechanics | 16:30 |
| dansmith | mriedem: docker doesn't use libvirt for its containers | 16:30 |
| dansmith | mriedem: parallels doesn't use namespaces for its containerization | 16:31 |
| *** stefan_amann has quit IRC | 16:31 | |
| dansmith | well, it might use some of it now | 16:31 |
| mriedem | i'm not sure why "docker doesn't use libvirt for its containers" is a valid reason | 16:31 |
| dansmith | historically it hasn't, and there is still a lot of special-sauce differences in paralles | 16:31 |
| dansmith | mriedem: because the docker driver requires all new nova code to talk to docker, but libvirt-lxc requires just a few differences in the libvirt driver | 16:32 |
| mriedem | yeah, i understand it's easier to add support for things that libvirt already supports per https://libvirt.org/drivers.html | 16:33 |
| mriedem | btw, i'm only playing devils advocate here | 16:33 |
| danpb | yeah, if you look at the changes needed to support parallels and lxc over what we had to kvm, it was trivial | 16:33 |
| *** annegentle has quit IRC | 16:33 | |
| danpb | most of the work in parallels was related to their special disk format in fact | 16:33 |
| ndipanov | jaypipes, might be tomorrow :( | 16:33 |
| danpb | and not directly related to the containre technology | 16:33 |
| *** rajesht_ has quit IRC | 16:34 | |
| danpb | i'm personally fine with having docker in tree, if people are will to maintain it | 16:34 |
| *** nic has joined #openstack-nova | 16:34 | |
| danpb | but despite there apparently being people who want it in nova, this never seems to turn in people willing to be maintainers for it | 16:34 |
| dansmith | and I'm only fine with lxc/parallels because it's such a small delta | 16:35 |
| dansmith | not because I think it's a godo thing :) | 16:35 |
| mriedem | ok, that was about what i figured on both fronts | 16:35 |
| mriedem | yes, nova-docker is bit rot city | 16:35 |
| * dansmith waits for the youtube link | 16:35 | |
| mriedem | and lxc/parallels are small deltas to the libvirt driver | 16:35 |
| mriedem | no link | 16:35 |
| *** sudipto has joined #openstack-nova | 16:35 | |
| mriedem | https://www.youtube.com/watch?v=0R5J6dvpujs ? | 16:36 |
| mriedem | if you need one | 16:36 |
| dansmith | there it is | 16:36 |
| danpb | i thought it was this link https://www.youtube.com/watch?v=dQw4w9WgXcQ | 16:36 |
| * dansmith isn't fooled | 16:36 | |
| mriedem | danpb: since your here still :) | 16:36 |
| mriedem | danpb: do you have any reason i shouldn't -2 this due to feature freeze? https://review.openstack.org/#/c/184153/ | 16:36 |
| mriedem | *you're | 16:37 |
| *** kiran_c has quit IRC | 16:37 | |
| danpb | ah its one of those fun lack-of-a-feature-implementation bugs | 16:38 |
| mriedem | yeah | 16:38 |
| mriedem | totally wishlist | 16:38 |
| mriedem | despite popular opinion, i don't like stomping on people's balls over this kind of thing, | 16:39 |
| danpb | if we're consistent with past practice we'd -2 it | 16:39 |
| mriedem | but kind of need to tow the line | 16:39 |
| *** boris-42 has joined #openstack-nova | 16:39 | |
| mriedem | yeah | 16:39 |
| danpb | though personally i would prefer to see us be more forgiving an accept this kind of thing | 16:39 |
| *** jgriffit1 has quit IRC | 16:39 | |
| *** doude has quit IRC | 16:40 | |
| mriedem | i just dropped the hammer | 16:40 |
| *** zhenguo has quit IRC | 16:41 | |
| *** ctrath has quit IRC | 16:41 | |
| johnthetubaguy | mriedem: nova-docker talks to the docker API I think | 16:43 |
| *** derekh has quit IRC | 16:44 | |
| *** skylerberg has quit IRC | 16:44 | |
| johnthetubaguy | oops, client no scroll down fail | 16:44 |
| *** lucasagomes has quit IRC | 16:44 | |
| *** geekinutah has quit IRC | 16:45 | |
| *** links has quit IRC | 16:45 | |
| *** yamahata has quit IRC | 16:45 | |
| * johnthetubaguy nods at the scrollback | 16:46 | |
| bauzas | johnthetubaguy: thanks for having reviewed the allocation-ratios changes | 16:46 |
| bauzas | johnthetubaguy: saw some comments, basically nits right? | 16:47 |
| *** browne has quit IRC | 16:47 | |
| *** browne has joined #openstack-nova | 16:47 | |
| bauzas | johnthetubaguy: do I need another iteration for having you happy? | 16:48 |
| johnthetubaguy | bauzas: I had to run to an hours worth of meetings, I escaped, so going to head back in there for a proper look, it seemed mostly sane I think | 16:49 |
| bauzas | johnthetubaguy: no worries | 16:49 |
| bauzas | johnthetubaguy: I'm currently working on rebasing the ReqSpec BP, but I was wondering if you were happy or not since you left some comments | 16:50 |
| *** eglynn has quit IRC | 16:50 | |
| johnthetubaguy | bauzas: I just wondered about that config change, might be easier to have in the third or forth patch rather than the first, but I am OK with it in context | 16:50 |
| bauzas | btw. the XenServer CI seems down... :( | 16:50 |
| *** ndipanov has quit IRC | 16:50 | |
| johnthetubaguy | bauzas: what did it say? | 16:50 |
| johnthetubaguy | BobBall: there is roomer of a dead CI system, is that true? | 16:50 |
| bauzas | johnthetubaguy: I need to look at those before calling BobBall :) | 16:51 |
| *** annegentle has joined #openstack-nova | 16:51 | |
| bauzas | johnthetubaguy: but I had no time since my Spec BP just takes all my time again... | 16:51 |
| *** annegentle has quit IRC | 16:51 | |
| *** EinstCrazy has joined #openstack-nova | 16:51 | |
| *** browne has quit IRC | 16:51 | |
| *** agireud has joined #openstack-nova | 16:52 | |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: support snapshots with parallels virt_type https://review.openstack.org/184153 | 16:52 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: snapshot creation now works for virtuozzo https://review.openstack.org/217797 | 16:52 |
| *** atuvenie has joined #openstack-nova | 16:52 | |
| *** sudipto has quit IRC | 16:52 | |
| *** lyanchih has joined #openstack-nova | 16:52 | |
| *** krtaylor has joined #openstack-nova | 16:52 | |
| *** afazekas_ has joined #openstack-nova | 16:52 | |
| *** annegentle has joined #openstack-nova | 16:54 | |
| *** browne has joined #openstack-nova | 16:55 | |
| *** ctrath has joined #openstack-nova | 16:55 | |
| johnthetubaguy | bauzas: no worries, thats the right thing to focus on right now, its crunch time | 16:55 |
| bauzas | yeah, 16 changes in a series :(' | 16:55 |
| *** marzif has quit IRC | 16:55 | |
| *** harlowja has joined #openstack-nova | 16:55 | |
| *** vilobhmm1 has joined #openstack-nova | 16:55 | |
| *** agireud has quit IRC | 16:56 | |
| *** aix has quit IRC | 16:58 | |
| *** unicell has joined #openstack-nova | 16:59 | |
| *** ccarmack has joined #openstack-nova | 16:59 | |
| BobBall | nice spelling johnthetubaguy | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Scheduler RPC API select_dests() https://review.openstack.org/199205 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Consider that all scheduler calls are IO Ops https://review.openstack.org/188486 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler https://review.openstack.org/188487 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Prepare filters for using RequestSpec object https://review.openstack.org/197912 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify left filters for RequestSpec https://review.openstack.org/202678 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify conductor to use RequestSpec object https://review.openstack.org/203078 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify metric-related filters for RequestSpec https://review.openstack.org/202676 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec in the ChanceScheduler https://review.openstack.org/199051 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify NUMA, PCI and num_instances filters for RequestSpec https://review.openstack.org/202677 | 17:00 |
| BobBall | and btw, I am not a happy bunny. | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Aggregate filters for RequestSpec https://review.openstack.org/202674 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify default filters for RequestSpec https://review.openstack.org/202675 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add RequestSpec methods for primitiving into dicts https://review.openstack.org/187197 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Modify Affinity filter for RequestSpec https://review.openstack.org/202673 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec object in HostManager https://review.openstack.org/191251 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager https://review.openstack.org/191250 | 17:00 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Remove reservation_id from the logs when a schedule fails https://review.openstack.org/217798 | 17:00 |
| BobBall | I was _JUST_ about to go home | 17:00 |
| BobBall | http://zuul.openstack.xenproject.org/scoreboard/?project=openstack/nova&user=jenkins,citrix_xenserver_ci,XenProject-CI&timeframe=24 Rumours are correct - xenserver CI is down | 17:00 |
| *** jgriffith_ has joined #openstack-nova | 17:00 | |
| bauzas | alaski: you should be happy with ^ (rebase of Spec object) | 17:01 |
| *** kylek3h has quit IRC | 17:01 | |
| BobBall | Stopped it running new tests | 17:01 |
| bauzas | alaski: I had to rebase my code and touch some stuff, since some changes were merged that impacted my branch | 17:01 |
| bauzas | BobBall: I apologize :) | 17:01 |
| bauzas | even :' | 17:01 |
| bauzas | or :( rather | 17:01 |
| *** lyanchih has quit IRC | 17:02 | |
| johnthetubaguy | BobBall: no worries, stopping the bad votes is good, you may now enjoy your trip home :) | 17:02 |
| BobBall | Looks like someone added a new BFV test and ignored the CI's -1 vote :) | 17:03 |
| BobBall | I can't because I'm on holiday tomorrow | 17:03 |
| BobBall | so I need to fix the CI now :D | 17:03 |
| *** lpetrut has joined #openstack-nova | 17:03 | |
| *** lpetrut has quit IRC | 17:03 | |
| *** jgriffith_ is now known as jgriffith | 17:03 | |
| *** lpetrut has joined #openstack-nova | 17:04 | |
| BobBall | https://review.openstack.org/#/c/213621/ | 17:04 |
| BobBall | The test landed and is broken so a revert is being pushed | 17:04 |
| *** geekinutah has joined #openstack-nova | 17:04 | |
| *** smatzek has joined #openstack-nova | 17:05 | |
| *** lpetrut has left #openstack-nova | 17:06 | |
| *** kiran_c has joined #openstack-nova | 17:07 | |
| mriedem | BobBall: https://review.openstack.org/#/c/182994/ was already fixed last week | 17:07 |
| *** afazekas_ has quit IRC | 17:07 | |
| mriedem | https://review.openstack.org/#/c/213723/ | 17:08 |
| *** Nikolay_St has quit IRC | 17:08 | |
| *** kiran_c has left #openstack-nova | 17:08 | |
| BobBall | sorry mriedem - is this related to the CI? | 17:08 |
| *** Marga_ has quit IRC | 17:08 | |
| mriedem | BobBall: that's in the context of https://review.openstack.org/#/c/213621/ | 17:09 |
| BobBall | Ah yes | 17:09 |
| *** eharney has joined #openstack-nova | 17:09 | |
| BobBall | I understand now | 17:09 |
| *** afazekas_ has joined #openstack-nova | 17:09 | |
| BobBall | But the comment on https://review.openstack.org/#/c/213621/13 suggests that the tempest test is going to be reverted until it is fixed | 17:09 |
| BobBall | But it's fine | 17:10 |
| BobBall | I've just excluded the test from the XS CI for now | 17:10 |
| BobBall | I'll re-enable it when I'm back from holiday and either the test is fixed or we can investigate + fix ourselves. | 17:10 |
| mriedem | tempest seems fine, cinder made a backward incompatible change to their api | 17:10 |
| mriedem | it's actually good that we have that test in tempest now :) | 17:10 |
| jgriffith | mriedem which cinder change? | 17:11 |
| mriedem | jgriffith: https://review.openstack.org/#/c/182994/ | 17:11 |
| mriedem | jgriffith: which you fixed in https://review.openstack.org/#/c/213723/ | 17:11 |
| jgriffith | mriedem: Yeah, that one!!! | 17:11 |
| mriedem | BobBall: by all means though, enjoy your time off | 17:12 |
| BobBall | mriedem: The issue is that the tempest test is broken for 3rd party CIs - probably because of a missing check in it; did you see the comment on https://review.openstack.org/#/c/213621/13? If you disagree with the comment, might be worth weighing in? | 17:14 |
| BobBall | Just so happens that the gate doesn't tickle this particular race condition | 17:15 |
| mriedem | BobBall: ah ok, i didn't see that, i agree it's a bug | 17:15 |
| *** afazekas_ has quit IRC | 17:15 | |
| mriedem | i'll leave it up to the QA team to sort that out if they revert or not | 17:16 |
| BobBall | Indeed. | 17:16 |
| BobBall | I'm just excluding the test to get the CI back up quickly :) | 17:16 |
| BobBall | Where quickly is some function of a very overloaded zuul | 17:16 |
| *** yamahata has joined #openstack-nova | 17:16 | |
| mriedem | sure, sounds reasonable | 17:17 |
| *** BobBall is now known as BobBall-AWOL | 17:17 | |
| dansmith | johnthetubaguy: not sure I can ever forgive you for the naming of this: https://blueprints.launchpad.net/nova/+spec/liberty-bump-object-and-rpcapi-versions | 17:17 |
| BobBall-AWOL | Will check + re-enable tonight bauzas: Feel free to xenserver: recheck and it will get queued for when I re-start running jobs. | 17:17 |
| johnthetubaguy | dansmith: its one of my most awesome-est whatsits ever | 17:19 |
| dansmith | johnthetubaguy: it's exercising my copy-paste skills :) | 17:19 |
| johnthetubaguy | dansmith: oh yeah, the commit message, oops | 17:19 |
| *** danpb has quit IRC | 17:20 | |
| johnthetubaguy | dansmith: I meant to ping you about that, yeah lets get that in earlier this time :) | 17:21 |
| dansmith | :) | 17:21 |
| *** bnemec has quit IRC | 17:21 | |
| dansmith | well, we're not going to get it all for sure | 17:21 |
| dansmith | but I'm doing instance right now | 17:21 |
| dansmith | which is the hardest | 17:21 |
| dansmith | I dont really think it's worth bumping the RPCs because there are like <=2 changes each | 17:21 |
| *** smatzek has quit IRC | 17:21 | |
| johnthetubaguy | dansmith: yeah, that makes sense, except maybe conductor, I think there stuff we can drop there? | 17:22 |
| * dansmith looks | 17:22 | |
| *** kiran_c has joined #openstack-nova | 17:22 | |
| johnthetubaguy | the removals that didn't require a bump | 17:22 |
| dansmith | yeah, true I guess | 17:22 |
| *** achanda has joined #openstack-nova | 17:22 | |
| dansmith | but, it's minor stuff, so probably lower priority than other drops | 17:22 |
| dansmith | but sure | 17:22 |
| johnthetubaguy | yeah, very true | 17:23 |
| openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: support snapshots with parallels virt_type https://review.openstack.org/184153 | 17:23 |
| dansmith | this instance bump is dropping a bunch of stuff | 17:23 |
| dansmith | and it's fairly hairy | 17:23 |
| dansmith | so that's my top priority... about to post that in the next hour or so | 17:23 |
| dansmith | the manifest version backports is going to make sooo much stuff easier | 17:23 |
| * johnthetubaguy makes note for the morning | 17:24 | |
| dansmith | really looking forward to dropping the old stuff in M | 17:24 |
| johnthetubaguy | yeah, its amazing to see how far we have come with all this | 17:24 |
| * johnthetubaguy uses the term "we" like a manager | 17:25 | |
| *** irenab has quit IRC | 17:25 | |
| *** atuvenie has quit IRC | 17:25 | |
| dansmith | heh | 17:25 |
| *** kylek3h has joined #openstack-nova | 17:25 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Pull out the common bits of Instance into _BaseInstance https://review.openstack.org/217815 | 17:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Fix an issue with NovaObjectRegistry hook https://review.openstack.org/217816 | 17:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Refactor Instance tests to use objects.Instance https://review.openstack.org/217817 | 17:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Update objects test infrastructure for multiple versions https://review.openstack.org/217818 | 17:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance v2.0 object https://review.openstack.org/217819 | 17:25 |
| *** kiran_c has left #openstack-nova | 17:26 | |
| dansmith | jaypipes: you wanted that real bad, so I expect you to review it :) ^^ | 17:27 |
| *** afazekas_ has joined #openstack-nova | 17:27 | |
| *** garyk has quit IRC | 17:29 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: Hyper-V: determine live migration failures https://review.openstack.org/217822 | 17:30 |
| *** reed has joined #openstack-nova | 17:31 | |
| ftersin | alaski: around? | 17:32 |
| *** angdraug has quit IRC | 17:32 | |
| alaski | ftersin: yes | 17:32 |
| jaypipes | dansmith: will do shortly. | 17:32 |
| *** atuvenie has joined #openstack-nova | 17:33 | |
| ftersin | alaski: hi, i hope you understand something around https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L147 | 17:33 |
| *** browne has quit IRC | 17:33 | |
| *** mriedem1 has joined #openstack-nova | 17:33 | |
| ftersin | alaski: now we have a problem with cells: http://logstash.openstack.org/#eyJzZWFyY2giOiJcIkJsb2NrIERldmljZSBNYXBwaW5nIGlzIEludmFsaWQ6IEJvb3Qgc2VxdWVuY2UgZm9yIHRoZSBpbnN0YW5jZSBhbmQgaW1hZ2UvYmxvY2sgZGV2aWNlIG1hcHBpbmcgY29tYmluYXRpb24gaXMgbm90IHZhbGlkLlwiIiwiZmllbGRzIjpbXSwib2Zmc2V0IjowLCJ0aW1lZnJhbWUiOiIxNDQwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0NDA2OTU5ODk1NTd9 | 17:33 |
| ftersin | alaski: looks like an instance has two similar bdms | 17:33 |
| *** rdopiera has quit IRC | 17:34 | |
| ftersin | alaski: both bdms are root and have the same volume_id and device_name | 17:34 |
| *** Marga_ has joined #openstack-nova | 17:34 | |
| ftersin | alaski: is it possible, if device_name is changed to None and to /dev/vda during booting? | 17:35 |
| *** mriedem has quit IRC | 17:36 | |
| *** annegentle has quit IRC | 17:36 | |
| ftersin | alaski: i mean that initially device name is None, then it is updated to /dev/vda | 17:37 |
| alaski | ftersin: it shouldn't be, due to the code you linked first | 17:38 |
| ftersin | alaski: other case is that device name is initially /dev/vda, then it's updated to None, and finally - to /dev/vda | 17:38 |
| j_king | Fixed up issues and got this patch passing tempest tests, looking for some reviews https://review.openstack.org/#/c/210126/ | 17:39 |
| alaski | ftersin: the bdm is looked up by instance uuid and device name, so there's something really off if you're seeing two bdms with the same device name for an instance | 17:39 |
| alaski | ftersin: because the code is supposed to do an update or create | 17:39 |
| ftersin | alaski: i don't know where to find the bdm list in logs, but i found bdms of an snapshot of the instance | 17:41 |
| ftersin | alaski: http://logs.openstack.org/43/170243/13/gate/gate-tempest-dsvm-cells/4f030d7/logs/tempest.txt.gz#_2015-08-27_14_45_33_117 | 17:41 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove v3 references in unit test 'contrib' https://review.openstack.org/212851 | 17:45 |
| *** VW_ has quit IRC | 17:45 | |
| *** meghal has joined #openstack-nova | 17:45 | |
| *** VW_ has joined #openstack-nova | 17:46 | |
| alaski | ftersin: that's an image request call. I'm not sure where it pulls the bdm data from for that | 17:46 |
| alaski | I expect that's something stored in glance | 17:47 |
| ftersin | alaski: look at http://logs.openstack.org/43/170243/13/gate/gate-tempest-dsvm-cells/4f030d7/logs/screen-n-api.txt.gz#_2015-08-27_14_45_31_665 | 17:47 |
| ftersin | alaski: this is a log of creation of this image | 17:48 |
| *** afazekas_ has quit IRC | 17:48 | |
| *** exploreshaifali has quit IRC | 17:48 | |
| *** EinstCrazy has quit IRC | 17:48 | |
| ftersin | alaski: search there volume id: 450c797f-b05b-41d2-b1ba-82993ba29b3b | 17:48 |
| ftersin | alaski: two snapshots are created there from this volume | 17:49 |
| ftersin | alaski: the code which does it: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2327 | 17:50 |
| *** VW_ has quit IRC | 17:50 | |
| *** tjones has joined #openstack-nova | 17:51 | |
| ftersin | alaski: i don't believe that this code can create two snapshots from one bdm | 17:52 |
| alaski | ftersin: is this happening consistently for you on a patch | 17:52 |
| ftersin | alaski: not for me only | 17:53 |
| *** sachin has joined #openstack-nova | 17:53 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: Don't trace on InstanceInfoCacheNotFound when refreshing network info_cache https://review.openstack.org/217826 | 17:53 |
| alaski | okay, I was hoping to add some logging in | 17:53 |
| ftersin | alaski: and it's randomly | 17:53 |
| *** rods has joined #openstack-nova | 17:54 | |
| ftersin | alaski: since that this test was merged today | 17:54 |
| alaski | to prevent duplication the code at https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L124 keeps it from being created in the api first. then https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L151 ensures we only sync up a change when we can uniquely look up a bdm to update it | 17:55 |
| alaski | ftersin: which test? | 17:55 |
| ftersin | alaski: https://github.com/openstack/tempest/blob/master/tempest/scenario/test_volume_boot_pattern.py#L188 | 17:55 |
| openstackgerrit | Cale Rath proposed openstack/nova: Make generate_password more secure https://review.openstack.org/210092 | 17:56 |
| ftersin | alaski: you can look at logstash on the link above | 17:56 |
| *** kumar has quit IRC | 17:59 | |
| ftersin | alaski: iiuc db at api cell doesn't have bdm with no device name, does it? | 18:01 |
| *** meghal has quit IRC | 18:01 | |
| *** Kennan has quit IRC | 18:03 | |
| *** tonytan4ever has quit IRC | 18:03 | |
| *** Piet has joined #openstack-nova | 18:03 | |
| alaski | ftersin: no, it should not. That was fixed with the code I added | 18:03 |
| openstackgerrit | Andrew Laski proposed openstack/nova: DNM Adding logging around bdm actions for cells https://review.openstack.org/217829 | 18:03 |
| *** Kennan has joined #openstack-nova | 18:03 | |
| ftersin | alaski: thanks, will look extended logs | 18:07 |
| alaski | I'll keep rechecking that to try to hit the error | 18:10 |
| alaski | I'm also now wondering if we should have a unique constraint on device_name and instance uuid | 18:10 |
| alaski | but I would need ndipanov to chime in on that | 18:11 |
| *** ociuhandu has quit IRC | 18:11 | |
| *** atuvenie has quit IRC | 18:11 | |
| mriedem1 | i smell pending lawsuit https://www.youtube.com/watch?v=GNCd_ERZvZM | 18:11 |
| mriedem1 | https://www.youtube.com/watch?v=ABrSYqiqvzc | 18:11 |
| *** rfolco has quit IRC | 18:11 | |
| *** meghal has joined #openstack-nova | 18:15 | |
| *** apuimedo_ has joined #openstack-nova | 18:15 | |
| *** apuimedo_ has quit IRC | 18:20 | |
| *** SourabhP has joined #openstack-nova | 18:21 | |
| *** takedakn1 has quit IRC | 18:22 | |
| openstackgerrit | garyk proposed openstack/nova: VMware: enable a resize of instance with no root disk https://review.openstack.org/173802 | 18:23 |
| *** annegentle has joined #openstack-nova | 18:23 | |
| *** yassine has quit IRC | 18:25 | |
| *** annegentle has quit IRC | 18:25 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: Don't trace on InstanceInfoCacheNotFound when refreshing network info_cache https://review.openstack.org/217840 | 18:30 |
| *** meghal1 has joined #openstack-nova | 18:34 | |
| *** meghal has quit IRC | 18:34 | |
| *** browne has joined #openstack-nova | 18:35 | |
| *** tonytan4ever has joined #openstack-nova | 18:35 | |
| *** ctrath has quit IRC | 18:37 | |
| *** atuvenie has joined #openstack-nova | 18:38 | |
| *** exploreshaifali has joined #openstack-nova | 18:40 | |
| mriedem1 | dansmith: sdague: while looking at this https://review.openstack.org/#/c/213433/ i got thinking about that migration.status thing the other day - migration.status would be better named as migration.display_status and a unicode type in the database schema | 18:40 |
| mriedem1 | just thinking if someone comes along later and says https://review.openstack.org/#/c/213433/ broke some API b/c we all of a sudden started naming the volumes that nova creates | 18:41 |
| mriedem1 | we can say, well it's display_name which is user-editable, so it's not an API breakage | 18:41 |
| dansmith | yeah, it's unfortunately named | 18:41 |
| mriedem1 | could we add a new display_status column to the migration table that just mirrors migration.status for now in it's value? | 18:42 |
| mriedem1 | i assume this is all probably nothing no one wants to deal with right now | 18:43 |
| *** jichen has joined #openstack-nova | 18:46 | |
| *** annegentle has joined #openstack-nova | 18:46 | |
| *** angdraug has joined #openstack-nova | 18:49 | |
| *** Marga_ has quit IRC | 18:50 | |
| dansmith | does that help? all that matters is what the user sees, right? | 18:51 |
| *** xyhuang has quit IRC | 18:53 | |
| *** ctrath has joined #openstack-nova | 18:56 | |
| *** rushil has joined #openstack-nova | 18:56 | |
| *** atuvenie has quit IRC | 18:57 | |
| mriedem1 | dansmith: it may only help in making it more clear that 'this field is display status only, not part of some api contract, don't treat it as such' | 19:01 |
| mriedem1 | even though people probably still would | 19:01 |
| mriedem1 | mtreinish: went through https://review.openstack.org/#/c/210684/ | 19:01 |
| mriedem1 | mtreinish: you're not going to be happy | 19:01 |
| mriedem1 | mtreinish: i'll see if i can dig up where that ec2 api code is unit tested though | 19:01 |
| dansmith | mriedem1: well, we're going to introduce a state element, so hopefully state will look solid and status will seem fuzzy :) | 19:01 |
| mriedem1 | also, i thought about the unicode type on the column b/c of what sdague was saying about that value should be a translatable thing using _() | 19:02 |
| mriedem1 | looks like tags are the only things in the nova schema that use unicode types | 19:02 |
| mriedem1 | you know, for our friends in asia | 19:03 |
| dansmith | really? names should all be unicode | 19:03 |
| *** jichen_ has joined #openstack-nova | 19:03 | |
| dansmith | thought we had a push to clean that up a while back | 19:03 |
| mriedem1 | they are strings | 19:03 |
| dansmith | well, if they're UTF-8 then they're good, right? | 19:03 |
| *** jichen has quit IRC | 19:03 | |
| *** jichen_ is now known as jichen | 19:04 | |
| mriedem1 | http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/models.py#n286 | 19:04 |
| mriedem1 | yeah i assume things are encoded in the code before putting into the db and decoded when pulling out | 19:04 |
| mriedem1 | or whatever | 19:04 |
| mriedem1 | seems like that could still blow up the size limit on that column though | 19:04 |
| mriedem1 | since 'a' would be like what 4 chars? | 19:04 |
| dansmith | two but yeah | 19:05 |
| mriedem1 | was thinking u0041 | 19:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add Instance v2.0 object https://review.openstack.org/217819 | 19:05 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make the conductor fixture use version manifests https://review.openstack.org/217851 | 19:05 |
| dansmith | although jaypipes did say something about how mysql does silly things when encoding utf-8 stuff I think | 19:05 |
| dansmith | or that's just storing actually, so unrelated to the size limit | 19:06 |
| mriedem1 | i know you need to have your database or tables configured properly for unicode/utf-8 | 19:06 |
| dansmith | I'm pretty sure that accepting english into your culture is easier than unicode | 19:06 |
| mriedem1 | ha, +1 to that | 19:06 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Limit instances pulled in _heal_instances https://review.openstack.org/176925 | 19:07 |
| *** vivekd has quit IRC | 19:08 | |
| rlrossit | dansmith: instance 2.0? what's the fanciness that we're adding to it? | 19:08 |
| dansmith | rlrossit: the "fanciness" is "less baggage" | 19:10 |
| rlrossit | dansmith: lame | 19:10 |
| *** aix has joined #openstack-nova | 19:11 | |
| mriedem1 | rlrossit: is obviously a horder | 19:11 |
| rlrossit | mriedem1: you're not wrong | 19:11 |
| dansmith | heh | 19:12 |
| *** tjones has left #openstack-nova | 19:12 | |
| rlrossit | I tend to be lazy in the organizing things area. I went through my mail for the first time in 2 months last night | 19:12 |
| *** skylerberg has joined #openstack-nova | 19:14 | |
| mriedem1 | mtreinish: i think i found where that ec2 api Executor stuff is unit tested | 19:15 |
| *** dprince has quit IRC | 19:20 | |
| *** rushil has quit IRC | 19:26 | |
| *** jkomg has joined #openstack-nova | 19:28 | |
| *** aysyd has quit IRC | 19:29 | |
| *** smatzek has joined #openstack-nova | 19:30 | |
| mtreinish | mriedem1: heh, yeah not happy | 19:34 |
| mtreinish | I'll work on respinning it now | 19:34 |
| *** fawadkhaliq has quit IRC | 19:35 | |
| *** nic has quit IRC | 19:35 | |
| *** annegentle has quit IRC | 19:36 | |
| *** annegentle has joined #openstack-nova | 19:37 | |
| *** burgerk has joined #openstack-nova | 19:38 | |
| *** jichen has quit IRC | 19:39 | |
| *** aysyd has joined #openstack-nova | 19:39 | |
| *** gsilvis has joined #openstack-nova | 19:40 | |
| *** agireud has joined #openstack-nova | 19:41 | |
| *** skylerberg has quit IRC | 19:44 | |
| *** matrohon has joined #openstack-nova | 19:45 | |
| *** agireud has quit IRC | 19:46 | |
| *** nic has joined #openstack-nova | 19:47 | |
| *** agireud has joined #openstack-nova | 19:50 | |
| *** IanGovett has quit IRC | 19:51 | |
| *** agireud has quit IRC | 19:57 | |
| *** achanda has quit IRC | 19:58 | |
| *** smatzek has quit IRC | 20:01 | |
| *** fawadkhaliq has joined #openstack-nova | 20:06 | |
| *** xyang1 has quit IRC | 20:06 | |
| *** jmccrory has quit IRC | 20:07 | |
| *** whenry__ has quit IRC | 20:08 | |
| *** rushil has joined #openstack-nova | 20:09 | |
| *** matrohon has quit IRC | 20:09 | |
| *** atuvenie has joined #openstack-nova | 20:10 | |
| *** jmccrory has joined #openstack-nova | 20:11 | |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Handle neutron exception on bad floating ip create request https://review.openstack.org/210684 | 20:12 |
| *** baoli has quit IRC | 20:14 | |
| j_king | just wondering if anyone has time for a review https://review.openstack.org/#/c/210126/ cheers | 20:15 |
| *** eharney has quit IRC | 20:15 | |
| *** matrohon has joined #openstack-nova | 20:16 | |
| *** atuvenie has quit IRC | 20:16 | |
| *** reed has quit IRC | 20:16 | |
| *** yamahata has quit IRC | 20:16 | |
| *** adam_g has quit IRC | 20:17 | |
| *** mriedem1 is now known as mriedem | 20:17 | |
| *** rlrossit has left #openstack-nova | 20:17 | |
| *** adam_g has joined #openstack-nova | 20:21 | |
| *** adam_g has quit IRC | 20:21 | |
| *** adam_g has joined #openstack-nova | 20:21 | |
| *** reed has joined #openstack-nova | 20:21 | |
| *** achanda has joined #openstack-nova | 20:22 | |
| *** achanda has quit IRC | 20:22 | |
| *** achanda has joined #openstack-nova | 20:23 | |
| *** matrohon has quit IRC | 20:23 | |
| *** sachin has quit IRC | 20:24 | |
| mriedem | j_king: i'm not sure about that change | 20:24 |
| mriedem | some comments inline | 20:24 |
| j_king | mriedem: ty | 20:24 |
| *** rlrossit has joined #openstack-nova | 20:25 | |
| mriedem | also, don't we test rebuild in the gate? | 20:25 |
| mriedem | in the neutron job | 20:25 |
| *** matrohon has joined #openstack-nova | 20:25 | |
| *** sachin has joined #openstack-nova | 20:25 | |
| *** alex_klimov has joined #openstack-nova | 20:26 | |
| mriedem | j_king: http://logs.openstack.org/59/215859/7/check/gate-tempest-dsvm-neutron-full/7167f62/console.html#_2015-08-25_20_21_15_829 | 20:27 |
| mriedem | ^ indicates that the rebuild test works with neutron | 20:27 |
| *** vladikr has quit IRC | 20:29 | |
| vishwanathj | Hi, I am trying to understand and use the user_data option via heat by following the steps in blog https://developer.rackspace.com/blog/openstack-orchestration-in-depth-part-2-single-instance-deployments/ however, I see the following message in the instance log "/sbin/cirros-userdata: line 1: /run/cirros/datasource/data/user-data: not found", not sure what I am doing wrong.... | 20:29 |
| vishwanathj | any guidance would be highly appreciated | 20:29 |
| *** exploreshaifali has quit IRC | 20:30 | |
| j_king | mriedem: working on pulling some logs. we've had to float our own patch to support this in production for some time but we are still on an older stable branch. | 20:33 |
| j_king | mriedem: thanks for taking the time. | 20:33 |
| *** markvoelker has quit IRC | 20:33 | |
| *** dustins has quit IRC | 20:34 | |
| mriedem | j_king: it's possible you're missing some other recent changes for rebuild + neutron that cfriesen/sdague worked on | 20:36 |
| mriedem | which branch are you on? | 20:36 |
| *** dave-mccowan has quit IRC | 20:37 | |
| *** exploreshaifali has joined #openstack-nova | 20:39 | |
| j_king | mriedem: we're running stable/juno. we've been applying this patch to log/warn about stray ports being left out when instances are deleted during rebuild, etc. | 20:41 |
| *** geekinutah has quit IRC | 20:42 | |
| mriedem | j_king: https://review.openstack.org/#/c/169827/ | 20:44 |
| mriedem | that's what i was thinking of | 20:44 |
| mriedem | j_king: also, are you sure it's rebuild and not evacuate? | 20:44 |
| bauzas | mriedem: heard about a possible grenade issue with -infra ? | 20:45 |
| mriedem | no | 20:45 |
| *** exploreshaifali has quit IRC | 20:45 | |
| bauzas | mriedem: per the logs, out of servers | 20:45 |
| bauzas | thru ansible I mean | 20:45 |
| bauzas | mriedem: okay, lemme check your magic logstash query | 20:45 |
| mriedem | well that would suck | 20:45 |
| bauzas | hope it's transient | 20:46 |
| *** krtaylor has quit IRC | 20:46 | |
| dansmith | I have seen several grenade runs today that didn't start the new side | 20:46 |
| *** signed8bit has joined #openstack-nova | 20:48 | |
| bauzas | well, logstash shows it's pretty small | 20:48 |
| bauzas | http://logs.openstack.org/86/188486/30/check/gate-grenade-dsvm/ce74f1c/console.html as a pattern for matching | 20:49 |
| bauzas | I mean http://logs.openstack.org/86/188486/30/check/gate-grenade-dsvm/ce74f1c/console.html#_2015-08-27_18_29_39_581 | 20:49 |
| *** ildikov has quit IRC | 20:50 | |
| *** rushil has quit IRC | 20:50 | |
| mriedem | omg omg omg nova meeting in 10 minutes | 20:50 |
| * mriedem dances around and flutters hands | 20:50 | |
| bauzas | http://logstash.openstack.org/#eyJzZWFyY2giOiJidWlsZF9uYW1lOlwiZ2F0ZS1ncmVuYWRlLWRzdm1cIiBBTkQgbWVzc2FnZTpcIk5vIGhvc3RzIG1hdGNoZWRcIiBBTkQgYnVpbGRfc3RhdHVzOlwiRkFJTFVSRVwiIiwiZmllbGRzIjpbXSwib2Zmc2V0IjowLCJ0aW1lZnJhbWUiOiI4NjQwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0NDA3MDg1MTc5MDJ9 | 20:50 |
| *** mwagner_lap has quit IRC | 20:51 | |
| mriedem | bauzas: logstash is backed up so it could be worse than what logstash shows | 20:51 |
| *** n0ano has joined #openstack-nova | 20:51 | |
| bauzas | mriedem: erm | 20:51 |
| bauzas | mriedem: how can I know that ? | 20:51 |
| *** gyee has joined #openstack-nova | 20:51 | |
| *** rushil has joined #openstack-nova | 20:51 | |
| bauzas | mriedem: unless hassling infra ? | 20:51 |
| mriedem | you have to know the oracle | 20:51 |
| *** gibi has quit IRC | 20:52 | |
| bauzas | I won't say candyman 3 times | 20:52 |
| *** rushil_ has joined #openstack-nova | 20:52 | |
| mriedem | bauzas: so a few things | 20:53 |
| mriedem | http://status.openstack.org/zuul/ - at the bottom is a logstash worker queue graph | 20:54 |
| mriedem | when that's all spiky, things are possibly backed up | 20:54 |
| mriedem | i know that some things are backed up b/c of another gate failure that i wasn't getting hits on but i know is failing in some jobs | 20:54 |
| reed | bauzas, I will | 20:55 |
| reed | candyman | 20:55 |
| reed | candyman | 20:55 |
| reed | ... | 20:55 |
| * bauzas violently runs | 20:55 | |
| reed | :) | 20:55 |
| bauzas | mriedem: oh yeah | 20:55 |
| bauzas | mriedem: good point about the spikes | 20:55 |
| *** rushil has quit IRC | 20:56 | |
| *** raildo is now known as raildo-afk | 20:56 | |
| melwitt | I'm seeing a failure in the cells job that's not quite the same as the existing bug, Block Device Mapping is Invalid: Boot sequence for the instance and image/block device mapping combination is not valid.', u'code': 400 for test_create_ebs_image_and_check_boot | 20:57 |
| mriedem | melwitt: known issue | 20:57 |
| *** Marga_ has joined #openstack-nova | 20:57 | |
| mriedem | melwitt: well https://bugs.launchpad.net/tempest/+bug/1489581 | 20:58 |
| openstack | Launchpad bug 1489581 in tempest "test_create_ebs_image_and_check_boot is race failing" [High,In progress] - Assigned to Andrey Pavlov (apavlov-e) | 20:58 |
| melwitt | mriedem: okay, I thought it was different than https://bugs.launchpad.net/tempest/+bug/1489581 because that one says waiting for the snapshot to be available | 20:58 |
| bauzas | melwitt: yeah that too | 20:58 |
| bauzas | melwitt: I noticed the cells job less happy | 20:58 |
| mriedem | melwitt: do you have a link to some cells job logs that have a trace or something? | 20:58 |
| melwitt | mriedem: I was looking at the detail view of this so far http://logs.openstack.org/78/203078/20/check/gate-tempest-dsvm-cells/3f431ee/logs/testr_results.html.gz | 20:59 |
| bauzas | mriedem: http://logs.openstack.org/86/188486/30/check/gate-tempest-dsvm-cells/78a34c9/ sounds a good candidate | 20:59 |
| mriedem | melwitt: bauzas: is there an error in the nova logs? | 21:00 |
| mriedem | or anything INFO+ | 21:00 |
| melwitt | mriedem: I'm looking for that, not found yet | 21:00 |
| *** fawadkhaliq has quit IRC | 21:00 | |
| johnthetubaguy | mikal tjones cburgess jgrimm adrian_otto funzo mjturek jcookekhugen irina_pov krtaylor danpb alexpilotti flip214 jaypipes garyk edleafe dims moshele anteaya Nisha sileht claudiub lxsli neiljerram markus_z swamireddy alevine tonyb andreykurilin ndipanov sc68cal akuriata artom jlvillal mnestratov kashyap aloga rgeragnov bauzas xyang tpatil med_ nic scottda nagyz dannywilson belliott dguitarbite its nova meeting time #openstack-meeting | 21:00 |
| mriedem | yeah, i know there is a failure in the c-vol logs, but that wasn't showing up in logstash | 21:01 |
| mriedem | b/c it's backed up | 21:01 |
| mriedem | and it's super generic | 21:01 |
| mtreinish | mriedem: what did I screw up: http://logs.openstack.org/84/210684/4/check/gate-nova-python27/3d9afd6/console.html#_2015-08-27_20_31_27_075 | 21:01 |
| mtreinish | it works on the 2.1 case | 21:01 |
| *** thedodd has quit IRC | 21:01 | |
| mriedem | probably missing a mock | 21:02 |
| mriedem | nova-meeting | 21:02 |
| mriedem | the other test class does a shitload of mox stubbing in setup | 21:02 |
| mriedem | so it probably handles some mocks for you there, but it's super brittle | 21:02 |
| mtreinish | mriedem: :/ look at what you made me do | 21:03 |
| mriedem | hey, as top test guy in openstack land i expect you to be able to sort it out :P | 21:03 |
| melwitt | mriedem: found an INFO http://logs.openstack.org/78/203078/20/check/gate-tempest-dsvm-cells/3f431ee/logs/screen-n-api.txt.gz#_2015-08-27_18_42_37_934 (to look at after meeting) | 21:04 |
| *** aysyd has quit IRC | 21:04 | |
| mtreinish | mriedem: heh, I suck at writing tests. Have you ever looked at the code for my projects? | 21:07 |
| mriedem | disgraceful | 21:07 |
| dims | haha | 21:07 |
| *** rushil_ has quit IRC | 21:10 | |
| *** jcoufal has quit IRC | 21:10 | |
| mriedem | melwitt: yeah http://goo.gl/fJVVT5 | 21:12 |
| mriedem | nice find | 21:12 |
| mriedem | that's only for the cells job htough | 21:12 |
| *** thorst1 has quit IRC | 21:13 | |
| *** Piet has quit IRC | 21:13 | |
| *** tjones has joined #openstack-nova | 21:14 | |
| *** apuimedo_ has joined #openstack-nova | 21:16 | |
| *** shz has quit IRC | 21:19 | |
| *** rlrossit has left #openstack-nova | 21:20 | |
| *** shz has joined #openstack-nova | 21:20 | |
| *** skylerberg has joined #openstack-nova | 21:20 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: :Don't trace on InstanceInfoCacheNotFound when refreshing network info_cache https://review.openstack.org/217840 | 21:21 |
| *** burgerk has quit IRC | 21:25 | |
| *** rushil has joined #openstack-nova | 21:28 | |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Handle neutron exception on bad floating ip create request https://review.openstack.org/210684 | 21:29 |
| mtreinish | mriedem: got it ^^^ | 21:29 |
| mtreinish | missed a tab offset so the assert call wasn't under the mock | 21:29 |
| *** Piet has joined #openstack-nova | 21:30 | |
| *** sachin has quit IRC | 21:32 | |
| mriedem | weak | 21:33 |
| mtreinish | mriedem: it's all your fault anyway, the mox stuff didn't have to worry about tab offsets | 21:34 |
| mriedem | suuuure | 21:35 |
| mriedem | just if you're using the 4th or 5th index in the stubbed port list from the setup | 21:35 |
| mriedem | b/c if you mess that up the tests fall over | 21:36 |
| mtreinish | heh, fair enough | 21:36 |
| mriedem | anyway, i have to head out - i'll try to get back to that tonight | 21:36 |
| mtreinish | mriedem: no rush, it took me like 2 weeks to respin | 21:36 |
| mriedem | well stop traveling | 21:37 |
| mtreinish | also I'm sure oomichi will find something else wrong with it overnight | 21:37 |
| mriedem | yes, he will | 21:37 |
| mtreinish | mriedem: well after qa sprint nothing until tokyo | 21:37 |
| mriedem | travel or work? | 21:37 |
| mtreinish | travel | 21:37 |
| mtreinish | I don't think i can skip work for the 6 weeks before summit | 21:38 |
| mtreinish | not everyone gets to go to disney world | 21:38 |
| *** sachin has joined #openstack-nova | 21:38 | |
| mriedem | gets to? | 21:38 |
| mriedem | has to | 21:38 |
| mriedem | alright, really leaving | 21:39 |
| *** sachin has quit IRC | 21:39 | |
| *** mriedem has quit IRC | 21:39 | |
| *** sachin has joined #openstack-nova | 21:39 | |
| openstackgerrit | Aditi Rajagopal proposed openstack/nova: Rm openstack/common/versionutils from setup.cfg https://review.openstack.org/217924 | 21:39 |
| alaski | melwitt: ftersin brought up the cells failure earlier and I spent some time looking at it but haven't gotten anywhere yet | 21:40 |
| alaski | I did throw https://review.openstack.org/#/c/217829/ up to add some more logging if I can hit the failure there | 21:40 |
| alaski | but I added some code a while ago to prevent what's supposedly happening from happening, so I'm not sure what's up yet | 21:41 |
| melwitt | alaski: okay. I just barely looked at it and the invalid block device doesn't make any sense to me yet | 21:43 |
| alaski | what was pointed out to me is that a snapshot is writing info for two block devices to glance | 21:44 |
| *** thorst1 has joined #openstack-nova | 21:44 | |
| *** agireud has joined #openstack-nova | 21:44 | |
| melwitt | alaski: oh, that sounds familiar... | 21:44 |
| *** krtaylor has joined #openstack-nova | 21:44 | |
| *** annegentle has quit IRC | 21:45 | |
| alaski | I've been focusing on how two block devices might be in the nova db. I haven't looked at the snapshot process itself closely yet | 21:45 |
| melwitt | alaski: L449, pretty unhelpful comment from me https://etherpad.openstack.org/p/nova-cells-testing but I observed this before back when we were trying to get the job to pass | 21:46 |
| melwitt | alaski: and that sounds like the patch you had to avoid writing it initially if it doesn't have device set or something | 21:47 |
| alaski | melwitt: yeah. that should be preventing multiple bdms with the same device name being written to the db | 21:48 |
| alaski | but there are a couple of conditionals when writing bdms so it's possible that some path isn't being covered properly | 21:48 |
| *** rushil has quit IRC | 21:50 | |
| *** agireud has quit IRC | 21:54 | |
| *** rushil has joined #openstack-nova | 21:56 | |
| *** burt has quit IRC | 21:56 | |
| alaski | one potential problem I see is that there's a create call and an update_or_create call. and if a create and then update_or_create happened everything would be fine, but if they got processed backwards it would be a problem. however I don't see where create is used in this flow | 21:58 |
| *** jkomg has quit IRC | 21:59 | |
| melwitt | alaski: good point. makes one wonder why there is a separate create :P | 22:01 |
| *** rushil has quit IRC | 22:04 | |
| *** rushil has joined #openstack-nova | 22:04 | |
| *** rushil_ has joined #openstack-nova | 22:06 | |
| alaski | huh, I hit the failure on the second try with https://review.openstack.org/#/c/217829/ | 22:06 |
| *** penick has joined #openstack-nova | 22:06 | |
| melwitt | alaski: you're good | 22:06 |
| alaski | but there's food afoot so I will have to dig into it tomorrow | 22:06 |
| alaski | melwitt: I'm not sure whether to consider myself lucky or unlucky on that | 22:07 |
| *** annegentle has joined #openstack-nova | 22:07 | |
| melwitt | alaski: haha :) okay, seeya | 22:08 |
| *** rushil has quit IRC | 22:09 | |
| dansmith | food. afoot. | 22:09 |
| dansmith | such vocabulary and grace | 22:09 |
| melwitt | dansmith: see, someone else says afoot other than me | 22:09 |
| dansmith | yes, but he has much less impressive footwear | 22:10 |
| melwitt | :) | 22:10 |
| alaski | hah, very true | 22:10 |
| *** mdrabe has quit IRC | 22:13 | |
| *** rushil_ has quit IRC | 22:14 | |
| *** sachin has quit IRC | 22:15 | |
| *** rushil has joined #openstack-nova | 22:15 | |
| *** ctrath has quit IRC | 22:15 | |
| *** rushil_ has joined #openstack-nova | 22:16 | |
| *** rushil has quit IRC | 22:20 | |
| *** matrohon has quit IRC | 22:22 | |
| *** annegentle has quit IRC | 22:24 | |
| *** SourabhP has quit IRC | 22:24 | |
| *** annegentle has joined #openstack-nova | 22:25 | |
| *** sachin has joined #openstack-nova | 22:26 | |
| *** zzzeek has quit IRC | 22:26 | |
| *** SourabhP has joined #openstack-nova | 22:26 | |
| *** signed8bit has quit IRC | 22:26 | |
| *** nic has quit IRC | 22:29 | |
| *** annegentle has quit IRC | 22:29 | |
| *** signed8bit has joined #openstack-nova | 22:31 | |
| *** sachin has quit IRC | 22:33 | |
| *** kylek3h has quit IRC | 22:33 | |
| *** sachin has joined #openstack-nova | 22:33 | |
| *** tonytan4ever has quit IRC | 22:36 | |
| *** sachin has quit IRC | 22:37 | |
| *** sachin has joined #openstack-nova | 22:38 | |
| *** annegentle has joined #openstack-nova | 22:39 | |
| *** unicell1 has joined #openstack-nova | 22:42 | |
| *** unicell has quit IRC | 22:42 | |
| *** alex_klimov has quit IRC | 22:47 | |
| *** thangp has quit IRC | 22:47 | |
| *** yamahata has joined #openstack-nova | 22:47 | |
| *** edmondsw has quit IRC | 22:51 | |
| *** sachin has quit IRC | 22:51 | |
| *** sachin has joined #openstack-nova | 22:59 | |
| *** tpeoples has quit IRC | 22:59 | |
| *** jwcroppe has quit IRC | 23:00 | |
| *** liusheng has quit IRC | 23:02 | |
| *** liusheng has joined #openstack-nova | 23:03 | |
| *** jwcroppe_ has joined #openstack-nova | 23:03 | |
| *** rushil_ has quit IRC | 23:04 | |
| *** asettle has joined #openstack-nova | 23:07 | |
| *** mriedem has joined #openstack-nova | 23:08 | |
| *** claudiub|2 has quit IRC | 23:09 | |
| *** jwcroppe_ has quit IRC | 23:09 | |
| *** nelsnels_ has joined #openstack-nova | 23:10 | |
| *** jwcroppe has joined #openstack-nova | 23:10 | |
| *** hemna is now known as hemnafk | 23:12 | |
| *** shoutm has joined #openstack-nova | 23:12 | |
| *** nelsnelson has quit IRC | 23:12 | |
| *** annegentle has quit IRC | 23:14 | |
| *** jwcroppe has quit IRC | 23:14 | |
| *** VW_ has joined #openstack-nova | 23:16 | |
| mriedem | mtreinish: one little thing in that patch and then i'm +2 | 23:16 |
| *** VW_ has quit IRC | 23:18 | |
| *** dims_ has joined #openstack-nova | 23:18 | |
| *** takedakn has joined #openstack-nova | 23:20 | |
| *** apuimedo_ has quit IRC | 23:22 | |
| *** dims has quit IRC | 23:22 | |
| *** takedakn has quit IRC | 23:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/216968 | 23:25 |
| mtreinish | mriedem: err, one sec | 23:26 |
| *** Marga_ has quit IRC | 23:27 | |
| openstackgerrit | Matthew Treinish proposed openstack/nova: Handle neutron exception on bad floating ip create request https://review.openstack.org/210684 | 23:30 |
| mtreinish | mriedem: ^^^ | 23:30 |
| mriedem | +2 | 23:30 |
| *** jaypipes has quit IRC | 23:31 | |
| gmann | alex_xu: johnthetubaguy : yea, this is last one on tests collapse thing - https://review.openstack.org/#/c/217606/ | 23:35 |
| *** bnemec has joined #openstack-nova | 23:35 | |
| mtreinish | mriedem: thanks | 23:37 |
| mtreinish | one step closer to an ipv6 only dsvm env working | 23:37 |
| clarkb | mtreinish: are you still using my change to check on that? | 23:37 |
| mtreinish | clarkb: this won't actually fix that, tempest will still try to floating ip on ipv6 | 23:38 |
| mtreinish | but at least now nova won't 500 | 23:38 |
| clarkb | ah | 23:38 |
| *** dims_ has quit IRC | 23:38 | |
| mtreinish | clarkb: heh, it was supposed to be the easier fix | 23:39 |
| mtreinish | the tempest one is a bit more involved | 23:39 |
| *** vilobhmm1 has quit IRC | 23:39 | |
| *** vilobhmm1 has joined #openstack-nova | 23:39 | |
| mtreinish | clarkb: https://bugs.launchpad.net/tempest/+bug/1482817 is the one which when fixed will unblock the ipv6 job | 23:40 |
| openstack | Launchpad bug 1482817 in tempest "Add config option for floating ip support" [Medium,Triaged] - Assigned to Shuquan Huang (shuquan) | 23:40 |
| mtreinish | I was hoping someone else would take care of it, but I haven't seen any patches | 23:41 |
| mtreinish | sc68cal: ^^^ seems like something you'd like :) | 23:41 |
| *** asettle has quit IRC | 23:42 | |
| *** takedakn has joined #openstack-nova | 23:43 | |
| *** SourabhP has quit IRC | 23:43 | |
| sc68cal | mtreinish: I'll check scrollback and see | 23:44 |
| sc68cal | mtreinish: I put down my drink, that's how much I believe you | 23:45 |
| *** angdraug has quit IRC | 23:45 | |
| clarkb | sc68cal: be careful its an ipv6 trap | 23:45 |
| mtreinish | sc68cal: heh, just a tempest patch to add a flag to disable doing floating ip anything | 23:45 |
| mtreinish | so it'll work in an ipv6 only env | 23:45 |
| *** vilobhmm1 has quit IRC | 23:46 | |
| sc68cal | my heart has grown 3 sizes too large, like the Grinch | 23:48 |
| mtreinish | heh | 23:49 |
| *** achanda_ has joined #openstack-nova | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!