*** dave-mccowan has quit IRC | 00:14 | |
*** dave-mccowan has joined #openstack-oslo | 00:30 | |
*** dave-mccowan has quit IRC | 01:10 | |
*** phuongnh has joined #openstack-oslo | 01:20 | |
*** bobh has quit IRC | 01:24 | |
*** dave-mccowan has joined #openstack-oslo | 01:25 | |
*** bobh has joined #openstack-oslo | 01:43 | |
*** bobh has quit IRC | 01:47 | |
openstackgerrit | ayoung proposed openstack/oslo.policy master: namespaced flag on checker CLI https://review.openstack.org/610402 | 01:50 |
---|---|---|
*** bobh has joined #openstack-oslo | 01:51 | |
*** bobh has quit IRC | 02:37 | |
*** bobh has joined #openstack-oslo | 02:41 | |
*** dave-mccowan has quit IRC | 03:11 | |
*** bobh has quit IRC | 03:18 | |
*** bobh has joined #openstack-oslo | 03:20 | |
*** bobh has quit IRC | 03:25 | |
*** chhagarw has joined #openstack-oslo | 03:38 | |
*** hoonetorg has quit IRC | 04:48 | |
*** hoonetorg has joined #openstack-oslo | 05:01 | |
*** hoonetorg has quit IRC | 05:12 | |
*** hoonetorg has joined #openstack-oslo | 05:33 | |
*** pooja_jadhav has joined #openstack-oslo | 06:02 | |
*** pcaruana has joined #openstack-oslo | 06:21 | |
*** jbadiapa has joined #openstack-oslo | 06:42 | |
*** a-pugachev has joined #openstack-oslo | 06:44 | |
*** e0ne has joined #openstack-oslo | 06:45 | |
*** pcaruana has quit IRC | 06:58 | |
*** rcernin has quit IRC | 06:59 | |
*** rcernin has joined #openstack-oslo | 07:05 | |
*** rcernin has quit IRC | 07:07 | |
*** pcaruana has joined #openstack-oslo | 07:13 | |
*** hberaud has joined #openstack-oslo | 07:15 | |
*** pooja_jadhav has quit IRC | 07:32 | |
*** pooja_jadhav has joined #openstack-oslo | 07:35 | |
*** tosky has joined #openstack-oslo | 07:42 | |
*** lpetrut has joined #openstack-oslo | 07:50 | |
*** rpittau has joined #openstack-oslo | 08:02 | |
*** shardy has joined #openstack-oslo | 08:09 | |
*** shardy has quit IRC | 08:16 | |
*** finucannot is now known as stephenfin | 08:20 | |
*** shardy has joined #openstack-oslo | 08:31 | |
*** hberaud has quit IRC | 08:43 | |
*** hberaud has joined #openstack-oslo | 08:45 | |
*** bobh has joined #openstack-oslo | 08:45 | |
*** bobh has quit IRC | 08:50 | |
*** lpetrut has quit IRC | 09:06 | |
*** chhavi__ has joined #openstack-oslo | 09:10 | |
*** sambetts|afk is now known as sambetts | 09:30 | |
*** zzzeek_ has quit IRC | 09:37 | |
*** zzzeek has joined #openstack-oslo | 09:40 | |
*** bnemec has joined #openstack-oslo | 09:45 | |
*** hoonetorg has quit IRC | 09:51 | |
*** bnemec has quit IRC | 09:53 | |
openstackgerrit | Merged openstack/oslo.i18n master: Don't quote {posargs} in tox.ini https://review.openstack.org/612181 | 10:01 |
*** sean-k-mooney has joined #openstack-oslo | 10:03 | |
*** hoonetorg has joined #openstack-oslo | 10:08 | |
*** phuongnh has quit IRC | 10:12 | |
*** shardy has quit IRC | 10:13 | |
*** shardy has joined #openstack-oslo | 10:16 | |
*** dave-mccowan has joined #openstack-oslo | 10:21 | |
*** pbourke has quit IRC | 10:23 | |
*** cdent has joined #openstack-oslo | 10:30 | |
*** pbourke has joined #openstack-oslo | 10:47 | |
*** cdent has quit IRC | 10:50 | |
*** bobh has joined #openstack-oslo | 10:57 | |
*** bobh has quit IRC | 11:02 | |
openstackgerrit | Thomas Herve proposed openstack/oslo.service master: Use eventletutils.Event for loopingcall events https://review.openstack.org/611807 | 11:17 |
*** pcaruana has quit IRC | 11:26 | |
*** ansmith has quit IRC | 11:27 | |
*** bobh has joined #openstack-oslo | 11:27 | |
*** pcaruana has joined #openstack-oslo | 11:48 | |
*** cdent has joined #openstack-oslo | 11:52 | |
*** pcaruana has quit IRC | 12:12 | |
*** pcaruana has joined #openstack-oslo | 12:12 | |
*** raildo has joined #openstack-oslo | 12:14 | |
*** bobh has quit IRC | 12:16 | |
*** smcginnis_vaca is now known as smcginnis | 12:26 | |
openstackgerrit | Thomas Herve proposed openstack/oslo.i18n master: Override getttext.find to cache result https://review.openstack.org/612376 | 12:30 |
*** chhavi__ has quit IRC | 12:51 | |
*** ansmith has joined #openstack-oslo | 12:52 | |
*** bobh has joined #openstack-oslo | 12:52 | |
*** kgiusti has joined #openstack-oslo | 12:52 | |
*** jroll has quit IRC | 12:53 | |
*** jroll has joined #openstack-oslo | 12:54 | |
zaneb | therve: where are we at with https://review.openstack.org/611807 ? | 13:07 |
therve | zaneb: I made the suggested change, but I was expected the overall approach to not work | 13:14 |
therve | zaneb: I did some manual testing, and it seems it does | 13:14 |
zaneb | therve: I think it actually might | 13:14 |
therve | Not sure why though :) | 13:14 |
zaneb | https://bugs.launchpad.net/oslo.messaging/+bug/1518430/comments/11 kind-of explains it | 13:14 |
openstack | Launchpad bug 1518430 in Ubuntu Cloud Archive kilo "liberty: ~busy loop on epoll_wait being called with zero timeout" [Medium,Fix committed] | 13:14 |
therve | zaneb: Obviously the original bug is not tested... | 13:15 |
openstackgerrit | Ilya Shakhat proposed openstack/osprofiler master: [TEST] Add a job to run Tempest tests on py27 https://review.openstack.org/609959 | 13:15 |
therve | zaneb: It explains something, not sure we're talking about the same though :) | 13:15 |
therve | zaneb: I'm wondering if my patch doesn't break https://bugs.launchpad.net/oslo.service/+bug/1660210 | 13:16 |
openstack | Launchpad bug 1660210 in oslo.service "LoopingCall sleep causes graceful process shutdown delay" [Undecided,Fix released] - Assigned to Allain Legacy (alegacy) | 13:16 |
zaneb | I don't know how the monkeypatching works, but it seems from that that just using the threading.Event interface, even when monkeypatches, causes the CPU usage | 13:16 |
zaneb | therve: I think that one is fine | 13:16 |
therve | OK | 13:17 |
zaneb | therve: your patch is equivalent to the original version of that fix | 13:17 |
therve | Well, kinda | 13:17 |
therve | zaneb: The original used threading.Event | 13:17 |
zaneb | it's doing wait() on the event instead of sleep(), so it won't go back to sleep after it's interrupted | 13:17 |
therve | Right that's probably why it works | 13:18 |
therve | I tried reimplemeting with spawn_after and it didn't | 13:18 |
therve | So hooray, I guess | 13:20 |
therve | zaneb: Created https://review.openstack.org/612376 while stracing processes btw | 13:20 |
*** a-pugachev has quit IRC | 13:26 | |
zaneb | therve: have you tried running the heat unit tests with this patch? I think that's what might break | 13:36 |
therve | zaneb: https://review.openstack.org/611807 ? | 13:36 |
zaneb | yes | 13:36 |
therve | Let me check | 13:37 |
zaneb | because eventletutils is going to call threading.Event when thread is not monkeypatched | 13:37 |
zaneb | which was the thing that caused problems with the original patch | 13:37 |
therve | OK let's see | 13:37 |
*** elbragstad is now known as lbragstad | 13:39 | |
stephenfin | kgiusti: Fancy taking a look at this? https://review.openstack.org/#/c/611310/ | 13:57 |
kgiusti | stephenfin: I'm on it... | 14:00 |
*** lpetrut has joined #openstack-oslo | 14:03 | |
therve | zaneb: Yeah it breaks... | 14:03 |
zaneb | dammit | 14:04 |
zaneb | therve: I think we just want to always use the eventletutils._Event class, regardless of monkeypatching status | 14:04 |
zaneb | but that's not a public interface... | 14:05 |
zaneb | either that or keep the two Event classes in oslo.service and just substitute eventletutils.Event for threading.Event in the ThreadingEvent one | 14:06 |
zaneb | which is altogether too many Events but I'll take whatever works at this point | 14:06 |
*** SteelyDan is now known as dansmith | 14:11 | |
*** a-pugachev has joined #openstack-oslo | 14:18 | |
kgiusti | stephenfin: LGTM - approved | 14:19 |
stephenfin | kgiusti: Ta :) | 14:31 |
openstackgerrit | Thomas Herve proposed openstack/oslo.service master: Use eventlet Event for loopingcall events https://review.openstack.org/611807 | 14:40 |
therve | zaneb: This one seems good ^^^ | 14:40 |
therve | Well except the whole shitty duplicated code | 14:41 |
zaneb | meh | 14:44 |
zaneb | +2 | 14:44 |
openstackgerrit | Thomas Herve proposed openstack/oslo.i18n master: Override getttext.find to cache result https://review.openstack.org/612376 | 14:47 |
therve | We should check with the projects that got the issue with the first patch | 14:47 |
*** zaneb has quit IRC | 15:10 | |
*** pcaruana has quit IRC | 15:12 | |
*** e0ne has quit IRC | 15:15 | |
*** zaneb has joined #openstack-oslo | 15:17 | |
*** a-pugachev has quit IRC | 15:18 | |
*** a-pugachev has joined #openstack-oslo | 15:18 | |
*** e0ne has joined #openstack-oslo | 15:28 | |
*** e0ne has quit IRC | 15:32 | |
*** bnemec has joined #openstack-oslo | 15:51 | |
*** bnemec has quit IRC | 15:59 | |
*** phuongnh has joined #openstack-oslo | 16:07 | |
*** phuongnh has quit IRC | 16:12 | |
*** mattgo has joined #openstack-oslo | 16:12 | |
*** shardy has quit IRC | 16:24 | |
*** zaneb has quit IRC | 16:45 | |
*** zaneb has joined #openstack-oslo | 16:46 | |
*** e0ne has joined #openstack-oslo | 16:52 | |
*** a-pugachev has quit IRC | 17:03 | |
*** e0ne has quit IRC | 17:06 | |
*** raildo has quit IRC | 17:14 | |
*** sambetts is now known as sambetts|afk | 17:18 | |
*** bnemec has joined #openstack-oslo | 17:19 | |
*** pcaruana has joined #openstack-oslo | 17:27 | |
*** e0ne has joined #openstack-oslo | 17:30 | |
*** mattgo has quit IRC | 17:32 | |
*** irclogbot_4 has joined #openstack-oslo | 17:34 | |
*** raildo has joined #openstack-oslo | 17:37 | |
*** raildo has quit IRC | 17:43 | |
*** irclogbot_4 has quit IRC | 17:56 | |
*** pcaruana has quit IRC | 17:57 | |
*** raildo has joined #openstack-oslo | 18:01 | |
*** irclogbot_4 has joined #openstack-oslo | 18:06 | |
*** raildo has quit IRC | 18:11 | |
*** eandersson has joined #openstack-oslo | 18:19 | |
*** e0ne has quit IRC | 18:24 | |
eandersson | Does anyone know why virtual hosts now defaults to None, instead of / for RabbitMQ? | 18:28 |
kgiusti | eandersson: bug - fix was merged on master last week | 18:32 |
eandersson | ah I see - I was just gonna add a test for it :D | 18:32 |
eandersson | thanks kgiusti | 18:33 |
kgiusti | eandersson: the work around is to append a '/' to the configured transport_url (if your config has a transport_url specified) | 18:33 |
eandersson | It would be nice to have a test to catch this | 18:34 |
kgiusti | eandersson: https://bugs.launchpad.net/oslo.messaging/+bug/1798241 | 18:34 |
openstack | Launchpad bug 1798241 in oslo.messaging "transport_url without trailing '/' considered virtual host 'None'" [High,Fix released] - Assigned to Ken Giusti (kgiusti) | 18:34 |
kgiusti | eandersson: yes - there's a patch to test this in flight - https://review.openstack.org/#/c/611870/ | 18:35 |
eandersson | Yep - I fixed it in Magnum by just using the default devstack construct (which adds a trailing /) | 18:35 |
eandersson | Sweet | 18:35 |
*** chhagarw has quit IRC | 18:51 | |
*** HenryG has quit IRC | 18:51 | |
*** flaper87 has quit IRC | 18:51 | |
*** HenryG has joined #openstack-oslo | 18:51 | |
*** flaper87 has joined #openstack-oslo | 18:51 | |
*** lpetrut has quit IRC | 18:56 | |
*** irclogbot_4 has quit IRC | 19:14 | |
*** cdent has quit IRC | 20:22 | |
*** zaneb has quit IRC | 20:45 | |
*** kgiusti has left #openstack-oslo | 20:53 | |
*** ansmith has quit IRC | 20:56 | |
*** bobh has quit IRC | 21:08 | |
*** bobh has joined #openstack-oslo | 21:09 | |
*** bobh has quit IRC | 21:13 | |
*** zaneb has joined #openstack-oslo | 21:21 | |
openstackgerrit | Merged openstack/oslo.service master: Use eventlet Event for loopingcall events https://review.openstack.org/611807 | 21:26 |
*** zaneb has quit IRC | 21:27 | |
*** dmellado has quit IRC | 21:32 | |
*** bnemec has quit IRC | 22:18 | |
*** rcernin has joined #openstack-oslo | 22:46 | |
*** tosky has quit IRC | 23:09 | |
*** ansmith has joined #openstack-oslo | 23:18 | |
*** zaneb has joined #openstack-oslo | 23:23 | |
*** zaneb has quit IRC | 23:23 | |
*** zaneb has joined #openstack-oslo | 23:24 | |
*** rcernin_ has joined #openstack-oslo | 23:28 | |
*** rcernin has quit IRC | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!