| *** zhiyan_ is now known as zhiyan | 00:10 | |
| *** zzzeek has quit IRC | 00:18 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 00:40 |
|---|---|---|
| *** yamahata has quit IRC | 00:52 | |
| *** arnaud has joined #openstack-oslo | 01:10 | |
| *** arnaud has quit IRC | 01:31 | |
| *** yamahata has joined #openstack-oslo | 01:37 | |
| *** arnaud has joined #openstack-oslo | 02:33 | |
| *** praneshp has quit IRC | 03:03 | |
| *** lbragstad has quit IRC | 03:11 | |
| *** zhiyan is now known as zhiyan_ | 03:20 | |
| *** zhiyan_ is now known as zhiyan | 03:26 | |
| *** gcb has joined #openstack-oslo | 03:41 | |
| *** arnaud has quit IRC | 03:42 | |
| *** dims_ has quit IRC | 03:53 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add basic support for conditional execution https://review.openstack.org/98946 | 03:55 |
| *** lbragstad has joined #openstack-oslo | 04:38 | |
| *** dims_ has joined #openstack-oslo | 04:46 | |
| *** dims_ has quit IRC | 04:50 | |
| *** ildikov has quit IRC | 04:51 | |
| *** harlowja is now known as harlowja_away | 04:59 | |
| *** arnaud has joined #openstack-oslo | 05:02 | |
| *** lbragstad has quit IRC | 05:13 | |
| *** gcb has quit IRC | 05:14 | |
| *** ildikov has joined #openstack-oslo | 05:15 | |
| *** lbragstad has joined #openstack-oslo | 05:15 | |
| *** dims_ has joined #openstack-oslo | 05:47 | |
| *** dims_ has quit IRC | 05:51 | |
| *** arnaud has quit IRC | 06:08 | |
| *** dims_ has joined #openstack-oslo | 06:17 | |
| *** arnaud has joined #openstack-oslo | 06:17 | |
| *** arnaud__ has joined #openstack-oslo | 06:19 | |
| *** mkoderer has joined #openstack-oslo | 06:21 | |
| *** dims_ has quit IRC | 06:22 | |
| *** arnaud__ has joined #openstack-oslo | 06:22 | |
| *** arnaud has quit IRC | 06:22 | |
| *** arnaud__ has quit IRC | 06:22 | |
| *** arnaud has joined #openstack-oslo | 06:23 | |
| *** arnaud has quit IRC | 06:23 | |
| *** jaosorior has joined #openstack-oslo | 06:41 | |
| openstackgerrit | Shuangtai Tian proposed a change to openstack/stevedore: use six.add_metaclass https://review.openstack.org/98963 | 06:41 |
| *** lbragstad has quit IRC | 06:49 | |
| *** lbragstad has joined #openstack-oslo | 06:50 | |
| *** flaper87|afk is now known as flaper87 | 07:06 | |
| *** dims_ has joined #openstack-oslo | 07:17 | |
| *** dims_ has quit IRC | 07:22 | |
| *** bogdando has quit IRC | 07:23 | |
| *** tkelsey has joined #openstack-oslo | 07:24 | |
| *** bogdando has joined #openstack-oslo | 07:24 | |
| openstackgerrit | Flavio Percoco proposed a change to openstack/oslo-specs: Add graduate-oslo-text spec to juno https://review.openstack.org/98965 | 07:27 |
| rpodolyaka | morning | 07:28 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/oslo-specs: Add graduate-oslo-text spec to juno https://review.openstack.org/98965 | 07:28 |
| openstackgerrit | Marc Koderer proposed a change to openstack/oslotest: Add known issue about time.time mocking https://review.openstack.org/95411 | 07:39 |
| viktors | hello folks! Does anybody know, how gates feel? | 07:49 |
| *** ajo|mac has joined #openstack-oslo | 07:50 | |
| *** stannie has joined #openstack-oslo | 07:55 | |
| *** ihrachyshka has joined #openstack-oslo | 07:59 | |
| *** gcb has joined #openstack-oslo | 08:08 | |
| *** dims_ has joined #openstack-oslo | 08:18 | |
| *** dims_ has quit IRC | 08:22 | |
| *** ajo|mac has quit IRC | 08:31 | |
| *** ajo|mac has joined #openstack-oslo | 08:33 | |
| *** lbragstad has quit IRC | 08:43 | |
| *** lbragstad has joined #openstack-oslo | 08:43 | |
| jaosorior | haypo, are you around? | 08:43 |
| haypo | jaosorior: hi | 08:45 |
| haypo | jaosorior: i added some comments to your last patch | 08:45 |
| openstackgerrit | Juan Antonio Osorio Robles proposed a change to openstack/oslo.i18n: Trivial refactors for gettextutils https://review.openstack.org/98429 | 08:45 |
| jaosorior | haypo: some things you commented from my last patch are addressed by the following commit by bknudson https://review.openstack.org/#/c/98542/1/oslo/i18n/gettextutils.py | 08:46 |
| jaosorior | haypo: I asked him to rebase the changes, though I'm not sure if I should just add the changes in the patch, or should I wait for him to rebase | 08:47 |
| haypo | jaosorior: oh, i didn't see his change | 08:56 |
| haypo | jaosorior: i voted +2 for this change. it's safe, i don't see the need to rebase, other patches are not merged yet | 08:59 |
| jaosorior | Alrighty then | 09:01 |
| haypo | jaosorior: IMO adding dependencies makes patches more difficult to modify and review | 09:02 |
| haypo | i prefer small independent patches, but others may disagree :) | 09:02 |
| jaosorior | To be honest I do too, but I was asked to add the dependency so I thought it's the way it worked for graduating from incubation... Or something | 09:03 |
| *** ihrachyshka has quit IRC | 09:10 | |
| *** zhiyan is now known as zhiyan_ | 09:18 | |
| *** dims_ has joined #openstack-oslo | 09:19 | |
| *** dims_ has quit IRC | 09:25 | |
| *** nacim has joined #openstack-oslo | 09:35 | |
| *** ihrachyshka has joined #openstack-oslo | 09:53 | |
| *** gcb has quit IRC | 10:04 | |
| *** gcb has joined #openstack-oslo | 10:16 | |
| tkelsey | hello oslo people, i have a question about oslo.messaging, hopefully this is the reight place to ask it. When using cast messages is there any mechanism to communicate an error back to the sender | 10:21 |
| *** dims_ has joined #openstack-oslo | 10:21 | |
| tkelsey | call is blocking and will rasie exceptions ect, but cast seems to be fire and forget, is there a standard pattern for communicating failure when using cast ? | 10:22 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 10:23 |
| ihrachyshka | tkelsey: yes, it's the right place for the question. I think there is no such option avail, but I may be wrong. | 10:26 |
| tkelsey | ihrachyshka: thanks, im thinking the same but wanted to ask the experts :-) | 10:26 |
| *** dims_ has quit IRC | 10:27 | |
| ihrachyshka | there was a multicall concept before, it could call multiple methods and return results when ready, but I think it's removed now in oslo.messaging | 10:27 |
| ihrachyshka | as per https://wiki.openstack.org/wiki/Oslo/Messaging, 'Finally, note that we previously had a concept of multicall where the method is invoked synchronously and multiple results are returned to the caller, each result returned as soon as it's ready. However, given that we don't have any current use cases for this, we're dropping support for it. | 10:27 |
| ihrachyshka | ' | 10:27 |
| tkelsey | ihrachyshka: i see, thanks for the info. Im not so much interested in multiple returned values as in knowing that a messaged is considered to have "failed". That is, was in some way rejected by the server. | 10:29 |
| ihrachyshka | tkelsey: I think before oslo.messaging, you could utilise multicall doing single call to do what you want, though I'm not sure | 10:31 |
| ihrachyshka | all in all, if there is a valid case for that, you may be interested in pushing a patch for that | 10:31 |
| ihrachyshka | that doesn't sound crazy on first sight | 10:31 |
| tkelsey | ihrachyshka: I can see how that could work, im not entirely sure multicall would be the right tool though, since it seems to be solving a slightly different problem really. | 10:32 |
| tkelsey | it sound like a call type for use in a generator or similar | 10:33 |
| tkelsey | of course I could be completely wrong :-) | 10:34 |
| ihrachyshka | tkelsey: well, anyway, it's gone now, so it's just history :) | 10:38 |
| ihrachyshka | I'm not best in archaeology ;) | 10:38 |
| tkelsey | ihrachyshka: heh well im still learning the lay of the land, bound to find a fossil sooner or later :-) | 10:39 |
| tkelsey | humm, it almost seems like some sort of out of bounds communication could do this, somthing to say "hey that message failed" but it feels like it might be adverse to the intended use of cast. | 10:41 |
| ihrachyshka | yes, instead, for your use case, we would need some async call | 10:43 |
| *** ajo|mac has quit IRC | 10:43 | |
| ihrachyshka | btw you can always hadk your way by creating two exchanges, then casting on the 1st peer exchange and retrieving results on another one | 10:44 |
| ekarlso | ihrachyshka: is the stuff ^ you are talking about a way to do for example a sync call to n servers ? | 10:44 |
| ihrachyshka | s/hadk/hack | 10:44 |
| ihrachyshka | ekarlso: tkelsey's case is 'doing call to one server, bailing out right away doing other work, then getting result in some other green thread once it's avail' | 10:45 |
| ihrachyshka | (at least that's my understanding) | 10:45 |
| ekarlso | ah ok | 10:46 |
| ekarlso | ls | 10:46 |
| ihrachyshka | ekarlso: what's the problem with doing N sync calls with e.g. for loop? :) | 10:46 |
| tkelsey | ihrachyshka: not quite, sorry for the miscommunication, to give some background, I am looking at message level security. This is very preliminary but im just thinking about the case where a message is delivered perfectly fine by the transport layer in a "cast" style but the sec layer says its bad. | 10:47 |
| tkelsey | im thinking about how this failure could be communicated back to the sender | 10:48 |
| ekarlso | ihrachyshka: ya | 10:48 |
| ekarlso | that's what we're doing in designate atm, just wondered if this was the topic ;) | 10:48 |
| ihrachyshka | tkelsey: via another 'error' exchange? | 10:49 |
| ihrachyshka | tkelsey: but why not just using .call? This would make your flow easy to trace and react on errors | 10:50 |
| tkelsey | ihrachyshka: yes call would work nicely, but cast is problematic | 10:50 |
| *** ekarlso has quit IRC | 10:57 | |
| *** ekarlso has joined #openstack-oslo | 10:58 | |
| tkelsey | ihrachyshka: my end goal is to create some sort of mechanism in oslo.messaging to support plugable sec policies for interacting with systems like Kite. More info on Kite is here https://blog-nkinder.rhcloud.com/?p=62 | 11:00 |
| bogdando | hi. Regarding https://review.openstack.org/#/c/95479/. Could you guys please contact the Nachi Ueno to remove -2, if appropriate? | 11:01 |
| tkelsey | ihrachyshka: of course others may already be thinking about the same problem, but i figgured someone in here would probably know about that if so :-) | 11:01 |
| *** bogdando has quit IRC | 11:04 | |
| *** bogdando has joined #openstack-oslo | 11:04 | |
| ihrachyshka | bogdando: hm, have you pushed it to oslo-incubator first? | 11:08 |
| ihrachyshka | bogdando: also I think full sync is the preferred way to copy the code now | 11:08 |
| ihrachyshka | tkelsey: thanks, sounds interesting, I've added the link to my reading list :) | 11:09 |
| bogdando | it is already in oslo-incubator, the patch is just a cherry-pick for critical issue fixing | 11:09 |
| ihrachyshka | bogdando: I can't find it with the same Change-Id. Please provide a link | 11:09 |
| tkelsey | ihrachyshka: :-) | 11:10 |
| bogdando | ihrachyshka, https://review.openstack.org/#/c/76686/ is it an appropriate place? | 11:10 |
| ihrachyshka | bogdando: the fix is in oslo.messaging. have you synced the fix to oslo-incubator? these are two different code bases. | 11:10 |
| bogdando | ihrachyshka, oh | 11:11 |
| ihrachyshka | the path is: oslo.messaging -> oslo-incubator -> neutron | 11:11 |
| ihrachyshka | bogdando: yes, weird, I know, so no worries that you didn't get it right ;) | 11:11 |
| ihrachyshka | bogdando: btw neutron is near getting rid of oslo-incubator RPC layer and moving to oslo.messaging (patches have already started to merge), so maybe in a week or two the change will be unneeded. Anyway, if that's critical, we may push it now. But first incubator. | 11:12 |
| bogdando | ihrachyshka, I'm sorry but I cannot find impl_rabbit.py in stable/icehouse of Oslo-incubator, nor in master | 11:16 |
| bogdando | ihrachyshka, should I cherry pick it to impl_kombu.py instead? | 11:17 |
| ihrachyshka | it was renamed from impl_kombu.py | 11:17 |
| ihrachyshka | yes, though I wouldn't name it cherry-pick because cherry-pick obviously fails... | 11:17 |
| ihrachyshka | I call it 'charry-pick' as in 'picking needed characters' :D | 11:17 |
| bogdando | ihrachyshka, that's how I call it... not an exact cherry-pick of cause... | 11:18 |
| bogdando | ihrachyshka, :D indeed | 11:18 |
| ihrachyshka | bogdando: fyi some background about incubation: https://wiki.openstack.org/wiki/Oslo#Incubation | 11:19 |
| *** dims_ has joined #openstack-oslo | 11:23 | |
| bogdando | ok, here is an Icehouse targeted one https://review.openstack.org/#/c/99007/ ... | 11:26 |
| ihrachyshka | bogdando: wait, you may cherry-pick to stable only once master is fixed | 11:28 |
| openstackgerrit | Bogdan Dobrelya proposed a change to openstack/oslo-incubator: Sync kombu_reconnect_delay from Oslo https://review.openstack.org/99009 | 11:29 |
| ihrachyshka | otherwise we may introduce regressions | 11:29 |
| bogdando | ihrachyshka, sure, I will submit for master as well | 11:29 |
| ihrachyshka | bogdando: and all the fixes should have the same change-id as oslo.messaging one | 11:30 |
| bogdando | here it is https://review.openstack.org/#/c/99009/. All are viewable by the change ID also | 11:30 |
| bogdando | ihrachyshka, btw, does the Oslo incubation sync assume the Oslo-incubator -> Project path only? | 11:31 |
| bogdando | so, the Oslo.x -> Oslo-incubator part is up to contributor efforts? | 11:31 |
| ihrachyshka | yes. but to get anything in those projects, you should copy the fix from oslo.messaging to oslo-incubator | 11:32 |
| ihrachyshka | yes, you obviously may avoid doing the sync if you're not interested in it | 11:32 |
| bogdando | ihrachyshka, got it , thanks. So, please review guys... | 11:32 |
| *** yamahata has quit IRC | 11:32 | |
| ihrachyshka | bogdando: see comments | 11:37 |
| openstackgerrit | Bogdan Dobrelya proposed a change to openstack/oslo-incubator: Slow down Kombu reconnect attempts https://review.openstack.org/99017 | 11:45 |
| *** gcb has quit IRC | 11:47 | |
| openstackgerrit | Bogdan Dobrelya proposed a change to openstack/oslo-incubator: Slow down Kombu reconnect attempts https://review.openstack.org/99017 | 12:00 |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslo-specs: graduate-oslo-local https://review.openstack.org/99028 | 12:17 |
| haypo | jaosorior: last comment? :-p https://review.openstack.org/#/c/98429/5 | 12:25 |
| *** dhellmann has quit IRC | 12:28 | |
| *** bknudson has quit IRC | 12:34 | |
| jaosorior | It could work. But I'll do it tomorrow. I'll be off my computer for the whole day now | 12:36 |
| jaosorior | Thanks for the input :) | 12:36 |
| haypo | jaosorior: except of my last comment, the change now looks good to me | 12:36 |
| haypo | jaosorior: so see you tomorrow ;) | 12:37 |
| jaosorior | Wha? | 12:37 |
| haypo | jaosorior: you didn't understand what i wrote? | 12:38 |
| jaosorior | Not the part: "except my last comment..." | 12:41 |
| haypo | jaosorior: if you solve my last comment, i'm ok to vote +2 | 12:41 |
| haypo | jaosorior: but i'm not sure that the find subfunction is an issue | 12:42 |
| jaosorior | Neither am I. I actually think that it's a good approach | 12:43 |
| *** lbragstad has quit IRC | 12:45 | |
| jaosorior | O_o | 12:49 |
| *** bknudson has joined #openstack-oslo | 12:52 | |
| *** HenryG has quit IRC | 12:53 | |
| *** dims_ has quit IRC | 12:59 | |
| *** yamahata has joined #openstack-oslo | 13:05 | |
| *** jecarey has quit IRC | 13:06 | |
| *** erecio has quit IRC | 13:07 | |
| *** erecio has joined #openstack-oslo | 13:08 | |
| *** zzzeek has joined #openstack-oslo | 13:15 | |
| *** mriedem has joined #openstack-oslo | 13:16 | |
| *** lbragstad has joined #openstack-oslo | 13:17 | |
| *** jecarey has joined #openstack-oslo | 13:18 | |
| *** lbragstad1 has joined #openstack-oslo | 13:26 | |
| *** lbragstad has quit IRC | 13:28 | |
| *** lbragstad has joined #openstack-oslo | 13:29 | |
| *** lbragstad1 has quit IRC | 13:30 | |
| *** dims_ has joined #openstack-oslo | 13:30 | |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Build locale dir env var name consistently https://review.openstack.org/99054 | 13:38 |
| *** lbragstad has quit IRC | 13:48 | |
| *** lbragstad has joined #openstack-oslo | 13:48 | |
| *** lbragstad1 has joined #openstack-oslo | 13:57 | |
| *** lbragstad has quit IRC | 13:58 | |
| viktors | bnemec: Hi! Can you please remove your WIP on patch https://review.openstack.org/#/c/93396/ (Remove common context usage from db model_query()). It's seems to be, that gates are working now :) | 14:13 |
| bnemec | viktors: Done | 14:14 |
| viktors | bnemec: thanks! | 14:15 |
| *** lbragstad1 has quit IRC | 14:16 | |
| openstackgerrit | Christian Berendt proposed a change to openstack/oslo-incubator: switch from mox to mox3 https://review.openstack.org/93729 | 14:17 |
| *** dhellmann has joined #openstack-oslo | 14:20 | |
| dhellmann | looks like my znc server died, so resending: | 14:22 |
| dhellmann | mestery: thanks for the heads-up! | 14:22 |
| dhellmann | bnemec, dims_, flaper87: looking at strutils again, I wonder if we should just toss it into oslo.utils instead of making a standalone lib? | 14:22 |
| mestery | dhellmann: No problem. We merged a few patches yesterday, hoping to land a few more and get this done very early in Juno-2. | 14:23 |
| flaper87 | dhellmann: That should work although I think there will be more things "text-related" that would fit into oslo.text | 14:23 |
| openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo.db: Handle slave database connection in EngineFacade https://review.openstack.org/94593 | 14:29 |
| dhellmann | flaper87: what do you have in mind? | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/99084 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements https://review.openstack.org/99085 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/99086 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Updated from global requirements https://review.openstack.org/96239 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/99087 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslotest: Updated from global requirements https://review.openstack.org/99088 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/99089 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslosphinx: Updated from global requirements https://review.openstack.org/99090 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 14:39 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 14:40 |
| flaper87 | dhellmann: mmh, actually, nevermind. I guess if there are more things to add they would be added to the strutils module anyway. | 14:41 |
| dhellmann | yeah, although see my comment about splitting that up into a few modules | 14:41 |
| haypo | dhellmann: hi. i saw https://review.openstack.org/#/c/97932/ and i dislike it :-p | 14:42 |
| haypo | dhellmann: why not keeping the code in a single file? | 14:42 |
| dhellmann | haypo: a lot of those functions shouldn't be part of the public API | 14:42 |
| haypo | dhellmann: ok, but so just add an underscore prefix | 14:43 |
| haypo | Message => _Message, that's all | 14:43 |
| haypo | we are all consentent adults here ;) | 14:43 |
| dhellmann | you're assuming well-behaved consumers of the library | 14:43 |
| haypo | dhellmann: if your users uses private things, how a submodule would prevent this? | 14:44 |
| haypo | dhellmann: you can import oslo.i18n._locale | 14:44 |
| dhellmann | yeah | 14:44 |
| dhellmann | haypo: can you explain more than "I don't like it"? | 14:45 |
| haypo | i'm not against splitting code into submodules, but usually i try to group similar functions & classes, here you put a single function in a file. IMO it's harder to navigate in the code which such file hierarchy | 14:45 |
| haypo | in python, the rule is that there is no stability guarantee on the private API. a whole module can go away, same for private functions and classes. private means anything called with a name starting with an underscore | 14:47 |
| openstackgerrit | Ilya Shakhat proposed a change to openstack/cliff: CSV formatter should use system-dependent line ending https://review.openstack.org/99106 | 14:47 |
| haypo | dhellmann: i have a much shorter experience than you in OpenStack, so i don't know which issues you had in the past with the API stability | 14:48 |
| sileht | jd__, can you take a look to https://review.openstack.org/#/c/90046/ | 14:49 |
| haypo | dhellmann: well, maybe i shouldn't vote -1. i just wanted to talk with you | 14:49 |
| dhellmann | haypo: when we split out oslo.messaging, we tried to keep the public API surface as small as possible | 14:50 |
| dhellmann | no, a -1 is fine, this is good feedback | 14:50 |
| jd__ | sileht: doing so | 14:50 |
| dhellmann | haypo: with oslo.messaging, we did that using private packages for some of the implementation details, and I was following that example here | 14:50 |
| haypo | dhellmann: in olso messaging, there is much more code in the private packages | 14:50 |
| dhellmann | haypo: the enable_lazy function has some global state that I was trying to hid more completely than just prefixing the variable name | 14:51 |
| haypo | dhellmann: it makes sense to move the code in private modules | 14:51 |
| dhellmann | s/hid/hide/ | 14:51 |
| dhellmann | the _i18n.py module contains instances of the translator that should be private to this package | 14:51 |
| dhellmann | get_locale_dir_variable_name is only one function, but it is used in a couple of the other modules so it can't go in gettextutils because of a circular import problem | 14:52 |
| dhellmann | the message and factory classes seemed big enough to warrant their own files | 14:52 |
| dhellmann | translate_args is used in a couple of places inside the package, so again it introduced circular imports | 14:53 |
| dhellmann | haypo: so I could basically have one big file, as before, or I could have a couple of files that are smaller than I'd otherwise make them normally | 14:53 |
| haypo | dhellmann: i was shocked by _locale.py which only contains 2 lines of code | 14:53 |
| haypo | dhellmann: i didn't read each file carefully yet | 14:53 |
| haypo | dhellmann: do you think that users will modify directly _USE_LAZY if it's not in an "hidden" module? | 14:54 |
| dhellmann | haypo: yeah, that function is tiny but may grow and it's used in _factory | 14:54 |
| dhellmann | haypo: quite possibly | 14:54 |
| haypo | dhellmann: you should warn users that private symbols are private, there is no guarantee of the stability of these variables, functions, etc. | 14:55 |
| haypo | but i expect that any python developers know that | 14:55 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Fix the notify method of the routing notifier https://review.openstack.org/92403 | 14:56 |
| openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Transport reconnection retries for notification https://review.openstack.org/92373 | 14:56 |
| dhellmann | haypo: yes, we're working on educating everyone about that | 14:56 |
| haypo | dhellmann: maybe you should rename these private symbols in each minor version to check how many people rely on them :-D | 14:57 |
| dims_ | dhellmann, +1 to throw strutils into oslo.utils | 14:57 |
| dhellmann | haypo: don't tempt me :-) | 14:58 |
| dhellmann | dims_, flaper87 : if we put strutils in oslo.utils, could you two work on the graduation together? | 15:01 |
| dims_ | yep | 15:01 |
| flaper87 | dhellmann: dims_ absolutely, I've put the olso.text code here: https://github.com/FlaPer87/oslo.text | 15:02 |
| flaper87 | at least the tests will be useful from there | 15:02 |
| dhellmann | flaper87: true, good | 15:03 |
| dhellmann | dims_, flaper87 : can one of you take over the utils spec and update it with the strutils file(s)? We should go ahead and think about splitting strutils up into groups that make more sense, too. | 15:03 |
| *** ihrachyshka has quit IRC | 15:04 | |
| haypo | dhellmann: i removed my -1 vote. as i wrote, i don't want to convince you, just to discuss ;) | 15:04 |
| *** HenryG has joined #openstack-oslo | 15:04 | |
| dhellmann | haypo: ok, thanks, this was good feedback and I'll take another look at the code and see if maybe I can make less drastic changes :-) | 15:06 |
| *** ildikov_ has joined #openstack-oslo | 15:08 | |
| *** ildikov has quit IRC | 15:09 | |
| *** praneshp has joined #openstack-oslo | 15:13 | |
| *** viktors is now known as viktors|afk | 15:30 | |
| *** nacim has quit IRC | 15:43 | |
| *** tkelsey has quit IRC | 15:55 | |
| *** jaosorior has quit IRC | 16:12 | |
| *** arnaud has joined #openstack-oslo | 16:18 | |
| *** ihrachyshka has joined #openstack-oslo | 16:29 | |
| openstackgerrit | Qin Zhao proposed a change to openstack/oslo-incubator: Log the function name of looping call https://review.openstack.org/99152 | 16:35 |
| *** mriedem has quit IRC | 16:35 | |
| *** markmcclain has joined #openstack-oslo | 16:36 | |
| openstackgerrit | Alex Gaynor proposed a change to openstack/oslo-incubator: Fixed a new pep8 error and a small typo https://review.openstack.org/99158 | 16:41 |
| therve | dhellmann, I got up on friday's meeting | 16:41 |
| therve | s/got/caught | 16:41 |
| therve | No red flags from Heat | 16:42 |
| therve | The config generation problem is worked around in master just now as you noticed | 16:42 |
| therve | And we'll work on oslo.messaging soon as it's fixed | 16:43 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Further improvements to documentation https://review.openstack.org/99160 | 16:48 |
| openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Improve initial documentation https://review.openstack.org/96961 | 16:49 |
| *** mriedem has joined #openstack-oslo | 16:49 | |
| *** stannie has quit IRC | 16:57 | |
| *** czzl has joined #openstack-oslo | 16:58 | |
| *** harlowja_away is now known as harlowja | 17:08 | |
| *** HenryG has quit IRC | 17:19 | |
| *** HenryG has joined #openstack-oslo | 17:20 | |
| *** markmcclain has quit IRC | 17:25 | |
| *** dims_ has quit IRC | 17:28 | |
| *** markmcclain has joined #openstack-oslo | 17:29 | |
| *** dims_ has joined #openstack-oslo | 17:42 | |
| *** rpodolyaka1 has joined #openstack-oslo | 17:42 | |
| rpodolyaka1 | Hey, oslo cores, can I get +A for a patch that had been approved but failed to merge? (https://review.openstack.org/#/c/94593/) :) | 17:45 |
| *** ildikov_ has quit IRC | 17:53 | |
| *** ildikov_ has joined #openstack-oslo | 17:54 | |
| *** ihrachyshka has quit IRC | 17:57 | |
| dhellmann | therve: cool, thanks for verifying that | 18:05 |
| dhellmann | rpodolyaka1: did that just need a rebase? | 18:06 |
| *** ildikov_ has quit IRC | 18:06 | |
| *** arnaud has quit IRC | 18:12 | |
| rpodolyaka1 | dhellmann: yes | 18:13 |
| dhellmann | rpodolyaka1: ok | 18:13 |
| rpodolyaka1 | dhellmann: thanks! | 18:13 |
| *** rpodolyaka1 has quit IRC | 18:17 | |
| *** harlowja is now known as harlowja_away | 18:25 | |
| *** dims_ has quit IRC | 18:25 | |
| *** jecarey has quit IRC | 18:29 | |
| *** jecarey has joined #openstack-oslo | 18:31 | |
| *** ildikov has joined #openstack-oslo | 18:33 | |
| *** jecarey_ has joined #openstack-oslo | 18:33 | |
| *** jecarey has quit IRC | 18:37 | |
| *** erecio has quit IRC | 18:48 | |
| *** erecio has joined #openstack-oslo | 18:49 | |
| *** markmcclain has quit IRC | 18:56 | |
| *** bknudson has quit IRC | 19:02 | |
| *** arnaud has joined #openstack-oslo | 19:03 | |
| *** harlowja_away is now known as harlowja | 19:03 | |
| *** rpodolyaka1 has joined #openstack-oslo | 19:05 | |
| *** praneshp_ has joined #openstack-oslo | 19:06 | |
| *** praneshp has quit IRC | 19:08 | |
| *** praneshp_ is now known as praneshp | 19:08 | |
| *** erecio has quit IRC | 19:10 | |
| *** dims_ has joined #openstack-oslo | 19:10 | |
| *** erecio has joined #openstack-oslo | 19:11 | |
| *** czzl has left #openstack-oslo | 19:22 | |
| *** amotoki has quit IRC | 19:22 | |
| *** stannie has joined #openstack-oslo | 19:26 | |
| *** arnaud has quit IRC | 19:38 | |
| *** bknudson has joined #openstack-oslo | 19:40 | |
| *** czzl has joined #openstack-oslo | 19:56 | |
| *** czzl has left #openstack-oslo | 19:58 | |
| *** czzl has joined #openstack-oslo | 20:00 | |
| *** gpocentek has joined #openstack-oslo | 20:02 | |
| openstackgerrit | Gauvain Pocentek proposed a change to openstack/oslo.db: Fix the installation documentation https://review.openstack.org/99195 | 20:02 |
| *** markmcclain has joined #openstack-oslo | 20:02 | |
| *** rpodolyaka1 has quit IRC | 20:03 | |
| *** jecarey has joined #openstack-oslo | 20:14 | |
| *** jecarey_ has quit IRC | 20:14 | |
| *** arnaud has joined #openstack-oslo | 20:27 | |
| *** kmartin has quit IRC | 20:33 | |
| *** kmartin has joined #openstack-oslo | 20:33 | |
| *** kmartin has quit IRC | 20:33 | |
| *** kmartin has joined #openstack-oslo | 20:33 | |
| *** dhellmann_ has joined #openstack-oslo | 20:41 | |
| *** HenryG has quit IRC | 20:45 | |
| *** dhellmann_ has quit IRC | 20:48 | |
| *** dhellmann_ has joined #openstack-oslo | 20:48 | |
| *** bknudson has quit IRC | 20:54 | |
| *** dhellmann has quit IRC | 20:54 | |
| *** bogdando has quit IRC | 20:54 | |
| *** openstackgerrit has quit IRC | 20:54 | |
| *** dhellmann_ is now known as dhellmann | 20:54 | |
| *** dhellmann has quit IRC | 20:56 | |
| *** bogdando has joined #openstack-oslo | 20:56 | |
| *** bknudson has joined #openstack-oslo | 21:08 | |
| *** openstackgerrit has joined #openstack-oslo | 21:08 | |
| *** markmc has quit IRC | 21:20 | |
| *** flaper87 is now known as flaper87|afk | 21:32 | |
| openstackgerrit | lifeless proposed a change to openstack/oslo-specs: Semantic version support for pbr https://review.openstack.org/96608 | 21:37 |
| *** jecarey has quit IRC | 21:47 | |
| *** mriedem has quit IRC | 21:47 | |
| openstackgerrit | Rick Harris proposed a change to openstack/oslo-incubator: Restore UUID and human-ID bash completion https://review.openstack.org/99216 | 21:50 |
| *** dims__ has joined #openstack-oslo | 21:52 | |
| *** dims_ has quit IRC | 21:54 | |
| *** dhellmann has joined #openstack-oslo | 21:55 | |
| openstackgerrit | Rick Harris proposed a change to openstack/oslo-incubator: Restore UUID and human-ID bash completion https://review.openstack.org/99216 | 22:03 |
| *** bknudson has quit IRC | 22:22 | |
| *** bogdando has quit IRC | 23:04 | |
| *** dims__ has quit IRC | 23:06 | |
| *** bogdando has joined #openstack-oslo | 23:07 | |
| *** dims__ has joined #openstack-oslo | 23:08 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add basic support for conditional execution https://review.openstack.org/98946 | 23:09 |
| *** stannie has left #openstack-oslo | 23:12 | |
| *** praneshp has quit IRC | 23:30 | |
| *** praneshp has joined #openstack-oslo | 23:33 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add basic support for conditional execution https://review.openstack.org/98946 | 23:34 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add basic support for conditional execution https://review.openstack.org/98946 | 23:37 |
| *** yamahata has quit IRC | 23:46 | |
| *** bogdando has quit IRC | 23:54 | |
| *** bogdando has joined #openstack-oslo | 23:57 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Update zake to requirements version https://review.openstack.org/99237 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!