| *** yogeshmehra has quit IRC | 00:01 | |
| *** Barker has joined #openstack-trove | 00:22 | |
| *** shakamunyi has joined #openstack-trove | 00:23 | |
| *** amytron has joined #openstack-trove | 00:27 | |
| *** yogeshmehra has joined #openstack-trove | 00:38 | |
| *** ams0 has joined #openstack-trove | 00:38 | |
| *** achampion has quit IRC | 00:41 | |
| *** ams0 has quit IRC | 00:43 | |
| *** achampion has joined #openstack-trove | 00:46 | |
| *** shakamunyi has quit IRC | 00:48 | |
| *** yogeshmehra has quit IRC | 00:56 | |
| *** shakamunyi has joined #openstack-trove | 01:13 | |
| *** erkules has quit IRC | 01:24 | |
| *** amytron has quit IRC | 01:29 | |
| *** amytron has joined #openstack-trove | 01:30 | |
| *** amytron has quit IRC | 01:32 | |
| *** erkules has joined #openstack-trove | 01:37 | |
| *** ams0 has joined #openstack-trove | 01:38 | |
| *** Barker has quit IRC | 01:41 | |
| *** ams0 has quit IRC | 01:43 | |
| *** ViswaV_ has joined #openstack-trove | 02:15 | |
| *** grapex has quit IRC | 02:18 | |
| *** ViswaV has quit IRC | 02:19 | |
| *** coolsvap|afk is now known as coolsvap | 02:35 | |
| *** robertmyers has joined #openstack-trove | 02:46 | |
| *** shakamunyi has quit IRC | 02:57 | |
| *** jasonb365 has joined #openstack-trove | 03:15 | |
| *** ams0 has joined #openstack-trove | 03:39 | |
| *** ams0 has quit IRC | 03:43 | |
| *** robertmyers has quit IRC | 04:06 | |
| *** robertmyers has joined #openstack-trove | 04:06 | |
| *** robertmyers has quit IRC | 04:10 | |
| *** jasonb365 has quit IRC | 04:11 | |
| *** ViswaV_ has quit IRC | 04:18 | |
| *** ViswaV has joined #openstack-trove | 04:19 | |
| *** jasonb365 has joined #openstack-trove | 04:19 | |
| *** jasonb365 has quit IRC | 04:26 | |
| *** achampion has quit IRC | 04:26 | |
| *** achampion has joined #openstack-trove | 04:28 | |
| *** jasonb365 has joined #openstack-trove | 04:30 | |
| *** ViswaV has quit IRC | 04:30 | |
| *** ViswaV has joined #openstack-trove | 04:31 | |
| *** haomaiwang has joined #openstack-trove | 04:31 | |
| *** haomaiwang has quit IRC | 04:32 | |
| *** haomaiwang has joined #openstack-trove | 04:33 | |
| *** jasonb365 has quit IRC | 04:40 | |
| *** achampion has quit IRC | 04:43 | |
| *** SushillKM has joined #openstack-trove | 04:43 | |
| *** jasonb365 has joined #openstack-trove | 04:44 | |
| *** achampion has joined #openstack-trove | 04:45 | |
| *** achampion has quit IRC | 04:55 | |
| *** achampion has joined #openstack-trove | 04:56 | |
| *** jasonb365 has quit IRC | 05:01 | |
| *** ams0 has joined #openstack-trove | 05:08 | |
| *** amytron has joined #openstack-trove | 05:10 | |
| *** sgotliv has joined #openstack-trove | 05:20 | |
| *** haomai___ has joined #openstack-trove | 05:22 | |
| *** haomaiwang has quit IRC | 05:22 | |
| *** ams0 has quit IRC | 05:26 | |
| *** haomaiwang has joined #openstack-trove | 05:26 | |
| *** ams0 has joined #openstack-trove | 05:26 | |
| *** ViswaV_ has joined #openstack-trove | 05:28 | |
| *** ViswaV has quit IRC | 05:28 | |
| *** haomai___ has quit IRC | 05:29 | |
| *** ams0 has quit IRC | 05:30 | |
| *** sgotliv has quit IRC | 05:40 | |
| *** shivam_ has joined #openstack-trove | 05:40 | |
| *** sgotliv has joined #openstack-trove | 05:41 | |
| *** ViswaV_ has quit IRC | 05:43 | |
| *** ViswaV has joined #openstack-trove | 05:43 | |
| *** ViswaV has quit IRC | 05:51 | |
| *** ViswaV has joined #openstack-trove | 05:51 | |
| *** ams0 has joined #openstack-trove | 05:56 | |
| *** yogeshmehra has joined #openstack-trove | 05:57 | |
| *** ViswaV_ has joined #openstack-trove | 06:00 | |
| *** ViswaV has quit IRC | 06:00 | |
| *** SnowDust has joined #openstack-trove | 06:02 | |
| *** ashishsnigam has joined #openstack-trove | 06:05 | |
| *** sgotliv has quit IRC | 06:18 | |
| *** yogeshmehra has quit IRC | 06:24 | |
| *** ViswaV_ has quit IRC | 06:29 | |
| *** ViswaV has joined #openstack-trove | 06:29 | |
| *** yogeshmehra has joined #openstack-trove | 06:32 | |
| *** ams0 has quit IRC | 06:32 | |
| *** global_ has joined #openstack-trove | 06:39 | |
| *** ViswaV_ has joined #openstack-trove | 06:41 | |
| *** ViswaV has quit IRC | 06:44 | |
| openstackgerrit | Shalini khandelwal proposed a change to openstack/trove-integration: Initial support for Vertica Datastore in trove https://review.openstack.org/89804 | 06:49 |
|---|---|---|
| *** ViswaV_ has quit IRC | 07:23 | |
| *** ViswaV has joined #openstack-trove | 07:24 | |
| *** ViswaV_ has joined #openstack-trove | 07:25 | |
| *** ViswaV has quit IRC | 07:25 | |
| *** flaper87|afk is now known as flaper87 | 07:29 | |
| *** ViswaV_ has quit IRC | 07:32 | |
| *** ViswaV has joined #openstack-trove | 07:32 | |
| *** global_ is now known as shalini_ | 07:33 | |
| openstackgerrit | Shalini khandelwal proposed a change to openstack/trove-integration: Added datastore type in test.conf https://review.openstack.org/84964 | 07:35 |
| openstackgerrit | shivam shukla proposed a change to openstack/trove: Added check for not use heat in security group delete https://review.openstack.org/69176 | 07:36 |
| *** ams0 has joined #openstack-trove | 07:38 | |
| *** yogeshmehra has quit IRC | 07:42 | |
| *** openstackgerrit has quit IRC | 07:53 | |
| *** ViswaV has quit IRC | 07:53 | |
| *** ViswaV has joined #openstack-trove | 07:56 | |
| *** ams0 has quit IRC | 08:27 | |
| *** ViswaV has quit IRC | 08:29 | |
| *** ViswaV has joined #openstack-trove | 08:29 | |
| *** ams0 has joined #openstack-trove | 08:34 | |
| *** ViswaV has quit IRC | 08:34 | |
| *** ViswaV has joined #openstack-trove | 08:35 | |
| *** ams0 has quit IRC | 08:39 | |
| *** ams0 has joined #openstack-trove | 08:40 | |
| *** ViswaV has quit IRC | 08:40 | |
| *** ams0 has quit IRC | 08:44 | |
| *** ams0_ has joined #openstack-trove | 08:47 | |
| *** ams0_ has quit IRC | 08:59 | |
| *** ams0 has joined #openstack-trove | 09:00 | |
| *** ams0 has quit IRC | 09:05 | |
| *** ams0 has joined #openstack-trove | 09:05 | |
| *** ams0_ has joined #openstack-trove | 09:07 | |
| *** ams0 has quit IRC | 09:10 | |
| *** ViswaV has joined #openstack-trove | 09:10 | |
| *** sgotliv has joined #openstack-trove | 09:12 | |
| *** ViswaV has quit IRC | 09:22 | |
| *** ViswaV has joined #openstack-trove | 09:23 | |
| *** haomaiwang has quit IRC | 09:25 | |
| *** ViswaV has quit IRC | 09:27 | |
| *** ViswaV has joined #openstack-trove | 09:28 | |
| *** ViswaV has quit IRC | 09:39 | |
| *** ViswaV has joined #openstack-trove | 09:40 | |
| *** ViswaV_ has joined #openstack-trove | 09:46 | |
| *** ViswaV has quit IRC | 09:46 | |
| *** ViswaV_ has quit IRC | 09:56 | |
| *** ViswaV has joined #openstack-trove | 09:57 | |
| *** ViswaV has quit IRC | 10:19 | |
| *** ViswaV has joined #openstack-trove | 10:19 | |
| *** SnowDust has quit IRC | 10:22 | |
| *** ViswaV_ has joined #openstack-trove | 10:27 | |
| *** ViswaV has quit IRC | 10:27 | |
| *** ViswaV_ has quit IRC | 10:29 | |
| *** ViswaV has joined #openstack-trove | 10:30 | |
| *** SnowDust has joined #openstack-trove | 10:36 | |
| *** coolsvap is now known as coolsvap|afk | 10:40 | |
| *** ViswaV has quit IRC | 10:46 | |
| *** ViswaV has joined #openstack-trove | 10:46 | |
| *** ViswaV_ has joined #openstack-trove | 10:47 | |
| *** ViswaV has quit IRC | 10:47 | |
| *** ViswaV has joined #openstack-trove | 11:00 | |
| *** ViswaV_ has quit IRC | 11:00 | |
| *** ams0_ has quit IRC | 11:02 | |
| *** ams0 has joined #openstack-trove | 11:02 | |
| *** ViswaV has quit IRC | 11:04 | |
| *** ViswaV has joined #openstack-trove | 11:06 | |
| *** ams0 has quit IRC | 11:06 | |
| *** ViswaV_ has joined #openstack-trove | 11:17 | |
| *** ViswaV has quit IRC | 11:17 | |
| *** haomaiwa_ has joined #openstack-trove | 11:21 | |
| *** ViswaV_ has quit IRC | 11:33 | |
| *** ViswaV has joined #openstack-trove | 11:36 | |
| *** SushillKM has quit IRC | 11:43 | |
| *** SushillKM has joined #openstack-trove | 11:48 | |
| *** sgotliv has quit IRC | 11:48 | |
| *** SnowDust has quit IRC | 11:56 | |
| *** ViswaV has quit IRC | 11:56 | |
| *** ViswaV has joined #openstack-trove | 12:00 | |
| *** ams0 has joined #openstack-trove | 12:00 | |
| *** ams0_ has joined #openstack-trove | 12:02 | |
| *** ViswaV has quit IRC | 12:03 | |
| *** ams0 has quit IRC | 12:06 | |
| *** ams0_ has quit IRC | 12:15 | |
| *** ams0 has joined #openstack-trove | 12:16 | |
| *** ams0_ has joined #openstack-trove | 12:30 | |
| *** ams0 has quit IRC | 12:33 | |
| *** ViswaV has joined #openstack-trove | 12:39 | |
| *** NehaV has joined #openstack-trove | 12:40 | |
| *** ashishsnigam has quit IRC | 12:43 | |
| *** NehaV has quit IRC | 12:44 | |
| *** achampion has quit IRC | 12:48 | |
| *** ViswaV_ has joined #openstack-trove | 12:49 | |
| *** ViswaV has quit IRC | 12:50 | |
| *** flaper87 has quit IRC | 12:52 | |
| *** NehaV has joined #openstack-trove | 12:52 | |
| *** flaper87|afk has joined #openstack-trove | 12:56 | |
| *** flaper87|afk is now known as flaper87 | 12:56 | |
| *** flaper87 has quit IRC | 12:56 | |
| *** flaper87 has joined #openstack-trove | 12:56 | |
| *** NehaV1 has joined #openstack-trove | 13:02 | |
| *** NehaV has quit IRC | 13:05 | |
| *** sgotliv has joined #openstack-trove | 13:06 | |
| *** ViswaV_ has quit IRC | 13:15 | |
| *** ViswaV has joined #openstack-trove | 13:16 | |
| *** jcru has joined #openstack-trove | 13:23 | |
| *** jcru has joined #openstack-trove | 13:23 | |
| *** radez_g0n3 is now known as radez | 13:24 | |
| *** ViswaV has quit IRC | 13:31 | |
| *** ViswaV_ has joined #openstack-trove | 13:31 | |
| *** openstackgerrit has joined #openstack-trove | 13:36 | |
| *** SushillKM has quit IRC | 13:37 | |
| *** SushillKM has joined #openstack-trove | 13:37 | |
| *** grapex has joined #openstack-trove | 13:37 | |
| *** yidclare has quit IRC | 13:38 | |
| *** yidclare has joined #openstack-trove | 13:39 | |
| *** yidclare has left #openstack-trove | 13:39 | |
| *** Barker has joined #openstack-trove | 13:44 | |
| *** SushillKM has quit IRC | 13:50 | |
| *** openstackgerrit has quit IRC | 13:51 | |
| *** achampion has joined #openstack-trove | 13:52 | |
| *** ams0_ has quit IRC | 13:54 | |
| *** ams0 has joined #openstack-trove | 13:54 | |
| *** grapex has quit IRC | 13:56 | |
| *** grapex has joined #openstack-trove | 13:56 | |
| *** ams0 has quit IRC | 13:57 | |
| *** ams0 has joined #openstack-trove | 13:57 | |
| *** ams0 has quit IRC | 13:58 | |
| *** ams0 has joined #openstack-trove | 13:58 | |
| *** mattgriffin has joined #openstack-trove | 14:07 | |
| *** freyes has joined #openstack-trove | 14:09 | |
| *** kevinconway has joined #openstack-trove | 14:16 | |
| *** rueb7363 has joined #openstack-trove | 14:17 | |
| *** rwsu has joined #openstack-trove | 14:18 | |
| *** ViswaV_ has quit IRC | 14:25 | |
| *** ViswaV has joined #openstack-trove | 14:26 | |
| *** jasonb365 has joined #openstack-trove | 14:37 | |
| *** NehaV1 has quit IRC | 14:37 | |
| *** NehaV has joined #openstack-trove | 14:37 | |
| *** sgotliv has quit IRC | 14:41 | |
| *** dmakogon has joined #openstack-trove | 14:50 | |
| *** denis_makogon has quit IRC | 14:50 | |
| *** dmakogon is now known as denis_makogon | 14:50 | |
| *** dmakogon_ has joined #openstack-trove | 14:51 | |
| denis_makogon | SlickNik, ping | 14:51 |
| *** jasonb365_ has joined #openstack-trove | 14:57 | |
| *** jasonb365 has quit IRC | 14:58 | |
| *** jasonb365_ is now known as jasonb365 | 14:58 | |
| *** NehaV1 has joined #openstack-trove | 15:00 | |
| *** robertmyers has joined #openstack-trove | 15:01 | |
| *** NehaV has quit IRC | 15:03 | |
| *** thedodd has joined #openstack-trove | 15:06 | |
| *** sgotliv has joined #openstack-trove | 15:06 | |
| *** pdmars has joined #openstack-trove | 15:09 | |
| *** robertmyers has quit IRC | 15:15 | |
| *** robertmyers has joined #openstack-trove | 15:16 | |
| *** doddstack has joined #openstack-trove | 15:16 | |
| *** ashishsnigam has joined #openstack-trove | 15:18 | |
| *** thedodd has quit IRC | 15:19 | |
| *** robertmyers has quit IRC | 15:20 | |
| *** jasonb365 has quit IRC | 15:29 | |
| *** haomaiwa_ has quit IRC | 15:29 | |
| *** Barker has quit IRC | 15:29 | |
| *** haomaiwang has joined #openstack-trove | 15:30 | |
| iccha1 | NehaV1: around? | 15:32 |
| *** sgotliv has quit IRC | 15:34 | |
| *** haomaiwang has quit IRC | 15:37 | |
| *** mattgriffin has quit IRC | 15:38 | |
| *** haomaiwa_ has joined #openstack-trove | 15:38 | |
| *** mattgriffin has joined #openstack-trove | 15:40 | |
| *** radez is now known as radez_g0n3 | 15:41 | |
| *** openstackgerrit has joined #openstack-trove | 15:42 | |
| *** haomaiwa_ has quit IRC | 15:43 | |
| *** radez_g0n3 is now known as radez | 15:44 | |
| *** coolsvap|afk is now known as coolsvap | 15:45 | |
| *** amytron has quit IRC | 15:45 | |
| *** shivamshukla_ has joined #openstack-trove | 15:50 | |
| *** demorris has joined #openstack-trove | 15:54 | |
| *** sgotliv has joined #openstack-trove | 15:56 | |
| hub_cap | grapex: wrt the "show stack traces in fake mode", can we not fix it in olso? I dont know what the issue is but the subtext of your commit says something is wrong w oslo | 15:59 |
| *** robertmy_ has joined #openstack-trove | 16:00 | |
| *** Barker has joined #openstack-trove | 16:02 | |
| *** jmontemayor has joined #openstack-trove | 16:04 | |
| *** robertmyers has joined #openstack-trove | 16:04 | |
| *** robertmy_ has quit IRC | 16:05 | |
| *** shivamshukla_ has quit IRC | 16:05 | |
| *** robertmyers has joined #openstack-trove | 16:05 | |
| *** NehaV1 has quit IRC | 16:06 | |
| *** NehaV has joined #openstack-trove | 16:06 | |
| *** jasonb365 has joined #openstack-trove | 16:07 | |
| *** ams0__ has joined #openstack-trove | 16:09 | |
| *** ams0 has quit IRC | 16:09 | |
| *** ViswaV has quit IRC | 16:12 | |
| *** ViswaV has joined #openstack-trove | 16:13 | |
| *** ViswaV_ has joined #openstack-trove | 16:15 | |
| *** sgotliv has quit IRC | 16:16 | |
| *** ViswaV has quit IRC | 16:18 | |
| openstackgerrit | Peter Stachowski proposed a change to openstack/python-troveclient: Added check for empty attributes in trove CLI https://review.openstack.org/92162 | 16:22 |
| *** ams0__ has quit IRC | 16:31 | |
| *** SushillKM has joined #openstack-trove | 16:38 | |
| *** SushillKM has quit IRC | 16:41 | |
| *** SushillKM has joined #openstack-trove | 16:41 | |
| SushillKM | hello cores | 16:42 |
| SushillKM | please review https://review.openstack.org/80315 | 16:42 |
| *** ashishsnigam has quit IRC | 16:44 | |
| *** jmontemayor has quit IRC | 16:50 | |
| *** jasonb365 has quit IRC | 16:51 | |
| *** jasonb365 has joined #openstack-trove | 16:54 | |
| *** SushillKM has quit IRC | 16:58 | |
| esmute | Happy monday everyone. Hope you guys had an awesome weekend. Just to get prepared for the week, can you review these patches? They are short and sweet. https://review.openstack.org/#/c/81379/ https://review.openstack.org/#/c/89717/ https://review.openstack.org/#/c/89719/ https://review.openstack.org/#/c/90462/. Many thanks! | 16:58 |
| *** SushillKM has joined #openstack-trove | 16:59 | |
| *** ramashri has joined #openstack-trove | 17:03 | |
| *** Barker has quit IRC | 17:05 | |
| *** ashishsnigam has joined #openstack-trove | 17:11 | |
| *** Barker has joined #openstack-trove | 17:15 | |
| *** harlowja_away is now known as harlowja | 17:16 | |
| *** shivamshukla_ has joined #openstack-trove | 17:18 | |
| grapex | hub_cap: Yo- about that fix stack traces for fake mode pull request | 17:19 |
| grapex | First off, I swear it worked when I left on Friday, but today I can see it obviously doesn't. | 17:19 |
| grapex | As for fixing it in Oslo, I'm going to put a pull request in there, but the answer is I'm not sure if they'll accept it so I thought I'd patch it for the time being | 17:20 |
| *** openstackgerrit has quit IRC | 17:20 | |
| *** openstackgerrit has joined #openstack-trove | 17:24 | |
| *** sbfox has joined #openstack-trove | 17:28 | |
| *** ashishsnigam has quit IRC | 17:30 | |
| *** ashishsnigam has joined #openstack-trove | 17:30 | |
| *** doddstack has quit IRC | 17:34 | |
| *** grapex has quit IRC | 17:36 | |
| *** grapex has joined #openstack-trove | 17:36 | |
| *** Barker has quit IRC | 17:37 | |
| *** jmontemayor has joined #openstack-trove | 17:37 | |
| *** amcrn has joined #openstack-trove | 17:46 | |
| SushillKM | https://bugs.launchpad.net/trove-integration/+bug/1316080 | 17:49 |
| SushillKM | regarding this bug, i thnk we shud put performance_schema in the ignore_dbs | 17:49 |
| SushillKM | since this is a special schema | 17:50 |
| dougshelley66 | that seems to be a dupe of https://bugs.launchpad.net/trove/+bug/1297000 | 17:50 |
| *** ramashri has quit IRC | 17:51 | |
| denis_makogon | dougshelley66, ++ | 17:52 |
| SushillKM | yeah its similar .... and the solution lies in putting an entry to ignore_dbs | 17:52 |
| *** GeorgeLorch has joined #openstack-trove | 17:52 | |
| denis_makogon | SushillKM, simple answer - yes, of course | 17:53 |
| ashishsnigam | https://etherpad.openstack.org/p/TroveTempestTesting | 17:53 |
| SushillKM | cool | 17:53 |
| SushillKM | thanks | 17:53 |
| *** jasonb365 has quit IRC | 17:54 | |
| ashishsnigam | Hi folks.. We need to change devstack to have a default datastore. | 17:55 |
| ashishsnigam | So that tempest codes for trove can be written. | 17:56 |
| ashishsnigam | Any comments? | 17:56 |
| *** jasonb365 has joined #openstack-trove | 17:56 | |
| denis_makogon | ashishsnigam, you could use tempest tests inside the trove-integration | 17:56 |
| denis_makogon | ashishsnigam, 'cuz we're planning to move datastore/version registration from trove-manage to API | 17:57 |
| SlickNik | ashishsnigam: Sounds reasonable. Check with peterstac to see if he's working on something similar so that you don't step on each other's toes. | 17:57 |
| SlickNik | #startmeeting trove-bp-review | 18:00 |
| openstack | Meeting started Mon May 5 18:00:07 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 |
| denis_makogon | SlickNik, are we still going to move all possible features from -manage to public API ? | 18:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 18:00 |
| openstack | The meeting name has been set to 'trove_bp_review' | 18:00 |
| *** PeterSchwaller_ has joined #openstack-trove | 18:00 | |
| SlickNik | Giving folks a couple of minutes to trickle in to the bp review. | 18:00 |
| cp16net | yea | 18:00 |
| * juice is trickling | 18:01 | |
| SlickNik | Just wanted to mention a point of order. | 18:01 |
| dougshelley66 | denis_makogon, SlickNik - isn't the point to move away from trove-integration to tempest with devstack CI? | 18:01 |
| openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Adds performance_schema to ignore_dbs https://review.openstack.org/92176 | 18:01 |
| denis_makogon | dougshelley66, of course, but it's like temporary solution | 18:01 |
| SlickNik | dougshelley66: Yes, the plan is to move away from trove-integration. | 18:02 |
| juice | slicknik - what was your point of order | 18:02 |
| SlickNik | dougshelley66: Let's chat after the bp meeting. | 18:02 |
| dougshelley66 | SlickNik certainly | 18:02 |
| denis_makogon | mattgriffin, it's your turn, i guess | 18:03 |
| mattgriffin | denis_makogon, hello! | 18:03 |
| denis_makogon | mattgriffin, hello, Mat =) | 18:03 |
| mattgriffin | so i've proposed 3 BPs | 18:03 |
| mattgriffin | 1. https://blueprints.launchpad.net/trove/+spec/support-percona-xtrabackup-2.2 | 18:03 |
| denis_makogon | #link https://blueprints.launchpad.net/trove/+spec/support-percona-xtrabackup-2.2 | 18:04 |
| mattgriffin | there was a question last week about backwards compability | 18:04 |
| mattgriffin | of backups | 18:04 |
| SlickNik | mattgriffin: hang on one sec. | 18:04 |
| mattgriffin | SlickNik, ack | 18:04 |
| *** grapex_ has joined #openstack-trove | 18:05 | |
| *** NehaV1 has joined #openstack-trove | 18:05 | |
| SlickNik | I don't think cores are around yet. | 18:05 |
| grapex_ | Sorry, got disconnected | 18:05 |
| SlickNik | grapex just got here. | 18:05 |
| SlickNik | np. | 18:05 |
| SlickNik | So just a point of order before we start. | 18:05 |
| *** demorris has quit IRC | 18:05 | |
| SlickNik | To be fair to all parties involved — if you submit more than one unrelated bp per review meeting — only the first one will be talked about as part of the queue. | 18:05 |
| *** michael-yu has joined #openstack-trove | 18:06 | |
| openstackgerrit | Sushil Kumar proposed a change to openstack/trove: Resolves volume resize issue https://review.openstack.org/80315 | 18:06 |
| SlickNik | We can talk about the other ones if we have time. | 18:06 |
| *** demorris has joined #openstack-trove | 18:06 | |
| grapex_ | SlickNik: I like it | 18:06 |
| SlickNik | Okay, with that out of the way. | 18:07 |
| SlickNik | #topic Percona support | 18:07 |
| *** grapex has quit IRC | 18:07 | |
| SlickNik | mattgriffin: | 18:07 |
| mattgriffin | SlickNik, hi :) | 18:07 |
| SlickNik | go ahead | 18:07 |
| *** NehaV has quit IRC | 18:07 | |
| mattgriffin | hi all. i created some Summit proposals but was told that the best route was to submit BPs | 18:08 |
| *** NehaV has joined #openstack-trove | 18:08 | |
| SlickNik | So a couple of things regarding the bps. | 18:08 |
| mattgriffin | ok | 18:08 |
| *** NehaV has left #openstack-trove | 18:09 | |
| *** NehaV has joined #openstack-trove | 18:09 | |
| SlickNik | It's still early days for supporting xtradbcluster (https://blueprints.launchpad.net/trove/+spec/support-pxc-5.5-and-5.6) since we don't have a clustering API in trove yet. | 18:09 |
| mattgriffin | SlickNik, ack | 18:10 |
| denis_makogon | SlickNik, agreed, looks like another package for current percona support | 18:10 |
| konetzed | SlickNik: *fingers crossed* for clustering :D | 18:10 |
| mattgriffin | SlickNik, just looking for ways that percona can plug into the team and help ... perhaps that API is a way forward during Juno | 18:10 |
| denis_makogon | mattgriffin, we can ask them to join amcrn's design sessions | 18:11 |
| SlickNik | We have a summit session on the clustering API design, and plan to have it nailed by then. | 18:11 |
| mattgriffin | cool | 18:11 |
| SlickNik | (amcrn is doing a great job of leading that) | 18:11 |
| SlickNik | As for support for percona-server. THe plan is to support it through the mysql-manager itself. | 18:12 |
| *** NehaV1 has quit IRC | 18:12 | |
| denis_makogon | SlickNik, sound very reasonable | 18:12 |
| SlickNik | There's a session on unifying guest-agents that will discuss that and hub_cap's driving that one. | 18:12 |
| mattgriffin | SlickNik, any work needed to get PS 5.6 into the mix for support? | 18:12 |
| cp16net | yeah it should be similar enough not to have another impl for percona | 18:12 |
| mattgriffin | SlickNik, ok... then chat about that in ATL | 18:13 |
| mattgriffin | ? | 18:13 |
| SlickNik | mattgriffin: Should be fairly straightforward; don't think there's an issue there. | 18:13 |
| SlickNik | cp16net: Yes, the idea is to have the mysql guest support percona as well. | 18:14 |
| *** yogeshmehra has joined #openstack-trove | 18:14 | |
| SlickNik | mattgriffin: So for xtrabackup, was there a backward compat issue? | 18:15 |
| mattgriffin | SlickNik, i checked into it. no issue but GeorgeLorch is here if there are additional questions | 18:15 |
| *** yogeshmehra has quit IRC | 18:16 | |
| *** yogeshme_ has joined #openstack-trove | 18:16 | |
| *** PeterSchwaller_ is now known as PeterSchwaller | 18:16 | |
| SlickNik | mattgriffin: Okay, sounds good. In that case, it's probably okay for us to move to xtrabackup- 2.2 for backup/restore. | 18:17 |
| mattgriffin | SlickNik, with xtrabackup, there are likely features that trove isn't using that would be helpful to users. | 18:17 |
| mattgriffin | SlickNik, cool | 18:17 |
| SlickNik | Will follow up with GeorgeLorch about new features. | 18:17 |
| mattgriffin | SlickNik, great! | 18:17 |
| *** SushillKM has quit IRC | 18:17 | |
| SlickNik | Thanks mattgriffin. | 18:17 |
| mattgriffin | SlickNik, thank you | 18:17 |
| cp16net | i see last time juice asked about direct integration with XtraBackup to s3/swift | 18:18 |
| mattgriffin | cp16net, that's unfortunately not going to be available until late 2014 in 3.0 | 18:18 |
| SlickNik | Okay, let's move on. | 18:19 |
| SlickNik | #topic Instance database log manipulations | 18:19 |
| denis_makogon | #link https://blueprints.launchpad.net/trove/+spec/dbinstance-log | 18:19 |
| denis_makogon | last time this one appeared, were asked several questions | 18:19 |
| denis_makogon | you can find them at the wiki page | 18:20 |
| *** yogeshme_ has quit IRC | 18:20 | |
| *** yogeshmehra has joined #openstack-trove | 18:20 | |
| *** ramashri has joined #openstack-trove | 18:20 | |
| denis_makogon | questions are related to log rotation | 18:20 |
| denis_makogon | and sys log | 18:20 |
| denis_makogon | *sys log service | 18:20 |
| *** doddstack has joined #openstack-trove | 18:21 | |
| denis_makogon | short answer to rotation - it can be done within scheduled task | 18:21 |
| *** ranjitha has joined #openstack-trove | 18:21 | |
| konetzed | why cant you use logrotate? | 18:21 |
| denis_makogon | konetzed, we can, that's what i wrote | 18:21 |
| konetzed | denis_makogon: sorry just saw that link, must have glanced over it first time | 18:22 |
| denis_makogon | konetzed, but to keep full log history we need to push log to Swift | 18:22 |
| denis_makogon | once rotation time comes, guest pushes database log into the Swift | 18:22 |
| denis_makogon | but this task can be accomplished once scheduled task appears | 18:23 |
| denis_makogon | we had design session for scheduled tasks | 18:23 |
| denis_makogon | *have | 18:23 |
| *** shivamshukla_ has quit IRC | 18:23 | |
| esmute | denis_makogon: How can you configure the the DB log config? rotation, file size and all the other logging goodness | 18:23 |
| denis_makogon | esmute, everything is written at wiki page | 18:24 |
| denis_makogon | esmute, size, schedule, file | 18:24 |
| denis_makogon | etc | 18:24 |
| esmute | denis_makogon: ok.. i just read the API portion | 18:24 |
| denis_makogon | esmute, cool =) | 18:25 |
| amcrn | denis_makogon: your wiki mentions the template, it doesn't mention how it's actually rendered with dynamic values | 18:25 |
| denis_makogon | amcrn, https://wiki.openstack.org/wiki/Trove/DBInstanceLogOperation#Log_files_rotation | 18:25 |
| denis_makogon | {{ how_often }} | 18:26 |
| amcrn | where does "actual rotation count" come from | 18:26 |
| denis_makogon | amcrn, from guest config | 18:26 |
| juice | denis_makogon: I still feel that either this is a generic component that could be added to dbaas as something installable or a 3rd party component - i.e. why are we building this? | 18:26 |
| SlickNik | denis_makogon: Your wiki doesn't mention any API endpoints for this. What's the endpoint that I need to actually hit? | 18:26 |
| denis_makogon | juice, from PaaS level perspective, you cannot access the VM directly, only through some API | 18:27 |
| amcrn | denis_makogon: https://wiki.openstack.org/wiki/Trove/DBInstanceLogOperation#Configuration is missing your guest config | 18:27 |
| denis_makogon | amcrn, that's true | 18:27 |
| juice | denis_makogon: I understand that but log shipping is a solved issue | 18:27 |
| denis_makogon | amcrn, thanks for catch | 18:27 |
| vipul | denis_makogon: We should forget about iteration 2 for now.. just focus on iteration 1. I'm missing how the user would save off a log file.. and how the deployer would choose which files can be saved.. and what timestamp a saved log file represents | 18:27 |
| denis_makogon | juice, we cannot use syslog server | 18:28 |
| denis_makogon | vipul, log files will be shipped to Swift container | 18:28 |
| cp16net | denis_makogon: can how do you add log files that are not specified in this "default" setup of log files? | 18:28 |
| vipul | denis_makogon: i understand that.. i feel like this can be simplified sooo much for v1 | 18:29 |
| denis_makogon | cp16net, i'm specifying at database logs only | 18:29 |
| denis_makogon | vipul, how ? | 18:29 |
| SlickNik | Okay, so I don't want this to turn into a design session. | 18:29 |
| denis_makogon | SlickNik, agreed | 18:29 |
| kevinconway | SlickNik: what if.... | 18:29 |
| denis_makogon | lets talk about meeting | 18:29 |
| denis_makogon | #action Fix wiki page (conf. section) | 18:30 |
| vipul | 1. we need an endpoint that let's a user 'save' a file.. given the name of a file.. 2. we need an endpiont that lists all saved files, and their timestamps.. Done | 18:30 |
| SlickNik | So I think we all agree that the BP needs some better definition around phase 1 of this. | 18:30 |
| SlickNik | denis_makogon: thanks. | 18:30 |
| denis_makogon | vipul, that's what i proposed as v1 | 18:30 |
| esmute | denis_makogon: so only the logs that are created/saved will be the ones accessible? | 18:31 |
| *** radez is now known as radez_g0n3 | 18:31 | |
| denis_makogon | esmute, yes | 18:31 |
| denis_makogon | esmute, only database logs | 18:31 |
| SlickNik | Let's defer this until next week, when denis_makogon updates the page with this info. | 18:31 |
| denis_makogon | let's move forward | 18:31 |
| esmute | so if i want daily logs (or hourly) i would have to add a cronjob/script to run every hour and invoke the dblog-create api? | 18:31 |
| SlickNik | thanks #denis_makogon | 18:31 |
| denis_makogon | SlickNik, agreed | 18:31 |
| denis_makogon | esmute, lets talk after | 18:32 |
| SlickNik | #topic Update database instance name | 18:32 |
| SlickNik | nehav around? | 18:32 |
| NehaV | hey | 18:32 |
| NehaV | its a bp to allow users to rename db instance | 18:32 |
| denis_makogon | one question, what's the justification of it ? | 18:32 |
| denis_makogon | what it stands for ? | 18:32 |
| NehaV | a change to the existing update instance call | 18:32 |
| NehaV | Users have requested the ability to rename instances after they are created | 18:33 |
| denis_makogon | NehaV, nova allows it ? | 18:33 |
| NehaV | yes | 18:33 |
| iccha1 | yes | 18:33 |
| denis_makogon | NehaV, why does DNS is not enough? | 18:33 |
| iccha1 | all projects allow name changes | 18:33 |
| iccha1 | its id changes which are not allowed | 18:33 |
| cp16net | i think a good example is i created a dev-instance and i'm now using it in prod so i'd like to change the name to prod-instance | 18:33 |
| NehaV | a user should be allowed to change the name of their instance | 18:34 |
| denis_makogon | cp16net, thanks for an example | 18:34 |
| grapex_ | Sounds pretty simple. Even when you're just making test instances, it can be painful sometimes when you can't rename them | 18:34 |
| amcrn | makes sense to me | 18:34 |
| denis_makogon | NehaV, does heat allows that ? | 18:34 |
| SlickNik | one question. | 18:35 |
| cp16net | yeah and nothing should be tied directly to the name any way | 18:35 |
| SlickNik | NehaV: So the bp mentions it being a PUT, but I think it's more likely a PATCH since you're not including all the json that specifies the resource. | 18:35 |
| NehaV | i m not sure about heat | 18:35 |
| denis_makogon | SlickNik, ++ | 18:35 |
| iccha1 | cp16net: +1 | 18:35 |
| cp16net | tru | 18:35 |
| vipul | is the plan to propogate the name change to the nova instance? | 18:35 |
| denis_makogon | NehaV, please take a look at heat, since we're planning to move at it, as soon as possible | 18:35 |
| iccha1 | SlickNik: then the config call wpould also have to change ? | 18:35 |
| *** saurabhs has joined #openstack-trove | 18:36 | |
| vipul | the one issue i see is the nova instance name is usually also the hostname of the VM | 18:36 |
| denis_makogon | vipul, ++ | 18:36 |
| vipul | so we are indirectly relying on it | 18:36 |
| esmute | vipul: Does renaming a nova instance, rename the hostname? | 18:36 |
| vipul | esmute: good question, i'm not sure | 18:36 |
| denis_makogon | esmute, yes | 18:36 |
| iccha1 | do u propagate instance name to nova server name? | 18:37 |
| denis_makogon | iccha1, yes, we're doing it | 18:37 |
| vipul | iccha1: we do.. unless DNS is enabled | 18:37 |
| denis_makogon | vipul, that's also true | 18:37 |
| grapex_ | Maybe I'm missing something- why is it important to rename the Nova instance name? | 18:37 |
| cp16net | yeah then its the dns name | 18:37 |
| grapex_ | The issue is though DNS names are configured via a strategy | 18:38 |
| grapex_ | At Rax they are these weird guid looking things that don't have to be the same as anything else | 18:38 |
| vipul | grapex_: if you don't use dns in Trove, the instance name = nova instance name | 18:38 |
| denis_makogon | also, heat changes host name "as it want's", by adding huge hash to resource name | 18:38 |
| grapex_ | vipul: Trust, but I guess I don't get why the nova and Trove names would have to be the same. | 18:39 |
| vipul | well the only issue i see is the hostname of the VM will not be tied to the Trove instance name | 18:39 |
| denis_makogon | NehaV, are you planning to rename instance hostname, or just Trove instance name ? | 18:39 |
| denis_makogon | vipul, same for me | 18:40 |
| NehaV | trove instance name | 18:40 |
| vipul | so for example.. we use that hostname + uuid + other stuff today to generate a 'salt key' | 18:40 |
| denis_makogon | in this case, it's valid | 18:40 |
| vipul | we can probably work around that.. | 18:40 |
| vipul | but just raising it as a potential issue | 18:40 |
| denis_makogon | i guess we can changes Trove instances name, since it's not chained with compute instance name | 18:41 |
| grapex_ | vipul: I see. | 18:41 |
| saurabhs | like nova when you change name of the instance, nothing on the instance is affected, updated name is visible in nova list, but if you do 'hostname' on the instance you will continue to see the old name. | 18:41 |
| iccha1 | we should be relying only on ids, and never on names. | 18:41 |
| saurabhs | we should do something similar change it it in trove database only and not change anything in nova | 18:41 |
| denis_makogon | iccha1, we just doing it | 18:41 |
| denis_makogon | saurabhs, that's valid | 18:42 |
| esmute | saurabhs: so nova rename does not update the hostname? | 18:42 |
| kevinconway | iccha1: but ba3e352a-577f-4a40-9646-aaca5acf86f8 is so hard to pronounce | 18:42 |
| saurabhs | it updates it only in nova list I guesst. on instance for sure 'hostname' command returns you old name of the instance | 18:42 |
| denis_makogon | kevinconway, at least you can try ;) | 18:42 |
| iccha1 | shakespeare said whats in a name kevinconway | 18:43 |
| SlickNik | Okay, I think we know pretty well what this entails. | 18:43 |
| SlickNik | Let's get a quick vote: | 18:43 |
| vipul | yea we should be able to work around it.. | 18:43 |
| esmute | +1 update both trove and nova instance | 18:43 |
| denis_makogon | +1 only for Trove instances | 18:44 |
| openstackgerrit | Anna Shen proposed a change to openstack/trove-integration: Trove guestagent should not use sample conf https://review.openstack.org/88478 | 18:44 |
| openstackgerrit | Anna Shen proposed a change to openstack/trove-integration: Add neutron switch for int tests https://review.openstack.org/87856 | 18:44 |
| openstackgerrit | Anna Shen proposed a change to openstack/trove-integration: Add support for a neutron-based install https://review.openstack.org/78123 | 18:44 |
| cp16net | #vote yes | 18:44 |
| annashen | sorry guys | 18:44 |
| SlickNik | #vote update instance name? yes-only-trove, yes-trove-and-nova, no | 18:44 |
| *** jasonb365 has quit IRC | 18:44 | |
| SlickNik | #startvote update instance name? yes-only-trove, yes-trove-and-nova, no | 18:45 |
| openstack | Begin voting on: update instance name? Valid vote options are yes-only-trove, yes-trove-and-nova, no. | 18:45 |
| denis_makogon | #vote yes-only-trove | 18:45 |
| openstack | Vote using '#vote OPTION'. Only your last vote counts. | 18:45 |
| grapex_ | I'd like to make it an option | 18:45 |
| esmute | #vote yes-trove-and-nova | 18:45 |
| NehaV | #vote yes-only-trove | 18:45 |
| grapex_ | #vote yes-only-trove | 18:45 |
| saurabhs | #vote yes-only-trove | 18:45 |
| robertmyers | #vote yes-only-trove | 18:45 |
| amcrn | #vote yes-only-trove | 18:45 |
| iccha1 | #vote yes-only-trove | 18:45 |
| grapex_ | I want other deployers to be able to change the Nova name and do other things if they need to | 18:45 |
| vipul | #vote yes-only-trove | 18:45 |
| grapex_ | Maybe we could add a function name to the configs that gets called when the name is changed, and by default its None | 18:45 |
| SlickNik | I'm with grapex_ on this one. | 18:45 |
| cp16net | #vote yes-only-trove | 18:45 |
| denis_makogon | #vote yes-only-trove | 18:46 |
| grapex_ | SlickNik: The options didn't entail that | 18:46 |
| grapex_ | but I'm sure it could be a fast-follow | 18:46 |
| SlickNik | sure | 18:46 |
| *** radez_g0n3 is now known as radez | 18:46 | |
| SlickNik | #endvote | 18:46 |
| openstack | Voted on "update instance name?" Results are | 18:46 |
| kevinconway | grapex_: you mean like an event callback? | 18:46 |
| openstack | yes-trove-and-nova (1): esmute | 18:46 |
| openstack | yes-only-trove (9): iccha1, robertmyers, saurabhs, denis_makogon, amcrn, cp16net, grapex_, NehaV, vipul | 18:46 |
| esmute | landslide | 18:46 |
| iccha1 | SlickNik: i would still like your put vs patch concern addressed though | 18:47 |
| SlickNik | iccha1: same here | 18:47 |
| SlickNik | IIRC, config groups used patch, but I'll have to check the code. | 18:47 |
| cp16net | yeah but i think it should be ok to start on the small change | 18:47 |
| SlickNik | cp16net might have a better idea. | 18:48 |
| SlickNik | Let's take that offline and work it out. | 18:48 |
| cp16net | yeah we have added patch for that | 18:48 |
| cp16net | for attach/detach | 18:48 |
| iccha1 | cp16net: configurations uses patch? | 18:48 |
| NehaV | the current update instance call has put for updating a config group to an instance | 18:48 |
| iccha1 | cool the docs are updated then | 18:48 |
| cp16net | iccha1: yup | 18:48 |
| iccha1 | *outdated | 18:48 |
| * cp16net thinks i'm up to date :-P | 18:49 | |
| iccha1 | https://wiki.openstack.org/wiki/Trove/Configurations#Update_an_Instance_.28PUT.29 | 18:49 |
| SlickNik | grapex_: I'm okay with having the trove-only option go in. We can fast follow with a config and nova rename if needed. :) | 18:49 |
| SlickNik | Let's move on. | 18:49 |
| SlickNik | #topic Pluggable conductor manager | 18:49 |
| SlickNik | boden? | 18:49 |
| denis_makogon | seems he's out | 18:50 |
| SlickNik | I'm not sure what his IRC nick is. | 18:50 |
| SlickNik | #topic Allow configs to be rendered based on datastore version | 18:50 |
| denis_makogon | SlickNik, boden | 18:50 |
| SlickNik | cp16net: all yours | 18:51 |
| denis_makogon | ++ for this BP | 18:51 |
| SlickNik | I think this really is a bug. | 18:51 |
| cp16net | yeah this needs to change a bit | 18:51 |
| cp16net | grapex_: passed this off to me and i think i am a little behind... :-p | 18:51 |
| cp16net | sorry | 18:51 |
| vipul | wasn't there talk of collapsing the version + datastore into a single field | 18:52 |
| vipul | if that happens, is this a solved problem | 18:52 |
| grapex_ | vipul: A single field in the class named "Datastore"? | 18:52 |
| cp16net | well the templates we have are stored in /tempatles/{manager}/ | 18:52 |
| cp16net | right now and we cant have like multiple tempaltes for different versions... | 18:52 |
| vipul | grapex_: yes.. datastore_name may imply 'datastore + version' | 18:52 |
| cp16net | like mysql 5.1 or 5.5 | 18:53 |
| grapex_ | vipul: Well today the template is only picked using the datastore's manager | 18:53 |
| cp16net | this makes the path easier to follow and make configurations for each version | 18:53 |
| denis_makogon | as for me, we should have root template /template/{datastore}/root.config | 18:53 |
| cp16net | but still defaulting back to the manager if the others are not found | 18:53 |
| denis_makogon | and other templates are extending the root.config | 18:53 |
| robertmyers | more options are better | 18:54 |
| cp16net | so after explaining that part... are there any questions about this ? | 18:54 |
| SlickNik | cp16net / denis_makogon: I think both of you are saying basically the same thing. | 18:54 |
| grapex_ | robermyers: ++ | 18:54 |
| vipul | cp16net: so you think it's better to derive the tempalte from datastore_version and if it does collapse into a single record.. then this would just follow? | 18:54 |
| grapex_ | I'm sure we'll change this and then next week a deployer will wish we'd added something else | 18:54 |
| denis_makogon | SlickNik, cp16net if that's so, than cool =) | 18:54 |
| grapex_ | vipul: the point here is to get the template from both the datastore name and the version | 18:55 |
| denis_makogon | grapex_, ++ | 18:55 |
| grapex_ | if it becomes a single record, that's ok, because the blueprint currently tries multiple paths | 18:55 |
| grapex_ | the first is something like /template/{datastore_name}/{datastore_version} | 18:55 |
| cp16net | vipul: yeah this makes the deployer able to make the templates in a more specific place | 18:55 |
| vipul | grapex_: yep, i got that.. i'm all for it.. just might become moot if someone does implement the single record solution | 18:55 |
| amcrn | i generally agree with the proposal, but have a few minor nits (but that can be discussed at a later time in a smaller setting) | 18:56 |
| robertmyers | we can always change the paths we check | 18:56 |
| grapex_ | vipul: Sure | 18:56 |
| denis_makogon | robertmyers, agreed | 18:56 |
| grapex_ | but that's relying on a pretty huge refactor to the datastore stuff | 18:56 |
| cp16net | yeah thats just a list of configuration paths | 18:56 |
| grapex_ | I am a horrible cynical man but I don't know if I believe that will happen | 18:56 |
| grapex_ | Maybe we can have a hack-a-thon at the summit and change datastores. :) | 18:56 |
| vipul | grapex_: fair enough it probably won't anytime soon | 18:57 |
| cp16net | amcrn: we can chat later and make sure we are on the same page | 18:57 |
| amcrn | cp16net: sounds good | 18:57 |
| cp16net | :) | 18:57 |
| grapex_ | vipul: Cool. Not kidding about the hackathon btw | 18:57 |
| SlickNik | So I'm good with this one as well. | 18:57 |
| cp16net | i think its straight forward | 18:57 |
| robertmyers | +1 | 18:58 |
| SlickNik | #startvote Allow configs to be rendered based on datastore version? yes, no | 18:58 |
| openstack | Begin voting on: Allow configs to be rendered based on datastore version? Valid vote options are yes, no. | 18:58 |
| openstack | Vote using '#vote OPTION'. Only your last vote counts. | 18:58 |
| robertmyers | #vote yes | 18:58 |
| cp16net | #vote yes | 18:58 |
| SlickNik | #vote yes | 18:58 |
| amcrn | #vote yes | 18:58 |
| vipul | #vote yes | 18:58 |
| esmute | #vote yes | 18:58 |
| denis_makogon | #vote yes | 18:58 |
| grapex_ | #vote yes | 18:58 |
| NehaV | #vote yes | 18:58 |
| SlickNik | #endvote | 18:58 |
| openstack | Voted on "Allow configs to be rendered based on datastore version?" Results are | 18:58 |
| openstack | yes (9): SlickNik, robertmyers, denis_makogon, amcrn, cp16net, esmute, NehaV, vipul, grapex_ | 18:58 |
| SlickNik | Okay, go for it cp16net | 18:59 |
| SlickNik | And that's all we have time for this week. | 18:59 |
| SlickNik | #endmeeting | 18:59 |
| openstack | Meeting ended Mon May 5 18:59:37 2014 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:59 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-05-05-18.00.html | 18:59 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-05-05-18.00.txt | 18:59 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/trove_bp_review/2014/trove_bp_review.2014-05-05-18.00.log.html | 18:59 |
| cp16net | thanks ev1 | 18:59 |
| SlickNik | Thanks all! | 18:59 |
| SlickNik | I'll update the BP wiki page with notes / and move the unfinished business to next week | 19:00 |
| denis_makogon | if anyone want to discuss dblog API, i'm here =) | 19:00 |
| denis_makogon | btw, next week is summit | 19:00 |
| SlickNik | next meeting* | 19:00 |
| dougshelley66 | SlickNik will there be IRC meetings next week? | 19:02 |
| ashishsnigam | SlickNik, denis_makogon: Can we now have a discussion on tempest? | 19:02 |
| denis_makogon | ashishsnigam, of course | 19:03 |
| ashishsnigam | peterstac: u there? | 19:03 |
| denis_makogon | ashishsnigam, as SlickNik said, you can add default datastore inside the devstack | 19:03 |
| peterstac | ashishsnigam: hi | 19:03 |
| SlickNik | dougshelley66: Probably not, since most people will be at the summit. Will update everyone on Wednesday with concrete plans. | 19:03 |
| denis_makogon | ashishsnigam, future plans should not bother you, if it blocks you from significant work (tests) | 19:04 |
| dougshelley66 | SlickNik great thx | 19:04 |
| SlickNik | ashishsnigam: Sure thing. | 19:05 |
| SlickNik | ashishsnigam: So if you are writing datastore tests, and need a default datastore set, I would update the devstack trove install in lib/trove to do that. | 19:07 |
| ashishsnigam | SlickNik: Thanks a lot | 19:08 |
| ashishsnigam | I think only after that we can proceed with most of trove tests. | 19:09 |
| SlickNik | ashishsnigam / peterstac: I'm really close to getting a ci-infra job set up for building a guest-image that devstack will be able to use as well. Hope to have it done by mid-end of this week. | 19:09 |
| peterstac | SlickNik: Sounds good | 19:09 |
| *** grapex has joined #openstack-trove | 19:09 | |
| SlickNik | ashishsnigam: yes, as peterstac earlier pointed out you're going to need to be able to have a way of actually spinning up a guest for most other tests. | 19:10 |
| ashishsnigam | SlickNik: Cool | 19:10 |
| ashishsnigam | SlickNik: Ok | 19:10 |
| denis_makogon | vipul, ping | 19:11 |
| vipul | denis_makogon: what's up | 19:11 |
| denis_makogon | vipul, i update wiki page, have you got 2-3 mins to review changes ? | 19:12 |
| vipul | db logs? | 19:12 |
| denis_makogon | vipul, https://wiki.openstack.org/wiki/Trove/DBInstanceLogOperation | 19:12 |
| denis_makogon | vipul, yes | 19:12 |
| vipul | denis_makogon: what are the endponts you are introducing? | 19:12 |
| denis_makogon | vipul, create, list with filters | 19:13 |
| *** grapex_ has quit IRC | 19:13 | |
| denis_makogon | vipul, create, list, show | 19:13 |
| vipul | what is the exact URL? | 19:14 |
| *** jasonb365 has joined #openstack-trove | 19:14 | |
| *** yogeshmehra has quit IRC | 19:14 | |
| denis_makogon | vipul, /{tenant_id}/dblogs | 19:15 |
| vipul | so why would you choose to make it a high-level resource vs. a subresource of an instance | 19:16 |
| ashishsnigam | SlickNik, peterstac: https://etherpad.openstack.org/p/TroveTempestTesting | 19:16 |
| denis_makogon | vipul, this one for listing the logs | 19:16 |
| ashishsnigam | Are we following this BP? | 19:16 |
| denis_makogon | vipul, /{tenant_id}/instances/{id}/dblog/{db_log}/create - for creating the log | 19:17 |
| vipul | denis_makogon: to me, it seems like logs are owned by instances, so all operations should be as subresources of instances | 19:17 |
| vipul | let me gist something up really quick | 19:17 |
| denis_makogon | vipul, datastore owns the log description, instance owns the log files | 19:17 |
| peterstac | ashishsnigam, SlickNik: Yes, I believe so - but it might be a bit out of date | 19:18 |
| peterstac | ashishsnigam, SlickNik: We're also using https://etherpad.openstack.org/p/trove-tempest-items to keep track (I think you know that though ;) ) | 19:20 |
| SlickNik | ashishsnigam: These notes from the latest midcycle meetup are somewhat more recent: https://etherpad.openstack.org/p/trove-2014-meetup-tempest | 19:21 |
| vipul | denis_makogon: https://gist.github.com/vipulsabhaya/c5191159612179b685a3 | 19:21 |
| *** radez is now known as radez_g0n3 | 19:22 | |
| ashishsnigam | Thanks peterstac, SlickNik | 19:22 |
| denis_makogon | vipul, that's what i meant https://gist.github.com/denismakogon/e92b83f745abb0939190 | 19:23 |
| denis_makogon | vipul, we're not tracking the db logs at the backend | 19:24 |
| *** coolsvap is now known as coolsvap|afk | 19:24 | |
| denis_makogon | vipul, we're tracking the available logs, that were written by the deployer at conf file | 19:24 |
| vipul | denis_makogon: sure ok, so even if we're not saving dblogs are records.. how does that impact the API i posted | 19:25 |
| denis_makogon | vipul, https://gist.github.com/vipulsabhaya/c5191159612179b685a3#file-gistfile1-txt-L29 has another look | 19:26 |
| vipul | BTW, these are important points to call out in a BP as well.. | 19:26 |
| vipul | let me edit the swift URL | 19:26 |
| denis_makogon | vipul, Database section is empty - it means that logs are not tracked | 19:27 |
| denis_makogon | vipul, https://wiki.openstack.org/wiki/Trove/DBInstanceLogOperation#Get_the_list_of_all_available_database_logging_files_for_all_datastore_versions._HTTP_method_GET this is how it looks like | 19:27 |
| vipul | denis_makogon: i think it would be more meaningful for others to say.. database changes are not required because dblogs are not tracked as a resource or something like that | 19:29 |
| denis_makogon | vipul, ok | 19:29 |
| denis_makogon | vipul, dblog itself is a sub-resource of the datastore, but actual log file is a sub-resource of the instance of course | 19:29 |
| *** ranjitha has quit IRC | 19:29 | |
| *** sbfox has quit IRC | 19:30 | |
| vipul | ok so.. why is it necessary to provide a list operation where you get all datastores + all logs files in a single call | 19:30 |
| openstackgerrit | Peter Stachowski proposed a change to openstack/python-troveclient: Fix trove CLI help UUID references https://review.openstack.org/92189 | 19:30 |
| *** aratim has joined #openstack-trove | 19:30 | |
| vipul | as opposed to, having /datastores/{datastore}/logs as the only way to available logs, for a single datastore | 19:30 |
| denis_makogon | vipul, when i wrote that i thought that it will be very informal to user | 19:31 |
| ashishsnigam | SlickNik peterstac: latest midcycle notes are really good points for me. | 19:32 |
| vipul | seems unnecessary to me.. | 19:32 |
| denis_makogon | vipul, but, i guess, it can be eliminated in favor of show operation | 19:32 |
| denis_makogon | vipul, you're right | 19:32 |
| vipul | ok denis_makogon -- now moving on to save operation | 19:32 |
| denis_makogon | vipul, i'm gonna drop that section | 19:32 |
| vipul | that route... i don't quite understand it | 19:33 |
| peterstac | ashishsnigam: Good stuff - if you need any clarifications, you know where to find SlickNik ;) | 19:33 |
| ashishsnigam | :) | 19:33 |
| vipul | seems like all you need is /tenant_id/instances/{id}/logs and a request payload that has the name(s) of the log files you wanna save | 19:33 |
| vipul | denis_makogon why is the filename, instance_id redundant in the body + url | 19:34 |
| peterstac | ashishsnigam: Will you be at the summit next week in ATL? | 19:34 |
| denis_makogon | vipul, seems valid | 19:35 |
| denis_makogon | vipul, thanks | 19:35 |
| *** ranjitha has joined #openstack-trove | 19:35 | |
| *** sbfox has joined #openstack-trove | 19:35 | |
| denis_makogon | vipul, done | 19:35 |
| vipul | ok so now you shoudl talk about how does one configure what logs are available to save | 19:36 |
| vipul | and what about the swift object naming? it seems like container/file you have in the location attribute.. should really indicate timestamp or something | 19:37 |
| denis_makogon | vipul, it can be accomplished through dict option, specific to datastore | 19:37 |
| *** ranjitha has quit IRC | 19:37 | |
| vipul | can you please give those examples | 19:37 |
| vipul | and why are we calling everything a 'dblog' -- isn't the idea to allow system logs to be saved as well if we want? | 19:37 |
| vipul | just call it 'logs' | 19:37 |
| denis_makogon | allowed_logs = {general_log': '/var/log/mysql/mysql.log', 'log_slow_queries': '/var/log/mysql/mysql-slow.log',} | 19:37 |
| denis_makogon | vipul, {instance_id} + {path according to income filename} + {daytime}.log | 19:39 |
| denis_makogon | vipul, i think it's more than enough for name convention | 19:39 |
| vipul | why isn't instance_id just a container | 19:39 |
| vipul | is naming convention something that you configure in guestagent.conf? | 19:40 |
| *** sbfox has quit IRC | 19:40 | |
| denis_makogon | yes | 19:40 |
| vipul | if so, what are available params i can compose a convention with | 19:40 |
| vipul | ie. is the Guest keeping a list of keys that can be subsituted? | 19:41 |
| denis_makogon | %(instance_id)s + %(log_filename)s + %(daytime)s.log | 19:41 |
| ashishsnigam | peterstac: no | 19:41 |
| vipul | so if i want %(tenant_id)s is that allowed | 19:41 |
| denis_makogon | i guess no | 19:41 |
| vipul | how does someone know that | 19:41 |
| vipul | why not keep it simple | 19:42 |
| denis_makogon | filename + timestamp ? | 19:42 |
| vipul | save it as 'dblogcontainer/{instance_id}/filename.timestamp | 19:42 |
| peterstac | ashishsnigam: ok, thought we might touch base there to co-ordinate the trove/tempest stuff | 19:42 |
| vipul | it's fixed.. it's always the same | 19:42 |
| peterstac | ashishsnigam: we'll do it through the etherpad then :) | 19:43 |
| denis_makogon | vipul, ok | 19:43 |
| vipul | what does manifest convention mean | 19:43 |
| ashishsnigam | sure :) | 19:44 |
| denis_makogon | vipul, container entity file type | 19:45 |
| ashishsnigam | peterstac: Thanks a lot and good night. | 19:45 |
| denis_makogon | vipul, plain text file, archive | 19:45 |
| vipul | denis_makogon: are you planning to zip the log files? | 19:45 |
| *** ashishsnigam has quit IRC | 19:45 | |
| denis_makogon | vipul, yes | 19:45 |
| denis_makogon | vipul, it depends of conf values, that used for backups | 19:46 |
| denis_makogon | *depends on | 19:46 |
| vipul | what is the difference between allow_database_logging and 'allow_database_log_files_audit' | 19:48 |
| vipul | denis_makogon | 19:48 |
| denis_makogon | vipul, i renamed that one to Storage container file extension | 19:48 |
| *** michael-yu has quit IRC | 19:48 | |
| denis_makogon | vipul, allow_database_logging used for rendering database config file | 19:48 |
| denis_makogon | allow_database_log_files_audit - chained with allowed_files, | 19:49 |
| vipul | i still don't know what the difference is | 19:49 |
| vipul | denis_makogon: please be clear when you state things.. don't just assume we all knkow | 19:50 |
| vipul | it might take 20 seconds to add a sentence of how/why you'd set it | 19:50 |
| denis_makogon | vipul, i'd better re-write configuration according to current changes | 19:51 |
| denis_makogon | vipul, will do that by the tomorrow | 19:51 |
| vipul | ok | 19:51 |
| denis_makogon | vipul, can we talk tomorrow ? | 19:52 |
| vipul | sure i'll be arond | 19:52 |
| denis_makogon | like at the same time | 19:52 |
| denis_makogon | vipul, thanks for your help | 19:52 |
| vipul | yup | 19:52 |
| imsplitbit | amcrn: you around? | 19:55 |
| *** michael-yu has joined #openstack-trove | 19:56 | |
| *** ams0 has joined #openstack-trove | 20:01 | |
| *** grapex has quit IRC | 20:08 | |
| *** grapex has joined #openstack-trove | 20:08 | |
| *** NehaV has quit IRC | 20:10 | |
| *** ams0 has quit IRC | 20:11 | |
| iccha1 | hey SlickNik around? cp16net | 20:11 |
| SlickNik | iccha1: here, 'sup? | 20:11 |
| *** ranjitha has joined #openstack-trove | 20:15 | |
| *** michael-yu has quit IRC | 20:23 | |
| *** achampion has quit IRC | 20:27 | |
| *** ams0 has joined #openstack-trove | 20:28 | |
| *** ranjitha has quit IRC | 20:29 | |
| *** michael-yu has joined #openstack-trove | 20:33 | |
| *** ranjitha has joined #openstack-trove | 20:33 | |
| *** ramashri has quit IRC | 20:35 | |
| cp16net | iccha1: hollar | 20:47 |
| iccha1 | SlickNik: cp16net so I was looking at the code and looks like the only patch call is for changing configurations and update instance is a put call https://github.com/openstack/trove/blob/1f3a4fda8712dadaa4f2fe743ac566907d31d3c1/trove/common/api.py#L181 | 20:49 |
| *** aratim1 has joined #openstack-trove | 20:50 | |
| iccha1 | so there are couple of approaches we can take here, One is we keep the PUT call and while updating name, the caller has to ensure the configuration reference is present as well | 20:50 |
| iccha1 | we cannot modify this to patch call, since it would be breaking api contract | 20:51 |
| iccha1 | we could add an additional patch call apart from put if we would like, to allow users to specify only one field to modify | 20:51 |
| *** aratim has quit IRC | 20:52 | |
| dougshelley66 | iccha1 - for changing configurations, couldn't we add the new call and eventually deprecate patch? | 20:54 |
| SlickNik | iccha1: I'd be in favor of doing the latter (and frankly deprecate the current PUT call in juno) | 20:54 |
| *** ams0 has quit IRC | 20:54 | |
| iccha1 | dougshelley66: changing configurations is a sperate resource path, this is changing instance | 20:54 |
| dougshelley66 | iccha1: i understand; i was commenting on the question of consistency | 20:55 |
| iccha1 | dougshelley66: u mean deprecate put? | 20:55 |
| iccha1 | SlickNik: dougshelley66 yes i like the notion of deprecating put as well | 20:55 |
| dougshelley66 | iccha1: i assume we only want to have one path to execute a "modify" and currently we have 2? | 20:56 |
| dougshelley66 | so we should pick one and move forward with it? | 20:56 |
| iccha1 | so the question being do we just add the patch call, and our put works only for configuration? | 20:56 |
| iccha1 | dougshelley66: yes if we add patch we would have two | 20:56 |
| iccha1 | for instance update | 20:56 |
| dougshelley66 | kind of funny because the trove cli calls that modify for config "configuration-patch" | 20:57 |
| dougshelley66 | i assume at that level is should be "configuration-modify" | 20:57 |
| dougshelley66 | ? | 20:57 |
| dougshelley66 | anyone, don't mean to derail your discussion here | 20:57 |
| dougshelley66 | s/anyone/anyway | 20:57 |
| SlickNik | dougshelley66: this is for attach not for patch, I think. | 20:57 |
| SlickNik | dougshelley66: i.e. attach config to an instance, not configuration-patch. | 20:58 |
| iccha1 | yup | 20:58 |
| SlickNik | dougshelley66: The configuration-patch call is actually a PATCH call. | 20:58 |
| SlickNik | iccha1: Yes; no point in adding functionality to a call we're planning on deprecating. | 20:58 |
| iccha1 | SlickNik: just wondering from a rest stand point if it is sound | 20:59 |
| iccha1 | put call working only on partial fields of an instance | 20:59 |
| iccha1 | but it is already like that | 20:59 |
| SlickNik | iccha1: The PUT call as it is today isn't sound. | 20:59 |
| iccha1 | SlickNik: hehe u said exactly the same thing | 20:59 |
| iccha1 | cool glad to know we re on the same page | 21:00 |
| *** ramashri has joined #openstack-trove | 21:10 | |
| *** ams0 has joined #openstack-trove | 21:13 | |
| openstackgerrit | Kevin Conway proposed a change to openstack/trove: Fix missing use of test config datastore version in tests https://review.openstack.org/92209 | 21:22 |
| kevinconway | SlickNik: grapex: one liner bug fix for the trove tests: https://review.openstack.org/#/c/92209/ | 21:24 |
| *** ams0 has quit IRC | 21:24 | |
| *** ams0 has joined #openstack-trove | 21:24 | |
| grapex | kevinconway: Oh man | 21:24 |
| grapex | vipul amcrn hub_cap: Can you all look at this? ^ | 21:25 |
| *** ams0_ has joined #openstack-trove | 21:30 | |
| *** ams0 has quit IRC | 21:30 | |
| *** denis_makogon has quit IRC | 21:30 | |
| openstackgerrit | Peter Stachowski proposed a change to openstack/python-troveclient: Correct grammar in trove CLI help https://review.openstack.org/92211 | 21:32 |
| *** ams0_ has quit IRC | 21:35 | |
| *** ams0 has joined #openstack-trove | 21:55 | |
| *** ams0 has quit IRC | 21:58 | |
| *** ams0 has joined #openstack-trove | 21:58 | |
| SlickNik | grapex: Just noticed that these values are coming from the defaults in trove/tests/config.py, and that the default version there (mysql-5.5) doesn't match the version created by redstack during kick-start (5.5) | 21:59 |
| SlickNik | grapex / kevinconway: if the version is not specified, it uses the default version. But in this case, the wrong version is explicitly specified, so the create-instance call will fail. | 22:00 |
| grapex | SlickNik: Ugh? | 22:01 |
| SlickNik | ugh indeed :( | 22:02 |
| *** ams0 has quit IRC | 22:03 | |
| SlickNik | grapex / kevinconway: Put in a comment to this effect in the review. | 22:07 |
| SlickNik | Thanks for catching the issue. | 22:07 |
| grapex | SlickNik: Actually- you're saying the defaults in trove/tests/config.py don't match redstack? | 22:08 |
| grapex | I think the better approach is to leave this review unmerged but change trove-integration to generate a correct test file | 22:09 |
| grapex | Either way I guess | 22:09 |
| grapex | It might make more sense for the default to do nothing | 22:09 |
| grapex | As in, not explicitly specify the datastore_version | 22:10 |
| *** jasonb365 has quit IRC | 22:17 | |
| *** kevinconway has quit IRC | 22:18 | |
| *** jcru has quit IRC | 22:21 | |
| *** robertmyers has quit IRC | 22:21 | |
| *** rueb7363 has quit IRC | 22:21 | |
| *** rueb7363 has joined #openstack-trove | 22:21 | |
| *** rueb7363 has quit IRC | 22:21 | |
| *** rueb7363 has joined #openstack-trove | 22:21 | |
| *** robertmyers has joined #openstack-trove | 22:21 | |
| *** demorris has quit IRC | 22:22 | |
| *** pdmars has quit IRC | 22:22 | |
| *** robertmyers has quit IRC | 22:26 | |
| *** ams0 has joined #openstack-trove | 22:26 | |
| *** aratim1 has quit IRC | 22:30 | |
| *** ranjitha has quit IRC | 22:32 | |
| *** doddstack has quit IRC | 22:36 | |
| *** michael-yu has quit IRC | 22:36 | |
| *** ramashri has quit IRC | 22:37 | |
| *** michael-yu has joined #openstack-trove | 22:39 | |
| *** grapex has quit IRC | 22:47 | |
| *** rueb7363 has quit IRC | 22:49 | |
| openstackgerrit | Ramashri Umale proposed a change to openstack/trove: Fix prepare call for redis guest agent https://review.openstack.org/84863 | 22:51 |
| *** mattgriffin has quit IRC | 22:53 | |
| *** jmontemayor has quit IRC | 22:59 | |
| *** ams0_ has joined #openstack-trove | 23:13 | |
| *** ams0_ has quit IRC | 23:15 | |
| *** ams0_ has joined #openstack-trove | 23:15 | |
| *** ams0 has quit IRC | 23:17 | |
| *** aratim has joined #openstack-trove | 23:17 | |
| *** aratim has quit IRC | 23:19 | |
| *** ams0_ has quit IRC | 23:20 | |
| *** amytron has joined #openstack-trove | 23:31 | |
| *** flaper87 is now known as flaper87|afk | 23:43 | |
| *** radez_g0n3 is now known as radez | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!