*** zzzeek has quit IRC | 00:07 | |
*** bknudson has joined #openstack-oslo | 00:08 | |
*** zzzeek has joined #openstack-oslo | 00:13 | |
*** zzzeek has quit IRC | 00:14 | |
*** zzzeek has joined #openstack-oslo | 00:33 | |
*** markmcclain has quit IRC | 00:48 | |
*** arnaud has quit IRC | 01:06 | |
*** zzzeek has quit IRC | 01:26 | |
*** Volkard has joined #openstack-oslo | 01:29 | |
*** zzzeek has joined #openstack-oslo | 01:29 | |
*** dims__ has quit IRC | 01:35 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/99084 | 01:37 |
---|---|---|
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements https://review.openstack.org/99085 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/99086 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Updated from global requirements https://review.openstack.org/96239 | 01:37 |
*** Volkard has left #openstack-oslo | 01:37 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/99087 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslotest: Updated from global requirements https://review.openstack.org/99088 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/99089 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslosphinx: Updated from global requirements https://review.openstack.org/99090 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 01:37 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 01:38 |
*** zzzeek has quit IRC | 01:50 | |
*** HenryG has joined #openstack-oslo | 01:58 | |
*** arnaud has joined #openstack-oslo | 02:05 | |
*** bnemec is now known as bnemec_away | 02:24 | |
*** gcb has joined #openstack-oslo | 02:44 | |
*** gcb has quit IRC | 03:12 | |
*** yamahata has joined #openstack-oslo | 03:15 | |
*** yamahata has quit IRC | 03:16 | |
*** zzzeek has joined #openstack-oslo | 03:20 | |
*** gcb has joined #openstack-oslo | 03:24 | |
*** harlowja is now known as harlowja_away | 03:24 | |
*** yamahata has joined #openstack-oslo | 03:35 | |
*** jecarey has joined #openstack-oslo | 03:51 | |
*** arnaud has quit IRC | 04:00 | |
*** praneshp has quit IRC | 04:25 | |
*** praneshp has joined #openstack-oslo | 04:51 | |
*** praneshp_ has joined #openstack-oslo | 04:56 | |
*** praneshp has quit IRC | 04:58 | |
*** praneshp_ is now known as praneshp | 04:58 | |
*** ildikov has quit IRC | 05:00 | |
openstackgerrit | Joe Gordon proposed a change to openstack/oslo-incubator: Make policy debug logging less verbose https://review.openstack.org/99269 | 05:01 |
*** zzzeek has quit IRC | 05:16 | |
*** arnaud has joined #openstack-oslo | 05:30 | |
*** arnaud has quit IRC | 05:36 | |
*** arnaud has joined #openstack-oslo | 05:42 | |
*** ildikov has joined #openstack-oslo | 05:46 | |
*** arnaud has quit IRC | 05:51 | |
*** arnaud has joined #openstack-oslo | 05:54 | |
*** arnaud has quit IRC | 06:07 | |
*** praneshp has quit IRC | 06:20 | |
openstackgerrit | Juan Antonio Osorio Robles proposed a change to openstack/oslo.i18n: Trivial refactors for gettextutils https://review.openstack.org/98429 | 06:22 |
*** praneshp has joined #openstack-oslo | 06:23 | |
*** jaosorior has joined #openstack-oslo | 06:23 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Denote that other projects can use this library https://review.openstack.org/98624 | 06:26 |
jaosorior | haypo, are you around? | 06:28 |
*** arnaud has joined #openstack-oslo | 06:34 | |
openstackgerrit | Steve Kowalik proposed a change to openstack-dev/pbr: Register testr as a distutil entry point https://review.openstack.org/99277 | 06:34 |
*** zhiyan_ is now known as zhiyan | 06:37 | |
*** arnaud has quit IRC | 06:50 | |
*** viktors|afk is now known as viktors | 07:03 | |
openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo-specs: Add ConfigFilter wrapper class for ConfigOpts https://review.openstack.org/97228 | 07:18 |
*** tkelsey has joined #openstack-oslo | 07:21 | |
*** praneshp has quit IRC | 07:24 | |
*** flaper87|afk is now known as flaper87 | 07:25 | |
*** markmc has joined #openstack-oslo | 07:32 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/oslo-incubator: Add URI validation method https://review.openstack.org/99288 | 07:39 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/oslo-incubator: Add URI validation method https://review.openstack.org/99288 | 07:40 |
*** mrda is now known as mrda-away | 07:42 | |
haypo | jaosorior: hello | 07:52 |
therve | sileht, Hey, do you know about the allowed_rpc_exception_modules options in oslo.messaging? | 07:55 |
therve | It seems to be defined but not registered | 07:55 |
sileht | therve, the rule about cfg option of oslo.messaging is, none of them should be read into the application, if you need to acces to one of them, perhaps a public API in the library is missing | 07:56 |
haypo | jaosorior: "system_locale[0] or 'en_US'" is nice, it avoids to compute system_locale[0] twice :) | 07:56 |
haypo | jaosorior: i didn't agree fully with your change, but i don't want to spend too much time on a minor refactoring patch :) your change is good enough | 07:57 |
haypo | jaosorior: (and we don't have to agree fully on the coding style ;)) | 07:57 |
therve | sileht, Interestingly it seems to be defined in oslo.messaging but not used :) | 07:58 |
therve | Still, I find it strange that it's not registered | 07:58 |
sileht | therve, yes, I just see that | 07:58 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Removes unused config options https://review.openstack.org/99291 | 08:01 |
sileht | therve, removed ^^ | 08:01 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Removes unused config option https://review.openstack.org/99291 | 08:01 |
therve | Weird I can't even +1 that project | 08:02 |
jaosorior | haypo: oh, I see you already reviewed the change. Thanks for the feedback :) | 08:03 |
haypo | jaosorior: no problem | 08:03 |
haypo | jaosorior: do you know if oslo.i18n is already used? | 08:04 |
jaosorior | Not in the projects I have contributed to | 08:05 |
jaosorior | But after graduation It should be easy enough to start using it | 08:05 |
haypo | it looks like doug wants to cleanup which symbols are exported, and do other cleanup, before using this new library | 08:06 |
jaosorior | I commented on his patch | 08:08 |
jaosorior | As mentioned there, i would prefer the access to objects would be done in the __init__.py instead of the separation and reliance on the underscore notation | 08:09 |
jaosorior | haypo: do you have some insights? | 08:15 |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/oslo-incubator: Add URI validation method https://review.openstack.org/99288 | 08:21 |
*** ihrachyshka has joined #openstack-oslo | 08:22 | |
*** ihrachyshka has quit IRC | 08:27 | |
*** ihrachyshka has joined #openstack-oslo | 08:27 | |
haypo | jaosorior: i don't think that __init__ (oslo.i18n) is currently the main API, but oslo.i18n.gettextutils | 08:35 |
haypo | jaosorior: i'm not sure that we need a directory, a simple long i18n.py file may be enough | 08:35 |
haypo | (and hide private classes, functions and constants using an underscore prefix in names) | 08:35 |
*** gcb has quit IRC | 08:38 | |
openstackgerrit | Ken'ichi Ohmichi proposed a change to openstack/oslo-incubator: Add URI validation method https://review.openstack.org/99288 | 08:45 |
sileht | markmc, Have you find some times for https://review.openstack.org/#/c/75365/ ? | 08:45 |
jaosorior | Wouldn't it be easier to actually have oslo.i18n as thr main API and manage private objects using __init__.py? | 08:45 |
*** amotoki has joined #openstack-oslo | 08:46 | |
*** gcb has joined #openstack-oslo | 08:52 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Removes unused config option https://review.openstack.org/99291 | 09:13 |
openstackgerrit | Ilya Shakhat proposed a change to openstack/cliff: CSV formatter should use system-dependent line ending https://review.openstack.org/99106 | 09:48 |
*** gcb has quit IRC | 09:54 | |
*** zhiyan is now known as zhiyan_ | 10:00 | |
ihrachyshka | dhellmann: hey, I have a back port for stable (not json, another one) that was already handled incorrectly (pushed to stable neutron and ceilometer before reaching incubator stable branch). Can we give it a priority to avoid inconsistencies? | 10:02 |
*** zhiyan_ is now known as zhiyan | 10:03 | |
*** yamahata has quit IRC | 10:23 | |
*** HenryG has quit IRC | 10:25 | |
*** zhiyan is now known as zhiyan_ | 10:28 | |
*** ajo has quit IRC | 10:51 | |
*** dims__ has joined #openstack-oslo | 11:01 | |
*** jaosorior has quit IRC | 11:14 | |
*** ujjain has quit IRC | 11:14 | |
*** ujjain has joined #openstack-oslo | 11:14 | |
*** jaosorior has joined #openstack-oslo | 11:15 | |
*** jecarey has quit IRC | 11:18 | |
*** jecarey has joined #openstack-oslo | 11:18 | |
*** jecarey has quit IRC | 11:19 | |
*** jecarey has joined #openstack-oslo | 11:19 | |
*** krotscheck has quit IRC | 11:19 | |
*** krotscheck has joined #openstack-oslo | 11:21 | |
*** tkelsey has quit IRC | 11:30 | |
viktors | dhellmann: hi! Please ping me or rpodolyaka when you're available. | 11:40 |
*** ihrachyshka has quit IRC | 11:40 | |
*** HenryG has joined #openstack-oslo | 11:43 | |
*** yamahata has joined #openstack-oslo | 11:45 | |
*** jecarey_ has joined #openstack-oslo | 11:47 | |
*** ihrachyshka has joined #openstack-oslo | 11:48 | |
*** jecarey has quit IRC | 11:50 | |
*** anteaya has quit IRC | 11:58 | |
*** erecio has quit IRC | 12:05 | |
*** erecio has joined #openstack-oslo | 12:09 | |
*** erecio has quit IRC | 12:15 | |
*** zzzeek has joined #openstack-oslo | 12:28 | |
*** dims__ has quit IRC | 12:29 | |
*** zzzeek has quit IRC | 12:31 | |
*** dims__ has joined #openstack-oslo | 12:36 | |
*** anteaya has joined #openstack-oslo | 12:39 | |
*** oomichi_ has joined #openstack-oslo | 12:40 | |
*** oomichi has quit IRC | 12:41 | |
*** lbragstad has joined #openstack-oslo | 12:46 | |
*** oomichi_ is now known as oomichi_sleeping | 12:47 | |
*** tkelsey has joined #openstack-oslo | 12:57 | |
*** jraim has quit IRC | 13:00 | |
*** zhiyan_ is now known as zhiyan | 13:00 | |
*** jraim has joined #openstack-oslo | 13:01 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Add a base test case for DB schema comparison https://review.openstack.org/93398 | 13:05 |
*** mriedem has joined #openstack-oslo | 13:05 | |
*** I159 has joined #openstack-oslo | 13:06 | |
*** I159 has quit IRC | 13:06 | |
*** ujjain has quit IRC | 13:07 | |
*** I159 has joined #openstack-oslo | 13:07 | |
*** I159 has quit IRC | 13:07 | |
*** markmc has quit IRC | 13:08 | |
*** markmc has joined #openstack-oslo | 13:09 | |
*** jecarey_ has quit IRC | 13:10 | |
*** ujjain has joined #openstack-oslo | 13:10 | |
*** jraim has quit IRC | 13:15 | |
*** jraim has joined #openstack-oslo | 13:17 | |
sileht | some cores to take a look to this topics https://review.openstack.org/#/q/status:open+project:openstack/oslo.messaging+branch:master+topic:bug/1282639,n,z ? | 13:30 |
*** bknudson has quit IRC | 13:39 | |
*** yamahata has quit IRC | 13:47 | |
*** yamahata has joined #openstack-oslo | 13:47 | |
*** zzzeek has joined #openstack-oslo | 13:53 | |
openstackgerrit | Brent Eagles proposed a change to openstack/oslo-incubator: Override DeprecationWarning visibility https://review.openstack.org/99378 | 13:55 |
*** bknudson has joined #openstack-oslo | 13:56 | |
viktors | dhellmann: around? | 14:07 |
openstackgerrit | Brent Eagles proposed a change to openstack/oslo.db: Override DeprecationWarning visibility https://review.openstack.org/99394 | 14:22 |
*** tsekiyama has joined #openstack-oslo | 14:24 | |
*** zhiyan is now known as zhiyan_ | 14:26 | |
*** czzl has quit IRC | 14:27 | |
openstackgerrit | Nataliia Uvarova proposed a change to openstack/oslo-incubator: Fix common.log.ContextFormatter for Python 3 https://review.openstack.org/95439 | 14:44 |
*** jecarey has joined #openstack-oslo | 15:04 | |
dhellmann | ihrachyshka: have a link to that backport? | 15:29 |
dhellmann | viktors: hi | 15:30 |
*** ihrachyshka_ has joined #openstack-oslo | 15:31 | |
ihrachyshka_ | dhellmann: actually, it was pushed today, so no problem here. but please also check the same patch for havana: https://review.openstack.org/99317 | 15:32 |
*** ihrachyshka_ has quit IRC | 15:32 | |
dhellmann | ihrachyshka: approved | 15:34 |
*** ihrachyshka has quit IRC | 15:35 | |
openstackgerrit | Thomas Herve proposed a change to openstack/oslo-incubator: Add a RequestContext.from_dict method https://review.openstack.org/99409 | 15:46 |
therve | sileht, ^^^ | 15:46 |
sileht | therve, cool | 15:47 |
*** jaosorior has quit IRC | 15:52 | |
ttx | dhellmann: around? | 15:57 |
viktors | dhellmann: are you still there? | 16:11 |
*** mriedem has quit IRC | 16:13 | |
*** mriedem has joined #openstack-oslo | 16:21 | |
dhellmann | ttx: back, what's up? | 16:30 |
dhellmann | viktors: here | 16:30 |
viktors | dhellmann: I have a few questions as for oslo.db release | 16:30 |
viktors | dhellmann: recently oslo.db release blockers from list https://etherpad.openstack.org/p/oslo.db-release-blockers was merged | 16:31 |
viktors | dhellmann: so can we finally tag a release? | 16:31 |
dhellmann | viktors: good! | 16:31 |
dhellmann | viktors: I think it is safe to tag an alpha release. | 16:32 |
*** jaosorior has joined #openstack-oslo | 16:32 | |
*** jaosorior has left #openstack-oslo | 16:32 | |
dhellmann | viktors: you tell me, though :-) are you confident in what you have in the lib now? | 16:32 |
viktors | dhellmann: yes | 16:33 |
viktors | dhellmann: at least for alpha release ) | 16:33 |
dhellmann | viktors: ok, so let's tag 1.0.0a1 and see how things go | 16:34 |
dhellmann | viktors: do you know how to do that, or would you like me to do it? | 16:34 |
* dhellmann checks if viktors has permission | 16:34 | |
*** markmcclain has joined #openstack-oslo | 16:35 | |
dhellmann | viktors: you should have permission to do it yourself | 16:35 |
dhellmann | viktors: let me find the directions in the wiki | 16:35 |
* viktors looking for "how to release" manual :) | 16:36 | |
viktors | ok | 16:36 |
viktors | btw, why 1.0.0a1, not 0.1.0 | 16:36 |
viktors | http://docs.openstack.org/developer/pbr/semver.html#how-should-i-deal-with-revisions-in-the-0-y-z-initial-development-phase | 16:36 |
dhellmann | viktors: we need the a1 at the end to indicate an alpha to ensure only a wheel is published and the final version for this cycle should be 1.0.0 | 16:37 |
viktors | dhellmann: ok | 16:37 |
dhellmann | viktors: I know the current oslo docs say to use 0.1.0, but now that we can make alpha releases I need to update those instructions | 16:37 |
dhellmann | viktors: these instructions look good: https://wiki.openstack.org/wiki/Sahara/How_To_Release | 16:38 |
viktors | dhellmann: thanks | 16:39 |
*** jaosorior has joined #openstack-oslo | 16:39 | |
viktors | dhellmann: just for sure - so now release number for oslo db will be "1.0.0a1" | 16:39 |
dhellmann | viktors: yes, use 1.0.0a1 today, 1.0.0a2 next, etc. | 16:40 |
viktors | dhellmann: ok, will do. It looks simple :) | 16:40 |
dhellmann | viktors: the only tricky part is making sure you use a *signed* tag, otherwise when you push nothing happens and you lose a version number :-) | 16:40 |
viktors | dhellmann: it seems to be, that I don't have gpg key ( | 16:49 |
*** arnaud has joined #openstack-oslo | 16:51 | |
*** harlowja_away is now known as harlowja | 16:53 | |
viktors | dhellmann: I suppose to do it tomorrow with the fresh mind | 16:54 |
*** arnaud has quit IRC | 16:55 | |
*** arnaud has joined #openstack-oslo | 16:55 | |
*** praneshp has joined #openstack-oslo | 16:57 | |
*** praneshp_ has joined #openstack-oslo | 16:57 | |
*** viktors is now known as viktors|afk | 16:58 | |
dhellmann | viktors|afk: ok | 16:59 |
boris-42 | dhellmann around? | 17:01 |
*** praneshp has quit IRC | 17:01 | |
*** praneshp_ is now known as praneshp | 17:01 | |
*** zzzeek has quit IRC | 17:01 | |
*** zzzeek has joined #openstack-oslo | 17:06 | |
boris-42 | dhellmann you should be still here lol=) | 17:07 |
dhellmann | boris-42: sorry, meeting, what's up? | 17:11 |
*** ildikov has quit IRC | 17:11 | |
*** erecio has joined #openstack-oslo | 17:14 | |
dhellmann | boris-42: grabbing lunch, but I'll be back | 17:24 |
*** praneshp has quit IRC | 17:30 | |
boris-42 | dhellmann ping me whn you'll back | 17:38 |
boris-42 | dhellmann need to talk a bit about osprofiler and dependencies | 17:38 |
*** HenryG has quit IRC | 17:42 | |
dhellmann | boris-42: ok, let's try this again :-) | 17:45 |
boris-42 | dhellmann ok, so is it okay if osprofiler will be in requirements of oslo incubator? | 17:46 |
boris-42 | dhellmann and oslo messaging | 17:46 |
dhellmann | boris-42: those libraries depend on other things, so I think so. | 17:47 |
boris-42 | dhellmann I mean theoretically it is possible to make them indented using lazy stuff | 17:47 |
dhellmann | boris-42: so we could make them optional? | 17:48 |
boris-42 | dhellmann I need to put this line in oslo https://github.com/boris-42/nova/blob/1bcc59fd5bc2897d151f513118d484e9aa68ca48/nova/openstack/common/db/sqlalchemy/session.py#L648 | 17:49 |
boris-42 | dhellmann theoretically with some kind of monkey patching i can do this | 17:49 |
boris-42 | dhellmann not sure actually that I am able to add new listeners after engine.connect() call | 17:49 |
dhellmann | boris-42: no, no monkey patching -- remind me again how osprofile is turned on and off at runtime. is there an environment variable? | 17:49 |
boris-42 | dhellmann now it can be turned always on | 17:50 |
boris-42 | dhellmann and only person that knows secret key, is able to sign trace info (and make traces) | 17:50 |
*** ildikov has joined #openstack-oslo | 17:50 | |
dhellmann | boris-42: but if I don't want tracing, and I would prefer not to have the performance impact of having tracing on, how do I turn it off? | 17:50 |
boris-42 | dhellmann lemme expalin | 17:51 |
boris-42 | dhellmann it's lazy turned on | 17:51 |
boris-42 | dhellmann you are tracing only requests that have special header | 17:51 |
dhellmann | ok, so if it's not actually being used for a request there is no performance impact? | 17:51 |
boris-42 | dhellmann yep | 17:51 |
dhellmann | ok | 17:51 |
boris-42 | dhellmann probably couple of ifs | 17:51 |
boris-42 | if's* | 17:51 |
dhellmann | yeah, that's ok | 17:52 |
boris-42 | if not None* consturctions | 17:52 |
boris-42 | =) | 17:52 |
boris-42 | dhellmann as well it can be turned of fully (so even signed keys won't work) | 17:52 |
boris-42 | off* | 17:52 |
boris-42 | dhellmann and configuration of keys/enabled is in wsgi api-paste.ini | 17:53 |
boris-42 | so simple to change | 17:53 |
dhellmann | ok | 17:53 |
boris-42 | and you only need to restart actually API services | 17:53 |
dhellmann | is all of this information in the osprofiler documentation? | 17:54 |
boris-42 | dhellmann I am refactoring API in this moment | 17:55 |
boris-42 | dhellmann and I just recently put it on https://github.com/stackforge/osprofiler | 17:55 |
dhellmann | ok | 17:55 |
boris-42 | dhellmann It will be better to make it clean as possible before adding it everywhere=) | 17:55 |
boris-42 | dhellmann so unit test coverage is about 90% going to bring it to 100% | 17:56 |
boris-42 | dhellmann and it support py26,27,33 now | 17:56 |
dhellmann | I don't have any objection in principle to having oslo-incubator, oslo.messaging, and oslo.db import osprofiler when the API is stable. I'd like to get it into the oslo program at some point when you think it's stable, too (or the operators program if you want it there). | 17:56 |
boris-42 | dhellmann yep it should be somewhere | 17:57 |
boris-42 | dhellmann I will play during this week and going to make some read me file | 17:57 |
*** praneshp has joined #openstack-oslo | 17:57 | |
dhellmann | ok | 17:57 |
boris-42 | dhellmann as well we will try to make dsvm gate for it | 17:58 |
boris-42 | dhellmann something wired custom* | 17:58 |
dhellmann | you won't be able to gate openstack projects on the library until it is managed within an incubated or integrated program | 17:58 |
dhellmann | you can go the other way, though | 17:58 |
boris-42 | dhellmann just .sh script | 17:59 |
boris-42 | dhellmann that runs my devstack with my localrc | 17:59 |
boris-42 | dhellmann and then runs rally and plots that profile | 17:59 |
boris-42 | dhellmann I was thinking about something like that | 17:59 |
dhellmann | ok, I was talking more about the policy on adding gate jobs than the implementation | 18:00 |
*** markmcclain has quit IRC | 18:01 | |
boris-42 | dhellmann ah and I was just about checking that patch works* not about common gate job=) | 18:01 |
*** markmcclain has joined #openstack-oslo | 18:02 | |
dhellmann | ok, you can test changes to osprofiler that way, sure | 18:02 |
boris-42 | dhellmann yep cause it's a bit hard now=) | 18:03 |
*** markmc has quit IRC | 18:13 | |
*** tkelsey has quit IRC | 18:16 | |
*** tkelsey has joined #openstack-oslo | 18:20 | |
*** HenryG has joined #openstack-oslo | 18:20 | |
*** tkelsey has quit IRC | 18:21 | |
*** arnaud has quit IRC | 18:22 | |
*** markmcclain has quit IRC | 18:27 | |
*** tkelsey has joined #openstack-oslo | 18:27 | |
ttx | dhellmann: about oslo tags | 18:30 |
ttx | just let me know what i should tag for oslo-incubator as juno-1 | 18:31 |
ttx | don't think we need to tag anything for oslo.messaging ? | 18:31 |
*** markmcclain has joined #openstack-oslo | 18:31 | |
*** jraim has quit IRC | 18:39 | |
*** jraim has joined #openstack-oslo | 18:40 | |
*** jaosorior has quit IRC | 18:42 | |
*** tkelsey has quit IRC | 18:47 | |
openstackgerrit | Nataliia Uvarova proposed a change to openstack/oslo-incubator: Fix common.log.ContextFormatter for Python 3 https://review.openstack.org/95439 | 18:53 |
*** zzzeek has quit IRC | 19:09 | |
*** arnaud has joined #openstack-oslo | 19:12 | |
*** czzl has joined #openstack-oslo | 19:15 | |
*** arnaud has quit IRC | 19:25 | |
*** markmcclain has quit IRC | 19:27 | |
*** bnemec has joined #openstack-oslo | 19:34 | |
ttx | dhellmann: ? | 19:38 |
openstackgerrit | Alex Gaynor proposed a change to openstack-dev/hacking: Fixed warning H302 when used with six.moves https://review.openstack.org/99467 | 19:57 |
*** markmcclain has joined #openstack-oslo | 20:02 | |
*** markmcclain has quit IRC | 20:09 | |
*** bnemec has quit IRC | 20:13 | |
*** bnemec has joined #openstack-oslo | 20:20 | |
*** arnaud__ has joined #openstack-oslo | 20:22 | |
*** arnaud__ has quit IRC | 20:23 | |
*** arnaud__ has joined #openstack-oslo | 20:23 | |
*** flaper87 is now known as flaper87|afk | 20:32 | |
openstackgerrit | Brant Knudson proposed a change to openstack/oslo.i18n: Remove Babel version workaround code https://review.openstack.org/98542 | 20:35 |
*** epopt37 has quit IRC | 20:39 | |
*** bnemec_away has quit IRC | 20:39 | |
*** jd__ has quit IRC | 20:39 | |
*** epopt37 has joined #openstack-oslo | 20:40 | |
*** jd__ has joined #openstack-oslo | 20:40 | |
*** beekneemech has joined #openstack-oslo | 20:41 | |
*** jecarey has quit IRC | 20:57 | |
*** bnemec has quit IRC | 21:03 | |
openstackgerrit | Brant Knudson proposed a change to openstack/oslo.i18n: Remove Babel version workaround code https://review.openstack.org/98542 | 21:10 |
*** mrda-away is now known as mrda | 21:13 | |
*** jecarey has joined #openstack-oslo | 21:18 | |
*** SlickNik has quit IRC | 21:18 | |
*** SlickNik has joined #openstack-oslo | 21:19 | |
*** Guest81438 has joined #openstack-oslo | 21:24 | |
*** redrobot is now known as Guest57969 | 21:25 | |
*** Guest81438 is now known as redrobot | 21:27 | |
*** mriedem has quit IRC | 21:49 | |
*** markmcclain has joined #openstack-oslo | 21:54 | |
*** Guest57969 has quit IRC | 21:58 | |
bknudson | anybody know what sqlite_db is used for? | 21:58 |
bknudson | I don't see it read anywhere | 21:58 |
*** redrobot has left #openstack-oslo | 22:02 | |
harlowja | bknudson in what context? | 22:05 |
bknudson | harlowja: http://git.openstack.org/cgit/openstack/oslo.db/tree/oslo/db/options.py#n19 | 22:06 |
bknudson | oslo.db has a config option sqlite_db, but I don't see it used | 22:06 |
harlowja | hmmm, probably someone that uses oslo.db uses those configs? | 22:06 |
bknudson | you can set the sqlite file using the connection string | 22:06 |
harlowja | bknudson ya, my guess is nova or other project want to use those configs, so they ended up in oslo.db (even though oslo.db doesn't care about them itself) | 22:07 |
harlowja | i don't personally like that kind of action at a distance :-/ | 22:08 |
bknudson | harlowja: the keystone conf file is generated from the registered configs, so now we have an option that's unused (and I guess it confused someone) | 22:08 |
harlowja | bknudson ya, that is the flaw with doing configs like this | 22:08 |
harlowja | all sorta of actions at a distance | 22:09 |
harlowja | *all sorts | 22:09 |
harlowja | bknudson cinder seems to use it, so thats where the issue comes into play with configs done like this, if its a config for set(cinder, nova, ...) but not set(keystone) it sure doesn't feel like to me that it shouldn't be in a commmon library (but should instead be in the set of projects that use it) | 22:10 |
harlowja | *it sure doesn't feel right to me that it should be in a common library | 22:11 |
harlowja | (can't spell, lol) | 22:11 |
bknudson | harlowja: when we're putting code to a library would be a good time to get rid of at least this one | 22:11 |
harlowja | i agree | 22:11 |
harlowja | bknudson https://etherpad.openstack.org/p/oslo.db-release-blockers maybe we should put your note there? | 22:12 |
bknudson | harlowja: I'll add it... also, filing a bug | 22:12 |
harlowja | k | 22:12 |
harlowja | i have a feeling that the way this config stuff is done won't last long term (other people have different views though) | 22:13 |
bknudson | we should at least have the option of having the lib handle config or the app does it. | 22:13 |
bknudson | if the apps are doing it then we'll wind up with inconsistent config options | 22:14 |
harlowja | agreed | 22:14 |
bknudson | which would be less than ideal | 22:14 |
harlowja | *and then in my ideal world the app would do all of the configuration (cause the app knows best) | 22:14 |
openstackgerrit | Brant Knudson proposed a change to openstack/oslo.db: Remove sqlite_db config option https://review.openstack.org/99493 | 22:18 |
*** markmcclain has quit IRC | 22:19 | |
bknudson | I also thought oslo.db would get rid of slave_connection | 22:19 |
bknudson | because it's racist | 22:19 |
harlowja | lol | 22:19 |
harlowja | hmmm | 22:20 |
harlowja | people i think just need to use a different dictionary when looking at computer terms :-P | 22:20 |
harlowja | don't use oxford one | 22:21 |
harlowja | *my 2 cents* | 22:21 |
bknudson | keystone doesn't have any use for the slave_connection | 22:21 |
bknudson | so another option that doesn't make sense for it | 22:22 |
harlowja | agreed, sounds like another case of the disjoint set | 22:22 |
harlowja | i'd almost want there to be just one oslo.db config option, db_uri, u'd think u could provide most of the configuration specifics via that (or via paramters of the url) | 22:24 |
harlowja | get_db_thingy(db_uri) | 22:24 |
harlowja | end of api | 22:24 |
bknudson | that's pretty much how odbc works | 22:25 |
harlowja | righto ;) | 22:26 |
*** dims__ has quit IRC | 22:28 | |
*** markmcclain has joined #openstack-oslo | 22:30 | |
*** arnaud__ has quit IRC | 22:30 | |
*** bknudson has quit IRC | 22:45 | |
*** arnaud__ has joined #openstack-oslo | 22:55 | |
*** arnaud__ has quit IRC | 22:56 | |
*** arnaud__ has joined #openstack-oslo | 23:01 | |
*** zzzeek has joined #openstack-oslo | 23:08 | |
*** yamahata has quit IRC | 23:21 | |
*** dims__ has joined #openstack-oslo | 23:30 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!