Wednesday, 2017-08-02

*** rajathagasthya has quit IRC00:20
*** kiennt has joined #openstack-oslo00:37
*** rajathagasthya has joined #openstack-oslo00:43
openstackgerritVipin Balachandran proposed openstack/osprofiler master: Improve unit test coverage  https://review.openstack.org/48973901:11
*** https_GK1wmSU has joined #openstack-oslo01:32
*** https_GK1wmSU has left #openstack-oslo01:34
*** rajathagasthya has quit IRC01:55
*** xiaoyang has joined #openstack-oslo02:19
*** yamamoto has quit IRC02:24
*** yamamoto has joined #openstack-oslo02:30
*** yamamoto has quit IRC02:35
*** yamamoto has joined #openstack-oslo02:36
*** yamamoto has quit IRC02:41
*** dave-mcc_ has joined #openstack-oslo02:41
*** dave-mccowan has quit IRC02:42
*** yamamoto has joined #openstack-oslo02:45
*** nicolasbock has joined #openstack-oslo02:57
*** dave-mccowan has joined #openstack-oslo03:11
*** dave-mcc_ has quit IRC03:14
*** gtrxcb has joined #openstack-oslo03:21
*** rajathagasthya has joined #openstack-oslo03:22
*** yamamoto has quit IRC03:33
*** dave-mccowan has quit IRC03:33
*** carolina1 has joined #openstack-oslo03:34
*** carolina1 has quit IRC03:43
*** links has joined #openstack-oslo03:48
*** yamamoto has joined #openstack-oslo03:49
*** rajathagasthya has quit IRC04:03
*** yamamoto has quit IRC04:05
*** yamamoto has joined #openstack-oslo04:10
*** kornicameister has quit IRC04:31
*** kornicameister has joined #openstack-oslo04:32
*** yamamoto has quit IRC04:50
*** yamamoto has joined #openstack-oslo05:07
*** kiennt has quit IRC05:36
*** kiennt has joined #openstack-oslo05:56
*** e0ne has joined #openstack-oslo06:01
*** hoonetorg has quit IRC06:02
*** e0ne has quit IRC06:03
*** kiennt has quit IRC06:10
*** kiennt has joined #openstack-oslo06:12
*** hoonetorg has joined #openstack-oslo06:15
*** jaosorior has quit IRC06:25
openstackgerritAbhishek Sharma M proposed openstack/pycadf master: Adding panko_api_audit_map.conf to pycadf  https://review.openstack.org/48968006:28
openstackgerritAbhishek Sharma M proposed openstack/pycadf master: Adding panko_api_audit_map.conf to pycadf  https://review.openstack.org/48968006:30
*** jaosorior has joined #openstack-oslo06:34
*** pcaruana has joined #openstack-oslo06:34
*** rcernin has joined #openstack-oslo06:46
*** aarefiev_afk is now known as aarefiev07:07
openstackgerritAbhishek Sharma M proposed openstack/pycadf master: Removed references to events api  https://review.openstack.org/48989207:21
*** lpetrut has joined #openstack-oslo07:37
*** gtrxcb has quit IRC07:44
*** ralonsoh has joined #openstack-oslo07:46
*** gcb_ has quit IRC07:48
*** tesseract has joined #openstack-oslo07:48
*** pblaho has joined #openstack-oslo07:49
*** ralonsoh has quit IRC08:12
*** xiaoyang has quit IRC08:13
*** ralonsoh has joined #openstack-oslo08:13
*** xiaoyang has joined #openstack-oslo08:13
*** ralonsoh has quit IRC08:15
*** ralonsoh has joined #openstack-oslo08:18
*** https_GK1wmSU has joined #openstack-oslo08:24
*** https_GK1wmSU has left #openstack-oslo08:25
*** lucas-afk is now known as lucasagomes08:32
*** openstackgerrit has quit IRC08:33
*** ralonsoh_ has joined #openstack-oslo08:40
*** ralonsoh has quit IRC08:40
*** openstackgerrit has joined #openstack-oslo08:46
openstackgerritTovin Seven proposed openstack/osprofiler master: Add function/sql results to trace info  https://review.openstack.org/45007208:46
openstackgerritTovin Seven proposed openstack/osprofiler master: [WiP] OSprofiler with OpenTracing compatible  https://review.openstack.org/48001808:47
*** gcb has joined #openstack-oslo09:28
*** sdague has joined #openstack-oslo09:32
*** e0ne has joined #openstack-oslo09:53
*** yassine has quit IRC09:59
*** kiennt has quit IRC10:01
*** sambetts|afk is now known as sambetts10:09
*** jaosorior has quit IRC10:10
*** ansmith has joined #openstack-oslo10:40
*** ansmith_ has joined #openstack-oslo10:41
*** ansmith_ has quit IRC10:53
*** ansmith has quit IRC10:53
*** jaosorior has joined #openstack-oslo11:02
*** lucasagomes is now known as lucas-hungry11:37
*** nicolasbock_ has joined #openstack-oslo11:41
*** nicolasbock__ has joined #openstack-oslo11:42
*** nicolasbock has quit IRC11:44
*** nicolasbock has joined #openstack-oslo11:45
*** nicolasbock_ has quit IRC11:46
*** nicolasbock__ has quit IRC11:47
*** raildo has joined #openstack-oslo12:00
*** yassine has joined #openstack-oslo12:01
*** lucas-hungry is now known as lucasagomes12:39
*** sdague_ has joined #openstack-oslo12:43
*** HenryG has quit IRC12:45
*** gordc has joined #openstack-oslo12:45
*** HenryG has joined #openstack-oslo12:47
*** yamamoto has quit IRC13:02
*** gordc has quit IRC13:09
*** yamamoto has joined #openstack-oslo13:18
*** kgiusti has joined #openstack-oslo13:21
*** ansmith_ has joined #openstack-oslo13:38
*** ansmith has joined #openstack-oslo13:39
*** ansmith__ has joined #openstack-oslo13:40
*** ansmith has quit IRC13:40
*** ansmith_ has quit IRC13:41
*** ansmith__ has quit IRC13:41
*** sdague has quit IRC13:43
*** ansmith has joined #openstack-oslo13:44
*** aarefiev is now known as aarefiev_afk13:51
openstackgerritAbhishek Sharma M proposed openstack/pycadf master: Removed references to events api  https://review.openstack.org/48989213:54
*** yamamoto has quit IRC13:55
*** sdague has joined #openstack-oslo13:56
-openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud.13:57
*** nicolasbock_ has joined #openstack-oslo14:04
*** nicolasbock__ has joined #openstack-oslo14:07
*** nicolasbock has quit IRC14:07
*** nicolasbock_ has quit IRC14:10
*** xiaoyang has quit IRC14:20
*** xiaoyang has joined #openstack-oslo14:20
*** nicolasbock_ has joined #openstack-oslo14:21
*** nicolasbock__ has quit IRC14:25
*** seba has joined #openstack-oslo14:26
*** janzian has joined #openstack-oslo14:33
*** ralonsoh_ has quit IRC14:39
*** gordc has joined #openstack-oslo14:41
*** yamamoto has joined #openstack-oslo14:55
openstackgerritAbhishek Sharma M proposed openstack/pycadf master: Removed references to events api  https://review.openstack.org/48989214:59
*** yamamoto has quit IRC15:00
*** links has quit IRC15:01
*** e0ne has quit IRC15:16
*** nicolasbock__ has joined #openstack-oslo15:21
*** nicolasbock_ has quit IRC15:24
*** ralonsoh has joined #openstack-oslo15:31
*** sdague_ has quit IRC15:43
*** e0ne has joined #openstack-oslo15:46
*** ralonsoh has quit IRC15:47
*** ralonsoh has joined #openstack-oslo15:47
*** aselius has joined #openstack-oslo15:58
*** links has joined #openstack-oslo15:58
*** janzian has quit IRC15:59
*** ihrachys has joined #openstack-oslo16:00
*** sdague has quit IRC16:03
*** lpetrut has quit IRC16:07
*** rcernin has quit IRC16:08
*** Rockyg has joined #openstack-oslo16:09
*** pcaruana has quit IRC16:14
*** janzian has joined #openstack-oslo16:15
*** yassine has quit IRC16:20
*** lucasagomes is now known as lucas-afk16:28
*** hoonetorg has quit IRC16:30
*** sdague has joined #openstack-oslo16:40
*** hoonetorg has joined #openstack-oslo16:42
*** e0ne has quit IRC16:57
*** e0ne has joined #openstack-oslo16:58
*** rajathagasthya has joined #openstack-oslo16:58
*** e0ne has quit IRC16:58
*** ralonsoh has quit IRC17:00
*** tesseract has quit IRC17:07
*** harlowja has joined #openstack-oslo17:13
*** rajathagasthya has quit IRC17:30
*** sambetts is now known as sambetts|afk17:30
*** rajathagasthya has joined #openstack-oslo17:48
openstackgerritMerged openstack/pycadf master: Removed references to events api  https://review.openstack.org/48989218:02
openstackgerritMerged openstack/pycadf master: Adding panko_api_audit_map.conf to pycadf  https://review.openstack.org/48968018:16
*** Rockyg has quit IRC18:25
*** links has quit IRC18:31
*** ihrachys has quit IRC18:42
*** ihrachys has joined #openstack-oslo18:42
*** nicolasbock__ has quit IRC19:10
ihrachyskgiusti, I know you were looking at handling NotFound raised on missing fanout exchange in oslo.messaging19:18
ihrachysplease have a look at https://bugs.launchpad.net/nova/+bug/1707160 comment 1619:18
openstackLaunchpad bug 1707160 in OpenStack Compute (nova) "test_create_port_in_allowed_allocation_pools test fails on ironic grenade" [Undecided,New]19:18
ihrachysI think we may have more serious problems with NotFound than just a programmatic exception raised19:18
kgiustiihrachys: thanks for the analysis - I'm blocked getting any patches through oslo.messaging CI due to https://review.openstack.org/#/c/488563/19:22
ihrachyskgiusti, I am not saying you should fix it. I am more interested in someone just looking whether what we see suggests a oslo/rabbitmq issue and not ironic/nova/neutron19:23
ihrachysin neutron, we now catch NotFound for all cast calls19:23
ihrachysbut it does seem to leave some issues with timeouts of channels used when NotFound was raised19:24
kgiustiihrachys: let me take a look into oslo - see what if anything is done to the channel on NotFound...19:25
*** e0ne has joined #openstack-oslo19:27
*** e0ne has quit IRC19:34
*** rajathagasthya has quit IRC19:35
*** rajathagasthya has joined #openstack-oslo19:38
*** sileht has quit IRC19:55
*** rajathagasthya has quit IRC20:00
*** e0ne has joined #openstack-oslo20:01
*** rajathagasthya has joined #openstack-oslo20:02
*** hoonetorg has quit IRC20:03
*** e0ne has quit IRC20:15
*** hoonetorg has joined #openstack-oslo20:20
kgiustiihrachys: ping - looks like the channel is not cleaned up for the generic exception path https://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/_drivers/impl_rabbit.py#n81020:23
kgiustiihrachys: perhaps that needs to be done for NotFound?20:23
*** ansmith has quit IRC20:25
ihrachyskgiusti, is it the self._set_current_channel(None) call?20:26
kgiustiihrachys: yep.  But another question: when did these NotFound errors start manifesting?   Kombu 4.1.0 recently rolled out....20:27
*** e0ne has joined #openstack-oslo20:27
ihrachyskgiusti, in neutron, it started manifesting when we migrated our agents from one RPC fanout exchange to another20:27
ihrachysso no component now listens to some exchanges20:27
ihrachysin which case you get NotFound20:27
*** raildo has quit IRC20:28
kgiustiihrachys: so the exchanges don't exist, and the fanout attempts to send to them, correct?20:28
* kgiusti is looking for an easy reproducer....20:29
ihrachysyes20:29
ihrachysreproducer should be just send to a random topic20:29
ihrachysthen you get NotFound20:29
ihrachyscatch it and try to send something proper with the same connection20:29
kgiustiihrachys: let me give that a try....20:30
ihrachyskgiusti, having an RPCClient/_Context, is it possible to get to the Connection object to call that internal method resetting it?20:31
*** https_GK1wmSU has joined #openstack-oslo20:35
kgiustiihrachys: it's a bit of a mess since the connection is selected from a pool on each call and not referenced from the transport20:36
ihrachyskevinbenton, fyi we may have broken api calls that trigger NotFound after which consequent attempts to use amqp by the same thread result in timeouts ^20:37
*** https_GK1wmSU has left #openstack-oslo20:37
ihrachyskgiusti, looks like we can't work that around from neutron side while you are fixing it20:37
kgiustiihrachys: probably not easily, no20:38
ihrachyskgiusti, I assume that even if we have a fix, we would need to release a new library meaning FFE for the new release?20:40
kgiustiihrachys: Running a quick test here - sending a fanout to a non existent exchange/topic actually causes the exchange to be created on rabbitmq20:41
kgiustiihrachys: so there must be some race where the exchange is deleted while the fanout is in-progress...20:42
ihrachyskgiusti, the trace http://logs.openstack.org/01/489701/1/check/gate-grenade-dsvm-ironic-ubuntu-xenial/50785e5/logs/new/screen-q-svc.txt.gz#_2017-08-01_22_48_58_43420:42
ihrachyskgiusti, note it's grenade20:43
ihrachysand old agents used the topic20:43
ihrachysso MAYBE it was created by older agent; then it expired (?) and was removed in the middle of fanout?20:43
ihrachysrather rare scenario I would assume20:43
ihrachysbecause we would need to catch the exact moment of expiration20:44
kgiustiihrachys: rare yes, but it sounds like you're hitting quite often.20:44
ihrachysyep, they say it's ~50% runs20:44
ihrachysofc there are multiple tests to fail on20:44
ihrachysso it shouldn't necessarily be every second api request :)20:45
ihrachyskgiusti, also it looks like we have same topic used before as well as after the failure with no issue20:48
ihrachysso maybe we indeed hit some race between cleanup after old agents and creating a new one before fanout?20:49
kgiustiihrachys: perhaps - I'm trying to hack out the creation of the exchange to see if it fails in a similar manner....20:50
*** e0ne has quit IRC20:52
ihrachyskgiusti, I checked, and this happens ~30 minutes after upgrade of neutron-server20:54
ihrachys~26 minutes in new neutron-server run20:54
ihrachysit does correlate with rabbit_transient_queues_ttl = 30 minutes20:54
ihrachys(it was 10 minutes but then bumped to 30 minutes in N)20:56
ihrachysI updated the bug https://bugs.launchpad.net/nova/+bug/1707160 with gist of the discussion above21:02
openstackLaunchpad bug 1707160 in neutron "test_create_port_in_allowed_allocation_pools test fails on ironic grenade" [Critical,Confirmed] - Assigned to Ihar Hrachyshka (ihar-hrachyshka)21:02
ihrachysI will need to drop in 5 mins21:02
kgiustiihrachys: yeah me also, but to confirm I get a NOT FOUND if I hack out the exchange creation during the fanout send...21:04
ihrachyswhere is the code creating the missing topic?21:04
kgiustihttps://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/_drivers/impl_rabbit.py#n127921:05
kgiustiihrachys: that creates the exchange, then it is bound to the channel here: https://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/_drivers/impl_rabbit.py#n117221:06
kgiustiihrachys: removing that .declare() and I can simulate the not found21:06
ihrachysI see. this declare() probably doesn't bump expiration window for existing topic21:07
ihrachyssorry, exchange21:07
kgiustiihrachys: it's possible also that the list of _declared_exchanges is stale21:08
*** yamamoto_ has joined #openstack-oslo21:08
kgiustiihrachys: which would certainly be the case if the exchange was deleted at some point21:08
kgiustiihrachys: and clearing the channel does reset that cache, more evidence!21:09
kgiustiihrachys: we're not setting channel to None when NotFound hits....21:09
ihrachyskgiusti, you can push a change; I only wonder if we can test ironic grenade gate before committing it21:10
ihrachyskgiusti, since oslo.msg is consumed via reqs...21:10
kgiustiihrachys: never done that before - perhaps infra can help us here?21:11
ihrachysI will need to drop now; if you don't reach them till tomorrow, I will chase that. if you can push a WIP patch to test against till then, that would be great.21:12
ihrachysthat's probably one of the blocker issues we will need to solve somehow for neutron this cycle21:12
ihrachysand reverting the changes that triggered it on neutron side is a PITA21:12
ihrachysa huge PITA21:13
kgiustiihrachys: Let's avoid the huge PITA21:13
ihrachysit will result in like 5-10 patches reverted :)21:13
ihrachysyeah, that's why I really want the oslo.msg fix21:13
ihrachyschasing a FFE should be easier21:13
kgiustiihrachys: I can push to a branch when I have a simple candidate21:13
ihrachysok, thanks21:14
ihrachyslet's sync tomorrow21:14
kgiustiihrachys: and the WIP patch too21:14
kgiustiihrachys: see you tomorrow21:14
ihrachyskevinbenton, I highly suggest you to read that ^; it's release affecting.21:14
*** yamamoto_ has quit IRC21:15
*** yamamoto_ has joined #openstack-oslo21:17
*** lpetrut has joined #openstack-oslo21:21
*** janzian has quit IRC21:34
kevinbentonihrachys: doesn't your general patch cover us if that doesn't make it?21:49
openstackgerritKen Giusti proposed openstack/oslo.messaging master: Prevent rabbit from raising kombu or py-amqp exceptions  https://review.openstack.org/48670621:49
*** kgiusti has left #openstack-oslo21:53
*** lpetrut has quit IRC22:14
*** ansmith has joined #openstack-oslo22:15
*** https_GK1wmSU has joined #openstack-oslo22:19
*** https_GK1wmSU has left #openstack-oslo22:21
*** gordc has quit IRC22:28
*** yamamoto_ has quit IRC22:29
*** yamamoto has joined #openstack-oslo22:32
*** yamamoto has quit IRC22:36
*** yamamoto has joined #openstack-oslo22:41
*** yamamoto has quit IRC22:44
*** sdague has quit IRC22:50
*** yamamoto has joined #openstack-oslo22:56
*** yassine has joined #openstack-oslo23:02
*** gtrxcb has joined #openstack-oslo23:06
*** yamamoto has quit IRC23:11
*** yamamoto has joined #openstack-oslo23:14
*** rajathagasthya has quit IRC23:29
*** ansmith has quit IRC23:54
*** rajathagasthya has joined #openstack-oslo23:54
*** ansmith has joined #openstack-oslo23:54

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