Wednesday, 2018-04-18

openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vmops.py  https://review.openstack.org/56184600:01
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vmops.py  https://review.openstack.org/56184600:01
*** yamamoto has joined #openstack-nova00:03
*** tianhui_ has joined #openstack-nova00:03
*** r-daneel has joined #openstack-nova00:05
*** tianhui has quit IRC00:05
*** r-daneel has quit IRC00:05
*** fragatina has quit IRC00:07
*** yamamoto has quit IRC00:07
*** tbachman_ has joined #openstack-nova00:11
*** tbachman has quit IRC00:13
*** tbachman_ is now known as tbachman00:13
*** liusheng has quit IRC00:20
*** liusheng has joined #openstack-nova00:21
*** Tom-Tom has joined #openstack-nova00:22
*** yamahata has quit IRC00:23
*** yamamoto has joined #openstack-nova00:24
*** Tom-Tom has quit IRC00:27
*** hoangcx has joined #openstack-nova00:28
*** yamamoto has quit IRC00:29
*** Tom-Tom has joined #openstack-nova00:31
*** Tom-Tom has quit IRC00:36
*** Dinesh_Bhor has joined #openstack-nova00:36
*** gjayavelu has joined #openstack-nova00:36
*** wolverin_ has quit IRC00:39
*** wolverineav has joined #openstack-nova00:40
*** markmcclain has quit IRC00:44
*** yamamoto has joined #openstack-nova00:44
*** hshiina|afk is now known as hshiina00:46
*** hiro-kobayashi has joined #openstack-nova00:49
*** markmcclain has joined #openstack-nova00:49
*** yamamoto has quit IRC00:51
*** dougshelley66 has quit IRC00:51
*** chenyb4 has joined #openstack-nova00:51
openstackgerritTetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs  https://review.openstack.org/56184200:54
openstackgerritMerged openstack/nova stable/ocata: Fix wrapping of neutron forbidden error  https://review.openstack.org/56008800:54
*** stakeda has joined #openstack-nova00:56
*** jichen has joined #openstack-nova00:58
*** Dinesh__Bhor has joined #openstack-nova00:58
Kevin_Zhengmelwitt Thanks for the info, I will fix the tests ASAP01:01
*** Dinesh_Bhor has quit IRC01:01
*** liuzz has joined #openstack-nova01:01
*** gjayavelu has quit IRC01:02
*** salv-orlando has joined #openstack-nova01:02
*** yamahata has joined #openstack-nova01:04
*** salv-orl_ has quit IRC01:04
*** phuongnh has joined #openstack-nova01:05
*** yamamoto has joined #openstack-nova01:07
*** yamamoto has quit IRC01:12
*** zhaochao has joined #openstack-nova01:15
*** harlowja has quit IRC01:16
*** tetsuro has joined #openstack-nova01:19
*** yamamoto has joined #openstack-nova01:28
*** armaan has quit IRC01:32
*** yamamoto has quit IRC01:32
*** janki has joined #openstack-nova01:33
*** takashin has quit IRC01:35
*** Tom-Tom has joined #openstack-nova01:38
*** AlexeyAbashkin has joined #openstack-nova01:39
*** hongbin_ has joined #openstack-nova01:40
*** sdeath has quit IRC01:41
*** salv-orlando has quit IRC01:41
*** salv-orlando has joined #openstack-nova01:42
*** AlexeyAbashkin has quit IRC01:43
openstackgerritMerged openstack/nova master: Deduplicate notification samples Rocky - 6  https://review.openstack.org/56073701:45
naichuansjaypipes: Hi, Jay, for vGPU nested resource provider, should we just keep the old rps when the hypervisor return no information about them?01:49
*** yamamoto has joined #openstack-nova01:49
*** slaweq has joined #openstack-nova01:50
*** spsurya has joined #openstack-nova01:50
*** wolverineav has quit IRC01:54
*** wolverineav has joined #openstack-nova01:54
*** slaweq has quit IRC01:55
*** yamamoto has quit IRC01:55
*** Tom-Tom has quit IRC01:58
*** wolverineav has quit IRC01:59
*** salv-orl_ has joined #openstack-nova02:00
*** salv-orlando has quit IRC02:03
*** sdague has quit IRC02:04
*** nicolasbock has quit IRC02:06
*** Nel1x has quit IRC02:11
*** yamamoto has joined #openstack-nova02:12
*** jmlowe_ has quit IRC02:15
*** yamamoto has quit IRC02:17
*** Tom-Tom has joined #openstack-nova02:18
*** Tom-Tom has quit IRC02:18
*** Tom-Tom has joined #openstack-nova02:18
*** phuongnh has quit IRC02:19
*** phuongnh has joined #openstack-nova02:21
*** takashin has joined #openstack-nova02:27
*** Nel1x has joined #openstack-nova02:29
*** dklyle has joined #openstack-nova02:31
*** yamamoto has joined #openstack-nova02:32
*** artom_ has joined #openstack-nova02:34
*** artom_ has quit IRC02:34
*** artom_ has joined #openstack-nova02:35
*** artom has quit IRC02:36
*** Guest4072 has quit IRC02:36
*** yamamoto has quit IRC02:39
*** sdeath has joined #openstack-nova02:40
*** Guest4072 has joined #openstack-nova02:41
*** jmlowe has joined #openstack-nova02:41
*** Dinesh__Bhor has quit IRC02:41
*** Dinesh__Bhor has joined #openstack-nova02:42
*** psachin has joined #openstack-nova02:49
*** jchhatbar has joined #openstack-nova02:51
*** janki has quit IRC02:53
*** yamamoto has joined #openstack-nova02:55
*** fragatina has joined #openstack-nova03:00
*** yamamoto has quit IRC03:00
*** takashin has left #openstack-nova03:00
*** fragatin_ has joined #openstack-nova03:01
*** dave-mccowan has quit IRC03:02
*** fragatina has quit IRC03:04
openstackgerritZhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications  https://review.openstack.org/55328803:06
*** TobbeCN has joined #openstack-nova03:13
*** yamamoto has joined #openstack-nova03:16
*** TobbeCN has quit IRC03:18
*** yamamoto has quit IRC03:21
*** hoangcx_ has joined #openstack-nova03:36
*** yamamoto has joined #openstack-nova03:37
openstackgerritZhenyu Zheng proposed openstack/nova master: Send out notification when instance metadata updated  https://review.openstack.org/55814003:39
*** hoangcx has quit IRC03:40
*** lpetrut_ has joined #openstack-nova03:41
*** harlowja has joined #openstack-nova03:43
*** yamamoto has quit IRC03:43
*** chenyb4 has quit IRC03:47
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748603:48
*** chenyb4 has joined #openstack-nova03:48
*** slaweq has joined #openstack-nova03:51
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647503:54
*** slaweq has quit IRC03:56
*** Dinesh__Bhor has quit IRC03:58
*** yamamoto has joined #openstack-nova04:00
*** Dinesh__Bhor has joined #openstack-nova04:01
*** hiro-kobayashi has quit IRC04:02
*** salv-orl_ has quit IRC04:04
*** yamamoto has quit IRC04:04
*** salv-orlando has joined #openstack-nova04:04
*** hongbin_ has quit IRC04:07
*** harlowja has quit IRC04:12
*** lpetrut_ has quit IRC04:13
*** Dinesh__Bhor has quit IRC04:14
*** Dinesh__Bhor has joined #openstack-nova04:14
*** lpetrut_ has joined #openstack-nova04:19
*** gyee has quit IRC04:20
*** yamamoto has joined #openstack-nova04:21
*** udesale has joined #openstack-nova04:21
*** yamamoto has quit IRC04:25
*** zhaochao has quit IRC04:30
*** zhaochao has joined #openstack-nova04:31
*** abhishekk has joined #openstack-nova04:35
*** Eran_Kuris has joined #openstack-nova04:37
*** hoangcx_ is now known as hoangcx04:38
*** fragatin_ has quit IRC04:41
*** fragatina has joined #openstack-nova04:41
*** yamamoto has joined #openstack-nova04:41
*** lpetrut_ has quit IRC04:44
*** yamamoto has quit IRC04:48
*** yamamoto has joined #openstack-nova04:48
*** Guest4072 has quit IRC04:50
*** claudiub|2 has joined #openstack-nova04:51
*** gjayavelu has joined #openstack-nova04:54
*** ratailor has joined #openstack-nova04:55
openstackgerritMerged openstack/nova master: Remove placement config check  https://review.openstack.org/55808904:56
*** Guest4072 has joined #openstack-nova04:57
*** jchhatbar has quit IRC04:59
*** Dinesh__Bhor has quit IRC05:00
*** jchhatbar has joined #openstack-nova05:00
*** jchhatbar has quit IRC05:01
*** jchhatbar has joined #openstack-nova05:01
*** jchhatbar has quit IRC05:02
*** jchhatbar has joined #openstack-nova05:03
*** Dinesh__Bhor has joined #openstack-nova05:05
*** pcaruana has joined #openstack-nova05:05
*** Dinesh__Bhor has quit IRC05:14
*** pcaruana has quit IRC05:19
*** lpetrut_ has joined #openstack-nova05:23
*** sridharg has joined #openstack-nova05:23
*** bkopilov has quit IRC05:24
*** Dinesh_Bhor has joined #openstack-nova05:26
openstackgerritjichenjc proposed openstack/nova master: Fix docs for confirmResize action  https://review.openstack.org/56206205:29
*** udesale_ has joined #openstack-nova05:33
*** lpetrut_ has quit IRC05:35
*** sar has quit IRC05:35
*** zhurong has joined #openstack-nova05:36
*** udesale has quit IRC05:36
*** hamzy has quit IRC05:37
*** krtaylor has quit IRC05:37
*** bingqianglin has joined #openstack-nova05:39
*** hamzy has joined #openstack-nova05:47
*** krtaylor has joined #openstack-nova05:50
*** lajoskatona has joined #openstack-nova05:58
*** sidx64 has joined #openstack-nova06:10
*** psachin has quit IRC06:11
*** kholkina has joined #openstack-nova06:13
*** Tom-Tom_ has joined #openstack-nova06:13
*** Tom-Tom has quit IRC06:16
*** do3meli has quit IRC06:18
*** armaan has joined #openstack-nova06:18
*** do3meli has joined #openstack-nova06:19
*** yinxq has quit IRC06:20
*** gmann has quit IRC06:21
*** TobbeCN has joined #openstack-nova06:21
*** lpetrut has joined #openstack-nova06:26
*** TobbeCN has quit IRC06:26
*** Dinesh__Bhor has joined #openstack-nova06:28
*** Dinesh_Bhor has quit IRC06:28
*** jchhatba_ has joined #openstack-nova06:30
*** jmlowe_ has joined #openstack-nova06:31
*** dklyle has quit IRC06:32
*** dklyle has joined #openstack-nova06:32
*** jchhatbar has quit IRC06:33
*** jmlowe has quit IRC06:33
*** tianhui has joined #openstack-nova06:33
*** dpawlik has joined #openstack-nova06:34
*** yikun_ has joined #openstack-nova06:34
*** tianhui_ has quit IRC06:34
*** yikun has quit IRC06:36
*** dpawlik has quit IRC06:38
*** sidx64_ has joined #openstack-nova06:38
*** bingqianglin has quit IRC06:40
*** sidx64 has quit IRC06:40
*** zhaochao has quit IRC06:42
*** sidx64 has joined #openstack-nova06:42
*** ildikov has joined #openstack-nova06:43
*** TobbeCN has joined #openstack-nova06:43
*** jchhatba_ is now known as janki06:44
*** sidx64_ has quit IRC06:44
*** mordred has quit IRC06:45
*** links has joined #openstack-nova06:46
*** dklyle has quit IRC06:47
*** mordred has joined #openstack-nova06:48
*** pcaruana has joined #openstack-nova06:50
*** blkart has quit IRC06:53
*** dineshbhor__ has joined #openstack-nova06:56
*** alexchadin has joined #openstack-nova06:59
*** rcernin has quit IRC07:00
*** Dinesh__Bhor has quit IRC07:00
*** gjayavelu has quit IRC07:03
*** cfriesen has quit IRC07:06
*** zigo has quit IRC07:11
*** gmann has joined #openstack-nova07:13
*** zigo has joined #openstack-nova07:14
*** damien_r has joined #openstack-nova07:17
*** links has quit IRC07:29
*** tesseract has joined #openstack-nova07:29
*** udesale__ has joined #openstack-nova07:29
*** AlexeyAbashkin has joined #openstack-nova07:30
*** zhurong has quit IRC07:30
*** udesale_ has quit IRC07:32
*** psachin has joined #openstack-nova07:34
*** dineshbhor__ has quit IRC07:35
*** dpawlik has joined #openstack-nova07:35
*** Dinesh_Bhor has joined #openstack-nova07:35
*** amoralej|off is now known as amoralej07:38
*** dpawlik has quit IRC07:40
*** pooja-jadhav has quit IRC07:43
*** dpawlik has joined #openstack-nova07:45
*** links has joined #openstack-nova07:46
*** jpena|off is now known as jpena07:47
*** yikun__ has joined #openstack-nova07:48
*** yamamoto has quit IRC07:49
*** yamamoto has joined #openstack-nova07:49
*** yikun_ has quit IRC07:51
*** links has quit IRC07:52
*** sidx64 has quit IRC07:52
*** tssurya has joined #openstack-nova07:52
*** tianhui_ has joined #openstack-nova07:52
*** tianhui has quit IRC07:55
*** yamamoto_ has joined #openstack-nova07:56
*** ccamacho has joined #openstack-nova07:59
*** dpawlik has quit IRC08:00
*** sidx64 has joined #openstack-nova08:00
*** yamamoto has quit IRC08:00
*** mdnadeem has joined #openstack-nova08:04
*** zhaochao has joined #openstack-nova08:04
*** lucas-afk is now known as lucasagomes08:05
*** sidx64 has quit IRC08:05
*** dpawlik has joined #openstack-nova08:05
*** sidx64 has joined #openstack-nova08:08
*** dpawlik has quit IRC08:09
*** mgoddard has joined #openstack-nova08:10
openstackgerritsahid proposed openstack/nova master: compute: introduce cpu_shared_set option  https://review.openstack.org/56191208:13
openstackgerritsahid proposed openstack/nova master: libvirt: always pass emulator threads policy  https://review.openstack.org/56191308:13
openstackgerritsahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set  https://review.openstack.org/51089708:13
*** sahid has joined #openstack-nova08:14
*** sidx64 has quit IRC08:14
*** avolkov has joined #openstack-nova08:17
*** mdbooth has joined #openstack-nova08:17
*** mdbooth_ has joined #openstack-nova08:18
*** bkopilov has joined #openstack-nova08:19
*** sambetts|afk is now known as sambetts08:19
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Return resources of entire trees in Placement  https://review.openstack.org/55946608:20
openstackgerritFlorian Haas proposed openstack/nova stable/ocata: Increase cpu time for image conversion  https://review.openstack.org/56214508:21
*** ktibi has joined #openstack-nova08:22
*** mdbooth has quit IRC08:22
*** stakeda has quit IRC08:23
*** slaweq has joined #openstack-nova08:23
*** zhaochao has quit IRC08:30
*** zhaochao has joined #openstack-nova08:30
*** Tom-Tom_ has quit IRC08:34
*** phuongnh has quit IRC08:35
*** markus_z has joined #openstack-nova08:35
*** udesale_ has joined #openstack-nova08:35
*** Tom-Tom has joined #openstack-nova08:36
*** Eran_Kuris has quit IRC08:37
*** udesale__ has quit IRC08:38
*** zhurong has joined #openstack-nova08:39
*** derekh has joined #openstack-nova08:43
*** sar has joined #openstack-nova08:45
*** markus_z has left #openstack-nova08:48
*** links has joined #openstack-nova08:48
*** pooja_jadhav has joined #openstack-nova08:50
*** hshiina is now known as hshiina|afk08:54
*** dr_gogeta86 has joined #openstack-nova08:56
*** dr_gogeta86 has quit IRC08:57
*** dr_gogeta86 has joined #openstack-nova08:57
openstackgerritZhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications  https://review.openstack.org/55328809:01
*** yikun__ has quit IRC09:02
*** yikun__ has joined #openstack-nova09:03
*** annp has quit IRC09:04
*** hoangcx has quit IRC09:04
*** tetsuro has left #openstack-nova09:04
*** M0z122 has quit IRC09:04
openstackgerritjichenjc proposed openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215409:04
*** hoangcx has joined #openstack-nova09:05
*** annp has joined #openstack-nova09:05
openstackgerritjichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url  https://review.openstack.org/51140009:05
openstackgerritjichenjc proposed openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215409:06
*** elmaciej has joined #openstack-nova09:06
*** phuongnh has joined #openstack-nova09:07
*** sahid has quit IRC09:08
*** mdnadeem_ has joined #openstack-nova09:08
*** mdnadeem has quit IRC09:10
*** trinaths has joined #openstack-nova09:11
openstackgerritjichenjc proposed openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215409:13
*** HW_Peter has quit IRC09:13
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration  https://review.openstack.org/55415409:15
*** bingqianglin has joined #openstack-nova09:16
*** alexchadin has quit IRC09:24
*** alexchadin has joined #openstack-nova09:26
openstackgerritjichenjc proposed openstack/nova-specs master: Add additional information for z/VM spec.  https://review.openstack.org/56215409:27
*** sidx64 has joined #openstack-nova09:29
*** Dinesh_Bhor has quit IRC09:30
*** sidx64 has quit IRC09:30
*** mdnadeem_ has quit IRC09:32
*** mdnadeem has joined #openstack-nova09:32
*** zhaochao has quit IRC09:34
*** elmaciej_ has joined #openstack-nova09:34
*** elmaciej has quit IRC09:34
*** sidx64 has joined #openstack-nova09:34
*** cdent has joined #openstack-nova09:35
*** sahid has joined #openstack-nova09:39
*** zhaochao has joined #openstack-nova09:40
*** itlinux has joined #openstack-nova09:40
*** phuongnh has quit IRC09:41
*** phuongnh has joined #openstack-nova09:41
*** kholkina has quit IRC09:42
*** yikun_ has joined #openstack-nova09:42
*** kholkina has joined #openstack-nova09:42
*** dougshelley66 has joined #openstack-nova09:44
*** yikun__ has quit IRC09:45
*** alexchadin has quit IRC09:49
*** itlinux has quit IRC09:49
*** jichen has quit IRC09:49
*** bhagyashris is now known as neha_alhat09:51
*** neha_alhat is now known as bhagyashris09:52
*** kholkina has quit IRC09:55
openstackgerritsahid proposed openstack/nova master: compute: introduce cpu_shared_set option  https://review.openstack.org/56191209:57
openstackgerritsahid proposed openstack/nova master: libvirt: always pass emulator threads policy  https://review.openstack.org/56191309:57
openstackgerritsahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set  https://review.openstack.org/51089709:57
*** zhurong has quit IRC09:57
*** sdague has joined #openstack-nova10:00
*** itlinux has joined #openstack-nova10:01
stephenfinbauzas: Could you take a look at https://review.openstack.org/#/c/379525 today?10:05
bauzasstephenfin: ack10:05
*** tianhui has joined #openstack-nova10:12
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky"  https://review.openstack.org/55878310:13
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_{LIBVIRT_KVM,QEMU}_S390_VERSION  https://review.openstack.org/56036310:13
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_KVM_PPC64_VERSION  https://review.openstack.org/56036410:13
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_VIRTUOZZO_VERSION  https://review.openstack.org/56036510:13
*** tianhui_ has quit IRC10:14
*** chenyb4 has quit IRC10:17
*** udesale__ has joined #openstack-nova10:21
*** udesale_ has quit IRC10:24
*** elmaciej_ has quit IRC10:24
*** hoangcx has quit IRC10:25
openstackgerritFlorian Haas proposed openstack/nova stable/ocata: Increase cpu time for image conversion  https://review.openstack.org/56214510:26
*** elmaciej has joined #openstack-nova10:27
*** tbachman has quit IRC10:27
*** fghaas has joined #openstack-nova10:27
*** pcaruana has quit IRC10:27
*** tbachman has joined #openstack-nova10:28
*** tbachman has quit IRC10:29
*** nicolasbock has joined #openstack-nova10:31
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031310:31
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table.  https://review.openstack.org/56083210:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy  https://review.openstack.org/55377610:32
*** abhishekk has quit IRC10:33
kashyaplyarwood: Is this Ocata (trivial) backport still acceptable under stable branch policies -- https://review.openstack.org/#/c/562145/10:34
kashyapIt should be accepted & solves a real problem.10:36
kashyapefried: Fixed the tests.  I went with:10:42
kashyap    @mock.patch.object(nova.virt.libvirt.migration,10:42
kashyap                       'get_updated_guest_xml', return_value='')10:42
kashyapLine 9179: https://review.openstack.org/#/c/558783/9/nova/tests/unit/virt/libvirt/test_driver.py10:43
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create  https://review.openstack.org/52031310:43
*** slagle has quit IRC10:44
*** Tom-Tom has quit IRC10:47
*** Tom-Tom has joined #openstack-nova10:47
fghaaslyarwood: I proposed the backport that kashyap just mentioned; I'm here if you need additional info on how exactly we bumped into this in Ocata10:47
lyarwoodsorry just on a call, I'll catch up in ~50mins10:49
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Use XAPI pool instead of aggregate pool for shared SR migration  https://review.openstack.org/55415410:49
fghaaslyarwood: no rush at all, just wanted to let you know — will be here for several hours10:50
*** zz9pzza has quit IRC10:51
*** Tom-Tom has quit IRC10:51
*** tbachman has joined #openstack-nova10:52
*** tbachman_ has joined #openstack-nova10:54
*** zz9pzza has joined #openstack-nova10:56
*** tbachman has quit IRC10:56
*** tbachman_ is now known as tbachman10:56
jaypipesnaichuans, jianghuaw_: still around?11:05
*** phuongnh has quit IRC11:07
*** Nisha_Agarwal has joined #openstack-nova11:08
*** mvk has quit IRC11:08
*** slagle has joined #openstack-nova11:11
*** Tom-Tom has joined #openstack-nova11:13
*** sar has quit IRC11:15
*** udesale__ has quit IRC11:16
*** Tom-Tom has quit IRC11:17
*** lucasagomes is now known as lucas-hungry11:18
*** Shilpa has quit IRC11:19
*** sar has joined #openstack-nova11:21
*** Kevin_Zheng has quit IRC11:30
jaypipessahid: reviewing the cpu_shared_set patches again now.11:31
*** bhagyashris has quit IRC11:32
*** itlinux has quit IRC11:34
*** mdbooth_ has quit IRC11:36
jaypipessahid: +2 on all.11:36
*** yamamoto_ has quit IRC11:36
*** dougshelley66 has quit IRC11:39
*** mdbooth_ has joined #openstack-nova11:39
sahidjaypipes: whaa thanks, i did not expected you to be so active on it :)11:40
jaypipessahid: let's "git r done" ;)11:42
*** sean-k-mooney has quit IRC11:46
*** gmann has quit IRC11:51
*** armaan has quit IRC11:52
*** armaan has joined #openstack-nova11:53
*** cdent has quit IRC11:56
*** Nisha_Agarwal has quit IRC11:56
*** Nisha_Agarwal has joined #openstack-nova11:57
*** itlinux has joined #openstack-nova11:58
*** jpena is now known as jpena|lunch12:00
*** QianYu has joined #openstack-nova12:01
*** Nisha_Agarwal has quit IRC12:02
*** yamamoto has joined #openstack-nova12:03
*** QianYu has quit IRC12:05
*** armaan has quit IRC12:06
*** armaan has joined #openstack-nova12:06
*** alexchadin has joined #openstack-nova12:08
*** amoralej is now known as amoralej|lunch12:10
*** trinaths has quit IRC12:14
*** gmann has joined #openstack-nova12:15
*** edmondsw has joined #openstack-nova12:15
*** pcaruana has joined #openstack-nova12:19
*** mvk has joined #openstack-nova12:20
efriedkashyap: Good deal man.12:24
*** alexchadin has quit IRC12:26
*** dougshelley66 has joined #openstack-nova12:26
*** pchavva has joined #openstack-nova12:27
*** QianYu has joined #openstack-nova12:29
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy  https://review.openstack.org/55377612:29
*** Alexey_Abashkin has joined #openstack-nova12:31
*** lucas-hungry is now known as lucasagomes12:31
*** QianYu has quit IRC12:33
*** AlexeyAbashkin has quit IRC12:33
*** Alexey_Abashkin is now known as AlexeyAbashkin12:33
*** edleafe- has joined #openstack-nova12:35
*** jmlowe_ has quit IRC12:35
*** edleafe has quit IRC12:35
*** edleafe- is now known as edleafe12:35
*** jmlowe has joined #openstack-nova12:35
*** alexchadin has joined #openstack-nova12:36
naichuansjaypipes: Hi, Jay, I'm still here12:38
*** armaan has quit IRC12:40
*** tbachman has quit IRC12:44
*** sidx64 has quit IRC12:46
openstackgerritBalazs Gibizer proposed openstack/nova-specs master: Suggest when to propose notification changes  https://review.openstack.org/56226512:48
*** artom_ has quit IRC12:48
jaypipesnaichuans: hi! so my primary concern is that the virt driver will be adding and removing resource provider records based on a "blip" of some device turning on and off.12:49
kashyapefried: Still ~14 more clean-ups to do.  Since this time the (libvirt / QEMU) version increment been a larger jump (because we didn't bump in "Queens"), need to clean up a lot more constants & compat code.12:49
* kashyap bbiab12:49
*** jbernard has quit IRC12:49
*** jbernard has joined #openstack-nova12:50
*** liuzz_ has joined #openstack-nova12:51
*** liuzz has quit IRC12:51
*** itlinux has quit IRC12:51
*** bhagyashris has joined #openstack-nova12:52
*** ratailor has quit IRC12:52
naichuansjaypipes: understood :) we decide to skip removing rp operation, and keep adding.We can't get allocation information in update_provider_tree. So it is difficult to check if the GPU is used by a VM or not. I remove the unavailable rps removing code. If nova schedule to a unavailable rp, then would be a failure and re-schedule12:53
*** armaan has joined #openstack-nova12:53
*** sidx64 has joined #openstack-nova12:53
naichuansjaypipes: Any suggestion?12:53
*** liverpooler has joined #openstack-nova12:57
bauzasjaypipes: naichuans: I think we agreed to ask operators to not modify the config opt if so12:59
bauzasat least because it would trample the compute service to understand how many vGPUs are used13:00
bauzasif you want to modify that opt, destroy all your instances using vGPUs IMHO13:00
naichuansAh, if operators don't modify the config files, then the issue should not happen~13:01
openstackgerritMatt Riedemann proposed openstack/nova master: Fix docs for confirmResize action  https://review.openstack.org/56206213:01
*** jpena|lunch is now known as jpena13:01
*** QianYu has joined #openstack-nova13:02
openstackgerritNguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build  https://review.openstack.org/55180213:04
*** amoralej|lunch is now known as amoralej13:05
naichuansjaypipes: bauzas: done?13:05
bauzasdone by what?13:05
naichuansI mean we will assume operators would not change config opts?13:06
*** dave-mccowan has joined #openstack-nova13:06
*** tbachman has joined #openstack-nova13:08
openstackgerritMatt Riedemann proposed openstack/nova master: Fix docs for confirmResize action  https://review.openstack.org/56206213:09
*** dave-mccowan has quit IRC13:11
*** asettle has left #openstack-nova13:14
*** dave-mccowan has joined #openstack-nova13:17
*** Shilpa has joined #openstack-nova13:30
*** germs has joined #openstack-nova13:31
*** germs has quit IRC13:31
*** germs has joined #openstack-nova13:31
*** itlinux has joined #openstack-nova13:32
*** fabian_ has joined #openstack-nova13:33
Shilpastephenfin: hi13:34
Shilpastephenfin: changes https://review.openstack.org/#/c/550172/3/ not working, any further inputs to resolve this?13:35
*** sar has quit IRC13:39
*** itlinux has quit IRC13:40
*** awaugama has joined #openstack-nova13:42
jaypipesnaichuans, bauzas: sorry, I really don't understand what you mean by config opts and operators.13:44
*** Kevin_Zheng has joined #openstack-nova13:45
bauzasI mean that enabled_vgpu_types shouldn't be modified13:45
*** udesale__ has joined #openstack-nova13:45
bhagyashrisefried; Hi,13:45
efriedbhagyashris: Hello13:46
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs  https://review.openstack.org/55292413:46
*** psachin has quit IRC13:46
*** fabian_ is now known as chenyb413:46
efriedbhagyashris: It looked like you were getting ready to talk to me yesterday, and then... didn't.13:46
jaypipesbauzas: that's not what we've been discussing, though. we've been discussing removing resource providers that temporarily become "unavailable" due to maintenance or down state, etc. And I have been telling naichuans and jianghuaw_ that it would be best to not auto-remove those resource providers based on some transient state.13:47
bhagyashrisefried: Regarding the patches https://review.openstack.org/#/c/560444/ , https://review.openstack.org/#/c/560459/2 I have fixed the unit test case part. so should i need to write the functional test as well?13:47
*** sidx64 has quit IRC13:47
efriedbhagyashris: I think that would be nice, certainly.13:48
efriedbhagyashris: A functional test that behaves exactly like your live setup did.13:48
naichuansjaypipes: like jianghua's comment, we mean modify the nova.conf to change supported vgpu types13:48
efriedbhagyashris: I would even say you should submit that functional test in its own patch, *before* the above changes, demonstrating that it produces the *wrong* result; and then with the above patches, flip it over to demonstrate the *right* result.13:49
efriedbhagyashris: Also, if we don't already have a LP bug open for this, we should do that.13:49
efrieddammit bauzas, I was in the middle of reviewing that!13:49
Kevin_Zhenggibi efried , Hi thanks for the reviews, I’m totally fine with adding req Id also to update notifications, I will do it tomorrow, could you guys also check https://review.openstack.org/#/c/560748 it make update notification tests using common sample, so if it got merged I don’t have to rebase ;)13:49
bauzasefried: thank Zuul13:50
efriedsigh13:50
bauzasjust a doc -1 because I was missing 2 blank lines13:50
efriedKevin_Zheng: I don't know if adding req IDs to those other objects is appropriate - gibi what do you think?13:50
bhagyashrisefried: yes and there is test for update_provider_tree in nova/tests/functional/test_servers.py so accordingly i will need to add fuctional test write13:50
jaypipesnaichuans: and what happens if there are allocations on a pGPU that you end up removing from the enabled_vgpu_types CONF?13:50
openstackgerritLee Yarwood proposed openstack/nova master: Add regression test for bug #1764883  https://review.openstack.org/56207213:50
openstackgerritLee Yarwood proposed openstack/nova master: WIP compute: Ignore active evacuations during _init_instance  https://review.openstack.org/56228413:50
openstackbug 1764883 in OpenStack Compute (nova) "Evacuation fails if the source host returns while the migration is still in progress" [Undecided,New] https://launchpad.net/bugs/176488313:50
bauzasjaypipes: I'm fine with your point too13:51
*** sidx64 has joined #openstack-nova13:51
efriedbhagyashris: Yes.  I don't know where libvirt-specific functional tests live, but that would be the idea.13:51
jaypipesnaichuans: at a bare minimum, I'd like to see some handling of a failed delete of a resource provider due to, say, an allocation existing against that provider.13:51
efriedbhagyashris: But I'm saying, also a functional test that hits the specific scenario you demonstrated - where the sharing provider was ignored before the fix (the DISK_GB shows up in the compute node RP) but is observed after the fix (shows up in the sharing provider).13:52
naichuansjaypipes: will check it and add the error handling13:52
Kevin_Zhengefried: gibi replied in the patch, he thought it is better to add it for InstanceUpdatePayload but not in status update13:53
*** eharney has joined #openstack-nova13:53
*** itlinux has joined #openstack-nova13:54
bhagyashrisefried: ok, any idea or is there any reference that the test written for libvirt13:56
efriedbhagyashris: I have no idea.  I know nothing about libvirt :)13:56
efriedbhagyashris: I thought *you* were the expert there :P13:57
efriedMaybe kashyap knows...13:57
stephenfinShilpa: No idea, sadly. Could you pull it down and deploy it locally. See what happens13:57
stephenfinShilpa: I imagine the issue is with nova and how it's parsing the URL. Can't say for sure though13:57
jaypipesnaichuans: ty!13:57
naichuansnp :)13:58
openstackgerritMatt Riedemann proposed openstack/nova-specs master: Spec for volume multiattach enhancements  https://review.openstack.org/55207813:59
bhagyashrisefried: ohh :) i will try my best14:00
Shilpastephenfin: i have tested tempst locally for patch set 2, for noVNC v1.0.0, test_novnc failed with 'Not a valid html document in the response.' error and for stable/v0.6, test_novnc failed with 'AssertionError: True is not false : Token must be invalid because the connection closed.'14:00
*** esberglu has joined #openstack-nova14:03
*** lajoskatona has quit IRC14:04
*** sar has joined #openstack-nova14:06
* kashyap blinks and looks up for context14:08
gibiKevin_Zheng: thanks. I will look at https://review.openstack.org/#/c/560748 shortly14:08
stephenfinjaypipes, sahid: I have small questions on the first and last of these three patches. Fancy taking a look so I can go/no-go? https://review.openstack.org/#/q/topic:bp/overhead-pin-set+status:open14:09
efriedkashyap: bhagyashris was looking for guidance as to where one might find/add libvirt-specific functional tests.  I allowed as how I know nothing about libvirt.... but you might.14:10
jaypipesstephenfin: sure thing14:10
*** artom has joined #openstack-nova14:11
*** dave-mccowan has quit IRC14:13
jaypipesstephenfin: answered.14:13
stephenfinta14:13
*** dklyle has joined #openstack-nova14:15
*** chenyb4 has quit IRC14:15
*** itlinux has quit IRC14:15
*** mlavalle has joined #openstack-nova14:18
*** itlinux has joined #openstack-nova14:19
openstackgerritStephen Finucane proposed openstack/nova master: Add CPUWeigher  https://review.openstack.org/37952514:22
kashyapefried: Sorry, being pinged on 4 places, just noticed your response.14:30
kashyapefried: This is the place where I see libvirt functional tests: nova/nova/tests/functional/libvirt14:30
efriedwell that would make sense :)14:31
efriedbhagyashris: ^14:31
sahidstephenfin: stephenfin, jaypipes I replied to you about rasing an exception14:36
sahidit's not something we should do actually14:36
* stephenfin goes to look14:36
*** cfriesen has joined #openstack-nova14:38
*** lpetrut has quit IRC14:40
*** hongbin_ has joined #openstack-nova14:40
jaypipessahid: yep, you made a good point.14:41
bhagyashrisefried: thank you :)14:41
bhagyashriskashyap: Thank you :)14:42
arvindn05bauzas: are you available for the review of https://review.openstack.org/#/c/560718/ - Handle rebuild of instance with new image14:44
arvindn05?14:44
bauzasarvindn05: for 15 mins because of an appointment but yes14:45
arvindn05cool...14:45
sahidjaypipes: cool14:46
arvindn05did you see alex_xu latest comment? he suggests rejecting images with new traits14:46
sahidstephenfin: i will respin the fix put the option under [compute] section. I'm not sure i understand why of your comments about '::'14:46
stephenfinsahid: Sweet, thanks :) Just replying to https://review.openstack.org/#/c/510897 now14:47
arvindn05IMO, i think its too much of an edge case to put in logic to handle it...14:47
*** alexchadin has quit IRC14:47
stephenfinsahid: If you put '::' at the end of the line, the next line is formatted as monospace text, i.e. code. I think that's what you want there, rather than the block comment14:47
stephenfine.g.14:47
stephenfinhere's some descriptive text::14:47
stephenfin14:48
stephenfin   config_option = config_value14:48
bauzasarvindn05: the problem is that we had a huuuuuge problem during Queens about whether we should accept to check the image when rebuilding14:48
arvindn05i would just go back to the original behaviour of trying to find destinations via placement if the image changes....14:48
bauzasarvindn05: we had a CVE for that14:48
bauzasarvindn05: then we found some problems with the CVE new behaviour14:48
bauzasarvindn05: then we tried to fix that14:48
sahidoh i think i understand what you mean now14:48
bauzasarvindn05: now we're like middleground14:48
arvindn05can you reference the conversation for me? i dont have that history unfortunately14:49
bauzasarvindn05: https://etherpad.openstack.org/p/nova-ptg-rocky L317 and below14:51
bauzaswe discussed about that during the PTG14:51
bauzasas a retrospective if you wish14:51
arvindn05will take a look14:51
*** fghaas has quit IRC14:51
bauzasarvindn05: so, see, before adding *more* to rebuild about scheduler check, maybe let's discuss about the above14:52
arvindn05i am trying to understand what the problem is with going to placement always. Only in case that the current host is at capacity, the current host may not get picked during placement. I guess we can add preferred destinations to the placement api so we can prioritize current host if it matches14:52
bauzasarvindn05: there is nothing really related to placement14:53
*** yamamoto has quit IRC14:53
bauzasarvindn05: it's more about the main issue said in L32114:53
bauzascalling placement for checking traits is just a new scheduler check14:54
*** felipemonteiro has joined #openstack-nova14:54
bauzasbut looks like mriedem isn't around14:54
bauzaswe agreed during the PTG to defer the instance.save() *after* rebuilding and mriedem said he was volunteering thart14:55
arvindn05yup...if you can make it to the nova meeting tommorow....maybe we can discuss in open topics...14:55
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info  https://review.openstack.org/56230414:56
bauzasAFAIR, the nova meeting is EU-compatible this week14:56
bauzasso, yup14:56
arvindn05great :)14:56
arvindn05i am not sure mreidem patch would effect me..14:57
arvindn05i think we first need to get concensus on whether to invoke placement or not....14:57
bauzasarvindn05: that's correct14:57
bauzasarvindn05: but until we fix the original bug, adding more checks to the scheduler means more problems14:58
bauzassee my problem ?14:58
bauzass/my problem/my concern14:58
bauzasjaypipes: efried: oh shit, saw the ML thread14:58
bauzasI thought there was consensus on numbered req groups14:59
bauzasjaypipes: efried: so I wrote my NUMA proposal accordingly14:59
efriedbauzas: Meet us in #openstack-placement14:59
arvindn05bauzas: yup...it does muddy the waters...but still not seeing how it would effect my changes though. how does instance.save() call effect my flow?14:59
*** rajinir has joined #openstack-nova15:00
bauzasarvindn05: your flow is about adding an extra check to the scheduler which eventually calls placement for traits query, right?15:01
openstackgerritsahid proposed openstack/nova master: compute: introduce cpu_shared_set option  https://review.openstack.org/56191215:02
openstackgerritsahid proposed openstack/nova master: libvirt: always pass emulator threads policy  https://review.openstack.org/56191315:02
openstackgerritsahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set  https://review.openstack.org/51089715:02
arvindn05bauzas: kindof. The extra check calls the GET /resource_providers/{hostUUID}/traits to get traits and match them against the image15:02
*** alexchadin has joined #openstack-nova15:02
arvindn05so we dont call placemnt allocation candiadates...but make a query to find the traits of the current host and match them against the image traits15:03
*** zhaochao has quit IRC15:04
arvindn05bauzas: essentially its this check set(image_traits_required).issubset(set(current_host.traits))15:04
*** mchlumsky has quit IRC15:05
bauzasarvindn05: sure, but that restricts the subset of hosts right?15:07
bauzasarvindn05: so a host that'd be accepted in the past wouldn't be now accepted, leading to possibly more NoValidHosts15:07
*** mchlumsky has joined #openstack-nova15:08
arvindn05bauzas: we are not checking agaisnt a subset of hosts though...only the current host.15:08
*** fghaas has joined #openstack-nova15:09
arvindn05we would get novalid hosts only if the current host the instance is residing upon does not have the traits15:09
*** TobbeCN has quit IRC15:09
arvindn05the hostuuid in the RP call basically comes from the resource spec `spec_obj.force_hosts[0], spec_obj.force_nodes[0]`15:10
arvindn05*request spec15:10
bauzasarvindn05: okay here is the thign15:10
bauzasI need to disappear15:11
bauzasbut I'll consider your spec15:11
cfriesenthis may be a weird question, but why does nova have entries for other services in nova.conf rather than using the keystone endpoint list?15:11
bauzasand comment it15:11
* bauzas flies away15:11
arvindn05bauzas: no problem...thanks.15:11
melwittgibi: hey, I'm going to be around early tomorrow, so I'll be able to run the meeting. thanks for the offer :)15:15
*** alexchadin has quit IRC15:16
*** tbachman has quit IRC15:16
*** do3meli has quit IRC15:17
efriedcfriesen: What "entries"?15:23
efriedcfriesen: The auth stuff is necessary on a per-service basis, obviously.15:23
efriedcfriesen: To identify endpoints, we should be able to use the service catalog for everything other than cinder at this point.  I don't think we've fixed up devstack to do that for everything, but I know we've at least hit glance.15:24
gibimelwitt: OK, cool.15:24
fghaaslyarwood: I'll have to drop off shortly for family obligations — I've requested a review from you on https://review.openstack.org/#/c/562145/ which kashyap mentioned earlier; hope that's OK with you. No rush on this at all of course; we do have a workaround in place.15:25
efriedWhere's ol mriedem today?15:25
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky"  https://review.openstack.org/55878315:25
openstackgerritKashyap Chamarthy proposed openstack/nova master: Drop MIN_LIBVIRT_SET_ADMIN_PASSWD  https://review.openstack.org/56231215:25
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD  https://review.openstack.org/56231315:25
gibimelwitt, efried: I've got the approval today to travel to the Vancouver summit so I can help with the onboarding session15:25
efriedcool15:25
*** alexchadin has joined #openstack-nova15:25
melwittgibi: woot, coolness15:26
gibimelwitt, efried: If you have any material about past onboarding sessions then link it to me so I can try thinkig about what I will talk about there15:26
efrieddansmith: Should this return True for the 404 case?  https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L1849-L185515:29
efrieddansmith: I noticed the return value is only checked in this call: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L3799-L380815:30
gibimelwitt: do we plan to have a nova-neutron cross project discussion on the summit?15:30
efrieddansmith: So I guess my question is, can/should we still short-circuit there if the instance is already gone?15:31
melwittgibi: yes, we have this http://forumtopics.openstack.org/cfp/details/12415:31
*** itlinux has quit IRC15:31
gibimelwitt: cool, it even contains the bandwidth topic I wanted to add15:31
melwittgibi: as for past onboarding session material, I'm not sure anyone has anything saved. last time's session was ad hoc, and the time before I wasn't helping with (I was in the room for only the end of it). dansmith, is there any linkable material from the boston onboarding session that gibi could look at?15:32
dansmithefried: well, does 404 there always refer to the allocation? or could it refer to the RP?15:32
dansmithmelwitt: we posted an annotated log of an instance boot in devstack15:33
efrieddansmith: It refers to the consumer being gone.15:33
dansmithmelwitt: that might still exist, but it's probably worth re-generating it at this point, if you're going to do that same schtick15:33
dansmithefried: you mean the allocation yeah/15:34
gibidansmith, melwitt: thanks15:34
melwittgibi: this time I'm going to try to prepare some slides in case we need them. apparently this is the sort of thing the foundation recommends we do or at least have available if the room is total newcomers https://www.slideshare.net/JayBryant2/openstack-cinder-onboarding-education-boston-summit-201715:34
*** lpetrut has joined #openstack-nova15:34
efrieddansmith: Looking at what compute manager does after that, it'll try to get that allocation again, and short-circuit out at *that* point.15:35
dansmithefried: tbh, I would still raise a NotFound there instead of returning a boolean, but it should return False, since the code is using that to determine if the other side of the migration was new or not.. "it's already gone" is not what it's checking for15:35
efrieddansmith: So the harm is only one redundant placement call (and an info log).15:35
dansmithefried: see above.. this is not just trying to nuke the allocation, it needs to know whether it existed or not15:36
cfriesenefried: I'm thinking of the [glance]/api_servers or [neutron]/url15:36
efrieddansmith: Okay.  So the report client method is doing the right thing; it's just that that thing is a bit confusing.15:36
*** fghaas has quit IRC15:37
dansmithefried: well, like I said, if it raised NotFound there and we were catching it, I think it might be a little more obvious15:37
efrieddansmith: The docstring would say:15:37
efried:return: True if the instance's allocations were deleted; False if they were not (which is also the case if the allocations weren't there to begin with)15:37
cfriesenefried: although, I guess those are now optional rather than required15:37
dansmithefried: instead of "did we delete it?" we'd be saying "We deleted it" or "it didn't exist" which is what we're really after15:37
efriedcfriesen: Correct.15:37
efriedcfriesen: deprecated, in the latter case.  In the former case, only use api_servers if you actually *need* multiple endpoints.15:38
*** dklyle has quit IRC15:38
efriedcfriesen: There was work over the past couple of releases to make that happen.  I can find the bp if you like...15:38
efriedcfriesen: https://review.openstack.org/#/q/topic:bp/use-ksa-adapter-for-endpoints15:39
efriedcfriesen: Note the one that's still open.  I would welcome someone jumping in and finishing that one...15:39
cfriesenefried: thanks15:39
*** Guest4072 has quit IRC15:40
openstackgerritMerged openstack/nova master: Fix docs for confirmResize action  https://review.openstack.org/56206215:41
*** yamahata has quit IRC15:42
gibimelwitt: OK. If you need help perparing or reviewing such slideset then I'm happy to help15:42
melwittthanks15:43
openstackgerritNguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build  https://review.openstack.org/55180215:43
*** tbachman has joined #openstack-nova15:43
*** armaan has quit IRC15:43
*** armaan has joined #openstack-nova15:43
*** beagles has left #openstack-nova15:45
*** damien_r has quit IRC15:49
arvindn05jaypipes: wanted to get your thoughts on role of placement during rebuild15:49
openstackgerritNguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build  https://review.openstack.org/55180215:50
openstackgerritNguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build  https://review.openstack.org/55180215:50
*** udesale__ has quit IRC15:52
*** hemna_ has quit IRC15:52
*** alexchadin has quit IRC15:53
*** yamamoto has joined #openstack-nova15:53
*** links has quit IRC15:55
*** Guest4072 has joined #openstack-nova15:55
jaypipesarvindn05: currently deep in the middle of teasing apart the @safe_connect decorator from the placement report client... may be some time until I can get to that :(15:56
*** itlinux has joined #openstack-nova15:58
arvindn05ahh..ok...will bounce the idea off matt for now :)15:59
*** yamamoto has quit IRC16:00
*** ktibi has quit IRC16:02
*** randomhack has quit IRC16:04
*** jackie-truong has joined #openstack-nova16:05
*** harlowja has joined #openstack-nova16:06
*** slaweq has quit IRC16:07
*** phuongnh has joined #openstack-nova16:07
*** r-daneel has joined #openstack-nova16:07
*** efried is now known as efried_rollin16:07
*** slaweq has joined #openstack-nova16:07
*** r-daneel has quit IRC16:08
*** owalsh is now known as owalsh_afk16:08
*** slaweq has quit IRC16:13
*** yamahata has joined #openstack-nova16:17
openstackgerritZhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 7  https://review.openstack.org/56074816:17
*** AlexeyAbashkin has quit IRC16:23
*** cdent has joined #openstack-nova16:24
*** ktibi has joined #openstack-nova16:29
*** elmaciej has quit IRC16:29
*** gyee has joined #openstack-nova16:30
*** lpetrut has quit IRC16:35
*** lucasagomes is now known as lucas-afk16:35
*** sdague has quit IRC16:40
*** gjayavelu has joined #openstack-nova16:41
openstackgerritJay Pipes proposed openstack/nova master: mock utils.execute() in qemu-img unit test  https://review.openstack.org/56233916:42
cfriesenmelwitt: had a thought on bug 1764556.   If we deleted the compute node,  recreated it, and migrated the instance back prior to the upgrade to Pike I think we could end up with the DB in the problem state.  On the other hand, that seems like a pretty unlikely scenario.16:44
openstackbug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Undecided,Fix released] https://launchpad.net/bugs/176455616:44
*** sidx64 has quit IRC16:44
melwittcfriesen: what would be the full sequence there? sorry, I forgot what the original sequence was regarding deleting of the service, migration, re-create service, migrate etc16:47
*** tbachman has quit IRC16:47
cfriesenmelwitt: boot instance, migrate instance, delete compute node, create new compute node with same name, migrate instance to new compute node, upgrade to pike16:48
*** tbachman has joined #openstack-nova16:49
melwittokay. does seem unlikely but I wouldn't be surprised if someone someday would happen to do it16:50
*** sdeath has quit IRC16:51
*** szaher has quit IRC16:53
cfriesenmelwitt: our local solution is to modify the online data migration in pike to put uuids on deleted services16:54
*** bmace has joined #openstack-nova16:54
openstackgerritMerged openstack/os-vif master: Add lower-constraints job  https://review.openstack.org/55605616:54
melwittack16:55
*** yamamoto has joined #openstack-nova16:56
phuongnhhi all, I would like to add one more line in file https://review.openstack.org/#/c/462759/3/os_traits/hw/cpu/x86.py for CPU_FPGA16:56
phuongnhthe link to this cpu type is: https://en.wikipedia.org/wiki/Field-programmable_gate_array16:56
bmacedoes anyone know or have any pointers to why stable/queens global-requirements is pinned to version 9.1 of python-novaclient when 10.1 seems to be the queens version of python-novaclient from what I can see?16:56
phuongnhDo I need to propose a blueprint in launchpad?16:56
melwittjaypipes: what's the procedure for proposing a new cpu type to os-traits? ^16:57
jaypipesphuongnh: that's not a CPU type.16:58
jaypipesmelwitt: the procedure to add any trait is simply to propose a patch to os-traits.16:58
jaypipesmelwitt: the issue here is that CPU_FPGA isn't a trait.16:58
melwittthanks16:59
phuongnhthanks jaypipes16:59
*** bhujay has joined #openstack-nova17:00
*** derekh has quit IRC17:00
*** yamamoto has quit IRC17:00
*** mgoddard has quit IRC17:01
melwittbmace: I think there was a problem with 10.1 not working with queens openstackclient. I have to look to see where/how that was noted (probably in some review somewhere)17:01
*** elmaciej has joined #openstack-nova17:01
bmacemelwitt: if you have more info or it was just certain projects that would be very helpful.  i think without it there are some issues getting multi-attach to work correctly through heat.17:02
*** bhujay has quit IRC17:03
*** szaher has joined #openstack-nova17:04
*** bhujay has joined #openstack-nova17:04
melwittbmace: okay, let me look into it and get back to you. I think mriedem might be able to point to some details about it but he's not around at the moment17:04
bmacemelwitt: sure thing, thanks very much for your time looking into it.17:05
*** amoralej is now known as amoralej|off17:07
*** gyee has quit IRC17:12
*** jpena is now known as jpena|off17:12
*** gjayavelu has quit IRC17:14
*** mdbooth_ has quit IRC17:14
*** elmaciej has quit IRC17:20
*** elmaciej has joined #openstack-nova17:21
*** tssurya has quit IRC17:23
*** sambetts is now known as sambetts|afk17:23
openstackgerritsahid proposed openstack/nova master: compute: introduce cpu_shared_set option  https://review.openstack.org/56191217:24
*** mvk has quit IRC17:25
*** wolverineav has joined #openstack-nova17:30
*** wolverineav has quit IRC17:32
*** wolverineav has joined #openstack-nova17:32
*** mgoddard has joined #openstack-nova17:33
*** Nisha_Agarwal has joined #openstack-nova17:33
*** Kevin_Zheng has quit IRC17:34
*** esberglu has quit IRC17:35
*** mdnadeem has quit IRC17:35
*** phuongnh has quit IRC17:35
*** janki has quit IRC17:37
*** harlowja has quit IRC17:44
*** AlexeyAbashkin has joined #openstack-nova17:46
*** esberglu has joined #openstack-nova17:47
*** Tom-Tom has joined #openstack-nova17:48
cfriesenmelwitt: bmace: I think this might be it: https://review.openstack.org/#/c/538070/17:48
*** itlinux has quit IRC17:48
melwittcfriesen: looks like it, thank you17:49
*** gjayavelu has joined #openstack-nova17:49
bmacecfriesen: melwitt: thanks much.  interesting comment from matt.  i'm pretty sure there are issues with multi-attach through heat not working without the 10.x novaclient17:50
*** sahid has quit IRC17:50
melwittyeah, I'm wondering what in 10.0.0 is required for multi-attach17:51
*** AlexeyAbashkin has quit IRC17:51
*** sridharg has quit IRC17:52
*** mvk has joined #openstack-nova17:52
*** Tom-Tom has quit IRC17:52
bmacemelwitt: i'll chat with the person that was investigating the issue and get some more details.  i think i recall it being at least in part related to the microversion or some such.17:53
*** itlinux has joined #openstack-nova17:54
melwittbmace: okay. yeah, the client itself has a version range it supports, maybe that's the missing piece. the microversion needed for multi-attach might have been added in 10.0.0, so even though there might not be anything else special in the client needed about it, the fact that 9.1.1 can't send the required microversion could break it. we have to look at it17:55
bmacemelwitt: i think that was the issue, and from what i recall the microversion may not have been right until 10.117:55
melwittguh, okay. I'm not immediately sure what we can do about this. I'm not sure if we could just bump the microversion in a 9.1.2 to deal with this17:56
*** yamamoto has joined #openstack-nova17:56
bmaceright, and trying to bump up the global-requirement to 10.1 back in stable queens might be a bigger change than people are comfortable with.17:57
*** dave-mccowan has joined #openstack-nova17:57
*** tbachman has quit IRC17:57
melwittyeah. I don't know the details around why osc couldn't work with 10.1.0 and what it would take to make it work. or the other way around, how we could get 10.1.0 to work with osc17:58
melwittbut that's another angle to look at17:58
bmacefrom what i saw in the review it seemed like the only reason the requirement wasn't approved was because matt said it wasn't needed?17:58
bmaceoh, nevermind, reading through that link to the january mailing list now where there was some reported breakage17:59
melwittyeah17:59
melwittfrom the comments, it sounds like it's fixable, just that it wasn't going to make it in time for queens and also it was thought nothing requires >= 10.0.0 in queens18:01
melwitt*fixing osc wasn't going to make it in time for queens18:01
* bmace nods.18:01
melwittbut if the latter isn't really true, we could fix it, AFAIK18:02
*** yamamoto has quit IRC18:02
*** QianYu_ has joined #openstack-nova18:03
bmacesure, it comes down to how much people actually want multi-attach to work in queens.18:03
*** lpetrut has joined #openstack-nova18:03
melwittfix osc, release osc, bump the queens requirement. I want to check with mriedem to see if that's the best way forward or if there's another solution18:03
bmacesounds good, thanks again.18:03
melwittwe definitely want multi-attach to work in queens. just a matter of how to go about fixing it18:03
melwittthanks for letting us know about this problem18:04
bmacesure thing! :)18:04
*** fragatina has quit IRC18:04
*** QianYu has quit IRC18:05
*** lpetrut has quit IRC18:10
*** pcaruana has quit IRC18:12
*** bhujay has quit IRC18:17
*** elmaciej has quit IRC18:17
*** tbachman has joined #openstack-nova18:18
*** lpetrut has joined #openstack-nova18:18
*** harlowja has joined #openstack-nova18:18
*** tssurya has joined #openstack-nova18:20
*** fghaas has joined #openstack-nova18:21
*** harlowja_ has joined #openstack-nova18:23
*** sdague has joined #openstack-nova18:24
*** Nisha_Agarwal has quit IRC18:24
*** harlowja has quit IRC18:24
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1764556  https://review.openstack.org/56204118:25
openstackbug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Undecided,New] https://launchpad.net/bugs/176455618:25
*** fghaas has quit IRC18:26
*** mlavalle has quit IRC18:35
*** itlinux has quit IRC18:35
*** mlavalle has joined #openstack-nova18:37
*** itlinux has joined #openstack-nova18:40
*** fragatina has joined #openstack-nova18:40
*** fghaas has joined #openstack-nova18:42
*** luzC_ has joined #openstack-nova18:46
*** mgoddard has quit IRC18:56
*** yamamoto has joined #openstack-nova18:57
openstackgerritmelanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells  https://review.openstack.org/54025818:58
melwittdansmith: updated to scope the query only for targeted cell for move operations, else scatter-gather ^18:59
dansmithmelwitt: tab queued19:00
dansmithmelwitt: tssurya: need a cells meeting today? I again have nothing new19:00
tssuryadansmith: me neither19:00
melwitt(was about to ask the same thing) are we having a cells meeting today? that was the only thing I think I wanted to highlight. mriedem isn't around on IRC today19:01
*** felipemonteiro_ has joined #openstack-nova19:01
dansmithsweet19:01
melwittokay, sounds like we're good to skip today. only thing to keep in mind is anything you'd like to get in for r-1 tomorrow19:01
melwittlettuce know19:02
tssuryaalso question about this nova-manage bug -> https://bugs.launchpad.net/nova/+bug/1746530 I marked it as won't fix because it felt more like an oslo_db issue, however please revert the status if there is a possible fix in nova part19:02
openstackLaunchpad bug 1746530 in OpenStack Compute (nova) "nova-manage api_db sync - NotSupportedWarning ['use_tpool']" [Medium,Won't fix]19:02
*** yamamoto has quit IRC19:03
tssuryainfact we would be interested if there is a fix on the nova side for the warning that comes up in Queens all the time19:03
*** felipemonteiro has quit IRC19:05
melwittlooks like that oslo.db fix landed in queens, but we didn't bump our requirements for oslo.db and we can't now that queens is a stable branch. so yeah, would have to do something in nova if anything19:07
tssuryamelwitt: oh I didn't know that fix landed in queens as well,19:08
melwitter, maybe it didn't. sorry, I got confused looking at the nova commit19:08
tssuryayea I guess the oslo_db fix is also only in master19:08
melwittyeah if it was fixed recently then it would be for rocky and even if it was backported, we couldn't bump the requirement on stable to consume it19:09
*** artom has quit IRC19:09
tssuryaI know that its just a warning, but was just wondering if there was a work around by any chance in nova19:10
*** artom has joined #openstack-nova19:10
melwittyeah. it's possible, I dunno off the top of my head19:10
tssuryaoh okay, shall I keep it as won't fix ?19:10
melwittyeah, I would since it's fixed for master and probably not worth trying to do a one-off for queens since it's just a warning IMHO19:11
tssuryayep,19:11
tssuryacool19:11
*** QianYu_ has quit IRC19:11
*** elmaciej has joined #openstack-nova19:12
*** mgoddard has joined #openstack-nova19:13
*** pcaruana has joined #openstack-nova19:13
*** tesseract has quit IRC19:15
*** felipemonteiro__ has joined #openstack-nova19:16
openstackgerritMerged openstack/nova-specs master: Fix typo in GET os-instance-actions url.  https://review.openstack.org/55714319:18
*** felipemonteiro_ has quit IRC19:20
*** pcaruana has quit IRC19:20
*** efried_rollin is now known as efried19:24
*** elmaciej has quit IRC19:26
*** Nel1x has quit IRC19:34
*** imacdonn has quit IRC19:35
*** imacdonn has joined #openstack-nova19:36
*** artom has quit IRC19:38
*** mgoddard has quit IRC19:38
*** felipemonteiro__ has quit IRC19:39
*** felipemonteiro__ has joined #openstack-nova19:40
*** awaugama has quit IRC19:43
cdentmelwitt: Is there still a gate job that runs with postgres? I seem to recall an experimental one or something?19:45
openstackgerritMerged openstack/osc-placement master: Initialize 'result' variable in functional.base  https://review.openstack.org/56000419:48
melwitthm, not sure. looking19:49
cdentthe reason I ask is because I've stumbled upon a postgres-related problem in some of the placement sql and I can't figure out when it started and I'm exploring my options for finding it19:52
*** Nel1x has joined #openstack-nova19:53
melwittI see. having trouble finding where our list of experimental jobs is, I can't remember where things are supposed to be since zuul v319:53
cdentyeah, had the same problem19:54
melwittbut not finding hardly anything related to postgres in openstack-zuul-jobs19:54
melwittor project-config19:55
cdentditto19:56
openstackgerritJackie Truong proposed openstack/nova master: Implement certificate_utils  https://review.openstack.org/47994919:56
openstackgerritJackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths  https://review.openstack.org/56126219:56
cdentso it's either gone or hiding. I know Matt was keen on it existing, but don't know if it stayed aline19:56
openstackgerritJackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620419:56
openstackgerritJackie Truong proposed openstack/nova master: Add certificate validation docs  https://review.openstack.org/56015819:56
cdenti'll try faking it locally19:57
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: libvirt: Block swap volume attempts with encrypted volumes prior to Queens  https://review.openstack.org/56160419:57
*** yamamoto has joined #openstack-nova19:59
*** eharney has quit IRC20:01
*** yamamoto has quit IRC20:04
spotzHey all do you want me to file a bug for nova where the error message in the nova client is typoed? I'm guessing I can track it down and fix it possibly20:05
melwittspotz: sure, you can file a novaclient bug here https://bugs.launchpad.net/python-novaclient20:07
melwitt(assuming the typo is in novaclient)20:08
spotzThanks melwitt, trying to track it down. It might be a little larger then just the message. stable/ocata won't associate a floating IP even though it says after Pike20:08
spotzIf I change the api version as the message suggests it works so, I'lljust bug the typo20:11
melwittcdent: you might be able to use a technique similar to this if you can find a postgres job in another project https://review.openstack.org/#/c/42766820:12
*** wolverin_ has joined #openstack-nova20:14
*** wolverineav has quit IRC20:14
*** avolkov has quit IRC20:14
cdentthanks melwitt20:15
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Modify incorrect debug meaasge in _inject_data  https://review.openstack.org/51995020:15
melwittspotz: okay. link me the bug after you file it, I'm not sure what message this is about. starting in newton, the nova API stopped proxying to neutron so floating IP association has to be done through neutron or openstackclient. so I'm not sure what message is saying "after Pike"20:18
spotzmelwitt https://bugs.launchpad.net/python-novaclient/+bug/176519220:19
openstackLaunchpad bug 1765192 in python-novaclient "floating-ip-associate deprecation message typo" [Undecided,New]20:19
*** bmace has left #openstack-nova20:19
spotzmelwitt: I can't find it in the code base, could it possibly be something added by devstack?20:19
cfriesenspotz: msg_deprecate_net in novaclient/v2/shell.py20:23
spotzcfriesen: And no errors in stable/ocata. Let me peek at Pike and master20:25
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Handle spawning error on unshelving  https://review.openstack.org/54862220:25
melwittthanks cfriesen. found one here https://github.com/openstack/python-novaclient/blob/stable/pike/novaclient/v2/shell.py#L8820:25
*** Nel1x has quit IRC20:25
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Clean up volumes on boot failure  https://review.openstack.org/54508620:26
melwittspotz: ^20:26
spotzmelwitt: I can patch if you like20:27
melwittspotz: so, you used stable/ocata novaclient and saw that message? seems like the message only started in pike20:27
spotzmelwitt: Yeah just realized this is a Pike packstack, not the Ocata devstack I thought I was on:)20:27
spotzI have too many servers running:(20:27
melwittspotz: heh, okay. glad it's just the typo then. yeah, if you want to propose a patch to stable/pike, we can review it20:29
openstackgerritChris Dent proposed openstack/nova master: WIP: Add root provider uuid to group by clause  https://review.openstack.org/56237920:29
*** lpetrut has quit IRC20:30
spotzmelwitt: Only other odd behavior is I had to set the api-version to less then 2.44 but that's part of the warning and I seem to be running 2.5320:30
*** liverpooler has quit IRC20:31
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Clean up volumes on boot failure  https://review.openstack.org/54508620:32
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Detach volumes when VM creation fails  https://review.openstack.org/54508720:32
melwittspotz: the novaclient CLI will default to the latest available nova server API microversion (through version discovery) that it supports. on the nova server, the add/remove fixed/floating IP were removed in microversion 2.44, that's why you have to tell novaclient to request an earlier microversion of the API in order to do it20:32
melwitthttps://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id3920:32
spotzmelwitt: So expected behavior:)20:33
melwittso if you're working with pike, the max microversion in pike is 2.53, so that's what novaclient will default to20:33
melwittyes, because of the CLI defaulting to latest available and supportable20:34
*** mriedem has joined #openstack-nova20:34
*** sar has quit IRC20:34
cfriesenmelwitt: I wonder if there should be a checker for implicitly continued strings that don't have a space either at the end of one line or the beginning of the next.20:35
melwittspotz: the recommended way to associate IPs these days is to use openstackclient as it will make the appropriate calls to neutron (because the association actually happens in neutron)20:35
spotzmelwitt: Good to know, I'm trying to be more thorough about using the openstack CLI and the various project CLIs20:35
spotzmelwitt: And looks like Neutron is looking for a total depcreation of their client in the near future20:35
melwittcfriesen: there could be. but I don't wanna figure out how to write it20:36
cfriesenmelwitt: have you noticed that openstackclient won't let you live-migrate unless you specify a destination host?20:36
melwittspotz: yeah, that's the direction, for openstackclient to be the way to do everything. unfortunately for certain nova operations, the openstackclient doesn't support things properly or requires extra steps whereas the novaclient does one step. we need to fix that stuff :(20:37
openstackgerritChris Dent proposed openstack/nova master: WIP: Add root and parenet provider uuid to group by clause  https://review.openstack.org/56237920:37
melwittcfriesen: I haven't. but I do know that the one-step boot-from-volume in openstackclient was removed relatively recently. and we need to get that fixed too20:38
spotzmelwitt: There's a lot of disparity in the commands as far as functionality. We'll get there eventually, I just try to teach folks as many ways as possible20:38
mriedemdansmith: melwitt: is there a cells meeting happening today?20:40
melwittmriedem: nay, we didn't have anything for today. do you have anything for today?20:41
mriedemtwo things kind of20:41
mriedem1. wanted to know if dansmith is ok with the plan laid out in https://review.openstack.org/#/c/560674/2/nova/api/openstack/compute/services.py@22220:41
openstackgerritChris Dent proposed openstack/nova master: WIP: Add root and parenet provider uuid to group by clause  https://review.openstack.org/56237920:41
mriedemwrt the get_uuids_by_host performance thing20:42
melwittspotz: yeah. one thing to know about openstackclient and nova is that the openstackclient does *not* discover the latest version, so if you want to do anything newer in the nova API, you have to pass along the desired microversion on the command line. that trips people up a lot20:42
mriedem2. was wondering if tssurya was investigating the functional test failures in https://review.openstack.org/#/c/554920/ because they look unrelated but i don't know why they'd fail here20:42
*** liverpooler has joined #openstack-nova20:43
mriedemnvm i think i just figured out #220:43
melwittsweet20:44
mriedemhave to step away for a few days to realize your mistakes20:44
*** yamamoto has joined #openstack-nova20:44
cdentmriedem: while you're back: are you aware of the life or death of the experimental job that once tested nova with postgres? Is that still around somewhere?20:45
mriedemcdent: neutron has one in their experimental queue20:45
mriedemwe don't, but we easily could20:45
*** Nel1x has joined #openstack-nova20:45
cdentsomething in a recent placement change tweaked things: https://review.openstack.org/#/c/562379/20:45
mriedemok, give me a sec and i can wip something up20:46
*** gouthamr has quit IRC20:46
cdentcool, thanks, i'll be back in a few, I need to do dishes etc20:46
*** sidx64 has joined #openstack-nova20:47
*** armaan has quit IRC20:47
*** armaan has joined #openstack-nova20:48
melwittmriedem: earlier today, bmace reported that novaclient 9.1.1 (queens) does not work with multi-attach, making it not possible to use multi-attach with the client in queens. I assumed this is because the max supported microversion in the client is < the multi-attach microversion20:48
*** QianYu has joined #openstack-nova20:49
*** sidx64_ has joined #openstack-nova20:49
mriedemit's in 10.x20:49
mriedemhttps://github.com/openstack/releases/blob/master/deliverables/queens/python-novaclient.yaml#L1620:49
melwittyeah, but we couldn't use 10.x for queens because it didn't work with osc, right?20:49
mriedem10.x isn't in upper-constraints for stable/queens correct20:50
mriedemb/c of osc and horizon20:50
melwittright20:50
mriedembecause i waited until too late to release it20:50
*** sidx64 has quit IRC20:51
openstackgerritMatt Riedemann proposed openstack/nova master: Add neutron-tempest-postgres-full to experimental queue  https://review.openstack.org/56238320:52
mriedemcdent: ^20:52
melwittis there anything we can do to fix it? they ran into a problem trying to use heat with the queens requirements and couldn't do multiattach http://eavesdrop.openstack.org/irclogs/%23openstack-nova/latest.log.html#t2018-04-18T17:50:3220:52
*** fghaas has quit IRC20:52
*** armaan has quit IRC20:53
cdentmriedem: so the job names are global to a zuul instance and you can choose to incorporate whichever ones you like?20:53
mriedemcdent: yeah. e.g. nova-multiattach is defined in the nova repo, but also used in cinder and tempest repos20:54
cdentrad20:54
*** QianYu has quit IRC20:54
mriedemmelwitt: well, if the horizon and osc fixes could be backported and released on stable/queens, you could maybe make the argument that upper-constraints on stable/queens could be bumped to include novaclient 10.x20:54
mriedemotherwise, heat on stable/queens would likely have to monekypatch novaclient's upper version limit when making that specific call20:55
cdentI will poke at this more closely in my morning20:55
mriedemthe horizon fixes might have made it by queens GA, would have to look20:55
*** armaan has joined #openstack-nova20:56
openstackgerritChris Dent proposed openstack/nova master: WIP: Add root and parenet provider uuid to group by clause  https://review.openstack.org/56237920:56
openstackgerritEd Leafe proposed openstack/nova master: Add columns for generation to consumers  https://review.openstack.org/55795820:56
openstackgerritEd Leafe proposed openstack/nova master: Add the Consumer object  https://review.openstack.org/56140620:56
openstackgerritEd Leafe proposed openstack/nova master: Add Consumer objects to Allocations  https://review.openstack.org/56140720:56
openstackgerritEd Leafe proposed openstack/nova master: Add generation checking logic to Consumer  https://review.openstack.org/56140820:56
openstackgerritEd Leafe proposed openstack/nova master: Add a microversion for consumer generation  https://review.openstack.org/56200020:56
melwittmriedem: I see, okay. I'll go look at that, thanks for the pointer20:57
*** dave-mccowan has quit IRC20:59
mriedemmelwitt: https://review.openstack.org/#/c/538070/21:00
mriedemso horizon is ok in queens per https://review.openstack.org/#/c/538472/21:00
mriedemosc was fixed in https://review.openstack.org/#/c/550105/21:01
mriedemi don't know how feasible backporting that is21:01
melwittalright, so osc needs a backport and if that could go smoothly, we could try to bump upper-constraints21:01
melwittyeah, it looks big21:01
mriedemyeah, if there are no API behavior changes in the osc patch then it could maybe be backported21:02
dansmithmriedem: I'm not sure I get the plan from reading those comments21:02
mriedemi think it was just supposed to be a ton of internal plumbing21:02
mriedemdansmith: it was in an earlier patch set21:02
mriedemhttps://review.openstack.org/#/c/560674/1/nova/api/openstack/compute/services.py@22121:03
*** dougshelley66 has quit IRC21:03
mriedemheat could also use KSA and just not rely on novaclient for that one call altogether...21:03
mriedemwhich might be in their best interest long term21:03
*** tbachman has quit IRC21:04
*** ktibi has quit IRC21:05
melwitthow does KSA avoid novaclient?21:05
openstackgerritArvind Nadendla proposed openstack/nova-specs master: Handle rebuild of instance with new image  https://review.openstack.org/56071821:05
openstackgerritChris Dent proposed openstack/nova master: WIP: Add root and parent provider uuid to group by clause  https://review.openstack.org/56237921:07
*** slaweq has joined #openstack-nova21:08
mriedemksa is just the auth wrapper around the request21:08
mriedemwe use ksa exclusively for talking to the placement api for example21:09
mriedemsince there is no python-placementclient21:09
*** bnemec has quit IRC21:09
melwittoh, I see. I need to look more at that in placement. I haven't known how that works21:10
*** eandersson has quit IRC21:10
*** bnemec has joined #openstack-nova21:11
mriedemit's in nova.scheduler.client.report.SchedulerReportClient21:11
arvindn05mriedem: wanted to discuss rebuilding of instance since a lot of comments in the code seem to be added by you on the conductor->scheduler flow21:11
melwittthanks21:11
arvindn05mriedem: https://review.openstack.org/#/c/560718/ is updated with the approach so wanted to check i am on right path21:12
*** pchavva has quit IRC21:12
*** slaweq has quit IRC21:12
openstackgerritmelanie witt proposed openstack/nova stable/pike: only increment disk address unit for scsi devices  https://review.openstack.org/56161121:13
*** eandersson has joined #openstack-nova21:13
arvindn05basically if image properties have changed we will trigger a scheduler -> placement -> filter flow for all cases where image properties havent changed we will use the existing scheduler -> filter flow(skipping placement)21:16
openstackgerritmelanie witt proposed openstack/nova stable/ocata: only increment disk address unit for scsi devices  https://review.openstack.org/56161321:16
cfriesencould one of the cores take a look at https://review.openstack.org/#/c/560690/ ?  It's already been approved by lyarwood and melwitt21:19
mriedemarvindn05: ok, i can't really get into that today, but can try to look it over tomorrow21:20
*** cdent has quit IRC21:22
arvindn05mriedem: sure. I will be bringing this up in open discussion tommorow in the nova meeting FYI21:22
openstackgerritMatt Riedemann proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4)  https://review.openstack.org/51118321:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: RP delete inventories (v1.5)  https://review.openstack.org/51464221:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6)  https://review.openstack.org/51464321:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7)  https://review.openstack.org/51464421:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9)  https://review.openstack.org/51464621:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10)  https://review.openstack.org/51464721:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12)  https://review.openstack.org/54281921:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13)  https://review.openstack.org/54667421:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667521:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804321:26
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832621:26
openstackgerritMerged openstack/nova stable/queens: Avoid showing password in log  https://review.openstack.org/55960321:28
*** mchlumsky has quit IRC21:32
*** dougshelley66 has joined #openstack-nova21:33
*** armaan has quit IRC21:42
*** eharney has joined #openstack-nova21:46
*** Tom-Tom has joined #openstack-nova21:48
*** elmaciej has joined #openstack-nova21:50
*** sidx64_ has quit IRC21:52
openstackgerritMerged openstack/nova stable/ocata: libvirt: mask InjectionInfo.admin_pass  https://review.openstack.org/54831421:53
*** Tom-Tom has quit IRC21:53
*** takashin has joined #openstack-nova21:53
*** hemna_ has joined #openstack-nova21:57
openstackgerritAmy Marrich (spotz) proposed openstack/python-novaclient stable/pike: Correct typo in deprecation for floating-ip commands  https://review.openstack.org/56239221:59
openstackgerritMerged openstack/nova stable/ocata: Fix possible TypeError in VIF.fixed_ips  https://review.openstack.org/53174622:00
*** liverpooler has quit IRC22:04
*** yamamoto_ has joined #openstack-nova22:04
*** felipemonteiro_ has joined #openstack-nova22:06
*** yamamoto has quit IRC22:08
*** felipemonteiro__ has quit IRC22:10
openstackgerritMerged openstack/nova stable/ocata: Proper error handling by _ensure_resource_provider  https://review.openstack.org/52530922:10
*** mlavalle has quit IRC22:22
*** esberglu has quit IRC22:22
*** Guest34655 has quit IRC22:23
*** claudiub|2 has quit IRC22:29
*** felipemonteiro_ has quit IRC22:29
*** felipemonteiro_ has joined #openstack-nova22:29
*** rcernin has joined #openstack-nova22:31
*** itlinux has quit IRC22:32
*** edmondsw has quit IRC22:41
*** sdague has quit IRC22:41
*** mvk has quit IRC22:44
*** AlexeyAbashkin has joined #openstack-nova22:45
*** mvk has joined #openstack-nova22:49
*** AlexeyAbashkin has quit IRC22:50
*** tssurya has quit IRC22:51
*** arvindn0_ has joined #openstack-nova23:02
*** gjayavel_ has joined #openstack-nova23:05
*** gjayavelu has quit IRC23:06
*** slaweq has joined #openstack-nova23:09
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for deleting a compute service  https://review.openstack.org/56062623:12
openstackgerritMatt Riedemann proposed openstack/nova master: Block deleting compute services which are hosting instances  https://review.openstack.org/56067423:12
openstackgerritMatt Riedemann proposed openstack/nova master: Delete allocations from API if nova-compute is down  https://review.openstack.org/56070623:12
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup ugly stub in TestLocalDeleteAllocations  https://review.openstack.org/56199723:12
*** slaweq has quit IRC23:13
*** hongbin_ has quit IRC23:18
*** andreaf has quit IRC23:18
*** andreaf has joined #openstack-nova23:18
*** elmaciej has quit IRC23:19
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for deleting a compute service  https://review.openstack.org/56062623:26
openstackgerritMatt Riedemann proposed openstack/nova master: Block deleting compute services which are hosting instances  https://review.openstack.org/56067423:26
openstackgerritMatt Riedemann proposed openstack/nova master: Delete allocations from API if nova-compute is down  https://review.openstack.org/56070623:26
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup RP and HM records while deleting a compute service.  https://review.openstack.org/55492023:26
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup ugly stub in TestLocalDeleteAllocations  https://review.openstack.org/56199723:26
*** shaohe_feng has quit IRC23:29
*** shaohe_feng has joined #openstack-nova23:30
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Detach volumes when VM creation fails  https://review.openstack.org/54508723:31
*** mriedem has quit IRC23:32
*** felipemonteiro_ has quit IRC23:35
*** Zames has joined #openstack-nova23:41
*** Nel1x has quit IRC23:43
*** Zames has quit IRC23:44
*** eharney has quit IRC23:49
*** QianYu has joined #openstack-nova23:49

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!