Monday, 2012-03-19

notmynamewhat we do for cloud files at rax is deploy the code separate from running the code. ie apt-get install doesn't start any services. so install it and then start it00:00
justinsb_notmyname: I think what you do at cloud files is in a totally different league to what we're talking about with a typical Ubuntu user's installation :-)00:01
notmynameheh. I'll buy that :-)00:01
justinsb_notmyname: If they install with replicas=1, are they "stuck" at the moment if they want to go to e.g. replicas="2".  They'd have to dump all their files out and reload them?00:02
notmynamethere is no way to change the replica count right now. if you change it, you have to migrate the data so that it gets rehashed into the right location00:03
notmynamesame with the partition count (set by the ring's partition power)00:04
justinsb_notmyname: Does changing the replica count cause a rehash?  I thought it just added another location?  If it requires a rehash, my testing isn't likely to work!00:05
notmynameno, not directly. but since there is no way to add a replica now, it would mean building a new ring. and since volumes have a random component to their placement, things would need to get rehashed00:06
notmynamebut if you are adding a replica (or removing) to the existing ring data structure, that could still work without rehashing all the data00:06
justinsb_notmyname: OK!  I think maybe I'll test it out, and see if it works (this may be academic anyway).  It may be that we can keep this as an 'experimental, dangerous' program like those programs that resized your disk partitions back in the day00:07
*** Mandell has quit IRC00:08
justinsb_notmyname: I've proposed a discussion of packaging issues for the design summit (like this one).  Maybe we can tack this on or have a separate discussion just about growing swift from first install to happy cluster user00:08
justinsb_notmyname: I'm pretty new to swift, so I'm not sure if there are other issues that people encounter along the way00:08
justinsb_notmyname: And maybe we can have either replica count increment or as unique as possible in Folsom00:09
notmynamejustinsb_: ya, I'd love to have a talk of 2 at the summit on it. please submit it :-)00:12
justinsb_notmyname: "Of 2" = "Growing swift from first install to happy cluster" ?00:13
notmynameoops00:14
*** btorch_ has joined #openstack-dev00:14
notmynames/a talk of 2/a talk or 2/00:14
justinsb_notmyname: OK!  I'll submit a discussion topic along those lines and if you want to add to it feel free!00:15
notmynamethanks00:15
*** thingee_zzz has quit IRC00:20
*** btorch_ has quit IRC00:24
*** ncode has joined #openstack-dev00:31
*** dolphm has joined #openstack-dev00:33
justinsb_notmyname: Just tried it using devstack and it does seem to work!  Cool!00:44
*** dolphm has quit IRC00:49
*** dolphm has joined #openstack-dev00:54
justinsb_notmyname: I think I'll try a real test with two machines, and then shut down the first one.  But it's a credit to swift that something this evil does seem to work!00:54
*** flaviamissi has quit IRC01:01
*** flaviamissi has joined #openstack-dev01:01
*** flaviamissi has quit IRC01:06
*** dprince has joined #openstack-dev01:19
*** rkukura has joined #openstack-dev01:21
*** dprince has quit IRC01:25
*** dolphm has quit IRC01:58
*** dolphm_ has joined #openstack-dev02:00
*** dtroyer_zzz is now known as dtroyer02:14
*** ncode has quit IRC02:48
*** jakedahn_zz is now known as jakedahn03:06
*** dolphm has joined #openstack-dev03:27
*** dolphm_ has quit IRC03:30
*** deshantm has quit IRC03:34
*** deshantm has joined #openstack-dev03:41
*** ywu has quit IRC03:51
*** dtroyer is now known as dtroyer_zzz03:53
*** Mandell has joined #openstack-dev03:53
*** mikal has quit IRC03:59
*** dolphm has quit IRC04:19
*** mdomsch has joined #openstack-dev04:31
*** anotherjesse is now known as anotherjesse_zz04:40
*** ghanima has joined #openstack-dev04:45
*** anotherjesse_zz is now known as anotherjesse04:46
*** Mandell has quit IRC04:50
*** ghanima has left #openstack-dev04:52
*** dolphm has joined #openstack-dev04:52
*** dolphm has quit IRC04:52
*** Mandell has joined #openstack-dev04:55
*** hattwick has quit IRC05:02
*** mikal has joined #openstack-dev05:16
*** mdomsch has quit IRC05:19
*** Mandell has quit IRC05:19
*** sniperd has quit IRC05:22
*** Mandell has joined #openstack-dev05:28
*** Mandell has quit IRC05:36
*** deshantm has quit IRC05:40
*** Mandell has joined #openstack-dev05:51
*** neodosh has joined #openstack-dev06:04
*** jakedahn is now known as jakedahn_zz06:12
*** dachary has quit IRC06:15
*** halfss has joined #openstack-dev06:22
vishytr3buchet: we have a potentially very nasty performance issue in the dhcp code in linux_net.  We are doing 2 queries for every running instance.  I think we may have to add in a nasty join to get reasoable performance out of it06:22
vishytr3buchet: I know we want to keep the stuff separate, but doing 2000 queries for every instance launch or terminate if there are 1000 instances running is not acceptable06:23
*** anotherjesse is now known as anotherjesse_zz06:34
*** jakedahn_zz is now known as jakedahn06:45
*** journeeman has joined #openstack-dev06:53
journeemanHi all. Didn't Openstack make it to GSoC 2012?06:53
journeemanI was really looking forward to participate :/06:55
journeemanWell, the Ideas page is still there so I'll continue looking into the ideas to work on this summer.06:57
*** Mandell has quit IRC07:10
*** jakedahn is now known as jakedahn_zz07:14
*** halfss has quit IRC07:29
*** apevec has joined #openstack-dev07:44
*** danwent has quit IRC07:48
*** hattwick has joined #openstack-dev07:59
*** popux has joined #openstack-dev08:10
*** neodosh has quit IRC08:18
*** anotherjesse_zz is now known as anotherjesse08:25
*** popux has quit IRC08:30
*** anotherjesse is now known as anotherjesse_zz08:33
*** dneary has joined #openstack-dev08:38
*** dneary has quit IRC08:38
*** dneary has joined #openstack-dev08:38
*** rbasak has joined #openstack-dev08:39
*** halfss has joined #openstack-dev08:40
*** garyk has joined #openstack-dev09:05
*** Ryan_Lane has quit IRC09:07
*** koolhead11 has joined #openstack-dev09:08
*** Gordonz has joined #openstack-dev09:10
*** Gordonz has quit IRC09:10
*** Gordonz has joined #openstack-dev09:10
*** Gordonz has quit IRC09:23
*** LinuxJedi has joined #openstack-dev09:34
*** reidrac has joined #openstack-dev09:58
*** zigo has joined #openstack-dev10:02
*** oneiroi has joined #openstack-dev10:06
*** shevek_ has quit IRC10:26
*** kyriakos has left #openstack-dev10:31
*** hattwick has quit IRC10:32
*** paulormg has joined #openstack-dev10:46
*** rkukura has quit IRC10:58
*** milner has joined #openstack-dev11:10
*** crobinso has joined #openstack-dev11:17
*** dachary has joined #openstack-dev11:20
*** zigo has quit IRC11:21
*** zigo has joined #openstack-dev11:21
*** zul has joined #openstack-dev11:24
*** dneary has quit IRC11:26
*** zigo has quit IRC11:26
*** Shrews has joined #openstack-dev11:28
*** bsza has joined #openstack-dev11:38
*** journeeman has quit IRC11:39
*** vizsla has joined #openstack-dev11:46
*** vizsla has left #openstack-dev11:47
*** vizsla has joined #openstack-dev11:47
*** vizsla has quit IRC11:57
*** vizsla has joined #openstack-dev11:57
*** asalkeld has quit IRC12:14
*** oneiroi has quit IRC12:20
*** oneiroi has joined #openstack-dev12:20
*** bsza has quit IRC12:24
*** maploin has joined #openstack-dev12:38
*** maploin has quit IRC12:38
*** maploin has joined #openstack-dev12:38
*** andrewsmedina has quit IRC12:43
*** refactor has joined #openstack-dev13:01
*** sandywalsh has quit IRC13:01
*** berendt has joined #openstack-dev13:03
*** dprince has joined #openstack-dev13:06
*** mattray has joined #openstack-dev13:08
*** davidkranz has joined #openstack-dev13:08
*** andrewsmedina has joined #openstack-dev13:14
*** mattray has quit IRC13:14
*** sandywalsh has joined #openstack-dev13:15
*** mattray has joined #openstack-dev13:16
*** kbringard has joined #openstack-dev13:18
*** zigo has joined #openstack-dev13:23
*** blamar has joined #openstack-dev13:25
*** deshantm has joined #openstack-dev13:25
*** berendt has quit IRC13:28
*** kbringard has quit IRC13:29
*** kbringard has joined #openstack-dev13:30
*** kbringard has quit IRC13:31
*** rkukura has joined #openstack-dev13:40
*** kbringard has joined #openstack-dev13:40
jaypipesblamar: mornin.13:40
blamarjaypipes: greetings.13:41
jaypipesblamar: I'm good with your KeyError patch. just get eglynn__ or bcwaldon for another +1\13:41
jaypipesblamar: and thanks!13:41
blamarjaypipes: many many thanks to you sir13:41
blamarwill do13:41
*** dneary has joined #openstack-dev13:43
*** dneary has quit IRC13:43
*** dneary has joined #openstack-dev13:43
*** gpernot has joined #openstack-dev13:47
jaypipesdprince: around?13:47
kbringardhmmm, what is the status of the nova2ools stuff?13:49
*** mattray has quit IRC14:01
*** mattray has joined #openstack-dev14:02
*** berendt has joined #openstack-dev14:02
*** hattwick has joined #openstack-dev14:02
*** berendt has quit IRC14:02
*** AlanClark has joined #openstack-dev14:02
*** blamar has quit IRC14:06
*** ayoung is now known as ayoung-afk14:07
*** deshantm_ has joined #openstack-dev14:10
*** sdake has quit IRC14:10
*** deshantm has quit IRC14:11
*** deshantm_ is now known as deshantm14:11
*** dolphm has joined #openstack-dev14:12
*** sdake has joined #openstack-dev14:12
*** ayoung-afk is now known as ayoung14:22
*** zzed has joined #openstack-dev14:23
*** mdomsch has joined #openstack-dev14:23
*** lts has joined #openstack-dev14:25
*** dolphm has quit IRC14:31
*** tryggvil_ has joined #openstack-dev14:33
*** andrewbogott has joined #openstack-dev14:34
*** andrewbogott has joined #openstack-dev14:34
eglynn__jeblair: ping14:34
andrewbogottI'm having a rotten time trying to get essex installed via apt.  Does essex pretty much require a Precise beta in order to install?  (On oneiric I see quite a few unmet and un-meetable dependencies.)14:35
davidkranzIs cloud-publish-tarball supposed to work in essex with  ubuntu-11.10-server-cloudimg-amd64.tar.gz? I am doing the same as I did with oneiric/diablo (no keystone) but now get trace back in compute log about DecryptionFailure.14:37
davidkranzOther euca commands work.14:37
davidkranzI deployed essex with precise packages.14:38
davidkranzUsing credentials from novarc generated by nova-manage14:41
*** dtroyer_zzz is now known as dtroyer14:44
*** rnirmal has joined #openstack-dev14:45
ttxandrewbogott: do you have an example of those unmet/unmeetable deps in Oneiric ?14:55
andrewbogottttx:  I had to melt down my test machine, but I should be back to that point shortly.14:56
andrewbogottttx:  You would expect it to work on oneiric?14:57
ttxandrewbogott: my understanding is that it's mostly usable under Oneiric, as evidenced by Devstack14:57
andrewbogottdoesn't devstack install essex as source though, rather than using apt?14:57
andrewbogottdevstack works great for me; it's the dependency-checking in 'apt-get install glance' that's failing.14:58
bcwaldonandrewbogott: (yes, it uses source)14:58
ttxandrewbogott: there may have been some runtime-deps that crept in14:58
ttxandrewbogott: hence my question :)14:58
andrewbogottok -- lemme catch up and I'll let you know when I hit a wall.14:58
ttxandrewbogott: what packages are you using ?14:59
jeblaireglynn__: pong14:59
jeblair jeblair: could you re-enable GLANCE_TEST_S3_CONF & GLANCE_TEST_SWIFT_CONF for the glance builds in Jenkins?14:59
jeblaireglynn__: ^ i'll do that now14:59
andrewbogottttx:  nova-core/trunk15:01
jeblaireglynn__: done15:02
ttxandrewbogott: <jeditrick>that's not the PPA you're looking for.</jeditrick>15:03
andrewbogottare there stable essex packages someplace?15:03
ttxandrewbogott: try http://wiki.openstack.org/PPAs15:03
ttx"stable essex" haha.15:03
*** tryggvil_ has quit IRC15:03
*** tryggvil__ has joined #openstack-dev15:04
ttxI need to find a nova-core volunteer to mark the nova-core ppas deprecated15:04
andrewbogottOh, so nova-core/trunk isn't just bleeding edge, it's completely unmaintained?15:04
ttxandrewbogott: it's not refreshed anymore15:04
andrewbogottok!  Well, that's encouraging then :)15:04
ttxautomatic packages build go to the other (common) one15:05
ttxandrewbogott: sent an email about it but you may have missed it15:05
mtaylorttx: send me an email to remind me and I'll takecare of it15:05
ttxmtaylor: was about to ask you to do the PROJECT-core PPA tour :)15:05
ttxmtaylor: willdo15:05
andrewbogottttx:  I believe that this is the page that misled me (or, one of them):  http://docs.openstack.org/bexar/openstack-compute/admin/content/ch03s02.html15:06
andrewbogottOh, I guess 'bexar' should've tipped me off :(15:06
eglynn__jeblair: thanks!15:06
ttxandrewbogott: it used to be all the rage15:06
mtaylorttx: perhaps we should remove bexar and cactus info?15:07
mtaylorannegentle: ^^^15:07
*** vincentricci has joined #openstack-dev15:07
andrewbogottHm... actually, I was also looking at this page, which is outside our sphere of influence... http://www.zeroaccess.org/2011/03/setting-up-openstack-compute/15:10
andrewbogottBasically my error was searching for 'package' and rather than 'ppa'.  It's only when I google for 'ppa' that http://wiki.openstack.org/PPAs shows up.15:11
*** rnirmal has quit IRC15:11
annegentleandrewbogott: yeah I think i'm going to remove /bexar and /cactus from the docs site and the Google CSE this week15:11
annegentleandrewbogott: blogs I can't do much about right :)15:11
*** rnirmal has joined #openstack-dev15:11
*** devananda has joined #openstack-dev15:12
andrewbogottannegentle:  Maybe add some keywords to that wiki page like 'debian package' and 'apt' and 'ubuntu' &c.15:12
andrewbogottNot that any of these changes will solve the problem of users not thinking ( <- the actual source of my problem)15:12
*** markwash has joined #openstack-dev15:13
bcwaldonjaypipes: https://review.openstack.org/#change,550915:17
bcwaldonchmouel: https://review.openstack.org/#change,550915:17
jaypipesjeblair: looks like the GLANCE_TEST_ env variables are still not unset on the py2.7 tox builder: https://jenkins.openstack.org/job/gate-glance-python27/144/console15:17
ttxandrewbogott: feel free to SEO that wikipage with anything necessary :)15:18
bcwaldonjaypipes: I just ran devstack locally with that change and everything went smoothly. We need to get this review in for my other glance change to land15:18
jaypipesbcwaldon: ++15:18
*** dolphm has joined #openstack-dev15:20
jeblairjaypipes: i just set the variables at eglynn__'s request.15:21
jaypipeseglynn__: just now?15:22
jeblairjaypipes: i completed setting them at 2 minutes past the hour.15:22
*** phorgh has joined #openstack-dev15:23
jaypipesjeblair: k. I fear we need to unset them again... eglynn__, got a sec?15:24
jeblairjaypipes: ok.  are you ready for me to do that now?15:24
eglynn__jaypipes: yep15:24
jaypipesjeblair: yes please. we need to push through some keystone-related critical fixes.15:25
jeblairjaypipes, eglynn__: unset15:25
jaypipeseglynn__: want to chat about latest status with this bug about S3 and 400 vs 200 return?15:25
jaypipesjeblair: ty15:25
eglynn__jaypipes: reason I wanted those variables set was to preproduce the S3 issue with the new log capture15:25
jaypipeseglynn__: k. did you see the output?15:26
eglynn__jaypipes: 400 versus 200 status in what context?15:26
jaypipeseglynn__: https://jenkins.openstack.org/job/gate-glance-python27/144/console15:27
*** bsza has joined #openstack-dev15:27
jaypipessorry, 404 != 20015:27
jaypipeseglynn__: I don't see any further output in the log capture there, do you?15:27
eglynn__jaypipes: yep, you're right15:28
* eglynn__ confused ... looking into it15:28
jeblairjaypipes, eglynn__: i set up two new jobs for you:15:29
jeblairhttps://jenkins.openstack.org/job/dev-gate-glance-python26-swifts3/15:29
jeblairhttps://jenkins.openstack.org/job/dev-gate-glance-python27-swifts3/15:29
jaypipesjeblair: ah, thank you jim15:30
eglynn__jeblair: thank you!15:30
jeblairthey are just like the normal unit test gate jobs, but have the variables set, and they run in silent mode15:30
jeblairso they'll be trigged by all the same changes, but won't affect jenkins vote15:30
jeblairyou'll have to visit the jenkins page for the jobs to see them (they won't show up in gerrit at all)15:30
eglynn__jeblair: excellent15:31
jeblairgood luck!15:31
eglynn__(now to figure out why the log capture isn't capturing ...)15:31
*** danwent has joined #openstack-dev15:31
bcwaldoncan a keystoner look at the 'configuringservices.rst' docs today? They seem to be out of date (wrong configuration values, preferring admin_token over full creds, etc)15:34
bcwaldondolphm, heckj: ^15:34
*** troytoman-away is now known as troytoman15:37
*** rods has joined #openstack-dev15:38
dolphmbcwaldon: i'll check it out15:39
bcwaldondolphm: thanks!15:39
*** heckj has joined #openstack-dev15:42
*** koolhead11 has quit IRC15:42
*** littleidea has joined #openstack-dev15:43
*** sandywalsh has quit IRC15:47
*** maploin has quit IRC15:48
*** gpernot has quit IRC15:50
*** vizsla has quit IRC15:50
*** littleidea has quit IRC15:50
*** gpernot has joined #openstack-dev15:50
bcwaldonjaypipes: this might conflict with your tenant-name -> id work: https://review.openstack.org/#change,551715:53
*** anotherjesse_zz is now known as anotherjesse15:53
*** littleidea has joined #openstack-dev15:55
jaypipesbcwaldon: actually, I just abandoned the original patchset I made for the tenant name bug... since it modded glance_auth_token that is now removed.15:55
bcwaldonjaypipes: ok, I wasnt sure if you had started on anything locally, that's what I was warning you about :)15:56
jaypipesbcwaldon: you want to take the tenant name bug? I think the thing that needs to be done is just a migration script..15:56
* annegentle laughs at keystoner15:56
bcwaldonjaypipes: yeah, I can do it15:56
dolphmi want a tshirt15:56
dolphmjust sayin15:56
bcwaldonannegentle: I'm surprised we havent made that connection yet15:56
jaypipesbcwaldon: cool, thx man. I'm currently swamped working on https://review.openstack.org/#change,529415:56
bcwaldonjaypipes: I'll gladly help you today15:57
bcwaldonjaypipes: for a hamburger tomorrow15:57
bcwaldonor something like that15:57
*** thingee has joined #openstack-dev15:59
*** pixelbeat has joined #openstack-dev15:59
*** sandywalsh has joined #openstack-dev16:01
jaypipesbcwaldon: thx Wimpy :)16:03
bcwaldonthat's his name! I couldnt remember what it was16:03
bcwaldonbefore my time16:03
jaypipes:)16:03
heckjjaypipes: just settling in to do some reviews - are you expecting to do more work on https://review.openstack.org/#change,5294, or is it ready for some eyeballs?16:06
*** zaitcev has joined #openstack-dev16:08
jaypipesheckj: just now pushed the final patchset16:10
heckjsweet - I'll catch it in my run this morning!16:10
jaypipescheers16:12
annegentleAnyone who wants to review the essex version of the install guide, have at it at https://review.openstack.org/#change,519016:14
*** danwent has quit IRC16:14
*** bengrue has quit IRC16:15
*** sandywalsh has quit IRC16:18
*** bengrue has joined #openstack-dev16:24
chmoueldolphm, heckj: https://review.openstack.org/#change,551116:30
* heckj lookin16:30
*** openstackjenkins has quit IRC16:30
*** openstackjenkins has joined #openstack-dev16:31
*** sandywalsh has joined #openstack-dev16:33
eglynn__jaypipes / jeblair: gah! the reason that extra glance log capture didn't appear is that Jenkins builds is driven by tox which calls the nosetests cmd directly instead of glance/run_tests.sh16:34
eglynn__(the glance logcapture plugin is loaded in run_tests.py)16:34
eglynn__OTOH running nosetests --glance-logcapture would require the plugin to be installed setuptools stylee instead of being picked up dynamically as it is now16:34
*** zul has quit IRC16:34
eglynn__jeblair: would it be poss for tox to just call glance/run_tests.sh?16:35
*** spiffxp has joined #openstack-dev16:38
*** sandywalsh has quit IRC16:38
*** dubsquared has joined #openstack-dev16:39
jeblaireglynn__: i don't want to step on mtaylor's toes here, he's leading the tox stuff.  let's see if we can get his opinion.16:40
jeblairmtaylor: ^16:40
eglynn__jeblair: cool16:40
*** sniperd has joined #openstack-dev16:40
*** Gordonz has joined #openstack-dev16:42
heckjbcwaldon: when you have a moment, would you mind giving https://review.openstack.org/#change,5350 a look16:43
*** paulormg has quit IRC16:45
*** jdurgin has joined #openstack-dev16:45
bcwaldonheckj: yep, going to look over all your docs changes this morning16:46
heckjthxusir!16:46
*** PotHix has joined #openstack-dev16:47
*** novas0x2a|laptop has quit IRC16:48
jaypipesjeblair: just pushed a proposed change to openstack-ci/devstack-gate. Really just looking for some initial feedback... bash scripting ain't my specialty ;)16:50
jaypipesjeblair: changes are attempting to get to the point where we can run tempest against the devstack-vm16:50
*** sandywalsh has joined #openstack-dev16:51
*** zul has joined #openstack-dev16:51
*** garyk has quit IRC16:51
*** sdake has quit IRC16:52
*** sdake has joined #openstack-dev16:53
*** gyee has joined #openstack-dev16:55
*** shevek_ has joined #openstack-dev16:56
mtaylorola16:56
*** gyee has quit IRC16:56
mtayloreglynn__: I would very much like to delete all of hte run_tests.py that exist anywhere16:56
mtayloreglynn__: so if we could get glance-logcapture published to pypi so that wecan properly build-depend on it, that would make me a happy camper16:57
*** adjohn has joined #openstack-dev16:57
mtayloreglynn__: also, if we're going to do that, we should make it a gerrit-controlled repo like we did for openstack.nose_plugin, no?16:58
*** gyee has joined #openstack-dev16:58
eglynn__mtaylor: well its a tiny plugin, seems a bit OTT to set up a repo and publish to pypi etc. ... I'll check first if I can figure out how to load it directly with nosetests16:59
mtayloreglynn__: ok. I mean, we can also have tox call something else ... but I'd really prefer to mve us away from custom test runners17:02
eglynn__mtaylor: cool, I'll have a dig and see if I figure out something cleaner than calling run_tests.sh directly17:03
mtayloreglynn__: cool. thanks!17:09
jeblairjaypipes: cool! left some feedback17:09
*** mattstep has quit IRC17:10
*** mattstep has joined #openstack-dev17:11
jaypipesjeblair: thx man!17:11
*** dneary has quit IRC17:18
heckjmarkmc: ping17:20
*** dachary has quit IRC17:21
*** shevek_ has quit IRC17:25
eglynn__heckj: it's a public holiday today in Ireland, so he may not be around ...17:26
*** andrewsmedina has quit IRC17:26
heckjeglynn__ - well darnit. Thanks for the head's up!17:27
*** andrewsmedina has joined #openstack-dev17:27
bcwaldoneglynn__: which holiday is it?17:28
bcwaldoneglynn__: off today for st. patty's day?17:28
eglynn__bcwaldon: in lieu of St Patricks day (fell on Saturday this year)17:28
eglynn__yep17:28
*** bengrue has quit IRC17:29
KiallYea - Its unlikely anyone in Ireland (bar me) is working today ;)17:31
*** anotherjesse is now known as anotherjesse_zz17:31
*** anotherjesse_zz is now known as anotherjesse17:32
Kiallalso.. bcwaldon its St. Paddy's day.. (cant help it - seen "Patty's Day" way too much this year!)17:33
bcwaldonKiall: I apologize for my American ignorance :)17:33
bcwaldonKiall: but how do you get Paddy from Patrick?17:33
KiallThat I can't explain! But patty would be even harder ;) St Patrick is a guy17:34
bcwaldonKiall: well Patty and Patrick both start with Pat, that's why that confusion exists17:34
bcwaldonKiall: anywho, thanks for correcting me :)17:34
KiallLol - As I said, I couldn't help myself!17:35
bcwaldonKiall: no worries17:35
KiallOn that note.. its time to head home.. cyas17:35
bcwaldonKiall: until next time...17:35
*** mdomsch has quit IRC17:37
*** dachary has joined #openstack-dev17:37
*** vizsla has joined #openstack-dev17:37
*** jakedahn_zz is now known as jakedahn17:38
*** bsza has quit IRC17:42
*** jakedahn is now known as jakedahn_zz17:43
*** vizsla has quit IRC17:46
*** rnirmal_ has joined #openstack-dev17:46
*** rnirmal_ has quit IRC17:47
*** rnirmal has quit IRC17:50
*** novas0x2a|laptop has joined #openstack-dev17:50
*** dtroyer is now known as dtroyer_zzz17:50
*** jakedahn_zz is now known as jakedahn17:50
*** mattray has quit IRC18:01
*** reidrac has quit IRC18:01
*** jakedahn is now known as jakedahn_zz18:02
*** mattray has joined #openstack-dev18:03
*** jakedahn_zz is now known as jakedahn18:06
*** johnpostlethwait has joined #openstack-dev18:11
*** bengrue has joined #openstack-dev18:12
*** n0ano has joined #openstack-dev18:14
rmkUgh, I forgot to register for the developer summit and now it's full.18:24
eglynn__mtaylor: I'm thinking the best way to package glance logcapture plugin would be to add it to openstack-nose as an additional plugin18:25
eglynn__mtaylor: after recasting that to be more of a collection of plugins as opposed to *the* openstack-nose plugin (singular)18:25
eglynn__mtaylor: so instead of just with-openstack, we'd have with-nova-test-report, with-glance-logcapture, with-keystone-something-else ...18:26
eglynn__mtaylor: there's a bit of work there tho' and the initial point of the exercise was to get closure on the mysterious S3-related failures we've been seeing on Jenkins18:26
eglynn__mtaylor: so in the interests of making headway on the test failures, I'd propose to temporarily switch over to run_tests.sh from nosetests18:26
eglynn__(with the understanding that this will be reverted once the S3 issue is isolated)18:27
*** relateable has joined #openstack-dev18:27
*** lts has quit IRC18:28
eglynn__mtaylor: i.e. this simple change to tox.ini: http://fpaste.org/S5is/18:28
*** rnirmal has joined #openstack-dev18:28
*** anotherjesse is now known as anotherjesse_zz18:30
mtayloreglynn__: ok. (you want to probably add -N to that)18:30
mtayloreglso that it doesn't try to make another venv18:30
mtaylorbut I'm ok with that for the short term as long as we have an exit strategry18:30
eglynn__mtaylor: cool, thanks!18:31
rmkIs there any way to still attend the developer summit?18:38
*** lts has joined #openstack-dev18:40
eglynn__jaypipes: the logcapture saga continues ...  https://review.openstack.org/552318:42
zaitcevrmk: Write to events@rackspace.com, ask. At some point there was a mix-up and I registered twice. What if they didn't expunge the bad registration?18:43
zaitcevrmk: Not necesserily my fault, but perhaps the total count is conservative.18:44
rmkcool -- I've give that a shot, thanks18:44
*** mnewby has joined #openstack-dev18:49
*** anotherjesse_zz is now known as anotherjesse18:49
*** Ryan_Lane has joined #openstack-dev18:55
*** jdg has joined #openstack-dev18:59
*** anotherjesse is now known as anotherjesse_zz19:03
*** anotherjesse_zz is now known as anotherjesse19:06
*** oneiroi has joined #openstack-dev19:07
*** dtroyer_zzz is now known as dtroyer19:07
*** oneiroi has quit IRC19:08
*** oneiroi has joined #openstack-dev19:08
*** dolphm has quit IRC19:09
*** dolphm has joined #openstack-dev19:12
*** dtroyer is now known as dtroyer_zzz19:14
*** adjohn has quit IRC19:17
anotherjesseheckj: https://review.openstack.org/#change,552519:18
*** lts has quit IRC19:20
*** zul has quit IRC19:21
*** aclark_ has joined #openstack-dev19:25
*** zul has joined #openstack-dev19:26
*** dtroyer_zzz is now known as dtroyer19:27
*** AlanClark has quit IRC19:28
*** anotherjesse is now known as anotherjesse_zz19:28
*** lts has joined #openstack-dev19:32
*** vincentricci has quit IRC19:36
*** rkukura has quit IRC19:38
jaypipeseglynn__, jeblair: do we have any idea what is going on with https://jenkins.openstack.org/job/gate-glance-python26/164/console19:39
eglynn__ jaypipes: just looking at it now ...19:40
*** dalang has joined #openstack-dev19:44
*** aclark_ has quit IRC19:45
*** asalkeld has joined #openstack-dev19:46
*** aclark_ has joined #openstack-dev19:47
*** anotherjesse_zz is now known as anotherjesse19:50
*** vincentricci has joined #openstack-dev19:51
*** anotherjesse is now known as anotherjesse_zz19:52
andrewbogottttx:  I'm hoping that there will be one or more sessions at the summit to discuss writing a nova plugin framework and/or establishing ways to isolate extension code from core code.  Is http://summit.openstack.org/sessions/view/21 that?19:52
*** adjohn has joined #openstack-dev19:53
ttxandrewbogott: yes19:53
andrewbogottCool.  55 minutes seems optimistic :)19:53
ttxandrewbogott: we can have follow-up sessions programmed on the spot19:54
andrewbogottAh, true.19:54
ttxbut if there is more room, we can definitely extend :)19:54
ttxThe Nova tracks are generally quite full.19:54
andrewbogottDo you know if anyone is planning to show up with an actual plugin-interface design in hand?  Or if that would be useful?  (Might be putting the cart before the horse.)19:55
ttxandrewbogott: i don't know. I won't have a design :)19:56
andrewbogottok19:56
ttxbut it can't hurt to prepare.19:56
Ashit's called CartHorse19:56
*** aclark__ has joined #openstack-dev19:57
andrewbogottI'll try to at least have a list of demands, if not a design :)19:57
*** aclark_ has quit IRC19:57
Ash<andrewbogott> one.. MILLION dollars19:57
*** markmc has joined #openstack-dev19:57
markmcheckj, dolphm, hey19:58
markmcheckj, dolphm, thanks for pushing ahead19:58
heckjmarkmc: arent you supposed to be holidaying ?19:58
dolphmmarkmc: np, hope you approve the giganto patch :)19:58
markmcheckj, yeah :)19:58
markmcdolphm, saw that19:58
markmccan we back off from the giant patch?19:58
* markmc thinks the history of individual commits is useful19:58
markmcit took a bit of work to keep them separate19:59
markmchappy to do whatever rebases to get it in now19:59
heckjmarkmc: I think dolphm just reapproved, but not sure of the status of that through jenkins19:59
markmcheckj, don't we need a devstack change too?19:59
* markmc looks at the devstack failure19:59
heckjmarkmc: didn't expect to need one for this20:00
markmcheckj, https://review.openstack.org/#change,528120:00
dolphmheckj: markmc: the first attempt only failed pep820:01
markmcdolphm, devstack too: https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/2727/20:01
dolphmmarkmc: ugh, totally missed that20:01
heckjthey need to be able to go in sequentially - submit both at once I'm afraid20:02
heckj(er, can't submit both at once)20:02
markmcyeah20:04
markmcwe had this with glance too20:04
dolphmdevstack needs to support both for a moment?20:04
heckjbasically, yeah20:05
markmchttps://bugs.launchpad.net/bugs/91759320:05
uvirtbotLaunchpad bug 917593 in openstack-ci "Co-ordinating glance/nova/keystone changes with devstack" [Critical,Won't fix]20:05
heckjintentionally set up so we walk the code forward as we go, making sure we dont introduce breaking changes20:06
markmceglynn__, there by any chance?20:07
eglynn__markmc: yep20:08
markmceglynn__, see above? need devstack to support both20:08
markmceglynn__, what did you do for glance again?20:08
bcwaldonmarkmc: how do I make a cli option required?20:08
bcwaldonmarkmc: with connon config stuff20:08
bcwaldonmarkmc: common*20:08
*** dolphm has quit IRC20:09
markmcbcwaldon, at the moment, just check if it's set, raise an exception if not20:09
markmcbcwaldon, we could add required=True flag or something, though20:09
eglynn__markmc: the chicken and egg issue when I reworked the paste config?20:09
markmceglynn__, yeah20:09
bcwaldonmarkmc: I would consider that a necessary feature20:09
markmcbcwaldon, does optparse have it?20:09
bcwaldonmarkmc: argparse does20:10
markmcbcwaldon, feel free to file against openstack-common20:10
eglynn__markmc: in that case we just copied the paste config in the *.conf as well as putting in the separate *-paste.ini20:10
markmcbcwaldon, right, I know argparse has it20:10
bcwaldonmarkmc: argparse is replacing optparse, anywho20:10
markmcbcwaldon, yep20:10
bcwaldonmarkmc: reason we didnt go with that?20:10
markmcbcwaldon, yes, can't remember offhand though20:10
eglynn__markmc: sorrta a special case I guess, just duplicating the paste config was sufficient to work around20:10
markmcbcwaldon, looked into it, pretty confident we can switch to it later20:10
markmcbcwaldon, think it was just I didn't want to switch both nova and glance to something new and glance was using optparse20:11
eglynn__markmc: whereas your keystone cfg changes are more intrusive, right?20:11
bcwaldonmarkmc: ok20:11
*** anotherjesse_zz is now known as anotherjesse20:11
markmceglynn__, I made identical changes to glance before your stuff, not sure how that was pushed through20:11
eglynn__markmc: was devstack gating merges at that time?20:11
markmceglynn__, yeah20:12
markmceglynn__, I'll find it, one sec20:12
*** troytoman is now known as troytoman-away20:12
*** danwent has joined #openstack-dev20:13
heckjmarkmc: since that big one failed, I'm fine with going back to the single commits and doing them individually.20:13
markmcheckj, ok, pushed another rebase20:13
markmcheckj, should be able to get them all in except the last two without the devstack change20:14
* markmc pokes at devstack20:14
markmcah, we weren't gating on devstack when we did this for glance20:15
*** novas0x2a|laptop has quit IRC20:16
*** novas0x2a|laptop has joined #openstack-dev20:16
* heckj nods20:17
markmcglance devstack change was https://review.openstack.org/234320:17
eglynn__markmc: a-ha20:17
eglynn__markmc: resurrect your idea on pairing the devstack patch with the keystone/glance/nova change that would break the old devstack?20:19
markmceglynn__, think we want to move quicker than that20:19
markmcsomething like20:19
markmcin glance checkout20:19
markmcif ! git rev-parse 35372906d >/dev/null 2>&!; do sed -i .... etc/keystone.conf; done20:20
markmcand then after we've merged the keystone changes20:20
markmcmerge your proper config file update?20:20
eglynn__markmc: you meant 'in keystone checkout' or?20:21
eglynn__(as opposed to glance)20:22
markmcyep, sorry20:22
*** anotherjesse is now known as anotherjesse_zz20:22
*** anotherjesse_zz is now known as anotherjesse20:23
eglynn__markmc can you explain: if ! git rev-parse 35372906d ...20:24
markmceglynn__, should check whether the commit 35372906d exists in the repo20:24
eglynn__markmc: gotcha, nice20:24
markmceglynn__, really we want a clever way of saying "is 35372906d an ancestor of HEAD"20:24
markmceglynn__, but I can't think of something nice offhand20:24
*** anotherjesse is now known as anotherjesse_zz20:24
eglynn__markmc: so devstack uses a pre-canned version of the keystone.conf, so that rev-parse could go into stack.sh, and drive selection between a new or old config file template20:26
eglynn__(i.e. devstack clones keystone, then copies over a keystone.conf it prepared earlier ...)20:27
markmcright20:28
eglynn__should work20:28
eglynn__should run the idea by some devstack core folks, dtroyer say ...20:28
markmcactually, can't do a sed20:29
markmcit would need to be something like cp files/keystone.conf.new files/keystone.conf20:29
eglynn__yep20:29
markmcdtroyer, there?20:30
dtroyermarkmc: yo, catching up20:30
markmcdtroyer, basically just need to switch to a new keystone conf file20:30
markmcdtroyer, but the new one doesn't work with current keystone20:30
eglynn__the keystone.conf.new could just be pulled from: https://review.openstack.org/#change,528120:31
dtroyermarkmc: ok.  how do we know which one to use?20:31
markmcdtroyer,  if ! git rev-parse 35372906d >/dev/null 2>&!; then ...20:31
*** dolphm has joined #openstack-dev20:31
*** zigo has quit IRC20:31
*** bengrue has quit IRC20:32
markmcdtroyer, well,  if git rev-parse 35372906d >/dev/null 2>&!; then use_new_conf; fi20:32
*** dprince has quit IRC20:34
*** Vek has quit IRC20:35
markmcdtroyer, simpler, if grep keystone.common.wsgi etc/keystone.conf >/dev/null; then use_new_conf; fi20:35
markmcdtroyer, i.e. check whether the keystone checkout has the new config20:36
jeblairmarkmc: there's another pattern for handling this kind of change that i think is pretty elegant20:36
jeblairmarkmc: check out https://github.com/openstack-dev/devstack/blob/master/stack.sh#L88120:36
jeblairwhere the nova api-paste.ini file from _nova_ is used as the template20:37
jeblairrather than one that lives with devstack.20:37
markmcabsolutely, would much rather that20:37
dtroyerok, I think that will work.  we'll want to carry the new file inside devstack rather than pull it from gerrit just to keep the external dependencies down20:37
dtroyerjeblair, markmc: I actually have a bug on doing exactly that20:38
dtroyerI think it's within reach for keystone20:38
jeblairdtroyer: cool.  i think that will reduce the number of times we run into this in the long run.20:38
jeblaironly _really fundamental_ changes would need switching logic in devstack20:38
jeblairyou guys seem on it.  i'll leave you to it; just wanted to make sure that wasn't overlooked.20:39
dtroyerI'd love to do it for all projects, but following the changes to glance recently shows me that some of them are just not going to be possible because of the variations in pipeline configs, etc.20:39
markmccurrent diff between devstack and keystone confs20:40
markmchttp://paste.openstack.org/show/11137/20:40
markmcadmin_token, compute_port, sql.connection, sql.idle_timeout, catalog.template_file20:42
markmcthat looks to be the only important changes20:42
justinsb_You could use this script to set config values cleanly: https://github.com/justinsb/openstack-simple-config/blob/master/utils/openstack-config-set20:42
justinsb_I borrowed the idea from Fedora's packaging20:43
markmcjustinsb_, yeah, we should really have that upstream somewhere20:44
justinsb_markmc: I'm hoping to create a project / area within Openstack that will make packaging easier20:44
markmcjustinsb_, ok, cool20:44
justinsb_markmc: http://summit.openstack.org/sessions/view/2620:45
markmcjustinsb_, heh, user config in paste config20:46
markmcjustinsb_, definite bugbear of mine too :)20:46
markmcheckj, there?20:48
markmchmm, lunch time on west coast20:49
* markmc assumes heckj has gone for a surf or something20:49
markmcthat's what you guys all do, right? :)20:49
*** anotherjesse_zz is now known as anotherjesse20:50
*** anotherjesse is now known as anotherjesse_zz20:51
justinsb_markmc: Yes, but we normally take our phones with us.20:52
heckjmarkmc: sorry, here - in a meeting20:52
markmcjustinsb_, we're not so considerate when we go for our lunchtime 4 pints of guinness :)20:53
markmcheckj, np20:53
heckj4 pints of guiness? Sounds damn good right now20:53
russellb+120:53
markmcheckj, was just going to say, you can approve https://review.openstack.org/4557 and everything below it20:54
markmcheckj, then we can figure out something about devstack20:54
heckjon it20:54
*** dneary has joined #openstack-dev20:54
ttxvishy: yo20:54
markmcheckj, cool20:55
heckjmarkmc: take a look at https://review.openstack.org/#q,status:open+keystone,n,z - is that the right set20:56
heckjmarkmc: do I need to hit the others in that big set? I just followed "needed by" and it was a short list...20:57
markmcheckj, this shouldn't be in it: https://review.openstack.org/455820:57
*** dolphm has quit IRC20:57
*** rkukura has joined #openstack-dev20:57
markmcheckj, nor should this: https://review.openstack.org/455920:57
markmcheckj, everything in https://review.openstack.org/#q,status:open+project:openstack/keystone+owner:markmc,n,z except those two20:58
heckjpulled 4558, 455920:58
markmcok, I think I have a solution21:02
markmcI'll add the old factory methods back to https://review.openstack.org/455821:02
markmcso it will support the old and new config21:02
heckjmarkmc: Ok - take a look now - should be set.21:03
markmcand remove them again in https://review.openstack.org/455921:03
markmcwe approve 455821:03
markmcget the new config into devstack21:03
*** zzed has quit IRC21:03
markmcthen approve 455921:03
*** Gordonz has quit IRC21:03
heckjmarkmc: sounds like a lovely waltz21:03
*** aclark__ has quit IRC21:03
markmcheckj, looks good21:03
*** zzed has joined #openstack-dev21:04
notmynamettx: swift 1.4.8 is at commit b8e1513ad7de8b88c95986e20b9a03d5b8207fd621:06
ttxnotmyname: ok, will cut milestone-proposed in my tomorrow morning21:07
notmynamettx: thanks. I'll try to get a go-nogo by wed pm for a release on thurs21:07
ttxnotmyname: sounds good.21:08
eglynn__markmc: yep, thats way neater21:09
markmcheckj, ok, pushed new versions of 4558 and 455921:09
markmcheckj, I guess it's best to just wait until all the others are merged before doing the dance21:09
heckjmarkms: yeah, sounds look a good plan21:09
markmcheckj, https://review.openstack.org/5281 is the devstack commit21:09
*** zzed has quit IRC21:18
*** lts has quit IRC21:19
*** dolphm has joined #openstack-dev21:19
*** jdg has quit IRC21:24
*** mancdaz1203 has quit IRC21:28
*** mancdaz1203 has joined #openstack-dev21:30
*** johnpostlethwait has quit IRC21:30
*** johnpostlethwait has joined #openstack-dev21:31
eglynn__jaypipes: found the issue that was causing https://review.openstack.org/5523 to bomb out on 2.6, looks like setUp() wasn't called reliably for skipped tests21:36
eglynn__fixed in patch set 221:36
*** rnirmal has quit IRC21:44
*** vizsla has joined #openstack-dev21:46
*** utlemming has quit IRC21:48
*** adjohn has quit IRC21:50
*** vizsla has quit IRC21:50
*** utlemming has joined #openstack-dev21:52
vishycomstud: can you check this for any issues: https://review.openstack.org/#change,5518 ?21:52
*** rgoodwin is now known as rgoodwin_away21:54
*** rgoodwin_away is now known as rgoodwin21:54
*** apevec has quit IRC21:56
vishypvo, comstud: I think we are shipping rc-1 tonight unless one of you guys has a reason that we need to delay...21:58
*** dolphm has quit IRC22:00
*** adjohn has joined #openstack-dev22:00
*** rgoodwin is now known as rgoodwin_away22:02
*** rgoodwin_away is now known as rgoodwin22:04
justinsb_vishy: You're talking about nova, right?  I looked through my bugs.  I'm wondering if bug #939386 could be a sign of bigger escaping problems in XML.  I'm checking it out.22:08
uvirtbotLaunchpad bug 939386 in nova "Console-Output does not escape XML correctly" [Medium,Confirmed] https://launchpad.net/bugs/93938622:08
*** Mandell has joined #openstack-dev22:09
*** crobinso has quit IRC22:09
*** kbringard has quit IRC22:10
vishyjustinsb_: are you going to submit a patch for that?22:11
justinsb_vishy: I'm evaluating it now.  If it's just console-output, I don't think it's too serious.  If it's all XML output, I'll report back.22:12
vishyjustinsb_: thanks22:13
*** dolphm has joined #openstack-dev22:17
*** ywu has joined #openstack-dev22:19
rmkRunning the Glance RC package with precise -- on install it attempts to do a db_sync and throws an exception:22:29
rmkTypeError: Invalid argument(s) for Table: ['extend_existing', 'mysql_engine']22:29
rmkGranted, the default config is sqlite22:29
rmkSo that looks like a possible bug?22:29
*** Mandell has quit IRC22:32
comstudvishy: We had a patch coming for fixing an issue searching by IPs in nova compute API... not sure if that's made it up for review yet22:33
vishycomstud: haven't seen it22:33
*** dolphm has quit IRC22:33
comstudvishy: checking22:33
vishyrmk: old version of sqlalchemy22:33
vishyrmk: needs useexisting22:34
vishyrmk: I guess the patch to glance to switch it should be reverted22:34
rmkvishy: Thanks -- this was on 11.1022:34
vishybcwaldon, jaypipes: ^^22:34
vishyrmk: ah that makes sense, yes oneiric that would happen22:34
rmkI'd rebuilt all the packages from Precise for it.  I can upgrade sqlalchemy but yeah it'll break oneiric people.22:34
*** markmc has quit IRC22:34
rmkThough hopefully most are looking to go to 12.04 now that it's an LTS.22:35
bcwaldonfile a bug for us :)22:35
*** dolphm has joined #openstack-dev22:35
rmkSure, though is it a bug or do you want to require a newer sqlalchemy?22:35
bcwaldoncan you file a bug with what you see as broken, then we'll be able to discuss the appropriate fix when everyone understands22:36
rmkcool will do22:36
bcwaldonI havent been following your convo with vishy, so I don't really know the problem yet22:36
*** Vek has joined #openstack-dev22:36
*** Mandell has joined #openstack-dev22:36
bcwaldonrmk: thanks!22:37
comstudvishy: the search-by-IP bug is only a bug when using quantum/melange..  i guess it'll be fixed with a larger re-work22:39
vishybcwaldon: it is the issue with sqlalchemy versions useexisting vs extend_existing22:39
comstudvishy: i doubt anyone really uses the search-by-ip function anyway, so I think it's okay to live with that bug being in essex22:39
vishybcwaldon: glance was updated to the new non-deprecated version which breaks with old versions of sqlalchemy22:39
vishycomstud: ok22:39
bcwaldonvishy: I don't have any previous context related to that...22:39
bcwaldonvishy: ok, sqlalchemy bug22:39
comstudso, i think as far as we're concerned, we're good to go22:39
bcwaldonvishy: great...22:39
*** dachary has quit IRC22:42
vishyjaypipes: in regards to your ratelimit bug22:42
vishyjaypipes: I put a bug on nova to have a way to disable rate limiting when I broke that flag in devstack22:43
vishyjaypipes: but I was told that it isn't vital for tempest because there is a workaround, so I untargeted it22:43
jaypipesvishy: hmm :)22:43
*** Mandell has quit IRC22:44
*** mattray has quit IRC22:44
jaypipesvishy: well, the "workaround" is to let the tempest client sleep for the number of seconds needed, which makes tempest run in about an hour. ;)22:44
vishyjaypipes: we could do a hacky simple version and just add a new pipeline22:44
jaypipesvishy: running out to dinner with the wifey... we'll chat later about it? :)22:44
vishysure22:45
*** Mandell has joined #openstack-dev22:53
justinsb_vishy: It's a xml / minidom bug I think: http://paste.openstack.org/show/11168/22:54
justinsb_vishy: Some characters escaped, some not escaped22:54
justinsb_vishy: Trying to figure out if it's minidom or our usage that's to blame22:54
*** dtroyer is now known as dtroyer_zzz22:56
vishyjustinsb_: we shouldn't be using xml.dom.minidom22:59
vishyjustinsb_: it should be lxml only22:59
vishythere is a patch out to convert most of it22:59
*** dolphm has quit IRC23:00
*** Mandell has quit IRC23:01
*** Mandell has joined #openstack-dev23:01
justinsb_vishy: Do you have a link on that patch?  wsgi.py still using minidom...23:02
*** dolphm has joined #openstack-dev23:03
rmkbcwaldon: https://bugs.launchpad.net/glance/+bug/95978723:03
uvirtbotLaunchpad bug 959787 in glance "Glance database creation fails using RC1 on Ubuntu 11.10" [Undecided,New]23:03
*** Mandell has quit IRC23:06
*** andrewsmedina has quit IRC23:06
vishyjustinsb_: https://review.openstack.org/#change,542223:07
vishyjustinsb_: i think it missed the minidom usages though23:08
justinsb_vishy: Yes - I don't see nova/api/openstack/wsgi.py in that list23:08
justinsb_vishy: I think this is the bug http://bugs.python.org/issue575223:09
justinsb_vishy: I don't think it's particularly serious; I think the XML produced is just invalid; I don't think it's a vulnerability or anything like that.23:10
justinsb_vishy: But it does mean we're potentially producing invalid XML anywhere we're outputing a string23:11
justinsb_vishy: In this case the problematic character is 0x1B, the escape character23:11
justinsb_vishy: I don't think tabs and newlines create invalid documents, just documents that don't exactly reflect the input23:11
justinsb_vishy: I think how you want to fix this is going to be your call23:12
justinsb_vishy: If you want to fix it at all!23:12
vishyconsole output should strip ctrl characters23:12
justinsb_vishy: OK ... I can implement that pretty easily.23:12
justinsb_vishy: We'll just have to remember that this isn't really fixed, just hidden!23:13
vishyjaypipes: when you get back: does this work: https://review.openstack.org/5543 ?23:13
*** thingee has quit IRC23:16
*** yoshiotu has joined #openstack-dev23:25
*** dachary has joined #openstack-dev23:26
*** yoshiotu has left #openstack-dev23:29
*** Kiall has quit IRC23:32
*** Gordonz has joined #openstack-dev23:32
*** Kiall has joined #openstack-dev23:33
justinsb_vishy: Stripping tricky characters out of console output https://review.openstack.org/#change,554523:34
justinsb_vishy: I don't really mind if that makes it in or not.  I can just use the JSON output.23:34
*** phorgh has quit IRC23:34
*** andrewsmedina has joined #openstack-dev23:37
*** test400 has joined #openstack-dev23:41
*** Gordonz has quit IRC23:41
*** vincentricci has quit IRC23:47
*** adjohn has quit IRC23:47
*** adjohn has joined #openstack-dev23:50
*** vricci has joined #openstack-dev23:50
pvobcwaldon: vishy: no huge blockers here.23:54
pvopull the chute!23:54
*** novas0x2a|laptop has quit IRC23:56
*** adjohn has quit IRC23:58

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