Thursday, 2011-09-15

mtaylorjeblair: on hold/abandoned for now00:00
mtaylorjeblair: instead of splitting into series branches we decided to block/wait on an upstream backporting bug00:00
mtaylorjeblair: bug 78852400:00
uvirtbotLaunchpad bug 788524 in python-defaults "backport dh_python2 to lucid (and maverick if appropriate)" [High,In progress] https://launchpad.net/bugs/78852400:00
jeblairok.  maybe abandon 351, or vote -2 on it?00:00
mtayloryes00:00
mtaylordone00:01
openstackgerritA change was merged to openstack/openstack-ci: Added support for diablo/trunk packaging split.  https://review.openstack.org/44700:02
tr3buchethas anyone done an rpc call with no metho args?00:09
tr3buchetlooks like it's required00:11
tr3buchetinteresting00:11
tr3buchetoops, was wrong. not required00:13
*** nati_ has joined #openstack-dev00:30
*** nati has quit IRC00:33
openstackgerritA change was merged to openstack/glance: Fix keystone paste config for functional tests  https://review.openstack.org/44300:37
*** bengrue has quit IRC00:40
*** novas0x2a|laptop has quit IRC00:41
*** Tushar has quit IRC00:51
*** nati_ has quit IRC01:06
*** jakedahn has quit IRC01:06
*** novas0x2a|laptop has joined #openstack-dev01:12
*** jdurgin has quit IRC01:18
HugoKuo_morning01:43
*** Mussi has joined #openstack-dev01:55
*** novas0x2a|laptop has quit IRC01:57
*** nati has joined #openstack-dev02:05
*** ecarlin has joined #openstack-dev02:15
*** nati has quit IRC02:19
*** nati has joined #openstack-dev02:19
*** nati has quit IRC02:36
HugoKuo_anyone around ?02:56
dolphmsort of02:56
dolphmti's 10pm here :)02:56
dolphmit is*02:56
*** martine has joined #openstack-dev03:02
*** Mussi has quit IRC03:09
HugoKuo_it's 11 AM here03:20
*** nati has joined #openstack-dev03:36
*** jeremyb has quit IRC03:47
*** jeremyb has joined #openstack-dev03:47
*** HugoKuo has joined #openstack-dev03:54
*** HugoKuo_ has quit IRC03:58
*** ecarlin has quit IRC04:08
*** dolphm has quit IRC05:09
*** jaypipes has quit IRC05:34
*** martine has quit IRC05:35
*** nati_ has joined #openstack-dev05:38
*** nati has quit IRC05:41
*** novas0x2a|laptop has joined #openstack-dev05:47
*** jdag has quit IRC06:12
*** nati_ has quit IRC06:13
*** jdag has joined #openstack-dev06:13
*** zaitcev has quit IRC06:44
vishytpatil: sure.: if you could get someone to merge that ubuntu package change that would be awesome06:56
vishymonty: ??06:56
*** reidrac has joined #openstack-dev07:02
*** mnour has joined #openstack-dev07:05
*** AhmedSoliman has joined #openstack-dev07:51
*** Vek has quit IRC08:04
*** darraghb has joined #openstack-dev08:51
*** bourke_ is now known as bourke09:18
*** bourke has quit IRC09:22
*** bourke has joined #openstack-dev09:23
ttxvishy: you want that change in Diablo ?09:25
ttxfor bug 782364: nova branch and packaging branch need to fix, commented on both merge props09:28
uvirtbotLaunchpad bug 782364 in nova "better to send gratuitous ARPs to support HA" [Wishlist,Triaged] https://launchpad.net/bugs/78236409:28
vishyi could go either way.  It is really important to make floating ips work properly09:32
ttxvishy: this ARP thing is related to floating IPs ?09:50
*** tryggvil_ has joined #openstack-dev09:53
*** Vek has joined #openstack-dev09:59
vishyttx: yes, if you don't gratuitous arp, you have to wait for the switch arp cache to clear10:00
vishywhich can take some time depending on settings10:01
ttxvishy: just concerned about the new dep, which might take some downstreams by surprise.10:01
vishyyeah, hmm10:02
vishyit defaults to false10:02
ttxvishy: is this optional behavior, or does it affect everyone ?10:02
vishyso it is only needed if the flag is true10:02
ttxthen I think it's ok10:02
vishy--send_arp_for_ha10:02
ttxit won't break those who haven't planned for it10:02
ttxstill needs a fix though, uses "sudo" directly :)10:04
*** Vek has quit IRC10:12
vishyagreed10:13
*** AhmedSoliman has quit IRC10:13
*** villep has joined #openstack-dev10:23
*** jeffjapan has quit IRC10:52
*** lts has joined #openstack-dev11:11
smoserttx, around ?11:58
smoseri need to bother someone on bug 82759012:00
uvirtbotLaunchpad bug 827590 in nova "cloud-init does not mount ephemeral0 on /mnt in nova" [Medium,Fix committed] https://launchpad.net/bugs/82759012:00
smoserand request that we get that into diablo12:00
*** markmc has joined #openstack-dev12:01
ttxsmoser: I am12:03
* ttx looks12:04
smoserttx, that branch that landed improved several things. metadata service *and* ephemeral devices now have a filesystem.12:05
ttxyes, the only drawback I see is the DB version bump12:05
smoseryes.12:06
ttxsmoser: any chance you could backport the fix to milestone-proposed and propose for merging there ?12:06
smoserwhich is the only drawback for us to carry it12:06
ttxsmoser: I can do it, but in a couple hours12:06
smoseri can test milestone-proposed, yes.12:06
smoseri will do that.12:07
ttxi'll target the bug12:07
ttxvishy: agree the bugfix is worth the DB bump ? ^12:08
*** Vek has joined #openstack-dev12:09
smosercan i also have someone look at/consider bug 837100 and bug 83710212:09
uvirtbotLaunchpad bug 837100 in nova "nova does not uncompress qcow compressed images" [Wishlist,Confirmed] https://launchpad.net/bugs/83710012:09
uvirtbotLaunchpad bug 837102 in nova "nova writes libvirt xml 'driver_type' based only on FLAGS.use_cow_images" [Low,Triaged] https://launchpad.net/bugs/83710212:09
smoseri believe that bug 837102 means that if the end user uploads a qcow disk, but does not have qcow enabled, it will not work.12:09
smoser(app armor will block it)12:09
smoserbug 837100 means if someone uploads an ubuntu image without qcow-converting it first, all reads will go through decompression12:10
uvirtbotLaunchpad bug 837100 in nova "nova does not uncompress qcow compressed images" [Wishlist,Confirmed] https://launchpad.net/bugs/83710012:10
*** mfer has joined #openstack-dev12:22
*** markmc has quit IRC12:25
*** markmc has joined #openstack-dev12:31
*** lorin1 has joined #openstack-dev12:39
ttxpvo: progress on bug 833552 ?12:44
uvirtbotLaunchpad bug 833552 in nova "No easy migration from old auth to keystone" [Critical,In progress] https://launchpad.net/bugs/83355212:44
ttxI'd like to see it enter the review tube asap12:44
*** AhmedSoliman has joined #openstack-dev12:46
ttxsoren: interested in fixing bug 845866 ? getting correct exceptions sounds like a nice to have before release12:47
uvirtbotLaunchpad bug 845866 in nova "exception tracebacks are lost when using raise" [Low,Triaged] https://launchpad.net/bugs/84586612:47
*** mfer has quit IRC12:48
*** mfer has joined #openstack-dev12:50
*** bsza has joined #openstack-dev12:52
sorenttx: Erk.12:53
sorenttx: I'm trying to address some security group bugs right now. Perhaps once I'm done.12:54
ttxok12:54
sorenIn related news, I'm working up some motivation to speed up our test suite. Sheesh!12:55
*** martine has joined #openstack-dev13:03
*** joesavak has joined #openstack-dev13:15
*** kbringard has joined #openstack-dev13:21
*** dprince has joined #openstack-dev13:30
*** zul has quit IRC13:30
*** zul has joined #openstack-dev13:37
vishysmoser, ttx: yes i'm good with it13:37
openstackjenkinsProject nova build #1,373: SUCCESS in 3 min 47 sec: https://jenkins.openstack.org/job/nova/1373/13:40
openstackjenkinsTarmac: The 1.1 API specifies that two vendor content types are allowed in addition to the standard JSON and XML content types.13:40
openstackjenkinsThis branch adds support for application/vnd.openstack.compute+json and application/vnd.openstack.compute+xml.13:40
vishysmoser: those other two bugs would be good fixes, but I don't have the bandwidth to handle them right now13:43
vishyttx: the wrong branch was linked to pvo's bug13:45
vishyi fixed13:45
smoservishy, i'll work on them.13:46
mtaylorvishy, ttx: morning13:46
smoserbiggest blocker right now is getting code to determine "is the qcow image compressed"13:46
mtaylorttx: I got the change to ppa_script.sh pushed, need to go fix a jenkins job then we should be good to go13:46
smoserthat is not a simple boolean, each cluster can be. although i'd be happy to say "if any clusters are compressed, decompress it"13:47
ttxmtaylor: cool!13:48
vishysmoser: can't you just do a qemu-img convert always13:49
vishyif it is compressed it will uncompress and if it is not-compressed it will just end up copying it?13:49
smoservishy, i can do that, yes.13:50
smoserbut that just seems wasteful. you'd be ok with it?13:50
smoseri'd just do it when it pulls down to the node13:50
smosera large uncompressed image could be 5G (or anything) and then we'd be doing extra IO on all of that.13:51
*** jaypipes has joined #openstack-dev14:02
smoseri found 3 pep8 errors on milestone branch using oneiric pep814:09
smoseris that known?14:09
mtaylorsmoser: pep8 has been slipping in and out a little bit recently - should get sorted when we do the gerrit transition and have a proper job dealing with pep814:10
smoserthanks.14:10
mtaylorttx: jenkins jobs changed - diablo packaging branches pushed for nova/swift/glance14:15
dprincemtaylor: does that mean we can merge glance scrubber packaging fixes?14:16
mtaylordprince: those are the ones that are for essex and not for diablo, right?14:16
mtaylordprince: then yes14:17
ttxyes14:17
dprincemtaylor: lp:~dan-prince/glance/ubuntu-glance-scrubber14:18
*** nati has joined #openstack-dev14:19
*** cp16net has joined #openstack-dev14:23
mtaylordprince: done14:24
dprincemtaylor: thanks dude14:24
*** rohitk has joined #openstack-dev14:29
*** joesavak has quit IRC14:31
rohitkHi Soren, the openstack-integration-tests is quite useful, from what I have checked it's independant of the OS API version right?14:33
*** joesavak has joined #openstack-dev14:33
*** nati has quit IRC14:37
*** nati has joined #openstack-dev14:37
*** code_franco has joined #openstack-dev14:42
*** mnour has quit IRC14:43
ttxmtaylor: do we need to retrigger PPA jobs ?14:49
mtaylorttx: yeah.14:50
ttxmtaylor: I can do that.14:51
mtaylorttx: which reminds me ... I should really make those jobs trigger on packaging AND on tarballs ...14:51
*** dolphm has joined #openstack-dev14:57
jk0.c14:57
dprince.rb14:59
mtaylor.asdf15:04
*** rnirmal has joined #openstack-dev15:05
*** amccabe has joined #openstack-dev15:06
*** bcwaldon has joined #openstack-dev15:11
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Added a new packaging script which uses bzr bd.  https://review.openstack.org/44815:15
*** novas0x2a|lapto1 has joined #openstack-dev15:16
jk0FYI, trunk depends on feedparser now (found this out by running unit tests)15:16
mtaylorjeblair, ttx: if you get a chance, look over the above ^^^ it's a stab at making a new version of ppa_script.sh which relies a little more on jenkins to do a few work tasks, and then uses bzr builddeb to run the actually building15:16
*** novas0x2a|laptop has quit IRC15:16
ttxmtaylor: upload rejected, due to different tarball with same name -- I guess we'll have to wait for the next change15:16
mtaylorttx: well poo. hrm. that shouldn't happen... will investigate in a bit15:17
ttxmtaylor: or... I could make a new tarball -- the date would be incremented.15:17
mtaylorttx: try that ... although I'm a little unhappy that something in our pipeline is re-making a tarball incorrectly15:18
*** cp16net has quit IRC15:18
*** cp16net has joined #openstack-dev15:18
*** nati has quit IRC15:19
*** rnorwood has joined #openstack-dev15:22
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Added a new packaging script which uses bzr bd.  https://review.openstack.org/44815:24
villepIs there going to be ACLs for objects/containers in Keystone ? ie. in swift one user in account could access the container and/or some file in it and some other account might not access, due rules from acl ?15:32
*** novas0x2a|lapto1 has quit IRC15:33
*** ameade has joined #openstack-dev15:35
*** markmc has quit IRC15:36
jaypipesglenc: heyo, feel free to call me after 11am CDT to chat about the protected-properties BP. can conf in Vek too if you want.15:37
Veknote I normally grab lunch around 11:3015:37
jaypipesrohitk: I'd advise emailing soren that question... it's his first day on the job at his new job I believe... I'll PM you his email address.15:38
rohitkthx jay15:39
rohitksoren has been kind enough to catch me on private chat15:39
rohitkjk0: i hit the feedparser errors and installed the package, also noticed lxml was needed15:41
*** heckj has joined #openstack-dev15:42
openstackgerritBrian Lamar proposed a change to openstack/glance: Fixes bug 851016.  https://review.openstack.org/44915:48
uvirtbotLaunchpad bug 851016 in glance "test_migrations.py isn't working properly" [Medium,Triaged] https://launchpad.net/bugs/85101615:48
blamarjaypipes: ... you're waaaay too quick sometimes15:50
jaypipesblamar: heh, I try :)15:50
*** markmc has joined #openstack-dev15:51
ttxmtaylor: problem solved for maverick/natty, still failing for oneiric/lucid. Updated https://bugs.launchpad.net/openstack-ci/+bug/84897315:52
uvirtbotLaunchpad bug 848973 in openstack-ci "Nova fails to build in milestone-proposed PPA" [Critical,In progress]15:52
blamarjaypipes: btw that migrations tests is awesome if you did that. walking up and down the migrations + the ability to test locally with mysql easily is amazing15:52
jaypipesblamar: yes, that was me :) lots of things flying around my desk in frustration as I coded that. ;)15:52
blamarhaha I bet15:53
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Fix logic problem with waiting for puppet.  https://review.openstack.org/45015:59
dolphmvillep: it's been discussed, and the current solution is to allow keystone consumers to "implement" ACL's using Keystone Roles & Tenants (if they really need ACL's today)... alternatively (and preferably, IMO), ACL's could be provided by a Keystone extension16:00
openstackgerritBrian Waldon proposed a change to openstack/glance: Adding git-specific Authors file test  https://review.openstack.org/45116:00
openstackgerritBrian Waldon proposed a change to openstack/glance: Adding git-specific Authors file test  https://review.openstack.org/45116:01
dolphmmtaylor: just dreaming, but would it be possible to link gerrit reviews together, so that one will not be merged until both can be merged? (into seperate repos... e.g. to sync changes between identity-api & keystone)16:05
*** reidrac has quit IRC16:10
*** nati has joined #openstack-dev16:11
*** markmc has quit IRC16:11
glencjaypipes: let me know when you have a moment; I'd like to discuss the system/protected properties BP with you. Ok, actually, I want to bludgeon you over the head until you see it my way, but let's be polite about it, ok?16:12
jaypipesglenc: bludgeon away. :)16:13
*** bcwaldon has quit IRC16:13
*** mattray has joined #openstack-dev16:17
villepdolphm: ok, sounds good16:19
villepdolphm: how would you do it, wsgi between keystone and proxy or inside in keystone?16:20
jeblairdolphm: i don't beleive gerrit has such a feature.  we could probably abuse jenkins for that purpose, but it would be complicated.  in the mean time, you could make sure that both changes have the same topic (by making them in the same branch name, or better yet -- flagging them as being for the same blueprint).  you can also make direct links to changes by mentioning the change id (or at least the first 9 characters) in commit messages16:20
*** bcwaldon has joined #openstack-dev16:21
*** jsavak has joined #openstack-dev16:21
openstackgerritBrian Waldon proposed a change to openstack/glance: Adding git-specific Authors file test  https://review.openstack.org/45116:23
*** AhmedSoliman has quit IRC16:24
*** joesavak has quit IRC16:25
*** markmc has joined #openstack-dev16:32
*** rohitk has quit IRC16:34
openstackgerritVerification of a change to openstack/glance failed: Fixes bug 851016.  https://review.openstack.org/44916:34
uvirtbotLaunchpad bug 851016 in glance "test_migrations.py isn't working properly" [Medium,Triaged] https://launchpad.net/bugs/85101616:34
Davieyhow cute, uvirtbot responds to openstackgerrit's bug number references.16:34
Davieyvishy: nova/test.py breaks the unittest feature of skipping class of tests.. this seems to be an issue.16:36
Daviey(unittest supports this feature)16:36
*** tryggvil_ has quit IRC16:39
*** jdurgin has joined #openstack-dev16:41
*** ohnoimdead has joined #openstack-dev16:54
openstackgerritA change was merged to openstack/openstack-ci: Fix logic problem with waiting for puppet.  https://review.openstack.org/45016:54
*** novas0x2a|laptop has joined #openstack-dev17:04
*** bengrue has joined #openstack-dev17:17
*** mszilagyi has joined #openstack-dev17:18
*** mfer has quit IRC17:29
dolphmvillep: either way would work, although to do it "inside keystone," we're interested in creating an extension-loading framework so you can tie into keystone directly from a discrete codebase... as of today you'd basically be contributing to the core keystone codebase17:33
*** darraghb has quit IRC17:33
*** mdomsch has joined #openstack-dev17:42
*** lorin1 has quit IRC17:43
mtaylorttx: awesome17:44
mtaylorttx: I think we need to fix the pep8 errors that oneiric pep8 is uncovering ... and perhaps I should backport oneiric pep8 into our ppas for the other serieses17:46
mtaylorttx: I've got a lucid backport of python-unittest2, I can get that copied in to the release ppa17:46
mtaylorttx: the python-all-dev issue in bug 848971 on the other hand, there are not a bunch of great options for that everyone likes at the moment ... but I will come up with something17:48
uvirtbotLaunchpad bug 848971 in openstack-ci "Swift fails to build on maverick/lucid in trunk / milestone-proposed / release PPAs" [High,New] https://launchpad.net/bugs/84897117:48
openstackgerritAlex Meade proposed a change to openstack/glance: Added min_disk and min_ram properties to images  https://review.openstack.org/45217:53
openstackgerritAlex Meade proposed a change to openstack/glance: Added min_disk and min_ram properties to images  https://review.openstack.org/42917:55
*** lorin1 has joined #openstack-dev17:55
*** patelna has joined #openstack-dev17:56
*** dolphm has quit IRC17:56
*** lorin1 has quit IRC17:58
*** lorin1 has joined #openstack-dev18:00
*** harlowja has joined #openstack-dev18:08
*** mfer has joined #openstack-dev18:13
openstackgerritDolph Mathews proposed a change to openstack/keystone: Backend-managed role ID's (bug #834683)  https://review.openstack.org/43118:30
uvirtbotLaunchpad bug 834683 in keystone "RoleID and RoleName" [Undecided,In progress] https://launchpad.net/bugs/83468318:30
bcwaldonvishy: I sent an email to the mailing list earlier this morning but I haven't seen it go through18:36
bcwaldonvishy: any clue why?18:36
vishybcwaldon: no idea18:39
vishybcwaldon: I mean I had the ml gremlin delete it because it was a terrible email18:39
bcwaldonvishy: screw you, man18:40
bcwaldonI'l go resend it :)18:40
vishyjaypipes: bug 83328518:42
uvirtbotLaunchpad bug 833285 in glance "swift store delete only removes manifest" [High,Confirmed] https://launchpad.net/bugs/83328518:42
vishythat one is nasty18:42
openstackgerritA change was merged to openstack/glance: Fixes bug 851016.  https://review.openstack.org/44918:43
uvirtbotLaunchpad bug 851016 in glance "test_migrations.py isn't working properly" [Medium,Triaged] https://launchpad.net/bugs/85101618:43
smoservishy, what do you think about bug 849154 for milestone ?18:43
uvirtbotLaunchpad bug 849154 in nova "swap is interpreted in GB, not MB" [Undecided,In progress] https://launchpad.net/bugs/84915418:43
vishysmoser: fine with me18:43
smoserok. i will get that fixed in trun and asked the proposed for milestone.18:44
vishylooks like you will have to backport it for milestone18:44
vishydue to conflicts18:44
*** mszilagyi has quit IRC18:45
rmkNot sure if this is a development question/comment (forgive me if not) -- The latest Ubuntu packages want to start nova-vncproxy as the "nova" user, even though it's attempting to bind to port 843.  It obviously fails unless I allow it to run as root.18:45
openstackjenkinsProject nova build #1,374: SUCCESS in 3 min 53 sec: https://jenkins.openstack.org/job/nova/1374/18:45
openstackjenkinsTarmac: Removing toprettyxml from OSAPI xml serialization in favor of toxml.18:45
rmkThis is using the milestone PPA.18:46
vishyrmk: yes this is an issue18:47
vishymtaylor: ping18:47
vishyDaviey, smoser, mtaylor: is there a standard way in ubuntu to deal with workers that need to bind ports < 1024 ?18:48
*** dolphm has joined #openstack-dev18:48
rmkok thanks, I'll look for a bug or create one if it doesn't exist18:50
smoseri'm not really sure. i think most would drop permissins themselves. but i'm not too familiar.18:50
vishyguys: I have 3 bugfixes I'd like to get in diablo.  They need reviews: https://code.launchpad.net/~vishvananda/nova/lp851157 https://code.launchpad.net/~vishvananda/nova/lp829020 https://code.launchpad.net/~vishvananda/nova/lp82443318:51
Davieyvishy: inetd style?18:53
vishyDaviey: that doesn't mean much to me, but nova-vncproxy needs to bind port 84318:54
Davieyvishy: As smoser said, it's common for applications to drop their own privs18:56
vishyDaviey: so it binds the port and then drops privs?18:58
vishyso we could add that to nova-vncproxy18:58
vishybut i guess we need to change the upstart script to run it as root in the packaging then18:58
openstackgerritAlex Meade proposed a change to openstack/glance: Added min_disk and min_ram properties to images  https://review.openstack.org/42918:58
openstackgerritJay Pipes proposed a change to openstack/glance: Fixes bug 851016.  https://review.openstack.org/45318:58
uvirtbotLaunchpad bug 851016 in glance "test_migrations.py isn't working properly" [Medium,Triaged] https://launchpad.net/bugs/85101618:58
Davieyvishy: yeah19:03
Davieyvishy: Out of interest, why does it need to be a privileged port?19:03
bcwaldonjaypipes: wait!19:04
jaypipesbcwaldon: quoi?19:04
Davieyvishy: I assume you came across, http://stackoverflow.com/questions/2699907/dropping-root-permissions-in-python ?19:04
bcwaldonbuild is failing because test_migrations.conf isn't in the tarball19:04
bcwaldoni think19:04
openstackgerritBrian Waldon proposed a change to openstack/glance: Adding test_migrations.conf to tarball for testing  https://review.openstack.org/45419:04
openstackgerritA change was merged to openstack/glance: Fixes bug 851016.  https://review.openstack.org/45319:05
uvirtbotLaunchpad bug 851016 in glance "test_migrations.py isn't working properly" [Medium,Fix committed] https://launchpad.net/bugs/85101619:05
bcwaldonjaypipes: i'll destroy that review if it needs to be solved differently19:05
jaypipesbcwaldon: looking... shouldn't be an issue...19:05
bcwaldonjaypipes: I think we have to explicitly include any non py files in MANIFEST, right?19:06
jaypipesbcwaldon: you have ameade's commit as a dependency in your review...19:06
bcwaldonjaypipes: crap19:06
jaypipesbcwaldon: and no, that file should not be included in MANIFEST.in (which is what gets included in tarballs, which test_migrations.conf doesn't need to be in..)19:07
jaypipesmtaylor: ping19:07
bcwaldonjaypipes: ok, then ignore me19:07
*** nati has quit IRC19:07
*** nati has joined #openstack-dev19:07
openstackjenkinsProject nova build #1,375: SUCCESS in 3 min 50 sec: https://jenkins.openstack.org/job/nova/1375/19:08
openstackjenkinsTarmac: Adding flavor extra data extension.19:08
vishyDaviey: 843 is the port that flash checks for css security policy19:10
Davieyahh19:11
vishyDaviey: I suppose we could have it listen on another port and set an iptables rule to forward 843 to the legal port19:15
*** deshantm_laptop has joined #openstack-dev19:16
*** harlowja has quit IRC19:23
Davieyvishy: meh, sounds like it is overcomplicating it tbh.. Whilst it is good to have less stuff started as root, it's not uncommon.19:26
vishyis it ok to change the package now19:26
vishyand put in a bug about dropping privileges for essex?19:27
Davieyvishy: Did you see my ping about nova/test.py breaking the skip class of tests?19:27
vishyDaviey: yeah, do you know how to fix it?19:27
Davieyvishy: Hmm, is this for essex or diablo?19:27
vishysmoser: can you backport this one? https://code.launchpad.net/~smoser/nova/lp849160/+merge/7523719:28
Davieyvishy: no. :(19:30
Davieyit was a ping-for-help :)19:30
smoservishy, hm... i can.19:31
smoseri'd like to have that one,19:31
smoserbut it was a PITA19:31
smoserand i have ohter ones that i'd like to get in too... the swap-as-MB for one.19:32
smoserbut yes, i'll attempt a backport of that. after i fix bug 84915419:32
uvirtbotLaunchpad bug 849154 in nova "swap is interpreted in GB, not MB" [Undecided,In progress] https://launchpad.net/bugs/84915419:32
Davieyvishy: The reasoning was conditional testing if carrot is installed, http://pb.daviey.com/wSg9/19:32
smoservishy, is this right: https://code.launchpad.net/~smoser/nova/milestone-block-device-mappings/+merge/7561519:33
smoseris that proposed to the right place19:34
smoserit does suck that every time you fix a bug, you find more.19:34
dprincecomstud: you around?19:35
vishysmoser that is proposed right19:35
vishysmoser it looks like you merged r 1564 though?19:35
vishyand it needs a commit message19:35
vishyDaviey: i understand why you want it, but I don't understand why it doesn't work19:36
smoservishy, i'm pretty sure it sright19:37
smoseri intended to get 1564, not 156319:37
smoserand i did not pull in the changes for 1563 (so either i did that correct, or bzr realized that one was already there)19:37
vishysmoser: https://code.launchpad.net/~smoser/nova/milestone-block-device-mappings/+merge/75615 diff seems ok?19:38
smoseryeah, it looks like i did it right.19:39
vishysmoser: k merging it19:39
*** lorin1 has joined #openstack-dev19:39
smoserit does to me, yes.19:39
smoserdo yo uwant me to write a commit message?19:39
smoseri tried to do a good commit message on that one commit. i'd just copy and paste if you wanted something there.19:40
vishynah i just wrote a short one19:40
smosercan i get bug 851145 on the milestone radar ?19:41
uvirtbotLaunchpad bug 851145 in nova "ephemeral device (local_gb) does not get a filesystem" [Undecided,New] https://launchpad.net/bugs/85114519:41
smoseri suck and missed the fact that that was there when verifying the rest of tha tmerge.19:41
smoseri have an idea on how it could be fixed fairly un-intrusively19:41
Davieyvishy: upstream unittst reports it as a feature :/19:46
vishysmoser: do it19:47
openstackjenkinsProject nova-milestone build #41: SUCCESS in 21 sec: https://jenkins.openstack.org/job/nova-milestone/41/19:57
openstackjenkinsTarmac: backport of 3 ephemeral related bug fixes.19:57
comstuddprince: yep.. just got out of a meeting19:57
comstudabout to get on another call, but I'll be online for this one19:58
dprincecomstud: question about unplugging vbd's19:59
dprincecomstud: I'm having an issue when 'unrescueing' an instance.19:59
dprincecomstud: If I rescue an instance and immediately unrescue it (without logging into it and doing anything) it works fine.19:59
comstuddprince: ok.. when does it break?20:00
dprincecomstud: If I log into the rescue VM and mount the original image and then try to unrescue (before or after unmounting) it breaks.20:00
dprincecomstud: essentially the unplug vbd function fails.20:01
comstudah interesting20:01
dprincecomstud: seems like hot unplugging should be failsafe. I mean if it fails we delete the rescue instance anyway. So who cares?20:01
comstudyeah, i agree.  although.. i wonder why we even bother unplugging until it's shutdown20:02
dprincecomstud: I'll play with it. So you think a shutdown first would fix it?20:03
*** mfer has quit IRC20:04
dprincecomstud: what I'm getting at is a hot unplug seems sort of silly. It seems to fail if the user mounts it inside the VM (presumably to fix things, etc)20:04
comstuddprince: Yeah, i also think that's silly...20:04
dprincecomstud: Okay. I'll look at it.20:05
comstuddprince: ok.. i can talk more when I'm off this call, if needed20:05
zulttx: are you still around?20:05
dprincecomstud: sure. NP. Thanks.20:05
comstudtrying to multitask isn't working so well20:05
comstud:)20:05
*** nati has quit IRC20:11
*** nati has joined #openstack-dev20:11
*** harlowja has joined #openstack-dev20:17
openstackjenkinsProject nova build #1,376: SUCCESS in 4 min 0 sec: https://jenkins.openstack.org/job/nova/1376/20:18
openstackjenkins* Tarmac: if no public-key is given (--key), do not show public-keys in metadata service20:18
openstackjenkins* Tarmac: This makes the OS api extension for booting from volumes work.20:18
openstackjenkinsThe _get_view_builder method was replaced in the parent class, but the BootFromVolume controller was not updated to use the new method.20:18
*** jsavak has quit IRC20:24
vishysoren: lp:~soren/nova/secgroup-fixes had a test failure.  Looks like a pretty simple fix20:26
*** dolphm has quit IRC20:29
openstackjenkinsProject nova-milestone build #42: SUCCESS in 17 sec: https://jenkins.openstack.org/job/nova-milestone/42/20:32
openstackjenkinsTarmac: This is the same fix that was merged to trunk for lp:850305. I just cherry-picked the commits.20:32
openstackjenkinsProject nova-milestone build #43: SUCCESS in 17 sec: https://jenkins.openstack.org/job/nova-milestone/43/20:42
openstackjenkinsTarmac: if no public-key is given (--key), do not show public-keys in metadata service20:42
johan_-_how does the openstack project feel about using a couple of functions from a BSD licensed project?20:43
vishyjohan_-_: bsd is a compatible license for apache afaik20:46
*** dprince has quit IRC20:48
johan_-_ok. i think i might bring it up on the list when i'm ready just to be sure20:48
sorenvishy: Weirdness.20:48
vishysoren: you don't get the same failure?20:49
sorenvishy: We'll see in about 5 minutes.20:49
openstackjenkinsProject nova-milestone build #44: SUCCESS in 17 sec: https://jenkins.openstack.org/job/nova-milestone/44/20:51
openstackjenkinsTarmac: backport of inject hostname for xenstore.20:52
vishysoren: our unit tests take more than 10 minutes now :(20:52
*** lorin1 has quit IRC20:53
sorenvishy: I know a shortcut.20:53
*** zaitcev has joined #openstack-dev20:54
vishysoren: any idea why nova-coverage appears to be broken, btw?20:54
sorenWhat, again? Darn it!20:54
sorenvishy: I mean: No.20:54
sorenNo idea.20:54
sorenvishy: Oh.20:55
sorenYeah, I think I can guess.20:55
sorenI just wish there was a more useful error message, but meh.20:56
* soren verifies hypothesis real quick20:56
sorenvishy: Ran 1712 tests in 412.926s20:58
sorenvishy: More than 5 minutes, but certainly less than 10.20:58
vishydamn, your computer is faster than mine20:58
sorenvishy: Hardly.20:59
sorenvishy: Take a look at libeatmydata20:59
vishymine ran in 600 s20:59
vishybut it was in a vm...20:59
mtaylorsoren: any idea why nova-coverage doesn't show the output of ./run_tests.sh when it runs it?20:59
sorenvishy: If you're on Ubuntu, just install eatmydata and run: "eatmydata ./run_tests.sh -N"20:59
sorenmtaylor: Looking at that *right now*, actually.20:59
jk0soren: I've seen them take 10+ minutes21:00
*** ohnoimdead_ has joined #openstack-dev21:00
*** ohnoimdead has quit IRC21:00
*** ohnoimdead_ is now known as ohnoimdead21:00
* jk0 looks into eatmydata21:00
vishysoren: nice21:00
sorenjk0: eatmydata will cut probably 40% off of that.21:00
jk0I think eday recommended that a year or so, but completely forgot about it21:01
vishysoren: damn, doesn't look like it is in brew21:01
sorenIt basically intercepts fsync() calls and stuff like that.21:01
sorenSo if anything tries to force a write to disk, it just goes "meh" and moves on.21:01
sorenWhen so much time is spent in sqlalchemy, and by extension in sqlite, this makes a world of difference.21:02
jk0I wonder if it would help to use this on jenkins?21:02
mtaylorjk0: saw just thinking that myself21:02
sorenmtaylor: /usr/lib/pymodules/python2.7/nose/plugins/manager.py:383: RuntimeWarning: Unable to load plugin xcover = nosexcover:XCoverage: coverage>=3.421:03
sorenmtaylor: That's why nova-coverage fails.21:03
mtaylorsoren: well fuck a duck21:03
mtaylorsoren: so I have two qestions now - a) why isn't nosexcover there nad b) why the hell does that output go away when run in the jenkins job21:04
sorenmtaylor: Would you be very upset if I did: sudo apt-get --reinstall install python-coverage?21:04
vishymeh, i have to port the freebsd port to osx…looks dangerous21:04
mtaylorsoren: well... hang on a sec. I'd like to figure out what's wrong there21:04
mtaylorsoren: so that we don't just get right back into this situation AGAIn21:04
sorenmtaylor: Well, if you look at "dpkg -L python-coverage" it lists a bunch of files that aren't actually on the filesystem.21:05
mtaylorsoren: ok. go for it21:05
mtaylorsoren: check python-nosexcover too21:05
sorenmtaylor: It's working now.21:06
mtaylorgreat. now what the hell happened there21:06
* soren blames pip21:07
sorenJust because.21:07
mtaylorsoren: probably right - although we're finally off of pip there21:08
sorenAnyways, it works now.21:11
sorenvishy: ^21:11
vishycool21:13
mtaylorthanks soren21:14
sorenSure thing.21:14
*** lts has quit IRC21:14
smoserdevcamcar, https://code.launchpad.net/~smoser/nova/lp849154/+merge/7537821:22
smoserthats updated. anyone else who wanted to review that also against trunk21:22
smoseri'm hoping to get that into milestone21:22
smoserto make the interpretation change of "swap" to be MB rather than GB this release rather than next (as then there is more baggage)21:22
*** martine has quit IRC21:24
openstackgerritBrian Waldon proposed a change to openstack/glance: Fixing bug 794582 - Now able to stream http(s) images  https://review.openstack.org/45521:30
uvirtbotLaunchpad bug 794582 in glance "glance sets image size to 0 for http images -> yields connection reset error on GET" [Medium,Confirmed] https://launchpad.net/bugs/79458221:30
*** bsza has quit IRC21:46
*** cp16net has quit IRC21:47
*** ameade has quit IRC21:50
openstackgerritA change was merged to openstack/openstack-ci: Added a new packaging script which uses bzr bd.  https://review.openstack.org/44821:55
*** kbringard has quit IRC22:04
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: close pull requests from openstack-manuals  https://review.openstack.org/45622:12
*** rnirmal has quit IRC22:12
*** patelna has quit IRC22:13
openstackgerritA change was merged to openstack/openstack-ci-puppet: close pull requests from openstack-manuals  https://review.openstack.org/45622:15
openstackjenkinsProject nova build #1,377: SUCCESS in 3 min 45 sec: https://jenkins.openstack.org/job/nova/1377/22:35
openstackjenkinsTarmac: updates Exception.NoMoreFixedIps to subclass NovaException instead of Error.22:35
*** rnorwood has quit IRC22:42
*** Guest51235 has joined #openstack-dev22:44
*** mattray has quit IRC22:50
openstackgerritJay Pipes proposed a change to openstack/glance: Fixes Bug #851216  https://review.openstack.org/45722:52
uvirtbotLaunchpad bug 851216 in glance "TestMigrations should default to sqlite" [Undecided,New] https://launchpad.net/bugs/85121622:52
*** code_franco has quit IRC22:58
vishysmoser: https://bugs.launchpad.net/nova/+bug/851301 is the same as https://bugs.launchpad.net/nova/+bug/82763423:03
uvirtbotLaunchpad bug 851301 in nova "in libvirt, local_gb is used incorrectly" [Undecided,New]23:03
vishysmoser: i'm not sure that resizing the root drive is the right approach.  Amazon does not do this AFAIK23:04
*** amccabe has quit IRC23:07
openstackgerritKevin L. Mitchell proposed a change to openstack/glance: Add protected properties  https://review.openstack.org/45823:21
*** bcwaldon has quit IRC23:23
*** ohnoimdead has quit IRC23:28
*** harlowja has left #openstack-dev23:28
*** jeffjapan has joined #openstack-dev23:29
openstackgerritAnne Gentle proposed a change to openstack/compute-api: Fixes pom file to use section numbering again  https://review.openstack.org/45923:32
openstackgerritA change was merged to openstack/compute-api: Fixes pom file to use section numbering again  https://review.openstack.org/45923:32
*** nati_ has joined #openstack-dev23:34
*** nati has quit IRC23:37
openstackgerritAnne Gentle proposed a change to openstack/compute-api: Fixes pom file to use section numbering and fix errors  https://review.openstack.org/46023:37
openstackgerritAnne Gentle proposed a change to openstack/compute-api: Fixes pom file to use section numbering and fix error pointing source instead of src  https://review.openstack.org/46123:39
openstackgerritA change was merged to openstack/compute-api: Fixes pom file to use section numbering and fix errors  https://review.openstack.org/46023:40
openstackgerritA change was merged to openstack/compute-api: Fixes pom file to use section numbering and fix error pointing source instead of src  https://review.openstack.org/46123:40
tr3buchetvishy: backported that merge and proposed to milestone23:42
openstackgerritAnne Gentle proposed a change to openstack/compute-api: Fixes pom file to use section numbering and fix error pointing to src docbkx  https://review.openstack.org/46223:44
openstackgerritA change was merged to openstack/compute-api: Fixes pom file to use section numbering and fix error pointing to src docbkx  https://review.openstack.org/46223:45

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