Monday, 2014-12-08

« Sunday, 2014-12-07 Index Tuesday, 2014-12-09 »
*** harlowja_at_home has joined #openstack-oslo00:02
*** e0ne has joined #openstack-oslo00:08
*** alexpilotti has quit IRC00:08
*** e0ne has quit IRC00:10
*** e0ne has joined #openstack-oslo00:14
*** e0ne has quit IRC00:19
*** e0ne has joined #openstack-oslo00:23
*** e0ne has quit IRC00:26
*** e0ne has joined #openstack-oslo00:30
*** oomichi has joined #openstack-oslo00:35
*** stevemar has quit IRC00:40
*** e0ne has quit IRC00:49
openstackgerritJoshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module  https://review.openstack.org/10324500:50
openstackgerritJoshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module  https://review.openstack.org/10324500:52
*** boris-42 has quit IRC00:52
*** e0ne has joined #openstack-oslo00:53
*** e0ne has quit IRC01:06
*** e0ne has joined #openstack-oslo01:09
*** e0ne has quit IRC01:11
*** git-harry has quit IRC01:12
*** git-harry has joined #openstack-oslo01:12
*** arnaud__ has joined #openstack-oslo01:16
openstackgerritJoshua Harlow proposed openstack/taskflow: Use condition variables using 'with'  https://review.openstack.org/13989501:17
openstackgerritJoshua Harlow proposed openstack/taskflow: Use condition variables using 'with'  https://review.openstack.org/13989501:21
*** harlowja_at_home has quit IRC01:37
*** stevemar has joined #openstack-oslo02:05
openstackgerritJoshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module  https://review.openstack.org/10324502:39
openstackgerritJoshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module  https://review.openstack.org/10324502:43
*** ihrachyshka has quit IRC03:21
*** sabari is now known as zz_sabari03:31
*** david-lyle_afk has quit IRC03:31
*** zz_sabari is now known as sabari03:31
*** sabari is now known as zz_sabari03:36
*** miqui_ has quit IRC03:40
*** amotoki__away is now known as amotoki03:41
*** david-lyle_afk has joined #openstack-oslo03:43
*** david-lyle_afk has quit IRC03:50
*** arnaud__ has quit IRC04:00
*** zz_sabari is now known as sabari04:02
*** david-lyle_afk has joined #openstack-oslo04:02
*** david-lyle_afk has quit IRC04:02
*** sabari is now known as zz_sabari04:07
*** dimsum__ has quit IRC04:11
*** arnaud__ has joined #openstack-oslo04:15
*** jecarey has joined #openstack-oslo04:20
*** ishant has joined #openstack-oslo04:28
*** ishant has quit IRC04:28
*** ckmvishnu has joined #openstack-oslo04:29
*** zz_sabari is now known as sabari04:45
*** jecarey has quit IRC04:53
*** david-lyle_afk has joined #openstack-oslo05:01
*** dimsum__ has joined #openstack-oslo05:11
*** dimsum__ has quit IRC05:17
*** arnaud__ has quit IRC05:18
*** arnaud__ has joined #openstack-oslo05:30
*** arnaud__ has quit IRC05:32
*** arnaud__ has joined #openstack-oslo05:34
*** arnaud__ has quit IRC05:38
*** arnaud__ has joined #openstack-oslo05:41
*** arnaud__ has quit IRC05:42
*** rpodolyaka1 has joined #openstack-oslo06:01
*** k4n0 has joined #openstack-oslo06:01
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/13656606:13
*** ckmvishnu has quit IRC06:15
*** vigneshvar has joined #openstack-oslo06:38
*** liusheng has joined #openstack-oslo06:39
*** liusheng has quit IRC06:51
*** liusheng has joined #openstack-oslo06:52
*** stevemar has quit IRC07:03
*** arnaud__ has joined #openstack-oslo07:06
*** subscope has joined #openstack-oslo07:40
*** i159 has joined #openstack-oslo07:48
*** tsufiev has quit IRC07:58
*** mfedosin_ has quit IRC07:59
*** dteselkin has quit IRC07:59
*** liusheng has quit IRC08:02
*** liusheng has joined #openstack-oslo08:03
*** ihrachyshka has joined #openstack-oslo08:08
*** mfedosin has joined #openstack-oslo08:10
*** dteselkin has joined #openstack-oslo08:10
*** rpodolyaka1 has quit IRC08:11
*** tsufiev has joined #openstack-oslo08:14
*** pblaho has joined #openstack-oslo08:42
*** gothicmindfood has quit IRC08:48
*** exploreshaifali has joined #openstack-oslo08:50
*** arnaud__ has quit IRC08:51
*** arnaud__ has joined #openstack-oslo08:52
openstackgerritSabari proposed openstack/oslo.vmware: Update urllib3 requirements  https://review.openstack.org/13929608:52
openstackgerritFrode Nordahl proposed openstack/oslo.messaging: Enable IPv6-support in libzmq by default  https://review.openstack.org/13807508:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.vmware: Updated from global requirements  https://review.openstack.org/13994508:58
*** exploreshaifali has quit IRC09:12
*** dimsum__ has joined #openstack-oslo09:15
*** gothicmindfood has joined #openstack-oslo09:15
*** dimsum__ has quit IRC09:20
*** oomichi has quit IRC09:22
*** dtantsur|afk is now known as dtantsur09:34
*** arnaud__ has quit IRC09:38
*** e0ne has joined #openstack-oslo09:48
*** sabari is now known as zz_sabari09:48
*** e0ne has quit IRC10:04
*** e0ne has joined #openstack-oslo10:06
openstackgerritIhar Hrachyshka proposed openstack/oslo-incubator: Delete graduated concurrency files  https://review.openstack.org/12279610:15
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging functional tests proposal  https://review.openstack.org/13996410:41
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging functional tests proposal  https://review.openstack.org/13996410:43
*** andreykurilin_ has joined #openstack-oslo10:44
openstackgerritOleksii Zamiatin proposed openstack/oslo-specs: oslo.messaging functional tests proposal  https://review.openstack.org/13996410:48
*** noelbk has joined #openstack-oslo11:17
*** ihrachyshka has quit IRC11:25
*** ihrachyshka has joined #openstack-oslo11:29
*** noelbk has quit IRC11:39
*** dimsum__ has joined #openstack-oslo11:47
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: more precise iterconsume timeout  https://review.openstack.org/13998011:59
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: qpid: honor iterconsume timeout  https://review.openstack.org/13998111:59
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: The executor don't need the set the timeout  https://review.openstack.org/13998211:59
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: The executor doesn't need to set the timeout  https://review.openstack.org/13998212:00
ihrachyshkadimsum__: hey. did you have a change to fix the bug in oslo.concurrency with substitutions?12:04
dimsum__ihrachyshka: hey, i have a small test case to reproduce the situation, don't have an actual fix yet12:04
*** dimsum__ is now known as dims12:05
ihrachyshkadims: yeah, I actually wanted to start from a test case for that :) is it for oslo.concurrency?12:10
dimseven more basic... in oslo.config - http://paste.openstack.org/show/147086/12:11
ihrachyshkadims: ah yeah. iiuc there is no obvious fix for that12:12
ihrachyshkadims: unless we introduce a special case for 'accessing configs via substitute'12:13
*** andreykurilin_ has quit IRC12:14
*** viktors|afk is now known as viktors12:48
*** takedakn has joined #openstack-oslo12:51
*** amotoki has quit IRC12:54
*** vigneshvar has quit IRC12:58
*** exploreshaifali has joined #openstack-oslo13:03
*** alexpilotti has joined #openstack-oslo13:08
*** bknudson has quit IRC13:08
*** ihrachyshka_ has joined #openstack-oslo13:22
*** ihrachyshka has quit IRC13:23
*** kgiusti has joined #openstack-oslo13:36
*** amrith is now known as _amrith_13:36
*** gordc has joined #openstack-oslo13:38
dimsihrachyshka_: seems to go against the design of the cfg filter unfortunately13:42
ihrachyshka_dims: yes, that's why I said that the whole idea seems to be contrary to substitute feature13:43
ihrachyshka_which is essential13:43
dimsihrachyshka_: y13:44
*** jeckersb_gone is now known as jeckersb13:47
ihrachyshka_dims: I'll raise that on the meeting today13:49
ihrachyshka_dims: for oslo.concurrency, I would just go with switching back to cfg.CONF13:50
ihrachyshka_dims: thoughts?13:50
dimslet's talk to dhellmann, who should be here soon...ish13:51
*** pblaho has quit IRC14:06
*** pblaho has joined #openstack-oslo14:08
*** mjturek has joined #openstack-oslo14:23
*** mriedem has joined #openstack-oslo14:35
*** jaypipes has joined #openstack-oslo14:40
openstackgerritChuck Short proposed openstack/oslo.serialization: Fix strings for python3  https://review.openstack.org/14002014:41
*** takedakn has quit IRC14:48
*** zzzeek has joined #openstack-oslo14:50
*** pradk has joined #openstack-oslo15:00
*** bknudson has joined #openstack-oslo15:00
*** arnaud__ has joined #openstack-oslo15:02
dhellmannviktors & zzzeek : can you review the alembic runner spec again, please? https://review.openstack.org/#/c/112842/1015:04
zzzeekhookay15:05
dhellmannit seems like it's a lot simpler than the original version, but you both understand what we need better than I do15:05
*** _amrith_ is now known as amrith15:06
*** arnaud__ has quit IRC15:07
*** pblaho_ has joined #openstack-oslo15:10
viktorsdhellmann: ok, will look15:14
*** exploreshaifali has quit IRC15:14
dhellmannzzzeek, viktors: thanks15:14
*** pblaho has quit IRC15:14
*** jecarey has joined #openstack-oslo15:19
*** gordc has quit IRC15:20
*** gordc has joined #openstack-oslo15:23
*** e0ne has quit IRC15:28
viktorszzzeek: hi!15:28
dhellmannharlowja_away: did you release taskflow last week?15:30
zzzeekviktors: hey viktors did u see my reviews15:30
viktorszzzeek: yes, as for issue with OperationalError, which was mentioned on Friday - I tested with SQLA from master, and DBConnectionError was raised properly.15:31
viktorszzzeek: thanks a lot!15:31
zzzeekviktors: yes but also the gerrit makes it work in 0.915:31
zzzeekviktors: they are all V+1 so lets push them up15:31
viktorszzzeek: I've tested your changes to oslo.db and I got same issue, unfortunately, can you please check it on your side?15:32
zzzeekviktors: OK15:32
viktorszzzeek: my test was looks like this - http://paste.openstack.org/show/147249/15:32
*** ihrachyshka_ has quit IRC15:33
*** arnaud__ has joined #openstack-oslo15:35
*** e0ne has joined #openstack-oslo15:39
*** k4n0 has quit IRC15:40
*** arnaud__ has quit IRC15:42
dhellmannreminder, meeting starting in 15 minutes15:45
*** zz_jgrimm is now known as jgrimm15:45
openstackgerritJulien Danjou proposed openstack/tooz: memcached: add support for group deletion  https://review.openstack.org/14003715:47
openstackgerritJulien Danjou proposed openstack/tooz: tests: minor code simplification  https://review.openstack.org/14003815:47
*** arnaud__ has joined #openstack-oslo15:47
openstackgerritJulien Danjou proposed openstack/tooz: memcached: add support for _destroy_group  https://review.openstack.org/14003915:47
openstackgerritQin Zhao proposed openstack/oslo.concurrency: Move retrying dependency to version 1.2.3  https://review.openstack.org/14004015:48
*** amotoki has joined #openstack-oslo15:51
zzzeekviktors: that test case is a different one.  for that one I’m not sure we have a workaround short of upgrading to SQLAlhcemy 1.015:51
zzzeekviktors: that’s the same SQLA issue but that’s not the “ping” handler, there is no opportunity for us to patch around that _revalidate_connection15:51
*** rpodolyaka1 has joined #openstack-oslo15:52
zzzeekviktors: this is because you are running on the same checked-out connection, rather than checking out fresh15:52
zzzeekviktors: in nova api etc. it wouldn’t usually look like this, it would be on a new API hit and the ping handler would be hitting it15:52
*** kbyrne has quit IRC15:55
*** ihrachyshka_ has joined #openstack-oslo15:57
*** mtanino has joined #openstack-oslo15:57
*** jaypipes has quit IRC15:59
viktorszzzeek: got it, I should check your changes with the real DBAPI hits instead of executing "select 1" on connections15:59
zzzeekviktors: im looking to see if i can also wrap the pool connection function in < SQLA 1.016:00
*** kbyrne has joined #openstack-oslo16:00
zzzeekviktors: because yes, when it first connects, we aren’t catching that at all16:00
zzzeekviktors: if this is critical for SQLA 0.9 i can try to patch into the actual connect fn16:00
viktorszzzeek: I'm not really sure, is it critical, we need to check some usecases16:01
viktorszzzeek: btw, it's meeting time )16:01
zzzeekviktors: yes16:01
*** ihrachyshka_ is now known as ihrachyshka16:02
*** harlowja_at_home has joined #openstack-oslo16:03
zzzeekviktors: it’s also not working in SQLA 0.8 :(  http://logs.openstack.org/38/139738/3/check/gate-oslo.db-tox-sqla_08/06edb8d/console.html16:04
*** david-lyle_afk is now known as david-lyle16:06
*** tsekiyama has joined #openstack-oslo16:10
*** gordc has quit IRC16:10
viktorszzzeek: :(16:13
zzzeekviktors: is there a bug report illusrating what ppl definitely need here?16:13
zzzeeki can keep patching this in, not a problem16:13
*** arnaud__ has quit IRC16:13
zzzeeki forgot to make it work on 0.8 i see why that is16:14
*** gordc has joined #openstack-oslo16:15
*** gordc has quit IRC16:15
viktorszzzeek: do know, when we should bump SQLA version in global requirements?16:18
zzzeekviktors: im happy bumping to 0.9, if we haven’t already16:19
viktorszzzeek: we haven't - see https://github.com/openstack/requirements/blob/master/global-requirements.txt#L14016:19
zzzeekviktors: i dont know whats involved, but I’m +1 to bump16:19
openstackgerritQin Zhao proposed openstack/oslo.concurrency: Move retrying dependency to version 1.2.3  https://review.openstack.org/14004016:21
*** exploreshaifali has joined #openstack-oslo16:23
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.db: Updated from global requirements  https://review.openstack.org/14005816:23
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: kombu: fix driver loading with kombu+qpid scheme  https://review.openstack.org/14005916:23
*** gordc has joined #openstack-oslo16:23
*** jaypipes has joined #openstack-oslo16:25
viktorszzzeek: the fresh one - https://review.openstack.org/#/c/139880/ :)16:30
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: kombu: fix driver loading with kombu+qpid scheme  https://review.openstack.org/14005916:30
zzzeekviktors: wow that just came out ?16:30
viktorszzzeek: yes16:31
zzzeekviktors: ok how does this work, this requirements file gets merged into all projects ?16:31
viktorszzzeek: openstack bot will sync this requirement by projects16:32
zzzeekviktors: woop!16:32
zzzeekI can yank a crapload out when that hits !16:32
zzzeekviktors: need a +2 for https://review.openstack.org/#/c/139725/ and you probably want https://review.openstack.org/#/c/139733/16:32
viktors:-D16:33
viktorszzzeek: ok, will do16:33
viktorszzzeek: you probably want to +2A this https://review.openstack.org/#/c/140058/16:35
zzzeekviktors: done16:36
*** amrith is now known as _amrith_16:37
openstackgerritMerged openstack/oslo.vmware: Update urllib3 requirements  https://review.openstack.org/13929616:42
*** arnaud__ has joined #openstack-oslo16:44
*** jgrimm has quit IRC16:44
*** i159 has quit IRC16:45
*** arnaud__ has quit IRC16:47
*** exploreshaifali has quit IRC16:49
*** amotoki has quit IRC16:53
*** pblaho_ has quit IRC16:55
zzzeekdhellmann: suppose oslo.db requirements.txt says: “sqlalchemy >= 0.9.7”17:02
zzzeekdhellmann: then, in that same oslo.db revision, we look at:  https://github.com/openstack/oslo.db/blob/master/oslo/db/sqlalchemy/compat/handle_error.py#L3917:02
zzzeekdhellmann: under what circumstances would that conditional be False ?17:02
*** harlowja_at_home has quit IRC17:02
zzzeekthe source to “utils.sqla_097” is: https://github.com/openstack/oslo.db/blob/master/oslo/db/sqlalchemy/compat/utils.py17:03
dhellmannzzzeek: the question isn't so much about removing that code, but about actually being able to say that the requirements can change to sqlalchemy >= 0.9717:04
*** sigmavirus24_awa is now known as sigmavirus2417:04
zzzeekdhellmann: that’s outside of my wheelhouse :)   that decision is based on what I see at https://review.openstack.org/#/c/139880/ and at what point that gets propagated17:05
dhellmannzzzeek: yeah, I need you to learn about and understand the requirements stuff -- you can't shrug it off17:05
dhellmannI cannot be the only one in Oslo who knows how the requirements updates are managed and what implications changing requirements has on our project17:05
zzzeekdhellmann: it has to do with downstream vendors and packaging17:05
dhellmannno, that is not all17:05
dhellmannit also has to do with the way upgrades are tested17:06
dhellmanngrenade installs everything, runs some tests, upgrades things, then runs more tests17:06
*** zz_sabari is now known as sabari_17:06
dhellmannit also ensures that the requirements overlap between the old version and the new version of the WHOLE system17:06
dhellmannthe packagers have some say about when we can bump requirements, but their voices are not the only voices with say17:07
zzzeekdhellmann: wouldnt that grenade test be a simple block for such a change ?17:07
zzzeekdhellmann: e.g. i propose library >= X.  grenade fails because project Q needs library < X.17:07
dhellmannyes, but the point of us having this conversation is to ensure that you understand that the test exists, and to make sure you're looking at how it works17:08
dhellmannit is quite likely that everything is technically OK17:08
zzzeekdhellmann: OK.   im not following what you want me to be doing here, I’m not proposing any changes to requirements.txt.  are you asking that I be involved in deciding when requirements.txt is safe to be bumped?17:09
dhellmannit wasn't clear from what you said in the meeting if the requirements change had already merged, so I wanted to make sure you understood the whole process for that update17:09
dhellmannsince the change has merged, that means some version of grenade was able to run the upgrade test with that version of the lib, and so if oslo.db has that as a minimum it should be safe to remove the obsolete code17:10
*** sabari_ is now known as zz_sabari_17:10
zzzeekdhellmann: yes, that discussion was only about, once oslo.db/requriements.txt has a version in it, the code immediately desecending from that file in that revision can assume that version17:11
dhellmannyes, I believe that is safe17:12
*** viktors is now known as viktors|afk17:16
*** openstackgerrit has quit IRC17:19
*** openstackgerrit has joined #openstack-oslo17:19
*** noelbk has joined #openstack-oslo17:20
dhellmanndims: it looks like most of the pre-release code work for oslo.context is done, but there are some project management tasks to do: https://blueprints.launchpad.net/oslo-incubator/+spec/graduate-oslo-context17:25
dhellmanndims: some of that may be done, I only marked off the things I was sure of17:26
dimsdhellmann: if you are happy with the code, i can work on the rest17:26
*** rharwood has joined #openstack-oslo17:29
*** ViswaV has joined #openstack-oslo17:31
rharwoodhey all, oslo.messaging question.  Looking at https://github.com/openstack/oslo.messaging/blob/a7f3ecb36041538026de8f095db773afec3534fc/oslo/messaging/_drivers/impl_rabbit.py#L70-72 suggests that rabbit_hosts should be set to `$rabbit_host:$rabbit_port` in the single-host case.  Is that correct?17:31
dhellmanndims: yeah, I think this is good for a first version17:32
dhellmannrharwood: that's the default, so you shouldn't have to set it explicitly if that's what you mean17:33
rharwooddhellmann: so it default to something with dollar signs in it, or does it do the substitution of the rabbit_host and rabbit_port values into that expression?17:35
dhellmannrharwood: oh, $foo substitutes the value of foo in that place17:35
*** ViswaV has quit IRC17:35
rharwooddhellmann: great, thanks!17:36
*** ViswaV has joined #openstack-oslo17:36
*** e0ne has quit IRC17:46
*** stevemar has joined #openstack-oslo17:52
openstackgerritMerged openstack/oslo.db: Updated from global requirements  https://review.openstack.org/14005817:59
*** _amrith_ is now known as amrith18:04
*** ViswaV has quit IRC18:04
*** ViswaV has joined #openstack-oslo18:06
openstackgerritgordon chung proposed openstack/oslo-specs: messaging: notification dispatcher filter  https://review.openstack.org/13537918:12
*** denis_makogon has joined #openstack-oslo18:16
*** harlowja_away is now known as harlowja18:20
*** e0ne has joined #openstack-oslo18:23
*** mriedem has quit IRC18:28
*** andreykurilin_ has joined #openstack-oslo18:33
*** e0ne has quit IRC18:43
*** jaosorior has joined #openstack-oslo18:45
*** e0ne has joined #openstack-oslo18:49
*** vigneshvar has joined #openstack-oslo18:52
openstackgerritJoshua Harlow proposed openstack/taskflow: Standardize bases in the pluggable modules  https://review.openstack.org/13974118:53
dimsdhellmann: pushed 0.1.0, should i add it to global requirements?19:05
palendaedhellmann: You stated last week for https://bugs.launchpad.net/python-cliff/+bug/1379055 you were thinking about moving to cmd instead of cmd2 for - were you thinking of recreating any cmd2 features?19:05
dimsdhellmann: updated the work items in the blueprint page as well19:05
palendaeOr simply dropping those?19:05
*** raildo has joined #openstack-oslo19:07
*** dtantsur is now known as dtantsur|afk19:07
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add a deprecation helper/utility  https://review.openstack.org/14011919:10
harlowjadims let me know what u think of ^19:10
harlowjathe talk of deprecation and helpers and such makes me think others need that19:10
harlowjaor at least a common utility to provide common deprecation patterns...19:11
*** rpodolyaka2 has joined #openstack-oslo19:11
dimsharlowja: we have a @deprecated in oslo-incubator/openstack/common/versionutils.py - fyi19:13
harlowjayes, thats not the same though19:13
dimsharlowja: y looking at yours now.19:13
harlowjait uses pyhton logging, not what i think is the more common warnings module (which explicily is i think made for this stuff)19:13
harlowja* aka https://docs.python.org/2/library/exceptions.html#exceptions.DeprecationWarning19:14
harlowjaand such19:14
harlowjanot saying the versionutils isn't useful, its just umm, different19:14
harlowja* https://docs.python.org/2/library/warnings.html#warning-categories19:15
harlowjai sorta think the versionutils one is ok for applications (nova, glance...) but for libraries they should prefer the warnings/depreciation warnings path19:15
dimsharlowja: yes, this will definitely be handy19:18
harlowjaya, might as well provide some common patterns if we can19:19
harlowjathey may not be perfect, but i've found them usful19:19
dimsharlowja: what i would really love to have...is something like jdiff - http://javadiff.sourceforge.net/ so i can browse what changed and drill down into each class/method/parameter19:20
dimshaven't found one yet :(19:20
harlowjainteresting19:20
harlowjadoes sound nice19:21
dimstoo bad python is dynamic :)19:21
harlowja;)19:22
harlowjai hear thats becoming more and more blurry :(19:22
harlowjadoesn't python 3 have type annotations or something19:23
* dims is clueless19:23
harlowja* http://www.infoq.com/news/2014/08/python-type-annotation-proposal19:23
harlowja:-/19:23
harlowjadef fib(n: int) -> None:19:24
harlowjai think http://legacy.python.org/dev/peps/pep-3107/ already exists19:24
harlowjathat would ummm, be interesting19:25
harlowjazzzeek would u be ok with a dogpile backend that uses pymemcache (i see there is one there for memcached  and Pylibmc but not for pymemcache) * https://pypi.python.org/pypi/pymemcache19:27
harlowja* https://pypi.python.org/pypi/pymemcache#comparison-with-other-libraries19:28
harlowjadims the other neat things about the warnings module is https://docs.python.org/2/library/logging.html#logging.captureWarnings19:29
harlowjaso if people really still want to use logging (like the versionutils one), then sure they can19:29
*** zzzeek has quit IRC19:29
dimsharlowja: nice! would be handy for the gate/check19:30
harlowjaya19:30
*** vigneshvar has quit IRC19:31
harlowjaalthough afaik the gate has warnings on19:31
harlowjai believe19:31
*** liusheng has quit IRC19:32
*** liusheng has joined #openstack-oslo19:32
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add a deprecation helper/utility  https://review.openstack.org/14011919:32
harlowjai can add some tests for that stuff to if desired19:32
*** zzzeek has joined #openstack-oslo19:32
harlowjathere is a way to locally capture all the warnings19:33
harlowja* https://docs.python.org/2/library/warnings.html#testing-warnings19:33
*** zz_jgrimm has joined #openstack-oslo19:33
harlowjawith warnings.catch_warnings(record=True) or something19:33
zzzeekharlowja: sure, but also you can make a backend for that and use it externally too, doesnt need to be committed for you to use it19:33
harlowjakk19:33
harlowjafair nuff19:33
zzzeekis that a newer backend19:33
*** andreykurilin_ has quit IRC19:33
zzzeeknot sure why i dont have that already19:33
harlowjanewer library i think19:33
harlowjajd__ is the expert in it, ha19:34
harlowjatooz uses that19:34
harlowjabut other projects in openstack use python-memcached so therein lies the conundrum19:34
harlowjaand global requirements approvers don't like dups19:35
harlowja*for better or worse19:35
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add a deprecation helper/utility  https://review.openstack.org/14011919:40
*** zz_jgrimm is now known as jgrimm19:40
*** vigneshvar has joined #openstack-oslo19:42
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add deprecation + reflection auto documentation  https://review.openstack.org/14012919:46
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a moved_inheritable_class deprecation helper  https://review.openstack.org/14013219:54
openstackgerritJoshua Harlow proposed openstack/taskflow: Standardize bases in the pluggable modules  https://review.openstack.org/13974119:55
dhellmannharlowja: how does https://review.openstack.org/#/c/140119/3 related to versionutils?19:59
harlowjanot related, imho different usages19:59
harlowjathe versionutils one is tightly coupled to logging and the specific depreciation logging there, this one isn't20:00
harlowjawhich i feel isn't appropriate for libraries to use (since i feel its not the common practice to do that)20:00
harlowja140119 uses the warnings module (which can be plugged into logging if so chosen)20:01
*** stannie1 has joined #openstack-oslo20:01
harlowjaand uses the common https://docs.python.org/2/library/exceptions.html#exceptions.DeprecationWarning and such20:01
harlowjaso that libraries can follow the more standard practices here (vs the versionutils one)20:02
dhellmannok20:02
harlowjai think the version utils one ok for applications that have used it and such, those that follow openstack releases... (its got its usages),20:02
harlowjathats my 2 cents20:02
dhellmannI don't know how I feel about having 2 versions of this sort of thing, though20:03
dhellmannI like some of the features of this new one20:03
harlowjai feel the newer one is the right way, but i might be just to opinonated ;)20:03
harlowjavs using the logging module to replicate some of the warnings module20:03
dhellmannyeah, we we discussed today, we can't get rid of the old one because that API is what people are using now.20:03
harlowjathe warning module shows location of code that is depreciated and such20:03
harlowjai'm not suggesting get rid of20:03
dhellmannI guess in this case we're warning developers and in the other case we're warning deployers20:04
harlowjabut 2 likely have to coexist at some point if u ever want to move things away to something else...20:04
harlowjaya, maybe at that point its just a usage documentation20:04
dhellmannyeah20:04
harlowjaversionutils for deployers, warnings (or variation of 140119) for deveopers20:04
harlowjadepending on who u are targeting20:04
harlowjaex @ http://paste.ubuntu.com/9431427/20:07
harlowjabut probably u've already seen similar stuff20:07
harlowjawarnings shows u the code location20:07
harlowjawhich deployers don't care about20:07
harlowjaso perhaps thats how these co-exist20:07
harlowjawarnings already have the 'warn once' stuff built-in, it looks like oslo.log rebuilt that20:08
harlowja*or rebuilt something similar20:09
dhellmannyeah, I get that it's not a perfect thing, but we have to keep supporting it while people are using it20:11
dhellmannharlowja: https://blueprints.launchpad.net/oslo-incubator/+spec/fix-import-cycle-log-and-versionutils20:12
harlowjasure so perhaps that can just be fixed by using the warnings interconnect, idk20:13
harlowjahttps://docs.python.org/2/library/logging.html#logging.captureWarnings20:13
harlowjai'm not saying not support the other mechanism, its just they aren't the same usecases and not really the same result20:13
dhellmannyeah, I don't know if we want all deprecations logged20:13
dhellmannright, that much is clear now :-)20:14
harlowjakk20:14
*** subscope_ has joined #openstack-oslo20:14
harlowjai believe libraries like sqlalchemy, and many others already use the warnings library, so it'd seem nice to use it for when its appropriate20:14
dhellmannYeah, the question is when is it appropriate? We only want to log warnings for things that deployers can fix. Otherwise, having warnings show up in unit test output, or even have them tripped as errors at some point in tests, brings them to the attention of developers.20:15
harlowjasure, so maybe big warnings in unit test output20:17
harlowjawith red blinky colors20:17
*** mriedem has joined #openstack-oslo20:17
harlowjathen turn that off on release20:17
harlowjaor yellow blinky (i'm not sure, can't decide)20:17
dhellmannheh20:17
*** raildo has quit IRC20:18
harlowjaanyways, something to think about; i find it useful, for developer <-> developer kind of stuff, that avoids breaking crap20:21
harlowjaor at least break crap in a more elegant manner, ha20:21
harlowjamaybe just turn on https://docs.python.org/2/library/logging.html#logging.captureWarnings in the CI system, turn it off outside of it20:23
*** noelbk has quit IRC20:23
harlowjahave things that showup as 'py.warnings' blinky red/yellow (or maybe red/green to match the season)20:23
*** alexpilotti has quit IRC20:27
*** e0ne has quit IRC20:32
harlowjaex, http://paste.ubuntu.com/9431665/ could be something that is plugged in (does not include blinky colors)20:32
*** rpodolyaka2 has quit IRC20:45
openstackgerritJoshua Harlow proposed openstack/taskflow: Add and use a new simple helper logging module  https://review.openstack.org/12300720:47
*** jamespage_ has joined #openstack-oslo20:52
*** e0ne has joined #openstack-oslo20:59
*** jamespage_ has quit IRC21:00
openstackgerritDoug Hellmann proposed openstack/oslo.config: Raise TemplateSubstitutionError from _substitute  https://review.openstack.org/14014321:00
*** e0ne has quit IRC21:02
palendaedhellmann: So I'm looking at removing the cmd2 dependency in cliff, and it appears right now interactive.py expects arguments passed around to be cmd2 classes (like ParsedString)21:02
palendaedhellmann: Were you thinking about pulling in some of the cmd2 logic/classes when dropping back to stdlib cmd?21:03
*** amrith is now known as _amrith_21:03
*** jamespage_ has joined #openstack-oslo21:06
*** e0ne has joined #openstack-oslo21:06
*** kgiusti has left #openstack-oslo21:09
dhellmannpalendae: let me look at that code21:10
palendaedhellmann: http://git.openstack.org/cgit/openstack/cliff/tree/cliff/interactive.py#n101 specifically21:10
dhellmannpalendae: oh, cmd supports an interactive mode, so we might just have to update to use cmd instead of cmd221:11
palendaedhellmann: Right, I actually just did that - however, the `statement` variable is  cmd2.ParsedString21:11
dhellmannright, I think in cmd you'll just get a string21:11
palendaeIf I simply change cmd2 -> cmd, I just get a str21:11
palendaeWasn't sure if you planned on replicating any of cmd2's features21:12
dhellmannI was not planning to, no21:12
palendaeI'm ok with writing a spec for it, just want to make sure I'm on the same general page21:12
dhellmannhave you contacted catherine about a release?21:12
palendaeI have not21:12
dhellmannpalendae: yeah, it's been a while since I've thought about this and apparently I didn't write down the plan21:12
palendaeI'm kind of picking this up from sigmavirus24; but I can certainly reach out to her21:12
palendaeCame from https://bugs.launchpad.net/python-cliff/+bug/137905521:13
dhellmannright21:13
sigmavirus24palendae: I'll assign that to you if you haven't already reassigned it21:13
palendaesigmavirus24: Go for it21:13
sigmavirus24palendae: lp id?21:13
palendaesigmavirus24: Actually I just did it21:14
sigmavirus24good21:14
dhellmannpalendae: oh, you know, she might have given me some rights on that repo the last time we had an issue, hang on21:14
*** mriedem1 has joined #openstack-oslo21:15
dhellmannpalendae: it looks like I can merge changes but not make releases on pypi21:16
*** mriedem has quit IRC21:17
dhellmannpalendae, sigmavirus24 : https://bitbucket.org/catherinedevlin/cmd2/pull-request/6/in-the-event-which-is-not-installed-return/diff is merged21:17
sigmavirus24dhellmann: awesome!21:18
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.log: Switch to oslo.context  https://review.openstack.org/14014721:18
palendaedhellmann: Thanks21:18
dhellmannpalendae, sigmavirus24 : so now we just need a release. I'll email Catherine.21:18
palendaedhellmann: Cool21:18
sigmavirus24:thumbsup:21:18
*** denis_makogon has quit IRC21:20
*** andreykurilin_ has joined #openstack-oslo21:22
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo-incubator: Switch over to oslo.context  https://review.openstack.org/14014921:23
*** e0ne has quit IRC21:29
*** e0ne has joined #openstack-oslo21:34
*** jecarey has quit IRC21:39
*** subscope_ has quit IRC21:48
dhellmanndims: you're tearing through the steps for creating oslo.context! :-)21:54
*** jgrimm is now known as zz_jgrimm21:57
*** zz_jgrimm is now known as jgrimm21:58
dimsdhellmann: :) muscle memory now21:58
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add a deprecation helper/utility  https://review.openstack.org/14011921:59
*** noelbk has joined #openstack-oslo21:59
*** boris-42 has joined #openstack-oslo22:00
*** pradk has quit IRC22:04
*** jamespage_ has quit IRC22:05
openstackgerritayoung proposed openstack/oslo-specs: Graduate Policy  https://review.openstack.org/14016122:05
*** _amrith_ is now known as amrith22:10
*** oomichi has joined #openstack-oslo22:11
openstackgerritJoshua Harlow proposed openstack/taskflow: Add and use a new simple helper logging module  https://review.openstack.org/12300722:17
*** jeckersb is now known as jeckersb_gone22:20
openstackgerritMerged openstack/taskflow: Use condition variables using 'with'  https://review.openstack.org/13989522:25
openstackgerritJoshua Harlow proposed openstack/taskflow: Add appropriate links into README.rst  https://review.openstack.org/14016722:28
openstackgerritMerged openstack/tooz: tests: do not skip test on connection error  https://review.openstack.org/13806022:29
*** andreykurilin_ has quit IRC22:30
*** andreykurilin_ has joined #openstack-oslo22:31
openstackgerritJoshua Harlow proposed openstack/oslo-incubator: Remove whitespace after project description  https://review.openstack.org/14017022:33
openstackgerritJoshua Harlow proposed openstack/oslo-incubator: Remove whitespace after project description + before range  https://review.openstack.org/14017022:35
openstackgerritJoshua Harlow proposed openstack/oslo-incubator: Exit the script early if something fails  https://review.openstack.org/14017122:38
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add deprecation + reflection auto documentation  https://review.openstack.org/14012922:39
openstackgerritMerged openstack/tooz: tests: remove check_port  https://review.openstack.org/13806222:40
openstackgerritMerged openstack/tooz: memcached: delete stale/dead group members on get_members()  https://review.openstack.org/13806422:40
openstackgerritMerged openstack/tooz: tests: check for leave group events on dead members cleanup  https://review.openstack.org/13837722:40
*** ihrachyshka has quit IRC22:40
openstackgerritJoshua Harlow proposed openstack/oslo.utils: Add deprecation + reflection auto documentation  https://review.openstack.org/14012922:41
*** alexpilotti has joined #openstack-oslo22:47
*** gordc has quit IRC22:50
*** noelbk has quit IRC22:53
openstackgerritMichael Bayer proposed openstack/oslo.db: Ensure DBConnectionError is raised on failed revalidate  https://review.openstack.org/13973822:54
*** bknudson has quit IRC22:55
*** vigneshvar has quit IRC22:58
openstackgerritJoshua Harlow proposed openstack/taskflow: Tidy up the WBE cache (now WBE types) module  https://review.openstack.org/10324522:58
openstackgerritJoshua Harlow proposed openstack/taskflow: Standardize bases in the pluggable modules  https://review.openstack.org/13974123:01
openstackgerritJoshua Harlow proposed openstack/taskflow: Add a moved_inheritable_class deprecation helper  https://review.openstack.org/14013223:01
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow loading conductors via entrypoints  https://review.openstack.org/12461723:01
*** oslonorway has joined #openstack-oslo23:02
*** oslonorway has left #openstack-oslo23:03
openstackgerritJoshua Harlow proposed openstack/taskflow: Add and use a new simple helper logging module  https://review.openstack.org/12300723:03
*** alexpilotti has quit IRC23:11
*** jaosorior has quit IRC23:23
openstackgerritJoshua Harlow proposed openstack/oslo-specs: config: Add a spec for a oslo.config proxy config opts  https://review.openstack.org/13004723:33
*** jgrimm is now known as zz_jgrimm23:41
harlowjajaypipes u mind looking over ^ ; seems like u looked it over once, just adjusted it to hopefully be more nice and such, ha23:43
harlowjawhen u get some time23:43
harlowjahopefully its more clear and more agreeable and such23:44
harlowjaalthough i do understand dhellmann points23:44
« Sunday, 2014-12-07 Index Tuesday, 2014-12-09 »

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