Thursday, 2017-07-20

openstackgerritMichael Still proposed openstack/nova master: Only setup iptables for metadata if using nova-net  https://review.openstack.org/48076500:00
*** lyan has joined #openstack-nova00:01
*** edmondsw has quit IRC00:03
*** owalsh_ has joined #openstack-nova00:07
*** ckopper has joined #openstack-nova00:08
*** yingjun has joined #openstack-nova00:08
*** tbachman_ has quit IRC00:09
*** owalsh- has joined #openstack-nova00:10
*** owalsh has quit IRC00:10
*** esberglu has joined #openstack-nova00:10
*** owalsh has joined #openstack-nova00:11
*** owalsh| has joined #openstack-nova00:12
*** owalsh_ has quit IRC00:13
*** gmann has quit IRC00:13
*** gmann has joined #openstack-nova00:13
*** owalsh_ has joined #openstack-nova00:14
*** owalsh- has quit IRC00:14
*** owalsh- has joined #openstack-nova00:15
mikalDoes anyone here understand how os-vif is tested in the gate? I'm trying to understand how it configures privsep.00:15
mikalI'm hoping that will explain why my privsep change fails with something which looks at lot like a configuration error.00:15
*** esberglu has quit IRC00:15
*** owalsh has quit IRC00:15
*** thorst has joined #openstack-nova00:17
*** owalsh| has quit IRC00:18
*** owalsh_ has quit IRC00:18
*** tbachman has joined #openstack-nova00:19
*** thorst has quit IRC00:19
*** randomhack has joined #openstack-nova00:20
*** owalsh has joined #openstack-nova00:22
*** masber has joined #openstack-nova00:23
*** lucasxu has joined #openstack-nova00:24
*** owalsh- has quit IRC00:25
*** randomhack has quit IRC00:27
*** crushil has quit IRC00:31
*** crushil has joined #openstack-nova00:32
*** zhurong has joined #openstack-nova00:32
*** armax has quit IRC00:35
*** edmondsw has joined #openstack-nova00:37
*** lucasxu has quit IRC00:39
*** fragatina has joined #openstack-nova00:40
*** Shunli has joined #openstack-nova00:40
*** mdnadeem has joined #openstack-nova00:41
*** edmondsw has quit IRC00:41
*** fragatin_ has quit IRC00:42
*** lucasxu has joined #openstack-nova00:43
*** rajathag_ has joined #openstack-nova00:43
*** rajathagasthya has quit IRC00:44
*** fragatina has quit IRC00:45
*** fragatina has joined #openstack-nova00:45
*** fragatina has quit IRC00:46
*** fragatina has joined #openstack-nova00:46
openstackgerritMerged openstack/nova master: Remove mox from nova.tests.unit.api.openstack.compute.test_limits  https://review.openstack.org/47778100:46
*** sbezverk has joined #openstack-nova00:49
*** fragatina has quit IRC00:50
*** thorst has joined #openstack-nova00:50
*** thorst has quit IRC00:50
*** vladikr has joined #openstack-nova00:52
*** fragatina has joined #openstack-nova00:53
*** lucasxu has quit IRC00:53
*** ckopper has quit IRC00:54
*** lucasxu has joined #openstack-nova00:54
*** Nel1x has joined #openstack-nova00:54
openstackgerritMerged openstack/nova master: api-ref: Fix an expand button in os-quota-sets  https://review.openstack.org/48507500:55
*** owalsh has quit IRC00:56
*** owalsh has joined #openstack-nova00:56
*** yamamoto has joined #openstack-nova00:56
*** rajathag_ has quit IRC00:56
*** ckopper has joined #openstack-nova00:56
*** litao__ has joined #openstack-nova00:57
*** fragatina has quit IRC00:57
*** rajathagasthya has joined #openstack-nova00:57
*** tbachman has quit IRC00:59
openstackgerritMerged openstack/nova master: Use uuid for id in os-services API  https://review.openstack.org/46428001:02
openstackgerritMerged openstack/nova master: Handle uuids in os-hypervisors API  https://review.openstack.org/47189901:03
*** chyka has joined #openstack-nova01:04
*** yamamoto has quit IRC01:05
*** rajathagasthya has quit IRC01:05
*** phuongnh has joined #openstack-nova01:06
*** yamamoto has joined #openstack-nova01:06
*** pramodrj07 has quit IRC01:08
*** MasterOfBugs has quit IRC01:08
*** chyka has quit IRC01:09
*** xyang1 has quit IRC01:12
*** yamamoto has quit IRC01:18
*** yamamoto has joined #openstack-nova01:19
*** tbachman has joined #openstack-nova01:20
openstackgerritLars Kellogg-Stedman proposed openstack/nova master: provide interface-scoped nameserver information  https://review.openstack.org/46769901:20
Kevin_Zhengmriedem: should https://review.openstack.org/#/c/459493/ depend on https://review.openstack.org/#/c/484170/ ?01:24
*** yamamoto has quit IRC01:24
*** kiennt has joined #openstack-nova01:25
mriedemKevin_Zheng: either that or we have to set instance.tags = instance_tags in conductor manager schedule_and_build_instance before it sends the instance to the compute01:28
mriedemwhere the instance.create notification happens01:28
mriedemthat would be a workaround until my change01:28
*** edmondsw has joined #openstack-nova01:31
*** lucasxu has quit IRC01:31
*** harlowja has quit IRC01:32
*** yamamoto has joined #openstack-nova01:32
*** Apoorva_ has joined #openstack-nova01:32
openstackgerritMerged openstack/nova master: Count instances to check quota  https://review.openstack.org/41652101:33
*** Daisy_ has joined #openstack-nova01:34
mriedemmelwitt: boing ^01:34
*** edmondsw has quit IRC01:35
*** yamamoto has quit IRC01:35
*** Apoorva has quit IRC01:36
*** Apoorva_ has quit IRC01:37
*** mdnadeem has quit IRC01:41
*** mdnadeem has joined #openstack-nova01:41
*** randomhack has joined #openstack-nova01:41
*** mdnadeem is now known as mdnadeem|afk01:41
*** yamamoto has joined #openstack-nova01:41
*** yamamoto has quit IRC01:44
*** Daisy_ has quit IRC01:45
*** yamamoto has joined #openstack-nova01:45
*** Daisy_ has joined #openstack-nova01:46
*** randomhack has quit IRC01:46
*** mdnadeem|afk has quit IRC01:46
*** yamamoto has quit IRC01:47
*** gbarros has quit IRC01:49
*** yamamoto has joined #openstack-nova01:49
*** Daisy_ has quit IRC01:50
*** yamamoto has quit IRC01:50
*** gbarros has joined #openstack-nova01:51
*** edmondsw has joined #openstack-nova01:51
*** yamamoto has joined #openstack-nova01:52
*** yamamoto has quit IRC01:54
*** edmondsw has quit IRC01:56
*** jmlowe has quit IRC01:58
*** mdnadeem has joined #openstack-nova01:59
*** sree has joined #openstack-nova01:59
*** Daisy_ has joined #openstack-nova02:02
*** armax has joined #openstack-nova02:10
*** edmondsw has joined #openstack-nova02:12
*** yamamoto has joined #openstack-nova02:13
*** jwcroppe has joined #openstack-nova02:13
*** yamamoto has quit IRC02:13
*** swamireddy has joined #openstack-nova02:14
mriedembauzas: https://review.openstack.org/#/c/483565/ is ready for your "Flux de travail" when you get up02:15
*** bkopilov has quit IRC02:16
*** edmondsw has quit IRC02:16
*** yamahata has quit IRC02:17
*** mtanino_ has joined #openstack-nova02:18
*** burt has quit IRC02:18
*** baoli has quit IRC02:20
*** zsli_ has joined #openstack-nova02:21
*** yamamoto has joined #openstack-nova02:22
*** chyka has joined #openstack-nova02:23
*** Shunli has quit IRC02:23
*** burt has joined #openstack-nova02:24
*** yushb has joined #openstack-nova02:24
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48535802:24
*** huanxie has quit IRC02:26
*** chyka has quit IRC02:28
*** huanxie has joined #openstack-nova02:28
melwittmriedem: yeeeeeahhhh \o/02:28
*** yamamoto has quit IRC02:29
*** sree has quit IRC02:31
*** sree has joined #openstack-nova02:32
*** yamamoto has joined #openstack-nova02:34
*** yamamoto has quit IRC02:36
*** sree has quit IRC02:36
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Expect id and disabled_reason in GET /os-services response  https://review.openstack.org/48540902:37
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48541002:37
*** acormier has joined #openstack-nova02:38
*** yamamoto has joined #openstack-nova02:40
*** acormier has quit IRC02:40
*** tuanluong has joined #openstack-nova02:41
*** yamamoto has quit IRC02:41
*** crushil has quit IRC02:44
*** acormier has joined #openstack-nova02:45
*** crushil has joined #openstack-nova02:45
*** s1061123 has quit IRC02:47
*** gbarros has quit IRC02:49
*** thorst has joined #openstack-nova02:51
*** harlowja has joined #openstack-nova02:52
*** edmondsw has joined #openstack-nova02:54
*** acormier has quit IRC02:55
*** sree has joined #openstack-nova02:56
*** acormier has joined #openstack-nova02:56
*** thorst has quit IRC02:56
melwittmriedem: have you ever seen a job failure like this before? trying to figure out if it's me or not http://logs.openstack.org/73/484973/3/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/9ec53dc/console.html#_2017-07-20_01_23_56_83145802:58
mriedemyes i think so02:58
mriedemit's a timeout02:58
*** edmondsw has quit IRC02:58
melwitthm, so entire job timeout02:59
*** acormier has quit IRC02:59
mriedemsomething like that02:59
*** acormier has joined #openstack-nova02:59
*** Sukhdev_ has joined #openstack-nova03:00
*** Sukhdev has quit IRC03:01
*** acormier has quit IRC03:04
*** yamahata has joined #openstack-nova03:07
*** s1061123 has joined #openstack-nova03:08
*** markvoelker has quit IRC03:09
*** edmondsw has joined #openstack-nova03:10
*** edmondsw has quit IRC03:14
TheJuliamriedem: re https://review.openstack.org/#/c/215385 Closer, is_volume is blowing up with a stack trace in the nova compute log. I don't have a good handle on the interior of the block device objects in nova, but I partially wonder if it is the tempest test scenario. A glance/sanity check would be greatly appreciated.03:23
openstackgerritMerged openstack/nova master: remove the very old unmaintained wsgi scripts  https://review.openstack.org/47902703:23
mriedemTheJulia: i think it's that the DriverVolumeBlockDevice object that wraps a BlockDeviceMapping object doesn't proxy the is_volume attribute03:24
TheJuliaThat was kind of what I was suspecting at a glance, but I should have gone to sleep like an hour ago03:25
mriedemhttps://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L23103:25
*** tbachman has quit IRC03:25
*** edmondsw has joined #openstack-nova03:25
TheJuliajoy!03:26
*** sree has quit IRC03:26
*** tbachman has joined #openstack-nova03:26
*** s1061123 has quit IRC03:27
mriedemi'll leave comments03:28
TheJuliais_block_device_mapping ?03:28
TheJuliamriedem: thanks!03:28
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48541003:28
mriedemdone03:28
*** s1061123 has joined #openstack-nova03:29
mriedemit's my fault for asking to use the mappings in the block_device_info list passed into spawn() to save the db roundtrip03:29
mriedemi'd say we workaround it in pike and cleanup it up with a TODO in queens03:30
mriedem*clean it up03:30
*** edmondsw has quit IRC03:30
*** tbachman_ has joined #openstack-nova03:32
*** Nel1x has quit IRC03:33
*** tbachman has quit IRC03:33
*** tbachman_ is now known as tbachman03:33
TheJuliamriedem: I'm always for saving a roundtrip to the database03:39
*** syqian has joined #openstack-nova03:44
*** s1061123_ has joined #openstack-nova03:45
*** s1061123 has quit IRC03:45
openstackgerritMerged openstack/nova master: Remove key_manager.api_class hack  https://review.openstack.org/48450103:45
openstackgerritMerged openstack/nova master: Remove reverts_task_state decorator from swap_volume  https://review.openstack.org/48144703:46
*** rajathagasthya has joined #openstack-nova03:47
*** sree has joined #openstack-nova03:50
*** vishwanathj has quit IRC03:52
*** thorst has joined #openstack-nova03:52
*** tiendc has joined #openstack-nova03:53
*** rajathagasthya has quit IRC03:54
*** sree_ has joined #openstack-nova03:54
*** sree_ is now known as Guest8475903:55
*** Daisy_ has quit IRC03:55
*** sree has quit IRC03:55
*** Daisy_ has joined #openstack-nova03:56
*** thorst has quit IRC03:58
*** cfriesen has quit IRC03:58
*** Guest84759 has quit IRC03:59
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.53 - services and hypervisors using UUIDs  https://review.openstack.org/48543504:01
*** crushil has quit IRC04:02
*** crushil has joined #openstack-nova04:03
openstackgerritMerged openstack/nova master: Use URIOpt  https://review.openstack.org/47921504:04
*** edmondsw has joined #openstack-nova04:05
*** s1061123 has joined #openstack-nova04:05
*** fragatina has joined #openstack-nova04:05
*** s1061123_ has quit IRC04:05
*** fragatina has quit IRC04:07
*** fragatina has joined #openstack-nova04:08
*** udesale has joined #openstack-nova04:08
*** edmondsw has quit IRC04:09
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48541004:10
*** Daisy_ has quit IRC04:12
*** mtanino_ has quit IRC04:12
*** Daisy_ has joined #openstack-nova04:12
*** chyka has joined #openstack-nova04:13
*** mriedem has quit IRC04:14
*** yushb has quit IRC04:15
*** Daisy_ has quit IRC04:17
*** chyka has quit IRC04:17
*** harlowja has quit IRC04:18
*** edmondsw has joined #openstack-nova04:20
*** gyee has quit IRC04:21
openstackgerritJulia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538504:23
*** dave-mccowan has quit IRC04:23
*** yushb has joined #openstack-nova04:24
*** edmondsw has quit IRC04:25
*** s1061123_ has joined #openstack-nova04:25
*** lbragstad has quit IRC04:25
*** s1061123 has quit IRC04:26
*** harlowja has joined #openstack-nova04:30
*** adisky__ has joined #openstack-nova04:33
*** rcernin has joined #openstack-nova04:34
*** edmondsw has joined #openstack-nova04:36
*** Daisy_ has joined #openstack-nova04:40
*** edmondsw has quit IRC04:40
*** tiendc has quit IRC04:41
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659504:43
*** sree has joined #openstack-nova04:44
* TheJulia finally gets local unit test results due to overheating laptop... and cries a little04:44
*** fragatina has quit IRC04:47
*** yushb has quit IRC04:48
*** sree has quit IRC04:49
*** nicolasbock has joined #openstack-nova04:53
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659504:57
*** crushil has quit IRC04:58
*** crushil has joined #openstack-nova04:59
*** yamamoto has joined #openstack-nova04:59
*** kristian__ has joined #openstack-nova05:00
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659505:02
*** kristian__ has quit IRC05:05
*** markvoelker has joined #openstack-nova05:10
*** fragatina has joined #openstack-nova05:11
*** sridharg has joined #openstack-nova05:16
openstackgerritJohn Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter  https://review.openstack.org/46659505:16
openstackgerritJulia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538505:18
*** randomhack has joined #openstack-nova05:23
*** randomhack has quit IRC05:28
*** jjung has joined #openstack-nova05:28
*** yamamoto has quit IRC05:31
*** harlowja has quit IRC05:33
*** armax has quit IRC05:34
*** rcernin has quit IRC05:36
*** trinaths has joined #openstack-nova05:37
*** fragatina has quit IRC05:41
*** edmondsw has joined #openstack-nova05:43
*** gouthamr has quit IRC05:43
*** kiennt has quit IRC05:44
*** jwcroppe has quit IRC05:44
*** markvoelker has quit IRC05:44
*** edmondsw has quit IRC05:47
*** yushb has joined #openstack-nova05:47
*** Daisy_ has quit IRC05:48
*** Daisy_ has joined #openstack-nova05:48
openstackgerritJulia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538505:51
*** Daisy_ has quit IRC05:53
*** thorst has joined #openstack-nova05:53
*** Sukhdev_ has quit IRC05:56
*** psachin has joined #openstack-nova05:57
*** mamandle has joined #openstack-nova05:57
*** thorst has quit IRC05:58
*** kiennt has joined #openstack-nova05:58
*** rcernin has joined #openstack-nova05:59
*** edmondsw has joined #openstack-nova05:59
*** markvoelker has joined #openstack-nova06:01
*** mamandle_ has joined #openstack-nova06:03
*** edmondsw has quit IRC06:03
*** moshele has joined #openstack-nova06:04
*** mamandle has quit IRC06:04
*** mamandle_ is now known as mamandle06:04
*** yamamoto has joined #openstack-nova06:05
*** Daisy_ has joined #openstack-nova06:05
*** markvoelker has quit IRC06:06
*** tuanluong has quit IRC06:09
*** Oku_OS-away is now known as Oku_OS06:09
*** chyka has joined #openstack-nova06:10
*** chyka has quit IRC06:14
*** andreas_s has joined #openstack-nova06:15
*** lpetrut_ has joined #openstack-nova06:18
*** sree has joined #openstack-nova06:20
*** yamamoto has quit IRC06:23
*** sree has quit IRC06:24
*** tuanluong has joined #openstack-nova06:27
*** FL1SK has quit IRC06:35
openstackgerritiswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values  https://review.openstack.org/48146506:38
*** edmondsw has joined #openstack-nova06:46
*** kaisers has quit IRC06:46
openstackgerritBéla Vancsics proposed openstack/nova master: ransform instance.resize_confirm notification  https://review.openstack.org/48255706:50
*** edmondsw has quit IRC06:51
openstackgerritBéla Vancsics proposed openstack/nova master: Transform instance.resize_confirm notification  https://review.openstack.org/48255706:51
*** kaisers has joined #openstack-nova06:52
*** priteau has joined #openstack-nova06:52
*** Daisy__ has joined #openstack-nova06:52
*** Daisy___ has joined #openstack-nova06:54
*** Daisy__ has quit IRC06:54
*** Daisy_ has quit IRC06:55
*** Daisy_ has joined #openstack-nova07:00
*** Daisy_ has quit IRC07:00
*** Daisy_ has joined #openstack-nova07:01
*** markvoelker has joined #openstack-nova07:02
*** Daisy___ has quit IRC07:03
openstackgerritMoshe Levi proposed openstack/nova master: hardware offload support for openvswitch  https://review.openstack.org/39826507:03
*** itlinux has quit IRC07:04
*** Daisy__ has joined #openstack-nova07:05
*** sree has joined #openstack-nova07:06
*** Daisy_ has quit IRC07:08
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949307:11
*** sree has quit IRC07:11
*** crushil has quit IRC07:12
*** crushil has joined #openstack-nova07:12
openstackgerritjichenjc proposed openstack/nova master: Check root disk size when doing resize in API  https://review.openstack.org/48466407:15
*** crushil has quit IRC07:15
*** yamamoto has joined #openstack-nova07:16
*** crushil has joined #openstack-nova07:16
*** crushil has quit IRC07:17
*** crushil has joined #openstack-nova07:18
*** zsli__ has joined #openstack-nova07:18
openstackgerritBéla Vancsics proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402307:18
*** avolkov has quit IRC07:19
*** tesseract has joined #openstack-nova07:19
*** zsli_ has quit IRC07:21
*** yamamoto has quit IRC07:25
*** edmondsw has joined #openstack-nova07:30
*** edmondsw has quit IRC07:34
*** markvoelker has quit IRC07:36
*** lpetrut_ has quit IRC07:37
*** kristian__ has joined #openstack-nova07:39
*** kristian__ has quit IRC07:40
*** crushil has quit IRC07:40
*** kristian__ has joined #openstack-nova07:40
*** alexchadin has joined #openstack-nova07:40
*** aarefiev_afk is now known as aarefiev07:43
*** damien_r has joined #openstack-nova07:44
*** kristian__ has quit IRC07:44
openstackgerritSylvain Bauza proposed openstack/nova master: Accept any scheduler driver entrypoint  https://review.openstack.org/48482807:45
openstackgerritJan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement  https://review.openstack.org/48345907:50
*** edmondsw has joined #openstack-nova07:50
*** thorst has joined #openstack-nova07:54
*** edmondsw has quit IRC07:55
*** thorst has quit IRC07:58
*** chyka has joined #openstack-nova07:59
*** ralonsoh has joined #openstack-nova08:00
*** sshwarts has joined #openstack-nova08:00
*** edmondsw has joined #openstack-nova08:03
*** chyka has quit IRC08:04
*** clarkb has quit IRC08:04
*** clarkb has joined #openstack-nova08:05
*** rmart04 has joined #openstack-nova08:06
*** edmondsw has quit IRC08:07
*** sree has joined #openstack-nova08:08
*** nicolasbock_ has joined #openstack-nova08:12
*** nicolasbock has quit IRC08:12
*** sree has quit IRC08:12
*** gcb has joined #openstack-nova08:21
*** lucas-afk is now known as lucasgomes08:22
*** lpetrut_ has joined #openstack-nova08:25
*** Daisy_ has joined #openstack-nova08:30
*** priteau has quit IRC08:31
*** abhishekk has joined #openstack-nova08:31
*** Daisy___ has joined #openstack-nova08:33
*** markvoelker has joined #openstack-nova08:33
*** Daisy__ has quit IRC08:34
*** FL1SK has joined #openstack-nova08:35
*** Daisy_ has quit IRC08:36
*** edmondsw has joined #openstack-nova08:39
*** fragatina has joined #openstack-nova08:40
*** lpetrut_ has quit IRC08:40
*** edmondsw has quit IRC08:43
*** Daisy_ has joined #openstack-nova08:47
*** udesale__ has joined #openstack-nova08:49
*** edmondsw has joined #openstack-nova08:51
*** Daisy___ has quit IRC08:51
*** VAhl has quit IRC08:52
*** udesale has quit IRC08:52
*** alexchadin has quit IRC08:52
*** edmondsw has quit IRC08:55
openstackgerritBéla Vancsics proposed openstack/nova master: Transform servergroup.delete notification  https://review.openstack.org/48390208:57
*** avolkov has joined #openstack-nova08:59
*** nicolasbock_ has quit IRC09:01
*** nicolasbock has joined #openstack-nova09:01
*** nicolasbock has quit IRC09:01
*** markvoelker has quit IRC09:07
*** avolkov` has joined #openstack-nova09:08
*** hoonetorg has joined #openstack-nova09:10
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949309:13
*** sambetts|afk is now known as sambetts09:16
*** randomhack has joined #openstack-nova09:24
*** randomhack has quit IRC09:28
openstackgerritStephen Finucane proposed openstack/python-novaclient master: doc: Remove Makefile  https://review.openstack.org/48009609:29
*** zsli__ has quit IRC09:30
openstackgerritStephen Finucane proposed openstack/python-novaclient master: Fix up release note  https://review.openstack.org/47044609:30
*** yingjun has quit IRC09:31
*** fragatina has quit IRC09:34
*** takedakn has joined #openstack-nova09:36
*** mvk has quit IRC09:43
*** trungnv has quit IRC09:44
*** takedakn has quit IRC09:46
openstackgerritsahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs  https://review.openstack.org/48552209:46
*** elod has quit IRC09:48
*** cdent has joined #openstack-nova09:49
*** elod has joined #openstack-nova09:50
openstackgerritBalazs Gibizer proposed openstack/nova master: add status field to the instance notifications  https://review.openstack.org/48552509:50
*** yamahata has quit IRC09:51
*** thorst has joined #openstack-nova09:55
*** udesale has joined #openstack-nova09:58
*** acormier has joined #openstack-nova10:00
*** mamandle has quit IRC10:00
*** udesale__ has quit IRC10:00
*** jjung has quit IRC10:01
*** dtantsur|afk is now known as dtantsur10:01
*** yushb has quit IRC10:02
*** tuanluong has quit IRC10:03
*** markvoelker has joined #openstack-nova10:03
*** acormier has quit IRC10:04
*** sree has joined #openstack-nova10:06
*** yushb has joined #openstack-nova10:10
*** mvk has joined #openstack-nova10:10
*** sree has quit IRC10:11
*** damien_r has quit IRC10:12
*** thorst has quit IRC10:12
*** yushb has quit IRC10:12
*** jjung has joined #openstack-nova10:14
*** mnestratov has joined #openstack-nova10:15
*** huanxie has quit IRC10:19
*** FL1SK has quit IRC10:21
*** edmondsw has joined #openstack-nova10:24
*** Daisy_ has quit IRC10:25
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying  https://review.openstack.org/34539610:27
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539710:27
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539810:27
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539910:27
*** ansiwen has quit IRC10:28
*** edmondsw has quit IRC10:28
*** xinliang has quit IRC10:29
*** kiennt has quit IRC10:30
gibibauzas, cdent: I think10:31
*** ociuhandu_ has quit IRC10:31
gibibauzas, cdent: I think I found a bug in https://review.openstack.org/#/c/483565/4/nova/scheduler/manager.py@146 but it is already on the gate10:31
cdenttherefore you are10:31
gibisometimes I don't want to be :)10:32
*** ansiwen has joined #openstack-nova10:32
gibihonestly I don't want to stop the claim patches I would fix this in a follow up instead10:33
cdentyeah agree10:33
gibithe effect is small as the exception will stop the instance boot and that is correct as there is no resources left for the instance10:34
cdentdo you have a replication scenario already?10:34
*** ansiwen has quit IRC10:34
gibinot in code yet10:34
*** sdague has joined #openstack-nova10:34
cdentwere you experimenting and then boom?10:34
cdentwe need more of that10:34
gibiyes, in devstack. I consumed all my MAGIC then the next boot produced the stacktrace10:35
gibifirst I was happy as the instance didn't booted and that is correct10:35
gibibut then I checked the logs and the stacktrace was starange10:35
gibistrange10:35
gibiI can try to put together a test for this scenario10:36
gibibut this time the problem is not on the placement side but on the scheduler/manager side10:36
*** markvoelker has quit IRC10:37
*** GnomeKing has joined #openstack-nova10:37
cdentgibi: I reckon a bug report describing what you just said is sufficient10:38
cdent“I consumed all my MAGIC” would make a good t-shirt or epitath10:39
gibiOK I file a bug10:40
cdentI’ve added a link to this talk to the review, for visibility10:41
gibiand I made a note about the t-shirt10:41
*** xinliang has joined #openstack-nova10:41
cdentI’m _really_ glad you are tracking this stuff so closely. I’d been planning to, but haven’t had the time. I think there are going to be lots of edge cases and other weirdnesses. Which is all fine. We’ll make it happy soon enough.10:42
*** ansiwen has joined #openstack-nova10:43
openstackgerritStephen Finucane proposed openstack/os-vif master: doc: Remove cruft from releasenotes conf.py  https://review.openstack.org/48009210:43
abhishekkhi can any one confirm that uwsgi will only be used for development purpose or it can be used in production as well?10:44
*** smatzek has joined #openstack-nova10:46
*** mdnadeem has quit IRC10:47
gibibauzas, cdent: bug has been reported https://bugs.launchpad.net/nova/+bug/170544610:47
openstackLaunchpad bug 1705446 in OpenStack Compute (nova) "filter scheduler raises TypeError: argument of type 'NoneType' is not iterable when placement returns no allocation candidates" [Undecided,New]10:47
cdentexcellent thanks10:48
stephenfinabhishekk: I can't confirm it, but I'm pretty sure any WSGI server can be used, yes10:48
gibicdent: yes, we just figure out that we might need to use the custom resource feature to build up a solution so I started building a PoC out of what nova has10:48
abhishekkstephenfin: ok, thank you10:49
cdentabhishekk: the idea with the new wsgi script is to make it so it can be run with any wsgi server10:49
cdentgibi: I’ve been hearing that quite a lot: custom resource classes are going to allow a lot of interesting flexibility for other hypervisors10:50
abhishekkcdent: ok, got it, thank you :)10:51
gibicdent: I think it will be a buzzword soon in the telco world10:51
cdentgibi: I hope we don’t run into too many people who are trying to do what nested resource providers is supposed to provide, with custom resource classes instead10:51
*** edmondsw has joined #openstack-nova10:52
*** hoonetorg has quit IRC10:52
cdentgibi: it won’t be the end of the world or anything, but it will be bad for interop10:52
cdent(which I guess in the telco world is not a huge concern, but still)10:52
openstackgerritMerged openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure  https://review.openstack.org/48498810:52
*** ociuhandu_ has joined #openstack-nova10:53
gibicdent: does nested resources support different provider for the parent and the child resource?10:54
gibicdent: I mean my MAGIC is provided by the compute node so in theory this can be nested under the compute resource somehow but the logic of providing MAGIC does not fitt nova so I need a separate provider10:55
*** edmondsw has quit IRC10:56
openstackgerritDao Cong Tien proposed openstack/nova master: Fix invalid getattr/setattr for DriverBlockDevice  https://review.openstack.org/48555010:59
*** gszasz has joined #openstack-nova10:59
openstackgerritMerged openstack/nova master: add dict of allocation requests to select_dests()  https://review.openstack.org/48356511:00
*** phuongnh has quit IRC11:00
cdentgibi: yeah, one example is compute node provider -> pci devices that provide physical or virtual functions (and have N of them)11:01
cdentand mix numa cells into that11:01
*** takashin has joined #openstack-nova11:02
gibicdent: sounds cool.11:03
*** mkucia has joined #openstack-nova11:03
cdentit is, but you know how the sql is already sort of mind blowing? now add a graph of providers ot that...11:03
openstackgerritStephen Finucane proposed openstack/nova master: remove un-necessary update() in _init_compute_node of rt  https://review.openstack.org/48350611:04
*** edmondsw has joined #openstack-nova11:04
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove cpuset_reserved from __init__ of InstanceNUMATopology  https://review.openstack.org/46603011:06
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/48555311:06
openstackgerritStephen Finucane proposed openstack/nova master: objects: remove related pinning from __init__ of InstanceNUMATopology  https://review.openstack.org/48555411:06
*** edmondsw has quit IRC11:09
*** markvoelker has joined #openstack-nova11:11
openstackgerritAndrey Volkov proposed openstack/nova master: Change default policy to view quota details  https://review.openstack.org/38600811:18
gibicdent: that SQL is something I cannot troubleshoot with my current SQL experiences so we need a cloning machine to replicate Jay11:19
cdentit sort of makes sense when it is broken down into pieces, but it can be a bit difficult to see that, and then the ways in which those pieces assemble can be a bit weird, as that bug you found shows11:20
*** dave-mccowan has joined #openstack-nova11:21
openstackgerritChris Dent proposed openstack/nova master: remove un-necessary update() in _init_compute_node of rt  https://review.openstack.org/48350611:22
*** edmondsw has joined #openstack-nova11:24
openstackgerritChris Dent proposed openstack/nova master: [placement] fix rp create failed when late placement endpoint  https://review.openstack.org/48346011:28
*** edmondsw has quit IRC11:28
*** jmlowe has joined #openstack-nova11:29
*** hoonetorg has joined #openstack-nova11:30
*** rgerganov has joined #openstack-nova11:30
*** rgerganov has quit IRC11:32
*** rgerganov has joined #openstack-nova11:32
*** gbarros has joined #openstack-nova11:36
*** jjung has quit IRC11:37
gibicdent:11:37
gibicdent: I think I can fix the bug by moving the proper initialization from here https://review.openstack.org/#/c/483565/4/nova/scheduler/manager.py@13911:38
gibicdent: to here https://review.openstack.org/#/c/483565/4/nova/scheduler/manager.py@11311:38
cdentthat would address your particular issue, but my understanding is that “None” is being used in some contexts (the caching scheduler?) to mean something. None is sometimes expected in the _schedule method of filter_scheduler11:42
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060811:42
cdentgibi: I’ve got a different batch of code checked out at the moment, digging around in another bug, so I’ve not got the right mental state for this but will in a little while11:44
openstackgerritAlexandru Muresan proposed openstack/nova master: Pass config object to oslo_reports  https://review.openstack.org/48557511:45
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed)  https://review.openstack.org/45063611:45
gibicdent: sure. I will try to propose something as a patch in the meantime11:46
openstackgerritChris Dent proposed openstack/nova master: placement: proper JOIN order for shared resources  https://review.openstack.org/48508811:48
openstackgerritChris Dent proposed openstack/nova master: [placement] gabbi tests for shared custom resource class  https://review.openstack.org/48520911:48
openstackgerritChris Dent proposed openstack/nova master: placement: alloc candidates only shared resources  https://review.openstack.org/48490011:48
cdentdammit11:48
cdentthat wasn’t supposed to rebase everything11:49
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709111:49
*** ralonsoh has quit IRC11:50
*** ralonsoh has joined #openstack-nova11:50
*** adisky__ has quit IRC11:50
mkuciaheh, unintended rebases happen time to time :)11:52
*** takedakn has joined #openstack-nova11:52
*** kristian__ has joined #openstack-nova11:55
*** udesale__ has joined #openstack-nova11:56
*** amuresan has joined #openstack-nova11:56
*** thorst has joined #openstack-nova11:56
*** thorst_ has joined #openstack-nova11:57
*** amuresan has quit IRC11:57
*** udesale has quit IRC11:58
*** damien_r has joined #openstack-nova11:58
*** amuresan has joined #openstack-nova11:58
*** kristian__ has quit IRC11:59
*** thorst has quit IRC12:01
*** lyan has quit IRC12:02
*** tbachman_ has joined #openstack-nova12:02
*** lyan has joined #openstack-nova12:02
*** tbachman has quit IRC12:02
*** tbachman_ is now known as tbachman12:02
*** udesale has joined #openstack-nova12:03
openstackgerritBalazs Gibizer proposed openstack/nova master: placement: init alloc_reqs earlier  https://review.openstack.org/48558512:03
*** hoonetorg has quit IRC12:03
*** MVenesio has joined #openstack-nova12:04
*** litao__ has quit IRC12:04
gibicdent: this ^^ is the patch I put together12:04
cdentcool, wil look in a sec12:05
*** udesale__ has quit IRC12:05
gibicdent: thanks12:05
*** amuresan has quit IRC12:06
*** thorst_ has quit IRC12:11
*** thorst has joined #openstack-nova12:11
*** thorst has quit IRC12:12
openstackgerritMerged openstack/nova master: Log compute node uuid when the record is created  https://review.openstack.org/48444712:12
openstackgerritBalazs Gibizer proposed openstack/nova master: placement: init alloc_reqs earlier  https://review.openstack.org/48558512:13
*** baoli has joined #openstack-nova12:15
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications  https://review.openstack.org/46552612:16
*** jmlowe has quit IRC12:16
*** amuresan has joined #openstack-nova12:17
*** amuresan has quit IRC12:18
*** amuresan has joined #openstack-nova12:18
*** amuresan has quit IRC12:19
gibicdent: thanks for the comment on the patch. Let's wait for the others to wake up to discuss if this is the good fix for the problem12:20
cdentyeah, that’s pretty much what I was trying to say in my comment, in too many words12:21
gibicdent: then I got your message :)12:21
*** thorst has joined #openstack-nova12:22
*** FL1SK has joined #openstack-nova12:22
openstackgerritChris Dent proposed openstack/nova master: Add functional test for local delete allocations  https://review.openstack.org/47057812:22
openstackgerritAlex Szarka proposed openstack/nova master: Raise Exception instead of Exception method call  https://review.openstack.org/48220012:22
*** randomhack has joined #openstack-nova12:23
*** syqian has quit IRC12:25
*** lpetrut has joined #openstack-nova12:25
*** edmondsw has joined #openstack-nova12:25
*** syqian has joined #openstack-nova12:25
openstackgerritMerged openstack/nova master: Fix indentation in policy doc  https://review.openstack.org/48464612:26
*** thorst has quit IRC12:26
openstackgerritBalazs Gibizer proposed openstack/nova master: placement: init alloc_reqs earlier  https://review.openstack.org/48558512:26
*** JoseMello has joined #openstack-nova12:26
*** thorst has joined #openstack-nova12:26
*** randomhack has quit IRC12:28
*** gbarros_ has joined #openstack-nova12:28
openstackgerritMerged openstack/nova master: Only setup iptables for metadata if using nova-net  https://review.openstack.org/48076512:30
openstackgerritMerged openstack/nova master: Add log info in scheduler to mark start of scheduling  https://review.openstack.org/48134012:31
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/48541012:32
*** gbarros has quit IRC12:32
openstackgerritMerged openstack/nova master: VStorage: changed default log path  https://review.openstack.org/45855712:32
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276612:33
*** takedakn has quit IRC12:34
*** abhishekk has quit IRC12:35
*** catintheroof has joined #openstack-nova12:36
*** alexchadin has joined #openstack-nova12:39
*** markvoelker has quit IRC12:40
*** coreywright has quit IRC12:48
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948312:49
*** thorst has quit IRC12:49
openstackgerritMatthew Booth proposed openstack/nova master: Allow wrapping of closures  https://review.openstack.org/47980112:50
openstackgerritMatthew Booth proposed openstack/nova master: Use _error_out_instance_on_exception in finish_resize  https://review.openstack.org/48560112:50
*** thorst has joined #openstack-nova12:50
*** nicolasbock has joined #openstack-nova12:50
*** nicolasbock has quit IRC12:51
openstackgerritSean Dague proposed openstack/nova master: WIP: request_log addition for running under uwsgi  https://review.openstack.org/48560212:51
*** nicolasbock has joined #openstack-nova12:51
*** kylek3h has joined #openstack-nova12:52
*** markvoelker has joined #openstack-nova12:52
*** thorst has quit IRC12:56
*** thorst has joined #openstack-nova12:58
*** _ix has joined #openstack-nova12:58
*** trinaths has quit IRC12:58
*** gbarros_ has quit IRC13:00
*** syqian_ has joined #openstack-nova13:00
*** coreywright has joined #openstack-nova13:00
cdentsdague: I wrote a couple of alternative ideas (to avoid paste changes) on ^. If it seems useful and you haven’t got the time I can probably look into tomorrow.13:01
*** rmart04_ has joined #openstack-nova13:01
*** sree has joined #openstack-nova13:01
*** thorst has quit IRC13:02
*** syqian has quit IRC13:02
*** gcb has quit IRC13:02
*** gcb has joined #openstack-nova13:03
*** mriedem has joined #openstack-nova13:03
*** tylerderosagrund has joined #openstack-nova13:03
*** rmart04 has quit IRC13:04
*** sree has quit IRC13:06
mdboothdansmith: https://review.openstack.org/#/c/479802/13:08
*** thorst has joined #openstack-nova13:08
*** rmart04_ has quit IRC13:08
mdboothYou wanted me to break it up, but as I put in a comment I don't think that works.13:08
* cdent runs errands13:10
*** esberglu has joined #openstack-nova13:10
*** jmlowe has joined #openstack-nova13:10
cdentgibi: if you learn something while I’m away, let me know13:10
*** cdent has quit IRC13:10
*** kristian__ has joined #openstack-nova13:10
*** ociuhandu_ has quit IRC13:11
mdboothdansmith: Anyway, working on it now so I'd really like to make sure I've understood you.13:11
openstackgerritiswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values  https://review.openstack.org/48146513:11
*** xyang1 has joined #openstack-nova13:12
dansmithmdavidson: what doesn't work?13:13
dansmithheh, mdbooth ^13:14
bauzasgibi: so, could you please tl:dr the problem about https://bugs.launchpad.net/nova/+bug/1705446 ?13:14
openstackLaunchpad bug 1705446 in OpenStack Compute (nova) "filter scheduler raises TypeError: argument of type 'NoneType' is not iterable when placement returns no allocation candidates" [High,In progress] - Assigned to Balazs Gibizer (balazs-gibizer)13:14
openstackgerritAlex Szarka proposed openstack/nova master: Transform the transformed notifications functional tests  https://review.openstack.org/48344813:16
gibibauzas: hi! so when placement returns no allocation candidates because there is no resource left, the scheduler/manager calls the scheduler drivers with None in alloc_reqs_by_rp_uuid13:16
gibibauzas: then the FilterScheduler tries to iterate on alloc_reqs_by_rp_uuid and blows up13:17
gibibauzas: see also https://review.openstack.org/#/c/485585/3/nova/scheduler/manager.py13:17
*** gouthamr has joined #openstack-nova13:17
gibibauzas: alloc_reqs_by_rp_uuid onyl properly initialized in the else branch at line 14713:18
mdboothdansmith: I think it's an atomic change13:18
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366013:18
mdboothdansmith: I appreciate it's obtuse13:18
*** zhurong has quit IRC13:19
*** pchavva has joined #openstack-nova13:19
gibibauzas: as soon as I moved the initialization up to the top the FilterScheduler was happy again13:19
mdboothBut being obtuse is the problem13:19
*** lucasxu has joined #openstack-nova13:19
mdboothWe've got this 1 thing which is there for no immediately discernable reason. After a mountaintop retreat, I have communed with the spirits and believe I have discerned a reason.13:20
*** kristian__ has quit IRC13:20
mdboothIf I remove it, I have to simultaneously replace its function, right?13:20
*** udesale has quit IRC13:21
mdboothIt looks like it's all over the place, but only because the code was previously all over the place.13:21
mdbooth...unless the spirits were lying to me, and I haven't correctly discerned its purpose.13:21
dansmiththe refactor of the decorator into a context manager does not have to be atomic with the rest of it (although I think I'd rather not even have that refactor, tbh)13:21
mdboothI could do that, but there would be no user of the context manager without the rest of it.13:22
*** eharney has joined #openstack-nova13:22
dansmithbut that refactor has nothing to do with the removal of the two state exclusion right?13:22
mdboothYeah, it does. The only reason I added a context manager is so I could reduce the scope.13:22
*** zzzeek has quit IRC13:23
mdboothAnd the only reason I want to do that is to replace (my guest guess of) that exclusion block's purpose.13:23
dansmithyou can refactor it to a context manager first and then remove the state exclusion13:23
bauzasgibi: sure, the point I had for the change adding that was about having different values13:23
*** alexchadin has quit IRC13:23
mdboothdansmith: Yep, but the context manager wouldn't have a user.13:24
bauzasgibi: if placement wasn't supported, returning None13:24
*** moshele has quit IRC13:24
mdboothI guess I'm ok with that, but we just normally don't do it.13:24
dansmithmdbooth: sure it would, the decorator13:24
dansmithmdbooth: we do it constantly13:24
bauzasgibi: if placement was supported but returning no candidates, then an empty dict13:24
*** lbragstad has joined #openstack-nova13:24
mdboothOh, ok13:24
dansmithwe do transitions as setup, migration, cleanup all the time13:24
bauzasgibi: FWIW, your change is saying that if we don't have placement, it would return a None13:24
gibibauzas: sure that makes sense13:24
bauzasoops13:25
bauzasit would return {}13:25
mdboothYeah, sure I can do that. Not sure it's worth it for a relatively small patch, but it's no great hassle.13:25
bauzaswhile I was wanting a None13:25
*** zzzeek has joined #openstack-nova13:25
gibibauzas: so you want to make a bit more finegrained fix13:25
dansmithafter the refactor,13:25
bauzasgibi: https://review.openstack.org/#/c/485585/3/nova/tests/unit/scheduler/test_scheduler.py@135 tbc13:25
gibibauzas: where {} is returned only when we got empty allocations back13:25
dansmithmdbooth: you can apply it to a smaller scope to avoid the race with the rpc call/notification stuff,13:25
mdboothdansmith: I thought you wanted me to split out the individual function changes.13:25
dansmithand after that you can remove the state exclusion rght?13:26
bauzasgibi: yup, that13:26
dansmithmdbooth: well, I do, if possible13:26
gibibauzas: OK, I will rework the patch13:26
gibibauzas: thanks13:26
mdboothThose 2 really are the same issue.13:26
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366013:26
dansmithmdbooth: how?13:26
bauzasgibi: thanks for working on it :)13:27
mdboothBecause the exclusion is the current mechanism by which we reduce the scope of errors_out_migration13:27
*** jamesdenton has joined #openstack-nova13:27
*** moshele has joined #openstack-nova13:28
mdboothIf you do it lexically, the exclusion no longer has a user.13:28
dansmithmdbooth: but your commit message says there's no reason for that code at all right?13:28
mdboothThe exclusion code?13:28
gibibauzas: I have another placement question if you have a minute13:28
dansmithmdbooth: the state exclusion yeah13:28
mdboothIts purpose is undocumented, and I've only guessed what it is13:29
bauzasgibi: sure13:29
mdboothBut I'm assuming that the person who put it there did it deliberately13:29
gibibauzas: when the scheduler claims the resources it is possible that there is a conflict13:29
mdboothIt appears to be to prevent the migration being put in an error state after certain points in the task13:29
gibibauzas: I see that placement return http 40913:29
gibibauzas: and that seems correct to me13:30
gibibauzas: I mean when there is really a conflict13:30
bauzasgibi: yup, correct13:30
dansmithmdbooth: right but I don't see how removing that (which you think needs to happen) is atomically related to the other scope change13:30
dansmithmdbooth: I gotta join this terrible 6:30am call now, so do whatever you think is right I guess13:30
gibibauzas: but at the same time placement logs a ERROR log about the invalid inventory and that seems not that nice as it suggest a software error for me13:30
mdboothdansmith: Enjoy :)13:31
bauzasgibi: when PUT /allocation ?13:31
bauzasgibi: if so, yup, it seems weirdo to pass a error level log13:31
*** jangutter has quit IRC13:31
*** mtanino__ has joined #openstack-nova13:31
gibibauzas: here is the stack trace https://pastebin.com/cJP97xQr13:32
openstackgerritAndrey Volkov proposed openstack/nova master: PoC: Select PCI devices with distinct tag values  https://review.openstack.org/44800813:32
gibibauzas: hm, it might not coming from the claiming13:33
*** baoli has quit IRC13:33
gibibauzas: ahh I found it, it is a "PUT /placement/allocations/485c7480-939c-4b88-8c00-0f346dc6a924"13:33
gibibauzas: so yes13:33
gibibauzas: then I will file a bug for this as well13:34
gibibauzas: should this log be on info or debug level?13:34
bauzasdebug IMHO13:34
bauzasbecause it just means that's a race condition13:35
gibibauzas: OK, thanks13:35
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563413:37
*** yamamoto has joined #openstack-nova13:39
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366013:39
*** Swami has joined #openstack-nova13:40
*** takashin has left #openstack-nova13:42
*** felipemonteiro has joined #openstack-nova13:43
gibibauzas: here is the bug report for the error log https://bugs.launchpad.net/nova/+bug/170548713:44
openstackLaunchpad bug 1705487 in OpenStack Compute (nova) "placement logs an ERROR when PUT /allocation result in an invalid inventory" [Undecided,New]13:44
*** felipemonteiro_ has joined #openstack-nova13:44
*** felipemonteiro has quit IRC13:49
*** yamamoto has quit IRC13:50
*** yingjun_1 has joined #openstack-nova13:51
*** jangutter has joined #openstack-nova13:53
*** jwcroppe has joined #openstack-nova13:53
*** crushil has joined #openstack-nova13:54
ftersinmdbooth: hi. If you have a minute, could you look at ScaleIO review (https://review.openstack.org/#/c/407440/)? I addressed your (and mriedem) comments there, and now ready for new ones.13:55
*** tbachman_ has joined #openstack-nova14:02
*** tbachman has quit IRC14:02
*** tbachman_ is now known as tbachman14:02
*** prateek has quit IRC14:05
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed)  https://review.openstack.org/45063614:05
*** awaugama has joined #openstack-nova14:06
*** crushil has quit IRC14:06
*** crushil has joined #openstack-nova14:06
mriedemandreykurilin: melwitt: we need to get https://review.openstack.org/#/c/484152/ in to unblock novaclient tests14:08
mriedemnow that the counting instance quotas change merged in nova14:08
mriedemor sdague ^14:08
openstackgerritMerged openstack/nova master: [placement] cover deleting standard trait  https://review.openstack.org/48415314:09
andreykurilinmriedem: done14:09
mriedemgibi: just wanted to say thanks for kicking the tires on the scheduler + placement stuff, you're finding some nice hairy bugs14:09
mriedemandreykurilin: thanks14:09
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366014:11
*** dillaman has joined #openstack-nova14:17
*** jdillaman has quit IRC14:17
*** dillaman has quit IRC14:17
*** jdillaman has joined #openstack-nova14:17
*** yingjun_1 has quit IRC14:17
openstackgerritKaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code  https://review.openstack.org/43985514:18
*** jaypipes has joined #openstack-nova14:19
gibimriedem: my pleasure :)14:19
*** Daisy_ has joined #openstack-nova14:20
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948314:21
sdaguemriedem: lookin14:21
sdaguemriedem: ah, andreykurilin already snagged it14:22
sdaguemriedem: you got a few moments to think about path forward on request logging?14:22
*** jmlowe has quit IRC14:22
mriedemsure, although it's over my head14:23
sdagueok, well do you get my concern that we're not logging it through the python subsystem any more?14:24
*** Daisy_ has quit IRC14:24
mriedemyeah it's definitely a problem, and as noted i'm seeing weird formatting14:24
mriedemlike parts of the log message are chopped off14:24
sdaguemriedem: you see my follow ups?14:25
mriedemreading those now14:25
*** acormier has joined #openstack-nova14:25
sdagueok, do that, then we can chat, so I don't repeat myself :)14:25
*** moshele has quit IRC14:27
mriedemsdague: wait, say that again14:28
* dansmith snickers14:28
mriedemsdague: oh man ok so that debug log message starts here then http://logs.openstack.org/65/483565/4/check/gate-tempest-dsvm-py35-ubuntu-xenial/9921636/logs/screen-n-sch.txt.gz#_Jul_19_20_17_18_80046714:29
openstackgerritAlex Szarka proposed openstack/nova master: Reduce code complexity - manager.py  https://review.openstack.org/35986814:29
mriedemand takes 3 full chunks14:29
sdaguemriedem: yep14:29
mriedemmake that 4 chunks14:29
*** fragatina has joined #openstack-nova14:29
mriedemyikes, maybe we shouldn't log full instances on a cron in the scheduler14:29
jaypipesmriedem: fyi, in and out this morning... more plumbing debacles at chez pipes.14:30
mriedemthat's only 3 instances,14:30
mriedemthink if you have 10K computes sending the update_instance_info periodic to the scheduler every minute, which is the default14:30
mriedemwith 1 million VMs14:30
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Handle missing volume vmdk during detach  https://review.openstack.org/48467514:31
mriedemjaypipes: ok. btw, i started my morning with an internal email from a guy doing scale testing asking about issues with booting 3000+ VMs at once14:31
*** tbachman has quit IRC14:31
mriedemand multiple scheduler workers, in mitaka14:31
jaypipesmriedem: lovely.14:31
jaypipesmriedem: I have an afternoon call to talk about why Nova AZs aren't AWS AZs.14:31
dansmithjaypipes: chez pipes sounds like a slum14:32
*** annegentle has joined #openstack-nova14:32
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects  https://review.openstack.org/47489214:33
*** armax has joined #openstack-nova14:33
mriedembtw this is a good example of why you can't run the scheduler with debug logging enabled14:34
mriedemin prod14:34
dansmithbut you also can't run it without debug in prod14:34
mriedemthis reminds me of a bug rlrossit found14:34
mriedemall the logging that happens in here https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L16614:35
mriedemwhen we turned on debug it killed the scheduler in our pre-prod cloud14:36
mriedemsdague: so you've got my attention14:37
*** yamamoto has joined #openstack-nova14:37
sdaguemriedem: ok, so the challenges / questions14:37
sdague1) do we need something like the request_log middleware? (assume answer is yes)14:38
sdague2) do we force people to do a paste pipeline update or do we hide this in existing middleware to help with upgrade?14:38
jaypipesdansmith: it is.14:39
dansmithjaypipes: :P14:39
sdague3) do we want to bring back in request timing (eventlet has it, this does not yet)?14:39
*** lucasxu has quit IRC14:39
mriedemwhat is request timing?14:40
mriedemif the answer to getting the request id in the logs under uwsgi is #1, then yes for sure14:41
*** marst has joined #openstack-nova14:41
sdaguehttp://logs.openstack.org/02/485602/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0c29b8a/logs/screen-c-api.txt.gz#_Jul_20_13_20_35_27950714:41
mriedemeven w/o global request id, we need the request id for basic debug14:41
mriedemis that "time: 0.5609260" ?14:42
sdaguethat's what a current eventlet wsgi log line looks like14:42
sdagueyeh14:42
mriedemi don't think i ever use that14:42
mriedemit would be nice to know for profiling i suppose14:42
sdaguehttp://logs.openstack.org/02/485602/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/0c29b8a/logs/screen-n-api.txt.gz#_Jul_20_13_20_03_249571 is what the request_log patch makes the nova lines look like14:42
mriedemif you have some requests taking 30 seconds or something14:42
sdagueso, 3 is pretty minor14:43
sdaguethe real issue is #2, in how we get this in the hands of people14:43
sdaguebecause right now, the WIP patch requires this - https://review.openstack.org/#/c/485602/1/etc/nova/api-paste.ini14:43
mriedemwell, there is something to consider there,14:44
mriedemwhich is,14:44
mriedemwait for it,14:44
mriedemthis isn't a problem if you're running nova-api under eventlet, yes?14:44
sdaguecorrect14:44
mriedemso if/when you move nova-api to uwsgi or apache,14:44
mriedemyou're going to be changing some config anyway, yes?14:44
sdaguemaybe, though up until this point we've never told people they have to change this stuff14:45
mriedemthis stuff == api-paste.ini?14:45
sdagueyes14:45
mriedembecause changing nova-api to run under uwsgi is going to require changes to someone's config management / deploy tooling14:45
mriedemjust like it did in devstack14:45
sdaguesure14:46
mriedemso it's not like this is some trivial change that doesn't require manual intervention to start running nova-api this way14:46
*** yingjun has joined #openstack-nova14:46
sdaguesure, so you are feeling that upgrade note and the paste change is fine?14:46
*** cfriesen has joined #openstack-nova14:46
mriedemso i don't think it's the end of the world to say, if you're going to start running nova-api under uwsgi AND you want the request IDs logged for debug (which we assume you do if you're not crazy), then you have to also do this other thing14:46
mriedemwith 8 days to FF, i'm feeling fine about that yes14:46
mriedemwait what day is it?14:47
mriedem7 days to FF14:47
sdagueok, I'll proceed forward based on that14:47
*** vishwanathj has joined #openstack-nova14:47
mriedemif you wanted to tinker with sliding something into existing middleware that is smart enough to know if it's running under eventlet or not and auto enable, that'd be cool but probably queens at this point14:47
sdaguethe functional parts of the patch should be ready for review - https://review.openstack.org/#/c/485602/1/nova/api/openstack/requestlog.py14:47
sdagueyeh, I'll put that as stretch goal14:48
mriedemis there any existing oslo.middleware stuff that all/most services use that we could do the auto enable thing in there so everyone else gets it for free?14:48
mriedemand doesn't have to copy the solution around?14:49
mriedemnoting that today is release freeze for oslo lis14:49
mriedem*libs14:49
sdagueI don't think that there is14:49
mriedemhttps://github.com/openstack/oslo.middleware/blob/master/oslo_middleware/request_id.py ?14:49
sdaguemaybe, not everyone uses that14:50
sdaguebut, regardless, I don't think we could get it in and through that process14:50
mriedemsure,14:50
sdaguein the time alloted14:50
sdagueone of the challenges about fixed time boxes14:50
dansmith...with 146 things in the check queue14:50
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667114:50
mriedemso maybe that's the other thing for queens is moving this into oslo.middleware so others can just pick it up from there14:50
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028514:50
jangutterNewbie question: https://review.openstack.org/#/c/485382/ failed a check in the gate (last time it failed a different check). Does this kinda thing happen often when upping requirements.txt?14:52
*** yamamoto has quit IRC14:55
*** cdent has joined #openstack-nova14:56
*** hoonetorg has joined #openstack-nova14:56
*** hoonetorg has quit IRC14:57
*** yamamoto has joined #openstack-nova14:57
*** hoonetorg has joined #openstack-nova14:57
sdaguemriedem: yeh, it's kind of going to depend on the way wsgi bits are done, which is not standardized14:59
sdaguehonestly, the ideal might be if oslo.service actually abstracted this stuff away15:00
sdaguebut... that's got other challenges15:00
*** yamamoto has quit IRC15:03
dansmithmelwitt: remind me why those need to be remotable? https://review.openstack.org/#/c/410945/58/nova/objects/quotas.py@6215:05
*** Oku_OS is now known as Oku_OS-away15:06
dansmithlike, I thought we had removed all of the quota calls by this point that need to be done from compute or non-db-having services15:07
gibimriedem: I cannot joint to the nova meeting today. Could you cover the notification subteam part?15:09
mdboothmelwitt: Addressed your comments on this patch with a test: https://review.openstack.org/#/c/479801/15:10
mdboothJenkins -1 is bogus, btw. Rechecked it, but following patch has passed.15:10
*** felipemonteiro_ has quit IRC15:11
mdboothmelwitt: Incidentally following patch is https://review.openstack.org/#/c/485601/ which is a minor cleanup to your quotas patch15:11
*** thorst has quit IRC15:11
mdbooths/minor/trivial/15:11
*** Daisy_ has joined #openstack-nova15:15
*** damien_r has quit IRC15:15
*** Daisy_ has quit IRC15:20
*** lucasxu has joined #openstack-nova15:22
mriedemgibi: sure15:22
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028515:23
*** jmlowe has joined #openstack-nova15:24
*** moshele has joined #openstack-nova15:24
*** GnomeKing has quit IRC15:25
*** annegentle has quit IRC15:25
*** rcernin has quit IRC15:25
*** moshele has quit IRC15:26
mriedemsdague: are you going to recap the plan to the ML?15:27
gibimriedem: thanks15:27
openstackgerritMerged openstack/nova master: placement: alloc candidates only shared resources  https://review.openstack.org/48490015:27
*** hoonetorg has quit IRC15:27
*** armax has quit IRC15:27
*** annegentle has joined #openstack-nova15:27
mriedemjaypipes: do we need anything in here to get released in os-traits today? https://review.openstack.org/#/q/os-traits+status:open15:28
mriedembecause today is the big day15:28
openstackgerritMerged openstack/python-novaclient master: Adjust test_resize_down_revert to account for counting quotas  https://review.openstack.org/48415215:28
mriedemmaybe the reqs update15:28
sdaguemriedem: I can15:28
sdagueI'm going to figure out test failures first15:28
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object  https://review.openstack.org/47491415:28
jaypipesmriedem: nope15:29
mriedemok i'll get the req update in and then release it15:29
jaypipesmriedem: danke15:29
*** armax has joined #openstack-nova15:29
*** mkucia has quit IRC15:32
*** ckopper has quit IRC15:32
*** tylerder_ has joined #openstack-nova15:33
mriedemsdague: another thing to consider, report a bug for this request id logging thing and then we can tag it pike-rc-potential for tracking15:33
mriedemi'm starting to tag things for the rc15:33
*** thorst has joined #openstack-nova15:33
*** Apoorva has joined #openstack-nova15:35
*** chyka has joined #openstack-nova15:35
jaypipessean-k-mooney: ralonsoh's https://review.openstack.org/#/c/474914/ is good to go. feel free to +2 if agree.15:35
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563415:35
ralonsohjaypipes, sean-k-moone: thanks!15:35
*** Apoorva_ has joined #openstack-nova15:35
mriedemjaypipes: sean-k-mooney: ralonsoh: does ^ need to be in an os-vif release for pike?15:35
mriedembecause we released os-vif yesterday15:35
*** hoonetorg has joined #openstack-nova15:36
jaypipesmriedem: I don't believe so, no.15:36
*** tylerderosagrund has quit IRC15:36
mriedemok, it's also dependent on a neutron change with -1s and test failures15:36
jaypipesmriedem: right.15:36
jaypipesmriedem: os-vif was good yesterday when it was released.15:36
ralonsohmriedem, jaypipes: well, this patch solves a bug for OVS userspace15:37
mriedemralonsoh: at this point it's not going to get released for pike15:37
mriedemunless it's later backported15:37
ralonsohmriedem; no, not now15:37
sdaguemriedem: sure15:37
jaypipesralonsoh: as mriedem said, there is a dependent neutron patch that would need to be merged for that os-vif patch to even be workable though.15:38
sean-k-mooneymriedem: need no nice to have definetly15:38
*** Apoorva has quit IRC15:39
jaypipesralonsoh: well, I shouldn't say "even be workable". I meant "even be useful".15:39
*** Apoorva_ has quit IRC15:39
ralonsohjaypipes: yes, this bug needed changes in 4 projects. I don't think is going to land in Pike15:39
*** Apoorva has joined #openstack-nova15:39
jaypipescdent: hey, just to repeat what I said on that bug report... really excellent investigative work on that sorted(sharing_providers.items()..) thing.15:39
*** andreas_s has quit IRC15:39
jaypipescdent: I had spent hours looking at the SQL generated going to myself "there's nothing wrong with this SQL... :("15:40
*** damien_r has joined #openstack-nova15:40
cdentthanks jaypipes, it was really because of gibi15:40
jaypipescdent: and your comment sparked a renewed look at the JOIN order.15:40
cdenthe had pasted up things to look at and started wondering “why is that different"15:40
sean-k-mooneymriedem: it would be good to backport https://review.openstack.org/#/c/474914 all the way to ocata but not sure if we can due to ovo chagnes15:40
cdentalso having the gabbi test (which makes things easier for me to toy with) helped15:40
jaypipeswell, thank you cdent and gibi both :)15:40
jaypipesI have a feeling that little patch may fix some of the odd random NoValidHosts failures we were seeing intermittently.15:41
*** psachin has quit IRC15:41
jaypipessince it produces an incorrect false negative on providers meeting requested resources when including shared resources.15:41
cdentjaypipes: you’ve seen today’s bug that gibi found, yes?15:43
cdentI’ve now become expectant of him finding one every day15:43
*** felipemonteiro has joined #openstack-nova15:43
*** yingjun has quit IRC15:43
*** damien_r has quit IRC15:43
jaypipescdent: :(15:43
*** Apoorva has quit IRC15:44
jaypipescdent: you mean the "init alloc_reqs earler" one?>15:44
cdentyeah15:44
cdentIt’s not :( it’s :). Find them now when they are fresh and new before they go hurt someone.15:44
*** felipemonteiro_ has joined #openstack-nova15:44
mriedemfinal os-traits release https://review.openstack.org/48572215:44
mriedembauzas: ^15:44
bauzasmriedem: looking15:45
gibicdent, jaypipes: I guess I will fail on the bug / day expectation as I will be out tomorrow15:47
cdentI will try to stand in for you15:47
jaypipesgibi: that's no excuse. we expect at least one bug a day now.15:47
gibicdent, jaypipes: however there is one more for today https://bugs.launchpad.net/nova/+bug/170548715:47
openstackLaunchpad bug 1705487 in OpenStack Compute (nova) "placement logs an ERROR when PUT /allocation result in an invalid inventory" [Undecided,New]15:47
*** Apoorva has joined #openstack-nova15:47
gibicdent, jaypipes: I'm hesitant if this is just a logging thing or something a bit deeper15:48
*** felipemonteiro has quit IRC15:48
cdentgibi: yay! more bugly goodness.15:48
jaypipesgibi: looking ...15:48
gibijaypipes, cdent: If I'm not mistaken then the exception is coming from here https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L160715:49
gibijaypipes: can be that it is connected to the fact that I'm using shared custom resource15:50
*** tuanla_fujitsu has joined #openstack-nova15:50
*** tuanla__ has joined #openstack-nova15:50
jaypipesgibi: do you have a test case for that newest bug? I'm not entirely sure how an invalid resource class can be allocated by the scheduler, but perhaps it could.15:50
gibijaypipes: not yet unfortunately and I have to leave soon.15:51
*** lucasgomes is now known as lucas-afk15:51
jaypipesgibi: ah, I see now... it's a race condition that we should retry on instead of logging an exception I guess15:51
gibijaypipes: I can try to craft the test case on Monday15:52
jaypipesgibi: cool. enjoy your long weekend! :)15:52
*** baoli has joined #openstack-nova15:53
gibijaypipes: thanks!15:53
*** hshiina has joined #openstack-nova15:54
*** Redcavalier has joined #openstack-nova15:56
*** mgiles has joined #openstack-nova15:57
*** ckopper has joined #openstack-nova15:58
openstackgerritJay Pipes proposed openstack/nova master: Remove improper LOG.exception() calls in placement  https://review.openstack.org/48572616:00
jaypipesgibi: fixed already :) ^^16:00
*** annegentle has quit IRC16:01
*** Apoorva has quit IRC16:02
*** Apoorva has joined #openstack-nova16:03
*** tuanla_fujitsu has quit IRC16:04
*** tuanla__ has quit IRC16:04
*** tuanla_fujitsu has joined #openstack-nova16:06
*** jmlowe has quit IRC16:15
mriedem+216:16
*** dfisher has joined #openstack-nova16:17
openstackgerritStephen Finucane proposed openstack/nova master: DNM: Try to figure out what the tenant is returning  https://review.openstack.org/48573316:19
*** itlinux has joined #openstack-nova16:19
mriedemwould be nice to get some eyes on this https://review.openstack.org/#/c/446671/16:20
mriedemwe've got 3 patches left for the cinder attachment stuff16:21
mriedemand this is probably the least complicated16:21
*** mvk has quit IRC16:21
*** itlinux has quit IRC16:21
*** itlinux has joined #openstack-nova16:22
dfishermriedem: is this for multi-attach?16:22
mriedemit's for using the new cinder 3.27 attachments api which is a prereq for multiattach16:23
openstackgerritMatthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager  https://review.openstack.org/48573416:23
dfishermriedem: i assume with all this changing, https://bugs.launchpad.net/nova/+bug/1527623 may get another look?16:23
openstackLaunchpad bug 1527623 in OpenStack Compute (nova) "Nova might orphan volumes when it's racing to delete a volume-backed instance" [Medium,In progress] - Assigned to ChangBo Guo(gcb) (glongwave)16:23
mriedemw/o loading a bunch of context into my head right now i can't say16:24
* dfisher nods16:24
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028516:25
*** oorgeron has joined #openstack-nova16:26
*** tylerder_ has quit IRC16:26
oorgeronHi folks, I'd like to get https://review.openstack.org/#/c/446643/ reviewed. Anyone around to help?16:27
*** tylerderosagrund has joined #openstack-nova16:27
*** rajathagasthya has joined #openstack-nova16:27
mriedemoorgeron: that's not something we need to spend time reviewing when we're 7 days from the pike feature freeze16:27
mriedemand is not a bug fix16:28
dansmith+116:28
oorgeronso you want me to shift it to blueprint or rfe?16:28
mriedemit's already got a blueprint tied to it16:28
*** tylerderosagrund has quit IRC16:28
mriedemso bring it up when we start talking about blueprints for queens16:28
mriedemin a few weeks please16:29
openstackgerritSean Dague proposed openstack/nova master: WIP: request_log addition for running under uwsgi  https://review.openstack.org/48560216:29
*** tylerderosagrund has joined #openstack-nova16:30
*** dfisher has left #openstack-nova16:30
*** tylerder_ has joined #openstack-nova16:31
oorgeronmriedem: I'll hold off on it for now.16:31
*** tylerder_ has quit IRC16:32
mriedemthanks16:32
*** acormier has quit IRC16:32
mriedemif there are going to be oracle people at the ptg in denver, i think it would also be good to bring this up as a topic in a cross-project session16:32
*** tesseract has quit IRC16:32
mriedemsince this is more than just nova16:32
mriedemand all projects should be on the same page16:32
*** tylerder_ has joined #openstack-nova16:32
mriedemotherwise you get 50% of the projects accepting this and others don't and you're stuck16:32
mriedemoorgeron: ^16:33
*** rcernin has joined #openstack-nova16:33
oorgeronmriedem: it's already in other projects and most don't require any changes beyond having oslo.db 4.24 or above.16:33
jaypipesoorgeron: wish I had been aware of https://review.openstack.org/#/c/427970/ :(16:33
mriedemthis is what i'm talking about16:33
mriedempoorly communicated crosss-project effort16:33
*** tylerder_ is now known as tylerderosagrun_16:34
*** tylerderosagrund has quit IRC16:34
*** tylerderosagrun_ has quit IRC16:34
*** tylerderosagrun_ has joined #openstack-nova16:34
*** tylerderosagrun_ is now known as tylerderosagrund16:35
*** smatzek has quit IRC16:36
TheJuliamriedem: following up from last night re ironic bfv, we have achieved a successful boot, and teardown, at least as far as nova is concerned.  Our test vm never booted back up for disk cleaning. hshiina has volunteered to take a stab at fixing the unit tests since the tooling doesn't make creating a wrapped object easy.16:42
mriedemildikov: comments in https://review.openstack.org/#/c/446671/16:42
mriedemildikov: only thing i worry about is the VolumeUnattached case16:42
mriedemTheJulia: can't just use bdm._bdm_obj.is_volume?16:43
*** dtantsur is now known as dtantsur|afk16:43
*** fragatina has quit IRC16:43
mriedemthe unit test should just pass in a block_device_info['mappings'] = [DriverVolumeBlockDevice(BlockDeviceMapping)]16:43
TheJuliaWell, if that works, then that should be a stupidly quick fix16:44
TheJuliahshiina: ^^^16:44
*** fragatina has joined #openstack-nova16:46
*** lyan has quit IRC16:49
*** Apoorva has quit IRC16:50
*** lucasxu has quit IRC16:51
*** dfisher has joined #openstack-nova16:52
*** thorst has quit IRC16:52
*** david-lyle has quit IRC16:52
*** david-lyle has joined #openstack-nova16:52
*** lpetrut has quit IRC16:53
*** prateek has joined #openstack-nova16:53
sean-k-mooneyanyone know is there a way to convert a vm in to a glance image other then creating a snapshot.16:54
*** bbansal has joined #openstack-nova16:56
bbansalguys, can anyone tell whom to contact in order to update about requirement consume?16:56
mriedembbansal: can you reword your question?16:57
bbansali see nova/requirements.txt:tooz!=1.57.4,>=1.47.0 # Apache-2.016:57
bbansaland my CI fails because version of tooz which gets installed as per upper-constraints.txt is 1.57.416:57
bbansalthe openstack-requirements channel mentioned that project which still have this !=1.57.4 needs to consume the requirement update16:58
cdentsdague: did you see my comments from ps1 on https://review.openstack.org/#/c/485602/ ? There’s a confused sentence in the commit message, and I also suggested some ways to avoid modifying paste.ini (if that turns out to be important, your email suggests it’s not).16:58
*** lucasxu has joined #openstack-nova16:58
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Read datapath_type from VIF object  https://review.openstack.org/47491416:58
*** trinaths has joined #openstack-nova16:59
bbansalmanila project has updated their changeset for the same17:00
bbansalhttps://review.openstack.org/#/c/48540617:00
*** baoli has quit IRC17:01
mriedembbansal: there is a bot that syncs the global-requirements changes to nova17:01
mriedemlike this https://review.openstack.org/#/c/455970/17:01
*** baoli has joined #openstack-nova17:01
*** baoli has quit IRC17:01
ildikovmriedem: checking17:01
bbansalokay, so i need to wait until changes are updated on all projects17:01
*** ralonsoh has quit IRC17:02
*** trinaths1 has joined #openstack-nova17:02
*** sambetts is now known as sambetts|afk17:03
bbansal@mriedem: yesterday u filed a bug related to n-net if u remember, can u please provide change set again17:03
bbansalor the bug id17:04
sdaguecdent: yeh, I did, I just wanted to get things passing first.17:05
sdagueI'll fix the commit message before moving it out of WIP17:05
*** trinaths has quit IRC17:05
*** alex_xu has quit IRC17:05
*** bmace has joined #openstack-nova17:06
*** hshiina has quit IRC17:07
*** Yingxin has quit IRC17:07
*** Yingxin has joined #openstack-nova17:07
larsksdansmith: btw, i just rebased https://review.openstack.org/#/c/467699/, which we had previously discussed. I see a jenkins failure but pretty sure it's unrelated.17:08
*** shaohe_feng has quit IRC17:09
*** Swami has quit IRC17:09
dansmithlarsks: okay, I'm trying to think of who the best person to review that is17:09
dansmithmriedem: ^17:09
*** shaohe_feng has joined #openstack-nova17:09
*** thorst has joined #openstack-nova17:10
*** baoli has joined #openstack-nova17:11
*** felipemonteiro_ has quit IRC17:12
*** felipemonteiro has joined #openstack-nova17:12
*** thorst_ has joined #openstack-nova17:12
*** alex_xu has joined #openstack-nova17:13
*** thorst has quit IRC17:15
*** avolkov` has quit IRC17:15
*** avolkov` has joined #openstack-nova17:16
*** thorst_ has quit IRC17:17
*** lyan has joined #openstack-nova17:18
*** avolkov` has quit IRC17:18
ildikovmriedem: if you got a moment, I got lost a bit on the VolumeUnattached stuff in remove check_detach17:18
*** tesseract has joined #openstack-nova17:18
*** Apoorva has joined #openstack-nova17:19
*** Apoorva has quit IRC17:19
*** Apoorva has joined #openstack-nova17:20
*** sridharg has quit IRC17:21
*** imacdonn has quit IRC17:21
*** imacdonn has joined #openstack-nova17:22
*** felipemonteiro_ has joined #openstack-nova17:22
*** prateek has quit IRC17:22
ildikovmriedem: in check_detach the VolumeUnattached exception should not be raised without multi-attach and I think even with that it would mean that something strange is going on17:22
ildikovmriedem: without it we hit InvalidVolume17:23
*** kfarr has joined #openstack-nova17:23
*** harlowja has joined #openstack-nova17:23
*** tuanla_fujitsu has quit IRC17:24
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt/driver.py: Set cache value for Cinder volume post-migration  https://review.openstack.org/48575217:26
*** david-lyle has quit IRC17:26
mriedembbansal: https://review.openstack.org/#/c/485300/17:26
*** felipemonteiro has quit IRC17:26
mriedemdansmith: larsks: does this go into config drive / metadata service API?17:27
dansmithyeah17:27
*** harlowja has quit IRC17:27
bbansalthanks17:27
mriedemhmm17:28
mriedemso just our usual old random shove whatever we want in there i guess?17:28
larsksmriedem: note that this change preserve existing behavior.17:28
*** shaohe_feng has quit IRC17:28
larsks...and implements more correct behavior for those who want it.17:28
dansmithmriedem: yeah, I think it's pretty compatible17:28
*** shaohe_feng has joined #openstack-nova17:28
dansmithmriedem: we've been doing no more fine-grained changes than one per release right? in terms of versions, I mean17:29
dansmithand we've not really been super consistent on that, AfAIK17:29
mriedemwe haven't been versioning the metadata api at all when network_data.json changes as far as i know17:30
mriedemthere was some other change or two earlier in pike that didn't have a version bump17:30
dansmithoh okay17:30
mriedemfiled under bug fixes17:30
dansmitheven better then17:30
dansmith"better"17:31
mriedemright. if this were the rest api, it would be a response body change17:31
mriedem*compute api17:31
*** smatzek has joined #openstack-nova17:32
mriedemleft comments17:34
mriedemthis isn't so much a bug as a wishlist item17:35
mriedembut i didn't -117:35
dansmithmriedem: did you read the bug?17:36
dansmithIIRC, you can't properly configure redhat-style interface definitions with just the global list17:37
dansmithif you use the smashed together version, then you break resolution when you bring interfaces down17:37
dansmith*some interfaces I mean17:37
larsksmriedem: I claim this is absolutely a bug.17:38
mriedemok, i didn't -117:38
mriedemi left some concerns17:38
* larsks reads17:38
dansmithmriedem: it's totally not wishlist, IMHO :)17:38
mriedemi have to run for a bit17:38
mriedemok change the severity in LP17:39
mriedemgotta run17:39
*** mriedem is now known as mriedem_afk17:39
*** tylerderosagrund has quit IRC17:39
*** tylerderosagrund has joined #openstack-nova17:40
*** sshwarts has quit IRC17:40
*** Swami has joined #openstack-nova17:41
*** breton has quit IRC17:42
*** breton has joined #openstack-nova17:43
*** fragatina has quit IRC17:48
*** yamahata has joined #openstack-nova17:53
trinaths1Hi, I have installed libvirt 1.3.5 and Qemu 2.6 in compute. When I start libvirtd service, I get this error http://paste.openstack.org/show/616064/  please help me solve this issue.17:55
*** krtaylor has quit IRC17:56
*** cdent_ has joined #openstack-nova17:56
*** tylerderosagrund has quit IRC17:58
*** oorgeron has left #openstack-nova17:58
*** tylerderosagrund has joined #openstack-nova17:58
sean-k-mooneytrinaths1: did you also install the correct version of libvirt python to match your libvirt. it looks like there is an issue with your libvirt config/path also as it is trying use qemu 2.4 not 2.617:59
*** cdent has quit IRC17:59
*** cdent_ is now known as cdent17:59
jaypipesdansmith: what the... ? http://logs.openstack.org/42/446242/41/check/gate-nova-tox-functional-ubuntu-xenial/30ad602/testr_results.html.gz18:00
jaypipesdansmith: I swear, gate functional tests are getting randomer and randomer failures :(18:00
dansmithjaypipes: access was denied to this resource18:00
dansmithduh18:00
jaypipesheh18:00
trinaths1sean-k-mooney: ok. is there any misconfiguration i'm doing ?18:01
cdentthat’s the thing where requests to placement go to compute (for reasons unknown)18:01
sean-k-mooneytrinaths1: did you build qemu and libvirt yourself. if you are using packages it shoudl have fix this for you18:01
cdentfixtures not playing well with one another18:01
*** thorst has joined #openstack-nova18:02
*** jamesdenton has quit IRC18:02
trinaths1sean-k-mooney: yes. I have build from git source code. qemu  2.4 and Libvirt 1.3.518:02
*** david-lyle has joined #openstack-nova18:03
trinaths1sean-k-mooney: but i have installed libvirt-python using dnf. on fedora 2318:04
sean-k-mooneytrinaths1: does virsh capabilities  work for you?18:05
*** harlowja has joined #openstack-nova18:06
sean-k-mooneytrinaths1: actully was that form the libvirtd unit in journalctl? http://paste.openstack.org/show/616064/18:07
*** jamesdenton has joined #openstack-nova18:07
sean-k-mooneytrinaths1: i taught the log was from nova at first hench question regarding libvirt python binding18:08
trinaths1sean-k-mooney: no.  it did give complete details. when I start nova-compute, I get this error:  http://paste.openstack.org/show/616067/18:08
sean-k-mooneytrinaths1: this actully looks like the libvirt version you are using is not compatible with the qemu you have installed.18:08
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667118:09
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028518:09
trinaths1sean-k-mooney: oops! please guide me to understand proper qemu for libvirt 1.3.518:10
sean-k-mooneytrinaths1: well it looks like you have 2 issues 1st No compute node record for host fedora23-1046: ComputeHostNotFound_Remote: Compute host fedora23-1046 could not be found.18:10
trinaths1sean-k-mooney:  i resolved it now. its working fine now.18:11
trinaths1sean-k-mooney: but 2nd issue is still pending ..18:11
sean-k-mooneytrinaths1: was that caused by cells?18:11
trinaths1sean-k-mooney: no. we are using mitaka compute.18:11
*** yamahata has quit IRC18:11
*** eharney has quit IRC18:13
*** thorst has quit IRC18:16
*** acormier has joined #openstack-nova18:16
trinaths1sean-k-mooney: see this, https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix I think 1.3.5 can be compatible with qemu 2.418:17
trinaths1sean-k-mooney: I aslo tried libvirt 1.3.5 with qemu 2.6 . it doesn't work well18:18
sean-k-mooneytrinaths1: it may be compatible on x86 not sure about arm. it may be down to how you compiled qemu18:19
trinaths1sean-k-mooney: ok18:19
sean-k-mooneyare you deploying with devstack out of interest?18:20
trinaths1sean-k-mooney: I'm using manual installation.18:20
trinaths1sean-k-mooney: not using devstack18:20
sean-k-mooneyah we wrote a devstack plugin to compile libvirt and qemu from source but i have not tested it on arm or we qemu and libvirt versions that old. https://github.com/openstack/devstack-plugin-libvirt-qemu.18:21
sean-k-mooneytrinaths1: it might still be good to compare how we do the compilation https://github.com/openstack/devstack-plugin-libvirt-qemu/blob/master/devstack/libs/libvirt and see if you did anything different18:22
trinaths1sean-k-mooney: oh ok. but as you have said above. i require libvirt 1.3.5 and qemu 2.418:22
openstackgerritMerged openstack/nova master: Make key_pairs use check_deltas() for quota  https://review.openstack.org/47769918:23
sean-k-mooneytrinaths1: do you need exactly those version or are they the minium. fedroas rpms should actully be newer then that but if you are trying to reporduce an issue that wont help you.18:24
trinaths1sean-k-mooney: agree. but libvirt 1.3.5 is mandatory.18:25
trinaths1sean-k-mooney: but qemu can be something different above 2.218:25
sean-k-mooneytrinaths1: it might be worth installing qemu from dnf then instead and see if it resolves the issue.18:27
sean-k-mooneyactully18:27
sean-k-mooneytrinaths1: do you have selinux enabled18:28
sean-k-mooneyits a long shot but that could be blocking the request18:28
*** krtaylor has joined #openstack-nova18:28
*** Sukhdev has joined #openstack-nova18:28
sean-k-mooneytrinaths1: your main error though is internal error: No JSON parser implementation is available18:29
*** annegentle has joined #openstack-nova18:29
trinaths1sean-k-mooney: SELinux status:                 disabled18:29
sean-k-mooneytrinaths1: that is why i taught it might be related to how you compiled libvirt18:29
trinaths1sean-k-mooney: yes. True said. but I find no help from google on this.18:30
trinaths1sean-k-mooney: so finally i have come here.18:30
sean-k-mooneytrinaths1: if you still have it i would check the configure output ans see if there is is any reference to json18:30
trinaths1sean-k-mooney: sure. I'm currently rebuilding libvirt fresh with qemu 2.4 refering the devstack plugin link you have provided.18:32
sean-k-mooneytrinaths1: in our devstack pluging we do "yum-builddep -y libvirt" and "yum-builddep -y qemu" and then install  libnuma-devel gcc make gettext-devel pkgconfig. my guess would be "yum-builddep -y libvirt" is installing a json lib and enableing json support when we run configure and you are missing the lib18:33
trinaths1sean-k-mooney: yes true said. and its giving a parsing error18:36
trinaths1sean-k-mooney: here is libvirt configure summary http://paste.openstack.org/show/616072/18:37
*** pchavva has quit IRC18:37
melwittdansmith: hm, I don't remember. maybe to make them match create_limit and update_limit, which I don't think were called from compute or non-db-having services in the first place either. but I need to think about it more18:39
*** tylerder_ has joined #openstack-nova18:39
*** mnestratov has quit IRC18:39
dansmithmelwitt: well, if you can make them not remotable, then they don't need a version bump and we don't have to have an awkward conversation about the naughtyness that lies within :)18:40
sean-k-mooneytrinaths1: i think you are missing yajl18:40
sean-k-mooneytrinaths1: https://github.com/libvirt/libvirt/blob/e64f2fab920fe8a08e97b790cb5d959f3566aea3/m4/virt-yajl.m4#L24-L4518:40
trinaths1sean-k-mooney: I need that lib installed before libvirt configure ?18:41
sean-k-mooneytrinaths1: i would try installing yajl-devel18:41
sean-k-mooneytrinaths1: yes18:42
*** tylerderosagrund has quit IRC18:42
trinaths1sean-k-mooney: ok. doing that now.18:43
sean-k-mooneytrinaths1: just so you are aware without also installing libnuma-devel you will not be able to use hugepages or cpu pinning18:43
trinaths1sean-k-mooney: agree.18:43
melwittdansmith: heh. I shall sanity check that. I expect they don't need to be remotable18:45
trinaths1sean-k-mooney: its numactl-devel for fedora 2318:45
dansmithmelwitt: cool18:46
sean-k-mooneytrinaths1: ah i just have centos and ubunut vms to hand currently so i assumed it would be the same as centos18:46
*** rtjure has quit IRC18:47
*** yamahata has joined #openstack-nova18:47
*** rtjure has joined #openstack-nova18:47
trinaths1sean-k-mooney: oh okay.18:47
trinaths1sean-k-mooney: even yum is moved to dnf18:48
*** hoonetorg has quit IRC18:49
sean-k-mooneytrinaths1: yes i think you can use dnf on centos too. they have a compaitblity shim on fedora to allow yum to still work but dnf is faster at installing things18:49
trinaths1sean-k-mooney: agree18:50
*** baoli has quit IRC18:52
*** Daisy_ has joined #openstack-nova18:53
*** baoli has joined #openstack-nova18:53
*** fragatina has joined #openstack-nova18:54
*** boris-42____ has joined #openstack-nova18:54
sean-k-mooneytrinaths1: anyway, that is my best quess of how to fix the issue. if you reconfig and recomiple then restart libvirtd then it might solve your issue18:54
*** baoli has quit IRC18:54
*** cdent has quit IRC18:55
trinaths1sean-k-mooney: ok. re-configuring and re-compiling libvirt now.18:55
trinaths1sean-k-mooney: i will share the configure summary in a while.18:55
trinaths1sean-k-mooney: http://paste.openstack.org/show/616077/18:56
sean-k-mooneytrinaths1: well that looks a little better numa and yajl are now enabled.18:57
trinaths1sean-k-mooney: yes. any more options/packages i need to enable to compile ?18:57
*** Daisy_ has quit IRC18:57
*** gszasz has quit IRC18:58
*** jdwidari has joined #openstack-nova18:59
*** jdwidari has quit IRC18:59
sean-k-mooneytrinaths1: from what i can see you should have enough libvires installed for it to work with openstack.18:59
trinaths1sean-k-mooney: did i miss other libraries ?19:00
sean-k-mooneytrinaths1: if you want to use glsterfs or ceph you will need to install extra packages19:00
*** eharney has joined #openstack-nova19:01
sean-k-mooneytrinaths1: but for a normal install i dont think you need any other packages19:01
mriedem_afkildikov: so i think we can remove handling VolumeUnattached from the rest api controller for delete19:01
mriedem_afk(detach_volume)19:01
*** thorst has joined #openstack-nova19:01
*** mriedem_afk is now known as mriedem19:01
trinaths1sean-k-mooney: we did not more to ceph end. i have started re-compile of libvirt. I will update when its completed.19:04
*** baoli has joined #openstack-nova19:04
*** baoli has quit IRC19:04
mriedemildikov: ok comments inline19:08
mriedemwhich are hopefully clear19:08
*** bmace has quit IRC19:10
*** thorst_ has joined #openstack-nova19:11
*** bmace has joined #openstack-nova19:11
*** thorst has quit IRC19:12
openstackgerritJay Pipes proposed openstack/nova master: style-only: s/context/ctx/  https://review.openstack.org/48579119:13
openstackgerritJay Pipes proposed openstack/nova master: use os_traits.MISC_SHARES_VIA_AGGREGATE  https://review.openstack.org/48579219:13
*** baoli has joined #openstack-nova19:14
ildikovmriedem: ah ok, I didn't get to the point of fully removing it from the previous comments as it wasn't introduced in check_detach19:17
ildikovmriedem: thanks for the comments, I'll update the patch19:17
ildikovmriedem: cleaning up is always good, so yeah, let's do it properly19:17
mriedemjaypipes: i for one love trailing parans19:17
mriedem*parens19:17
mriedem'know': {19:18
mriedem    'what': 'i mean?'19:18
mriedem}19:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563419:19
janguttermriedem: fun fact, when I signed my current employment contract, there was an unmatched parenthesis. I still am uncertain whether I am actually employed.19:19
mriedemha19:20
openstackgerritSean Dague proposed openstack/nova master: DNM: WIP adding tests for requestlog  https://review.openstack.org/48579319:21
*** hughsaunders has quit IRC19:26
*** hughsaunders has joined #openstack-nova19:29
*** jwcroppe has quit IRC19:31
*** jwcroppe_ has joined #openstack-nova19:34
*** itlinux has quit IRC19:38
*** cdent has joined #openstack-nova19:41
*** awaugama has quit IRC19:41
*** bbansal has quit IRC19:45
*** nicolasbock has quit IRC19:46
jaypipesmriedem: :)19:49
*** dfisher has left #openstack-nova19:50
*** tylerder_ has quit IRC19:50
*** tylerderosagrund has joined #openstack-nova19:50
*** _pewp_ has quit IRC19:51
*** moshele has joined #openstack-nova19:53
*** tylerderosagrund has quit IRC19:55
*** tylerderosagrund has joined #openstack-nova19:55
*** openstack has joined #openstack-nova19:59
*** Apoorva_ has joined #openstack-nova20:03
*** _pewp_ has joined #openstack-nova20:04
*** cdent has quit IRC20:06
*** rajathagasthya has quit IRC20:06
*** Apoorva has quit IRC20:06
openstackgerritRobert Ellis proposed openstack/nova master: Changed code or added clarifying documentation where nodename is used in ironic driver.  https://review.openstack.org/48580320:09
openstackgerritRobert Ellis proposed openstack/nova master: Clarifying node_uuid usage in ironic driver.  https://review.openstack.org/48580320:13
*** gouthamr has quit IRC20:14
*** armax has quit IRC20:15
*** armax has joined #openstack-nova20:15
*** openstackgerrit has quit IRC20:17
*** bbansal has joined #openstack-nova20:20
*** smatzek has quit IRC20:22
*** hamzy has quit IRC20:22
*** gouthamr has joined #openstack-nova20:27
*** _pewp_ has quit IRC20:31
*** liangy has joined #openstack-nova20:36
*** moshele has quit IRC20:36
*** eharney has quit IRC20:38
*** lyan has quit IRC20:38
*** rajathagasthya has joined #openstack-nova20:41
*** rajathagasthya has quit IRC20:42
*** Daisy_ has joined #openstack-nova20:42
*** gbarros has joined #openstack-nova20:43
*** rajathagasthya has joined #openstack-nova20:43
*** crushil has quit IRC20:46
*** Daisy_ has quit IRC20:47
*** jwcroppe_ has quit IRC20:47
*** jwcroppe has joined #openstack-nova20:51
*** openstackgerrit has joined #openstack-nova20:51
openstackgerritmelanie witt proposed openstack/nova master: Detach volume after deleting instance with no host  https://review.openstack.org/34061420:52
mriedemnova team meeting in 8 minutes20:52
jaypipessupernovas... UNITE!20:53
*** Sukhdev has quit IRC20:53
*** takashin has joined #openstack-nova20:53
openstackgerritSean Dague proposed openstack/nova master: WIP: request_log addition for running under uwsgi  https://review.openstack.org/48560220:53
*** tylerder_ has joined #openstack-nova20:54
*** mgiles has quit IRC20:55
* dansmith thinks jaypipes has been watching too much Power Rangers20:56
jaypipesdansmith: Voltron. :)20:56
jangutterhey, I just got that.20:56
janguttercoffee levels at 10%20:56
dansmithI'20:56
dansmithjaypipes: I'm proud to have no idea what that is20:56
jaypipesheh20:56
*** smatzek has joined #openstack-nova20:57
*** tylerderosagrund has quit IRC20:57
melwittI always thought power rangers were a rip off voltron20:58
mriedemthey are all a rip off of some thing that started in japan20:58
mriedemand are all terrible20:58
janguttermriedem: "Voltron - The End" is amazing though.20:58
*** tylerder_ has quit IRC20:59
melwittnova meeting in one minute20:59
mriedemis the end of voltron better then the end of seinfeld?20:59
mriedembecause that's not hard20:59
janguttermriedem: it's a 4 minute youtube video.20:59
*** gbarros has quit IRC20:59
mriedemno time20:59
mriedemthat's 3:45 too long21:00
*** crushil has joined #openstack-nova21:00
*** vladikr has quit IRC21:01
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup unnecessary logic in os-volume_attachments controller code  https://review.openstack.org/48582321:02
*** pchavva has joined #openstack-nova21:02
*** lucasxu has quit IRC21:02
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094521:03
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094621:04
*** thorst_ has quit IRC21:05
*** hamzy has joined #openstack-nova21:06
*** krtaylor has quit IRC21:07
*** thorst has joined #openstack-nova21:09
*** Apoorva_ has quit IRC21:10
*** Apoorva has joined #openstack-nova21:10
*** catintheroof has quit IRC21:10
*** jwcroppe has quit IRC21:12
*** thorst has quit IRC21:13
*** jwcroppe has joined #openstack-nova21:14
*** itlinux has joined #openstack-nova21:19
*** tbachman has joined #openstack-nova21:19
*** mvk has joined #openstack-nova21:20
cfriesenIf anyone wants a short break from what they're doing, I'd welcome some additional reviewers for https://review.openstack.org/#/c/468203 and https://review.openstack.org/#/c/461456 .  Sahid doesn't really like the proposed tweaks to cpu_realtime_mask but sfinucan is fine with it...would be useful to get a few more opinions.21:22
*** fragatin_ has joined #openstack-nova21:23
*** randomhack has joined #openstack-nova21:23
*** smatzek has quit IRC21:25
*** fragatina has quit IRC21:26
*** gouthamr has quit IRC21:26
*** randomhack has quit IRC21:27
*** dave-mccowan has quit IRC21:31
*** Redcavalier has quit IRC21:31
*** esberglu has quit IRC21:32
*** JoseMello has quit IRC21:36
*** jamesdenton has quit IRC21:38
*** thorst has joined #openstack-nova21:38
*** thorst has quit IRC21:42
melwittdansmith: I removed the remotables from the Quotas object patch. those methods are only called in the quota classes API21:45
melwittthanks for pointing that out21:45
dansmithmelwitt: oh sweet21:45
melwittI must have had remotable fever when I first added them21:45
dansmitha common affliction21:46
melwitthaha, yup!!21:46
*** fragatin_ has quit IRC21:46
mriedemcfriesen: so you mean, take a break from reviewing code to review some code?21:47
*** fragatina has joined #openstack-nova21:47
cfriesenmriedem: but it's different code!21:47
mriedemOH!21:47
dansmithless critical for the release code?21:48
dansmithsign me up.21:48
cfriesenless stressful, more laid-back21:48
mriedemcfriesen: did you miss this while you were on vacation?21:48
mriedemthis abuse21:48
cfriesenall the time.  had to go search out random strangers21:48
mriedemrandom...canadian strangers?21:49
cfriesenno, they're too polite21:49
mriedemright21:49
*** avolkov has quit IRC21:49
cfriesenso are multi-cells actually going to be completely usable in pike?  anything that needs some reviews to make sure that happens?21:50
mriedemthe last big thing is moving quotas to the api21:50
mriedemstarts here https://review.openstack.org/#/c/410945/21:51
*** esberglu has joined #openstack-nova21:51
mriedemcfriesen: and this doesn't have an owner https://bugs.launchpad.net/nova/+bug/170293521:52
openstackLaunchpad bug 1702935 in OpenStack Compute (nova) "Unshelving an offloaded server does not restrict to the original cell" [High,Confirmed]21:52
mriedemotherwise i21:52
mriedemotherwise i'd like to see someone doing some performance testing, maybe with rally, comparing stable/ocata to current master21:52
mriedemnow that we have the counting quotas stuff in and some changes to the filter scheduler for using placement21:53
mriedemreally just try to break things21:53
cfriesenour arch guys have decided to skip ocata and jump from newton right to pike (yeah, I know), so I won't be able to compare against ocata.  We should never have made kilo->mitaka work.21:53
melwittat least you're upgrading? :P21:54
cfriesenthere is that21:55
melwittskipping releases the right way just means you pull the middle code and just run migrations, then move to the next I thought. I would think that shouldn't be too painful or is there more to it?21:55
cfriesenhas anyone looked at allowable latency between the API cell and the compute cells?21:55
*** esberglu has quit IRC21:56
*** xyang1 has quit IRC21:56
cfriesenmelwitt: we're going to have computes running newton live migrating to computes running pike21:56
melwittoh, live, okay21:56
cfriesenI expect, anyway.  since I think we'll probably be doing an OS upgrade too21:56
melwittcfriesen: what do you mean by latency? there's no syncing of data back and forth so there's no latency in that sense21:57
dansmiththe latency requirements are exactly the same as the requirements today between your api services and the database server21:57
*** yamamoto has joined #openstack-nova21:57
cfriesenmelwitt: I mean ping time basically.   given the architecture I wouldn't expect it to be all that sensitive21:58
*** jwcroppe has quit IRC21:58
dansmithi.e. "don't use this for geographically distributed stuff"21:58
cfriesendansmith: that's exactly what I'm wondering about. :)21:58
*** esberglu has joined #openstack-nova22:00
*** yamamoto has quit IRC22:00
*** felipemonteiro_ has quit IRC22:01
*** tbachman has quit IRC22:01
openstackgerritIldiko Vancsa proposed openstack/nova master: Remove check_detach  https://review.openstack.org/44667122:02
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028522:02
*** baoli has quit IRC22:03
ildikovmriedem: VolumeUnattached removed ^^22:04
*** baoli has joined #openstack-nova22:04
*** baoli has quit IRC22:05
*** dave-mccowan has joined #openstack-nova22:06
*** kylek3h has quit IRC22:06
*** kfarr has quit IRC22:10
*** acormier has quit IRC22:16
*** acormier has joined #openstack-nova22:16
*** marst has quit IRC22:19
*** acormier has quit IRC22:21
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.exists notification  https://review.openstack.org/40366022:23
*** dave-mccowan has quit IRC22:24
*** vishwanathj has quit IRC22:26
*** Daisy_ has joined #openstack-nova22:26
*** vishwanathj has joined #openstack-nova22:26
*** mtanino__ has quit IRC22:27
mriedemack, will review later22:27
*** tylerderosagrund has joined #openstack-nova22:28
*** pchavva has quit IRC22:30
*** Daisy_ has quit IRC22:31
*** vishwana_ has joined #openstack-nova22:32
*** tylerderosagrund has quit IRC22:33
*** coreywright has quit IRC22:33
*** jmlowe has joined #openstack-nova22:34
*** vishwanathj has quit IRC22:34
*** tesseract has quit IRC22:37
* melwitt stabs the functional tests22:46
*** baoli has joined #openstack-nova22:50
*** coreywright has joined #openstack-nova22:51
*** annegentle has quit IRC22:51
*** rajathagasthya has quit IRC22:53
imacdonnthink I found a little doc-bug ... https://docs.openstack.org/ocata/config-reference/compute/config-options.html talks about the reserved_host_cpus option, but https://review.openstack.org/#/c/460113/ is only in master (post ocata-release)22:57
*** annegentle has joined #openstack-nova22:58
*** yamamoto has joined #openstack-nova23:01
*** edmondsw has quit IRC23:04
*** takashin has quit IRC23:05
*** bbansal has quit IRC23:06
*** yamamoto has quit IRC23:07
*** bmace has left #openstack-nova23:07
*** edmondsw_ has joined #openstack-nova23:07
*** gouthamr has joined #openstack-nova23:08
*** bbansal has joined #openstack-nova23:08
*** baoli has quit IRC23:09
*** jmlowe has quit IRC23:10
*** edmondsw_ has quit IRC23:11
*** Sukhdev has joined #openstack-nova23:15
*** Sukhdev has quit IRC23:15
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356623:15
jaypipesmriedem, dansmith, edleafe: k, fixed up comment from review ^23:16
*** esberglu has quit IRC23:16
*** Sukhdev has joined #openstack-nova23:16
*** jmlowe has joined #openstack-nova23:21
*** acormier_ has joined #openstack-nova23:23
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356623:24
jaypipesforgot to git add the releasenote...23:24
jaypipesdone.23:24
*** jaypipes has quit IRC23:25
*** acormier_ has quit IRC23:28
*** edmondsw has joined #openstack-nova23:29
*** _ix has quit IRC23:31
*** jwcroppe has joined #openstack-nova23:33
*** armax has quit IRC23:33
*** edmondsw has quit IRC23:33
*** rajathagasthya has joined #openstack-nova23:35
*** thorst has joined #openstack-nova23:37
*** Swami has quit IRC23:37
*** Apoorva_ has joined #openstack-nova23:37
*** thorst has quit IRC23:38
*** chyka has quit IRC23:39
*** Apoorva has quit IRC23:41
*** _ix has joined #openstack-nova23:44
*** annegentle has quit IRC23:48
*** Sukhdev has quit IRC23:48
*** liangy has quit IRC23:49
*** bbansal has quit IRC23:51
*** rajathagasthya has quit IRC23:53
*** rajathagasthya has joined #openstack-nova23:54
*** david-lyle has quit IRC23:54
mriedemimacdonn: hmm, weird23:57
mriedemyeah, i'm not really sure how those config docs are generated23:58
mriedemstephenfin: ^?23:58
imacdonnheh, yeah, I was just going to say the same thing (about the process for generating docs)23:58
mriedemi'm asking in #openstack-doc23:59
imacdonnok23:59
mriedemthose are all getting moved into nova now rather than openstack-manuals23:59
mriedembut stephenfin has been the one working on it23:59
*** oomichi has quit IRC23:59

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