Wednesday, 2014-03-12

*** yamahata has joined #openstack-oslo00:18
*** zhiyan_ has quit IRC00:57
*** zhiyan` has joined #openstack-oslo00:58
*** GheRivero has quit IRC00:58
*** GheRivero has joined #openstack-oslo00:58
*** sileht has quit IRC00:59
*** openstack has joined #openstack-oslo01:05
*** mattymo has joined #openstack-oslo01:05
*** wingwj has joined #openstack-oslo01:19
*** gcb has joined #openstack-oslo01:24
openstackgerritJoshua Harlow proposed a change to openstack/oslo.messaging: Add a zookeeper based matchmaker  https://review.openstack.org/7982501:28
*** rpodolyaka has quit IRC02:38
*** harlowja is now known as harlowja_away02:48
*** mriedem has quit IRC03:15
*** arnaud has joined #openstack-oslo03:48
*** sileht has quit IRC03:56
*** sileht has joined #openstack-oslo03:58
*** zhiyan` is now known as zhiyan04:02
openstackgerritVui Lam proposed a change to openstack/oslo.vmware: Fix unit tests running unnecessarily slow  https://review.openstack.org/7740505:03
*** rpodolyaka has joined #openstack-oslo05:31
openstackgerritlifeless proposed a change to openstack-dev/pbr: Fix up some docstrings.  https://review.openstack.org/7985506:19
openstackgerritlifeless proposed a change to openstack-dev/pbr: Return the real class in VersionInfo __repr__.  https://review.openstack.org/7985606:19
*** arnaud has quit IRC06:58
*** rpodolyaka has quit IRC07:00
*** bknudson has quit IRC07:24
*** flaper87|afk is now known as flaper8707:24
*** bknudson has joined #openstack-oslo07:26
*** morganfainberg is now known as morganfainberg_Z08:17
*** I159 has joined #openstack-oslo08:24
*** ekarlso has quit IRC08:47
*** ekarlso has joined #openstack-oslo08:47
openstackgerritStanislav Kudriashev proposed a change to openstack/taskflow: Rework proxy publish functionality  https://review.openstack.org/7960108:54
openstackgerritStanislav Kudriashev proposed a change to openstack/taskflow: Rename remote task to request  https://review.openstack.org/7968508:54
*** viktors_ is now known as viktors09:04
*** zhiyan is now known as zhiyan_09:06
openstackgerritJulien Danjou proposed a change to openstack/oslo-incubator: policy: rename Exception to avoid nose regression  https://review.openstack.org/7988309:08
*** yamahata has quit IRC10:17
*** ChanServ changes topic to "test/gate jobs are queuing now in preparation for gerrit maintenance at 12:00 utc (eta to resume is 12:30 utc)"11:27
openstackgerritVictor Stinner proposed a change to openstack/oslo.messaging: Add an optional timeout parameter to Listener.poll  https://review.openstack.org/7100312:01
*** ChanServ changes topic to "gerrit on review.openstack.org is down for maintenance (revised eta to resume is 13:00 utc)"12:24
openstackgerritStanislav Kudriashev proposed a change to openstack/taskflow: Introduce message types for WBE protocol  https://review.openstack.org/7971212:46
*** zhiyan_ is now known as zhiyan12:47
*** openstackgerrit has quit IRC12:54
*** openstackgerrit has joined #openstack-oslo12:54
*** openstackgerrit has quit IRC13:08
*** openstackgerrit has joined #openstack-oslo13:08
*** mriedem has joined #openstack-oslo13:18
openstackgerritVictor Sergeyev proposed a change to openstack/oslo-incubator: Remove common context usage from db model_query()  https://review.openstack.org/7965313:23
*** yamahata has joined #openstack-oslo13:57
*** beav has joined #openstack-oslo14:01
*** dims_ has quit IRC14:03
*** openstack has joined #openstack-oslo14:04
*** dims_ has joined #openstack-oslo14:16
bknudsonhttp://logs.openstack.org/41/78441/9/check/gate-oslo-incubator-python33/6640a2b/console.html14:30
bknudsonTypeError: __class__ assignment: 'UnboundSelf' object layout differs from 'TestException'14:30
*** fungi has joined #openstack-oslo14:30
bknudsonHow does that happen?14:31
*** ChanServ sets mode: +o fungi14:31
dhellmannbknudson: https://review.openstack.org/#/c/79883/14:32
dhellmannbknudson: new version of nose had a behavior change; jd__'s fix is a work-around in the incubator14:32
dhellmannbknudson: rebase on top of that (it *just* landed)14:32
bknudsondhellmann: cool, thanks14:32
*** fungi changes topic to "#openstack-oslo"14:33
*** fungi has left #openstack-oslo14:33
bknudsonthat's funny.14:33
bknudsonkeystone had some functions called test_directory or something that were getting run as tests when we switched to testr.14:34
dhellmann'explicit is better than implicit'14:35
openstackgerritBrant Knudson proposed a change to openstack/oslo-incubator: Fix excessive logging from db.sqlalchemy.session  https://review.openstack.org/7844114:36
openstackgerritVictor Sergeyev proposed a change to openstack/oslo-incubator: Remove common context usage from db model_query()  https://review.openstack.org/7965314:37
openstackgerritAnastasia Karpinska proposed a change to openstack/taskflow: Add retry action to execute retries  https://review.openstack.org/6811214:41
openstackgerritAnastasia Karpinska proposed a change to openstack/taskflow: Flow smart revert with retry controller  https://review.openstack.org/7162114:41
openstackgerritAnastasia Karpinska proposed a change to openstack/taskflow: Add atom intentions for tasks and retries  https://review.openstack.org/7101314:41
openstackgerritAnastasia Karpinska proposed a change to openstack/taskflow: Extend logbook and storage to work with retry  https://review.openstack.org/6811114:41
openstackgerritJames Carey  proposed a change to openstack/oslo-incubator: Fix gettextutil.Message handling of deep copy failures  https://review.openstack.org/7966714:44
viktorsdhellmann: around?14:45
dhellmannhi, viktors14:45
viktorsdhellmann: hello14:46
dhellmannviktors: I wanted to talk about timing for the oslo.db graduation14:46
viktorsdhellmann: yes14:46
dhellmannwe've had some changes to the incubator code, bug fixes and that sort of thing, so I think any version you've extracted may be out of date14:46
viktorsagree with you14:47
dhellmannit might be best to wait to start after the release candidates are all cut, when we know the incubator can be locked down for a little while14:47
dhellmannI had been thinking *now* would be a good time, but I wasn't anticipating so many changes :-/14:47
viktorsdhellmann: so when should we start?14:48
dhellmannviktors: I think after we hear from ttx that all of the integrated project release candidates have been successfully cut over, and we have a stable/icehouse branch in the incubator14:49
* viktors looking for release schedule14:50
dhellmannviktors: that should be some time around april 17, which is a long wait but I would rather wait than miss a change in the library or slow down the releases because of having to update the library before the incubator14:51
viktorsdhellmann: yes, in a such case we should wait more than month (14:54
viktorsdhellmann: at april 17 should be final release, yes? Can we start after some release candidate?14:56
dhellmannviktors: well, release "candidate" isn't quite the same as "release" so I think it will be safer to wait for the release to be declared final14:57
viktorsdhellmann: yes, it sounds reasonable. Let it be15:00
viktorsdhellmann: so what can we do now from graduation checklist https://wiki.openstack.org/wiki/Oslo/CreatingANewLibrary#Checklist ?15:02
dhellmannviktors: you could prepare all of the changes to infra, but I think you've already done most of that15:04
dhellmannviktors: there will be some changes needed after the new repository is created to set up the documentation build, so you could work all of that out15:05
dhellmannbasically, prepare a "dry run" of the export -- the only piece you would have to redo is creating the repository to be imported into gerrit15:05
bknudsonhttps://review.openstack.org/#/c/78441/ is working again -- has 2 +215:07
viktorsdhellmann: ok, thanks, will wait15:08
bknudsononce that's merged I'll sync with keystone15:09
dhellmannbknudson: is that still needed? I thought bnemec changed that code so it used the connect callback?15:10
bnemecdhellmann: I think we still need that change.  Otherwise we're going to log the message for every connection that gets created, which I think by default is like 50.15:10
bknudsondhellmann: good question... if something else merged I can try it.15:11
dhellmannbnemec: ah, ok15:11
bnemecI was planning to rebase the checkout->connect change on bknudson's when it merges.15:11
dhellmannbnemec: ok, got it15:11
dhellmannI'm going to fast-track the lock fix in https://review.openstack.org/#/c/79655/3 so we get rid of the instability in the gate15:11
bnemec+1 from me15:12
dhellmannit's in the gate queue now15:13
bnemec\o/15:13
dhellmannbknudson: after that lock fix merges, we can continue normal reviewing of your change (and others)15:13
bknudsondhellmann: ok, thanks.15:14
dhellmannhttps://review.openstack.org/#/c/79655/ has landed15:19
dhellmannrelease the hounds!15:19
*** rpodolyaka has joined #openstack-oslo15:25
*** rpodolyaka has quit IRC15:40
openstackgerritCyril Roelandt proposed a change to openstack/oslo-incubator: Python 3: enable more tests  https://review.openstack.org/7978115:42
openstackgerritZhongyue Luo proposed a change to openstack/oslo-incubator: Adds test condition in test_periodic  https://review.openstack.org/7207115:43
openstackgerritZhongyue Luo proposed a change to openstack/oslo-incubator: Remove redundant test cases in test_imageutils  https://review.openstack.org/7703015:43
openstackgerritZhongyue Luo proposed a change to openstack/oslo-incubator: Clean up test_imageutils  https://review.openstack.org/7809315:43
openstackgerritIlya Pekelny proposed a change to openstack/oslo-incubator: Fix migrations changing the type of deleted column  https://review.openstack.org/7241915:44
*** yamahata has quit IRC15:52
*** rpodolyaka has joined #openstack-oslo15:58
*** rpodolyaka has quit IRC16:00
*** rpodolyaka has joined #openstack-oslo16:00
openstackgerritBen Nemec proposed a change to openstack/oslo-incubator: Set sql_mode callback on connect instead of checkout  https://review.openstack.org/7862816:04
* dhellmann wants an openstackgerrit alert for "all of the check tests passed on a change..."16:05
*** rpodolyaka has quit IRC16:27
*** rpodolyaka has joined #openstack-oslo16:29
*** saikrishna_ has joined #openstack-oslo16:35
zhiyandhellmann: hello, just response your comments in https://bugs.launchpad.net/glance/+bug/1279000 , it will be cool if you can take a look.16:55
openstackgerritDoug Hellmann proposed a change to openstack/oslo.version: Switch to oslosphinx  https://review.openstack.org/7999816:55
openstackgerritDoug Hellmann proposed a change to openstack/oslo.version: Switch to oslosphinx  https://review.openstack.org/7999816:56
*** arnaud has joined #openstack-oslo17:00
*** harlowja_away is now known as harlowja17:04
*** viktors is now known as viktors_17:10
*** morganfainberg_Z is now known as morganfainberg17:13
openstackgerritDoug Hellmann proposed a change to openstack/oslo.version: Switch to oslosphinx  https://review.openstack.org/7999817:14
*** I159 has quit IRC17:34
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001817:36
*** openstackstatus has quit IRC17:38
*** openstackstatus has joined #openstack-oslo17:39
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001817:47
*** rpodolyaka has quit IRC17:52
*** rpodolyaka has joined #openstack-oslo17:56
*** rpodolyaka has quit IRC17:57
*** rpodolyaka has joined #openstack-oslo18:01
arnauddhellmann: hi18:08
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001818:16
openstackgerritNikola Dipanov proposed a change to openstack-dev/hacking: Add an import whitelist helper factory  https://review.openstack.org/8001818:17
*** rpodolyaka has quit IRC18:20
*** rpodolyaka has joined #openstack-oslo18:23
*** rpodolyaka has quit IRC18:47
*** saikrishna_ has quit IRC19:03
*** rpodolyaka has joined #openstack-oslo19:05
rpodolyakabnemec: do you mind if I update your sql-mode patch in-place instead of leaving a nit comment? (I want to add  another execute() statement to the test case, so we actually check that connection have been checked out from the pool twice for execution of two sql queries, but sql_mode variable has been set only once)19:09
bnemecrpodolyaka: Sure, go for it.19:10
rpodolyakabnemec: ok, thx19:10
bnemecIt's not a terribly good test anyway since it's more testing the Fake classes, but I figured at least if someone starts monkeying around with this it will let them know how it's supposed to work.19:10
openstackgerritRoman Podoliaka proposed a change to openstack/oslo-incubator: Set sql_mode callback on connect instead of checkout  https://review.openstack.org/7862819:10
rpodolyakabnemec: yeah, kind of19:11
*** zhiyan is now known as zhiyan_19:20
bnemecHeh, so that's why I keep getting pep8 failures locally: https://bugzilla.redhat.com/show_bug.cgi?id=105559719:24
*** rpodolyaka has quit IRC19:31
*** rpodolyaka has joined #openstack-oslo19:31
rpodolyakawow, that's interesting19:32
rpodolyakayou can no longer trust sort these days :)19:33
bnemecYeah, I kept meaning to look into whether there was another switch we should add to the sort command, but then I saw a commit that mentioned this bug.19:34
bnemec+1 to procrastination :-)19:34
rpodolyakaI'm just used to think that coreutils are rock solid, and when there is a bug in grep/sort/find/etc, I'm just like, wtf?! :)19:46
arnauddhellmann: dims_: we would need an oslo.vmware 0.2.120:06
arnaudcontains support for storage policy based management20:07
arnaudand bug fixes20:07
dhellmannarnaud: are you ready with what's merged now, or do you have something else to land first?20:10
arnaudready with what is merged now20:10
dhellmannarnaud: commit 6246abd659b5e6e00a473faf4be46c081d122fd5 ?20:11
arnaudyes20:11
dhellmannarnaud: I see "add PBM client" and a couple of requirements updates20:12
dhellmannperhaps this should be 0.3?20:12
arnaudit depends the meaning20:12
arnaudif switching from 0.2 to 0.3 means add a new feature to the lib20:12
dhellmann0.2.1 should have only bug fixes for 0.220:13
arnaudyes it should be 0.320:13
dhellmannno new features20:13
arnaudok20:13
dhellmannso 0.3?20:13
arnaud0.3 makes more sense I agree20:13
dhellmannok, tagging...20:13
arnaudthank you dhellmann20:13
dhellmann0.3 tag pushed, and the jobs are running to create the release20:14
arnaudcool thanks20:14
dhellmannarnaud: you can watch on http://status.openstack.org/zuul/ and let me know if you don't see it on pypi in an hour or so20:14
arnaudok20:14
dims_+1 to 0.3 (a bit late)20:14
dims_:)20:14
dims_arnaud, you will need to update requirements repo20:15
arnaudyes20:15
arnaudI will do it20:15
dhellmannarnaud: at some point we should talk about when you're going to be ready to tag a 1.x20:16
dims_arnaud, here's my plug to the nova guys to switch to oslo.vmware - http://markmail.org/message/vgeqak5cinwnjb6j20:17
arnaudok dhellmann : I think at this point, there is still some work to do to know what needs to be extracted from nova (and useful for Glance, Cinder and others)20:17
arnauddims_: we have a patch out for review to use oslo.vmware in nova20:17
arnaudcurrently glance is the only one fully integrated with it afaik20:18
dims_arnaud, ah cool, got a review #?20:18
dhellmannarnaud: ok, perhaps for juno, then?20:18
arnaudlet me find it20:18
dims_thanks20:18
arnaudyes dhellmann juno is what we expect20:18
dhellmannmarkmc has (mostly) convinced me that oslo libs should version our releases based on the 6 month cycle, where possible20:19
arnaudhttps://review.openstack.org/#/c/70175/1420:19
dims_thanks arnaud20:19
arnaud+dependencies on this patch20:19
arnaudthere is a chain of patches adding support for vsan+oslo.vmware integration in nova20:19
dhellmannarnaud: are those part of a ffe for rc1?20:20
arnaudno20:20
arnaudunfortunately20:20
*** rpodolyaka has quit IRC20:20
*** rpodolyaka has joined #openstack-oslo20:21
*** rpodolyaka has quit IRC20:21
*** rpodolyaka1 has joined #openstack-oslo20:21
dhellmannarnaud: ok20:25
dhellmanndims_, arnaud : how are you handling the feature freeze for the library?20:27
*** rpodolyaka1 has quit IRC21:05
*** rpodolyaka has joined #openstack-oslo21:09
*** mriedem has quit IRC21:45
*** rpodolyaka has quit IRC21:52
*** rpodolyaka has joined #openstack-oslo21:56
*** rpodolyaka has quit IRC22:03
*** rpodolyaka has joined #openstack-oslo22:10
*** rpodolyaka has quit IRC22:28
*** rpodolyaka has joined #openstack-oslo22:30
*** harlowja has quit IRC22:30
*** harlowja has joined #openstack-oslo22:31
*** dims_ has quit IRC23:02
*** mriedem has joined #openstack-oslo23:09
*** dims_ has joined #openstack-oslo23:16
boris-42dhellmann hi23:48
*** yamahata has joined #openstack-oslo23:50

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