Monday, 2014-10-20

*** amrith is now known as _amrith_00:13
*** Longgeek has joined #openstack-trove00:23
*** newb has quit IRC00:23
*** Longgeek has quit IRC00:27
*** haomaiwa_ has quit IRC01:08
*** amcrn has joined #openstack-trove01:08
*** haomaiwang has joined #openstack-trove01:08
*** haomaiwang has quit IRC01:13
*** erkules_ has joined #openstack-trove01:23
*** erkules has quit IRC01:26
*** haomaiwang has joined #openstack-trove01:27
*** Longgeek has joined #openstack-trove01:38
*** Longgeek has quit IRC01:43
*** nosnos has joined #openstack-trove01:54
*** _amrith_ is now known as amrith01:59
*** haomaiwang has quit IRC02:00
*** haomaiwang has joined #openstack-trove02:01
*** haomaiw__ has joined #openstack-trove02:04
*** haomaiwang has quit IRC02:08
*** kevinconway has joined #openstack-trove02:17
*** jasonb365 has joined #openstack-trove02:21
*** kevinconway has quit IRC02:23
*** ramishra has joined #openstack-trove02:28
*** mrsnivvel has quit IRC02:47
*** haomaiw__ has quit IRC02:55
*** haomaiwang has joined #openstack-trove02:56
*** mrsnivvel has joined #openstack-trove02:59
*** haomaiw__ has joined #openstack-trove03:11
*** haomaiwang has quit IRC03:15
*** jasonb365 has quit IRC03:18
*** haomaiw__ has quit IRC03:20
*** haomaiwang has joined #openstack-trove03:20
*** Longgeek has joined #openstack-trove03:22
*** Longgeek has quit IRC03:24
*** Longgeek has joined #openstack-trove03:24
*** nosnos has quit IRC03:24
*** nosnos has joined #openstack-trove03:25
*** nosnos has quit IRC03:29
*** nosnos has joined #openstack-trove03:48
*** ramishra has quit IRC03:52
*** ramishra has joined #openstack-trove03:58
*** ramishra has quit IRC04:10
*** ramishra has joined #openstack-trove04:16
openstackgerritSushil Kumar proposed a change to openstack/trove: Imports troveclient.compat.client as module  https://review.openstack.org/6636204:40
*** sgotliv has joined #openstack-trove04:44
*** fifieldt has joined #openstack-trove04:48
*** hollandais has quit IRC04:57
*** hollandais has joined #openstack-trove04:59
*** vkmc has quit IRC05:03
*** Longgeek has quit IRC05:08
*** Longgeek has joined #openstack-trove05:23
*** tshefi has joined #openstack-trove05:24
*** ramishra_ has joined #openstack-trove05:24
*** ramishra has quit IRC05:28
openstackgerritZhi Yan Liu proposed a change to openstack/trove: Integrate OSprofiler and Trove  https://review.openstack.org/11665305:33
*** amcrn has quit IRC05:34
*** nosnos has quit IRC05:59
*** nosnos has joined #openstack-trove06:00
openstackgerritOpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex  https://review.openstack.org/12680706:09
*** k4n0 has joined #openstack-trove06:11
*** ramishra has joined #openstack-trove06:16
*** ramishra_ has quit IRC06:16
openstackgerritSergey Gotliv proposed a change to openstack/trove: Yum install should get a list of packages as a string  https://review.openstack.org/12952306:29
*** haomaiwang has quit IRC06:34
*** haomaiwa_ has joined #openstack-trove06:34
*** ramishra has quit IRC06:38
*** nosnos has quit IRC06:39
*** nosnos has joined #openstack-trove06:39
openstackgerritSergey Gotliv proposed a change to openstack/trove: Rename generic variable named with mysql specific name  https://review.openstack.org/12952806:47
*** sgotliv has quit IRC06:52
*** X019 has quit IRC07:02
*** X019 has joined #openstack-trove07:09
*** rushiagr_away is now known as rushiagr07:10
*** Longgeek has quit IRC07:16
*** Longgeek has joined #openstack-trove07:19
*** Longgeek has quit IRC07:24
*** ramishra has joined #openstack-trove07:43
*** boblebauce has joined #openstack-trove07:46
*** haomaiw__ has joined #openstack-trove07:48
*** haomaiwa_ has quit IRC07:51
*** ramishra has quit IRC08:09
*** ramishra has joined #openstack-trove08:15
openstackgerritSergey Gotliv proposed a change to openstack/trove: Rename generic variable named with mysql specific name  https://review.openstack.org/12952808:19
*** sgotliv has joined #openstack-trove08:20
*** Longgeek has joined #openstack-trove08:20
*** ramishra has quit IRC08:25
*** Longgeek has quit IRC08:26
*** sgotliv has quit IRC08:30
*** isviridov_away is now known as isviridov08:36
*** rushiagr is now known as rushiagr_away08:38
*** Longgeek has joined #openstack-trove08:45
*** Longgeek has quit IRC08:47
*** Longgeek has joined #openstack-trove08:47
*** sgotliv has joined #openstack-trove08:53
*** rushiagr_away is now known as rushiagr08:57
openstackgerritA change was merged to openstack/trove: Allow users the ability to update an instance name  https://review.openstack.org/9270109:10
*** ramishra has joined #openstack-trove09:22
*** vkmc has joined #openstack-trove09:22
*** vkmc has joined #openstack-trove09:22
*** k4n0 has quit IRC09:32
*** exploreshaifali has joined #openstack-trove09:33
*** k4n0 has joined #openstack-trove09:34
*** denis_makogon_ has joined #openstack-trove09:57
*** sgotliv has quit IRC10:03
*** Longgeek has quit IRC10:14
*** sgotliv has joined #openstack-trove10:14
*** Longgeek has joined #openstack-trove10:14
*** rushiagr is now known as rushiagr_away10:17
*** exploreshaifali has quit IRC10:18
openstackgerritA change was merged to openstack/trove: Logging audit for guestagent/redis  https://review.openstack.org/10780510:19
*** ankush has joined #openstack-trove10:32
*** newb has joined #openstack-trove10:33
*** sgotliv has quit IRC10:43
*** newb has quit IRC10:55
*** exploreshaifali has joined #openstack-trove10:56
*** denis_makogon has quit IRC11:11
*** denis_makogon_ is now known as denis_makogon11:11
*** dmakogon_ has joined #openstack-trove11:12
*** Longgeek has quit IRC11:14
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Update some log calls for translation and lazyness  https://review.openstack.org/11940211:14
*** Longgeek has joined #openstack-trove11:15
*** IanGovett has joined #openstack-trove11:16
denis_makogonguys, feel free to review update manual for Trove installation https://review.openstack.org/#/c/129573/ thanks11:20
*** rushiagr_away is now known as rushiagr11:21
ankushhi denis, thanks for all your guidance, I am able to fix all the issues related to trove on my box and now I am able to create a db instance11:25
*** tomblank has quit IRC11:27
*** sgotliv has joined #openstack-trove11:30
*** Longgeek has quit IRC11:30
*** vkmc has quit IRC11:30
*** Longgeek has joined #openstack-trove11:31
*** Longgeek_ has joined #openstack-trove11:32
*** exploreshaifali has quit IRC11:32
*** amrith is now known as _amrith_11:32
*** Longgeek has quit IRC11:36
openstackgerritA change was merged to openstack/trove: Increase test rate limit to avoid rate limit error  https://review.openstack.org/12561111:39
*** haomaiw__ has quit IRC11:41
denis_makogonankush, so, what's was wrong ?11:46
*** IanGovett has quit IRC11:54
ankushone type mistake  network_label_regex = ^novanetwork$ was having space at the end.  present of this in network_driver = trove.network.nova.NovaNetwork in trove.conf  which confused whether it was nova or neutron network11:56
ankush3rd  ubuntu image has password set for root even though I have put the password in my.cnf still the trove was not able to create dbs and users in mysql. Reinstall mysql without any password , created the image, updated the trove-manage with the latest image and it started working11:57
openstackgerritSylvain Baubeau proposed a change to openstack/trove: Install Redis before changing owner of mount point  https://review.openstack.org/12722112:08
denis_makogonankush, i see, nice to hear that you was able to deal with it =)12:12
ankushthanks denis :)12:13
*** jcru has joined #openstack-trove12:16
*** newb has joined #openstack-trove12:16
*** flaper87|afk is now known as flaper8712:23
*** sgotliv has quit IRC12:30
*** erkules_ is now known as erkules12:34
*** ramishra has quit IRC12:36
denis_makogonankush, you're welcome12:40
*** IanGovett has joined #openstack-trove12:40
*** sgotliv has joined #openstack-trove12:42
*** tomblank has joined #openstack-trove12:43
*** pboros has joined #openstack-trove12:45
*** flaper87 is now known as flaper87|afk12:49
*** radez_g0n3 is now known as radez12:55
*** ankush has quit IRC12:55
*** mrsnivvel has quit IRC13:04
*** miqui has joined #openstack-trove13:06
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Update some log calls for translation and lazyness  https://review.openstack.org/11940213:07
*** _amrith_ is now known as amrith13:16
*** AJaeger has joined #openstack-trove13:23
AJaegerdenis_makogon, I fixed your comment about multiple imports. Thanks for the review13:23
AJaegerWould a patch to fix some more of these problems be welcome?13:23
*** mrsnivvel has joined #openstack-trove13:24
*** exploreshaifali has joined #openstack-trove13:29
denis_makogonAJaeger, we have opened bug for H302, feel free to work on it =)13:46
denis_makogonAJaeger, btw, thanks for taking into account my suggestion13:46
AJaegerdenis_makogon, thanks, will grab it...13:47
denis_makogonAJaeger, take a look at new comment =)13:51
denis_makogonAJaeger, it'll be far more easier to re-write to oslo.i18n once it's in13:52
AJaegerdenis_makogon, you keep me busy ;)13:52
AJaegerdenis_makogon, ok, I'll change it...13:53
denis_makogonAJaeger, don't get me wrong, i still +1 it, each time =)13:53
AJaegerdenis_makogon, I know - but I agree with you that it should be as good as possible ;)13:53
denis_makogonAJaeger, thanks13:54
*** k4n0 has quit IRC13:55
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Enable H301  https://review.openstack.org/12962413:56
*** rushiagr is now known as rushiagr_away13:58
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Update some log calls for translation and lazyness  https://review.openstack.org/11940213:59
*** thedodd has joined #openstack-trove14:01
AJaegerdenis_makogon, the trove imports are quite a mess from a hacking perspective ;)14:01
denis_makogonAJaeger, i know =(14:02
*** kevinconway has joined #openstack-trove14:06
*** robertmyers has joined #openstack-trove14:07
*** nosnos has quit IRC14:09
*** nosnos has joined #openstack-trove14:09
*** nosnos has quit IRC14:14
*** jasonb365 has joined #openstack-trove14:26
*** ramishra has joined #openstack-trove14:27
*** mattgriffin has joined #openstack-trove14:29
*** rwsu has joined #openstack-trove14:30
AJaegerdenis_makogon, ;) I might tackle a few more checks if 129624 goes in quickly, so please review14:39
*** grapex has joined #openstack-trove14:42
*** thedodd has quit IRC14:44
*** Riddhi has joined #openstack-trove14:44
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Enable H301  https://review.openstack.org/12962414:45
denis_makogonAJaeger, sure14:45
*** georgelorch2 has joined #openstack-trove14:51
*** Riddhi has quit IRC14:51
*** Riddhi has joined #openstack-trove14:52
*** georgelorch has quit IRC14:52
*** tshefi has quit IRC14:57
*** georgelorch2 has quit IRC15:00
*** haomaiwang has joined #openstack-trove15:00
*** georgelorch has joined #openstack-trove15:02
*** Barker has joined #openstack-trove15:04
*** jmontemayor has joined #openstack-trove15:07
*** Riddhi has quit IRC15:11
*** haomaiwang has quit IRC15:11
*** tomblank has quit IRC15:11
*** haomaiwa_ has joined #openstack-trove15:12
*** Riddhi has joined #openstack-trove15:26
*** sgotliv has quit IRC15:37
*** tomblank has joined #openstack-trove15:40
*** Riddhi has quit IRC15:43
*** Riddhi has joined #openstack-trove15:44
*** todd_dsm has joined #openstack-trove15:45
*** exploreshaifali has quit IRC15:51
*** georgelorch has quit IRC15:51
*** georgelorch has joined #openstack-trove15:51
*** eghobo has joined #openstack-trove15:58
*** jasonb365_ has joined #openstack-trove15:59
*** Barker has quit IRC16:00
*** ramishra has quit IRC16:00
*** jasonb365 has quit IRC16:00
*** jasonb365_ is now known as jasonb36516:00
*** fifieldt has quit IRC16:03
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - unused modules  https://review.openstack.org/12929216:04
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929416:04
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - lockutils  https://review.openstack.org/12929516:04
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - importutils  https://review.openstack.org/12937816:04
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - excutils  https://review.openstack.org/12965416:04
*** Barker has joined #openstack-trove16:04
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - unused modules  https://review.openstack.org/12929216:05
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929416:06
*** tomblank has quit IRC16:06
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - lockutils  https://review.openstack.org/12929516:07
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - importutils  https://review.openstack.org/12937816:08
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - excutils  https://review.openstack.org/12965416:08
*** tomblank has joined #openstack-trove16:11
*** ramishra has joined #openstack-trove16:13
*** georgelorch has quit IRC16:14
*** haomaiw__ has joined #openstack-trove16:16
*** haomaiwa_ has quit IRC16:18
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - strutils  https://review.openstack.org/12966316:19
*** eghobo has quit IRC16:23
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - network_utils (now netutils)  https://review.openstack.org/12966416:23
*** georgelorch has joined #openstack-trove16:30
*** ramishra has quit IRC16:30
*** exploreshaifali has joined #openstack-trove16:32
*** haomaiw__ has quit IRC16:37
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - jsonutils (now oslo.serialization)  https://review.openstack.org/12966816:37
amrithAJaeger, are you there?16:39
*** newb has quit IRC16:40
*** sriram_tesora has joined #openstack-trove16:40
*** boblebauce has quit IRC16:41
*** todd_dsm has quit IRC16:42
*** thedodd has joined #openstack-trove16:47
*** vkmc has joined #openstack-trove16:58
*** AJaeger_ has joined #openstack-trove16:59
*** mattgriffin has quit IRC16:59
AJaeger_Hi amrith, I'm here now...16:59
*** Longgeek_ has quit IRC17:02
*** Barker has quit IRC17:04
*** radez is now known as radez_g0n317:04
*** Longgeek has joined #openstack-trove17:05
*** mattgriffin has joined #openstack-trove17:06
*** Longgeek has quit IRC17:06
*** dkehn has quit IRC17:08
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929417:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - lockutils  https://review.openstack.org/12929517:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - importutils  https://review.openstack.org/12937817:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - excutils  https://review.openstack.org/12965417:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - strutils  https://review.openstack.org/12966317:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - network_utils (now netutils)  https://review.openstack.org/12966417:09
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - jsonutils (now oslo.serialization)  https://review.openstack.org/12966817:09
*** thedodd has quit IRC17:11
*** tomblank has quit IRC17:13
*** harlowja_away is now known as harlowja17:14
*** kevinconway has quit IRC17:20
*** kevinconway has joined #openstack-trove17:20
*** kevinconway has quit IRC17:23
*** edmondk has joined #openstack-trove17:24
*** saurabhs has joined #openstack-trove17:26
openstackgerritA change was merged to openstack/trove: Miscellaneous Cluster Fixes  https://review.openstack.org/11925917:28
*** isviridov is now known as isviridov_away17:28
*** tomblank has joined #openstack-trove17:35
*** flaper87|afk is now known as flaper8717:39
*** johnma has joined #openstack-trove17:42
*** newb has joined #openstack-trove17:43
*** georgelorch2 has joined #openstack-trove17:43
*** amcrn has joined #openstack-trove17:43
*** georgelorch has quit IRC17:43
*** sgotliv has joined #openstack-trove17:44
openstackgerritamrith proposed a change to openstack/trove-specs: Blueprints for oslo-incubator module migration activities  https://review.openstack.org/12357117:47
*** annashen has joined #openstack-trove17:49
*** georgelorch2 is now known as georgelorch17:52
openstackgerritamrith proposed a change to openstack/trove-specs: Blueprints for oslo-incubator module migration activities  https://review.openstack.org/12357117:56
*** miqui has quit IRC17:58
SlickNik#startmeeting trove-bp-review18:00
openstackMeeting started Mon Oct 20 18:00:09 2014 UTC and is due to finish in 60 minutes.  The chair is SlickNik. Information about MeetBot at http://wiki.debian.org/MeetBot.18:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.18:00
openstackThe meeting name has been set to 'trove_bp_review'18:00
SlickNikWaiting a couple of minutes for folks to trickle in.18:00
SlickNikAgenda at18:01
SlickNik#link https://wiki.openstack.org/wiki/Meetings/TroveBPMeeting18:01
amrith./18:01
dougshelley66 o/18:01
denis_makogono/18:01
vgnbkrtrickle, trickle18:01
amcrnpercolate percolate18:01
schango/18:02
SlickNikOkay, let's get started.18:03
SlickNik#topic Handle oslo-incubator module graduation18:03
amrithhello18:03
amriththat's me18:03
amrithhere comes a paste-bomb18:03
SlickNik#link https://review.openstack.org/#/c/123571/18:03
amriththe proposal in this blueprint initially came from the observation that oslo.concurrency has graduated and now we need to get code from there (not oslo-incubator).18:03
amrithsubsequent to writing this, a new proposal from oslo team indicates that they are going to more aggressively delete code from oslo-incubator and therefore I expect that in addition there will have to be a wider set of projects to address this change.18:03
amrithhttp://markmail.org/message/3w7mcgfdwc4cll7f18:03
amrithRobert's comment re: nova led to some email exchange where we (Robert and I) concluded that we have no dependency on lockutils so I'll update the bp to reflect that.18:03
amrithI have submitted some sample code in the form of four reviews.18:03
amrithhttps://review.openstack.org/#/c/129292/18:03
amrithhttps://review.openstack.org/#/c/129294/18:03
amrithhttps://review.openstack.org/#/c/129295/18:03
amrithhttps://review.openstack.org/#/c/129378/18:03
amrithOne thing to keep in mind with these merges is that it is ok to change code in trove/openstack/common as far as it relates to repointing libraries to a different location for a graduated oslo library.18:03
amrithA similar thing can be seen in this review:18:04
*** dkehn has joined #openstack-trove18:04
amrithhttps://review.openstack.org/#/c/128644/18:04
amrithI've got most of the changes ready to go, right now I'm trying to figure out why rdjenkins is failing for three of them; the tests work for me locally, not sure why they're failing on gate. Hypothesis is that the guest image doesn't have the dependencies installed.18:04
amrith<end paste bomb>18:04
*** tomblank has quit IRC18:04
amrithso, I've pushed a bunch of sample code modeled on other projects18:04
amrithI'm just wrapping up i18n18:04
amrithgettextutils18:04
amrithfeedback on the bp18:04
amrithis primarily the purpose of this meeting18:04
georgelorcho/18:04
amrithbut I felt that feedback on the bp may also depend on some idea of what the implementation looks like18:05
amrithhence the sample code18:05
amrithnow I'll shut up and listen18:05
*** todd_dsm has joined #openstack-trove18:05
SlickNikAwesome, thanks for getting started on this Amrith.18:06
*** tomblank has joined #openstack-trove18:06
*** Barker has joined #openstack-trove18:06
SlickNikWe've got quite a bit of code that is being retired from oslo-incubator in favor of graduating oslo projects.18:06
SlickNikSo it will be good to move to them.18:06
*** jmontemayor has quit IRC18:06
denis_makogoncorrect, but we can't get rid of wsgi stuff in terms of this BP18:07
*** jmontemayor has joined #openstack-trove18:07
SlickNikSo one question I have is whether these are BPs and bugs.18:08
SlickNikor*18:08
peterstaco/18:08
grapexo/18:08
amrithSlickNik, happy to go either way.18:08
amrithI'd started with a BP for o-i -> o.c18:09
amrithso I stuck with that18:09
amrithother projects opted for bugs18:09
SlickNikProbably best to figure out how the other projects are handling them, and stay consistent.18:09
amrithIf you follow the thread: http://markmail.org/message/3w7mcgfdwc4cll7f18:10
amrithmany projects have replied with a bug number and some patch sets18:10
amrithI was going to reply with the patch sets once the BP's are discussed here18:10
*** thedodd has joined #openstack-trove18:10
amrithone thing that I will point out18:10
amrithis that we (trove) appear to have a lot more of this18:11
amriththan other projects18:11
amrithso, one way or another, we're going to have a lot more files changing18:11
*** sgotliv_ has joined #openstack-trove18:11
amrithnot a lot of lines of change for the ones I'm dealing with18:11
amrithbut I know that sgotliv is working on rpc and that's a beast18:11
amriththere are a couple of specific things I'd like people to consider18:12
amrith1. note that in this process, we're going to be updating files in trove/openstack/common18:12
amriththings that were in o-i.18:12
amriththis is to reflect the new dependencies.18:12
SlickNikYup, so I think it's probably best to split up the work into bugs (one per each module).18:12
amrith2. i've opted to do this as multiple chkins, one per graduated module18:12
amrithand that way reviewers get to see commonality18:13
amrithI've also made them a long dependency trail18:13
amrith3. finally, I'm having rdjenkins failures18:13
SlickNikThat way it's fairly easy to split up the work into different patchsets (from independent contributors) if need be.18:13
amrithand I'm thinking this is because of the guest agent18:13
amrithnot having the required dependencies.18:13
*** sgotliv has quit IRC18:13
amrithSlickNik, re: multiple contributors; for the work in this BP, I've got most of the changes submitted at this point18:14
amriththere are two more18:14
amriththat I'm still tweaking a bit18:14
amrithbut I'll have those up in the next couple of days.18:14
amriththen I get to wait for oslo.concurrency to graduate18:14
amrithand then I can push that change set up as well.18:14
amrithI'd appreciate thoughts on the BP, and especially concurrency on #1 above, and any thoughts on #3 above.18:15
amriths/concurrency/agreement/18:15
SlickNik3 is likely the missing dependencies. So we'll have to work on getting them in the cached image.18:16
amrithok. just to be clear, int-tests pass locally.18:16
SlickNikamrith: Yes, but we need to make sure the gate isn't borked.18:17
amrithyup18:17
amrithalso, the chain of chkins makes it easy to review18:18
SlickNikSo for #1 - if I read correctly, we will move from spec to bug?18:18
amrithbut for commit, do you want me to squash?18:18
amrith#1: yes, I will enter a bug18:18
amrithand sorry18:18
amrithyou entered a bug18:18
amrithhttps://bugs.launchpad.net/trove/+bug/138078918:18
amrithI've been using that in commits18:18
amrithas partial-bug: #138078918:18
amrithI'll stick with that.18:18
amrith#1: ok with that bug?18:20
SlickNikSure thing. If any of them (like RPC) is a beast that warrants a separate bug, feel free to open a new one for it.18:20
SlickNikBut I'm okay with using a common bug to tackle a bunch of the smaller dependencies.18:20
amrithRPC (I believe) is already being tracked by bp rpc-versioning18:21
amrithsee https://review.openstack.org/#/c/9448418:21
amrithhttps://blueprints.launchpad.net/trove/+spec/rpc-versioning18:21
SlickNikyes, I guess that's slightly different because moving to oslo.messaging also gives us RPC versioning.18:22
SlickNikwhereas for these other ones, we're just going for equivalence18:22
amrithok, with https://bugs.launchpad.net/trove/+bug/1380789 we have equivalence18:23
amrithWe can punt this bp if that is preferable18:23
SlickNikOkay sounds good.18:24
SlickNikWill work offline with you on #3.18:24
amrithundersood.18:25
amrithI think I have what I need to move forward. If people want to start reviewing the code, a list of change sets is available in the etherpad https://etherpad.openstack.org/p/trove-kilo-do-not-use-obsolete-oslo-modules18:25
SlickNikSounds good — thanks amrith!18:25
amriththanks.18:25
SlickNik#topic Cassandra clustering18:26
denis_makogonmine18:26
*** thedodd has quit IRC18:27
denis_makogon#link https://review.openstack.org/#/c/122736/5/specs/kilo/cassandra-cluster.rst,cm18:27
vgnbkrIt seems that there are 2 components here: changes to APIs, and a Cassandra implementation of those APIs.  I would like to see separate blueprints for API changes so that they can be discussed with thought to how they affect all datastores.18:28
vgnbkr(I mean that genearally, not just in this specific instance)18:29
denis_makogonvgnbkr, those API already exists18:29
amcrni wouldn't mind if they stayed the same blueprint, but having a dedicated section that succinctly explains the api changes would be helpful.18:29
SlickNikAre there really changes to the API that are needed for this?18:29
amcrnex: isn't returning shard_Id18:29
denis_makogonSlickNik, no18:30
amcrnwell, not "changes", but differences i suppose is a better term18:30
SlickNikOr is this an oversight with the BP, and the BP needs to be updated?18:30
*** jmontemayor has quit IRC18:30
vgnbkrThere is at small section at the end about making changes to APIs.  I thought API changes deserved more than a footnote.18:30
SlickNikvgnbkr / amcrn: I haven't gotten there yet. +1 about calling those out upfront, since we need to make sure the API changes are backwards compatible.18:32
denis_makogonthose API changes are specific to C*, there's enough info about why those changes were made18:32
denis_makogonSlickNik, API changes - is nothing else than another View, but specific for C*, same as amcrn did for Mongo18:33
denis_makogonnothing else18:33
amcrnas far as i can tell, the only noteable differences from the mongo spec are: not returning shard_id, using add_node vs. add_shard, returning all node's ip's vs. just query routers as seen in mongodb's case, naming of the instances as "-node-" vs. what's seen in mongodb.18:33
denis_makogonamcrn, correct18:33
dougshelley66wouldn't it make sense for that detail to be in the spec?18:33
amcrn^^18:34
denis_makogondougshelley66, what kind of details ?18:34
dougshelley66denis_makogon, the details that you indicated about + what amcrn just stated18:34
dougshelley66s/about/above18:34
SlickNikdenis_makogon: Details calling out what the _exact_ API differences are.18:34
amcrnvs. having folks extrapolate them.18:35
amcrnoverall looks pretty good18:35
denis_makogonSlickNik, we don't have generic API for clustering, we datastore-specific API18:35
denis_makogonand don't see a value-add on comparing C* clustering API changes against Mongo clustering API18:36
amcrndenis_makogon: the only point here is that it takes too long to answer the question "what changes in behavior will i see as a user compared to the mongodb clustering implementation?". all of the information is in there, it just has to be teased out.18:36
SlickNikdenis_makogon: We spent a lot of time making sure that the API we're using for clustering is similar for across datastores.18:37
amrithone question which I have about this spec is this. In contrast to mysql and mongodb where we have end-users who can validate the implementation, do we have confidence in this implementation/design path? I'd hate to see us go down this path to find that the cassandra community prefers or recommends some other approach.18:37
SlickNikSo I'm not sure I understand your comment about "datastore-specific" API18:37
denis_makogonSlickNik, current clustering framework allows to build datastore-specific response views, that's all18:38
amriththe design choices we made around replication and clustering were intended to provide a common framework that would work for multiple datastores. So I'm a little concerned that we may be diverging from the framework we started with on MongoDB and if we are, I'd like to make sure we are all in agreement that the divergence is worthwhile, the best way to do things etc.,18:39
denis_makogonamrith, i have lots of DBA of C* in production, they know best practices on how to do C* clustering, also there's enough docs with HOWTO (taking into account major DataStax docs)18:40
denis_makogonthere's only one way to do clustering for C*18:40
SlickNikdenis_makogon: Awesome, can we add one of these DBAs as a co-owner in the docs, and get him to the BP meeting to talk about the design tradeoffs?18:40
denis_makogonSlickNik, not quite sure, but will ask18:41
SlickNikClustering is a critical area for Trove, so want to make sure we're doing the right thing here.18:41
denis_makogonthere's no way to make it worth than it is =)18:41
*** AJaeger_ has left #openstack-trove18:42
amrithalso in mongo and mysql we had people who used it in production and had experience operating in production. I'd like to make sure that this implementation will meet a real need completely.18:42
denis_makogonplease leave your comments/concerns on spec review, i will take them into account18:42
vgnbkrMy reading about C* suggested there are different implementation choices, though there are some best practices.18:42
denis_makogonvgnbkr, let's talk about it offline18:43
SlickNikdenis_makogon: I'm also a bit concerned that you're the only primary for both https://review.openstack.org/#/c/122736/ and https://review.openstack.org/#/c/122767/. What's the priority here? Both of these seem like they would take a team of folks some time to implement, so what are the time-frames?18:43
vgnbkrFor example, I don't know that I agree that topology strategy, replication factor etc won't need to be configurable by the user.  Your spec suggests it would be pre-configured by the provider, unless I missed something.18:44
denis_makogonSlickNik, C* clustering K-2, Oracle K-318:44
denis_makogonvgnbkr, replication factor is controlled by keyspace owner, topology strategy by a deployer18:45
denis_makogonvgnbkr, in general, correct, it's up to provider18:45
vgnbkrIf you could clarify in your spec how those would be configured that would be a great help.18:46
denis_makogonvgnbkr, ok, could you please leave a comment on the review18:47
vgnbkrSure.18:47
denis_makogonvgnbkr, thanks18:47
denis_makogonany other thoughts/suggestions?18:48
amrithseveral but I'll put them in review comments.18:49
amriththey were smaller18:49
amrithand not worth taking everyones time ehre.18:49
amriths/ehre/here/18:49
denis_makogonamrith, thanks, a lot =)18:49
SlickNikOkay, sounds good.18:50
denis_makogonone more thing18:50
denis_makogonit's about all spec reviews18:50
denis_makogonIt's pretty hard to gather feedbacks during meeting. Why can't we do spec reviewing offline and once it's ready ask PTL and core to take a look at it and say if it's ready to go public18:51
SlickNikdenis_makogon: That's the point of doing the spec reviews as .rst files. When they're ready to go public, they will be +2 and approved.18:52
SlickNikgo public = ready to be implemented.18:52
denis_makogonthen we need to spend _lot_ more time doing reviews18:53
denis_makogonnot just 1 hour per 2-3 weeks18:53
denis_makogonthat's pretty much all that i wanted to say18:53
denis_makogonthanks folks for feedback18:53
SlickNikdenis_makogon: I'm sure that all of us spend a _LOT_ more time than that doing reviews.18:54
vgnbkrI understand what Denis is saying, though I still think the meeting can be useful.  Maybe we need to say the specs must be up by Friday morning so that people have a chance to review before the meeting, then we can concentrate on high level discussion rather than implementation details?18:55
denis_makogonlet's officially finish the meeting and then we can discuss it18:56
SlickNikvgnbkr: The way I see it, it's for getting face-face feedback and for resolving contentious issues.18:57
SlickNik#endmeeting18:57
openstackMeeting ended Mon Oct 20 18:57:09 2014 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:57
*** rushiagr_away is now known as rushiagr18:57
openstackMinutes:        http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-10-20-18.00.html18:57
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-10-20-18.00.txt18:57
*** saurabhs1 has joined #openstack-trove18:57
openstackLog:            http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-10-20-18.00.log.html18:57
vgnbkrSlickNik: Exactly.18:57
*** saurabhs has quit IRC18:59
openstackgerritRiddhi Shah proposed a change to openstack/trove: Add visibility filter to datastore versions  https://review.openstack.org/11019719:01
openstackgerritAndreas Jaeger proposed a change to openstack/trove: Update some log calls for translation and lazyness  https://review.openstack.org/11940219:03
*** vkmc has quit IRC19:10
*** jmontemayor has joined #openstack-trove19:10
*** rushiagr is now known as rushiagr_away19:13
*** X019 has quit IRC19:30
*** anna_ has joined #openstack-trove19:30
*** radez_g0n3 is now known as radez19:30
*** vkmc has joined #openstack-trove19:36
*** georgelorch has quit IRC19:36
*** miqui has joined #openstack-trove19:38
*** georgelorch has joined #openstack-trove19:39
exploreshaifalihello I am trying to install trove, while running *./redstack  kick-start mysql* getting "invalid argument, not mounted error"19:45
exploreshaifalihere is full output http://pastebin.com/MmH74m2M19:45
denis_makogonexploreshaifali, i'd try to guess, how big your VM ?19:47
denis_makogonexploreshaifali, i mean in GBs ?19:47
exploreshaifalidenis_makogon, around 4 GB19:47
denis_makogonexploreshaifali, is it RAM or disk space?19:47
denis_makogonexploreshaifali, you need from 10Gb up to 20Gb of disk space for devstack installation and image building19:48
exploreshaifaliiccha1, ^19:48
denis_makogonexploreshaifali, it may work for you19:49
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - gettextutils (now oslo.i18n)  https://review.openstack.org/12971419:50
exploreshaifalidenis_makogon, Thanks, here is output that shows disk space infohttp://pastebin.com/2kzFg8nS19:52
denis_makogonexploreshaifali, yeah it looks like you need bigger one =)19:53
exploreshaifalidenis_makogon, why I already have 40G disk space19:54
exploreshaifalidenis_makogon, the error is not talking about space probably, as it says "'There is a duplicated hook in your elements: dpkg/pre-install.d/00-disable-apt-recommends'19:59
exploreshaifaliThere is a duplicated hook in your elements: dpkg/pre-install.d/00-disable-apt-recommends"19:59
openstackgerritEdmond Kotowski proposed a change to openstack/trove: Configuration group checking 0 validation rules  https://review.openstack.org/12736020:04
*** harlowja is now known as harlowja_away20:10
*** IanGovett has quit IRC20:12
denis_makogonexploreshaifali, try to reclone your environment20:16
denis_makogonexploreshaifali, iirc, you need to set environment variable "export RECLONE=yes; ./redstack install; ./redstack kick-start mysql"20:17
denis_makogonexploreshaifali, wait =)20:18
denis_makogonexploreshaifali, we have a bug in trove-integration =))20:18
denis_makogonSlickNik, amrith yt?20:19
*** jmontemayor has quit IRC20:19
exploreshaifalidenis_makogon, okay I should retry to install it again :)20:21
exploreshaifalidenis_makogon, Thanks!!20:21
denis_makogonexploreshaifali, you might wait20:21
denis_makogonexploreshaifali, we need to get appropriate fix to trove-integration20:21
SlickNikdenis_makogon: here, what's up?20:21
denis_makogonSlickNik, we have broken gate =(20:22
denis_makogonSlickNik, due to recent merge https://github.com/openstack/diskimage-builder/commit/2ea4f6ec8a5d4ed651275db35779d929a1d04b9020:22
denis_makogonSlickNik, exploreshaifali faced with it see logs http://pastebin.com/2kzFg8nS20:22
denis_makogonSlickNik, DIB dpkg elements also has 00-disable-apt-recommends20:23
denis_makogonSlickNik, https://github.com/openstack/trove-integration/blob/master/scripts/files/elements/ubuntu-guest/pre-install.d/00-disable-apt-recommends20:23
exploreshaifalidenis_makogon, I am using remote server to install and try trove20:24
denis_makogonexploreshaifali, you can't build image yet20:24
denis_makogonexploreshaifali, because you would get the same error20:24
exploreshaifaliimage of what?20:25
denis_makogonexploreshaifali, image for Trove20:25
*** georgelorch has quit IRC20:25
SlickNikdenis_makogon: Where's the duplicate element?20:25
SlickNikduplicate hook*20:25
denis_makogonSlickNik, one it DIB/dpkg and one in ubuntu-guest20:25
*** georgelorch has joined #openstack-trove20:26
exploreshaifalidenis_makogon, redstack install worked, but kick-start mysql not working20:26
denis_makogonexploreshaifali, correct20:26
denis_makogonSlickNik, https://github.com/openstack/diskimage-builder/blob/master/elements/dpkg/pre-install.d/00-disable-apt-recommends and https://github.com/openstack/trove-integration/blob/master/scripts/files/elements/ubuntu-guest/pre-install.d/00-disable-apt-recommends20:26
abramleydenis_makogon - yes the elements are identical and I think we can remove the one out of ubuntu-guest20:26
denis_makogonabramley, i'm working on it right now20:26
SlickNikdenis_makogon: The gate uses a cached image — so I don't think it's broken. But it does look like it'd be an issue trying to rebuild the image.20:27
abramleyThe change to DIB that broke it only went in today so it hasn't been broken long20:28
*** grapex has quit IRC20:30
denis_makogonabramley, yes, as i said, recent commit =)20:31
denis_makogonlooks like 00-disable-apt-recommended is the only one duplicated hook, so submitting fix for it20:33
denis_makogonSlickNik, could you please review bug-report https://bugs.launchpad.net/trove-integration/+bug/1383467 ?20:34
openstackgerritDenis M. proposed a change to openstack/trove-integration: Remove duplicated hook 00-disable-apt-recommended  https://review.openstack.org/12972320:34
denis_makogonabramley, please review this one https://review.openstack.org/129723 ^^20:35
*** dkehnm has quit IRC20:36
*** kevinconway has joined #openstack-trove20:41
exploreshaifalidenis_makogon, so I should wait for how much time?20:42
exploreshaifalior I can clone any other branch to get trove?20:42
openstackgerritBob Thyne proposed a change to openstack/python-troveclient: Adds support for Keystone v3 API  https://review.openstack.org/10231520:45
*** dkehnm has joined #openstack-trove20:45
denis_makogonexploreshaifali, try this one "git pull ssh://dmakogon@review.openstack.org:29418/openstack/trove-integration refs/changes/23/129723/1"20:45
denis_makogonexploreshaifali, sorry, this one "git pull https://dmakogon@review.openstack.org/openstack/trove-integration refs/changes/23/129723/1"20:46
exploreshaifalidenis_makogon, okay :)20:48
exploreshaifaliThanks! :)20:48
exploreshaifalidenis_makogon, and now after pulling it I need to run kick-start20:49
*** georgelorch has quit IRC20:49
*** georgelorch has joined #openstack-trove20:50
denis_makogonexploreshaifali, yes20:50
exploreshaifalicool :)20:51
*** tomblank has quit IRC20:51
*** radez is now known as radez_g0n320:52
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929420:54
*** radez_g0n3 is now known as radez20:57
*** jcru has quit IRC20:59
*** edmondk has quit IRC20:59
*** mattgriffin has quit IRC21:01
*** sriram_ has joined #openstack-trove21:04
*** sriram_tesora has quit IRC21:05
openstackgerritAuston McReynolds proposed a change to openstack/trove-specs: Quota For RAM, Cores, & Volume Count Spec  https://review.openstack.org/12973421:17
*** pboros has quit IRC21:17
*** flaper87 is now known as flaper87|afk21:20
-openstackstatus- NOTICE: Zuul erroneously marked some changes as having merge conflicts. Those changes have been added to the check queue to be rechecked and will be automatically updated when complete.21:22
*** robertmyers has quit IRC21:23
*** grapex has joined #openstack-trove21:24
exploreshaifalidenis_makogon, after running kick-start mysql getting this output http://pastebin.com/50N5pfqg21:25
*** grapex_ has joined #openstack-trove21:26
exploreshaifalidenis_makogon, see at line 148, getting invalid arguments21:27
*** grapex has quit IRC21:29
openstackgerritAuston McReynolds proposed a change to openstack/trove-specs: Quota For RAM, Cores, & Volume Count Spec  https://review.openstack.org/12973421:31
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - unused modules  https://review.openstack.org/12929221:34
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - timeutils  https://review.openstack.org/12929421:34
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - lockutils  https://review.openstack.org/12929521:34
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - importutils  https://review.openstack.org/12937821:34
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - excutils  https://review.openstack.org/12965421:34
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - strutils  https://review.openstack.org/12966321:35
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - network_utils (now netutils)  https://review.openstack.org/12966421:35
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - jsonutils (now oslo.serialization)  https://review.openstack.org/12966821:35
openstackgerritamrith proposed a change to openstack/trove: Obsolete oslo-incubator modules - gettextutils (now oslo.i18n)  https://review.openstack.org/12971421:35
*** vkmc has quit IRC21:35
*** openstackgerrit has quit IRC21:40
*** Barker has quit IRC21:42
*** todd_dsm has quit IRC21:46
*** amrith is now known as _amrith_21:47
*** radez is now known as radez_g0n321:48
*** tomblank has joined #openstack-trove21:52
*** todd_dsm has joined #openstack-trove21:52
*** annashen has quit IRC21:54
*** edmondk has joined #openstack-trove21:56
*** jasonb365 has quit IRC22:03
*** exploreshaifali has quit IRC22:09
*** exploreshaifali has joined #openstack-trove22:16
*** grapex_ has quit IRC22:16
*** denis_makogon has quit IRC22:20
*** eghobo has joined #openstack-trove22:22
*** eghobo has quit IRC22:29
*** eghobo has joined #openstack-trove22:32
*** anna_ has quit IRC22:35
*** Barker has joined #openstack-trove22:35
*** kevinconway has quit IRC22:36
*** Barker has quit IRC22:36
*** openstackgerrit has joined #openstack-trove22:42
*** eghobo has quit IRC22:44
*** harlowja_away is now known as harlowja22:48
*** anna_ has joined #openstack-trove22:48
*** eghobo has joined #openstack-trove22:48
*** eghobo has quit IRC22:49
*** eghobo has joined #openstack-trove22:53
*** vkmc has joined #openstack-trove22:59
openstackgerritAuston McReynolds proposed a change to openstack/trove-specs: Enhance Mgmt-Show To Support Deleted Spec  https://review.openstack.org/12975223:03
*** anna_ has quit IRC23:11
*** saurabhs1 has left #openstack-trove23:14
*** exploreshaifali has quit IRC23:14
*** X019 has joined #openstack-trove23:14
*** todd_dsm has quit IRC23:24
*** Riddhi has quit IRC23:32
*** sgotliv_ has quit IRC23:43
*** sriram_ has quit IRC23:44
*** Barker has joined #openstack-trove23:48
*** todd_dsm has joined #openstack-trove23:48
openstackgerritAuston McReynolds proposed a change to openstack/trove: Instance-Delete Should Stop the Database First  https://review.openstack.org/12910423:50
openstackgerritAuston McReynolds proposed a change to openstack/trove: Config Group Load Fails If DS Version Inactive  https://review.openstack.org/12914523:51
openstackgerritAuston McReynolds proposed a change to openstack/trove: Couchbase Root Password Can Go Out Of Sync  https://review.openstack.org/12909523:51
openstackgerritAuston McReynolds proposed a change to openstack/trove-integration: Make Rsync For Guest Optional  https://review.openstack.org/11948823:52

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