Friday, 2011-08-19

*** deshantm has quit IRC00:04
*** jakedahn has quit IRC00:04
openstackjenkinsProject nova-tarball-bzr-delta build #497: FAILURE in 11 sec: https://jenkins.openstack.org/job/nova-tarball-bzr-delta/497/00:14
openstackjenkins* Tarmac: Makes all of the binary services launch using the same strategy.00:14
openstackjenkins��* Removes helper methods from utils for loading flags and logging00:14
openstackjenkins��* Changes service.serve to use Launcher00:14
openstackjenkins��* Changes service.wait to actually wait for all the services to exit00:14
openstackjenkins��* Changes nova-api to explicitly load flags and logging and use service.serve00:14
openstackjenkins* Fixes the annoying IOError when /etc/nova/nova.conf doesn't exist00:14
openstackjenkins* Tarmac: Fixes issue where ServersXMLSerializer was missing a method for update actions00:14
openstackjenkinsProject nova build #1,270: SUCCESS in 3 min 32 sec: https://jenkins.openstack.org/job/nova/1270/00:26
openstackjenkins* Tarmac: Currently, rescue/unrescue is only available over the admin API. Non-admin tenants also need to be able to access this functionality. This patch adds rescue functionality over an API extension.00:26
openstackjenkins* Tarmac: Makes all of the binary services launch using the same strategy.00:26
openstackjenkins��* Removes helper methods from utils for loading flags and logging00:26
openstackjenkins��* Changes service.serve to use Launcher00:26
openstackjenkins��* Changes service.wait to actually wait for all the services to exit00:26
openstackjenkins��* Changes nova-api to explicitly load flags and logging and use service.serve00:26
openstackjenkins* Fixes the annoying IOError when /etc/nova/nova.conf doesn't exist00:26
openstackgerritA change was merged to openstack/keystone: Refactored manage.py to be both testable and useful for testing  https://review.openstack.org/27600:31
openstackgerritVerification of a change to openstack/keystone failed: Cleaned up come issues with python2.6.  https://review.openstack.org/27000:31
openstackgerritVerification of a change to openstack/keystone failed: Changes to allow test to work on python 2.6.*.  https://review.openstack.org/27100:31
openstackjenkinsProject burrow build #33: SUCCESS in 11 sec: https://jenkins.openstack.org/job/burrow/33/00:31
openstackjenkinsTarmac: Backend module cleanup.00:31
*** mfer has quit IRC00:42
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Cleaned up come issues with python2.6.  https://review.openstack.org/27000:44
openstackgerritA change was merged to openstack/keystone: Cleaned up come issues with python2.6.  https://review.openstack.org/27000:47
*** bengrue has quit IRC00:50
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Changes to allow test to work on python 2.6.*.  https://review.openstack.org/27101:02
thickskinI see01:03
znsthickskin: was that part of a conversation I missed?01:04
openstackgerritA change was merged to openstack/keystone: hanges to allow test to work on python 2.6.*.  https://review.openstack.org/27101:04
thickskinzns:You don't have to worry about it.01:06
*** mfer has joined #openstack-dev01:10
*** mfer has quit IRC01:12
*** zns has quit IRC01:13
*** rods has quit IRC01:15
*** Tushar has quit IRC01:17
*** bcwaldon has quit IRC01:19
*** mfer has joined #openstack-dev01:37
*** mfer has quit IRC01:40
*** deshantm has joined #openstack-dev01:46
*** bengrue has joined #openstack-dev01:46
*** lorin1 has joined #openstack-dev01:48
lorin1ttx: I think you were asking me a question the other day, as I was on my way out...01:49
*** RobertLaptop has quit IRC01:55
*** RobertLaptop has joined #openstack-dev02:07
openstackjenkinsProject nova build #1,271: SUCCESS in 3 min 39 sec: https://jenkins.openstack.org/job/nova/1271/02:22
openstackjenkinsTarmac: Bugfix for lp 828429.  Its still not clear to me exactly how this code path is actually invoked when nova is used, so I'm looking for input on whether we should be adding a test case for this, removing the code as unused, etc.  Thanks.02:22
uvirtbotLaunchpad bug 828429 in nova "nova-dhcpbridge: interface causes KeyError - bug added in v.1449" [High,In progress] https://launchpad.net/bugs/82842902:22
*** bengrue has quit IRC02:29
*** bengrue has joined #openstack-dev02:31
*** RobertLaptop has quit IRC02:33
*** RobertLaptop has joined #openstack-dev02:34
*** lorin1 has quit IRC02:59
*** martine has joined #openstack-dev03:01
openstackgerritJustin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n  https://review.openstack.org/27703:16
openstackgerritJustin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n  https://review.openstack.org/27703:25
*** zul has joined #openstack-dev04:18
*** deshantm has quit IRC04:22
*** dolphm_ has quit IRC05:31
*** martine_ has joined #openstack-dev05:49
*** martine has quit IRC05:51
*** martine_ has quit IRC05:53
openstackgerritA change was merged to openstack/openstack-ci-puppet: Update rfc.sh.  https://review.openstack.org/27306:24
openstackgerritA change was merged to openstack/openstack-ci: Update rfc.sh to assert divergence.  https://review.openstack.org/27406:25
openstackgerritA change was merged to openstack/openstack-ci-puppet: Update rfc.sh to assert divergence.  https://review.openstack.org/27506:45
*** reidrac has joined #openstack-dev07:02
openstackgerritMonty Taylor proposed a change to openstack/keystone: Added more missing files to MANIFEST.in.  https://review.openstack.org/27807:06
*** timr has quit IRC07:34
*** timr has joined #openstack-dev08:10
ttxsoren: looks like https://code.launchpad.net/~midokura/nova/nova-quantum-vifid/+merge/71990 just needs a reapprove08:27
*** reidrac has quit IRC08:30
* soren looks08:45
ttxwas approved, but caught in a python-migrate race08:45
sorenttx: Yup. Approved.08:46
ttxthx08:46
zykes-what version of .py are currently spported ?08:47
openstackjenkinsProject nova build #1,272: SUCCESS in 3 min 23 sec: https://jenkins.openstack.org/job/nova/1272/09:00
openstackjenkinsTarmac: Added uuid column in virtual_interfaces table, and an OpenStack extension API for virtual interfaces to expose these IDs.  Also set this UUID as one of the external IDs in the OVS vif driver.09:00
*** nickon has joined #openstack-dev09:05
ttxsoren: could you have a quick look at https://bugs.launchpad.net/nova/+bug/828841 ?09:20
uvirtbotLaunchpad bug 828841 in nova "Launching VMs in a security group with a source group set with ecua2ools 1.3 causes VMs to fail to spawn" [Undecided,New]09:20
sorenzykes-: 2.6 and onwards.09:22
sorener..09:22
ttx2.x with x>6 ?09:22
sorenzykes-: Sorry, 2.X for X >= 6.09:22
*** YorikSar has quit IRC09:33
sorenttx: Do you know if there's a way to make pep8 ignore a line?10:05
sorenttx: I have a line that needs to be long.10:05
sorenhmmm... or do I?10:06
*** reidrac has joined #openstack-dev10:31
*** lorin1 has joined #openstack-dev10:54
*** lorin1 has quit IRC10:54
*** rods has joined #openstack-dev11:01
*** markvoelker has joined #openstack-dev11:06
*** bsza has joined #openstack-dev11:09
*** timr has quit IRC11:14
*** anp has quit IRC11:33
*** mfer has joined #openstack-dev11:43
*** anp has joined #openstack-dev11:46
*** anp has quit IRC11:54
*** anp has joined #openstack-dev12:01
*** lorin1 has joined #openstack-dev12:03
*** lorin1 has left #openstack-dev12:03
*** lts has joined #openstack-dev12:43
*** martine_ has joined #openstack-dev12:43
*** martine_ has quit IRC12:48
*** joesavak has joined #openstack-dev12:50
*** bourke_ has joined #openstack-dev12:57
bourke_Hi, just wondering has anyone else experienced the fact that the swift CLI tool doesn't seem to return any errno's other than 0 ?12:58
*** chipc has joined #openstack-dev12:59
*** zns has joined #openstack-dev13:07
*** mfer has quit IRC13:12
*** mfer has joined #openstack-dev13:12
*** martine has joined #openstack-dev13:20
*** bcwaldon has joined #openstack-dev13:21
*** YorikSar has joined #openstack-dev13:25
*** mattray has joined #openstack-dev13:35
*** anp has quit IRC13:35
*** bcwaldon_ has joined #openstack-dev13:37
*** bcwaldon_ has quit IRC13:37
*** bcwaldon has quit IRC13:39
annegentlemtaylor: around?13:42
*** chipc has quit IRC13:45
annegentleI need help troubleshooting a build, anyone else got some Jenkins knowhow?13:45
*** chipc has joined #openstack-dev13:47
*** mnour has quit IRC13:48
*** bcwaldon has joined #openstack-dev13:48
*** amccabe has joined #openstack-dev13:51
*** zns has quit IRC14:00
*** zns has joined #openstack-dev14:00
*** bourke_ has left #openstack-dev14:03
*** anp has joined #openstack-dev14:05
mtaylorannegentle: whazzup?14:07
*** mattray has quit IRC14:26
*** mattray has joined #openstack-dev14:28
*** clayg_ has joined #openstack-dev14:30
*** nickon has quit IRC14:34
*** bcwaldon has quit IRC14:39
*** bcwaldon_ has joined #openstack-dev14:39
*** anp has quit IRC14:39
*** nmistry has joined #openstack-dev14:48
*** zul has quit IRC14:52
*** rnirmal has joined #openstack-dev14:54
*** rnirmal_ has joined #openstack-dev14:55
*** rnirmal has quit IRC14:59
*** rnirmal_ is now known as rnirmal14:59
*** reidrac has left #openstack-dev15:05
*** dragondm has joined #openstack-dev15:08
*** heckj has joined #openstack-dev15:14
*** nmistry has quit IRC15:21
ttxvishy: ping15:38
*** lorin1 has joined #openstack-dev15:43
*** vladimir3p has joined #openstack-dev15:43
mtaylordolphm: ping15:46
ttxvishy: see pm15:47
*** zns has quit IRC15:51
dolphmmtaylor: > 1s15:52
mtaylordolphm: no rush - just pinging you to approve my change :)15:53
notmynamemtaylor: do you use any of the vcs stuff found in setup.py for packaging?15:53
mtaylornotmyname: no15:53
dolphmmtaylor: approved15:53
notmynamemtaylor: we don't use it at all (AFAIK), and since it's bzr-specific, I was hopign to remove it15:54
mtaylornotmyname: ++15:54
notmynamemtaylor: cool15:54
openstackgerritA change was merged to openstack/keystone: Added more missing files to MANIFEST.in.  https://review.openstack.org/27815:55
*** rods has quit IRC15:58
*** lorin1 has left #openstack-dev15:58
*** lorin1 has joined #openstack-dev15:58
*** zns has joined #openstack-dev15:59
*** mattray has quit IRC16:00
*** rods has joined #openstack-dev16:04
mtaylordolphm: thanks!16:08
dolphmmtaylor: np16:08
mtaylordolphm: I think that's the last of the fixes we need pre-packaging16:08
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Fix Bug 825344 - Only set project watches once  https://review.openstack.org/27916:11
uvirtbotLaunchpad bug 825344 in openstack-ci "project watches incorrectly added to gerrit" [Medium,New] https://launchpad.net/bugs/82534416:11
openstackgerritDolph Mathews proposed a change to openstack/keystone: XSD & sample updates  https://review.openstack.org/28016:18
*** chomping has joined #openstack-dev16:26
vishysoren: https://bugs.launchpad.net/nova/+bug/829609 So it looks like we are supposed to allow everything if just source group is specified16:27
uvirtbotLaunchpad bug 829609 in ensemble "EC2 compatibility describe security group returns erroneous value for group ip permissions" [Critical,New]16:27
*** bengrue has quit IRC16:45
*** bengrue has joined #openstack-dev16:46
*** Tushar has joined #openstack-dev16:48
vladimir3pvishy: ping. any update on volume types status?16:56
dolphmmtaylor: revised gerrit workflow doesn't have the direct command to install the commit-msg hook, and keystone doesn't have a rfc.sh file yet??16:57
vishyvladimir3p: the volume cleanup is taking a lot more than I expected16:58
vishyvladimir3p: if you want to keep going on types/extra_types/meta that would be awesome16:58
vladimir3pok, I will take care of this part, but a bit later today16:58
Tusharbcwaldon_: Are you there?16:59
bcwaldon_Tushar: yep, I'm about to sit down for a quick meeting, though. Give me 5 minutes?17:00
Tusharbawaldon_:OK17:00
*** bcwaldon_ has quit IRC17:00
*** bengrue has quit IRC17:07
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Will this work?  https://review.openstack.org/28117:07
*** bcwaldon has joined #openstack-dev17:12
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Will this work?  https://review.openstack.org/28117:13
bcwaldonTushar: what can I help you with?17:13
Tusharbcwaldon:I want to discuss with you on your review comments on my merge prop # link https://code.launchpad.net/~tpatil/nova/add-options-network-create-os-apis/+merge/6829217:13
bcwaldonsure17:13
Tusharbcwaldon: line #74717:14
bcwaldonokay, I'm with you17:14
Tusharbcwaldon: I also wanted to aviod RPC call but it's simply inevitable17:14
Tusharbcwaldon: *avoid17:14
*** dprince has joined #openstack-dev17:14
bcwaldonTushar: okay, can you help me understand why?17:15
bcwaldonTushar: I haven't spent much time in the network manager17:15
Tusharbcwaldon: OK17:16
Tusharbcwaldon: #line 52017:16
bcwaldonTushar: yep17:16
Tusharbcwaldon: Heere I want to assign specific address instead of getting available address from the network pool17:18
bcwaldonTushar: I definitely understand the need for the function. I'm questioning the behavior when 'address' is None. I would prefer for this function to require a non-None 'address' and force the client to use the fixed_ip_associate_pool function directly17:19
dprincemtaylor: you around?17:20
Tusharbcwaldon: Ok, I got you.17:21
vishybtw, in case anyone decides they want to try it, nova does not run in pypy :(17:22
Tusharbcwaldon: line#454, there is already a function with the name  fixed_ip_associate, that's why I appended by_address, the purpose of both these functions are different17:22
dprincemtaylor: Jenkins tarmac job is still hosed. It actually looks like a LP issue again. Honestly I'd reboot the nova slave if I had access.17:23
bcwaldonTushar: let me look real fast, one sec17:23
dprinceAlso. I've noticed when the nova merge props stack up Tarmac processes them in reverse order... which could get a bit frustrating with DB migration numbers, etc.17:24
bcwaldonTushar: it looks like fixed_ip_associate is a simpler (dumber) version of fixed_ip_associate_by_address17:27
bcwaldonTushar: any way you can replace fixed_ip_associate with your new function?17:27
Tusharbcwaldon: Ok, I will fixed all remaining comments given by you and propose it again for merging by end of today. Thank you very much for your time.17:31
bcwaldonTushar: I should be thanking you for being so patient waiting for me to review it!17:31
bcwaldonTushar: Hopefully it won't take so long for me to do it next time :)17:32
Tusharbcwaldon: Thanks17:32
openstackgerritDolph Mathews proposed a change to openstack/keystone: XSD & sample updates  https://review.openstack.org/28017:34
*** deshantm has joined #openstack-dev17:38
*** anotherjesse has joined #openstack-dev17:46
anotherjesseblamar_ / bcwaldon time to talk about https://code.launchpad.net/~cloudbuilders/nova/os-floating-ips-redux/+merge/7192217:49
anotherjesse?17:49
bcwaldonanotherjesse: yessir17:49
blamar_anotherjesse, yup yup yup17:49
anotherjesseto the question about where the actions go - we put them in the contrib module mostly because it kept the code related to floating ips seperated17:50
bcwaldonanotherjesse: I don't follow, can you explain? Not sure I know what question you're answering17:51
anotherjesse"I think it makes even more sense to have a disassociate/associate action on a server"17:51
bcwaldonOkay, well you could keep the code in contrib. See jk0's rescue extension17:52
anotherjessebcwaldon: k!17:52
anotherjessekinda weird that rescue is an extension ;)17:53
bcwaldonyeah, but all admin functionality is going to move into an extension17:53
bcwaldonanything that isn't in the spec *has* to be an extension17:53
bcwaldonat least that's the hole we've dug ourselves into at this point17:53
jk0anotherjesse: I just do what product tells me :)17:53
anotherjessejk0: so that's the secret ;)17:54
anotherjessebcwaldon - this is just a refactor17:54
anotherjesseI'll look at the rescue though17:54
anotherjessebcwaldon: interesting - so - personally I don't want to get into a battle about what proper restful structure17:55
blamar_anotherjesse: not restful, just consistent with other current features?17:55
bcwaldonyeah, I wouldn't call /action restful ;)17:55
*** mgius has joined #openstack-dev17:57
anotherjesseblamar_: / bcwaldon moving to to /servers/:id/associate/(floating_ip?) (post?)   (or is floating ip a param in the body?)17:57
bcwaldonI would go with curl -X POST -d '{"associate": {"address":"192.168.0.4"}}' http://localhost:8774/v1.1/servers/12/action17:58
blamar_s/associate/associate_floating_ip17:58
blamar_or leave it associate and change "address" to "floating_ip"17:59
blamar_tons of ways! :\17:59
bcwaldonjust have to think about future usage of the 'associate' action17:59
anotherjessethe uri path is actually v1.1/:tenant_id/servers/:server_id/action ??18:00
bcwaldonyep18:00
anotherjessethe word action?18:00
bcwaldonthats how resize, confirmResize, revertResize, changePassword, reboot work18:00
bcwaldonyes18:00
anotherjesserescue/reboot don't do that though - it uses rescue/unrescue instead of action?18:00
bcwaldonreboot does18:01
bcwaldonrescue is in the admin api18:01
bcwaldonthe admin api is implemented as a token in the uri, not through an 'action'18:01
bcwaldonbut now we have the os-rescue extension which adds 'rescue' and 'unrescue' actions18:01
anotherjessek18:02
anotherjesseI am reading the exts.ActionExtension code now18:02
anotherjessethanks for clearing up :)18:02
bcwaldonthat's exactly what you need18:02
bcwaldonno problem, it will be changing again18:02
anotherjessebcwaldon: while I'm here - about the keypair integration merge18:02
bcwaldonyep18:03
anotherjesseI am leaning towards removing the automatic adding of the first keypair in the project18:03
bcwaldonanotherjesse: yeah, I think I'm for that. I just wanted to make sure we were all on the same page. I like being explicit18:04
anotherjessecool - then if you give me an approve we can merge that sucker :)18:04
bcwaldonI'll approve, but tarmac is borked18:04
bcwaldonprobably won't merge for a while18:05
anotherjesseno landing for you … 4 hours18:05
bcwaldonexactly18:05
*** lorin1 has quit IRC18:06
bcwaldonanotherjesse: dprince actually has something he wanted to bring up about that branch18:08
bcwaldonanotherjesse: he'll be here in a minute18:08
anotherjessecool18:08
dprinceyou guys are winksters today ;)18:10
anotherjessenot getting much sleep ;p18:10
dprinceanotherjesse: So. Would it be a good idea to leave the old keypair behaviour in place for the v1.0 API?18:11
anotherjessedprince: the idea of grabbing the first keypair from the project?18:12
dprinceanotherjesse: and require it to be specified when using the v1.1 API, via the extension.18:12
dprinceanotherjesse: Yes.18:12
anotherjessedprince: if we do that should we make it be the first keypair for the user?18:12
dprinceanotherjesse: Removing that behavoiur will break some tests suites.18:12
*** dolphm has quit IRC18:13
dprinceanotherjesse: Yes.18:13
anotherjessedprince: deal18:13
sorendprince: Hey. I had accidentally included a part of a different patch in https://code.launchpad.net/~soren/nova/virt-layer-cleanup/+merge/72173 which made the test suite hang. Would you mind giving it another quick look and reapprove, it you're happy with it?18:13
sorens/, it/, if/18:13
anotherjessedprince: still new to os api layer - did my change remove from both versions?18:13
bcwaldonanotherjesse: it did18:14
anotherjessebcwaldon: any pointers on the clean way to check versions at the location I changed it?18:14
bcwaldonanotherjesse: figure out how to add it in the ControllerV11 class in api.openstack.servers18:16
dprinceanotherjesse: I say add a parameter to the helper.create_instance call. Then each ControllerV11 and ControllerV10 controller can have their own methods to get the keypairs?18:17
bcwaldonthat'll work18:18
anotherjessek - will make a stab at that .. thanks - (wink)18:18
jeblairdprince: it looks like the nova build slave is swapping.18:18
jeblairi'm going to collect some data, then probably expand the image and reboot18:19
dprincejeblair: Like I said. If I had access I'd reboot it.18:19
bcwaldonlike a boss18:19
dprincejeblair: making it bigger is good too.18:19
dprincejeblair: Is there an easy way to make it so the nova-tarmac and nova-coverage jobs don't run at the same times as well?18:20
dprincejeblair: Both of those can do a run_tests.sh -N and that seems worrisome.18:20
*** bengrue has joined #openstack-dev18:21
jeblairi think we can reduce the number of executors to 1.  that's caused problems with gerrit/git slaves, but i don't know if bzr/tarmac has them.18:21
jeblairis your concern resource usage, or something more like resource collision?18:22
jeblairif it's usage, then maybe just expanding memory is the answer.  if it's collision, ie, the test can't be run twice on the same machine at the same time, that's harder. :/18:22
*** rnirmal has quit IRC18:23
*** mattray has joined #openstack-dev18:27
*** markvoelker has quit IRC18:27
dprincejeblair: My concern is resource collision between the various nova jobs.18:27
jeblairdo you know of something specific that collides?18:28
dprincejeblair: run_tests -N does a mix of unit and functional tests. It actually runs some of the API code on ports, etc.18:28
bcwaldonvishy: When you say 'we just need to change noauth...', you mean 'vishy needs to change noauth...', right? Just want to be clear18:28
jeblairok.  that does sound problematic.18:29
*** alekibango has joined #openstack-dev18:30
*** uvirtbot` has joined #openstack-dev18:30
*** lorin11 has joined #openstack-dev18:31
*** colinnich_ has joined #openstack-dev18:31
jeblairdprince: i've set the # of executors to 1 for now.  and the resize is in progress.18:31
*** lorin11 has left #openstack-dev18:31
vishybcwaldon: depends who gets merged first...18:31
vishy:)18:31
dprincejeblair: Thanks man. Sounds good.18:31
bcwaldonvishy: I see...the race is on!18:31
*** comstud_ has joined #openstack-dev18:34
*** jsavak has joined #openstack-dev18:34
dprincejeblair: previously a lot of fingers were pointing at Launchpad as being the cause of the hang. The jobs say 'Connection to bazaar.launchpad.net closed by remote host.'. The thing is I also run a cloud server (SmokeStack) that pulls quite a bit of data from Launchpad as well. And I never see it hang. Just got me looking a bit closer as these jobs seem to hang all the time now.18:34
*** kpepple_ has joined #openstack-dev18:35
jeblairi know that error occured when launchpad changed their connection timeouts recently18:35
jeblairi believe tarmac was using bzrlib which held its connection to lp open during its work18:36
jeblairwhich can take a while18:36
*** amccabe_ has joined #openstack-dev18:36
jeblairnonetheless, if the test suite has a race condition, that will cause that error too, as you suggest18:36
jeblair(likely regardless of launchpad's timeout)18:36
dprincejeblair: Sure. Anyway. Hopefully this gets us over the hump. I think the swap space issue might be the big one causing the recent hangs.18:37
jeblairof course the swapping is a problem as well; if the tests are proceeding, they are doing so very slowly.  lots of problems at once!18:37
*** joesavak has quit IRC18:37
*** tr3buche` has joined #openstack-dev18:37
*** dragondm has quit IRC18:38
*** amccabe has quit IRC18:38
*** mnaser has quit IRC18:38
*** sandywalsh has quit IRC18:38
*** uvirtbot has quit IRC18:38
*** jk0 has quit IRC18:38
*** comstud has quit IRC18:38
*** smoser has quit IRC18:38
*** colinnich has quit IRC18:38
*** tr3buchet has quit IRC18:38
*** dabo has quit IRC18:38
*** kpepple has quit IRC18:38
*** smoser has joined #openstack-dev18:39
*** mwhooker has joined #openstack-dev18:39
vladimir3pvishy: I remember I asked about it, but I don't remember the answer. Why virtio is disabled for bridge interface (I see it enabled for some other things)18:41
vishyvladimir3p: there are issues with attaching volumes with older kernels18:42
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Wrong common namespace.  https://review.openstack.org/28218:42
vladimir3pvishy: even if we will enable it for bridge? (I'm not talking about disks/devices)18:43
vishythe issue is that virtio net devices will fail if a block device is hot attached18:43
vishywith older kernels18:43
*** sandywalsh has joined #openstack-dev18:46
vladimir3pvishy: hmm... so if we will enable virtio for bridged NIC interfaces it will cause problems when block devices will be hot-added ... ?18:46
vladimir3pvishy: seems weird ..18:46
openstackgerritJustin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n  https://review.openstack.org/27718:50
openstackjenkinsProject nova build #1,273: SUCCESS in 3 min 41 sec: https://jenkins.openstack.org/job/nova/1273/18:53
openstackjenkinsTarmac: Removes the incorrect hard-coded filter path.18:53
*** anotherjesse_ has joined #openstack-dev18:54
*** anotherjesse has quit IRC18:55
*** anotherjesse_ is now known as anotherjesse18:55
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Update rfc.sh to add change-id to commit message.  https://review.openstack.org/28318:57
*** mattfarina has joined #openstack-dev18:58
*** mfer has quit IRC18:58
*** mattfarina is now known as mfer18:58
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: Update rfc.sh to add change-id to commit message.  https://review.openstack.org/28418:58
openstackgerritJames E. Blair proposed a change to openstack/keystone: Add rfc.sh for git review.  https://review.openstack.org/28518:59
openstackgerritJames E. Blair proposed a change to openstack/glance: Add rfc.sh for git review.  https://review.openstack.org/28619:00
openstackgerritDolph Mathews proposed a change to openstack/keystone: Banished .DS_Store  https://review.openstack.org/28719:01
*** tr3buche` is now known as tr3buchet19:02
tr3buchettest19:03
tr3buchetweird19:03
openstackgerritA change was merged to openstack/keystone: Add rfc.sh for git review.  https://review.openstack.org/28519:05
*** rnirmal has joined #openstack-dev19:05
vishyvladimir3p: yup that is what I'm saying19:05
bengruevladimir3p: I'm going to start in on the nova api for the volume_type changes…. as long as you haven't started on them.19:05
bengrue(Have you started on them?)19:05
jeblairthe nova build slave is up and running again, with more memory, but it's currently only running one job at a time, so it may take a bit to work through the backlog.19:05
vishyvladimir3p: if you hot-add a block device the virio kernel model crashes and you lose networking19:05
*** zns1 has joined #openstack-dev19:06
openstackgerritA change was merged to openstack/keystone: Wrong common namespace.  https://review.openstack.org/28219:07
*** zns has quit IRC19:08
*** amccabe_ has quit IRC19:08
vladimir3pvishy: I was thinking about enabling it for network ifaces, but we can keep it disabled for block19:13
vishyvladimir3p: don't think that matters19:13
vishyvladimir3p: virito net breaks when you hot add19:13
vishyvladimir3p: IIRC i tested hot adding a non-virtio block device and it still crashes19:14
vladimir3pvishy: :-)19:14
vishyvladimir3p: although I'm not totally sure19:14
vishyvladimir3p: anyway it doesn't matter if you don't need to support lucid /centos 5.5 guests19:14
vladimir3pvishy: at least we can try to make it configurable.19:14
vishyvladimir3p: +1 there is even a bug that has been around for about 6 mo+19:14
vladimir3pvishy: :-) ok, will try to find it19:15
vishyhttps://bugs.launchpad.net/nova/+bug/70204019:15
uvirtbot`Launchpad bug 702040 in nova "Slow network performance since Virtio net drivers are always disabled" [Wishlist,Confirmed]19:15
bengruejeblair: where's the nova build slave?19:16
bengruevladimir3p: ping, re: my earlier question?19:17
*** amccabe has joined #openstack-dev19:17
vladimir3pbengrue: sorry, I missed it19:17
vladimir3pbengrue: pls take a look at...19:17
vladimir3pbengrue: lp:~vladimir.p/nova/volume_type_extradata19:18
vladimir3pbengrue: it has all required DB schemas, migration and DB APIs. Some tests as well19:18
bengrueokay!19:19
bengrueBut, it's cool if I write the APIs for you, right?19:19
vladimir3pbengrue: I've not started any API work, so you are welcome19:19
bengrueyou've not started that work?19:19
bengrueexcellent.19:19
vladimir3pbengrue: let's stay in touch & updated on progress19:19
bengruevladimir3p: cool, what's your preferred method of communication?19:20
vladimir3pbengue: any... email might be better as I could miss IRC (vladimir@zadarastorage.com)19:22
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Remove more group stuff  https://review.openstack.org/28819:29
*** mattray has quit IRC19:33
*** mattray has joined #openstack-dev19:34
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Documentation cleanups.  https://review.openstack.org/28919:38
*** anotherjesse has quit IRC19:40
*** anotherjesse has joined #openstack-dev19:42
*** mattray has quit IRC19:45
*** anotherjesse has quit IRC19:46
*** ohnoimdead has joined #openstack-dev19:48
*** anotherjesse has joined #openstack-dev19:48
vladimir3pfolks, I'm getting a strange error on one of servers while trying to spawn a new instance: Error: 'V1Client' object has no attribute 'set_auth_token'. All servers working from the same source tree and it seems like some pre-requisites are not installed. Any clue?19:49
*** mattray has joined #openstack-dev19:49
bcwaldonvladimir3p: upgrade glance19:50
bcwaldonvladimir3p: on the node you are getting the error on19:50
vladimir3pbcwaldon: hmm... but glance is running on a different server19:50
dprincemake sure you upgrade the glance paste configs in your config file too.19:50
bcwaldonit's glance client that is causing the problem19:50
vladimir3pbcwaldon: ok, thanks will do it.19:51
*** anotherjesse has quit IRC19:57
*** anotherjesse has joined #openstack-dev19:58
*** rnorwood has joined #openstack-dev19:59
*** dprince has quit IRC20:01
vladimir3pbcwaldon: it works, thanks20:03
bcwaldonvladimir3p: cool, glad I could help20:03
openstackgerritMonty Taylor proposed a change to openstack/keystone: Removed OSX files that shouldn't be in git.  https://review.openstack.org/29020:05
openstackgerritA change was merged to openstack/keystone: XSD & sample updates  https://review.openstack.org/28020:06
openstackgerritA change was merged to openstack/keystone: Banished .DS_Store  https://review.openstack.org/28720:08
openstackgerritA change was merged to openstack/keystone: Removed OSX files that shouldn't be in git.  https://review.openstack.org/29020:11
*** mfer has quit IRC20:13
openstackgerritA change was merged to openstack/keystone: Documentation cleanups.  https://review.openstack.org/28920:21
*** jakedahn has joined #openstack-dev20:24
openstackgerritDolph Mathews proposed a change to openstack/keystone: Remove more group stuff  https://review.openstack.org/28820:24
openstackgerritA change was merged to openstack/keystone: Remove more group stuff  https://review.openstack.org/28820:27
*** troytoman-away is now known as troytoman20:32
sandywalshany keystone people in the house?20:36
jsavakhi sandy20:36
jsavaki'm a keystone peep20:36
sandywalshhey jsavak ... got a sec for a quick question ?20:36
jsavaksure20:36
sandywalshI think something has changed wrt keystone and novaclient and I'm trying to put my finger on it20:37
sandywalshhere's what I'm seeing20:37
sandywalshwe auth against nova and it, correctly, 305's back to keystone20:37
anotherjessesandywalsh: they are working on rewriting the api now I think20:37
sandywalshah20:37
jsavakyup - we have a working session going on right now20:37
sandywalshso it's busted is a known issue :) ?20:38
anotherjessesandywalsh: recommendation with keystone is to stick with a branch that works until they finish the api redux20:38
anotherjesse(my personal)20:38
jsavakif you pulled recently - that might be causing some issues20:38
sandywalshheh  ... ok, cool. Thought I was going crazy there for a bit, since it was working a little while ago20:38
anotherjessesandywalsh: I can get you the rev we are using for integration20:38
jsavaki'd like to know the issue though just to be sure20:38
sandywalshk20:38
anotherjessesandywalsh: and you can test against our deploy if you want (pm-ing creds)20:38
sandywalshwe get the 305 to keystone correctly x.x.x.x:500120:38
sandywalshbut the POST /tokens gives a 40420:39
anotherjesseare you posting /v2.0/tokens20:39
sandywalshPOST /v2.0/tokens gives bad credentials20:39
sandywalshbut it used to be that it was POST /tokens (and, as per the spec)20:39
jsavakare you including a tenant in the post?20:39
sandywalsh(I hacked the v2.0 part, since that's what I thought was happening)20:39
sandywalshyes20:40
sandywalshlemme get you a novaclient debug paste20:40
anotherjessethe spec says it is just /tokens, but the keystone endpoint should include the version number20:40
jsavakyeah - we are moving tenant id up to root right now and not be part of passwordCredentials20:40
sandywalshah, that could be it20:40
anotherjessejsavak: um - pinging ziad/jorge20:40
anotherjessek20:40
jsavaksure20:40
sandywalshI think also there is some confusion in novaclient re versioning (OS API 1.0/1.1 vs. Keystone auth 1.0/2.0)20:42
anotherjessesandywalsh: how so?20:42
anotherjessesandywalsh: the token generate code has 2.0 vs 1.x paths20:42
anotherjessesandywalsh: depending on what api the endpoint you specify supports20:43
sandywalshanotherjesse, are they meant to refer to the same thing or are there different versioning schemes (auth vs. OS API)?20:43
anotherjesseauth has its own api rev that is COMPLETELY independant of every other api20:44
sandywalshif I hit /v1.1, I'm get v1 auth and not v2 auth20:44
anotherjesseif you hit /v1.1 on what endpoint? nova or keystone?20:44
sandywalshI think it's getting muddled in novaclient20:44
sandywalshnova20:44
anotherjessesandywalsh: I wrote that code - so .. :)20:44
sandywalshI hit nova on 1.1 and the 305 bounces it back to keystone, which should be using 2.0, but it's seeing the 1.1 and using 1.020:45
sandywalsh(heh, if you can follow that :)20:45
anotherjesseso - where do you configure nova to do the 305 bounce?20:45
anotherjesseyes I do20:45
sandywalshit's done in the middleware I believe, so keystone.conf20:45
zns1The 305 was a mistake we made intially in the code anotherjesse started with. It's supposed to be a 401. That has been fixed in the latest code.20:46
anotherjessesandywalsh: if it is redirecting to the v1.x api, then it is what is wrong (the keystone middleware in nova)20:47
sandywalshhere's with POST /v2.0/token and POST /token http://paste.openstack.org/show/2227/20:47
zns1Line of code: https://github.com/openstack/keystone/blob/master/keystone/middleware/auth_token.py#L21420:47
sandywalshignore my PART # debug messages20:47
anotherjessezns1: so it needs to add a auth_Version?20:48
zns1sandywalsh: and this is where the reponse URL gets set: https://github.com/openstack/keystone/blob/master/examples/paste/nova-api-paste.ini#L9320:50
anotherjessezns1: why/when did tenants get removed from authenticate?20:54
anotherjessezns1: so that should be updated to have v2.0/ on it?20:55
*** lts has quit IRC21:02
*** rnorwood has quit IRC21:05
*** clayg_ has left #openstack-dev21:06
zns1anotherjesse: not sure what you mean. Which "tenants" are you looking for?21:07
anotherjessezns1: jsavak was saying you were removing tenant from being passed in the password credentials json ?21:08
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Reworked XSDs and WADL to support auth and access elements.  https://review.openstack.org/29121:13
zns1anotherjesse: yes. Tenant is NOT actually a credential. It's part of your request but not a credential. So we're keeping it in the request, but moving it up a level.21:13
*** amccabe has quit IRC21:13
zns1{21:16
zns1  "auth": {21:16
zns1    "passwordCredentials": {21:16
zns1      "username": "test_user",21:16
zns1      "password": "a86850deb2742ec3cb41518e26aa2d89"21:16
zns1    },21:16
zns1    "tenantId": "1234"21:16
zns1  }21:16
zns1}21:16
anotherjessezns1: k - so not in the uri - but in the json still .. k21:16
anotherjessethx21:16
anotherjessewas afaraid21:16
zns1We considered putting it in the uri like this - POST /tokens/?tenantId=1234 - but that wouldn't allow us to extend requests without breaking the contract. Adding the auth at the root does.21:17
openstackgerritDolph Mathews proposed a change to openstack/keystone: Fixed reference to unassigned variable  https://review.openstack.org/29221:26
*** troytoman is now known as troytoman-away21:26
openstackgerritA change was merged to openstack/keystone: Fixed reference to unassigned variable  https://review.openstack.org/29221:29
*** chipc has quit IRC21:31
*** chipc has joined #openstack-dev21:31
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Updated samples  https://review.openstack.org/29321:35
anotherjessebcwaldon: the comment about ignore me - does that mean it is in the spec?21:36
openstackgerritDolph Mathews proposed a change to openstack/keystone: Removed redundant function from base user api  https://review.openstack.org/29421:36
*** martine has quit IRC21:39
*** AhmedSoliman has joined #openstack-dev21:39
*** mattray1 has joined #openstack-dev21:50
*** jsavak has quit IRC21:50
*** mattray has quit IRC21:53
*** bcwaldon has quit IRC21:57
openstackgerritA change was merged to openstack/keystone: Reworked XSDs and WADL to support auth and access elements.  https://review.openstack.org/29121:58
openstackgerritA change was merged to openstack/keystone: Updated samples  https://review.openstack.org/29322:01
openstackgerritA change was merged to openstack/keystone: Removed redundant function from base user api  https://review.openstack.org/29422:03
*** chipc has quit IRC22:03
*** mattray1 has quit IRC22:03
*** mattray has joined #openstack-dev22:08
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Cleanup service it endpoint catalog.  https://review.openstack.org/29622:28
*** deshantm has quit IRC22:30
*** bsza has quit IRC22:33
*** mattray has quit IRC22:38
*** anotherjesse_ has joined #openstack-dev22:44
*** rnirmal has quit IRC22:46
openstackgerritJorge L. Williams proposed a change to openstack/keystone: Atom links on Token.  https://review.openstack.org/29722:46
*** rnorwood has joined #openstack-dev22:47
*** bsza has joined #openstack-dev22:47
*** anotherjesse has quit IRC22:48
*** anotherjesse_ is now known as anotherjesse22:48
openstackgerritA change was merged to openstack/keystone: Cleanup service it endpoint catalog.  https://review.openstack.org/29622:48
*** bsza3 has joined #openstack-dev22:51
*** bsza has quit IRC22:52
*** bsza3 has quit IRC22:52
*** bsza has joined #openstack-dev22:53
*** zns1 has quit IRC22:53
*** rnorwood has quit IRC22:54
openstackgerritA change was merged to openstack/keystone: Atom links on Token.  https://review.openstack.org/29722:54
*** anotherjesse has quit IRC23:16
*** anotherjesse has joined #openstack-dev23:16
*** bengrue has quit IRC23:22
*** zaitcev has joined #openstack-dev23:24
*** AhmedSoliman has quit IRC23:25
Tusharbcwaldon: I have fixed review comments given by you on my merg prop. Request you to please review it. Thanks.23:34
anotherjessecan someone kick jenkins?23:41
anotherjesseit isn't merging stuff23:41
anotherjessejeblair / mtaylor ^23:41
jeblairanotherjesse: i'll take a look23:43
anotherjessejeblair: thx!23:43
jeblairit looks like the nova-coverage job was spinning (ie, the tests were taking forever)23:49
*** zns has joined #openstack-dev23:51
mtaylorjeblair: was it still trying to do soren's thing?23:55
jeblairhttps://jenkins.openstack.org/job/nova-coverage/1084/console23:56
jeblairunfortunately, since we only have one executor, if any nova job blocks now, they are all stuck.23:57
*** bsza has quit IRC23:57
mtaylorbleh23:57
jeblairif we have a job with a high probability of getting stuck, then maybe we should chance the race condition and run 2 executors23:57
mtaylorjeblair: we could also spin up more nova nodes23:58
jeblairyes23:58
mtaylorjeblair: the puppet module matches nova-1. as well, yeah?23:59
jeblairyep23:59

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