Monday, 2015-03-09

*** achanda has quit IRC00:04
* gus is seeing double.00:06
gusThe oslo.rootwrap tests seem to be mostly-but-not-entirely duplicated between tests/ and oslo_rootwrap/tests/00:07
guswhich is the true location?  Why do there seem to be changes going into both?00:07
*** amrith is now known as _amrith_00:11
openstackgerritMerged openstack/oslo.rootwrap: Remove env changing support in daemon mode  https://review.openstack.org/16118300:19
*** achanda has joined #openstack-oslo00:39
*** e0ne has quit IRC00:49
*** takedakn has joined #openstack-oslo01:14
*** takedakn has quit IRC01:17
*** achanda has quit IRC01:25
*** mriedem1 has quit IRC01:26
openstackgerritJoshua Harlow proposed openstack/taskflow: Add read/write lock decorators  https://review.openstack.org/16250301:29
openstackgerritJoshua Harlow proposed openstack/taskflow: Add no double writers thread test  https://review.openstack.org/16250601:42
*** liusheng has quit IRC01:46
openstackgerritzhangjl proposed openstack/oslo.messaging: Fix the bug redis do not delete the expired keys  https://review.openstack.org/15247101:57
openstackgerritJoshua Harlow proposed openstack/taskflow: Add no double writers thread test  https://review.openstack.org/16250602:10
*** achanda has joined #openstack-oslo02:31
*** Krast has joined #openstack-oslo02:32
*** achanda has quit IRC02:38
*** achanda has joined #openstack-oslo02:38
*** enikanorov has quit IRC02:40
*** enikanorov has joined #openstack-oslo02:40
*** achanda has quit IRC02:42
*** achanda has joined #openstack-oslo02:42
*** pmathews has joined #openstack-oslo02:53
openstackgerritMerged openstack/debtcollector: Add a removed module deprecation helper  https://review.openstack.org/15429803:01
*** dimsum__ has quit IRC03:02
*** pmathews1 has joined #openstack-oslo03:20
*** pmathews has quit IRC03:21
*** pmathews has joined #openstack-oslo03:23
*** pmathews1 has quit IRC03:24
*** achanda has quit IRC03:31
*** pmathews1 has joined #openstack-oslo03:31
*** pmathews1 has quit IRC03:33
*** pmathews1 has joined #openstack-oslo03:33
*** pmathews has quit IRC03:34
*** pmathews has joined #openstack-oslo03:36
*** pmathews1 has quit IRC03:38
*** pmathews has joined #openstack-oslo03:41
*** dimsum__ has joined #openstack-oslo04:03
*** achanda has joined #openstack-oslo04:04
*** dimsum__ has quit IRC04:09
*** achanda has quit IRC04:16
*** achanda has joined #openstack-oslo04:20
*** pmathews has quit IRC04:44
*** achanda has quit IRC04:46
*** achanda has joined #openstack-oslo04:48
*** achanda has quit IRC04:57
*** exploreshaifali has joined #openstack-oslo05:00
openstackgerritJoshua Harlow proposed openstack/taskflow: Add + use read/write lock decorators  https://review.openstack.org/16250305:22
*** achanda has joined #openstack-oslo05:40
openstackgerritMerged openstack/taskflow: Add no double writers thread test  https://review.openstack.org/16250605:49
*** ajo has joined #openstack-oslo05:50
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.vmware: Imported Translations from Transifex  https://review.openstack.org/16253606:04
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.concurrency: Imported Translations from Transifex  https://review.openstack.org/16253806:08
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Imported Translations from Transifex  https://review.openstack.org/16064906:12
*** yamahata has joined #openstack-oslo06:15
*** nikhil_k has quit IRC06:18
*** nikhil_k has joined #openstack-oslo06:18
*** achanda has quit IRC06:37
*** exploreshaifali has quit IRC06:39
*** inc0 has joined #openstack-oslo06:40
*** achanda has joined #openstack-oslo06:44
*** nikhil_k has quit IRC06:48
*** exploreshaifali has joined #openstack-oslo07:02
*** miqui has quit IRC07:04
*** achanda has quit IRC07:40
*** dimsum__ has joined #openstack-oslo07:41
*** dimsum__ has quit IRC07:46
*** exploreshaifali has quit IRC07:49
*** YorikSar has joined #openstack-oslo07:51
*** achanda has joined #openstack-oslo07:52
*** yamahata has quit IRC07:58
*** stevemar has quit IRC08:01
*** achanda has quit IRC08:14
*** achanda has joined #openstack-oslo08:16
*** achanda has quit IRC08:28
*** amotoki has joined #openstack-oslo08:33
*** ihrachyshka has joined #openstack-oslo08:35
*** andreykurilin_ has joined #openstack-oslo08:41
*** ihrachyshka has quit IRC09:00
*** sdague has quit IRC09:04
*** hogepodge has quit IRC09:04
*** sdague has joined #openstack-oslo09:06
*** hogepodge has joined #openstack-oslo09:06
*** ihrachyshka has joined #openstack-oslo09:11
*** inc0_ has joined #openstack-oslo09:17
*** inc0 has quit IRC09:19
*** andreykurilin_ has quit IRC09:21
*** exploreshaifali has joined #openstack-oslo09:33
*** pblaho has joined #openstack-oslo09:33
openstackgerritMerged openstack/oslo.concurrency: Imported Translations from Transifex  https://review.openstack.org/16253809:36
*** yassine_ has joined #openstack-oslo09:59
*** dims_ has joined #openstack-oslo10:16
*** YorikSar has quit IRC10:16
*** inc0_ has quit IRC10:33
*** inc0 has joined #openstack-oslo10:33
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604710:37
*** inc0 has quit IRC10:40
*** inc0 has joined #openstack-oslo10:41
*** serg_ has joined #openstack-oslo10:45
*** serg_ has quit IRC10:46
*** inc0 has quit IRC10:47
*** inc0 has joined #openstack-oslo10:47
*** sreshetn1ak has joined #openstack-oslo10:49
*** andreykurilin_ has joined #openstack-oslo10:53
*** sreshetn1ak has quit IRC11:00
*** inc0 has quit IRC11:04
*** _amrith_ is now known as amrith11:10
*** exploreshaifali has quit IRC11:16
haypoihrachyshka, jd__ : hi. fun with date&time, today: "no DST during ramadan", ok... http://www.timeanddate.com/news/time/egypt-morocco-dst-ramadan-2014.html11:32
ihrachyshkaWAT11:33
haypoihrachyshka: it's a new law of june 201411:33
*** inc0 has joined #openstack-oslo11:34
ihrachyshkaI think Egypt should just shift their time for ~12 hours, so that official day is during the night. I fast like that every day with no problems!11:36
*** cdent has joined #openstack-oslo11:52
*** kgiusti has joined #openstack-oslo12:21
*** dulek has joined #openstack-oslo12:23
yassine_haypo: lol :)12:31
*** gordc has joined #openstack-oslo12:34
*** sigmavirus24_awa is now known as sigmavirus2412:39
*** freyes has joined #openstack-oslo12:40
jd__haypo: waho that's gonna be nice to support :D12:42
*** amrith is now known as _amrith_12:42
*** dims_ has quit IRC12:44
*** dimsum__ has joined #openstack-oslo12:44
*** exploreshaifali has joined #openstack-oslo12:47
*** andreykurilin_ has quit IRC12:48
*** ccrouch is now known as ccrouch-biab12:52
*** stpierre has joined #openstack-oslo12:52
*** dimsum__ is now known as dims13:01
*** bknudson has left #openstack-oslo13:03
*** amotoki has quit IRC13:04
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088913:11
dhellmanngus: There are 2 sets of tests. The ones in the top level /tests directory are for the old import names. The ones under oslo_rootwrap/tests are for the new package name. If you're adding features, you only need to update oslo_rootwrap/tests.13:13
dhellmanndims, beekneemech: I'm going to have to leave the meeting early today to go to a doctor's appointment. Can one of you chair?13:14
*** amotoki has joined #openstack-oslo13:14
*** bknudson has joined #openstack-oslo13:24
dimsdhellmann: yep, will do13:24
dhellmanndims: thanks13:24
dhellmanndims: the main topic is obviously the feature freeze, but we should also start identifying bugs we need to fix post-freeze13:25
dimssounds good dhellmann13:25
* dhellmann starts working making notes about what to release today13:27
*** ccrouch-biab is now known as ccrouch13:32
dimsdhellmann: will there be a oslo.utils release today? (interested in the 'cwd' fix for process execution for Nova)13:33
dhellmanndims: I'll put it on the list13:34
*** superdan is now known as dansmith13:34
dimsthanks!13:34
dhellmannI'm pulling fresh copies of everything so I can see what's changed13:34
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add tox env for listing unreleased changes  https://review.openstack.org/16263413:39
*** YorikSar has joined #openstack-oslo13:41
dhellmanndims: release notes: https://etherpad.openstack.org/p/EqsRdKPo0213:41
dhellmannttx: do you have a minute to talk about library stable releases? I think I missed a detail in my spec and want to make sure it doesn't affect us this cycle: https://review.openstack.org/#/c/155072/13:44
dimsdhellmann: looks the the one i needed is already in 1.3.0, but +1 to a new oslo.utils13:44
dhellmanndims: ok, good13:44
dhellmannttx: we can't use ~= in the global requirements list because the oslo library stable releases won't have the just-released versions in their requirements lists. So we'll have to stick to >=,< for managing caps13:45
dhellmanndims: I'm planning to call oslo.log 1.0.0, does that seem OK to you?13:46
dhellmanngordc: I'm planning to call oslo.middleware 1.0.0, is that OK?13:47
dimsdhellmann: +1 to oslo.log 1.0.013:47
* gordc goes to do a quick check13:47
dhellmanngordc: my working release notes page: https://etherpad.openstack.org/p/EqsRdKPo0213:48
dimsdhellmann: we have heat, keystone, nova already using oslo.log13:48
dhellmanndims: that's even better than I thought!13:48
dimsi was pleasantly surprised as well :)13:49
dhellmannjd__: it looks like we could use a tooz release? https://etherpad.openstack.org/p/EqsRdKPo0213:49
gordcdhellmann: 1.0.0 is ok with me.13:49
dhellmanngordc: cool, thanks13:49
jd__dhellmann: adding to my TODO :)13:50
dhellmannjd__: I'm about to do a bunch, so if you agree I can just do it13:50
jd__dhellmann: oh sure go ahead!13:50
dhellmannjd__: is it 1.0.0 or 0.13.0?13:50
jd__dhellmann: 0.13 (I think we stuck to 1 point only, though change if you prefer)13:51
*** _amrith_ is now known as amrith13:52
dhellmannI moved that etherpad over to a place with a real name: https://etherpad.openstack.org/p/oslo-kilo-feature-freeze-releases13:52
dhellmannjd__: we should probably talk about what's left to reach a 1.0, since some projects are using the lib already13:52
jd__dhellmann: yeah, harlowj might have some ideas too13:53
dhellmannjd__: maybe start a ML thread? I'd like to to get tooz and taskflow to 1.0 by the end of next cycle so people have confidence that they are stable.13:54
dhellmannthey don't have to be feature complete, just stable13:54
jd__dhellmann: sure I'll start a thread13:54
dhellmannjd__: thanks13:55
jd__dhellmann: done :)13:56
jd__I expect eglynn and harlowj to jump in13:56
*** inc0 has quit IRC13:56
dhellmannjd__: ok, sounds good13:57
ttxdhellmann: I think that works too13:59
dhellmannttx: ok, I'll update the spec14:00
openstackgerritDoug Hellmann proposed openstack/oslo-incubator: Add tox env for listing unreleased changes  https://review.openstack.org/16263414:01
*** salv-orlando has joined #openstack-oslo14:05
*** jecarey has joined #openstack-oslo14:12
*** ChuckC_ has joined #openstack-oslo14:13
*** ChuckC has quit IRC14:14
*** daniel3_ has joined #openstack-oslo14:15
*** ChuckC_ has quit IRC14:18
bknudson“‘Chuffed’ means pleased… or displeased.14:26
bknudsonhttps://britishisms.wordpress.com/2013/07/03/chuffed/14:26
bknudsonthat's a new one for me... not british.14:27
openstackgerritMerged openstack/taskflow: Use condition context manager instead of acquire/release  https://review.openstack.org/16238314:29
*** GheRivero has joined #openstack-oslo14:31
*** achanda has joined #openstack-oslo14:33
boris-42dhellmann: hi there14:38
boris-42dhellmann: one question regarding to openstackclient14:39
boris-42dhellmann: is it ready for use ?14:39
*** mriedem has joined #openstack-oslo14:39
boris-42dhellmann: and is it backward compable, I mean can we run master of it against grizzle for example?14:40
*** mtanino has joined #openstack-oslo14:42
*** achanda has quit IRC14:42
*** zz_jgrimm is now known as jgrimm14:43
dhellmannboris-42: that's the goal, but you'll find the dev team in #openstack-sdks so you might want to ask the question there again14:45
boris-42dhellmann: thanks14:47
openstackgerritMerged openstack/taskflow: Rename the timing listeners to duration listeners  https://review.openstack.org/16182914:47
*** beekneemech is now known as bnemec14:49
* bnemec is double booked for the Oslo meeting again :-(14:49
bnemecSeriously, what is it with people at Red Hat and one off meetings Monday at 10 AM central?14:50
*** ChuckC_ has joined #openstack-oslo14:54
*** ChuckC_ has quit IRC14:54
*** ChuckC_ has joined #openstack-oslo14:54
dhellmannbnemec: My meeting invitations always have a "Decline" button ;-)14:55
bnemecdhellmann: Yeah, I was supposed to present at this one though.14:56
bnemecAlthough I found out this morning that I'm off the hook for that, so I guess I don't actually _have_ to be there now. :-)14:56
*** zzzeek has joined #openstack-oslo15:01
dhellmanndansmith: do we need an oslo.versionedobjects release today?15:02
* bnemec just realized the oslo meeting isn't at 10 AM central anymore.15:03
*** stevemar has joined #openstack-oslo15:03
dimsi was just checking that15:04
dims12 noon my time :)15:04
*** achanda has joined #openstack-oslo15:04
*** exploreshaifali has quit IRC15:05
dansmithdhellmann: ...I don't think so, why?15:05
dhellmanndansmith: this week is feature freeze, so I'm releasing all the things15:05
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: rabbit: heartbeat implementation  https://review.openstack.org/14604715:06
dansmithdhellmann: well, not that I know of, and haven't heard any complaints from the heat guys, so I assume everything is fine15:06
dhellmanndansmith: ok, cool, I was just checking15:06
openstackgerritAnita Kuno proposed openstack/oslo-incubator: Add two additional emotions to release_notes  https://review.openstack.org/16265115:07
*** prad has joined #openstack-oslo15:07
*** achanda has quit IRC15:09
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Update to oslo.context  https://review.openstack.org/15977815:09
openstackgerritMehdi Abaakouk proposed openstack/oslo.messaging: Deprecates the localcontext API  https://review.openstack.org/14061615:09
zzzeekoh crap UTC has no DST.   i hate computer programmers15:18
jd__dhellmann: looks like your 'z' key is broken ;)15:20
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088915:20
dhellmannjd__: auto-complete failure :-/15:20
dimszzzeek: so what's the story behind your handle? :)15:21
zzzeekdims: oh it goes back to when i was a teenager practically, not really anything special15:21
zzzeekdims: just a succession of nicknames15:21
zzzeekthough “zzzeek” was the first one that got linked to my real identity, so the whole “hide behind your nickname” thing was over.15:22
dhellmannttx: I'm going to go ahead and try the ~= thing and see how it works. I don't think we need to worry about the minimums after all, but we'll see. https://review.openstack.org/16265615:23
dimszzzeek: :)15:24
*** tsekiyama has joined #openstack-oslo15:26
*** pmathews has joined #openstack-oslo15:28
*** harlowja_at_home has joined #openstack-oslo15:32
*** david-lyle_afk is now known as david-lyle15:38
openstackgerritJoshua Harlow proposed openstack/taskflow: Add another probabilistic rw-lock test  https://review.openstack.org/16266515:43
*** yamahata has joined #openstack-oslo15:45
openstackgerritMerged openstack/taskflow: Add + use read/write lock decorators  https://review.openstack.org/16250315:47
jd__dhellmann: FWIW looks like we have a small regression in Gnocchi due to one of the new oslo release :)15:54
jd__likely oslo.log or oslo.config, cf http://logs.openstack.org/49/162649/1/check/gate-gnocchi-python27/466f3b4/console.html15:55
jd__gordc: ^15:55
dhellmannjd__: le sigh15:59
*** sputnik13 has joined #openstack-oslo16:00
amrithis it oslo meeting time ...16:01
*** sputnik13 has quit IRC16:05
jd__I bisect the culprit to https://review.openstack.org/#/c/156033/16:06
jd__but honestly it makes no sense16:06
jd__I reverted it and it seems to fix the issue we have16:06
jd__gordc: could you confirm?16:06
jd__this might not be a regression but something weird we do that now triggers a bug16:06
*** daniel3_ has quit IRC16:07
dhellmannjd__: the test requirements for gnocchi install ceilometer, which has the incubator version of the log code, with a different version of that option definition than is found in oslo.log now16:11
*** jaypipes has joined #openstack-oslo16:14
jd__arf :(16:15
cdentbest solution to just un-incubate ceilo?16:17
dhellmanncdent: that's likely the simplest solution, yes16:20
cdentneeds to happen eventually anyway, so may as well look on this as an opportunity16:21
dhellmannanother option is to update the definition of the option in the ceilometer module, but procedurally that requires backporting the change to the stable/juno branch of the incubator first16:21
dhellmannyeah, it shouldn't be too much work, and I can help with reviews16:21
*** yamahata has quit IRC16:22
*** salv-orlando has quit IRC16:22
*** exploreshaifali has joined #openstack-oslo16:25
*** pmathews has quit IRC16:31
*** ChuckC_ has quit IRC16:34
*** andreykurilin_ has joined #openstack-oslo16:35
*** harlowja_at_home has quit IRC16:39
*** dulek has quit IRC16:41
amrithdims16:46
amrithping16:46
dimsamrith: pong16:46
amrithFor Trove, oslo.log https://review.openstack.org/#/c/162676/ and https://review.openstack.org/#/c/162677/16:46
amriththird one to come for trove-client16:46
*** daniel3_ has joined #openstack-oslo16:47
amrithif required16:47
amriththx16:47
dimsamrith: thanks! nice16:48
*** Krast has quit IRC16:55
*** Krast has joined #openstack-oslo16:56
*** tsekiyama has quit IRC17:07
*** pblaho has quit IRC17:12
*** zzzeek has quit IRC17:16
*** daniel3_ has quit IRC17:17
cdentdhellmann: there's an existing patchset in progress but it has some issue: https://review.openstack.org/#/c/147810/17:18
*** daniel3_ has joined #openstack-oslo17:19
*** ihrachyshka has quit IRC17:20
*** zzzeek has joined #openstack-oslo17:21
*** salv-orlando has joined #openstack-oslo17:23
openstackgerritDan Krause proposed openstack/taskflow: Writers can now claim a read lock in ReaderWriterLock  https://review.openstack.org/16269517:24
openstackgerritDan Krause proposed openstack/taskflow: Writers can now claim a read lock in ReaderWriterLock  https://review.openstack.org/16269517:25
*** yassine_ has quit IRC17:26
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088917:27
*** sputnik13 has joined #openstack-oslo17:36
*** salv-orlando has quit IRC17:36
*** harlowja has joined #openstack-oslo17:40
*** harlowja_ has joined #openstack-oslo17:43
*** harlowja has quit IRC17:45
*** yamahata has joined #openstack-oslo17:51
*** saikrishna has joined #openstack-oslo18:00
harlowja_lifeless will https://pypi.python.org/pypi/traceback2/1.4.0 be the offical backport for 2.6+ ?18:01
harlowja_or is that more of just an experimental thing18:02
*** yamahata has quit IRC18:03
*** yamahata has joined #openstack-oslo18:04
harlowja_for your traceback stuff/work18:05
*** crc32 has joined #openstack-oslo18:05
*** tsekiyama has joined #openstack-oslo18:08
*** amotoki has quit IRC18:09
*** amotoki has joined #openstack-oslo18:10
*** tsekiyama has quit IRC18:13
*** ChuckC has joined #openstack-oslo18:17
*** achanda has joined #openstack-oslo18:18
*** achanda has quit IRC18:19
*** achanda has joined #openstack-oslo18:19
harlowja_dhellmann http://lists.openstack.org/pipermail/openstack-dev/2015-March/058589.html (done)18:22
harlowja_+ any others dims bnemec  ... ^18:22
harlowja_will try to get out taskflow tommorow (or day after i guess)18:23
dimsharlowja_: thanks18:23
harlowja_np18:23
harlowja_it'd be nice to absorb https://review.openstack.org/#/c/162644/18:23
harlowja_*and avoid duplicates18:23
harlowja_but lets see18:23
openstackgerritJoshua Harlow proposed openstack/taskflow: Remove support for 3.3  https://review.openstack.org/16270818:25
openstackgerritJoshua Harlow proposed openstack/tooz: Remove support for 3.3  https://review.openstack.org/16270918:27
harlowja_probably a few other libraries that should have that change also18:27
kragnizwhat do we do when the incubator can be totally removed from a project?18:29
kragnizjust remove openstack-common.conf and openstack/common?18:30
dimskragniz: yes18:33
kragnizcool18:33
*** tsekiyama has joined #openstack-oslo18:33
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088918:36
*** exploreshaifali has quit IRC18:42
dimsdhellmann: None in choices is triggering a tempest pep8 failure - https://github.com/openstack/tempest/blob/master/tempest/config.py#L460 because of a oslo.config change - https://github.com/openstack/oslo.config/commit/0f550d7c79249900c7f23fdd53723a1f2443e01d18:44
*** salv-orlando has joined #openstack-oslo18:46
openstackgerritMerged openstack/taskflow: Add another probabilistic rw-lock test  https://review.openstack.org/16266518:46
*** salv-orlando has quit IRC18:51
*** serg1 has joined #openstack-oslo18:52
*** serg1 has quit IRC18:54
*** serg1 has joined #openstack-oslo18:54
*** salv-orlando has joined #openstack-oslo18:55
*** salv-orlando has quit IRC19:00
*** salv-orlando has joined #openstack-oslo19:00
dhellmannharlowja_: thanks19:02
*** daniel3__ has joined #openstack-oslo19:02
dhellmanncdent: added to my review list19:02
*** daniel3_ has quit IRC19:02
cdentthanks dhellmann. it's looking like the swift issues that are mentioned in the comments remain, so it's probably going to fail its current jenkins run19:03
dhellmanndims: is None a valid value for a string option?19:03
dhellmanndims: it should be easy enough to fix oslo.config if you haven't already prepared a patch19:04
*** serg1 has quit IRC19:07
dimsdhellmann: on its way19:07
dhellmanndims: even if we fix it, tempest's pep8 check is going to fail because the output of the file will be different19:08
dimstrue19:08
dimswill cross that bridge :)19:08
dhellmannthat's on mtreinish though :-)19:08
*** sreshetn1 has joined #openstack-oslo19:09
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: None in config choices breaks oslo-generator  https://review.openstack.org/16272819:10
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: None in config choices breaks oslo-config-generator  https://review.openstack.org/16272819:10
openstackgerritDan Krause proposed openstack/taskflow: add _listeners_for_job method to Conductor base  https://review.openstack.org/16272919:10
dhellmanndims: how about making that: six.text_type(choice) if choice is not None else '<None>'19:12
dhellmanndims: that will protect us against folks who have numbers as choices, too19:13
dhellmanndims: is there a bug for this, or should I open one?19:13
dhellmanndims: https://bugs.launchpad.net/oslo.config/+bug/142998119:15
openstackLaunchpad bug 1429981 in oslo.config "non-string choices for StrOpt break config generator" [Undecided,New]19:15
*** amrith is now known as _amrith_19:18
*** ihrachyshka has joined #openstack-oslo19:23
dimsdhellmann: sorry, stepped out for a cup of joe. thanks for the bug, will update review with six.text_type and the bug #19:33
*** exploreshaifali has joined #openstack-oslo19:35
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: None in config choices breaks oslo-config-generator  https://review.openstack.org/16272819:35
openstackgerritMerged openstack/taskflow: Writers can now claim a read lock in ReaderWriterLock  https://review.openstack.org/16269519:37
*** e0ne has joined #openstack-oslo19:38
*** marun has joined #openstack-oslo19:40
marunzzzeek: hi!19:40
marunzzzeek: I'd like to speak to you about the oslo.db test fixture's coupling to the testcase, and whether you'd be amenable to changing that.19:47
zzzeekmarun: i believe there’s already a WIP for that19:47
marunzzzeek: link?19:47
zzzeekhttps://review.openstack.org/#/c/159883/19:48
zzzeekmarun: feel free to work with that19:48
marunzzzeek: I'm also looking into making a new long-lived db fixture for functional testing19:48
zzzeekmarun: this fixture should do that19:48
marunzzzeek: ah, ok.19:48
marunzzzeek: thanks for the pointer!19:48
zzzeekmarun: it uses testresources now19:48
*** achanda has quit IRC19:49
marunzzzeek: regarding fixture cleanup, what do you think of deleting all records from the tables?19:49
marunzzzeek: we already do that in neutron unit tests (likely as a stopgap until the transactional stuff lands)19:50
zzzeekmarun: the approach in oslo.db uses transacitons, cleaner and simpler19:50
marunzzzeek: for functional testing, we won't have that option19:50
zzzeekmarun: why not ?19:50
marunzzzeek: we'll have an out-of-process daemon interacting with the db19:50
zzzeekmarun: ah OK19:50
marunzzzeek: it seems to work fine for sqlite, any gotchas you know about wrt mysql or pgsql?19:51
zzzeekmarun: well oslo.db can support it that way, not sure if i built that in already you should poke around provision.py19:51
zzzeekmarun: well you need to delete in order19:51
zzzeekmarun: if you have self-referential tables this can be challenging19:51
zzzeekmarun: on mysql at least…i think it might complain if you delete from a table like that.  maybe not, i forget19:52
marunzzzeek: doesn't meta.sorted_tables help with that?19:52
zzzeekmarun: yes19:52
marunsorry, metadata.sorted_tables?19:52
*** dhellman_ has joined #openstack-oslo19:52
zzzeekmarun: the case is if two tables are mutually dependent on each other, then its harder.19:52
*** dhellman_ has quit IRC19:52
marunzzzeek: hmmm...19:52
zzzeekmarun: its a less likely scenario.19:53
*** daniel3__ is now known as daniel3_19:53
marunzzzeek: I guess I'll have to see. Moving api tests in-tree allows us to do db-level cleanup rather than through the api, but maybe we'll have to continue using tenant isolation for the most part.19:53
marunzzzeek: table creation in mysql ain't exactly cheap, though.19:54
marunzzzeek: will poke around, thank you :)19:54
zzzeekmarun: the testresources appraoch requires you to sort your tests with an OptimizingTestSuite which will group tests using common resources together19:55
marunzzzeek: ah, I forgot about that wrinkle...19:55
zzzeekmarun: i had a diferent appraoch in mind but thats how we do it in openstack :)19:57
marunzzzeek: some of it is good, some of it less so....19:57
marungood with the bad, as always19:57
*** dims_ has joined #openstack-oslo19:58
*** pradk has joined #openstack-oslo19:59
*** dims has quit IRC20:00
*** achanda has joined #openstack-oslo20:01
*** pradk has quit IRC20:02
*** ajo has quit IRC20:02
*** prad has quit IRC20:02
*** ajo has joined #openstack-oslo20:02
*** e0ne has quit IRC20:06
*** e0ne has joined #openstack-oslo20:06
*** marun has quit IRC20:07
openstackgerritJoshua Harlow proposed openstack/tooz: Add a driver feature compatibility matrix  https://review.openstack.org/16274220:08
openstackgerritJoshua Harlow proposed openstack/tooz: Add a driver feature compatibility matrix  https://review.openstack.org/16274220:09
dhellmanndims_: I went ahead and +2a https://review.openstack.org/#/c/162728/3 -- we can deal with empty string later20:10
dims_dhellmann: ack thanks20:11
*** achanda has quit IRC20:11
dhellmanndims_: I'm going to step AFK for a few minutes20:11
*** achanda has joined #openstack-oslo20:14
*** kgiusti has left #openstack-oslo20:15
openstackgerritJoshua Harlow proposed openstack/tooz: Add a driver feature compatibility matrix  https://review.openstack.org/16274220:15
*** andreykurilin_ has quit IRC20:16
*** andreykurilin_ has joined #openstack-oslo20:17
*** achanda has quit IRC20:28
*** cdent has quit IRC20:30
dims_dhellmann: do we need to rev-up oslo.config?20:33
*** jungleboyj has quit IRC20:33
*** dims_ has quit IRC20:33
*** dims has joined #openstack-oslo20:34
dimsah looks like you did dhellmann :)20:38
*** achanda has joined #openstack-oslo20:38
dimsah that's a note talking about the bug itself on the -dev mailing list20:39
openstackgerritDan Krause proposed openstack/taskflow: Persistence backend refactor  https://review.openstack.org/16088920:39
openstackgerritMerged openstack/tooz: Remove support for 3.3  https://review.openstack.org/16270920:41
dhellmanndims: when the patch merges, we'll cut the next release and then update https://review.openstack.org/16265620:41
dhellmanndims: brb, I have to reboot20:42
dimsack dhellmann20:42
*** ajo has quit IRC20:55
*** ajo has joined #openstack-oslo20:58
*** amuller has joined #openstack-oslo20:59
amullerdhellmann: Hello Doug, can you point me towards someone who can answer some questions about the fake driver for Oslo messaging?21:01
dhellmannamuller: you probably want sileht or jd__, but they're in europe so it's late in the day for them21:01
amuller yeah it'd be late for them21:02
amullerTrying to figue out how/if the fake driver should be used in testing21:02
amullerI'll try tomorrow morning21:02
amullerdhellmann: Thank you21:03
dhellmannamuller: yes, I think that's the point of the fake driver, to be used in tests21:03
* dhellmann runs an errand21:03
*** jungleboyj has joined #openstack-oslo21:03
*** prad has joined #openstack-oslo21:05
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: print better message when choices has an empty string  https://review.openstack.org/16276021:13
dimsbnemec: around? ^^21:15
*** marun has joined #openstack-oslo21:16
harlowja_lifeless for https://github.com/testing-cabal/traceback2 will there be an api to reattach a FrameSummary or whatever to a exception so that it can be re-thrown (with the reattached info)21:25
harlowja_*if u know21:25
harlowja_*useful for remoting...21:26
harlowja_or similar kind of stuff21:26
*** GheRivero has left #openstack-oslo21:28
harlowja_seems like there is no current re-attach/rethrow kind of ability21:29
*** sreshetn1 has quit IRC21:31
*** ChuckC has quit IRC21:31
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: print better message when choices has an empty string  https://review.openstack.org/16276021:37
openstackgerritMin Pae proposed openstack/taskflow: Allow using ordered(set/dict) to retain ordering  https://review.openstack.org/16055821:39
*** daniel3_ has quit IRC21:39
*** stpierre has quit IRC21:40
*** sigmavirus24 is now known as sigmavirus24_awa21:41
dimsdhellmann: posted a follow up 16276021:41
dhellmanndims: I was just looking. I also hit "reveriify" on the bug fix21:42
dimsack. thanks21:42
openstackgerritJoshua Harlow proposed openstack/tooz: Add a driver feature compatibility matrix  https://review.openstack.org/16274221:43
bnemecdims: That seems like a rather verbose way of saying if choice == '':21:43
bnemec:-)21:43
dimsbnemec: i am over engineering!21:44
dimsthanks fixing21:44
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: print better message when choices has an empty string  https://review.openstack.org/16276021:45
*** saikrishna has quit IRC21:45
bnemecdims: Thanks, +221:46
*** ihrachyshka has quit IRC21:47
dhellmannbnemec, dims : +2a21:47
*** achanda has quit IRC21:48
dimsdhellmann: stepping out for a bit21:48
dhellmanndims: I have dinner guests coming in a few minutes, so it might be a tomorrow before I could cut a release. If you're still around when those merge and want to do it, go for it.21:48
dimsdhellmann: which script do you use?21:48
dhellmanndims: in the openstack-infra/release-tools repo there's a script called "release_library.sh" for doing the release dance with launchpad. Then in oslo-incubator/tools there's a script for building the release notes email (release_notes.py I think)21:49
dhellmanndims: https://review.openstack.org/16263421:49
dimsthanks, will take care of it. have a good evening21:50
dhellmanndims: thanks!21:50
*** achanda has joined #openstack-oslo21:52
openstackgerritJoshua Harlow proposed openstack/oslo-incubator: By default close *most* fds in child processes  https://review.openstack.org/15760821:57
elarsonjust in case anyone was interested - https://github.com/ionrock/lodgeit-el22:01
harlowja_dhellmann alright lets see what happens with https://review.openstack.org/#/c/162781/ and https://review.openstack.org/#/c/162780/22:02
*** ChuckC has joined #openstack-oslo22:02
harlowja_think i did that right so it should depend on correctly, lol22:02
*** crc32 has quit IRC22:02
*** gordc has quit IRC22:05
*** jungleboyj has quit IRC22:06
lifelessharlowja_: no remoting stuff planned at the moment. That would be PEP-worthy in cPYthon22:10
harlowja_lifeless fair enough22:10
*** marun has quit IRC22:10
harlowja_durn :-P22:10
lifelessharlowja_: if you were to do it, I'd be happy to do the backporting work of course - and help review and design the pep22:10
harlowja_hmmm, possibly22:10
harlowja_seems like openstack woudl like that, to get away from the various 'hacks' around this22:10
lifelessits not on my immediate radar - in fact the new API only exists because what I did want (https://github.com/testing-cabal/testtools/pull/129) was overlapping with Guido's desire for an overhaul of traceback22:11
*** andreykurilin_ has quit IRC22:11
lifelessthere are security implications with remoting of any sort22:11
lifelesswhich is one major reason I'm saying it would need a PEP22:11
*** andreykurilin_ has joined #openstack-oslo22:11
harlowja_agreed22:12
lifelessharlowja_: traceback2 and linecache2 are as official as any backport project :)22:12
harlowja_:-P22:12
*** ajo has quit IRC22:12
lifelessand yes, I support 2.6 and will until we stop supporting 2.6 in testtools22:13
lifelesspossibly longer than that22:13
lifelessbut no shorter22:13
harlowja_:)22:13
harlowja_https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/_drivers/common.py#L203 is one of the things it would be nice to get rid of22:14
harlowja_^ madness lies there, lol22:14
lifelessyeah22:15
lifelessWAT22:15
*** jgrimm is now known as zz_jgrimm22:15
*** dims_ has joined #openstack-oslo22:15
lifelessfailure.__class__ = new_ex_type22:15
harlowja_:-/22:15
* harlowja_ runs away22:16
*** ChuckC has quit IRC22:16
lifelessthere's a backport of the new exception stuff to 2.x22:16
lifelessshould be using that instead22:16
harlowja_taskflow failure object is imho slightly better, at least it doesn't do that stuff; but it ends up having to throw a kind of wrapped exception with failure objects in it (and the failure objects have the string versions of tracebacks....)22:16
harlowja_so a nice python supported something or other would be cool :-P22:17
lifelessI generally think string tracebacks are better than full ones anyway22:17
lifelessbut the new summary classes in traceback would be a good fit I think22:17
lifelessand they should naively serialise just fine22:17
lifelessw/out loops or references to worry about22:17
*** dims has quit IRC22:17
harlowja_its version instead @ https://github.com/openstack/taskflow/blob/master/taskflow/types/failure.py#L77 (if u haven't seen, think u might of)22:17
*** amuller has left #openstack-oslo22:18
lifelessharlowja_: 17911 is merged :)22:18
harlowja_ya, the other part that would be nice is to have exceptions + tracebacks go correctly across multiprocessing boundary; instead of afaik losing it22:18
harlowja_ya ya22:18
*** rushiagr_away has quit IRC22:18
harlowja_'Versions: Python 3.5'22:19
harlowja_lol22:19
harlowja_on 17911 :-P22:19
lifelessyeah - it doesn't qualify for stable updates to 3.4 etc22:19
lifelessbut its in traceback222:19
harlowja_yup, i'm gonna try to figure out the usage of traceback2; if it helps cut some code away and such22:20
lifelessplease22:20
harlowja_having the reconstructable stuff would help with oslo.messaging stuff that u must not look at, lol22:20
lifelessalso you don't handle __cause__ and __context__ in that code22:20
lifelesswell22:20
lifelessI kindof think the oslo.messaging thing is an antifeature22:20
harlowja_which code u looking at?22:20
harlowja_:-/22:21
lifelesshttps://github.com/openstack/taskflow/blob/master/taskflow/types/failure.py#L77 and the olso one as well22:21
lifelessthe messaging thing is tying together code across process boundaries without actually defining an API22:21
lifelessusually we consider that bad :)22:21
harlowja_ya; traceback2 will help there for taskflow at least22:21
harlowja_*releated to __cause__ and such22:22
harlowja_six afaik also got a six.reraise_from that does the right thing on py2/py3 (assocating a cause) so that will be useful also22:22
*** openstack has joined #openstack-oslo22:25
openstackgerritMin Pae proposed openstack/taskflow: Allow using ordered(set/dict) to retain ordering  https://review.openstack.org/16055822:25
openstackgerritDavanum Srinivas (dims) proposed openstack/oslo.config: print better message when choices has an empty string  https://review.openstack.org/16276022:26
*** sreshetn1 has joined #openstack-oslo22:27
*** prad has quit IRC22:27
*** bknudson has quit IRC22:28
lifelessharlowja_: yeah, it has reraise and raise_from (which I added)22:32
harlowja_ah, thx lifeless !22:32
harlowja_:)22:32
*** sreshetn1 has quit IRC22:32
lifelesshttps://bitbucket.org/gutworth/six/issue/102/raise-foo-from-bar-is-a-syntax-error-on-2722:32
harlowja_got my taskflow code updated to use a little helper that uses it https://review.openstack.org/#/c/155187/ (so that it works ok on py2 and py3 with exceptions)22:32
harlowja_little helper @ https://github.com/openstack/taskflow/blob/master/taskflow/exceptions.py#L24 (that uses reraise_from)22:33
harlowja_first oslo library that probably does that right :-P22:34
harlowja_without losing associated causes22:34
lifelessyah22:35
*** jecarey has quit IRC22:39
*** exploreshaifali has quit IRC22:41
openstackgerritJoshua Harlow proposed openstack/taskflow: Allow using ordered(set/dict) to retain ordering  https://review.openstack.org/16055823:05
openstackgerritJoshua Harlow proposed openstack/taskflow: Added a map and a reduce task  https://review.openstack.org/15767723:06
*** ChuckC has joined #openstack-oslo23:12
*** e0ne has quit IRC23:13
*** sigmavirus24_awa is now known as sigmavirus2423:20
*** openstack has joined #openstack-oslo23:24
openstackgerritJoshua Harlow proposed openstack/taskflow: Denote issue 17911 has been merged/accepted  https://review.openstack.org/16279623:31
harlowja_lifeless ^ there, ha23:31
*** sigmavirus24 is now known as sigmavirus24_awa23:33
harlowja_lifeless is https://github.com/testing-cabal/traceback2/blob/master/traceback2/__init__.py#L229 meant to be a public api?23:41
harlowja_seems useful :-P23:42
lifelessit is23:50
lifelessits useless unles the interpreter supports it23:50
lifelessI need to do some more digging on some subtle interactions there too23:50
lifelessbut yes, its public23:50

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