*** zykes has joined #openstack-dev | 00:16 | |
*** ayoung has quit IRC | 00:20 | |
*** zykes- has joined #openstack-dev | 00:20 | |
*** pixelbeat has quit IRC | 00:23 | |
*** zykes- has quit IRC | 00:23 | |
*** zykes has quit IRC | 00:24 | |
*** sdake has quit IRC | 00:30 | |
*** sdake has joined #openstack-dev | 00:31 | |
*** dtroyer has quit IRC | 00:33 | |
*** winston-d has joined #openstack-dev | 00:33 | |
*** rods has quit IRC | 00:38 | |
*** dtroyer has joined #openstack-dev | 00:40 | |
*** pixelbeat has joined #openstack-dev | 00:54 | |
*** littleidea has joined #openstack-dev | 00:55 | |
*** littleidea has quit IRC | 00:57 | |
*** littleidea has joined #openstack-dev | 01:01 | |
*** dwalleck has quit IRC | 01:17 | |
*** dwalleck has joined #openstack-dev | 01:29 | |
*** hub_cap has joined #openstack-dev | 01:32 | |
*** dtroyer has quit IRC | 01:39 | |
*** pixelbeat has quit IRC | 01:48 | |
*** dwalleck has quit IRC | 02:02 | |
*** martine has joined #openstack-dev | 02:03 | |
*** tomoe_ has joined #openstack-dev | 02:09 | |
*** ckleban has quit IRC | 02:09 | |
*** VinayP__ has quit IRC | 02:21 | |
*** dtroyer has joined #openstack-dev | 02:27 | |
*** eglynn_ has joined #openstack-dev | 02:29 | |
*** eglynn has quit IRC | 02:33 | |
*** littleidea has quit IRC | 02:38 | |
*** martine has quit IRC | 02:58 | |
*** littleidea has joined #openstack-dev | 03:16 | |
*** vizsla has quit IRC | 03:20 | |
*** Gordonz has joined #openstack-dev | 03:21 | |
*** dubsquared has joined #openstack-dev | 03:22 | |
*** Gordonz has quit IRC | 03:23 | |
*** Gordonz has joined #openstack-dev | 03:23 | |
*** jakedahn has joined #openstack-dev | 03:33 | |
*** blamar_ has joined #openstack-dev | 03:40 | |
*** dtroyer has quit IRC | 03:47 | |
*** hub_cap has quit IRC | 03:56 | |
*** dubsquared has quit IRC | 04:06 | |
*** dubsquared1 has joined #openstack-dev | 04:06 | |
*** blamar_ has quit IRC | 04:46 | |
*** dtroyer has joined #openstack-dev | 04:47 | |
*** dayou has joined #openstack-dev | 05:02 | |
*** dtroyer has quit IRC | 05:09 | |
*** dwalleck has joined #openstack-dev | 05:15 | |
*** jakedahn has quit IRC | 05:15 | |
*** dwalleck has quit IRC | 05:18 | |
*** aweiss has joined #openstack-dev | 05:21 | |
*** Gordonz has quit IRC | 05:41 | |
*** cp16net has quit IRC | 05:53 | |
*** danwent has quit IRC | 06:17 | |
*** utlemming has quit IRC | 06:22 | |
*** utlemming has joined #openstack-dev | 06:26 | |
*** journeeman has joined #openstack-dev | 06:30 | |
*** bepernoot has joined #openstack-dev | 06:34 | |
*** aweiss has left #openstack-dev | 06:46 | |
*** danwent has joined #openstack-dev | 06:57 | |
*** bepernoot has quit IRC | 06:59 | |
*** sniperd has joined #openstack-dev | 07:01 | |
*** sniperd_ has quit IRC | 07:03 | |
*** bencherian has joined #openstack-dev | 07:14 | |
*** rbasak has joined #openstack-dev | 07:35 | |
*** tserong has joined #openstack-dev | 07:37 | |
*** littleidea has quit IRC | 07:42 | |
*** bencherian has quit IRC | 07:59 | |
*** bepernoot has joined #openstack-dev | 08:03 | |
*** jeroenhn has joined #openstack-dev | 08:13 | |
*** Ryan_Lane has quit IRC | 08:25 | |
*** pixelbeat has joined #openstack-dev | 08:32 | |
*** hashar has joined #openstack-dev | 08:37 | |
LarsErikP | what's this all about? (nova): TRACE: Failure: Unable to log in to XenAPI (is the Dom0 disk full?) | 08:54 |
---|---|---|
LarsErikP | the disk isn't full at all... | 08:54 |
*** martine has joined #openstack-dev | 08:54 | |
*** mancdaz1203 has joined #openstack-dev | 08:55 | |
*** darraghb has joined #openstack-dev | 09:00 | |
*** martine has quit IRC | 09:01 | |
*** journeeman has quit IRC | 09:07 | |
*** danwent has quit IRC | 09:13 | |
*** bepernoot has quit IRC | 09:15 | |
*** bepernoot has joined #openstack-dev | 09:16 | |
*** vizsla has joined #openstack-dev | 09:16 | |
*** popux has joined #openstack-dev | 09:17 | |
*** derekh has joined #openstack-dev | 09:19 | |
*** vizsla has quit IRC | 09:20 | |
*** ghe_rivero has joined #openstack-dev | 09:23 | |
*** journeeman has joined #openstack-dev | 09:30 | |
*** Mkenneth has joined #openstack-dev | 09:31 | |
*** tomoe_ has quit IRC | 09:37 | |
*** Ryan_Lane has joined #openstack-dev | 09:41 | |
*** Mandell has quit IRC | 09:58 | |
*** apevec has joined #openstack-dev | 10:15 | |
*** paulormg has joined #openstack-dev | 10:16 | |
*** Mkenneth has quit IRC | 10:18 | |
*** maploin has joined #openstack-dev | 10:19 | |
*** maploin has quit IRC | 10:19 | |
*** maploin has joined #openstack-dev | 10:19 | |
*** Mkenneth has joined #openstack-dev | 10:32 | |
*** popux has quit IRC | 10:42 | |
*** markmc has joined #openstack-dev | 10:44 | |
*** shevek_ has quit IRC | 10:45 | |
*** popux has joined #openstack-dev | 11:13 | |
*** hashar has quit IRC | 11:16 | |
*** bepernoot has quit IRC | 11:33 | |
*** rods has joined #openstack-dev | 11:36 | |
*** deshantm has joined #openstack-dev | 11:53 | |
*** popux has quit IRC | 11:58 | |
*** vizsla has joined #openstack-dev | 12:05 | |
*** deshantm has quit IRC | 12:06 | |
*** deshantm has joined #openstack-dev | 12:19 | |
*** bepernoot has joined #openstack-dev | 12:23 | |
*** bepernoot has quit IRC | 12:26 | |
*** hashar has joined #openstack-dev | 12:31 | |
*** hashar has left #openstack-dev | 12:37 | |
*** hub_cap has joined #openstack-dev | 12:58 | |
*** journeeman has quit IRC | 13:16 | |
*** hashar_ has joined #openstack-dev | 13:17 | |
*** hashar_ has quit IRC | 13:18 | |
*** hashar has joined #openstack-dev | 13:18 | |
*** crobinso has joined #openstack-dev | 13:19 | |
*** markvoelker has joined #openstack-dev | 13:19 | |
*** hashar has quit IRC | 13:21 | |
*** hashar has joined #openstack-dev | 13:23 | |
*** vizsla has quit IRC | 13:32 | |
*** kbringard has joined #openstack-dev | 13:32 | |
*** rkukura has joined #openstack-dev | 13:42 | |
LinuxJedi | annegentle: I think I've fixed your swift doc archive problem, I've put some comments in the bug, let me know how it works out for you | 13:45 |
*** hub_cap has quit IRC | 13:46 | |
*** sandywalsh has quit IRC | 13:47 | |
*** sandywalsh has joined #openstack-dev | 13:50 | |
*** bepernoot has joined #openstack-dev | 13:51 | |
*** davidkranz has joined #openstack-dev | 13:52 | |
*** dayou has quit IRC | 13:57 | |
*** dayou has joined #openstack-dev | 13:58 | |
*** martine has joined #openstack-dev | 14:00 | |
*** bepernoot has quit IRC | 14:01 | |
*** martine has quit IRC | 14:03 | |
*** Mkenneth has quit IRC | 14:03 | |
*** sandywalsh has quit IRC | 14:04 | |
*** mdomsch has joined #openstack-dev | 14:08 | |
*** dprince has joined #openstack-dev | 14:08 | |
*** sandywalsh has joined #openstack-dev | 14:16 | |
*** j05h has quit IRC | 14:17 | |
*** mattray has joined #openstack-dev | 14:18 | |
*** Mkenneth has joined #openstack-dev | 14:21 | |
*** stuntmachine has joined #openstack-dev | 14:22 | |
*** bepernoot has joined #openstack-dev | 14:35 | |
*** lts has joined #openstack-dev | 14:37 | |
*** andrewbogott_afk is now known as andrewbogott | 14:38 | |
*** Mkenneth has quit IRC | 14:40 | |
*** nikhil has joined #openstack-dev | 14:46 | |
annegentle | LinuxJedi: that looks awesome. I'll run the job for Swift 1.4.4 and 1.4.5 too. | 14:51 |
annegentle | LinuxJedi: and understood about the tag and false/true setting | 14:51 |
andrewbogott | ironcamel and/or ironcame12: Are you available to answer a question about sql encoding? | 14:55 |
*** Gordonz has joined #openstack-dev | 14:57 | |
*** bepernoot has quit IRC | 14:57 | |
*** bepernoot has joined #openstack-dev | 14:58 | |
*** AlanClark has joined #openstack-dev | 15:00 | |
*** Gordonz has quit IRC | 15:00 | |
*** dayou has quit IRC | 15:02 | |
*** Mkenneth has joined #openstack-dev | 15:02 | |
*** bepernoot has quit IRC | 15:03 | |
*** stuntmachine has quit IRC | 15:05 | |
*** dubsquared has joined #openstack-dev | 15:07 | |
*** j05h has joined #openstack-dev | 15:08 | |
*** stuntmachine has joined #openstack-dev | 15:11 | |
*** jdg has joined #openstack-dev | 15:18 | |
annegentle | is it possible to use a patch file submitted via launchpad? see https://bugs.launchpad.net/openstack-manuals/+bug/941978 | 15:20 |
uvirtbot` | Launchpad bug 941978 in openstack-manuals "[PATCH] Docs: (compute) FlatDHCP disallows IP injection" [Undecided,New] | 15:21 |
annegentle | mtaylor: or LinuxJedi or CI team: Patches in Launchpad, what can I do with them? | 15:21 |
*** ayoung has joined #openstack-dev | 15:21 | |
*** cp16net has joined #openstack-dev | 15:22 | |
*** Mkenneth1 has joined #openstack-dev | 15:23 | |
*** Mkenneth1 has left #openstack-dev | 15:23 | |
*** zzed has joined #openstack-dev | 15:23 | |
*** Mkenneth has quit IRC | 15:26 | |
*** littleidea has joined #openstack-dev | 15:27 | |
*** kyriakos has left #openstack-dev | 15:30 | |
*** dolphm has joined #openstack-dev | 15:33 | |
*** Gordonz has joined #openstack-dev | 15:35 | |
*** dtroyer has joined #openstack-dev | 15:37 | |
*** andrewsben has joined #openstack-dev | 15:39 | |
*** maploin has quit IRC | 15:42 | |
LinuxJedi | annegentle: they need to be applied to a git tree and reviews in the normal way | 15:43 |
*** hub_cap has joined #openstack-dev | 15:45 | |
annegentle | LinuxJedi: ok, that's what I thought, thanks for the confirmation | 15:45 |
ironcamel | andrewbogott: sure | 15:45 |
andrewbogott | ironcamel: I'm looking at your patch to convert nova tables to utf-8. There seems to be a similar issue in keystone. | 15:46 |
YorikSar | vishy, mtaylor: I've moved the localization testing to tools/hacking.py. Why it is not being checked on every change? | 15:46 |
andrewbogott | Before I came across your patch, I struck upon this solution for keystone: https://review.openstack.org/#change,4577 | 15:46 |
andrewbogott | Now I'm trying to understand what my patch does vs. what yours does. | 15:46 |
andrewbogott | Also, I don't understand why your patch worked without adding convert_unicode: True. Was that somehow already set in nova? | 15:47 |
*** Gordonz_ has joined #openstack-dev | 15:47 | |
ironcamel | andrewbogott: the nova database is created via migrate scripts | 15:47 |
andrewbogott | sure. | 15:48 |
ironcamel | so for nova, the sqlalchemy data models dont matter | 15:48 |
ironcamel | as far as generating the schema | 15:48 |
andrewbogott | My understanding is that your patch changed the format of the database storage, and mine changes the pathway by which unicode strings are inserted into the database. | 15:49 |
andrewbogott | So, I don't understand why it works to not have both things... | 15:49 |
andrewbogott | sqlalchemy in nova still matters when someone changes something via e.g. nova-manage doesn't it? | 15:50 |
*** zaitcev has joined #openstack-dev | 15:50 | |
andrewbogott | Maybe we've just gotten luck in nova and nothing is getting passed in as 16-bit unicode string. | 15:50 |
*** Gordonz has quit IRC | 15:51 | |
andrewbogott | ironcamel: Would it alarm you if I add convert_unicode to the python sqlalchemy engine, just as a safeguard? | 15:53 |
crobinso | so new keystone-manage doesn't allow creating users, but in order to use the keystoneclient tool we need an existing user/tenant. how is this supposed to work for new deployments, or am I missing something? | 15:54 |
crobinso | (this is based on git snapshots in fedora, so things may have changed upstream) | 15:54 |
ironcamel | andrewbogott: not at all | 15:54 |
andrewbogott | ironcamel: OK. I will see if I can produce a test case to demonstrate that it matters. | 15:54 |
andrewbogott | If not, *shrug* | 15:54 |
ironcamel | andrewbogott: would not alarm me at all i meant | 15:55 |
* andrewbogott nods | 15:55 | |
*** Gordonz_ has quit IRC | 15:56 | |
ironcamel | andrewbogott: just fyi, i have tested creating new instances with unicode names | 15:56 |
ironcamel | through the api | 15:56 |
ironcamel | and was able to create them and list them just fine | 15:56 |
andrewbogott | OK. Do you have any idea if the instance name is getting stored as a unicode or as utf-8 within python? | 15:56 |
andrewbogott | It's the former case that I'm worried about. | 15:56 |
ironcamel | unicode or utf8? aren't those kinda the same? | 15:57 |
andrewbogott | No, in python u'foo' is a 16-bit string of type unicode. 'foo' is an 8-bit string of ambiguous encoding. | 15:57 |
andrewbogott | sqlmagic will pass on an 8-bit string without comment, regardless of encoding. | 15:57 |
andrewbogott | when it gets a unicode it will, by default, try to encode it as 'latin-1' which fails if there are any non-latin characters involved. | 15:58 |
andrewbogott | Awkwardly, python treats u'strings' and 'strings' the same so most of the time developers don't notice the difference. But that can backfire. | 15:59 |
ironcamel | andrewbogott: u'foo' is not utf8 encoded? | 15:59 |
andrewbogott | No... | 15:59 |
ironcamel | how is it encoded? | 16:00 |
andrewbogott | when we talk about 'encoding', that's the process of converting a 16-bit string into an 'escaped' 8-bit string. | 16:00 |
ironcamel | ah | 16:00 |
andrewbogott | utf-8 is a method of storing unicode data in an 8-bit string. Python has a data type called 'unicode' which uses actual wide strings. | 16:00 |
andrewbogott | So... confusingly, utf-8 and u'strings' are both an implementation of unicode-the-concept. | 16:01 |
andrewbogott | Sorry, I'm not doing a great job of explaining this. | 16:01 |
ironcamel | no, you are doing great, thanks! | 16:01 |
dtroyer | crobinso: there is a shiny new tools/sample_data.sh that demonstrates this: https://github.com/openstack/keystone/blob/master/tools/sample_data.sh | 16:02 |
dtroyer | crobinso: note that it requires a recent python-keystoneclient. the concept works with the older client, but the exact syntax has changed in a couple of places | 16:02 |
crobinso | dtroyer: cool I'll take a look | 16:03 |
andrewbogott | ironcamel: I believe that python 3 makes all of this explicit. As it is, we have a few landmines hiding here and there. | 16:04 |
berendt | dtroyer: it's the shiny from from devstack, isn't it? | 16:04 |
dtroyer | berendt: basically, yes | 16:05 |
berendt | wouldn't it make sense to use the same script for both? | 16:05 |
dtroyer | berendt: yes, and we will at some point | 16:05 |
berendt | dtroyer: fine :) | 16:05 |
dtroyer | berendt: sometimes moving too fast on these things bites in the wrong place ;) | 16:06 |
berendt | is it possible to receive a notification from jenkins when a specific job was finished? | 16:08 |
*** eglynn__ has joined #openstack-dev | 16:08 | |
*** eglynn_ has quit IRC | 16:09 | |
*** bencherian has joined #openstack-dev | 16:10 | |
*** danwent has joined #openstack-dev | 16:13 | |
*** x86brandon has joined #openstack-dev | 16:15 | |
*** PotHix has joined #openstack-dev | 16:17 | |
*** jdg has quit IRC | 16:17 | |
*** jeroenhn has quit IRC | 16:19 | |
berendt | problem with dashboard: "ImproperlyConfigured: Error importing middleware horizon.middleware: "cannot import name client" | 16:22 |
berendt | what could be wrong here? (i have installed python-novaclient, python-keystoneclient, glance..) | 16:22 |
*** maploin has joined #openstack-dev | 16:24 | |
*** maploin has quit IRC | 16:24 | |
*** maploin has joined #openstack-dev | 16:24 | |
berendt | berendt: you have to install python-quantumclient, too... | 16:29 |
berendt | this should be no hard requirement.. | 16:29 |
*** jdg has joined #openstack-dev | 16:34 | |
*** maplebed has quit IRC | 16:34 | |
*** aweiss has joined #openstack-dev | 16:35 | |
*** Mandell has joined #openstack-dev | 16:38 | |
*** Yak-n-Yeti has joined #openstack-dev | 16:40 | |
*** Mandell has quit IRC | 16:40 | |
*** Mandell_ has joined #openstack-dev | 16:45 | |
*** eglynn has joined #openstack-dev | 16:45 | |
*** eglynn__ has quit IRC | 16:46 | |
*** Mkenneth has joined #openstack-dev | 16:50 | |
*** cjwilson has joined #openstack-dev | 16:53 | |
*** newadmin has joined #openstack-dev | 16:57 | |
*** vincentr has joined #openstack-dev | 16:59 | |
*** vincentr is now known as vricci | 16:59 | |
*** andrewsmedina has joined #openstack-dev | 16:59 | |
*** vricci has quit IRC | 16:59 | |
*** stuntmachine has quit IRC | 16:59 | |
*** vricci has joined #openstack-dev | 16:59 | |
*** vricci is now known as vincentricci | 16:59 | |
*** vincentricci has left #openstack-dev | 17:00 | |
*** x86brandon_ has joined #openstack-dev | 17:01 | |
*** Mkenneth1 has joined #openstack-dev | 17:01 | |
*** x86brandon has quit IRC | 17:01 | |
*** x86brandon_ is now known as x86brandon | 17:01 | |
*** Mkenneth has quit IRC | 17:06 | |
*** dolphm has quit IRC | 17:06 | |
*** dubsquared has quit IRC | 17:06 | |
*** dubsquared has joined #openstack-dev | 17:08 | |
*** hashar has quit IRC | 17:08 | |
*** maploin has quit IRC | 17:08 | |
*** shevek_ has joined #openstack-dev | 17:09 | |
*** dolphm has joined #openstack-dev | 17:12 | |
*** bencherian has quit IRC | 17:12 | |
*** openstackjenkins has quit IRC | 17:15 | |
*** openstackjenkins has joined #openstack-dev | 17:16 | |
*** dolphm has quit IRC | 17:17 | |
*** hhoover has joined #openstack-dev | 17:21 | |
*** hhoover has left #openstack-dev | 17:21 | |
*** bencherian has joined #openstack-dev | 17:22 | |
*** statik has joined #openstack-dev | 17:22 | |
*** aweiss has quit IRC | 17:28 | |
*** aweiss has joined #openstack-dev | 17:31 | |
*** maplebed has joined #openstack-dev | 17:31 | |
*** jog0 has joined #openstack-dev | 17:33 | |
*** jog0 has quit IRC | 17:33 | |
*** jog0 has joined #openstack-dev | 17:34 | |
*** jdurgin has joined #openstack-dev | 17:40 | |
*** novas0x2a|laptop has quit IRC | 17:44 | |
*** jog0 has left #openstack-dev | 17:45 | |
*** shevek_ has quit IRC | 17:45 | |
*** jog0 has joined #openstack-dev | 17:47 | |
*** bepernoot has joined #openstack-dev | 17:50 | |
*** derekh has quit IRC | 17:52 | |
*** bencherian has quit IRC | 17:55 | |
*** heckj has joined #openstack-dev | 17:58 | |
*** mattray has quit IRC | 17:59 | |
*** markmc has quit IRC | 17:59 | |
*** stuntmachine has joined #openstack-dev | 18:02 | |
*** reed has joined #openstack-dev | 18:04 | |
*** maplebed has quit IRC | 18:11 | |
*** bencherian has joined #openstack-dev | 18:11 | |
*** darraghb has quit IRC | 18:12 | |
*** aweiss has quit IRC | 18:12 | |
*** Mkenneth1 has quit IRC | 18:12 | |
*** ohnoimdead has joined #openstack-dev | 18:19 | |
*** rbasak has quit IRC | 18:33 | |
jaypipes | eglynn: k, both Copy-From branches reviewed... | 18:36 |
jaypipes | eglynn: sorry for the delay. | 18:36 |
*** torgomatic has joined #openstack-dev | 18:37 | |
*** armaan has joined #openstack-dev | 18:38 | |
*** aweiss has joined #openstack-dev | 18:39 | |
*** jakedahn has joined #openstack-dev | 18:41 | |
bcwaldon | jaypipes: you aren't allowed to disagree with me | 18:42 |
*** littleidea has quit IRC | 18:45 | |
*** littleidea has joined #openstack-dev | 18:46 | |
*** novas0x2a|laptop has joined #openstack-dev | 18:46 | |
*** dolphm has joined #openstack-dev | 18:48 | |
*** vladimir3p has joined #openstack-dev | 18:48 | |
*** Ryan_Lane has joined #openstack-dev | 18:49 | |
jaypipes | bcwaldon: ah, damn, missed that memo :( | 18:49 |
bcwaldon | jaypipes: I'll let it go this time | 18:50 |
jaypipes | bcwaldon: k, thx mate :) | 18:50 |
*** shevek__ has joined #openstack-dev | 18:52 | |
bcwaldon | jaypipes: have you tried the copy_from stuff? It's awesome | 18:54 |
*** littleidea has left #openstack-dev | 18:54 | |
*** eglynn has quit IRC | 18:59 | |
*** hub-cap has joined #openstack-dev | 18:59 | |
jaypipes | bcwaldon: yep, had some for lunch. | 18:59 |
*** eglynn has joined #openstack-dev | 19:00 | |
bcwaldon | jaypipes: yes... | 19:00 |
*** hub_cap has quit IRC | 19:03 | |
*** hub-cap is now known as hub_cap | 19:03 | |
*** bencherian has quit IRC | 19:05 | |
*** maplebed has joined #openstack-dev | 19:07 | |
*** utlemming has quit IRC | 19:12 | |
*** sandywalsh has quit IRC | 19:13 | |
*** utlemming has joined #openstack-dev | 19:15 | |
*** littleidea has joined #openstack-dev | 19:16 | |
*** sandywalsh has joined #openstack-dev | 19:16 | |
*** nati has joined #openstack-dev | 19:19 | |
*** vincentricci has joined #openstack-dev | 19:19 | |
*** vincentricci has left #openstack-dev | 19:23 | |
*** adjohn has joined #openstack-dev | 19:24 | |
openstackgerrit | Verification of a change to openstack/keystone failed: XML de/serialization (bug 928058) https://review.openstack.org/4297 | 19:27 |
uvirtbot` | Launchpad bug 928058 in keystone "support XML output for Keystone API" [High,In progress] https://launchpad.net/bugs/928058 | 19:27 |
*** aweiss has quit IRC | 19:28 | |
*** aweiss has joined #openstack-dev | 19:29 | |
*** vincentricci has joined #openstack-dev | 19:31 | |
*** vincentricci has left #openstack-dev | 19:31 | |
dolphm | mtaylor: jeblair: any idea what's going on here? https://jenkins.openstack.org/job/gate-keystone-unittests/530/console | 19:39 |
mtaylor | dolphm: /me looks | 19:39 |
dolphm | mtaylor: same review, slightly different failure: https://jenkins.openstack.org/job/gate-keystone-pep8/940/console | 19:40 |
mtaylor | dolphm: sigh. yeah - one sec | 19:41 |
dolphm | mtaylor: thanks! | 19:42 |
*** mszilagyi has joined #openstack-dev | 19:43 | |
mtaylor | dolphm: wow. what a weird edge case... | 19:43 |
dolphm | mtaylor: what's going on? | 19:43 |
eglynn | jaypipes: thanks for the review! | 19:44 |
eglynn | jaypipes: that get_copy_from() "helper" method turned out to be more of a hinderance ;) | 19:44 |
mtaylor | dolphm: it has to do with the pip bundle caching | 19:44 |
mtaylor | dolphm: and responding to changes in pip-requires in a particular change | 19:44 |
eglynn | (I've removed it from the latest patch set) | 19:44 |
jaypipes | eglynn: :) | 19:45 |
jaypipes | eglynn: k, thx! | 19:45 |
dolphm | mtaylor: after adding lxml? | 19:45 |
mtaylor | dolphm: yeah. | 19:46 |
mtaylor | dolphm: we pre-build the venvs out of band so that we're not building them during test run | 19:46 |
mtaylor | dolphm: but that's problematic if one of the requirements has changed in the patcheset to be tested | 19:46 |
dolphm | mtaylor: rebuilding it on demand then? | 19:47 |
mtaylor | yes | 19:47 |
mtaylor | BUT - there's a bug in rebuilding it on demand (or in pip) | 19:47 |
mtaylor | in that we build the venv from a frozen bundle, but there are git urls in the require file | 19:48 |
mtaylor | so pip doesn't know how to run pip install --upgrade -r tools/pip-requires on that venv, because the bundle-installed depends don't match what it expects for a git require | 19:49 |
mtaylor | gigth | 19:49 |
mtaylor | sigh | 19:49 |
mtaylor | dolphm: seems to be fixed | 19:50 |
dolphm | mtaylor: retrigger? | 19:50 |
mtaylor | dolphm: already re-triggered | 19:50 |
dolphm | nice, thanks | 19:51 |
dolphm | mtaylor: merged == happiness :D | 19:53 |
*** armaan has left #openstack-dev | 19:54 | |
*** mattray has joined #openstack-dev | 19:54 | |
*** aweiss has quit IRC | 19:55 | |
*** aweiss has joined #openstack-dev | 19:56 | |
*** davidkranz has quit IRC | 20:01 | |
*** hashar has joined #openstack-dev | 20:02 | |
*** hub_cap has quit IRC | 20:04 | |
*** hub_cap has joined #openstack-dev | 20:04 | |
*** mnewby has joined #openstack-dev | 20:05 | |
openstackgerrit | Verification of a change to openstack/glance failed: Remove usage of the deprecated pip -E option. https://review.openstack.org/4363 | 20:06 |
*** gabrielhurley has joined #openstack-dev | 20:08 | |
*** paulormg has quit IRC | 20:09 | |
openstackgerrit | Verification of a change to openstack/glance failed: Split requires. https://review.openstack.org/4325 | 20:11 |
*** bepernoot has joined #openstack-dev | 20:16 | |
*** eglynn has quit IRC | 20:18 | |
openstackgerrit | Verification of a change to openstack/keystone failed: Add Vary header (bug 928057) https://review.openstack.org/4460 | 20:18 |
uvirtbot` | Launchpad bug 928057 in keystone "provide Vary header" [Medium,In progress] https://launchpad.net/bugs/928057 | 20:18 |
*** vricci has joined #openstack-dev | 20:19 | |
*** adjohn_ has joined #openstack-dev | 20:19 | |
*** vricci has left #openstack-dev | 20:20 | |
*** aweiss has quit IRC | 20:20 | |
*** maplebed has quit IRC | 20:21 | |
*** adjohn has quit IRC | 20:21 | |
*** adjohn_ is now known as adjohn | 20:21 | |
*** maplebed has joined #openstack-dev | 20:21 | |
*** maplebed_ has joined #openstack-dev | 20:23 | |
*** maplebed has quit IRC | 20:23 | |
openstackgerrit | Verification of a change to openstack/keystone failed: Add HEAD /tokens/{token_id} (bug 933587) https://review.openstack.org/4371 | 20:23 |
uvirtbot` | Launchpad bug 933587 in keystone "HEAD /v2.0/tokens/{token_id} returns 404" [Medium,In progress] https://launchpad.net/bugs/933587 | 20:23 |
dolphm | mtaylor: subsequent & dependent job failed b/c lxml not in venv? ^^^ | 20:24 |
*** maplebed_ is now known as maplebed | 20:29 | |
*** maplebed has joined #openstack-dev | 20:29 | |
*** dtroyer has quit IRC | 20:31 | |
mtaylor | dolphm: race condition I think | 20:36 |
mtaylor | dolphm: the first change landing will trigger the creation of a new venv ... but that takes a few minutes... so if the dependent change fired right away, the new venv won't be there | 20:37 |
* mtaylor punches self | 20:37 | |
mtaylor | dolphm: I'm on it | 20:38 |
dolphm | mtaylor: /salute | 20:39 |
*** creiht has joined #openstack-dev | 20:43 | |
*** ChanServ sets mode: +v creiht | 20:43 | |
creiht | vishy: around??? | 20:43 |
*** dtroyer has joined #openstack-dev | 20:43 | |
*** eglynn has joined #openstack-dev | 20:45 | |
annegentle | ttx mtaylor notmyname reed or vishy: can I get a CLA admin to add koolhead17 to openstack-cla? https://launchpad.net/~openstack-cla/+members#proposed | 20:47 |
vishy | creiht: yes | 20:48 |
creiht | vishy: https://github.com/openstack/python-novaclient/commit/03f54c57e13f27324f0ac5aa556bf4ca94e3ed0e | 20:48 |
creiht | that changes the volumes endpoint from /os-volumes to /volumes | 20:49 |
*** dolphm has quit IRC | 20:49 | |
vishy | creiht: correct | 20:49 |
creiht | any way to override that? | 20:49 |
vishy | creiht: no, I had it in initially supporting both but we decided since os-volumes was an extension | 20:49 |
vishy | that we should just force people to use the new one | 20:49 |
creiht | an extension that a lot of people are probably already using ;) | 20:50 |
reed | annegentle, what's his/her name? | 20:50 |
vishy | the addition is relatively simple, so you think i should add it back in? | 20:50 |
vishy | creiht: ^^ | 20:50 |
creiht | vishy: it would be nice for the short term at least | 20:51 |
creiht | until people can get migrated over | 20:51 |
annegentle | reed: it's Atul Jha | 20:51 |
creiht | for example, I'm having to (once again) update my vtypes merge | 20:51 |
creiht | and I can't test it at the moment | 20:51 |
vishy | creiht: can you just use an older commit? | 20:51 |
creiht | that would be kind of painful | 20:52 |
vishy | creiht: why can't you test with the new version? | 20:52 |
reed | annegentle, found himt | 20:52 |
creiht | because all of our test setup is still listening to /os-volumes | 20:52 |
annegentle | reed: thanks | 20:52 |
reed | annegentle, found his CLA... he signed the old form... | 20:52 |
vishy | creiht: i see | 20:52 |
reed | damn, nobody has removed it yet | 20:52 |
annegentle | reed: do folks need to resign? | 20:52 |
creiht | I'm a manager now, I don't have time to spin up my own test stuff :) | 20:52 |
*** eglynn has quit IRC | 20:52 | |
vishy | creiht: so you aren't running the volumes endpoint in your test cluster? | 20:52 |
reed | annegentle, no, it's the same identical document, only it goes in the wrong bucket | 20:53 |
creiht | what started out as a simple one day quick fix has turned into a multi-day ordeal :/ | 20:53 |
creiht | vishy: we are running the extension | 20:53 |
justinsb | vishy: How can I get the extension os-volumes if you install a system os-volumes over the top? | 20:53 |
reed | annegentle, I haven't managed to get the right people to take that down | 20:53 |
creiht | multi-week even | 20:53 |
annegentle | reed: bumemr | 20:53 |
vishy | creiht: right, but the volumes endpoint runs by default as well... | 20:53 |
reed | annegentle, is it referenced somewhere else but the wiki? | 20:53 |
justinsb | vishy: (I think this is a general extensions question, not just volumes) | 20:53 |
reed | besides the wiki I mean | 20:53 |
annegentle | reed: hm. thinking | 20:53 |
vishy | justinsb: I don't understand the question | 20:54 |
vishy | justinsb: what is a "system os-volumes" | 20:54 |
*** davidkranz has joined #openstack-dev | 20:54 | |
justinsb | vishy: Sorry, let me catch up on the thread here | 20:55 |
vishy | creiht: the concern is that if we leave it in, people will continue to use it | 20:55 |
creiht | vishy: I guess I am a bit confused as well? | 20:55 |
vishy | creiht: we want people to start using the volumes endpoint | 20:55 |
vishy | instead of the compute extension | 20:55 |
creiht | vishy: it would be nice if that were communicated somwhere :) | 20:56 |
reed | is cheyenne Bryant on this channel? | 20:56 |
vishy | creiht: that is true. Perhaps I should send an email to the ml | 20:56 |
creiht | that would be nice | 20:56 |
vishy | creiht: the types extension you were working on should also go to the volume endpoint i would think | 20:57 |
creiht | yeah that is fine | 20:57 |
justinsb | vishy: If someone wants to keep using the os-volumes extension, shouldn't they be able to do so? | 20:57 |
vishy | justinsb: they can, but do you think we should provide bindings for it in python-novaclient? | 20:58 |
justinsb | vishy: So you can't grab /os-volumes, because an extension is using that, right? | 20:58 |
creiht | vishy: so back to the vtypes stuff for the command line | 20:58 |
creiht | vishy: any way you can pick up that merge, and fix the stuff with the new endpoint? | 20:58 |
vishy | creiht: sure | 20:58 |
creiht | I don't have the bandwidth to try to keep up | 20:58 |
creiht | thx | 20:58 |
vishy | creiht: send me the review num? | 20:58 |
justinsb | vishy: If a cloud is using it, then the client bindings should support it | 20:58 |
creiht | https://review.openstack.org/#change,4293 | 20:58 |
*** x86brandon__ has joined #openstack-dev | 20:59 | |
*** dolphm has joined #openstack-dev | 20:59 | |
creiht | vishy: it is almost there, it needs to be updated for the service_type stuff, and a fix to the devstack tests | 20:59 |
justinsb | vishy: The approach we're trying to take in the Java bindings is that extensions can be passed in separately and maintained separately from the core API binding | 20:59 |
justinsb | vishy: Although that is very nascent (attributes only, so far) | 20:59 |
*** anotherjesse has joined #openstack-dev | 21:00 | |
vishy | justinsb: yeah we would like that in novaclient as well | 21:00 |
creiht | vishy: and speaking of the devstack tests, those things are way too fragile | 21:00 |
vishy | justinsb: but unfortunately we are not there | 21:00 |
*** anotherjesse has left #openstack-dev | 21:00 | |
*** jog0 has left #openstack-dev | 21:00 | |
*** anotherjesse has joined #openstack-dev | 21:00 | |
justinsb | vishy: But you really can't have your new volumes endpoint listen on os-volumes, unless I can continue to access the extension on e.g. extensions/os-volumes | 21:00 |
creiht | they shouldn't break because I add a column to a user-readable table :) | 21:00 |
*** andrewsben has quit IRC | 21:00 | |
vishy | the new volumes endpoint is a separate api | 21:01 |
anotherjesse | creiht: the fact that they are gating tests are while tempest is merged in | 21:01 |
vishy | justinsb: it listens on volumes-endpoint/volumes | 21:01 |
vishy | so there is no conflict there | 21:01 |
justinsb | vishy: Ah OK! Sorry! | 21:01 |
anotherjesse | creiht: the primary reason for exercises is to show people how the clients work | 21:01 |
*** bepernoot has quit IRC | 21:01 | |
*** jog0_ has joined #openstack-dev | 21:01 | |
*** x86brandon has quit IRC | 21:02 | |
*** x86brandon__ is now known as x86brandon | 21:02 | |
creiht | anotherjesse: I understand that part | 21:02 |
creiht | I'm just saying of you are going to gate on something, don't gate on flakey tests ;) | 21:02 |
creiht | just my opinion of course | 21:02 |
*** torgomatic has quit IRC | 21:02 | |
*** jog0_ has quit IRC | 21:02 | |
*** bencherian has joined #openstack-dev | 21:02 | |
*** torgomatic has joined #openstack-dev | 21:03 | |
openstackgerrit | Verification of a change to openstack/glance failed: Support copy-from for queued images. https://review.openstack.org/4446 | 21:03 |
*** zykes- has joined #openstack-dev | 21:03 | |
*** ohnoimdead has quit IRC | 21:04 | |
anotherjesse | creiht: I wouldn't say they are flakey - they are brittle | 21:04 |
anotherjesse | creiht: if they were flakey I wouldn't gate | 21:04 |
bcwaldon | jaypipes: https://review.openstack.org/#change,4596 <- fix for that unowned image deletion bug | 21:05 |
zykes- | soren: around? | 21:08 |
jaypipes | bcwaldon: see question on review... | 21:09 |
*** vincentricci has joined #openstack-dev | 21:11 | |
*** x86brandon has quit IRC | 21:11 | |
creiht | vishy: thanks for picking up the vtypes stuff, it will make our testing a lot easier | 21:11 |
creiht | plus make it easier of course for other people to use volume types :) | 21:12 |
*** markvoelker has quit IRC | 21:13 | |
*** bepernoot has joined #openstack-dev | 21:13 | |
vishy | creiht: in the meantime you should get your test cluster running the volumes endpoint if it isn't already | 21:14 |
*** ohnoimdead has joined #openstack-dev | 21:14 | |
vishy | (it might just be adding an extra endpoint in your keystone catalog) | 21:14 |
creiht | vishy: yeah we will get there | 21:14 |
*** vincentricci has quit IRC | 21:15 | |
creiht | it wouldn't be that big of a deal, just that the timing coincided with me trying to hack something together real quick | 21:16 |
creiht | which has turned into taking much more time than I originally planned :) | 21:16 |
*** ohnoimdead has quit IRC | 21:17 | |
* creiht goes back to managing stuff :/ | 21:18 | |
*** creiht has left #openstack-dev | 21:19 | |
*** eglynn has joined #openstack-dev | 21:25 | |
*** vincentricci has joined #openstack-dev | 21:28 | |
*** stuntmachine has quit IRC | 21:28 | |
jaypipes | bcwaldon: you see my little question on https://review.openstack.org/#change,4596,patchset=2? | 21:37 |
*** ohnoimdead has joined #openstack-dev | 21:46 | |
bcwaldon | jaypipes: yes, the answer is yes but I found a failing tests I need to fix :( | 21:50 |
bcwaldon | jaypipes: the authorization stuff we have in glance needs to be redone | 21:50 |
bcwaldon | jaypipes: it's a mess | 21:50 |
*** bepernoot has quit IRC | 21:51 | |
*** crobinso has quit IRC | 21:52 | |
*** milner has joined #openstack-dev | 21:53 | |
*** mikeyp has joined #openstack-dev | 21:53 | |
heckj | termie: ping | 21:53 |
*** sandywalsh has quit IRC | 21:56 | |
termie | heckj: I AM HERE | 21:57 |
termie | heckj: WHAT DO YOU ASK OF ME | 21:57 |
*** jakedahn has quit IRC | 21:57 | |
*** jakedahn has joined #openstack-dev | 21:57 | |
*** torgomatic_ has joined #openstack-dev | 21:58 | |
heckj | termie: I'm reading through https://review.openstack.org/#change,4464, trying to figure out how adam_g got into a circular dependency, but appeared to follow your code pattern. | 21:58 |
termie | heckj: the three separate APIs all use each other | 21:59 |
termie | heckj: normally one gets around this by moving that logic to a single place and making the other objects dumber | 21:59 |
termie | heckj: i think we should do that too, but my tired eyes weren't up to the task, so i solved the dependency problem and wrote TODOs | 21:59 |
*** torgomatic has quit IRC | 21:59 | |
*** torgomatic_ is now known as torgomatic | 21:59 | |
termie | heckj: i don't like putting much logic in my data objects | 22:00 |
anotherjesse | heckj: I reviewed ldap https://review.openstack.org/#change,4362 - concentrating on the code impact outside of ldap (eg, I'm not an ldap expert) | 22:00 |
bcwaldon | termie: that's what she said | 22:00 |
anotherjesse | heckj: seems ready for merge imho | 22:00 |
heckj | termie: makes sense. Logic into the Controllers. | 22:00 |
heckj | Adam's change works right now - removing catalog_api from identity (where it's not yet used). That made me generally uncomfy, but should we let it roll through as is and plan to refactor the logic where the API's use each? | 22:01 |
heckj | anotherjesse: running through 4464 now | 22:02 |
termie | heckj: catalog_api from identity? i don't think there is a catalog api in there... is there? | 22:02 |
termie | heckj: don't remember seeing one when i was changing code | 22:03 |
heckj | there was - never used, so Adam removed it to solve the circ dep | 22:03 |
termie | oh way | 22:03 |
termie | wait | 22:03 |
termie | i am talking about the wrong thing | 22:03 |
heckj | (see: https://review.openstack.org/#patch,sidebyside,4464,4,keystone/identity/core.py) | 22:03 |
termie | i am talking about ayoung's branch | 22:03 |
termie | retract all my statements so far | 22:03 |
heckj | Okay - retracting away (there they go!) | 22:03 |
termie | i now have two minutes before i have to drive somebody to the airport | 22:03 |
termie | so lesse, in the catalog sql branch, something about circular dependencies | 22:04 |
termie | we haven't done anything to resolve those, especially, no | 22:04 |
termie | right now we just happen to not quite have any, but that could change at any moment | 22:04 |
zykes- | anotherjesse: ? | 22:04 |
termie | and then we'd have to put some more thinking into it | 22:04 |
termie | probably similar to the shim i did in ldap, as a possible short-term | 22:05 |
bcwaldon | jeblair: you might want to take a peek at these failures: https://jenkins.openstack.org/job/gate-glance-python27/ | 22:05 |
termie | okay rolling out now | 22:05 |
bcwaldon | jeblair: I retriggered latest | 22:05 |
termie | back online in probably an hour | 22:05 |
bcwaldon | eglynn: can you rebase this -> https://review.openstack.org/#change,4363 | 22:05 |
zykes- | notmyname: here ? :p | 22:05 |
eglynn | bcwaldon: sure | 22:06 |
jaypipes | bcwaldon: yeah, no crap... it's wreaking havoc for nati and me right now... | 22:06 |
bcwaldon | jaypipes: I think I figured out my problem, but we need to really think this out when we implement v2 | 22:07 |
openstackgerrit | Verification of a change to openstack/glance failed: Support copy-from for queued images. https://review.openstack.org/4446 | 22:07 |
jaypipes | bcwaldon: we still talking about keystone here? | 22:07 |
heckj | ayoung: you around? | 22:08 |
bcwaldon | jaypipes: glance + keystone integration | 22:08 |
jeblair | bcwaldon: huh. | 22:08 |
ayoung | heckj, yes | 22:08 |
heckj | is the python-ldap dependency specific to a version (in https://review.openstack.org/#patch,sidebyside,4362,6,tools/pip-requires-test)? Or general? | 22:09 |
heckj | ayoung: pip-requires and pip-requires-test are slightly different there | 22:09 |
bcwaldon | jaypipes: my branch is good for another round | 22:10 |
jeblair | heckj: can you call it test-requires? | 22:10 |
jeblair | i think that's what mtaylor has been assuming (and what's happening in glance) | 22:10 |
heckj | jeblair: yeah, I don't see any reason we couldn't rename it if it makes something easier for you | 22:10 |
jaypipes | bcwaldon: approved. off to the pit of despair.. | 22:10 |
ayoung | heckj, I think we can go with the less restrictive version for both | 22:10 |
jeblair | heckj: thanks | 22:11 |
*** sandywalsh has joined #openstack-dev | 22:11 | |
bcwaldon | jaypipes: might not be functional right now, seeing unittest failures | 22:11 |
jaypipes | bcwaldon: https://bugs.launchpad.net/keystone/+bug/942247 | 22:11 |
uvirtbot` | Launchpad bug 942247 in keystone "Admin token auth completely broken" [Critical,Confirmed] | 22:11 |
heckj | ayoung: good either way, just wanted to know. Approving as is - I can make it general in a follow-up checkin (which I'm about to do for jeblair) | 22:11 |
jaypipes | bcwaldon: is what nati and I are currently running into ... | 22:11 |
ayoung | heckj, latest version has them in sync | 22:11 |
bcwaldon | jaypipes: neat | 22:12 |
ayoung | I'll repost with that | 22:12 |
ayoung | I keep finding more cleanup to do | 22:12 |
heckj | ayoung: I just approved it as it - let that be a follow up commit. I think we're good to go with the existing 4362 | 22:13 |
ayoung | heckj, OK | 22:13 |
eglynn | bcwaldon: that patch needing a rebase has actually been superceeded by Monty's change: https://github.com/openstack/glance/commit/d26f66bc | 22:13 |
openstackgerrit | Verification of a change to openstack/keystone failed: Add Vary header (bug 928057) https://review.openstack.org/4460 | 22:13 |
uvirtbot` | Launchpad bug 928057 in keystone "provide Vary header" [Medium,In progress] https://launchpad.net/bugs/928057 | 22:13 |
bcwaldon | eglynn: okay, you can abandon your review, then | 22:13 |
eglynn | bcwaldon: so I'm just gonna abandon the gerrit review | 22:13 |
eglynn | yep | 22:13 |
*** andrewsmedina has quit IRC | 22:14 | |
*** j05h has left #openstack-dev | 22:14 | |
jaypipes | annegentle: ping. looking for the old compute 1.0 API docs... know where I can find those? | 22:18 |
*** anotherjesse1 has joined #openstack-dev | 22:18 | |
annegentle | jaypipes: looking | 22:18 |
jaypipes | annegentle: ty gentle lady. | 22:18 |
annegentle | jaypipes: http://docs.openstack.org/cactus/openstack-compute/developer/openstack-compute-api-1.0/content/ | 22:19 |
jeblair | bcwaldon: a change from monty was recently merged to glance that pulled some reqs out of pip-requires and into test-requires. monty didn't update the glance venv build job to use that, so tests using the newest venv fail | 22:20 |
bcwaldon | jeblair: yay! | 22:20 |
jeblair | bcwaldon: i've updated the glance-venv job to include test-requires | 22:20 |
jeblair | bcwaldon: it will take a little while to run, once it's done, we can retrigger the failed changes | 22:20 |
zykes- | anyone hiring pythoneers or eager to help persons ? | 22:21 |
heckj | jeblair: I'm running under the assumption that test-requires should be a superset of pip-requires (everything plus some) - is that matching what you're thinking? | 22:22 |
*** anotherjesse has quit IRC | 22:22 | |
*** hub_cap has quit IRC | 22:22 | |
jeblair | heckj: no, i think mtaylor is thinking of it an add-on (no duplication) | 22:23 |
jeblair | heckj: so we would use pip-requires AND test-requires to build a venv for testing | 22:23 |
jaypipes | annegentle: thx! | 22:23 |
openstackgerrit | Verification of a change to openstack/glance failed: Complete fix for modification of unowned image https://review.openstack.org/4596 | 22:23 |
heckj | jeblair - that's not how the tox.ini file we have is using it - are you still running with tox, or is this something new? | 22:24 |
heckj | (not that we're really using tox so heavily right now - but it's there) | 22:24 |
adam_g | jaypipes: ping, wondering if theres been any progress on this one https://bugs.launchpad.net/glance/+bug/824794 if not, is your proposed solution still worth looking into? (im assuming auto-creation of tables is not going away) | 22:24 |
uvirtbot` | Launchpad bug 824794 in glance "Tables are generated outside of migration process" [Low,In progress] | 22:24 |
anotherjesse1 | keystone folks (dolphm, heckj, termie, bcwaldon) any working on https://bugs.launchpad.net/keystone/+bug/942247 ? | 22:24 |
uvirtbot` | Launchpad bug 942247 in keystone "Admin token auth completely broken" [Critical,Confirmed] | 22:24 |
bcwaldon | anotherjesse1: not I | 22:25 |
jeblair | heckj: hrm, that's interesting. we better ask mtaylor, as I'm only going based on what i've seen recently and in glance | 22:26 |
jeblair | mtaylor: ^ ? | 22:26 |
heckj | anotherjesse1: nopes - just saw it a few sec ago | 22:26 |
jaypipes | adam_g: unfortunately, no more progress, sorry :( I think we (OpenStack community) really needs to discuss SQLalchemy-migrate at the summit and standardize on how we do migrations. I know that dprince and others advocate not auto-creating databases ... frankly, I could easily come around to that solution as long as we had a consistent way of handling migrations and db_sync across all the projects | 22:26 |
anotherjesse1 | heckj: have two worries - one that we are barfing a stacktrace — the other that the gate tests didn't hit this | 22:27 |
jaypipes | adam_g: If you would like to tackle the problem of standardizing migrations, I would certainly not try to discourage you ;) | 22:27 |
*** torgomatic has quit IRC | 22:27 | |
dolphm | anotherjesse1: jaypipes: i'm not - but I assume that's the middleware from redux? when was it pulled from keystone heavy? it seems like it's rather outdated | 22:28 |
jaypipes | dolphm: from latest commit of keystone... | 22:28 |
*** torgomatic has joined #openstack-dev | 22:28 | |
*** bencherian has quit IRC | 22:29 | |
openstackgerrit | Verification of a change to openstack/keystone failed: LDAP Identity backend https://review.openstack.org/4362 | 22:30 |
anotherjesse1 | jaypipes: can you try running tools/info.sh on the install? | 22:30 |
anotherjesse1 | https://github.com/openstack-dev/devstack/blob/master/tools/info.sh | 22:30 |
jaypipes | anotherjesse1: yup, one sec | 22:30 |
heckj | jeblair: https://review.openstack.org/4362 failed because it didn't have python-ldap, but we added that to the pip-requires in the commit. Anything special we need to do? | 22:31 |
heckj | anotherjesse: 2nd concern worries me a bit more... | 22:31 |
heckj | anotherjesse1: ^ | 22:31 |
jaypipes | anotherjesse1: http://paste.openstack.org/show/5240 | 22:31 |
zykes- | too bad noone is hiring .eu yet more actively | 22:32 |
dolphm | heckj: mtaylor just worked on a similar issue for me, but i just had the same problem in a different build | 22:32 |
heckj | dolphm: when we add deps, was there anything we needed to do other than edit the requires? | 22:33 |
adam_g | jaypipes: yes, i second all of that. ill consider driving something at the summit. in the meantime, there are several duplicate ubuntu bugs resulting from the issue of table auto-creation. would the try: execpt: you described on that report be the correct work around? ill spend some time this week on that if so (still learning SQLalchmey as i go..) | 22:33 |
heckj | jaypipes: what repo for keystone? I'm not seeing that committish in my log | 22:33 |
*** anotherjesse1 has quit IRC | 22:34 | |
*** anotherjesse has joined #openstack-dev | 22:34 | |
*** cp16net has quit IRC | 22:34 | |
jaypipes | heckj: see paste above... keystone master github repo, commit 746cd92 | 22:34 |
heckj | jaypipes: yeah, that's what I was looking at. Must be doing something stupid. git log | grep 746cd92 on my keystone isn't showing me anything | 22:35 |
heckj | jaypipes: foudn it in github | 22:35 |
jaypipes | heckj: git pull? ;P | 22:36 |
heckj | jaypipes: yep, #PROTIP! | 22:36 |
heckj | duh | 22:36 |
* jaypipes scurries back into his hole. | 22:36 | |
jaypipes | jeblair: ping. any chance we can have you or mtaylor look into https://jenkins.openstack.org/job/gate-glance-python26/42/console ? Looks like nose isn't installed properly on the unit test nodes | 22:37 |
jeblair | heckj: nothing else should need to be added | 22:37 |
jeblair | jaypipes: i've been working on that for several minutes, see conversation in this channel with bcwaldon | 22:38 |
jaypipes | jeblair: sorry mate... | 22:38 |
jeblair | jaypipes: np, i'll check back in on it in a sec | 22:39 |
*** andrewsben has joined #openstack-dev | 22:39 | |
jeblair | heckj: does run_tests.sh in keystone use a venv by default? | 22:39 |
jeblair | jaypipes: https://jenkins.openstack.org/job/gate-glance-python27/43/ | 22:39 |
jeblair | fixed | 22:39 |
*** armaan has joined #openstack-dev | 22:40 | |
jeblair | jaypipes: I retriggered 4446 (complete) and 4596 (running now) | 22:40 |
comstud | vishy: https://review.openstack.org/#change,4477 | 22:40 |
*** lts has quit IRC | 22:41 | |
heckj | anotherjesse1: there's nothing in our test suite that's driving tests across that piece of middleware that I can see. (auth_token) major hole! | 22:41 |
heckj | jeblair: yes | 22:41 |
jaypipes | jeblair: cheerio. | 22:41 |
jeblair | heckj: okay, i'm not seeing the problem yet. it installed ldap in the venv... | 22:42 |
jaypipes | heckj: alright, then let's plug the hole :) what can nati and I do to help? | 22:42 |
jeblair | heckj: search for "NEED TO UPGRADE PIP-REQUIRES" in the console output, and you'll see that it detected the pip-requires change and re-build the venv based on that | 22:42 |
*** bencherian has joined #openstack-dev | 22:44 | |
heckj | jaypipes: we need to extend tests/test_middleware.py with a new test class and tests(methods) to load and test the auth_token middle so we work it. Looks like exactly the right place, just not being excercised. | 22:44 |
heckj | jeblair: thanks - looking through it | 22:44 |
*** dprince has quit IRC | 22:46 | |
jaypipes | heckj: FYI, this is exactly why the Glance pip-requires file is still pegged on an old Keystone Git commit... :( | 22:47 |
*** torgomatic has quit IRC | 22:47 | |
*** apevec has quit IRC | 22:47 | |
jaypipes | heckj: since this happened with an earlier keystone version too. | 22:47 |
*** jog0 has joined #openstack-dev | 22:47 | |
heckj | jaypipes: we'll get it fixed and excercise it properly so it doesn't happen again. | 22:47 |
jaypipes | heckj: which explains, of course, why Glance's tests (which definitely DO stress this middleware) are continuing to pass. | 22:47 |
vishy | comstud: doesn't look like you updated the commit message or the docs? | 22:48 |
jaypipes | heckj: so, I think we need to port the Glance middleware tests to Keystone... | 22:48 |
jaypipes | heckj: and just stub out the glance client.. | 22:48 |
*** torgomatic has joined #openstack-dev | 22:48 | |
*** hub_cap has joined #openstack-dev | 22:49 | |
*** andrewsmedina has joined #openstack-dev | 22:49 | |
comstud | vishy: The commit message is still valid... but I can add more information to it. | 22:49 |
comstud | vishy: As far as docs.. do you mean.. add documentation for this new flag? | 22:50 |
heckj | jaypipes: what's the dependency on glanceclient in the tests? | 22:50 |
jaypipes | heckj: Oh, i's just that glance's client is used in the tests to call to a server that has the middleware installed.. nothing that can't be removed | 22:52 |
heckj | jaypipes: then I'd love to port those tests right into keystone! | 22:52 |
jaypipes | heckj: i.e. it can just be a straight httplib2 call... | 22:52 |
jaypipes | heckj: working on it... | 22:52 |
vishy | comstud: sorry maybe i don't understand what you changed | 22:53 |
vishy | comstud: does it support full.path.to.filter.Class now? | 22:53 |
comstud | vishy: Essentially what I changed is: added --scheduler_available_filters to be able to specify filters that are made available to the scheduler (vs the hardcoding in init.py) | 22:53 |
comstud | vishy: --scheduler_available_filters does | 22:54 |
jeblair | heckj: ldap is installed inside the venv; "import ldap" works | 22:54 |
comstud | --scheduler_default_filters keeps the short versions | 22:54 |
comstud | I think maybe what's confusing is that there are 2 lists of filters? | 22:54 |
heckj | jeblair - it's good, I'm runing those tests myself to find the error. Something in the code, definitely | 22:54 |
comstud | 1 is "what is available" (that's what I added) | 22:54 |
comstud | 2nd is "what should I use by defualt from the available" | 22:54 |
jeblair | heckj: okay, cool, i'll stand down. | 22:54 |
comstud | (which pre-existed) | 22:54 |
vishy | ah gotcha | 22:54 |
heckj | jeblair: I thought it was the venv based on a quick read of the messages popping out, but it's got to me something more/else | 22:55 |
vishy | comstud: now i understand | 22:55 |
comstud | i just updated the commit message | 22:55 |
jeblair | heckj: understandable. "ImportError (No module named ldap)" seems pretty clear at first glance. :) | 22:55 |
jeblair | "from keystone.common.ldap import fakeldap" is an important detail though | 22:56 |
vishy | comstud: can you add a paragraph to the .rst about how to add additional filters? | 22:56 |
*** Yak-n-Yeti has quit IRC | 22:56 | |
heckj | jeblair: yeah | 22:56 |
vishy | using --scheduler_available_filters | 22:56 |
comstud | vishy: yeah, i'll take a look | 22:56 |
vishy | (mentioning that they can be external) | 22:56 |
vishy | cool thx | 22:56 |
*** Yak-n-Yeti has joined #openstack-dev | 22:56 | |
*** armaan has left #openstack-dev | 23:00 | |
*** Yak-n-Yeti has quit IRC | 23:00 | |
openstackgerrit | Verification of a change to openstack/keystone failed: Add Vary header (bug 928057) https://review.openstack.org/4460 | 23:04 |
uvirtbot` | Launchpad bug 928057 in keystone "provide Vary header" [Medium,In progress] https://launchpad.net/bugs/928057 | 23:04 |
milner | I pushed an updated patchset to gerrit. Any trick to get SmokeStack to re-run tests? https://review.openstack.org/#change,4459 | 23:05 |
dolphm | jeblair: mtaylor: 'g-api did not start' in devstack tests... something intermittent? https://jenkins.openstack.org/view/Keystone/job/gate-integration-tests-devstack-vm/1876/console | 23:07 |
*** tomoe_ has joined #openstack-dev | 23:12 | |
*** hashar has quit IRC | 23:13 | |
*** zzed has quit IRC | 23:14 | |
eglynn | bcwaldon: fix for #942118 is out of review | 23:15 |
eglynn | https://review.openstack.org/#change,4602 | 23:15 |
bcwaldon | eglynn: will look shortly | 23:16 |
eglynn | bcwaldon: cool, thx ... bbiab | 23:16 |
*** bencherian has quit IRC | 23:17 | |
*** openstackjenkins has quit IRC | 23:18 | |
*** openstackjenkins has joined #openstack-dev | 23:19 | |
openstackgerrit | Verification of a change to openstack/keystone failed: Add Vary header (bug 928057) https://review.openstack.org/4460 | 23:19 |
uvirtbot` | Launchpad bug 928057 in keystone "provide Vary header" [Medium,In progress] https://launchpad.net/bugs/928057 | 23:19 |
*** Mandell_ has quit IRC | 23:19 | |
*** openstackjenkins has quit IRC | 23:22 | |
*** bencherian has joined #openstack-dev | 23:22 | |
*** openstackjenkins has joined #openstack-dev | 23:22 | |
*** torgomatic has quit IRC | 23:24 | |
*** torgomatic has joined #openstack-dev | 23:24 | |
adam_g | heckj: regarding https://review.openstack.org/#change,4464 i found some chatter in scrollback about it, should i leave as-is and wait to refactor *_api later? or are you waiting on an update from me? | 23:28 |
dtroyer | jaypipes: I am digging in to the keystone middleware problem, wanted to confirm that the 'openstack' token in the bug report is what is configured as admin_token. | 23:29 |
*** danwent has quit IRC | 23:30 | |
*** hub_cap has quit IRC | 23:32 | |
heckj | adam_g: I think you can leave it as is - we'll need to do some refactoring for the circ dependecy issue you saw as we get more advanced. | 23:33 |
jaypipes | dtroyer: yes, it is. | 23:34 |
jaypipes | dtroyer: it is what is in localrc.. | 23:34 |
heckj | dtroyer: thank you! | 23:34 |
jaypipes | indeed, thanks dean! | 23:34 |
adam_g | heckj: sounds good, thanks | 23:35 |
*** novas0x2a|laptop has quit IRC | 23:35 | |
*** danwent has joined #openstack-dev | 23:38 | |
*** jakedahn has quit IRC | 23:41 | |
*** novas0x2a|laptop has joined #openstack-dev | 23:41 | |
comstud | vishy: ok, i updated that doc more: https://review.openstack.org/#change,4477 | 23:43 |
anotherjesse | heckj: regarding the merge of sql catalog - does that mean we have "service-CRUD", "endpoint-CRUD" and a config file? | 23:48 |
heckj | it means that we can do either - default is the catalog, but this enables SQL CRUD backend to the catalog service | 23:49 |
heckj | anotherjesse: ^ | 23:49 |
*** AlanClark has quit IRC | 23:51 | |
*** Mandell has joined #openstack-dev | 23:51 | |
*** gyee has joined #openstack-dev | 23:52 | |
*** mattray has quit IRC | 23:53 | |
openstackgerrit | Verification of a change to openstack/nova failed: Do not hit the network_api every poll https://review.openstack.org/4494 | 23:56 |
*** andrewsben has quit IRC | 23:57 | |
jeblair | dolphm: when something fails to start in devstack, the only artifact is a screen session running with a backtrace; it would be really useful if devstack put that somewhere useful, like a logfile or the stdout of stack.sh. if it did, you'd be able to see why g-api didn't start | 23:57 |
jeblair | dolphm: File "/opt/stack/keystone/keystone/common/wsgi.py", line 457 | 23:58 |
jeblair | <<<<<<< HEAD | 23:58 |
jeblair | dolphm: is what was left there in build 1876; i think there are still conflicts in the patch | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!