*** deshantm has quit IRC | 00:04 | |
*** jakedahn has quit IRC | 00:04 | |
openstackjenkins | Project 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 logging | 00:14 |
openstackjenkins | ��* Changes service.serve to use Launcher | 00:14 |
openstackjenkins | ��* Changes service.wait to actually wait for all the services to exit | 00:14 |
openstackjenkins | ��* Changes nova-api to explicitly load flags and logging and use service.serve | 00:14 |
openstackjenkins | * Fixes the annoying IOError when /etc/nova/nova.conf doesn't exist | 00:14 |
openstackjenkins | * Tarmac: Fixes issue where ServersXMLSerializer was missing a method for update actions | 00:14 |
openstackjenkins | Project 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 logging | 00:26 |
openstackjenkins | ��* Changes service.serve to use Launcher | 00:26 |
openstackjenkins | ��* Changes service.wait to actually wait for all the services to exit | 00:26 |
openstackjenkins | ��* Changes nova-api to explicitly load flags and logging and use service.serve | 00:26 |
openstackjenkins | * Fixes the annoying IOError when /etc/nova/nova.conf doesn't exist | 00:26 |
openstackgerrit | A change was merged to openstack/keystone: Refactored manage.py to be both testable and useful for testing https://review.openstack.org/276 | 00:31 |
openstackgerrit | Verification of a change to openstack/keystone failed: Cleaned up come issues with python2.6. https://review.openstack.org/270 | 00:31 |
openstackgerrit | Verification of a change to openstack/keystone failed: Changes to allow test to work on python 2.6.*. https://review.openstack.org/271 | 00:31 |
openstackjenkins | Project burrow build #33: SUCCESS in 11 sec: https://jenkins.openstack.org/job/burrow/33/ | 00:31 |
openstackjenkins | Tarmac: Backend module cleanup. | 00:31 |
*** mfer has quit IRC | 00:42 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Cleaned up come issues with python2.6. https://review.openstack.org/270 | 00:44 |
openstackgerrit | A change was merged to openstack/keystone: Cleaned up come issues with python2.6. https://review.openstack.org/270 | 00:47 |
*** bengrue has quit IRC | 00:50 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Changes to allow test to work on python 2.6.*. https://review.openstack.org/271 | 01:02 |
thickskin | I see | 01:03 |
zns | thickskin: was that part of a conversation I missed? | 01:04 |
openstackgerrit | A change was merged to openstack/keystone: hanges to allow test to work on python 2.6.*. https://review.openstack.org/271 | 01:04 |
thickskin | zns:You don't have to worry about it. | 01:06 |
*** mfer has joined #openstack-dev | 01:10 | |
*** mfer has quit IRC | 01:12 | |
*** zns has quit IRC | 01:13 | |
*** rods has quit IRC | 01:15 | |
*** Tushar has quit IRC | 01:17 | |
*** bcwaldon has quit IRC | 01:19 | |
*** mfer has joined #openstack-dev | 01:37 | |
*** mfer has quit IRC | 01:40 | |
*** deshantm has joined #openstack-dev | 01:46 | |
*** bengrue has joined #openstack-dev | 01:46 | |
*** lorin1 has joined #openstack-dev | 01:48 | |
lorin1 | ttx: I think you were asking me a question the other day, as I was on my way out... | 01:49 |
*** RobertLaptop has quit IRC | 01:55 | |
*** RobertLaptop has joined #openstack-dev | 02:07 | |
openstackjenkins | Project nova build #1,271: SUCCESS in 3 min 39 sec: https://jenkins.openstack.org/job/nova/1271/ | 02:22 |
openstackjenkins | Tarmac: 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 |
uvirtbot | Launchpad bug 828429 in nova "nova-dhcpbridge: interface causes KeyError - bug added in v.1449" [High,In progress] https://launchpad.net/bugs/828429 | 02:22 |
*** bengrue has quit IRC | 02:29 | |
*** bengrue has joined #openstack-dev | 02:31 | |
*** RobertLaptop has quit IRC | 02:33 | |
*** RobertLaptop has joined #openstack-dev | 02:34 | |
*** lorin1 has quit IRC | 02:59 | |
*** martine has joined #openstack-dev | 03:01 | |
openstackgerrit | Justin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n https://review.openstack.org/277 | 03:16 |
openstackgerrit | Justin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n https://review.openstack.org/277 | 03:25 |
*** zul has joined #openstack-dev | 04:18 | |
*** deshantm has quit IRC | 04:22 | |
*** dolphm_ has quit IRC | 05:31 | |
*** martine_ has joined #openstack-dev | 05:49 | |
*** martine has quit IRC | 05:51 | |
*** martine_ has quit IRC | 05:53 | |
openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Update rfc.sh. https://review.openstack.org/273 | 06:24 |
openstackgerrit | A change was merged to openstack/openstack-ci: Update rfc.sh to assert divergence. https://review.openstack.org/274 | 06:25 |
openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Update rfc.sh to assert divergence. https://review.openstack.org/275 | 06:45 |
*** reidrac has joined #openstack-dev | 07:02 | |
openstackgerrit | Monty Taylor proposed a change to openstack/keystone: Added more missing files to MANIFEST.in. https://review.openstack.org/278 | 07:06 |
*** timr has quit IRC | 07:34 | |
*** timr has joined #openstack-dev | 08:10 | |
ttx | soren: looks like https://code.launchpad.net/~midokura/nova/nova-quantum-vifid/+merge/71990 just needs a reapprove | 08:27 |
*** reidrac has quit IRC | 08:30 | |
* soren looks | 08:45 | |
ttx | was approved, but caught in a python-migrate race | 08:45 |
soren | ttx: Yup. Approved. | 08:46 |
ttx | thx | 08:46 |
zykes- | what version of .py are currently spported ? | 08:47 |
openstackjenkins | Project nova build #1,272: SUCCESS in 3 min 23 sec: https://jenkins.openstack.org/job/nova/1272/ | 09:00 |
openstackjenkins | Tarmac: 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-dev | 09:05 | |
ttx | soren: could you have a quick look at https://bugs.launchpad.net/nova/+bug/828841 ? | 09:20 |
uvirtbot | Launchpad 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 |
soren | zykes-: 2.6 and onwards. | 09:22 |
soren | er.. | 09:22 |
ttx | 2.x with x>6 ? | 09:22 |
soren | zykes-: Sorry, 2.X for X >= 6. | 09:22 |
*** YorikSar has quit IRC | 09:33 | |
soren | ttx: Do you know if there's a way to make pep8 ignore a line? | 10:05 |
soren | ttx: I have a line that needs to be long. | 10:05 |
soren | hmmm... or do I? | 10:06 |
*** reidrac has joined #openstack-dev | 10:31 | |
*** lorin1 has joined #openstack-dev | 10:54 | |
*** lorin1 has quit IRC | 10:54 | |
*** rods has joined #openstack-dev | 11:01 | |
*** markvoelker has joined #openstack-dev | 11:06 | |
*** bsza has joined #openstack-dev | 11:09 | |
*** timr has quit IRC | 11:14 | |
*** anp has quit IRC | 11:33 | |
*** mfer has joined #openstack-dev | 11:43 | |
*** anp has joined #openstack-dev | 11:46 | |
*** anp has quit IRC | 11:54 | |
*** anp has joined #openstack-dev | 12:01 | |
*** lorin1 has joined #openstack-dev | 12:03 | |
*** lorin1 has left #openstack-dev | 12:03 | |
*** lts has joined #openstack-dev | 12:43 | |
*** martine_ has joined #openstack-dev | 12:43 | |
*** martine_ has quit IRC | 12:48 | |
*** joesavak has joined #openstack-dev | 12:50 | |
*** bourke_ has joined #openstack-dev | 12: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-dev | 12:59 | |
*** zns has joined #openstack-dev | 13:07 | |
*** mfer has quit IRC | 13:12 | |
*** mfer has joined #openstack-dev | 13:12 | |
*** martine has joined #openstack-dev | 13:20 | |
*** bcwaldon has joined #openstack-dev | 13:21 | |
*** YorikSar has joined #openstack-dev | 13:25 | |
*** mattray has joined #openstack-dev | 13:35 | |
*** anp has quit IRC | 13:35 | |
*** bcwaldon_ has joined #openstack-dev | 13:37 | |
*** bcwaldon_ has quit IRC | 13:37 | |
*** bcwaldon has quit IRC | 13:39 | |
annegentle | mtaylor: around? | 13:42 |
*** chipc has quit IRC | 13:45 | |
annegentle | I need help troubleshooting a build, anyone else got some Jenkins knowhow? | 13:45 |
*** chipc has joined #openstack-dev | 13:47 | |
*** mnour has quit IRC | 13:48 | |
*** bcwaldon has joined #openstack-dev | 13:48 | |
*** amccabe has joined #openstack-dev | 13:51 | |
*** zns has quit IRC | 14:00 | |
*** zns has joined #openstack-dev | 14:00 | |
*** bourke_ has left #openstack-dev | 14:03 | |
*** anp has joined #openstack-dev | 14:05 | |
mtaylor | annegentle: whazzup? | 14:07 |
*** mattray has quit IRC | 14:26 | |
*** mattray has joined #openstack-dev | 14:28 | |
*** clayg_ has joined #openstack-dev | 14:30 | |
*** nickon has quit IRC | 14:34 | |
*** bcwaldon has quit IRC | 14:39 | |
*** bcwaldon_ has joined #openstack-dev | 14:39 | |
*** anp has quit IRC | 14:39 | |
*** nmistry has joined #openstack-dev | 14:48 | |
*** zul has quit IRC | 14:52 | |
*** rnirmal has joined #openstack-dev | 14:54 | |
*** rnirmal_ has joined #openstack-dev | 14:55 | |
*** rnirmal has quit IRC | 14:59 | |
*** rnirmal_ is now known as rnirmal | 14:59 | |
*** reidrac has left #openstack-dev | 15:05 | |
*** dragondm has joined #openstack-dev | 15:08 | |
*** heckj has joined #openstack-dev | 15:14 | |
*** nmistry has quit IRC | 15:21 | |
ttx | vishy: ping | 15:38 |
*** lorin1 has joined #openstack-dev | 15:43 | |
*** vladimir3p has joined #openstack-dev | 15:43 | |
mtaylor | dolphm: ping | 15:46 |
ttx | vishy: see pm | 15:47 |
*** zns has quit IRC | 15:51 | |
dolphm | mtaylor: > 1s | 15:52 |
mtaylor | dolphm: no rush - just pinging you to approve my change :) | 15:53 |
notmyname | mtaylor: do you use any of the vcs stuff found in setup.py for packaging? | 15:53 |
mtaylor | notmyname: no | 15:53 |
dolphm | mtaylor: approved | 15:53 |
notmyname | mtaylor: we don't use it at all (AFAIK), and since it's bzr-specific, I was hopign to remove it | 15:54 |
mtaylor | notmyname: ++ | 15:54 |
notmyname | mtaylor: cool | 15:54 |
openstackgerrit | A change was merged to openstack/keystone: Added more missing files to MANIFEST.in. https://review.openstack.org/278 | 15:55 |
*** rods has quit IRC | 15:58 | |
*** lorin1 has left #openstack-dev | 15:58 | |
*** lorin1 has joined #openstack-dev | 15:58 | |
*** zns has joined #openstack-dev | 15:59 | |
*** mattray has quit IRC | 16:00 | |
*** rods has joined #openstack-dev | 16:04 | |
mtaylor | dolphm: thanks! | 16:08 |
dolphm | mtaylor: np | 16:08 |
mtaylor | dolphm: I think that's the last of the fixes we need pre-packaging | 16:08 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Fix Bug 825344 - Only set project watches once https://review.openstack.org/279 | 16:11 |
uvirtbot | Launchpad bug 825344 in openstack-ci "project watches incorrectly added to gerrit" [Medium,New] https://launchpad.net/bugs/825344 | 16:11 |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: XSD & sample updates https://review.openstack.org/280 | 16:18 |
*** chomping has joined #openstack-dev | 16:26 | |
vishy | soren: https://bugs.launchpad.net/nova/+bug/829609 So it looks like we are supposed to allow everything if just source group is specified | 16:27 |
uvirtbot | Launchpad bug 829609 in ensemble "EC2 compatibility describe security group returns erroneous value for group ip permissions" [Critical,New] | 16:27 |
*** bengrue has quit IRC | 16:45 | |
*** bengrue has joined #openstack-dev | 16:46 | |
*** Tushar has joined #openstack-dev | 16:48 | |
vladimir3p | vishy: ping. any update on volume types status? | 16:56 |
dolphm | mtaylor: 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 |
vishy | vladimir3p: the volume cleanup is taking a lot more than I expected | 16:58 |
vishy | vladimir3p: if you want to keep going on types/extra_types/meta that would be awesome | 16:58 |
vladimir3p | ok, I will take care of this part, but a bit later today | 16:58 |
Tushar | bcwaldon_: 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 |
Tushar | bawaldon_:OK | 17:00 |
*** bcwaldon_ has quit IRC | 17:00 | |
*** bengrue has quit IRC | 17:07 | |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Will this work? https://review.openstack.org/281 | 17:07 |
*** bcwaldon has joined #openstack-dev | 17:12 | |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Will this work? https://review.openstack.org/281 | 17:13 |
bcwaldon | Tushar: what can I help you with? | 17:13 |
Tushar | bcwaldon: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/68292 | 17:13 |
bcwaldon | sure | 17:13 |
Tushar | bcwaldon: line #747 | 17:14 |
bcwaldon | okay, I'm with you | 17:14 |
Tushar | bcwaldon: I also wanted to aviod RPC call but it's simply inevitable | 17:14 |
Tushar | bcwaldon: *avoid | 17:14 |
*** dprince has joined #openstack-dev | 17:14 | |
bcwaldon | Tushar: okay, can you help me understand why? | 17:15 |
bcwaldon | Tushar: I haven't spent much time in the network manager | 17:15 |
Tushar | bcwaldon: OK | 17:16 |
Tushar | bcwaldon: #line 520 | 17:16 |
bcwaldon | Tushar: yep | 17:16 |
Tushar | bcwaldon: Heere I want to assign specific address instead of getting available address from the network pool | 17:18 |
bcwaldon | Tushar: 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 directly | 17:19 |
dprince | mtaylor: you around? | 17:20 |
Tushar | bcwaldon: Ok, I got you. | 17:21 |
vishy | btw, in case anyone decides they want to try it, nova does not run in pypy :( | 17:22 |
Tushar | bcwaldon: 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 different | 17:22 |
dprince | mtaylor: 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 |
bcwaldon | Tushar: let me look real fast, one sec | 17:23 |
dprince | Also. 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 |
bcwaldon | Tushar: it looks like fixed_ip_associate is a simpler (dumber) version of fixed_ip_associate_by_address | 17:27 |
bcwaldon | Tushar: any way you can replace fixed_ip_associate with your new function? | 17:27 |
Tushar | bcwaldon: 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 |
bcwaldon | Tushar: I should be thanking you for being so patient waiting for me to review it! | 17:31 |
bcwaldon | Tushar: Hopefully it won't take so long for me to do it next time :) | 17:32 |
Tushar | bcwaldon: Thanks | 17:32 |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: XSD & sample updates https://review.openstack.org/280 | 17:34 |
*** deshantm has joined #openstack-dev | 17:38 | |
*** anotherjesse has joined #openstack-dev | 17:46 | |
anotherjesse | blamar_ / bcwaldon time to talk about https://code.launchpad.net/~cloudbuilders/nova/os-floating-ips-redux/+merge/71922 | 17:49 |
anotherjesse | ? | 17:49 |
bcwaldon | anotherjesse: yessir | 17:49 |
blamar_ | anotherjesse, yup yup yup | 17:49 |
anotherjesse | to the question about where the actions go - we put them in the contrib module mostly because it kept the code related to floating ips seperated | 17:50 |
bcwaldon | anotherjesse: I don't follow, can you explain? Not sure I know what question you're answering | 17:51 |
anotherjesse | "I think it makes even more sense to have a disassociate/associate action on a server" | 17:51 |
bcwaldon | Okay, well you could keep the code in contrib. See jk0's rescue extension | 17:52 |
anotherjesse | bcwaldon: k! | 17:52 |
anotherjesse | kinda weird that rescue is an extension ;) | 17:53 |
bcwaldon | yeah, but all admin functionality is going to move into an extension | 17:53 |
bcwaldon | anything that isn't in the spec *has* to be an extension | 17:53 |
bcwaldon | at least that's the hole we've dug ourselves into at this point | 17:53 |
jk0 | anotherjesse: I just do what product tells me :) | 17:53 |
anotherjesse | jk0: so that's the secret ;) | 17:54 |
anotherjesse | bcwaldon - this is just a refactor | 17:54 |
anotherjesse | I'll look at the rescue though | 17:54 |
anotherjesse | bcwaldon: interesting - so - personally I don't want to get into a battle about what proper restful structure | 17:55 |
blamar_ | anotherjesse: not restful, just consistent with other current features? | 17:55 |
bcwaldon | yeah, I wouldn't call /action restful ;) | 17:55 |
*** mgius has joined #openstack-dev | 17:57 | |
anotherjesse | blamar_: / bcwaldon moving to to /servers/:id/associate/(floating_ip?) (post?) (or is floating ip a param in the body?) | 17:57 |
bcwaldon | I would go with curl -X POST -d '{"associate": {"address":"192.168.0.4"}}' http://localhost:8774/v1.1/servers/12/action | 17:58 |
blamar_ | s/associate/associate_floating_ip | 17:58 |
blamar_ | or leave it associate and change "address" to "floating_ip" | 17:59 |
blamar_ | tons of ways! :\ | 17:59 |
bcwaldon | just have to think about future usage of the 'associate' action | 17:59 |
anotherjesse | the uri path is actually v1.1/:tenant_id/servers/:server_id/action ?? | 18:00 |
bcwaldon | yep | 18:00 |
anotherjesse | the word action? | 18:00 |
bcwaldon | thats how resize, confirmResize, revertResize, changePassword, reboot work | 18:00 |
bcwaldon | yes | 18:00 |
anotherjesse | rescue/reboot don't do that though - it uses rescue/unrescue instead of action? | 18:00 |
bcwaldon | reboot does | 18:01 |
bcwaldon | rescue is in the admin api | 18:01 |
bcwaldon | the admin api is implemented as a token in the uri, not through an 'action' | 18:01 |
bcwaldon | but now we have the os-rescue extension which adds 'rescue' and 'unrescue' actions | 18:01 |
anotherjesse | k | 18:02 |
anotherjesse | I am reading the exts.ActionExtension code now | 18:02 |
anotherjesse | thanks for clearing up :) | 18:02 |
bcwaldon | that's exactly what you need | 18:02 |
bcwaldon | no problem, it will be changing again | 18:02 |
anotherjesse | bcwaldon: while I'm here - about the keypair integration merge | 18:02 |
bcwaldon | yep | 18:03 |
anotherjesse | I am leaning towards removing the automatic adding of the first keypair in the project | 18:03 |
bcwaldon | anotherjesse: yeah, I think I'm for that. I just wanted to make sure we were all on the same page. I like being explicit | 18:04 |
anotherjesse | cool - then if you give me an approve we can merge that sucker :) | 18:04 |
bcwaldon | I'll approve, but tarmac is borked | 18:04 |
bcwaldon | probably won't merge for a while | 18:05 |
anotherjesse | no landing for you … 4 hours | 18:05 |
bcwaldon | exactly | 18:05 |
*** lorin1 has quit IRC | 18:06 | |
bcwaldon | anotherjesse: dprince actually has something he wanted to bring up about that branch | 18:08 |
bcwaldon | anotherjesse: he'll be here in a minute | 18:08 |
anotherjesse | cool | 18:08 |
dprince | you guys are winksters today ;) | 18:10 |
anotherjesse | not getting much sleep ;p | 18:10 |
dprince | anotherjesse: So. Would it be a good idea to leave the old keypair behaviour in place for the v1.0 API? | 18:11 |
anotherjesse | dprince: the idea of grabbing the first keypair from the project? | 18:12 |
dprince | anotherjesse: and require it to be specified when using the v1.1 API, via the extension. | 18:12 |
dprince | anotherjesse: Yes. | 18:12 |
anotherjesse | dprince: if we do that should we make it be the first keypair for the user? | 18:12 |
dprince | anotherjesse: Removing that behavoiur will break some tests suites. | 18:12 |
*** dolphm has quit IRC | 18:13 | |
dprince | anotherjesse: Yes. | 18:13 |
anotherjesse | dprince: deal | 18:13 |
soren | dprince: 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 |
soren | s/, it/, if/ | 18:13 |
anotherjesse | dprince: still new to os api layer - did my change remove from both versions? | 18:13 |
bcwaldon | anotherjesse: it did | 18:14 |
anotherjesse | bcwaldon: any pointers on the clean way to check versions at the location I changed it? | 18:14 |
bcwaldon | anotherjesse: figure out how to add it in the ControllerV11 class in api.openstack.servers | 18:16 |
dprince | anotherjesse: 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 |
bcwaldon | that'll work | 18:18 |
anotherjesse | k - will make a stab at that .. thanks - (wink) | 18:18 |
jeblair | dprince: it looks like the nova build slave is swapping. | 18:18 |
jeblair | i'm going to collect some data, then probably expand the image and reboot | 18:19 |
dprince | jeblair: Like I said. If I had access I'd reboot it. | 18:19 |
bcwaldon | like a boss | 18:19 |
dprince | jeblair: making it bigger is good too. | 18:19 |
dprince | jeblair: 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 |
dprince | jeblair: Both of those can do a run_tests.sh -N and that seems worrisome. | 18:20 |
*** bengrue has joined #openstack-dev | 18:21 | |
jeblair | i 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 |
jeblair | is your concern resource usage, or something more like resource collision? | 18:22 |
jeblair | if 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 IRC | 18:23 | |
*** mattray has joined #openstack-dev | 18:27 | |
*** markvoelker has quit IRC | 18:27 | |
dprince | jeblair: My concern is resource collision between the various nova jobs. | 18:27 |
jeblair | do you know of something specific that collides? | 18:28 |
dprince | jeblair: run_tests -N does a mix of unit and functional tests. It actually runs some of the API code on ports, etc. | 18:28 |
bcwaldon | vishy: When you say 'we just need to change noauth...', you mean 'vishy needs to change noauth...', right? Just want to be clear | 18:28 |
jeblair | ok. that does sound problematic. | 18:29 |
*** alekibango has joined #openstack-dev | 18:30 | |
*** uvirtbot` has joined #openstack-dev | 18:30 | |
*** lorin11 has joined #openstack-dev | 18:31 | |
*** colinnich_ has joined #openstack-dev | 18:31 | |
jeblair | dprince: i've set the # of executors to 1 for now. and the resize is in progress. | 18:31 |
*** lorin11 has left #openstack-dev | 18:31 | |
vishy | bcwaldon: depends who gets merged first... | 18:31 |
vishy | :) | 18:31 |
dprince | jeblair: Thanks man. Sounds good. | 18:31 |
bcwaldon | vishy: I see...the race is on! | 18:31 |
*** comstud_ has joined #openstack-dev | 18:34 | |
*** jsavak has joined #openstack-dev | 18:34 | |
dprince | jeblair: 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-dev | 18:35 | |
jeblair | i know that error occured when launchpad changed their connection timeouts recently | 18:35 |
jeblair | i believe tarmac was using bzrlib which held its connection to lp open during its work | 18:36 |
jeblair | which can take a while | 18:36 |
*** amccabe_ has joined #openstack-dev | 18:36 | |
jeblair | nonetheless, if the test suite has a race condition, that will cause that error too, as you suggest | 18:36 |
jeblair | (likely regardless of launchpad's timeout) | 18:36 |
dprince | jeblair: 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 |
jeblair | of 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 IRC | 18:37 | |
*** tr3buche` has joined #openstack-dev | 18:37 | |
*** dragondm has quit IRC | 18:38 | |
*** amccabe has quit IRC | 18:38 | |
*** mnaser has quit IRC | 18:38 | |
*** sandywalsh has quit IRC | 18:38 | |
*** uvirtbot has quit IRC | 18:38 | |
*** jk0 has quit IRC | 18:38 | |
*** comstud has quit IRC | 18:38 | |
*** smoser has quit IRC | 18:38 | |
*** colinnich has quit IRC | 18:38 | |
*** tr3buchet has quit IRC | 18:38 | |
*** dabo has quit IRC | 18:38 | |
*** kpepple has quit IRC | 18:38 | |
*** smoser has joined #openstack-dev | 18:39 | |
*** mwhooker has joined #openstack-dev | 18:39 | |
vladimir3p | vishy: 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 |
vishy | vladimir3p: there are issues with attaching volumes with older kernels | 18:42 |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Wrong common namespace. https://review.openstack.org/282 | 18:42 |
vladimir3p | vishy: even if we will enable it for bridge? (I'm not talking about disks/devices) | 18:43 |
vishy | the issue is that virtio net devices will fail if a block device is hot attached | 18:43 |
vishy | with older kernels | 18:43 |
*** sandywalsh has joined #openstack-dev | 18:46 | |
vladimir3p | vishy: hmm... so if we will enable virtio for bridged NIC interfaces it will cause problems when block devices will be hot-added ... ? | 18:46 |
vladimir3p | vishy: seems weird .. | 18:46 |
openstackgerrit | Justin Shepherd proposed a change to openstack/glance: Addresses glance/+spec/i18n https://review.openstack.org/277 | 18:50 |
openstackjenkins | Project nova build #1,273: SUCCESS in 3 min 41 sec: https://jenkins.openstack.org/job/nova/1273/ | 18:53 |
openstackjenkins | Tarmac: Removes the incorrect hard-coded filter path. | 18:53 |
*** anotherjesse_ has joined #openstack-dev | 18:54 | |
*** anotherjesse has quit IRC | 18:55 | |
*** anotherjesse_ is now known as anotherjesse | 18:55 | |
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci: Update rfc.sh to add change-id to commit message. https://review.openstack.org/283 | 18:57 |
*** mattfarina has joined #openstack-dev | 18:58 | |
*** mfer has quit IRC | 18:58 | |
*** mattfarina is now known as mfer | 18:58 | |
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci-puppet: Update rfc.sh to add change-id to commit message. https://review.openstack.org/284 | 18:58 |
openstackgerrit | James E. Blair proposed a change to openstack/keystone: Add rfc.sh for git review. https://review.openstack.org/285 | 18:59 |
openstackgerrit | James E. Blair proposed a change to openstack/glance: Add rfc.sh for git review. https://review.openstack.org/286 | 19:00 |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Banished .DS_Store https://review.openstack.org/287 | 19:01 |
*** tr3buche` is now known as tr3buchet | 19:02 | |
tr3buchet | test | 19:03 |
tr3buchet | weird | 19:03 |
openstackgerrit | A change was merged to openstack/keystone: Add rfc.sh for git review. https://review.openstack.org/285 | 19:05 |
*** rnirmal has joined #openstack-dev | 19:05 | |
vishy | vladimir3p: yup that is what I'm saying | 19:05 |
bengrue | vladimir3p: 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 |
jeblair | the 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 |
vishy | vladimir3p: if you hot-add a block device the virio kernel model crashes and you lose networking | 19:05 |
*** zns1 has joined #openstack-dev | 19:06 | |
openstackgerrit | A change was merged to openstack/keystone: Wrong common namespace. https://review.openstack.org/282 | 19:07 |
*** zns has quit IRC | 19:08 | |
*** amccabe_ has quit IRC | 19:08 | |
vladimir3p | vishy: I was thinking about enabling it for network ifaces, but we can keep it disabled for block | 19:13 |
vishy | vladimir3p: don't think that matters | 19:13 |
vishy | vladimir3p: virito net breaks when you hot add | 19:13 |
vishy | vladimir3p: IIRC i tested hot adding a non-virtio block device and it still crashes | 19:14 |
vladimir3p | vishy: :-) | 19:14 |
vishy | vladimir3p: although I'm not totally sure | 19:14 |
vishy | vladimir3p: anyway it doesn't matter if you don't need to support lucid /centos 5.5 guests | 19:14 |
vladimir3p | vishy: at least we can try to make it configurable. | 19:14 |
vishy | vladimir3p: +1 there is even a bug that has been around for about 6 mo+ | 19:14 |
vladimir3p | vishy: :-) ok, will try to find it | 19:15 |
vishy | https://bugs.launchpad.net/nova/+bug/702040 | 19:15 |
uvirtbot` | Launchpad bug 702040 in nova "Slow network performance since Virtio net drivers are always disabled" [Wishlist,Confirmed] | 19:15 |
bengrue | jeblair: where's the nova build slave? | 19:16 |
bengrue | vladimir3p: ping, re: my earlier question? | 19:17 |
*** amccabe has joined #openstack-dev | 19:17 | |
vladimir3p | bengrue: sorry, I missed it | 19:17 |
vladimir3p | bengrue: pls take a look at... | 19:17 |
vladimir3p | bengrue: lp:~vladimir.p/nova/volume_type_extradata | 19:18 |
vladimir3p | bengrue: it has all required DB schemas, migration and DB APIs. Some tests as well | 19:18 |
bengrue | okay! | 19:19 |
bengrue | But, it's cool if I write the APIs for you, right? | 19:19 |
vladimir3p | bengrue: I've not started any API work, so you are welcome | 19:19 |
bengrue | you've not started that work? | 19:19 |
bengrue | excellent. | 19:19 |
vladimir3p | bengrue: let's stay in touch & updated on progress | 19:19 |
bengrue | vladimir3p: cool, what's your preferred method of communication? | 19:20 |
vladimir3p | bengue: any... email might be better as I could miss IRC (vladimir@zadarastorage.com) | 19:22 |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Remove more group stuff https://review.openstack.org/288 | 19:29 |
*** mattray has quit IRC | 19:33 | |
*** mattray has joined #openstack-dev | 19:34 | |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Documentation cleanups. https://review.openstack.org/289 | 19:38 |
*** anotherjesse has quit IRC | 19:40 | |
*** anotherjesse has joined #openstack-dev | 19:42 | |
*** mattray has quit IRC | 19:45 | |
*** anotherjesse has quit IRC | 19:46 | |
*** ohnoimdead has joined #openstack-dev | 19:48 | |
*** anotherjesse has joined #openstack-dev | 19:48 | |
vladimir3p | folks, 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-dev | 19:49 | |
bcwaldon | vladimir3p: upgrade glance | 19:50 |
bcwaldon | vladimir3p: on the node you are getting the error on | 19:50 |
vladimir3p | bcwaldon: hmm... but glance is running on a different server | 19:50 |
dprince | make sure you upgrade the glance paste configs in your config file too. | 19:50 |
bcwaldon | it's glance client that is causing the problem | 19:50 |
vladimir3p | bcwaldon: ok, thanks will do it. | 19:51 |
*** anotherjesse has quit IRC | 19:57 | |
*** anotherjesse has joined #openstack-dev | 19:58 | |
*** rnorwood has joined #openstack-dev | 19:59 | |
*** dprince has quit IRC | 20:01 | |
vladimir3p | bcwaldon: it works, thanks | 20:03 |
bcwaldon | vladimir3p: cool, glad I could help | 20:03 |
openstackgerrit | Monty Taylor proposed a change to openstack/keystone: Removed OSX files that shouldn't be in git. https://review.openstack.org/290 | 20:05 |
openstackgerrit | A change was merged to openstack/keystone: XSD & sample updates https://review.openstack.org/280 | 20:06 |
openstackgerrit | A change was merged to openstack/keystone: Banished .DS_Store https://review.openstack.org/287 | 20:08 |
openstackgerrit | A change was merged to openstack/keystone: Removed OSX files that shouldn't be in git. https://review.openstack.org/290 | 20:11 |
*** mfer has quit IRC | 20:13 | |
openstackgerrit | A change was merged to openstack/keystone: Documentation cleanups. https://review.openstack.org/289 | 20:21 |
*** jakedahn has joined #openstack-dev | 20:24 | |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Remove more group stuff https://review.openstack.org/288 | 20:24 |
openstackgerrit | A change was merged to openstack/keystone: Remove more group stuff https://review.openstack.org/288 | 20:27 |
*** troytoman-away is now known as troytoman | 20:32 | |
sandywalsh | any keystone people in the house? | 20:36 |
jsavak | hi sandy | 20:36 |
jsavak | i'm a keystone peep | 20:36 |
sandywalsh | hey jsavak ... got a sec for a quick question ? | 20:36 |
jsavak | sure | 20:36 |
sandywalsh | I think something has changed wrt keystone and novaclient and I'm trying to put my finger on it | 20:37 |
sandywalsh | here's what I'm seeing | 20:37 |
sandywalsh | we auth against nova and it, correctly, 305's back to keystone | 20:37 |
anotherjesse | sandywalsh: they are working on rewriting the api now I think | 20:37 |
sandywalsh | ah | 20:37 |
jsavak | yup - we have a working session going on right now | 20:37 |
sandywalsh | so it's busted is a known issue :) ? | 20:38 |
anotherjesse | sandywalsh: recommendation with keystone is to stick with a branch that works until they finish the api redux | 20:38 |
anotherjesse | (my personal) | 20:38 |
jsavak | if you pulled recently - that might be causing some issues | 20:38 |
sandywalsh | heh ... ok, cool. Thought I was going crazy there for a bit, since it was working a little while ago | 20:38 |
anotherjesse | sandywalsh: I can get you the rev we are using for integration | 20:38 |
jsavak | i'd like to know the issue though just to be sure | 20:38 |
sandywalsh | k | 20:38 |
anotherjesse | sandywalsh: and you can test against our deploy if you want (pm-ing creds) | 20:38 |
sandywalsh | we get the 305 to keystone correctly x.x.x.x:5001 | 20:38 |
sandywalsh | but the POST /tokens gives a 404 | 20:39 |
anotherjesse | are you posting /v2.0/tokens | 20:39 |
sandywalsh | POST /v2.0/tokens gives bad credentials | 20:39 |
sandywalsh | but it used to be that it was POST /tokens (and, as per the spec) | 20:39 |
jsavak | are 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 |
sandywalsh | yes | 20:40 |
sandywalsh | lemme get you a novaclient debug paste | 20:40 |
anotherjesse | the spec says it is just /tokens, but the keystone endpoint should include the version number | 20:40 |
jsavak | yeah - we are moving tenant id up to root right now and not be part of passwordCredentials | 20:40 |
sandywalsh | ah, that could be it | 20:40 |
anotherjesse | jsavak: um - pinging ziad/jorge | 20:40 |
anotherjesse | k | 20:40 |
jsavak | sure | 20:40 |
sandywalsh | I 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 |
anotherjesse | sandywalsh: how so? | 20:42 |
anotherjesse | sandywalsh: the token generate code has 2.0 vs 1.x paths | 20:42 |
anotherjesse | sandywalsh: depending on what api the endpoint you specify supports | 20:43 |
sandywalsh | anotherjesse, are they meant to refer to the same thing or are there different versioning schemes (auth vs. OS API)? | 20:43 |
anotherjesse | auth has its own api rev that is COMPLETELY independant of every other api | 20:44 |
sandywalsh | if I hit /v1.1, I'm get v1 auth and not v2 auth | 20:44 |
anotherjesse | if you hit /v1.1 on what endpoint? nova or keystone? | 20:44 |
sandywalsh | I think it's getting muddled in novaclient | 20:44 |
sandywalsh | nova | 20:44 |
anotherjesse | sandywalsh: I wrote that code - so .. :) | 20:44 |
sandywalsh | I 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.0 | 20:45 |
sandywalsh | (heh, if you can follow that :) | 20:45 |
anotherjesse | so - where do you configure nova to do the 305 bounce? | 20:45 |
anotherjesse | yes I do | 20:45 |
sandywalsh | it's done in the middleware I believe, so keystone.conf | 20:45 |
zns1 | The 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 |
anotherjesse | sandywalsh: if it is redirecting to the v1.x api, then it is what is wrong (the keystone middleware in nova) | 20:47 |
sandywalsh | here's with POST /v2.0/token and POST /token http://paste.openstack.org/show/2227/ | 20:47 |
zns1 | Line of code: https://github.com/openstack/keystone/blob/master/keystone/middleware/auth_token.py#L214 | 20:47 |
sandywalsh | ignore my PART # debug messages | 20:47 |
anotherjesse | zns1: so it needs to add a auth_Version? | 20:48 |
zns1 | sandywalsh: and this is where the reponse URL gets set: https://github.com/openstack/keystone/blob/master/examples/paste/nova-api-paste.ini#L93 | 20:50 |
anotherjesse | zns1: why/when did tenants get removed from authenticate? | 20:54 |
anotherjesse | zns1: so that should be updated to have v2.0/ on it? | 20:55 |
*** lts has quit IRC | 21:02 | |
*** rnorwood has quit IRC | 21:05 | |
*** clayg_ has left #openstack-dev | 21:06 | |
zns1 | anotherjesse: not sure what you mean. Which "tenants" are you looking for? | 21:07 |
anotherjesse | zns1: jsavak was saying you were removing tenant from being passed in the password credentials json ? | 21:08 |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Reworked XSDs and WADL to support auth and access elements. https://review.openstack.org/291 | 21:13 |
zns1 | anotherjesse: 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 IRC | 21: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 |
anotherjesse | zns1: k - so not in the uri - but in the json still .. k | 21:16 |
anotherjesse | thx | 21:16 |
anotherjesse | was afaraid | 21:16 |
zns1 | We 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 |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Fixed reference to unassigned variable https://review.openstack.org/292 | 21:26 |
*** troytoman is now known as troytoman-away | 21:26 | |
openstackgerrit | A change was merged to openstack/keystone: Fixed reference to unassigned variable https://review.openstack.org/292 | 21:29 |
*** chipc has quit IRC | 21:31 | |
*** chipc has joined #openstack-dev | 21:31 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Updated samples https://review.openstack.org/293 | 21:35 |
anotherjesse | bcwaldon: the comment about ignore me - does that mean it is in the spec? | 21:36 |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Removed redundant function from base user api https://review.openstack.org/294 | 21:36 |
*** martine has quit IRC | 21:39 | |
*** AhmedSoliman has joined #openstack-dev | 21:39 | |
*** mattray1 has joined #openstack-dev | 21:50 | |
*** jsavak has quit IRC | 21:50 | |
*** mattray has quit IRC | 21:53 | |
*** bcwaldon has quit IRC | 21:57 | |
openstackgerrit | A change was merged to openstack/keystone: Reworked XSDs and WADL to support auth and access elements. https://review.openstack.org/291 | 21:58 |
openstackgerrit | A change was merged to openstack/keystone: Updated samples https://review.openstack.org/293 | 22:01 |
openstackgerrit | A change was merged to openstack/keystone: Removed redundant function from base user api https://review.openstack.org/294 | 22:03 |
*** chipc has quit IRC | 22:03 | |
*** mattray1 has quit IRC | 22:03 | |
*** mattray has joined #openstack-dev | 22:08 | |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Cleanup service it endpoint catalog. https://review.openstack.org/296 | 22:28 |
*** deshantm has quit IRC | 22:30 | |
*** bsza has quit IRC | 22:33 | |
*** mattray has quit IRC | 22:38 | |
*** anotherjesse_ has joined #openstack-dev | 22:44 | |
*** rnirmal has quit IRC | 22:46 | |
openstackgerrit | Jorge L. Williams proposed a change to openstack/keystone: Atom links on Token. https://review.openstack.org/297 | 22:46 |
*** rnorwood has joined #openstack-dev | 22:47 | |
*** bsza has joined #openstack-dev | 22:47 | |
*** anotherjesse has quit IRC | 22:48 | |
*** anotherjesse_ is now known as anotherjesse | 22:48 | |
openstackgerrit | A change was merged to openstack/keystone: Cleanup service it endpoint catalog. https://review.openstack.org/296 | 22:48 |
*** bsza3 has joined #openstack-dev | 22:51 | |
*** bsza has quit IRC | 22:52 | |
*** bsza3 has quit IRC | 22:52 | |
*** bsza has joined #openstack-dev | 22:53 | |
*** zns1 has quit IRC | 22:53 | |
*** rnorwood has quit IRC | 22:54 | |
openstackgerrit | A change was merged to openstack/keystone: Atom links on Token. https://review.openstack.org/297 | 22:54 |
*** anotherjesse has quit IRC | 23:16 | |
*** anotherjesse has joined #openstack-dev | 23:16 | |
*** bengrue has quit IRC | 23:22 | |
*** zaitcev has joined #openstack-dev | 23:24 | |
*** AhmedSoliman has quit IRC | 23:25 | |
Tushar | bcwaldon: I have fixed review comments given by you on my merg prop. Request you to please review it. Thanks. | 23:34 |
anotherjesse | can someone kick jenkins? | 23:41 |
anotherjesse | it isn't merging stuff | 23:41 |
anotherjesse | jeblair / mtaylor ^ | 23:41 |
jeblair | anotherjesse: i'll take a look | 23:43 |
anotherjesse | jeblair: thx! | 23:43 |
jeblair | it looks like the nova-coverage job was spinning (ie, the tests were taking forever) | 23:49 |
*** zns has joined #openstack-dev | 23:51 | |
mtaylor | jeblair: was it still trying to do soren's thing? | 23:55 |
jeblair | https://jenkins.openstack.org/job/nova-coverage/1084/console | 23:56 |
jeblair | unfortunately, since we only have one executor, if any nova job blocks now, they are all stuck. | 23:57 |
*** bsza has quit IRC | 23:57 | |
mtaylor | bleh | 23:57 |
jeblair | if we have a job with a high probability of getting stuck, then maybe we should chance the race condition and run 2 executors | 23:57 |
mtaylor | jeblair: we could also spin up more nova nodes | 23:58 |
jeblair | yes | 23:58 |
mtaylor | jeblair: the puppet module matches nova-1. as well, yeah? | 23:59 |
jeblair | yep | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!