Monday, 2016-01-11

*** chlong has quit IRC00:32
*** chlong has joined #openstack-trove00:49
*** cheneydc has joined #openstack-trove01:02
*** chlong has quit IRC01:09
*** chlong has joined #openstack-trove01:22
*** jengeb has quit IRC01:47
*** johnma has quit IRC01:54
*** yanna has joined #openstack-trove02:12
*** yanna has quit IRC02:15
*** itlinux has joined #openstack-trove02:24
*** itlinux has quit IRC02:30
*** cheneydc has quit IRC02:47
*** cheneydc has joined #openstack-trove02:51
*** haomaiwang has joined #openstack-trove03:16
*** csoukup has joined #openstack-trove03:20
*** sgotliv_ has joined #openstack-trove03:41
*** sgotliv has quit IRC03:43
*** links has joined #openstack-trove03:48
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-trove04:01
*** haomaiwang has quit IRC04:37
*** coolsvap|away is now known as coolsvap04:41
*** csoukup has quit IRC04:49
cp16netdloi: mind rebasing and getting this patchset passing? https://review.openstack.org/#/c/19602504:57
*** david-lyle has quit IRC05:00
*** david-lyle has joined #openstack-trove05:01
*** imandhan has joined #openstack-trove05:39
*** imandhan has quit IRC05:42
*** imandhan has joined #openstack-trove05:43
*** imandhan has quit IRC05:48
*** mingdang1 has joined #openstack-trove05:59
openstackgerritMerged openstack/trove: Test: make enforce_type=True in CONF.set_override  https://review.openstack.org/24745106:27
openstackgerritMerged openstack/trove: Add debug testenv in tox  https://review.openstack.org/26552406:28
*** sgotliv_ has quit IRC07:36
*** chlong has quit IRC07:39
*** k4n0 has joined #openstack-trove07:39
*** csoukup has joined #openstack-trove07:45
*** csoukup has quit IRC07:50
*** haomaiwang has joined #openstack-trove08:12
*** haomaiwang has quit IRC08:27
*** openstackgerrit has quit IRC08:32
*** openstackgerrit has joined #openstack-trove08:32
*** k4n0 has quit IRC08:41
openstackgerritMerged openstack/trove-specs: Add 'Dashboard Impact' to default spec  https://review.openstack.org/26430609:05
*** sgotliv_ has joined #openstack-trove09:38
*** itlinux has joined #openstack-trove09:39
*** itlinux has quit IRC09:44
*** cheneydc has quit IRC10:02
*** qianli has quit IRC10:31
*** jengeb has joined #openstack-trove10:42
*** mingdang1 has quit IRC10:45
*** tellesnobrega_af is now known as tellesnobrega10:55
*** coolsvap is now known as coolsvap|away11:17
*** bhaskarduvvuri has joined #openstack-trove11:53
*** chlong has joined #openstack-trove11:57
*** mingdang1 has joined #openstack-trove11:57
*** tellesnobrega is now known as tellesnobrega_af12:02
*** haomaiwang has joined #openstack-trove12:04
openstackgerritMerged openstack/trove: Fixed 'client connection lost' bug in test_helper  https://review.openstack.org/26432712:17
*** sonali5 has joined #openstack-trove12:20
*** tellesnobrega_af is now known as tellesnobrega12:25
*** tellesnobrega is now known as tellesnobrega_af12:25
*** sonali5 has quit IRC12:37
*** sonali5 has joined #openstack-trove12:45
*** haomaiwang has quit IRC13:01
*** haomaiwang has joined #openstack-trove13:01
*** sonali5 has quit IRC13:06
*** sonali5 has joined #openstack-trove13:09
*** bhaskarduvvuri has quit IRC13:10
*** haomaiwang has quit IRC13:11
*** haomaiwang has joined #openstack-trove13:12
*** links has quit IRC13:13
*** amrith is now known as _amrith_13:40
*** pmackinn has joined #openstack-trove13:44
*** haomaiwang has quit IRC14:01
*** haomaiwang has joined #openstack-trove14:01
*** tellesnobrega_af is now known as tellesnobrega14:26
*** mingdang1 has quit IRC14:29
openstackgerritMerged openstack/python-troveclient: Trival: Remove 'MANIFEST.in'  https://review.openstack.org/26556914:43
sonali5pmalik please review this spec https://review.openstack.org/#/c/263980/14:44
sonali5^peterstac14:44
*** miqui has joined #openstack-trove14:49
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-trove15:01
*** haomaiwang has quit IRC15:01
*** haomaiwang has joined #openstack-trove15:02
*** _amrith_ is now known as amrith15:05
openstackgerritMatthew Van Dijk proposed openstack/trove: Backup to swift using wrong large object type  https://review.openstack.org/22275215:06
*** tellesnobrega is now known as tellesnobrega_af15:14
*** johnma has joined #openstack-trove15:41
*** csoukup has joined #openstack-trove16:08
*** sabeen has joined #openstack-trove16:24
*** tosky has joined #openstack-trove16:27
*** sgotliv_ has quit IRC17:03
*** csoukup has left #openstack-trove17:13
*** tellesnobrega_af is now known as tellesnobrega17:18
*** amrith is now known as _amrith_17:39
*** _amrith_ is now known as amrith17:40
cp16netpmalik: or peterstac i'll need some help figureing out how to make a new datastore client connection with different credentials17:41
cp16netlooks like its currently only setup now for a single user/pass17:41
pmalikcp16net Are you talking about the Trove client or the helper connection?17:44
cp16nethelper connection17:44
cp16netits only using the _get_connection_string() method17:45
cp16netwhen enabling root for a cluster i need to be able to use a root user/pass17:45
cp16netto verify17:45
*** imandhan has joined #openstack-trove17:47
pmalikcp16net Give me a sec I may have something.17:48
cp16netok cool17:48
cp16neti wasnt sure if you already had something done for it17:48
cp16neti saw a bug related to adding root scenario tests17:48
openstackgerritPetr Malik proposed openstack/trove: Add root-actions int-tests  https://review.openstack.org/26595717:56
pmalikcp16net I wrote new root scenario tests ( https://review.openstack.org/#/c/265957/ ). They are for single instance, but I guess you may use them somehow. I was actually thinking about adding another test which would connect as root to verify it is (is not) accessible. I guess we could add new helper methods 'get_root_credentials' and 'get_root_client' which could be facilitaded by these tests... Maybe let's wait for pete18:00
pmalikrstac he may have more ideas.18:00
pmalikpeterstac ^^^18:01
cp16netpmalik: yeah thats simliar to what i was thinking as well18:01
cp16neti feel like the verify methods are kinad magic18:01
*** _mattf is now known as mattf18:02
cp16netlooks like the assert_root_enable method is not called right from the other tests18:02
*** mattf is now known as _mattf18:02
cp16netthe 2nd arg should be root_password and i dont see it being called like that in other methods18:03
pmalikcp16net 'assert_root_enable' is called from 'run_enable_root_no_password' and 'run_enable_root_with_password'18:06
*** sharikapongubala has joined #openstack-trove18:06
pmalikcp16net The naming is a bit unfortunate as there is also another method called 'assert_root_enabled' ('d' at the end)... ;-/18:07
cp16netoh18:07
cp16netdamn i didnt realize that18:07
cp16neti think that should change18:07
pmalikcp16net i agree :-)18:07
cp16netthats difficult to realize ;-)18:08
openstackgerritPetr Malik proposed openstack/trove: Add root-actions int-tests  https://review.openstack.org/26595718:13
*** sonali5 has quit IRC18:21
*** davideag_ has quit IRC18:46
*** davideagnello has joined #openstack-trove18:50
*** barra204 has joined #openstack-trove18:55
*** shakamunyi has quit IRC18:56
cp16netpmalik: doesnt looks like it helps as much18:58
cp16neti'll finish what i was thinking and then maybe we can work it together18:59
cp16netmine is a WIP18:59
pmalikcp16net we discussed it with peterstac We will be pushing something up into the root patch set. The idea is essentially, to add another helper method (similar to add_data) for testing connections (say called 'ping', would perform 'SELECT 1' on MySQL). That method would accept credentials. You get the credentials from root_create, so you could just call 'ping' with those. If not given the ping would use the 'default' cred19:03
pmalikentials from the helper. We will need to update the get_client method to accept credentials, but that should not be difficult.19:03
*** sabeen has quit IRC19:05
cp16netok yeah its similar19:06
cp16netpmalik: is there work already done for this?19:06
cp16neti have something similar thats close to that19:06
pmalikcp16net We were planning to include it in the root patch set. Maybe if you push up your version we can discuss it and don't need to do anything more?19:08
pmalikcp16net * No work done at this instance.19:08
*** shakamunyi has joined #openstack-trove19:12
*** barra204 has quit IRC19:12
*** amrith is now known as _amrith_19:13
*** csoukup has joined #openstack-trove19:14
*** sabeen1 has joined #openstack-trove19:19
*** sabeen2 has joined #openstack-trove19:21
*** sabeen1 has quit IRC19:24
pmalikcp16net You may actually push your solution into my root patch set. Once it works there we can then extend it to clusters.19:27
*** _amrith_ is now known as amrith19:37
*** sharikapongubala has quit IRC19:37
cp16netok i think i have something done now19:37
cp16neti havnt ran the tests yet so its WIP but you can see what i was thinking.19:38
openstackgerritCraig Vyvial proposed openstack/trove: refactor root enable for pxc clustered datastore  https://review.openstack.org/26600519:40
*** jengeb1 has joined #openstack-trove19:43
*** jengeb has quit IRC19:48
*** sgotliv_ has joined #openstack-trove20:23
openstackgerritDuk Loi proposed openstack/trove: Add support for root-disable  https://review.openstack.org/19602520:56
openstackgerritDuk Loi proposed openstack/trove: Add support for root-disable  https://review.openstack.org/19602520:58
openstackgerritMerged openstack/trove-dashboard: Fix unit tests  https://review.openstack.org/26531921:28
dougshelley66cp16net yt?21:44
openstackgerritDuk Loi proposed openstack/trove: Add support for root-disable  https://review.openstack.org/19602521:53
dougshelley66cp16net - you might be interested in this: https://bugs.launchpad.net/trove/+bug/153296921:54
openstackLaunchpad bug 1532969 in Trove "Percona XtraDB Cluster not working with v5.6" [Undecided,New] - Assigned to Doug Shelley (0-doug)21:54
dougshelley66i have fixes for it, will push up soon21:55
cp16netdougshelley66: hey21:57
cp16netdougshelley66: ok21:57
dougshelley66cp16net i believe i have the issues in the LP bug fixed and can now get a pxc 5.6 cluster to start in redstack22:01
dougshelley66just need to get the unit tests passing etc and will push up22:01
cp16netdougshelley66: so the pxb2.3 is using socket instead of the host ip?22:03
-openstackstatus- NOTICE: Gerrit is restarting to resolve java memory issues22:06
*** dalees has joined #openstack-trove22:14
*** mingdang1 has joined #openstack-trove22:28
*** sgotliv_ has quit IRC22:31
dougshelley66cp16net it looks like it was using host="localhost" which causes the mysql client to attempt to be smart and switch to unix domain socket22:50
dougshelley66i'm not completely certain what the specific change was in pxb2.3 but george told me it was a complete rewrite22:51
*** tosky has quit IRC23:07
openstackgerritVictoria Martinez de la Cruz proposed openstack/trove-specs: Add support for MariaDB GTID Replication  https://review.openstack.org/26069023:11
*** tellesnobrega is now known as tellesnobrega_af23:16
*** csoukup has quit IRC23:17
*** mingdang1 has quit IRC23:17
*** miqui has quit IRC23:17
*** tellesnobrega_af is now known as tellesnobrega23:17
cp16netdougshelley66: interesting23:19
cp16netnice work debugging it though! :)23:19
*** sonali5 has joined #openstack-trove23:25
openstackgerritVictoria Martinez de la Cruz proposed openstack/trove: Adds support for MariaDB GTID replication  https://review.openstack.org/26607123:25
dougshelley66cp16net, thanks - but we will wait until you see the fixes :)23:37
*** sabeen2 has quit IRC23:50

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