| *** tosky___ has quit IRC | 00:04 | |
| openstackgerrit | Jonathan Halterman proposed openstack/trove: Percona Xtradb Cluster implementation https://review.openstack.org/208073 | 00:04 |
|---|---|---|
| cp16net | saurabhs: nice! | 00:09 |
| cp16net | stuff got merged! :) | 00:09 |
| openstackgerrit | Saurabh Surana proposed openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 00:13 |
| *** sharikapongubala has quit IRC | 00:13 | |
| openstackgerrit | Saurabh Surana proposed openstack/trove: Add support for extended_properties for clusters https://review.openstack.org/206931 | 00:14 |
| openstackgerrit | Ishita Mandhan proposed openstack/trove-integration: Remove directory check in mysql redstack install https://review.openstack.org/220362 | 00:25 |
| openstackgerrit | Jonathan Halterman proposed openstack/trove: Add support for extended_properties for clusters https://review.openstack.org/206931 | 00:28 |
| openstackgerrit | Ishita Mandhan proposed openstack/trove-integration: Remove directory check in mysql redstack install https://review.openstack.org/220362 | 00:28 |
| openstackgerrit | Saurabh Surana proposed openstack/trove: Modifying Vertica clusters to have a master node https://review.openstack.org/204271 | 00:32 |
| openstackgerrit | Saurabh Surana proposed openstack/trove: Add floating ip to vertica master node https://review.openstack.org/207578 | 00:33 |
| openstackgerrit | Merged openstack/python-troveclient: Implements Datastore Registration API https://review.openstack.org/214510 | 00:34 |
| openstackgerrit | Merged openstack/trove: Implements Datastore Registration API https://review.openstack.org/195153 | 00:34 |
| openstackgerrit | Saurabh Surana proposed openstack/trove: made vertica debian package path configurable https://review.openstack.org/214990 | 00:34 |
| *** sgotliv has quit IRC | 00:43 | |
| *** saurabhs1 has left #openstack-trove | 00:46 | |
| *** imandhan has quit IRC | 00:54 | |
| *** _amrith_ is now known as amrith | 01:17 | |
| *** erkules_ has joined #openstack-trove | 01:17 | |
| amrith | saurabhs, the rebase to make dependent on pxc just messed me up big time. | 01:19 |
| amrith | between running out of disk space and being half way through a merge, I'm somewhere in git hell. | 01:20 |
| amrith | time to start over. | 01:20 |
| *** erkules has quit IRC | 01:20 | |
| amrith | cp16net, what got merged? | 01:23 |
| amrith | pxc? | 01:23 |
| cp16net | amrith: no not that yet | 01:26 |
| cp16net | i think thats going to be one of the last ones since i had to change the commit msg | 01:26 |
| amrith | ok | 01:26 |
| SlickNik | amrith: It was just datastore registration that got merged. | 01:26 |
| amrith | ran out of space on vm in the middle of rebasing saurabs change on top of pxc | 01:27 |
| amrith | and git went into a tailspin | 01:27 |
| cp16net | and now the image build and tests should work | 01:27 |
| SlickNik | That sounds like fun. | 01:27 |
| amrith | it is a vm ... | 01:27 |
| amrith | so reset to last known good state (thankfully I had images loaded) and trying to clean up and rebase. | 01:28 |
| cp16net | yikes | 01:28 |
| cp16net | nice | 01:28 |
| amrith | should be back up and running in a couple of minutes. | 01:28 |
| amrith | I love vm snapshots ;) | 01:28 |
| cp16net | yeah they can save save your @$$ | 01:29 |
| *** sharikapongubala has joined #openstack-trove | 02:00 | |
| *** sharikapongubala has quit IRC | 02:01 | |
| *** sharikapongubala has joined #openstack-trove | 02:02 | |
| cp16net | expect to see some more merges here in a few... | 02:09 |
| cp16net | wow the merge-check queue is huge... 1221 right now | 02:17 |
| SlickNik | coming down slowly ~300 right now. | 02:47 |
| cp16net | heh yeah little by little | 02:49 |
| *** imandhan has joined #openstack-trove | 02:50 | |
| *** imandhan_ has joined #openstack-trove | 02:50 | |
| saurabhs | amrith: my patch already depends on pxc so you don't need to do anything extra. Just pull in my patch and you should get pxc too | 02:51 |
| saurabhs | If you want to test it with Maria then you will need to rebase it with that patch but then you also need the other mariadb patch I uploaded this morning which adds a root controller entry for mariadb | 02:53 |
| *** openstackgerrit_ has joined #openstack-trove | 03:11 | |
| *** sharikapongubala has quit IRC | 03:28 | |
| *** eglute has quit IRC | 03:28 | |
| *** sharikapongubala has joined #openstack-trove | 03:30 | |
| *** eglute has joined #openstack-trove | 03:33 | |
| *** links has joined #openstack-trove | 04:12 | |
| *** openstackgerrit_ has quit IRC | 04:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-troveclient: Updated from global requirements https://review.openstack.org/217214 | 04:19 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Updated from global requirements https://review.openstack.org/216572 | 04:20 |
| *** rjaiswal has joined #openstack-trove | 04:33 | |
| openstackgerrit | Merged openstack/trove: Add Redis backup/restore functionality https://review.openstack.org/185892 | 04:50 |
| openstackgerrit | Merged openstack/trove-integration: MariaDB 5.5 image building scripts https://review.openstack.org/211791 | 04:51 |
| openstackgerrit | Merged openstack/trove: Add support for MariaDB datastore in Trove https://review.openstack.org/217451 | 04:56 |
| *** haomaiwa_ has quit IRC | 04:57 | |
| openstackgerrit | Merged openstack/python-troveclient: Redis Clustering Initial Implementation https://review.openstack.org/203701 | 04:59 |
| cp16net | W00T! | 04:59 |
| cp16net | more stuff is merging! :) | 04:59 |
| *** imandhan_ has quit IRC | 05:01 | |
| *** imandhan has quit IRC | 05:01 | |
| *** imandhan_ has joined #openstack-trove | 05:02 | |
| *** imandhan has joined #openstack-trove | 05:02 | |
| openstackgerrit | Merged openstack/python-troveclient: Use more appropriate exceptions for validation https://review.openstack.org/210004 | 05:03 |
| cp16net | hmm gate is clear for trove stuff now... | 05:04 |
| *** imandhan_ has quit IRC | 05:07 | |
| *** imandhan has quit IRC | 05:07 | |
| *** sharikapongubala has quit IRC | 05:12 | |
| *** sharikapongubala has joined #openstack-trove | 05:12 | |
| cp16net | i'll work on rebasing https://review.openstack.org/#/c/203702/ | 05:12 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Redis Cluster Initial Implementation https://review.openstack.org/203702 | 05:42 |
| *** mingdang1 has joined #openstack-trove | 06:04 | |
| *** k4n0 has joined #openstack-trove | 06:06 | |
| SlickNik | cp16net: Nice! Thanks for rebasing the Redis patch | 06:20 |
| cp16net | yeah working on the next one... | 06:20 |
| cp16net | btw look that over and make sure i didnt screw it up | 06:20 |
| cp16net | i rebasing the redis replication on that cluster patch | 06:20 |
| cp16net | this is proving a bit painful | 06:21 |
| cp16net | working on this one. https://review.openstack.org/#/c/203278/ | 06:22 |
| SlickNik | Looks like we need to fix the pxc patch as well. | 06:23 |
| cp16net | do we? | 06:23 |
| cp16net | i think someone rebased it | 06:23 |
| cp16net | ahh crap | 06:23 |
| SlickNik | https://review.openstack.org/#/c/208073/ => Patch in merge conflict | 06:23 |
| cp16net | ok i'll do that in a sec | 06:23 |
| cp16net | almost done with this one... | 06:24 |
| SlickNik | cp16net: no worries, thanks. | 06:25 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Percona Xtradb Cluster implementation https://review.openstack.org/208073 | 06:29 |
| cp16net | ok so i think there could be another rebase... | 06:31 |
| cp16net | maybe i should just line these patches on each other if i did it right... | 06:31 |
| SlickNik | That would prevent us from having to rebase again. | 06:32 |
| SlickNik | sushilkm: around? | 06:32 |
| cp16net | yeah i think that might be a good idea | 06:32 |
| cp16net | he was chatting with me | 06:32 |
| SlickNik | sushilkm: Any ideas what this unit test failure is about? http://logs.openstack.org/55/205355/15/check/gate-trove-python27/f1d91b0/console.html#_2015-09-04_05_56_39_184 | 06:33 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Add support for Redis replication https://review.openstack.org/203278 | 06:34 |
| cp16net | ok i think that chained those 2... | 06:35 |
| cp16net | yes it did. | 06:35 |
| cp16net | oh nice... | 06:35 |
| cp16net | i'll add the pxc after | 06:35 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Percona Xtradb Cluster implementation https://review.openstack.org/208073 | 06:38 |
| sushilkm | hello SlickNik | 06:43 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 06:43 |
| cp16net | ok i think i got a chain done... | 06:44 |
| sushilkm | not sure about that, i have tried a sample run, did not get that error, also did not see that in any other test runs, but still i wud look to how i can suppress that | 06:44 |
| cp16net | SlickNik: http://logs.openstack.org/55/205355/15/gate/gate-trove-python27/b37398d/console.html#_2015-09-04_02_23_03_652 | 06:44 |
| cp16net | that was the run before | 06:44 |
| sushilkm | i guess it cud be occurring because of several other ds-versions getting created durin complete test run | 06:44 |
| cp16net | concurrency race condition? | 06:45 |
| cp16net | :-( | 06:45 |
| sushilkm | i guess that too it cud be because of some concurrency problems | 06:45 |
| sushilkm | anyways i would have a look on this to find out how could it be eradicated | 06:46 |
| sushilkm | let me quickly file up a bug for this | 06:46 |
| *** mingdang1 has quit IRC | 06:48 | |
| SlickNik | sushilkm: We need to get it fixed pretty quickly since it looks like it will cause random breaks in the gate and prevent other changes from merging. | 06:49 |
| cp16net | omg did i screw up something on the redis cluster patch now? | 06:49 |
| cp16net | damnit looks like something got screwed up... | 06:50 |
| sushilkm | ok luking into the same | 06:50 |
| *** rjaiswal has quit IRC | 06:50 | |
| cp16net | trying to run it manaully... | 06:52 |
| cp16net | wtf it worked | 06:53 |
| SlickNik | cp16net: what happened? Anything I can help with? | 06:53 |
| cp16net | look at zuul | 06:53 |
| cp16net | am i crazy that tox failed? with some user change test? | 06:54 |
| cp16net | oh shit i think i checked out the wrong ###### | 06:55 |
| cp16net | its getting late... | 06:55 |
| cp16net | i'm looking forward to the 3 day weekend... | 06:56 |
| cp16net | i need it | 06:56 |
| SlickNik | word | 06:57 |
| SlickNik | you should get some rest man. | 06:57 |
| sushilkm | SlickNik, I can grab the reason, and the fix for http://logs.openstack.org/55/205355/15/check/gate-trove-python27/f1d91b0/console.html#_2015-09-04_05_56_39_184 | 06:58 |
| sushilkm | but not sure about http://logs.openstack.org/55/205355/15/gate/gate-trove-python27/b37398d/console.html#_2015-09-04_02_23_03_652 | 06:58 |
| sushilkm | because this one is giving error in setup.py | 06:59 |
| sushilkm | where it tries to load a ds_version which was just created two lines earlier than failing line | 06:59 |
| sushilkm | https://github.com/openstack/trove/blob/master/trove/tests/unittests/mgmt/test_datastores.py#L31-L40 | 07:00 |
| cp16net | i'm thinking depending on a database in a unittest is dumb | 07:01 |
| cp16net | and biting us in the ass | 07:01 |
| sushilkm | yeah more because we dont have any method to clear database | 07:02 |
| cp16net | well i think we do... | 07:02 |
| cp16net | its calling util.init_db() | 07:02 |
| cp16net | it recreates the db | 07:02 |
| cp16net | and its called in multiple places | 07:02 |
| cp16net | a unit test shouldnt deal with a db | 07:03 |
| cp16net | fakemode should be the only thing using a db | 07:03 |
| cp16net | and "real" mode tests | 07:03 |
| cp16net | because you dont need to test the db queries | 07:04 |
| cp16net | just the data that you expect | 07:04 |
| * cp16net gets off soap box... | 07:04 | |
| sushilkm | i dont think so it recreates | 07:04 |
| sushilkm | i guess it only creates | 07:05 |
| sushilkm | and moreover this becomes more difficult as these failures are hard to reproduce sometimes | 07:06 |
| openstackgerrit | Sushil Kumar proposed openstack/trove: Fixes the mgmt.test_datastores errors https://review.openstack.org/220419 | 07:22 |
| sushilkm | SlickNik, cp16net i pushed up a fix for those datastore test problems | 07:23 |
| cp16net | nice | 07:26 |
| sushilkm | i saw ur bug for removing test dependancy on db | 07:26 |
| *** pmackinn has quit IRC | 07:32 | |
| cp16net | sushilkm: could you look at this gate issue? https://review.openstack.org/#/c/205355/ | 07:37 |
| cp16net | i've run it manually and got no errors | 07:37 |
| cp16net | i might be losing my mind | 07:38 |
| sushilkm | sure let me look | 07:39 |
| cp16net | Ran 396 tests in 151.890s | 07:41 |
| cp16net | alright so i just ran it like 3 times with that patchset and its worked everytime for me and also ran it on master with out any issues. | 07:45 |
| cp16net | i'm out. talk to you in the morning | 07:45 |
| cp16net | i hope you can replicate it because i cant. :( | 07:46 |
| cp16net | nite | 07:46 |
| sushilkm | cp16net r u there | 07:49 |
| cp16net | i am | 07:49 |
| sushilkm | i got the problem | 07:49 |
| cp16net | yea? | 07:49 |
| sushilkm | https://review.openstack.org/#/c/210004/ | 07:49 |
| sushilkm | https://review.openstack.org/#/c/210004/6/troveclient/v1/users.py | 07:49 |
| sushilkm | this has updated the exception | 07:50 |
| sushilkm | shall i push up a bug fix quickly so that we can go further | 07:50 |
| cp16net | nice catch | 07:51 |
| cp16net | umm | 07:51 |
| cp16net | i'll update them | 07:51 |
| cp16net | theres a long chain that have to be updated... | 07:51 |
| cp16net | unless you want to do it :-D | 07:52 |
| cp16net | i got it started... | 07:53 |
| cp16net | confused on how this got in tho | 07:57 |
| cp16net | i thought SlickNik had a fix that resolved this type of issue | 07:57 |
| sushilkm | r u working on the fix | 08:02 |
| cp16net | https://review.openstack.org/#/c/218701/ | 08:02 |
| cp16net | its already there i just saw | 08:02 |
| sushilkm | ok | 08:02 |
| sushilkm | cool | 08:02 |
| cp16net | sushilkm: sorry for this.. | 08:07 |
| sushilkm | np | 08:07 |
| cp16net | could you take it? | 08:07 |
| sushilkm | what | 08:07 |
| cp16net | get that patch to pass and dependent on the others | 08:07 |
| *** sharikapongubala has quit IRC | 08:08 | |
| sushilkm | do u mean to create a dependency on all patches | 08:09 |
| *** sharikapongubala has joined #openstack-trove | 08:09 | |
| cp16net | yeah | 08:09 |
| sushilkm | and which patches, do we have any list | 08:09 |
| cp16net | https://review.openstack.org/#/c/203278/17 | 08:10 |
| cp16net | all the related patches there | 08:10 |
| cp16net | get them to pass check | 08:10 |
| sushilkm | i think the simpler way wud be get the fixin patch merged and then let others go after that | 08:10 |
| cp16net | if that works | 08:11 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 08:26 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Expect ValidationError on an empty user update https://review.openstack.org/218701 | 08:26 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Add support for Redis replication https://review.openstack.org/203278 | 08:26 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Redis Cluster Initial Implementation https://review.openstack.org/203702 | 08:26 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Percona Xtradb Cluster implementation https://review.openstack.org/208073 | 08:26 |
| *** lexholden has joined #openstack-trove | 08:27 | |
| cp16net | i dont see a wf+1 on this one so i'm holding off https://review.openstack.org/#/c/196913/ | 08:27 |
| cp16net | but the others should all get in now... | 08:27 |
| cp16net | done.. | 08:28 |
| cp16net | recheck if needed | 08:28 |
| * cp16net drops the mic | 08:28 | |
| openstackgerrit | Sushil Kumar proposed openstack/trove: Fixes the mgmt.test_datastores errors https://review.openstack.org/220419 | 08:51 |
| *** sharikapongubala has quit IRC | 08:56 | |
| *** coolsvap_ has joined #openstack-trove | 09:01 | |
| *** tosky has joined #openstack-trove | 09:25 | |
| openstackgerrit | Merged openstack/python-troveclient: Updated from global requirements https://review.openstack.org/217214 | 09:33 |
| *** erkules_ is now known as erkules | 09:41 | |
| *** erkules has joined #openstack-trove | 09:41 | |
| *** sharikapongubala has joined #openstack-trove | 09:56 | |
| *** coolsvap_ has quit IRC | 09:58 | |
| *** sharikapongubala has quit IRC | 10:01 | |
| *** IanGovett has joined #openstack-trove | 10:21 | |
| openstackgerrit | Merged openstack/trove: Expect ValidationError on an empty user update https://review.openstack.org/218701 | 10:32 |
| openstackgerrit | Merged openstack/trove: Redis Cluster Initial Implementation https://review.openstack.org/203702 | 10:35 |
| openstackgerrit | Merged openstack/trove: Add support for Redis replication https://review.openstack.org/203278 | 10:35 |
| openstackgerrit | Merged openstack/trove: Percona Xtradb Cluster implementation https://review.openstack.org/208073 | 10:35 |
| *** links has quit IRC | 10:49 | |
| sushilkm | :64 | 11:52 |
| amrith | I don't believe it ... gate passed on https://review.openstack.org/#/c/220288/ | 11:53 |
| amrith | :65 | 11:53 |
| amrith | sushilkm, some more explanation of the race condition in 220419 would be good. | 11:55 |
| sushilkm | amrith, what more shall i add, whats the demand | 11:56 |
| amrith | i don't know that it is a demand, just that from the description and the change you are leaving it up to the reviewer to figure out what you think the race condition is. | 11:57 |
| sushilkm | oh do you mean to define the race condition | 11:58 |
| *** mingdang1 has joined #openstack-trove | 12:06 | |
| openstackgerrit | Sushil Kumar proposed openstack/trove: Fixes the mgmt.test_datastores errors https://review.openstack.org/220419 | 12:06 |
| sushilkm | amrith, does that luks better | 12:07 |
| openstackgerrit | Sushil Kumar proposed openstack/trove: Mongodb Cluster Scaling https://review.openstack.org/205355 | 12:08 |
| amrith | not really, to describe a race condition involving (say) two concurrent players, you say what happens in sequence. you identify which action is done by which player and what the actions are that conflict with each other. your new commit message just describes a theoretical race condition using more words. Yes, I get it that setUp is doing something that is messing with a function. And I get that it has something to do | 12:10 |
| amrith | with one of the datastore loads. But, there's another one there as well, why isn't that a race condition etc., | 12:10 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Mongodb Cluster Scaling https://review.openstack.org/205355 | 12:15 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Mongodb Cluster Scaling https://review.openstack.org/205355 | 12:18 |
| mvandijk | sushilkm, i just pushed up the same changes you did | 12:20 |
| sushilkm | no there are some diffrences in what u sent | 12:21 |
| sushilkm | the pathcset you have pushed might fail on jenkins | 12:21 |
| mvandijk | let me know next time you decide to do that :/ | 12:21 |
| *** links has joined #openstack-trove | 12:21 | |
| sushilkm | also i had fixed the comment petr had on the patchset | 12:22 |
| mvandijk | yes same | 12:22 |
| mvandijk | k respinning | 12:22 |
| mvandijk | one more time | 12:22 |
| openstackgerrit | Sushil Kumar proposed openstack/trove: Fixes the mgmt.test_datastores errors https://review.openstack.org/220419 | 12:23 |
| *** k4n0 has quit IRC | 12:23 | |
| amrith | y'all guys race to a condition where you agree ... ;) | 12:23 |
| sushilkm | amrith, updated the commit message I think one can now understand the problem and the fix better | 12:23 |
| sushilkm | everyone has a job, someone has a job to race :P | 12:25 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Mongodb Cluster Scaling https://review.openstack.org/205355 | 12:26 |
| sushilkm | mvandijk, yeah now its better :) | 12:27 |
| *** arist has quit IRC | 12:30 | |
| *** arist has joined #openstack-trove | 12:31 | |
| *** mattgriffin has joined #openstack-trove | 12:57 | |
| *** pmackinn has joined #openstack-trove | 12:59 | |
| amrith | except for people without a job | 13:00 |
| *** mattgriffin has quit IRC | 13:04 | |
| *** mattgriffin has joined #openstack-trove | 13:07 | |
| *** coolsvap_ has joined #openstack-trove | 13:08 | |
| *** mattgriffin has quit IRC | 13:11 | |
| *** haomaiwang has joined #openstack-trove | 13:21 | |
| *** links has quit IRC | 13:27 | |
| *** sharikapongubala has joined #openstack-trove | 13:59 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-trove | 14:01 | |
| *** sharikapongubala has quit IRC | 14:03 | |
| *** csoukup has joined #openstack-trove | 14:13 | |
| *** mingdang1 has quit IRC | 14:17 | |
| *** coolsvap_ has quit IRC | 14:41 | |
| *** _crobertsrh is now known as crobertsrh | 14:44 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-trove | 15:01 | |
| *** lexholden has quit IRC | 15:16 | |
| *** mattgriffin has joined #openstack-trove | 15:18 | |
| *** erkules has left #openstack-trove | 15:24 | |
| cp16net | hey yall | 15:50 |
| *** ashleighfarnham has joined #openstack-trove | 15:52 | |
| cp16net | amrith: how goes it? | 15:53 |
| openstackgerrit | Petr Malik proposed openstack/trove: Add user and database actions int-tests https://review.openstack.org/210094 | 15:53 |
| amrith | cp16net, good | 15:57 |
| cp16net | amrith: great! | 15:57 |
| cp16net | i had to do some crazy rebasing last night but we got a bunch in | 15:57 |
| cp16net | sushilkm: thanks for watching them last night | 15:58 |
| amrith | i saw some of them. | 15:59 |
| amrith | had some trouble with the merging last night | 15:59 |
| cp16net | yeah | 15:59 |
| cp16net | i ended up chaining them together | 15:59 |
| cp16net | it helped because redis cluster/replication changed things in the same places | 15:59 |
| cp16net | then to make sure there were no other issues i chained the ones we approved already together | 16:00 |
| *** haomaiwang has quit IRC | 16:01 | |
| cp16net | amrith: so what is your status on the vertica root enable patches? | 16:01 |
| *** haomaiwang has joined #openstack-trove | 16:01 | |
| amrith | cp16net, I'm going to +2 them | 16:01 |
| amrith | cp16net, do you want to take a gander at them as well? | 16:02 |
| amrith | they have merge relative to pxc | 16:02 |
| amrith | and I've tried mysql | 16:02 |
| amrith | NOT vertica | 16:02 |
| amrith | not pxc ;) | 16:02 |
| *** sabeen has joined #openstack-trove | 16:03 | |
| cp16net | i'm looking at them now | 16:03 |
| cp16net | i have a feeling we will need to fix some issues after all is said and done with these changes | 16:04 |
| cp16net | i'll update it with the major things i see | 16:05 |
| cp16net | right now mariadb doesnt have a cfg since it was merged | 16:06 |
| *** mattgriffin has quit IRC | 16:09 | |
| openstackgerrit | Jonathan Halterman proposed openstack/trove: Add support for extended_properties for clusters https://review.openstack.org/206931 | 16:13 |
| *** sabeen has quit IRC | 16:14 | |
| amrith | yes | 16:16 |
| amrith | and there is a corresponding need to abstract the root controller for that. | 16:16 |
| amrith | as best as I can tell, this change hasn't broken mysql and other existing databases that I have played with. | 16:17 |
| *** sabeen1 has joined #openstack-trove | 16:17 | |
| cp16net | thats good. | 16:17 |
| *** sharikapongubala has joined #openstack-trove | 16:18 | |
| amrith | i just posted my comments on it. | 16:18 |
| cp16net | amrith: https://review.openstack.org/#/c/196913/36/trove/guestagent/datastore/experimental/couchbase/manager.py,cm | 16:21 |
| cp16net | no logging ? | 16:21 |
| cp16net | just an observation? | 16:21 |
| cp16net | looks like it to me. | 16:22 |
| *** saurabhs1 has joined #openstack-trove | 16:23 | |
| cp16net | i updated it tho | 16:24 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Disable MongoDB cluster security https://review.openstack.org/220574 | 16:25 |
| mvandijk | cp16net, amrith, peterstac https://review.openstack.org/#/c/205355/ has been rebased and bug fixed | 16:26 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 16:26 |
| cp16net | amrith: i made those commented changes | 16:26 |
| cp16net | so that should be good now | 16:26 |
| saurabhs1 | cp16net just logged in let me know if I can help. | 16:28 |
| saurabhs1 | cp16net thanks for updating those root-enable patches | 16:28 |
| cp16net | np | 16:28 |
| cp16net | amrith: 16:21 < cp16net> just an observation? | 16:31 |
| cp16net | doh.. | 16:31 |
| cp16net | amrith: https://review.openstack.org/#/c/196913/ | 16:31 |
| cp16net | look at it again | 16:31 |
| peterstac | mvandijk, done | 16:31 |
| saurabhs1 | cp16net I see you added root-controller configuration for mariadb. Since mariadb patch was not merged earlier, I created another patch set which depends on mariadb and root-enable https://review.openstack.org/#/c/220280/. I will abandon this now. | 16:32 |
| cp16net | ah ok | 16:32 |
| cp16net | yeah i added it to be complete in that patch | 16:32 |
| amrith | done | 16:33 |
| cp16net | thx | 16:33 |
| saurabhs1 | amrith cp16net thanks for review, rebase and merge on root-enable patches | 16:40 |
| *** imandhan has joined #openstack-trove | 16:58 | |
| openstackgerrit | Petr Malik proposed openstack/trove: Defer revision dir initialization in GA https://review.openstack.org/218382 | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-trove | 17:01 | |
| *** openstackgerrit_ has joined #openstack-trove | 17:02 | |
| cp16net | alright it looks like after the 3 patches we should be good on the blueprints (mongodb cluster scaling) (root enable for vertica trove and client) | 17:02 |
| *** exploreshaifali has joined #openstack-trove | 17:09 | |
| *** tosky has quit IRC | 17:11 | |
| *** rjaiswal has joined #openstack-trove | 17:11 | |
| *** tosky has joined #openstack-trove | 17:15 | |
| SlickNik | cp16net: w00t! | 17:20 |
| pmackinn | SlickNik, trove-int coast clear? | 17:23 |
| vipul | hey so i heard Trove hit the nova instance naming issue too | 17:26 |
| vipul | is that a breaking API change? | 17:26 |
| vipul | or is that something with the v2.1 of the api? | 17:26 |
| SlickNik | It's in v2.1 | 17:26 |
| vipul | what did you guys do to work around it | 17:26 |
| SlickNik | They switched from v2.0 to v2.1 by default. | 17:27 |
| SlickNik | We're still trying to figure out the best way to fix the issue. | 17:27 |
| vipul | is v2.1 the recommended nova api version for liberty? | 17:27 |
| SlickNik | vipul: FYI https://bugs.launchpad.net/nova/+bug/1491511 | 17:29 |
| openstack | Launchpad bug 1491511 in OpenStack Compute (nova) "Behavior change with latest nova paste config" [Critical,In progress] - Assigned to John Garbutt (johngarbutt) | 17:29 |
| SlickNik | vipul: also - https://review.openstack.org/#/c/220279/ | 17:30 |
| SlickNik | They're planning on fixing it. | 17:30 |
| SlickNik | This is the change that switched it over — https://github.com/openstack/nova/commit/a9090bb5f50310702408aee340af9d4dad9474cf | 17:31 |
| SlickNik | From the change "starting in Liberty the v21 implementation replaces the v2 implementation and is suggested that you use it as the default" | 17:32 |
| openstackgerrit | Pete MacKinnon proposed openstack/trove-integration: Modifications for Fedora 21 support in redstack https://review.openstack.org/214195 | 17:32 |
| vipul | SlickNik: awesome thanks | 17:43 |
| SlickNik | vipul: Not a problem. | 17:45 |
| *** crobertsrh has quit IRC | 17:45 | |
| *** crobertsrh has joined #openstack-trove | 17:46 | |
| *** crobertsrh has quit IRC | 17:48 | |
| *** crobertsrh has joined #openstack-trove | 17:51 | |
| *** crobertsrh has quit IRC | 17:52 | |
| *** crobertsrh has joined #openstack-trove | 17:52 | |
| *** devlaps has joined #openstack-trove | 17:56 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-trove | 18:01 | |
| *** mattgriffin has joined #openstack-trove | 18:07 | |
| *** IanGovett has quit IRC | 18:18 | |
| *** itlinux has joined #openstack-trove | 18:24 | |
| *** crobertsrh has quit IRC | 18:27 | |
| *** crobertsrh has joined #openstack-trove | 18:29 | |
| *** sharikapongubala has quit IRC | 18:29 | |
| *** sharikapongubala has joined #openstack-trove | 18:30 | |
| *** itlinux has quit IRC | 18:31 | |
| amrith | vipul, I know someone by that name ... | 18:34 |
| vipul | vipul: yea me too haha | 18:34 |
| cp16net | stanger danger! | 18:36 |
| amrith | yes, that vipul guy. danger is right ... | 18:37 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Disable MongoDB cluster security https://review.openstack.org/220574 | 18:54 |
| openstackgerrit | Matthew Van Dijk proposed openstack/trove: Disable MongoDB cluster security https://review.openstack.org/220574 | 18:56 |
| *** haomaiwang has quit IRC | 19:01 | |
| *** 5EXAB2JHN has joined #openstack-trove | 19:01 | |
| *** tosky has quit IRC | 19:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Updated from global requirements https://review.openstack.org/216572 | 19:41 |
| *** mattgriffin has quit IRC | 19:45 | |
| *** 5EXAB2JHN has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-trove | 20:01 | |
| *** openstackgerrit_ has quit IRC | 20:03 | |
| openstackgerrit | Peter Stachowski proposed openstack/trove: Add support for configuration groups in int-tests https://review.openstack.org/213388 | 20:03 |
| *** openstackgerrit_ has joined #openstack-trove | 20:04 | |
| *** laurelm has quit IRC | 20:16 | |
| *** schang has quit IRC | 20:16 | |
| *** laurelm_ has joined #openstack-trove | 20:16 | |
| *** schang has joined #openstack-trove | 20:16 | |
| *** laurelm_ is now known as laurelm | 20:16 | |
| *** exploreshaifali has quit IRC | 20:29 | |
| pmackinn | could i please get a Workflow on https://review.openstack.org/#/c/214195/ ? with sugar on it | 20:35 |
| vkmc | pmackinn, coming right up! | 20:38 |
| vkmc | pmackinn, would you like to add some fries to it? | 20:38 |
| pmackinn | vkmc, bless you...deliver me from this madness | 20:38 |
| *** openstackgerrit_ has quit IRC | 20:47 | |
| *** openstackgerrit_ has joined #openstack-trove | 20:47 | |
| *** openstackgerrit_ has quit IRC | 20:48 | |
| *** crobertsrh has quit IRC | 20:48 | |
| *** openstackgerrit_ has joined #openstack-trove | 20:48 | |
| cp16net | anyone know why in the zuul gate you see some patches say the tests are SKIPPED? | 20:49 |
| *** crobertsrh has joined #openstack-trove | 20:50 | |
| saurabhs1 | cp16net are you referring to root-enable patch in zuul. I think that poor thing is goona fail again to get merged. I think its gonna get in merge conflict with mongodb patch its like 2 patches above it in the queue. | 20:51 |
| saurabhs1 | they both change cfg.py, last line in mongodb section | 20:52 |
| *** crobertsrh is now known as _crobertsrh | 20:52 | |
| saurabhs1 | I bet its skipped those tests because it detected a merge conflict | 20:52 |
| *** openstackgerrit_ has quit IRC | 20:53 | |
| openstackgerrit | Merged openstack/python-troveclient: Root enablement for Vertica clusters/instances https://review.openstack.org/218548 | 20:59 |
| openstackgerrit | Merged openstack/trove: Mongodb Cluster Scaling https://review.openstack.org/205355 | 20:59 |
| cp16net | yeah that might happen | 21:00 |
| cp16net | saurabhs1: thats why i rebased all the patches last night on each other so there wouldnt be any conflicts on merge | 21:00 |
| openstackgerrit | Merged openstack/trove: Fixed redeclared CONF = cfg.CONF https://review.openstack.org/219209 | 21:00 |
| cp16net | yup | 21:00 |
| cp16net | merge conflict | 21:00 |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-trove | 21:01 | |
| cp16net | i got it rebased. | 21:02 |
| openstackgerrit | Craig Vyvial proposed openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 21:02 |
| *** imandhan has quit IRC | 21:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Updated from global requirements https://review.openstack.org/216572 | 21:08 |
| *** shayneburgess has quit IRC | 21:09 | |
| *** whydidyoustealmy has quit IRC | 21:11 | |
| *** shayneburgess has joined #openstack-trove | 21:14 | |
| *** arist has quit IRC | 21:15 | |
| *** whydidyoustealmy has joined #openstack-trove | 21:18 | |
| *** arist has joined #openstack-trove | 21:19 | |
| openstackgerrit | Merged openstack/trove-integration: Modifications for Fedora 21 support in redstack https://review.openstack.org/214195 | 21:50 |
| *** davideag_ has joined #openstack-trove | 21:53 | |
| davideag_ | hello, has anyone noticed cloud-images.ubuntu.com issue today, ssl error? | 21:54 |
| davideag_ | curl: (35) Unknown SSL protocol error in connection to cloud-images.ubuntu.com:443 | 21:54 |
| cp16net | umm i was waiting on building an image a min ago | 21:55 |
| cp16net | let me check... | 21:55 |
| cp16net | yeah i see an error with the service as well | 21:57 |
| cp16net | just trying again right now seem to work | 21:57 |
| cp16net | must have been a hickup | 21:57 |
| cp16net | davideag_: ^^ | 21:57 |
| pmackinn | vkmc, yaaaaaaaaaay! | 21:57 |
| vkmc | o.o | 21:58 |
| cp16net | pmackinn: w00t! got that in :-P | 21:58 |
| vkmc | pmackinn, wohooooo | 21:58 |
| * pmackinn retires from all things trove-ish, even pirate booty | 21:58 | |
| cp16net | lolz | 21:58 |
| cp16net | its crazy that the gate is almost down to single digits now | 22:00 |
| *** haomaiwang has quit IRC | 22:01 | |
| *** 17SADHVEF has joined #openstack-trove | 22:01 | |
| *** pmackinn_ has joined #openstack-trove | 22:03 | |
| *** pmackinn has quit IRC | 22:07 | |
| *** pmackinn_ is now known as pmackinn | 22:09 | |
| pmackinn | amrith, do we have a "novice" or "starter" bug tag or some such? | 22:24 |
| vkmc | pmackinn, we actually don't use it... low-hanging-fruit | 22:24 |
| pmackinn | k sold | 22:25 |
| vkmc | actually we use it, we have two https://bugs.launchpad.net/trove/+bugs?field.tag=low-hanging-fruit | 22:27 |
| cp16net | #agreed | 22:27 |
| davideag_ | @cp16net ok, thank you! | 22:27 |
| cp16net | davideag_: np | 22:28 |
| *** sabeen1 has quit IRC | 22:28 | |
| *** csoukup has quit IRC | 22:30 | |
| openstackgerrit | Sharika Pongubala proposed openstack/python-troveclient: Accepting network and availability zone for instances in cluster https://review.openstack.org/184349 | 22:38 |
| cp16net | vkmc: we need to get the gate working now that all the features are in | 22:45 |
| vkmc | cp16net, we do | 22:45 |
| cp16net | but i think its going to have to require a patch from nova | 22:45 |
| cp16net | this is the error i'm seeing still http://paste.openstack.org/show/445851/ | 22:45 |
| cp16net | from nova api | 22:46 |
| vkmc | this right here? Invalid input for field/attribute 0. | 22:47 |
| cp16net | ye | 22:47 |
| vkmc | weird | 22:47 |
| cp16net | i think SlickNik said it was related to this nova bug 1491325 | 22:48 |
| openstack | bug 1491325 in OpenStack Compute (nova) "nova api v2.1 does not allow to use autodetection of volume device path" [Critical,Fix committed] https://launchpad.net/bugs/1491325 - Assigned to Davanum Srinivas (DIMS) (dims-v) | 22:48 |
| vkmc | yes | 22:48 |
| vkmc | I was going to paste that | 22:48 |
| vkmc | hehe | 22:48 |
| cp16net | maybe it was a side effect | 22:48 |
| vkmc | what version of nova api do we use? | 22:48 |
| cp16net | i believe its 2.0 | 22:50 |
| cp16net | watch this merge come... | 22:50 |
| vkmc | hmm | 22:51 |
| vkmc | I'm not sure its the same thing | 22:52 |
| cp16net | i wasnt sure either i asked in the nova chan | 22:52 |
| cp16net | no reply | 22:52 |
| cp16net | probably because its late on friday after release | 22:52 |
| vkmc | yeah | 22:52 |
| vkmc | I'm inspecting the wsgi.py file | 22:53 |
| vkmc | for nova | 22:53 |
| cp16net | as soon as this last patch merges for trove i'm going to recheck the gate enable patch https://review.openstack.org/#/c/219972/ | 22:54 |
| cp16net | i'm sure it will fail but at least its showing the status of the gate after everything | 22:54 |
| vkmc | yeah | 22:55 |
| cp16net | this nova patch is taking for ever to merge tho... | 22:55 |
| vkmc | you are not feeling THAT curious then | 22:55 |
| cp16net | omg.... the anticipation is killing me | 22:55 |
| vkmc | to wait for it to merge on a Friday night | 22:55 |
| vkmc | hahahaha | 22:55 |
| cp16net | its the last patch from the features we needed | 22:56 |
| cp16net | and i couldnt get it to work on my vm | 22:57 |
| cp16net | but i'm about to head out for the weekend as well | 22:57 |
| cp16net | 3 day weekend | 22:57 |
| cp16net | i'm starting to think the gate took the weekend off from merging as well | 22:58 |
| cp16net | lol | 22:58 |
| openstackgerrit | Merged openstack/trove: Root enablement for Vertica clusters/instances https://review.openstack.org/196913 | 22:58 |
| cp16net | yay! | 22:58 |
| *** ashleighfarnham has quit IRC | 22:59 | |
| vkmc | :D | 23:00 |
| vkmc | so, you are moving forward the revert... right? | 23:01 |
| *** 17SADHVEF has quit IRC | 23:01 | |
| cp16net | alright i'm out yall. have a good one. | 23:01 |
| *** itlinux has joined #openstack-trove | 23:01 | |
| cp16net | i tried doing a recheck | 23:01 |
| cp16net | dont see it showing up yet in the queue | 23:01 |
| vkmc | k | 23:01 |
| *** haomaiwang has joined #openstack-trove | 23:01 | |
| vkmc | I'm going to be around for a while | 23:01 |
| vkmc | I'll check what happens | 23:01 |
| cp16net | ok thx! :) | 23:01 |
| vkmc | :) | 23:02 |
| cp16net | peace out! | 23:02 |
| vkmc | have a great weekend! recover some sleep | 23:02 |
| cp16net | w00t i see it. | 23:02 |
| cp16net | yeah for sure | 23:02 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Updated from global requirements https://review.openstack.org/216572 | 23:10 |
| *** itlinux has quit IRC | 23:25 | |
| *** itlinux has joined #openstack-trove | 23:27 | |
| *** IanGovett has joined #openstack-trove | 23:38 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!