| *** claudiub|2 has quit IRC | 00:01 | |
| *** yangyapeng has joined #openstack-nova | 00:01 | |
| *** Guest27 has joined #openstack-nova | 00:04 | |
| *** Guest27 has quit IRC | 00:05 | |
| *** yangyapeng has quit IRC | 00:06 | |
| *** salv-orlando has joined #openstack-nova | 00:06 | |
| *** salv-orlando has quit IRC | 00:11 | |
| *** stakeda has joined #openstack-nova | 00:17 | |
| *** sapd has quit IRC | 00:19 | |
| *** hshiina has joined #openstack-nova | 00:24 | |
| Spazmotic | Morning again | 00:26 |
|---|---|---|
| *** yassine has joined #openstack-nova | 00:26 | |
| *** slaweq has joined #openstack-nova | 00:28 | |
| *** edmondsw has joined #openstack-nova | 00:31 | |
| *** slaweq has quit IRC | 00:33 | |
| *** edmondsw has quit IRC | 00:35 | |
| *** slaweq has joined #openstack-nova | 00:38 | |
| *** itlinux has quit IRC | 00:39 | |
| *** BrinZhang has joined #openstack-nova | 00:39 | |
| *** slaweq has quit IRC | 00:43 | |
| *** tbachman has quit IRC | 00:44 | |
| *** slaweq has joined #openstack-nova | 00:49 | |
| *** slaweq has quit IRC | 00:54 | |
| *** hongbin has joined #openstack-nova | 00:54 | |
| *** jichen has joined #openstack-nova | 00:58 | |
| *** phuongnh has joined #openstack-nova | 00:58 | |
| *** zhaochao has joined #openstack-nova | 00:59 | |
| *** tiendc has joined #openstack-nova | 01:01 | |
| *** salv-orlando has joined #openstack-nova | 01:06 | |
| *** salv-orlando has quit IRC | 01:11 | |
| *** amodi has joined #openstack-nova | 01:15 | |
| *** liuzz has joined #openstack-nova | 01:17 | |
| *** artom has quit IRC | 01:21 | |
| *** suresh12 has joined #openstack-nova | 01:23 | |
| *** suresh12 has quit IRC | 01:28 | |
| *** pengdake has joined #openstack-nova | 01:30 | |
| *** yangyapeng has joined #openstack-nova | 01:32 | |
| *** tbachman has joined #openstack-nova | 01:34 | |
| *** pengdake has quit IRC | 01:38 | |
| *** pengdake has joined #openstack-nova | 01:38 | |
| *** amodi has quit IRC | 01:40 | |
| *** lei-zh has joined #openstack-nova | 01:40 | |
| *** amodi has joined #openstack-nova | 01:48 | |
| *** tiendc has quit IRC | 01:48 | |
| *** tiendc has joined #openstack-nova | 01:49 | |
| *** slaweq has joined #openstack-nova | 01:50 | |
| *** hiro-kobayashi has joined #openstack-nova | 01:50 | |
| *** slaweq has quit IRC | 01:54 | |
| *** vladikr has joined #openstack-nova | 01:56 | |
| *** sapd has joined #openstack-nova | 01:58 | |
| *** slaweq has joined #openstack-nova | 02:00 | |
| *** Dinesh_Bhor has joined #openstack-nova | 02:03 | |
| *** slaweq has quit IRC | 02:05 | |
| openstackgerrit | Jake Yip proposed openstack/nova master: Update deprecated log-config option in docs https://review.openstack.org/551825 | 02:07 |
| *** salv-orlando has joined #openstack-nova | 02:07 | |
| *** gcb has joined #openstack-nova | 02:08 | |
| *** hoangcx has joined #openstack-nova | 02:11 | |
| *** salv-orlando has quit IRC | 02:12 | |
| *** edmondsw has joined #openstack-nova | 02:19 | |
| *** edmondsw has quit IRC | 02:24 | |
| *** namnh has joined #openstack-nova | 02:28 | |
| *** amodi has quit IRC | 02:34 | |
| *** slaweq has joined #openstack-nova | 02:40 | |
| *** pengdake has quit IRC | 02:43 | |
| *** psachin has joined #openstack-nova | 02:44 | |
| *** amodi has joined #openstack-nova | 02:45 | |
| *** jichen has quit IRC | 02:45 | |
| *** slaweq has quit IRC | 02:45 | |
| *** jichen has joined #openstack-nova | 02:45 | |
| *** vladikr has quit IRC | 02:50 | |
| *** vladikr has joined #openstack-nova | 02:51 | |
| *** slaweq has joined #openstack-nova | 02:51 | |
| *** sapd has quit IRC | 02:55 | |
| *** sapd has joined #openstack-nova | 02:55 | |
| *** slaweq has quit IRC | 02:56 | |
| *** vladikr has quit IRC | 02:59 | |
| *** vladikr has joined #openstack-nova | 02:59 | |
| *** salv-orlando has joined #openstack-nova | 03:08 | |
| *** licanwei has joined #openstack-nova | 03:10 | |
| *** salv-orlando has quit IRC | 03:12 | |
| *** sapd1 has joined #openstack-nova | 03:15 | |
| *** psachin has quit IRC | 03:18 | |
| *** BrinZhang has quit IRC | 03:31 | |
| *** BrinZhang has joined #openstack-nova | 03:31 | |
| *** psachin has joined #openstack-nova | 03:33 | |
| *** zhangbailin_ has joined #openstack-nova | 03:34 | |
| *** BrinZhang has quit IRC | 03:38 | |
| *** dave-mccowan has quit IRC | 03:38 | |
| *** pooja_jadhav has joined #openstack-nova | 03:40 | |
| *** vladikr has quit IRC | 03:45 | |
| *** vladikr has joined #openstack-nova | 03:45 | |
| *** _pewp_ has quit IRC | 03:48 | |
| *** _pewp_ has joined #openstack-nova | 03:49 | |
| *** vivsoni has quit IRC | 03:51 | |
| *** vivsoni_ has joined #openstack-nova | 03:51 | |
| *** suresh12 has joined #openstack-nova | 04:01 | |
| *** slaweq has joined #openstack-nova | 04:02 | |
| *** hongbin has quit IRC | 04:02 | |
| *** zhurong has joined #openstack-nova | 04:06 | |
| *** suresh12 has quit IRC | 04:06 | |
| *** slaweq has quit IRC | 04:07 | |
| *** udesale has joined #openstack-nova | 04:07 | |
| *** edmondsw has joined #openstack-nova | 04:07 | |
| *** salv-orlando has joined #openstack-nova | 04:09 | |
| *** suresh12 has joined #openstack-nova | 04:09 | |
| *** amodi has quit IRC | 04:09 | |
| *** edmondsw has quit IRC | 04:12 | |
| *** slaweq has joined #openstack-nova | 04:12 | |
| *** salv-orlando has quit IRC | 04:14 | |
| *** slaweq has quit IRC | 04:17 | |
| *** stakeda has quit IRC | 04:21 | |
| *** jmlowe_ has joined #openstack-nova | 04:21 | |
| *** gjayavelu has joined #openstack-nova | 04:22 | |
| *** suresh12 has quit IRC | 04:23 | |
| *** jmlowe has quit IRC | 04:24 | |
| *** lpetrut has joined #openstack-nova | 04:24 | |
| *** jmlowe_ has quit IRC | 04:26 | |
| *** wolverineav has joined #openstack-nova | 04:27 | |
| *** wolverineav has quit IRC | 04:27 | |
| *** wolverineav has joined #openstack-nova | 04:28 | |
| *** wolverineav has joined #openstack-nova | 04:28 | |
| *** wolverineav has quit IRC | 04:30 | |
| *** wolverineav has joined #openstack-nova | 04:31 | |
| *** wolverineav has quit IRC | 04:35 | |
| *** suresh12 has joined #openstack-nova | 04:35 | |
| *** lei-zh has quit IRC | 04:35 | |
| *** slaweq has joined #openstack-nova | 04:43 | |
| *** slaweq has quit IRC | 04:48 | |
| *** suresh12 has quit IRC | 04:49 | |
| *** lpetrut has quit IRC | 04:51 | |
| *** suresh12 has joined #openstack-nova | 04:58 | |
| *** Dinesh_Bhor has quit IRC | 04:59 | |
| *** lei-zh has joined #openstack-nova | 05:04 | |
| *** suresh12 has quit IRC | 05:04 | |
| *** germs has joined #openstack-nova | 05:06 | |
| *** germs has quit IRC | 05:06 | |
| *** germs has joined #openstack-nova | 05:06 | |
| *** zhurong has quit IRC | 05:09 | |
| *** salv-orlando has joined #openstack-nova | 05:10 | |
| *** sridharg has joined #openstack-nova | 05:10 | |
| *** germs has quit IRC | 05:11 | |
| *** sridharg has quit IRC | 05:11 | |
| *** sridharg has joined #openstack-nova | 05:13 | |
| *** salv-orlando has quit IRC | 05:15 | |
| *** yangyapeng has quit IRC | 05:21 | |
| *** janki has joined #openstack-nova | 05:22 | |
| *** fragatina has quit IRC | 05:30 | |
| *** fragatina has joined #openstack-nova | 05:31 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:35 | |
| *** sidx64 has joined #openstack-nova | 05:38 | |
| *** sridhargaddam has joined #openstack-nova | 05:40 | |
| *** sridharg has quit IRC | 05:41 | |
| *** mdnadeem has joined #openstack-nova | 05:43 | |
| *** deepak_ has joined #openstack-nova | 05:44 | |
| *** yangyapeng has joined #openstack-nova | 05:45 | |
| *** claudiub|2 has joined #openstack-nova | 05:50 | |
| *** yangyapeng has quit IRC | 05:51 | |
| *** suresh12 has joined #openstack-nova | 05:51 | |
| *** sapd1 has quit IRC | 05:52 | |
| *** edmondsw has joined #openstack-nova | 05:55 | |
| *** suresh12 has quit IRC | 05:56 | |
| *** ratailor has joined #openstack-nova | 05:57 | |
| *** yamahata has quit IRC | 05:59 | |
| vivsoni_ | Hi Team, tempest.api.volume.test_volumes_extend.VolumesExtendAttachedTest.test_extend_attached_volume is failing | 06:00 |
| vivsoni_ | we already have a launchpad bug raised for it | 06:00 |
| vivsoni_ | https://bugs.launchpad.net/nova/+bug/1732199 | 06:00 |
| openstack | Launchpad bug 1732199 in OpenStack Compute (nova) "test_extend_attached_volume fails with Unexpected compute_extend_volume result 'Error'" [Medium,Confirmed] | 06:00 |
| *** david-lyle has quit IRC | 06:00 | |
| *** edmondsw has quit IRC | 06:00 | |
| *** annp has joined #openstack-nova | 06:01 | |
| *** yangyapeng has joined #openstack-nova | 06:03 | |
| *** jaosorior has joined #openstack-nova | 06:06 | |
| *** zhurong has joined #openstack-nova | 06:06 | |
| *** david-lyle has joined #openstack-nova | 06:07 | |
| *** yangyapeng has quit IRC | 06:07 | |
| *** yangyapeng has joined #openstack-nova | 06:07 | |
| *** salv-orlando has joined #openstack-nova | 06:11 | |
| *** slaweq has joined #openstack-nova | 06:15 | |
| *** salv-orlando has quit IRC | 06:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 06:18 |
| *** slaweq has quit IRC | 06:20 | |
| *** moshele has joined #openstack-nova | 06:21 | |
| *** salv-orlando has joined #openstack-nova | 06:23 | |
| *** slaweq has joined #openstack-nova | 06:25 | |
| *** udesale has quit IRC | 06:25 | |
| *** ratailor_ has joined #openstack-nova | 06:26 | |
| *** threestrands_ has joined #openstack-nova | 06:26 | |
| *** amotoki_ has joined #openstack-nova | 06:27 | |
| *** hoangcx_ has joined #openstack-nova | 06:29 | |
| *** alexchadin has joined #openstack-nova | 06:29 | |
| *** slaweq has quit IRC | 06:29 | |
| *** BlackDex_ has joined #openstack-nova | 06:30 | |
| *** andymccr_ has joined #openstack-nova | 06:33 | |
| *** lei-zh has quit IRC | 06:33 | |
| *** lei-zh has joined #openstack-nova | 06:34 | |
| *** ratailor has quit IRC | 06:34 | |
| *** hoangcx has quit IRC | 06:34 | |
| *** gcb has quit IRC | 06:34 | |
| *** threestrands has quit IRC | 06:34 | |
| *** xinliang has quit IRC | 06:34 | |
| *** NostawRm has quit IRC | 06:34 | |
| *** andymccr has quit IRC | 06:34 | |
| *** mdrabe has quit IRC | 06:34 | |
| *** BlackDex has quit IRC | 06:34 | |
| *** amotoki has quit IRC | 06:34 | |
| *** hemna has quit IRC | 06:34 | |
| *** NostawRm has joined #openstack-nova | 06:35 | |
| *** hemna has joined #openstack-nova | 06:35 | |
| *** slaweq has joined #openstack-nova | 06:35 | |
| *** mdrabe has joined #openstack-nova | 06:35 | |
| *** tuanla____ has joined #openstack-nova | 06:38 | |
| *** slaweq has quit IRC | 06:40 | |
| *** gcb has joined #openstack-nova | 06:40 | |
| *** xinliang has joined #openstack-nova | 06:40 | |
| *** sridhargaddam has quit IRC | 06:40 | |
| *** sridharg has joined #openstack-nova | 06:41 | |
| *** lajoskatona has joined #openstack-nova | 06:43 | |
| *** phuongnh has quit IRC | 06:44 | |
| *** phuongnh has joined #openstack-nova | 06:45 | |
| *** Dinesh_Bhor has quit IRC | 06:45 | |
| *** slaweq has joined #openstack-nova | 06:45 | |
| *** hoangcx_ is now known as hoangcx | 06:47 | |
| *** sar has quit IRC | 06:47 | |
| *** gjayavelu has quit IRC | 06:47 | |
| *** bauwser has quit IRC | 06:48 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:49 | |
| *** slaweq has quit IRC | 06:50 | |
| *** Shilpa has joined #openstack-nova | 06:50 | |
| *** Dinesh_Bhor has quit IRC | 06:51 | |
| *** sidx64 has quit IRC | 06:51 | |
| *** sidx64 has joined #openstack-nova | 06:53 | |
| *** tiendc has quit IRC | 06:55 | |
| *** tiendc has joined #openstack-nova | 06:55 | |
| *** slaweq has joined #openstack-nova | 06:55 | |
| *** namnh has quit IRC | 06:57 | |
| *** tuanla____ has quit IRC | 06:57 | |
| *** annp has quit IRC | 06:57 | |
| *** tuanla____ has joined #openstack-nova | 06:57 | |
| *** namnh has joined #openstack-nova | 06:57 | |
| *** annp has joined #openstack-nova | 06:57 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:59 | |
| *** slaweq has quit IRC | 07:00 | |
| *** tuanla____ has quit IRC | 07:04 | |
| *** Eran_Kuris has joined #openstack-nova | 07:05 | |
| *** slaweq has joined #openstack-nova | 07:06 | |
| *** germs has joined #openstack-nova | 07:07 | |
| *** germs has quit IRC | 07:07 | |
| *** germs has joined #openstack-nova | 07:07 | |
| *** rcernin has quit IRC | 07:09 | |
| *** slaweq has quit IRC | 07:10 | |
| *** germs has quit IRC | 07:11 | |
| *** sidx64_ has joined #openstack-nova | 07:13 | |
| *** threestrands_ has quit IRC | 07:13 | |
| *** sidx64 has quit IRC | 07:14 | |
| *** gjayavelu has joined #openstack-nova | 07:14 | |
| *** gjayavelu has quit IRC | 07:16 | |
| *** slaweq has joined #openstack-nova | 07:16 | |
| *** sidx64 has joined #openstack-nova | 07:16 | |
| *** sar has joined #openstack-nova | 07:17 | |
| *** sidx64_ has quit IRC | 07:17 | |
| *** lpetrut has joined #openstack-nova | 07:20 | |
| *** slaweq has quit IRC | 07:21 | |
| *** Dinesh_Bhor has quit IRC | 07:22 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:25 | |
| *** slaweq has joined #openstack-nova | 07:26 | |
| *** trinaths has joined #openstack-nova | 07:29 | |
| *** slaweq has quit IRC | 07:31 | |
| *** trinaths has quit IRC | 07:33 | |
| *** phuongnh has quit IRC | 07:33 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:35 | |
| *** slaweq has joined #openstack-nova | 07:36 | |
| *** jafeha__ is now known as jafeha | 07:37 | |
| *** udesale has joined #openstack-nova | 07:40 | |
| *** pasuder has joined #openstack-nova | 07:40 | |
| *** slaweq has quit IRC | 07:40 | |
| *** sahid has joined #openstack-nova | 07:42 | |
| *** phuongnh has joined #openstack-nova | 07:43 | |
| *** edmondsw has joined #openstack-nova | 07:44 | |
| *** sidx64 has quit IRC | 07:47 | |
| *** ttsiouts has joined #openstack-nova | 07:47 | |
| *** edmondsw has quit IRC | 07:48 | |
| *** slaweq has joined #openstack-nova | 07:53 | |
| *** migi has joined #openstack-nova | 07:54 | |
| *** alexchadin has quit IRC | 07:55 | |
| *** pcaruana has joined #openstack-nova | 07:56 | |
| *** slaweq__ has joined #openstack-nova | 07:56 | |
| *** migi has quit IRC | 07:57 | |
| *** Dinesh_Bhor has quit IRC | 07:57 | |
| *** migi has joined #openstack-nova | 07:58 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:59 | |
| *** slaweq__ has quit IRC | 08:01 | |
| *** migi has quit IRC | 08:02 | |
| *** alexchadin has joined #openstack-nova | 08:05 | |
| *** migi has joined #openstack-nova | 08:05 | |
| *** slaweq__ has joined #openstack-nova | 08:07 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:08 | |
| *** Dinesh_Bhor has quit IRC | 08:09 | |
| *** tesseract has joined #openstack-nova | 08:10 | |
| *** tianhui_ is now known as tianhui | 08:11 | |
| *** slaweq__ has quit IRC | 08:11 | |
| *** kaisers has quit IRC | 08:13 | |
| *** jpena|off is now known as jpena | 08:14 | |
| *** slaweq__ has joined #openstack-nova | 08:17 | |
| *** ccamacho has joined #openstack-nova | 08:18 | |
| *** ShilpaSD has joined #openstack-nova | 08:19 | |
| *** bhagyashri_s has joined #openstack-nova | 08:19 | |
| *** pooja-jadhav has joined #openstack-nova | 08:19 | |
| *** kaisers has joined #openstack-nova | 08:19 | |
| *** Shilpa has quit IRC | 08:21 | |
| *** pooja_jadhav has quit IRC | 08:21 | |
| *** bhagyashris has quit IRC | 08:21 | |
| *** slaweq__ has quit IRC | 08:22 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:23 | |
| *** slaweq__ has joined #openstack-nova | 08:27 | |
| *** ispp has joined #openstack-nova | 08:27 | |
| *** salv-orlando has quit IRC | 08:28 | |
| *** slaweq__ has quit IRC | 08:31 | |
| *** hoonetorg has quit IRC | 08:36 | |
| *** slaweq__ has joined #openstack-nova | 08:37 | |
| *** trinaths has joined #openstack-nova | 08:38 | |
| *** slaweq__ has quit IRC | 08:42 | |
| *** slaweq__ has joined #openstack-nova | 08:47 | |
| *** amoralej|off is now known as amoralej | 08:49 | |
| *** damien_r has joined #openstack-nova | 08:49 | |
| *** hoonetorg has joined #openstack-nova | 08:50 | |
| *** slaweq__ has quit IRC | 08:52 | |
| *** sidx64 has joined #openstack-nova | 08:57 | |
| *** slaweq__ has joined #openstack-nova | 08:58 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move configurable mkfs to privsep. https://review.openstack.org/551921 | 09:01 |
| *** tssurya has joined #openstack-nova | 09:01 | |
| *** hiro-kobayashi has quit IRC | 09:02 | |
| *** slaweq__ has quit IRC | 09:03 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Adds instance live-resize https://review.openstack.org/248581 | 09:03 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | 09:03 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: conductor: add live_resize task https://review.openstack.org/248579 | 09:03 |
| *** salv-orlando has joined #openstack-nova | 09:04 | |
| *** lucas-afk is now known as lucasagomes | 09:07 | |
| *** germs has joined #openstack-nova | 09:08 | |
| *** germs has quit IRC | 09:08 | |
| *** germs has joined #openstack-nova | 09:08 | |
| *** slaweq__ has joined #openstack-nova | 09:08 | |
| *** sridharg has quit IRC | 09:08 | |
| *** sridharg has joined #openstack-nova | 09:08 | |
| *** germs has quit IRC | 09:12 | |
| *** alexchadin has quit IRC | 09:13 | |
| *** slaweq__ has quit IRC | 09:13 | |
| *** slaweq__ has joined #openstack-nova | 09:18 | |
| *** hshiina has quit IRC | 09:22 | |
| *** slaweq__ has quit IRC | 09:22 | |
| openstackgerrit | do3meli proposed openstack/nova-specs master: Adds resize on shared storage without ssh keys https://review.openstack.org/551927 | 09:26 |
| *** slagle has quit IRC | 09:26 | |
| *** lyarwood is now known as lyaaaaaaaarwood | 09:27 | |
| *** finucannot is now known as stephenfin | 09:28 | |
| *** slaweq__ has joined #openstack-nova | 09:28 | |
| openstackgerrit | do3meli proposed openstack/nova-specs master: Adds resize on shared storage without ssh keys https://review.openstack.org/551927 | 09:29 |
| *** mgoddard_ has joined #openstack-nova | 09:31 | |
| *** giblet is now known as gibi | 09:31 | |
| gibi | morning | 09:32 |
| *** edmondsw has joined #openstack-nova | 09:32 | |
| *** Dinesh_Bhor has quit IRC | 09:32 | |
| *** ralonsoh has joined #openstack-nova | 09:33 | |
| *** slaweq__ has quit IRC | 09:33 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:33 | |
| *** salv-orlando has quit IRC | 09:34 | |
| *** edmondsw has quit IRC | 09:36 | |
| *** salv-orlando has joined #openstack-nova | 09:36 | |
| *** slaweq__ has joined #openstack-nova | 09:38 | |
| openstackgerrit | Alex Xu proposed openstack/nova-specs master: Report CPU features to placement service by traits API https://review.openstack.org/497733 | 09:38 |
| *** lyaaaaaaaarwood is now known as lyarwood | 09:38 | |
| *** slaweq__ has quit IRC | 09:43 | |
| *** danpawlik has joined #openstack-nova | 09:44 | |
| *** mgoddard_ has quit IRC | 09:47 | |
| *** bauzas has joined #openstack-nova | 09:47 | |
| *** sambetts_ is now known as sambetts | 09:48 | |
| *** ragiman has joined #openstack-nova | 09:48 | |
| bauzas | good morning Nova | 09:48 |
| bauzas | just spent 2 hours of investigation because had a severe network outage | 09:48 |
| bauzas | found the root cause https://twitter.com/sylvainbauza/status/973127051074031616 | 09:48 |
| *** slaweq__ has joined #openstack-nova | 09:49 | |
| *** oanson has quit IRC | 09:51 | |
| *** fragatina has quit IRC | 09:51 | |
| *** slaweq__ has quit IRC | 09:53 | |
| *** mgoddard_ has joined #openstack-nova | 09:53 | |
| *** oanson has joined #openstack-nova | 09:57 | |
| *** jichen has quit IRC | 09:57 | |
| *** slaweq__ has joined #openstack-nova | 09:59 | |
| gibi | bauzas: nice start for a Monday | 09:59 |
| bauzas | yeah, it just killed my morning :( | 09:59 |
| bauzas | in general, I always walk the dog in the mornings | 10:00 |
| bauzas | but today, I was wanting to work earlier | 10:00 |
| gibi | bauzas: but you solved a hard problem already so you were productive | 10:00 |
| bauzas | meh | 10:01 |
| bauzas | I haven't understood why the Vera Plus is saturing the switch | 10:01 |
| *** pasuder has quit IRC | 10:02 | |
| *** yamamoto has quit IRC | 10:02 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | 10:03 |
| *** udesale has quit IRC | 10:03 | |
| *** slaweq__ has quit IRC | 10:04 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | 10:04 |
| *** pasuder has joined #openstack-nova | 10:05 | |
| *** annp has quit IRC | 10:05 | |
| *** hrw has left #openstack-nova | 10:07 | |
| *** slagle has joined #openstack-nova | 10:08 | |
| *** slaweq__ has joined #openstack-nova | 10:09 | |
| *** udesale has joined #openstack-nova | 10:12 | |
| *** udesale has quit IRC | 10:13 | |
| *** udesale has joined #openstack-nova | 10:14 | |
| *** slaweq__ has quit IRC | 10:14 | |
| *** slagle has quit IRC | 10:14 | |
| *** lei-zh has quit IRC | 10:16 | |
| *** dtantsur|afk is now known as dtantsur | 10:16 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551948 | 10:16 |
| Spazmotic | Well that sounds like a gross start to the week. | 10:19 |
| *** slaweq__ has joined #openstack-nova | 10:19 | |
| *** namnh has quit IRC | 10:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Avoid exploding if guest refuses to detach a volume https://review.openstack.org/551950 | 10:21 |
| *** zhangbailin_ has quit IRC | 10:21 | |
| *** jangutter has joined #openstack-nova | 10:22 | |
| *** alexchadin has joined #openstack-nova | 10:23 | |
| Spazmotic | Hmm there's a similar event for that in the XenAPI Drivers that I should add that into | 10:24 |
| *** slaweq__ has quit IRC | 10:24 | |
| *** jangutter_ has joined #openstack-nova | 10:26 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | 10:26 |
| *** jangutter has quit IRC | 10:29 | |
| *** slaweq__ has joined #openstack-nova | 10:29 | |
| *** mgoddard_ has quit IRC | 10:29 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | 10:31 |
| *** slaweq__ has quit IRC | 10:34 | |
| *** zhurong has quit IRC | 10:37 | |
| *** phuongnh has quit IRC | 10:39 | |
| *** slaweq__ has joined #openstack-nova | 10:39 | |
| *** trinaths has quit IRC | 10:43 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Marker reset option for nova-manage map_instances https://review.openstack.org/539501 | 10:43 |
| *** slaweq__ has quit IRC | 10:44 | |
| *** sdague has joined #openstack-nova | 10:45 | |
| *** slaweq__ has joined #openstack-nova | 10:50 | |
| *** slaweq__ has quit IRC | 10:54 | |
| *** slaweq__ has joined #openstack-nova | 11:00 | |
| *** yamamoto has joined #openstack-nova | 11:03 | |
| *** slaweq__ has quit IRC | 11:04 | |
| *** pasuder has quit IRC | 11:05 | |
| *** germs has joined #openstack-nova | 11:08 | |
| *** germs has quit IRC | 11:08 | |
| *** germs has joined #openstack-nova | 11:08 | |
| *** slaweq__ has joined #openstack-nova | 11:10 | |
| *** yamamoto has quit IRC | 11:11 | |
| *** germs has quit IRC | 11:13 | |
| *** yamamoto has joined #openstack-nova | 11:14 | |
| *** slaweq__ has quit IRC | 11:15 | |
| *** pasuder has joined #openstack-nova | 11:18 | |
| *** mgoddard_ has joined #openstack-nova | 11:18 | |
| *** slaweq__ has joined #openstack-nova | 11:20 | |
| *** yikun has joined #openstack-nova | 11:22 | |
| *** slaweq__ has quit IRC | 11:25 | |
| *** BlackDex_ is now known as BlackDex | 11:30 | |
| *** slaweq__ has joined #openstack-nova | 11:30 | |
| johnthetubaguy | jaypipes: I am wondering about traits on glance images, should the flavor be allowed to deny some / give permission for some (thinking of billing being linked to flavors really) Maybe it goes back to that forbidden traits configuration idea? | 11:31 |
| Spazmotic | Glance Metadata already has restrictions built into its metadata such as minRam, MinDisk. Any reason to duplicate that? | 11:34 |
| johnthetubaguy | well it has protected properties, its just you likely want that per flavor | 11:34 |
| Spazmotic | That's true enough, they are the protected ones | 11:35 |
| johnthetubaguy | well, or rather you want a whitelist not a blacklist I think | 11:35 |
| *** slaweq__ has quit IRC | 11:35 | |
| *** lajoskatona has quit IRC | 11:35 | |
| Spazmotic | I see, I guess I could see the useful ness for that on some image types. Was just curious mostly :) | 11:36 |
| *** takedakn has joined #openstack-nova | 11:37 | |
| *** ratailor_ has quit IRC | 11:38 | |
| *** lucasagomes is now known as lucas-hungry | 11:39 | |
| *** slaweq__ has joined #openstack-nova | 11:41 | |
| *** tiendc has quit IRC | 11:41 | |
| *** danpawlik has quit IRC | 11:42 | |
| *** slaweq__ has quit IRC | 11:45 | |
| openstackgerrit | Merged openstack/nova-specs master: Return Generation from Resource Provider Creation https://review.openstack.org/548903 | 11:48 |
| *** lajoskatona has joined #openstack-nova | 11:49 | |
| *** slaweq__ has joined #openstack-nova | 11:51 | |
| *** stvnoyes1 has left #openstack-nova | 11:53 | |
| stephenfin | gibi, jaypipes: Can we jump on a Hangout sometime later this afternoon when you're both about? I'd like to hash out [1] before I write anything more for it [1] https://review.openstack.org/#/c/541290/ | 11:54 |
| gibi | stephenfin: sure, we can | 11:55 |
| gibi | stephenfin: after the scheduler meeting maybe? | 11:55 |
| *** slaweq__ has quit IRC | 11:56 | |
| *** danpawlik has joined #openstack-nova | 11:58 | |
| *** yikun has quit IRC | 11:59 | |
| *** cdent has joined #openstack-nova | 11:59 | |
| *** yikun has joined #openstack-nova | 11:59 | |
| *** slaweq__ has joined #openstack-nova | 12:01 | |
| *** tssurya has quit IRC | 12:02 | |
| *** janki has quit IRC | 12:02 | |
| *** jmlowe has joined #openstack-nova | 12:03 | |
| *** slaweq__ has quit IRC | 12:06 | |
| johnthetubaguy | stephenfin: just spotted some changes left in limbo, I am tempted to push these forward for you, but should it say rocky now? https://review.openstack.org/#/c/499179/ | 12:06 |
| *** jmlowe has quit IRC | 12:08 | |
| *** jmlowe has joined #openstack-nova | 12:10 | |
| *** slaweq__ has joined #openstack-nova | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:13 | |
| *** alexchadin has quit IRC | 12:15 | |
| openstackgerrit | Merged openstack/nova-specs master: Re-propose convert consoles code to use objects framework https://review.openstack.org/543662 | 12:15 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add request_id filed to InstanceAction versioned notifications https://review.openstack.org/551982 | 12:15 |
| *** slaweq__ has quit IRC | 12:15 | |
| *** tssurya has joined #openstack-nova | 12:15 | |
| openstackgerrit | Merged openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/540879 | 12:18 |
| *** mvk has quit IRC | 12:21 | |
| *** slaweq__ has joined #openstack-nova | 12:21 | |
| *** salv-orlando has quit IRC | 12:23 | |
| *** pasuder has quit IRC | 12:24 | |
| *** yamamoto has quit IRC | 12:26 | |
| *** slaweq__ has quit IRC | 12:26 | |
| *** Eran_Kuris has quit IRC | 12:28 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: db: Adds live-resize to Migration model migration_type https://review.openstack.org/185961 | 12:30 |
| *** slaweq__ has joined #openstack-nova | 12:32 | |
| *** efried has joined #openstack-nova | 12:32 | |
| *** salv-orlando has joined #openstack-nova | 12:33 | |
| * efried waves | 12:33 | |
| * edleafe waves back | 12:33 | |
| *** slagle has joined #openstack-nova | 12:34 | |
| *** janki has joined #openstack-nova | 12:34 | |
| efried | edleafe: Before I start filtering through a week worth of emails, anything earth-shattering I should know about from last week? | 12:35 |
| Spazmotic | johnthetubaguy, thanks sir :) | 12:36 |
| efried | This is literally the first time I've booted up my computer since the PTG | 12:36 |
| *** slaweq__ has quit IRC | 12:36 | |
| edleafe | efried: glad to hear you know how to do PTO | 12:37 |
| efried | ~800 unread emails. Sigh. | 12:37 |
| edleafe | efried: but no, nothing other than the usual | 12:37 |
| *** mvk has joined #openstack-nova | 12:37 | |
| efried | edleafe: Cool, thanks. | 12:37 |
| *** lei-zh has joined #openstack-nova | 12:39 | |
| * gibi waves to efried | 12:40 | |
| jaypipes | stephenfin: sure. after scheduler meeting works for me. | 12:40 |
| efried | Hi gibi | 12:40 |
| *** slaweq__ has joined #openstack-nova | 12:42 | |
| jaypipes | johnthetubaguy: currently, we have nothing in the os-traits library or in nova that identifies "conflicting" traits. for example, there's nothing preventing an admin from decorating a compute node with both the STORAGE_DISK_HDD and STORAGE_DISK_SSD traits at the same time. We will need some logic somewhere (not in the virt driver I would hope) to process conflicts in requested traits between the flavor and image I would think? | 12:42 |
| openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | 12:43 |
| cdent | can we call that rope? If you try it, it doesn't work, you investigate, you see why, don't do that again | 12:44 |
| *** Eran_Kuris has joined #openstack-nova | 12:44 | |
| cdent | putting _meaning_ into the traits (rather than just beign symbols) will get messy | 12:44 |
| cdent | jaypipes: ^ | 12:44 |
| efried | jaypipes, cdent: Yeah, I agree we don't want to try to get too clever - *especially* outside of virt driver - about inter-trait semantics like that. | 12:44 |
| efried | For the SSD/HDD example, if I'm not using trees, and I have both kinds of disk on my compute host, that could be a valid configuration. | 12:45 |
| cdent | welcome back efried | 12:45 |
| efried | Even if I am using trees, it's not certain I would split the local disk out from the compute node RP | 12:46 |
| efried | Thanks cdent! | 12:46 |
| efried | oh, you mean in the request | 12:46 |
| efried | So if I make a resource request that has both of those traits, in one request group, it doesn't make sense. | 12:46 |
| *** slaweq__ has quit IRC | 12:47 | |
| efried | ...and would *usually* fail in GET /a_c. Unless you've got the aforementioned host with both kinds of disk on it. | 12:47 |
| * efried stops blathering and goes back to deleting emails. | 12:48 | |
| jaypipes | efried: for sure. | 12:50 |
| *** slaweq__ has joined #openstack-nova | 12:52 | |
| *** mgoddard_ has quit IRC | 12:53 | |
| edleafe | I agree that there shouldn't be meaning in traits. If there needs to be logic added, it should be in the layer that is applying the traits, not in the trait API. | 12:54 |
| efried | What we NEED is trait metadata | 12:55 |
| efried | And aggregate metadata. | 12:55 |
| edleafe | somebody shoot that guy | 12:55 |
| *** pasuder has joined #openstack-nova | 12:56 | |
| *** slaweq__ has quit IRC | 12:56 | |
| jaypipes | heh | 12:57 |
| cdent | actually, I'd prefer trait traits | 12:58 |
| *** gcb has quit IRC | 12:59 | |
| jaypipes | cdent: lol | 13:00 |
| edleafe | that's silly. What we need are aggregates of traits | 13:01 |
| edleafe | with metadata, of course | 13:01 |
| *** slaweq__ has joined #openstack-nova | 13:02 | |
| cdent | why have aggregates when we can have trait traits traits? | 13:03 |
| *** yangyapeng has quit IRC | 13:03 | |
| cdent | (I actually do think it would be much more easy to reason about the world if we only had traits and not aggregates) | 13:04 |
| cdent | but I'm a faceted classification kind of guy | 13:04 |
| edleafe | we've been through this. There really is no difference. I prefer traits because it's shorter to type | 13:04 |
| jaypipes | buffalo buffalo buffalo buffalo buffalo buffalo buffalo | 13:05 |
| *** yangyapeng has joined #openstack-nova | 13:05 | |
| *** trozet has quit IRC | 13:05 | |
| cdent | parkay | 13:06 |
| cdent | If traits is shorter than aggregates and that's good, then _clearly_ tags are the way to go | 13:07 |
| * cdent pulls up his trouser legs | 13:07 | |
| *** READ10 has joined #openstack-nova | 13:07 | |
| *** slaweq__ has quit IRC | 13:07 | |
| arvindn05 | continue the discussion on #openstack-meeting-alt ? | 13:07 |
| jaypipes | hehe | 13:07 |
| jaypipes | arvindn05: in 55 minutes, no? | 13:08 |
| *** trozet has joined #openstack-nova | 13:08 | |
| edleafe | arvindn05: in 52 minutes, sure | 13:08 |
| edleafe | jinx | 13:08 |
| jaypipes | arvindn05: daylight savings... :) | 13:08 |
| arvindn05 | ah...DST :) | 13:08 |
| jaypipes | yup | 13:08 |
| jaypipes | gets me every time. | 13:08 |
| *** mgoddard_ has joined #openstack-nova | 13:08 | |
| *** yangyapeng has quit IRC | 13:09 | |
| arvindn05 | would have thought outlook adjusted...oh well...extra hour for me :) | 13:09 |
| arvindn05 | edleafe: cdent: latest patch set on https://review.openstack.org/#/c/541507/ addresses other concerns..can you add your review and original +1's? | 13:11 |
| edleafe | arvindn05: I have that and a bunch of others open for review today. | 13:11 |
| cdent | yup, already have that open in tab | 13:12 |
| arvindn05 | great...thanks. | 13:12 |
| *** slaweq__ has joined #openstack-nova | 13:12 | |
| *** salv-orlando has quit IRC | 13:13 | |
| arvindn05 | i think i need a workflow +1...i am guessing that should come from matt? | 13:13 |
| *** liverpooler has joined #openstack-nova | 13:14 | |
| *** dtantsur is now known as dtantsur|brb | 13:16 | |
| *** slaweq__ has quit IRC | 13:17 | |
| *** slagle has quit IRC | 13:18 | |
| *** amoralej is now known as amoralej|lunch | 13:19 | |
| *** slaweq__ has joined #openstack-nova | 13:22 | |
| *** yamamoto has joined #openstack-nova | 13:23 | |
| *** lyan has joined #openstack-nova | 13:23 | |
| *** lyan is now known as Guest18434 | 13:24 | |
| stephenfin | johnthetubaguy: They should. I'll fix them up and ping you then, if that's OK? | 13:24 |
| *** slagle has joined #openstack-nova | 13:24 | |
| Spazmotic | Welcome home, BTW, efried | 13:25 |
| efried | Spazmotic: Thanks! | 13:25 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove '[conductor] topic' opt https://review.openstack.org/499179 | 13:26 |
| stephenfin | johnthetubaguy: Actually, it was just the commit message so I stripped out that line (there's no rocky equivalent) and sent it through. Fancy hitting https://review.openstack.org/#/c/508487 though? | 13:27 |
| *** slaweq__ has quit IRC | 13:27 | |
| *** yamamoto has quit IRC | 13:28 | |
| Spazmotic | c'mon tempest-full.. don't play with me | 13:30 |
| *** awaugama has joined #openstack-nova | 13:31 | |
| Spazmotic | My achey breaky heart and all that. | 13:31 |
| *** slagle has quit IRC | 13:31 | |
| *** dave-mccowan has joined #openstack-nova | 13:31 | |
| sahid | jaypipes: we have some kind of agrements on this spec: https://review.openstack.org/#/c/539605/ if we can have you to make a first +2 | 13:32 |
| *** slaweq__ has joined #openstack-nova | 13:33 | |
| sahid | cfriesen: any chance you have a look at this https://review.openstack.org/#/c/511188/ ? | 13:33 |
| *** burt has joined #openstack-nova | 13:33 | |
| *** yangyapeng has joined #openstack-nova | 13:33 | |
| *** salv-orlando has joined #openstack-nova | 13:35 | |
| *** yangyapeng has quit IRC | 13:35 | |
| *** yangyapeng has joined #openstack-nova | 13:36 | |
| openstackgerrit | Merged openstack/nova master: Raise a proper exception in unit test https://review.openstack.org/550914 | 13:37 |
| johnthetubaguy | stephenfin: I think all the others need the same treatment? | 13:37 |
| johnthetubaguy | stephenfin: happy to hit the whole chain though, all seem like good tidy ups | 13:37 |
| *** slaweq__ has quit IRC | 13:37 | |
| *** efried has quit IRC | 13:39 | |
| *** esberglu has joined #openstack-nova | 13:40 | |
| *** READ10 has quit IRC | 13:41 | |
| *** READ10 has joined #openstack-nova | 13:41 | |
| *** slaweq__ has joined #openstack-nova | 13:43 | |
| *** gongysh has joined #openstack-nova | 13:44 | |
| ShilpaSD | stephenfin: Hi | 13:44 |
| *** jackie-truong has joined #openstack-nova | 13:45 | |
| *** gouthamr has joined #openstack-nova | 13:45 | |
| *** mriedem has joined #openstack-nova | 13:46 | |
| *** tianhui has quit IRC | 13:46 | |
| jaypipes | sahid: done | 13:46 |
| kashyap | jaypipes, johnthetubaguy: Does this require a specification? Also a config change: https://review.openstack.org/#/c/511188/ | 13:47 |
| *** tianhui has joined #openstack-nova | 13:47 | |
| kashyap | Err, wrong link | 13:47 |
| sahid | jaypipes: thanks | 13:47 |
| kashyap | jaypipes: johnthetubaguy Correct one: https://review.openstack.org/#/c/534384/ ("Allow to specify granular CPU feature flags") | 13:48 |
| *** slaweq__ has quit IRC | 13:48 | |
| *** Eran_Kuris has quit IRC | 13:48 | |
| johnthetubaguy | jaypipes: +1 on the conflict resolution, I was more thinking about operator per flavor restrictions, like allow trait overrides on a specific flavor | 13:48 |
| johnthetubaguy | kashyap: that is libvirt only, and via config, I think, so you could argue it was a specless blueprint | 13:49 |
| jaypipes | kashyap: I prefer https://review.openstack.org/#/c/497733/. | 13:49 |
| kashyap | johnthetubaguy: Right; I'll go file that spec-less Blueprint | 13:49 |
| *** r-daneel has joined #openstack-nova | 13:50 | |
| kashyap | jaypipes: Hi, this is a bit more imminent, the commit message gives the rationale: https://review.openstack.org/#/c/534384/ | 13:50 |
| kashyap | jaypipes: I see the spec you linked to is the long-term solution | 13:50 |
| kashyap | But to solve the immediate performance problems for all the users, I'd argue the above config approach is the cleanest | 13:51 |
| johnthetubaguy | not sure, there is make available to guest and require from guest so restrict to hosts that have it | 13:51 |
| johnthetubaguy | I guess its just two sides of the problem? | 13:51 |
| kashyap | johnthetubaguy: Can you expand on this: "require from guest so restrict to hosts that have it" | 13:52 |
| jaypipes | kashyap: adding yet more configuration options to nova.conf? that is specific to a single virt driver and doesn't solve the problem for the rest of them? | 13:52 |
| johnthetubaguy | an image or flavor might "require" pcid | 13:52 |
| johnthetubaguy | regardless the operator might want to provide pcid to the guest | 13:52 |
| johnthetubaguy | seems like two different things | 13:53 |
| kashyap | jaypipes: I'm with you on not sleep-walking into adding more and more config options. | 13:53 |
| kashyap | jaypipes: But ... | 13:53 |
| *** slaweq__ has joined #openstack-nova | 13:53 | |
| *** yamamoto has joined #openstack-nova | 13:53 | |
| kashyap | jaypipes: This is an exceptional scenario. The executive summary is: In light of applying the "Meltdown" CVE fixes, those guests using custom CPU models will incur 30% additional performance penalty! | 13:53 |
| kashyap | jaypipes: So, I don't take the adding a new config option approach very lightly | 13:54 |
| *** beagles is now known as beagles|brb | 13:54 | |
| kashyap | jaypipes: Yeah, there is a case to be made for that, too: allowing operators to configure _per_ flavor | 13:54 |
| kashyap | (But that's a different problem. I wanted to address this immediate isolated problem.) | 13:54 |
| jaypipes | kashyap: it's more of an image thing than a flavor thing, right? | 13:55 |
| johnthetubaguy | neither, its an operator problem here | 13:55 |
| johnthetubaguy | kashyap: you need more context in the git commit message, and less about those additional use cases | 13:55 |
| dansmith | yeah, this is a pretty important one, IMHO | 13:55 |
| johnthetubaguy | ideally we would turn it on everywhere to avoid the performance issue, but its not available everywhere, and that would break live-migrate | 13:55 |
| dansmith | and on par with other virt conf options we have | 13:55 |
| *** takedakn has quit IRC | 13:56 | |
| johnthetubaguy | dansmith: +1 | 13:56 |
| *** eharney has joined #openstack-nova | 13:56 | |
| kashyap | johnthetubaguy: I wrote additional context in my first commit message. | 13:56 |
| kashyap | johnthetubaguy: Here's all the additional context: https://review.openstack.org/#/c/534384/2//COMMIT_MSG | 13:56 |
| kashyap | (But I was asked to remove it from the commit by mdbooth.) | 13:56 |
| kashyap | Damned if I did, damned if I didn't | 13:56 |
| johnthetubaguy | kashyap: maybe somewhere in between, without the sub headings | 13:57 |
| *** yamahata has joined #openstack-nova | 13:58 | |
| *** slaweq__ has quit IRC | 13:58 | |
| kashyap | johnthetubaguy: Can modify it; I used sub headings just to make it more readable. Wonder if you think the core content is okay? | 13:58 |
| *** yamamoto has quit IRC | 13:58 | |
| stephenfin | gibi, jaypipes: Cool. Ping me whenever that is :) | 13:59 |
| stephenfin | ShilpaSD: Hey, what's up? | 13:59 |
| kashyap | dansmith: Since you also seem to be amenable to the idea, mind adding an Acked-By or some such in the review? | 14:00 |
| gibi | stephenfin: scheduler meeting starts now, I will ping you when it is over | 14:00 |
| dansmith | kashyap: I have it open, I'll look at it after I dig out from the morning | 14:00 |
| *** Eran_Kuris has joined #openstack-nova | 14:00 | |
| dansmith | johnthetubaguy: you gonna hit it first? | 14:00 |
| edleafe | Scheduler subteam meeting running now in #openstack-meeting-alt | 14:00 |
| kashyap | dansmith: Sure, take your sweet time. I need to rework the unit tests anyway. | 14:00 |
| dansmith | kashyap: it would really help if it was passing unit tests :) | 14:01 |
| kashyap | dansmith: Also, additional context (that I cut down) is here: https://review.openstack.org/#/c/534384/2//COMMIT_MSG | 14:01 |
| dansmith | hah, yeah | 14:01 |
| kashyap | dansmith: Sure, don't open it until I fix the unit tests :-) | 14:01 |
| *** pcaruana has quit IRC | 14:01 | |
| ShilpaSD | Stephenfin: hi, actually want to inform you that i am working on issue https://github.com/novnc/noVNC/issues/967, and for this updating nova\compute\manager.py for path as suugested on github issue | 14:02 |
| ShilpaSD | so is it okay to chang path in such way? or is there any way around? | 14:02 |
| *** mgoddard_ has quit IRC | 14:02 | |
| johnthetubaguy | dansmith: yeah, looking at it now | 14:03 |
| *** slaweq__ has joined #openstack-nova | 14:03 | |
| ShilpaSD | https://github.com/openstack/nova/blob/d7c46b279686308f5431410bfccb3e40031e380a/nova/compute/manager.py#L5058 | 14:03 |
| kashyap | johnthetubaguy: Many thanks. ACK / NACK on the core change in driver.py would be useful. (I'm still looking at what tests got broken & a new unit test.) | 14:04 |
| *** amoralej|lunch is now known as amoralej | 14:04 | |
| stephenfin | ShilpaSD: Yeah, I tried to reproduce that and couldn't. I only needed to make one change to the configuration. See my notes here https://review.openstack.org/#/c/483994/ | 14:05 |
| *** efried has joined #openstack-nova | 14:05 | |
| stephenfin | ShilpaSD: It's the comments from March 6th | 14:06 |
| kashyap | johnthetubaguy: But you're right that I should've retained the URL to the perf degradation analysis. Feel free to call that out, I'm fixing it locally | 14:06 |
| stephenfin | ShilpaSD: Perhaps I missed something though. Could you provide compare my steps to yours to see if this is the case? | 14:06 |
| stephenfin | johnthetubaguy: Sure, that would be great. I'll go through and fix all those up now :) | 14:07 |
| *** tianhui has quit IRC | 14:07 | |
| *** yassine has quit IRC | 14:07 | |
| *** tianhui has joined #openstack-nova | 14:07 | |
| *** slaweq__ has quit IRC | 14:08 | |
| openstackgerrit | Merged openstack/nova master: XenAPI: XCP2.1+ Swallow VDI_NOT_IN_MAP Exception https://review.openstack.org/538415 | 14:08 |
| openstackgerrit | Merged openstack/nova master: Make nova build reproducible https://review.openstack.org/551269 | 14:08 |
| ShilpaSD | stephenfin: hi, i have gone through https://review.openstack.org/#/c/483994/9/nova/conf/vnc.py | 14:09 |
| *** slagle has joined #openstack-nova | 14:09 | |
| ShilpaSD | we need to use vnc_lite.html | 14:09 |
| ShilpaSD | i have verified using vnc.html | 14:10 |
| ShilpaSD | and there we have issue | 14:10 |
| stephenfin | ShilpaSD: Oh, ok. So if you use 'vnc_lite.html', it works as expected? | 14:10 |
| stephenfin | ShilpaSD: Does 'vnc.html' work with noVNC < 1.0.0? If so, we might have a migration path to avoid people breaking when they upgrade to noVNC 1.0 | 14:11 |
| ShilpaSD | not tested with vnc_lite.html yet | 14:11 |
| ShilpaSD | i have gone through your changes, and from there i thought you verified for vnc_lite.html, wait will check with vnc_lite and get back to you. | 14:12 |
| *** slaweq__ has joined #openstack-nova | 14:13 | |
| stephenfin | ShilpaSD: So they renamed 'vnc_auto.html' to 'vnc_lite.html' in 1.0. Other than the rename, it works exactly the same as before | 14:13 |
| *** jangutter has joined #openstack-nova | 14:14 | |
| stephenfin | ShilpaSD: However, if 'vnc.html' works for both 0.6 and 1.0, we might need your fix and then we can change the default from 'vnc_auto.html' to 'vnc.html' | 14:14 |
| *** jpena is now known as jpena|lunch | 14:14 | |
| stephenfin | That means people could use either noVNC 0.6 or 1.0 with no other config changes, which would make upgrading much easier | 14:14 |
| stephenfin | Let me know what you find :) | 14:14 |
| ShilpaSD | stephenfin: okay, will test and get back to you | 14:15 |
| *** pcaruana has joined #openstack-nova | 14:15 | |
| *** jangutter_ has quit IRC | 14:17 | |
| *** slaweq__ has quit IRC | 14:18 | |
| *** esberglu_ has joined #openstack-nova | 14:19 | |
| *** esberglu has quit IRC | 14:21 | |
| *** salv-orlando has quit IRC | 14:22 | |
| *** lei-zh has quit IRC | 14:24 | |
| *** slaweq__ has joined #openstack-nova | 14:24 | |
| *** amodi has joined #openstack-nova | 14:25 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt https://review.openstack.org/499612 | 14:27 |
| sean-k-mooney | stephenfin: is novnc running under a webserver? if so adding a redirect from vnc_auto.html to vnc_lite.html should be pretty simple | 14:27 |
| *** moshele has quit IRC | 14:28 | |
| stephenfin | sean-k-mooney: That...is not a bad idea | 14:28 |
| stephenfin | Lemme have a look at how DevStack is deploying it | 14:28 |
| arvindn05 | mriedem: https://review.openstack.org/#/c/541507/ - can you take a look at the latest patchset? I think it should address your concerns | 14:28 |
| *** slaweq__ has quit IRC | 14:29 | |
| *** pcaruana has quit IRC | 14:30 | |
| *** beagles|brb is now known as beagles | 14:32 | |
| *** mlavalle has joined #openstack-nova | 14:33 | |
| *** slaweq__ has joined #openstack-nova | 14:34 | |
| *** tidwellr has joined #openstack-nova | 14:36 | |
| *** yamamoto has joined #openstack-nova | 14:38 | |
| *** slaweq__ has quit IRC | 14:39 | |
| *** sridharg has quit IRC | 14:39 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'allow_instance_snapshots' opt https://review.openstack.org/499621 | 14:40 |
| *** jangutter has quit IRC | 14:40 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | 14:41 |
| *** sridharg has joined #openstack-nova | 14:41 | |
| *** sahid has quit IRC | 14:41 | |
| cdent | stephenfin++++++++++ | 14:41 |
| * bauzas raises fist at Credit Innovation Recherche | 14:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation of database options https://review.openstack.org/443097 | 14:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | 14:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | 14:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | 14:42 |
| bauzas | heh, Credit Impot Recherche, rather | 14:43 |
| *** jackie-truong has quit IRC | 14:43 | |
| *** yamamoto has quit IRC | 14:43 | |
| *** pcaruana has joined #openstack-nova | 14:43 | |
| stephenfin | johnthetubaguy: I think I've most of those rebased and the out-of-date blueprint line removed. If you fancy pushing them through, that would be appreciated :) | 14:44 |
| *** slaweq__ has joined #openstack-nova | 14:44 | |
| *** gongysh has quit IRC | 14:44 | |
| johnthetubaguy | stephenfin: on a call, but will try hit those, do they still have some shared topic? | 14:44 |
| stephenfin | johnthetubaguy: Sure. I kept the topic the same https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/centralize-config-options-queens | 14:44 |
| edmondsw | stephenfin I had to manually rebase after you +Wd... can you take another look at https://review.openstack.org/#/c/539314/ ? | 14:45 |
| *** mgoddard_ has joined #openstack-nova | 14:46 | |
| stephenfin | edmondsw: Sure | 14:47 |
| *** esberglu_ is now known as esberglu | 14:47 | |
| efried | melwitt: The "runways" thing - is that going to be tracked on https://etherpad.openstack.org/p/rocky-nova-priorities-tracking or is there a separate etherpad? | 14:47 |
| *** hongbin has joined #openstack-nova | 14:47 | |
| edmondsw | stephenfin ty sir | 14:47 |
| * johnthetubaguy wonders about using storyboard for runways, and then runs away | 14:48 | |
| ShilpaSD | stephenfin: Hi, tested vnc console access using 'vnc_lite.html' for version 1.0, its working without any changes at nova-compute services, and able to access console | 14:48 |
| stephenfin | ShilpaSD: Excellent. So there's nothing more needed, if we decide to use that? | 14:49 |
| ShilpaSD | but if configured 'vnc.html' for version 1.0, then require changes at nova-compute service | 14:49 |
| stephenfin | What about version 0.6? | 14:49 |
| *** slaweq__ has quit IRC | 14:49 | |
| *** sar has quit IRC | 14:49 | |
| ShilpaSD | 0.6 not yet checked, will do that in next 5 min and get back to you | 14:49 |
| stephenfin | (y) | 14:49 |
| ShilpaSD | should i use v v0.6.1 or v0.6.2? | 14:50 |
| *** mdnadeem has quit IRC | 14:53 | |
| *** yamamoto has joined #openstack-nova | 14:53 | |
| *** yamamoto has quit IRC | 14:53 | |
| *** yamamoto has joined #openstack-nova | 14:53 | |
| ShilpaSD | stephenfin: actually earlier we had 'vnc_auto.html', and that removed in currunt vnc 1.0 release, so we have to make configuration chnages | 14:53 |
| *** slaweq__ has joined #openstack-nova | 14:54 | |
| stephenfin | ShilpaSD: Yup, that what's I found. What I'm suggesting is that maybe 'vnc.html' works in both 0.6 and 1.0. If so, we might start using that instead of 'vnc_auto' or 'vnc_lite' | 14:55 |
| stephenfin | Also, use 0.6.2 I guess. Either would be OK though | 14:55 |
| ShilpaSD | okay, will check on 0.6.0 with 'vn.html' and get back to you with outcome | 14:56 |
| ShilpaSD | 'vnc.html'* | 14:56 |
| *** yamamoto has quit IRC | 14:58 | |
| *** slaweq__ has quit IRC | 14:59 | |
| *** lucas-hungry is now known as lucasagomes | 15:00 | |
| jaypipes | bauzas, cdent: can you guys give me 45 minutes? | 15:01 |
| bauzas | efried: I don't see why it could be "madness" to accept operator's defined traits | 15:01 |
| cdent | jaypipes: you mean before continuing this topic? I can, sure | 15:01 |
| bauzas | the munging logic should be very explicit | 15:01 |
| jaypipes | yes | 15:01 |
| bauzas | jaypipes: yup, I can hold | 15:01 |
| * bauzas goes back to paper filling of shit | 15:01 | |
| * cdent makes coffee and snack | 15:02 | |
| gibi | jaypipes, stephenfin: so the scheduler meeting is over. I have about two hours left from my workday to jump on a hangouts about the numa aware vswitches | 15:03 |
| stephenfin | gibi: I think jaypipes is busy for a bit so maybe once he's free? I've no other meeting scheduled for the day | 15:04 |
| gibi | stephenfin: OK. I just noted down my boundaries :) | 15:04 |
| *** slaweq__ has joined #openstack-nova | 15:04 | |
| stephenfin | gibi: Sure, we'll fit it in in that time :) | 15:05 |
| *** jpena|lunch is now known as jpena | 15:09 | |
| *** jackie-truong has joined #openstack-nova | 15:09 | |
| *** slaweq__ has quit IRC | 15:09 | |
| ShilpaSD | stephenfin: Hi, for 'vnc.html' with noVnc version 0.6.2 and websockify-0.8.0, it failed to access, error at do_handshake() | 15:12 |
| stephenfin | ShilpaSD: Is that the same issue you saw with 1.0? | 15:12 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraint' functions https://review.openstack.org/385071 | 15:13 |
| *** sidx64 has quit IRC | 15:13 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move availability zones opts to a group https://review.openstack.org/462469 | 15:13 |
| *** slaweq__ has joined #openstack-nova | 15:15 | |
| *** psachin has quit IRC | 15:16 | |
| *** tbachman has quit IRC | 15:17 | |
| ShilpaSD | for 1.0, https://github.com/novnc/noVNC/issues/967 | 15:17 |
| *** gjayavelu has joined #openstack-nova | 15:17 | |
| ShilpaSD | actually issue for noVNC=v1.0.0-testing.2 | 15:17 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology https://review.openstack.org/485553 | 15:19 |
| ShilpaSD | for 1.0, token is not available as query parameter as well as it is not set in cookie. | 15:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology https://review.openstack.org/485554 | 15:19 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology https://review.openstack.org/466030 | 15:19 |
| *** slaweq__ has quit IRC | 15:19 | |
| *** Nil_ has joined #openstack-nova | 15:19 | |
| ShilpaSD | and if we configure 'vnc.html' | 15:20 |
| *** READ10 has quit IRC | 15:20 | |
| stephenfin | ShilpaSD: So they're different issues? | 15:22 |
| *** salv-orlando has joined #openstack-nova | 15:22 | |
| *** chyka has joined #openstack-nova | 15:23 | |
| *** slaweq__ has joined #openstack-nova | 15:25 | |
| *** yamamoto has joined #openstack-nova | 15:26 | |
| ShilpaSD | stephenfin: No, its same. token is not getting as query parameter as well as it is not set in cookie. | 15:26 |
| ShilpaSD | for vnc.html configuration | 15:26 |
| ShilpaSD | stephenfinr: please firstly confirm me that are we going to use vnc_lite.html or vnc_html for version 1.0? | 15:27 |
| ShilpaSD | token unavailable issue is with vnc.html only | 15:28 |
| *** salv-orlando has quit IRC | 15:28 | |
| *** hemna_ has joined #openstack-nova | 15:28 | |
| *** sahid has joined #openstack-nova | 15:29 | |
| *** zhaochao has quit IRC | 15:29 | |
| *** slaweq__ has quit IRC | 15:30 | |
| *** yikun has quit IRC | 15:30 | |
| *** felipemonteiro has joined #openstack-nova | 15:31 | |
| *** yikun has joined #openstack-nova | 15:31 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Don't use single backticks in man pages https://review.openstack.org/540887 | 15:33 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using openstackdoctheme's extlink extension https://review.openstack.org/540888 | 15:33 |
| stephenfin | ShilpaSD: So, this is my idea. We fix the issue you're seeing and change the default to 'vnc.html'. That means people can use the same configuration file with both 0.6 and 1.0 | 15:33 |
| *** yamamoto has quit IRC | 15:34 | |
| stephenfin | I don't know if that's a good idea but, in my mind, this would make for the easiest upgrade path for operators | 15:34 |
| *** eharney has quit IRC | 15:35 | |
| *** slaweq__ has joined #openstack-nova | 15:35 | |
| *** tbachman has joined #openstack-nova | 15:36 | |
| sean-k-mooney | stephenfin: since upgrades of openstack servers are supposed to be doable without any config changes from n to n+1 i think it makes sense | 15:36 |
| *** yangyapeng has quit IRC | 15:36 | |
| stephenfin | sean-k-mooney: Right. We could conceivably backport that fix too to make things even easier for the N to N+M cases | 15:37 |
| *** yangyapeng has joined #openstack-nova | 15:37 | |
| ShilpaSD | stephenfin: but earlier its 'vnc_auto.html' for 0.6, so configuration file changes are require with my fixes too | 15:37 |
| sean-k-mooney | stephenfin: well for n to n+2 it is expect that config change could be required but that should be achive via an itermediate change during n+1 | 15:38 |
| *** eharney has joined #openstack-nova | 15:38 | |
| stephenfin | ShilpaSD: True. However, operators could make that change incrementally and keep noVNC versions the same. When they do decide to update noVNC, the same config file will work with both | 15:38 |
| *** oanson has quit IRC | 15:38 | |
| stephenfin | Otherwise they need to both update the config file and noVNC version in lockstep | 15:38 |
| stephenfin | Then again, maybe I'm overthinking this and the operator could just place a redirect in place, as sean-k-mooney suggested :) | 15:39 |
| *** Eran_Kuris has quit IRC | 15:39 | |
| * stephenfin wonders what the difference between (vnc_auto|vnc_lite).html and vnc.html is | 15:39 | |
| *** slaweq__ has quit IRC | 15:39 | |
| sean-k-mooney | stephenfin: redirect for appache/nginx or simlink for python webserver "should" work | 15:40 |
| *** tbachman has quit IRC | 15:41 | |
| *** gyee has joined #openstack-nova | 15:41 | |
| *** yangyapeng has quit IRC | 15:42 | |
| *** eharney has quit IRC | 15:42 | |
| *** lpetrut has quit IRC | 15:44 | |
| *** eharney has joined #openstack-nova | 15:44 | |
| *** slaweq__ has joined #openstack-nova | 15:45 | |
| *** dtantsur|brb is now known as dtantsur | 15:45 | |
| *** tbachman has joined #openstack-nova | 15:46 | |
| sean-k-mooney | stephenfin: looking at the two i think the lite version just uses less css/javascript effects | 15:47 |
| stephenfin | sean-k-mooney: So maybe the redirect is the best option? | 15:48 |
| stephenfin | i.e. leave it to the deployment tool | 15:48 |
| *** READ10 has joined #openstack-nova | 15:48 | |
| *** jackie-truong has quit IRC | 15:49 | |
| *** claudiub|2 has quit IRC | 15:49 | |
| *** slaweq__ has quit IRC | 15:50 | |
| *** slaweq has quit IRC | 15:50 | |
| ShilpaSD | stephenfin: understood what you want to mentioned here, so for fix i am updating access URL at https://github.com/openstack/nova/blob/d7c46b279686308f5431410bfccb3e40031e380a/nova/compute/manager.py#L5058 | 15:50 |
| ShilpaSD | as below | 15:50 |
| ShilpaSD | access_url = '%s?path=websockify?token=%s' % (CONF.vnc.novncproxy_base_url, token) | 15:50 |
| *** gjayavelu has quit IRC | 15:50 | |
| *** amotoki_ is now known as amotoki | 15:50 | |
| sean-k-mooney | stephenfin: perhaps. the vnc.html seams to support audio and addtional key bindings + some transitions. | 15:50 |
| ShilpaSD | stephenfin: adding '?path=websockify', in access URL, is it okay? | 15:51 |
| sean-k-mooney | stephenfin: also i think it adds copy past support | 15:51 |
| *** fragatina has joined #openstack-nova | 15:51 | |
| ShilpaSD | its resolve my issue with 'vnc.html' | 15:51 |
| *** gcb has joined #openstack-nova | 15:51 | |
| stephenfin | ShilpaSD: If that works, push up a patch and I can review | 15:52 |
| stephenfin | ShilpaSD: Does that also work with 'vnc_lite.html'? | 15:53 |
| stephenfin | If so, we could make it the default | 15:53 |
| stephenfin | noVNC 0.6 is quite old. We could say we require it | 15:53 |
| sean-k-mooney | stephenfin: at least in devstack the version of novnc is set here https://github.com/openstack-dev/devstack/blob/master/stackrc#L606 so you could add a new config option that would set the defualt html file based on this | 15:54 |
| *** slaweq has joined #openstack-nova | 15:55 | |
| jaypipes | cdent, bauzas: hey, back now. thanks for your patience. | 15:56 |
| cdent | hola | 15:56 |
| jaypipes | stephenfin, gibi: we could discuss stephenfin's thing tomorrow my morning if that works for you | 15:56 |
| stephenfin | sean-k-mooney: I had a patch up to change the devstack default but it's failing for reason's I haven't investigated | 15:56 |
| ShilpaSD | stephenfin: yes, it works with 'vnc_lite.html' also, will upload patch for you review and get back to you on same ASAP | 15:57 |
| gibi | jaypipes: it works for me | 15:57 |
| stephenfin | jaypipes: Works for me too. Wanna stick something in someone's calendar | 15:58 |
| *** damien_r has quit IRC | 15:58 | |
| * stephenfin is eager to write code that won't be immediately discarded :) | 15:58 | |
| jaypipes | stephenfin: go for it. | 15:59 |
| jaypipes | stephenfin: jaypipes@gmail.com | 15:59 |
| gibi | stephenfin: gibizer@gmail.com | 15:59 |
| stephenfin | Ta! | 16:00 |
| *** slaweq has quit IRC | 16:00 | |
| *** pasuder has quit IRC | 16:01 | |
| cdent | jaypipes: was the stuff you wanted to talk with me about on friday eve this traits stuff, or something else? | 16:02 |
| jaypipes | cdent: yeah | 16:02 |
| ShilpaSD | stephenfin: bye, leaving for the day | 16:02 |
| stephenfin | ShilpaSD: Ciao. Let me know when you have that review up and I can review it | 16:03 |
| ShilpaSD | stephenfin: yes sure, working on TOX, once done, will do so | 16:03 |
| *** karimull1 has quit IRC | 16:04 | |
| gibi | Kevin_Zheng: I just noticed that you published a spec https://review.openstack.org/#/c/551982/ for add-request-id-to-instance-action-notifications I think the bp is already approved as specless so we don't need to have a spec | 16:05 |
| gibi | Kevin_Zheng: you can go for the implementation | 16:05 |
| *** karimull has joined #openstack-nova | 16:05 | |
| *** slaweq has joined #openstack-nova | 16:05 | |
| *** udesale_ has joined #openstack-nova | 16:06 | |
| *** udesale has quit IRC | 16:06 | |
| cdent | jaypipes: I guess bauzas is still in his pile of paperwork. I don't really have much to add to the topic than what was said on Friday and in those comments I menionted in the meetings. | 16:07 |
| bauzas | cdent: well, I can stop paperworking, y'know | 16:07 |
| jaypipes | cdent: my primary concern is how do we prevent situations where different actors are constantly overwriting each other... | 16:08 |
| bauzas | honestly, in between writing silly stuff and discussing with you folks, you got my opinion I guess | 16:08 |
| bauzas | MHO is that I agree with jaypipes and dansmith, I'd like to see the compute service reconciling traits provided by operators with traits set by the virt driver | 16:09 |
| bauzas | if that's the question | 16:09 |
| cdent | jaypipes: are we able to enumerate who the legit actors are? | 16:09 |
| dansmith | I thought we were going to have a hangout about this? | 16:09 |
| jaypipes | cdent: for both traits and allocation ratios. either we have some agreement about what is always the source of truth for certain attributes, or we never have the automated agents (like resource tracker in compute manager) ever *delete* things... only add them. and then rely on manual deletes to remove things. but then we still need to solve the case of an agent continually adding some trait or setting some allocation ratio when it's been | 16:10 |
| jaypipes | overridden or deleted by an external actor. | 16:10 |
| dansmith | IMHO, if the operator removes a trait owned by the virt driver directly in placement, the proper thing to do is overwrite that change | 16:10 |
| dansmith | the operator has configured the virt driver with details about the supported cpu model, for example, and it makes no sense for her to then go tweak placement after the fact | 16:10 |
| *** slaweq has quit IRC | 16:10 | |
| efried | Meaning the trait gets restored, yes? | 16:10 |
| jaypipes | dansmith: so the virt driver should know not to write CPU traits in that case, yes? | 16:11 |
| *** udesale_ has quit IRC | 16:11 | |
| bauzas | I was only thinking about additive traits | 16:11 |
| dansmith | jaypipes: no, the virt driver should be authoritative for cpu flag traits | 16:11 |
| efried | No, the opposite. The virt driver is the Source of Truth for CPU traits. | 16:11 |
| *** oanson has joined #openstack-nova | 16:11 | |
| efried | (and others, for that matter) | 16:11 |
| dansmith | efried: right | 16:11 |
| jaypipes | dansmith, efried: ok. fair enough about cpu traits. what about allocation ratios? | 16:11 |
| bauzas | if the operator wants to remove a trait, why shouldn't we not accepting that ? | 16:12 |
| bauzas | it's his responsibility | 16:12 |
| dansmith | jaypipes: today, compute owns those, but I think who owns it in the future depends on what we do with our current config | 16:12 |
| jaypipes | bauzas: dansmith and efried are saying that the virt driver should own the cpu traits. and I don't disagree with that, BTW. | 16:12 |
| bauzas | the compute service *is* the source of truth, not the virt driver IMHO | 16:12 |
| *** pcaruana has quit IRC | 16:12 | |
| bauzas | that's where I see the difference | 16:13 |
| dansmith | bauzas: compute does not own cpu flag traits | 16:13 |
| bauzas | meaning that some operator could want to hide a trait if they'd like | 16:13 |
| jaypipes | dansmith: right, and aggregate stuff complicates that a bit, too. | 16:13 |
| dansmith | bauzas: it does the setting based on what virt says, but it doesn't own the decision about which ones to set | 16:13 |
| efried | ++ | 16:14 |
| bauzas | dansmith: the problem I see with that approach is that we have different traits | 16:14 |
| dansmith | jaypipes: yup, but the ratio thing is pretty straightforward I think.. we just decide what we're going to do at first startup, and what to do on subsequent ones | 16:14 |
| bauzas | dansmith: some that we don't want the operator to modify, some we accept that | 16:14 |
| jaypipes | dansmith: ack. and I'm cool with your default_xxx_allocation_ratio suggestion. | 16:14 |
| bauzas | anyway, it's a set, right | 16:14 |
| bauzas | ? | 16:14 |
| dansmith | bauzas: I don't understand what you're saying | 16:14 |
| bauzas | so, it's either an additive or a removal operation for a trait | 16:15 |
| jaypipes | dansmith: do you think the default_xxx_allocation_ratio solution warrants a spec? | 16:15 |
| dansmith | jaypipes: I bet mriedem does | 16:15 |
| jaypipes | yeah, makes sense | 16:15 |
| jaypipes | I'll hack one together quick. | 16:15 |
| jaypipes | scoped only to this narrow use case. | 16:16 |
| dansmith | maybe it would help if I wrote some example code for how I think we should handle the trait thing? | 16:16 |
| *** slaweq has joined #openstack-nova | 16:16 | |
| bauzas | dansmith: the point I'm saying is that if we don't accept operators to remove a trait for a specific usecase (here a CPU feature), we shouldn't accept that for *any* trait too | 16:16 |
| dansmith | between virt traits, compute traits, and operator-set traits? | 16:16 |
| dansmith | bauzas: that makes no sense to me | 16:16 |
| jaypipes | dansmith: sure, that would help | 16:16 |
| efried | It seems to be a requirement for the admin to be able to set/unset at least *certain* traits. And I personally don't want to get into a situation where we treat some traits differently from others. Having to figure out and enforce which is which == nightmare. So we need the admin-override mechanism to be generic. And if they want to remove something that clearly should be owned by the virt driver (like a CPU trait), th | 16:16 |
| jaypipes | (the sample code) | 16:16 |
| dansmith | bauzas: virt can be authoritative for TRAIT_VMX and not be involved in TRAIT_SOMETHINGELSE | 16:16 |
| bauzas | dansmith: so the compute would know which resource is authoritative for a trait, then ? | 16:17 |
| jaypipes | efried: did you cut off that last sentence? | 16:17 |
| efried | dansmith: IMO that's crazypants, to try to make lists of traits that can or can't be controlled by the admin, or are "owned" by virt vs. not, etc. | 16:17 |
| dansmith | bauzas: does't need to be by any set of complex rules, IMHO | 16:17 |
| dansmith | efried: I don't think we need a set of rules | 16:17 |
| dansmith | efried: I think the virt driver and compute services can just be authoritative for the ones they need to care about, and any others will be left alone | 16:18 |
| edleafe | efried: if the virt driver sets traits A,B,C, and the admin sets traits X,Y,Z, there is no problem, right? It's only when the admin wants to set/clear one of A, B, or C | 16:18 |
| bauzas | dansmith: I mean, how the compute can know whether it can accept an operator's defined list of traits to remove if one of those traits is TRAIT_VMX for example ? | 16:18 |
| jaypipes | efried: dansmith is saying "whatever the virt driver returns as traits should be authoritative for *those* traits" | 16:18 |
| dansmith | jaypipes: exactly | 16:18 |
| efried | I don't see how that can work in practice. | 16:18 |
| edleafe | jaypipes: yes | 16:18 |
| efried | Thinking about how the virt driver is going to discover traits... | 16:18 |
| efried | In the general case, it gets its info from the platform and translates to trait strings. | 16:19 |
| dansmith | efried: the virt driver is write-only for traits, right? it doesn't care what traits are currently set, it only generates a list of traits that *should* be set based on talking to the hypervisor | 16:19 |
| efried | Yes, but we're talking about the virt driver having to *know* about things it simply doesn't get from the platform as still being under its control. | 16:19 |
| dansmith | I'm not sure I understand what you mean, but.. I don't think I agree | 16:20 |
| bauzas | dansmith: so, if we accept some mechanism for the operator to remove some trait, it would just be a conditional in the compute service that would say 'don't touch what the virt driver returns", right? | 16:20 |
| jaypipes | efried: but dan isn't suggesting that the virt driver overwrite the complete set of traits, only that it will ensure that some traits are added to the provider | 16:20 |
| efried | Especially if some new CPU trait (or whatever) comes into existence that the platform didn't know about, but is still expected to be the authority on. | 16:20 |
| bauzas | can I backup for a second ? | 16:20 |
| *** slaweq has quit IRC | 16:20 | |
| jaypipes | efried: and if the admin added such a trait, the virt driver wouldn't remove it. | 16:20 |
| bauzas | after all, do we need to provide some mechanism for an operator to *remove* a trait from what the compute service will report ? | 16:21 |
| dansmith | efried: are you concerned about the operator adding that new cpu trait to the compute node thinking it will magically make it work? | 16:21 |
| dansmith | because that's not a problem we need to solve, IMHO | 16:21 |
| bauzas | if we only allow the operator to give a list of human-defined traits, I don't see the problem | 16:21 |
| efried | dansmith: No | 16:21 |
| jaypipes | bauzas: I believe what dansmith is saying is "no, we don't need such functionality". | 16:21 |
| bauzas | jaypipes: then, I agree | 16:21 |
| cdent | just to repeat my concern, so it's on the table, before I forget: The thing that I wonder being a problem is if the virtdriver reports too many traits. Is that ever going to be an issue? | 16:21 |
| bauzas | adding a possibility to remove a trait opens a lot of problems | 16:22 |
| efried | dansmith: But traits don't always equate to CPU features. | 16:22 |
| bauzas | (17:11:22) bauzas: I was only thinking about additive traits | 16:22 |
| jaypipes | cdent: I don't think so, no. | 16:22 |
| bauzas | defined by the operator | 16:22 |
| jaypipes | efried: yes, traits aren't always CPU features. and nobody is saying that the only thing the virt driver would return is CPU features. | 16:23 |
| efried | It sounds like dansmith is simply advocating that, whatever mechanism we provide for admins to add or remove traits, they can't remove traits that the virt driver sets. (Or rather, they'll be restored the next time u_p_t runs.) And I'm on board with that. | 16:23 |
| dansmith | efried: yes, that's what I'm saying | 16:23 |
| dansmith | but I think that's all we need | 16:23 |
| jaypipes | efried: all that is being stipulated is that the virt driver will return a set of traits that it feels should always be set on the provider. it will not remove any traits manually added by an admin. | 16:23 |
| dansmith | the only other interface we need for admins is placement | 16:23 |
| bauzas | +1 to the idea of keeping things somple | 16:24 |
| bauzas | simple | 16:24 |
| efried | dansmith: Via osc, yah? | 16:24 |
| efried | that's sufficient? | 16:24 |
| *** salv-orlando has joined #openstack-nova | 16:24 | |
| dansmith | efried: yeah | 16:24 |
| dansmith | totally sufficient, IMHO | 16:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec for volume multiattach enhancements https://review.openstack.org/552078 | 16:25 |
| jaypipes | OK, I will attempt to summarize here. | 16:25 |
| bauzas | dansmith: but tbc, we agree on if the admin stupidely deletes a trait in the placement API, then on the next virt driver call, the compute service will add again that trait | 16:25 |
| bauzas | I now understand what you're saying | 16:25 |
| efried | Okay, I'm on board now. I think this works. The stipulation that traits owned by the virt driver will magically reappear in some indeterminate amount of time - that's going to be a little weird to describe. But if that's the only wrinkle, I can accept it. | 16:25 |
| jaypipes | bauzas: if it's a trait the virt driver reports, yep. | 16:25 |
| bauzas | jaypipes: that slight detail makes me a little wondering | 16:25 |
| efried | Yeah | 16:26 |
| *** slaweq has joined #openstack-nova | 16:26 | |
| efried | It's going to make the admin ask the question, "Okay, how do I find out which traits I'm not allowed to remove?" | 16:26 |
| bauzas | I don't see a reason why we shouldn't consider the compute service as the single source of truth for any compute-defined trait | 16:26 |
| *** lajoskatona has quit IRC | 16:26 | |
| dansmith | efried: I'm less worried about that, | 16:26 |
| dansmith | efried: because they're going to be adding some trait because a doc told them to, like "add a trait to this if it is a trusted PF" | 16:27 |
| jaypipes | efried: it's a valid concern but I'm not particularly worried about that either. | 16:27 |
| * efried zeroes the counter of bugs opened by ops with titles like "Deleted trait reappears" | 16:27 | |
| efried | but sure. | 16:27 |
| dansmith | that is easy to document, | 16:27 |
| bauzas | dansmith: adding a trait is totally fine by me | 16:27 |
| efried | dansmith: I agree it should be documented. I don't agree it's easy to document. | 16:27 |
| bauzas | dansmith: that's the removal case that makes me wondering why we should special case the virt ones | 16:27 |
| dansmith | the "what new stuff can I add" is a little harder, but really not a huge deal I think | 16:27 |
| efried | But we'll burn that bridge when we cross it. | 16:27 |
| dansmith | I'm writing a really simple example bit of code to explain what we just discussed I think | 16:28 |
| jaypipes | bauzas: if "special casing" == "the virt driver understands these traits should be applied to this provider", then I don't agree it's an issue. | 16:28 |
| *** salv-orlando has quit IRC | 16:29 | |
| efried | but wait | 16:29 |
| bauzas | jaypipes: no | 16:30 |
| *** tssurya has quit IRC | 16:30 | |
| bauzas | jaypipes: the special case I mentioned would be "the compute manager would see a removed trait, and only add it again to Placement API if that's coming from the virt driver" | 16:30 |
| bauzas | that's the "if..." I wonder | 16:31 |
| *** felipemonteiro has quit IRC | 16:31 | |
| *** slaweq has quit IRC | 16:31 | |
| *** gcb has quit IRC | 16:31 | |
| efried | We still have the case where u_p_t is called, and the virt driver decides that it wants to assert a set of traits, but something has changed since the last time, and that set of traits is a *subset* of its previous. Then it has no way to know that it should remove the extraneous traits. | 16:31 |
| *** felipemonteiro has joined #openstack-nova | 16:31 | |
| bauzas | for me, the compute service should provide all the traits that are reported from various ways, and shouldn't care whether it's not existing in Placement or not | 16:31 |
| jaypipes | efried: what we're saying is that it should never "remove the extraneous traits". | 16:31 |
| bauzas | ++ | 16:32 |
| efried | jaypipes: And that's a problem. | 16:32 |
| dansmith | efried: wait for my example code | 16:32 |
| cdent | jaypipes: that's difficult in the cluster virt drivers | 16:32 |
| efried | Because something may have changed that legitimately warrants the removal of such a trait. | 16:32 |
| cdent | where hardware is added on the fly, or hot pluggable hardware | 16:32 |
| efried | Otherwise why would we be bothering to do this in a periodic? | 16:32 |
| *** yamamoto has joined #openstack-nova | 16:32 | |
| efried | cdent: Yes, or *removed* | 16:32 |
| cdent | yes | 16:32 |
| *** damien_r has joined #openstack-nova | 16:33 | |
| efried | So I go back to asserting this doesn't work for the general case. | 16:33 |
| jaypipes | efried: "the general case" <-- what is that? | 16:34 |
| efried | jaypipes: I just mean "for all cases". | 16:34 |
| edleafe | If an admin wants to "remove" a trait FOO set by the virt driver, she could add a CUSTOM_FOO_BAD trait, and then make that a forbidden trait | 16:34 |
| efried | edleafe: I think we're past that one. It doesn't have a realistic use case, most likely. | 16:35 |
| edleafe | efried: good | 16:35 |
| *** slaweq has joined #openstack-nova | 16:36 | |
| *** yangyapeng has joined #openstack-nova | 16:37 | |
| efried | dansmith, jaypipes: Specifically the case I'm talking about is, for example, hot-pluggable storage. Let's say you start with some disk. Virt driver presents inventory of DISK_GB and trait STORAGE_DISK_SSD. Then you pull out the disk. The next u_p_t needs to be able to remove the DISK_GB inventory *and* remove the STORAGE_DISK_SSD trait. With this design, it can't do the latter. The admin would have to do it manually. | 16:37 |
| jaypipes | efried: it's not that I don't see your use case (for example, dynamically marking some child providers representing a PF with a CUSTOM_TRUSTED trait or something). But I also don't necessarily think that what dansmith is proposing would *preclude* us from moving in a more complicated direction if and when such a direction was determined to be required. | 16:37 |
| *** yamamoto has quit IRC | 16:38 | |
| dansmith | this is what I think we need: https://pastebin.com/mM6NMzQ4 | 16:39 |
| *** ragiman has quit IRC | 16:39 | |
| dansmith | note the operator's traits are untouched, yet the virt and compute-owned traits are asserted or de-asserted as things change | 16:40 |
| *** slaweq has quit IRC | 16:41 | |
| cdent | oh that's interesting | 16:41 |
| *** yangyapeng has quit IRC | 16:41 | |
| efried | dansmith: What I'm asserting is that the virt driver has no way of knowing it wants to -SOMETHING | 16:41 |
| cdent | but can/does that interop with how u_p_t is planned to behave | 16:42 |
| *** chyka has quit IRC | 16:42 | |
| *** ygl has joined #openstack-nova | 16:42 | |
| ygl | hi all | 16:42 |
| dansmith | efried: it totally does | 16:42 |
| *** chyka has joined #openstack-nova | 16:42 | |
| efried | cdent: What we decided on DublinFriday is that we're going to merge u_p_t as currently written and then bolt this on after. | 16:42 |
| *** jpena is now known as jpena|brb | 16:42 | |
| jaypipes | dansmith: that's essentially what my traits:always and traits:ignore in the provider config file format were doing. :) | 16:42 |
| *** tssurya has joined #openstack-nova | 16:42 | |
| dansmith | efried: I assert we don't have to worry about cases where the operator adds a trait to an x86 compute node of TRAIT_CPU_POWER7_THINGY | 16:43 |
| ygl | i want to launch a vm with a dummy nic interface without an IP assigned to it . can someone help me how to do it please | 16:43 |
| jaypipes | dansmith: but clearly I don't segregate between what the compute and the virt think of as asserted/removed | 16:43 |
| efried | ygl: Try #openstack | 16:43 |
| dansmith | jaypipes: this is not always and ignore, this is always and remove, or something | 16:43 |
| jaypipes | dansmith: understood. | 16:43 |
| dansmith | jaypipes: and this is exposed from the virt driver, not encoded in a separate editable file | 16:44 |
| jaypipes | dansmith: would this be exposed as a config option (or 2 options)? | 16:44 |
| dansmith | jaypipes: no | 16:44 |
| efried | dansmith: I'm not talking about the admin adding traits - we already decided she gets to do that. I'm talking about a trait the virt driver needs to remove because e.g. someone hot-unplugged a disk. | 16:45 |
| dansmith | jaypipes: the virt driver would return things like this, which may vary based on virt driver config and talking to the hypevisor | 16:45 |
| efried | The virt driver operates by discovering what it has and asserting inventory/traits accordingly. | 16:45 |
| jaypipes | k | 16:45 |
| efried | If it discovers at time T and has a disk, it will assert some DISK_GB inventory and the STORAGE_DISK_SSD trait. | 16:45 |
| jaypipes | dansmith: a perfectly satisfactory solution. | 16:46 |
| dansmith | jaypipes: this is what I was describing from my corner on friday while we were discussing this | 16:46 |
| *** slaweq has joined #openstack-nova | 16:46 | |
| efried | Then it discovers at time T+1 and it doesn't have a disk; it will not assert DISK_GB inventory (which works, because it's the only source of inventory info) and it will not assert the STORAGE_DISK_SSD trait. | 16:46 |
| efried | But it *doesn't* know that it needs to *de*assert the STORAGE_DISK_SSD trait. | 16:46 |
| efried | Because it has no memory that it asserted it at time T. It just knows it isn't there now. | 16:47 |
| *** chyka has quit IRC | 16:47 | |
| dansmith | efried: I completely don't understand that | 16:47 |
| jaypipes | dansmith: yes, I recognize the +/- syntax now. just didn't have in my mind how the virt driver would expose this (automatically vs. config-based) | 16:47 |
| dansmith | jaypipes: ack | 16:47 |
| dansmith | efried: if you as a virt driver ever expose trait FOO, then you need to de-assert FOO if you determine there is no reason to assert it, right? | 16:47 |
| efried | That goes exactly back to my earlier statement about how the virt driver would have to know the full set of traits it *might* be responsible for, and then what subset of those it thinks should be set, so that it can explicitly deassert the remainder. | 16:49 |
| dansmith | why is that unreasonable? | 16:49 |
| dansmith | it doesn't have to be the full set of traits that any virt driver might expose, | 16:49 |
| dansmith | just the ones _it_ might expose | 16:49 |
| dansmith | we could make a turbo simple data structure that they all use, which requires them to declare any traits they may want to use, then assert the ones it finds, and the rest will be de-asserted | 16:50 |
| dansmith | to avoid leaking some asserted ones that are never de-asserted | 16:50 |
| dansmith | I must be missing why this is a hard problem | 16:50 |
| bauzas | dansmith: so the virt driver would provide all the possible CPU traits, either with a "+" or a "-" prefix, depending on what it knows to support ? | 16:50 |
| dansmith | bauzas: all the cpu traits it knows about yeah | 16:50 |
| bauzas | that would work then | 16:51 |
| bauzas | okay, I see your idea | 16:51 |
| dansmith | bauzas: it doesn't need to worry about traits for PPC processors, for example | 16:51 |
| *** fragatina has quit IRC | 16:51 | |
| *** slaweq has quit IRC | 16:51 | |
| dansmith | if the operator added one of those then, sucks for that operator | 16:51 |
| bauzas | yeah, but it would still report "negative" traits | 16:51 |
| dansmith | yeah | 16:51 |
| bauzas | cool with me then | 16:51 |
| dansmith | so if you changed the cpu_model later, it would sync up with all the flags that are now legit | 16:52 |
| bauzas | based on the specific compute version it runs | 16:52 |
| ygl | bauzas: can you help me with my issue please if you can | 16:52 |
| dansmith | sure, but as time goes forward, the cpu flag set only grows | 16:52 |
| bauzas | that's a reasonable assumption | 16:52 |
| dansmith | and for non-cpu flag things, | 16:52 |
| ygl | bauzas: i want to launch a vm with a dummy nic interface without an IP assigned to it | 16:53 |
| dansmith | it becomes a compatibility thing kinda like our db schema, | 16:53 |
| bauzas | and honestly, who cares if the operator sets a trait that the virt driver doesn't know ? | 16:53 |
| dansmith | where you need to not just add/remove traits willy-nilly over releases without doing some cleanup | 16:53 |
| dansmith | ygl: this channel is for development, see topic please | 16:53 |
| bauzas | the operator would suck less if they would use a custom trait for that | 16:53 |
| bauzas | yeah, I like that idea actually | 16:53 |
| ygl | dansmith: i tried other channels but they r not responding | 16:53 |
| bauzas | because we explicitly then provide what the virt driver knows | 16:53 |
| dansmith | bauzas: yup, we could have an audit mode where the compute node logs any traits it didn't calculate, which should ideally be just the ones the operator set | 16:53 |
| *** tssurya has quit IRC | 16:54 | |
| dansmith | so any that got leaked over time would be easy to identify | 16:54 |
| *** chyka has joined #openstack-nova | 16:56 | |
| *** chyka has quit IRC | 16:56 | |
| ygl | bauzas: sorry to disturb you. can you help me with my issue, if you can | 16:56 |
| *** slaweq has joined #openstack-nova | 16:56 | |
| *** chyka has joined #openstack-nova | 16:57 | |
| efried | Note that the ironic virt driver is gleaning traits from inspector. So that interface will need to change: either to support +/-; or to add a separate "get all the traits I might care about" call. | 16:57 |
| *** oanson has quit IRC | 16:57 | |
| *** awaugama has quit IRC | 16:57 | |
| dansmith | efried: the interface from the ironic driver to ironic? | 16:57 |
| efried | yeah | 16:57 |
| sean-k-mooney | dansmith: the cpu traits would be reported on the CPU resouce providers not the compute node correct nulless the cpu inventory is under the compute node. just thinking of the numa case where the inventory would be per numa node | 16:57 |
| dansmith | efried: depends on the traits I guess, and what is going on right now.. if the ironic driver is exposing traits that don't need to be deleted later, then it won't really matter | 16:58 |
| dansmith | sean-k-mooney: yeah, that doesn't change this though | 16:58 |
| sean-k-mooney | dansmith: yep just checking. i would expect most operator traits to be applied to the compute node RP but they may want to tag sub resouce providres also. | 16:59 |
| dansmith | sean-k-mooney: yeah, network-related traits will need to be below compute node | 17:00 |
| dansmith | efried: ah, looks like we're just proxying any trait they want? | 17:00 |
| efried | yup. | 17:00 |
| *** admin__ has joined #openstack-nova | 17:00 | |
| dansmith | efried: yeah, well, that kinda sucks, but it's resolvable I think | 17:00 |
| efried | Assuming we go forward with this, all the merge logic needs to be handled by the individual virt drivers. Unless we're going to invent some interface outside of u_p_t. | 17:00 |
| efried | except for get_traits, which I freakin knew was going to bite us in the ass. | 17:00 |
| dansmith | I guess we could also provide the current set of traits to the virt driver so it gets to decide if it thinks any of the ones set are important to ack or nak | 17:01 |
| *** slaweq has quit IRC | 17:01 | |
| *** salv-orlando has joined #openstack-nova | 17:01 | |
| efried | dansmith: That's what we do. | 17:01 |
| efried | in u_p_t | 17:01 |
| dansmith | efried: I don't think the virt drivers should be doing merging in the general case | 17:02 |
| *** lpetrut has joined #openstack-nova | 17:02 | |
| dansmith | efried: just being write-only for the majority of things, this ironic thing notwithstanding | 17:02 |
| efried | dansmith: Then we need nontrivial redesign of u_p_t. | 17:02 |
| dansmith | well I dunno what to say dude.. this is clearly a thing we missed in the design :) | 17:02 |
| efried | The way it's currently designed: The resource tracker builds the ProviderTree, which includes all the providers' traits as known by placement, and passes that to u_p_t. The virt driver mungs the ProviderTree as it sees fit and then returns. Then resource tracker flushes those changes back to placement. | 17:03 |
| *** moshele has joined #openstack-nova | 17:03 | |
| dansmith | yeah, well, we can provide merging routines to the virt drivers to use on the tree, | 17:03 |
| *** yamahata has quit IRC | 17:03 | |
| dansmith | and expect that most just use those | 17:04 |
| efried | Sure we can. But the merging still has to be the responsibility of the virt driver. | 17:04 |
| sean-k-mooney | efried: perhaps but it would be nicer if it was the respociblity of the compute manager or some other componet just above the virt driver so it could be shared acrosss all virtdirvers | 17:05 |
| dansmith | sean-k-mooney: yeah, that's what I'd like, to expose some augmented topology from the virt driver so the compute can update the tree, | 17:05 |
| efried | sean-k-mooney: And I'm saying that would require some semantic that's not compatible with u_p_t as designed. | 17:05 |
| efried | The ProviderTree business was so we didn't have to have separate virt driver methods for get_inventory, get_traits, get_aggregates, and (this is the main one) somehow_structure_the_provider_tree_hierarchy | 17:05 |
| dansmith | but it sounds like we've already exposed this all the way down | 17:05 |
| *** damien_r has quit IRC | 17:06 | |
| bauzas | efried: dansmith's proposal patch is simple | 17:06 |
| dansmith | so, whatever, we can argue about cleaning that up later if we want, unrelated to fixing this merging thing | 17:06 |
| efried | dansmith: The code isn't merged. We could still rip it all out and change our minds. But I believe we'd be getting a lot more complicated. | 17:06 |
| admin__ | mridem: http://logs.openstack.org/24/550324/2/gate/legacy-tempest-dsvm-neutron-full/211271c/job-output.txt.gz#_2018-03-12_16_02_09_694703 failed | 17:06 |
| bauzas | it doesn't require to modify what we currently have | 17:06 |
| bauzas | it's just a convention | 17:06 |
| dansmith | efried: oh, I thought you meant they're already getting the tree | 17:06 |
| *** slaweq has joined #openstack-nova | 17:07 | |
| *** admin__ has quit IRC | 17:07 | |
| dansmith | I dunno, exposing that all the way down isn't my preference, but if that's what we've agreed on so far, I wouldn't hold it up because I don't like it | 17:07 |
| bauzas | like, in a nested RP world, I'd just see the tree with the root RP having a set of traits, each of those be ether prefixed by plus or minus | 17:07 |
| bauzas | or, say a NUMA node | 17:07 |
| dansmith | we just need to integrate this wrinkle at least | 17:07 |
| *** vivsoni has joined #openstack-nova | 17:07 | |
| dansmith | bauzas: it's more complicated than that | 17:07 |
| efried | dansmith: The code was written in Q, but the series didn't merge in time. So we wrote and approved the spec for R (it didn't have a bp/spec in Q - was just kinda folded into the NRP work). The series starts here: https://review.openstack.org/#/c/537648/ | 17:07 |
| dansmith | we need a full topology back from the virt driver with traits at every level It hink | 17:08 |
| dansmith | ack | 17:08 |
| bauzas | mmm | 17:08 |
| bauzas | IIRC, the proposal is the virt driver that passes a tree that has RPS with traits attached, right? | 17:08 |
| efried | dansmith: Yes, the full topology. Does that topology include +/- prefixes? In which case it's not a ProviderTree but something different (ProviderTreeDelta?) | 17:08 |
| *** damien_r has joined #openstack-nova | 17:08 | |
| bauzas | a trait is attached to a RP, right? | 17:09 |
| dansmith | efried: in the model you have proposed, | 17:09 |
| sean-k-mooney | dansmith: you may not have traits at all levels but there is noting preventing that either. it just depends on the resouces but in all likely hood most RPs will have traits | 17:09 |
| efried | dansmith: Otherwise, like I say, we have to have granular methods that return deltas for traits, inventories, aggregates, and (somehow) the tree structure itself. | 17:09 |
| bauzas | the fact that the RP is a child or a root RP, is just a detail | 17:09 |
| dansmith | efried: we just need a provider.merge_traits(my_assertions) method that takes the +/- things right? | 17:09 |
| bauzas | yeah | 17:09 |
| bauzas | that's a compute method | 17:10 |
| jaypipes | efried: we would want to change the ProviderTree.set_traits() stuff to use the merge traits +/- approach | 17:10 |
| bauzas | no need to change the virt driver interface | 17:10 |
| efried | dansmith: Yes. And virt drivers have to use it. Which is okay with me. | 17:10 |
| dansmith | efried: where provider would be any element in the tree | 17:10 |
| jaypipes | doh, jinx | 17:10 |
| dansmith | efried: yeah, I think that's fine | 17:10 |
| bauzas | the merge_traits() thing is a compute manager method, right? | 17:10 |
| efried | dansmith: But to clarify, it still means the virt driver is responsible for merging traits. | 17:10 |
| dansmith | bauzas: no | 17:10 |
| jaypipes | bauzas: no. ProviderTree | 17:10 |
| efried | ^ | 17:10 |
| bauzas | meh to that | 17:11 |
| dansmith | efried: understood, and I don't like that as much, but I'm willing to defer that argument until later :) | 17:11 |
| *** felipemonteiro_ has joined #openstack-nova | 17:11 | |
| bauzas | I mean, the caller would be the compute manager ? | 17:11 |
| dansmith | efried: so I still don't think this is a major disruption, just a change in protocol a little bit | 17:11 |
| dansmith | bauzas: in some cases | 17:11 |
| *** slaweq has quit IRC | 17:11 | |
| dansmith | bauzas: but not the ones we're talking about here | 17:11 |
| *** lucasagomes is now known as lucas-afk | 17:11 | |
| bauzas | I'm missing some cases then | 17:11 |
| efried | dansmith: Not really even that. The only thing we would need to change about what's proposed is the u_p_t docstring. | 17:11 |
| bauzas | but I don't want to rathole | 17:11 |
| efried | Adding the ProviderTree.merge_traits method is a nice-to-have, but until then, u_p_t could still do that manually. | 17:12 |
| bauzas | oh, unrelated, I'll take a couple of days off | 17:12 |
| dansmith | efried: um, not sure about that | 17:12 |
| *** Swami has joined #openstack-nova | 17:12 | |
| jaypipes | bauzas: technically, the resource tracker is the caller of update_provider_tree() on a virt driver, but the goal is to have update_provider_tree() API be what other agents (including Neutron) use for managing their hierarchy of providers, inventories, and traits. | 17:12 |
| dansmith | efried: but get some code up so we can argue over that, which will be more clear right? | 17:12 |
| bauzas | jaypipes: dansmith: cdent: mriedem: others: I'll be on PTO thursday to monday included | 17:12 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Fix N358 hacking check https://review.openstack.org/547670 | 17:12 |
| * dansmith has a call in a few minutes | 17:12 | |
| jaypipes | bauzas: ack | 17:12 |
| bauzas | disneyland... | 17:12 |
| bauzas | not for the dog this time | 17:13 |
| efried | dansmith: I'm rebasing the u_p_t series now. I'll throw a comment in the appropriate spot. And then at some point I guess I can propose the ProviderTree.merge_traits method. | 17:13 |
| dansmith | okay | 17:13 |
| bauzas | jaypipes: ack, yeah, makes sense | 17:13 |
| * efried 's cup runneth over | 17:13 | |
| bauzas | it's 6:13pm here, my stomatch starts to tell me it's time to hang off | 17:14 |
| bauzas | but I'll be back tonight | 17:14 |
| bauzas | \o | 17:14 |
| jaypipes | ciao | 17:14 |
| *** felipemonteiro has quit IRC | 17:14 | |
| * bauzas has a couple of specs to review tonight | 17:14 | |
| *** edmondsw has quit IRC | 17:15 | |
| *** edmondsw has joined #openstack-nova | 17:15 | |
| *** felipemonteiro_ has quit IRC | 17:17 | |
| *** slaweq has joined #openstack-nova | 17:17 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:17 | |
| *** moshele has quit IRC | 17:17 | |
| *** pcaruana has joined #openstack-nova | 17:18 | |
| *** edmondsw has quit IRC | 17:19 | |
| *** slaweq has quit IRC | 17:20 | |
| *** slaweq has joined #openstack-nova | 17:21 | |
| *** AlexeyAbashkin has quit IRC | 17:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | 17:23 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 17:23 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 17:23 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 17:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 17:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324 | 17:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | 17:24 |
| *** jpena|brb is now known as jpena | 17:24 | |
| *** ygl has quit IRC | 17:24 | |
| efried | ^ rebase. The first patch should still be good to go regardless. The next one still needs some work. | 17:26 |
| *** pcaruana has quit IRC | 17:27 | |
| mriedem | is someone going to summarize all of the 'merge traits' discussion/decisions/etc from today into the ML per the retrospective at the ptg? | 17:28 |
| *** fragatina has joined #openstack-nova | 17:28 | |
| *** sridharg has quit IRC | 17:29 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move resource class fields https://review.openstack.org/540049 | 17:29 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move resource provider objects into placement hierarchy https://review.openstack.org/551528 | 17:29 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Reparent placement objects to oslo_versionedobjects https://review.openstack.org/551529 | 17:29 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 17:29 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 17:29 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 17:29 |
| *** damien_r has quit IRC | 17:30 | |
| *** sahid has quit IRC | 17:30 | |
| efried | mriedem: Well, I was planning to propose an update to the update-provider-tree spec. | 17:30 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates https://review.openstack.org/552098 | 17:31 |
| edleafe | dansmith: ^^ | 17:31 |
| edleafe | dansmith: Still have to do tests, but I wanted jaypipes to review the SQL join code | 17:31 |
| efried | mriedem: Once I've put that up, I can write something up for the ML. | 17:31 |
| *** eharney has quit IRC | 17:33 | |
| *** oanson has joined #openstack-nova | 17:35 | |
| *** slaweq__ has joined #openstack-nova | 17:37 | |
| *** slaweq__ has quit IRC | 17:37 | |
| *** yamahata has joined #openstack-nova | 17:43 | |
| *** wolverineav has joined #openstack-nova | 17:43 | |
| *** imacdonn has quit IRC | 17:43 | |
| *** yamahata has quit IRC | 17:44 | |
| *** imacdonn has joined #openstack-nova | 17:44 | |
| *** yamahata_ has joined #openstack-nova | 17:44 | |
| *** oanson has quit IRC | 17:44 | |
| *** chyka has quit IRC | 17:44 | |
| *** oanson_ has joined #openstack-nova | 17:44 | |
| *** chyka has joined #openstack-nova | 17:45 | |
| *** oanson_ is now known as oanson | 17:46 | |
| efried | dansmith: Are we going to have this same issue for aggregates? | 17:47 |
| *** deepak_ has quit IRC | 17:47 | |
| efried | Rephrase: We're going to have this same issue for aggregates. | 17:47 |
| efried | I hesitate to wonder if we've got the same issue for inventories. | 17:47 |
| efried | Golly, yes, allocation_ratio... | 17:47 |
| *** slaweq__ has joined #openstack-nova | 17:48 | |
| *** slaweq__ has quit IRC | 17:48 | |
| sean-k-mooney | efried: same issue? | 17:48 |
| *** slaweq has quit IRC | 17:49 | |
| *** itlinux has joined #openstack-nova | 17:49 | |
| *** chyka has quit IRC | 17:49 | |
| efried | sean-k-mooney: Yeah; does the operator get to make changes to a provider's aggregates? | 17:49 |
| *** edmondsw has joined #openstack-nova | 17:49 | |
| efried | Since we're mirroring host aggregates, the answer is clearly yes; though those are going to be funneled through the compute service somehow. | 17:50 |
| sean-k-mooney | efried: they get to create new ones. not sure they are allowed to modify existing ones | 17:50 |
| efried | sean-k-mooney: Then we have exactly the same problem - how does the virt driver know it gets to remove an aggregate association? | 17:50 |
| sean-k-mooney | it cant unless it created it | 17:51 |
| *** suresh12 has joined #openstack-nova | 17:51 | |
| *** slaweq has joined #openstack-nova | 17:51 | |
| sean-k-mooney | other service can create aggregates also not just the operator | 17:51 |
| *** slaweq__ has joined #openstack-nova | 17:51 | |
| *** slaweq___ has joined #openstack-nova | 17:52 | |
| arvindn05 | mriedem: https://review.openstack.org/#/c/541507/ - can you take a look at the latest patchset? I think it should address your concerns | 17:52 |
| *** r-daneel has quit IRC | 17:52 | |
| *** pcaruana has joined #openstack-nova | 17:53 | |
| *** slaweq__ has quit IRC | 17:54 | |
| *** slaweq__ has joined #openstack-nova | 17:54 | |
| *** slaweq has quit IRC | 17:56 | |
| *** vivsoni has quit IRC | 17:57 | |
| *** slaweq__ has quit IRC | 17:58 | |
| *** felipemonteiro__ has joined #openstack-nova | 17:58 | |
| *** deepak_ has joined #openstack-nova | 17:59 | |
| *** chyka has joined #openstack-nova | 17:59 | |
| *** RaoulHC has joined #openstack-nova | 18:00 | |
| *** dave-mccowan has quit IRC | 18:00 | |
| *** dtantsur is now known as dtantsur|afk | 18:01 | |
| *** felipemonteiro_ has quit IRC | 18:02 | |
| *** dave-mccowan has joined #openstack-nova | 18:04 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Skip placement on rebuild in same host https://review.openstack.org/546357 | 18:04 |
| *** slaweq has joined #openstack-nova | 18:05 | |
| *** slaweq has quit IRC | 18:06 | |
| *** tesseract has quit IRC | 18:06 | |
| efried | I don't suppose there's a way to permalink an etherpad line | 18:06 |
| *** suresh12 has quit IRC | 18:07 | |
| *** slaweq has joined #openstack-nova | 18:07 | |
| *** slaweq has quit IRC | 18:07 | |
| sean-k-mooney | efried: i dont think the lines have css id so no | 18:07 |
| efried | sean-k-mooney: Or even permalink the etherpad as a whole at a point-in-time. | 18:08 |
| *** suresh12 has joined #openstack-nova | 18:08 | |
| edmondsw | efried I wouldn't think so | 18:08 |
| sean-k-mooney | efried: you could use http://archive.org/web/ | 18:08 |
| efried | mmm, nice | 18:08 |
| sean-k-mooney | efried: make a snap shot of the doc then link to the snap shot and give the line number | 18:09 |
| *** sambetts is now known as sambetts|afk | 18:09 | |
| efried | sean-k-mooney: Yeah, I'll pastebin just the relevant section. | 18:09 |
| efried | I guess. | 18:09 |
| sean-k-mooney | efried: you can create readonly share links in etherpad like this one https://etherpad.openstack.org/p/r.2bb9e3053658b6ddf5e73d31045dcfba | 18:11 |
| *** Swami has quit IRC | 18:11 | |
| *** Swami_ has joined #openstack-nova | 18:11 | |
| *** Swami_ has quit IRC | 18:11 | |
| *** Swami has joined #openstack-nova | 18:12 | |
| efried | sean-k-mooney: That doesn't actually freeze it. Just makes it so you can't edit. | 18:13 |
| efried | sean-k-mooney: Changes still show up. | 18:13 |
| *** slaweq has joined #openstack-nova | 18:13 | |
| sean-k-mooney | efried: ah ok ya maybe with the version specified https://etherpad.openstack.org/p/r.2bb9e3053658b6ddf5e73d31045dcfba/timeslider#43302 | 18:14 |
| sean-k-mooney | still pastbin might be cleaner | 18:14 |
| efried | aha, #timeslider is probably what I want. | 18:14 |
| *** slaweq has quit IRC | 18:14 | |
| efried | 'cept no line numbers! | 18:14 |
| efried | gaaahhh! | 18:14 |
| *** gjayavelu has joined #openstack-nova | 18:15 | |
| *** felipemonteiro__ has quit IRC | 18:15 | |
| *** yangyapeng has joined #openstack-nova | 18:16 | |
| *** felipemonteiro__ has joined #openstack-nova | 18:16 | |
| sean-k-mooney | efried: ya all the etherpads are version controled but yes the etherpad line number are added with javascipt renderer and are not part of the document | 18:16 |
| *** gouthamr has quit IRC | 18:16 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support default allocation ratios https://review.openstack.org/552105 | 18:18 |
| jaypipes | dansmith: ^ | 18:18 |
| -openstackstatus- NOTICE: Most jobs in zuul are currently failing due to a recent change to zuul; we are evaluating the issue and will follow up with a recommendation shortly. For the moment, please do not recheck. | 18:18 | |
| *** ChanServ changes topic to "Most jobs in zuul are currently failing due to a recent change to zuul; we are evaluating the issue and will follow up with a recommendation shortly. For the moment, please do not recheck." | 18:18 | |
| *** ralonsoh has quit IRC | 18:20 | |
| dansmith | efried: I don't see the same problem with aggregates | 18:20 |
| dansmith | efried: and aggregate changes wouldn't go through the compute service | 18:20 |
| *** gouthamr has joined #openstack-nova | 18:21 | |
| dansmith | efried: adds or removes would get mirrored to placement,m | 18:21 |
| *** yangyapeng has quit IRC | 18:21 | |
| dansmith | but if the operator changed them in placement, we wouldn't necessarily un-do that with a sync based on what jaypipes has proposed as I understand it | 18:21 |
| *** slaweq has joined #openstack-nova | 18:22 | |
| *** eharney has joined #openstack-nova | 18:22 | |
| *** slaweq has quit IRC | 18:23 | |
| efried | dansmith: I think, as with the "driver capabilities" traits, the mirrored host aggregates will be a special case controlled by compute manager (!virt). But I mean in general. If an admin adds/removes an aggregate in placement directly, how does virt know not to remove/restore that aggregate association? | 18:23 |
| sean-k-mooney | dansmith: i thikn efried main concern was how does nova know if it is allowed to remove a RP it created from an aggregate. my answer would be unless it created the aggregate it should not add or remove RPs from it | 18:23 |
| *** janki has quit IRC | 18:23 | |
| *** slaweq has joined #openstack-nova | 18:24 | |
| *** slaweq has quit IRC | 18:24 | |
| dansmith | efried: the compute manager can't know what aggregate it's in | 18:24 |
| dansmith | efried: it can't even access the database that has those details, nor talk to conductors that can | 18:25 |
| *** amoralej is now known as amoralej|off | 18:25 | |
| efried | okay, sorry, "...special case controlled by conductor". But the point about the general case remains. | 18:25 |
| dansmith | efried: I'm missing something | 18:26 |
| efried | dansmith: I.e. do we need to implement ProviderTree.merge_aggregates ? | 18:26 |
| dansmith | efried: at steady state, nova wouldn't add or remove providers from aggregates | 18:26 |
| sean-k-mooney | efried: in the general case nova cant assume it own RPs or Aggregate in placement | 18:26 |
| dansmith | sean-k-mooney: agree | 18:26 |
| efried | sean-k-mooney: Disagree. Sigh. | 18:26 |
| sean-k-mooney | efried: well the simple case would be that neutron may want to add compute nodes to a aggregate to model there availablity zones which do not have to map direclty to novas | 18:27 |
| dansmith | efried: can you state that disagreement as a full statement? like, you disagree that... | 18:27 |
| sean-k-mooney | same for cinder | 18:27 |
| dansmith | sean-k-mooney: yep | 18:27 |
| dansmith | sean-k-mooney: or the operator may do so for any other reason | 18:27 |
| *** moshele has joined #openstack-nova | 18:28 | |
| *** mgoddard_ has quit IRC | 18:28 | |
| dansmith | efried: I just wonder which part you disagree with | 18:28 |
| sean-k-mooney | dansmith: yes for failure domains suchs as power or ha failover or other usecse that openstck can discover | 18:28 |
| dansmith | sean-k-mooney: yup | 18:28 |
| sean-k-mooney | s/ can discover/ can't discover/ | 18:29 |
| *** RaoulHC has quit IRC | 18:29 | |
| efried | The virt driver knows of a certain set of aggregates of which its providers (the ones it "owns") are members. What it does not necessarily know about is the set of aggregates of which its providers are *not* members. Today, as designed, u_p_t is supposed to just overwrite aggregate associations. So there's no affordance for outside agents to manipulate them. | 18:29 |
| *** dtruong has joined #openstack-nova | 18:30 | |
| efried | The question is: do we need such affordance? At least for the host agg mirroring, it sounds like it. | 18:30 |
| dansmith | efried: ah are you talking about an aggregate around a compute node and its children, for example? | 18:30 |
| dansmith | efried: not equal to nova's host aggregates? | 18:30 |
| sean-k-mooney | efried: yes we do for the neutron/cinder case and for failure domains. the virt driver should simple add/remove aggregates it owns but not overrite all aggregates for a RP | 18:31 |
| dansmith | right | 18:31 |
| efried | dansmith: Could be intra-tree; could also be inter-tree (e.g. a shared storage pool). The argument applies to the host agg mirrors as well as whatever else we want to do with aggs. | 18:31 |
| efried | I.e. if the virt driver is not the single SoT for agg associations of the RPs it "owns", we have to have the exact same discussion as we just did with traits. | 18:32 |
| sean-k-mooney | efried: for a shared storage pool nova does not really own that RP cinder likely does | 18:32 |
| efried | sean-k-mooney: Sometimes. PowerVM SSPs will be (co-)owned by the virt driver(s). | 18:32 |
| dansmith | efried: well, I agree the virt driver can draw its own aggregates around things and manage those itself, and maybe tell compute that it should join an aggregate for some shared storage it's connected to, | 18:32 |
| dansmith | but nothing more authoritative than that | 18:32 |
| efried | dansmith: In which case we need to amend the u_p_t charter for aggregates in the same way we just did for traits. Because as currently designed, virt just overwrites the aggregates. | 18:34 |
| dansmith | efried: okay I'm not sure how that was ever considered a reasonable thing to do, but.. yes, agree it needs to not do that :) | 18:34 |
| efried | And we have the same quandary as to which aggregates the virt driver "owns", so it knows which ones to de-assert. | 18:34 |
| *** lbragstad has quit IRC | 18:35 | |
| sean-k-mooney | efried: this feels like a case of placement is used almost entirely by nova today and we need to make it support other services. | 18:35 |
| efried | And I contend that that's a much stickier problem than the traits one. Because we don't have a static list of agg UUIDs that virt "might" use. | 18:35 |
| sean-k-mooney | efried: that said we should have hit this already with routed networks in neutron | 18:35 |
| dansmith | efried: okay, I guess I'm surprised that aggregate membership is part of provider tree, but fair eough | 18:35 |
| *** jackie-truong has joined #openstack-nova | 18:35 | |
| efried | sean-k-mooney: We're not talking about changing anything on the placement side. Just nova's consumption thereof. | 18:35 |
| dansmith | efried: right, if it's doing that, then it's going to need some persistence to keep track of which ones it has created | 18:36 |
| dansmith | sean-k-mooney: right his point is that we need to not be so aggressive in nova, | 18:36 |
| dansmith | er, efried : | 18:36 |
| dansmith | because we can't trample what other users would have done, human or otherwise | 18:36 |
| sean-k-mooney | efried: yes but neutron is modeling routed netwoks in placement today using aggreagtes to express what node can connet to routed subnets correct? | 18:36 |
| efried | I don't know that answer. (But probably not "today" if you mean "code that's merged".) | 18:37 |
| dansmith | it doesn't matter, | 18:37 |
| dansmith | that's a thing that will happen | 18:37 |
| dansmith | neutron, cinder, et al | 18:38 |
| *** awaugama has joined #openstack-nova | 18:38 | |
| sean-k-mooney | efried: well neutorn has been tracking routed network in placement since pike i think | 18:38 |
| efried | sean-k-mooney: By creating placement aggregates around compute node resource providers? | 18:38 |
| sean-k-mooney | efried: i think so im currently checking | 18:39 |
| dansmith | it doesn't matter, that's a thing we must support | 18:39 |
| jaypipes | efried: how about just relaxing the aggregate overwrite thing to instead just add aggregates (and potentially remove aggregates that the virt driver is 100% sure it owns fully)? | 18:39 |
| *** moshele has quit IRC | 18:40 | |
| efried | jaypipes: Which is exactly what we're doing for traits. The difference is that knowing that second thing is way harder for aggs. | 18:40 |
| *** tssurya has joined #openstack-nova | 18:40 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Rocky_Release_Schedule" | 18:41 | |
| -openstackstatus- NOTICE: Zuul has been restarted without the breaking change; please recheck any changes which failed tests with the error "Accessing files from outside the working dir ... is prohibited." | 18:41 | |
| efried | jaypipes: Kinda one step harder than the trait thing will be for the ironic case... | 18:41 |
| jaypipes | efried: if the virt driver created the agg, then it knows that, no? | 18:42 |
| efried | jaypipes: No. | 18:42 |
| efried | jaypipes: u_p_t has no memory. | 18:42 |
| jaypipes | efried: right, but the virt driver could persist that data somewhere itself.. | 18:42 |
| efried | jaypipes: Would have to be a file; else this doesn't work across compute restarts. | 18:42 |
| efried | Are we in the practice of letting virt drivers write arbitrary files?? | 18:43 |
| jaypipes | efried: IMHO, the virt drivers can do whatever they want w.r.t. persisting local information. | 18:43 |
| efried | eek | 18:44 |
| jaypipes | efried: hell, that's pretty much what the libvirt XML files are ;) | 18:44 |
| cdent | except write allocations! ) | 18:44 |
| dansmith | well, | 18:44 |
| * cdent runs away | 18:44 | |
| jaypipes | cdent: well... good point :) | 18:44 |
| *** felipemonteiro_ has joined #openstack-nova | 18:44 | |
| dansmith | anything we do in that regard has to *really* be sure that it supports upgrade properly, | 18:44 |
| dansmith | and we don't have a lot of tooling around that | 18:44 |
| dansmith | so I would be hesitant to approve any new things around that without a plan | 18:44 |
| dansmith | right now, all the data we store is managed by nova and tested (to the limit it's possible) by our upgrade tooling | 18:45 |
| jaypipes | dansmith: agreed. but, if powervm wants to use etcd running on a mainframe in a datacenter in Cambodia, I honestly couldn't care less. :) | 18:45 |
| sean-k-mooney | efried: this covers the aggregate usecase for neutron reouted networks. https://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/neutron-routed-networks.html#proposed-change | 18:45 |
| dansmith | jaypipes: well, I care, because I care about users' experience with nova being positive when possible | 18:45 |
| sean-k-mooney | efried: the first paragrph directly covers the use of aggregated to model network segments | 18:45 |
| dansmith | jaypipes: which is why I care about virt drivers merging random crap that affects that view | 18:45 |
| jaypipes | dansmith: I, of course, *would* care about the libvirt driver doing such a thing. | 18:45 |
| dansmith | jaypipes: it's hard to explain why we hold libvirt to a different standard once we let things like that into other drivers, but anyway.. | 18:46 |
| jaypipes | fair enough. | 18:46 |
| sean-k-mooney | jaypipes: well libvirt xmls are generated at runtime so they are not really persiting state | 18:47 |
| *** felipemonteiro__ has quit IRC | 18:48 | |
| sean-k-mooney | jaypipes: we can recompute them form info in the nova db if they were deleted | 18:48 |
| dansmith | sean-k-mooney: their compatibility is also managed by libvirt | 18:48 |
| *** claudiub|2 has joined #openstack-nova | 18:49 | |
| sean-k-mooney | dansmith: yes that too. we did discuss brefily that we could add a table to the nova db for virt dirvers to store instance specific info. provided we used ovo or something similar to version the data they stored for upgrades | 18:49 |
| dansmith | yep, that would be better from an upgrade point of view | 18:50 |
| *** harlowja has joined #openstack-nova | 18:50 | |
| sean-k-mooney | in this case we would have to simple presetit a list of virt_driver owned aggregats or something along those lines? | 18:50 |
| sean-k-mooney | personally i dislike storing json blobs in the db but if its somting we will never query against and is just used as a keyvalue store for the virt dirver then as long as its versioned in some way i personally dont care as much | 18:52 |
| dansmith | that's pretty specific to this case | 18:52 |
| dansmith | I'm not opposed to virt drivers storing things locally, I just think we need a plan to avoid letting it get out of hand and breaking during an upgrade | 18:52 |
| jaypipes | dansmith: I don't disagree with you. I was being a bit tongue in cheek earlier about the datacenter in Cambodia. | 18:54 |
| *** lbragstad has joined #openstack-nova | 18:55 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:56 | |
| *** sar has joined #openstack-nova | 18:59 | |
| *** AlexeyAbashkin has quit IRC | 19:00 | |
| *** moshele has joined #openstack-nova | 19:01 | |
| *** openstackgerrit has quit IRC | 19:04 | |
| *** openstackgerrit has joined #openstack-nova | 19:05 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Support default allocation ratios https://review.openstack.org/552105 | 19:05 |
| cfriesen | dansmith: as per discussion at PTG: https://bugs.launchpad.net/nova/+bug/1754782 I won't have any time to work on it till next week at the earliest. | 19:06 |
| openstack | Launchpad bug 1754782 in OpenStack Compute (nova) "we skip critical scheduler filters when forcing the host on instance boot" [Undecided,New] | 19:07 |
| *** salv-orlando has quit IRC | 19:08 | |
| *** moshele has quit IRC | 19:11 | |
| *** jpena is now known as jpena|off | 19:12 | |
| melwitt | jaypipes: thanks for writing up the PTG summary for placement, good stuff | 19:13 |
| jaypipes | melwitt: no prob. thanks to mriedem and gibi who double-checked stuff on it. | 19:14 |
| melwitt | mriedem++ gibi++ | 19:14 |
| sean-k-mooney | QQ just working on https://bugs.launchpad.net/nova/+bug/1747496 did we remove the old default config values from MTU when not set on the port from nova? i think we did but just setting | 19:16 |
| openstack | Launchpad bug 1747496 in OpenStack Compute (nova) "MTUs are not set for VIFs if using kernel ovs + hybrid plug = false" [Medium,Confirmed] - Assigned to sean mooney (sean-k-mooney) | 19:16 |
| sean-k-mooney | *checking | 19:16 |
| *** suresh12 has quit IRC | 19:17 | |
| *** suresh12 has joined #openstack-nova | 19:17 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Allow for merging traits https://review.openstack.org/552122 | 19:20 |
| efried | dansmith, jaypipes, cdent, sean-k-mooney, edleafe: ^ | 19:20 |
| sean-k-mooney | hum it does appare to be in https://github.com/openstack/nova/blob/master/nova/conf/network.py or in .../neutron.py so i guess they are gone and were move to os-vif https://github.com/openstack/os-vif/search?utf8=%E2%9C%93&q=network_device_mtu&type= | 19:20 |
| efried | I didn't make any changes for aggregates. | 19:20 |
| cdent | efried: thanks, in the queue | 19:21 |
| *** maciejjozefczyk has quit IRC | 19:21 | |
| sean-k-mooney | efried: im just heading home but i addem myself to the review and ill leave it open in a tab for tomorow. ill give it a quick skim before i leave however | 19:23 |
| dansmith | efried: looks okay to me | 19:24 |
| *** gouthamr has quit IRC | 19:25 | |
| sean-k-mooney | efried: why did you start with "Traits are special. Rather than overwriting the entire set of traits" this statement also applies to RPs and Aggregates | 19:25 |
| efried | sean-k-mooney: Because I wrote this before it occurred to me that we should be doing the same for the other bits. | 19:26 |
| sean-k-mooney | efried: the compute node RP is not owned by nova so nova cant override all child RPs of the compute node either | 19:26 |
| sean-k-mooney | efried: ah ok | 19:26 |
| efried | sean-k-mooney: But I also wanted to do this one isolated because I don't think we're done discussing the others. | 19:26 |
| *** suresh12 has quit IRC | 19:27 | |
| efried | The compute node RP *is* owned by nova. | 19:27 |
| efried | sean-k-mooney: And *some* of its children may also be. | 19:27 |
| dansmith | yeah, the compute node RP is _definitely_ owned by nova :) | 19:27 |
| efried | In general virt needs to be smart enough not to muck with stuff it doesn't own. But originally we *thought* that meant virt owns everything about the RPs it owns. | 19:28 |
| sean-k-mooney | efried: that has issues wich other services want to tag it with traits | 19:28 |
| dansmith | agreed nova can't blow away all the providers underneath, unless it's deleting the compute node | 19:28 |
| *** gouthamr has joined #openstack-nova | 19:28 | |
| sean-k-mooney | dansmith: well event then i dont think it can | 19:28 |
| efried | Which is why we already have ProviderTree methods to add/remove providers in the tree. | 19:28 |
| efried | ...individually. | 19:28 |
| dansmith | sean-k-mooney: it has to be able to delete it and the subtree, IMHO | 19:28 |
| sean-k-mooney | dansmith: if i have a converged deployment where my compute nodes are also cinder storage pool providers the the root node of the tree would be a parent of the nova resources and the cinder resouces | 19:29 |
| efried | Agree. E.g. if neutron created bandwidth RPs under a PF, and compute removes the PF, the bandwidth providers are no longer relevant. | 19:29 |
| sean-k-mooney | which is why i dont think the root node can be owned by nova | 19:30 |
| efried | sean-k-mooney: I don't think that's a model we should support. | 19:30 |
| dansmith | sean-k-mooney: in that case the cinder providers are not servicing other computes, right? then it's okay to delete it | 19:30 |
| dansmith | sean-k-mooney: if they are servicing others, then the cinder pool should not be a child of the compute node | 19:30 |
| sean-k-mooney | dansmith: no the could be and associated with other computes via a sharing aggregate | 19:30 |
| dansmith | sean-k-mooney: if they are, they should not be a child of the compute IMHO | 19:30 |
| jaypipes | efried: I agree with both edleafe and dansmith on that update to the u-p-t spec | 19:31 |
| dansmith | sean-k-mooney: the compute node RP is not the physical computer, it's the nova service and the hypervisor/virt driver underneath | 19:31 |
| sean-k-mooney | dansmith: perhaps i had always just envisioned that the server itself was a resouce provider all services could create childeren under without having to worry about it beeing deleted by e.g. nova | 19:32 |
| *** READ10 has quit IRC | 19:32 | |
| dansmith | sean-k-mooney: not, IMHO.. if we need to model that relationship then the compute node RP would be a child to "the computer" I think | 19:32 |
| jaypipes | dansmith: well, in the case of a hypervisor host, yes. :) for ironic, of course, that's different. | 19:32 |
| dansmith | but I hope we don't need to do that | 19:32 |
| sean-k-mooney | dansmith: yes well that has some advantages | 19:32 |
| dansmith | jaypipes: yeah I know, but thats special | 19:32 |
| jaypipes | ack | 19:33 |
| dansmith | jaypipes: ironic nodes wouldn't be cinder providers in the same hierarchy, and even better, the computer running the nova service for those, if it were a cinder pool shared with other nodes, should be modeled peer to the compute service's RP I think | 19:34 |
| sean-k-mooney | dansmith: the only issue i have really with saying that nova owns the root RP is that we may need to have multiple root nodes for the same server depending on the service | 19:34 |
| dansmith | sean-k-mooney: no, nova owns the compute RP.. if you decide to make that the root of your cinder provider, then it is a root :) | 19:34 |
| dansmith | and you get what's coming to you in that case :) | 19:34 |
| dansmith | which is nova may delete the root that it owns | 19:35 |
| sean-k-mooney | dansmith: yes but the we have too trees for the same phyical server and we dont currently have a way to model that they are the same server | 19:35 |
| sean-k-mooney | i guess maybe with an aggregate | 19:35 |
| dansmith | sean-k-mooney: they're not the same server, | 19:35 |
| sean-k-mooney | dansmith: why not? | 19:36 |
| dansmith | sean-k-mooney: they represent services that happen to be on the same box.. if we need to model that they're on the same box (i.e. below a parent provider) then that's a thing | 19:36 |
| efried | dansmith, jaypipes: Please confirm: you want two separate methods, with *traits args | 19:36 |
| dansmith | efried: that's fine | 19:36 |
| jaypipes | efried: yes please | 19:36 |
| efried | ight | 19:36 |
| jaypipes | add_traits() and remove_traits() would be my preference. | 19:36 |
| jaypipes | efried: ^ | 19:36 |
| dansmith | efried: btw, excellent job not having your head explode here. kudos :) | 19:36 |
| efried | jaypipes: Cool, swhere I was gonna go. dansmith: ack, thx; head may yet explode over aggregates thing, but so far so good. | 19:37 |
| dansmith | efried: on friday of PTG, I thought maybe your eyeballs were about to shoot from your head at lethal velocity, which scared me when they were pointed at me | 19:37 |
| sean-k-mooney | dansmith: ok anyway thats a little off topic. i do think we may want to have a parent of the compute node RP at some point then but let cross that bridge only when we need too | 19:37 |
| dansmith | sean-k-mooney: we may and that would solve the problem of this ownership of the tree for deletes problem, that's what I'm saying :) | 19:38 |
| dansmith | until that point, if you create a provider underneath someone else's provider, I think you need to know that you're at the other's mercy | 19:38 |
| efried | dansmith: It ain't my eyeballs that should worry you :P FWIW, I'm still uncomfortable with this, but I don't see a better alternative. | 19:38 |
| dansmith | um. | 19:38 |
| * dansmith thinks about other body parts | 19:38 | |
| * efried 's whole body is a lethal weapon | 19:39 | |
| * dansmith calls HR to report a threat | 19:39 | |
| * efried chokes out HR too | 19:39 | |
| dansmith | oof | 19:39 |
| sean-k-mooney | dansmith: oh yes it would solve that issue. it would raise the question of which tree i shoudl create the resouce other for thinks like bandwith. anyway i better run before the stores close at 8 | 19:40 |
| dansmith | yeah | 19:41 |
| * sean-k-mooney rereads my last sentence and think my dyslxia is getting worse. littrally types different words /other/under /thinks/things then i taught in my head... | 19:43 | |
| *** suresh12 has joined #openstack-nova | 19:47 | |
| *** slaweq has joined #openstack-nova | 19:47 | |
| *** slaweq has quit IRC | 19:49 | |
| *** slaweq has joined #openstack-nova | 19:50 | |
| *** r-daneel has joined #openstack-nova | 19:50 | |
| mnaser | hm, does anyone have any idea behind the historical reason why ComputeCapabilitiesFilter is hard-coded to use instancetypes (not taking image properties into consideration?) | 19:52 |
| mnaser | context: customer *needs* instances with a specific cpu flag, i'd like to minimize the number of flavors so i was hoping i can have a image flag requesting that cpu flag | 19:52 |
| mnaser | vs having to create a new instance type for this use case specifically | 19:53 |
| efried | jaypipes: Considering removing ProviderTree.set_traits/set_aggregates entirely. | 19:53 |
| *** danpawlik has quit IRC | 19:54 | |
| *** gouthamr has quit IRC | 19:54 | |
| *** moshele has joined #openstack-nova | 19:55 | |
| *** suresh12 has quit IRC | 19:57 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Allow for merging traits and aggregates https://review.openstack.org/552122 | 20:00 |
| efried | dansmith, jaypipes, cdent, edleafe: ^ with changes as requested, plus for aggregates | 20:00 |
| *** r-daneel_ has joined #openstack-nova | 20:00 | |
| *** mvk has quit IRC | 20:01 | |
| *** r-daneel has quit IRC | 20:02 | |
| *** r-daneel_ is now known as r-daneel | 20:02 | |
| *** slaweq___ has quit IRC | 20:02 | |
| cdent | ✔ | 20:03 |
| *** Sukhdev has joined #openstack-nova | 20:04 | |
| *** salv-orlando has joined #openstack-nova | 20:08 | |
| *** jackie-truong has quit IRC | 20:09 | |
| *** slaweq has quit IRC | 20:09 | |
| *** slaweq has joined #openstack-nova | 20:10 | |
| *** esberglu_ has joined #openstack-nova | 20:10 | |
| *** tssurya has quit IRC | 20:11 | |
| *** lpetrut has quit IRC | 20:11 | |
| *** liverpooler has quit IRC | 20:12 | |
| *** esberglu has quit IRC | 20:12 | |
| *** tssurya has joined #openstack-nova | 20:12 | |
| *** salv-orlando has quit IRC | 20:13 | |
| *** jackie-truong has joined #openstack-nova | 20:15 | |
| jaypipes | efried: +2 from me. thx | 20:16 |
| efried | thx | 20:16 |
| *** suresh12 has joined #openstack-nova | 20:16 | |
| *** tidwellr has quit IRC | 20:19 | |
| *** tidwellr_ has joined #openstack-nova | 20:19 | |
| *** mgoddard_ has joined #openstack-nova | 20:21 | |
| edleafe | efried: a little late, but added my +1 for good measure :) | 20:21 |
| efried | thx | 20:22 |
| dansmith | efried: consider me suitably thrown under the bus for my method naming | 20:23 |
| efried | dansmith: You're not held accountable for sample code in pastebin. Else I would have griped at you about having the + case first and the - case the default. | 20:24 |
| dansmith | else should have raised | 20:25 |
| mriedem | efried: with ^ how can i integrate that with https://review.openstack.org/#/c/538498/ where i'm not in a virt driver with a provider tree - or will that RT code eventually have the provider tree object so i can call add_traits there? | 20:26 |
| mriedem | alternatively, if the RT isn't the place to do that and the virt driver is, i could add a generic method in the base ComputeDriver class that handles adding traits from the capabilities dict | 20:28 |
| dansmith | mriedem: virt driver can do that from u_p_t he's describing here | 20:28 |
| mriedem | right, but i don't want to copy the same thing in all virt drivers | 20:28 |
| dansmith | but better for compute to do it right where it's calling into the virt driver | 20:28 |
| mriedem | so RT or base class, i'm ok with either | 20:28 |
| dansmith | call the capabilities compute-owned | 20:28 |
| efried | mriedem: I'm glad you asked. I believe you should get the traits from placement and add in those you got from _get_traits (which is the union of those from virt & capabilities). | 20:29 |
| mriedem | the compute isn't calling this, the RT is | 20:29 |
| dansmith | yeah, wherever we call that in the virt driver from | 20:29 |
| mriedem | i don't know what the RT will have the UPT | 20:29 |
| dansmith | efried: eh? | 20:29 |
| *** Sukhdev has quit IRC | 20:29 | |
| mriedem | maybe it will and i'm just behind | 20:29 |
| *** nicolasbock has joined #openstack-nova | 20:29 | |
| mriedem | efried: i'm not sure why i need to call placement, | 20:29 |
| dansmith | efried: we get them from the virt driver structure, we can just generate the traits that go with them and tack them onto the provider tree for the compute node RP | 20:29 |
| mriedem | i want to *always* report these capabilities | 20:29 |
| dansmith | mriedem: you wouldn't | 20:29 |
| dansmith | right | 20:29 |
| dansmith | these are compute-asserted traits, IMHO | 20:30 |
| mriedem | telling me, "go look at the UPT series of changes and figure it out" is acceptable also | 20:30 |
| efried | mriedem: If we can slot your change into the UPT series, that would make things easier to reason about. Having to decide what it would look like in all three possible permutations is harder. | 20:31 |
| *** slaweq has quit IRC | 20:31 | |
| *** slaweq has joined #openstack-nova | 20:32 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Network interface attach/detach https://review.openstack.org/546813 | 20:32 |
| dansmith | mriedem: https://review.openstack.org/#/c/520246/52/nova/compute/resource_tracker.py L884 there I think | 20:32 |
| dansmith | logically it's compute, but mechanically it's RT because that's what owns the compute node RP | 20:32 |
| mriedem | ok that works for me, i can rebase on top of that | 20:32 |
| efried | mriedem: If we do your change *after* https://review.openstack.org/#/c/520246/ then you can use the prov_tree to merge in the capability traits and then call update_from_provider_tree. | 20:32 |
| mriedem | yeah i'll plan on that, | 20:33 |
| mriedem | no rush | 20:33 |
| *** yangyapeng has joined #openstack-nova | 20:34 | |
| *** esberglu_ has quit IRC | 20:34 | |
| *** esberglu has joined #openstack-nova | 20:35 | |
| *** moshele has quit IRC | 20:35 | |
| efried | mriedem: I'm about to do some ka-shuffle-fu. Want me to fold that in? | 20:35 |
| mriedem | no leave it to me | 20:36 |
| mriedem | please | 20:36 |
| mriedem | that also leaves you on the hook for reviewing it when i do it | 20:37 |
| *** felipemonteiro_ has quit IRC | 20:37 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:37 | |
| mikal | Deleted nova-net yet? | 20:38 |
| *** esberglu_ has joined #openstack-nova | 20:38 | |
| *** yangyapeng has quit IRC | 20:39 | |
| *** esberglu has quit IRC | 20:39 | |
| *** prateek has quit IRC | 20:41 | |
| *** salv-orlando has joined #openstack-nova | 20:42 | |
| mriedem | no | 20:48 |
| mriedem | working on it | 20:48 |
| mriedem | need to get https://review.openstack.org/#/c/549789/ going | 20:48 |
| mriedem | but i have to go through the test failures there and figure out what (and how) to blacklist | 20:49 |
| *** hoangcx has quit IRC | 20:49 | |
| mriedem | mikal: ^ | 20:49 |
| mriedem | the failures are mostly just due to stuff like not supporting security groups in the cells v1 api | 20:49 |
| *** hoangcx has joined #openstack-nova | 20:49 | |
| mriedem | https://review.openstack.org/#/c/225199/ would have handled that nicely but it got bogged down in committee | 20:50 |
| *** burt has quit IRC | 20:50 | |
| *** wolverineav has quit IRC | 20:51 | |
| *** wolverineav has joined #openstack-nova | 20:52 | |
| *** amodi has quit IRC | 20:56 | |
| *** wolverineav has quit IRC | 20:56 | |
| *** itlinux has quit IRC | 20:57 | |
| *** tblakes has joined #openstack-nova | 21:00 | |
| *** mvk has joined #openstack-nova | 21:03 | |
| *** tblakes has quit IRC | 21:05 | |
| edmondsw | does tagged_attach require the metadata service? Because cloud-init would only be an option during boot, not after, right? | 21:06 |
| *** amodi has joined #openstack-nova | 21:08 | |
| *** ameeda has joined #openstack-nova | 21:08 | |
| ameeda | Hi, I try to call heat template by rest api, when I try to do GET http://<IP>:<PORT>/v1/<Tenent_ID>/stacks, I got this message HTTP request sent, awaiting response... 401 Unauthorized | 21:10 |
| edmondsw | and by cloud-init I should have said config drive... | 21:10 |
| *** gouthamr has joined #openstack-nova | 21:12 | |
| *** r-daneel_ has joined #openstack-nova | 21:14 | |
| *** suresh12 has quit IRC | 21:15 | |
| *** suresh12 has joined #openstack-nova | 21:15 | |
| *** suresh12 has quit IRC | 21:15 | |
| *** mgoddard_ has quit IRC | 21:15 | |
| *** suresh12 has joined #openstack-nova | 21:16 | |
| *** r-daneel has quit IRC | 21:16 | |
| *** r-daneel_ is now known as r-daneel | 21:16 | |
| *** artom has joined #openstack-nova | 21:18 | |
| *** sar has quit IRC | 21:19 | |
| *** ameeda has quit IRC | 21:19 | |
| *** Sukhdev has joined #openstack-nova | 21:20 | |
| mriedem | edmondsw: tagged attach does not require the metadata service, the tagged devices will show up in the metadata blob in the config drive | 21:21 |
| mriedem | the config drive will only have the create-time tagged devices, correct | 21:21 |
| mriedem | if you need on-the-fly changes to tagged devices then you'd need the metadata service | 21:22 |
| edmondsw | mriedem config drive is available beyond the first boot? | 21:22 |
| mriedem | no it's not | 21:22 |
| edmondsw | ok, that's what I was thinking | 21:22 |
| edmondsw | boo... but tx for confirming | 21:23 |
| mriedem | now if you're using configuration strategy, | 21:23 |
| mriedem | who knows | 21:23 |
| mriedem | :) | 21:24 |
| edmondsw | "configuration strategy"? | 21:24 |
| mriedem | https://www.ibm.com/developerworks/community/forums/html/topic?id=1918efbc-ae12-44a3-8075-a6023831d334&ps=100 | 21:27 |
| mriedem | https://www.ibm.com/support/knowledgecenter/en/SST55W_4.2.0/liaca/liacaimportingstrategy.html is better | 21:28 |
| mriedem | i.e. config-drive for ibm i and aix images right? | 21:29 |
| *** gouthamr has quit IRC | 21:30 | |
| mriedem | boot me an ibm i guest with 500 volumes attached in a single call please | 21:30 |
| *** wolverineav has joined #openstack-nova | 21:32 | |
| *** Guest18434 has quit IRC | 21:33 | |
| *** jackie-truong has quit IRC | 21:34 | |
| *** pcaruana has quit IRC | 21:36 | |
| *** wolverineav has quit IRC | 21:36 | |
| *** wolverineav has joined #openstack-nova | 21:39 | |
| openstackgerrit | Merged openstack/nova-specs master: Report CPU features to placement service by traits API https://review.openstack.org/497733 | 21:39 |
| *** gouthamr has joined #openstack-nova | 21:40 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Snapshot https://review.openstack.org/543023 | 21:41 |
| *** felipemonteiro__ has joined #openstack-nova | 21:43 | |
| *** yangyapeng has joined #openstack-nova | 21:43 | |
| *** liusheng has quit IRC | 21:45 | |
| *** liusheng has joined #openstack-nova | 21:46 | |
| *** edmondsw has quit IRC | 21:46 | |
| *** suresh12 has quit IRC | 21:46 | |
| *** felipemonteiro_ has quit IRC | 21:46 | |
| *** edmondsw has joined #openstack-nova | 21:46 | |
| *** suresh12 has joined #openstack-nova | 21:47 | |
| *** Sukhdev has quit IRC | 21:47 | |
| *** liusheng has quit IRC | 21:48 | |
| *** tbachman has quit IRC | 21:48 | |
| *** yangyapeng has quit IRC | 21:48 | |
| *** liusheng has joined #openstack-nova | 21:49 | |
| *** salv-orlando has quit IRC | 21:49 | |
| *** blkart has quit IRC | 21:49 | |
| *** krtaylor has quit IRC | 21:49 | |
| *** kashyap has quit IRC | 21:50 | |
| *** Swami has quit IRC | 21:50 | |
| *** Swami has joined #openstack-nova | 21:51 | |
| *** blkart has joined #openstack-nova | 21:52 | |
| *** kashyap has joined #openstack-nova | 21:52 | |
| *** krtaylor has joined #openstack-nova | 21:53 | |
| *** salv-orlando has joined #openstack-nova | 21:55 | |
| *** cdent has quit IRC | 21:56 | |
| openstackgerrit | Merged openstack/nova-specs master: Express forbidden traits in placement API https://review.openstack.org/548915 | 22:03 |
| *** gouthamr has quit IRC | 22:03 | |
| *** esberglu_ has quit IRC | 22:04 | |
| *** esberglu has joined #openstack-nova | 22:05 | |
| *** tbachman has joined #openstack-nova | 22:08 | |
| *** esberglu has quit IRC | 22:09 | |
| *** suresh12 has quit IRC | 22:10 | |
| *** tbachman_ has joined #openstack-nova | 22:10 | |
| *** tasker has joined #openstack-nova | 22:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix spec name for "report CPU features to as traits" https://review.openstack.org/552159 | 22:12 |
| tasker | trying to instruct tox to run "nova.tests.unit.compute.test_compute_api" and it fails, complaining about being unable to import "nova.tests.unit.network.test_neutronv2", "nova.tests.unit.virt.test_block_device", and "nova.tests.unit.virt.test_imagecache". any quick thoughts on what I've done wrong? | 22:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix spec name for "report CPU features as traits" https://review.openstack.org/552159 | 22:13 |
| tasker | I've rebased my work on master a few minutes ago and still getting the above. | 22:13 |
| *** tbachman has quit IRC | 22:13 | |
| *** tbachman_ is now known as tbachman | 22:13 | |
| mriedem | tasker: did you rebuild the tox venv? | 22:13 |
| *** gouthamr has joined #openstack-nova | 22:13 | |
| mriedem | tox -r -e py27 | 22:13 |
| tasker | no. i was unaware of that. doing so now. | 22:13 |
| mriedem | tox -r -e py27 -- nova.tests.unit.compute.test_compute_api | 22:14 |
| *** suresh12 has joined #openstack-nova | 22:14 | |
| edleafe | /away evening | 22:14 |
| edleafe | doh! | 22:14 |
| *** tssurya has quit IRC | 22:15 | |
| *** idlemind has joined #openstack-nova | 22:17 | |
| *** mlavalle has quit IRC | 22:18 | |
| *** tidwellr_ has quit IRC | 22:21 | |
| *** tidwellr has joined #openstack-nova | 22:21 | |
| *** edmondsw has quit IRC | 22:24 | |
| *** rcernin has joined #openstack-nova | 22:24 | |
| *** edmondsw has joined #openstack-nova | 22:24 | |
| *** tidwellr has quit IRC | 22:25 | |
| tasker | mriedem: that did the trick. thanks! | 22:26 |
| tasker | now I just need to fix my test. | 22:27 |
| *** edmondsw has quit IRC | 22:29 | |
| *** yangyapeng has joined #openstack-nova | 22:32 | |
| *** hongbin has quit IRC | 22:33 | |
| *** Sukhdev has joined #openstack-nova | 22:33 | |
| *** yangyapeng has quit IRC | 22:36 | |
| openstackgerrit | Merged openstack/nova master: Make the nova-next job voting and gating https://review.openstack.org/549893 | 22:38 |
| *** AlexeyAbashkin has joined #openstack-nova | 22:38 | |
| *** elmaciej has joined #openstack-nova | 22:41 | |
| *** AlexeyAbashkin has quit IRC | 22:42 | |
| *** suresh12 has quit IRC | 22:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 22:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 22:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | 22:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 22:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_bindings" network API method https://review.openstack.org/552170 | 22:50 |
| *** felipemonteiro__ has quit IRC | 22:54 | |
| *** amodi has quit IRC | 22:56 | |
| *** elmaciej has quit IRC | 22:56 | |
| *** yamamoto has joined #openstack-nova | 22:57 | |
| openstackgerrit | Merged openstack/nova master: setup.cfg: Explicitly set [build_sphinx] builder https://review.openstack.org/508483 | 22:59 |
| openstackgerrit | Merged openstack/nova master: Check the return code when forcing TCG mode with libguestfs https://review.openstack.org/524727 | 22:59 |
| openstackgerrit | Merged openstack/nova master: Remove vestigial extra_info update in PciDevice.save() https://review.openstack.org/523919 | 22:59 |
| openstackgerrit | Merged openstack/nova master: conf: Deprecate 'keymap' options https://review.openstack.org/483994 | 22:59 |
| openstackgerrit | Eric M Gonzalez (tasker) proposed openstack/nova master: unquiesce instance after quiesce failure https://review.openstack.org/550865 | 23:01 |
| tasker | with tests this time! | 23:02 |
| *** suresh12 has joined #openstack-nova | 23:03 | |
| *** tidwellr has joined #openstack-nova | 23:04 | |
| *** chyka_ has joined #openstack-nova | 23:05 | |
| *** suresh12_ has joined #openstack-nova | 23:07 | |
| *** suresh12 has quit IRC | 23:07 | |
| openstackgerrit | Giridhar Jayavelu proposed openstack/nova-specs master: VMware: place instances on resource pool https://review.openstack.org/549067 | 23:08 |
| *** chyka has quit IRC | 23:08 | |
| *** chyka_ has quit IRC | 23:09 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Allow for merging traits and aggregates https://review.openstack.org/552122 | 23:10 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | 23:10 |
| *** r-daneel has quit IRC | 23:12 | |
| *** germs has joined #openstack-nova | 23:14 | |
| *** germs has quit IRC | 23:14 | |
| *** germs has joined #openstack-nova | 23:14 | |
| *** tidwellr has quit IRC | 23:15 | |
| *** tidwellr has joined #openstack-nova | 23:15 | |
| *** tidwellr has quit IRC | 23:16 | |
| *** germs has quit IRC | 23:18 | |
| *** yassine has joined #openstack-nova | 23:19 | |
| cfriesen | is it possible to query reviews that haven't been merged based on Change-Id? | 23:20 |
| *** Sukhdev has quit IRC | 23:20 | |
| cfriesen | never mind, figured it out. works every time, post a question after digging for a while, then find it immediately after posting. | 23:20 |
| *** salv-orlando has quit IRC | 23:29 | |
| *** tidwellr has joined #openstack-nova | 23:30 | |
| *** yangyapeng has joined #openstack-nova | 23:32 | |
| *** yangyapeng has quit IRC | 23:36 | |
| *** artom has quit IRC | 23:37 | |
| *** r-daneel has joined #openstack-nova | 23:39 | |
| *** edmondsw has joined #openstack-nova | 23:41 | |
| *** tidwellr has quit IRC | 23:42 | |
| *** liverpooler has joined #openstack-nova | 23:44 | |
| *** r-daneel has quit IRC | 23:44 | |
| Spazmotic | Morning friends | 23:45 |
| *** r-daneel has joined #openstack-nova | 23:46 | |
| *** edmondsw has quit IRC | 23:46 | |
| Spazmotic | jianghuaw_, you home? :D I'm going to head to the gym, would like to talk to ya later! Has been too long! | 23:46 |
| *** yassine has quit IRC | 23:46 | |
| Spazmotic | cfriesen, you figure everything out? i've been digging Gerritt api all week so I feel pretty strong it lol | 23:47 |
| Spazmotic | on it* | 23:47 |
| *** yassine has joined #openstack-nova | 23:48 | |
| *** wolverineav has quit IRC | 23:50 | |
| *** itlinux has joined #openstack-nova | 23:51 | |
| *** sdague has quit IRC | 23:51 | |
| *** wolverineav has joined #openstack-nova | 23:52 | |
| *** tbachman has quit IRC | 23:56 | |
| *** liusheng has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!