*** Guest74389 has quit IRC | 00:11 | |
*** salv-orlando has joined #openstack-oslo | 00:14 | |
*** salv-orlando has quit IRC | 00:36 | |
*** dimtruck is now known as zz_dimtruck | 01:00 | |
*** mtanino has quit IRC | 01:00 | |
*** manjeets has left #openstack-oslo | 01:11 | |
*** ozamiatin__ has quit IRC | 01:13 | |
*** salv-orlando has joined #openstack-oslo | 01:18 | |
*** zz_dimtruck is now known as dimtruck | 01:28 | |
*** yamahata has quit IRC | 01:34 | |
*** salv-orlando has quit IRC | 01:36 | |
-openstackstatus- NOTICE: Gerrit is about to be restarted to help with page timeouts | 01:47 | |
*** salv-orlando has joined #openstack-oslo | 02:12 | |
*** salv-orlando has quit IRC | 02:32 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/315748 | 02:39 |
---|---|---|
*** dimtruck is now known as zz_dimtruck | 02:58 | |
*** salv-orlando has joined #openstack-oslo | 03:05 | |
*** zz_dimtruck is now known as dimtruck | 03:14 | |
*** dimtruck is now known as zz_dimtruck | 03:16 | |
*** zz_dimtruck is now known as dimtruck | 03:18 | |
*** links has joined #openstack-oslo | 03:26 | |
*** salv-orlando has quit IRC | 03:28 | |
*** jkeen has joined #openstack-oslo | 03:36 | |
*** jkeen has quit IRC | 03:56 | |
*** jamespage has quit IRC | 04:00 | |
*** jamespage has joined #openstack-oslo | 04:01 | |
*** salv-orlando has joined #openstack-oslo | 04:05 | |
*** salv-orlando has quit IRC | 04:25 | |
*** amotoki has quit IRC | 04:26 | |
*** amotoki has joined #openstack-oslo | 04:26 | |
*** dimtruck is now known as zz_dimtruck | 04:29 | |
*** yamamoto has quit IRC | 04:30 | |
*** salv-orlando has joined #openstack-oslo | 04:58 | |
*** SlickN1k has joined #openstack-oslo | 05:06 | |
*** sabari- has joined #openstack-oslo | 05:06 | |
*** mikal_ has joined #openstack-oslo | 05:08 | |
*** haypo_ has joined #openstack-oslo | 05:08 | |
*** davidlenwell has quit IRC | 05:09 | |
*** fnordahl has quit IRC | 05:12 | |
*** mikal has quit IRC | 05:12 | |
*** SlickNik has quit IRC | 05:12 | |
*** sabari has quit IRC | 05:12 | |
*** Kennan has quit IRC | 05:12 | |
*** links has quit IRC | 05:12 | |
*** haypo has quit IRC | 05:12 | |
*** ericksonsantos has quit IRC | 05:12 | |
*** sabari- is now known as sabari | 05:12 | |
*** Kennan has joined #openstack-oslo | 05:13 | |
*** SlickN1k is now known as SlickNik | 05:13 | |
*** ericksonsantos has joined #openstack-oslo | 05:13 | |
*** yamamoto_ has joined #openstack-oslo | 05:13 | |
*** links has joined #openstack-oslo | 05:13 | |
*** fnordahl has joined #openstack-oslo | 05:15 | |
*** lpetrut has joined #openstack-oslo | 05:17 | |
*** davidlenwell has joined #openstack-oslo | 05:17 | |
*** salv-orlando has quit IRC | 05:18 | |
*** salv-orlando has joined #openstack-oslo | 05:18 | |
*** rcernin has joined #openstack-oslo | 05:47 | |
*** ozamiatin has joined #openstack-oslo | 05:51 | |
*** ozamiatin has quit IRC | 05:52 | |
*** david-lyle has quit IRC | 06:05 | |
*** ozamiatin has joined #openstack-oslo | 06:16 | |
*** salv-orlando has quit IRC | 06:30 | |
*** lpetrut has quit IRC | 06:41 | |
*** rcernin has quit IRC | 06:47 | |
*** nkrinner_afk is now known as nkrinner | 06:47 | |
*** dukhlov has joined #openstack-oslo | 06:48 | |
*** ozialien10 has quit IRC | 06:50 | |
*** rcernin has joined #openstack-oslo | 06:51 | |
*** lpetrut has joined #openstack-oslo | 06:52 | |
openstackgerrit | Merged openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/317496 | 06:55 |
*** dukhlov has quit IRC | 06:58 | |
*** dukhlov has joined #openstack-oslo | 07:00 | |
*** ozamiatin has quit IRC | 07:07 | |
*** salv-orlando has joined #openstack-oslo | 07:11 | |
*** mixos has quit IRC | 07:11 | |
*** ozamiatin has joined #openstack-oslo | 07:11 | |
*** lpetrut has quit IRC | 07:23 | |
openstackgerrit | Oleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Don't skip non-direct message types https://review.openstack.org/316800 | 07:26 |
*** gcb has quit IRC | 07:27 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: zeromq: Deprecates host, port options https://review.openstack.org/317878 | 07:28 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: deprecate usage of transport aliases https://review.openstack.org/180434 | 07:28 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Remove deprecated localcontext https://review.openstack.org/317303 | 07:28 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: rabbit: Deprecates host, port, auth options https://review.openstack.org/317285 | 07:28 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Make TransportURL.parse aware of transport_url https://review.openstack.org/316920 | 07:28 |
*** salv-orlando has quit IRC | 07:31 | |
*** lpetrut has joined #openstack-oslo | 07:35 | |
*** salv-orlando has joined #openstack-oslo | 07:35 | |
*** ihrachys has joined #openstack-oslo | 07:38 | |
*** dukhlov has quit IRC | 07:38 | |
*** gcb has joined #openstack-oslo | 07:41 | |
*** salv-orl_ has joined #openstack-oslo | 07:48 | |
*** jpena|off is now known as jpena | 07:48 | |
*** salv-orlando has quit IRC | 07:52 | |
*** shardy has joined #openstack-oslo | 07:53 | |
*** gcb has quit IRC | 07:54 | |
*** dukhlov has joined #openstack-oslo | 07:57 | |
*** david-lyle has joined #openstack-oslo | 07:57 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-oslo | 08:01 | |
*** gcb has joined #openstack-oslo | 08:11 | |
*** salv-orl_ has quit IRC | 08:27 | |
*** dukhlov has quit IRC | 08:29 | |
*** dukhlov has joined #openstack-oslo | 08:31 | |
*** lucas-dinner is now known as lucasagomes | 08:32 | |
*** dukhlov has quit IRC | 08:34 | |
*** pblaho has joined #openstack-oslo | 08:36 | |
*** yassine__ has joined #openstack-oslo | 08:37 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Fix bug with version_cap and target.version in RPCClient https://review.openstack.org/309956 | 08:42 |
*** vipuls has joined #openstack-oslo | 08:42 | |
*** pkholkin_ has joined #openstack-oslo | 08:42 | |
*** wendar_ has joined #openstack-oslo | 08:43 | |
*** jraim_ has joined #openstack-oslo | 08:44 | |
*** bapalm_ has joined #openstack-oslo | 08:44 | |
*** shardy_ has joined #openstack-oslo | 08:44 | |
*** hockeynut_afk has joined #openstack-oslo | 08:45 | |
*** lucasagomes_ has joined #openstack-oslo | 08:45 | |
*** haypo_ is now known as haypo | 08:45 | |
*** breton_ has joined #openstack-oslo | 08:45 | |
*** sigmavirus24_awa has quit IRC | 08:45 | |
*** hockeynut has quit IRC | 08:45 | |
*** pmalik has quit IRC | 08:45 | |
*** timsim has quit IRC | 08:45 | |
*** ogelbukh has quit IRC | 08:45 | |
*** d34dh0r53 has quit IRC | 08:45 | |
*** jraim has quit IRC | 08:45 | |
*** shardy has quit IRC | 08:45 | |
*** tpeoples has quit IRC | 08:45 | |
*** zz_dimtruck has quit IRC | 08:45 | |
*** vipul- has quit IRC | 08:45 | |
*** bapalm has quit IRC | 08:45 | |
*** melwitt has quit IRC | 08:45 | |
*** cloudnull has quit IRC | 08:45 | |
*** odyssey4me has quit IRC | 08:45 | |
*** pkholkin has quit IRC | 08:45 | |
*** rbradfor has quit IRC | 08:45 | |
*** lucasagomes has quit IRC | 08:45 | |
*** breton has quit IRC | 08:45 | |
*** Nakato has quit IRC | 08:45 | |
*** jamielennox has quit IRC | 08:45 | |
*** wendar has quit IRC | 08:45 | |
*** melwitt has joined #openstack-oslo | 08:45 | |
*** Nakato has joined #openstack-oslo | 08:45 | |
*** pmalik has joined #openstack-oslo | 08:45 | |
*** d34dh0r53 has joined #openstack-oslo | 08:45 | |
*** dimtruckl has joined #openstack-oslo | 08:45 | |
*** sigmavirus24_awa has joined #openstack-oslo | 08:45 | |
*** sigmavirus24_awa has quit IRC | 08:45 | |
*** sigmavirus24_awa has joined #openstack-oslo | 08:45 | |
*** haypo has quit IRC | 08:45 | |
*** haypo has joined #openstack-oslo | 08:45 | |
*** melwitt is now known as Guest37264 | 08:46 | |
*** dimtruckl is now known as dimtruck | 08:46 | |
*** timsim has joined #openstack-oslo | 08:46 | |
*** odyssey4me has joined #openstack-oslo | 08:46 | |
*** pkholkin_ is now known as pkholkin | 08:46 | |
*** rbradfor has joined #openstack-oslo | 08:46 | |
*** cloudkiller has joined #openstack-oslo | 08:47 | |
*** jraim_ is now known as jraim | 08:47 | |
*** ogelbukh has joined #openstack-oslo | 08:47 | |
*** dougwig has quit IRC | 08:48 | |
*** lucasagomes_ is now known as lucasagomes | 08:49 | |
*** rpodolyaka has quit IRC | 08:49 | |
*** tpeoples has joined #openstack-oslo | 08:52 | |
*** jamielennox has joined #openstack-oslo | 08:55 | |
*** dtardivel has joined #openstack-oslo | 08:55 | |
*** rpodolyaka has joined #openstack-oslo | 08:57 | |
*** dougwig has joined #openstack-oslo | 08:59 | |
*** salv-orlando has joined #openstack-oslo | 09:09 | |
*** lpetrut has quit IRC | 09:13 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Add configurable serialization to pika https://review.openstack.org/304172 | 09:16 |
*** zqfan has joined #openstack-oslo | 09:36 | |
openstackgerrit | Merged openstack/oslo.messaging: rabbit: Deprecates host, port, auth options https://review.openstack.org/317285 | 09:39 |
openstackgerrit | Merged openstack/oslo.messaging: Remove deprecated localcontext https://review.openstack.org/317303 | 09:41 |
*** cloudkiller is now known as cloudnull | 09:46 | |
*** sdague has joined #openstack-oslo | 10:01 | |
*** ihrachys has quit IRC | 10:05 | |
openstackgerrit | Merged openstack/oslo.messaging: Make TransportURL.parse aware of transport_url https://review.openstack.org/316920 | 10:08 |
*** salv-orlando has quit IRC | 10:13 | |
*** dukhlov has joined #openstack-oslo | 10:18 | |
*** salv-orlando has joined #openstack-oslo | 10:20 | |
*** cdent has joined #openstack-oslo | 10:23 | |
*** cdent has left #openstack-oslo | 10:35 | |
*** dimtruck is now known as zz_dimtruck | 10:36 | |
*** lpetrut has joined #openstack-oslo | 10:57 | |
openstackgerrit | Merged openstack/oslo.db: api: always enable retry_on_request https://review.openstack.org/317299 | 11:10 |
*** tedross has joined #openstack-oslo | 11:14 | |
*** tedross has left #openstack-oslo | 11:15 | |
openstackgerrit | Merged openstack/oslo.messaging: zeromq: Deprecates host, port options https://review.openstack.org/317878 | 11:26 |
openstackgerrit | Merged openstack/oslo.messaging: Fix bug with version_cap and target.version in RPCClient https://review.openstack.org/309956 | 11:26 |
*** dukhlov has quit IRC | 11:28 | |
*** dukhlov has joined #openstack-oslo | 11:34 | |
*** yamamoto_ has quit IRC | 11:38 | |
*** lucasagomes is now known as lucas-hungry | 11:38 | |
*** jpena is now known as jpena|lunch | 11:46 | |
*** sigmavirus24_awa is now known as sigmavirus24 | 12:12 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 12:20 |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 12:22 |
*** jdandrea_ has quit IRC | 12:22 | |
*** jdandrea has joined #openstack-oslo | 12:22 | |
*** gordc has joined #openstack-oslo | 12:23 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 12:28 |
*** kgiusti has joined #openstack-oslo | 12:28 | |
openstackgerrit | Alexis Lee proposed openstack/oslo.log: Reload log_config_append config on SIGHUP https://review.openstack.org/293561 | 12:28 |
*** links has quit IRC | 12:31 | |
*** lucas-hungry is now known as lucasagomes | 12:32 | |
*** zz_dimtruck is now known as dimtruck | 12:33 | |
*** links has joined #openstack-oslo | 12:35 | |
*** ihrachys has joined #openstack-oslo | 12:40 | |
*** salv-orlando has quit IRC | 12:42 | |
*** yamamoto has joined #openstack-oslo | 12:58 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 13:01 |
*** jpena|lunch is now known as jpena | 13:02 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 13:03 |
*** edmondsw has joined #openstack-oslo | 13:04 | |
*** rlrossit has joined #openstack-oslo | 13:11 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 13:13 |
*** links has quit IRC | 13:16 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 13:23 |
*** tonytan4ever has joined #openstack-oslo | 13:31 | |
*** dimtruck is now known as zz_dimtruck | 13:42 | |
*** salv-orlando has joined #openstack-oslo | 13:43 | |
*** mtanino has joined #openstack-oslo | 13:43 | |
*** lpetrut has quit IRC | 13:46 | |
openstackgerrit | Kirill Bespalov proposed openstack/oslo.messaging: Fix consuming from missing queues https://review.openstack.org/315700 | 13:48 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/315748 | 13:56 |
*** pradk has joined #openstack-oslo | 14:00 | |
*** yamahata has joined #openstack-oslo | 14:03 | |
*** lpetrut has joined #openstack-oslo | 14:06 | |
*** salv-orlando has quit IRC | 14:09 | |
*** links has joined #openstack-oslo | 14:10 | |
*** shardy_ is now known as shardy | 14:14 | |
*** rbrndt has joined #openstack-oslo | 14:15 | |
*** yamahata has quit IRC | 14:18 | |
openstackgerrit | Kirill Bespalov proposed openstack/oslo.messaging: Fix consuming from missing queues https://review.openstack.org/315700 | 14:27 |
openstackgerrit | Julien Danjou proposed openstack/oslo.db: api: use sane default in wrap_db_retry() https://review.openstack.org/318135 | 14:28 |
*** salv-orlando has joined #openstack-oslo | 14:35 | |
*** dukhlov has quit IRC | 14:35 | |
*** dukhlov has joined #openstack-oslo | 14:37 | |
*** links has quit IRC | 14:39 | |
*** takedakn has joined #openstack-oslo | 14:48 | |
*** takedakn has quit IRC | 14:56 | |
*** hockeynut_afk is now known as hockeynut | 14:56 | |
*** tonytan4ever has quit IRC | 15:01 | |
*** salv-orlando has quit IRC | 15:05 | |
*** mixos has joined #openstack-oslo | 15:06 | |
kevinbenton | harlowja: hey! | 15:09 |
*** dukhlov has quit IRC | 15:12 | |
*** rcernin has quit IRC | 15:24 | |
*** itisha has joined #openstack-oslo | 15:26 | |
*** salv-orlando has joined #openstack-oslo | 15:31 | |
*** salv-orlando has quit IRC | 15:41 | |
*** salv-orlando has joined #openstack-oslo | 15:42 | |
*** dukhlov has joined #openstack-oslo | 15:43 | |
*** ozamiatin has quit IRC | 15:43 | |
*** nkrinner is now known as nkrinner_afk | 15:43 | |
*** zz_dimtruck is now known as dimtruck | 15:51 | |
openstackgerrit | Ildar Svetlov proposed openstack/oslo.messaging: [kafka] Add several bootstrap servers support https://review.openstack.org/307668 | 15:52 |
*** tonytan4ever has joined #openstack-oslo | 15:53 | |
*** Rockyg has joined #openstack-oslo | 15:55 | |
*** dukhlov has quit IRC | 16:15 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception https://review.openstack.org/316069 | 16:21 |
openstackgerrit | Merged openstack/oslo.messaging: Reorganize the AMQP 1.0 driver source files https://review.openstack.org/314218 | 16:21 |
*** dimtruck is now known as zz_dimtruck | 16:27 | |
*** zz_dimtruck is now known as dimtruck | 16:29 | |
openstackgerrit | Alexis Lee proposed openstack/oslo.log: log: Spooling logger and manager https://review.openstack.org/316162 | 16:36 |
openstackgerrit | Alexis Lee proposed openstack/oslo.log: log: Extra spool overflow strategies https://review.openstack.org/318212 | 16:36 |
harlowja | kevinbenton yo yo | 16:38 |
harlowja | lxsli yt, how do u think https://review.openstack.org/#/c/308399/ would be implemented ontop of the refactored stuff | 16:39 |
patchbot | harlowja: patch 308399 - oslo.utils - specs_matcher: Allow passing custom op_methods | 16:39 |
lxsli | harlowja: wow that's pretty crazy | 16:43 |
harlowja | lol | 16:43 |
harlowja | ya, its like dynamic DSL :-/ | 16:44 |
harlowja | without a grammar, lol | 16:44 |
*** dukhlov has joined #openstack-oslo | 16:44 | |
lxsli | If there's really only going to be one alternative, pass in a flag that alters the BNF | 16:44 |
lxsli | Otherwise... maybe just don't? | 16:44 |
harlowja | :) | 16:44 |
*** ri0 has joined #openstack-oslo | 16:44 | |
harlowja | ya, that was my thinking, if u want to do this, probably u shouldn't, or u should make a real grammar so people know what the new things u are doing are | 16:45 |
harlowja | jroll yt | 16:45 |
lxsli | yeah with the patch as-is, you could do something crazy like make "=" mean ">=" | 16:45 |
lxsli | btw harlowja I'd love your thoughts on my log spooling work | 16:48 |
harlowja | def | 16:48 |
harlowja | lxsli saw that, it seems ok, idk about the exception thingy, is that normal? idk | 16:48 |
harlowja | does like the syslog stuff blow-up if it can't work? | 16:48 |
harlowja | or just it does drop things? | 16:48 |
lxsli | I've changed the default strategy to just flush | 16:48 |
lxsli | and put up a new patch with a ton of alternative strategies | 16:49 |
harlowja | kk | 16:49 |
harlowja | lxsli was there any pypi solutions that did spooling to, i would've thought there would have been | 16:49 |
harlowja | other thoughts, does systemd do all this spolling itself, idk | 16:50 |
lxsli | harlowja: there's bbrecorder but it doesn't really fit our needs imo | 16:51 |
*** Guest37264 is now known as melwitt | 16:51 | |
harlowja | kk | 16:51 |
lxsli | logrecord sinks don't know when a request succeeds so they can't drop those logs | 16:52 |
harlowja | gotcha | 16:53 |
*** dukhlov has quit IRC | 16:59 | |
kgiusti | hallelujah | 17:01 |
harlowja | lol | 17:01 |
kevinbenton | harlowja: yo | 17:04 |
kevinbenton | harlowja: so select.poll() seems to block even with eventlet | 17:04 |
harlowja | hmmm | 17:05 |
harlowja | don't do that :-P | 17:05 |
harlowja | lol | 17:05 |
kevinbenton | harlowja: if you turn on watch-file in oslo logging, it uses pyinotify, which does that | 17:07 |
kevinbenton | harlowja: it's a cool feature where the whole agent freezes for 500 milliseconds like every second | 17:07 |
harlowja | lol | 17:07 |
harlowja | i wonder if something not monkey patched somewhere? | 17:07 |
kevinbenton | harlowja: well i was looking around and it sounds like directly calling poll isn't monkey patched by eventlet? | 17:07 |
kevinbenton | harlowja: does that sound familiar? | 17:08 |
*** tonytan4ever has quit IRC | 17:08 | |
harlowja | hmmm, that'd seem odd | 17:08 |
*** lpetrut has quit IRC | 17:08 | |
kevinbenton | harlowja: all of the other monkey patched stuff works okay | 17:10 |
harlowja | hmmmm | 17:10 |
kevinbenton | harlowja: like oslo messaging stuff | 17:10 |
kevinbenton | harlowja: and i ensured that the monkey_patch was called way before pyinotify was even imported | 17:10 |
harlowja | ya, i wonder | 17:11 |
harlowja | https://github.com/eventlet/eventlet/pull/317 is interesting | 17:11 |
harlowja | haypo yt | 17:11 |
kevinbenton | harlowja: so i was wondering if we should at least change the poll timeout in oslo log to like 5ms for a quick backport | 17:12 |
harlowja | seems reasonable | 17:13 |
harlowja | or switch to not poll | 17:13 |
kevinbenton | pyinotify does that | 17:13 |
harlowja | k | 17:13 |
kevinbenton | i don't think we have an option there | 17:14 |
kevinbenton | harlowja: submitting a patch now and bug report | 17:14 |
harlowja | k | 17:14 |
harlowja | https://bitbucket.org/eventlet/eventlet/issues/167/monkey-patch-on-select-leaves-selectpoll i guess u saw also kevinbenton ? | 17:16 |
*** Rockyg has quit IRC | 17:17 | |
kevinbenton | harlowja: ah, interesting | 17:17 |
kevinbenton | harlowja: so now we will get an attribute error after that patch, right? | 17:17 |
harlowja | i thinks so | 17:18 |
*** shardy has quit IRC | 17:19 | |
*** jpena is now known as jpena|off | 17:22 | |
openstackgerrit | Kevin Benton proposed openstack/oslo.log: Reduce READ_FREQ and TIMEOUT for watch-file https://review.openstack.org/318256 | 17:39 |
kevinbenton | harlowja: ^^ | 17:39 |
*** lpetrut has joined #openstack-oslo | 17:40 | |
openstackgerrit | Dougal Matthews proposed openstack/mox3: Correct spelling of occurrences https://review.openstack.org/318257 | 17:40 |
*** rlrossit has quit IRC | 17:43 | |
*** spzala has joined #openstack-oslo | 17:43 | |
stevemar | dims_: harlowja spzala o/ | 17:43 |
stevemar | dims_: harlowja question about 'lib-forward-testing' | 17:44 |
spzala | stevemar: o/ | 17:44 |
spzala | dims_: o/ | 17:44 |
*** lucasagomes is now known as lucas-dinner | 17:44 | |
*** lpetrut has quit IRC | 17:44 | |
stevemar | does lib-forward-testing test dsvm against the master branch of oslo, instead of the latest release? | 17:44 |
stevemar | master branch of an oslo project* | 17:45 |
dims_ | stevemar : yes | 17:46 |
harlowja | (sorry on call) | 17:47 |
stevemar | harlowja: np | 17:47 |
stevemar | dims_: okay, but it just installs it and runs the dsvm job right? and tempest tests? | 17:47 |
dims_ | hang on let me check something | 17:48 |
*** rlrossit has joined #openstack-oslo | 17:49 | |
stevemar | dims_: spzala based on what i see here: https://github.com/openstack-infra/project-config/blob/35b38e4f8bf40999799fce1d2c3f92614c33358f/jenkins/jobs/devstack-gate.yaml#L128-L141 | 17:49 |
stevemar | i think it just sets up dvsm and runs tempest :P | 17:50 |
*** dimtruck is now known as zz_dimtruck | 17:50 | |
dims_ | stevemar : http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n399 | 17:50 |
dims_ | those are the 2 job names that will show up | 17:50 |
stevemar | dims_: yes, line 399 refers to the link i wrote 2 lines up | 17:51 |
dims_ | sorry, same name in gate and check | 17:51 |
dims_ | stevemar : yep | 17:51 |
stevemar | dims_: basically spzala has a similar problem, his heat-translator project depends on tosca-parser (these are all in the openstack namespace) | 17:52 |
stevemar | dims_: but i think both projects just run py27 tests | 17:52 |
*** zz_dimtruck is now known as dimtruck | 17:52 | |
spzala | stevemar: thanks, yes. dims_: going forward I want to support backward compatibility with our pypi releases | 17:53 |
stevemar | dims_: spzala wants to make sure the master branch of tosca-parser doesn't break heat-translator when released | 17:53 |
spzala | stevemar: by27 and py34 tests | 17:53 |
stevemar | not sure if theres a way to set up the py27 tests to install from git instead of latest release | 17:53 |
spzala | stevemar: dims_: ideally both, that master of tosca-parser doesn't break master of heat-translator and latest pypi of heat-translator | 17:53 |
stevemar | spzala: if it doesn't break the master of heat-translator, it shouldn't break the latest release of heat-translator :P | 17:54 |
spzala | stevemar: dims_: I believe it's safer that way to test against both | 17:55 |
spzala | stevemar: :-) well, what if any change in master of heat-translator is already broke previous latest release of heat-translator | 17:55 |
spzala | stevemar: I think that's totally possible | 17:56 |
stevemar | spzala: so i know you can create your own job and install it from git, but i'm wondering if there is something that's already available | 17:56 |
spzala | s/has/is | 17:56 |
openstackgerrit | Ken Giusti proposed openstack/oslo.messaging: Add the proper branch back to .gitreview https://review.openstack.org/318265 | 17:56 |
spzala | stevemar: sure, that definitely makes sense to use if something is already available | 17:57 |
spzala | steve_mar: does it make sense the possible scenario I mentioned that heat-translator master (any project that matter) can run into backward compatibility issue with a latest release of it's own so it it's good to test against master and latest release? | 17:59 |
spzala | i.e. for tosca-parser to test against master and latest release of heat-translator | 17:59 |
*** dtardivel has quit IRC | 18:00 | |
*** yamahata has joined #openstack-oslo | 18:00 | |
jroll | harlowja: here now | 18:02 |
stevemar | spzala: actually, i think you want something like "periodic-jobs-with-oslo-master" | 18:04 |
*** wendar_ is now known as wendar | 18:04 | |
*** tonytan4ever has joined #openstack-oslo | 18:18 | |
*** ihrachys has quit IRC | 18:18 | |
*** dimtruck is now known as zz_dimtruck | 18:19 | |
harlowja | jroll so lxsli and i were wondering about https://review.openstack.org/#/c/308399/ and whether that could just be simpler, if its just about 'integer-only' methods, probably easier to just enable that with https://review.openstack.org/#/c/313699/ (with a nice formalized spec) | 18:20 |
patchbot | harlowja: patch 308399 - oslo.utils - specs_matcher: Allow passing custom op_methods | 18:20 |
patchbot | harlowja: patch 313699 - oslo.utils - Use an actual well defined parser for spec matching | 18:20 |
harlowja | as like an option to match integer_only=True or something | 18:21 |
harlowja | vs full on change all the operations | 18:21 |
jroll | harlowja: that's what it's about today, not sure about the future | 18:23 |
jroll | harlowja: though I'm thinking more about this, it's unclear to me if we actually want this | 18:23 |
jroll | we should be doing this in the db for what we're using it for | 18:24 |
harlowja | jroll likely via sql and such? | 18:25 |
jroll | harlowja: yeah | 18:25 |
openstackgerrit | Ken Giusti proposed openstack/oslo.messaging: Refactor link management to support link recovery https://review.openstack.org/318275 | 18:26 |
harlowja | jroll fair enough | 18:26 |
*** isq has joined #openstack-oslo | 18:27 | |
*** dukhlov has joined #openstack-oslo | 18:28 | |
harlowja | jroll so whole spec stuff and madness there might not be needed, that correct | 18:28 |
harlowja | (just double-checking, i'm ok with that honestly) :-P | 18:28 |
jroll | harlowja: more like, we may or may not even need to move this code | 18:28 |
harlowja | right | 18:30 |
kgiusti | harlowja: hey - I'm having some problems using that feature branch (feature/amqp-dispatch-router) | 18:34 |
harlowja | kgiusti uhohs? | 18:34 |
kgiusti | harlowja: I've restored the .gitreview to include the branch= directive. | 18:34 |
kgiusti | harlowja: but whenever I do a git review, the review is against master, not the branch | 18:35 |
harlowja | hmmm | 18:35 |
kgiusti | harlowja: I created by local branch simply by checking out a branch that tracks remote/feature/amqp-dispatch-router | 18:35 |
harlowja | do u have a gerrit remote that might be off | 18:35 |
harlowja | i've seen that kind happen before | 18:36 |
harlowja | i'm wondering if your git config might have something weird in it | 18:36 |
harlowja | i've seen that before, unsure if this might be the cause | 18:37 |
harlowja | cat .git/config | 18:37 |
harlowja | might be a gerrit or branch in there that's not pointing at the right place? | 18:37 |
kgiusti | harlowja: git/config: http://paste.fedoraproject.org/368121/35966561/ | 18:38 |
kgiusti | harlowja: I'm trying to git review from the 'feature/amqp-dispatch-router' branch | 18:39 |
kgiusti | harlowja: the .gitreview file: https://paste.fedoraproject.org/368118/96608146/ | 18:39 |
*** ri0 has quit IRC | 18:39 | |
*** ri0 has joined #openstack-oslo | 18:40 | |
harlowja | ya, hmmm it seems right | 18:40 |
harlowja | want to jump on the infra channel, they might have someone who understands git-review better than myself for this | 18:41 |
harlowja | seems odd | 18:41 |
kgiusti | harlowja: kk - will do | 18:41 |
*** ri0 has quit IRC | 18:42 | |
*** dukhlov has quit IRC | 18:44 | |
*** lpetrut has joined #openstack-oslo | 18:47 | |
harlowja | kevinbenton https://ondergetekende.nl/using-pyinotify-with-eventlet.html lol | 18:49 |
harlowja | seems like someone else worked around this also | 18:49 |
kgiusti | harlowja: fixed - turns out the "branch" keyword in the .gitreview file is wrong - needs to be "defaultbranch" | 18:50 |
harlowja | and https://github.com/seb-m/pyinotify/pull/117 kevinbenton | 18:50 |
harlowja | kgiusti ahhh, good to know | 18:50 |
harlowja | kevinbenton i guess 117 there would also fix the issue | 18:51 |
harlowja | if it ever merges/gets released | 18:51 |
kevinbenton | harlowja: yeah | 18:54 |
kevinbenton | harlowja: another option would be to override the read_events method | 18:54 |
kevinbenton | harlowja: and write it not to use poll | 18:54 |
harlowja | ya, seems like we can go there if we have to :-/ | 18:54 |
*** amotoki has quit IRC | 18:56 | |
openstackgerrit | Dougal Matthews proposed openstack/mox3: Correct spelling of occurrences https://review.openstack.org/318257 | 19:41 |
*** yamamoto has quit IRC | 19:55 | |
*** salv-orlando has quit IRC | 19:59 | |
*** zz_dimtruck is now known as dimtruck | 20:06 | |
*** tonytan4ever has quit IRC | 20:08 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/315748 | 20:26 |
adam_g | anyone around familiar with the big refactor that happened here? https://review.openstack.org/#/c/300727 seems to have broken astara's ability to consume neutron notifications with the recent 5.x release | 20:32 |
adam_g | (oslo.messaging) | 20:32 |
*** ozamiatin has joined #openstack-oslo | 20:37 | |
*** lpetrut has quit IRC | 20:40 | |
*** lpetrut has joined #openstack-oslo | 20:41 | |
*** lpetrut has quit IRC | 20:46 | |
*** kgiusti has left #openstack-oslo | 20:51 | |
*** yamamoto has joined #openstack-oslo | 20:56 | |
*** yamamoto has quit IRC | 21:03 | |
harlowja | adam_g i don't have much expertise in that, but how did it break astra's ability ? | 21:04 |
harlowja | since afaik that was mostly internal refactoring | 21:04 |
harlowja | any more details? | 21:04 |
adam_g | harlowja: still digging in trying to figure that out | 21:04 |
adam_g | coming up with a minimal test that captures our usage pattern, but that commit definitely breaks us in some way | 21:05 |
*** spzala has quit IRC | 21:13 | |
*** spzala has joined #openstack-oslo | 21:13 | |
*** ozamiatin has quit IRC | 21:16 | |
*** spzala has quit IRC | 21:18 | |
*** salv-orlando has joined #openstack-oslo | 21:21 | |
*** spzala has joined #openstack-oslo | 21:22 | |
*** rlrossit has quit IRC | 21:30 | |
*** mixos has quit IRC | 21:46 | |
*** yassine__ has quit IRC | 21:49 | |
*** ozialien10 has joined #openstack-oslo | 22:03 | |
adam_g | harlowja: if yer interested, https://launchpadlibrarian.net/260251888/notification_test.py | 22:10 |
*** harlowja has quit IRC | 22:22 | |
*** gordc has quit IRC | 22:25 | |
*** itisha has quit IRC | 22:29 | |
*** edmondsw has quit IRC | 22:32 | |
*** pradk has quit IRC | 22:41 | |
*** rbrndt has quit IRC | 22:51 | |
*** yamahata has quit IRC | 22:58 | |
*** harlowja has joined #openstack-oslo | 23:00 | |
harlowja | adam_g ya, hmmm, that seems off | 23:00 |
*** jamielennox is now known as jamielennox|away | 23:04 | |
*** salv-orl_ has joined #openstack-oslo | 23:10 | |
harlowja | adam_g i'm not sure on the https://github.com/openstack/oslo.messaging/blob/4.6.0/oslo_messaging/server.py#L398 and https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/server.py#L415 | 23:11 |
harlowja | sileht when he gets online might have thoughts there | 23:11 |
harlowja | there seems to be an api switch there (from what is returned) | 23:11 |
adam_g | harlowja: yeah, actually the return there changed once more recently | 23:11 |
adam_g | https://review.openstack.org/#/c/306542/7/oslo_messaging/server.py | 23:12 |
patchbot | adam_g: patch 306542 - oslo.messaging - Refactor base interfaces (MERGED) | 23:12 |
adam_g | i think that is where the interface changed for reals | 23:13 |
*** salv-orlando has quit IRC | 23:13 | |
adam_g | is simple notification listener usage tested anywhere in the o.messagings' gate? i would have thought this could have been caught in something like ceilometer's tests | 23:14 |
harlowja | i would of thought so to | 23:14 |
*** sdague has quit IRC | 23:14 | |
*** spzala has quit IRC | 23:15 | |
*** spzala has joined #openstack-oslo | 23:16 | |
harlowja | unsure though if many folks start off a thread for listener.start like the example u did has | 23:16 |
adam_g | harlowja: oh, could have made that non-threading and just called listener.start(), same result | 23:18 |
harlowja | kk | 23:19 |
harlowja | adam_g can u try using eventlet instead of threading, i'm thinking that the blocking executor, which i think is the path u are hitting may not be functional | 23:21 |
*** spzala has quit IRC | 23:21 | |
harlowja | a guess at least | 23:21 |
adam_g | harlowja: oh, i think the code im hitting is functional, but is broken in its interface--if i just call listener.wait() instead of listener.start() from my script, i can start processing things but AFAICS that's not the correct usage | 23:23 |
adam_g | http://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/notify/listener.py#n42 | 23:24 |
harlowja | adam_g right it seems that previously start did wait also, where now u must do wait yourself right? | 23:30 |
adam_g | harlowja: yeah, or something similar | 23:32 |
adam_g | anyway gotta run, will pick up with this tomorrow hopefully | 23:35 |
harlowja | kk | 23:37 |
harlowja | ya sileht might be back soon, probably has some thoughts on that | 23:37 |
*** salv-orl_ has quit IRC | 23:46 | |
*** jamielennox|away is now known as jamielennox | 23:51 | |
*** zqfan has quit IRC | 23:53 | |
*** ekcs has joined #openstack-oslo | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!