*** harlowja has quit IRC | 00:02 | |
*** harlowja has joined #openstack-oslo | 00:02 | |
*** HenryG has quit IRC | 00:04 | |
*** beekneemech has joined #openstack-oslo | 00:09 | |
*** bnemec has quit IRC | 00:09 | |
*** takedakn has joined #openstack-oslo | 00:16 | |
*** yamahata has joined #openstack-oslo | 00:16 | |
*** tsekiyama has quit IRC | 00:16 | |
*** takedakn has quit IRC | 00:16 | |
*** takedakn has joined #openstack-oslo | 00:17 | |
*** noelbk has quit IRC | 00:17 | |
*** mtanino has quit IRC | 00:17 | |
*** takedakn has quit IRC | 00:18 | |
*** takedakn has joined #openstack-oslo | 00:22 | |
*** takedakn has quit IRC | 00:26 | |
openstackgerrit | Richard Jones proposed a change to openstack/oslo-specs: Add CORS middleware spec https://review.openstack.org/119485 | 01:10 |
---|---|---|
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add the database schema to the sqlalchemy persistence section https://review.openstack.org/127398 | 01:21 |
*** sunrenjie has joined #openstack-oslo | 01:38 | |
*** zzzeek has quit IRC | 01:45 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 02:10 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 02:14 | |
*** HenryG has joined #openstack-oslo | 02:26 | |
*** harlowja is now known as harlowja_away | 02:37 | |
*** sunrenjie has quit IRC | 02:45 | |
*** liusheng has quit IRC | 02:55 | |
*** liusheng has joined #openstack-oslo | 02:56 | |
*** openstackgerrit has quit IRC | 02:58 | |
*** yamahata has quit IRC | 03:05 | |
*** yamahata has joined #openstack-oslo | 03:05 | |
*** harlowja_away has quit IRC | 03:12 | |
*** jecarey has joined #openstack-oslo | 03:25 | |
*** andreykurilin_ has joined #openstack-oslo | 03:53 | |
*** andreykurilin_ has quit IRC | 04:10 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 04:12 | |
*** harlowja_at_home has joined #openstack-oslo | 04:18 | |
*** harlowja_at_home has quit IRC | 04:22 | |
*** harlowja_at_home has joined #openstack-oslo | 04:24 | |
*** vigneshvar has joined #openstack-oslo | 04:36 | |
*** jgrimm is now known as zz_jgrimm | 04:39 | |
*** harlowja_at_home has quit IRC | 04:40 | |
*** vigneshvar has quit IRC | 04:41 | |
*** stevemar has quit IRC | 04:43 | |
*** vigneshvar has joined #openstack-oslo | 04:57 | |
*** r1chardj0n3s_afk is now known as r1chardj0n3s | 05:02 | |
*** noelbk has joined #openstack-oslo | 05:27 | |
*** r1chardj0n3s is now known as r1chardj0n3s_afk | 05:29 | |
*** arosen1 has left #openstack-oslo | 05:58 | |
*** rpodolyaka1 has joined #openstack-oslo | 06:19 | |
*** noelbk has quit IRC | 06:29 | |
*** k4n0 has joined #openstack-oslo | 06:34 | |
*** flaper87|afk is now known as flaper87 | 06:46 | |
*** rpodolyaka has quit IRC | 06:49 | |
*** rpodolyaka has joined #openstack-oslo | 06:49 | |
*** kevinbenton has quit IRC | 06:50 | |
*** rpodolyaka1 has quit IRC | 07:14 | |
*** rpodolyaka1 has joined #openstack-oslo | 07:14 | |
*** rpodolyaka1 has quit IRC | 07:14 | |
*** k4n0 has quit IRC | 07:24 | |
*** liusheng has quit IRC | 07:28 | |
*** k4n0 has joined #openstack-oslo | 07:28 | |
*** liusheng has joined #openstack-oslo | 07:30 | |
*** ihrachyshka has joined #openstack-oslo | 08:01 | |
*** e0ne has joined #openstack-oslo | 08:34 | |
*** pblaho has joined #openstack-oslo | 08:41 | |
*** andreykurilin_ has joined #openstack-oslo | 09:54 | |
*** kevinbenton has joined #openstack-oslo | 10:03 | |
*** andreykurilin_ has quit IRC | 10:21 | |
*** alexpilotti has joined #openstack-oslo | 10:28 | |
*** pblaho has quit IRC | 10:49 | |
*** ihrachyshka has quit IRC | 10:55 | |
*** ihrachyshka has joined #openstack-oslo | 10:55 | |
*** yamahata has quit IRC | 11:09 | |
*** miqui has quit IRC | 11:34 | |
*** tedross has joined #openstack-oslo | 11:35 | |
*** ihrachyshka has quit IRC | 11:40 | |
*** k4n0 has quit IRC | 11:42 | |
*** ihrachyshka has joined #openstack-oslo | 11:58 | |
*** kgiusti has joined #openstack-oslo | 12:33 | |
*** dims has joined #openstack-oslo | 12:37 | |
dims | dhellmann: nova got rid of xmlutils, so jd__'s patch is good to go - https://review.openstack.org/#/c/125021/ | 12:38 |
*** rpodolyaka has quit IRC | 12:48 | |
*** rpodolyaka1 has joined #openstack-oslo | 12:51 | |
*** dims has quit IRC | 12:52 | |
*** dims has joined #openstack-oslo | 12:52 | |
*** miqui has joined #openstack-oslo | 12:56 | |
*** HenryG has quit IRC | 12:57 | |
*** jecarey has quit IRC | 13:00 | |
*** jeckersb_gone is now known as jeckersb | 13:00 | |
*** e0ne has quit IRC | 13:03 | |
*** e0ne has joined #openstack-oslo | 13:04 | |
*** bknudson has joined #openstack-oslo | 13:05 | |
*** rpodolyaka1 has quit IRC | 13:12 | |
*** jecarey has joined #openstack-oslo | 13:14 | |
*** gordc has joined #openstack-oslo | 13:31 | |
*** kgiusti has quit IRC | 13:31 | |
*** kgiusti has joined #openstack-oslo | 13:32 | |
*** takedakn has joined #openstack-oslo | 13:44 | |
gordc | ihrachyshka: sorry, i probably should've posted here before starting. | 13:54 |
ihrachyshka | gordc: hey :) np. | 13:55 |
gordc | ihrachyshka: i also put up patches for cinder... i haven't done anything for nova yet. | 13:55 |
ihrachyshka | gordc: I'm interested in neutron only (atm) | 13:55 |
gordc | ihrachyshka: ah i see. i'll remember to post here first next time i start with neutron | 13:55 |
ihrachyshka | gordc: I'm a neutron liaison for kilo, so it would be great to know in advance :) do you have oslo.i18n effort maybe? | 13:56 |
gordc | ihrachyshka: no. i was just trying to graduate oslo.middleware and i ended up having to pull/sync a versionutils and oslo.serialization. | 13:56 |
gordc | my work will stop there. | 13:57 |
*** sigmavirus24_awa is now known as sigmavirus24 | 13:57 | |
ihrachyshka | ok, got it | 13:57 |
ihrachyshka | gordc: for middleware, I wonder whether we're able to fix grenade by adding entry points | 13:58 |
gordc | ihrachyshka: hmm.. i didn't think of that. | 13:58 |
ihrachyshka | gordc: in setup.cfg, see oslo.messaging.notify.drivers | 13:58 |
ihrachyshka | I don't know whether it will really work, but it's worth trying. we better nuke the pile of code ;) | 13:59 |
gordc | the main reason i kept it was because when i tried to pull it out in cinder. the grenade tests failed because the api-paste.ini file referenced the openstack.common.middleware items | 13:59 |
gordc | ihrachyshka: yeah, that's a good point. i'll give it a try. | 13:59 |
ihrachyshka | yeah, but *maybe* entry points are useful here | 14:00 |
ihrachyshka | good, thanks | 14:00 |
*** e0ne has quit IRC | 14:09 | |
dhellmann | dims, jd__ : +2a | 14:12 |
*** e0ne has joined #openstack-oslo | 14:15 | |
*** openstackgerrit has joined #openstack-oslo | 14:15 | |
dims | dhellmann: thanks | 14:16 |
*** jaypipes is now known as leakypipes | 14:16 | |
*** vigneshvar has quit IRC | 14:17 | |
*** zzzeek has joined #openstack-oslo | 14:17 | |
*** e0ne has quit IRC | 14:17 | |
*** e0ne has joined #openstack-oslo | 14:18 | |
viktors | dims: hi! | 14:18 |
dims | viktors: hi | 14:19 |
viktors | dims: do you plan to continue your work on patch https://review.openstack.org/#/c/124096/ (Add uuidutils to oslo.utils) ? | 14:19 |
viktors | I had some ideas as for is_uuid_like() function | 14:19 |
dims | viktors: awesome, feel free to take over or start a new review. | 14:20 |
viktors | dims: I'll reload your patch and add co-authored. Is it ok for you? | 14:20 |
dims | sounds good | 14:20 |
dims | and thanks viktors | 14:21 |
viktors | no problem, I'll ping you when I'll upload net patch set | 14:22 |
*** mtanino has joined #openstack-oslo | 14:27 | |
*** zz_jgrimm is now known as jgrimm | 14:29 | |
*** HenryG has joined #openstack-oslo | 14:31 | |
*** takedakn has quit IRC | 14:36 | |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo.db: Set utf8 encoding for mysql and postgresql https://review.openstack.org/111236 | 14:37 |
*** stevemar has joined #openstack-oslo | 14:40 | |
*** dhellmann is now known as dhellmann_ | 14:40 | |
*** dhellmann_ is now known as dhellmann | 14:41 | |
gordc | ihrachyshka: do you know if setup.cfg tweak will work even if class names are different? | 14:47 |
gordc | https://review.openstack.org/#/c/127036/5/etc/api-paste.ini | 14:47 |
ihrachyshka | gordc: yeah, it was the case for oslo.messaging | 14:50 |
*** i159 has joined #openstack-oslo | 14:53 | |
*** ihrachyshka is now known as ihrachyshka|afk | 14:53 | |
gordc | ihrachyshka|afk: so something similar to this: neutron.openstack.common.middleware.request_id = oslo.middleware:RequestId? | 14:54 |
openstackgerrit | Victor Sergeyev proposed a change to openstack/oslo.utils: Add uuidutils to oslo.utils https://review.openstack.org/124096 | 15:01 |
viktors | dims: please tell, what do you think about ^^ | 15:01 |
*** kgiusti has quit IRC | 15:08 | |
*** kgiusti has joined #openstack-oslo | 15:09 | |
*** tsekiyama has joined #openstack-oslo | 15:10 | |
flaper87 | dhellmann: hey, question. stevedore caches loaded plugins, right? As in, if I request to load the same plugin twice, the second time it'll get it from a local cache. | 15:11 |
* flaper87 can't remember | 15:11 | |
dhellmann | flaper87: yes, once the set of plugins is scanned, that information is cached and when the module is imported that module stays in sys.modules | 15:14 |
flaper87 | dhellmann: right. cool! Thanks :) | 15:14 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.messaging: RabbitMQ heartbeat implementation https://review.openstack.org/126330 | 15:17 |
dims | viktors: left a comment. thanks again for picking this up | 15:18 |
openstackgerrit | Ken Giusti proposed a change to openstack/oslo.messaging: Implement message flow control using link credit https://review.openstack.org/127550 | 15:19 |
*** dims is now known as dimsum_ | 15:21 | |
viktors | dimsum_: thanks for comment, left an answer | 15:23 |
dimsum_ | replied :) | 15:25 |
dimsum_ | 0) remove "urn:" 1) remove "uuid:" 3) strip non-alnum | 15:27 |
dimsum_ | that should work right? | 15:27 |
viktors | dimsum_: yes, this should work | 15:28 |
*** krotscheck has quit IRC | 15:28 | |
*** noelbk has joined #openstack-oslo | 15:29 | |
viktors | dimsum_: can it be made in a single regexp? :) | 15:29 |
*** krotscheck has joined #openstack-oslo | 15:29 | |
dimsum_ | viktors: nice challenge :) | 15:30 |
dimsum_ | viktors: sometimes i use this tool to test regexp(s) http://regex101.com/#python | 15:30 |
*** noelbkdeux has joined #openstack-oslo | 15:32 | |
*** noelbk has quit IRC | 15:32 | |
*** noelbkdeux is now known as noelbk | 15:32 | |
viktors | dimsum_: thanks, interesting tool | 15:35 |
dimsum_ | viktors: pattern = re.compile('(uuid|urn)[:]*[\W_]+') | 15:39 |
dimsum_ | seems to do the trick | 15:39 |
*** kgiusti has quit IRC | 15:41 | |
* viktors checking | 15:42 | |
*** kgiusti has joined #openstack-oslo | 15:43 | |
*** harlowja_at_home has joined #openstack-oslo | 15:45 | |
*** andreykurilin has joined #openstack-oslo | 15:46 | |
andreykurilin | dhellmann, hi! | 15:47 |
andreykurilin | dhellmann, What do you think about openstack.common.cliutils ? I mean, graduation to oslo.utils... | 15:50 |
dhellmann | andreykurilin: if we graduate that, it will go into its own library | 15:50 |
andreykurilin | dhellmann, why not oslo.utils? | 15:50 |
dhellmann | andreykurilin: we need to determine whether that code is stable, used, etc. | 15:50 |
dhellmann | andreykurilin: because oslo.utils is a general purpose utility library, with a different reviewer base than cliutils | 15:51 |
dhellmann | andreykurilin: we split code out into separate libraries to manage dependencies, and to manage reviwer teams | 15:51 |
dhellmann | andreykurilin: we're working on our kilo plans in https://etherpad.openstack.org/p/kilo-oslo-library-proposals if you want to have a look | 15:52 |
andreykurilin | dhellmann: thanks for link | 15:53 |
viktors | dimsum_: pattern.sub('', '{12345678-1234-5678-1234-567812345678}') -> '{12345678-1234-5678-1234-567812345678}' | 15:55 |
andreykurilin | dhellmann, several functions from cliutils are used not only in clients code. dims have a patch to nova(sync latest code from incubator - https://review.openstack.org/#/c/126003/ ) and nova guys don't want to have additional dependency(they don't want apiclient code). | 15:56 |
dimsum_ | andreykurilin: y am looking at that. any suggestions? | 15:56 |
andreykurilin | dimsum_: hi! | 15:56 |
*** e0ne has quit IRC | 15:58 | |
*** stevemar has quit IRC | 15:58 | |
andreykurilin | dimsum_: imo, it would be better to split cliutils to two modules. Function cliutils.find_resource (and others) should go to apiclient.utils and exceptions, which are related to cliutils should go to cliutils | 15:59 |
*** stevemar has joined #openstack-oslo | 16:00 | |
dimsum_ | andreykurilin: want to file a review? | 16:01 |
andreykurilin | dimsum_: ok, I will do it | 16:02 |
dimsum_ | thanks andreykurilin | 16:02 |
*** yamahata has joined #openstack-oslo | 16:11 | |
openstackgerrit | Michael Bayer proposed a change to openstack/oslo-specs: Make EngineFacade a Facade https://review.openstack.org/125181 | 16:15 |
openstackgerrit | Andrey Kurilin proposed a change to openstack/oslo-incubator: Split cliutils https://review.openstack.org/127568 | 16:18 |
andreykurilin | dimsum_: ^ | 16:20 |
*** yamahata has quit IRC | 16:24 | |
dimsum_ | andreykurilin: do we need an alias in cliutils that points to find_resource in the new spot? | 16:25 |
andreykurilin | dimsum_: if we add alias, we will not have ability to remove dependency of apiclient in cliutils. | 16:28 |
dimsum_ | ah right | 16:29 |
dimsum_ | thanks | 16:29 |
andreykurilin | dimsum_: imo, aliases is not helpful, because developers will not switch to use something from another place until old implementation is work successful:) | 16:29 |
dimsum_ | right | 16:30 |
*** e0ne has joined #openstack-oslo | 16:46 | |
*** kgiusti has quit IRC | 16:52 | |
*** kgiusti has joined #openstack-oslo | 16:53 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 16:55 | |
*** hartsocks has left #openstack-oslo | 16:59 | |
*** rpodolyaka1 has joined #openstack-oslo | 17:02 | |
*** i159 has quit IRC | 17:04 | |
*** harlowja_at_home has quit IRC | 17:04 | |
dimsum_ | beekneemech: dhellmann: please take a look at this - https://review.openstack.org/#/c/127568/ - need it for Nova reviews. they don't like pulling in apiclient | 17:07 |
*** viktors is now known as viktors|afk | 17:08 | |
dhellmann | dimsum_: on my way to lunch, but I will after | 17:12 |
dimsum_ | thanks dhellmann | 17:12 |
*** kgiusti has quit IRC | 17:12 | |
dhellmann | dimsum_, they want cliutils but not apiutils? | 17:13 |
dhellmann | or apiclient, I mean | 17:13 |
dimsum_ | dhellmann: correct | 17:13 |
dhellmann | I'm not sure I understand why. Seems reasonable to have cliutils depend on apiutils. | 17:14 |
dhellmann | dimsum_: this patch introduces a dependency the other way, and that doesn't seem like it makes sense | 17:14 |
*** kgiusti has joined #openstack-oslo | 17:16 | |
*** jecarey has quit IRC | 17:16 | |
dhellmann | dimsum_: left a comment on the review, bbiab | 17:16 |
openstackgerrit | A change was merged to openstack/oslo.i18n: Make clear in docs to use _LE() when using LOG.exception() https://review.openstack.org/126419 | 17:24 |
stevemar | dimsum_, ping | 17:28 |
dimsum_ | stevemar: pong | 17:28 |
stevemar | dimsum_, i'm here to plead/bribe you for a review of https://review.openstack.org/#/c/120316/, name your price | 17:29 |
dimsum_ | stevemar: ouch apologies | 17:29 |
*** takedakn has joined #openstack-oslo | 17:34 | |
dimsum_ | stevemar: the last bit threw me off..what does the line with the grep do? 34,35 in oslo_debug_helper | 17:34 |
*** zzzeek has quit IRC | 17:35 | |
stevemar | dimsum_, that's there from the old version (line 14) | 17:36 |
stevemar | basically, if the user passes in oslo_debug_helper test_identity_v3, it'll do a grep for that against the whole list of possible tests | 17:37 |
stevemar | and use that result to only run those tests | 17:37 |
*** takedakn has quit IRC | 17:38 | |
dimsum_ | instead of specifying one single test, you are specifying a pattern | 17:38 |
dimsum_ | thanks | 17:39 |
stevemar | thanks dimsum_ ! | 17:40 |
dimsum_ | stevemar: don't wait too long next time, just ping :) | 17:41 |
*** zzzeek has joined #openstack-oslo | 17:42 | |
stevemar | dimsum_, haha, i feel bad bugging folks | 17:42 |
*** jeckersb is now known as jeckersb_gone | 17:44 | |
dimsum_ | stevemar: i don't mind at all | 17:46 |
*** exploreshaifali has joined #openstack-oslo | 17:53 | |
*** harlowja has joined #openstack-oslo | 17:54 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 17:58 | |
*** jecarey has joined #openstack-oslo | 18:10 | |
*** e0ne has quit IRC | 18:21 | |
*** andreykurilin_ has joined #openstack-oslo | 18:25 | |
harlowja | bknudson do u have any more details on the 'encryption of config options' topic? | 18:26 |
harlowja | i know its been a request at y! | 18:26 |
*** vigneshvar has joined #openstack-oslo | 18:28 | |
harlowja | we have a system that appears similar to https://wiki.openstack.org/wiki/Barbican and being able to plug oslo.config into that would make certain security people be able to sleep better | 18:30 |
bknudson | harlowja: I don't have more details... the goal is to have a discussion and come up with something that will work for people | 18:32 |
bknudson | if it involves barbican or something that would be interesting | 18:32 |
bknudson | but maybe all we do is have a plugin that allows you to get the config value | 18:33 |
bknudson | and that plugin can do whatever... | 18:34 |
bknudson | do some simple obfuscation, do some encryption, talk to barbican, whatever. | 18:34 |
*** stevemar has quit IRC | 18:43 | |
*** stevemar has joined #openstack-oslo | 18:44 | |
*** kgiusti has quit IRC | 18:45 | |
*** jogo is now known as flashgordon | 18:46 | |
*** stevemar2 has joined #openstack-oslo | 18:46 | |
*** stevemar has quit IRC | 18:47 | |
*** openstackgerrit has quit IRC | 18:48 | |
*** kgiusti has joined #openstack-oslo | 18:49 | |
*** openstackgerrit has joined #openstack-oslo | 18:54 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo.utils: Add uuidutils to oslo.utils https://review.openstack.org/124096 | 18:57 |
dimsum_ | viktors|afk: i've added some more tests | 18:57 |
*** rpodolyaka2 has joined #openstack-oslo | 18:59 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/oslo-incubator: Split cliutils https://review.openstack.org/127568 | 19:06 |
harlowja | bknudson cool, sounds good | 19:07 |
*** rpodolyaka2 has quit IRC | 19:38 | |
*** liusheng has quit IRC | 19:39 | |
*** liusheng has joined #openstack-oslo | 19:39 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo-incubator: Remove middleware https://review.openstack.org/123656 | 19:41 |
*** e0ne has joined #openstack-oslo | 19:41 | |
*** kgiusti has quit IRC | 19:43 | |
*** kgiusti has joined #openstack-oslo | 19:44 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Allow dictionary lookup in credentials with dot notation https://review.openstack.org/123909 | 19:53 |
*** stevemar2 is now known as stevemar | 19:57 | |
*** rpodolyaka2 has joined #openstack-oslo | 19:59 | |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo-incubator: Remove middleware https://review.openstack.org/123656 | 20:04 |
*** kgiusti has quit IRC | 20:05 | |
*** kgiusti has joined #openstack-oslo | 20:06 | |
*** alexpilotti has quit IRC | 20:06 | |
dimsum_ | dhellmann: resurrected jd's oslo.middleware cleanup patch - https://review.openstack.org/#/c/123656/ | 20:06 |
dhellmann | dimsum_: +2 | 20:09 |
*** ihrachyshka|afk has quit IRC | 20:09 | |
dimsum_ | thanks! | 20:10 |
*** e0ne has quit IRC | 20:10 | |
*** stevemar has quit IRC | 20:12 | |
gordc | dhellmann: i'm trying to graduate the middleware in the projects but i've no idea how to use entry_points. is there a way to link openstack.common.middleware.request_id:RequestIdMiddleware to oslo.middleware:RequestId ? | 20:13 |
dhellmann | gordc: I'm not sure how anyone is using the middleware now, can you point me at some code that you're working on? | 20:14 |
gordc | https://review.openstack.org/#/c/127028/ | 20:15 |
dhellmann | gordc: it looks like you're updating the paste.ini as expected. Where's the problem? | 20:16 |
gordc | dhellmann: the issue arises in grenade tests because api-paste.ini points to openstack.common in icehouse/juno but if we remove the code in kilo, the upgrade will fail because the paste file still points to openstack.common | 20:17 |
gordc | ie. it's fine for a free install but an upgrade will have issues. | 20:17 |
gordc | fresh* | 20:17 |
dhellmann | gordc: ok, so 2 issues there. First, the paste file isn't using entry points afaik. It's parsing the value given and doing the import itself. Second, grenade should update the config file with the code, maybe? | 20:17 |
openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo-incubator: Mask keystone token in debug output https://review.openstack.org/122977 | 20:18 |
dhellmann | hrm, maybe paste does use entry points | 20:18 |
dhellmann | well, not "does" but "can" http://pythonpaste.org/deploy/#egg-uris | 20:19 |
dhellmann | gordc: ^^ | 20:19 |
dhellmann | gordc: unfortunately that won't help if the old version of the file doesn't already use that syntax | 20:19 |
gordc | dhellmann: hmm. i'll take a look at that. | 20:19 |
dimsum_ | gordc: can you look at https://review.openstack.org/#/c/123656/ as well please :) | 20:20 |
dhellmann | gordc: if *that* doesn't work, we'll have to create some modules in the incubator so the import aliases will work, and then you can sync those into cinder | 20:20 |
gordc | dhellmann: yeah... i think i'll have an issue justifying changing the values in grenade. as i understand it, it's to make sure this backward compat scenario works | 20:21 |
gordc | dhellmann: you mean change current oslo-incubator code to just point to oslo.middleware? | 20:22 |
gordc | dimsum_: i think the patch is fine assuming we figure a way to resolve upgrade path for existing api-paste.ini files | 20:22 |
dhellmann | gordc: part of the upgrade is upgrading the config, though, right? | 20:22 |
dhellmann | gordc: yeah, if we have to, we can create some dummy modules in the incubator that just import from oslo.middleware | 20:23 |
* gordc is a pessimist | 20:24 | |
gordc | dhellmann: i'll try asking qa folks to see what they think. | 20:24 |
*** bknudson has quit IRC | 20:24 | |
gordc | or i'll check to see if there's precedence | 20:24 |
dhellmann | gordc: ok | 20:24 |
*** kgiusti has quit IRC | 20:25 | |
*** kgiusti has joined #openstack-oslo | 20:26 | |
*** kgiusti has left #openstack-oslo | 20:32 | |
*** e0ne has joined #openstack-oslo | 20:38 | |
*** harlowja is now known as harlowja_away | 20:43 | |
*** rpodolyaka2 has quit IRC | 20:44 | |
*** harlowja_away is now known as harlowja | 20:47 | |
*** e0ne has quit IRC | 20:48 | |
*** jgrimm is now known as zz_jgrimm | 20:52 | |
*** e0ne has joined #openstack-oslo | 20:52 | |
*** e0ne has quit IRC | 20:53 | |
*** dhellmann has quit IRC | 20:56 | |
*** dhellmann has joined #openstack-oslo | 20:57 | |
*** miqui has quit IRC | 20:57 | |
*** dhellmann-phone has joined #openstack-oslo | 21:02 | |
*** jecarey has quit IRC | 21:02 | |
*** dhellmann-phone has quit IRC | 21:02 | |
*** rpodolyaka2 has joined #openstack-oslo | 21:07 | |
*** rpodolyaka2 has quit IRC | 21:08 | |
*** jecarey has joined #openstack-oslo | 21:08 | |
*** jecarey_ has joined #openstack-oslo | 21:18 | |
harlowja | anyone feel like doing a simple doc review @ https://review.openstack.org/#/c/127398/ ; nice for taskflow docs | 21:19 |
harlowja | could be useful for others docs also i think | 21:19 |
harlowja | https://review.openstack.org/#/c/127398/3/tools/schema_generator.py isn't really taskflow specific | 21:19 |
dimsum_ | gordc: dhellmann: Any notes we can leave on this review in Nova for the api-paste.ini problem? https://review.openstack.org/#/c/127057/ | 21:20 |
*** jecarey has quit IRC | 21:20 | |
dimsum_ | harlowja: ack | 21:20 |
dimsum_ | harlowja: output looks nice! | 21:21 |
dhellmann | dimsum_: gordc was looking at something like that earlier, and was having some trouble with grenade not updating the cinder config files when it updates the code, so the paste references to the old middleware weren't working | 21:22 |
harlowja | dimsum_ thx, that generator maybe others can use, it should work with anything that can create a sqlite schema and pop out a view of those tables schemas in rst | 21:23 |
harlowja | i didn't find something that simple (found http://schemaspy.sourceforge.net/ but thats way to much overkill) | 21:23 |
openstackgerrit | A change was merged to openstack/oslo-incubator: Remove middleware https://review.openstack.org/123656 | 21:32 |
gordc | dimsum_: i don't know how to snip parts of conversation but if you read eavesdrop between me and mtreinish i think we have a solution: http://eavesdrop.openstack.org/irclogs/%23openstack-qa/%23openstack-qa.2014-10-10.log | 21:34 |
gordc | requires grenade changes though. | 21:34 |
gordc | definitely won't be a easy clean swap. | 21:35 |
dimsum_ | gordc: ack. will cut n paste into abishek's nova review | 21:36 |
gordc | dimsum_: sounds good. thanks for keeping track of that (before i wasted time and tried to update nova myself) | 21:38 |
dimsum_ | yw! and have a nice thanksgiving | 21:42 |
*** gordc has quit IRC | 21:46 | |
*** zzzeek has quit IRC | 21:46 | |
openstackgerrit | Amaury Medeiros proposed a change to openstack/oslo-incubator: Fix attributes ordering at common/imageutils.py https://review.openstack.org/127050 | 21:46 |
*** andreykurilin_ has quit IRC | 22:08 | |
*** AAzza has quit IRC | 22:29 | |
*** alexpilotti has joined #openstack-oslo | 22:30 | |
*** alexpilotti has quit IRC | 22:31 | |
*** AAzza_afk has joined #openstack-oslo | 22:32 | |
*** AAzza_afk is now known as AAzza | 22:32 | |
*** dimsum_ has quit IRC | 22:43 | |
*** flaper87 is now known as flaper87|afk | 22:46 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 22:47 | |
*** openstackgerrit has quit IRC | 22:54 | |
*** tedross has quit IRC | 23:13 | |
*** tsekiyama has quit IRC | 23:16 | |
*** dimsum_ has joined #openstack-oslo | 23:44 | |
*** dimsum_ has quit IRC | 23:49 | |
*** noelbk has quit IRC | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!