*** oomichi has joined #openstack-oslo | 00:10 | |
*** jaosorior has quit IRC | 01:22 | |
*** rpodolyaka has quit IRC | 01:29 | |
*** rpodolyaka1 has joined #openstack-oslo | 01:29 | |
*** harlowja_at_home has quit IRC | 01:34 | |
*** amotoki has quit IRC | 01:38 | |
*** yamahata has joined #openstack-oslo | 01:42 | |
openstackgerrit | A change was merged to openstack/oslotest: Add known issue about time.time mocking https://review.openstack.org/95411 | 01:43 |
---|---|---|
*** amotoki has joined #openstack-oslo | 01:46 | |
*** enykeev has quit IRC | 01:55 | |
*** dims has quit IRC | 02:18 | |
*** dstanek is now known as dstanek_zzz | 02:27 | |
openstackgerrit | A change was merged to openstack/oslo.config: generator: refactor MultiStrOpt handling https://review.openstack.org/104225 | 02:34 |
openstackgerrit | A change was merged to openstack/oslo.config: cfg,generator: add Opt.sample_default attribute https://review.openstack.org/104226 | 02:35 |
*** dstanek_zzz is now known as dstanek | 02:38 | |
*** dims has joined #openstack-oslo | 02:44 | |
*** dims has quit IRC | 02:50 | |
*** lbragstad_ has joined #openstack-oslo | 03:09 | |
openstackgerrit | A change was merged to openstack/oslo.rootwrap: Let tests pass on distros where "ip" is in /bin https://review.openstack.org/104865 | 03:27 |
openstackgerrit | A change was merged to openstack/oslo.vmware: Translations: make use of _LE, _LI and _LW https://review.openstack.org/101821 | 03:27 |
*** liusheng has quit IRC | 03:33 | |
*** liusheng has joined #openstack-oslo | 03:34 | |
*** zhiyan_ is now known as zhiyan | 03:40 | |
*** dims_ has joined #openstack-oslo | 03:46 | |
*** lbragstad_ has left #openstack-oslo | 03:48 | |
*** dims_ has quit IRC | 03:51 | |
*** lbragstad_ has joined #openstack-oslo | 03:58 | |
*** zhiyan is now known as zhiyan_ | 04:20 | |
*** arnaud has joined #openstack-oslo | 04:31 | |
*** SridharG has joined #openstack-oslo | 04:36 | |
*** dims_ has joined #openstack-oslo | 04:47 | |
*** dims_ has quit IRC | 04:52 | |
*** ajc_ has joined #openstack-oslo | 04:52 | |
*** harlowja_at_home has joined #openstack-oslo | 04:58 | |
*** arnaud has quit IRC | 05:04 | |
*** arnaud has joined #openstack-oslo | 05:10 | |
openstackgerrit | A change was merged to openstack/taskflow: Remove pbr as a runtime dependency https://review.openstack.org/105043 | 05:10 |
*** arnaud has quit IRC | 05:16 | |
*** ildikov has joined #openstack-oslo | 05:18 | |
*** ildikov_ has quit IRC | 05:22 | |
*** ildikov has quit IRC | 05:23 | |
*** yamahata has quit IRC | 05:44 | |
*** dims_ has joined #openstack-oslo | 05:47 | |
*** dims_ has quit IRC | 05:53 | |
*** arnaud has joined #openstack-oslo | 06:28 | |
*** ildikov_ has joined #openstack-oslo | 06:29 | |
*** AAzza_afk is now known as AAzza | 06:41 | |
*** harlowja_at_home has quit IRC | 06:47 | |
*** dims_ has joined #openstack-oslo | 06:48 | |
*** liusheng has quit IRC | 06:48 | |
*** liusheng has joined #openstack-oslo | 06:50 | |
*** dims_ has quit IRC | 06:53 | |
*** flaper87|afk is now known as flaper87 | 07:24 | |
*** arnaud has quit IRC | 07:29 | |
*** AAzza is now known as AAzza_afk | 07:31 | |
*** nacim has joined #openstack-oslo | 07:42 | |
*** AAzza_afk is now known as AAzza | 07:44 | |
*** yamahata has joined #openstack-oslo | 07:45 | |
openstackgerrit | Vipin Balachandran proposed a change to openstack/oslo.vmware: Fix wrong usage of assertRaises https://review.openstack.org/105077 | 07:46 |
*** sreshetn1ak has quit IRC | 07:51 | |
*** markmc has joined #openstack-oslo | 07:51 | |
*** linkid has joined #openstack-oslo | 08:01 | |
*** ajo has joined #openstack-oslo | 08:05 | |
*** dstanek is now known as dstanek_zzz | 08:08 | |
*** i159 has joined #openstack-oslo | 08:09 | |
*** ihrachyshka has joined #openstack-oslo | 08:18 | |
*** linkid has quit IRC | 08:18 | |
*** yamahata has quit IRC | 08:20 | |
*** dstanek_zzz is now known as dstanek | 08:29 | |
*** zhiyan_ is now known as zhiyan | 08:30 | |
openstackgerrit | Christian Berendt proposed a change to openstack/oslo.config: Replaced 'e.g.' with 'for example' https://review.openstack.org/105086 | 08:35 |
*** tkelsey has joined #openstack-oslo | 08:38 | |
*** dstanek is now known as dstanek_zzz | 08:39 | |
*** rpodolyaka1 has quit IRC | 08:42 | |
*** rpodolyaka has joined #openstack-oslo | 08:46 | |
*** pblaho has joined #openstack-oslo | 08:47 | |
*** rpodolyaka has quit IRC | 08:50 | |
*** rpodolyaka1 has joined #openstack-oslo | 08:50 | |
*** sreshetnyak has joined #openstack-oslo | 08:50 | |
*** ihrachyshka has quit IRC | 09:26 | |
*** ihrachyshka has joined #openstack-oslo | 09:26 | |
*** linkid has joined #openstack-oslo | 09:28 | |
linkid | HI | 09:29 |
linkid | hi | 09:29 |
linkid | I would like to modify a function to clarify an exception | 09:31 |
linkid | in oslo-incubator | 09:31 |
linkid | it is the function openstack.common.importutils.import_versioned_module, which will raise an ImportError instead of apiclient.exceptions.UnsupportedVersion | 09:34 |
linkid | is it relevant ? | 09:34 |
linkid | I think it is, because in my tests, I will check this exception instead of ImportError, due to the name of the function | 09:36 |
linkid | I would like your opinion before I submit a ticket and a review | 09:38 |
*** praneshp has quit IRC | 09:40 | |
therve | linkid, Well, you can't raise apiclient exceptions in importutils | 09:41 |
linkid | why ? | 09:41 |
therve | Because it can be used outside of the client | 09:41 |
therve | It would seem better to modify apiclient to catch ImportError and raise UnsupportedVersion | 09:43 |
linkid | but this function is never called in the apiclient | 09:45 |
linkid | so it will be in my api where I have to catch this exception ? | 09:48 |
linkid | it is weird | 09:48 |
linkid | no ? | 09:50 |
linkid | *api → client | 09:58 |
boris-42 | markmc hi | 10:00 |
boris-42 | markmc i have some strange behavior of oslo.messaging | 10:01 |
boris-42 | markmc in part of nova & cinder | 10:01 |
boris-42 | markmc unit tests | 10:01 |
boris-42 | markmc I am trying to initialize oslo.messaging.Notifier during start of service | 10:01 |
boris-42 | markmc and it says that | 10:02 |
boris-42 | TypeError: __init__() got an unexpected keyword argument 'topic' | 10:02 |
boris-42 | ^ Notifier.__init__() | 10:02 |
boris-42 | (all other tests + tempest are passing) | 10:02 |
boris-42 | markmc ^ haven't see such behavior before? | 10:02 |
boris-42 | seen* | 10:02 |
*** oomichi has quit IRC | 10:04 | |
boris-42 | markmc so this line fails https://github.com/stackforge/osprofiler/blob/master/osprofiler/_notifiers/messaging.py#L31-L32 | 10:05 |
*** pcm_ has joined #openstack-oslo | 10:08 | |
*** pcm_ has quit IRC | 10:12 | |
*** pcm_ has joined #openstack-oslo | 10:13 | |
ihrachyshka | boris-42: what is that Notifier? if it's unit tests, it may be some fake notifier that does not support topic argument | 10:27 |
boris-42 | ihrachyshka yep seems like so | 10:28 |
boris-42 | ihrachyshka I actually don't know let me just show patch set | 10:28 |
boris-42 | ihrachyshka https://review.openstack.org/#/c/103415/ | 10:28 |
boris-42 | ihrachyshka take a look here at py26 / py27 jobs | 10:28 |
ihrachyshka | boris-42: I had similar failure when I worked on neutron migration to oslo.msg | 10:28 |
ihrachyshka | lemme check | 10:28 |
boris-42 | ihrachyshka oh nice thank you | 10:28 |
ihrachyshka | yeah, right | 10:29 |
ihrachyshka | see https://github.com/openstack/cinder/blob/master/cinder/tests/fake_notifier.py | 10:29 |
ihrachyshka | FakeNotifier is not aware of topic | 10:29 |
ihrachyshka | and here is what we have in neutron: https://github.com/openstack/neutron/blob/master/neutron/tests/fake_notifier.py | 10:29 |
ihrachyshka | compare __init__ methods | 10:29 |
*** linkid has quit IRC | 10:31 | |
boris-42 | ihrachyshka thank you! | 10:37 |
boris-42 | ihrachyshka really thank you! | 10:37 |
boris-42 | =) | 10:37 |
boris-42 | ihrachyshka so I need just one line change =) | 10:37 |
boris-42 | ihrachyshka oh there are more bugs | 10:41 |
boris-42 | ihrachyshka e..g _notify now accepts priority as a last argument | 10:41 |
ihrachyshka | boris-42: you should fix your fake notifier to be more similar to a real one | 10:43 |
boris-42 | ihrachyshka yep I'll | 10:44 |
*** ndipanov has joined #openstack-oslo | 10:49 | |
*** dims_ has joined #openstack-oslo | 10:53 | |
*** mkoderer has joined #openstack-oslo | 10:54 | |
*** dims_ has quit IRC | 10:58 | |
*** dims_ has joined #openstack-oslo | 11:14 | |
boris-42 | ihrachyshka ohh | 11:33 |
boris-42 | ihrachyshka 2014-07-07 11:13:21.203 | ValueError: Circular reference detected | 11:33 |
boris-42 | ihrachyshka https://jenkins02.openstack.org/job/gate-cinder-python26/7180/console | 11:33 |
boris-42 | ihrachyshka this seems quite wired=) | 11:33 |
ihrachyshka | boris-42: check your payload, probably it | 11:34 |
ihrachyshka | boris-42: check your payload, probably it is not json-able | 11:35 |
boris-42 | ihrachyshka actually I don't send any notification | 11:35 |
boris-42 | ihrachyshka so it seems related to the project unit tests | 11:35 |
boris-42 | ihrachyshka but thanks I will try to figure out what is happening there | 11:35 |
*** linkid has joined #openstack-oslo | 11:39 | |
*** AAzza is now known as AAzza_afk | 11:41 | |
*** markmcclain has joined #openstack-oslo | 12:09 | |
*** viktors|afk is now known as viktors | 12:13 | |
*** dstanek_zzz is now known as dstanek | 12:20 | |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Fix AMQPListener for polling with timeout https://review.openstack.org/104964 | 12:26 |
*** jraim has quit IRC | 12:28 | |
*** mgagne has quit IRC | 12:28 | |
*** russellb has quit IRC | 12:29 | |
*** wendar has quit IRC | 12:29 | |
*** wendar has joined #openstack-oslo | 12:29 | |
*** russellb has joined #openstack-oslo | 12:30 | |
*** jgriffit1 has joined #openstack-oslo | 12:30 | |
*** jraim has joined #openstack-oslo | 12:30 | |
*** hartsocks has quit IRC | 12:30 | |
*** GheRiver1 has joined #openstack-oslo | 12:30 | |
*** GheRivero has quit IRC | 12:31 | |
*** jgriffith has quit IRC | 12:31 | |
*** mgagne has joined #openstack-oslo | 12:31 | |
*** jraim has quit IRC | 12:31 | |
*** jraim has joined #openstack-oslo | 12:31 | |
*** hartsocks has joined #openstack-oslo | 12:31 | |
*** hartsocks is now known as Guest69876 | 12:31 | |
*** ajc_ has quit IRC | 12:33 | |
*** dims_ has quit IRC | 12:37 | |
*** dims_ has joined #openstack-oslo | 12:37 | |
*** bknudson has quit IRC | 12:39 | |
*** lbragstad_ has quit IRC | 12:43 | |
*** AAzza_afk is now known as AAzza | 12:51 | |
*** bknudson has joined #openstack-oslo | 12:54 | |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: Added missing jsonutils.dump() function https://review.openstack.org/105161 | 13:07 |
*** ajo has quit IRC | 13:08 | |
*** joesavak has joined #openstack-oslo | 13:13 | |
*** jecarey has quit IRC | 13:15 | |
*** dstanek is now known as dstanek_zzz | 13:17 | |
*** yamahata has joined #openstack-oslo | 13:20 | |
*** ajo has joined #openstack-oslo | 13:23 | |
dhellmann_ | markmc: I saw that the config filter merged in oslo.config. Do you want to do a release this week? I'm going to be releasing a few other libs this morning, so I can do it if you think we're ready. | 13:29 |
*** dhellmann_ is now known as dhellmann | 13:29 | |
markmc | dhellmann, yep, had planned on doing a release | 13:30 |
markmc | dhellmann, I'd like the patch to remove the sanitizer merged first | 13:30 |
dhellmann | markmc: makes sense | 13:31 |
markmc | dhellmann, also, we're prevented from using it until infra do the work for mirroring alpha releases | 13:31 |
dhellmann | markmc: I thought that was done? | 13:31 |
markmc | dhellmann, it passed my by if it did | 13:32 |
* markmc looks | 13:32 | |
dhellmann | markmc: they switched the mirror to use bandersnatch instead of the home-grown thing they had put together | 13:32 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/105171 | 13:32 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/105173 | 13:32 |
dhellmann | markmc: I thought that fixed it | 13:32 |
markmc | dhellmann, didn't know that had landed, last I say was | 13:33 |
markmc | http://lists.openstack.org/pipermail/openstack-dev/2014-July/039089.html | 13:33 |
markmc | https://review.openstack.org/#/c/103256/ | 13:33 |
markmc | https://review.openstack.org/#/c/103536/ | 13:33 |
*** ajo has quit IRC | 13:34 | |
*** jecarey has joined #openstack-oslo | 13:34 | |
* dhellmann grumbles | 13:34 | |
*** zhiyan is now known as zhiyan_ | 13:37 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Added missing jsonutils.dump() function https://review.openstack.org/105161 | 13:37 |
dhellmann | markmc: why did you mark https://review.openstack.org/#/c/103256/ as WIP? | 13:38 |
dhellmann | because we needed to fix the mirroring script first? | 13:39 |
markmc | yes | 13:39 |
dhellmann | markmc: ok, it seems like we want to try to merge that to test whether alphas are being allowed, right? | 13:39 |
markmc | dhellmann, sure, if you're saying there's a reasonable chance it just works now ... I can approve again | 13:40 |
dhellmann | markmc: I'll go ask infra first, I guess | 13:40 |
markmc | dhellmann, got any link to this bandersnatch work ? | 13:40 |
markmc | "stuff happened on the infra channel, do keep up" :) | 13:41 |
*** sigmavirus24 has joined #openstack-oslo | 13:42 | |
dhellmann | markmc: I think it was just on irc | 13:42 |
dhellmann | markmc: our theme for the K summit should be "write it down" | 13:42 |
markmc | dhellmann, right :) | 13:43 |
haypo | hi markmc & dhellmann | 13:43 |
dhellmann | hi, haypo | 13:43 |
markmc | dhellmann, well, most of the other programs are doing pretty well on that in J :) | 13:43 |
dhellmann | markmc: meaning infra needs to work on it, oslo does, or both? :-) | 13:44 |
markmc | dhellmann, oh goodness, the former :) | 13:44 |
markmc | hi haypo | 13:44 |
dhellmann | markmc: ok, the i18n work came to mind | 13:45 |
markmc | dhellmann, meh, discussed on the mailing list, you wrote excellent docs ... it was grand in the end | 13:46 |
*** mriedem has joined #openstack-oslo | 13:46 | |
markmc | dhellmann, certainly no expectation that people should be following this channel if they wanted to know what's going on :) | 13:46 |
dhellmann | markmc: yeah, I don't expect anyone to follow conversations here | 13:49 |
*** dhellmann is now known as dhellmann_ | 13:51 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/105171 | 13:54 |
*** Guest69876 is now known as hartsocks | 13:55 | |
*** markmcclain has quit IRC | 13:58 | |
*** markmcclain has joined #openstack-oslo | 14:11 | |
*** dstanek_zzz is now known as dstanek | 14:13 | |
*** SridharG has quit IRC | 14:13 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests https://review.openstack.org/93424 | 14:16 |
*** jaosorior has joined #openstack-oslo | 14:17 | |
*** jgriffit1 has quit IRC | 14:19 | |
*** amotoki_ has joined #openstack-oslo | 14:21 | |
*** amotoki has quit IRC | 14:22 | |
*** tkelsey has quit IRC | 14:25 | |
*** ajo has joined #openstack-oslo | 14:25 | |
*** dhellmann_ is now known as dhellmann | 14:26 | |
openstackgerrit | Timur Sufiev proposed a change to openstack/oslo-incubator: Enhance versionutils.deprecated to work with classes https://review.openstack.org/104916 | 14:27 |
openstackgerrit | Victor Stinner proposed a change to openstack/oslo.messaging: Fix AMQPListener for polling with timeout https://review.openstack.org/104964 | 14:28 |
*** bswartz has joined #openstack-oslo | 14:33 | |
*** YorikSar has joined #openstack-oslo | 14:35 | |
YorikSar | bnemec: Hello. Around? | 14:35 |
*** bswartz has left #openstack-oslo | 14:35 | |
YorikSar | bnemec: Can you please take a look at https://review.openstack.org/94613/ | 14:37 |
YorikSar | bnemec: It already has all +1's and one +2 from Doug, so it would be great to land it so that specs for other projects could move on | 14:37 |
dhellmann | YorikSar: ttx's +1 counts as a +2 on that spec, too | 14:43 |
YorikSar | dhellmann: Then I need anyone to just approve it :) | 14:43 |
bnemec | YorikSar: I'm in a meeting right now, but I'll try to take a look today. | 14:47 |
dhellmann | YorikSar: I'll give bnemec and dims another day to review it and then approve it | 14:48 |
YorikSar | bnemec: Thanks :) | 14:48 |
YorikSar | dhellmann: Ok. Great. I'll have about 12 days till SAD to approve specs in other projects then :) | 14:50 |
*** joesavak has quit IRC | 14:53 | |
*** tsekiyama has joined #openstack-oslo | 14:56 | |
*** james_li has joined #openstack-oslo | 14:57 | |
*** joesavak has joined #openstack-oslo | 14:58 | |
*** zzzeek has joined #openstack-oslo | 15:01 | |
*** alexpilotti has quit IRC | 15:11 | |
openstackgerrit | A change was merged to openstack/oslo.i18n: Setup for translation https://review.openstack.org/98243 | 15:12 |
jecarey | dhellmann, Am I correct in assuming it isn't going to be required to move to the i18n library in Juno, so critical bugs should be fixed in oslo-incubator? | 15:12 |
dhellmann | jecarey: bugs should be fixed in the library first and then backported to the incubator | 15:13 |
jecarey | I think I need to backport https://review.openstack.org/#/c/97931/ | 15:13 |
*** alexpilotti has joined #openstack-oslo | 15:13 | |
dhellmann | jecarey: really? that shouldn't cause a problem for an application | 15:14 |
openstackgerrit | Mark McLoughlin proposed a change to openstack/oslo.messaging: zmq: provide sample_default for rpc_zmq_host opt https://review.openstack.org/105205 | 15:14 |
jecarey | The problem is that when a function is imported (_ in this case) it is cached so subsequent changes don't get picked up. | 15:14 |
dhellmann | jecarey: ah | 15:15 |
jecarey | dhellmann: Seeing a failure in a cinder test case when change to import _ | 15:15 |
ihrachyshka | zzzeek: hey, I have a question on sqlalchemy and sqlconnector | 15:15 |
zzzeek | yeah | 15:15 |
ihrachyshka | zzzeek: I get the following error when trying to spawn lots of parallel db workers and create ports inside of them: TimeoutError: QueuePool limit of size 10 overflow 20 reached, connection timed out, timeout 10 | 15:16 |
ihrachyshka | zzzeek: 10 workers are fine, but 1000 fails in that way | 15:16 |
ihrachyshka | zzzeek: ideas? | 15:16 |
zzzeek | um | 15:16 |
zzzeek | the pool has a limit of 30 connecvtions max | 15:16 |
zzzeek | and you’re using more workers than that | 15:17 |
zzzeek | perhaps you’d want to remove the limit off the connecvtion pool. but then again 1000 concurrent connecvtions is too much for the vast majority of database configurations that aren’t running on a giant mainframe | 15:17 |
ihrachyshka | zzzeek: yeah, but why not blocking them or I don't know what... it seems operations still succeed, though with delay | 15:18 |
zzzeek | the pool blocks on waiters | 15:18 |
zzzeek | and it has a timeout | 15:18 |
zzzeek | all configurable | 15:18 |
ihrachyshka | zzzeek: not that I consider it a real use case, I just test sqlconnector | 15:18 |
ihrachyshka | zzzeek: I think with mysqldb there is no way to get it | 15:18 |
zzzeek | ihrachyshka: take SQLA out of the equation for a minute | 15:18 |
zzzeek | ihrachyshka: you want to run 1000 workers, all talking to the DB | 15:18 |
ihrachyshka | zzzeek: because everything is blocked inside db | 15:18 |
zzzeek | you want 1000 DB connections at once? a DB conneciton takes up memory, do you know how much, and does the system have enough memory for 1000 of them ? | 15:19 |
ihrachyshka | zzzeek: well, I have 16 gigs here, so... :) | 15:19 |
zzzeek | did you configure your my.cnf for 1000 connections ? | 15:19 |
ihrachyshka | zzzeek: nope, I didn't. will it raise sqlalchemy limit? | 15:20 |
zzzeek | ihrachyshka: no. you need to read some docs :) | 15:21 |
ihrachyshka | zzzeek: ok... roger :) | 15:21 |
zzzeek | sqlalchemy’s docs, go to connection pooling | 15:22 |
ihrachyshka | tnx | 15:22 |
ihrachyshka | ok, one last thing: is timeout harmless? [expect the delay] | 15:23 |
openstackgerrit | James Carey proposed a change to openstack/oslo-incubator: Backport code for i18n to check lazy at runtime https://review.openstack.org/105207 | 15:24 |
openstackgerrit | Masaru Nomura proposed a change to openstack/oslo.vmware: Add API for introspecting OVAs https://review.openstack.org/105208 | 15:25 |
ihrachyshka | zzzeek: though it's not oslo specific, you may be interesting in that one: https://review.openstack.org/#/c/104905/ some interesting numbers for sqlconnector inside | 15:26 |
dhellmann | jecarey: why not go ahead and fully revert that change and remove the lazy argument to the factory? | 15:27 |
jecarey | dhellmann, I was trying to limit interface change. It would be cleaner to do that. | 15:28 |
dhellmann | jecarey: yeah, I'm worried that if we leave the argument it will be confusing | 15:29 |
jecarey | dhellmann: it also makes it a bit easier to move the the library. I'll go ahead and remove it. | 15:29 |
dhellmann | jecarey: ok | 15:30 |
*** markmcclain has quit IRC | 15:33 | |
openstackgerrit | Julien Danjou proposed a change to openstack/oslo.db: Raise DBReferenceError on foreign key violation https://review.openstack.org/94333 | 15:34 |
*** tmcpeak has joined #openstack-oslo | 15:37 | |
tmcpeak | jogo: you're back :) | 15:37 |
*** linkid has left #openstack-oslo | 15:42 | |
*** pblaho has quit IRC | 15:42 | |
*** ildikov_ has quit IRC | 15:47 | |
*** arnaud__ has joined #openstack-oslo | 15:52 | |
*** ihrachyshka has quit IRC | 15:56 | |
openstackgerrit | James Carey proposed a change to openstack/oslo-incubator: Backport code for i18n to check lazy at runtime https://review.openstack.org/105207 | 15:56 |
*** AAzza is now known as AAzza_afk | 16:00 | |
*** sigmavir1s24 has joined #openstack-oslo | 16:02 | |
*** sigmavirus24 has quit IRC | 16:02 | |
*** sigmavir1s24 is now known as sigmavirus24 | 16:03 | |
*** sigmavirus24 has quit IRC | 16:03 | |
*** sigmavirus24 has joined #openstack-oslo | 16:03 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Add a base test case for DB schema comparison https://review.openstack.org/93398 | 16:04 |
*** viktors is now known as viktors|afk | 16:09 | |
*** arnaud__ has quit IRC | 16:09 | |
viktors|afk | zzzeek: around? | 16:17 |
zzzeek | yeah | 16:17 |
*** viktors|afk is now known as viktors | 16:17 | |
viktors | zzzeek: recently, you provided a link, with gevent MySQL-python patching | 16:18 |
zzzeek | yes its on the wiki page i made | 16:18 |
viktors | oh | 16:18 |
*** i159 has quit IRC | 16:18 | |
zzzeek | https://wiki.openstack.org/wiki/OpenStack_and_SQLAlchemy#MySQLdb_.2B_eventlet_.3D_sad | 16:18 |
viktors | we with rpodolyaka1 tried to make a similar patching for eventlet. What do you think about it? | 16:19 |
zzzeek | did it work? great | 16:19 |
zzzeek | shrugs, all it has to do is work right ? | 16:19 |
viktors | https://github.com/vikt0rs/greenify/commit/5ebd5299a34d8e3711c0c903098744b8d727892a | 16:19 |
zzzeek | great :) i’m not familair with the internals of greening things | 16:20 |
viktors | me actually, too. It's based on ideas of psycopg2 patching :) | 16:21 |
viktors | yes, it's work, but we got no profit on the fast queries :( | 16:21 |
zzzeek | psycopg2 has a 1st class async API | 16:21 |
viktors | 1 min | 16:21 |
zzzeek | viktors: well you’d have to profile the blocking somehow | 16:21 |
viktors | see https://github.com/eventlet/eventlet/blob/master/eventlet/support/psycopg2_patcher.py | 16:22 |
zzzeek | oh right this | 16:22 |
zzzeek | yeah sure | 16:22 |
zzzeek | but, this is beacuse psyvcopg2 has an explicit async API | 16:22 |
zzzeek | the “extensions.set_wait_callback(eventlet_wait_callback)” changes the whole library’s behavior | 16:23 |
zzzeek | mysqldb doesnt have that | 16:23 |
zzzeek | http://initd.org/psycopg/docs/extensions.html#psycopg2.extensions.set_wait_callback | 16:23 |
viktors | zzzeek: yes, but we work with "greenif'ed" mysqldb | 16:25 |
zzzeek | viktors: OK | 16:26 |
zzzeek | viktors: doent that mean the C source code of mysqldb has to change ? | 16:26 |
viktors | zzzeek: mysql-connector-c was modified, similar to https://github.com/CMGS/greenify/blob/develop/README.rst | 16:27 |
zzzeek | viktors: OK | 16:28 |
zzzeek | viktors: well then the pieces are there, as far as “does it work” youd need to make a test that defintiely performs poorly due to blocking first, then see if the new approach fixes | 16:28 |
zzzeek | viktors: perhaps you are testing a case that doesnt have a blocking issue anyway | 16:28 |
*** pblaho has joined #openstack-oslo | 16:29 | |
zzzeek | viktors: there should be some way to detect when the interpreter is doing nothing, waiting on IO. an async system should never be waiting | 16:29 |
viktors | zzzeek: ! min, I'll show you test case | 16:31 |
*** sigmavirus24 has quit IRC | 16:33 | |
viktors | zzzeek: http://paste.openstack.org/show/85599/ | 16:33 |
viktors | zzzeek: so the issue is | 16:34 |
viktors | when you use ``select sleep(0)`` - it pass ~1.5 sec with monkey patching and 0.002 - without | 16:35 |
zzzeek | im not surprised it has quirks but thats nothing i have any specific insight on | 16:36 |
viktors | zzzeek: :) | 16:38 |
viktors | same thing ) | 16:38 |
zzzeek | here’s a test against PG that defintiely shows improvement: https://bitbucket.org/zzzeek/green_sqla/src/2732bb7ea9d06b9d4a61e8cd587a95148ce2599b/green_sqla/stress_sqla.py?at=default | 16:38 |
zzzeek | vs. plain threads | 16:39 |
viktors | zzzeek: thanks! Will try to test in a such way also | 16:41 |
*** ndipanov has quit IRC | 16:41 | |
*** SridharG has joined #openstack-oslo | 16:43 | |
openstackgerrit | Graham Hayes proposed a change to openstack/oslosphinx: Fixed link text colour for incubation variant https://review.openstack.org/105233 | 16:44 |
*** viktors is now known as viktors|afk | 16:44 | |
*** ildikov has joined #openstack-oslo | 16:44 | |
*** pblaho has quit IRC | 16:47 | |
*** tmcpeak has left #openstack-oslo | 16:48 | |
*** harlowja_away is now known as harlowja | 17:01 | |
*** james_li has quit IRC | 17:24 | |
*** arnaud__ has joined #openstack-oslo | 17:42 | |
*** arnaud__ has quit IRC | 17:46 | |
openstackgerrit | A change was merged to openstack/oslo-incubator: Backport code for i18n to check lazy at runtime https://review.openstack.org/105207 | 17:55 |
openstackgerrit | A change was merged to openstack/oslo-incubator: Ability to customize default_log_levels for each project https://review.openstack.org/94001 | 17:55 |
*** ajo has quit IRC | 17:57 | |
*** mriedem has left #openstack-oslo | 18:00 | |
*** amotoki_ has quit IRC | 18:02 | |
*** arnaud__ has joined #openstack-oslo | 18:06 | |
*** ajo has joined #openstack-oslo | 18:09 | |
*** james_li has joined #openstack-oslo | 18:13 | |
*** dims_ has quit IRC | 18:14 | |
*** mriedem has joined #openstack-oslo | 18:19 | |
*** dims has joined #openstack-oslo | 18:19 | |
dhellmann | dims: have a sec to talk about oslo.utils? | 18:40 |
dims | dhellmann, yep, am here | 18:40 |
dhellmann | I'm just looking at the github repo again, did you see my comments there? | 18:40 |
dhellmann | dims: it would probably be better to leave things like https://github.com/dims/oslo.utils/commit/6b9f2e9ec0a352e6c3ce862226ac05987d0823d7 for after the import | 18:41 |
dims | dhellmann, where can i see your comments? | 18:42 |
dhellmann | https://github.com/dims/oslo.utils/commit/01bca57862633ca43c8f1c05cef761acd85a7011 | 18:42 |
dims | right agree about flaper87's patch | 18:42 |
dhellmann | I think those can all be addressed after this repo is imported | 18:42 |
dhellmann | meh, I'll give you a +2 on that patch to go with our approval and we can just pull it in like it is now | 18:43 |
dims | ack | 18:43 |
dhellmann | dims: you saw jeblair's comments about how they are redoing the ACL stuff for https://review.openstack.org/#/c/101230/ right? | 18:44 |
dims | yes, i was waiting for that, not sure where it is now | 18:44 |
dims | i can ping jeblair | 18:45 |
dhellmann | dims: yeah, would be good to move this ahead | 18:45 |
*** praneshp has joined #openstack-oslo | 18:49 | |
*** pblaho has joined #openstack-oslo | 18:50 | |
dims | dhellmann, https://review.openstack.org/#/c/101230/ is ready again, had to patch up a few things | 19:02 |
dhellmann | dims: looking | 19:02 |
dhellmann | dims: I see a check job for python33, do we want a gate job, too? | 19:03 |
dhellmann | dims: 2 comments inline | 19:04 |
dims | dhellmann, i just copied the stuff from "openstack/oslo.version" | 19:05 |
dhellmann | dims: there's a "gate:" section for oslo.version, too | 19:06 |
dhellmann | dims: ironically, the name of the job doesn't really matter, what matters is which section it's put in | 19:06 |
dhellmann | dims: and it's possible someone cleaned up oslo.version since you copied it :-) | 19:07 |
*** SridharG has quit IRC | 19:07 | |
dims | lol, looks like it. reviewing :) | 19:07 |
dhellmann | dims: I run into this pretty often. These files see a lot of churn. | 19:07 |
dims | yep. updated. | 19:12 |
*** rpodolyaka1 has quit IRC | 19:24 | |
*** rpodolyaka has joined #openstack-oslo | 19:24 | |
boris-42 | markmc hi | 19:31 |
*** pblaho has quit IRC | 19:34 | |
*** erecio has joined #openstack-oslo | 19:49 | |
*** dstanek is now known as dstanek_zzz | 19:49 | |
*** dstanek_zzz is now known as dstanek | 19:59 | |
dhellmann | dims: do you know if the utils code all works under pypy? I would hesitate to add those jobs if we don't know they work | 20:06 |
*** james_li has quit IRC | 20:08 | |
*** james_li has joined #openstack-oslo | 20:10 | |
openstackgerrit | A change was merged to openstack/oslo.messaging: Fix AMQPListener for polling with timeout https://review.openstack.org/104964 | 20:16 |
openstackgerrit | A change was merged to openstack/oslo.config: generator: remove the default sanitizer feature https://review.openstack.org/104227 | 20:16 |
*** yamahata__ has joined #openstack-oslo | 20:16 | |
*** yamahata_ has quit IRC | 20:17 | |
*** pcm_ has quit IRC | 20:21 | |
*** pcm_ has joined #openstack-oslo | 20:22 | |
*** sigmavirus24 has joined #openstack-oslo | 20:24 | |
*** jecarey has quit IRC | 20:25 | |
*** markmc has quit IRC | 20:45 | |
openstackgerrit | Arnaud Legendre proposed a change to openstack/oslo.vmware: Share get_values_from_object_properties https://review.openstack.org/105274 | 20:46 |
openstackgerrit | A change was merged to openstack-dev/oslo-cookiecutter: Use spaces instead of tabs for indentation https://review.openstack.org/103872 | 20:54 |
*** markmcclain has joined #openstack-oslo | 21:00 | |
*** harlowja is now known as harlowja_away | 21:02 | |
*** GheRiver1 is now known as GheRivero | 21:05 | |
*** jecarey has joined #openstack-oslo | 21:14 | |
*** flaper87 is now known as flaper87|afk | 21:17 | |
*** joesavak has quit IRC | 21:32 | |
*** zzzeek has quit IRC | 21:33 | |
*** zzzeek has joined #openstack-oslo | 21:46 | |
*** dstanek is now known as dstanek_zzz | 21:50 | |
*** rmcall has joined #openstack-oslo | 21:54 | |
*** mriedem has quit IRC | 21:54 | |
*** alexpilotti has quit IRC | 22:03 | |
*** alexpilotti has joined #openstack-oslo | 22:04 | |
*** harlowja_away is now known as harlowja | 22:04 | |
*** sigmavirus24 has quit IRC | 22:07 | |
*** alexpilotti has quit IRC | 22:09 | |
*** markmcclain has quit IRC | 22:12 | |
*** ildikov has quit IRC | 22:15 | |
*** bknudson has quit IRC | 22:16 | |
*** asalkeld has joined #openstack-oslo | 22:29 | |
*** james_li has quit IRC | 22:42 | |
*** mriedem has joined #openstack-oslo | 22:48 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add a redis backed job/jobboard (WIP) https://review.openstack.org/105298 | 22:50 |
*** asalkeld has quit IRC | 23:03 | |
*** dims has quit IRC | 23:37 | |
*** oomichi has joined #openstack-oslo | 23:38 | |
*** jaosorior has quit IRC | 23:42 | |
openstackgerrit | pritesh proposed a change to openstack-dev/hacking: Add hacking check for editor configuration in source files https://review.openstack.org/101969 | 23:48 |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add a redis backed job/jobboard https://review.openstack.org/105298 | 23:48 |
openstackgerrit | Joshua Harlow proposed a change to openstack/oslo-specs: Add a redis backed job/jobboard https://review.openstack.org/105298 | 23:49 |
*** dims has joined #openstack-oslo | 23:57 | |
openstackgerrit | Michael Bayer proposed a change to openstack/oslo-specs: Add use-events-for-error-wrapping https://review.openstack.org/105306 | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!