| *** Isotopp_ has joined #openstack-trove | 00:03 | |
| *** pdmars_ has joined #openstack-trove | 00:04 | |
| *** hub_cap has joined #openstack-trove | 00:05 | |
| *** zacksh_ has joined #openstack-trove | 00:09 | |
| *** coolsvap_ has joined #openstack-trove | 00:13 | |
| *** georgelorch has quit IRC | 00:13 | |
| *** coolsvap has quit IRC | 00:13 | |
| *** hub_cap_ has quit IRC | 00:13 | |
| *** adam_g has quit IRC | 00:13 | |
| *** pdmars has quit IRC | 00:13 | |
| *** zacksh has quit IRC | 00:13 | |
| *** Isotopp has quit IRC | 00:13 | |
| *** coolsvap_ is now known as coolsvap | 00:13 | |
| *** georgelorch has joined #openstack-trove | 00:15 | |
| *** adam_g has joined #openstack-trove | 00:19 | |
| *** adam_g has quit IRC | 00:19 | |
| *** adam_g has joined #openstack-trove | 00:19 | |
| *** prasoon has joined #openstack-trove | 00:42 | |
| *** dkehnx1_ has quit IRC | 01:03 | |
| *** tomblank has quit IRC | 01:07 | |
| *** juantwo_ has joined #openstack-trove | 01:10 | |
| *** juantwo has quit IRC | 01:13 | |
| *** Longgeek has joined #openstack-trove | 01:15 | |
| *** prasoon has quit IRC | 01:18 | |
| *** juantwo_ has quit IRC | 01:21 | |
| *** juantwo has joined #openstack-trove | 01:21 | |
| *** georgelorch has quit IRC | 01:30 | |
| *** georgelorch has joined #openstack-trove | 01:31 | |
| *** mrsnivvel has joined #openstack-trove | 01:53 | |
| *** haomaiwang has joined #openstack-trove | 02:10 | |
| *** prasoon has joined #openstack-trove | 02:18 | |
| *** ramishra has joined #openstack-trove | 02:28 | |
| *** ramishra has quit IRC | 02:36 | |
| *** ramishra has joined #openstack-trove | 02:37 | |
| *** tomblank has joined #openstack-trove | 02:51 | |
| *** jasonb365 has joined #openstack-trove | 02:53 | |
| *** kfox1111 has quit IRC | 03:14 | |
| *** Longgeek_ has joined #openstack-trove | 03:19 | |
| *** Longgeek has quit IRC | 03:22 | |
| *** haomaiwang has quit IRC | 03:31 | |
| *** haomaiwang has joined #openstack-trove | 03:32 | |
| *** ramishra has quit IRC | 03:33 | |
| *** ramishra has joined #openstack-trove | 03:33 | |
| *** ramishra has quit IRC | 03:37 | |
| *** ramishra has joined #openstack-trove | 03:38 | |
| *** haomaiw__ has joined #openstack-trove | 03:38 | |
| *** ramishra has quit IRC | 03:41 | |
| *** ramishra has joined #openstack-trove | 03:42 | |
| *** ramishra has quit IRC | 03:42 | |
| *** haomaiwang has quit IRC | 03:42 | |
| *** ramishra has joined #openstack-trove | 03:42 | |
| *** ramishra has quit IRC | 03:44 | |
| *** ramishra has joined #openstack-trove | 03:44 | |
| *** jasonb365 has quit IRC | 03:59 | |
| *** Riddhi has joined #openstack-trove | 03:59 | |
| *** Riddhi has quit IRC | 04:04 | |
| *** sgotliv has joined #openstack-trove | 04:06 | |
| *** sgotliv_ has joined #openstack-trove | 04:06 | |
| *** achampion has joined #openstack-trove | 04:24 | |
| *** Longgeek_ has quit IRC | 04:27 | |
| *** achampio1 has quit IRC | 04:27 | |
| *** Longgeek has joined #openstack-trove | 04:27 | |
| *** Longgeek_ has joined #openstack-trove | 04:35 | |
| *** rushiagr_away is now known as rushiagr | 04:37 | |
| *** Longgeek has quit IRC | 04:38 | |
| *** ViswaV has joined #openstack-trove | 04:47 | |
| *** ViswaV has quit IRC | 04:52 | |
| *** Longgeek_ has quit IRC | 04:53 | |
| *** ViswaV has joined #openstack-trove | 04:53 | |
| *** ViswaV_ has joined #openstack-trove | 05:03 | |
| *** ViswaV has quit IRC | 05:05 | |
| *** ViswaV has joined #openstack-trove | 05:10 | |
| *** ViswaV_ has quit IRC | 05:12 | |
| *** juantwo has quit IRC | 05:19 | |
| *** rushiagr is now known as rushiagr_away | 05:24 | |
| *** Longgeek has joined #openstack-trove | 05:29 | |
| *** Longgeek has quit IRC | 05:31 | |
| *** Longgeek has joined #openstack-trove | 05:32 | |
| *** ajayaa has joined #openstack-trove | 05:49 | |
| *** tomblank has quit IRC | 05:53 | |
| *** sgotliv has quit IRC | 05:55 | |
| *** sgotliv_ has quit IRC | 05:56 | |
| *** k4n0 has joined #openstack-trove | 05:57 | |
| *** Longgeek has quit IRC | 06:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex https://review.openstack.org/117695 | 06:07 |
|---|---|---|
| *** Longgeek has joined #openstack-trove | 06:11 | |
| *** rushiagr_away is now known as rushiagr | 06:18 | |
| *** achampio1 has joined #openstack-trove | 06:18 | |
| *** achampion has quit IRC | 06:20 | |
| *** rushiagr is now known as rushiagr_away | 06:23 | |
| *** flaper87|afk is now known as flaper87 | 06:26 | |
| *** vigneshvar has joined #openstack-trove | 06:27 | |
| *** ViswaV has quit IRC | 06:39 | |
| *** ViswaV_ has joined #openstack-trove | 06:39 | |
| *** ajayaa has quit IRC | 06:59 | |
| *** ViswaV_ has quit IRC | 07:03 | |
| *** haomaiw__ has quit IRC | 07:13 | |
| *** haomaiwang has joined #openstack-trove | 07:13 | |
| *** ajayaa has joined #openstack-trove | 07:20 | |
| *** haomaiw__ has joined #openstack-trove | 07:29 | |
| *** haomaiwang has quit IRC | 07:32 | |
| *** ramishra has quit IRC | 07:39 | |
| *** ramishra has joined #openstack-trove | 07:39 | |
| *** vigneshvar has quit IRC | 07:41 | |
| *** sgotliv has joined #openstack-trove | 07:49 | |
| openstackgerrit | Sergey Gotliv proposed a change to openstack/trove: Make --repo-path an optional argument for db_recreate https://review.openstack.org/85700 | 07:56 |
| *** boblebauce has joined #openstack-trove | 07:59 | |
| *** coolsvap has quit IRC | 08:04 | |
| *** coolsvap has joined #openstack-trove | 08:04 | |
| *** k4n0 has quit IRC | 08:05 | |
| *** k4n0 has joined #openstack-trove | 08:05 | |
| *** ramishra has quit IRC | 08:06 | |
| *** ramishra has joined #openstack-trove | 08:06 | |
| *** sgotliv has quit IRC | 08:06 | |
| *** sgotliv has joined #openstack-trove | 08:06 | |
| *** sgotliv_ has joined #openstack-trove | 08:07 | |
| *** iartarisi has joined #openstack-trove | 08:22 | |
| *** ajayaa has quit IRC | 08:37 | |
| *** k4n0 has quit IRC | 08:48 | |
| *** k4n0 has joined #openstack-trove | 09:07 | |
| *** ramishra has quit IRC | 09:11 | |
| *** ramishra has joined #openstack-trove | 09:11 | |
| *** ramishra has quit IRC | 09:26 | |
| openstackgerrit | Sergey Gotliv proposed a change to openstack/trove: Port RPC API to use oslo.messaging library https://review.openstack.org/94484 | 09:26 |
| *** ramishra has joined #openstack-trove | 09:26 | |
| *** blipp has joined #openstack-trove | 09:27 | |
| *** blipp_ has quit IRC | 09:30 | |
| *** ramishra has quit IRC | 09:32 | |
| *** ramishra has joined #openstack-trove | 09:32 | |
| *** rushiagr_away is now known as rushiagr | 09:42 | |
| *** ajayaa has joined #openstack-trove | 09:56 | |
| *** sgotliv has quit IRC | 10:19 | |
| *** sgotliv_ has quit IRC | 10:20 | |
| *** vigneshvar has joined #openstack-trove | 10:23 | |
| *** ajayaa has quit IRC | 10:30 | |
| *** ajayaa has joined #openstack-trove | 10:31 | |
| *** Krast has joined #openstack-trove | 10:34 | |
| *** Krast has quit IRC | 10:35 | |
| *** rushiagr is now known as rushiagr_away | 10:37 | |
| *** k4n0 has quit IRC | 10:42 | |
| *** rushiagr_away is now known as rushiagr | 10:44 | |
| *** mrsnivvel has quit IRC | 10:47 | |
| *** ajayaa has quit IRC | 10:54 | |
| *** ajayaa has joined #openstack-trove | 10:54 | |
| *** sgotliv has joined #openstack-trove | 10:56 | |
| *** sgotliv_ has joined #openstack-trove | 10:56 | |
| *** rushiagr is now known as rushiagr_away | 10:56 | |
| *** IanGovett has joined #openstack-trove | 11:00 | |
| *** prasoon has quit IRC | 11:01 | |
| *** k4n0 has joined #openstack-trove | 11:03 | |
| *** vkmc has joined #openstack-trove | 11:06 | |
| *** vkmc has joined #openstack-trove | 11:06 | |
| *** vigneshvar has quit IRC | 11:09 | |
| *** vigneshvar has joined #openstack-trove | 11:20 | |
| *** ramishra has quit IRC | 11:24 | |
| *** _amrith_ is now known as amrith | 11:31 | |
| openstackgerrit | Sergey Gotliv proposed a change to openstack/trove: Port RPC API to use oslo.messaging library https://review.openstack.org/94484 | 11:41 |
| *** rushiagr_away is now known as rushiagr | 11:42 | |
| *** prasoon has joined #openstack-trove | 12:03 | |
| *** sgotliv_ has quit IRC | 12:05 | |
| *** denis_makogon has joined #openstack-trove | 12:11 | |
| *** Longgeek has quit IRC | 12:25 | |
| *** ramashri has joined #openstack-trove | 12:26 | |
| *** juantwo has joined #openstack-trove | 12:27 | |
| *** juantwo has joined #openstack-trove | 12:27 | |
| *** ajayaa has quit IRC | 12:31 | |
| *** ramishra has joined #openstack-trove | 12:33 | |
| *** tomblank has joined #openstack-trove | 12:37 | |
| *** Longgeek has joined #openstack-trove | 12:39 | |
| *** ajayaa has joined #openstack-trove | 12:44 | |
| *** jasonb365 has joined #openstack-trove | 13:00 | |
| openstackgerrit | A change was merged to openstack/trove: Imported Translations from Transifex https://review.openstack.org/117695 | 13:04 |
| *** k4n0 has quit IRC | 13:13 | |
| *** boblebauce has quit IRC | 13:13 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Fix master status validation while replica provisioning https://review.openstack.org/119770 | 13:18 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Forbid replica provisioning from non-root replica site https://review.openstack.org/119771 | 13:19 |
| *** achampio1 has quit IRC | 13:37 | |
| *** robertmyers has joined #openstack-trove | 13:39 | |
| *** achampion has joined #openstack-trove | 13:42 | |
| *** radez_g0n3 is now known as radez | 13:49 | |
| *** ramashri has quit IRC | 14:15 | |
| *** ramishra has quit IRC | 14:18 | |
| openstackgerrit | Sergey Gotliv proposed a change to openstack/trove: Port RPC API to use oslo.messaging library https://review.openstack.org/94484 | 14:21 |
| *** Longgeek has quit IRC | 14:21 | |
| *** rwsu has joined #openstack-trove | 14:21 | |
| *** grapex has joined #openstack-trove | 14:24 | |
| *** sgotliv has quit IRC | 14:29 | |
| *** Isotopp_ is now known as Isotopp | 14:33 | |
| *** jasonb365_ has joined #openstack-trove | 14:38 | |
| *** jasonb365 has quit IRC | 14:40 | |
| *** jasonb365_ is now known as jasonb365 | 14:40 | |
| *** kevinconway has joined #openstack-trove | 14:42 | |
| *** sgotliv has joined #openstack-trove | 14:44 | |
| *** ramishra has joined #openstack-trove | 14:49 | |
| *** ramashri has joined #openstack-trove | 14:59 | |
| *** Riddhi has joined #openstack-trove | 15:00 | |
| *** rushiagr is now known as rushiagr_away | 15:05 | |
| *** georgelorch has quit IRC | 15:06 | |
| *** georgelorch has joined #openstack-trove | 15:07 | |
| *** vigneshvar has quit IRC | 15:08 | |
| *** jmontemayor has joined #openstack-trove | 15:10 | |
| *** jmontemayor_ has joined #openstack-trove | 15:12 | |
| *** sgotliv has quit IRC | 15:13 | |
| *** jmontemayor has quit IRC | 15:15 | |
| *** tomblank has quit IRC | 15:20 | |
| *** kfox1111 has joined #openstack-trove | 15:40 | |
| *** ramashri has quit IRC | 15:41 | |
| *** tomblank has joined #openstack-trove | 15:50 | |
| *** newb has joined #openstack-trove | 15:52 | |
| *** mattgriffin has joined #openstack-trove | 15:52 | |
| *** jasonb365_ has joined #openstack-trove | 16:02 | |
| *** eghobo has joined #openstack-trove | 16:02 | |
| *** jasonb365 has quit IRC | 16:02 | |
| *** jasonb365_ is now known as jasonb365 | 16:02 | |
| *** ajayaa has quit IRC | 16:04 | |
| *** rushiagr_away is now known as rushiagr | 16:05 | |
| *** tomblank has quit IRC | 16:06 | |
| openstackgerrit | iccha-sethi proposed a change to openstack/trove: Delete backup created during replication https://review.openstack.org/119169 | 16:08 |
| *** tomblank has joined #openstack-trove | 16:09 | |
| *** vigneshvar has joined #openstack-trove | 16:28 | |
| *** ramashri has joined #openstack-trove | 16:28 | |
| *** kfox1111 has quit IRC | 16:28 | |
| *** iartarisi has quit IRC | 16:31 | |
| *** eghobo has quit IRC | 16:31 | |
| *** zacksh_ is now known as zacksh | 16:33 | |
| amrith | mat-lowery, got a sec? | 16:34 |
| mat-lowery | Yes amrith. | 16:36 |
| amrith | hi mat | 16:36 |
| amrith | saw your question re: strings, list and shell=True | 16:36 |
| amrith | thought I'd talk to you about that here | 16:36 |
| amrith | as it may be easier than a longish comment in review | 16:37 |
| *** eghobo has joined #openstack-trove | 16:37 | |
| amrith | ok? | 16:38 |
| mat-lowery | That would be great and I want to work in a manner that is convenient for you. However, I do fear that having conversation outside of Gerrit means that others cannot see the answers to the (dumb) questions I ask. | 16:38 |
| mat-lowery | Let's talk here and I'll summarize the answer in Gerrit? How about that? | 16:39 |
| amrith | no worries, I'll summarize for gerrit | 16:39 |
| amrith | the Popen() constructor that we use accepts a command | 16:39 |
| amrith | that can be either a string or a list | 16:39 |
| *** ramashri has quit IRC | 16:40 | |
| mat-lowery | gotcha so far | 16:40 |
| amrith | if you specify shell=False (the default) then it means that the elements of the list (or a string in its entirety) are to be used for execution. | 16:40 |
| amrith | so, for example, ['ls', '-al'] | 16:40 |
| amrith | would be executed as the command | 16:41 |
| amrith | ls -al | 16:41 |
| amrith | i.e. each element in the list would be part of the staetment that is executed and the first element is the 'command' | 16:41 |
| amrith | if you specify shell=True then you are required to pass a string that is (in its entirety) passed to the default shell (on *x that's sh, on Windows it is whatever COMSPEC is set to) | 16:42 |
| amrith | so subprocess.Popen("ls -al", shell=True, ...) | 16:42 |
| amrith | is equivalent to | 16:42 |
| amrith | subprocess.Popen(["/bin/sh", "ls", "-al"], shell=False, ...) | 16:43 |
| amrith | it is considered to be a security risk to use a complete string | 16:43 |
| amrith | because of string injection exploits | 16:43 |
| amrith | and it is preferred that you pass a list and set Shell to be false (or not specify) | 16:43 |
| amrith | see: https://docs.python.org/2/library/subprocess.html#subprocess.Popen | 16:44 |
| amrith | and https://docs.python.org/2/library/subprocess.html#frequently-used-arguments | 16:44 |
| amrith | hence the change from strings to lists and disabling shell = true | 16:44 |
| amrith | now, there are cases where you have no choice | 16:44 |
| amrith | such as when a command has indirection | 16:44 |
| amrith | ls -al | foo | bar | 16:44 |
| amrith | this is a command that you cannot pass down as ["ls", "-al", "|", "foo", "|", "bar"] etc | 16:45 |
| *** ramashri has joined #openstack-trove | 16:45 | |
| amrith | in these cases, shell = true is required | 16:45 |
| amrith | (for now) | 16:45 |
| mat-lowery | It sort of makes sense. I don't really get why you can't send a string in with shell=False. | 16:46 |
| amrith | because | 16:46 |
| amrith | the entire "string" | 16:47 |
| amrith | is passed to the interpreter | 16:47 |
| amrith | consider this | 16:47 |
| amrith | if I want to execute the command "ls -al" | 16:47 |
| amrith | then, you are searching for "ls" in the PATH | 16:47 |
| amrith | as you can imagine, you won't find "ls -al" in $PATH ;) | 16:48 |
| mat-lowery | OK. It makes sense now. Thank you for explaining it to me. | 16:48 |
| *** ViswaV has joined #openstack-trove | 16:49 | |
| amrith | your other question | 16:50 |
| amrith | about the future plans | 16:50 |
| amrith | yes, I have a sweep on for anything to do with "shell=true" | 16:50 |
| amrith | also for things that are doing sudo | 16:50 |
| mat-lowery | Gotcha. So are all the file perm issues taken care of? | 16:51 |
| amrith | not all | 16:55 |
| amrith | there is a future chkin that should deal with several of these things | 16:55 |
| *** sgotliv has joined #openstack-trove | 16:56 | |
| *** georgelorch has quit IRC | 16:57 | |
| *** georgelorch has joined #openstack-trove | 16:58 | |
| amrith | I will summarize this for review | 17:00 |
| mat-lowery | Thanks amrith! | 17:00 |
| amrith | np mat | 17:00 |
| *** harlowja has joined #openstack-trove | 17:04 | |
| amrith | mat-lowery, looks like you summarized for review ;) | 17:09 |
| amrith | thanks! | 17:09 |
| mat-lowery | Well the link you gave (and I posted in the review) was helpful but the real solution for me was the chat. So thanks! | 17:10 |
| *** ramishra has quit IRC | 17:12 | |
| *** johnma has joined #openstack-trove | 17:12 | |
| *** vigneshvar has quit IRC | 17:12 | |
| amrith | :) | 17:13 |
| *** eghobo has quit IRC | 17:16 | |
| *** georgelorch has quit IRC | 17:21 | |
| *** georgelorch has joined #openstack-trove | 17:22 | |
| *** eghobo has joined #openstack-trove | 17:22 | |
| *** vigneshvar has joined #openstack-trove | 17:25 | |
| *** amcrn has joined #openstack-trove | 17:26 | |
| *** ViswaV has quit IRC | 17:37 | |
| *** tkatarki has joined #openstack-trove | 17:40 | |
| *** tkatarki has quit IRC | 17:42 | |
| *** tkatarki has joined #openstack-trove | 17:44 | |
| *** georgelorch has quit IRC | 17:48 | |
| *** georgelorch has joined #openstack-trove | 17:48 | |
| *** tkatarki has quit IRC | 17:53 | |
| *** georgelorch has quit IRC | 17:53 | |
| *** ViswaV has joined #openstack-trove | 17:53 | |
| *** georgelorch has joined #openstack-trove | 17:54 | |
| *** tkatarki has joined #openstack-trove | 17:58 | |
| *** prasoon has quit IRC | 18:00 | |
| SlickNik | #startmeeting trove-bp-review | 18:00 |
| openstack | Meeting started Mon Sep 15 18:00:46 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 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 18:00 |
| *** prasoon has joined #openstack-trove | 18:00 | |
| openstack | The meeting name has been set to 'trove_bp_review' | 18:00 |
| iccha2 | o/ | 18:00 |
| grapex | o/ | 18:01 |
| denis_makogon | o/ | 18:01 |
| amrith | ./ | 18:01 |
| SlickNik | agenda at: | 18:01 |
| amcrn | O/ | 18:01 |
| SlickNik | #link https://wiki.openstack.org/wiki/Meetings/TroveBPMeeting | 18:01 |
| schang | o/ | 18:02 |
| SlickNik | #topic OSProfiler integration | 18:02 |
| SlickNik | zhiyan_: around? | 18:02 |
| *** rushiagr is now known as rushiagr_away | 18:03 | |
| SlickNik | Looks like not. | 18:03 |
| SlickNik | Let's come back to this one later. | 18:04 |
| SlickNik | #Topic Cassandra clustering | 18:04 |
| denis_makogon | it is mine | 18:04 |
| vgnbkr | o/ | 18:04 |
| SlickNik | #linkhttps://blueprints.launchpad.net/trove/+spec/cassandra-cluster | 18:04 |
| sgotliv | o/ | 18:04 |
| denis_makogon | initial framework was merged | 18:04 |
| *** prasoon has quit IRC | 18:05 | |
| denis_makogon | and as i can see, it's time to go forward with adding custering support for other datastores | 18:05 |
| denis_makogon | this topic is about Cassandra clustering | 18:05 |
| SlickNik | denis_makogon: https://wiki.openstack.org/wiki/Trove/cassandra-clustering seems a little light on details. | 18:06 |
| SlickNik | For example, what data partitioning strategy are we proposing to use? | 18:06 |
| amcrn | agreed, it needs to resemble https://wiki.openstack.org/wiki/Trove/Clusters-MongoDB | 18:07 |
| amrith | I posted some comments this AM | 18:07 |
| SlickNik | Are we doing replication? How are Snitches handled? | 18:07 |
| vigneshvar | amrith: denis_makogon SlickNik : pls give your valuable comments/suggestions on https://review.openstack.org/#/c/103186/ | 18:07 |
| *** tkatarki has quit IRC | 18:08 | |
| amrith | using the terminology of a code review; if this were put into an RST etc., I would believe that this needs more work | 18:08 |
| amrith | at this stage, I think it is premature to approve | 18:08 |
| amrith | because I'm not able to tell for sure whether cassandra will 'fit' in the current framework cleanly | 18:08 |
| amrith | and what the 'rough edges' could be. | 18:08 |
| amrith | but I think the bp is a start | 18:08 |
| amrith | and needs to be finished | 18:08 |
| iccha2 | also should we start using specs repo since we re looking at bps for kilo | 18:09 |
| SlickNik | amrith: +1 | 18:09 |
| georgelorch | o/ | 18:09 |
| dougshelley66 | o/ | 18:09 |
| SlickNik | iccha2: Yes, getting that repo up and going is in the works. Should be done for next week's BP meeting. | 18:09 |
| amrith | In particular, I would like to make sure that whatever we implement/extend of the framework is also compatible with things (like MySQL/Percona) | 18:10 |
| iccha2 | swwet thanks SlickNik | 18:10 |
| amrith | iccha2, +1 | 18:10 |
| SlickNik | iccha2 / amrith: https://review.openstack.org/#/c/121457/ | 18:10 |
| SlickNik | But coming back to this BP — denis_makogon can you switch to the rst format as you continue working on it? | 18:11 |
| SlickNik | I suspect folks will have quite a few comments on this, and it'll be easier to review the .rst | 18:11 |
| denis_makogon | ok | 18:12 |
| *** georgelorch2 has joined #openstack-trove | 18:12 | |
| SlickNik | denis_makogon: thanks. | 18:12 |
| SlickNik | #topic Clustering int-tests | 18:13 |
| denis_makogon | that one is pretty simple | 18:13 |
| SlickNik | #link https://blueprints.launchpad.net/trove/+spec/clustering-int-tests | 18:13 |
| denis_makogon | there's no int tests for mongo clustering at all | 18:14 |
| denis_makogon | it was added to be approved by PTL | 18:14 |
| denis_makogon | for now we have several API endpoints that sould be tested | 18:14 |
| *** georgelorch has quit IRC | 18:14 | |
| amcrn | denis_makogon: is the intent of adding this to the agenda to ask about its status or ? | 18:15 |
| SlickNik | denis_makogon: The design for this needs to be thought through as well. Should every int-tests be spinning up 5 instances for a cluster? How does this effect int-test workload, and run time? | 18:15 |
| denis_makogon | amcrn, i planned to work on it, if there's no objections | 18:17 |
| grapex | I think we should add int-tests that are optional- as in , we don't *have* to run them all the time and ruin the gate- but *could* be run | 18:17 |
| grapex | and could also run in fake mode- which would require fixing the event simulator. :/ I could do that. | 18:17 |
| SlickNik | denis_makogon: afaik amcrn, and mat-lowery have been trying to answer some of these design questions to come up with something that is acceptable to run in the gate. | 18:17 |
| amcrn | SlickNik: +1, but we wouldn't mind your help denis_makogon | 18:17 |
| denis_makogon | SlickNik, ok, thanks | 18:18 |
| amcrn | denis_makogon: we'll try to share some details fairly soon about the options we've been discussing | 18:18 |
| *** vigneshvar has quit IRC | 18:18 | |
| amrith | At this point, I think the work is "definition" | 18:18 |
| SlickNik | grapex: +1. Especially on getting fake mode to cover this so that we can test that in the gate. | 18:18 |
| denis_makogon | amcrn, awesome, thanks | 18:18 |
| amrith | I understood the approval to mean that in principle we agree with the proposal | 18:18 |
| amrith | but the devil is in the details | 18:18 |
| amrith | so who is ironing out the details ;) | 18:18 |
| amrith | I'm assuming this is mat-lowery amrith SlickNik ... | 18:19 |
| amrith | yes? | 18:19 |
| grapex | mat-lowery: didn't you have tests that were almost running until you found out about the fake mode limitation? | 18:19 |
| mat-lowery | Yeah due to the resources required for a real int test, I was planning on the fake mode-only route first. But then I ran into event simulator limitations. | 18:19 |
| denis_makogon | amrith, i guess amcrn and mat-lowery would answer all questions since they are already working on them, but i didn't know about that (my bad =( ) | 18:19 |
| grapex | Let's proceed with those but not add them to any groups which would run in the gate | 18:20 |
| grapex | then hopefully I can fix the event simulator limitations | 18:20 |
| amcrn | denis_makogon: we should have flipped the blueprint assignee, i screwed up there, apologies. | 18:20 |
| SlickNik | amrith: I believe in addition to amcrn and grapex as well. | 18:20 |
| grapex | SlickNik amcrn amrith: Would that work for you? | 18:20 |
| denis_makogon | amcrn, it's totally fine | 18:20 |
| amrith | grapex, absolutely | 18:21 |
| amrith | no objections from me. | 18:21 |
| SlickNik | grapex / mat-lowery: Yes, I think adding int-tests even if it's not part of the gate would be good. | 18:21 |
| amrith | I would like us to keep one thing in mind as we make these choices | 18:21 |
| amcrn | not sure what i'm agreeing to, but sure :) | 18:21 |
| amrith | for example | 18:22 |
| amrith | SUSE is goign to run their own CI | 18:22 |
| amrith | we should make sure that the things we choose will work on their CI as well, without too much of a burden | 18:22 |
| amrith | otherwise, things that we have approved, such as SUSE support would be impacted | 18:22 |
| amrith | because we implicitly are relying on their CI to find issues in code we commit | 18:23 |
| denis_makogon | amrith, not sure about that, since SUSE CI is a third-party CI, if they would have an issues - they would come to us with questions | 18:23 |
| denis_makogon | because we can't say for sure that it would work for them | 18:23 |
| amrith | denis_makogon, my issue was related to things like the numebr of instances we'd spin up | 18:24 |
| amrith | for each test | 18:24 |
| amrith | or things like that | 18:24 |
| denis_makogon | ok, i get that | 18:24 |
| amrith | if we expect that to test clustering, your CI infrastructure should be enormous | 18:24 |
| amrith | taht may have a complicating effect | 18:24 |
| *** georgelorch2 is now known as georgelorch | 18:24 | |
| amrith | on other database vendors who want to participate | 18:24 |
| amrith | I don't think there is an imminent threat of that happening | 18:25 |
| amrith | but just something to keep in mind | 18:25 |
| amrith | dougshelley66, has a CI setup that we're operating | 18:25 |
| amrith | I have to keep that in mind as well | 18:25 |
| amrith | similarly, Percona | 18:25 |
| amrith | georgelorch, will have one he ahs to operate | 18:25 |
| amrith | and so on | 18:25 |
| amcrn | i'm onboard for fake-mode, but real int-tests is difficult to get working on a single box for clusters; we have all sorts of hacks to get it to work on a fairly beefy box. | 18:25 |
| * georgelorch nods | 18:26 | |
| cp16net | i can imagine | 18:26 |
| amcrn | amrith: so if we can't run it, i'm not sure the real value of writing it | 18:26 |
| denis_makogon | we might also speak with infra guys to understand how much of resources we're able to use for our clustering tests | 18:26 |
| grapex | amrith: Agreed. I think we should accept having monstrous tests like clustering in the code even if we don't run them all the time, with the expectation that we can't expect them to always be catching bugs since they won't run frequently. But it would be nice to have them since they will run in fake mode, and we could still run them before the big lettered releases. | 18:26 |
| denis_makogon | for the real one | 18:26 |
| amrith | amcrn, I think the value of having the tests in there is that there will be runs of those tests, even if they are not on a per commit basis | 18:27 |
| amcrn | grapex: but we internally can't even run them on a single box without hacks to nova + diskimage-builder | 18:27 |
| amrith | as grapex says above | 18:27 |
| grapex | amcrn: Still, fake mode by itself will give a lot of feedback and keep smaller bugs from being introduced. | 18:27 |
| amrith | therefore to an earlier point that SlickNik made, there is the question of how these tests should be structured | 18:27 |
| grapex | The big risks left open is communication between Trove and the guest agent- but if people change clustering releated RPC calls they will hopefully expect risk in those areas. | 18:28 |
| amrith | and I think that at this point that thinking is something that you (grapex, amcrn, mat-lowery, SlickNik ...) are in the best position to do. | 18:28 |
| amcrn | grapex: right, i get fake-mode; i'm saying int-tests that aren't runnable. | 18:28 |
| grapex | (as if "hopefully expecting risk" has ever saved anyone in this profession) | 18:28 |
| amcrn | bah, i mean non-fake isn't really runnable on a single box setup*. | 18:28 |
| amrith | amcrn, I think that's fine | 18:28 |
| amcrn | wish irc had an edit button sometimes :) | 18:28 |
| amrith | I'm assuming that with replication and clustering there will be larger test infrastructures that are required for some testing. and that's goodness. | 18:29 |
| amcrn | so the tl;dr here is we're going to go with fake-mode first, which has a dependency on the event simulator fix. beyond this, we need larger deployed test infras to run non-fake int-tests. | 18:29 |
| amcrn | if i'm understanding correctly. | 18:29 |
| SlickNik | amcrn: I agree with that tl;dr | 18:30 |
| denis_makogon | guys, we might take a look at Sahara project, since they are deploying heavy clusters of Hadoop over infra gates | 18:30 |
| amcrn | denis_makogon: i believe they're doing single node, but good point, we should confirm. | 18:30 |
| SlickNik | In the meantime, I can start some conversations with the infra folks to see how we can get creative with testing this on infra. | 18:30 |
| denis_makogon | SlickNik, that would be awesome | 18:31 |
| *** openstackgerrit has quit IRC | 18:32 | |
| *** openstackgerrit has joined #openstack-trove | 18:32 | |
| denis_makogon | and when we'll be ready with fake-mode test we would proceed to real-mode ones, and that'll be a bit tough | 18:32 |
| SlickNik | denis_makogon / amcrn: I think that's the case too — denis_makogon, can you recheck with sergey? | 18:32 |
| denis_makogon | SlickNik, sure | 18:32 |
| SlickNik | thanks | 18:32 |
| denis_makogon | np | 18:32 |
| SlickNik | Alright, I think we have a plan in place for that. | 18:33 |
| SlickNik | zhiyan_: back yet? | 18:33 |
| SlickNik | . | 18:33 |
| SlickNik | #topic Open Discussion | 18:33 |
| amrith | question, what is tl;dr? | 18:33 |
| dougshelley66 | Too long; didn't read | 18:33 |
| SlickNik | too long; didn't read | 18:33 |
| amrith | HA! | 18:34 |
| SlickNik | aka summary | 18:34 |
| amrith | I love these calls | 18:34 |
| amrith | I learn something new | 18:34 |
| cp16net | too long; didn't read | 18:34 |
| cp16net | :-P | 18:35 |
| SlickNik | #endmeeting | 18:35 |
| openstack | Meeting ended Mon Sep 15 18:35:29 2014 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:35 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-09-15-18.00.html | 18:35 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-09-15-18.00.txt | 18:35 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-09-15-18.00.log.html | 18:35 |
| denis_makogon | go ahead and vote with +1 for https://review.openstack.org/#/c/121457/1 and we would proceed to working with specs repo | 18:35 |
| SlickNik | Quick one today. | 18:35 |
| amrith | SlickNik, thanks ... | 18:35 |
| SlickNik | Thanks folks! | 18:35 |
| *** vigneshvar has joined #openstack-trove | 18:39 | |
| *** boden has joined #openstack-trove | 18:39 | |
| *** tkatarki has joined #openstack-trove | 18:41 | |
| *** ramishra has joined #openstack-trove | 18:43 | |
| *** tkatarki has quit IRC | 18:43 | |
| *** ramishra has quit IRC | 18:47 | |
| *** vigneshvar has quit IRC | 18:48 | |
| *** vigneshvar has joined #openstack-trove | 18:48 | |
| *** rushiagr_away is now known as rushiagr | 18:51 | |
| *** mattgriffin has quit IRC | 18:52 | |
| *** tkatarki has joined #openstack-trove | 18:52 | |
| *** tkatarki has quit IRC | 19:04 | |
| *** rushiagr is now known as rushiagr_away | 19:08 | |
| *** amrith is now known as _amrith_ | 19:11 | |
| *** harlowja has quit IRC | 19:14 | |
| *** harlowja_ has joined #openstack-trove | 19:14 | |
| *** prasoon has joined #openstack-trove | 19:17 | |
| *** prasoon has quit IRC | 19:19 | |
| *** prasoon has joined #openstack-trove | 19:33 | |
| *** sgotliv has quit IRC | 19:40 | |
| *** ramishra has joined #openstack-trove | 19:44 | |
| *** ramishra has quit IRC | 19:48 | |
| *** tkatarki has joined #openstack-trove | 19:53 | |
| *** prasoon_ has joined #openstack-trove | 19:53 | |
| *** prasoon has quit IRC | 19:54 | |
| *** sgotliv has joined #openstack-trove | 19:59 | |
| *** tkatarki has quit IRC | 20:01 | |
| *** boden has quit IRC | 20:04 | |
| *** ramashri has quit IRC | 20:07 | |
| *** mattgriffin has joined #openstack-trove | 20:11 | |
| *** ViswaV has quit IRC | 20:30 | |
| *** ramishra has joined #openstack-trove | 20:44 | |
| *** ViswaV has joined #openstack-trove | 20:45 | |
| *** sriram_tesora has joined #openstack-trove | 20:45 | |
| *** prasoon_ has quit IRC | 20:48 | |
| *** harlowja_ is now known as harlowja_away | 20:49 | |
| *** ramishra has quit IRC | 20:49 | |
| *** sgotliv has quit IRC | 20:49 | |
| *** prasoon has joined #openstack-trove | 20:51 | |
| *** prasoon has quit IRC | 20:56 | |
| *** achampio1 has joined #openstack-trove | 20:56 | |
| *** ramashri has joined #openstack-trove | 20:56 | |
| *** juantwo has quit IRC | 20:57 | |
| *** achampion has quit IRC | 20:58 | |
| *** fifieldt_ has joined #openstack-trove | 20:59 | |
| *** eghobo has quit IRC | 21:00 | |
| *** sriram_tesora has quit IRC | 21:00 | |
| *** fifieldt has quit IRC | 21:03 | |
| *** grapex has quit IRC | 21:04 | |
| *** grapex has joined #openstack-trove | 21:05 | |
| *** grapex has quit IRC | 21:09 | |
| *** jdaggett has joined #openstack-trove | 21:11 | |
| *** vigneshvar has quit IRC | 21:14 | |
| *** tomblank has quit IRC | 21:15 | |
| *** robertmyers has quit IRC | 21:16 | |
| *** prasoon has joined #openstack-trove | 21:19 | |
| *** prasoon has quit IRC | 21:24 | |
| *** radez is now known as radez_g0n3 | 21:38 | |
| *** ramishra has joined #openstack-trove | 21:45 | |
| *** denis_makogon has quit IRC | 21:46 | |
| *** flaper87 is now known as flaper87|afk | 21:46 | |
| *** ramishra has quit IRC | 21:50 | |
| openstackgerrit | A change was merged to openstack/trove: Fix NoSuchOptError on Couchbase create https://review.openstack.org/119636 | 21:51 |
| *** jasonb365 has quit IRC | 22:00 | |
| *** georgelorch has quit IRC | 22:05 | |
| *** grapex has joined #openstack-trove | 22:07 | |
| *** grapex_ has joined #openstack-trove | 22:08 | |
| *** grapex has quit IRC | 22:11 | |
| *** prasoon has joined #openstack-trove | 22:21 | |
| *** harlowja_away is now known as harlowja_ | 22:24 | |
| *** prasoon has quit IRC | 22:25 | |
| *** achampio1 has quit IRC | 22:33 | |
| *** achampion has joined #openstack-trove | 22:35 | |
| *** ramishra has joined #openstack-trove | 22:46 | |
| *** juantwo has joined #openstack-trove | 22:50 | |
| *** juantwo has joined #openstack-trove | 22:51 | |
| *** ramishra has quit IRC | 22:51 | |
| *** vkmc has quit IRC | 22:52 | |
| *** mattgriffin has quit IRC | 22:53 | |
| *** vkmc has joined #openstack-trove | 22:55 | |
| *** vkmc has joined #openstack-trove | 22:55 | |
| *** ramishra has joined #openstack-trove | 22:56 | |
| *** grapex_ has quit IRC | 22:56 | |
| *** grapex has joined #openstack-trove | 22:57 | |
| *** grapex has quit IRC | 23:01 | |
| *** kevinconway has quit IRC | 23:01 | |
| *** boden has joined #openstack-trove | 23:06 | |
| *** ramishra has quit IRC | 23:06 | |
| *** boden has quit IRC | 23:10 | |
| *** ramishra has joined #openstack-trove | 23:18 | |
| *** ramishra has quit IRC | 23:19 | |
| *** prasoon has joined #openstack-trove | 23:22 | |
| *** tomblank has joined #openstack-trove | 23:24 | |
| *** prasoon has quit IRC | 23:26 | |
| *** jmontemayor_ has quit IRC | 23:32 | |
| *** ramishra has joined #openstack-trove | 23:50 | |
| *** Riddhi has quit IRC | 23:52 | |
| *** Riddhi has joined #openstack-trove | 23:53 | |
| *** ramishra has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!