Friday, 2017-07-21

*** rajathagasthya has quit IRC00:00
*** jwcroppe has quit IRC00:00
*** oomichi has joined #openstack-nova00:02
*** ckopper has quit IRC00:02
*** yamamoto has joined #openstack-nova00:03
*** david-lyle has joined #openstack-nova00:03
*** rajathagasthya has joined #openstack-nova00:03
*** Apoorva_ has quit IRC00:04
*** Apoorva has joined #openstack-nova00:05
*** yingjun has joined #openstack-nova00:05
*** s1061123_ has quit IRC00:06
*** yamamoto has quit IRC00:08
openstackgerritMonty Taylor proposed openstack/python-novaclient master: WIP Use keystoneauth for version discovery  https://review.openstack.org/48586800:09
*** xyang1 has joined #openstack-nova00:10
openstackgerritMerged openstack/nova master: Fix and optimize external_events for multiple cells  https://review.openstack.org/44514200:13
*** gmann has quit IRC00:13
*** gmann has joined #openstack-nova00:13
*** _ix has quit IRC00:15
*** itlinux has quit IRC00:17
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563400:17
openstackgerritFelipe Monteiro proposed openstack/nova master: Update policy description for 'instance_actions'  https://review.openstack.org/48460900:20
*** s1061123 has joined #openstack-nova00:23
*** ckopper has joined #openstack-nova00:24
trinaths1sean-k-mooney: ping00:25
trinaths1sean-k-mooney: http://paste.openstack.org/show/616099/  it went well.  but there was a iptables issues.00:26
openstackgerritSteve Noyes proposed openstack/nova master: update live migration to use v3 cinder api  https://review.openstack.org/46398700:26
*** kylek3h has joined #openstack-nova00:28
*** kylek3h has quit IRC00:29
*** kylek3h has joined #openstack-nova00:29
*** zhurong has joined #openstack-nova00:31
*** bbansal has joined #openstack-nova00:32
*** masber has quit IRC00:33
*** masber has joined #openstack-nova00:33
*** acormier has joined #openstack-nova00:35
*** marst has joined #openstack-nova00:35
*** acormier has quit IRC00:35
*** acormier has joined #openstack-nova00:35
*** mtanino has quit IRC00:35
*** thorst has joined #openstack-nova00:39
openstackgerritmelanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances  https://review.openstack.org/42848100:39
openstackgerritmelanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances  https://review.openstack.org/42850500:39
*** krtaylor has joined #openstack-nova00:39
*** Shunli has joined #openstack-nova00:42
*** thorst has quit IRC00:44
*** sree has joined #openstack-nova00:46
*** bbansal has quit IRC00:48
*** _ix has joined #openstack-nova00:51
*** fragatin_ has joined #openstack-nova00:51
*** sdague has quit IRC00:53
*** fragatina has quit IRC00:54
*** fragati__ has joined #openstack-nova00:54
*** fragatin_ has quit IRC00:55
*** fragatina has joined #openstack-nova00:56
*** fragatina has quit IRC00:57
*** fragatin_ has joined #openstack-nova00:57
*** kiennt has joined #openstack-nova00:57
*** trungnv has joined #openstack-nova00:57
*** fragatin_ has quit IRC00:59
*** fragatina has joined #openstack-nova00:59
*** fragati__ has quit IRC00:59
*** kylek3h has quit IRC00:59
*** dave-mcc_ has joined #openstack-nova01:00
*** ckopper has quit IRC01:01
*** Nel1x has joined #openstack-nova01:01
*** itlinux has joined #openstack-nova01:02
*** fragatina has quit IRC01:03
*** ckopper has joined #openstack-nova01:05
*** ckopper has quit IRC01:07
*** _ix has quit IRC01:08
*** thorst has joined #openstack-nova01:11
*** thorst has quit IRC01:11
*** rajathagasthya has quit IRC01:12
*** vishwanathj has joined #openstack-nova01:12
*** fragatina has joined #openstack-nova01:12
*** vishwana_ has quit IRC01:12
*** bbansal has joined #openstack-nova01:14
*** fragatina has quit IRC01:14
*** gmann_ has joined #openstack-nova01:16
*** ckopper has joined #openstack-nova01:16
*** crushil_ has joined #openstack-nova01:23
*** Daisy_ has joined #openstack-nova01:25
*** armax has joined #openstack-nova01:26
*** ckopper has quit IRC01:26
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup unnecessary logic in os-volume_attachments controller code  https://review.openstack.org/48582301:26
mriedemneed a +W on the change to remove check_detach from the code https://review.openstack.org/#/c/446671/01:29
*** ckopper has joined #openstack-nova01:33
*** sree has quit IRC01:33
*** sree has joined #openstack-nova01:34
*** ckopper has quit IRC01:35
*** smatzek has joined #openstack-nova01:37
openstackgerritTony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222901:39
openstackgerritTony Breeds proposed openstack/nova master: Move the last_bytes util method to the libvirt driver.  https://review.openstack.org/47222801:39
openstackgerritTony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916601:39
openstackgerritTony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197201:39
openstackgerritTony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py  https://review.openstack.org/48590701:39
openstackgerritTony Breeds proposed openstack/nova master: Decpouple the os-brick and nova privsep modules.  https://review.openstack.org/48590801:39
mriedemholy f it's a wild tonyb01:39
tonyb:D01:40
tonybmriedem: You can blame mikal01:40
mriedem"no one is reviewing my privsep thing so you do it and maybe they'll help"01:40
mriedemlike that?01:40
*** armax has quit IRC01:40
*** sree has quit IRC01:41
tonybMore like "why does my privsep stuff fail in the gate, can you help me replicate it"01:41
tonybmriedem: so I did that thing and then started playing01:41
*** crushil_ has quit IRC01:41
mriedemoh i see01:41
* tonyb learned al ot about privsep and how to debug it01:42
mriedemmaybe document that somewhere sometime01:42
mriedemsince no one else knows how01:42
tonybmriedem: If I could work out what to document sure.01:43
trinaths1sean-k-mooney: ping01:44
mriedemtonyb: FAQ?01:46
*** _ix has joined #openstack-nova01:46
mriedemwhen you see x, what does it mean - this is what i found out!01:46
tonybmriedem: Yeah that could work.01:46
tonybmriedem: or just a list of steps I took to narrow it down. and tools I used to test it.  I don't know01:47
tonybmriedem: but I wrote a privsep patch so that good right?01:47
*** sree has joined #openstack-nova01:48
mriedembut what have you done for me lately?01:48
tonybmriedem: I asked you to block oslo.serialization!01:48
*** gmann_ has quit IRC01:48
*** jmlowe has quit IRC01:49
*** ckopper has joined #openstack-nova01:49
mriedempsh01:49
mriedemthat was ages ago01:49
*** itlinux has quit IRC01:49
tonyb:)01:50
*** sree has quit IRC01:53
*** zsli_ has joined #openstack-nova01:53
*** Shunli has quit IRC01:56
*** bbansal has left #openstack-nova01:56
*** yamahata has quit IRC01:56
*** gbarros has joined #openstack-nova01:58
*** acormier_ has joined #openstack-nova01:58
*** yushb has joined #openstack-nova01:59
*** acormier has quit IRC02:01
*** itlinux has joined #openstack-nova02:04
*** baoli has joined #openstack-nova02:06
*** phuongnh has joined #openstack-nova02:06
*** crushil has quit IRC02:07
*** crushil has joined #openstack-nova02:07
*** markvoelker has quit IRC02:09
*** markvoelker has joined #openstack-nova02:09
*** mtanino_ has joined #openstack-nova02:12
*** markvoelker has quit IRC02:14
*** chyka has joined #openstack-nova02:14
*** acormier_ has quit IRC02:21
*** crushil has quit IRC02:22
*** crushil has joined #openstack-nova02:22
*** yangyapeng has joined #openstack-nova02:31
*** yushb has quit IRC02:33
*** yushb has joined #openstack-nova02:34
*** yamamoto has joined #openstack-nova02:36
*** crushil has quit IRC02:36
*** gmann_ has joined #openstack-nova02:39
*** coreywright has quit IRC02:39
*** gouthamr has quit IRC02:40
*** dave-mcc_ has quit IRC02:42
*** yamahata has joined #openstack-nova02:42
*** yingjun has quit IRC02:44
*** Nel1x has quit IRC02:46
*** felipemonteiro has joined #openstack-nova02:48
*** sree has joined #openstack-nova02:50
*** yingjun has joined #openstack-nova02:51
*** coreywright has joined #openstack-nova02:52
*** yufei has joined #openstack-nova02:54
*** smatzek has quit IRC03:04
*** r-daneel has joined #openstack-nova03:05
*** yamamoto has quit IRC03:08
*** baoli has quit IRC03:09
*** jwcroppe has joined #openstack-nova03:10
*** thorst has joined #openstack-nova03:12
*** xyang1 has quit IRC03:12
*** fragatina has joined #openstack-nova03:12
openstackgerritlu.li proposed openstack/python-novaclient master: Update the documentation link for doc migration  https://review.openstack.org/48592603:13
*** Sukhdev has joined #openstack-nova03:14
*** fragatina has quit IRC03:16
*** trinaths1 has left #openstack-nova03:17
*** trinaths1 has quit IRC03:17
*** thorst has quit IRC03:17
*** armax has joined #openstack-nova03:18
openstackgerritMerged openstack/nova master: Remove improper LOG.exception() calls in placement  https://review.openstack.org/48572603:19
*** ckopper has quit IRC03:22
*** gbarros has quit IRC03:26
*** gbarros has joined #openstack-nova03:27
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563403:28
*** gmann_ has quit IRC03:30
*** tylerderosagrund has joined #openstack-nova03:30
openstackgerrityuanyue proposed openstack/nova master: Whether to use configdrive disk depends on instance.configdrive  https://review.openstack.org/48593003:32
*** tylerderosagrund has quit IRC03:35
*** mriedem has quit IRC03:35
*** sree has quit IRC03:36
*** sree has joined #openstack-nova03:36
*** sree has quit IRC03:41
*** gcb has quit IRC03:49
*** gcb has joined #openstack-nova03:52
*** Daisy_ has quit IRC03:54
*** Daisy_ has joined #openstack-nova03:54
*** jwcroppe has quit IRC03:56
*** fragatina has joined #openstack-nova03:56
*** fragatina has quit IRC03:56
*** jwcroppe has joined #openstack-nova03:57
*** fragatina has joined #openstack-nova03:57
*** mtanino_ has quit IRC03:57
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356603:59
*** mtanino_ has joined #openstack-nova03:59
openstackgerritTony Breeds proposed openstack/nova master: Decouple the os-brick and nova privsep modules.  https://review.openstack.org/48590804:00
openstackgerritTony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222904:00
openstackgerritTony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916604:00
openstackgerritTony Breeds proposed openstack/nova master: Move the last_bytes util method to the libvirt driver.  https://review.openstack.org/47222804:00
openstackgerritTony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py  https://review.openstack.org/48590704:00
openstackgerritTony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197204:00
*** jwcroppe has quit IRC04:01
*** Daisy_ has quit IRC04:02
*** Daisy_ has joined #openstack-nova04:03
*** gouthamr has joined #openstack-nova04:05
*** yamahata has quit IRC04:06
openstackgerritTony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222904:06
openstackgerritTony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916604:06
openstackgerritTony Breeds proposed openstack/nova master: Move the last_bytes util method to the libvirt driver.  https://review.openstack.org/47222804:06
openstackgerritTony Breeds proposed openstack/nova master: Do not import nova.conf into nova/exception.py  https://review.openstack.org/48590704:06
openstackgerritTony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197204:06
*** yamahata has joined #openstack-nova04:07
*** gbarros has quit IRC04:07
*** Daisy_ has quit IRC04:08
*** mtanino_ has quit IRC04:10
*** markvoelker has joined #openstack-nova04:10
*** yushb has quit IRC04:13
*** yushb has joined #openstack-nova04:13
*** adisky__ has joined #openstack-nova04:18
*** acormier has joined #openstack-nova04:21
*** yushb has quit IRC04:22
*** _ix has quit IRC04:24
*** acormier has quit IRC04:26
*** yushb has joined #openstack-nova04:28
*** sree has joined #openstack-nova04:30
*** hoonetorg has joined #openstack-nova04:32
*** Daisy_ has joined #openstack-nova04:34
*** yufei has left #openstack-nova04:34
*** yufei has joined #openstack-nova04:34
*** chyka has quit IRC04:37
*** Daisy_ has quit IRC04:38
*** yamamoto has joined #openstack-nova04:39
*** felipemonteiro has quit IRC04:40
*** hshiina has joined #openstack-nova04:43
*** markvoelker has quit IRC04:45
openstackgerrityuanyue proposed openstack/nova master: Whether to use configdrive disk depends on instance.configdrive  https://review.openstack.org/48593004:46
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48595004:46
*** Daisy_ has joined #openstack-nova04:46
*** sridharg has joined #openstack-nova04:48
*** yamamoto has quit IRC04:48
*** psachin has joined #openstack-nova04:49
*** yamamoto has joined #openstack-nova04:51
*** nicolasbock has joined #openstack-nova04:54
openstackgerritMerged openstack/nova master: Make security_group_rules use check_deltas() for quota  https://review.openstack.org/47770004:54
*** sree has quit IRC04:56
*** sree has joined #openstack-nova04:57
*** syqian_ has quit IRC04:58
*** syqian_ has joined #openstack-nova04:58
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538504:59
openstackgerritiswarya vakati proposed openstack/nova master: Removed unused 'wrap' property  https://review.openstack.org/48146505:01
*** sree has quit IRC05:01
*** r-daneel has quit IRC05:01
*** r-daneel has joined #openstack-nova05:02
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563405:03
*** thorst has joined #openstack-nova05:13
*** fragatina has quit IRC05:15
*** yamamoto has quit IRC05:15
*** jjung has joined #openstack-nova05:16
*** thorst has quit IRC05:18
*** armax has quit IRC05:22
*** rcernin has quit IRC05:22
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563405:23
*** Sukhdev has quit IRC05:26
*** yufei has quit IRC05:27
openstackgerritTony Breeds proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222905:27
openstackgerritTony Breeds proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916605:27
openstackgerritTony Breeds proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197205:27
*** Daisy_ has quit IRC05:29
*** yushb has quit IRC05:33
*** rcernin has joined #openstack-nova05:36
*** abhishekk has joined #openstack-nova05:36
*** cfriesen has quit IRC05:38
*** markvoelker has joined #openstack-nova05:42
*** kiennt has quit IRC05:44
*** yangyapeng has quit IRC05:46
*** trinaths has joined #openstack-nova05:46
*** yangyapeng has joined #openstack-nova05:47
*** yufei has joined #openstack-nova05:47
*** syqian__ has joined #openstack-nova05:47
*** alex_xu has quit IRC05:48
*** alex_xu has joined #openstack-nova05:49
*** Daisy_ has joined #openstack-nova05:50
*** syqian_ has quit IRC05:51
*** yushb has joined #openstack-nova05:53
*** esberglu has joined #openstack-nova05:54
*** esberglu has quit IRC05:54
*** esberglu has joined #openstack-nova05:54
*** alex_xu has quit IRC05:56
*** alex_xu has joined #openstack-nova05:56
*** nicolasbock has quit IRC05:58
*** esberglu has quit IRC05:59
*** nicolasbock has joined #openstack-nova06:00
*** kiennt has joined #openstack-nova06:03
*** jwcroppe has joined #openstack-nova06:05
*** larainema has quit IRC06:07
*** syqian__ has quit IRC06:10
*** markvoelker has quit IRC06:14
*** psachin has quit IRC06:16
*** andreas_s has joined #openstack-nova06:17
openstackgerritMerged openstack/nova master: Use plain routes list for os-volume-attachments instead of stevedore  https://review.openstack.org/48482406:19
openstackgerritMerged openstack/nova master: Use plain routes list for os-snapshots instead of stevedore  https://review.openstack.org/48482506:20
*** s1061123_ has joined #openstack-nova06:22
*** s1061123 has quit IRC06:22
*** sree has joined #openstack-nova06:24
*** thorst has joined #openstack-nova06:27
*** sree has quit IRC06:28
*** phuongnh has quit IRC06:30
*** Oku_OS-away is now known as Oku_OS06:30
*** phuongnh has joined #openstack-nova06:31
*** thorst has quit IRC06:31
*** ralonsoh has joined #openstack-nova06:32
*** jhesketh has quit IRC06:35
openstackgerritMerged openstack/nova master: Use plain routes list for extension_info instead of stevedore  https://review.openstack.org/48482606:35
*** jhesketh has joined #openstack-nova06:35
*** jwcroppe has quit IRC06:36
*** gouthamr has quit IRC06:36
trinathsdid any one attend this event, https://www.meetup.com/Openstack-Boston/events/241558725/06:37
*** links has joined #openstack-nova06:37
*** sree has joined #openstack-nova06:37
*** sneti has quit IRC06:38
*** pumaranikar has quit IRC06:38
*** raj_singh has quit IRC06:38
*** siva_krishnan has quit IRC06:38
*** aunnam has quit IRC06:38
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add datapath type information to OVS vif objects  https://review.openstack.org/47489206:39
*** sree has quit IRC06:41
*** anilvenkata has joined #openstack-nova06:43
*** abhishek_k has joined #openstack-nova06:44
anilvenkataHi I need help,06:45
anilvenkata I am trying a  multinode devstack setup.06:45
*** ralonsoh has quit IRC06:45
anilvenkata In controller I have all the services.06:45
anilvenkata In compute, I am enabling "ENABLED_SERVICES=n-cpu,rabbit,neutron,q-agt,q-dhcp,q-l3" will that be sufficient to spawn vms in compute?06:45
*** ralonsoh has joined #openstack-nova06:46
anilvenkatawhen I try "nova hypervisor-list" , it is showing only controller and not compute host06:46
*** abhishekk has quit IRC06:47
*** yamahata has quit IRC06:50
trinathsanilvenkata: is u r compute node able to communicate with services in controller node.06:56
anilvenkatatrinaths, neutron services on compute are able to communicate with controller06:56
*** ralonsoh has quit IRC06:57
trinathsanilvenkata: nova-compute ?06:57
anilvenkatatrinaths, no06:57
anilvenkatatrinaths, on compute, I am not evening seeing nova service i.e ps -aef | grep nova06:57
anilvenkatathat is empty06:58
*** rcernin has quit IRC06:58
trinathsanilvenkata: mean in u r devstack localrc u did not enable nova  ?06:58
*** ralonsoh has joined #openstack-nova06:58
*** rcernin has joined #openstack-nova06:58
anilvenkatatrinaths, I enabled n-cpu06:58
anilvenkatawill that be sufficient?06:58
*** yufei has quit IRC06:59
trinathsanilvenkata: yes.07:00
*** dtantsur|afk is now known as dtantsur07:00
trinathsanilvenkata: use OFFLINE=no and RECLONE=yes and re stack it.07:00
trinathsanilvenkata: n-cpu must fix that.07:00
anilvenkatatrinaths, sure, thanks trinath07:01
trinathsanilvenkata: use n-api too07:01
anilvenkatatrinaths, why is it needed?07:01
trinathsanilvenkata: it will download the nova code. try it07:02
anilvenkatatrinaths, ok07:02
*** phuongnh has quit IRC07:03
*** yufei has joined #openstack-nova07:04
*** tylerderosagrund has joined #openstack-nova07:07
*** phuongnh has joined #openstack-nova07:07
openstackgerritjunbo.li proposed openstack/nova master: [DOC BLD FIX] Correct reST field lists in docstrings  https://review.openstack.org/48601007:07
*** vgadiraj has quit IRC07:08
*** tylerderosagrund has quit IRC07:11
*** markvoelker has joined #openstack-nova07:11
*** trungnv has quit IRC07:12
*** psachin has joined #openstack-nova07:13
*** tesseract has joined #openstack-nova07:16
*** zhurong has quit IRC07:18
*** bhagyashris has quit IRC07:18
*** trinaths is now known as trinaths_away07:19
*** yamamoto has joined #openstack-nova07:20
*** alexchadin has joined #openstack-nova07:26
*** itlinux has quit IRC07:26
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949307:31
*** VAhl has joined #openstack-nova07:33
openstackgerritJan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement  https://review.openstack.org/48345907:35
*** _ix has joined #openstack-nova07:36
*** yufei has quit IRC07:41
*** yufei has joined #openstack-nova07:42
*** yamamoto has quit IRC07:42
*** trinaths_away is now known as trinaths07:44
*** markvoelker has quit IRC07:45
*** yufei has left #openstack-nova07:45
*** yushb has quit IRC07:48
*** damien_r has joined #openstack-nova07:49
*** phuongnh has quit IRC07:49
*** yushb has joined #openstack-nova07:51
*** damien_r has quit IRC07:53
jangutterWould someone be able to trigger a +W on https://review.openstack.org/#/c/485634/ ?07:54
*** damien_r has joined #openstack-nova07:55
*** jwcroppe has joined #openstack-nova07:57
*** zhurong has joined #openstack-nova07:58
*** lucas-afk is now known as lucasagomes08:03
*** rmart04 has joined #openstack-nova08:04
*** jwcroppe has quit IRC08:07
*** dims has quit IRC08:09
*** dims has joined #openstack-nova08:11
openstackgerritsahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs  https://review.openstack.org/48552208:12
*** dims has quit IRC08:21
*** yushb has quit IRC08:21
*** dims has joined #openstack-nova08:23
*** thorst has joined #openstack-nova08:28
*** Apoorva_ has joined #openstack-nova08:31
*** _ix has quit IRC08:32
*** thorst has quit IRC08:33
*** mnestratov has joined #openstack-nova08:34
*** Apoorva has quit IRC08:34
*** Daisy_ has quit IRC08:39
*** Daisy_ has joined #openstack-nova08:40
*** derekh has joined #openstack-nova08:42
*** markvoelker has joined #openstack-nova08:42
*** Daisy_ has quit IRC08:44
*** avolkov has joined #openstack-nova08:46
*** Daisy_ has joined #openstack-nova08:46
*** Daisy_ has quit IRC08:50
*** gaurangt has quit IRC08:51
openstackgerritjunbo.li proposed openstack/nova master: [DOC BLD FIX] Correct reST field lists in docstrings  https://review.openstack.org/48601008:51
*** Kevin_Zheng has quit IRC08:51
*** Kevin_Zheng has joined #openstack-nova08:52
*** Daisy_ has joined #openstack-nova08:52
*** gaurangt has joined #openstack-nova08:53
*** esberglu has joined #openstack-nova08:54
*** Daisy_ has quit IRC08:57
*** acormier has joined #openstack-nova08:58
*** Daisy_ has joined #openstack-nova08:58
*** esberglu has quit IRC08:59
*** Daisy_ has quit IRC08:59
*** Daisy_ has joined #openstack-nova08:59
*** cdent has joined #openstack-nova09:02
*** acormier has quit IRC09:02
*** trinaths has left #openstack-nova09:04
openstackgerritsahid proposed openstack/nova master: Revert "Convert HostState.limits['numa_topology'] to primitive"  https://review.openstack.org/48605709:06
openstackgerritsahid proposed openstack/nova master: filter: NUMATopologyLimits to a primitve prior to pass it to HostState  https://review.openstack.org/48605809:06
*** zsli_ has quit IRC09:06
*** zsli_ has joined #openstack-nova09:06
*** zsli__ has joined #openstack-nova09:12
*** yingjun has quit IRC09:14
*** zsli_ has quit IRC09:16
*** markvoelker has quit IRC09:16
openstackgerritSylvain Bauza proposed openstack/nova master: Destroy the ReqSpec object when deleting the instance  https://review.openstack.org/39106009:18
*** bhagyashris has joined #openstack-nova09:18
*** lpetrut has joined #openstack-nova09:23
openstackgerritAlex Xu proposed openstack/nova master: Remove the unittest for plugin framework  https://review.openstack.org/48506009:24
openstackgerritAlex Xu proposed openstack/nova master: Using plain routes for the microversions test  https://review.openstack.org/48506109:24
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore  https://review.openstack.org/48501109:24
openstackgerritAlex Xu proposed openstack/nova master: Remove the useless extension block_device_mapping_v1 object  https://review.openstack.org/48606909:24
openstackgerritAlex Xu proposed openstack/nova master: Remove the code related to extension loading  https://review.openstack.org/48607009:24
openstackgerritAlex Xu proposed openstack/nova master: Move the note about '/os-volume_boot' to the correct place  https://review.openstack.org/48607109:24
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028509:25
*** Daisy_ has quit IRC09:26
*** Daisy_ has joined #openstack-nova09:27
alex_xugmann: sfinucan ^ last few patches for no more extension09:28
* stephenfin checking09:28
stephenfinalex_xu: It's stephenfin now ;)09:28
openstackgerritGábor Antal proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485109:29
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification  https://review.openstack.org/48021409:29
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification  https://review.openstack.org/47392909:29
alex_xustephenfin: ok :) why change a new name09:29
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508109:29
openstackgerritGábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification  https://review.openstack.org/46257609:29
*** yushb has joined #openstack-nova09:30
stephenfinalex_xu: So people stop dropping the 'e' from my last name (Finucane) :) Also, so I can use the same name _everywhere_ (GitHub, Twitter, etc.)09:30
*** Daisy_ has quit IRC09:31
openstackgerritZhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification  https://review.openstack.org/45949309:32
*** esberglu has joined #openstack-nova09:32
alex_xustephenfin: haha09:32
*** esberglu has quit IRC09:32
*** esberglu has joined #openstack-nova09:32
*** sambetts|afk is now known as sambetts09:33
*** esberglu has quit IRC09:37
*** jjung has quit IRC09:44
*** zsli__ has quit IRC09:44
*** jwcroppe has joined #openstack-nova09:46
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661209:47
*** yushb has quit IRC09:49
smcginnisI'm a little confused on a Nova unit test failure we are getting on the version bump of the cinderclient.09:51
smcginnishttp://logs.openstack.org/03/485903/2/check/gate-cross-nova-python27-ubuntu-xenial/bd55208/console.html#_2017-07-21_05_00_42_04909409:51
smcginnisThe Nova test code passes skip_version_check=True09:51
smcginnishttps://github.com/openstack/nova/blob/master/nova/tests/unit/volume/test_cinder.py#L85109:51
*** amuresan has joined #openstack-nova09:51
smcginnisBut then it blows up on checking the microversion here:09:52
smcginnishttps://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L14109:52
*** kiennt has quit IRC09:53
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API  https://review.openstack.org/48222609:53
*** jwcroppe has quit IRC09:55
*** klipka has joined #openstack-nova09:56
*** yushb has joined #openstack-nova09:58
*** larainema has joined #openstack-nova10:00
*** yushb has quit IRC10:01
*** elod has quit IRC10:01
*** jjung has joined #openstack-nova10:02
*** slagle has quit IRC10:04
*** sapcc-bot has quit IRC10:06
*** sdague has joined #openstack-nova10:08
fricklersmcginnis: you are looking at the wrong test is seems, this one is failing: https://github.com/openstack/nova/blame/master/nova/tests/unit/volume/test_cinder.py#L826 . does the new python-cinderclient provide a different 3.x range only?10:09
smcginnisfrickler: Shoot, you're right. Don't know how I messed that up when the line number is right in the traceback.10:11
fricklersmcginnis: ah, this is probably the default version changed here: https://github.com/openstack/nova/blame/master/nova/tests/unit/volume/test_cinder.py#L82310:11
*** alexchadin has quit IRC10:11
smcginnisfrickler: MAX_VERSION is now 3.40. Hmm.10:12
*** markvoelker has joined #openstack-nova10:12
fricklersmcginnis: seems wrong from nova to make assuptions about that being 3.27 permanently10:13
*** jaypipes has joined #openstack-nova10:13
smcginnisfrickler: Well, if I understand this correctly, unless we remove that microversion it should always be there.10:14
smcginnisfrickler: Looking at the check code to see what it's doing now.10:14
smcginnisLooks like we may still have a bug in python-cinderclient.10:19
smcginnisIt was supposed to have been fixed here it was treating version as a float.10:19
smcginnisSo 3.40 becomes 3.4, which makes the range checking off.10:19
fricklersmcginnis: the test mocks the answer in L823 by using the constant from python-cinderclient, so it will be 3.4. but it compares that result to the fixed string "3.27". so IMO either the mock or the comparison should be fixed10:21
smcginnisfrickler: Ah, you're right. Line 823 calls float() on that.10:22
smcginnisfrickler: I think the test needs to be fixed then.10:22
smcginnisfrickler: Thanks, I can fix that quick.10:22
smcginnisfrickler: Although I obviously need more coffee based on the number of obvious things you've had to point out to me. :)10:22
*** Apoorva_ has quit IRC10:28
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce basic framework for security proxying  https://review.openstack.org/34539610:28
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication  https://review.openstack.org/34539710:28
openstackgerritStephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme  https://review.openstack.org/34539810:28
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539910:28
*** cdent has quit IRC10:29
*** thorst has joined #openstack-nova10:29
openstackgerritSean McGinnis proposed openstack/nova master: Don't case cinderclient microversions to float  https://review.openstack.org/48609610:32
*** thorst has quit IRC10:33
openstackgerritSean McGinnis proposed openstack/nova master: Don't cast cinderclient microversions to float  https://review.openstack.org/48609610:33
*** cdent has joined #openstack-nova10:36
sdaguehey alex_xu, you still around today? If so, I'd like to get your take on - https://review.openstack.org/#/c/485602/10:36
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775110:36
*** yangyapeng has quit IRC10:43
*** sapcc-bot has joined #openstack-nova10:45
*** artom_ has quit IRC10:45
*** mkoderer_ has joined #openstack-nova10:45
*** dgonzalez_ has joined #openstack-nova10:45
*** markvoelker has quit IRC10:45
*** mkoderer_ has quit IRC10:47
*** dgonzalez_ has quit IRC10:47
*** yamamoto_ has joined #openstack-nova10:48
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache  https://review.openstack.org/47277310:49
*** zhurong has quit IRC10:50
*** elod has joined #openstack-nova10:55
*** dtantsur is now known as dtantsur|bbl10:57
cdentbauzas: is this still alive: https://review.openstack.org/#/c/427200/11:01
*** elod has quit IRC11:03
*** hieulq has quit IRC11:09
*** gcb has quit IRC11:14
*** rtjure has quit IRC11:16
*** trinaths1 has joined #openstack-nova11:20
jangutterWould someone be able to trigger +W on https://review.openstack.org/#/c/485634/ <--- updated from global requirements11:23
*** yamamoto_ has quit IRC11:32
*** jwcroppe has joined #openstack-nova11:35
*** elod has joined #openstack-nova11:36
*** hferenc has quit IRC11:38
sdaguejangutter: done11:42
*** markvoelker has joined #openstack-nova11:43
janguttersdague: thanks! as a bit of a newbie I am both in awe and scared snotless of the automation here.11:44
*** jwcroppe has quit IRC11:45
jangutterjust one or two more levels of indirection and you can trigger a job to write the code from specs.11:46
*** hferenc has joined #openstack-nova11:46
*** smatzek has joined #openstack-nova11:46
*** acormier has joined #openstack-nova11:46
*** lucasagomes is now known as lucas-hungry11:47
*** yamamoto has joined #openstack-nova11:52
*** tylerderosagrund has joined #openstack-nova11:55
*** yamamoto has quit IRC11:56
*** acormier has quit IRC11:57
*** acormier has joined #openstack-nova11:57
*** thorst has joined #openstack-nova11:59
*** thorst has quit IRC11:59
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification  https://review.openstack.org/48021412:03
*** amuresan has quit IRC12:04
*** thorst has joined #openstack-nova12:04
*** lyan has joined #openstack-nova12:05
*** artom has joined #openstack-nova12:07
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709112:11
*** fragatina has joined #openstack-nova12:14
*** acormier has quit IRC12:14
*** acormier has joined #openstack-nova12:15
*** slagle has joined #openstack-nova12:15
*** markvoelker has quit IRC12:16
bauzascdent: sorry was afk12:19
cdentno problem12:19
bauzascdent: we could yeah, totally12:19
bauzascdent: I just need to resurrect this one12:19
*** acormier has quit IRC12:19
cdentI was going through the links on the prevous rp update and wanted to make sure that anything there was still relevant before creating the new one12:20
*** baoli has joined #openstack-nova12:20
*** markvoelker has joined #openstack-nova12:26
bauzasnp12:28
openstackgerritSean Dague proposed openstack/nova master: request_log addition for running under uwsgi  https://review.openstack.org/48560212:28
*** _ix has joined #openstack-nova12:31
*** Drankis has joined #openstack-nova12:32
openstackgerritAlexandra Settle proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848512:32
*** esberglu has joined #openstack-nova12:33
*** _ix has quit IRC12:35
*** _ix has joined #openstack-nova12:36
*** elod has quit IRC12:36
*** esberglu has quit IRC12:37
*** trinaths1 has quit IRC12:38
*** jmlowe has joined #openstack-nova12:39
*** pchavva has joined #openstack-nova12:39
*** jmlowe has quit IRC12:40
*** dave-mccowan has joined #openstack-nova12:41
*** yamamoto has joined #openstack-nova12:42
*** cdent has quit IRC12:42
*** elod has joined #openstack-nova12:43
*** kylek3h has joined #openstack-nova12:43
*** andreas_s has quit IRC12:43
openstackgerritVladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997512:44
*** edmondsw has joined #openstack-nova12:44
*** mamandle has joined #openstack-nova12:44
*** lucas-hungry is now known as lucasagomes12:45
*** dave-mcc_ has joined #openstack-nova12:45
openstackgerritEd Leafe proposed openstack/nova master: Migrate Ironic Flavors  https://review.openstack.org/48494912:46
*** dave-mccowan has quit IRC12:48
*** xyang1 has joined #openstack-nova12:48
*** edleafe is now known as figleaf12:50
*** yamamoto has quit IRC12:52
*** yamamoto has joined #openstack-nova12:52
figleafleakypipes: Is https://review.openstack.org/#/c/483566/ the last one in your series?12:52
figleafleakypipes: if so, I can start working on adding alternates12:53
*** eharney has joined #openstack-nova12:54
*** esberglu has joined #openstack-nova12:56
*** gbarros has joined #openstack-nova13:01
*** mriedem has joined #openstack-nova13:06
*** adisky__ has quit IRC13:08
*** yangyapeng has joined #openstack-nova13:10
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48563413:13
Dinesh_Bhorstephenfin, mriedem: Hi, wanted to discuss this blueprint: https://blueprints.launchpad.net/nova/+spec/specify-keymap-on-server-boot13:13
stephenfinDinesh_Bhor: Are you referring to the bug I have opened to deprecate those options?13:14
Dinesh_Bhorstephenfin, mriedem: We started work on this But it seems to contradict with this bug: https://bugs.launchpad.net/nova/+bug/1682020/13:14
openstackLaunchpad bug 1682020 in OpenStack Compute (nova) "Remove nova default keymap option for qemu-kvm (deprecated)" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane)13:14
Dinesh_Bhorstephenfin: yes13:15
stephenfinIndeed you are :)13:15
stephenfinRight, so what do you want to know?13:15
Dinesh_Bhorstephenfin: Whether should we proceed work of that blueprint13:15
Dinesh_Bhor?13:15
*** mamandle has quit IRC13:16
*** crushil has joined #openstack-nova13:16
stephenfinDinesh_Bhor: Personally, I wouldn't13:17
stephenfinYou've seen the comment from danpb on that bug13:17
stephenfinWe shouldn't be doing that, and it's breaking stuff for us. The blueprint sounds like it would be putting lipstick on a pig, so to speak :)13:18
openstackgerritSteve Noyes proposed openstack/nova master: update live migration to use v3 cinder api  https://review.openstack.org/46398713:19
Dinesh_Bhorstephenfin: ohh, ohk. Understood. Could you please comment on those patches about your opinion?13:21
Dinesh_Bhorstephenfin: patches submitted against that blueprint13:21
mriedemkeymapping huh, huawei has interest in something similar for vnc consoles i think13:22
mriedembut i'd have to dig up the details13:22
*** randomhack has joined #openstack-nova13:23
mriedemvdrok: a couple of small things in https://review.openstack.org/#/c/419975/ just for docs13:23
vdrokmriedem: thx, updating13:24
*** catintheroof has joined #openstack-nova13:25
*** annegentle has joined #openstack-nova13:25
stephenfinmriedem, vdrok: Docs, you say? That's going to bring merge conflicts for doc-migration13:25
*** alexchadin has joined #openstack-nova13:25
stephenfinI assume it's too important to -2 until doc-migration is, as I've been doing of late?13:25
vdrokstephenfin: it's about comments in code and relnote update I think, so should not affect the doc-migration13:26
*** catinthe_ has joined #openstack-nova13:26
*** yamamoto_ has joined #openstack-nova13:28
*** yamamoto_ has quit IRC13:28
stephenfinvdrok: Yeah, it's just the support-matrix changes that would cause the issue13:28
*** yamamoto has quit IRC13:28
stephenfinvdrok: Unless mriedem disagrees, could I ask you to split that little bit out into a separate patch, and we'll merge it in after?13:28
vdrokstephenfin: oh, ok, I'll check the doc-migration patches then, can rebase on top of the relevant one13:28
stephenfinmaybe not, because I can't be sure they'll get in before feature freeze, and I'm sure you want it in then13:29
*** jwcroppe has joined #openstack-nova13:29
stephenfintwo separate patches would be better, because we can merge docs after feature freeze13:29
stephenfinvdrok: just the support-matrix doc. Release note is fine13:30
*** jmlowe has joined #openstack-nova13:30
vdrokokie13:30
*** catintheroof has quit IRC13:30
stephenfinta :)13:31
*** dtantsur|bbl is now known as dtantsur13:32
*** Drankis has quit IRC13:33
*** lpetrut has quit IRC13:35
*** lucasxu has joined #openstack-nova13:37
jangutterWould anyone have the time to give https://review.openstack.org/#/c/483459 (Netronome VIF type) a quick once-over? 62 lines of code, 290 lines of tests and 42 litres of tears.13:37
*** annegentle has quit IRC13:37
*** jwcroppe has quit IRC13:37
*** _ix has quit IRC13:39
*** links has quit IRC13:43
*** alexchadin has quit IRC13:44
*** Qiming has quit IRC13:46
openstackgerritVladyslav Drok proposed openstack/nova master: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997513:46
openstackgerritVladyslav Drok proposed openstack/nova master: Update ironic feature matrix  https://review.openstack.org/48614813:46
*** yamamoto has joined #openstack-nova13:47
*** nmathew has joined #openstack-nova13:48
*** baoli has quit IRC13:49
*** Qiming has joined #openstack-nova13:50
*** vishwanathj has quit IRC13:50
*** vishwanathj has joined #openstack-nova13:50
*** gouthamr has joined #openstack-nova13:51
*** jwcroppe has joined #openstack-nova13:57
*** dansmith is now known as superdan13:57
stephenfinDinesh_Bhor: done and done13:58
*** awaugama has joined #openstack-nova13:58
*** cfriesen has joined #openstack-nova13:59
stephenfinvdrok: docs patch +2'd14:00
stephenfinI'll leave the code one to mriedem for now14:00
vdrokstephenfin: thanks :)14:00
*** eharney has quit IRC14:02
Dinesh_Bhorstephenfin: thank you so much14:02
*** baoli has joined #openstack-nova14:02
openstackgerritAlexandra Settle proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848514:03
*** felipemonteiro has joined #openstack-nova14:03
*** jwcroppe has quit IRC14:04
*** jwcroppe has joined #openstack-nova14:05
jaypipesfigleaf: yes it is. please do work on alternates, that would be great14:05
jaypipesand thank you!14:05
jaypipeswill give me time to add func tests around failures on compute nodes14:06
*** psachin has quit IRC14:06
openstackgerritSean Dague proposed openstack/nova master: request_log addition for running under uwsgi  https://review.openstack.org/48560214:07
sdaguemriedem: that should be ready to go... I hope ^^^14:07
*** felipemonteiro_ has joined #openstack-nova14:07
openstackgerritFelipe Monteiro proposed openstack/nova master: Update policy description for 'instance_actions'  https://review.openstack.org/48460914:09
figleafjaypipes: on it14:09
*** jaypipes is now known as leakypipes14:09
*** _ix has joined #openstack-nova14:09
*** acormier has joined #openstack-nova14:10
*** felipemonteiro has quit IRC14:11
*** baoli has quit IRC14:12
*** baoli has joined #openstack-nova14:13
*** Daisy_ has joined #openstack-nova14:14
*** jmlowe has quit IRC14:15
*** armax has joined #openstack-nova14:16
*** eharney has joined #openstack-nova14:16
*** baoli has quit IRC14:16
*** baoli has joined #openstack-nova14:17
*** Daisy_ has quit IRC14:23
leakypipesmriedem, superdan, melwitt: super important bug fix: https://review.openstack.org/#/c/485088/614:25
leakypipesit addresses the transient NoValidHosts issue we've been seeing.14:25
leakypipesin some of the functional test failures.14:25
melwittnoice14:26
*** r-daneel has quit IRC14:27
*** mtanino_ has joined #openstack-nova14:28
*** jmlowe has joined #openstack-nova14:29
*** baoli has quit IRC14:33
*** hshiina has quit IRC14:33
leakypipesmriedem: btw, did you see cdent's ? to you on https://review.openstack.org/#/c/483566/?14:35
mriedemjust sat down with coffee for the first time today, so i haven't seen anything14:35
mriedemjust a lot of pings14:35
*** baoli has joined #openstack-nova14:36
mriedemleakypipes: it's a comment on that change14:38
mriedemleakypipes: looking at the grenade failure14:38
*** jkilpatr has joined #openstack-nova14:38
mriedemhttp://logs.openstack.org/66/483566/6/check/gate-grenade-dsvm-neutron-ubuntu-xenial/b0077c3/logs/new/screen-n-sch.txt.gz?level=TRACE#_2017-07-20_23_58_04_58914:38
mriedemwe have 1 node in this job14:38
mriedemthe scheduler goes to submit an allocation and it fails because something else slipped in and changed the inventory at the same time14:39
mriedemwith concurrently running tests in a single node job, if we merge this, i think it's going to kill the gate14:39
leakypipesgotcha14:39
*** efried is now known as fried_rice14:40
mriedemso i think that means if we have exhausted the list of filtered hosts in the filter scheduler,14:40
mriedemwe have to go back up to the scheduler manager and retry by getting a fresh set of allocation candidates14:40
mriedemuntil CONF.num_retries or whatever14:40
leakypipesmriedem: well, we could also retry the same host if we get that specific error.14:40
leakypipesmriedem: the issue is we'd need to put somehting into the claim_resources() report client method to distinguish between 409 Conflict for concurrent update and 409 Conflict for InvalidInventory (which is returned when the capacity was exceeded by another thread and thus the same claim request would fail)14:41
mriedemi'm happy with that,14:41
mriedemi just wasn't sure if we could do it14:42
*** marst has quit IRC14:42
leakypipesmriedem: yup. gimme about an hour. I'll add a dependent patch before that one that adds the error condition distinguishing thing14:43
leakypipesmriedem: and then mod the patch to retry same host on concurrent update14:43
*** jjung has quit IRC14:43
mriedemack14:43
*** _ix has quit IRC14:45
*** baoli has quit IRC14:46
figleafleakypipes: I have a small but significant bug in https://review.openstack.org/#/c/483566/ as long as you're fixing the 409 claim conflict14:46
figleafleakypipes: Also, did we agree that the number of alternates would be based on CONF.scheduler.max_attempts?14:47
leakypipesfigleaf: yeah14:47
figleafok14:47
*** eharney has quit IRC14:48
*** tylerderosagrund has quit IRC14:49
*** tylerderosagrund has joined #openstack-nova14:49
*** marst_ has joined #openstack-nova14:50
*** eharney has joined #openstack-nova14:50
mriedemvdrok: https://review.openstack.org/#/c/419975/18..19/doc/source/support-matrix.ini ?14:51
mriedemyou dropped that in PS1914:51
mriedemotherwise i'd +214:51
*** READ10 has joined #openstack-nova14:53
vdrokmriedem: stephenfin asked to move it to a separate change to avoid conflict with doc migration14:53
vdrokmriedem https://review.openstack.org/48614814:54
openstackgerritMatt Riedemann proposed openstack/nova master: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997514:55
mriedemvdrok: ack, +14:55
mriedem+214:55
mriedemvery simple +W for someone https://review.openstack.org/#/c/419975/14:56
vdrokThanks!14:56
mriedemyw14:56
melwittmriedem: does that one imply we also need to update the hypervisor matrix?14:58
mriedemmelwitt: see ^14:58
mriedem:)14:58
mriedemhttps://review.openstack.org/48614814:58
melwittoh, heh14:59
mriedemstephenfin, the kaiser of docs, asked to move it14:59
*** rmart04 has quit IRC14:59
melwittbrought down the hammer14:59
stephenfinAll Hail Stephen14:59
* stephenfin decided 'heil' was a stretch to far15:00
stephenfin*too15:00
mriedemonly if markus_z is around15:00
melwittmriedem: I didn't notice this till now, but do you think "other" is the right place for this type of release note? or should it be under "upgrade"? https://review.openstack.org/#/c/386008/10/releasenotes/notes/quota-show-detail-access-d6f37282d288fa33.yaml15:03
mriedemmelwitt: sdague asked me about this exact same one earlier in the week :)15:03
*** nmathew has quit IRC15:04
melwittgive me the scoop15:04
mriedemif it were a new rule, other would be fine i think,15:04
mriedemsince it's changing the default for an existing rule, upgrade seems more appropriate15:04
mriedemi think of it like config options15:04
melwittthat's what I thought, I hadn't noticed it was "other" when I +2ed it. guess I'll change it and re +215:05
*** hoonetorg has quit IRC15:05
*** READ10 has quit IRC15:07
openstackgerritmelanie witt proposed openstack/nova master: Change default policy to view quota details  https://review.openstack.org/38600815:10
*** ekhugen- has quit IRC15:10
*** ekhugen has joined #openstack-nova15:10
*** itlinux has joined #openstack-nova15:11
*** patriciadomin has quit IRC15:11
*** randomhack has quit IRC15:12
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48595015:12
*** Oku_OS is now known as Oku_OS-away15:14
*** rcernin has quit IRC15:14
*** rtjure has joined #openstack-nova15:16
*** randomhack has joined #openstack-nova15:18
*** r-daneel has joined #openstack-nova15:20
*** vishwanathj has quit IRC15:22
*** aarefiev is now known as aarefiev_afk15:23
*** vishwanathj has joined #openstack-nova15:23
*** _ix has joined #openstack-nova15:24
*** ekhugen_alt has quit IRC15:34
*** chyka has joined #openstack-nova15:36
*** trinaths has joined #openstack-nova15:38
*** damien_r has quit IRC15:39
*** cdent has joined #openstack-nova15:42
*** mlavalle has joined #openstack-nova15:43
*** klipka has quit IRC15:43
*** yamamoto has quit IRC15:44
*** yamamoto has joined #openstack-nova15:44
mriedemildikov: at some point we'll have to talk about the connection_info stuff going on in https://review.openstack.org/#/c/330285/ because i don't get it15:46
mriedemdid cinder regress that in the api15:46
mriedem?15:46
mriedemi'm not sure why nova needs to stitch things back together15:46
ildikovmriedem: the information that's coming back from Cinder in the new calls is everything in one dict15:47
ildikovmriedem: so there's no nested dict anymore15:47
ildikovmriedem: whatever was under the 'data' key is in connection_info15:47
*** baoli has joined #openstack-nova15:47
mriedemoh15:47
ildikovmriedem: so we put back the 'data' key for now15:47
mriedemok we need to separate that out into a different change then15:47
ildikovmriedem: too much effort to remove it...15:48
ildikovmriedem: it's already separated out15:48
mriedemwhere?15:48
ildikovmriedem: I just try to fix the old flow tests in the attach patch before upload the extended chain15:48
mriedemah ok15:48
*** annegentle has joined #openstack-nova15:48
mriedemok yeah the amount of test change in that patch really scared me15:48
ildikovmriedem: but I can upload where I am now if you want to take a look15:48
mriedemfinish up :)15:49
mriedemi don't have time to dig into it again today probably15:49
ildikovok :)15:49
ildikovI have functional tests failing, which is kinda interesting, but I will figure that out15:49
*** yamamoto has quit IRC15:49
ildikovunit tests are fine now15:49
*** sridharg has quit IRC15:49
ildikovI will upload when it's fixed or when I gave up for today... :)15:50
ildikovand then move on to the new tests15:50
ildikovmriedem: do you have any preference for the new tests?15:50
ildikovmriedem: like kinda duplicate what we have just with the new flow?15:50
mriedemildikov: that's probably what i'd do15:50
mriedemthe functional api sample tests don't need to change at all15:50
ildikovmriedem: do we want less code lines or complete separation?15:50
mriedemor shouldn't need to15:50
mriedemi want separation15:51
ildikovI have 15 tests failing...15:51
mriedemmucking with the old tests (1) loses coverage on the old flow and (2) makes it much harder to review IMO15:51
ildikovok, I prefer that too, so at least we are on the same page15:51
mriedemfwiw that's what i did in the swap volume new style attachments change15:51
ildikovhonestly I wanted to see tests working and deal with finalizing them when we are kinda fine with the code15:52
ildikovmaybe wasn't the best idea15:52
ildikovstill helped understanding a few things though, so oh well15:52
ildikovnext time I will do it differently... :)15:52
ildikovmriedem: I think we're good with the direction, I will ping you if I get this up at a reasonable time today so at least you know it's there15:53
mriedemhttps://bugs.launchpad.net/nova/+bug/170429315:54
openstackLaunchpad bug 1704293 in OpenStack Compute (nova) "We can not set volume's type when creating a vm from image by creating a volume" [Undecided,Won't fix] - Assigned to liuxiuli (liu-lixiu)15:54
mriedemblarg15:54
janguttermriedem: would something like this (create non-standard volume/port/etc and attach it to the created server) ever be covered by the osc CLI?16:00
openstackgerritJay Pipes proposed openstack/nova master: claim resources in placement API during schedule()  https://review.openstack.org/48356616:01
openstackgerritJay Pipes proposed openstack/nova master: placement: add retry tight loop claim_resources()  https://review.openstack.org/48617016:01
mriedemjangutter: that's the way you do it today16:01
leakypipesmriedem: ^16:01
mriedemwhich was my response in the bug report16:01
leakypipesmriedem: I ended up hiding the tight retry loop in the scheduler report client's claim_resources() method.16:02
leakypipesmriedem: so no changes needed to the scheduler patch.16:02
mriedemleakypipes: sneaky sis16:02
* mriedem goes to lunch16:02
*** mriedem is now known as mriedem_afk16:02
leakypipesmriedem_afk: :)16:02
janguttermriedem: argh, sorry, my verbage unclear... I meant, doing that in "one go"16:02
janguttermriedem: I think that's not just a can of worms, but a can of worm factory.16:03
*** itlinux has quit IRC16:03
melwittleakypipes: question for you, if this sort is to get non-shared resources first, does that mean "sharing_providers" has non-shared resources in it? https://review.openstack.org/#/c/485088/6/nova/objects/resource_provider.py@106016:03
melwittjust confused about what's going on there. and how does sorting them make non-shared be first?16:04
melwittcdent ^16:06
cdentmelwitt: the name of the variable might be misleading16:07
cdentmelwitt: just a sec I’ll refresh my memory16:09
*** lucasagomes is now known as lucas-afk16:09
cdentmelwitt: the value in the dictionary is the id of a provider of that resource class that makes it available via sharing16:10
melwittit looks like sharing_providers is a dict where the key is resource class ID and each value is a list of resource provider IDs?16:10
cdentof it items() is sorted by the value, then anything provider which is not one that shares comes first16:11
melwittnon sharing providers always have lower IDs?16:12
cdentno, the value will be None if it a non sharing provider16:14
melwittokay, so the Nones get sorted ahead of the lists of IDs16:14
cdentyeah16:15
cdentand it’s not none, sorry it’s an empty list16:15
cdentit’s hard to track :(16:15
melwittokay, thanks. now this makes a lot more sense16:16
* cdent blinks away the fog16:16
*** mlavalle has left #openstack-nova16:20
*** lyan has quit IRC16:22
*** _ix has quit IRC16:22
*** rtjure has quit IRC16:23
*** _ix has joined #openstack-nova16:24
*** yamahata has joined #openstack-nova16:27
*** itlinux has joined #openstack-nova16:29
cdentleakypipes: can you clarify your comment on https://review.openstack.org/#/c/485209/2/nova/tests/functional/api/openstack/placement/gabbits/shared-resources.yaml . Are you saying it should be 1 provider or something else? It currently returns 0, which doesn’t align with what you seem to be saying16:29
leakypipescdent: it's currently returning 1, which is correct.16:31
leakypipescdent:      $.resource_providers.`len`: 116:31
*** itlinux has quit IRC16:31
cdentit is not returning 1, that’s why the xfail: true is there16:32
*** lpetrut has joined #openstack-nova16:33
cdenteXpected fail16:33
leakypipescdent: oh, sorry16:35
leakypipescdent: didn't catch the xfail thing16:36
leakypipescdent: I can investigate that then.16:36
cdenti’ve got a better version of the test, will push it up without the xfail16:36
cdentleakypipes: I can do it if you like, now that I know what the correct thing is I should be able to some reasonable poking16:37
*** _ix has quit IRC16:37
leakypipescdent: sure, please do if you'd like16:37
*** sdague has quit IRC16:37
cdent16:38
cdentam I slow or is gerrit slow?16:38
figleafcdent: both are possible16:39
figleafleakypipes: making progress on the alternates, and then lost connectivity to my dev env. Will continue after lunch16:39
leakypipesfigleaf: sounds good, thanks Ed16:40
*** greghaynes has quit IRC16:40
*** r-daneel has quit IRC16:47
*** dave-mccowan has joined #openstack-nova16:47
melwittI wonder what the func test job fail rate is, it seems like it's got to be near 50%16:47
*** dave-mcc_ has quit IRC16:50
*** yamamoto has joined #openstack-nova16:50
*** lyan has joined #openstack-nova16:50
leakypipesmelwitt: yes, likely. I'm hoping the bug fix for that join order will help some.16:51
melwittyeah, true. most of the fails I see are "authentication error" and versioned notifications stuff16:53
*** derekh has quit IRC16:54
cdentmelwitt: I think that auth error is part of the compute and placement fixtures not working well together all the time16:56
melwittcdent: yeah, I've heard that. recently mriedem_afk tried adding the keepalive=False globally but it didn't seem to have helped16:57
cdentleakypipes: as far as I can tell the functionality for shared providers on /resource_providers was never implemented, only talked about: it’s talked about in the commit message of this: https://review.openstack.org/#/c/460798/ but I can’t find where it happened16:57
openstackgerritStephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation  https://review.openstack.org/34539916:57
leakypipescdent: heh, yes, that's correct. :)16:57
leakypipescdent: I'd forgotten about that16:57
*** acormier has quit IRC16:58
cdentso I reckon we’ve got at least two options: a) implement it b) decide that actually we never wanted that on resource providers anyway, that shared thing is sort of magical any way for “listing” resource providers, so doesn’t belong there16:58
*** r-daneel has joined #openstack-nova16:59
*** acormier has joined #openstack-nova16:59
cdentleakypipes: I think I actually prefer option b for that url. you?16:59
leakypipescdent: not sure I follow you... GET /resource_providers is intended to return a list of the resource providers that can accomodate the requested resources, and in this case, if one of the requested resources is a shared resource, we naturally need to account for sharing providers (we just don't need to return information that indicates a sharing provider would be used)17:01
*** rajathagasthya has joined #openstack-nova17:01
*** sambetts is now known as sambetts|afk17:02
*** acormier has quit IRC17:03
cdentWhat I’m saying is that maybe (now that we have /allocation_candidates) listing /resource_providers should just do that: list resource providers that match queries, where resources is just another type of query. the concept of a shared resource isn’t necessarily a concept that is “natural” when request a listing of resource provider for no specific purpose17:04
cdent(i’m not sold on this, just throwing it out there for thinking)17:04
*** ralonsoh has quit IRC17:04
*** yamamoto has quit IRC17:05
-openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup.17:06
leakypipescdent: I'm not actually sure what you're advocating :)17:06
cdenti’m advocating leaving /resource_providers as it is right now17:07
cdentleakypipes: or rather not advocating, wondering if it makes sense to do so17:08
leakypipescdent: I see. I don't suppose it would hurt to leave it as-is for a little while as we decide on that question.17:08
cdentfigleaf: you have any thoughts on the last few lines?17:11
*** dtantsur is now known as dtantsur|afk17:11
*** rajathagasthya has quit IRC17:15
*** greghaynes has joined #openstack-nova17:15
*** lpetrut has quit IRC17:16
*** trinaths has left #openstack-nova17:17
*** thorst has quit IRC17:17
*** swebster has joined #openstack-nova17:17
*** swebster_ has quit IRC17:18
*** thorst has joined #openstack-nova17:20
*** Swami has joined #openstack-nova17:21
*** imacdonn has quit IRC17:22
*** imacdonn has joined #openstack-nova17:22
* stephenfin is off on his holiers and will chat to y'all after feature freeze. Toodles17:23
*** dave-mccowan has quit IRC17:23
*** thorst has quit IRC17:24
*** dave-mccowan has joined #openstack-nova17:26
cdenthave fun stephenfin17:27
*** rajathagasthya has joined #openstack-nova17:33
*** mriedem_afk is now known as mriedem17:33
*** kylek3h has quit IRC17:34
mriedemmaybe we should put a dumb retry in the functional osapi client fixture if we get a 401 or 403 response for now?17:35
mriedemas a hack until we're to RC1?17:35
*** jkilpatr has quit IRC17:36
melwittyeah, I forgot about that idea17:38
melwittI don't understand what the right way to fix this would be17:41
*** fragatina has quit IRC17:41
mriedemrm -rf nova/tests/functional17:42
melwitthah17:42
cdentis the auth error the main offender at this point?17:42
mriedemit's what i've been noticing after the global keepalive=False change17:42
cdentcommon url, or multiple urls?17:43
mriedemwhich is odd since we use the noauth middleware i thought in all of the osapi fixture tests17:43
melwittyeah. I don't get it17:43
mriedemi'll find the last one i just rechecked17:43
mriedemhttp://logs.openstack.org/11/485011/4/gate/gate-nova-tox-functional-py35-ubuntu-xenial/42d69de/console.html#_2017-07-21_14_55_31_53290717:44
mriedemi've been seeing these in the functional jobs too17:45
mriedemsys:1: ResourceWarning: unclosed file <_io.FileIO name=1 mode='wb' closefd=True>17:45
*** thorst has joined #openstack-nova17:45
*** randomhack has quit IRC17:46
*** randomhack has joined #openstack-nova17:46
cdentmriedem: is that on both py27 and py35 or or just py35?17:46
mriedemonly seeing it on py317:47
mriedempy3 jobs are also way chattier about warnings17:47
*** anilvenkata has quit IRC17:47
melwittgdi looks like logstash.o.o is busted again17:48
melwittI wanted to check which jobs have "OpenStackApiAuthenticationException: Authentication error"17:48
cdentI get unclosed file errors all over the place in lots of other things besides nova in py3517:48
cdents/errors/warnings/17:48
*** Sukhdev has joined #openstack-nova17:48
mriedemso when we get that auth error we aren't providing the original response text17:48
mriedemi had that working in the functional tests but leakypipes removed it17:49
mriedemand then i raged17:49
leakypipeshmm?17:49
mriedemyou know what you did17:49
mriedemsec17:49
mriedemhttps://github.com/openstack/nova/commit/de8096a59d80d10ff1ccf14e0b345be641ba4f0717:49
mriedemthis is actually a bit different17:50
mriedemdo we have a bug for this anywhere?17:50
cdentwhich this?17:50
mriedemthe random auth failures in functional tests17:50
* cdent doesn’t know17:51
mriedemi don't see one17:51
melwittmriedem: oh, sorry. I think I approved that. I thought bc the jobs were passing it wasn't needed anymore, I didn't know it was a thing to get more info17:51
mriedemyeah it's for debug17:51
*** Daisy_ has joined #openstack-nova17:52
melwittI guess put it back, with a comment that says what it's for17:52
mriedemi assume the "BaseException.message has been deprecated as of Python 2.6" was for something else17:52
mriedemb/c i've seen that before too17:52
mriedemit wouldn't actually help in what we're seeing here17:52
mriedemso i'm going to push something else for that17:52
mriedempush17:52
mriedemit17:52
mriedemreal17:52
melwittI thought that's what was causing those warnings was setting of the message attribute17:53
mriedemi don't remember anymore, i added it here https://github.com/openstack/nova/commit/01dd1a05a213c0cbd0097188418cabe915291c8d17:57
mriedemanywho, not the issue here17:57
leakypipesmriedem: this tempest.api.identity.admin.v3.test_users.UsersV3TestJSON.test_password_history_not_enforced_in_admin_reset failure... grrr.17:58
mriedemi think we have a signature for that one17:59
mriedemhttp://status.openstack.org/elastic-recheck/#1702211 ?17:59
mriedemcdent: melwitt: https://bugs.launchpad.net/nova/+bug/170575317:59
openstackLaunchpad bug 1705753 in OpenStack Compute (nova) "Random OpenStackApiAuthenticationException: Authentication error in nova functional tests" [Undecided,New]17:59
*** Daisy_ has quit IRC17:59
cdentthanks, mriedem18:01
mriedemi'll push a debug patch18:01
cdentmriedem: do you want me to try the retry on auth fail thingie?18:02
mriedemsure18:03
cdentit seems like it might of some use but doesn’t really get at whatever the issue is, sadly18:03
cdentbut yeah, i’ll make one go18:03
melwittI wonder if this is relevant https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1436-L143818:04
mriedemprobably18:04
*** mnestratov has quit IRC18:04
mriedemthe spike in failures started when placement fixture was turned on globally in the IntegratedHelpers mixin18:04
melwittyeah, that lines up with the fact that "The current placement NoAuthMiddleware returns a 401 in case a token is not provided"18:05
melwittI just don't know what that means18:05
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/placement/auth.py#L3218:06
mriedemhttps://github.com/openstack/nova/blob/master/nova/api/openstack/placement/auth.py#L4118:06
*** nicolasbock has quit IRC18:06
cdentif that’s playing a part, then it is likely that the problem is when compute requests land on the placement api (which seems to be the core problem here)18:07
melwittso does that imply that a request is being made that does send the x-auth-token header? is there anything other than GET/PUT/DELETE/POST?18:07
melwitt*does not18:07
melwittoh. compute requests landing on placement api18:07
mriedemi think there is an eventlet switch that goofs things up18:07
mriedemor that's the theory18:07
melwittI guess I don't understand that18:08
mriedemthis is what compute does https://github.com/openstack/nova/blob/master/nova/api/openstack/auth.py#L3218:08
*** nicolasbock has joined #openstack-nova18:08
cdentplacement does what it does to behave like a normal auth middleware and not fake more than it should, it basically stripped that middleware back to the basics18:09
cdentchanging it would not fix the real problem here18:09
cdentit would mask it18:09
cdentand we don’t want to do that do we?18:09
mriedemright we do'nt send a fake token for compute requests https://github.com/openstack/nova/blob/master/nova/tests/functional/api/client.py#L14218:09
melwittI mean, how does something switch to the wrong api, how does a compute request end up going to the placement api18:10
superdanbad threading18:10
cdentmelwitt: the theory is that something is causing eventlet sockets to get confused18:10
superdanyeah18:10
melwitt\:| okay18:11
superdanmelwitt: your hair is messed up?18:11
* superdan tries to decode that emoticon18:11
melwittthat's my raised unibrow18:11
superdaneyebrows?18:12
superdanokay18:12
superdanheh18:12
cdentwe could run one of the apis (presunably placement) on wsgi intercept instead of a separate server thread, and then it wouldn’t be on threads?18:12
cdent(or rather not in the same way)18:12
*** baoli has quit IRC18:13
* melwitt googles18:14
melwittso that means we have two wsgi services total now? it seems like this intercept thing would allow us to set up each one separately (with intercept) right? (because of different host/port combos)18:16
openstackgerritChris Dent proposed openstack/nova master: DNM: retry on authentication failure in api_client  https://review.openstack.org/48619018:16
mriedemas far as i can tell these are started on the same host and port18:17
mriedem127.0.0.1:018:17
melwittjust thinking if it would still work if someday we had a third wsgi service18:17
cdentmelwitt: yes, that’s right, they would18:17
cdent0 means choose a port18:17
cdentmelwitt: yues18:17
melwittbecause we do need a real way to isolate these from each other. cool18:17
mriedem[nova.placement.wsgi.server] (4697) wsgi starting up on http://127.0.0.1:45989'18:18
mriedem[nova.osapi_compute.wsgi.server] (4697) wsgi starting up on http://127.0.0.1:33219'18:18
mriedemok18:18
melwittso I guess we could do the retry for now and then replace it with intercept whenever one of us gets it working18:18
melwittassuming that getting intercept to work might be not easy18:18
cdentmelwitt: it may require unwinding some of the fixture’s pieces, because the deal wsgi intercept is it takes away the need for fakes18:19
cdentfake requests I mean, you use a real http client to make real http requests to a fake socket18:19
mriedemi think i've got an easier workaround for now18:19
* cdent holds his breath18:20
melwittcdent: ah, okay18:20
cdentmelwitt: I think it is probably worth doing regardless of the outcome here, but I’m probably a bit too biased to be the decider on such thing18:20
*** pchavva has quit IRC18:20
cdentmriedem: i’m starting to die from lack of air, halp18:21
melwittcdent: I agree we should do it. just wanted to be clear that I wasn't suggesting we hold off on a workaround because of it18:21
* cdent nods18:22
openstackgerritMerged openstack/nova master: Don't cast cinderclient microversions to float  https://review.openstack.org/48609618:22
* cdent turns blue18:22
mriedemsec18:22
mriedempushing it pu18:22
mriedem*up18:22
melwittmriedem is trying to kill cdent18:22
cdentI knew it18:22
*** baoli has joined #openstack-nova18:23
*** kylek3h has joined #openstack-nova18:25
openstackgerritMatt Riedemann proposed openstack/nova master: Pass X-Auth-Token in TestOpenStackClient._authenticate  https://review.openstack.org/48619318:26
openstackgerritMatt Riedemann proposed openstack/nova master: Pass X-Auth-Token in TestOpenStackClient._authenticate  https://review.openstack.org/48619318:28
mriedemlogstash is back up18:28
melwittf yes18:28
mriedemlanguage18:29
mriedemdoesn't seem to be finding anything thoguh18:32
mriedem*though18:32
cdentmriedem: interesting. I’m not sure your solution will work. It’s trying to prevent the 401 response from placement happening (which it will) but placement’s no auth will not response with the response.headers that the _authenticate method is supposed to provide to its callers18:32
cdentplacement no auth middleware doesn’t not set response headers18:33
cdentso the retry thing that I did is more likely to get the desired outcome, isn’t it? (I’m not entirely sure, the gears withing gears isn’t clear)18:33
cdent(double negative above not intentional)18:34
mriedemyo'ure probably right, because the compute api noauth returns a request context,18:34
mriedemthat the compute api code requires18:34
* cdent nods18:34
mriedemso even though we avoid the 401, the request won't have a context in it18:34
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028518:34
openstackgerritIldiko Vancsa proposed openstack/nova master: Translate the return value of attachment_create and _update  https://review.openstack.org/48619418:34
ildikovmriedem: refactor + fixed old flow tests ^^18:35
*** acormier has joined #openstack-nova18:36
cdentwhatever the outcome, I can investigate the wsgi intercept stuff on monday, at least to spike it to see if it has any potential18:36
mriedemcdent: let's un-DNM yours18:36
mriedemand just push it in18:36
cdentk, on it18:36
*** baoli has quit IRC18:37
*** baoli has joined #openstack-nova18:38
*** mdrabe has quit IRC18:38
*** kylek3h has quit IRC18:39
-openstackstatus- NOTICE: docs.o.o is currently broken, we're investigating18:39
openstackgerritChris Dent proposed openstack/nova master: retry on authentication failure in api_client  https://review.openstack.org/48619018:39
*** baoli has quit IRC18:40
*** fragatina has joined #openstack-nova18:40
openstackgerritMatt Riedemann proposed openstack/nova master: retry on authentication failure in api_client  https://review.openstack.org/48619018:41
cdentpushed that before I saw your coments melwitt and mriedem, got the NOTE anyway by chance, but efect typo still there18:41
mriedemit is?!!?!??!?!18:41
melwittlol18:41
cdentdood, I’m just going to let you do everything henceforth18:41
cdentyou are fleet of foot18:42
mriedemi also have a light step18:42
mriedemyou'll never see it coming18:42
melwittwell, that was fun18:42
mriedemback to business18:43
cdentthis is me when you come for me mriedem https://www.youtube.com/watch?v=vZvbhy5lDgY18:43
melwittnot getting any results in logstash, I see they're talking about some missing entries in #openstack-infra18:44
*** mdrabe has joined #openstack-nova18:44
mriedemi don't want to be the guy from blue velvet18:44
melwitthaha18:44
mriedemthe 02 guy18:44
openstackgerritMerged openstack/nova master: Updated from global requirements  https://review.openstack.org/48563418:44
*** MasterOfBugs has joined #openstack-nova18:45
cdentI’ve always felt a kinship for that guy in the oil, waiting to blow up. I think there ought to be a DSM-5 entry for that specific mental illness.18:45
*** nicolasbock has quit IRC18:45
melwitthah18:46
openstackgerritIldiko Vancsa proposed openstack/nova master: Translate the return value of attachment_create and _update  https://review.openstack.org/48619418:48
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028518:48
*** baoli has joined #openstack-nova18:48
*** sdague has joined #openstack-nova18:48
*** kylek3h has joined #openstack-nova18:48
*** baoli has quit IRC18:49
ildikovmriedem: I cleaned up the above as much as I could, so you can review the refactor and the current content of the attach patch will not change either18:52
ildikovmriedem: I will add separate new tests in the next patch set18:53
*** kylek3h has quit IRC18:53
mriedemok18:53
*** baoli has joined #openstack-nova18:56
*** baoli has quit IRC18:57
*** baoli has joined #openstack-nova18:59
*** annegentle has quit IRC19:00
openstackgerritPeter Hamilton proposed openstack/nova master: WIP Add trusted_certificates to REST API  https://review.openstack.org/48620419:05
*** annegentle has joined #openstack-nova19:06
*** mdrabe has quit IRC19:06
*** acormier has quit IRC19:08
*** acormier has joined #openstack-nova19:08
*** lyan has quit IRC19:10
*** lyan has joined #openstack-nova19:12
*** acormier has quit IRC19:13
figleafcdent: sorry, been battling connectivity issues most of the day19:17
figleafcdent: if I understood you, you'd like to simplify GET /resource_providers to just return those RPs that can satisfy *all* the requested resources themselves, ignoring shared resources19:18
*** tylerder_ has joined #openstack-nova19:18
*** Guest16122 has quit IRC19:18
figleafIf so, then I would agree with that, if 'allocation_candidates' is how we're going to handle the fancy nova-specific stuff19:18
*** egarbade has joined #openstack-nova19:19
*** egarbade is now known as Guest3074119:19
*** crushil has quit IRC19:19
*** yamahata has quit IRC19:20
*** jangutter has quit IRC19:20
*** tylerderosagrund has quit IRC19:21
*** mdrabe has joined #openstack-nova19:21
*** gbarros has quit IRC19:22
*** tylerder_ has quit IRC19:27
*** tylerderosagrund has joined #openstack-nova19:27
melwittI feel like I'm seeing tempest jobs time out fairly often lately too19:28
*** mdrabe has quit IRC19:28
openstackgerritChris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped  https://review.openstack.org/48620819:29
cfriesenmriedem: dansmith: is this ^ the right way to handle the issue?  I need to deal with the testcases still.19:30
*** acormier has joined #openstack-nova19:33
sdaguemelwitt: yeh, I noticed that as well. I haven't gone and looked yet19:34
sdaguemelwitt: you have a link to some?19:35
*** tylerder_ has joined #openstack-nova19:35
melwittsdague: I think this is one, the most recent I've seen http://logs.openstack.org/75/419975/20/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/17f4349/19:35
*** tylerder_ has quit IRC19:36
*** randomhack has quit IRC19:36
*** tylerder_ has joined #openstack-nova19:36
sdaguemelwitt: also, if you are inclined, bringing back request logging under uwsgi - https://review.openstack.org/#/c/48560219:36
melwittsdague: this one says "Request timed out" http://logs.openstack.org/88/485088/6/gate/gate-tempest-dsvm-cells-ubuntu-xenial/17bb8bf/logs/testr_results.html.gz19:38
melwittk, looking19:38
*** tylerderosagrund has quit IRC19:38
*** patriciadomin has joined #openstack-nova19:39
*** ekhugen_alt has joined #openstack-nova19:39
*** vishwana_ has joined #openstack-nova19:41
*** vishwanathj has quit IRC19:41
*** mdrabe has joined #openstack-nova19:42
*** vishwana_ has quit IRC19:50
*** vishwanathj has joined #openstack-nova19:50
cdentfigleaf: that is what it is currently doing19:53
figleafcdent: ah, I thought it was taking shared resources into account, too19:54
*** vishwanathj has quit IRC19:54
cdentit was supposed to be but that was never actually implemented19:54
*** vishwanathj has joined #openstack-nova19:54
figleafcdent: then if it isn't needed, I would prefer to leave it as is19:54
figleafsince we're using allocation_candidates19:54
sdaguemelwitt: yeh, trying to figure out with infra if this is persistent issue or not19:54
*** xyang1 has quit IRC19:55
cdentfigleaf: yeah, that’s kinda where I landed too19:55
*** yamahata has joined #openstack-nova19:55
figleafcdent: maybe later if it is needed, we could add an additional query param to indicate "include shared"19:55
* cdent nods19:56
leakypipesmriedem, cdent, melwitt, superdan, figleaf: I swear I need to hire someone to just keep rechecking my patches through the gate :(19:58
*** Swami has quit IRC19:58
*** xyang1 has joined #openstack-nova19:58
leakypipeshttp://logs.openstack.org/66/483566/8/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/eda58dd/console.html19:58
leakypipeslatest random failure...19:58
melwittyou and me both19:58
melwittrecheck bots19:58
figleafleakypipes: I thought you had a team of lackeys for such mundane tasks19:58
*** catinthe_ has quit IRC19:58
leakypipeshonestly, our gate should see 1 test failure like that out of 1319 tempest tests and say to itself "fuck it, close enough. merge."19:59
melwittheh19:59
* leakypipes is kidding, of course. before someone berates me.19:59
openstackgerritChris Dent proposed openstack/nova master: retry on authentication failure in api_client  https://review.openstack.org/48619020:00
cdentmriedem: you want +w that ^ again, had a pep8 failure20:00
* figleaf was going to back leakypipes on the "close enough" patch20:00
*** dave-mccowan has quit IRC20:02
openstackgerritMerged openstack/nova master: Change default policy to view quota details  https://review.openstack.org/38600820:02
cdentleakypipes: that auth retry thing _may_ help the gate20:02
leakypipescdent: +Wallaby'd20:03
melwittwhoa something merged, amaze20:04
*** acormier has quit IRC20:04
*** acormier has joined #openstack-nova20:05
*** vishwana_ has joined #openstack-nova20:07
*** vishwanathj has quit IRC20:07
*** acormier_ has joined #openstack-nova20:08
openstackgerritEd Leafe proposed openstack/nova master: WIP - add alternate hosts  https://review.openstack.org/48621520:09
*** acormier has quit IRC20:09
figleafleakypipes: ^^ very rough for now. Just wanted to get feedback on direction20:09
figleafleakypipes: planning on a subsequent patch to return allocation_candidates along with hosts, and then return both the alternates with the candidates20:10
* figleaf hit return too soon20:11
figleafreturn both to the compute build_and_run_instances() call20:11
*** acormier_ has quit IRC20:12
*** eharney has quit IRC20:13
*** lpetrut has joined #openstack-nova20:15
openstackgerritMerged openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775120:16
melwittsdague: what's the idea here? if you get an exception while logging, log again? https://review.openstack.org/#/c/485602/5/nova/api/openstack/requestlog.py@9220:17
*** awaugama has quit IRC20:18
sdaguemelwitt: because of the way the paste pipelines work, if we get an unexpected exception it's going to end up going through that layer20:19
sdagueand handled at the fault layer above it20:19
sdagueif the log message was not emitted in the except block, you'd never have a record of the request20:20
sdagueyou'd just have the fault error20:20
melwittsdague: ah, cool. thanks20:20
mriedemsdague: in case you didn't know, the neutron linuxbridge job is the one that has been timing out all week20:21
mriedemkevinbenton: ^ know what's going on there?20:21
sdaguemriedem: it's also set at 130minutes20:21
sdaguewhich is shorter than the rest20:21
sdagueI've definitely seen passing neutron jobs at 1:45 and 2:0020:21
sdaguewhich means everything is just long20:22
sdagueso a 130 is going to fail just because of statistics on slow node events20:22
melwitthuh, interesting. I would have assumed they all had the same timeout20:22
sdagueI think at some point the linux bridge job was running less stuff20:22
sdaguebut who knows :)20:23
melwittheh20:23
*** tylerderosagrund has joined #openstack-nova20:23
*** tylerder_ has quit IRC20:24
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Expect id and disabled_reason in GET /os-services response  https://review.openstack.org/48540920:27
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.53 - services and hypervisors using UUIDs  https://review.openstack.org/48543520:27
*** boris-42____ has quit IRC20:28
*** mriedem has quit IRC20:30
*** MVenesio has quit IRC20:30
*** jmlowe has quit IRC20:32
*** avolkov has quit IRC20:42
*** gbarros has joined #openstack-nova20:43
*** vishwana_ has quit IRC20:46
*** baoli has quit IRC20:47
*** vishwanathj has joined #openstack-nova20:47
*** bnemec has quit IRC20:50
*** smatzek has quit IRC20:51
cfriesenhas anyone considered a "run this command in parallel on all cells and aggregate the responses" helper function?  there seem to be quite a few "for cell in cells:" loops in the code.20:52
cfriesenseparate issue...nova/compute/api.py has a CELLS variable.  How does this get refreshed to pick up new cells created after the variable was initially set?20:53
melwittcfriesen: yes https://github.com/openstack/nova/blob/master/nova/context.py#L53120:53
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028520:54
melwittcurrently you have to restart the services that are caching them. refreshing them is a TODO https://github.com/openstack/nova/blob/master/nova/context.py#L5020:55
cfriesenmelwitt: nice work20:55
melwittgdi and my patch just hit the linuxbridge job timeout20:57
cfriesenif we don't fix that refresh thing for Pike we should document it...I don't remember seeing any mention of that in the discussion about adding new cells20:57
cfriesenmake that documentation, not discussion20:57
melwittyeah, true. I'll put it on the TODOs etherpad20:58
kevinbentonsdague: where can we propose to increase that timeout?20:58
kevinbenton(for linux bridge job)20:58
*** vishwana_ has joined #openstack-nova20:59
*** vishwanathj has quit IRC20:59
melwittcfriesen: the todo etherpad is here if you think of other stuff that would help (documentation or otherwise) https://etherpad.openstack.org/p/nova-pike-cells-v2-todos20:59
*** baoli has joined #openstack-nova21:00
cfriesenmelwitt: good to know21:01
*** sapcc-bot has quit IRC21:05
*** sapcc-bot1 has joined #openstack-nova21:05
*** tylerderosagrund has quit IRC21:06
*** eharney has joined #openstack-nova21:07
cfriesenmelwitt: was there discusson on how to handle the refresh?  for what it's worth, I think a trigger would be more deterministic21:09
melwittcfriesen: not really yet. I was thinking we'd do it upon a SIGHUP or something, like we do for config file refreshes21:11
*** bnemec has joined #openstack-nova21:11
openstackgerritChris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped  https://review.openstack.org/48620821:14
*** gouthamr has quit IRC21:14
openstackgerritMerged openstack/nova master: Use plain routes list for versions instead of stevedore  https://review.openstack.org/48501121:18
openstackgerritMerged openstack/nova master: Remove the unittest for plugin framework  https://review.openstack.org/48506021:19
cfriesenmelwitt: what about making it an RPC call and hooking it into the "nova-manage cell_v2 create_cell" command so that all the necessary services would get kicked when we add a cell?  Otherwise it's a requirement on the operator to arrange for a SIGHUP to all the services which may be on separate physical servers.21:20
*** rnoriega has quit IRC21:20
*** lpetrut has quit IRC21:22
melwittcfriesen: nova-manage doesn't work that way, it just does direct DB access stuff, no RPC21:26
melwittbut yeah, I agree SIGHUP isn't ideal. probably we can come up with a better idea21:27
*** edmondsw has quit IRC21:30
*** annegentle has quit IRC21:31
*** rdo_ has quit IRC21:35
*** rdo has joined #openstack-nova21:35
*** edmondsw has joined #openstack-nova21:37
*** acormier has joined #openstack-nova21:37
*** acormier has quit IRC21:37
*** acormier has joined #openstack-nova21:38
*** edmondsw has quit IRC21:41
*** edmondsw has joined #openstack-nova21:42
*** acormier has quit IRC21:45
*** acormier has joined #openstack-nova21:45
*** baoli has quit IRC21:47
*** edmondsw has quit IRC21:47
*** edmondsw has joined #openstack-nova21:48
*** acormier has quit IRC21:50
*** lucasxu has quit IRC21:51
*** edmondsw has quit IRC21:53
*** edmondsw has joined #openstack-nova21:54
*** mtanino_ has quit IRC21:55
*** cdent has quit IRC21:58
*** edmondsw has quit IRC21:59
*** edmondsw has joined #openstack-nova22:00
openstackgerritChris Dent proposed openstack/nova master: WIP: Use wsgi_intercept in PlacementFixture  https://review.openstack.org/48623722:02
*** edmondsw has quit IRC22:05
*** fried_rice is now known as efried_afk22:06
*** edmondsw has joined #openstack-nova22:08
*** edmondsw_ has joined #openstack-nova22:10
*** MVenesio has joined #openstack-nova22:12
*** MVenesio has quit IRC22:12
*** edmondsw has quit IRC22:12
*** edmondsw_ has quit IRC22:14
*** Sukhdev has quit IRC22:17
*** annegentle has joined #openstack-nova22:23
*** tesseract has quit IRC22:25
*** marst_ has quit IRC22:28
*** marst has joined #openstack-nova22:29
*** annegentle has quit IRC22:31
*** sdague has quit IRC22:33
*** marst has quit IRC22:34
openstackgerritEd Leafe proposed openstack/nova master: WIP - add alternate hosts  https://review.openstack.org/48621522:34
openstackgerritEd Leafe proposed openstack/nova master: WIP - return alternates along with their allocations  https://review.openstack.org/48625322:34
figleafleakypipes: ^^ and with that I'm heading out22:34
*** figleaf is now known as edleafe22:35
*** edmondsw has joined #openstack-nova22:48
*** rajathagasthya has quit IRC22:51
*** sdague has joined #openstack-nova22:51
*** edmondsw has quit IRC22:52
*** lyan has quit IRC22:56
*** manjeets has quit IRC22:56
*** MVenesio has joined #openstack-nova22:57
*** MVenesio has quit IRC22:58
*** gouthamr has joined #openstack-nova22:58
*** esberglu has quit IRC23:05
*** felipemonteiro_ has quit IRC23:07
*** xyang1 has quit IRC23:11
*** tbachman has joined #openstack-nova23:14
openstackgerritMerged openstack/nova master: placement: proper JOIN order for shared resources  https://review.openstack.org/48508823:22
openstackgerritMerged openstack/nova master: Removed unused 'wrap' property  https://review.openstack.org/48146523:23
*** thorst has quit IRC23:24
*** r-daneel has quit IRC23:31
*** chyka has quit IRC23:34
*** tbachman has quit IRC23:34
*** tbachman has joined #openstack-nova23:35
*** markvoelker has quit IRC23:39
*** rajathagasthya has joined #openstack-nova23:40
*** rajathag_ has joined #openstack-nova23:43
*** rajathagasthya has quit IRC23:44
*** tbachman has quit IRC23:44
*** baoli has joined #openstack-nova23:52
*** gouthamr has quit IRC23:52
*** acormier has joined #openstack-nova23:54
*** acormier has quit IRC23:54
*** acormier has joined #openstack-nova23:54
openstackgerritChris Friesen proposed openstack/nova master: Ensure we unshelve in the cell the instance is mapped  https://review.openstack.org/48620823:55
*** baoli has quit IRC23:57
*** catintheroof has joined #openstack-nova23:58

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