Thursday, 2018-04-26

*** r-daneel has quit IRC00:00
openstackgerritMerged openstack/nova master: doc: Adds Hyper-V PCI passthrough details  https://review.openstack.org/51046700:06
openstackgerritMerged openstack/nova master: api-ref: mark block_device_mapping_v2.boot_index as required  https://review.openstack.org/56372800:07
*** takashin has joined #openstack-nova00:12
*** gjayavelu has quit IRC00:16
*** slaweq has joined #openstack-nova00:20
*** slaweq has quit IRC00:25
*** Tom-Tom has joined #openstack-nova00:27
*** Tom-Tom has quit IRC00:32
*** hemna_ has quit IRC00:32
*** hshiina|afk is now known as hshiina00:33
openstackgerritZack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory  https://review.openstack.org/56370400:35
*** Dinesh_Bhor has joined #openstack-nova00:36
*** gjayavelu has joined #openstack-nova00:44
*** yingjun has joined #openstack-nova00:46
*** liverpooler has quit IRC00:49
*** namnh has joined #openstack-nova00:51
*** ssurana has quit IRC00:53
*** yamamoto has quit IRC00:53
*** yamamoto has joined #openstack-nova00:53
*** hiro-kobayashi has joined #openstack-nova00:57
openstackgerritMerged openstack/nova master: xenapi: handle InstanceNotFound in detach_interface()  https://review.openstack.org/56283801:00
*** tiendc has joined #openstack-nova01:09
*** kmalloc has quit IRC01:10
*** hoangcx has joined #openstack-nova01:12
yikun_melwitt, OK, Thanks, I will update soon.01:14
*** moshele has joined #openstack-nova01:16
*** salv-orlando has joined #openstack-nova01:26
*** salv-orl_ has quit IRC01:29
*** zhaochao has joined #openstack-nova01:30
*** yamahata has quit IRC01:32
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: ServerActionsSampleJsonTest refactor  https://review.openstack.org/56181901:33
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API  https://review.openstack.org/55714501:33
*** sapd has joined #openstack-nova01:35
*** phuongnh has joined #openstack-nova01:38
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Support inherit api samples in ApiSampleTestBase  https://review.openstack.org/56254701:41
*** rajinir has quit IRC01:48
openstackgerritMerged openstack/nova master: Fix dropped check for boot_index 0 in _validate_bdm  https://review.openstack.org/56373201:53
*** chyka has joined #openstack-nova01:53
*** chyka has quit IRC01:58
*** gmann_ has joined #openstack-nova02:08
*** jamespage_ has joined #openstack-nova02:08
*** test222___ has joined #openstack-nova02:09
*** beekneemech has joined #openstack-nova02:10
*** breton has joined #openstack-nova02:11
*** _gryf has joined #openstack-nova02:12
*** markmcclain has quit IRC02:12
*** kashyap has quit IRC02:12
*** tv1 has quit IRC02:12
*** breton__ has quit IRC02:12
*** test222__ has quit IRC02:12
*** jamespage has quit IRC02:12
*** gryf has quit IRC02:12
*** mordred has quit IRC02:12
*** zhaochao has quit IRC02:12
*** gmann has quit IRC02:12
*** bnemec has quit IRC02:12
*** oanson has quit IRC02:12
*** ianw_pto has quit IRC02:12
*** jamespage_ is now known as jamespage02:13
*** tv2 has joined #openstack-nova02:13
*** ianw has joined #openstack-nova02:13
*** vladikr has quit IRC02:13
*** mordred has joined #openstack-nova02:13
*** oanson has joined #openstack-nova02:13
*** zhaochao has joined #openstack-nova02:13
*** markmcclain has joined #openstack-nova02:13
*** bandini has quit IRC02:14
*** FL1SK has quit IRC02:15
*** cburgess has quit IRC02:15
*** kashyap has joined #openstack-nova02:15
*** Tom-Tom has joined #openstack-nova02:17
*** sean-k-mooney[m] has quit IRC02:17
*** Tom-Tom has quit IRC02:17
*** bandini has joined #openstack-nova02:17
*** Tom-Tom has joined #openstack-nova02:18
*** cburgess has joined #openstack-nova02:18
*** gongysh has joined #openstack-nova02:19
*** edmondsw has joined #openstack-nova02:20
*** hongbin has joined #openstack-nova02:23
*** dougshelley66 has quit IRC02:24
*** cfriesen has joined #openstack-nova02:31
*** moshele has quit IRC02:47
*** psachin has joined #openstack-nova02:48
*** fragatina has quit IRC02:50
*** fragatina has joined #openstack-nova02:50
*** fragatina has quit IRC02:51
*** fragatina has joined #openstack-nova02:51
*** fragatina has quit IRC02:56
*** namnh_ has joined #openstack-nova03:06
*** namnh has quit IRC03:08
*** awaugama has quit IRC03:09
*** armaan has quit IRC03:16
*** armaan has joined #openstack-nova03:17
*** fragatina has joined #openstack-nova03:21
*** sree has joined #openstack-nova03:21
*** andreas_s has joined #openstack-nova03:23
*** sree has quit IRC03:26
*** dpawlik has joined #openstack-nova03:27
*** jenglisch has quit IRC03:27
*** dr_gogeta86 has quit IRC03:28
*** andreas_s has quit IRC03:28
*** jenglisch has joined #openstack-nova03:29
*** liuzz_ has joined #openstack-nova03:29
*** dr_gogeta86 has joined #openstack-nova03:30
*** dr_gogeta86 has quit IRC03:30
*** dr_gogeta86 has joined #openstack-nova03:30
*** liuzz has quit IRC03:30
*** dpawlik has quit IRC03:32
*** nicolasbock has quit IRC03:33
*** takashin has quit IRC03:34
*** Tom-Tom has quit IRC03:35
*** liuzz_ has quit IRC03:36
*** liuzz has joined #openstack-nova03:38
*** annp has joined #openstack-nova03:39
*** lpetrut has joined #openstack-nova03:44
*** chyka has joined #openstack-nova03:44
*** bkopilov has quit IRC03:45
*** chyka has quit IRC03:49
*** edmondsw has quit IRC03:50
*** janki has joined #openstack-nova03:55
*** chyka has joined #openstack-nova03:56
*** chyka has quit IRC03:57
*** chyka has joined #openstack-nova03:58
*** Dinesh_Bhor has quit IRC04:00
*** hongbin has quit IRC04:01
*** dklyle has joined #openstack-nova04:02
*** Dinesh_Bhor has joined #openstack-nova04:04
*** sree has joined #openstack-nova04:12
*** lpetrut has quit IRC04:13
*** sree_ has joined #openstack-nova04:14
*** sree_ is now known as Guest3552104:14
*** FL1SK has joined #openstack-nova04:15
*** yingjun has quit IRC04:17
*** sree has quit IRC04:17
*** Tom-Tom has joined #openstack-nova04:17
*** edmondsw has joined #openstack-nova04:18
*** edmondsw has quit IRC04:21
*** edmondsw has joined #openstack-nova04:21
*** edmondsw has quit IRC04:21
*** Tom-Tom has quit IRC04:22
*** edmondsw has joined #openstack-nova04:22
*** slaweq has joined #openstack-nova04:22
*** armaan has quit IRC04:22
*** armaan has joined #openstack-nova04:23
*** namnh has joined #openstack-nova04:23
*** namnh_ has quit IRC04:23
*** dklyle has quit IRC04:26
*** edmondsw has quit IRC04:26
*** slaweq has quit IRC04:26
*** lpetrut has joined #openstack-nova04:27
*** dklyle has joined #openstack-nova04:30
*** yikun__ has joined #openstack-nova04:32
*** dklyle has quit IRC04:35
*** yikun_ has quit IRC04:35
*** cfriesen has quit IRC04:38
*** jchhatbar has joined #openstack-nova04:41
*** janki has quit IRC04:44
*** lpetrut has quit IRC04:46
*** ccamacho has quit IRC04:48
*** links has joined #openstack-nova04:53
*** stakeda has joined #openstack-nova04:54
*** yingjun has joined #openstack-nova05:01
*** yamahata has joined #openstack-nova05:02
*** mdnadeem has joined #openstack-nova05:06
*** abhishekk has joined #openstack-nova05:08
*** chyka has quit IRC05:09
*** lpetrut has joined #openstack-nova05:13
*** ssurana has joined #openstack-nova05:14
*** sree has joined #openstack-nova05:18
*** Guest35521 has quit IRC05:19
*** belmoreira has joined #openstack-nova05:19
*** yingjun has quit IRC05:22
*** moshele has joined #openstack-nova05:25
*** moshele has quit IRC05:28
*** ssurana has left #openstack-nova05:29
*** bkopilov has joined #openstack-nova05:29
*** ratailor has joined #openstack-nova05:32
*** lpetrut has quit IRC05:32
*** takashin has joined #openstack-nova05:36
*** Tom-Tom has joined #openstack-nova05:38
*** shaohe_feng has quit IRC05:41
*** Tom-Tom has quit IRC05:43
*** liuzz_ has joined #openstack-nova05:52
*** yingjun has joined #openstack-nova05:52
*** Tom-Tom has joined #openstack-nova05:52
*** trinaths has joined #openstack-nova05:53
*** liuzz has quit IRC05:55
*** sridharg has joined #openstack-nova05:55
*** evin has joined #openstack-nova06:10
*** armaan has quit IRC06:12
*** armaan has joined #openstack-nova06:13
*** armaan has quit IRC06:13
*** armaan has joined #openstack-nova06:13
*** slaweq has joined #openstack-nova06:22
*** gibi has quit IRC06:24
*** tssurya has joined #openstack-nova06:32
*** shaohe_feng has joined #openstack-nova06:32
*** gjayavelu has quit IRC06:32
*** do3meli has joined #openstack-nova06:32
*** kholkina has joined #openstack-nova06:34
*** andreas_s has joined #openstack-nova06:35
*** lpetrut has joined #openstack-nova06:38
*** moshele has joined #openstack-nova06:41
*** tiendc has quit IRC06:43
*** hoangcx has quit IRC06:43
*** tiendc has joined #openstack-nova06:44
*** hoangcx has joined #openstack-nova06:44
*** trinaths has quit IRC06:44
*** sree_ has joined #openstack-nova06:48
*** sree_ is now known as Guest5042606:49
*** sree has quit IRC06:50
*** dpawlik has joined #openstack-nova06:52
openstackgerritZhenyu Zheng proposed openstack/nova master: WIP new migration threads control  https://review.openstack.org/56350506:52
*** amoralej|off is now known as amoralej06:55
*** dpawlik has quit IRC06:56
*** tesseract has joined #openstack-nova06:58
*** dpawlik has joined #openstack-nova06:58
*** tesseract has quit IRC07:00
*** diga has joined #openstack-nova07:01
*** tesseract has joined #openstack-nova07:02
*** ccamacho has joined #openstack-nova07:03
*** udesale has joined #openstack-nova07:03
*** sahid has joined #openstack-nova07:07
*** diga has quit IRC07:08
*** zz9pzza has joined #openstack-nova07:11
*** Dinesh_Bhor has quit IRC07:14
*** yamamoto has quit IRC07:16
*** yamamoto has joined #openstack-nova07:17
*** Dinesh_Bhor has joined #openstack-nova07:17
*** andreas_s has quit IRC07:18
*** andreas_s has joined #openstack-nova07:19
*** ralonsoh has joined #openstack-nova07:20
*** andreas_s_ has joined #openstack-nova07:21
*** andreas__ has joined #openstack-nova07:23
*** andreas_s has quit IRC07:23
*** udesale has quit IRC07:24
*** Guest50426 has quit IRC07:25
*** sree has joined #openstack-nova07:25
*** andreas__ has quit IRC07:26
*** andreas_s_ has quit IRC07:26
*** andreas_s has joined #openstack-nova07:30
*** pcaruana has joined #openstack-nova07:31
*** rcernin has quit IRC07:32
*** jpena|off is now known as jpena07:35
*** alexchadin has joined #openstack-nova07:36
*** sree has quit IRC07:36
*** damien_r has joined #openstack-nova07:38
*** andreas_s has quit IRC07:39
bauzasgood morning folks07:41
*** andreas_s has joined #openstack-nova07:44
*** AlexeyAbashkin has joined #openstack-nova07:45
kashyapMorning07:45
*** mdnadeem has quit IRC07:49
openstackgerritwanghongtao proposed openstack/nova master: Fix the metadata re to match the unicode  https://review.openstack.org/53623607:50
*** threestrands_ has quit IRC07:52
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py  https://review.openstack.org/56441507:53
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (1)  https://review.openstack.org/56441507:54
*** takashin has left #openstack-nova08:00
*** moshele has quit IRC08:01
*** moshele has joined #openstack-nova08:02
*** andreas_s has quit IRC08:02
*** ragiman has joined #openstack-nova08:05
*** ccamacho has quit IRC08:08
*** andreas_s has joined #openstack-nova08:09
*** ccamacho has joined #openstack-nova08:11
*** yamamoto has quit IRC08:11
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647508:13
*** lucas-afk is now known as lucasagomes08:15
*** sree has joined #openstack-nova08:15
*** moshele has quit IRC08:18
*** andreas_s has quit IRC08:18
*** owalsh is now known as owalsh_afk08:20
*** andreas_s has joined #openstack-nova08:23
openstackgerritBalazs Gibizer proposed openstack/nova master: Enhance service restart in functional env  https://review.openstack.org/51255208:23
openstackgerritBalazs Gibizer proposed openstack/nova master: Reproduce bug 1724172 in the functional test env  https://review.openstack.org/51255308:23
openstackbug 1724172 in OpenStack Compute (nova) "Allocation of an evacuated instance is not cleaned on the source host if instance is not defined on the hypervisor" [Medium,In progress] https://launchpad.net/bugs/1724172 - Assigned to Balazs Gibizer (balazs-gibizer)08:23
openstackgerritBalazs Gibizer proposed openstack/nova master: cleanup evacuated instances not on hypervisor  https://review.openstack.org/51262308:23
*** yingjun has quit IRC08:24
*** jaosorior has joined #openstack-nova08:24
*** others has joined #openstack-nova08:24
*** others is now known as gibi08:24
*** mdnadeem has joined #openstack-nova08:25
*** mdbooth has joined #openstack-nova08:25
lyarwoodstephenfin: https://review.openstack.org/#/c/564257/ - would you mind taking a swing at this today?08:26
lyarwoodstephenfin: awkward hard reboot regression we could really use backported to queens08:27
lyarwoodstephenfin: well, the fix not the bug ;)08:27
stephenfinWill do08:27
*** yamamoto has joined #openstack-nova08:27
lyarwoodcheers08:27
*** dtruong_ has joined #openstack-nova08:27
*** dtruong has quit IRC08:28
*** andreas_s has quit IRC08:29
*** andreas_s has joined #openstack-nova08:29
stephenfinlyarwood: aaand done. That was easy08:29
*** andreas_s has quit IRC08:30
*** yingjun has joined #openstack-nova08:31
*** andreas_s_ has joined #openstack-nova08:33
*** derekh has joined #openstack-nova08:34
*** yikun__ has quit IRC08:40
kashyapI know Matt is right "strictly speaking", this isn't a "bug": https://bugs.launchpad.net/nova/+bug/176620808:40
openstackLaunchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,Invalid] - Assigned to Kashyap Chamarthy (kashyapc)08:40
kashyapBut I filed it as a mostly "item tracker"08:40
kashyapMaybe it's a habit out of dealing with Bugzilla08:41
openstackgerritStephen Finucane proposed openstack/nova master: Handle deprecation of inspect.getargspec  https://review.openstack.org/53077708:42
*** rubasov has left #openstack-nova08:42
*** hiro-kobayashi has quit IRC08:46
*** udesale has joined #openstack-nova08:46
*** yingjun has quit IRC08:49
*** hshiina is now known as hshiina|afk08:49
*** sean-k-mooney[m] has joined #openstack-nova08:51
*** udesale has quit IRC08:51
*** yingjun has joined #openstack-nova08:51
*** jmccarthy has joined #openstack-nova08:52
*** jmccarthy has left #openstack-nova08:53
*** maciejjozefczyk has joined #openstack-nova08:54
*** maciejjozefczyk has quit IRC08:55
*** udesale has joined #openstack-nova08:55
*** maciejjozefczyk has joined #openstack-nova08:55
*** maciejjozefczyk1 has joined #openstack-nova08:56
*** maciejjozefczyk has quit IRC08:56
*** Alexey_Abashkin has joined #openstack-nova08:57
*** hoonetorg has quit IRC08:58
*** hiro-kobayashi has joined #openstack-nova09:00
*** AlexeyAbashkin has quit IRC09:01
*** Alexey_Abashkin is now known as AlexeyAbashkin09:01
bauzascores, easy +W for a High priority backport potential bugfix https://review.openstack.org/#/c/564257/09:02
*** sambetts|afk is now known as sambetts09:03
bauzasgibi, stephenfin: ^09:03
stephenfinbauzas: Um, I already did that?09:03
bauzasoops my bad09:03
*** moshele has joined #openstack-nova09:03
stephenfinSome people are so ungrateful09:03
bauzasstephenfin: <309:04
bauzasstephenfin: I can spoil you InfinityWars if you want09:04
stephenfinbauzas: I'm pretty sure I know where you live09:04
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647509:09
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307709:09
*** alexchadin has quit IRC09:12
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647509:14
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307709:14
*** hoonetorg has joined #openstack-nova09:15
bauzasstephenfin: /me is afraid09:15
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647509:17
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307709:17
*** alexchadin has joined #openstack-nova09:18
*** maciejjozefczyk1 has left #openstack-nova09:21
stephenfinHow to confuse oneself: fire up 'tox -e py35' that was recreated a while back and is missing dependencies. Try to figure out what's going on from the wall of text09:27
*** tiendc has quit IRC09:27
*** hoangcx has quit IRC09:27
stephenfinThat's got to be a bug with stestr09:27
*** tiendc has joined #openstack-nova09:28
*** hoangcx has joined #openstack-nova09:28
*** Dinesh_Bhor has quit IRC09:30
*** hiro-kobayashi has quit IRC09:31
*** maciejjozefczyk has joined #openstack-nova09:33
*** derekh_ has joined #openstack-nova09:34
*** dtantsur|afk is now known as dtantsur09:38
*** derekh has quit IRC09:39
*** sean-k-mooney[m] has quit IRC09:40
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path  https://review.openstack.org/49720109:40
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647509:40
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307709:40
*** mtreinish has quit IRC09:41
*** dpawlik has quit IRC09:43
*** mgoddard has joined #openstack-nova09:44
*** dpawlik has joined #openstack-nova09:46
*** mtreinish has joined #openstack-nova09:47
*** jaosorior has quit IRC09:49
lyarwoodstephenfin: thanks again09:49
*** yingjun has quit IRC09:50
*** udesale has quit IRC09:54
*** sdague has joined #openstack-nova09:58
*** dougshelley66 has joined #openstack-nova09:58
*** alexchadin has quit IRC09:59
*** gongysh has quit IRC10:01
*** cdent has joined #openstack-nova10:01
*** Tom-Tom has quit IRC10:02
*** Tom-Tom has joined #openstack-nova10:03
*** armaan has quit IRC10:03
*** Tom-Tom has quit IRC10:04
*** alexchadin has joined #openstack-nova10:04
*** ccamacho has quit IRC10:07
*** ccamacho has joined #openstack-nova10:08
*** ccamacho has quit IRC10:08
openstackgerritMerged openstack/nova master: libvirt: fix hard reboot issue with mdevs  https://review.openstack.org/56425710:08
*** ccamacho has joined #openstack-nova10:08
*** alexchadin has quit IRC10:09
*** hoangcx has quit IRC10:11
*** alexchadin has joined #openstack-nova10:11
*** vivsoni__ has quit IRC10:14
*** namnh has quit IRC10:14
*** alexchadin has quit IRC10:16
*** sean-k-mooney[m] has joined #openstack-nova10:21
stephenfinpatchbomb incoming10:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMANetworkTopology object  https://review.openstack.org/56443910:23
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444010:23
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.networks field  https://review.openstack.org/56444110:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NetworkRequest.physnet, tunnelled  https://review.openstack.org/56444210:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info  https://review.openstack.org/56444310:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Add 'populate_networks' to network API  https://review.openstack.org/56444410:23
openstackgerritStephen Finucane proposed openstack/nova master: network: Add '_get_network_tunnel_status'  https://review.openstack.org/56444510:23
openstackgerritStephen Finucane proposed openstack/nova master: db: Add network_requests to instance_extra  https://review.openstack.org/56444610:23
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add Instance.network_requests  https://review.openstack.org/56444710:24
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests  https://review.openstack.org/56444810:24
openstackgerritStephen Finucane proposed openstack/nova master: rt: Start utilizing Instance.network_requests  https://review.openstack.org/56444910:24
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.requested_networks  https://review.openstack.org/56445010:24
openstackgerritStephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter  https://review.openstack.org/56445110:24
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing network_requests  https://review.openstack.org/56445210:24
*** alexchadin has joined #openstack-nova10:24
*** moshele has quit IRC10:24
stephenfingibi, bauzas, jaypipes, sean-k-mooney[m]: voilà ^10:24
*** alexchadin has quit IRC10:28
*** alexchadin has joined #openstack-nova10:29
openstackgerritLee Yarwood proposed openstack/nova stable/queens: libvirt: fix hard reboot issue with mdevs  https://review.openstack.org/56445410:30
*** abhishekk has quit IRC10:31
*** nicolasbock has joined #openstack-nova10:33
*** alexchadin has quit IRC10:34
*** andreas_s_ has quit IRC10:37
*** udesale has joined #openstack-nova10:37
*** udesale_ has joined #openstack-nova10:45
*** ragiman_ has joined #openstack-nova10:46
*** cdent has quit IRC10:47
*** udesale has quit IRC10:47
*** mdnadeem has quit IRC10:47
*** ragiman has quit IRC10:47
*** mdnadeem has joined #openstack-nova10:48
*** cdent has joined #openstack-nova10:48
*** tbachman has quit IRC10:51
*** yamamoto has quit IRC10:55
*** dpawlik has quit IRC10:59
*** jpena is now known as jpena|lunch11:00
*** threestrands_ has joined #openstack-nova11:01
*** udesale_ has quit IRC11:01
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION  https://review.openstack.org/56398411:02
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC  https://review.openstack.org/56401011:02
openstackgerritKashyap Chamarthy proposed openstack/nova master: Drop BAD_LIBVIRT_NUMA_VERSIONS  https://review.openstack.org/56401111:02
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS  https://review.openstack.org/56401211:02
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD  https://review.openstack.org/56401311:02
kashyapDarn, one more time11:04
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_BLOCK_LM_WITH_VOLUMES_VERSION  https://review.openstack.org/56398411:04
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_NUMA_VERSION_PPC  https://review.openstack.org/56401011:04
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_NUMA_VERSIONS  https://review.openstack.org/56401111:04
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS  https://review.openstack.org/56401211:04
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_PARALLELS_SET_ADMIN_PASSWD  https://review.openstack.org/56401311:04
*** wolverineav has joined #openstack-nova11:04
*** mriedem has joined #openstack-nova11:05
*** alexchadin has joined #openstack-nova11:07
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Fix dropped check for boot_index 0 in _validate_bdm  https://review.openstack.org/56446111:11
*** pchavva has joined #openstack-nova11:11
*** armaan has joined #openstack-nova11:12
*** zz9pzza has quit IRC11:15
*** andreas_s has joined #openstack-nova11:17
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: xenapi: handle InstanceNotFound in detach_interface()  https://review.openstack.org/56446811:18
*** claudiub has joined #openstack-nova11:18
*** udesale_ has joined #openstack-nova11:20
*** ssurana has joined #openstack-nova11:22
*** lucasagomes is now known as lucas-hungry11:24
*** dougshelley66 has quit IRC11:24
*** dpawlik has joined #openstack-nova11:26
*** ssurana has quit IRC11:27
*** maciejjozefczyk has left #openstack-nova11:31
*** purplerbot has quit IRC11:32
*** purplerbot has joined #openstack-nova11:33
*** phuongnh has quit IRC11:33
*** efried has joined #openstack-nova11:35
efriedō/11:36
mriedemefried: ha, that inspect.getargspec warnings filter explodes on anything using mox11:36
efriedmriedem: Niiice.  I didn't look too closely at the failures.11:36
efriedmriedem: We should... get rid of mox.11:37
mriedem- Failed: 71811:38
*** bkopilov has quit IRC11:38
*** rm_work has quit IRC11:39
mriedemhttps://github.com/openstack/mox3/blob/285bc2a1760d460080df79f1ed3b0988cab44f7b/mox3/mox.py#L90911:39
*** dtantsur has quit IRC11:39
mriedemcould patch mox311:39
*** rm_work has joined #openstack-nova11:39
kashyapefried: Hehe, unrelated: I saw you comment about the typo: "Teset" -- https://review.openstack.org/#/c/554438/3/nova/tests/unit/test_utils.py11:40
kashyapefried: That is ... much better than the typo mriedem makes :P11:40
kashyap(You know what it is.)11:40
mriedemtestes11:41
kashyapIndeed, :D11:41
efriedWe *could* patch mox3 -- to use mock.11:41
*** dpawlik has quit IRC11:42
efriedsurely somebody's done that somewhere.11:42
mriedemit looks like it's usage is best effort and optional11:42
mriedemfor inspect.getargspec11:42
mriedemso easy thing would just be adding an "if six.PY3: raise ValueError()"11:42
*** dpawlik has joined #openstack-nova11:42
mriedemanywho, the warnings filter is low priority now11:42
*** chyka has joined #openstack-nova11:42
efriedYeah, totally.  But leaving the patch open is goodness.  And we can revisit it when mox is all gone.11:43
efriedtakashin has been doing a good job there.11:43
*** sean-k-mooney has quit IRC11:44
*** dtantsur has joined #openstack-nova11:44
*** dtantsur has quit IRC11:44
kashyapmriedem: Morning, meta comment: on your remark here, if you say: "I insist", I'll close it as you prefer.  I'm normally mindful of it, but in this instance I'd prefer to track it via LP bug too: https://bugs.launchpad.net/nova/+bug/176620811:44
openstackLaunchpad bug 1766208 in OpenStack Compute (nova) "Lift the restriction on choices for `cpu_model_extra_flags` config attribute" [Undecided,In progress] - Assigned to Kashyap Chamarthy (kashyapc)11:44
kashyapSee my last comment, before you say the phrase :-)11:44
*** dtantsur has joined #openstack-nova11:44
*** dtantsur has quit IRC11:44
*** armaan has quit IRC11:45
kashyapAnd thanks for the actual substantial review, though!  _That_ appreciate the most11:45
mriedemkashyap: i insist11:45
kashyapHeh, okay; moooooooving on.11:46
*** chyka has quit IRC11:47
*** alex_xu has quit IRC11:49
*** dtantsur has joined #openstack-nova11:49
*** dtantsur has quit IRC11:49
*** alex_xu has joined #openstack-nova11:49
*** dtantsur has joined #openstack-nova11:49
*** dtantsur has quit IRC11:49
*** gibi has left #openstack-nova11:50
*** yamamoto has joined #openstack-nova11:50
*** yamamoto has quit IRC11:50
openstackgerritMatt Riedemann proposed openstack/nova master: Handle deprecation of inspect.getargspec  https://review.openstack.org/53077711:51
*** alexchadin has quit IRC11:53
*** alexchadin has joined #openstack-nova11:53
*** dtantsur has joined #openstack-nova11:54
mriedemkashyap: i think you can throw https://review.openstack.org/#/c/563926/ into runways11:54
mriedempickings in the queue are starting to get slim11:54
mriedemfor non-blocked things11:54
*** StevenK has quit IRC11:55
openstackgerritMatt Riedemann proposed openstack/nova master: Add a warnings filter for inspect.getargspec usage  https://review.openstack.org/56434311:55
*** lpetrut_ has joined #openstack-nova11:56
*** StevenK has joined #openstack-nova11:56
*** edmondsw has joined #openstack-nova11:57
*** gibi has joined #openstack-nova11:58
*** edmondsw_ has joined #openstack-nova11:58
*** eharney has joined #openstack-nova11:59
*** damien_r1 has joined #openstack-nova11:59
kashyapmriedem: Hi, will click in a min; making some quick late lunch11:59
*** lpetrut has quit IRC11:59
*** dougshelley66 has joined #openstack-nova11:59
kashyapmriedem: Do you have a (non-null) pointer as to how I can "throw it into a runway"?  If you have to look up too, disregard; I'll get to it12:00
efriedkashyap: https://etherpad.openstack.org/p/nova-runways-rocky12:00
efriedkashyap: Add it at the bottom of the queue (L89)12:00
kashyapGracias12:01
openstackgerritMatt Riedemann proposed openstack/nova master: Remove explicit instance.info_cache.delete()  https://review.openstack.org/56368912:02
*** damien_r has quit IRC12:02
*** edmondsw has quit IRC12:02
kashyapDone12:03
kashyapThanks, Eric!12:03
*** damien_r1 has quit IRC12:04
*** tbachman has joined #openstack-nova12:04
efriedyahyoubetcha12:05
*** zz9pzza has joined #openstack-nova12:08
*** damien_r has joined #openstack-nova12:08
*** edmondsw_ is now known as edmondsw12:08
*** tbachman has quit IRC12:09
*** tiendc has quit IRC12:11
*** tbachman has joined #openstack-nova12:11
*** liuzz has joined #openstack-nova12:14
*** liverpooler has joined #openstack-nova12:15
kashyapHas anyone here tinkered with modifying the 'post-config' for Nova here: http://git.openstack.org/cgit/openstack-dev/devstack/tree/.zuul.yaml#n28112:15
* kashyap posts a DNM DevStack patch to try the Nova config by just appending it below Neutron12:16
mriedemsean-k-mooney[m]: i think https://review.openstack.org/#/c/551370/ is ready for your review. i've confirmed via the live migration CI run in the patch at the top of the series that enables this code that it's hitting _update_vif_xml correctly and linked the logs inline12:17
mriedemsean-k-mooney[m]: however, note the TODO i have in here https://review.openstack.org/#/c/551370/19/nova/virt/libvirt/migration.py@25212:17
mriedemabout vif type changing12:17
mriedemremember we talked about whether we could just ignore the original source_vif and just get a new vif config based on the destination vif to generate the new xml *except* for the pci address?12:18
*** liuzz_ has quit IRC12:18
mriedemthe problem with that is, the source_vif contains the 'network' info in the vif, which is what we need for things like source bridge and target dev12:18
mriedemso can we even have vif type changes for a vif on the same network?12:18
*** yamamoto has joined #openstack-nova12:18
mriedemsean-k-mooney[m]: what i really need is a 2-node deployment where i've got ovs on one node and linuxbridge on another, and live migrate between them, but getting that setup would probably take me awhile - i'm not sure how easily devstack can be configured to tell the subnode to use linuxbridge when the controller is configured for ovs (but doesn't neutron server by default support ML2 ovs and LB agents?)12:19
*** ralonsoh has quit IRC12:23
*** jpena|lunch is now known as jpena12:24
openstackgerritMatthew Booth proposed openstack/nova master: Simplify BDM boot index checking  https://review.openstack.org/56448012:28
kashyapmriedem: Re-thinking on your comment here on the confusion of `diff`s: https://review.openstack.org/#/c/563926/3/nova/tests/unit/virt/libvirt/test_driver.py@641812:28
kashyapmriedem: I added it in the existing way because it keeps all the *_custom* tests together12:28
mdboothmriedem: ^^^ it was a simple bug. Refactor remains worth it imho, as the old code is mind-bendingly obtuse.12:29
kashyapSo someone reading the test along will feel less "cognitive load" if I don't randomly add the new '_multiple_extra_flags' test _after_ '_host_model_with_extra_flags'12:29
*** lucas-hungry is now known as lucasagomes12:30
mriedemmdbooth: ack12:31
*** sq4ind has quit IRC12:32
*** udesale_ has quit IRC12:32
kashyapmriedem: Disregard me until the new iteration passes tests; I still have rearranging to do.12:33
*** dave-mccowan has joined #openstack-nova12:33
*** psachin has quit IRC12:38
*** sq4ind has joined #openstack-nova12:38
openstackgerritMartin Midolesov proposed openstack/nova master: Validating video ram against the max video ram allowed  https://review.openstack.org/56419312:39
*** dave-mcc_ has joined #openstack-nova12:42
*** sq4ind has quit IRC12:42
*** dave-mccowan has quit IRC12:43
*** lyan has joined #openstack-nova12:44
*** lyan is now known as Guest298112:45
*** sq4ind has joined #openstack-nova12:45
openstackgerritMatt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron  https://review.openstack.org/55633412:51
openstackgerritMatt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration  https://review.openstack.org/51542312:51
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration  https://review.openstack.org/55137012:51
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: compute: use port binding extended API during live migration  https://review.openstack.org/55137112:51
openstackgerritMatt Riedemann proposed openstack/nova master: Port binding based on events during live migration  https://review.openstack.org/43487012:51
openstackgerritMatt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate  https://review.openstack.org/52253712:51
*** sahid has quit IRC12:56
*** threestrands_ has quit IRC12:58
*** abhishekk has joined #openstack-nova12:59
*** dpawlik has quit IRC13:00
*** dpawlik has joined #openstack-nova13:01
*** dpawlik has quit IRC13:05
*** bhagyashris has quit IRC13:09
*** edmondsw has quit IRC13:12
*** dpawlik has joined #openstack-nova13:12
*** edmondsw has joined #openstack-nova13:12
*** edmondsw has quit IRC13:16
*** ratailor has quit IRC13:17
*** moshele has joined #openstack-nova13:17
TahvokHey guys! Is it possible to request a release for ocata branch?13:18
*** sahid has joined #openstack-nova13:18
mriedemTahvok: i think that's already up13:18
mriedemhttps://review.openstack.org/#/c/564044/13:18
TahvokOh, nice! I was waiting for some fix to be realesed for a couple of months now :)13:19
TahvokThanks!13:19
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags`  https://review.openstack.org/56392613:21
*** bkopilov has joined #openstack-nova13:23
*** ssurana has joined #openstack-nova13:25
mordredmriedem, jaypipes: how hard/expensive would it be to return a total count of objects for paginated collections?13:26
*** moshele has quit IRC13:26
*** cfriesen has joined #openstack-nova13:27
*** udesale_ has joined #openstack-nova13:27
mordred(just got a question about being able to get a count of items without having to loop over the iterator in the sdk - but we can't do it because pagination... thought I'd check to see how insane it would be to get that added13:27
*** sree_ has joined #openstack-nova13:28
*** sree_ is now known as Guest7713313:29
*** ssurana has quit IRC13:29
cdentmordred: you're after a 'total' bit of metadata?13:29
*** sree has quit IRC13:29
*** mvk has quit IRC13:29
mordredcdent: yah. atleast, maybe I'm after a total bit of metadata13:30
mordredcdent: I'm honestly not sure how muchit's a thing I want - but cost of implementation probably impacts how much I want it :)13:30
cdentI know how that can be. I'm afrait that I'm not familiar with the pagination handling in nova's api, so can't provide any quick and dirty guess13:31
*** alexchadin has quit IRC13:33
jaypipesmordred: it would be a total refactoring of the pagination system.13:34
mordredjaypipes: awesome.then I don't want it13:34
jaypipesha13:34
*** jaosorior has joined #openstack-nova13:35
*** kholkina has quit IRC13:36
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647513:36
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307713:36
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API  https://review.openstack.org/55714513:37
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Support inherit api samples in ApiSampleTestBase  https://review.openstack.org/56254713:38
*** yikun has joined #openstack-nova13:39
*** alexchadin has joined #openstack-nova13:40
*** lpetrut_ has quit IRC13:40
*** Nil_ has quit IRC13:41
*** awaugama has joined #openstack-nova13:42
*** ediardo has quit IRC13:44
*** esberglu has joined #openstack-nova13:46
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler  https://review.openstack.org/56307713:46
*** Guest77133 has quit IRC13:46
*** sree has joined #openstack-nova13:46
*** awaugama has quit IRC13:50
*** sree has quit IRC13:51
mriedemright we don't return a total in the links response today,13:51
mriedemand...we also limit the responses by default to 100013:51
*** yikun_ has joined #openstack-nova13:52
mriedemso based on that, the total we give might not be the actual total number of that resource in the db13:52
mriedemwell, you can pass in limit13:52
*** mlavalle has joined #openstack-nova13:54
*** yikun has quit IRC13:55
openstackgerritMerged openstack/nova master: ServerActionsSampleJsonTest refactor  https://review.openstack.org/56181913:56
*** yamamoto has quit IRC13:56
*** mvk has joined #openstack-nova13:58
openstackgerritMerged openstack/nova stable/queens: libvirt: fix hard reboot issue with mdevs  https://review.openstack.org/56445413:58
*** gokhan has quit IRC14:01
*** Shilpa has joined #openstack-nova14:05
*** tianhui_ has quit IRC14:07
*** tianhui has joined #openstack-nova14:08
*** chyka has joined #openstack-nova14:09
*** sq4ind has quit IRC14:11
*** jaosorior has quit IRC14:12
*** jaosorior has joined #openstack-nova14:13
mriedemkashyap: some small things to cleanup in https://review.openstack.org/#/q/topic:bump_min_libvirt_and_qemu+status:open14:13
*** yingjun has joined #openstack-nova14:13
kashyapmriedem: Will look14:13
*** chyka has quit IRC14:14
kashyapmriedem: On that other patch of `cpu_model_extra_flags` -- while context is fresh, a functional change comment:14:14
*** evin has quit IRC14:14
kashyapmriedem: I was just talking to QEMU / libvirt folks, and I was reminded that allowing / removing extra flags even for 'host-passthrough' mode makes sense.  Why?  An example:14:14
*** hemna_ has joined #openstack-nova14:15
kashyap(a) An Operator might want to explicitly disable nested virt for a Nova guest; then they can remove 'vmx' CPU flag14:15
kashyap(b) QEMU may disable some features unless you explicitly ask for them (like: "invtsc" CPU flag) -- in that case it makes sense to explicitly specify the flag14:16
kashyapI'll add a note in the code for a future patch.14:16
kashyapOr after some thinking ... I might add it in the current patch.  Haven't decided yet14:16
openstackgerritEric Fried proposed openstack/nova master: Add host/hostId to instance action events API  https://review.openstack.org/55714514:17
efriedmriedem: This should be a quick +A (trivial changes since your prior +2) ^14:17
mriedemkashyap: if you're going to drop the model restrictions when dropping the choices restriction, it'd be best to do that in the same change so you're not having to dump multiple feature release notes on people for the same config option14:19
*** edmondsw has joined #openstack-nova14:19
mriedemi will have to take your word on the host-passthrough thing14:19
kashyapYeah, exactly -- I too don't like find it appealing to make the feature release notes muddly14:20
openstackgerritJay Pipes proposed openstack/nova master: support multiple member_of qparams  https://review.openstack.org/56131514:20
kashyapmriedem: You don't have to; I can provide a minimal functional test case you can try yourself on a linux laptop14:20
kashyapWith bare QEMU.  That demonstrates it.14:21
*** rajinir has joined #openstack-nova14:22
kashyapmriedem: Sorry about this one: https://review.openstack.org/#/c/564013/3/nova/virt/libvirt/driver.py14:22
openstackgerritDan Smith proposed openstack/nova master: Make service all-cells min version helper use scatter-gather  https://review.openstack.org/53959014:22
openstackgerritDan Smith proposed openstack/nova master: Make host_manager use scatter-gather and ignore down cells  https://review.openstack.org/53961714:23
kashyapmriedem: Mixed up versions there; it's for Stein14:23
dansmithmriedem: ^14:23
*** tssurya has quit IRC14:23
mriedemkashyap: ok, s/have to//14:24
*** armaan has joined #openstack-nova14:24
*** tssurya has joined #openstack-nova14:30
kashyapmriedem: Here's the first piece of evidence on point (b) I noted above:14:31
kashyapStarting a guest with <cpu mode='host-passthrough' and feature policy='require' name='invtsc' does not provide cpu flag nonstop_tsc in the guest.14:31
kashyapSource: https://bugzilla.redhat.com/show_bug.cgi?id=138412414:31
openstackbugzilla.redhat.com bug 1384124 in qemu-kvm-rhev "cpu flag nonstop_tsc is not present in guest with host-passthrough and feature policy require invtsc" [High,Closed: errata] - Assigned to ehabkost14:31
*** bkopilov has quit IRC14:32
*** dpawlik has quit IRC14:32
*** hongbin_ has joined #openstack-nova14:32
kashyapWhere "invtsc" == Invariant TSC (Time Stamp Counter) -- there's a Wikipedia entry about it14:33
openstackgerritJay Pipes proposed openstack/nova master: support multiple member_of qparams  https://review.openstack.org/56131514:33
*** bkopilov has joined #openstack-nova14:35
openstackgerritEd Leafe proposed openstack/nova master: Add the Consumer object  https://review.openstack.org/56140614:36
*** sq4ind has joined #openstack-nova14:36
*** beekneemech is now known as bnemec14:37
*** sree has joined #openstack-nova14:37
openstackgerritChris Dent proposed openstack/nova master: Use nova.db.api directly  https://review.openstack.org/54326214:38
*** sree has quit IRC14:42
*** pcaruana has quit IRC14:42
*** moshele has joined #openstack-nova14:45
*** alexchadin has quit IRC14:45
*** r-daneel has joined #openstack-nova14:45
*** Nil_ has joined #openstack-nova14:46
*** awaugama has joined #openstack-nova14:46
*** dklyle has joined #openstack-nova14:47
*** udesale_ has quit IRC14:50
*** edmondsw has quit IRC14:54
*** edmondsw has joined #openstack-nova14:54
*** jaosorior has quit IRC14:55
*** yamamoto has joined #openstack-nova14:57
*** armaan has quit IRC14:58
kashyapmriedem: I made a test case, just for you: https://kashyapc.fedorapeople.org/CPU-flags-and-host-passthrough/readme.txt14:58
*** felipemonteiro_ has joined #openstack-nova14:58
*** sq4ind has quit IRC14:58
kashyapTranslation: libvirt's "host-passthrough" in QEMU paralance is '-cpu host'14:58
kashyapIf you have a CirrOS image called 'base.qcow2', then you can run the two scripts in the top-level directory and see the difference14:59
*** felipemonteiro__ has joined #openstack-nova14:59
*** abhishekk has quit IRC15:00
*** r-daneel has quit IRC15:01
*** sree has joined #openstack-nova15:01
kashyap(The above is a proper function test, BTW.)15:01
*** sq4ind has joined #openstack-nova15:02
openstackgerritBalazs Gibizer proposed openstack/nova master: Call generate_image_url only for legacy notification  https://review.openstack.org/56452815:02
*** yamamoto has quit IRC15:03
*** felipemonteiro_ has quit IRC15:03
*** inara has quit IRC15:03
*** jistr|mtgs is now known as jistr15:04
*** moshele has quit IRC15:04
*** sree has quit IRC15:04
*** dpawlik has joined #openstack-nova15:04
*** sree has joined #openstack-nova15:05
-openstackstatus- NOTICE: We've successfully troubleshooted the issue that prevented paste.openstack.org from loading and it's now back online, thank you for your patience.15:06
*** links has quit IRC15:06
*** lei-zh has joined #openstack-nova15:08
jaypipescdent: you may want to have a gander at https://review.openstack.org/#/c/524425/15:08
*** dpawlik has quit IRC15:09
cdentjaypipes: yeah, I'm aware of that15:09
*** lei-zh has quit IRC15:10
*** trinaths has joined #openstack-nova15:10
*** lei-zh has joined #openstack-nova15:10
cdentjaypipes: I still have the same question as ever on that stuff: what policy?15:11
*** lei-zh has quit IRC15:12
*** tbachman has quit IRC15:12
jaypipescdent: sorry, I'm not following you...15:12
*** lei-zh has joined #openstack-nova15:14
*** inara has joined #openstack-nova15:14
cdentjaypipes: I don't see the use case for granular authZ in placement (yet).15:14
mriedemi'm working on that right now, hence my questions about request context in -placement15:18
mriedemcdent: talk to mgagne, he wants granular policy for placement15:18
*** zhaochao has quit IRC15:19
cdentmriedem: I know, and the use cases that they've described do not provide any real protections15:19
cdentif you have access to modify inventories, you have access to breaking the whole thing15:20
*** lei-zh has quit IRC15:22
*** do3meli has quit IRC15:25
*** ssurana has joined #openstack-nova15:26
*** ssurana has quit IRC15:30
*** sree has quit IRC15:33
*** sree has joined #openstack-nova15:33
*** yingjun has quit IRC15:36
*** tbachman has joined #openstack-nova15:37
*** sree has quit IRC15:38
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types  https://review.openstack.org/56421415:39
openstackgerritSylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info  https://review.openstack.org/56230415:39
openstackgerritSylvain Bauza proposed openstack/nova master: WIP: libvirt: Get inventories per pGPU  https://review.openstack.org/56421515:39
*** yamahata has quit IRC15:39
*** gyee has joined #openstack-nova15:40
*** jmlowe has quit IRC15:44
*** tbachman has quit IRC15:45
dansmithmriedem: ugh, I think we have a race for service version checking somewhere in unit tests15:46
dansmitha bad mock or something15:46
dansmithI dunno why it's showing up in this patch15:46
*** ragiman_ has quit IRC15:48
mriedemcdent: you don't have to give access to modify inventories15:51
mriedemyou can give read-only access to non-admins for things like listing providers and showing inventory15:51
mriedemand leave POST and PUT to admin15:51
cdentmriedem: I believe the ask was to be able to control allocation ratios, but maybe I'm not remembering right15:52
wolverineavsahid: stephenfin - just trying to move the conversation here for faster convergence. if you're both online here. there's 2 open queries. (1) get_config_ivs() being removed. i'm not sure where its called or used. (2) #try os-vif codepath works or not - this is true and verified.15:53
stephenfinwolverineav: It's being called here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L653-L65415:55
sahidwolverineav: you have to convert this function to be something like _set_config_VIF15:55
stephenfin(Yes, that _is_ really non-obvious and I plan to fix it in the future)15:55
*** AlexeyAbashkin has quit IRC15:56
sahidso when osvif object is going to be generate, nova will call it based on the vif type15:56
sahidthat to configure domain xml15:56
sahidos-vif never configure domain xml15:56
sahidit's nova responsbility15:56
stephenfinwolverineav: Yeah, what sahid said. The path it will be taking is from [1] to [2]15:56
stephenfin[1] https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L64815:56
stephenfin[2] https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L616-L62215:57
openstackgerritBalazs Gibizer proposed openstack/nova master: DNM: Showing some strange behavior  https://review.openstack.org/56454115:57
*** dpawlik has joined #openstack-nova15:57
stephenfinwolverineav, sahid: We should probably document this once in os-vif we're done. It's certainly a little confusing15:58
*** hamzy has quit IRC15:59
*** yamamoto has joined #openstack-nova15:59
sahidstephenfin: yes it is16:00
*** andreas_s has quit IRC16:01
*** dpawlik has quit IRC16:01
*** jmlowe has joined #openstack-nova16:04
*** andreas_s_ has joined #openstack-nova16:04
wolverineavsahid: stephenfin - got it. so the get_config_* methods i should just simply remove, since they're for legacy path. that fixes (1). and for (2) - i installed the os-vif-bigswitch plugin locally and did some regression to bring up a VM. so that satisfies (2). makes sense?16:04
*** yamamoto has quit IRC16:04
*** jmlowe has quit IRC16:05
wolverineavi mean get_config_ivs related methods to be removed.16:05
stephenfinwolverineav: I think you need to remove the get_config_ivs methods and, if necessary, add some '_set_config_ivs' functions16:05
stephenfinAssuming you need to modify the libvirt XML for those interfaces16:06
stephenfinIf not, you're good16:06
stephenfinI think :)16:06
wolverineavstephenfin: ah. i'm fairly certain we don't. i'll confirm in a bit.16:07
wolverineavthanks sahid stephenfin!16:07
stephenfinwolverineav: Anytime!16:07
*** andreas_s_ has quit IRC16:08
*** owalsh_afk has quit IRC16:11
*** owalsh has joined #openstack-nova16:12
*** jchhatbar has quit IRC16:14
*** fragatina has quit IRC16:18
*** yamahata has joined #openstack-nova16:18
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: General policy sample file for placement  https://review.openstack.org/52442516:18
*** fragatina has joined #openstack-nova16:18
*** sahid has quit IRC16:23
*** owalsh has quit IRC16:24
*** owalsh has joined #openstack-nova16:26
*** moshele has joined #openstack-nova16:29
*** edmondsw has quit IRC16:31
*** ssurana has joined #openstack-nova16:32
*** damien_r has quit IRC16:37
*** mdnadeem has quit IRC16:37
*** amodi has joined #openstack-nova16:37
*** moshele has quit IRC16:37
*** amodi has quit IRC16:38
*** lucasagomes is now known as lucas-afk16:38
*** sridharg has quit IRC16:38
*** felipemonteiro__ has quit IRC16:45
*** felipemonteiro has joined #openstack-nova16:45
*** mdbooth has quit IRC16:45
*** jpena is now known as jpena|off16:45
*** tbachman has joined #openstack-nova16:49
*** germs has joined #openstack-nova16:50
*** germs has quit IRC16:50
*** germs has joined #openstack-nova16:50
*** gjayavelu has joined #openstack-nova16:52
*** lbragstad[m] has joined #openstack-nova16:55
*** ediardo has joined #openstack-nova16:55
*** yamamoto has joined #openstack-nova17:01
*** derekh_ has quit IRC17:03
*** yamamoto has quit IRC17:06
*** lbragstad has quit IRC17:07
*** baoli has joined #openstack-nova17:08
*** dtantsur is now known as dtantsur|afk17:10
*** dklyle has quit IRC17:12
*** hamzy has joined #openstack-nova17:12
*** germs has quit IRC17:13
*** germs has joined #openstack-nova17:13
*** germs has quit IRC17:13
*** germs has joined #openstack-nova17:13
*** ansiwen[q] has joined #openstack-nova17:15
*** baoli has quit IRC17:17
*** germs has quit IRC17:24
*** germs has joined #openstack-nova17:25
*** germs has quit IRC17:25
*** germs has joined #openstack-nova17:25
*** moshele has joined #openstack-nova17:25
*** ssurana has quit IRC17:25
*** germs has quit IRC17:26
*** germs has joined #openstack-nova17:26
*** germs has quit IRC17:26
*** germs has joined #openstack-nova17:26
*** bpoulos has joined #openstack-nova17:31
*** vladikr has joined #openstack-nova17:31
*** belmoreira has quit IRC17:32
*** moshele has quit IRC17:35
openstackgerritEric Fried proposed openstack/nova master: DNM: Showing some strange behavior  https://review.openstack.org/56454117:36
*** esberglu has quit IRC17:42
*** germs has quit IRC17:46
*** amoralej is now known as amoralej|off17:46
*** pchavva has quit IRC17:46
*** germs has joined #openstack-nova17:47
*** germs has quit IRC17:47
*** germs has joined #openstack-nova17:47
*** moshele has joined #openstack-nova17:51
*** felipemonteiro has quit IRC17:51
*** felipemonteiro has joined #openstack-nova17:51
*** tssurya has quit IRC17:53
*** phuongnh has joined #openstack-nova17:55
*** mvk has quit IRC17:55
*** trinaths has quit IRC17:56
*** moshele has quit IRC17:56
*** dpawlik has joined #openstack-nova17:58
*** yamamoto has joined #openstack-nova18:02
*** dpawlik has quit IRC18:03
*** yamamoto has quit IRC18:07
*** pchavva has joined #openstack-nova18:11
*** moshele has joined #openstack-nova18:13
*** fragatin_ has joined #openstack-nova18:15
*** fragatina has quit IRC18:17
*** felipemonteiro_ has joined #openstack-nova18:17
*** moshele has quit IRC18:18
*** felipemonteiro has quit IRC18:21
*** evin has joined #openstack-nova18:21
*** pchavva has quit IRC18:21
*** lbragstad has joined #openstack-nova18:22
*** dklyle has joined #openstack-nova18:22
*** sree has joined #openstack-nova18:23
*** tesseract has quit IRC18:24
*** phuongnh has quit IRC18:24
*** Phuongnh has joined #openstack-nova18:24
*** pchavva has joined #openstack-nova18:26
*** sree has quit IRC18:28
*** mvk has joined #openstack-nova18:31
*** tbachman has quit IRC18:31
*** lpetrut has joined #openstack-nova18:32
*** tssurya has joined #openstack-nova18:32
*** esberglu has joined #openstack-nova18:32
*** moshele has joined #openstack-nova18:40
melwittmriedem: do we know if anyone is working on a spec amendment for https://review.openstack.org/#/q/topic:bp/nova-validate-certificates ?18:43
*** pchavva has quit IRC18:43
cdentnice blog post mriedem18:44
openstackgerritJay Pipes proposed openstack/nova master: placement: resource requests for nested providers  https://review.openstack.org/55452918:45
*** Phuong_nh has joined #openstack-nova18:46
*** moshele has quit IRC18:46
*** harlowja has joined #openstack-nova18:46
*** Phuongnh has quit IRC18:48
dansmithmelwitt: yeah I think she said someone was18:48
mriedemmelwitt: i don't, i asked about it last week18:48
dansmithpeter?18:48
mriedemoh right18:48
mriedemcdent: oh did they post it?18:48
cdentmriedem: showed up in my planet feed18:48
openstackgerritJay Pipes proposed openstack/nova master: support multiple member_of qparams  https://review.openstack.org/56131518:48
melwittdansmith: okay, thanks. I thought I had read something like but second guessed it18:49
*** sree has joined #openstack-nova18:50
*** xinliang has quit IRC18:51
*** mgoddard has quit IRC18:52
*** cdent has quit IRC18:53
*** tbachman has joined #openstack-nova18:55
*** sree has quit IRC18:55
melwittesberglu: I'm going to move your blueprint out of the runway for now since the series is on hold while the CI is worked on. I'll put it at the front of the queue if we expect the CI to be adjusted soon18:57
*** mriedem has quit IRC18:58
esberglumelwitt: ack. Working on the CI pipeline now. I'm hoping to get the scenario testing up early next week, but depends on what kind of roadblocks I hit19:02
*** yamamoto has joined #openstack-nova19:04
melwittesberglu: k, sounds good. keep us posted19:04
*** mriedem has joined #openstack-nova19:06
*** xinliang has joined #openstack-nova19:07
*** Phuong_nh has quit IRC19:07
melwittnaichuans: hi, your blueprint https://blueprints.launchpad.net/nova/+spec/xenapi-image-handler-option-improvement has been added to a review runway. I know you have been active on the patch series already but please let us know if now is not a good time to quickly iterate on the series19:09
*** yamamoto has quit IRC19:09
melwittjianghuaw_: ^19:09
openstackgerritEric Fried proposed openstack/nova master: Base test module/class for functional placement db  https://review.openstack.org/56459019:09
*** sree has joined #openstack-nova19:10
mriedemchannel topic needs to be updated19:11
* melwitt summons dansmith ^19:11
*** ChanServ sets mode: +o dansmith19:12
melwittI was thinking I can probably remove "Add host/hostId to instance action events API" too since it's all approved but maybe should wait until merge19:12
mriedemthat still needs the novaclient change19:13
melwittokay, so keep it until novaclient done19:13
*** dansmith changes topic to "Current runways: xenapi-image-handler-option-improvement / convert-consoles-to-objects / add-host-to-instance-action-events -- This channel is for Nova development. For support of Nova deployments, please use #openstack."19:13
melwittthanks19:14
mriedemi think so, i assume yikun_ can crank out the novaclient change in the next day19:14
*** imacdonn has quit IRC19:14
mriedemif takashin doesn't beat him to it19:14
melwittyeah, that's cool. I wasn't sure if the novaclient thing was going to be off-runway but makes sense to be in-runway19:14
*** imacdonn has joined #openstack-nova19:14
mriedemlike off broadway?19:15
melwittyes exactly like that19:15
mriedemfwiw i always waited to complete a bp when the novaclient changes were done19:15
mriedemhelped remind me to pester people to do them19:15
melwittmakes sense19:15
*** yamahata has quit IRC19:15
*** armaan has joined #openstack-nova19:19
*** fragatin_ has quit IRC19:19
*** sree has quit IRC19:20
*** ssurana has joined #openstack-nova19:21
openstackgerritEric Fried proposed openstack/nova master: Base test module/class for functional placement db  https://review.openstack.org/56459019:21
openstackgerritEric Fried proposed openstack/nova master: Use test_base symbols directly  https://review.openstack.org/56459219:21
*** armaan has quit IRC19:22
*** ediardo has quit IRC19:23
*** eharney has quit IRC19:24
*** sree has joined #openstack-nova19:24
*** sree has quit IRC19:40
*** felipemonteiro__ has joined #openstack-nova19:46
*** felipemonteiro_ has quit IRC19:46
*** moshele has joined #openstack-nova19:58
*** dpawlik has joined #openstack-nova19:59
*** moshele has quit IRC20:01
*** lpetrut has quit IRC20:02
*** dpawlik has quit IRC20:03
*** yamamoto has joined #openstack-nova20:05
*** sree has joined #openstack-nova20:10
*** salv-orl_ has joined #openstack-nova20:11
*** salv-orlando has quit IRC20:11
*** yamamoto has quit IRC20:11
*** Sundar has joined #openstack-nova20:14
*** sree has quit IRC20:14
SundarHi edleafe20:15
mordredefried: I just pushed up an update to https://review.openstack.org/#/c/564299/ that should make it work - and also the previous patch has been updated with a co-authored-by for you :)20:15
efriedmordred: I already +1'd the latter.  Thanks for that.20:16
mordred\o/20:16
mordredefried: thanks, btw, for your excellent tests of the version normalization code - theymade adding testing here much easier20:16
efriedoh, good.20:17
openstackgerritDan Smith proposed openstack/nova master: Make service all-cells min version helper use scatter-gather  https://review.openstack.org/53959020:17
openstackgerritDan Smith proposed openstack/nova master: Make host_manager use scatter-gather and ignore down cells  https://review.openstack.org/53961720:17
jaypipesstephenfin: fyi, finally getting around to reviewing the numa-aware-vswitches spec again.20:18
efriedmordred: Reviewed20:30
*** jmlowe has joined #openstack-nova20:32
*** jmlowe has quit IRC20:32
mordredefried: thank you sir20:33
*** jmlowe has joined #openstack-nova20:33
edleafeSundar: hi - was away from my desk for a bit20:36
Sundaredlefe, NP. Thanks for reaching out20:38
Sundar*edleafe20:39
SundarI am trying to understand why nested Resource Provider support is not usable for Cyborg in Rocky. I understand it is incomplete. But could you give me some details or pointers?20:39
edleafeSundar: here's the current work for getting NRPs in allocation candidates: https://review.openstack.org/#/q/topic:bp/nested-resource-providers-allocation-candidates+(status:open+OR+status:merged)20:41
openstackgerritMerged openstack/nova master: Handle deprecation of inspect.getargspec  https://review.openstack.org/53077720:42
openstackgerritMerged openstack/nova master: Add host/hostId to instance action events API  https://review.openstack.org/55714520:42
edleafeSundar: so until that series is complete, we can't really begin to use it20:43
*** takashin has joined #openstack-nova20:43
*** eharney has joined #openstack-nova20:43
*** evin has quit IRC20:43
*** hamzy has quit IRC20:45
*** ssurana has quit IRC20:47
Sundaredleafe: It seems to me that https://review.openstack.org/#/c/554529/ is the key one. Others relate to nested+shared RPs etc., which may not be of immediate relevance to Cyborg. Does that sound right?20:47
melwitttakashin: hi, have you seen this ML thread about your blueprint bp/list-show-all-server-migration-types http://lists.openstack.org/pipermail/openstack-dev/2018-April/129736.html ? your response would be appreciated20:47
edleafeSundar: it's difficult to say which will impact Cyborg.20:48
edleafeSundar: is 1400 UTC a good time for you? We have the scheduler subteam meeting then20:49
edleafeSundar: on Mondays20:49
melwittnova meeting in 10 minutes20:50
Sundaredleafe: Sure, I can make it at 7 am PDT Monday (I am in SF Bay Area).20:50
takashinmelwitt: Yes.20:50
Sundaredleafe: Also, this series does not seem to address nested RPs + granular resource requests. Where is that tracked?20:51
*** artom has quit IRC20:53
edleafeSundar: hmmm... efried, are you around?20:53
melwitttakashin: okay, cool. if you could send a response to the mailing list, it would be helpful20:53
edleafeSundar: https://wiki.openstack.org/wiki/Meetings/NovaScheduler20:53
takashinmelwitt: okay.20:53
melwittthanks20:53
*** bpoulos has quit IRC20:57
*** belmoreira has joined #openstack-nova20:57
*** dougshelley66 has quit IRC20:57
*** sree has joined #openstack-nova20:58
Sundaredleafe: Thanks!20:59
*** awaugama has quit IRC20:59
*** sapcc-bot has quit IRC21:00
*** sapcc-bot1 has quit IRC21:00
*** sapcc-bot2 has joined #openstack-nova21:00
*** sapcc-bot has joined #openstack-nova21:00
*** arvindn0_ has joined #openstack-nova21:01
efriededleafe: sup21:02
*** fragatina has joined #openstack-nova21:02
*** sree has quit IRC21:03
edleafeCan you shed some light on Sundar's question above about nested RPs and granular requests?21:03
efriedSundar: https://review.openstack.org/#/c/554529/ is the first step in getting nested to work in GET /allocation_candidates.  Without it, nested resource providers aren't usable.21:05
efriedSundar: Granular will happen here: https://review.openstack.org/#/q/topic:bp/granular-resource-requests+status:open21:06
efriedSundar: There are some who think granular should wait until nested is done.  I don't necessarily agree, but we'll see how it shakes out.21:06
Sundarefried: Got it. Thanks!21:07
*** yamamoto has joined #openstack-nova21:07
*** dklyle has quit IRC21:08
*** david-lyle has joined #openstack-nova21:08
*** mgoddard has joined #openstack-nova21:09
openstackgerritLance Bragstad proposed openstack/nova master: Deprecate rule:context_is_admin  https://review.openstack.org/56434921:12
*** yamamoto has quit IRC21:13
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: General policy sample file for placement  https://review.openstack.org/52442521:16
openstackgerritLance Bragstad proposed openstack/nova master: Deprecate rule:context_is_admin  https://review.openstack.org/56434921:17
lbragstadmriedem: two tests are still failing and i'm not quite sure why, it seems unrelated but that's probably not the case ^21:17
lbragstadotherwise - i attempted to simplify it a bit more21:17
Sundaredleafe, efried: Apparently there is a convention/agreement that only the entity that created an RP can publish traits on it (apart from administrators). If so, Cyborg cannot publish traits on the compute node RP, as Nova created it. Can you throw some light on this?21:18
lbragstadi think nova was just using rule:context_is_admin to set is_admin on the context object (which makes sense given the name)21:18
lbragstadso - in order to deprecate it, that has to live somewhere21:18
openstackgerritAdam Harwell proposed openstack/nova stable/pike: Add support for listing hosts in cellv2  https://review.openstack.org/56379721:18
mriedemlbragstad: ok, i've made progress on the granular placement policy patch, except it breaks all placement-related tests now...21:18
lbragstadso i put it in the context object itself21:19
lbragstadand i have it checking for roles set by keystonemiddleware, which should hopefully make things more clear21:19
efriedSundar: That convention/agreement was reversed on Friday in Dublin.21:20
efriedSundar: You can put traits on the compute node RP if you want, as long as they're not traits that compute thinks it "owns".21:20
efriedSundar: but... why would you want to do that?  I would have expected you to want to put traits on the child providers representing your devices.21:21
edleafeefried: I think that the virt driver would be the "creator", since it reports its presence21:22
edleafeCyborg would be an outside agent in this case21:22
*** cfriesen has quit IRC21:22
Sundarefried: Yes, I'd prefer using nRPs. But edleafe has advised us not to rely upon that in Rocky. So, we are creating a backup plan to apply the traits on the compute node RP instead. That in turn has consequences for supporting multiple device types on the same node. It is getting a bit complicated.21:23
efriededleafe: Yes, but we decided to allow merging of traits supplied by outside parties.21:23
efriedSundar: Yeah, the only way you could reliably support multiple device types on the same provider would be via custom resource classes.  And you would then have to unwind that model when nrp is in place.21:23
*** mgoddard has quit IRC21:26
edleafeefried: that was why I advised Cyborg to wait until NRP was working.21:26
edleafeefried: do you have an educated guess as to how much of NRP will make it into Rocky?21:26
efriededleafe: It's really binary: will nrp-in-alloc-cands make it or not.21:27
edleafeefried: ok, then, rephrasing. Do you think NRP will make it into Rocky?21:28
efriededleafe: I can't answer that.  Question for jaypipes21:29
openstackgerritMatt Riedemann proposed openstack/nova master: Fix tox -e docs  https://review.openstack.org/56463121:31
arvindn0_mriedem: quick clarification. conductor.manager.ComputeTaskManager#rebuild_instance already expects and handles no valid host exception21:32
lbragstadmriedem: added a couple comments to your placement patch21:32
mriedemlbragstad: already replied :)21:33
*** dougshelley66 has joined #openstack-nova21:34
mriedemarvindn0_: yes, but that's already after nova-api rpc cast to nova-conductor21:34
mriedemand returned a 202 to the user21:34
arvindn0_https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L95321:34
mriedemarvindn0_: yes i realize21:34
mriedemit's best, imo, if we can fail fast in the API with a 400 - if we're going to do that21:34
mriedemlike trying to rebuild a volume-backed instance with a new image21:34
arvindn0_yea...makes sense21:35
mriedemi don't like the idea of making this an error condition though, because i think that's just technical debt we'll eventually have to fix21:35
mriedembut, like i said, i need to read the rest of the ML thread21:35
arvindn0_when you say the api do you mean near the API entry point?21:36
arvindn0_or still in the conductor API conductor.api.ComputeTaskAPI#rebuild_instance21:36
arvindn0_not sure where the volume backed instance example is21:37
mriedemnova.compute.api.API.rebuild_instance21:37
arvindn0_found it...21:40
arvindn0_my main concern is that if we extend image properties in future to allow granular/forbidden etc traits in future...this logic will need to be updated to handle all of those cases21:41
arvindn0_for granular traits for example, the code we need to add would be as complicated if not more incase we need to handle rebuild taking all those into account21:43
*** belmoreira has quit IRC21:46
jaypipesmriedem: "the great placement schism of 2018" lol :)21:46
mriedemlance stepped right into a big pile of it21:48
*** liverpooler has quit IRC21:58
*** slaweq has quit IRC22:00
*** dpawlik has joined #openstack-nova22:00
* lbragstad tries scrapping it off of his shoes22:02
*** dave-mcc_ has quit IRC22:02
*** arvindn0_ has quit IRC22:03
*** dpawlik has quit IRC22:05
*** burt has quit IRC22:06
*** Tom-Tom has joined #openstack-nova22:06
*** yamamoto has joined #openstack-nova22:09
*** Tom-Tom has quit IRC22:10
*** yamamoto has quit IRC22:14
*** moshele has joined #openstack-nova22:18
*** rcernin has joined #openstack-nova22:23
openstackgerritEric Fried proposed openstack/nova master: WIP: Use helpers in test_resource_provider (func)  https://review.openstack.org/56463822:27
*** felipemonteiro__ has quit IRC22:28
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1)  https://review.openstack.org/56463922:31
*** david-lyle is now known as dklyle22:33
*** tssurya has quit IRC22:34
openstackgerritEd Leafe proposed openstack/nova master: WIP - Add consumer generation checking logic  https://review.openstack.org/56464122:40
*** wolverineav has quit IRC22:42
*** wolverineav has joined #openstack-nova22:43
*** salv-orl_ has quit IRC22:45
*** Guest2981 has quit IRC22:45
*** salv-orlando has joined #openstack-nova22:45
melwittmriedem: did you mention sort of recently that we had a new dbdeadlock somewhere?22:45
mriedemyeah, but now i don't remember if i opened a bug for it, since i only see https://bugs.launchpad.net/nova/+bug/164253722:46
openstackLaunchpad bug 1642537 in OpenStack Compute (nova) "finish_resize fails with DBDeadlock on migrations table" [Undecided,Confirmed]22:46
melwittyeah, same22:47
melwittlogstashin it22:47
*** hongbin_ has quit IRC22:47
*** wolverineav has quit IRC22:47
melwittlots in neutron22:49
*** salv-orlando has quit IRC22:50
mriedemneutron has a lot of retry_on_deadlock decorators22:50
mriedemyou likely need to add: AND tags:"screen-n-cpu.txt"22:50
melwittthanks, was about to try that22:50
melwitt[instance: 6dcdb60e-e398-43c0-a6c0-21d0494185c1] RemoteError: Remote error: DBDeadlock (pymysql.err.InternalError) (1213, u'Deadlock found when trying to get lock; try restarting transaction') [SQL: u'UPDATE migrations SET updated_at=%(updated_at)s, status=%(status)s WHERE migrations.id = %(migrations_id)s'] [parameters: {'status': u'post-migrating', 'migrations_id': 4, 'updated_at': datetime.datetime(2018, 4, 26, 21, 16, 31, 790833)}]22:51
melwitt8 hits in the last 7 days but all on the same change and all in the ceph job. seems weird22:52
melwitthah, it's my backport https://review.openstack.org/56461522:53
*** esberglu has quit IRC22:54
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2)  https://review.openstack.org/56464322:55
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in unit/virt/xenapi/test_vmops.py  https://review.openstack.org/56184622:57
melwitthow does DBDeadlock work if there's already a @oslo_db_api.wrap_db_retry(max_retries=5, retry_on_deadlock=True) decorator on the method? does that mean it tried 5 times already and still got deadlocked?23:07
mriedemi think so23:10
*** yamamoto has joined #openstack-nova23:11
melwittokay. seeing it on an internal bug in queens during the service record report_count update by the conductor service (service heartbeat)23:11
*** Sundar has quit IRC23:11
melwittnot sure what could be clashing with that23:12
*** artom has joined #openstack-nova23:13
*** annp has quit IRC23:13
*** annp has joined #openstack-nova23:14
melwittalthough, this is a scale test situation and seeing messaging timeouts, so maybe it's just all of the various service heartbeats being affected by that23:14
*** dpawlik has joined #openstack-nova23:14
*** wolverineav has joined #openstack-nova23:15
*** yamamoto has quit IRC23:16
mriedemlbragstad: hitting something weird in a test for placement policy; it's a fake token / user context, but it has the 'admin' role in the context, and the rule it's checking against is "role:admin", and these are the policy values from the context:23:17
mriedem{'is_admin_project': True, 'user_id': 'admin', 'service_project_id': None, 'project_domain_id': None, 'service_user_domain_id': None, 'service_user_id': None, 'roles': ['admin'], 'service_project_domain_id': None, 'user_domain_id': None, 'project_id': 'admin', 'service_roles': []}23:18
mriedemis there something obvious i might not be thinking of?23:18
mriedemi figured 'roles': ['admin'] would be enough23:18
*** yamahata has joined #openstack-nova23:18
mriedemoh sorry, it's actually using this rule: rule:admin_api23:19
mriedemwhich is defined as: is_admin:True23:19
*** dpawlik has quit IRC23:19
mriedemand i'm guessing that's failing because the policy creds don't have an is_admin key?23:19
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3)  https://review.openstack.org/56464523:19
*** wolverineav has quit IRC23:20
*** fragatina has quit IRC23:20
mriedemi think this might be due to that goofy context_is_admin middleware thing23:20
mriedemwhich i'm not using in the placement policy stuff23:20
*** fragatina has joined #openstack-nova23:20
mriedemah yes, nova.context.RequestContext overrides to_policy_values to add is_admin https://github.com/openstack/nova/blob/master/nova/context.py#L26323:23
*** fragatina has quit IRC23:23
*** fragatina has joined #openstack-nova23:24
*** moshele has quit IRC23:25
*** wolverineav has joined #openstack-nova23:25
*** Guest84910 has quit IRC23:30
*** yassine has joined #openstack-nova23:31
*** yassine is now known as Guest1200523:31
*** sree has joined #openstack-nova23:36
*** sdague has quit IRC23:36
*** mlavalle has quit IRC23:39
*** claudiub has quit IRC23:39
*** sree has quit IRC23:40
*** wolverineav has quit IRC23:41
*** salv-orlando has joined #openstack-nova23:46
mriedemyup that was it, got it fixed now23:46
mriedemmelwitt: would be sweet if we could build docs locally again https://review.openstack.org/#/c/564631/23:47
melwitt+223:51
*** salv-orlando has quit IRC23:51
mriedemdanke23:54
*** gyee has quit IRC23:54
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Implement granular policy rules for placement  https://review.openstack.org/52442523:55

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