Tuesday, 2014-12-16

vkmchmm whats up with grenade-dsvm00:01
*** X019 has joined #openstack-trove00:08
adam_gvkmc, a new setuptools released over the weekend and broke the world00:12
vkmcadam_g, ohh, thanks for the info00:12
adam_gvkmc, http://pad.lv/1402747 specifically00:13
adam_ganyone know how to get apidocs updated to make the checkniceness test happy?00:14
adam_gfirst its failing jsoncheck() because of missing newlines at the end of file. i use openstack-jsoncheck to reformat them, and a later test fails because there are new ines00:15
*** david-lyle is now known as david-lyle_afk00:35
*** X019 has quit IRC01:27
*** Riddhi has quit IRC01:40
*** X019 has joined #openstack-trove01:42
*** haomaiwang has joined #openstack-trove01:57
*** erkules_ has joined #openstack-trove02:05
*** erkules has quit IRC02:08
*** Riddhi has joined #openstack-trove02:08
*** Riddhi has quit IRC02:10
*** Riddhi has joined #openstack-trove02:11
*** Barker has joined #openstack-trove03:00
*** eghobo has quit IRC03:01
*** rwsu has quit IRC03:09
*** erkules_ is now known as erkules03:22
*** vkmc has quit IRC03:30
*** radez is now known as radez_g0n304:06
*** haomaiwang has quit IRC04:22
*** haomaiwang has joined #openstack-trove04:23
*** flwang1 has quit IRC04:33
*** grapex has joined #openstack-trove04:34
*** grapex_ has joined #openstack-trove04:35
*** zacksh has quit IRC04:36
*** Isotopp has quit IRC04:36
*** zacksh has joined #openstack-trove04:38
*** Isotopp has joined #openstack-trove04:38
*** grapex_ has quit IRC04:39
*** grapex has quit IRC04:39
*** grapex has joined #openstack-trove04:40
*** grapex has quit IRC04:44
*** Barker has quit IRC04:45
*** tomblank has quit IRC04:45
openstackgerritSushil Kumar proposed openstack/trove: Fixes package configuration method in pkg.py  https://review.openstack.org/14176705:41
*** ajayaa has joined #openstack-trove06:05
*** vigneshvar has joined #openstack-trove06:12
*** nshah has left #openstack-trove06:18
*** ajayaa has quit IRC06:19
*** ajayaa has joined #openstack-trove06:26
*** vigneshvar has quit IRC06:28
*** ajayaa has quit IRC06:30
*** ajayaa has joined #openstack-trove06:32
*** ajayaa has quit IRC06:51
*** vigneshvar has joined #openstack-trove06:54
*** ajayaa has joined #openstack-trove07:04
*** pcaruana has joined #openstack-trove07:20
*** Longgeek has joined #openstack-trove07:24
*** flwang1 has joined #openstack-trove07:38
*** sgotliv has quit IRC07:42
openstackgerritSushil Kumar proposed openstack/trove: Corrects INCLUDE_MARKER_OPERATORS for MySQL  https://review.openstack.org/14202008:17
*** romainh has joined #openstack-trove08:21
*** boblebauce has joined #openstack-trove08:39
*** sgotliv has joined #openstack-trove08:41
*** Krast has joined #openstack-trove08:42
*** sgotliv has quit IRC08:44
*** k4n0 has joined #openstack-trove08:47
*** Longgeek has quit IRC08:49
*** Longgeek has joined #openstack-trove08:50
*** hollandais has quit IRC08:58
*** hollandais has joined #openstack-trove09:12
*** ekarlso- has quit IRC10:07
*** ekarlso- has joined #openstack-trove10:07
openstackgerritsarad patel proposed openstack/trove: Adds negative unittests to test_backup_controller.py  https://review.openstack.org/14205610:24
*** haomaiwang has quit IRC11:14
*** vkmc has joined #openstack-trove11:25
*** Riddhi has quit IRC11:54
*** nexusz99 has quit IRC12:18
*** sgotliv has joined #openstack-trove12:25
openstackgerritSushil Kumar proposed openstack/trove: Corrects error message for user-access-revoke  https://review.openstack.org/14209012:35
*** IanGovett has joined #openstack-trove12:56
*** grapex has joined #openstack-trove13:02
*** sgotliv has quit IRC13:03
*** grapex_ has joined #openstack-trove13:04
*** grapex has quit IRC13:08
*** sgotliv has joined #openstack-trove13:18
*** lifeless_ has joined #openstack-trove13:33
*** lifeless has quit IRC13:35
*** vigneshvar has quit IRC13:42
*** tomblank has joined #openstack-trove13:56
*** Riddhi has joined #openstack-trove14:03
*** Riddhi has quit IRC14:05
*** Riddhi has joined #openstack-trove14:05
*** sgotliv has quit IRC14:14
*** jwang__ has joined #openstack-trove14:20
*** amrith is now known as _amrith_14:22
*** sgotliv has joined #openstack-trove14:26
*** k4n0 has quit IRC14:30
*** jwang_ has quit IRC14:30
*** johnma has quit IRC14:31
*** k4n0 has joined #openstack-trove14:31
*** mattgriffin has joined #openstack-trove14:34
*** Barker has joined #openstack-trove14:34
*** Barker has quit IRC14:42
*** Riddhi has quit IRC14:46
*** Riddhi has joined #openstack-trove14:50
*** pboros has joined #openstack-trove14:50
*** jcru has joined #openstack-trove14:54
*** X019 has quit IRC14:55
*** X019 has joined #openstack-trove15:08
*** pboros has quit IRC15:08
*** robertmyers has joined #openstack-trove15:12
*** flaper87 has quit IRC15:18
*** flaper87 has joined #openstack-trove15:18
*** Riddhi has quit IRC15:20
icchaX019: around?15:24
*** sgotliv has quit IRC15:25
*** sgotliv has joined #openstack-trove15:25
*** k4n0 has quit IRC15:25
*** k4n0 has joined #openstack-trove15:25
*** Barker has joined #openstack-trove15:28
*** Barker has quit IRC15:28
*** sriram_tesora has joined #openstack-trove15:31
*** _amrith_ is now known as amrith15:34
*** newb_ has joined #openstack-trove15:36
*** Barker has joined #openstack-trove15:37
*** pcaruana has quit IRC15:42
*** Riddhi has joined #openstack-trove15:46
*** isviridov is now known as isviridov_away16:05
*** ajayaa has quit IRC16:07
*** jmontemayor has joined #openstack-trove16:15
*** nshah has joined #openstack-trove16:24
*** IanGovett has quit IRC16:42
*** david-lyle_afk is now known as david-lyle16:44
*** IanGovett has joined #openstack-trove16:50
*** vigneshvar has joined #openstack-trove16:51
amrithX019, ping ...16:51
X019I mean16:52
X019are we going to just get the logs directly to the console16:53
X019or go the alternative of just saving them to swift?16:53
X019https://review.openstack.org/#/c/131610/16:53
*** ajayaa has joined #openstack-trove16:54
zhiyanSlickNik: amrith hi, do you think we can make osprofiler integration change in k-1? https://review.openstack.org/#/c/116653/ i think it's ready for that. thx16:55
*** georgelorch has quit IRC17:04
*** grapex has joined #openstack-trove17:07
*** grapex__ has joined #openstack-trove17:08
*** grapex_ has quit IRC17:09
SlickNikzhiyan: kilo-1 is pretty locked down atm and we're hitting some gate issues. imo, the osprofiler changes should make it in early kilo-2.17:09
*** grapex__ has quit IRC17:11
*** grapex_ has joined #openstack-trove17:12
*** grapex has quit IRC17:12
icchaX019: i think the prev meeting we sotr of proposed oushing them directly to swigt17:13
iccha*swift17:13
icchaX019: you can look at online log meetings here: http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/17:14
*** vigneshvar has quit IRC17:14
SlickNikiccha / X019: I think the concern was sending messages that were too large over RPC.17:15
SlickNikCan alleviate that concern by either having a hard limiting on the log messages size returned, or copying to swift.17:16
icchaX019: feel free to upload what you have so far and share it :) SlickNik amrith and other folks can take a look and help shape it up for tomorrow's meeting :) the trove community is here to mentor you17:20
*** vigneshvar has joined #openstack-trove17:33
*** IanGovett has quit IRC17:33
*** boblebauce has quit IRC17:38
amrithSlickNik, got a second to chat about https://review.openstack.org/#/c/129714/ ?17:41
*** IanGovett has joined #openstack-trove17:42
zhiyanoh probably oslo.db caused sqlalchemy version issue on req installation.. SlickNik17:44
zhiyanSlickNik: bad news but i'm ok...continue to wait for a while..17:45
amrithzhiyan, trove doesn't use oslo.db ;)17:46
zhiyanamrith: heh, ok..17:47
zhiyanso SlickNik amrith for the change  https://review.openstack.org/#/c/116653/ do you think anything i need to address, like concerns?17:48
zhiyanoh, btw, the troveclient change is already merged week ago, so i'm not sure if  https://review.openstack.org/#/c/116653/ is ok to wait k-2...17:48
amrithadam_g, what operating system do you run on your machine17:54
amrithdoh, dumb question, sorry ... I just saw your email address ;)17:54
adam_gamrith, lol :)17:54
amrithso I'm assuming it is MacOS :) :) :)17:54
amrithok, there was a problem that I thought had been fixed17:54
amrithand it related to the doc files that are generated17:55
adam_gseems like that openstack-doc-tools thing is crazy17:55
amrithand the fact that they aren't identically generated on all os's17:55
amrithhave you rebased recently?17:58
adam_gamrith, yesterday, looks to be clean17:59
amrithrobertmyers, yt?17:59
*** sgotliv has quit IRC17:59
amrithrobertmyers, could you give an assist with https://review.openstack.org/#/c/115811/1118:00
adam_gamrith, the openstack-doc-tools jsoncheck checks and "fixes" the issue, then the same library's doctest fails on the thing that got fixed.18:00
amrithyes18:00
amriththere was an issue that robert and I discussed 12/0318:01
amrithhad to do with the rendering of the dictionary18:01
amrithI thought that may have been the issue till I saw your email address18:01
amrithso I'm assuming that it isn't.18:01
*** romainh has left #openstack-trove18:02
adam_gamrith, i can look into it,  just wasn't sure i was doing it right. first time i've touched os-doc-tools18:03
amrithwhat I understood was that if you regenerate the files and tox passes locally you should be all set.18:03
SlickNikamrith: About https://review.openstack.org/#/c/12971418:05
amrithyes18:05
*** k4n0 has quit IRC18:06
SlickNiktrove/common has the code that's common across the trove services — I don't think other OpenStack projects have an equivalent.18:07
amrithcinder and swift certainly do18:08
amrithcinder/cinder/common and swift/swift/common18:08
amrithso does keystone18:08
amrithnova does not18:08
SlickNikI was looking at nova, nvm18:08
SlickNikSo what's the reason the other projects don't choose to put it there?18:08
SlickNikI would think the i18n code would get used across the place?18:09
amrithI don't know but in one of the email threads about how to handle the i18n graduation there was a description of where to put i18n18:09
amrithlet me check if I bookmarked it18:09
amrithSlickNik, please see: http://docs.openstack.org/developer/oslo.i18n/usage.html18:12
amrithit indicates that the location is app/i18n.py18:13
amrithnot saying it is authoritative, or prescriptive18:13
amrithbut that appears to be the template that people have used18:13
amriththis link was referenced in the (now abandoned) blueprint in the references section.18:14
SlickNikYeah, it's totally possible that that's an example.18:15
SlickNikYou can do exactly the same thing using app/common/i18n18:15
amrithpersonally, I'd rather not for two reasons18:16
amrith1. it will put us out of step with the other projects18:16
amrith2. it will invalidate the reviews on all dependent changes18:16
amrithand given how fun it is to get gate to pass, I'm reluctant to do that.18:16
amrithbut it is your call.18:16
SlickNikYou do break the trove/common abstraction though, and I don't like that.18:17
SlickNikFor 1. I don't think that has any implications.18:18
SlickNik2 does have implications since the patches are now dependent on each other.18:19
amriththey would have, even if they weren't dependent18:20
amriththere are implications of moving the file because a bunch of imports would have to change18:20
amrithand since a lot of what I'm changing is imports18:20
amriththere will be merge conflicts18:21
amriththat have to be manually resolved18:21
amrithie. gerrit won't automagically wave its hands over this one ;)18:21
SlickNikWell, you're always going to have to do that in a project if you touch the same files.18:22
*** ajayaa has quit IRC18:22
amrithand that merge will impact 66 files in the primary patch18:23
SlickNikMy point was that by forcing a dependency on the changes when one doesn't exist, you have to do it even if you _don't_ touch the same files.18:23
SlickNikand that sucks ;)18:23
amrithand five patches that follow.18:23
amrithwell, it would have been easy to make it one patch but that didn't seem to fly either.18:26
SlickNikamrith: That seems like a horrible idea — you're having a hard enough time getting reviews on these smaller patches no? :)18:27
amrithyou bet ... but it'd make the merge so much easier!18:28
amrithso it is a trade ...18:28
amrith1. trove/common abstraction18:28
amrith2. painful merge18:28
amrith3. painful review18:28
amrithyou can't solve all three18:28
amrithpick the two you want to solve18:28
amrithI'd recommend 2 and 318:28
amrith;)18:28
amriththen, all I have to do is remove the wf -1 :)18:29
*** mriedem has joined #openstack-trove18:47
mriedemdoes anyone know if there is a bug reported for this trove failure in the gate?18:47
mriedemhttp://logs.openstack.org/21/140121/8/gate//gate-trove-functional-dsvm-mysql/29dacef/console.html18:47
mriedemit's blowing up that job http://status.openstack.org/elastic-recheck/data/uncategorized.html18:47
SlickNikmriedem: There isn't a bug — it's caused by a devstack issue.18:48
SlickNikmriedem: https://review.openstack.org/#/c/141564/18:48
SlickNikmriedem: actually nvm, there _is_ a bug18:49
mriedemso there is a bug :) https://bugs.launchpad.net/devstack/+bug/140222718:49
SlickNikAnd this is it https://bugs.launchpad.net/devstack/+bug/140222718:49
SlickNikYup18:49
mriedemSlickNik: thanks, i'll get an e-r query up18:49
SlickNikmriedem: Thanks!18:49
SlickNikbtw, amrith: I was looking into ^, and that's the reason that the newer trove patches (including the later oslo fixes) are failing functional-dsvm int-test jobs.18:51
*** mattgriffin has quit IRC18:51
SlickNikdoesn't have anything to do with the oslo fixes themselves.18:51
*** vigneshvar has quit IRC18:55
amriththx18:58
amrithsgotliv was also asking abotu those this morning18:59
amrithand it appears that grenade is failing for everyone as well18:59
SlickNikamrith: The grenade problems was an issue with the latest setuptools release over the weekend, I believe.19:00
amrithwas having the problem this morning as well19:00
vkmchttps://bugs.launchpad.net/glance/+bug/1402747 ^19:04
SlickNikokay, let me dig deeper to see if I can find out what's going on here.19:04
SlickNikamrith: For the i18n patch, I have a solution that addresses all 3 of the points you brought up.19:06
SlickNik1. move it to common, and 2. and 3. I can make the changes and rebase. ;)19:07
*** vigneshvar has joined #openstack-trove19:07
amrithSlickNik, that's not a solution, just moving it around. I'll move i18n.py as you suggest.19:08
SlickNikamrith: btw, just talked to dhellmann about the doc, The wiki page was meant just as an example — the main take-away is not to use the oslo.i18n module directly. The adapter can exist in the project where it makes sense.19:10
amrithok19:11
SlickNikamrith: also, almost done with the rebase and merge fixes — so I can push those up.19:15
amrithplease don't19:15
amrithif you don't mind19:16
*** mattgriffin has joined #openstack-trove19:16
SlickNikSure, np. The merge conflicts weren't too bad — just 3 files.19:18
SlickNikcommon/limits.py,  common/exception.py,  and common/utils.py19:18
amrithfinal tox running now19:19
amrithI'll be able to push the changes up shortly19:19
amrithyes, already addressed those19:19
vkmchey.. if someone has a moment.. where is it supposed to be trove_test.sqlite? http://docs.openstack.org/developer/trove/dev/manual_install.html#prepare-database19:27
*** mriedem has left #openstack-trove19:29
openstackgerritamrith proposed openstack/trove: Obsolete oslo-incubator modules - gettextutils (now oslo.i18n)  https://review.openstack.org/12971419:30
*** Barker has quit IRC19:30
*** Barker has joined #openstack-trove19:34
SlickNikvkmc: I'm not 100% sure since I've almost always used trove with mysql and haven't tried it with sqllite.19:38
vkmcSlickNik, no problem, I ran trove-manager without those params and appears to work19:39
SlickNikvkmc: Good to know, thanks. That doc probably needs to be cleaned up. denis_makogon was working on that, but I'm not sure how far he got.19:40
vkmcSlickNik, I could try to help with that :) I'm setting up Trove in Fedora manually and I already noticed some changes in OpenStack clients that impact that doc19:42
*** sgotliv has joined #openstack-trove19:43
SlickNikvkmc: That would be awesome. :) Please update that document with what you find.19:43
vkmcSlickNik, sure, I will catch up with denis_makogon first19:44
SlickNikvkmc: Awesome thanks!19:45
vkmcnp, glad to help19:45
SlickNikvkmc: If I recall correctly, he's currently on vacation but will be back on the 22nd.19:46
SlickNikjust fyi19:46
vkmcSlickNik, ohh good to know, thanks for the heads up19:48
openstackgerritamrith proposed openstack/trove: Obsolete oslo-incubator modules - exception  https://review.openstack.org/13305119:49
openstackgerritamrith proposed openstack/trove: Obsolete oslo-incubator modules - wsgi  https://review.openstack.org/13306819:55
openstackgerritamrith proposed openstack/trove: Obsolete oslo-incubator modules - processutils  https://review.openstack.org/14108120:02
*** exploreshaifali has joined #openstack-trove20:03
*** tomblank_ has joined #openstack-trove20:04
*** IanGovett has quit IRC20:04
*** tomblank has quit IRC20:05
*** tomblank_ is now known as tomblank20:05
*** IanGovett has joined #openstack-trove20:09
openstackgerritamrith proposed openstack/trove: Eliminate redundant modules from oslo-incubator  https://review.openstack.org/14109020:12
openstackgerritamrith proposed openstack/trove: resync oslo-incubator code  https://review.openstack.org/14188020:13
amrithSlickNik, all done.20:13
sgotlivSlickNik, ping20:16
*** flwang1 has quit IRC20:26
*** Barker has quit IRC20:35
*** Barker has joined #openstack-trove20:47
*** Barker has quit IRC20:51
*** flwang1 has joined #openstack-trove20:53
*** Barker has joined #openstack-trove20:55
*** jcru has quit IRC20:57
*** tomblank has quit IRC21:01
openstackgerritShayne Burgess proposed openstack/trove: Deletes volume on instance delete after resize  https://review.openstack.org/13745221:08
SlickNikamrith: Thanks!21:09
SlickNiksgotliv: here now, what's up?21:09
sgotlivSlickNik, how are you?21:09
*** tomblank has joined #openstack-trove21:09
SlickNiksgotliv: pretty good — how're things going with you?21:09
sgotlivSlickNik, it seems like I found a way to keep a remote debugger21:10
sgotlivSlickNik, actually flaper87 gave an idea :)21:10
SlickNikyeah, I saw flaper87's suggestion on the patch.21:10
SlickNikThx!21:10
sgotlivSlickNik, another thing21:11
sgotlivI have to provide our user a way to create their own images21:11
sgotlivso I thought about using trove-integration on Fedora/RHEL21:11
sgotlivthe thing is that its totally Ubuntu oriented, can we make it more Fedora friendly?21:12
sgotlivI mean, we'll do that. All we need are just reviews21:13
sgotlivSlickNik, what do you think?21:13
SlickNiksgotliv: Seems reasonable. I'm wondering if we almost need a separate way to build images that's abstracted out from the trove integration (redstack) script.21:14
sgotlivSlickNik, you'll make me a happy guy!21:15
sgotlivSlickNik, this is a next thing I want to discuss21:15
SlickNikThat way the build image piece can be extended, and documented separately for folks — and redstack can just call into that to build the images.21:15
sgotlivlet's take a sahara-image-elements as an example21:15
sgotlivSlickNik, +121:15
sgotlivSlickNik, so how about creating trove-image-elements?21:18
sgotlivand take a relevant code from trove-integration21:18
SlickNiksgotliv: We already have the image-elements in trove-integration — I'd suggest we keep it there for now.21:20
SlickNikWhat seems interesting is to have a script / entry-point that exists just to build images.21:20
SlickNikWhich can be used and extended in isolation.21:20
sgotlivSlickNik, ok, make sense21:21
SlickNik(in isolation of ./redstack)21:21
SlickNikSomething like https://github.com/openstack/sahara-image-elements/blob/master/diskimage-create/diskimage-create.sh21:21
sgotlivI understand21:21
sgotlivok, sounds perfect, I'll see how I can help21:22
*** flwang1 has quit IRC21:23
*** flwang2 has joined #openstack-trove21:23
SlickNiksgotliv: Sounds good -- thank you!21:23
*** sgotliv has quit IRC21:31
*** tomblank has quit IRC21:37
openstackgerritamrith proposed openstack/trove: Removal of uneeded guest mount_point from taskmgr  https://review.openstack.org/8680821:39
openstackgerritamrith proposed openstack/trove: Removal of uneeded guest mount_point from taskmgr  https://review.openstack.org/8680821:42
*** Longgeek has quit IRC21:53
vkmchey guys, I'm getting a 401 when trying to list instances... http://paste.openstack.org/show/152026/22:01
*** jmontemayor has quit IRC22:01
vkmcdid someone see something alike?22:01
*** johnma has joined #openstack-trove22:06
*** amrith is now known as _amrith_22:14
*** newb_ has quit IRC22:20
*** jmontemayor has joined #openstack-trove22:25
*** vigneshvar has quit IRC22:27
*** johnma has quit IRC22:35
openstackgerritMerged openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929422:39
*** robertmyers has quit IRC22:44
*** flwang2 has quit IRC22:45
*** jmontemayor has quit IRC22:48
*** jmontemayor has joined #openstack-trove22:52
*** jmontemayor_ has joined #openstack-trove22:53
*** jmontemayor has quit IRC22:56
*** sriram_tesora has quit IRC23:00
*** Barker has quit IRC23:13
*** IanGovett has quit IRC23:21
*** jmontemayor_ has quit IRC23:47
*** newb has joined #openstack-trove23:53
*** vkmc has quit IRC23:55
*** mattgriffin has quit IRC23:57
*** flwang1 has joined #openstack-trove23:59

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