*** tsekiyam_ has joined #openstack-oslo | 00:06 | |
*** tsekiyama has quit IRC | 00:09 | |
*** tsekiyam_ has quit IRC | 00:11 | |
*** dstanek_zzz is now known as dstanek | 00:35 | |
*** dims_ has joined #openstack-oslo | 00:39 | |
*** dstanek is now known as dstanek_zzz | 00:45 | |
*** yamahata has joined #openstack-oslo | 00:53 | |
*** bpokorny has joined #openstack-oslo | 00:57 | |
*** mriedem has joined #openstack-oslo | 01:03 | |
*** arnaud__ has quit IRC | 01:26 | |
*** HenryG_ has joined #openstack-oslo | 01:27 | |
*** HenryG has quit IRC | 01:29 | |
*** dstanek_zzz is now known as dstanek | 01:33 | |
*** markmcclain has quit IRC | 01:34 | |
*** mriedem has left #openstack-oslo | 02:17 | |
*** dims_ has quit IRC | 02:24 | |
*** HenryG_ has quit IRC | 02:37 | |
*** HenryG_ has joined #openstack-oslo | 02:38 | |
*** dims_ has joined #openstack-oslo | 02:39 | |
*** dims_ has quit IRC | 02:44 | |
*** zhiyan_ is now known as zhiyan | 02:45 | |
*** arnaud has joined #openstack-oslo | 02:45 | |
*** harlowja is now known as harlowja_away | 02:53 | |
*** praneshp has quit IRC | 03:05 | |
*** dims_ has joined #openstack-oslo | 03:10 | |
*** dims_ has quit IRC | 03:15 | |
openstackgerrit | David Stanek proposed a change to openstack/oslo.config: Refactors _Namespace to make the code clearer https://review.openstack.org/99831 | 03:20 |
---|---|---|
*** yamahata is now known as tacker-owner | 03:24 | |
*** arnaud has quit IRC | 03:26 | |
*** HenryG_ has quit IRC | 03:32 | |
*** arnaud has joined #openstack-oslo | 03:34 | |
*** tacker-owner is now known as tacker-owner_ | 03:37 | |
*** tacker-owner_ is now known as tacker-owner__ | 03:38 | |
*** tacker-owner__ is now known as tacker-owner | 03:45 | |
*** tacker-owner has quit IRC | 03:59 | |
*** yamahata has joined #openstack-oslo | 04:01 | |
*** yamahata is now known as tacker-owner | 04:02 | |
*** arnaud has quit IRC | 04:09 | |
*** dims_ has joined #openstack-oslo | 04:11 | |
*** dims_ has quit IRC | 04:16 | |
*** arnaud has joined #openstack-oslo | 04:18 | |
*** dstanek is now known as dstanek_zzz | 04:22 | |
*** arnaud has quit IRC | 04:30 | |
*** rpodolyaka1 has joined #openstack-oslo | 04:36 | |
*** tacker-owner has quit IRC | 04:38 | |
*** yamahata has joined #openstack-oslo | 04:38 | |
*** yamahata is now known as tacker-owner | 04:39 | |
*** zzzeek has quit IRC | 04:42 | |
*** dstanek_zzz is now known as dstanek | 04:43 | |
*** dims_ has joined #openstack-oslo | 04:46 | |
*** ildikov has quit IRC | 04:50 | |
*** dims_ has quit IRC | 04:51 | |
*** dstanek is now known as dstanek_zzz | 04:53 | |
*** tacker-owner has quit IRC | 04:53 | |
*** arnaud__ has joined #openstack-oslo | 05:06 | |
*** zhiyan is now known as zhiyan_ | 05:06 | |
*** zhiyan_ is now known as zhiyan | 05:13 | |
*** ihrachyshka has joined #openstack-oslo | 05:35 | |
*** rpodolyaka1 has quit IRC | 05:37 | |
*** dstanek_zzz is now known as dstanek | 05:44 | |
*** rpodolyaka1 has joined #openstack-oslo | 05:45 | |
*** dims_ has joined #openstack-oslo | 05:47 | |
*** ildikov has joined #openstack-oslo | 05:50 | |
*** dims_ has quit IRC | 05:51 | |
*** dstanek is now known as dstanek_zzz | 05:54 | |
*** rpodolyaka1 has quit IRC | 05:59 | |
*** praneshp has joined #openstack-oslo | 06:03 | |
*** yamahata has joined #openstack-oslo | 06:04 | |
*** arnaud__ has quit IRC | 06:06 | |
*** rpodolyaka1 has joined #openstack-oslo | 06:06 | |
*** rpodolyaka1 has quit IRC | 06:10 | |
*** praneshp_ has joined #openstack-oslo | 06:10 | |
*** praneshp has quit IRC | 06:13 | |
*** praneshp_ is now known as praneshp | 06:13 | |
*** ihrachyshka has quit IRC | 06:30 | |
*** zhiyan is now known as zhiyan_ | 06:32 | |
*** bogdando has joined #openstack-oslo | 06:42 | |
*** dstanek_zzz is now known as dstanek | 06:45 | |
*** dims_ has joined #openstack-oslo | 06:47 | |
*** flaper87|afk is now known as flaper87 | 06:49 | |
*** dims_ has quit IRC | 06:54 | |
*** dstanek is now known as dstanek_zzz | 06:54 | |
*** zhiyan_ is now known as zhiyan | 07:00 | |
*** mrda is now known as mrda-weekend | 07:25 | |
*** ihrachyshka has joined #openstack-oslo | 07:38 | |
*** dstanek_zzz is now known as dstanek | 07:45 | |
*** dims_ has joined #openstack-oslo | 07:50 | |
*** dims_ has quit IRC | 07:55 | |
*** dstanek is now known as dstanek_zzz | 07:55 | |
*** zhiyan is now known as zhiyan_ | 08:10 | |
*** tkelsey has joined #openstack-oslo | 08:13 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add an optional timeout parameter to Listener.poll https://review.openstack.org/71003 | 08:21 |
haypo | sileht: hi. could you take a look at this patch? ^^ | 08:21 |
*** zhiyan_ is now known as zhiyan | 08:30 | |
*** dstanek_zzz is now known as dstanek | 08:46 | |
*** dims_ has joined #openstack-oslo | 08:51 | |
*** zhiyan is now known as zhiyan_ | 08:52 | |
*** dims_ has quit IRC | 08:56 | |
*** dstanek is now known as dstanek_zzz | 08:56 | |
*** rpodolyaka is now known as malor | 08:59 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add a new asynchronous executor based on Trollius https://review.openstack.org/70948 | 08:59 |
*** zhiyan_ is now known as zhiyan | 09:00 | |
*** jaosorior has joined #openstack-oslo | 09:03 | |
*** zhiyan is now known as zhiyan_ | 09:09 | |
*** zhiyan_ is now known as zhiyan | 09:10 | |
*** ildikov has quit IRC | 09:17 | |
*** praneshp has quit IRC | 09:42 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add a new asynchronous executor based on Trollius https://review.openstack.org/70948 | 09:42 |
haypo | Trollius is not dead! :) | 09:43 |
*** dstanek_zzz is now known as dstanek | 09:47 | |
*** yamahata_ has quit IRC | 09:49 | |
*** dims_ has joined #openstack-oslo | 09:52 | |
*** dims_ has quit IRC | 09:56 | |
*** dstanek is now known as dstanek_zzz | 09:57 | |
*** zhiyan is now known as zhiyan_ | 10:06 | |
haypo | can someone review this simple python3 patch for the log module of oslo incubator? https://review.openstack.org/#/c/95439/ | 10:08 |
*** yamahata has quit IRC | 10:18 | |
*** markmc has joined #openstack-oslo | 10:33 | |
*** dstanek_zzz is now known as dstanek | 10:48 | |
*** dims_ has joined #openstack-oslo | 10:52 | |
*** dims_ has quit IRC | 10:57 | |
*** dstanek is now known as dstanek_zzz | 10:58 | |
openstackgerrit | Tom Cammann proposed a change to openstack/oslo-incubator: Make periodic tasks run on regular spacing interval https://review.openstack.org/99695 | 11:02 |
*** dims_ has joined #openstack-oslo | 11:16 | |
*** viktors|afk is now known as viktors | 11:26 | |
*** tkelsey has quit IRC | 11:27 | |
openstackgerrit | Tom Cammann proposed a change to openstack/oslo-incubator: Make periodic tasks run on regular spacing interval https://review.openstack.org/99695 | 11:41 |
*** ildikov_ has joined #openstack-oslo | 11:43 | |
*** yamahata has joined #openstack-oslo | 11:46 | |
*** yamahata is now known as tacker-owner | 11:47 | |
*** dstanek_zzz is now known as dstanek | 11:48 | |
*** tacker-owner is now known as yamahata | 11:49 | |
*** yamahata is now known as tacker-owner | 11:53 | |
*** tacker-owner has quit IRC | 11:55 | |
*** yamahata has joined #openstack-oslo | 11:56 | |
*** dstanek is now known as dstanek_zzz | 11:58 | |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo-incubator: Prevent races in opportunistic db test cases https://review.openstack.org/99592 | 12:02 |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo.db: Prevent races in opportunistic db test cases https://review.openstack.org/99608 | 12:06 |
*** ihrachyshka has quit IRC | 12:16 | |
*** ihrachyshka has joined #openstack-oslo | 12:16 | |
*** mriedem has joined #openstack-oslo | 12:22 | |
*** erecio has joined #openstack-oslo | 12:23 | |
openstackgerrit | Tom Cammann proposed a change to openstack/oslo-incubator: Make periodic tasks run on regular spacing interval https://review.openstack.org/99695 | 12:24 |
*** yamahata has quit IRC | 12:28 | |
*** tkelsey has joined #openstack-oslo | 12:38 | |
*** dstanek_zzz is now known as dstanek | 12:49 | |
*** bpokorny has quit IRC | 12:49 | |
*** ildikov has joined #openstack-oslo | 12:52 | |
*** ildikov_ has quit IRC | 12:52 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Add a new asynchronous executor based on Trollius https://review.openstack.org/70948 | 12:53 |
openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo-specs: Add ConfigFilter wrapper class for ConfigOpts https://review.openstack.org/97228 | 13:03 |
*** HenryG has joined #openstack-oslo | 13:07 | |
*** jecarey has quit IRC | 13:09 | |
viktors | haypo: around? | 13:11 |
openstackgerrit | Tim Kelsey proposed a change to openstack/oslo.db: Adding dependency documentation for MySQL. Merging change I347e1c235bc82d6e510761f4b1c3c3de62994c2f fixing pip info. https://review.openstack.org/98105 | 13:11 |
*** bpokorny has joined #openstack-oslo | 13:13 | |
haypo | viktors: hello | 13:13 |
viktors | haypo: I have a question as for Trollius | 13:17 |
viktors | haypo: why do you choose Tulip, not eventlet? | 13:17 |
haypo | viktors: hum, my colleague sileht just wrote "sileht have a big headache because of debugging an (other) eventlet issue in ceilometer" :) | 13:18 |
*** czzl has joined #openstack-oslo | 13:18 | |
haypo | viktors: does it answer to your question? :) | 13:18 |
haypo | viktors: i tried to explain why i prefer Trollius/asyncio in the blueprint: https://wiki.openstack.org/wiki/Oslo/blueprints/asyncio | 13:18 |
viktors | haypo: ok, got it, thanks :) | 13:19 |
haypo | viktors: and here: http://techs.enovance.com/6562/asyncio-openstack-python3 | 13:19 |
haypo | viktors: Trollius makes asynchronous programming explicit, it should be less surprising | 13:19 |
haypo | viktors: the code "looks" sequential, as normal code | 13:20 |
malor | been there too... eventlet is pure 'magic' | 13:20 |
viktors | haypo: yes, but we need to do a hard work to move OS to Trollius | 13:21 |
malor | but what are you going to do with libraries that don't support your event loop and do block the execution of a thread? | 13:22 |
haypo | the major difference with eventlet (eventlet with monkey-patching, greenlet and eventlet without monkey-patching don't have such issue) is that you *see* directly which parts of the code is asynchronous or not | 13:22 |
haypo | so it's easier to detect race conditions (where locks are needed) | 13:22 |
malor | e.g. DB API drivers or SQLAlchemy | 13:22 |
haypo | malor: the easiest option is to run blocking calls in a pool of threads, it's a builtin feature and well supported | 13:23 |
malor | so something like deferToThread()? | 13:23 |
malor | but with futures API | 13:23 |
malor | right? | 13:23 |
haypo | malor: the other option is to call the blocking call and so block the event loop, it's not a problem if it don't block longer than xxx milliseconds | 13:23 |
*** czzl has quit IRC | 13:24 | |
haypo | malor: the lastest option, the most complex, is to use new libraries async-friendly and patch existing libraries | 13:24 |
malor | heh, no that's not an option, IMO. we have a lot of SLOW queries in OS | 13:24 |
haypo | malor: yeah, it's exactly the same than deferToThread() | 13:24 |
haypo | (hum, i don't know if deferToThread() uses a pool of threads, but the idea and the API is the same) | 13:25 |
malor | I think, it does | 13:25 |
*** czzl has joined #openstack-oslo | 13:25 | |
malor | so I really like the idea of standardizing the event loop and making the code look synchronous while in fact it's asynchronous | 13:26 |
haypo | i understood that even if eventlet looks to run code concurrently, there are bottlenecks. for example, DB queries are usually executed in a sequential order (not in parallel) | 13:26 |
*** lbragstad has joined #openstack-oslo | 13:26 | |
malor | but I think this does not play nicely with a lot of existing blocking code | 13:27 |
malor | when eventlet doesn't not require you to change your code | 13:27 |
haypo | malor: IMO it's not a major issue if blocking calls make the applications slower. i don't plan to make Trollius mandatory | 13:28 |
malor | and deferToThread() is the same as if you were using native threads | 13:28 |
haypo | malor: i'm trying to find a way to make it optional. for example, for oslo messaging it's a new executor. you can choose between trollius & eventlet | 13:28 |
openstackgerrit | Tim Kelsey proposed a change to openstack/oslo.db: Adding dependency documentation for MySQL. Merging change I347e1c235bc82d6e510761f4b1c3c3de62994c2f fixing pip info. https://review.openstack.org/98105 | 13:29 |
malor | got it | 13:29 |
haypo | malor: if i reach my goal is replacing completly eventlet with trollius, i expect *better* performances because we will be able to run more tasks in parallel | 13:29 |
haypo | malor: like using real threads | 13:30 |
openstackgerrit | Tom Cammann proposed a change to openstack/oslo-incubator: generator: remove unused param, small clean up https://review.openstack.org/99707 | 13:30 |
haypo | in many cases, multiple python threads can run in parallel (even if there is a GIL) | 13:30 |
malor | why not just use threads and processes then, if we expect 90%+ calls to be blocking and executed in defereToThread() manner? :P | 13:30 |
*** lbragstad has quit IRC | 13:31 | |
malor | (I'm talking about API services here, which usually make a lot of DB calls in OS projects) | 13:32 |
openstackgerrit | Tim Kelsey proposed a change to openstack/oslo.db: Adding dependency documentation for MySQL. https://review.openstack.org/98105 | 13:32 |
malor | so yeah, in a perfect world, I'd like our code to be written in a 'concurrency-model agnostic' way, so that we could choose between eventlet/native threads/trollius/etc without having to modify the code actually. But I'm not sure that's possible. At least with trollius (or any other 'explicit' asynchronous model) | 13:34 |
haypo | malor: using multiple processes is a different question | 13:36 |
haypo | malor: i'm not sure that you can replace eventlet with multiple processes. waiting for blocking calls using processes is not efficient. if you will need a lot of them, which would waste memory | 13:36 |
haypo | malor: replacing eventlet with threads: hum... i don't like threads... :) and because of the Python GIL, using threads for anything is not efficient | 13:37 |
malor | right, that's why we've added eventlet in the first place :) | 13:37 |
haypo | malor: in Python, the most efficient option is to run all code in a single thread | 13:37 |
malor | yep. but that's in a perfect world, where all the blocking calls you do switch the execution context :) | 13:38 |
malor | what I'm just saying here, that for Nova API or any other API service we are going to use a lot of deferToThread() and I'm not sure that would be any better than using native threads | 13:39 |
malor | while using of 'prepared' libraries would definitely help us | 13:40 |
malor | but I doubt we have a lot of them now | 13:40 |
dhellmann | bnemec: do you mind if I fix the import grouping in https://review.openstack.org/#/c/92681/5/tests/test_handler.py in a follow-up patch to avoid rebasing that whole series? | 13:40 |
haypo | malor: you still have code between deferToThread(), and the idea is to drop these deferToThread() one by one :) | 13:40 |
haypo | between calls to* deferToThread() | 13:40 |
malor | haypo: well, yeah, but even if can process new requests between calls to deferToThread(), you are still going to run out of free threads in the pool eventually | 13:42 |
therve | sileht, Hey, I have an issue with that allowed_rpc_exception_modules options you removed | 13:48 |
sileht | therve, lol | 13:48 |
sileht | therve, tell me :) | 13:48 |
therve | sileht, So I need that option in heat, if only for backward compatibility | 13:49 |
therve | sileht, But the version without the option is not released, and I should probably support both | 13:49 |
therve | sileht, Any idea how I can check if the option is defined or not? | 13:50 |
therve | It's mostly for config generation stuff as it's not registered by oslo.messaging | 13:50 |
sileht | markmc, ^^ | 13:51 |
sileht | therve, I guess you can register the option yourself, no ? | 13:51 |
bnemec | dhellmann: Yeah, that works for me. | 13:51 |
therve | sileht, Yeah that part is fine. My issue is just that it's listed twice during config generation | 13:52 |
sileht | therve, arf, I see .... | 13:53 |
markmc | therve, you want to replace the use of allowed_rpc_exception_modules with the allowed_remote_exmods param to messaging.get_transport() | 13:53 |
bnemec | Oh, hmm. Not sure I should have approved that. The gate is still in pretty rough shape... | 13:54 |
therve | markmc, I'm not sure I understand what you mean. I want to use allowed_rpc_exception_modules to pass as allowed_remote_exmods param to get_transport | 13:54 |
markmc | therve, it shouldn't be user-configurable at all; heat knows the modules where it defines exceptions | 13:55 |
therve | markmc, I need to keep the option if only for backward compatibility | 13:56 |
sileht | therve, I think nobody change the default of the old 'allowed_rpc_exception_modules', because nova have not don't the backward compat, and nobody complains | 13:56 |
markmc | therve, you don't, there's no sane use case for it | 13:56 |
sileht | therve, this list should be defined by dev not deployer | 13:56 |
therve | Well the fact is that our deployed Heat has that options, and that we shouldn't break configuration file if we can | 13:57 |
sileht | therve, does heat works if the list is changed ? | 13:57 |
therve | sileht, There is one use case for heat, when as a deployer you have custom resources | 13:58 |
markmc | automatically add custom resource modules to the list ? | 13:59 |
*** yamahata has joined #openstack-oslo | 14:00 | |
therve | There is no way to know if it's the right thing to do or not | 14:00 |
*** yamahata has quit IRC | 14:01 | |
*** yamahata has joined #openstack-oslo | 14:01 | |
markmc | I'm struggling to think of a reason why that would be unsafe | 14:03 |
markmc | but anyway, if you don't want to take that advice | 14:03 |
markmc | have heat register the option | 14:04 |
markmc | wrap the register_opt() with a try/except to ignore allowed_remote_exmods | 14:04 |
markmc | ignore DuplicateOptError I mean | 14:04 |
therve | I'm a bit puzzled that you don't care about changing options | 14:04 |
markmc | I care about impacting users | 14:04 |
markmc | I'm pretty sure you won't impact any by removing the option | 14:05 |
markmc | it's pragmatism | 14:05 |
therve | Yeah the issue is that it's hard to know for sure | 14:06 |
haypo | markmc: hi. i updated my 2 patches for messaging: https://review.openstack.org/71003 & https://review.openstack.org/70948 -- can you take a look? | 14:06 |
markmc | haypo, yeah, I do want to get to them | 14:08 |
markmc | haypo, definitely high on my list | 14:08 |
sileht | markmc, don't forget the retry API too :p | 14:11 |
*** zzzeek has joined #openstack-oslo | 14:11 | |
markmc | sileht, looking at it right now | 14:11 |
markmc | sileht, just paging back in the past discussions is taking time :( | 14:12 |
markmc | sileht, damn volatile memory | 14:12 |
markmc | sileht, you need to rebase onto master, got a conflict with the default exchange thing | 14:17 |
sileht | arf | 14:18 |
markmc | :) | 14:18 |
*** ihrachyshka has quit IRC | 14:22 | |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Fix the notify method of the routing notifier https://review.openstack.org/92403 | 14:24 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Add transport reconnection retries https://review.openstack.org/75365 | 14:24 |
openstackgerrit | Mehdi Abaakouk proposed a change to openstack/oslo.messaging: Transport reconnection retries for notification https://review.openstack.org/92373 | 14:24 |
sileht | markmc, fixed | 14:24 |
markmc | sileht, thanks | 14:24 |
*** jecarey has joined #openstack-oslo | 14:37 | |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Build locale dir env var name consistently https://review.openstack.org/99054 | 14:37 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo.i18n: Fix import grouping in tests https://review.openstack.org/99948 | 14:42 |
*** zhiyan_ is now known as zhiyan | 14:46 | |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslo-specs: graduate-oslo-utils https://review.openstack.org/98431 | 14:50 |
*** ildikov has quit IRC | 14:52 | |
*** czzl has quit IRC | 15:01 | |
openstackgerrit | Matthew Booth proposed a change to openstack/oslo-specs: Add vmware-new-vim to juno https://review.openstack.org/99952 | 15:04 |
*** tsekiyama has joined #openstack-oslo | 15:06 | |
*** tsekiyama has quit IRC | 15:06 | |
*** tsekiyama has joined #openstack-oslo | 15:06 | |
*** zhiyan is now known as zhiyan_ | 15:09 | |
openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo.messaging: Add an example usage of RPCClient retry parameter https://review.openstack.org/99953 | 15:16 |
openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo.messaging: rabbit/qpid: remove the args/kwargs from ensure() https://review.openstack.org/99954 | 15:16 |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo.db: Add psycopg2 to test-requirements.txt https://review.openstack.org/99955 | 15:17 |
*** czzl has joined #openstack-oslo | 15:38 | |
*** erecio has quit IRC | 15:38 | |
*** bknudson has joined #openstack-oslo | 15:48 | |
*** bknudson has quit IRC | 15:48 | |
*** bknudson has joined #openstack-oslo | 15:49 | |
*** arnaud has joined #openstack-oslo | 16:00 | |
*** arnaud has quit IRC | 16:01 | |
*** arnaud has joined #openstack-oslo | 16:01 | |
*** jeffreycoho has quit IRC | 16:03 | |
*** dims_ has quit IRC | 16:07 | |
*** jeffreycoho has joined #openstack-oslo | 16:07 | |
*** dims_ has joined #openstack-oslo | 16:08 | |
*** jaosorior has quit IRC | 16:12 | |
*** I159 has joined #openstack-oslo | 16:20 | |
*** I159 has quit IRC | 16:20 | |
*** I159 has joined #openstack-oslo | 16:21 | |
*** viktors has quit IRC | 16:35 | |
*** I159 has quit IRC | 16:37 | |
openstackgerrit | Tomoki Sekiyama proposed a change to openstack/oslo-specs: Add ChainingRegExpFilter into rootwrap https://review.openstack.org/98536 | 16:38 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests https://review.openstack.org/93424 | 16:43 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Implementation Alembic as migration engine https://review.openstack.org/99965 | 16:43 |
* malor notices it's beer o'clock here | 16:54 | |
*** bknudson has left #openstack-oslo | 16:57 | |
openstackgerrit | Tomoki Sekiyama proposed a change to openstack/oslo.rootwrap: Add ChainingRegExpFilter for prefix utilities https://review.openstack.org/97336 | 16:59 |
*** dims_ has quit IRC | 17:04 | |
*** czzl has left #openstack-oslo | 17:05 | |
*** harlowja_away is now known as harlowja | 17:06 | |
*** arnaud has quit IRC | 17:19 | |
*** markmcclain has joined #openstack-oslo | 17:30 | |
*** praneshp has joined #openstack-oslo | 17:31 | |
*** markmc has quit IRC | 17:39 | |
*** ildikov has joined #openstack-oslo | 17:44 | |
*** rpodolyaka has joined #openstack-oslo | 17:45 | |
*** rpodolyaka has quit IRC | 17:46 | |
*** rpodolyaka has joined #openstack-oslo | 17:48 | |
*** praneshp has quit IRC | 17:55 | |
*** rpodolyaka has quit IRC | 17:56 | |
*** praneshp has joined #openstack-oslo | 18:03 | |
*** arnaud has joined #openstack-oslo | 18:14 | |
*** ihrachyshka has joined #openstack-oslo | 18:17 | |
*** flaper87 is now known as flaper87|afk | 18:20 | |
*** bpokorny has quit IRC | 18:24 | |
*** czzl has joined #openstack-oslo | 18:25 | |
openstackgerrit | George Peristerakis proposed a change to openstack/oslo-incubator: wrapping the mock module with six https://review.openstack.org/87375 | 18:34 |
*** dims_ has joined #openstack-oslo | 18:41 | |
*** bpokorny1 has joined #openstack-oslo | 18:42 | |
*** bpokorny1 is now known as bpokorny | 18:42 | |
*** tkelsey has quit IRC | 18:45 | |
*** rpodolyaka has joined #openstack-oslo | 18:46 | |
*** ihrachyshka has quit IRC | 18:54 | |
*** ihrachyshka has joined #openstack-oslo | 18:58 | |
*** dims has joined #openstack-oslo | 19:00 | |
*** dims_ has quit IRC | 19:03 | |
*** czzl has left #openstack-oslo | 19:18 | |
*** czzl has joined #openstack-oslo | 19:27 | |
*** czzl has quit IRC | 19:40 | |
ihrachyshka | dhellmann: hey. I have a question re oslo.messaging notifier types | 19:42 |
*** czzl has joined #openstack-oslo | 19:44 | |
dhellmann | ihrachyshka: I'll try to help but markmc and flaper87|afk know more about the lib than I do | 19:46 |
*** rpodolyaka has quit IRC | 19:48 | |
ihrachyshka | dhellmann: ok, so basically, I'm trying to 'upgrade' neutron.conf for neutron, and I'm interested in what I need to set for notification_driver | 19:52 |
ihrachyshka | atm it's neutron.openstack.common.notifier.rpc_notifier, and this probably maps to 'messaging' in oslo.messaging | 19:53 |
ihrachyshka | but there's also messagingv2 there, and it even goes as far as to say about 'messaging': This driver should only be used in cases where there are existing consumers | 19:53 |
ihrachyshka | deployed which do not support the 2.0 message format. | 19:53 |
ihrachyshka | (probably meaning, messagingv2 is somehow better) | 19:54 |
ihrachyshka | so the question is - what to set | 19:54 |
dhellmann | ihrachyshka: that's a good question, but I think you're going to have to wait for markmc or flaper87|afk because I don't know the answer myself | 19:54 |
dhellmann | ihrachyshka: you could look at what ceilometer is doing, since they deal with notifications (receiving), or nova (they send) | 19:55 |
ihrachyshka | dhellmann: in nova, they auto generate the conf I think, meaning, notification_driver value is default ([]), and requires update to be meaningful (I guess, I didn't actually run it myself). as for other projects, that's a good idea, I'll check them, thanks | 19:58 |
*** rpodolyaka has joined #openstack-oslo | 20:04 | |
*** czzl has left #openstack-oslo | 20:06 | |
ihrachyshka | dhellmann: it turns out most projects don't have it specified, instead, devstack sets it for them, and in case of oslo.messaging projects (glance and nova), it's 'messaging'. Looks like no one actually uses that 'recommended' messagingv2 notifier | 20:18 |
dhellmann | ihrachyshka: I don't know the history there. It may have something to do with message signing and encryption? If you send email to the -dev ML, we can both find out. :-) | 20:23 |
ihrachyshka | dhellmann: ok, let's do it :) | 20:24 |
ihrachyshka | dhellmann: I'll send it the next time I have access to my mailbox :) | 20:24 |
*** flaper87|afk is now known as flaper87 | 20:33 | |
*** jecarey has quit IRC | 20:40 | |
dhellmann | ihrachyshka: sounds good | 20:45 |
openstackgerrit | Doug Hellmann proposed a change to openstack/oslotest: Add API documentation https://review.openstack.org/98861 | 20:51 |
*** rpodolyaka has quit IRC | 20:54 | |
*** ihrachyshka has quit IRC | 21:05 | |
*** mgagne has quit IRC | 21:11 | |
*** mgagne has joined #openstack-oslo | 21:20 | |
*** mgagne is now known as Guest61486 | 21:20 | |
*** bpokorny has quit IRC | 21:45 | |
*** harlowja has quit IRC | 22:03 | |
*** zzzeek has quit IRC | 22:11 | |
*** flaper87 is now known as flaper87|afk | 22:13 | |
*** zzzeek has joined #openstack-oslo | 22:25 | |
*** zzzeek has quit IRC | 22:26 | |
*** mriedem has quit IRC | 22:26 | |
*** Guest61486 has quit IRC | 22:38 | |
*** mgagne has joined #openstack-oslo | 22:41 | |
*** mgagne is now known as Guest8031 | 22:41 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack-dev/hacking: Updated from global requirements https://review.openstack.org/100037 | 22:50 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/99084 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.config: Updated from global requirements https://review.openstack.org/99085 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/99086 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.i18n: Updated from global requirements https://review.openstack.org/96239 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/99087 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslotest: Updated from global requirements https://review.openstack.org/99088 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.version: Updated from global requirements https://review.openstack.org/100038 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/99089 | 22:56 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/pycadf: Updated from global requirements https://review.openstack.org/99091 | 22:57 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/stevedore: Updated from global requirements https://review.openstack.org/96273 | 22:57 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 22:58 |
*** markmcclain has quit IRC | 23:03 | |
*** dims has quit IRC | 23:47 | |
*** praneshp has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!