Wednesday, 2016-05-18

*** Guest74389 has quit IRC00:11
*** salv-orlando has joined #openstack-oslo00:14
*** salv-orlando has quit IRC00:36
*** dimtruck is now known as zz_dimtruck01:00
*** mtanino has quit IRC01:00
*** manjeets has left #openstack-oslo01:11
*** ozamiatin__ has quit IRC01:13
*** salv-orlando has joined #openstack-oslo01:18
*** zz_dimtruck is now known as dimtruck01:28
*** yamahata has quit IRC01:34
*** salv-orlando has quit IRC01:36
-openstackstatus- NOTICE: Gerrit is about to be restarted to help with page timeouts01:47
*** salv-orlando has joined #openstack-oslo02:12
*** salv-orlando has quit IRC02:32
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/31574802:39
*** dimtruck is now known as zz_dimtruck02:58
*** salv-orlando has joined #openstack-oslo03:05
*** zz_dimtruck is now known as dimtruck03:14
*** dimtruck is now known as zz_dimtruck03:16
*** zz_dimtruck is now known as dimtruck03:18
*** links has joined #openstack-oslo03:26
*** salv-orlando has quit IRC03:28
*** jkeen has joined #openstack-oslo03:36
*** jkeen has quit IRC03:56
*** jamespage has quit IRC04:00
*** jamespage has joined #openstack-oslo04:01
*** salv-orlando has joined #openstack-oslo04:05
*** salv-orlando has quit IRC04:25
*** amotoki has quit IRC04:26
*** amotoki has joined #openstack-oslo04:26
*** dimtruck is now known as zz_dimtruck04:29
*** yamamoto has quit IRC04:30
*** salv-orlando has joined #openstack-oslo04:58
*** SlickN1k has joined #openstack-oslo05:06
*** sabari- has joined #openstack-oslo05:06
*** mikal_ has joined #openstack-oslo05:08
*** haypo_ has joined #openstack-oslo05:08
*** davidlenwell has quit IRC05:09
*** fnordahl has quit IRC05:12
*** mikal has quit IRC05:12
*** SlickNik has quit IRC05:12
*** sabari has quit IRC05:12
*** Kennan has quit IRC05:12
*** links has quit IRC05:12
*** haypo has quit IRC05:12
*** ericksonsantos has quit IRC05:12
*** sabari- is now known as sabari05:12
*** Kennan has joined #openstack-oslo05:13
*** SlickN1k is now known as SlickNik05:13
*** ericksonsantos has joined #openstack-oslo05:13
*** yamamoto_ has joined #openstack-oslo05:13
*** links has joined #openstack-oslo05:13
*** fnordahl has joined #openstack-oslo05:15
*** lpetrut has joined #openstack-oslo05:17
*** davidlenwell has joined #openstack-oslo05:17
*** salv-orlando has quit IRC05:18
*** salv-orlando has joined #openstack-oslo05:18
*** rcernin has joined #openstack-oslo05:47
*** ozamiatin has joined #openstack-oslo05:51
*** ozamiatin has quit IRC05:52
*** david-lyle has quit IRC06:05
*** ozamiatin has joined #openstack-oslo06:16
*** salv-orlando has quit IRC06:30
*** lpetrut has quit IRC06:41
*** rcernin has quit IRC06:47
*** nkrinner_afk is now known as nkrinner06:47
*** dukhlov has joined #openstack-oslo06:48
*** ozialien10 has quit IRC06:50
*** rcernin has joined #openstack-oslo06:51
*** lpetrut has joined #openstack-oslo06:52
openstackgerritMerged openstack/oslo.vmware: Updated from global requirements  https://review.openstack.org/31749606:55
*** dukhlov has quit IRC06:58
*** dukhlov has joined #openstack-oslo07:00
*** ozamiatin has quit IRC07:07
*** salv-orlando has joined #openstack-oslo07:11
*** mixos has quit IRC07:11
*** ozamiatin has joined #openstack-oslo07:11
*** lpetrut has quit IRC07:23
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Don't skip non-direct message types  https://review.openstack.org/31680007:26
*** gcb has quit IRC07:27
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: zeromq: Deprecates host, port options  https://review.openstack.org/31787807:28
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: deprecate usage of transport aliases  https://review.openstack.org/18043407:28
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Remove deprecated localcontext  https://review.openstack.org/31730307:28
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: rabbit: Deprecates host, port, auth options  https://review.openstack.org/31728507:28
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Make TransportURL.parse aware of transport_url  https://review.openstack.org/31692007:28
*** salv-orlando has quit IRC07:31
*** lpetrut has joined #openstack-oslo07:35
*** salv-orlando has joined #openstack-oslo07:35
*** ihrachys has joined #openstack-oslo07:38
*** dukhlov has quit IRC07:38
*** gcb has joined #openstack-oslo07:41
*** salv-orl_ has joined #openstack-oslo07:48
*** jpena|off is now known as jpena07:48
*** salv-orlando has quit IRC07:52
*** shardy has joined #openstack-oslo07:53
*** gcb has quit IRC07:54
*** dukhlov has joined #openstack-oslo07:57
*** david-lyle has joined #openstack-oslo07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-oslo08:01
*** gcb has joined #openstack-oslo08:11
*** salv-orl_ has quit IRC08:27
*** dukhlov has quit IRC08:29
*** dukhlov has joined #openstack-oslo08:31
*** lucas-dinner is now known as lucasagomes08:32
*** dukhlov has quit IRC08:34
*** pblaho has joined #openstack-oslo08:36
*** yassine__ has joined #openstack-oslo08:37
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Fix bug with version_cap and target.version in RPCClient  https://review.openstack.org/30995608:42
*** vipuls has joined #openstack-oslo08:42
*** pkholkin_ has joined #openstack-oslo08:42
*** wendar_ has joined #openstack-oslo08:43
*** jraim_ has joined #openstack-oslo08:44
*** bapalm_ has joined #openstack-oslo08:44
*** shardy_ has joined #openstack-oslo08:44
*** hockeynut_afk has joined #openstack-oslo08:45
*** lucasagomes_ has joined #openstack-oslo08:45
*** haypo_ is now known as haypo08:45
*** breton_ has joined #openstack-oslo08:45
*** sigmavirus24_awa has quit IRC08:45
*** hockeynut has quit IRC08:45
*** pmalik has quit IRC08:45
*** timsim has quit IRC08:45
*** ogelbukh has quit IRC08:45
*** d34dh0r53 has quit IRC08:45
*** jraim has quit IRC08:45
*** shardy has quit IRC08:45
*** tpeoples has quit IRC08:45
*** zz_dimtruck has quit IRC08:45
*** vipul- has quit IRC08:45
*** bapalm has quit IRC08:45
*** melwitt has quit IRC08:45
*** cloudnull has quit IRC08:45
*** odyssey4me has quit IRC08:45
*** pkholkin has quit IRC08:45
*** rbradfor has quit IRC08:45
*** lucasagomes has quit IRC08:45
*** breton has quit IRC08:45
*** Nakato has quit IRC08:45
*** jamielennox has quit IRC08:45
*** wendar has quit IRC08:45
*** melwitt has joined #openstack-oslo08:45
*** Nakato has joined #openstack-oslo08:45
*** pmalik has joined #openstack-oslo08:45
*** d34dh0r53 has joined #openstack-oslo08:45
*** dimtruckl has joined #openstack-oslo08:45
*** sigmavirus24_awa has joined #openstack-oslo08:45
*** sigmavirus24_awa has quit IRC08:45
*** sigmavirus24_awa has joined #openstack-oslo08:45
*** haypo has quit IRC08:45
*** haypo has joined #openstack-oslo08:45
*** melwitt is now known as Guest3726408:46
*** dimtruckl is now known as dimtruck08:46
*** timsim has joined #openstack-oslo08:46
*** odyssey4me has joined #openstack-oslo08:46
*** pkholkin_ is now known as pkholkin08:46
*** rbradfor has joined #openstack-oslo08:46
*** cloudkiller has joined #openstack-oslo08:47
*** jraim_ is now known as jraim08:47
*** ogelbukh has joined #openstack-oslo08:47
*** dougwig has quit IRC08:48
*** lucasagomes_ is now known as lucasagomes08:49
*** rpodolyaka has quit IRC08:49
*** tpeoples has joined #openstack-oslo08:52
*** jamielennox has joined #openstack-oslo08:55
*** dtardivel has joined #openstack-oslo08:55
*** rpodolyaka has joined #openstack-oslo08:57
*** dougwig has joined #openstack-oslo08:59
*** salv-orlando has joined #openstack-oslo09:09
*** lpetrut has quit IRC09:13
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Add configurable serialization to pika  https://review.openstack.org/30417209:16
*** zqfan has joined #openstack-oslo09:36
openstackgerritMerged openstack/oslo.messaging: rabbit: Deprecates host, port, auth options  https://review.openstack.org/31728509:39
openstackgerritMerged openstack/oslo.messaging: Remove deprecated localcontext  https://review.openstack.org/31730309:41
*** cloudkiller is now known as cloudnull09:46
*** sdague has joined #openstack-oslo10:01
*** ihrachys has quit IRC10:05
openstackgerritMerged openstack/oslo.messaging: Make TransportURL.parse aware of transport_url  https://review.openstack.org/31692010:08
*** salv-orlando has quit IRC10:13
*** dukhlov has joined #openstack-oslo10:18
*** salv-orlando has joined #openstack-oslo10:20
*** cdent has joined #openstack-oslo10:23
*** cdent has left #openstack-oslo10:35
*** dimtruck is now known as zz_dimtruck10:36
*** lpetrut has joined #openstack-oslo10:57
openstackgerritMerged openstack/oslo.db: api: always enable retry_on_request  https://review.openstack.org/31729911:10
*** tedross has joined #openstack-oslo11:14
*** tedross has left #openstack-oslo11:15
openstackgerritMerged openstack/oslo.messaging: zeromq: Deprecates host, port options  https://review.openstack.org/31787811:26
openstackgerritMerged openstack/oslo.messaging: Fix bug with version_cap and target.version in RPCClient  https://review.openstack.org/30995611:26
*** dukhlov has quit IRC11:28
*** dukhlov has joined #openstack-oslo11:34
*** yamamoto_ has quit IRC11:38
*** lucasagomes is now known as lucas-hungry11:38
*** jpena is now known as jpena|lunch11:46
*** sigmavirus24_awa is now known as sigmavirus2412:12
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606912:20
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606912:22
*** jdandrea_ has quit IRC12:22
*** jdandrea has joined #openstack-oslo12:22
*** gordc has joined #openstack-oslo12:23
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606912:28
*** kgiusti has joined #openstack-oslo12:28
openstackgerritAlexis Lee proposed openstack/oslo.log: Reload log_config_append config on SIGHUP  https://review.openstack.org/29356112:28
*** links has quit IRC12:31
*** lucas-hungry is now known as lucasagomes12:32
*** zz_dimtruck is now known as dimtruck12:33
*** links has joined #openstack-oslo12:35
*** ihrachys has joined #openstack-oslo12:40
*** salv-orlando has quit IRC12:42
*** yamamoto has joined #openstack-oslo12:58
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606913:01
*** jpena|lunch is now known as jpena13:02
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606913:03
*** edmondsw has joined #openstack-oslo13:04
*** rlrossit has joined #openstack-oslo13:11
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606913:13
*** links has quit IRC13:16
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606913:23
*** tonytan4ever has joined #openstack-oslo13:31
*** dimtruck is now known as zz_dimtruck13:42
*** salv-orlando has joined #openstack-oslo13:43
*** mtanino has joined #openstack-oslo13:43
*** lpetrut has quit IRC13:46
openstackgerritKirill Bespalov proposed openstack/oslo.messaging: Fix consuming from missing queues  https://review.openstack.org/31570013:48
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/31574813:56
*** pradk has joined #openstack-oslo14:00
*** yamahata has joined #openstack-oslo14:03
*** lpetrut has joined #openstack-oslo14:06
*** salv-orlando has quit IRC14:09
*** links has joined #openstack-oslo14:10
*** shardy_ is now known as shardy14:14
*** rbrndt has joined #openstack-oslo14:15
*** yamahata has quit IRC14:18
openstackgerritKirill Bespalov proposed openstack/oslo.messaging: Fix consuming from missing queues  https://review.openstack.org/31570014:27
openstackgerritJulien Danjou proposed openstack/oslo.db: api: use sane default in wrap_db_retry()  https://review.openstack.org/31813514:28
*** salv-orlando has joined #openstack-oslo14:35
*** dukhlov has quit IRC14:35
*** dukhlov has joined #openstack-oslo14:37
*** links has quit IRC14:39
*** takedakn has joined #openstack-oslo14:48
*** takedakn has quit IRC14:56
*** hockeynut_afk is now known as hockeynut14:56
*** tonytan4ever has quit IRC15:01
*** salv-orlando has quit IRC15:05
*** mixos has joined #openstack-oslo15:06
kevinbentonharlowja: hey!15:09
*** dukhlov has quit IRC15:12
*** rcernin has quit IRC15:24
*** itisha has joined #openstack-oslo15:26
*** salv-orlando has joined #openstack-oslo15:31
*** salv-orlando has quit IRC15:41
*** salv-orlando has joined #openstack-oslo15:42
*** dukhlov has joined #openstack-oslo15:43
*** ozamiatin has quit IRC15:43
*** nkrinner is now known as nkrinner_afk15:43
*** zz_dimtruck is now known as dimtruck15:51
openstackgerritIldar Svetlov proposed openstack/oslo.messaging: [kafka] Add several bootstrap servers support  https://review.openstack.org/30766815:52
*** tonytan4ever has joined #openstack-oslo15:53
*** Rockyg has joined #openstack-oslo15:55
*** dukhlov has quit IRC16:15
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: Remove logging from serialize_remote_exception  https://review.openstack.org/31606916:21
openstackgerritMerged openstack/oslo.messaging: Reorganize the AMQP 1.0 driver source files  https://review.openstack.org/31421816:21
*** dimtruck is now known as zz_dimtruck16:27
*** zz_dimtruck is now known as dimtruck16:29
openstackgerritAlexis Lee proposed openstack/oslo.log: log: Spooling logger and manager  https://review.openstack.org/31616216:36
openstackgerritAlexis Lee proposed openstack/oslo.log: log: Extra spool overflow strategies  https://review.openstack.org/31821216:36
harlowjakevinbenton  yo yo16:38
harlowjalxsli yt, how do u think https://review.openstack.org/#/c/308399/ would be implemented ontop of the refactored stuff16:39
patchbotharlowja: patch 308399 - oslo.utils - specs_matcher: Allow passing custom op_methods16:39
lxsliharlowja: wow that's pretty crazy16:43
harlowjalol16:43
harlowjaya, its like dynamic DSL :-/16:44
harlowjawithout a grammar, lol16:44
*** dukhlov has joined #openstack-oslo16:44
lxsliIf there's really only going to be one alternative, pass in a flag that alters the BNF16:44
lxsliOtherwise... maybe just don't?16:44
harlowja:)16:44
*** ri0 has joined #openstack-oslo16:44
harlowjaya, 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 are16:45
harlowjajroll yt16:45
lxsliyeah with the patch as-is, you could do something crazy like make "=" mean ">="16:45
lxslibtw harlowja I'd love your thoughts on my log spooling work16:48
harlowjadef16:48
harlowjalxsli saw that, it seems ok, idk about the exception thingy, is that normal? idk16:48
harlowjadoes like the syslog stuff blow-up if it can't work?16:48
harlowjaor just it does drop things?16:48
lxsliI've changed the default strategy to just flush16:48
lxsliand put up a new patch with a ton of alternative strategies16:49
harlowjakk16:49
harlowjalxsli was there any pypi solutions that did spooling to, i would've thought there would have been16:49
harlowjaother thoughts, does systemd do all this spolling itself, idk16:50
lxsliharlowja: there's bbrecorder but it doesn't really fit our needs imo16:51
*** Guest37264 is now known as melwitt16:51
harlowjakk16:51
lxslilogrecord sinks don't know when a request succeeds so they can't drop those logs16:52
harlowjagotcha16:53
*** dukhlov has quit IRC16:59
kgiustihallelujah17:01
harlowjalol17:01
kevinbentonharlowja: yo17:04
kevinbentonharlowja: so select.poll() seems to block even with eventlet17:04
harlowjahmmm17:05
harlowjadon't do that :-P17:05
harlowjalol17:05
kevinbentonharlowja: if you turn on watch-file in oslo logging, it uses pyinotify, which does that17:07
kevinbentonharlowja: it's a cool feature where the whole agent freezes for 500 milliseconds like every second17:07
harlowjalol17:07
harlowjai wonder if something not monkey patched somewhere?17:07
kevinbentonharlowja: well i was looking around and it sounds like directly calling poll isn't monkey patched by eventlet?17:07
kevinbentonharlowja: does that sound familiar?17:08
*** tonytan4ever has quit IRC17:08
harlowjahmmm, that'd seem odd17:08
*** lpetrut has quit IRC17:08
kevinbentonharlowja: all of the other monkey patched stuff works okay17:10
harlowjahmmmm17:10
kevinbentonharlowja: like oslo messaging stuff17:10
kevinbentonharlowja: and i ensured that the monkey_patch was called way before pyinotify was even imported17:10
harlowjaya, i wonder17:11
harlowjahttps://github.com/eventlet/eventlet/pull/317 is interesting17:11
harlowjahaypo yt17:11
kevinbentonharlowja: so i was wondering if we should at least change the poll timeout in oslo log to like 5ms for a quick backport17:12
harlowjaseems reasonable17:13
harlowjaor switch to not poll17:13
kevinbentonpyinotify does that17:13
harlowjak17:13
kevinbentoni don't think we have an option there17:14
kevinbentonharlowja: submitting a patch now and bug report17:14
harlowjak17:14
harlowjahttps://bitbucket.org/eventlet/eventlet/issues/167/monkey-patch-on-select-leaves-selectpoll i guess u saw also kevinbenton ?17:16
*** Rockyg has quit IRC17:17
kevinbentonharlowja: ah, interesting17:17
kevinbentonharlowja: so now we will get an attribute error after that patch, right?17:17
harlowjai thinks so17:18
*** shardy has quit IRC17:19
*** jpena is now known as jpena|off17:22
openstackgerritKevin Benton proposed openstack/oslo.log: Reduce READ_FREQ and TIMEOUT for watch-file  https://review.openstack.org/31825617:39
kevinbentonharlowja: ^^17:39
*** lpetrut has joined #openstack-oslo17:40
openstackgerritDougal Matthews proposed openstack/mox3: Correct spelling of occurrences  https://review.openstack.org/31825717:40
*** rlrossit has quit IRC17:43
*** spzala has joined #openstack-oslo17:43
stevemardims_: harlowja spzala o/17:43
stevemardims_: harlowja question about 'lib-forward-testing'17:44
spzalastevemar: o/17:44
spzaladims_: o/17:44
*** lucasagomes is now known as lucas-dinner17:44
*** lpetrut has quit IRC17:44
stevemardoes lib-forward-testing test dsvm against the master branch of oslo, instead of the latest release?17:44
stevemarmaster branch of an oslo project*17:45
dims_stevemar : yes17:46
harlowja(sorry on call)17:47
stevemarharlowja: np17:47
stevemardims_: okay, but it just installs it and runs the dsvm job right? and tempest tests?17:47
dims_hang on let me check something17:48
*** rlrossit has joined #openstack-oslo17:49
stevemardims_: spzala based on what i see here: https://github.com/openstack-infra/project-config/blob/35b38e4f8bf40999799fce1d2c3f92614c33358f/jenkins/jobs/devstack-gate.yaml#L128-L14117:49
stevemari think it just sets up dvsm and runs tempest :P17:50
*** dimtruck is now known as zz_dimtruck17:50
dims_stevemar : http://git.openstack.org/cgit/openstack-infra/project-config/tree/zuul/layout.yaml#n39917:50
dims_those are the 2 job names that will show up17:50
stevemardims_: yes, line 399 refers to the link i wrote 2 lines up17:51
dims_sorry, same name in gate and check17:51
dims_stevemar : yep17:51
stevemardims_: basically spzala has a similar problem, his heat-translator project depends on tosca-parser (these are all in the openstack namespace)17:52
stevemardims_: but i think both projects just run py27 tests17:52
*** zz_dimtruck is now known as dimtruck17:52
spzalastevemar: thanks, yes. dims_: going forward I want to support backward compatibility with our pypi releases17:53
stevemardims_: spzala wants to make sure the master branch of tosca-parser doesn't break heat-translator when released17:53
spzalastevemar: by27 and py34 tests17:53
stevemarnot sure if theres a way to set up the py27 tests to install from git instead of latest release17:53
spzalastevemar: dims_: ideally both, that master of tosca-parser doesn't break master of heat-translator and latest pypi of heat-translator17:53
stevemarspzala: if it doesn't break the master of heat-translator, it shouldn't break the latest release of heat-translator :P17:54
spzalastevemar: dims_: I believe it's safer that way to test against both17:55
spzalastevemar: :-) well, what if any change in master of heat-translator is already broke previous latest release of heat-translator17:55
spzalastevemar: I think that's totally possible17:56
stevemarspzala: 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 available17:56
spzalas/has/is17:56
openstackgerritKen Giusti proposed openstack/oslo.messaging: Add the proper branch back to .gitreview  https://review.openstack.org/31826517:56
spzalastevemar: sure, that definitely makes sense to use if something is already available17:57
spzalasteve_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
spzalai.e. for tosca-parser to test against master and latest release of heat-translator17:59
*** dtardivel has quit IRC18:00
*** yamahata has joined #openstack-oslo18:00
jrollharlowja: here now18:02
stevemarspzala: actually, i think you want something like "periodic-jobs-with-oslo-master"18:04
*** wendar_ is now known as wendar18:04
*** tonytan4ever has joined #openstack-oslo18:18
*** ihrachys has quit IRC18:18
*** dimtruck is now known as zz_dimtruck18:19
harlowjajroll 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
patchbotharlowja: patch 308399 - oslo.utils - specs_matcher: Allow passing custom op_methods18:20
patchbotharlowja: patch 313699 - oslo.utils - Use an actual well defined parser for spec matching18:20
harlowjaas like an option to match integer_only=True or something18:21
harlowjavs full on change all the operations18:21
jrollharlowja: that's what it's about today, not sure about the future18:23
jrollharlowja: though I'm thinking more about this, it's unclear to me if we actually want this18:23
jrollwe should be doing this in the db for what we're using it for18:24
harlowjajroll likely via sql and such?18:25
jrollharlowja: yeah18:25
openstackgerritKen Giusti proposed openstack/oslo.messaging: Refactor link management to support link recovery  https://review.openstack.org/31827518:26
harlowjajroll fair enough18:26
*** isq has joined #openstack-oslo18:27
*** dukhlov has joined #openstack-oslo18:28
harlowjajroll so whole spec stuff and madness there might not be needed, that correct18:28
harlowja(just double-checking, i'm ok with that honestly) :-P18:28
jrollharlowja: more like, we may or may not even need to move this code18:28
harlowjaright18:30
kgiustiharlowja: hey - I'm having some problems using that feature branch (feature/amqp-dispatch-router)18:34
harlowjakgiusti uhohs?18:34
kgiustiharlowja: I've restored the .gitreview to include the branch= directive.18:34
kgiustiharlowja: but whenever I do a git review, the review is against master, not the branch18:35
harlowjahmmm18:35
kgiustiharlowja: I created by local branch simply by checking out a branch that tracks remote/feature/amqp-dispatch-router18:35
harlowjado u have a gerrit remote that might be off18:35
harlowjai've seen that kind happen before18:36
harlowjai'm wondering if your git config might have something weird in it18:36
harlowjai've seen that before, unsure if this might be the cause18:37
harlowjacat .git/config18:37
harlowjamight be a gerrit or branch in there that's not pointing at the right place?18:37
kgiustiharlowja: git/config: http://paste.fedoraproject.org/368121/35966561/18:38
kgiustiharlowja: I'm trying to git review from the 'feature/amqp-dispatch-router' branch18:39
kgiustiharlowja: the .gitreview file:  https://paste.fedoraproject.org/368118/96608146/18:39
*** ri0 has quit IRC18:39
*** ri0 has joined #openstack-oslo18:40
harlowjaya, hmmm it seems right18:40
harlowjawant to jump on the infra channel, they might have someone who understands git-review better than myself for this18:41
harlowjaseems odd18:41
kgiustiharlowja: kk - will do18:41
*** ri0 has quit IRC18:42
*** dukhlov has quit IRC18:44
*** lpetrut has joined #openstack-oslo18:47
harlowjakevinbenton  https://ondergetekende.nl/using-pyinotify-with-eventlet.html lol18:49
harlowjaseems like someone else worked around this also18:49
kgiustiharlowja: fixed - turns out the  "branch" keyword in the .gitreview file is wrong - needs to be "defaultbranch"18:50
harlowjaand https://github.com/seb-m/pyinotify/pull/117 kevinbenton18:50
harlowjakgiusti ahhh, good to know18:50
harlowjakevinbenton  i guess 117 there would also fix the issue18:51
harlowjaif it ever merges/gets released18:51
kevinbentonharlowja: yeah18:54
kevinbentonharlowja: another option would be to override the read_events method18:54
kevinbentonharlowja: and write it not to use poll18:54
harlowjaya, seems like we can go there if we have to :-/18:54
*** amotoki has quit IRC18:56
openstackgerritDougal Matthews proposed openstack/mox3: Correct spelling of occurrences  https://review.openstack.org/31825719:41
*** yamamoto has quit IRC19:55
*** salv-orlando has quit IRC19:59
*** zz_dimtruck is now known as dimtruck20:06
*** tonytan4ever has quit IRC20:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/31574820:26
adam_ganyone 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 release20:32
adam_g(oslo.messaging)20:32
*** ozamiatin has joined #openstack-oslo20:37
*** lpetrut has quit IRC20:40
*** lpetrut has joined #openstack-oslo20:41
*** lpetrut has quit IRC20:46
*** kgiusti has left #openstack-oslo20:51
*** yamamoto has joined #openstack-oslo20:56
*** yamamoto has quit IRC21:03
harlowjaadam_g i don't have much expertise in that, but how did it break astra's ability ?21:04
harlowjasince afaik that was mostly internal refactoring21:04
harlowjaany more details?21:04
adam_gharlowja: still digging in trying to figure that out21:04
adam_gcoming up with a minimal test that captures our usage pattern, but that commit definitely breaks us in some way21:05
*** spzala has quit IRC21:13
*** spzala has joined #openstack-oslo21:13
*** ozamiatin has quit IRC21:16
*** spzala has quit IRC21:18
*** salv-orlando has joined #openstack-oslo21:21
*** spzala has joined #openstack-oslo21:22
*** rlrossit has quit IRC21:30
*** mixos has quit IRC21:46
*** yassine__ has quit IRC21:49
*** ozialien10 has joined #openstack-oslo22:03
adam_gharlowja: if yer interested, https://launchpadlibrarian.net/260251888/notification_test.py22:10
*** harlowja has quit IRC22:22
*** gordc has quit IRC22:25
*** itisha has quit IRC22:29
*** edmondsw has quit IRC22:32
*** pradk has quit IRC22:41
*** rbrndt has quit IRC22:51
*** yamahata has quit IRC22:58
*** harlowja has joined #openstack-oslo23:00
harlowjaadam_g ya, hmmm, that seems off23:00
*** jamielennox is now known as jamielennox|away23:04
*** salv-orl_ has joined #openstack-oslo23:10
harlowjaadam_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#L41523:11
harlowjasileht when he gets online might have thoughts there23:11
harlowjathere seems to be an api switch there (from what is returned)23:11
adam_gharlowja: yeah, actually the return there changed once more recently23:11
adam_ghttps://review.openstack.org/#/c/306542/7/oslo_messaging/server.py23:12
patchbotadam_g: patch 306542 - oslo.messaging - Refactor base interfaces (MERGED)23:12
adam_gi think that is where the interface changed for reals23:13
*** salv-orlando has quit IRC23:13
adam_gis 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 tests23:14
harlowjai would of thought so to23:14
*** sdague has quit IRC23:14
*** spzala has quit IRC23:15
*** spzala has joined #openstack-oslo23:16
harlowjaunsure though if many folks start off a thread for listener.start like the example u did has23:16
adam_gharlowja: oh, could have made that non-threading and just called listener.start(), same result23:18
harlowjakk23:19
harlowjaadam_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 functional23:21
*** spzala has quit IRC23:21
harlowjaa guess at least23:21
adam_gharlowja: 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 usage23:23
adam_ghttp://git.openstack.org/cgit/openstack/oslo.messaging/tree/oslo_messaging/notify/listener.py#n4223:24
harlowjaadam_g right it seems that previously start did wait also, where now u must do wait yourself right?23:30
adam_gharlowja: yeah, or something similar23:32
adam_ganyway gotta run, will pick up with this tomorrow hopefully23:35
harlowjakk23:37
harlowjaya sileht might be back soon, probably has some thoughts on that23:37
*** salv-orl_ has quit IRC23:46
*** jamielennox|away is now known as jamielennox23:51
*** zqfan has quit IRC23:53
*** ekcs has joined #openstack-oslo23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!