Monday, 2012-11-05

*** tgall_foo has joined #openstack-dev00:01
*** tgall_foo has quit IRC00:01
*** tgall_foo has joined #openstack-dev00:01
*** jakedahn is now known as jakedahn_zz00:11
*** winston-d has joined #openstack-dev00:12
*** maxiz has quit IRC00:13
*** dachary has quit IRC00:15
*** jergerber has quit IRC00:16
*** devcamca1 has joined #openstack-dev00:21
*** jakedahn_zz is now known as jakedahn00:22
*** matiu has quit IRC00:26
*** devcamca1 has quit IRC00:31
*** colinmcnamara has joined #openstack-dev00:32
*** colinmcnamara has quit IRC00:37
*** drewlander has quit IRC00:38
*** matiu has joined #openstack-dev00:40
*** drewlander has joined #openstack-dev00:45
*** ayoung has quit IRC00:54
*** littleidea has quit IRC00:58
*** devcamca1 has joined #openstack-dev00:58
*** Ruetobas has quit IRC01:04
*** KurtMartin has joined #openstack-dev01:05
*** KurtMartin has quit IRC01:05
*** sunilthaha has quit IRC01:06
*** haleyb has quit IRC01:10
*** devcamca1 has quit IRC01:10
*** roge has quit IRC01:21
*** roge has joined #openstack-dev01:22
*** wenjianhn has joined #openstack-dev01:27
*** Ruetobas has joined #openstack-dev01:32
*** ewindisch has joined #openstack-dev01:32
*** colinmcnamara has joined #openstack-dev01:33
*** xchu has joined #openstack-dev01:36
*** devcamca1 has joined #openstack-dev01:36
*** colinmcnamara has quit IRC01:37
*** edygarcia has joined #openstack-dev01:40
*** blamar has quit IRC01:42
*** blamar has joined #openstack-dev01:45
*** matiu has quit IRC01:46
*** devcamca1 has quit IRC01:48
*** xarg has joined #openstack-dev01:49
*** edygarcia has quit IRC01:49
*** xchu has quit IRC02:07
*** brucer has quit IRC02:14
*** timello has joined #openstack-dev02:14
*** devcamca1 has joined #openstack-dev02:14
*** gongysh has joined #openstack-dev02:15
*** eglynn__ has joined #openstack-dev02:16
*** ben_duyujie has joined #openstack-dev02:18
*** eglynn_ has quit IRC02:19
*** dims has joined #openstack-dev02:21
*** timello has quit IRC02:24
*** timello has joined #openstack-dev02:26
*** devcamca1 has quit IRC02:26
*** ewindisch has quit IRC02:28
*** roge has quit IRC02:29
*** colinmcnamara has joined #openstack-dev02:33
*** littleidea has joined #openstack-dev02:38
*** markwash has quit IRC02:45
*** dims has quit IRC02:45
*** colinmcnamara has quit IRC02:48
*** devcamca1 has joined #openstack-dev02:54
*** ewindisch has joined #openstack-dev03:01
*** devcamca1 has quit IRC03:03
*** matiu has joined #openstack-dev03:05
*** matiu has quit IRC03:05
*** matiu has joined #openstack-dev03:05
*** lianhao has quit IRC03:11
*** danwent has joined #openstack-dev03:12
*** zaitcev has joined #openstack-dev03:21
*** lianhao has joined #openstack-dev03:26
*** lianhao has joined #openstack-dev03:29
*** devcamca1 has joined #openstack-dev03:30
*** lianhao has quit IRC03:33
*** sunilthaha has joined #openstack-dev03:35
*** lianhao has joined #openstack-dev03:39
*** anniec72 has quit IRC03:41
*** ben_duyujie has quit IRC03:41
*** devcamca1 has quit IRC03:42
*** blamar has quit IRC03:42
*** ben_duyujie has joined #openstack-dev03:45
*** ben_duyujie has left #openstack-dev03:47
*** Ruetobas has quit IRC03:52
*** spzala has joined #openstack-dev03:55
*** devcamca1 has joined #openstack-dev04:09
*** samkottler is now known as samkottler|out04:11
*** danwent has quit IRC04:14
*** blamar has joined #openstack-dev04:16
*** devcamca1 has quit IRC04:20
*** tgall_foo has quit IRC04:24
*** xchu has joined #openstack-dev04:25
*** littleidea has quit IRC04:29
*** sunilthaha has quit IRC04:30
*** littleidea has joined #openstack-dev04:32
*** spzala has quit IRC04:34
*** devcamca1 has joined #openstack-dev04:47
*** cyeoh has quit IRC04:51
*** xchu has quit IRC04:54
*** cyeoh has joined #openstack-dev04:55
*** devcamca1 has quit IRC04:59
*** xchu has joined #openstack-dev05:00
*** lianhao has left #openstack-dev05:00
cloudflyhrmm05:00
*** lianhao has joined #openstack-dev05:01
*** andrewbogott_afk is now known as andrewbogott05:08
*** andrewbogott is now known as andrewbogott_afk05:10
*** Mandell has joined #openstack-dev05:13
clarkbvishy: jgriffith can we take a look at why http://logs.openstack.org/15317/1/gate/gate-tempest-devstack-vm/16895/ and http://logs.openstack.org/15327/3/check/gate-tempest-devstack-vm/16842/ failed? both appear to have issues running the volumes devstack exercises05:14
*** brucer has joined #openstack-dev05:19
*** devcamca1 has joined #openstack-dev05:26
*** xchu has quit IRC05:32
*** devcamca1 has quit IRC05:36
*** nati_ueno has joined #openstack-dev05:40
*** blamar has quit IRC05:51
*** persia has joined #openstack-dev05:52
*** zaitcev has quit IRC05:56
*** xchu has joined #openstack-dev05:59
*** devcamca1 has joined #openstack-dev06:03
*** garyk has quit IRC06:04
*** afazekas has joined #openstack-dev06:07
*** devcamca1 has quit IRC06:14
*** zhidong has quit IRC06:22
*** danwent has joined #openstack-dev06:31
*** metral_ has joined #openstack-dev06:32
*** metral_ has quit IRC06:32
*** AnilV4 has joined #openstack-dev06:35
*** metral has quit IRC06:35
openstackgerritA change was merged to openstack/openstack-common: Fixes setup compatibility issue on Windows  https://review.openstack.org/1532906:38
*** cyeoh has quit IRC06:41
*** devcamca1 has joined #openstack-dev06:41
openstackgerritA change was merged to openstack/openstack-common: Creates uuidutils module  https://review.openstack.org/1465806:45
openstackgerritA change was merged to openstack/openstack-common: Relax the update.py replace regex.  https://review.openstack.org/1516006:46
*** markwash has joined #openstack-dev06:49
openstackgerritA change was merged to openstack/openstack-common: Log CONF from ProcessLauncher.wait, like ServiceLauncher  https://review.openstack.org/1446606:50
*** devcamca1 has quit IRC06:53
*** brucer has quit IRC07:01
*** mrunge has joined #openstack-dev07:02
*** bencherian has joined #openstack-dev07:06
openstackgerritA change was merged to openstack/openstack-common: Remove openstack.common.config and extensions.  https://review.openstack.org/1472907:06
*** danwent has quit IRC07:09
*** eglynn__ has quit IRC07:10
*** garyk has joined #openstack-dev07:12
*** nati_ueno has quit IRC07:13
*** cyeoh has joined #openstack-dev07:16
*** eglynn__ has joined #openstack-dev07:17
*** henrynash has joined #openstack-dev07:18
*** devcamca1 has joined #openstack-dev07:20
*** cyeoh has quit IRC07:21
*** ndipanov has joined #openstack-dev07:22
*** cyeoh has joined #openstack-dev07:23
*** devcamca1 has quit IRC07:32
*** mindpixel has joined #openstack-dev07:35
*** jakedahn is now known as jakedahn_zz07:35
*** Navneet has joined #openstack-dev07:36
*** Navneet has quit IRC07:40
*** Joe2 has joined #openstack-dev07:40
*** ondergetekende has joined #openstack-dev07:48
*** littleidea has quit IRC07:51
*** corXi has joined #openstack-dev07:51
mordredvishy, jgriffith: I second clarkb above. it's starting to be problematic for other projects too07:55
*** devcamca1 has joined #openstack-dev07:58
*** eglynn__ has quit IRC08:01
*** apevec has joined #openstack-dev08:06
*** apevec has joined #openstack-dev08:06
*** devcamca1 has quit IRC08:09
*** reidrac has joined #openstack-dev08:09
*** eglynn__ has joined #openstack-dev08:12
*** Tross has joined #openstack-dev08:17
*** apevec has quit IRC08:20
*** markwash has quit IRC08:21
*** Tross has quit IRC08:24
*** Hitesh has joined #openstack-dev08:25
*** Tross has joined #openstack-dev08:25
Hiteshgaryk: Hi Gary08:25
Hiteshgaryk: how was the weekend?08:25
Hitesharosen: Hi Aron08:28
Hitesharosen: U there?08:28
*** Tross has quit IRC08:28
*** henrynash has quit IRC08:30
*** apevec has joined #openstack-dev08:33
*** apevec has joined #openstack-dev08:33
*** aloga has joined #openstack-dev08:34
*** devcamca1 has joined #openstack-dev08:37
*** alexpilotti has joined #openstack-dev08:37
*** dachary has joined #openstack-dev08:38
*** apevec has quit IRC08:38
*** Dorogs has joined #openstack-dev08:40
*** cyeoh has quit IRC08:41
*** apevec has joined #openstack-dev08:42
*** apevec has joined #openstack-dev08:42
*** rbasak has joined #openstack-dev08:46
*** apevec has quit IRC08:47
*** Mandell has quit IRC08:48
*** devcamca1 has quit IRC08:48
*** metral has joined #openstack-dev08:49
*** cyeoh has joined #openstack-dev08:52
*** zing has joined #openstack-dev08:56
*** winston-d has quit IRC08:57
*** metral has quit IRC08:59
*** apevec has joined #openstack-dev09:00
*** apevec has joined #openstack-dev09:00
*** zing_ has joined #openstack-dev09:03
*** zing has quit IRC09:03
*** zing_ is now known as zing09:03
*** apevec has quit IRC09:06
*** eglynn__ is now known as eglynn09:12
*** apevec has joined #openstack-dev09:12
*** apevec has joined #openstack-dev09:12
*** reed has joined #openstack-dev09:13
*** devcamca1 has joined #openstack-dev09:16
*** Joe2 has left #openstack-dev09:17
*** eglynn_ has joined #openstack-dev09:18
*** eglynn has quit IRC09:18
*** darraghb has joined #openstack-dev09:19
*** apevec has quit IRC09:26
*** cyeoh has quit IRC09:26
*** devcamca1 has quit IRC09:27
*** markwash has joined #openstack-dev09:28
*** afazekas has quit IRC09:28
*** cyeoh has joined #openstack-dev09:29
*** Dorogs has quit IRC09:30
*** markwash has quit IRC09:32
*** afazekas has joined #openstack-dev09:33
*** apevec has joined #openstack-dev09:40
*** apevec has joined #openstack-dev09:40
*** mmagr has joined #openstack-dev09:41
*** Dorogs has joined #openstack-dev09:41
*** xchu has quit IRC09:41
*** devcamca1 has joined #openstack-dev09:53
*** Tross has joined #openstack-dev09:57
*** maploin has joined #openstack-dev09:57
*** maploin has quit IRC09:57
*** maploin has joined #openstack-dev09:57
*** alexpilotti has quit IRC10:01
*** afazekas has quit IRC10:02
*** devcamca1 has quit IRC10:05
*** apevec_ has joined #openstack-dev10:07
*** apevec_ has joined #openstack-dev10:07
*** apevec has quit IRC10:09
*** bencherian has quit IRC10:10
*** apevec_ has quit IRC10:12
*** pixelbeat has joined #openstack-dev10:14
*** reed has quit IRC10:15
*** andrea_ has quit IRC10:23
*** andrea_ has joined #openstack-dev10:24
openstackgerritA change was merged to openstack/quantum: Add uuidutils module  https://review.openstack.org/1537410:25
*** apevec has joined #openstack-dev10:30
*** apevec has joined #openstack-dev10:30
*** Dorogs has quit IRC10:31
*** devcamca1 has joined #openstack-dev10:32
*** reed has joined #openstack-dev10:35
dave_mcnAnyone available to approve a change? https://review.openstack.org/#/c/14769/10:35
*** sunilthaha has joined #openstack-dev10:37
*** devcamca1 has quit IRC10:43
*** Hitesh has quit IRC10:44
*** dachary has quit IRC10:47
openstackgerritA change was merged to openstack/quantum: Drop lxml dependency  https://review.openstack.org/1538310:51
*** wenjianhn has quit IRC10:59
*** markmc has joined #openstack-dev11:07
*** devcamca1 has joined #openstack-dev11:10
*** ondergetekende has quit IRC11:12
*** adalbas has joined #openstack-dev11:18
*** devcamca1 has quit IRC11:22
*** rburon has joined #openstack-dev11:24
*** dachary has joined #openstack-dev11:33
*** maxiz has joined #openstack-dev11:40
*** maurosr has joined #openstack-dev11:44
*** ondergetekende has joined #openstack-dev11:44
*** Tross1 has joined #openstack-dev11:47
*** Tross has quit IRC11:47
*** devcamca1 has joined #openstack-dev11:49
*** maurosr has quit IRC11:50
openstackgerritA change was merged to openstack/quantum: Use uuidutils for uuid validation  https://review.openstack.org/1537711:59
*** devcamca1 has quit IRC12:01
*** bencherian has joined #openstack-dev12:02
*** bencherian has quit IRC12:09
*** milner has joined #openstack-dev12:13
*** afazekas has joined #openstack-dev12:17
*** bencherian has joined #openstack-dev12:19
*** bencherian has quit IRC12:20
*** bencherian_ has joined #openstack-dev12:20
garykrussellb: ping12:20
*** bencherian has joined #openstack-dev12:23
*** bencherian_ has quit IRC12:23
*** apevec_ has joined #openstack-dev12:24
*** apevec has quit IRC12:27
*** devcamca1 has joined #openstack-dev12:28
*** devcamca1 has quit IRC12:39
*** bencherian has quit IRC12:41
*** bencherian_ has joined #openstack-dev12:41
*** cmagina has quit IRC12:41
*** dachary has quit IRC12:42
*** cmagina has joined #openstack-dev12:43
*** dachary has joined #openstack-dev12:46
*** bencherian_ has quit IRC12:52
*** gongysh has quit IRC12:52
*** tmishler has joined #openstack-dev12:53
*** markvoelker has joined #openstack-dev12:55
*** krtaylor has quit IRC12:55
*** dave_mcn has quit IRC13:01
*** radez_g0n3 is now known as radez13:05
*** eglynn_ is now known as hungry-eglynn13:06
*** devcamca1 has joined #openstack-dev13:07
*** alexpilotti has joined #openstack-dev13:09
*** samkottler|out is now known as samkottler13:12
*** samkottler has joined #openstack-dev13:12
*** apevec__ has joined #openstack-dev13:12
*** shardy is now known as shardy_afk13:13
*** tgall_foo has joined #openstack-dev13:14
*** apevec_ has quit IRC13:15
*** littleidea has joined #openstack-dev13:16
*** devcamca1 has quit IRC13:18
*** roge has joined #openstack-dev13:20
*** littleidea has quit IRC13:20
*** alunduil has quit IRC13:20
*** dims has joined #openstack-dev13:24
*** maxiz has quit IRC13:27
*** tgall_foo has quit IRC13:28
*** kgriffs has joined #openstack-dev13:30
*** ndipanov has quit IRC13:31
*** jaypipes has joined #openstack-dev13:31
*** krtaylor has joined #openstack-dev13:32
*** apevec__ has quit IRC13:34
*** ayoung has joined #openstack-dev13:36
*** sandywalsh has joined #openstack-dev13:39
*** CaptTofu has joined #openstack-dev13:45
*** boden has joined #openstack-dev13:45
CaptTofuhi all! Does anyone here have the reigns of openstack-cla ?13:46
*** devcamca1 has joined #openstack-dev13:46
*** ndipanov has joined #openstack-dev13:46
*** andrewbogott_afk is now known as andrewbogott13:46
*** mrunge is now known as mrunge_afk13:53
alexpilottirussellb: hi13:53
alexpilottirussellb: I just saw your comment13:53
alexpilottirussellb: can you plz tell me how to recommit when tempest fails?13:54
alexpilottirussellb: I tried to do a git commit -a --amend13:54
alexpilottirussellb: followed by a git review, but it fails with: "no changes made"13:55
*** hungry-eglynn is now known as eglynn13:55
alexpilottirussellb: that's why I abandoned the change and resubmittet13:55
alexpilottirussellb: I also wrote to jeblar, but no answer so far13:56
*** devcamca1 has quit IRC13:56
*** dprince has joined #openstack-dev13:57
*** maurosr has joined #openstack-dev13:57
alexpilottirussellb: ^^13:58
*** rpedde_away is now known as rpedde13:59
*** littleidea has joined #openstack-dev13:59
*** jcru has joined #openstack-dev14:00
*** reidrac has quit IRC14:00
*** jergerber has joined #openstack-dev14:01
*** maurosr has quit IRC14:01
Kiallalexpilotti, leave a comment saying "recheck" if it was pre-approval tests that failed or "reverify" if it was the post-approve/pre-merge tests14:02
*** tgall_foo has joined #openstack-dev14:02
*** tgall_foo has quit IRC14:02
*** tgall_foo has joined #openstack-dev14:02
Kiall(Saying just that!)14:02
*** kbringard has joined #openstack-dev14:02
alexpilottiKiall: tx!14:02
*** maurosr has joined #openstack-dev14:03
alexpilottiKiall: another question: can I "unabandon" a review?14:03
KiallYea - there should be a button on the review page14:04
alexpilottiKiall: "restore change"?14:04
Kiallthats the one14:04
alexpilottiKiall: tx, hopefully I finally got my way around the review system!14:04
*** apevec has joined #openstack-dev14:05
*** apevec has joined #openstack-dev14:05
garykrussellb: ping14:07
*** imsplitbit has joined #openstack-dev14:08
*** maurosr has quit IRC14:08
*** reidrac has joined #openstack-dev14:10
*** markmcclain has joined #openstack-dev14:13
*** markmcclain has quit IRC14:14
*** sc68cal has joined #openstack-dev14:16
*** m4xmr has joined #openstack-dev14:19
*** xarg has quit IRC14:19
*** Ruetobas has joined #openstack-dev14:20
*** alunduil has joined #openstack-dev14:23
*** devcamca1 has joined #openstack-dev14:23
*** alunduil has quit IRC14:24
*** alunduil has joined #openstack-dev14:25
*** m4xmr has left #openstack-dev14:28
*** rbasak has quit IRC14:28
*** rbasak has joined #openstack-dev14:28
jaypipesbcwaldon: FYI, that rabbitmq infinite loop is still occurring on a brand new devstack checkout...14:32
*** cppcabrera has joined #openstack-dev14:32
*** devcamca1 has quit IRC14:35
*** blamar has joined #openstack-dev14:36
*** mtreinish has joined #openstack-dev14:36
*** mrodden has joined #openstack-dev14:39
*** annegentle__ has joined #openstack-dev14:42
alexpilottiKiall: tx, "recheck" worked14:42
*** pvo_ has joined #openstack-dev14:43
*** blamar has quit IRC14:43
*** pvo has quit IRC14:45
*** pvo_ is now known as pvo14:45
*** pvo has joined #openstack-dev14:45
*** ChanServ sets mode: +v pvo14:45
*** blamar has joined #openstack-dev14:45
*** ndipanov has quit IRC14:45
*** ndipanov has joined #openstack-dev14:47
*** dolphm has joined #openstack-dev14:51
*** Mandell has joined #openstack-dev14:52
*** dolphm has quit IRC14:52
*** cppcabrera has left #openstack-dev14:53
*** danwent has joined #openstack-dev14:54
*** CaptTofu has quit IRC14:57
*** sc68cal has quit IRC14:57
russellbhi garyk14:58
garykrussellb: hi, regarding the monkey patch14:58
*** afazekas has quit IRC14:58
garykrussellb: it seems that when socket=False it "works". i agree that this is not good.14:58
*** sc68cal has joined #openstack-dev14:58
*** eharney has joined #openstack-dev14:58
*** eharney has joined #openstack-dev14:58
garykrussellb: do you understand what in qpid can or may cause the problem. it seems to hang when the monkey patching is done14:59
alexpilottigaryk: can you please explain?14:59
russellbgaryk: what other threads are running in quantum at the time14:59
garykalexpilotti: give me a sec and i'll post the bug.14:59
garykrussellb: none.15:00
alexpilottigaryk: but probably we are referring to two different patches with monkey patch issues :-)15:00
russellbah, so this is just the main thread, and that's it?15:00
*** jimfehlig has joined #openstack-dev15:00
garykrussellb: correct.15:00
garykrussellb: i reproduced it with a simple script - https://bugs.launchpad.net/quantum/+bug/107399915:00
uvirtbotLaunchpad bug 1073999 in quantum "Quantum service does not restart after reboot" [Critical,New]15:00
garykalexpilotti: https://bugs.launchpad.net/quantum/+bug/107399915:00
russellbgaryk: yeah, saw that.  the only thing with that is the cases where it works were monkey patched too15:01
garykrussellb: it works when all are patched except the sockets (does that make sense)15:01
russellbi'll play for a bit ...15:02
garykrussellb: ok, thanks. please let me know if you need anything.15:03
alexpilottirussellb: can you please take a look at https://review.openstack.org/#/c/15337/15:03
*** devcamca1 has joined #openstack-dev15:03
alexpilottirussellb: it's a blocking issue for volumes on Hyper-V15:03
*** markmcclain has joined #openstack-dev15:04
russellbalexpilotti: can you put a comment in the code about it?15:04
russellband it sounds like this is really an eventlet bug ... but we can put this workaround in for now15:04
*** TerryH has joined #openstack-dev15:04
garykalexpilotti: it seems like today is planet of the apes day15:04
russellbthis is going to be a problem on windows15:04
alexpilottigaryk: lol :-)15:05
russellbthis is removing a place where eventlet can schedule other threads15:05
alexpilottirussellb: sure15:05
*** mjfork has joined #openstack-dev15:05
*** edygarcia has joined #openstack-dev15:07
*** johngarbutt has joined #openstack-dev15:07
*** Tross1 has quit IRC15:07
*** sunilthaha has quit IRC15:10
russellbgaryk: your test code works fine for me, even with monkey patching on.15:13
*** SlickNik has joined #openstack-dev15:14
garykrussellb: this is odd. when you start qpidd do you see that it connects?15:14
*** devcamca1 has quit IRC15:14
russellbyes15:14
garykrussellb: which distribution are you using? i am using fedora 1715:15
russellbfedora 1715:15
russellbwant me to log in to a machine where you see the problem?15:15
garykrussellb: i will try and arrange remote access for you so that i can show it to you15:16
russellbok15:16
alexpilottirussellb: committed15:16
russellbalexpilotti: +2, thanks15:16
alexpilottirussellb: I'm also going to look for the eventlet manteirs, as I don't like the workaround as well15:17
russellbok15:17
alexpilottirussellb: tx!15:17
russellbno15:17
russellberr, np15:17
*** Tross has joined #openstack-dev15:19
*** SlickNik has left #openstack-dev15:19
*** SlickNik has joined #openstack-dev15:20
*** Tross has left #openstack-dev15:20
*** dhellmann has joined #openstack-dev15:21
*** rkukura has joined #openstack-dev15:23
*** Gordonz has joined #openstack-dev15:28
*** Gordonz has quit IRC15:28
ayoungewindisch, let me know when you have a minute to talk about your greenthreads patch15:28
*** gyee has joined #openstack-dev15:28
*** Gordonz has joined #openstack-dev15:28
ewindischayoung: sure - in a bit.15:29
ewindischnote that you don't need  to use the eventlet webserver to use eventlet.15:30
*** cp16net is now known as cp16net|away15:30
*** cp16net|away is now known as cp16net15:30
*** rnirmal has joined #openstack-dev15:31
*** cp16net is now known as cp16net|away15:32
ayoungewindisch, understood, but eventlet does not make sense when used in conjunction with non-eventlet based web servers15:34
ayounglet me know when you have time for a longer conversation.15:34
*** Tross has joined #openstack-dev15:37
*** zaitcev has joined #openstack-dev15:37
*** dolphm has joined #openstack-dev15:37
*** simonmcc has joined #openstack-dev15:39
ayoungdolphm, got a moment to talk monkeypatching and popen?15:39
ayoungdolphm, specifically what is done in  /usr/lib/python2.7/site-packages/eventlet/green/subprocess.py15:40
dolphmayoung: sure15:40
ayoungdolphm, I'm assuming you have that same file in roughly the same location.  Look at what they do at the bottom.  Can we do that smae hack?15:41
ayoungNamely15:41
ayoungcheck_call = new.function(subprocess_orig.check_call.func_code, globals())15:41
ayoungbut rather do it in the cms call?15:41
ayoungso change cms_verify and cms_sign_text to call the eventlet.green.subprocess.Popen without changing those files?15:42
*** andrewbogott is now known as andrewbogott_afk15:43
*** devcamca1 has joined #openstack-dev15:43
*** woodspa has joined #openstack-dev15:45
ewindischayoung: okay- available.15:48
ayoungewindisch, OK,  so there are a few things going on in service.py right now15:48
ayoungprobably the biggest is that I am refactoring the hell out of it15:48
*** Dorogs has joined #openstack-dev15:48
ayoungspecifically the authenticate call, which, as you can see has grown out of control15:49
openstackgerritA change was merged to openstack/nova: Imported Translations from Transifex  https://review.openstack.org/1534015:49
ewindischright. I actually was thinking of refactoring that, too - but I could only do so much in a single patch :)15:49
ayoungnow, where you are injecting greenthreads makes sense from a stricly eventlet based approach15:49
ewindischgood to know you're working on it15:49
ayounglet me finish the refactoring, first pass.  I was heads down working on it late last week, but got diverted to dealing with the zombie issue15:49
ayoungonce we have more fine grained methods to work with, you and I can talk about where and how to inject eventlet specific optimizations15:50
ayoungI will use your patch as a guide for where to break up the authenticate method so there are obvious functiosn to wrap.  That make sense?15:50
*** cp16net|away is now known as cp16net15:51
ewindischno problem. I'd like that method to be broken up - it needs it.15:51
ayounghowever, it might be possible to take the changes that you are making here and push them down a level, into the identity and token apis15:51
ewindischUnless you're moving more functionality into those APIs, they're not too bad right as they are right now.15:51
ayoungIt might make sense to try and slide in an eventlet shim layer between the callers of the identity methods and the implementation15:51
ewindischreally, the idea was just that if you're running a few non-related queries, we don't need to block on them.15:52
*** jpich has joined #openstack-dev15:52
ayoungno, not more functionality.  I might split up authenticate so that the same logic is always performed when building the authorization information15:52
ayoungewindisch, absolutely, and I like the idea of parallelizing the queries.15:53
*** devcamca1 has quit IRC15:53
ewindischayoung: but yeah, no rush on this patch. I just wanted to open the dialogue more than anything else, right now. None of the backends are greenlet friendly, anyway.15:53
ayoungyeah, there is that, too15:53
ayoungewindisch, OK, I think we are on the same page.  I'll ping you when the refactoring is up for review15:54
ewindischayoung: cool.15:54
*** danwent has quit IRC15:55
jeblairjgriffith: it looks there might be more non-deterministic volume test failures, have you seen this: http://logs.openstack.org/15317/1/gate/gate-tempest-devstack-vm/16899/console.html.gz15:57
*** mrunge_afk is now known as mrunge15:57
*** aeperezt has joined #openstack-dev15:58
*** colinmcnamara has joined #openstack-dev15:59
*** Mandell has quit IRC15:59
devanandaannegentle: ping15:59
*** Tross has quit IRC16:00
*** reidrac has quit IRC16:00
openstackgerritA change was merged to openstack/quantum: Implements _validate_uuid  https://review.openstack.org/1539616:00
*** winston-d has joined #openstack-dev16:01
*** datsun180b has joined #openstack-dev16:02
*** mrodden has quit IRC16:02
alogaayoung: good that you're working on refactoring authenticate()16:03
alexpilottirussellb: do you think we should consider the update.py script issue as a bug?16:04
ayoungaloga, I need to get back to that, but eventlet needs to be beaten into submission first16:04
alogaayoung: I've some code from my own16:04
russellbalexpilotti: what's the issue16:04
alexpilottirussellb: ref.: https://review.openstack.org/#/c/15401/16:04
alexpilottirussellb: see my answer to your comment16:05
alogaayoung: I can submit it so that you can get it if it is useful16:05
russellbalexpilotti: i guess it's a bug ...16:05
alexpilottirussellb: I just changed the EOL style manually16:05
russellbok16:05
*** colinmcnamara has left #openstack-dev16:06
alexpilottirussellb: for the rest the procedure is ok?16:06
ayoungaloga, you can send me your patch directly.16:06
*** Tross has joined #openstack-dev16:06
alexpilottirussellb: I mean, commit to openstack-common, running update.py and committing on the *client project?16:06
ayoungaloga, it is likely we have split things at different points, and it would have to be manually merged anyway, but I would love to see what you had in mind16:06
russellbyes that's the right process16:07
alogaayoung: it's just a bit of organization16:07
*** maurosr has joined #openstack-dev16:07
ayoungaloga, git format-patch it and send to me when you get a chance16:08
alexpilottirussellb: tx, going to update all the other clients16:08
*** zaneb has joined #openstack-dev16:10
*** zbitter has quit IRC16:11
*** mrodden has joined #openstack-dev16:12
bodenayoung -- hi, can I bounce a quick question off you?16:12
ayoungboden, please do16:12
*** Shrews has joined #openstack-dev16:13
bodenayoung -- has there been any discussion in the past regarding generating amqp notifications (http://wiki.openstack.org/NotificationSystem) from Keystone? for example a notification when a user or tenant is created16:14
ayoungboden, not that I am aware.  You have a need for that, it take it?16:14
*** spiffxp has joined #openstack-dev16:15
bodenayoung -- we might... in general it seems like 'evenones doing it'... I know glance/nova/etc are generating events for resource operations16:15
*** troytoman-away is now known as troytoman16:15
bodenayoung -- do you think such function is something keystone would entertain for master if IBM had the staffing to impl?16:16
ayoungboden, possibly.  We'd need to see that larger implications.16:16
*** nati_ueno has joined #openstack-dev16:17
bodenayoung -- fair enough... was just trying to get an initial feel for that space.... thx for your time16:17
*** anniec72 has joined #openstack-dev16:17
ayoungboden, so this kind of thing has come up in the past wrt to revocations16:18
*** roadTripper has joined #openstack-dev16:18
ayoungboden, There is the question of alternate back ends and what happens when a user is injected into LDAP directly16:18
bodenayoung -- a question regarding how you handle generating a notificaiton in that case you mean?16:19
*** devcamca1 has joined #openstack-dev16:20
ayoungboden, sure, or making sure that the remote systems don't break if a notification is not avaiable.  BTW, what use cases are you trying to support with notifications?16:20
ayoungboden, also, I am going to nix any RPC ties ins until we have a way to run the RPC code without eventlet16:20
ayoungWe'16:20
ayoungve worked to hard to get HTTPD support in there to break it now.16:21
*** danwent has joined #openstack-dev16:22
bodenayoung -- so a sample use case might be that I have a remote billing service and I want to be notified when a user/tenant is created/deleted/etc in keystone16:22
*** xarg has joined #openstack-dev16:22
ayoungboden, I'd almost prefer to make a use case like that be pull, not push.  Could you instead request a new REST API  which is "give me all users created after time t"?16:23
bodenayoung -- in my experience with these types of scenarios, pushing events has worked better although a consumer may have to do an initial 'pull' to sync up when its starting16:24
*** ondergetekende has quit IRC16:24
*** dhellmann has quit IRC16:26
*** maurosr has quit IRC16:27
*** maurosr has joined #openstack-dev16:27
*** corXi has quit IRC16:29
ayoungboden, understood.  But where we are talking about taking a purely API driven service like Keystone, we have to balance out the overhead of alternative mechanisms versus using the approaches we have.  Typically, with something message driven like that, you need a very high level of confidence that every message is delivered, and delivered once and only once.  Which means that typically you need to audit periodically to ensure you haven16:29
ayoung't lost anything anyway.  I'd be prone to write that auditing API first, and treat the event based one as a performance tune16:29
*** corXi has joined #openstack-dev16:29
*** markmcclain has quit IRC16:30
*** devcamca1 has quit IRC16:31
*** Tross1 has joined #openstack-dev16:32
*** Tross has quit IRC16:32
*** afazekas has joined #openstack-dev16:33
winston-drussellb, around?16:33
ewindischayoung: running RPC without eventlet will mean that we'd need to remove eventlet, period, everywhere.16:33
*** sc68cal has quit IRC16:33
ewindischOR have a separate set of RPC drivers that are eventlet-free.16:33
*** troytoman is now known as troytoman-away16:34
ayoungewindisch, you say that like it is a bad thing.16:34
ayoungewindisch, yes, I am aware of the repercussions16:34
openstackgerritA change was merged to openstack/quantum: Add check for cidr overrapping for adding external gateway  https://review.openstack.org/1349716:34
russellbwinston-d: yeah, kinda busy atm though16:34
openstackgerritA change was merged to openstack/quantum: Use uuidutils.is_uuid_like in quantum.extentions.l3  https://review.openstack.org/1540016:34
ayoungewindisch, that is why I want to be really careful about additional eventlet dependencies that we include.16:34
ewindischayoung: well, keep in mind that the patch I put forward could be just as easily done via threads as with eventlet.16:35
winston-drussellb, ok.  wondering if you'd help review https://review.openstack.org/#/c/14119/8 when u r free, please?16:35
ayoungSo far, Keystone has not really had anything Eventlet specific that we can't work around.  I think that RPC would change that.  However,  there is nothing in AMQP or in Zero MQ that would not work find in an Eventlet agnostic system16:35
ewindisch(in fact, I had thought of using the threading module instead, since eventlet would monkey_patch it anyway)16:35
russellbwinston-d: k16:36
winston-drussellb, great, thx!16:36
ayoungewindisch, yes, I was thinking along those lines.  Maybe have two shim layers that could be selected at Run time:  greenlet versus pthread16:36
ewindischayoung: I'd need a non-trivial rewrite for the zeromq driver to work outside eventlet.16:36
ayoungewindisch, Ah, I wasn't talking about the Messagin Zero MQ driver, but Zero MQ itslef16:37
ayoungitself16:37
ayounglet me try to make that clearer16:37
ewindischRight. I'm just clarifying that the messaging driver is built around eventlet. I'd have done things much differently (and better) were it not for eventlet.16:38
*** dani4571 has joined #openstack-dev16:39
ewindischayoung: also note that I'm putting forward a patch to (optionally) put logging into a thread. That uses greenthreads, since, well...16:39
ayoungewindisch, yep,  makes sense...ugh16:39
ayoungperhaps it does make sense to use the threading API for that stuff, and let greenlet monkeypatch it16:40
ayoungwe already have a command line switch to turn on/off monkeypatching thread,16:40
ewindischnote that eventlet only monkey-patches threading, not multiprocessing.16:41
openstackgerritA change was merged to openstack/quantum: Log loaded extension messages as INFO not WARNING.  https://review.openstack.org/1534616:41
*** nunosantos has joined #openstack-dev16:43
*** jcmartin has joined #openstack-dev16:43
ayoungewindisch, BTW,  did you see my question to Dolph?  It was around the same issue:16:45
*** SlickNik has quit IRC16:46
ayoungI want to keep the cms code non-eventlet specific, and was wondering if I could do the same hack as eventlet does in patching check_call etc16:46
*** andrewbogott_afk is now known as andrewbogott16:46
ewindischI didn't see it. I see a ML conv from September?16:47
*** heckj has joined #openstack-dev16:48
jgriffithjeblair: sighhh16:48
jgriffithjeblair: have you seen that on a regular basis?  First time I've seen a forbidden response (I think)16:48
bodenayoung -- understand your points, tho I'll point out that glance seems to be an api driven service and its creating events... I guess we can have this discussion later need be, for now I guess its just food for thought16:48
jgriffithor unauth16:48
*** mnewby has joined #openstack-dev16:48
jeblairjgriffith: notmyname and dansmith reported a few instances of that16:49
ewindischayoung: oh, the Popen thread?16:49
ayoungewindisch, yeah16:49
*** markwash has joined #openstack-dev16:50
jgriffithjeblair: hmmm... well that's bad16:50
ayoungewindisch, we can't just monkeypatch Popen, so I'd like to monkeypatch keystone/common/cms.py to use the green version instead16:50
*** dani4571 has quit IRC16:50
*** jtran has joined #openstack-dev16:51
maploinis there any precedent for writing database backend-specific tests? say for postgres?16:51
ewindischayoung: Is this running as a fastcgi script in Apache?16:53
ewindischoh, mod_wsgi, right?16:53
fungijgriffith: jeblair seems to have ticked up over the weekend, so hopefully a recently introduced race or something?16:53
notmynamejgriffith: for example, look at the recurring errors on https://review.openstack.org/#/c/15317/16:53
*** corXi has quit IRC16:53
*** corXi has joined #openstack-dev16:54
*** corXi has quit IRC16:54
jgriffithnotmyname: Yeah...16:54
ayoungewindisch, when running in Apache I was planning mod_wsgi in prefork mode.  Hadn't looked beyond that.16:54
jgriffithnotmyname: jeblair So we had a patch to do some retries for expired tokens and such16:54
*** corXi has joined #openstack-dev16:54
jgriffithnotmyname: jeblair It broke the gate tests though so it's been reverted, I'll see if I can get that going again16:54
jgriffithjgriffith: notmyname Of course that makes some assumptions about the error16:55
*** maploin has quit IRC16:55
*** nati_ueno has quit IRC16:55
*** corXi has quit IRC16:55
*** rburon has quit IRC16:56
*** corXi has joined #openstack-dev16:56
ewindischayoung: how are using greenthreads a problem for that? Each request goes to a separate single-threaded apache child process. That single-threaded child can use greenthreading internally.16:57
ewindischthat shouldn't block IO for other children or requests...16:57
*** koolhead17 has joined #openstack-dev16:57
*** devcamca1 has joined #openstack-dev16:58
alexpilottivishy: Can you please check this one? https://review.openstack.org/#/c/15337/16:59
claygso at onetime migrate.cfg used to be referenced in MANIFEST.in - but not anymore?  Is there something special in packaging that needs to happen to make that file get pushed out to /usr/lib/pymodules/python2.7/${PROJECT}/db/sqlalchemy/migrate_repo?17:05
dolphmheckj: finally showing passing tests on this https://review.openstack.org/#/c/15066/17:06
heckjdolphm: YEAH!!!17:06
dolphmheckj: same for client https://review.openstack.org/#/c/15297/17:06
heckjdolphm: (I was supposed to try that again friday, but lost track...)17:07
*** mindpixel has quit IRC17:07
dolphmheckj: no worries17:07
heckjdolphm: https://review.openstack.org/#/c/15066/ approved17:07
heckjhttps://review.openstack.org/#/c/15297/ approved17:08
*** devcamca1 has quit IRC17:10
ayoungewindisch, true, we could use greenthreads there, and it might be fine to leave them in place.17:11
ayoungBut I don't want to bank on it17:12
ayoungdolphm, rock on!17:12
openstackgerritA change was merged to openstack/nova: Add uuidutils module  https://review.openstack.org/1537817:15
*** sagar_nikam has joined #openstack-dev17:16
*** mmagr has quit IRC17:16
*** alop has joined #openstack-dev17:20
*** troytoman-away is now known as troytoman17:20
*** markwash has quit IRC17:22
*** anniec72 has quit IRC17:24
*** garyk has quit IRC17:24
*** shardy_afk is now known as shardy17:28
*** corXi has quit IRC17:30
*** metral has joined #openstack-dev17:31
*** Gordonz has quit IRC17:31
*** Gordonz has joined #openstack-dev17:32
*** jergerber has quit IRC17:33
ayoungheckj, dolphm ewindisch OK, I think I have a minorly elegant solution to the Zombie problem.17:34
*** metral has quit IRC17:34
ayoungIt involves breaking a coding convention17:34
ayoungin cms17:34
ayoungfrom subprocess import Popen17:34
*** SlickNik has joined #openstack-dev17:35
ayoungand in the wsgi startup code (project specific)17:35
ayoungfrom eventlet.green.subprocess import Popen17:35
ayoungthen later17:35
ayoung    cms.Popen = Popen17:35
ayoungI could probably change the second one to17:35
ayoung from eventlet.green import subprocess17:36
ayoungand then17:36
ayoung cms.Popen = subprocess.Popen17:36
*** otherwiseguy has joined #openstack-dev17:40
*** lloydde has joined #openstack-dev17:41
alexpilottidanwent: hi17:41
*** devcamca1 has joined #openstack-dev17:41
danwentalexpilotti: hey17:42
alexpilottidanwent: did you receive my email related to the quantum plugins?17:42
*** sniperd has joined #openstack-dev17:42
danwentalexpilotti: yeah, great to hear that you're planning on doing hyper-v work on it.17:43
danwentam just way behind on replying to mail :P17:43
alexpilottidanwent: np :-)17:43
danwentcan you do me a favor and create a blueprint for hyper-v pluign in quantum?17:43
alexpilottidanwent: we have it already, let me find the URL17:44
danwentah, maybe its new since I did my last sweep17:44
alexpilottidanwent: I'm also going to send you some docs17:44
alexpilottidanwent: btw when you have time, can you please check this review: https://review.openstack.org/#/c/15389/17:44
danwentk17:45
*** sagar_nikam has quit IRC17:45
alexpilottidanwent: it's just the neverending story of the openstack-common change that needs to propagate17:45
danwentbtw, i'm not seeing a blueprint for quantum on hyperv: https://blueprints.launchpad.net/quantum17:45
danwentalexpilotti: ah, so that change has now merged into common, and the link you sent out is just a refresh of common?17:46
danwentor rather, pulling a new version of common into python-quantumclient?17:46
*** dhellmann has joined #openstack-dev17:46
alexpilottidanwent: the latter, using the update.py tool in openstack-common17:47
danwentalexpilotti: ok, great17:47
danwentbtw, do you have link to the quantum bp your created?  I'm not seeing one.17:47
alexpilottidanwent: https://blueprints.launchpad.net/nova/+spec/quantum-plugin-hyper-v17:47
alexpilottidanwent: now I see, it went under nova for some reason, lol17:47
*** thingee_zz has joined #openstack-dev17:48
danwentalexpilotti: ah, that explains it.  I think you can re-target17:48
*** littleidea has quit IRC17:48
russellbi don't think blueprints can be moved17:48
russellbi was trying the other day :(17:48
heckjayoung: seems like a pretty reasonable solution to the ick17:48
*** devcamca1 has quit IRC17:48
alexpilottidanwent: done17:48
*** thingee_zz is now known as thingee17:48
alexpilottirussellb: it worked17:48
alexpilottidanwent: https://blueprints.launchpad.net/quantum/+spec/quantum-plugin-hyper-v17:49
russellbalexpilotti: oh reallllly, neat.  :)17:49
russellbmaybe i just couldn't move someone else's ... anyyyway.17:49
danwentalexpilotti: k, updated it.  what milestone are you targeting?  g-1 is in two weeks.17:49
alexpilottirussellb: I was also not expecting it to be so wasy ;-)17:49
ayoungheckj, I'll have a patch here shortly.17:49
danwentrussellb: yeah, you have to own it, i believe17:50
alexpilottidanwent: we are pushing for g-1, but g-2 is a more realisting target17:50
*** markmcclain has joined #openstack-dev17:51
danwentalexpilotti: ok.  and we'll also have to figure out who on the core team will be the maintainer for this plugin.  The usual model is either that you have someone who wrote the plugin do enough community work to become a core dev, or you have a maintainer that works with an existing core dev.17:51
openstackgerritA change was merged to openstack/openstack-common: Remove unused greenthread import in lockutils  https://review.openstack.org/1538217:52
alexpilottidanwent: we'd like to have somebody becoming a core dev (me, pnavarro, or ociuhandu)17:52
danwentalexpilotti hey, sorry, something urgent came up, I've got to run for a bit.17:52
alexpilottidanwent: np, ttyl!17:52
*** littleidea has joined #openstack-dev17:54
*** Tross1 has quit IRC17:56
*** vipul has joined #openstack-dev17:58
*** cp16net is now known as cp16net|away17:58
*** andrewbogott is now known as andrewbogott_afk18:00
*** sacharya has joined #openstack-dev18:03

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