Thursday, 2014-08-28

*** tkatarki has joined #openstack-trove00:11
*** topshare has joined #openstack-trove00:17
*** topshare has quit IRC00:18
*** amcrn has quit IRC00:19
*** tkatarki has quit IRC00:24
*** ViswaV has joined #openstack-trove00:27
*** ViswaV has quit IRC00:52
openstackgerritamrith proposed a change to openstack/trove: make backup_incremental_strategy a datastore specific option  https://review.openstack.org/11315500:59
*** grapex has quit IRC01:05
*** tkatarki has joined #openstack-trove01:05
openstackgerritamrith proposed a change to openstack/trove: Partially address concerns in Couchbase restore strategy  https://review.openstack.org/11717401:05
*** Barker has joined #openstack-trove01:09
*** eghobo has quit IRC01:10
*** eghobo has joined #openstack-trove01:11
*** Barker has quit IRC01:14
*** Barker has joined #openstack-trove01:16
*** Riddhi has quit IRC01:18
*** eguz has joined #openstack-trove01:21
*** Barker has quit IRC01:21
*** eguz has quit IRC01:21
*** eghobo has quit IRC01:25
amrithdenis_makogon ... you there? dmakogon_ ...01:26
amrithI guess not01:26
*** achampion has joined #openstack-trove01:33
*** _1_vigneshvar_ has joined #openstack-trove01:37
*** _1_vigneshvar_ has left #openstack-trove01:40
*** tkatarki has quit IRC01:49
*** nosnos has joined #openstack-trove01:49
*** tkatarki has joined #openstack-trove01:52
*** tkatarki has quit IRC01:54
*** tkatarki has joined #openstack-trove01:55
*** openstackgerrit has quit IRC02:02
*** openstackgerrit has joined #openstack-trove02:02
*** saurabhs has left #openstack-trove02:10
*** tkatarki has quit IRC02:10
*** tkatarki has joined #openstack-trove02:11
*** johnma has quit IRC02:13
*** vigneshvar has joined #openstack-trove02:16
*** tkatarki has quit IRC02:21
*** haomaiwa_ has quit IRC02:25
*** topshare has joined #openstack-trove02:25
*** 7YUAAI27X has joined #openstack-trove02:25
*** ramishra has joined #openstack-trove02:26
*** eghobo has joined #openstack-trove02:32
*** jasonb365 has joined #openstack-trove02:34
*** topshare has quit IRC02:36
*** eghobo has quit IRC02:37
*** vigneshvar has quit IRC02:38
*** eghobo has joined #openstack-trove02:38
*** dkehn has joined #openstack-trove02:39
*** Riddhi has joined #openstack-trove02:40
*** dkehn_ has quit IRC02:40
*** dkehnx has quit IRC02:40
*** haomaiwang has joined #openstack-trove02:41
*** Riddhi has quit IRC02:41
*** Riddhi has joined #openstack-trove02:42
*** dkehn_ has joined #openstack-trove02:42
*** ramishra has quit IRC02:43
*** 7YUAAI27X has quit IRC02:43
*** ramishra has joined #openstack-trove02:44
*** vigneshvar has joined #openstack-trove02:49
*** topshare has joined #openstack-trove02:54
*** eguz has joined #openstack-trove02:55
*** eghobo has quit IRC02:58
*** eguz has quit IRC03:01
vigneshvaramrith: hi03:03
*** mattgriffin has joined #openstack-trove03:05
*** juantwo has quit IRC03:16
*** juantwo has joined #openstack-trove03:18
*** topshare has quit IRC03:19
*** topshare has joined #openstack-trove03:22
*** nosnos has quit IRC03:26
*** nosnos has joined #openstack-trove03:27
*** Longgeek has joined #openstack-trove03:31
*** nosnos has quit IRC03:32
*** topshare has quit IRC03:37
*** Riddhi has quit IRC03:54
*** juantwo_ has joined #openstack-trove04:04
*** eghobo has joined #openstack-trove04:07
*** jasonb365 has quit IRC04:08
*** juantwo has quit IRC04:08
*** topshare has joined #openstack-trove04:08
*** juantwo_ has quit IRC04:11
*** eghobo has quit IRC04:12
*** ramishra has quit IRC04:19
*** nosnos has joined #openstack-trove04:27
*** ramishra has joined #openstack-trove04:34
*** eghobo has joined #openstack-trove04:34
*** ramishra has quit IRC04:42
*** ramishra has joined #openstack-trove04:45
*** Longgeek_ has joined #openstack-trove04:52
*** ramishra has quit IRC04:55
*** Longgeek has quit IRC04:56
*** ViswaV has joined #openstack-trove04:57
*** ViswaV_ has joined #openstack-trove04:58
*** ViswaV has quit IRC05:01
*** ramishra has joined #openstack-trove05:06
*** vigneshvar_ has joined #openstack-trove05:11
*** topshare has quit IRC05:22
*** topshare has joined #openstack-trove05:27
*** mattgriffin has quit IRC05:27
*** mikehn_ has quit IRC05:32
*** ViswaV_ has quit IRC05:37
*** mikehn has joined #openstack-trove05:39
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements  https://review.openstack.org/11677005:42
openstackgerritOpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements  https://review.openstack.org/11152905:43
*** k4n0 has joined #openstack-trove06:03
openstackgerritOpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex  https://review.openstack.org/11444206:07
*** ramishra has quit IRC06:17
*** topshare has quit IRC06:17
*** flaper87|afk is now known as flaper8706:27
*** sgotliv has joined #openstack-trove06:48
*** vkmc has quit IRC06:52
*** denis_makogon has joined #openstack-trove06:54
*** sgotliv has quit IRC06:57
*** sgotliv has joined #openstack-trove06:57
*** eghobo has quit IRC07:08
*** harlowja_ is now known as harlowja_away07:14
*** ramishra has joined #openstack-trove07:18
*** ramishra has quit IRC07:22
*** ramishra has joined #openstack-trove07:22
*** topshare has joined #openstack-trove07:23
*** ramishra has quit IRC07:24
*** todd_dsm has quit IRC07:24
*** ramishra has joined #openstack-trove07:24
*** topshare has quit IRC07:28
*** ramishra has quit IRC07:34
*** ramishra has joined #openstack-trove07:35
*** topshare has joined #openstack-trove07:35
*** topshare has quit IRC07:38
sgotlivSlickNik, good morning07:41
*** boblebauce has joined #openstack-trove07:53
*** topshare has joined #openstack-trove07:56
*** julienvey has joined #openstack-trove08:07
*** rushiagr_away is now known as rushiagr08:08
*** ajayaa has joined #openstack-trove08:09
*** vigneshvar has quit IRC08:11
*** rushiagr is now known as rushiagr_away08:12
*** AndChat|329121 has joined #openstack-trove08:14
*** vigneshvar_ has quit IRC08:15
*** flaper87 is now known as flaper87|afk08:21
*** boris-42 has quit IRC08:21
*** AndChat|329121 has quit IRC08:26
*** vigneshvar_ has joined #openstack-trove08:26
*** flaper87|afk is now known as flaper8708:30
*** boris-42 has joined #openstack-trove08:32
*** ramishra has quit IRC08:42
*** nosnos has quit IRC08:47
*** nosnos has joined #openstack-trove08:47
*** nosnos has quit IRC08:52
*** ramishra has joined #openstack-trove08:54
*** vigneshvar_ has quit IRC09:03
*** vigneshvar_ has joined #openstack-trove09:06
*** ramishra has quit IRC09:08
*** sgotliv has quit IRC09:20
*** sgotliv has joined #openstack-trove09:20
*** nosnos has joined #openstack-trove09:24
openstackgerritKevin Conway proposed a change to openstack/trove: Add PostgreSQL support  https://review.openstack.org/5760909:41
*** vigneshvar_ has quit IRC09:42
*** kevinconway has joined #openstack-trove10:00
*** haomaiwang has quit IRC10:00
*** isviridov is now known as isviridov_awa10:00
*** isviridov_awa is now known as isviridov_awat10:00
*** isviridov_awat is now known as isviridov_away10:00
*** haomaiwang has joined #openstack-trove10:01
*** kevinconway has quit IRC10:01
*** haomaiw__ has joined #openstack-trove10:07
*** haomaiwang has quit IRC10:10
*** denis_makogon has quit IRC10:13
*** tkatarki has joined #openstack-trove10:15
*** topshare has quit IRC10:18
*** nosnos has quit IRC10:20
*** nosnos has joined #openstack-trove10:21
*** nosnos has quit IRC10:25
*** tkatarki has quit IRC10:26
*** tkatarki has joined #openstack-trove10:28
*** georgelorch has quit IRC10:30
*** nosnos has joined #openstack-trove10:34
*** isviridov_away is now known as isviridov10:37
*** sgotliv has quit IRC10:42
*** IanGovett has joined #openstack-trove10:52
openstackgerritamrith proposed a change to openstack/trove: make backup_incremental_strategy a datastore specific option  https://review.openstack.org/11315511:00
*** nosnos has quit IRC11:18
*** nosnos has joined #openstack-trove11:20
*** tomblank has quit IRC11:26
*** ranjitha has joined #openstack-trove11:33
*** ranjitha has quit IRC11:37
*** achampion has quit IRC11:46
*** vkmc has joined #openstack-trove11:46
*** vkmc has joined #openstack-trove11:46
*** iartarisi has joined #openstack-trove11:47
*** kevinconway has joined #openstack-trove11:56
*** vigneshvar_ has joined #openstack-trove11:59
*** topshare has joined #openstack-trove12:04
*** georgelorch has joined #openstack-trove12:14
*** juantwo has joined #openstack-trove12:14
*** kevinconway_ has joined #openstack-trove12:16
*** kevinconway has quit IRC12:17
*** kevinconway_ is now known as kevinconway12:17
*** juantwo has quit IRC12:17
*** juantwo has joined #openstack-trove12:18
*** nosnos has quit IRC12:22
*** nosnos has joined #openstack-trove12:23
*** nosnos has quit IRC12:27
*** topshare has quit IRC12:28
*** topshare has joined #openstack-trove12:30
*** AndChat|329121 has joined #openstack-trove12:35
*** vigneshvar_ has quit IRC12:38
*** achampion has joined #openstack-trove12:41
*** georgelorch has quit IRC12:51
*** georgelorch has joined #openstack-trove12:51
*** pdmars has joined #openstack-trove12:53
*** pdmars has quit IRC12:54
*** pdmars has joined #openstack-trove12:54
*** sriram_tesora has quit IRC13:00
*** dmakogon_ is now known as denis_makogon13:01
*** topshare has quit IRC13:01
denis_makogonamrith, https://review.openstack.org/113155 could you please address the last one suggestion13:01
*** vigneshvar_ has joined #openstack-trove13:03
*** AndChat|329121 has quit IRC13:05
*** k4n0 has quit IRC13:10
*** ajayaa has quit IRC13:11
*** jasonb365 has joined #openstack-trove13:11
*** Barker has joined #openstack-trove13:17
*** ramishra has joined #openstack-trove13:22
*** radez_g0` is now known as radez13:24
kevinconwaydenis_makogon: when is the last time you used redstack?13:33
kevinconwayi had to recreate my dev env recently and the nova servers don't go active13:33
kevinconwayand i can't ssh into them13:33
*** ramishra has quit IRC13:40
*** Barker has quit IRC13:45
*** Barker has joined #openstack-trove13:46
denis_makogonkevinconway, to be honest, few days ago13:47
denis_makogonkevinconway, i'll try it out soon13:47
*** robertmyers has joined #openstack-trove13:48
*** sriram_tesora has joined #openstack-trove13:49
*** mattgriffin has joined #openstack-trove13:56
*** edmund2 has joined #openstack-trove14:00
*** vigneshvar_ has quit IRC14:00
*** jcru has joined #openstack-trove14:01
*** tomblank has joined #openstack-trove14:05
*** ramishra has joined #openstack-trove14:11
*** IanGovett1 has joined #openstack-trove14:13
*** PeterSchwaller_ has joined #openstack-trove14:15
*** ramishra has quit IRC14:16
*** IanGovett has quit IRC14:16
*** PeterSchwaller has quit IRC14:18
*** Longgeek_ has quit IRC14:24
*** Riddhi has joined #openstack-trove14:29
icchaglucas: why is there a -2 on the detach patch14:43
*** dkehn__ has joined #openstack-trove14:43
icchai mean workflow -114:45
*** dkehn_ has quit IRC14:45
*** dkehn has quit IRC14:46
openstackgerritDenis M. proposed a change to openstack/trove: Fix fake int-tests hosts tests assertion  https://review.openstack.org/11752414:46
denis_makogoniccha, nice to see you here, i want to talk about forbidding a deletion of master site14:47
*** dkehn has joined #openstack-trove14:47
denis_makogoniccha, do you have like 5 mins?14:47
icchahey yes denis_makogon14:49
denis_makogoniccha, i was thinking about how should we act with slaves when trying to delete them14:49
icchadelete slave or master?14:49
denis_makogoniccha, slaves14:49
icchaok go ahead denis_makogon14:50
denis_makogoniccha, maybe we need to forbid delete call for slaves too, since slave can be safely delete only when it gets detached, right?14:50
denis_makogoniccha, so, my proposal is to act with slaves in the same manner as with masters14:51
amrithdenis_makogon, re: https://review.openstack.org/113155. good suggestion. will do.14:51
denis_makogonamrith, thanks14:51
icchanothing depends on the slave. but i can see where you are coming from denis_makogon . i am open to either way. would like to know what other stake holders think glucas SlickNik14:52
denis_makogoniccha, sure, i asked you first since you've assigned to "delete master" bug14:53
icchathanks for bringing it up denis_makogon  :)14:53
denis_makogoniccha, np14:53
*** PeterSchwaller has joined #openstack-trove14:55
denis_makogonglucas, iccha, dougshelley66, amrith, SlickNik, guys, there are certain list of the bugs that somehow related to current client create call (in terms of slave creation), like users/schemes, flavors, volume, datastore and its version, so i'd like to proposed add another client call (refers to the same instance create call) that would look like replica-create <master-id> --flavor --volume14:57
*** PeterSchwaller_ has quit IRC14:58
denis_makogonglucas, iccha, dougshelley66, amrith, SlickNik it would give us good benefits, such as: we would not need to add additional logic that validates users/schemes, flavors, datastores, versions14:58
denis_makogonglucas, iccha, dougshelley66, amrith, SlickNik, to be precise, we would close next bugs: https://bugs.launchpad.net/trove/+bug/1357709, https://bugs.launchpad.net/trove/+bug/1357501, https://bugs.launchpad.net/trove/+bug/135941115:01
glucasdenis_makogon: the replica-create would also need to support nics, config groups, right?15:03
denis_makogonglucas, not sure about configuration group, but completely sure about nics15:03
glucasdenis_makogon: changing this may impact horizon create-instance support, if we drop support for slave_of in the normal create call.15:04
denis_makogonglucas, does horizon already supports slave_of ?15:04
glucasit's in review15:05
denis_makogonglucas, then we need to deal with it fast15:05
denis_makogonglucas, we need to make a conclusion today, maximum tomorrow, agreed?15:05
denis_makogonJ3 is too close15:05
vgnbkrThat sort of change should be in a blueprint, and subject to blueprint review, so can't really be even talked about 'til Monday.15:06
glucasvgnbkr: agreed, it needs wider discussion15:06
*** coolsvap has quit IRC15:07
*** amrith_ has joined #openstack-trove15:07
*** amrith_ is now known as Guest6969815:07
kevinconwayanyone spin up a redstack vm in the recent past?15:07
amrithkevinconway, yes. this morning15:08
kevinconwayi had to destroy my existing one today and the new one's nova servers don't go active15:08
*** cweid_ has joined #openstack-trove15:08
icchai also think that this somehow bleeds into concept of clusters vs replicas in mysql denis_makogon glucas vgnbkr15:08
*** cweid_ has quit IRC15:08
kevinconwayamrith: have you seen any issues with the nova servers never intializing/ not being to ssh into them?15:08
*** cweid has quit IRC15:08
amrithI have not had that recently15:09
amrithI did a while ago15:09
*** cweid has joined #openstack-trove15:09
amrithbut that was because I messed up my authorized_keys file15:09
amrithvery clever thing to do15:09
kevinconwayamrith: what hypervisor do you use for making the dev env? virtualbox?15:09
Guest69698vmware/kvm15:10
*** coolsvap has joined #openstack-trove15:10
Guest69698kevinconway, having nick issues ...15:11
*** ramishra has joined #openstack-trove15:12
vgnbkriccha: good point.  We'll need to be careful of our terminology to avoid confusion.15:12
glucasdenis_makogon: I suspect we want the create instance validation regardless, since both create calls would map to the same route?15:13
denis_makogonglucas, yes, each bug should be covered with specific validation step15:13
glucasbut I could see possibly adding a replicate-create command later as a convenience where more validation could be done on the client side.15:13
denis_makogonglucas, that would be preferable15:14
*** amrith has quit IRC15:15
*** Guest69698 is now known as amrith15:15
*** ramishra has quit IRC15:16
kevinconwayamrith: how do you set up your env? do you use the trove-vagrant-vmware?15:17
amrithkevinconway, no.15:19
amrithI don't use vagrant15:20
amrithI've been launching my setup in one of two ways15:20
amriththe hard way by following the long install process15:20
amriththree vm's15:20
amriththe other is to use redstack15:20
amrithall on one vm15:20
*** tomblank has left #openstack-trove15:21
*** tomblank has joined #openstack-trove15:23
openstackgerritGreg Lucas proposed a change to openstack/trove: Add detach-replica support  https://review.openstack.org/10198115:29
*** sriram_ has joined #openstack-trove15:35
openstackgerritCraig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db  https://review.openstack.org/7985015:38
*** edmund2 has quit IRC15:39
*** edmund3 has joined #openstack-trove15:39
glucasSlickNik or other core folks: can we get bug/1309030 on the j3 list? The related patches went through quite a bit of review feedback back in early june and are ready to go..15:40
glucas^^ the bug was originally triaged for juno-1 but that got lost when it was moved from trove-int to trove15:42
denis_makogonglucas, i'll mark target to j315:43
*** dkehn__ is now known as dkehnx15:48
glucasdenis_makogon: oh, guess I could have just done that. :)15:48
denis_makogonglucas, =)15:49
*** tomblank has quit IRC15:49
*** tomblank has joined #openstack-trove15:53
*** iartarisi has quit IRC15:54
*** eghobo has joined #openstack-trove15:54
*** todd_dsm has joined #openstack-trove15:55
*** Barker has quit IRC15:56
*** Barker has joined #openstack-trove16:03
denis_makogoniccha, are you there ?16:08
icchayes denis_makogon16:09
denis_makogoniccha, https://review.openstack.org/#/c/116369/3/troveclient/v1/instances.py,cm, do yo really think that raise of Exception would be ok from user perspective ?16:09
icchadenis_makogon: yes. maybe we could elaborate in the message and say please detach first and then attempt to attach16:10
denis_makogoniccha, if we're going to raise an exception we need to hide stack traces16:10
denis_makogoniccha, i'd suggest to use something less hardcore, what about use of print() ?16:11
denis_makogoniccha, and do validation before going into instances.py16:12
Riddhidenis_makogon: if the concern is stack traces being shown, with raise exception it does not seem like it shows a stack trace16:13
denis_makogoniccha, we should stop at shell.py once we have remove_configuration and configuration mentioned16:13
icchadenis_makogon: i think the logic for shell should not know anything about instances.16:14
denis_makogoniccha, i'd suggest to move validation logic to shell.py and use print instead of raising an exception16:14
Riddhisomething like here -https://github.com/openstack/python-troveclient/blob/master/troveclient/v1/users.py#L7016:14
Riddhi??16:14
Riddhithis is not being done in shell.py as well16:15
denis_makogonRiddhi, iccha, it should be like this https://github.com/openstack/python-troveclient/blob/master/troveclient/v1/shell.py#L245-L25416:15
*** PierreRambaud has quit IRC16:16
icchadenis_makogon: but that throws excpetion16:17
icchadenis_makogon: so the reason i think its more appropriate in instances.py is because the expetion is specific to how instances behave and that logic should be kept out of shell.py16:17
icchadenis_makogon: i shall pull down code and test if i see a stack trace or not16:18
*** isviridov is now known as isviridov_away16:18
denis_makogoniccha, oj16:18
denis_makogoniccha, ok16:18
denis_makogoniccha, Riddhi, i think we need to have this logic in both places and raise appropriate exception (Exception is too broad, we have more specific one - CommandError)16:20
icchadenis_makogon: if we catch in shell it ll never go to instances.py16:21
denis_makogoniccha, correct, fast-fail is alway good option =)16:22
*** kevinconway has quit IRC16:22
icchahmmm i see what you are saying denis_makogon16:23
denis_makogonSlickNik, i've just looked at https://etherpad.openstack.org/p/trove-juno3-reviews, and didn't find backup/restore for Cassandra, i think we've missed it16:23
*** todd_dsm has quit IRC16:23
*** redthrux has joined #openstack-trove16:23
denis_makogoniccha, glad to hear16:26
icchaglucas: what changed in the latest patchset u pushed up for detach-replica support?16:26
Riddhibut denis,  I also agree with iccha in the logic being resource specific and also, the implementation seems consistent with other resources16:26
*** todd_dsm has joined #openstack-trove16:28
denis_makogonRiddhi, consistent with something that was done long time ago and possible with some inaccuracies, if it work it  doesn't mean that it's correct, so i suggest to do it in another way =)16:29
Riddhihmm, in that case i will not maintain it in both places...ill have it only in shell16:30
*** kevinconway has joined #openstack-trove16:30
denis_makogonRiddhi, it must be in two places, since shell is available only if user uses python-troveclient CLI, but what about those who doesn't ?16:31
Riddhiahh i see that16:32
icchadenis_makogon: so then ctahcing it in instnaces.py catches both for us16:32
*** tomblank has quit IRC16:32
icchano need for duplication16:32
icchaanyways we re catching it before api call16:32
denis_makogoniccha, for shell users we need to raise CommandError instead of too broad exception16:33
*** ramishra has joined #openstack-trove16:33
*** todd_dsm has quit IRC16:33
openstackgerritDenis M. proposed a change to openstack/trove: Apply recursive initial master lookup while replica provisioning  https://review.openstack.org/11729816:34
openstackgerritDenis M. proposed a change to openstack/trove: Fix master status validation while replica provisioning  https://review.openstack.org/11725516:34
*** tomblank has joined #openstack-trove16:40
openstackgerritDenis M. proposed a change to openstack/trove: Fix fake int-tests hosts tests assertion  https://review.openstack.org/11752416:44
*** Barker has quit IRC16:45
openstackgerritDenis M. proposed a change to openstack/trove: Fix hosts fake int-tests assertion  https://review.openstack.org/11752416:47
*** redthrux has left #openstack-trove16:48
*** Barker has joined #openstack-trove16:54
*** tomblank has quit IRC16:58
*** eghobo has quit IRC16:59
*** eghobo has joined #openstack-trove16:59
*** ramishra has quit IRC17:02
*** harlowja_away is now known as harlowja_17:03
*** tomblank has joined #openstack-trove17:03
*** ramishra has joined #openstack-trove17:03
*** pdmars has quit IRC17:04
*** harlowja_ is now known as harlowja_away17:04
SlickNikdenis_makogon: The reason cassandra backup / restore wasn't in j3 is because I was talking with amcrn, and he mentioned that you had talked with the cassandra folks on the IRC channel and they had reservations about the current approach we are taking with this.17:05
*** saurabhs has joined #openstack-trove17:05
denis_makogonSlickNik, but we came into conclusion with amcrn and amrith17:06
*** ViswaV has joined #openstack-trove17:06
denis_makogonSlickNik, there was complain about use of system keyspace and we decided to drop it out17:06
*** pdmars has joined #openstack-trove17:06
*** ViswaV_ has joined #openstack-trove17:06
denis_makogonSlickNik, that's all, nothing else17:07
SlickNikWhen did you guys come to that conclusion? My last conversation with them was the the mid-cycle meetup.17:07
*** mmrtnz has joined #openstack-trove17:08
SlickNikWe can discuss what the reservations were once they're online.17:08
*** harlowja_away is now known as harlowja_17:09
denis_makogonSlickNik, conclusion was made right after discussion with cassandra folks17:09
denis_makogonSlickNik, i didn't know why Auston said that17:09
*** denis_makogon has quit IRC17:10
*** ViswaV has quit IRC17:11
glucasiccha: Latest replica patch set fixed the unit tests only17:15
openstackgerritSteve Leon proposed a change to openstack/trove: Make storage strategy available for trove API and TM  https://review.openstack.org/8624217:15
*** RicoLin has quit IRC17:17
*** Barker has quit IRC17:18
*** juantwo has quit IRC17:19
openstackgerritA change was merged to openstack/trove: Load trove API extensions using stevedore  https://review.openstack.org/11171417:25
*** ramishra has quit IRC17:28
*** dkehn_ has joined #openstack-trove17:39
*** shakamunyi has joined #openstack-trove17:40
*** dkehn_ has quit IRC17:41
*** dkehn_ has joined #openstack-trove17:41
*** dkehn has quit IRC17:42
*** dkehnx has quit IRC17:42
*** dkehn has joined #openstack-trove17:43
*** boblebauce has quit IRC17:50
*** amcrn has joined #openstack-trove17:51
*** tomblank has quit IRC17:51
*** pdmars has quit IRC17:56
amrithjust to be clear, I don't think anyone came to a 'conclusion' with me re: cassandra backup and restore.17:58
amrithif you had, and I concured with the decision/implementation, you'd find a +1 in the review.17:58
openstackgerritZhi Yan Liu proposed a change to openstack/trove: Integrate OSprofiler and Trove  https://review.openstack.org/11665317:58
*** tomblank has joined #openstack-trove18:00
amrithesmute, yt?18:12
esmutehello amrith18:13
amrithhi steve.18:13
amrithre: https://review.openstack.org/#/c/113155/518:13
amrithI see some of the config options you mentioned still being used18:13
amrithsee backup_runner_options, for example18:13
amrithI'd like to get this bug done and look at those more carefully18:13
amrithif that's ok with you.18:13
esmuteyeah that is fine18:14
amrithI'll look at the other two; the restore ones18:14
esmutethere are some configs are no longer used either like backup_strategy18:14
amrithyes, there needs to be a general cleanup there.18:14
esmuteor at least should not be used since it already has datastore specific one18:14
amrithshould have widened the scope of this bug to begin with18:15
amrithyes18:15
esmutecool.. np.. just wanted to point it out since you have started cleaning up these backup configs18:15
amrithso, I think [backup/restore]_namespace *should* be safe.18:16
*** pdmars has joined #openstack-trove18:16
amrithbackup_runner_options is still used which I find odd18:16
amrithbackup_runner isn't.18:16
amrithso maybe safer to put these on a list for later.18:16
esmuteyeah. i agree.. It is not as simple as to remove them or move it to datastore-specific groups18:17
*** vigneshvar has joined #openstack-trove18:18
vigneshvaramrith: hi18:18
openstackgerritamrith proposed a change to openstack/trove: make backup_incremental_strategy a datastore specific option  https://review.openstack.org/11315518:19
amrithvigneshvar, hi18:19
amrithI took a look at the proxy bug18:19
amriththat you pointed me to yesterday18:19
amrithand the older patches.18:19
amrithit occurs to me that what you are fixing looks very different from what the original person (the person who submitted the bug) is trying to fix18:19
amrithbut I'm still somewhat confused about what he is reporting as the problem18:20
vigneshvaramrith: ok. yes you are right18:20
vigneshvaramrith: what i am trying to fix is different18:20
amrithhe says the installation is failing because a proxy is set18:20
amrithand there there are cases where rest requests to the local openstack are going through a proxy18:20
vigneshvaramrith: yes, he says since proxy is set and no_proxy is not set18:20
amrithyou appear to be adding an HTTP_PROXY to sudo18:20
amrithwell, no_proxy is set to localhost only18:21
vigneshvaramrith: should i file the issue am trying to fix as different bug18:21
amrithso what he is talking about is that you should set no_proxy to include the range of addresses that is localhost, the IP of the OpenStack compute host and the floating IP range18:21
amrithwell, I don't know what the issue is that your fix would address.18:21
amrithso, you had for example, a pip install command18:22
amrithwhich was run with sudo.18:22
amrithI'm not sure that $HTTP_PROXY would impact that18:22
amrithin your case18:22
amrithyou seem to have an issue that you are behind a proxy server18:23
amrithand direct internet access is failing18:23
amrithbecause ReST calls are not using your proxy18:23
amrithis that right?18:23
vigneshvaramrith: if i dont set proxy globally and export it only on local terminal and run redstack18:23
vigneshvaramrith: then it is failing at sudo18:23
amrithwould you describe your network setup to me.18:23
amrithon the machine where you are running redstack18:24
vigneshvaramrith: sure18:24
amrithif you wanted to use a browser18:24
amrithand go to http://www.google.com18:24
amrithwould you have to set a proxy?18:24
vigneshvaramrith: yes you are right18:24
amrithso, on this machine, you would normally run with ${http_proxy} (lowercase) set, is that right?18:25
vigneshvaramrith: yes exactly18:25
vigneshvaramrith: my installation works only if http_proxy is set18:25
vigneshvarand no_proxy is required else it is failing during keystone calls18:26
amrithso the issue you are having is that the redstack script does not consistently set ${HTTP_PROXY} for all the sudo calls18:26
amrithand what you are fixing is a different bug18:26
amrithso, I'd suggest you do a couple of things.18:26
vigneshvaramrith: yes please18:26
amrithwait-a-sec18:26
vigneshvaramrith: sure18:26
amrithare you at HP?18:27
vigneshvaramrith: no , i work for reliance jio infocomm18:27
amrithok, one second18:28
amrithhmmm18:28
amrithare there any other people at your company who work on openstack?18:28
*** julienvey has quit IRC18:28
vigneshvaramrith: yap18:29
amrithone thing that would be good to do is to run the idea past them as well.18:29
amrithI would suggest that you search existing bugs18:30
amrithand confirm that there is no existing bug that fixes this18:30
amrithsorry, describes this18:30
amrithproblem18:30
vigneshvaramrith: ok ya i got it. i will do that first18:30
vigneshvaramrith: if it does not exist i will file a bug18:30
amrithyes, you should file it18:31
*** dkehn_ is now known as dkehnx18:31
*** julienvey has joined #openstack-trove18:32
vigneshvaramrith: i will try to fix the existing open bug as well18:32
vigneshvaramrith: quoting back to yesterdays question. no_proxy needs to be automated ?18:33
amriththat's a good question18:34
amrithso, you will have to fix not only your new bug18:35
amrithbut also the old bug18:35
*** julienve_ has joined #openstack-trove18:35
*** julienvey has quit IRC18:35
vigneshvaramrith: yes i would do that as well18:35
vigneshvaramrith: this is going to give me good experience18:35
amrithnow, if you look at that script18:35
amriththe one you are fixing18:35
amrithI believe there's another variable that lists the floating IP range18:36
amrithlet me look18:36
amrithI saw it somewhere18:36
amrithmaybe in the old bug fix18:36
amrithso if you can do both, then it would solve your problem18:36
amrithof being able to run behind a proxy18:36
amrithcompletely18:36
amrithvery cool!18:36
*** vigneshvar_ has joined #openstack-trove18:37
amrithsorry for the delay18:37
amrithplease ping me if it works (or if it doesn't).18:37
vigneshvaramrith: sure will check both and update18:37
*** vigneshvar_ has quit IRC18:38
*** vigneshvar has quit IRC18:38
*** vigneshvar has joined #openstack-trove18:38
vigneshvaramrith: thanks for helping me out18:39
amrithvigneshvar, don't say that yet. it hasn't yet worked ;)18:39
vigneshvaramrith: ;)18:40
vigneshvaramrith: i might bug you again for any clarifications18:40
openstackgerritMat Lowery proposed a change to openstack/trove: Clusters TaskManager Implementation  https://review.openstack.org/11369218:43
amrithvigneshvar, absolutely. happy to help.18:43
*** todd_dsm has joined #openstack-trove18:44
*** todd_dsm has quit IRC18:44
openstackgerritRamashri Umale proposed a change to openstack/trove: Clusters Guest Implementation  https://review.openstack.org/11372818:44
*** juantwo has joined #openstack-trove18:46
*** tomblank has quit IRC18:47
openstackgerritCraig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db  https://review.openstack.org/7985018:49
*** tomblank has joined #openstack-trove18:51
icchaglucas: any plans on adding intehgration tests for detach replica?18:53
*** georgelorch has quit IRC18:55
*** georgelorch has joined #openstack-trove18:55
*** tomblank has quit IRC18:56
*** georgelorch has quit IRC18:57
vigneshvaramrith: the bug i work for is in progress by someone, is it ok to take it up and fix18:57
amrithwhat's the bug #18:58
vigneshvarhttps://launchpad.net/bugs/129111118:58
vigneshvarhttps://review.openstack.org/#/c/94988/18:58
amrithone second, let me see if I can find an IRC name for this person18:59
*** georgelorch has joined #openstack-trove19:00
amrithit's been abandoned for long enough, I'd suggest you propose a fix and add the #Closes-Bug tag to your bug. Also, I'd email the other bugs owner; his email addresses are on his lp page19:00
amrithhttps://launchpad.net/~qiang-zu19:01
vigneshvaramrith: sure19:02
*** tomblank has joined #openstack-trove19:04
*** Barker has joined #openstack-trove19:09
*** tomblank has quit IRC19:19
*** tomblank has joined #openstack-trove19:19
*** flaper87 is now known as flaper87|afk19:21
*** pdmars_ has joined #openstack-trove19:29
*** pdmars_ has quit IRC19:29
*** openstackgerrit has quit IRC19:31
*** openstackgerrit has joined #openstack-trove19:32
*** sgotliv has joined #openstack-trove19:36
glucasiccha: I actually added a test to the api/replication.py, but it depends on the troveclient merging first so I did not push it up.19:36
glucasiccha: since those tests use the dbaas client, and the current client does not support edit (detach).19:37
icchaglucas: generally when patches depend on client we still add them to the patch. and there is a -1 on tests but we add in commit msg/comments saying it depends on pythonclient merge.19:37
glucasiccha: I can add it.. hopefully the client side goes through soon anyway.19:38
icchamaybe amrith can +2 and merge it :) now that he is core. i see him on list of reviewers on patch glucas19:39
glucasiccha: :-) Trying not to abuse the fact that I could throw something at amrith from where I set.19:40
icchahehe19:42
amriththe funny thing about throwing things from point A to point B is this. the person at point B is still close enough to throw things at point A.19:44
glucasouch!19:44
amrithcase in point, I have a bottle of honey.19:44
amrithwhat do you have?19:44
* glucas plans to work at home tomorrow19:44
openstackgerritA change was merged to openstack/trove: handle repeating mysqld options containing equals  https://review.openstack.org/10959519:58
*** jasonb365 has quit IRC19:59
vigneshvaramrith: added this bug https://bugs.launchpad.net/trove/+bug/136279720:00
amrithvigneshvar, looks good.20:01
vigneshvaramrith: i will now update the commit message and upload the patchset20:03
*** jasonb365 has joined #openstack-trove20:03
*** kevinconway has quit IRC20:10
*** tomblank has quit IRC20:18
*** Barker has quit IRC20:21
*** denis_makogon has joined #openstack-trove20:26
openstackgerritvigneshvar proposed a change to openstack/trove-integration: Carries the proxy variable while installing trove-integration  https://review.openstack.org/11253220:26
denis_makogonSlickNik, are you there?20:28
vigneshvaramrith: when you get time, kindly check my patch https://review.openstack.org/#/c/112532/20:28
*** jasonb365 has quit IRC20:29
denis_makogonamrith, take a look at https://bugs.launchpad.net/trove/+bug/1362318, please20:30
denis_makogonSlickNik, please take a look at https://bugs.launchpad.net/trove/+bug/136231820:31
SlickNikdenis_makogon: yes, what's up?20:31
*** esp has joined #openstack-trove20:32
vigneshvarSlickNik: Your comments on patchset 5 -  https://review.openstack.org/#/c/112532/ made me understand about the mistake. I have filed a new bug and filed my patch against it. Kindly review it20:32
*** Barker has joined #openstack-trove20:33
denis_makogonSlickNik, i've noticed randomly failing fake int-tests (described at bug report), and already proposed fix for it. I tried it multiple times and it didn't happen again20:33
*** tomblank has joined #openstack-trove20:33
*** jasonb365 has joined #openstack-trove20:33
SlickNikdenis_makogon: hmm, I haven't come across this failure. Is there a log with the failure somewhere perhaps during a gate job?20:33
denis_makogonSlickNik, yes, i added them into bug report20:34
SlickNikvigneshvar: Thanks! Will take a look with the other reviews.20:34
denis_makogonSlickNik, i came across this problem with recent patchset https://review.openstack.org/#/c/117255/220:35
denis_makogonSlickNik, i tried "recheck 3 times" and it helped20:35
vigneshvarSlickNik: thanks sure.20:36
denis_makogonSlickNik, even taking into account that recent patchset is #3 (pep8 fixs from #2)20:37
SlickNikdenis_makogon: How many times did you hit this? Is it something that's one off, that we aren't hitting that frequently.20:37
*** dkehn_ has joined #openstack-trove20:37
SlickNikAlso looking at the fix, it doesn't address the percent being at 102 (which the stack trace seems to suggest).20:38
denis_makogonSlickNik, actually it is20:38
*** vigneshvar has quit IRC20:38
denis_makogon102% = 2048/200420:38
SlickNikWhat about line 149?20:39
SlickNikThat would still fail20:39
*** julienve_ has quit IRC20:39
SlickNikSo it looks like you haven't hit the issue again, even in your re-runs.20:40
*** dkehn_ has quit IRC20:40
*** dkehn_ has joined #openstack-trove20:40
*** dkehn has quit IRC20:41
*** dkehnx has quit IRC20:41
*** vigneshvar_ has joined #openstack-trove20:41
SlickNikWe should probably monitor the issue and see if it happens again — no reason to try and expedite a fix for something that's seems to me a one-off issue..20:41
*** sgotliv has quit IRC20:41
denis_makogonSlickNik, take a look at patchset #120:41
denis_makogonSlickNik, same issue as for #220:42
*** fifieldt_ has quit IRC20:42
*** dkehn has joined #openstack-trove20:42
*** vigneshvar has joined #openstack-trove20:43
SlickNikdenis_makogon: maybe you should write an elastic recheck query to track how many times this is happening?20:44
SlickNikI don't think anyone else has run into this other than you.20:45
denis_makogonSlickNik, sounds like a plan20:45
*** julienvey has joined #openstack-trove20:45
denis_makogonSlickNik, i will look at how to write queries for elastic-recheck20:46
denis_makogonSlickNik, i'll abandon this fix lets see how often it would happen20:47
vigneshvardenis_makogon: pls review it when you get some spare time, https://review.openstack.org/#/c/112532/ made an update.20:52
*** vigneshvar has quit IRC20:52
denis_makogonvigneshvar_, sure20:52
vigneshvar_Thanks20:53
*** radez is now known as radez_g0n320:54
denis_makogonvigneshvar_, done20:55
*** AndChat|329121 has joined #openstack-trove20:56
*** fifieldt_ has joined #openstack-trove20:59
*** vigneshvar_ has quit IRC20:59
*** tomblank has quit IRC21:01
*** dkehn__ has joined #openstack-trove21:02
*** Riddhi has quit IRC21:03
*** dkehn__ has quit IRC21:05
*** dkehn__ has joined #openstack-trove21:05
*** dkehn_ has quit IRC21:05
*** dkehn has quit IRC21:05
*** dkehn has joined #openstack-trove21:07
*** dkehn__ is now known as dkehnx21:12
openstackgerritNikhil Manchanda proposed a change to openstack/trove: Cleaned up sample trove-guestagent.conf  https://review.openstack.org/10406221:15
*** jcru has quit IRC21:20
*** jcru has joined #openstack-trove21:21
*** kevinconway has joined #openstack-trove21:22
*** ViswaV has joined #openstack-trove21:36
*** ViswaV_ has quit IRC21:36
*** robertmyers has quit IRC21:36
*** Barker has quit IRC21:52
*** Barker has joined #openstack-trove21:53
*** jasonb365 has quit IRC21:59
*** denis_makogon has quit IRC22:05
*** sriram_ has quit IRC22:08
*** amcrn has quit IRC22:08
*** sriram_tesora has quit IRC22:09
*** amcrn has joined #openstack-trove22:14
*** IanGovett1 has quit IRC22:14
*** amcrn has quit IRC22:17
*** arist has quit IRC22:23
*** arist has joined #openstack-trove22:24
*** georgelorch has quit IRC22:30
*** georgelorch has joined #openstack-trove22:30
icchaSlickNik: alos regarding https://review.openstack.org/#/c/112995/ i think we need to deprecate config values is the protocol. not remove them immediately22:33
icchabut I may be wrong. so might be worth checking out22:34
*** dkehn_ has joined #openstack-trove22:39
*** dkehn_ has quit IRC22:42
*** dkehn has quit IRC22:42
*** dkehn_ has joined #openstack-trove22:42
*** dkehnx has quit IRC22:42
*** jcru has quit IRC22:44
*** dkehn has joined #openstack-trove22:44
*** mattgriffin has quit IRC22:54
*** esp has left #openstack-trove22:55
*** julienvey has quit IRC22:56
*** vkmc has quit IRC22:59
kevinconwayanyone around?23:02
*** Barker has quit IRC23:05
*** cweid has quit IRC23:11
*** topshare has joined #openstack-trove23:30
*** shakamunyi has quit IRC23:33
*** achampion has quit IRC23:38
*** amcrn has joined #openstack-trove23:40
*** topshare has quit IRC23:42
hub_capjust u23:43
*** mattgriffin has joined #openstack-trove23:45
*** openstackgerrit has quit IRC23:46
*** openstackgerrit has joined #openstack-trove23:46

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