Thursday, 2015-07-16

*** zz_dimtruck is now known as dimtruck00:11
*** mtanino has quit IRC00:13
*** tsekiyama has quit IRC00:15
*** pblaho has quit IRC00:18
*** manishg has joined #openstack-oslo00:19
*** salv-orlando has quit IRC00:24
*** manishg has quit IRC00:24
*** manishg has joined #openstack-oslo00:25
*** salv-orlando has joined #openstack-oslo00:26
*** achanda_ has quit IRC00:26
*** salv-orlando has quit IRC00:34
openstackgerritlifeless proposed openstack-dev/pbr: Export ChangeLog and AUTHORS in install  https://review.openstack.org/20233900:36
*** ujjain has quit IRC00:36
*** ujjain has joined #openstack-oslo00:37
*** ujjain has quit IRC00:37
*** ujjain has joined #openstack-oslo00:37
*** ozamiatin has quit IRC00:39
*** openstack has joined #openstack-oslo01:24
*** yamahata has quit IRC01:26
*** sputnik13 has quit IRC01:32
*** dims_ has quit IRC01:58
openstackgerritlifeless proposed openstack-dev/pbr: Export ChangeLog and AUTHORS in install  https://review.openstack.org/20233902:10
*** ozamiatin has joined #openstack-oslo02:16
*** dims__ has joined #openstack-oslo02:18
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737602:28
*** dims__ has quit IRC02:30
*** yamahata has joined #openstack-oslo02:34
*** yamahata has quit IRC02:36
*** yamahata has joined #openstack-oslo02:37
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737602:40
*** katyafervent is now known as katyafervent_awa02:41
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737602:43
openstackgerritJoshua Harlow proposed openstack/taskflow: Create and use a serial retry executor  https://review.openstack.org/19578402:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Separate revert/execute failure into different outcome constants  https://review.openstack.org/19984802:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Cache atom name -> actions and provide accessor function  https://review.openstack.org/19821102:46
*** yamahata has quit IRC02:50
*** sdake_ has joined #openstack-oslo02:53
*** sdake has quit IRC02:57
*** sputnik13 has joined #openstack-oslo03:05
*** crc32 has quit IRC03:10
*** ozamiatin has quit IRC03:19
*** jraim has quit IRC03:45
*** tchaypo has quit IRC03:45
*** zhiyan has quit IRC03:45
*** h00327910_ has quit IRC03:46
*** boris-42 has quit IRC03:46
*** stevemar has joined #openstack-oslo03:48
*** freyes has quit IRC03:52
*** zhiyan has joined #openstack-oslo03:54
*** sdake_ has quit IRC03:55
*** jraim has joined #openstack-oslo03:56
*** achanda has joined #openstack-oslo04:02
*** boris-42 has joined #openstack-oslo04:07
*** h00327910_ has joined #openstack-oslo04:10
*** harlowja_at_home has joined #openstack-oslo04:33
*** zzzeek has joined #openstack-oslo04:37
*** freyes has joined #openstack-oslo04:40
*** tchaypo_ has joined #openstack-oslo04:45
*** zzzeek has quit IRC04:52
*** rpodolyaka1 has joined #openstack-oslo05:07
*** manishg has quit IRC05:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737605:43
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737605:52
openstackgerritJoshua Harlow proposed openstack/automaton: Ensure doctesting and doc8 testing done in py27 env  https://review.openstack.org/20168705:55
openstackgerritJoshua Harlow proposed openstack/taskflow: Move doc8 to being a normal test requirement in test-requirements.txt  https://review.openstack.org/20239305:59
*** e0ne has joined #openstack-oslo06:00
*** lakshmiS has joined #openstack-oslo06:01
*** e0ne has quit IRC06:02
lakshmiSdoes anyone know why notifications in a topic are not sent to multiple listeners when using different oslo messaging pool names?06:02
*** rpodolyaka1 has quit IRC06:07
*** nkrinner_afk is now known as nkrinner06:07
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737606:09
*** harlowja_at_home has quit IRC06:10
*** rpodolyaka1 has joined #openstack-oslo06:39
*** stevemar has quit IRC06:42
*** tchaypo_ is now known as tchaypo06:52
openstackgerritMerged openstack/oslo.utils: Add oslo.config to test requirements  https://review.openstack.org/20125207:01
*** shardy has joined #openstack-oslo07:03
*** rpodolyaka1 has quit IRC07:28
*** ihrachyshka has joined #openstack-oslo07:32
*** rpodolyaka1 has joined #openstack-oslo07:35
*** rpodolyaka1 has quit IRC07:35
*** rpodolyaka1 has joined #openstack-oslo07:35
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Ensure callback variable capture is done correctly  https://review.openstack.org/20220207:37
*** stevemar has joined #openstack-oslo07:42
*** BrianShang has joined #openstack-oslo07:45
*** achanda has quit IRC07:46
*** stevemar has quit IRC07:47
*** rpodolyaka1 has quit IRC07:47
*** lakshmiS has quit IRC07:53
*** cdent has joined #openstack-oslo07:57
*** pblaho has joined #openstack-oslo08:13
*** enikanorov_ has joined #openstack-oslo08:15
*** enikanorov has quit IRC08:17
*** rpodolyaka1 has joined #openstack-oslo08:19
*** katyafervent_awa is now known as katyafervent08:26
*** rpodolyaka1 has quit IRC08:32
*** boris-42 has quit IRC08:32
*** cdent has quit IRC08:33
*** cdent has joined #openstack-oslo08:33
*** rpodolyaka1 has joined #openstack-oslo08:34
*** jecarey has quit IRC08:36
*** yassine has joined #openstack-oslo08:40
*** exploreshaifali has joined #openstack-oslo08:41
*** exploreshaifali has quit IRC08:46
*** salv-orlando has joined #openstack-oslo08:46
*** ozamiatin has joined #openstack-oslo09:09
*** ndipanov has joined #openstack-oslo09:14
*** stevemar has joined #openstack-oslo09:24
*** ozamiatin has quit IRC09:25
*** dimtruck is now known as zz_dimtruck09:27
*** stevemar has quit IRC09:28
*** ozamiatin has joined #openstack-oslo09:31
*** ozamiatin has quit IRC09:32
*** salv-orlando has quit IRC09:40
*** jaosorior has joined #openstack-oslo09:41
*** viktors has joined #openstack-oslo09:44
*** ozamiatin has joined #openstack-oslo09:44
viktorssileht: hi!09:44
silehtviktors, hi09:44
*** rpodolyaka1 has quit IRC09:45
viktorssileht: I'm a bit confused by the functional test behavioral. Can you please suggest?09:46
silehtviktors, what do you expect/not expect to occur ?09:47
viktorswhen we create a Target() object we cat set fanout=True and "a message be directed to all  servers listening on a topic"09:47
viktorssileht: but do we want to get the same if we set fanout=False and server=None  ?09:48
*** rpodolyaka1 has joined #openstack-oslo09:49
silehtviktors, in this case that should go in one random server listening to the topics09:49
viktorssileht: aha09:52
viktorssileht: thanks a lot!09:52
*** e0ne has joined #openstack-oslo10:02
*** rpodolyaka1 has quit IRC10:20
openstackgerritAngus Lees proposed openstack/oslo.db: Test that concurrent sqlalchemy transactions don't block  https://review.openstack.org/10443610:22
*** sputnik13 has quit IRC10:23
*** dims_ has joined #openstack-oslo10:24
*** e0ne is now known as e0ne_10:25
*** e0ne_ is now known as e0ne10:28
*** amotoki has joined #openstack-oslo10:32
*** dims_ has quit IRC10:39
*** salv-orlando has joined #openstack-oslo10:40
*** exploreshaifali has joined #openstack-oslo10:43
*** BrianShang has quit IRC10:45
*** ozamiatin has quit IRC10:55
*** ozamiatin has joined #openstack-oslo10:59
*** ozamiatin has quit IRC11:02
*** e0ne is now known as e0ne_11:13
*** e0ne_ is now known as e0ne11:30
*** shardy_ has joined #openstack-oslo11:39
*** shardy has quit IRC11:39
*** shardy_ has quit IRC11:44
*** shardy has joined #openstack-oslo11:45
*** rpodolyaka1 has joined #openstack-oslo11:45
*** ganso_ has joined #openstack-oslo11:48
sdaguehey dims, now that - https://review.openstack.org/#/c/195292/ is in pbr, can you coordinate another release of it? After that is out there we can start getting projects over to it11:56
*** amotoki has quit IRC12:04
*** prad has joined #openstack-oslo12:06
*** dims_ has joined #openstack-oslo12:14
*** nkrinner is now known as nkrinner_afk12:18
*** gordc has joined #openstack-oslo12:24
*** stevemar has joined #openstack-oslo12:24
openstackgerritPavel Kholkin proposed openstack/oslo.cache: Added NO_VALUE to core file  https://review.openstack.org/20253112:26
*** pradk has joined #openstack-oslo12:27
*** stevemar has quit IRC12:29
*** stevemar has joined #openstack-oslo12:30
*** dims_ has quit IRC12:30
*** salv-orlando has quit IRC12:31
*** dims__ has joined #openstack-oslo12:32
*** stevemar has quit IRC12:35
*** ozamiatin has joined #openstack-oslo12:38
*** dims__ has quit IRC12:39
*** rpodolyaka1 has quit IRC12:41
*** rpodolyaka1 has joined #openstack-oslo12:41
*** shardy_ has joined #openstack-oslo12:48
*** shardy has quit IRC12:49
*** stevemar has joined #openstack-oslo12:50
*** stevemar_ has joined #openstack-oslo12:51
*** shardy_ has quit IRC12:53
*** salv-orlando has joined #openstack-oslo12:53
*** shardy has joined #openstack-oslo12:54
*** stevemar has quit IRC12:54
*** stevemar_ has quit IRC12:55
*** stevemar has joined #openstack-oslo12:56
*** stevemar_ has joined #openstack-oslo12:57
*** stevemar has quit IRC13:01
*** haypo has joined #openstack-oslo13:01
hayposileht: hi. are you aware of oslo_messaging.tests.drivers.test_impl_rabbit.TestSendReceive.test_send_receive failures on py34?13:01
hayposileht: oh, and can you review https://review.openstack.org/#/c/202115/ ?13:02
silehthaypo, yes13:02
haypoi'm sad, the py34 is no more voting13:02
silehthaypo, me too13:02
hayposileht: what can we do for test_send_receive failures? why does the py34 gate not vote anymore?13:03
silehthaypo, but it segfault and we don't really known who is resposible to trigger this: https://bugs.launchpad.net/ubuntu/+source/python3.4/+bug/136790713:04
openstackLaunchpad bug 1367907 in python3.4 (Ubuntu Trusty) "Segfault in gc with cyclic trash" [Undecided,Triaged]13:04
hayposileht: the cause is known, canonical is unable to backport a simple fix, nor to upgrade their python3 package...13:04
hayposileht: this crash is really severe, i don't understand why it's not fixed13:05
silehthaypo, me too...13:05
hayposileht: ok. can we try by fixing //review.openstack.org/#/c/202115/ ?13:06
haypo(stupid gnome, it doesn't copy/paste https: prefix)13:07
*** bnemec has joined #openstack-oslo13:14
*** dims_ has joined #openstack-oslo13:16
*** zzzeek has joined #openstack-oslo13:24
*** bknudson has joined #openstack-oslo13:25
*** sdake has joined #openstack-oslo13:25
*** jaypipes has joined #openstack-oslo13:35
*** rpodolyaka1 has quit IRC13:39
dansmithdims_: on that dict bug13:41
*** rpodolyaka1 has joined #openstack-oslo13:41
dims_y, do we need a test?13:41
dansmithdims_: I'd like to write a test, but I can't run tests anymore because of tox or pbr or venv or something13:41
dansmithdims_: I asked you yesterday but you were busy I guess13:41
*** prad has quit IRC13:41
dims_ack dansmith. i'll take a crack at a test or will +A it later today13:41
dims_dansmith: sorry, in keystone mid-cycle13:42
dansmithdims_: no, I want to write the test, I just didn't because I can't run tests13:42
dansmithlet me get you a pastebin13:42
dims_ah ok13:42
*** prad has joined #openstack-oslo13:42
dansmithdims_: http://pastebin.com/U8jQ8tem13:43
dansmithdims_: figure we need to change something because of recent stuff or something?13:43
dims_let me try running tests...refreshing my sandbox now13:43
*** e0ne is now known as e0ne_13:44
dansmithokay, I've deleted .tox and still see it13:46
dansmithI think it happened after I upgraded venv to make my nova tests work :(13:46
*** exploreshaifali has quit IRC13:48
dims_dansmith: here's what i have - http://paste.openstack.org/show/380822/13:49
*** e0ne_ is now known as e0ne13:49
dims_what are your versions?13:49
*** pradk has quit IRC13:52
dims_dansmith: i tried pbr==1.3.0 as well, seems to work fine13:52
dansmithmy tox is 1.813:52
dansmithsigh13:52
dims_CI system seems to be on tox==2.0.113:54
*** mriedem_away is now known as mriedem13:56
dansmithtox 2.1.1 gives the same13:56
* dansmith tries 2.0.113:56
dansmithoh, you're on 2.1.113:57
openstackgerritMerged openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/20228713:59
*** yamahata has joined #openstack-oslo14:01
*** sigmavirus24_awa is now known as sigmavirus2414:02
*** Kennan has quit IRC14:02
*** Kennan has joined #openstack-oslo14:03
*** jecarey has joined #openstack-oslo14:11
openstackgerritMerged openstack/oslo.utils: versionutils: add version convert helper methods  https://review.openstack.org/19850014:25
*** mriedem has left #openstack-oslo14:26
openstackgerritRandall Nortman proposed openstack/oslo.messaging: Restore debug logging of each message in AMQPListener  https://review.openstack.org/19124214:27
*** mriedem has joined #openstack-oslo14:27
openstackgerritMerged openstack/oslo.service: Add oslo_debug_helper to tox.ini  https://review.openstack.org/20165214:27
openstackgerritMerged openstack/oslo.service: Add oslo_debug_helper to tox.ini  https://review.openstack.org/20165214:27
openstackgerritMerged openstack/oslo.versionedobjects: Replace 'hypervisor' with 'provided' version  https://review.openstack.org/20136714:28
mriedemhaypo: zzzeek: any ideas what might be causing this test to only fail with py34 in sqla-migrate? http://logs.openstack.org/70/202170/2/check/gate-sqlalchemy-migrate-python34/b8fe666/console.html#_2015-07-15_19_57_42_55714:32
mriedemtrace is here actually http://logs.openstack.org/70/202170/2/check/gate-sqlalchemy-migrate-python34/b8fe666/console.html#_2015-07-15_19_57_57_94314:32
zzzeekmriedem: migrate supports py3 ?14:33
mriedemzzzeek: yeah, it's had a py34 job for awhile14:33
mriedemwas working last week when i did a release14:33
zzzeekmriedem: module import mechanics?  looks likje migrate is doing some import stuff14:33
*** zz_dimtruck is now known as dimtruck14:34
mriedemi'm not familiar with py3 really, i haven't dug too far into the tests,14:34
mriedemwas trying to unblock migrate w/ mock 1.114:34
mriedemhttps://review.openstack.org/#/c/202170/14:34
zzzeekmriedem: does this fail on py3.3 or we are only testing on py3.4?14:34
mriedemb/c mock 1.1 doesn't support py26 and migrate has a py26 jo14:34
mriedem*job14:34
mriedemonly 3414:34
mriedemwell, we only test on 3414:34
zzzeekmriedem: maybe it has somethign to do with .pyc14:34
mriedemhrm14:35
zzzeekmriedem: months ago I got into a huge flamewar on the list that I really didnt think we should be deleting .pyc files and it was b.c. i had dealt a lot with this in alembic14:35
zzzeekmriedem: e.g. they are important14:35
zzzeekmriedem: anyway, the test here seems to be looking for self.assertRaises(exceptions.InvalidScriptError, self.cls.verify_module, path)14:35
zzzeekmriedem: and instead its getting ImportError14:35
mriedemwell, the migrate tests don't delete pycs first like nova does14:35
mriedemhttps://github.com/stackforge/sqlalchemy-migrate/blob/master/tox.ini14:35
mriedemhttps://github.com/stackforge/sqlalchemy-migrate/blob/master/tools/pretty_tox.sh14:36
zzzeekmriedem: so seems like that might be something simple, wrong kind of error14:36
*** e0ne is now known as e0ne_14:36
mriedemok, maybe changing to importlib.import_module() would do something, no idea really, just reading docs on https://docs.python.org/3/library/functions.html#__import__ says to not use it14:38
*** shardy_ has joined #openstack-oslo14:38
*** kairat_kushaev has quit IRC14:38
mriedemzzzeek: i opened a bug https://bugs.launchpad.net/sqlalchemy-migrate/+bug/1475339 to track it, if i can't get it sorted out today i'll probably push a change to skip the tests for now until it can be worked on since it's blocking all migrate changes and there is one that zigo was trying to get in14:39
openstackLaunchpad bug 1475339 in sqlalchemy-migrate "migrate.tests.versioning.test_script.TestPyScript.test_verify_nofuncs randomly fails on py34" [Undecided,New]14:39
*** shardy has quit IRC14:39
zigomriedem: Mine isn't urgent, I have it in my package, and it's not impacting anyone else.14:40
zzzeekmriedem: here’s alembic’s latest import stuff:  https://bitbucket.org/zzzeek/alembic/src/c1947910621ad283219c2730d9224644e51a816c/alembic/util/compat.py?at=master#cl-6314:40
mriedemzigo: ok14:40
zigomriedem: It's just that it needs to get in at some point to avoid me rebasing it constantly.14:40
zzzeekmriedem: as you can see, py33, *totally* different14:40
zzzeekmriedem: i’d recommend emulating that14:40
openstackgerritMerged openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/20182614:41
mriedemzzzeek: ok14:41
mriedemi'll make a note of it14:41
zigozzzeek: Did you see this one? https://bugs.launchpad.net/keystone/+bug/147476014:43
openstackLaunchpad bug 1474760 in Keystone "Unit test failures with sqlalchemy 1.0.6" [Undecided,Incomplete]14:43
zigoI wonder why it used to work with SQLA 0.9.714:43
*** shardy_ has quit IRC14:43
openstackgerritMerged openstack/tooz: Updated from global requirements  https://review.openstack.org/20225214:44
zzzeekzigo: this has been seen and should have been fixed14:44
openstackgerritMerged openstack/oslo.log: Updated from global requirements  https://review.openstack.org/20071314:44
*** shardy has joined #openstack-oslo14:44
zzzeekzigo: all the test suites have a thing where they go and detect all the FKs in a database to drop them as part of the test teardown14:44
zigozzzeek: In what component has this been fixed then?14:45
dansmithdims_: pbr 1.3 fixed it for me, not sure why it worked for you at 0.11, but oh well14:45
zigoFKs stands for what?14:45
zigoFixtures ? :)14:45
zzzeekzigo: its in the test fixture in the teardown14:45
zigoOk.14:45
dansmithzigo: foreign keys I think :)14:45
zigozzzeek: So, basically, the issue is in the teardown, meaning I can just ignore it?14:45
zigo(for the moment, at least...)14:46
dims_dansmith: cool14:46
dims_haypo: python-memcached is not py34 compatible? (https://github.com/linsomniac/python-memcached/issues/54)14:47
zigodims_: I can answer that one for him ! :)14:48
zigodims_: It's not if you don't apply haypo's patch on top of master.14:49
zigodims_: Upstream for memcached has been famously not reactive at all.14:49
zigoSo we cannot expect anything form him.14:49
zigoHowever, I've packaged master + haypo's patch in Debian already.14:49
dims_zigo: ouch!14:50
dims_zigo: pointer to haypo's patch please?14:50
zigodims_: https://github.com/linsomniac/python-memcached/pull/6714:51
zigodims_: IMO, there's a bunch of upstream which we should blacklist. L'insomniac is one of them, the guys doing requests, and Gabriel (the guy doing sure and others). Plus we should get rid of mysqldb in the favor of pymysql. Though not for a single of these issues I have hope ... :(14:54
*** rpodolyaka1 has quit IRC14:55
*** manishg has joined #openstack-oslo14:57
*** ashleighfarnham has quit IRC14:57
haypodims_, zigo : hey15:00
haypodims_, zigo : i didn't make a choice for python-memcached. there are two options: replace it with pymemcache or fork it15:00
dims_haypo: zigo: please respond on mailing list as this is a openstack-wide issue15:00
dims_i started a thread15:00
haypozigo: FYI a colleague may fork python-ldap to add python3 support. it will also help keystone15:01
zigodims_: What's the thread?15:01
dims_just sent email15:01
haypodims_: ah? i will find the thread15:01
dims_one sec15:01
dims_http://lists.openstack.org/pipermail/openstack-dev/2015-July/069746.html15:01
*** mriedem is now known as mriedem_away15:04
*** e0ne_ is now known as e0ne15:05
*** rpodolyaka1 has joined #openstack-oslo15:07
*** amotoki has joined #openstack-oslo15:10
haypodims_: ok, i replied to your mail15:11
haypodims_: i didn't take a decision yet because it was blocked by other things15:11
*** sdake has quit IRC15:11
*** rpodolyaka1 has quit IRC15:12
*** pblaho has quit IRC15:17
*** sdake has joined #openstack-oslo15:17
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Fix serializer supported version reporting in object_backport  https://review.openstack.org/20264115:19
dims_haypo: thanks15:19
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Remove dict assumption from remotable decorator  https://review.openstack.org/20220015:24
*** e0ne has quit IRC15:26
*** manishg has quit IRC15:26
*** e0ne has joined #openstack-oslo15:26
*** manishg has joined #openstack-oslo15:26
*** e0ne has quit IRC15:27
*** e0ne has joined #openstack-oslo15:27
*** rjaiswal has joined #openstack-oslo15:28
openstackgerritRandall Nortman proposed openstack/oslo.messaging: Restore debug logging of each message in AMQPListener  https://review.openstack.org/19124215:28
*** e0ne has quit IRC15:28
*** ozamiatin has quit IRC15:29
*** tsekiyama has joined #openstack-oslo15:30
*** _________ has joined #openstack-oslo15:30
*** _________ has quit IRC15:31
*** sputnik13 has joined #openstack-oslo15:32
*** e0ne has joined #openstack-oslo15:32
openstackgerritMerged openstack/oslo.cache: Updated from global requirements  https://review.openstack.org/20228815:34
*** ig0r_ has quit IRC15:40
*** rpodolyaka1 has joined #openstack-oslo15:42
*** harlowja_at_home has joined #openstack-oslo15:45
*** shardy_ has joined #openstack-oslo15:45
*** shardy has quit IRC15:46
harlowja_at_homearg, is https://review.openstack.org/#/c/202202/ stuck in no-mans-land, lol15:50
*** shardy_ has quit IRC15:50
harlowja_at_homenm, seems back in the queue, yippe15:51
*** shardy has joined #openstack-oslo15:51
openstackgerritMehdi Abaakouk (sileht) proposed openstack/oslo.messaging: Add notification_pools  https://review.openstack.org/20266615:54
openstackgerritJoshua Harlow proposed openstack/taskflow: Move doc8 to being a normal test requirement in test-requirements.txt  https://review.openstack.org/20239315:58
openstackgerritJoshua Harlow proposed openstack/taskflow: Move doc8 to being a normal test requirement in test-requirements.txt  https://review.openstack.org/20239315:59
openstackgerritIlya Shakhat proposed openstack/oslo.service: Allow LoopingCall to continue on exception in callee  https://review.openstack.org/19360516:00
*** ccrouch has left #openstack-oslo16:03
*** ihrachyshka has quit IRC16:05
*** sdake_ has joined #openstack-oslo16:07
*** ashleighfarnham has joined #openstack-oslo16:07
*** cdent has quit IRC16:08
*** sdake has quit IRC16:10
*** crc32 has joined #openstack-oslo16:11
*** mriedem_away is now known as mriedem16:13
*** harlowja_at_home has quit IRC16:16
*** ihrachyshka has joined #openstack-oslo16:17
*** achanda has joined #openstack-oslo16:28
*** achanda has quit IRC16:29
*** dims_ has quit IRC16:30
*** dims_ has joined #openstack-oslo16:32
*** stevemar_ has quit IRC16:32
*** dims_ has quit IRC16:32
*** achanda has joined #openstack-oslo16:35
*** rpodolyaka1 has quit IRC16:36
*** crc32 has quit IRC16:38
*** crc32 has joined #openstack-oslo16:39
*** ozamiatin has joined #openstack-oslo16:44
*** manishg has quit IRC16:46
*** crc32 has quit IRC16:56
openstackgerritMerged openstack/oslo.service: save docstring, name etc using six.wraps  https://review.openstack.org/20028716:59
*** pasquier-s has quit IRC17:01
*** pasquier-s has joined #openstack-oslo17:08
*** e0ne has quit IRC17:09
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Fix serializer supported version reporting in object_backport  https://review.openstack.org/20264117:12
*** pasquier-s has quit IRC17:15
*** ihrachyshka has quit IRC17:17
*** pasquier-s_ has joined #openstack-oslo17:22
*** achanda has quit IRC17:25
*** mtanino has joined #openstack-oslo17:25
gordci think we need to block oslo.messaging 1.17.x. there seems to be a massive memory leak when consuming off queue.17:28
openstackgerritMerged openstack/stevedore: Updated from global requirements  https://review.openstack.org/20186517:28
gordchttps://bugs.launchpad.net/ceilometer/+bug/147530717:28
openstackLaunchpad bug 1475307 in Ceilometer "oslo.messaging has possible memory leak" [Critical,New]17:28
harlowja_gordc massive memory leak!?17:29
harlowja_wonder whats happening there17:29
haypogordc: you should use tracemalloc!17:29
haypoi'm saying that, and now i'm leaving! bye17:29
*** haypo has left #openstack-oslo17:29
harlowja_lol17:29
gordcharlowja_: when i used 1.17.x, it's growing in memory by ~10MB/min17:29
harlowja_woah, thats alot17:30
harlowja_wonder what tracemalloc or other will show17:30
* gordc will google tracemalloc17:30
harlowja_someone holding onto something17:30
gordci'm just looking at htop right now.17:30
gordcbut when i downgrade to 1.16.0, it seems to be ok.17:31
gordcthe 10MB/min is when i send about 40msg/s17:31
*** yamahata has quit IRC17:31
*** dims_ has joined #openstack-oslo17:32
harlowja_ya, if u can figure out which object/module/other is holding it that'd be a start17:33
*** exploreshaifali has joined #openstack-oslo17:34
*** achanda has joined #openstack-oslo17:35
*** dims_ has quit IRC17:39
*** achanda has quit IRC17:40
*** sputnik13 has quit IRC17:40
*** ashleighfarnham has quit IRC17:40
*** sputnik13 has joined #openstack-oslo17:40
*** bknudson has quit IRC17:45
*** achanda has joined #openstack-oslo17:46
openstackgerritJeremy Stanley proposed openstack-dev/cookiecutter: Stop specifying requirements.txt in tox.ini  https://review.openstack.org/20272117:48
*** ashleighfarnham has joined #openstack-oslo17:50
*** ozamiatin has quit IRC17:51
*** dims_ has joined #openstack-oslo17:52
openstackgerritDoug Hellmann proposed openstack/oslo.messaging: Remove oslo namespace package  https://review.openstack.org/18197417:53
*** dims_ has quit IRC17:53
*** shardy_ has joined #openstack-oslo17:53
dhellmannharlowja_, haypo, dims, bnemec, jd__, gordc: I think we're ready to remove the oslo namespace from oslo.messaging ^^17:54
*** shardy has quit IRC17:54
gordccool cool. /me gets popcorn ready for the release :)17:56
*** dims_ has joined #openstack-oslo17:57
harlowja_ nice17:58
*** shardy_ has quit IRC17:58
*** shardy has joined #openstack-oslo17:59
*** amotoki has quit IRC18:00
*** Kennan has quit IRC18:02
openstackgerritMerged openstack/oslo.versionedobjects: Remove ununsed class in test.py  https://review.openstack.org/20131818:03
*** Kennan has joined #openstack-oslo18:03
zzzeekdims_: thanks for all the effort on that alembic patch !18:06
*** sripriya has joined #openstack-oslo18:07
*** stevemar has joined #openstack-oslo18:08
*** xek has quit IRC18:10
*** isq has quit IRC18:10
*** xek has joined #openstack-oslo18:12
*** isq has joined #openstack-oslo18:13
*** sdake has joined #openstack-oslo18:13
*** achanda has quit IRC18:16
*** sdake_ has quit IRC18:16
openstackgerritNikola Dipanov proposed openstack/oslo.versionedobjects: Fix deserialization with nested backleveling  https://review.openstack.org/20273218:18
*** mtanino has quit IRC18:22
*** crc32 has joined #openstack-oslo18:24
*** achanda has joined #openstack-oslo18:26
*** yassine has quit IRC18:34
dims_zzzeek: very welcome18:37
*** achanda has quit IRC18:38
*** jaosorior has quit IRC18:46
*** mtanino has joined #openstack-oslo18:47
*** dims_ has quit IRC18:51
*** bknudson has joined #openstack-oslo18:59
*** jecarey_ has joined #openstack-oslo19:01
*** jecarey has quit IRC19:02
*** crc32 has left #openstack-oslo19:06
*** sdake_ has joined #openstack-oslo19:08
*** sdake has quit IRC19:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Refactor argument and provider fetching  https://review.openstack.org/18737619:11
*** ozamiatin has joined #openstack-oslo19:12
harlowja_zzzeek i think everyone now confused on https://bitbucket.org/zzzeek/dogpile.cache/pull-request/32/add-a-cachetools-lru-lfu-in-memory-backend/diff#comment-8320635 :-P19:12
harlowja_*or at least some people are, hahaha19:12
harlowja_your decision, i don't mind either way, ha19:13
*** ganso_ has quit IRC19:18
*** bknudson has quit IRC19:19
*** ozamiatin has quit IRC19:47
*** harlowja has joined #openstack-oslo19:49
*** harlowja_ has quit IRC19:49
zzzeekits jsut more for me to have to maintain19:51
zzzeekif i accpted it, now that guy wants the pickle version, more PRs, blah19:51
*** jecarey_ has quit IRC19:52
*** harlowja has quit IRC19:52
*** jecarey has joined #openstack-oslo19:52
*** harlowja has joined #openstack-oslo19:53
*** jecarey_ has joined #openstack-oslo19:53
*** jecarey has quit IRC19:57
*** achanda has joined #openstack-oslo20:03
*** exploreshaifali has quit IRC20:04
*** ozamiatin has joined #openstack-oslo20:07
*** achanda has quit IRC20:09
*** salv-orlando has quit IRC20:11
*** achanda has joined #openstack-oslo20:19
*** sputnik13 has quit IRC20:20
*** sputnik13 has joined #openstack-oslo20:20
gordcharlowja: do you know what https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_executors/impl_pooledexecutor.py#L87 is for?20:20
gordcharlowja: i don't know if that's the reason for memory bump (it seems to be old code). that said, it never gets cleared and stores 10 of thousands of records in minutes.20:22
gordcsileht: ^ for when you wake up20:22
gordcor i guess it's still evning in France.20:22
*** e0ne has joined #openstack-oslo20:27
*** manishg has joined #openstack-oslo20:30
*** dims_ has joined #openstack-oslo20:37
*** salv-orlando has joined #openstack-oslo20:45
openstackgerritMerged openstack/oslo-specs: policy: Dynamic Policies Overlay  https://review.openstack.org/19675320:50
harlowjagordc hmmmm20:56
harlowjaso its used in https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_executors/impl_pooledexecutor.py#L10620:56
harlowjabut i do think i see what u are saying20:56
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Ensure callback variable capture + cleanup is done correctly  https://review.openstack.org/20220221:00
harlowjagordc ^21:00
harlowjacombined that with the other ongoing fixes21:00
dims_hmmm, this is the right kind of pbr! http://pabstblueribbon.com/products/21:01
harlowja:-P21:01
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Ensure callback variable capture + cleanup is done correctly  https://review.openstack.org/20220221:02
harlowjagordc ok, that should do that properly, guess that was another related bug on that code21:03
harlowjahopefully the last one, ha21:04
gordcharlowja: it's weird that didn't seem to be an issue in 1.16... it looks like it exists there too.21:09
*** stevemar has quit IRC21:09
harlowjagordc so the pooled executor probably wasn't as tested now as it is being right now :-P21:09
harlowjaits not the 'base' of all executors21:10
harlowja*its now21:10
harlowjai didn't think it was  before?21:10
harlowjaie https://github.com/openstack/oslo.messaging/commit/0055d79a21:10
silehtgordc, is this responsible of the ceilo memory leak ?21:10
*** ozamiatin has quit IRC21:11
harlowjagordc so that commit unified all the executors (yaaaa!)21:11
harlowjabut also unvealed a few 'bugs'21:11
harlowja*revealed21:11
harlowja*nothing to unexpected there (imho); just the way it goes21:11
gordcsileht: possibly. i'll comment on bug to see if it works21:12
*** stevemar has joined #openstack-oslo21:12
gordcharlowja: i thought the code was taken from eventlet executor (which we've been using in ceilometer). looks like it's from thread.21:12
harlowjagordc yes, thread, but thread was using an executor object to do all the work21:13
harlowjathe executor abstraction (not the oslo.messaging executors, name confusion there) works also for all the other models21:13
gordcharlowja: will give it a try. thanks for looking at it. would've taken me much longer.21:13
harlowjanp21:13
*** rjaiswal has quit IRC21:14
gordclucked out that a bunch of us are doing performance tests right now in ceilometer21:14
harlowja:)21:14
harlowjaeventual bugs, thanks for finding and allowing us to fix them :)21:14
*** e0ne has quit IRC21:16
silehtthanks, see you tomorrow21:16
gordclaters sileht21:16
*** stevemar has quit IRC21:17
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Fully use futurist code-base to abstract concurrent.futures away  https://review.openstack.org/20171621:19
openstackgerritJoshua Harlow proposed openstack/taskflow: Move bulk of storage '__init__' logic to new 'initialize' method  https://review.openstack.org/20279421:27
*** gordc has quit IRC21:33
*** mriedem has quit IRC21:38
*** gordc has joined #openstack-oslo21:42
*** boris-42 has joined #openstack-oslo21:42
openstackgerritJoshua Harlow proposed openstack/futurist: Move doc8 to being a normal test requirement in test-requirements.txt  https://review.openstack.org/20280321:45
*** bnemec has quit IRC21:46
openstackgerritJoshua Harlow proposed openstack/automaton: Ensure doctesting and doc8 testing done in py27 env  https://review.openstack.org/20168721:47
*** ozamiatin has joined #openstack-oslo21:47
elarsonis there a macro for copying parameters between jobs?21:54
openstackgerritMerged openstack/oslo.versionedobjects: Remove dict assumption from remotable decorator  https://review.openstack.org/20220021:54
elarsonsorry, in jenkins job builder21:54
harlowjaelarson not sure, might ask on #openstack-infra ?21:54
*** shardy has quit IRC22:00
*** ozamiatin has quit IRC22:01
*** jecarey_ has quit IRC22:01
gordcharlowja: had to restart, testing patch now. 5 min test seems promising.22:02
harlowjacools22:02
harlowjaya22:02
gordchttps://bugs.launchpad.net/ceilometer/+bug/1475307 you can target that bug if you want.22:03
openstackLaunchpad bug 1475307 in Ceilometer "oslo.messaging has possible memory leak" [Critical,New]22:03
*** ashleighfarnham has quit IRC22:04
*** sputnik13 has quit IRC22:04
*** sputnik13 has joined #openstack-oslo22:04
harlowjakk,22:05
harlowjawill target both :-P22:05
gordcharlowja: works for me22:05
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Ensure callback variable capture + cleanup is done correctly  https://review.openstack.org/20220222:05
harlowjaokie dokie22:06
*** achanda has quit IRC22:10
harlowjalifeless for https://review.openstack.org/#/c/201929/ should i be fine with just 'import mock' instead of trying to even bother using the unittest version of it?22:17
harlowjathink u said thats ok, just want to double-check22:17
*** ashleighfarnham has joined #openstack-oslo22:24
gordcharlowja: i'm going to let it run for a few hours but still looks good. thanks for the help. will review later.22:26
*** gordc has quit IRC22:26
*** vipul is now known as vipoul22:27
harlowja:)22:27
*** vipoul is now known as vipul22:27
*** achanda has joined #openstack-oslo22:27
lifelessharlowja: yes22:31
harlowjalifeless k22:32
harlowjawill update22:32
*** mriedem has joined #openstack-oslo22:32
openstackgerritJoshua Harlow proposed openstack/taskflow: Just use 'import mock' for all mock versions  https://review.openstack.org/20192922:34
openstackgerritJoshua Harlow proposed openstack/taskflow: Just use 'import mock' for all mock versions  https://review.openstack.org/20192922:34
*** achanda has quit IRC22:41
*** klindgren has left #openstack-oslo22:46
*** hogepodge has quit IRC23:02
*** zzzeek has quit IRC23:08
*** achanda has joined #openstack-oslo23:09
*** sdake_ has quit IRC23:12
*** salv-orlando has quit IRC23:16
openstackgerritOpenStack Proposal Bot proposed openstack/oslo-incubator: Updated from global requirements  https://review.openstack.org/20282723:19
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/20282823:19
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.versionedobjects: Updated from global requirements  https://review.openstack.org/20228923:20
*** zigo has quit IRC23:21
*** zigo has joined #openstack-oslo23:22
openstackgerritJoshua Harlow proposed openstack/taskflow: Add + use docstring retaining 'abc.ABCMeta' metaclass base  https://review.openstack.org/20283123:23
*** dimtruck is now known as zz_dimtruck23:25
openstackgerritMerged openstack/oslo.utils: Updated from global requirements  https://review.openstack.org/20270723:27
*** openstackstatus has joined #openstack-oslo23:34
*** ChanServ sets mode: +v openstackstatus23:34
*** manishg_ has joined #openstack-oslo23:36
*** manishg has quit IRC23:36
*** jaypipes has quit IRC23:50
*** barra204 has joined #openstack-oslo23:54
*** mriedem1 has joined #openstack-oslo23:55
*** mriedem has quit IRC23:55

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