| *** yogeshmehra has quit IRC | 00:00 | |
| *** paul_lodronio has left #openstack-trove | 00:00 | |
| *** rnirmal has quit IRC | 00:05 | |
| *** yogeshmehra has joined #openstack-trove | 00:06 | |
| *** tanisdl has quit IRC | 00:07 | |
| juice | what do you guys think about this as for getting consistent on imports | 00:12 |
|---|---|---|
| juice | https://gist.github.com/juicegit/6910847 | 00:12 |
| *** yogeshmehra has quit IRC | 00:13 | |
| juice | please comment without freaking out | 00:13 |
| juice | the items in never absolutely drive me crazy in python and for the most part I am finding the "from" syntax gets me queasy | 00:13 |
| hub_cap | juice: http://docs.openstack.org/developer/hacking/#human-alphabetical-order-examples | 00:14 |
| juice | this is good - thx hub_cap | 00:17 |
| juice | I want a heavier hammer though for those from imports but in a way the openstack guidance can provide that | 00:20 |
| *** matsuhashi has joined #openstack-trove | 00:29 | |
| *** tanisdl has joined #openstack-trove | 00:29 | |
| *** tanisdl has quit IRC | 00:37 | |
| hub_cap | ++ | 00:37 |
| *** adrian_otto has joined #openstack-trove | 00:48 | |
| *** amcrn[a] has quit IRC | 00:49 | |
| *** nosnos has joined #openstack-trove | 00:59 | |
| *** yogeshmehra has joined #openstack-trove | 01:07 | |
| *** jlucci has joined #openstack-trove | 01:08 | |
| *** yogeshmehra has quit IRC | 01:12 | |
| hub_cap | im pretty sure SlickNik left so he didnt have to review my code | 01:13 |
| SlickNik | nope, I'm still here and looking at it :P | 01:13 |
| hub_cap | https://review.openstack.org/#/c/48576/ | 01:13 |
| hub_cap | w00t | 01:14 |
| hub_cap | lets merge it yo! | 01:14 |
| *** jrodom has quit IRC | 01:18 | |
| *** erkules_ has joined #openstack-trove | 01:18 | |
| *** erkules has quit IRC | 01:21 | |
| hub_cap | SlickNik: got any questions so far? | 01:28 |
| SlickNik | nope, looks good. | 01:29 |
| SlickNik | taking it for a spin. :) | 01:29 |
| hub_cap | sweet! | 01:29 |
| hub_cap | dont forget to just . ~/devstack/openrc | 01:29 |
| hub_cap | and then take advantage of the awesome | 01:29 |
| hub_cap | bbiaf | 01:30 |
| imsplitbit | exit | 01:42 |
| imsplitbit | :) | 01:42 |
| imsplitbit | I'm out | 01:42 |
| *** erkules_ has quit IRC | 01:46 | |
| *** erkules_ has joined #openstack-trove | 01:47 | |
| *** amytron has joined #openstack-trove | 01:48 | |
| *** vipul is now known as vipul-away | 01:49 | |
| cp16net | workin hard or hardly workin | 01:50 |
| *** amytron has quit IRC | 01:50 | |
| *** amytron has joined #openstack-trove | 01:52 | |
| hub_cap | LOL imsplitbit | 01:57 |
| *** amytron has quit IRC | 01:57 | |
| hub_cap | its /wc | 01:57 |
| *** amytron has joined #openstack-trove | 01:58 | |
| *** vipul-away is now known as vipul | 01:59 | |
| *** amytron has quit IRC | 02:00 | |
| cp16net | hub_cap: you have the client workin | 02:01 |
| *** amytron has joined #openstack-trove | 02:01 | |
| hub_cap | there is one small issue cp16net | 02:02 |
| hub_cap | SlickNik: found wrt not using volumes | 02:02 |
| hub_cap | but hell ya its working | 02:02 |
| hub_cap | im trying to merge that shit | 02:02 |
| cp16net | ok i'll check it out | 02:02 |
| *** amytron has quit IRC | 02:04 | |
| hub_cap | plz do cuz itll be the client we use tomorrow ;) | 02:05 |
| *** amytron has joined #openstack-trove | 02:05 | |
| cp16net | heh | 02:06 |
| cp16net | ok | 02:06 |
| cp16net | hub_cap: did you have any ideas about the API path to get the default configuration of an instance? | 02:06 |
| cp16net | or what template is applied | 02:07 |
| cp16net | GET /instances/{id}/configuration | 02:07 |
| cp16net | does that sounds reasonable | 02:07 |
| hub_cap | def | 02:07 |
| hub_cap | u mean like if u havent generated one yet ya? | 02:07 |
| cp16net | i didnt see any feedback on the email | 02:07 |
| hub_cap | oh crap i forgot to reply | 02:07 |
| cp16net | :-P | 02:07 |
| *** yogeshmehra has joined #openstack-trove | 02:08 | |
| cp16net | this is like finding out the defaults are that are in my.cnf | 02:08 |
| cp16net | basically apply the template for the ram and use it | 02:09 |
| cp16net | rather than calling down to the guest to get it off the instance | 02:09 |
| hub_cap | ic | 02:10 |
| cp16net | but maybe thats what we should do so that its what is REALLY on the instance not just what the template "THINKS" is on the instance | 02:10 |
| hub_cap | well i think we should be applying the template to the instance maybe | 02:10 |
| *** jrodom has joined #openstack-trove | 02:10 | |
| hub_cap | wait we are ya? | 02:10 |
| cp16net | so thats what i have done | 02:10 |
| cp16net | but thinking about maybe we should call down to the guest and read the my.cnf | 02:10 |
| cp16net | i think that would be harder to do tho | 02:11 |
| cp16net | when we have the template versions that will have to be reterived from the guest updates | 02:11 |
| hub_cap | naw i think its ok for now to just erturn the template | 02:11 |
| cp16net | ok cool | 02:12 |
| cp16net | and the API path sounds sane :) | 02:12 |
| hub_cap | sane enough ;) | 02:14 |
| *** yogeshmehra has quit IRC | 02:14 | |
| *** mmcdaris has quit IRC | 02:15 | |
| cp16net | hub_cap: https://gist.github.com/cp16net/6911975 | 02:15 |
| cp16net | is that because of then entry point? | 02:16 |
| hub_cap | cp16net: | 02:18 |
| *** amytron has quit IRC | 02:18 | |
| hub_cap | trove | 02:18 |
| *** amytron has joined #openstack-trove | 02:18 | |
| hub_cap | trove-cli wont work anymore, unless you re-add it to setup.cfg | 02:20 |
| *** amytron has quit IRC | 02:20 | |
| hub_cap | cp16net: trove --help | 02:21 |
| openstackgerrit | SlickNik proposed a change to openstack/python-troveclient: Massive refactoring to the troveclient https://review.openstack.org/48576 | 02:21 |
| *** amytron has joined #openstack-trove | 02:21 | |
| hub_cap | <3 SlickNik | 02:21 |
| hub_cap | FRACK cp16net i just noticed your error... did i forget to remove cli | 02:24 |
| hub_cap | hmm no i moved it | 02:24 |
| cp16net | no i think that was because i changed from an old client | 02:24 |
| cp16net | the trove-cli didnt work | 02:24 |
| cp16net | oh... | 02:25 |
| cp16net | ok | 02:25 |
| hub_cap | lulz cp16net | 02:25 |
| cp16net | oh and i am using a different user by default right now | 02:28 |
| cp16net | so the sourcing of openrc didnt help | 02:28 |
| hub_cap | HAH | 02:29 |
| hub_cap | yes u wont see any of the other users stuff ;) | 02:29 |
| cp16net | ./redstack rd-client | 02:29 |
| hub_cap | oh ya we need to fix that | 02:29 |
| hub_cap | u dont need redstack anymore tho | 02:29 |
| hub_cap | just use the cli | 02:29 |
| cp16net | yeah thats what i am using | 02:29 |
| hub_cap | id prefer to just remove that redstack rd-client | 02:29 |
| hub_cap | its gross | 02:29 |
| hub_cap | ill leave the functionality as-is for rackers that use it that expect the old client to work | 02:30 |
| cp16net | i've grown close to it over time | 02:30 |
| cp16net | but yeah i know what you mean | 02:30 |
| hub_cap | ya i know dude :) | 02:30 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Massive refactoring to the troveclient https://review.openstack.org/48576 | 02:36 |
| *** amytron has quit IRC | 02:36 | |
| *** amytron has joined #openstack-trove | 02:37 | |
| cp16net | oh damn | 02:37 |
| hub_cap | yup | 02:39 |
| *** amytron has quit IRC | 02:41 | |
| hub_cap | gottta do it in the evening since its causing some blips | 02:42 |
| hub_cap | gate will fail till the other 2 projects are updated | 02:42 |
| *** amytron has joined #openstack-trove | 02:42 | |
| *** amytron has quit IRC | 02:44 | |
| *** amytron has joined #openstack-trove | 02:45 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/python-troveclient: Tweak pbr conf https://review.openstack.org/50805 | 02:47 |
| *** amytron has quit IRC | 02:49 | |
| *** amytron has joined #openstack-trove | 02:50 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Tweak pbr conf https://review.openstack.org/50805 | 03:02 |
| *** amytron has quit IRC | 03:02 | |
| *** amytron has joined #openstack-trove | 03:03 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/python-troveclient: Add support for a service type env var https://review.openstack.org/50807 | 03:04 |
| *** amytron has quit IRC | 03:05 | |
| *** amytron has joined #openstack-trove | 03:07 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Add support for a service type env var https://review.openstack.org/50807 | 03:08 |
| *** amytron has quit IRC | 03:08 | |
| *** amytron has joined #openstack-trove | 03:09 | |
| *** amytron has quit IRC | 03:12 | |
| *** amytron has joined #openstack-trove | 03:13 | |
| *** amytron has quit IRC | 03:16 | |
| *** amytron has joined #openstack-trove | 03:17 | |
| *** jrodom has quit IRC | 03:21 | |
| *** amytron has quit IRC | 03:21 | |
| *** amytron has joined #openstack-trove | 03:22 | |
| hub_cap | https://pypi.python.org/pypi/python-troveclient/ | 03:22 |
| hub_cap | 1.0.0 horray | 03:22 |
| *** amytron has quit IRC | 03:24 | |
| *** amytron has joined #openstack-trove | 03:25 | |
| *** amytron has quit IRC | 03:30 | |
| *** amytron has joined #openstack-trove | 03:32 | |
| *** amytron has quit IRC | 03:33 | |
| *** amytron has joined #openstack-trove | 03:34 | |
| *** amytron has quit IRC | 03:35 | |
| *** amytron has joined #openstack-trove | 03:37 | |
| *** amytron has quit IRC | 03:43 | |
| *** amytron has joined #openstack-trove | 03:44 | |
| *** amytron has quit IRC | 03:47 | |
| *** amytron has joined #openstack-trove | 03:48 | |
| hub_cap | juice: http://logs.openstack.org/27/50527/1/gate/gate-trove-python27/7da6794/console.html | 03:57 |
| *** vipul has quit IRC | 03:57 | |
| hub_cap | 2013-10-10 03:48:43.661 | AssertionError: 2013-10-10T03:48:00Z != 2013-10-10T03:48:02Z | 03:57 |
| *** haomaiwang has quit IRC | 03:57 | |
| *** vipul has joined #openstack-trove | 03:59 | |
| *** amytron has quit IRC | 03:59 | |
| *** amytron has joined #openstack-trove | 04:00 | |
| hub_cap | FRAAAAACJ | 04:02 |
| hub_cap | K | 04:02 |
| hub_cap | 2nd time | 04:02 |
| hub_cap | 2013-10-10 04:00:47.047 | AssertionError: 2013-10-10T04:00:29Z != 2013-10-10T04:00:30Z | 04:02 |
| *** amytron has quit IRC | 04:04 | |
| *** amytron has joined #openstack-trove | 04:05 | |
| *** grapex has quit IRC | 04:06 | |
| *** amytron has quit IRC | 04:06 | |
| cp16net | this alchemy stuff is goofy | 04:08 |
| *** amytron has joined #openstack-trove | 04:08 | |
| hub_cap | lol stop tryign to make gold | 04:08 |
| openstackgerrit | A change was merged to openstack/trove: Modifying tests to use the compat client https://review.openstack.org/50527 | 04:09 |
| *** yogeshmehra has joined #openstack-trove | 04:09 | |
| hub_cap | finally | 04:10 |
| cp16net | i feel like i am trying like hell to shine this thing | 04:11 |
| *** amytron has quit IRC | 04:11 | |
| hub_cap | u notice something? | 04:12 |
| hub_cap | do u see any other fkeys and shit | 04:12 |
| cp16net | nope | 04:12 |
| hub_cap | ;) | 04:12 |
| cp16net | heh | 04:12 |
| *** amytron has joined #openstack-trove | 04:13 | |
| *** amytron has quit IRC | 04:16 | |
| *** yogeshmehra has quit IRC | 04:17 | |
| *** jlucci has quit IRC | 04:17 | |
| *** amytron has joined #openstack-trove | 04:18 | |
| *** adrian_otto has quit IRC | 04:20 | |
| *** amytron has quit IRC | 04:22 | |
| *** amytron has joined #openstack-trove | 04:24 | |
| *** amytron has quit IRC | 04:31 | |
| *** amytron has joined #openstack-trove | 04:32 | |
| *** amytron has quit IRC | 04:34 | |
| *** amytron has joined #openstack-trove | 04:35 | |
| *** amytron has quit IRC | 04:39 | |
| *** amytron has joined #openstack-trove | 04:40 | |
| *** yogeshmehra has joined #openstack-trove | 04:48 | |
| *** amytron has quit IRC | 04:48 | |
| *** amytron has joined #openstack-trove | 04:49 | |
| *** yogeshmehra has quit IRC | 04:54 | |
| *** amytron has quit IRC | 04:54 | |
| *** amytron has joined #openstack-trove | 04:55 | |
| *** KapilSaxena has joined #openstack-trove | 04:56 | |
| *** amytron has quit IRC | 04:56 | |
| *** amytron has joined #openstack-trove | 04:58 | |
| *** amytron has quit IRC | 05:04 | |
| *** amytron has joined #openstack-trove | 05:05 | |
| *** SushilKM__ has joined #openstack-trove | 05:06 | |
| *** grapex has joined #openstack-trove | 05:07 | |
| *** amytron has quit IRC | 05:07 | |
| *** amytron has joined #openstack-trove | 05:08 | |
| *** amytron has quit IRC | 05:09 | |
| *** amytron has joined #openstack-trove | 05:11 | |
| *** SushilKM__ has quit IRC | 05:14 | |
| *** amytron has quit IRC | 05:15 | |
| *** SushilKM__ has joined #openstack-trove | 05:16 | |
| *** amytron has joined #openstack-trove | 05:16 | |
| *** grapex has quit IRC | 05:17 | |
| *** amytron has quit IRC | 05:19 | |
| *** amytron has joined #openstack-trove | 05:20 | |
| openstackgerrit | A change was merged to openstack/trove-integration: Adding changes to comply with the compat client. https://review.openstack.org/50677 | 05:27 |
| *** amytron has quit IRC | 05:27 | |
| *** amytron has joined #openstack-trove | 05:28 | |
| *** haomaiwang has joined #openstack-trove | 05:30 | |
| *** amytron has quit IRC | 05:30 | |
| *** amytron has joined #openstack-trove | 05:31 | |
| *** amytron has quit IRC | 05:35 | |
| *** amytron has joined #openstack-trove | 05:36 | |
| *** amytron has quit IRC | 05:40 | |
| *** amytron has joined #openstack-trove | 05:41 | |
| * hub_cap does a dance | 05:43 | |
| *** amytron has quit IRC | 05:43 | |
| hub_cap | merged and tests are back online | 05:43 |
| *** amytron has joined #openstack-trove | 05:44 | |
| * SlickNik turns up the music | 05:47 | |
| *** amytron has quit IRC | 05:47 | |
| SlickNik | hell yea | 05:47 |
| hub_cap | most def | 05:48 |
| *** amytron has joined #openstack-trove | 05:48 | |
| *** mmcdaris has joined #openstack-trove | 05:49 | |
| *** amytron has quit IRC | 05:51 | |
| *** amytron has joined #openstack-trove | 05:52 | |
| *** dmakogon has joined #openstack-trove | 05:52 | |
| *** amytron has quit IRC | 05:54 | |
| *** yogeshmehra has joined #openstack-trove | 05:55 | |
| *** amytron has joined #openstack-trove | 05:55 | |
| *** yogeshmehra has quit IRC | 05:59 | |
| *** amytron has quit IRC | 05:59 | |
| *** amytron has joined #openstack-trove | 06:00 | |
| openstackgerrit | Arata Notsu proposed a change to openstack/python-troveclient: Enable shell to resize and restart https://review.openstack.org/50831 | 06:02 |
| *** yogeshmehra has joined #openstack-trove | 06:05 | |
| *** SushilKM__ has quit IRC | 06:05 | |
| *** SushilKM__ has joined #openstack-trove | 06:07 | |
| *** amytron has quit IRC | 06:07 | |
| *** grapex has joined #openstack-trove | 06:07 | |
| *** amytron has joined #openstack-trove | 06:08 | |
| *** yogeshme_ has joined #openstack-trove | 06:08 | |
| *** ashestakov has joined #openstack-trove | 06:09 | |
| *** yogeshmehra has quit IRC | 06:10 | |
| hub_cap | dammit i cant believe i missed that^ ^ ^ | 06:11 |
| hub_cap | :) | 06:11 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Enable shell to resize and restart https://review.openstack.org/50831 | 06:11 |
| openstackgerrit | Arata Notsu proposed a change to openstack/python-troveclient: Correct handling args.backup in do_create https://review.openstack.org/50834 | 06:11 |
| *** grapex has quit IRC | 06:12 | |
| *** SushilKM__ has quit IRC | 06:18 | |
| *** amytron has quit IRC | 06:18 | |
| *** amytron has joined #openstack-trove | 06:18 | |
| openstackgerrit | SlickNik proposed a change to openstack/python-troveclient: Fixed bug with showing backups https://review.openstack.org/50837 | 06:19 |
| *** amytron has quit IRC | 06:21 | |
| *** amytron has joined #openstack-trove | 06:22 | |
| *** vipul has quit IRC | 06:23 | |
| *** amytron has quit IRC | 06:23 | |
| *** vipul has joined #openstack-trove | 06:24 | |
| *** amytron has joined #openstack-trove | 06:25 | |
| hub_cap | heh how can there be bugs in the client?!?!? | 06:29 |
| hub_cap | ;) | 06:29 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Correct handling args.backup in do_create https://review.openstack.org/50834 | 06:29 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Fixed bug with showing backups https://review.openstack.org/50837 | 06:31 |
| *** amytron has quit IRC | 06:31 | |
| *** amytron has joined #openstack-trove | 06:32 | |
| hub_cap | looks like ill be pushing 1.0.1 soon ;) | 06:34 |
| SlickNik | heh, might be a good idea. :) | 06:34 |
| hub_cap | ill let it bake for another ~24 hrs id think | 06:35 |
| dmakogon | wow, goodmorning | 06:35 |
| SlickNik | good morning dmakogon | 06:35 |
| dmakogon | glad to see you all up so early) | 06:35 |
| hub_cap | nope im going to bed | 06:35 |
| hub_cap | its late ;) | 06:36 |
| SlickNik | Same here | 06:36 |
| dmakogon | ag, timezones, always confuses me | 06:36 |
| dmakogon | SLickNik: could you take a look at several reviews . | 06:36 |
| dmakogon | ? | 06:36 |
| hub_cap | hahaah we are goijg to bed dmakogon | 06:37 |
| hub_cap | *going | 06:37 |
| dmakogon | but you still here )) | 06:37 |
| * hub_cap leaves | 06:37 | |
| dmakogon | i just got up, sun shines bright | 06:37 |
| hub_cap | :) peace everyone | 06:38 |
| dmakogon | okay | 06:38 |
| hub_cap | dmakogon fyi gate is broken | 06:38 |
| dmakogon | gate is broken ?? | 06:38 |
| hub_cap | it might jut be stable/grizzly | 06:38 |
| hub_cap | not our fault | 06:38 |
| hub_cap | but upstream like nova | 06:38 |
| dmakogon | sh*t | 06:38 |
| hub_cap | keep watching (check your email) | 06:38 |
| hub_cap | also rebase all your changes | 06:38 |
| hub_cap | the new client is merged | 06:38 |
| dmakogon | ok | 06:39 |
| hub_cap | <3 gnite | 06:39 |
| dmakogon | i have no reviews related to client, so, i'm ok)) | 06:39 |
| *** amytron has quit IRC | 06:40 | |
| *** amytron has joined #openstack-trove | 06:41 | |
| *** vipul is now known as vipul-away | 06:43 | |
| *** amytron has quit IRC | 06:43 | |
| *** mmcdaris has quit IRC | 06:43 | |
| *** amytron has joined #openstack-trove | 06:44 | |
| *** amytron has quit IRC | 06:45 | |
| *** amytron has joined #openstack-trove | 06:47 | |
| *** amytron has quit IRC | 06:49 | |
| *** amytron has joined #openstack-trove | 06:50 | |
| *** debasish has joined #openstack-trove | 06:50 | |
| *** mmcdaris has joined #openstack-trove | 06:52 | |
| *** amytron has quit IRC | 06:53 | |
| *** mmcdaris has quit IRC | 06:54 | |
| *** erkules_ is now known as erkules | 06:54 | |
| *** amytron has joined #openstack-trove | 06:55 | |
| SlickNik | dmakogon: just reviewed a couple changes. Most of the others are waiting for changes based on comments. | 06:57 |
| SlickNik | Will take another look tomorrow. | 06:57 |
| SlickNik | Heading out; see you tomorrow. | 06:58 |
| SlickNik | G'night! | 06:58 |
| *** vipul-away is now known as vipul | 07:06 | |
| *** amytron has quit IRC | 07:06 | |
| *** amytron has joined #openstack-trove | 07:07 | |
| *** grapex has joined #openstack-trove | 07:08 | |
| *** amytron has quit IRC | 07:09 | |
| *** amytron has joined #openstack-trove | 07:11 | |
| *** grapex has quit IRC | 07:12 | |
| *** amytron has quit IRC | 07:12 | |
| *** amytron has joined #openstack-trove | 07:13 | |
| *** amytron has quit IRC | 07:17 | |
| *** amytron has joined #openstack-trove | 07:19 | |
| *** vipul is now known as vipul-away | 07:22 | |
| *** amytron has quit IRC | 07:22 | |
| *** amytron has joined #openstack-trove | 07:23 | |
| *** vipul-away is now known as vipul | 07:30 | |
| *** amytron has quit IRC | 07:30 | |
| *** amytron has joined #openstack-trove | 07:31 | |
| *** amytron has quit IRC | 07:32 | |
| *** amytron has joined #openstack-trove | 07:34 | |
| *** dafter has joined #openstack-trove | 07:37 | |
| *** amytron has quit IRC | 07:38 | |
| *** amytron has joined #openstack-trove | 07:39 | |
| *** yogeshme_ has quit IRC | 07:40 | |
| *** amytron has quit IRC | 07:41 | |
| *** amytron has joined #openstack-trove | 07:42 | |
| *** SushilKM__ has joined #openstack-trove | 07:46 | |
| *** amytron has quit IRC | 07:46 | |
| *** amytron has joined #openstack-trove | 07:47 | |
| *** amytron has quit IRC | 07:49 | |
| *** amytron has joined #openstack-trove | 07:50 | |
| *** amytron has quit IRC | 08:07 | |
| *** amytron has joined #openstack-trove | 08:08 | |
| *** grapex has joined #openstack-trove | 08:08 | |
| *** dafter has quit IRC | 08:09 | |
| *** grapex has quit IRC | 08:13 | |
| *** amytron has quit IRC | 08:13 | |
| *** ashestakov has quit IRC | 08:14 | |
| *** amytron has joined #openstack-trove | 08:14 | |
| *** krow has joined #openstack-trove | 08:25 | |
| *** amytron has quit IRC | 08:25 | |
| *** ashestakov has joined #openstack-trove | 08:26 | |
| *** amytron has joined #openstack-trove | 08:26 | |
| *** amytron has quit IRC | 08:32 | |
| *** amytron has joined #openstack-trove | 08:33 | |
| *** yogeshmehra has joined #openstack-trove | 08:40 | |
| *** amytron has quit IRC | 08:42 | |
| *** amytron has joined #openstack-trove | 08:43 | |
| *** dmakogon is now known as dmakogon_home | 08:43 | |
| *** yogeshmehra has quit IRC | 08:45 | |
| *** amytron has quit IRC | 08:48 | |
| *** amytron has joined #openstack-trove | 08:48 | |
| *** KapilSaxena has quit IRC | 08:48 | |
| *** amytron has quit IRC | 08:50 | |
| *** yogeshmehra has joined #openstack-trove | 08:51 | |
| *** amytron has joined #openstack-trove | 08:51 | |
| *** krow has quit IRC | 08:51 | |
| *** amytron has quit IRC | 08:53 | |
| *** amytron has joined #openstack-trove | 08:54 | |
| *** yogeshmehra has quit IRC | 08:55 | |
| *** amytron has quit IRC | 08:55 | |
| *** amytron has joined #openstack-trove | 08:57 | |
| *** amytron has quit IRC | 09:05 | |
| *** nosnos has quit IRC | 09:06 | |
| *** amytron has joined #openstack-trove | 09:06 | |
| *** grapex has joined #openstack-trove | 09:09 | |
| *** amytron has quit IRC | 09:09 | |
| *** dmakogon_home has quit IRC | 09:11 | |
| *** amytron has joined #openstack-trove | 09:11 | |
| *** amytron has quit IRC | 09:13 | |
| *** grapex has quit IRC | 09:13 | |
| *** amytron has joined #openstack-trove | 09:14 | |
| *** amytron has quit IRC | 09:18 | |
| *** amytron has joined #openstack-trove | 09:19 | |
| *** dafter has joined #openstack-trove | 09:20 | |
| *** amytron has quit IRC | 09:22 | |
| *** Kapil-Away has joined #openstack-trove | 09:22 | |
| *** amytron has joined #openstack-trove | 09:23 | |
| *** dafter has quit IRC | 09:25 | |
| *** amytron has quit IRC | 09:25 | |
| *** debasish has quit IRC | 10:03 | |
| *** dafter has joined #openstack-trove | 10:05 | |
| *** grapex has joined #openstack-trove | 10:09 | |
| *** dmakogon_work is now known as dmakogon | 10:13 | |
| *** grapex has quit IRC | 10:14 | |
| *** ppenjoy has joined #openstack-trove | 10:14 | |
| *** SushilKM__ has quit IRC | 10:43 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: GA refactoring https://review.openstack.org/50686 | 10:55 |
| *** jaishanker has joined #openstack-trove | 11:10 | |
| *** grapex has joined #openstack-trove | 11:10 | |
| *** grapex has quit IRC | 11:15 | |
| *** dafter has quit IRC | 11:25 | |
| *** yogeshmehra has joined #openstack-trove | 11:52 | |
| *** yogeshmehra has quit IRC | 11:58 | |
| *** pdmars has joined #openstack-trove | 12:03 | |
| *** ppenjoy has left #openstack-trove | 12:03 | |
| *** Kapil-Away has quit IRC | 12:03 | |
| *** dafter has joined #openstack-trove | 12:05 | |
| *** dafter has quit IRC | 12:05 | |
| *** dafter has joined #openstack-trove | 12:05 | |
| *** grapex has joined #openstack-trove | 12:10 | |
| dmakogon | hub_cap: i know you told that gate is broken and you told that i possibly nova upstream issue, but take a look at this one, https://rdjenkins.dyndns.org/job/Trove-Gate/1217/console - it tells that there is a problem with troveclient | 12:13 |
| *** grapex has quit IRC | 12:15 | |
| *** dafter has quit IRC | 12:29 | |
| dmakogon | hub_cap: oh, i figured out, my review head if behind one commit of master and troveclient got an update | 12:46 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: GA refactoring https://review.openstack.org/50686 | 12:48 |
| *** yogeshmehra has joined #openstack-trove | 12:54 | |
| *** yogeshmehra has quit IRC | 13:00 | |
| *** adrian_otto has joined #openstack-trove | 13:06 | |
| *** adrian_otto has quit IRC | 13:07 | |
| *** jrodom has joined #openstack-trove | 13:08 | |
| *** grapex has joined #openstack-trove | 13:11 | |
| *** grapex has quit IRC | 13:15 | |
| *** robertmyers has joined #openstack-trove | 13:17 | |
| *** jlucci has joined #openstack-trove | 13:22 | |
| *** demorris has joined #openstack-trove | 13:26 | |
| *** radez_g0n3 is now known as radez | 13:27 | |
| *** dafter has joined #openstack-trove | 13:37 | |
| *** jlucci has quit IRC | 13:38 | |
| *** amytron has joined #openstack-trove | 14:03 | |
| *** Barker has joined #openstack-trove | 14:13 | |
| *** KapilSaxena has joined #openstack-trove | 14:17 | |
| *** dukhlov has quit IRC | 14:23 | |
| *** dukhlov has joined #openstack-trove | 14:23 | |
| *** jasonb365 has joined #openstack-trove | 14:27 | |
| *** datsun180b has joined #openstack-trove | 14:28 | |
| *** matsuhashi has quit IRC | 14:30 | |
| *** demorris has quit IRC | 14:30 | |
| *** matsuhashi has joined #openstack-trove | 14:31 | |
| *** demorris has joined #openstack-trove | 14:31 | |
| *** matsuhashi has quit IRC | 14:35 | |
| *** jlucci has joined #openstack-trove | 14:42 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Update h.template with default resources https://review.openstack.org/50620 | 14:44 |
| *** grapex has joined #openstack-trove | 14:45 | |
| *** grapex has quit IRC | 14:45 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Update statuses on GA timeout https://review.openstack.org/45723 | 14:46 |
| *** grapex has joined #openstack-trove | 14:46 | |
| *** ioggstream has joined #openstack-trove | 14:49 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Security groups workflow update https://review.openstack.org/50944 | 14:50 |
| *** adrian_otto has joined #openstack-trove | 14:52 | |
| *** yogeshmehra has joined #openstack-trove | 14:56 | |
| *** yidclare has joined #openstack-trove | 15:00 | |
| *** yogeshmehra has quit IRC | 15:01 | |
| *** adrian_otto1 has joined #openstack-trove | 15:05 | |
| *** jmontemayor has quit IRC | 15:05 | |
| *** adrian_otto has quit IRC | 15:06 | |
| *** adrian_otto1 has quit IRC | 15:08 | |
| *** paul_lodronio has joined #openstack-trove | 15:08 | |
| *** adrian_otto has joined #openstack-trove | 15:09 | |
| *** jasonb365 has quit IRC | 15:10 | |
| *** jasonb365 has joined #openstack-trove | 15:11 | |
| *** tanisdl has joined #openstack-trove | 15:12 | |
| *** adrian_otto has quit IRC | 15:21 | |
| dmakogon | hub_cap: please, restart this build https://rdjenkins.dyndns.org/job/Trove-Gate/1221 | 15:23 |
| openstackgerrit | Illia Khudoshyn proposed a change to openstack/trove: Update trove-guestagent.conf.sample https://review.openstack.org/50955 | 15:26 |
| *** jlucci has quit IRC | 15:27 | |
| *** adrian_otto has joined #openstack-trove | 15:29 | |
| *** shakayumi has joined #openstack-trove | 15:29 | |
| *** Nate2 has quit IRC | 15:34 | |
| *** Nate2 has joined #openstack-trove | 15:35 | |
| *** Nate3 has joined #openstack-trove | 15:35 | |
| *** Nate2 has quit IRC | 15:35 | |
| *** shakayumi has quit IRC | 15:37 | |
| *** jcru has joined #openstack-trove | 15:37 | |
| *** KapilSaxena has quit IRC | 15:45 | |
| *** Nate3 has quit IRC | 15:49 | |
| *** jmontemayor has joined #openstack-trove | 15:49 | |
| *** adrian_otto has quit IRC | 15:50 | |
| openstackgerrit | Robert Myers proposed a change to openstack/trove: Fixing restart tests https://review.openstack.org/50971 | 15:51 |
| *** Nate2 has joined #openstack-trove | 15:54 | |
| *** dafter has quit IRC | 15:57 | |
| *** adrian_otto has joined #openstack-trove | 16:00 | |
| *** dafter has joined #openstack-trove | 16:02 | |
| *** adrian_otto has quit IRC | 16:03 | |
| openstackgerrit | Robert Myers proposed a change to openstack/trove: Fixing restart tests https://review.openstack.org/50971 | 16:06 |
| *** dukhlov has quit IRC | 16:06 | |
| *** dafter has quit IRC | 16:06 | |
| *** dafter has joined #openstack-trove | 16:06 | |
| *** jasonb365 has quit IRC | 16:07 | |
| *** jodom has joined #openstack-trove | 16:08 | |
| *** jodom has joined #openstack-trove | 16:09 | |
| *** jasonb365 has joined #openstack-trove | 16:09 | |
| *** jrodom has quit IRC | 16:11 | |
| *** yidclare has quit IRC | 16:12 | |
| *** jasonb365 has quit IRC | 16:14 | |
| *** KapilSaxena has joined #openstack-trove | 16:15 | |
| *** jasonb365 has joined #openstack-trove | 16:17 | |
| *** jasonb365 has quit IRC | 16:27 | |
| dmakogon | hub_cap: https://rdjenkins.dyndns.org/job/Trove-Gate/1223 - somehow it got failed at image building | 16:29 |
| *** KapilSaxena_ has joined #openstack-trove | 16:29 | |
| *** KapilSaxena has quit IRC | 16:31 | |
| *** rnirmal has joined #openstack-trove | 16:41 | |
| *** dafter has quit IRC | 16:42 | |
| *** jodom has quit IRC | 16:46 | |
| *** yidclare has joined #openstack-trove | 16:51 | |
| *** vipul is now known as vipul-away | 16:51 | |
| *** adrian_otto has joined #openstack-trove | 16:55 | |
| *** KapilSaxena_ has quit IRC | 16:57 | |
| *** tanisdl has quit IRC | 16:59 | |
| *** yogesh has joined #openstack-trove | 17:07 | |
| *** yogeshmehra has joined #openstack-trove | 17:07 | |
| *** yogesh has quit IRC | 17:08 | |
| *** SushilKM has joined #openstack-trove | 17:10 | |
| *** yogesh has joined #openstack-trove | 17:13 | |
| *** ioggstream has quit IRC | 17:15 | |
| *** yogesh has quit IRC | 17:15 | |
| *** dafter has joined #openstack-trove | 17:26 | |
| david-lyle | wanted to give a heads up... The 1.0 release of python-troveclient broke the Horizon integration in Havana. I am trying to temporarily cap the version of python-troveclient < 1.0 in openstack/requirements and horizon/requirements until Havana ships | 17:31 |
| david-lyle | Then would remove the cap | 17:31 |
| SlickNik | david-lyle: Thanks for the heads up. | 17:35 |
| *** vipul-away is now known as vipul | 17:36 | |
| *** dafter has quit IRC | 17:42 | |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Removing lxml/httplib2 depends https://review.openstack.org/50994 | 17:50 |
| hub_cap | david-lyle: poop.. want us to push a fast review to fix it? its likely just to move from troveclient.* to troveclient.v1.* | 17:56 |
| david-lyle | hub_cap: we're at RC2 and trying to get it out today, I think the safest thing for Havana is cap the version and update the integration in Icehouse. | 17:58 |
| david-lyle | we can also backport the fix to Havana at the next stable release | 17:59 |
| *** pdmars has quit IRC | 17:59 | |
| hub_cap | k | 17:59 |
| *** pdmars has joined #openstack-trove | 17:59 | |
| hub_cap | thats likely smartest anyway due to the fact that its an update to the client during requirement freeze | 17:59 |
| imsplitbit | https://wiki.openstack.org/wiki/Trove-Replication-And-Clustering-API | 18:00 |
| imsplitbit | errbuddy ^^^ | 18:00 |
| imsplitbit | vipul: ^^^ | 18:00 |
| imsplitbit | hub_cap: ^^^ | 18:00 |
| imsplitbit | isviridov: ^^^^ | 18:00 |
| hub_cap | imsplitbit: ^^^^^ | 18:00 |
| imsplitbit | start reading bizzle | 18:00 |
| *** Barker has quit IRC | 18:01 | |
| hub_cap | robertmyers: hey, horizon+db support is broken w/. the new client... can u take a look @ it? | 18:01 |
| robertmyers | hub_cap: I can | 18:02 |
| robertmyers | hub_cap: did you add support for token auth? | 18:03 |
| robertmyers | that might simplify the horizon api | 18:03 |
| *** jmontemayor has quit IRC | 18:03 | |
| *** pdmars has quit IRC | 18:04 | |
| hub_cap | if by "add support for token auth" u mean make it behave like the other clients, then yes | 18:05 |
| david-lyle | robertmyers: mostly the way the client is instantiated needs to be updated, but the test data set up is also broken | 18:05 |
| hub_cap | yes the client.Dbaas is no more | 18:05 |
| hub_cap | it should look _exactly_ like cinder/nova clients | 18:05 |
| *** jmontemayor has joined #openstack-trove | 18:05 | |
| robertmyers | ok, that should help | 18:05 |
| *** adrian_otto has quit IRC | 18:06 | |
| robertmyers | I had to hack it before :) | 18:06 |
| *** amcrn[a] has joined #openstack-trove | 18:06 | |
| *** amcrn[a] is now known as amcrn | 18:06 | |
| hub_cap | robertmyers: dude im sure | 18:07 |
| *** pdmars has joined #openstack-trove | 18:07 | |
| hub_cap | now they play in perfect harmony | 18:07 |
| *** Barker has joined #openstack-trove | 18:07 | |
| *** adrian_otto has joined #openstack-trove | 18:09 | |
| hub_cap | hello amcrn :) | 18:09 |
| amcrn | hola | 18:09 |
| hub_cap | new client has landed, feel free to take it for a spin | 18:09 |
| hub_cap | weve found a few bugs, so pull from src plz | 18:09 |
| amcrn | definitely smiled once I saw it merge :) | 18:09 |
| hub_cap | id like to get a decent ammt of people to run it thru the paces so that we can push a 1.0.1 | 18:09 |
| amcrn | will do | 18:09 |
| hub_cap | <3 | 18:12 |
| vipul | hub_cap: is the order of verb+noun configurable in the client? | 18:15 |
| vipul | at least in some other clients.. | 18:15 |
| vipul | i've seen things like 'instance-create' | 18:15 |
| vipul | instead of 'create-instance' | 18:15 |
| vipul | allows you to group operations per noun a bit better | 18:15 |
| *** arborism has joined #openstack-trove | 18:15 | |
| datsun180b | that does read better tbh | 18:17 |
| *** adrian_otto has quit IRC | 18:17 | |
| *** amcrn has quit IRC | 18:18 | |
| hub_cap | it is | 18:19 |
| hub_cap | but the other clients dont work like that do they? | 18:19 |
| *** arborism is now known as amcrn | 18:20 | |
| vipul | nova client sure does | 18:20 |
| datsun180b | for organization sake i'm on the resource-action / noun-verb side though | 18:20 |
| vipul | floating-ip-create Allocate a floating IP for the current tenant. | 18:20 |
| vipul | floating-ip-delete De-allocate a floating IP. | 18:20 |
| vipul | floating-ip-list List floating ips for this tenant. | 18:20 |
| kevinconway | nova client uses "image-list" and "flavor-list" | 18:20 |
| vipul | aggregate-details Show details of the specified aggregate. | 18:20 |
| vipul | aggregate-list Print a list of all aggregates. | 18:20 |
| hub_cap | sure we can change em | 18:20 |
| hub_cap | submit a review.. thats a good catch :) | 18:21 |
| kevinconway | won't you have to rev the major API version? | 18:21 |
| hub_cap | lol no kevinconway | 18:21 |
| hub_cap | cuz no one has used it yet | 18:21 |
| hub_cap | and its only a cli | 18:21 |
| hub_cap | shhhhh | 18:21 |
| datsun180b | let's also use different verbs for each resource to "reduce" confusion | 18:22 |
| *** dafter has joined #openstack-trove | 18:22 | |
| *** dafter has joined #openstack-trove | 18:22 | |
| hub_cap | lets push it rather quickly tho | 18:22 |
| hub_cap | what do u mean datsun180b? | 18:22 |
| kevinconway | instead of list use "enumerate" | 18:22 |
| datsun180b | i was joking, i meant use verbs other than "create" or "list" | 18:22 |
| hub_cap | show, list, create, delete are used by all | 18:22 |
| hub_cap | dorks | 18:22 |
| hub_cap | lol | 18:22 |
| hub_cap | ok ill change quickly | 18:22 |
| vipul | kevinconway: ++ | 18:22 |
| kevinconway | do any of the other CLI's have sound effects? | 18:23 |
| kevinconway | that sounds like a feature we need | 18:23 |
| datsun180b | instance-zap database-pewpewpew | 18:23 |
| *** adrian_otto has joined #openstack-trove | 18:24 | |
| datsun180b | i'd love to build a CLI that uses NLP so you can do things like "trove-client hey can i have the biggest instance i'm allowed and call it dave please" | 18:24 |
| robertmyers | datsun180b: lol | 18:24 |
| hub_cap | ok give me ~5 min to push a review | 18:25 |
| datsun180b | "trove-client what are my instances; trove-client okay delete the oldest one" | 18:25 |
| datsun180b | but who am i kidding, who uses the CLI | 18:26 |
| kevinconway | trove-client open the pod bay doors | 18:26 |
| datsun180b | 400 | 18:26 |
| datsun180b | Bad request, Dave. | 18:26 |
| *** jasonb365 has joined #openstack-trove | 18:27 | |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Removing lxml/httplib2 depends https://review.openstack.org/50994 | 18:32 |
| hub_cap | so do i do | 18:38 |
| hub_cap | flavor-list | 18:38 |
| hub_cap | flavor-show | 18:38 |
| hub_cap | or | 18:38 |
| hub_cap | flavorSSSS-list | 18:38 |
| hub_cap | flavor-show | 18:38 |
| hub_cap | as in, do i add the letter s? | 18:38 |
| *** krow1 has joined #openstack-trove | 18:38 | |
| *** SushilKM has quit IRC | 18:40 | |
| hub_cap | no S | 18:41 |
| amcrn | hub_cap: i re-joined the network in the midst of kevinconway+you+datsun180b's comments, but it looks like your flipping to backup-create/backup-delete vs. delete-backup/create-backup yes? | 18:44 |
| amcrn | you're* | 18:45 |
| datsun180b | hub_cap: no S | 18:45 |
| datsun180b | amcrn: looks like it | 18:45 |
| amcrn | ok cool :) | 18:45 |
| hub_cap | ok almost done w/ it | 18:46 |
| hub_cap | also helping demorris w auth+rax | 18:46 |
| hub_cap | amcrn: datsun180b vipul kevinconway etc etc https://gist.github.com/hub-cap/6923440 | 18:48 |
| hub_cap | once over it, and ill push it | 18:48 |
| kevinconway | hub_cap: that looks pretty good | 18:49 |
| vipul | hub_cap: Looks good to me | 18:49 |
| kevinconway | i think nova uses "boot" rather than "create" for a new instance if that is important | 18:49 |
| kevinconway | but i like create better | 18:49 |
| hub_cap | crap limit- limits | 18:49 |
| hub_cap | ill change to limit | 18:49 |
| datsun180b | "boot" can also mean "to vomit" | 18:49 |
| hub_cap | ya screw boot | 18:49 |
| vipul | you have limits-list.. looks fine | 18:50 |
| datsun180b | wait why is "list" a standalone command | 18:50 |
| kevinconway | nova uses plural | 18:50 |
| kevinconway | oh wait | 18:50 |
| kevinconway | does it? | 18:50 |
| hub_cap | volume-type-create Create a new volume type. | 18:50 |
| hub_cap | volume-type-delete Delete a specific flavor | 18:50 |
| hub_cap | ^ ^ 2 nove args | 18:50 |
| hub_cap | *nova | 18:50 |
| hub_cap | cell-show Show details of a given cell. | 18:51 |
| hub_cap | it does not use plurals | 18:51 |
| datsun180b | wait hub_cap lines 9, 13, 17, 20 and others | 18:51 |
| datsun180b | why is that just a verb and no noun | 18:51 |
| hub_cap | cuz thats how the base resource works | 18:51 |
| hub_cap | look @ nova | 18:51 |
| hub_cap | ssh SSH into a server. | 18:51 |
| hub_cap | start Start a server. | 18:51 |
| hub_cap | those are _instance_ things | 18:52 |
| datsun180b | oh that's the <subcommand> part i'm guessing | 18:52 |
| hub_cap | list List active servers. | 18:52 |
| datsun180b | gotcha, i was having a forest and trees sort of moment | 18:52 |
| vipul | what about security-groups (plural) | 18:53 |
| hub_cap | did i do that? | 18:53 |
| vipul | nova: secgroup-list List security groups for the current tenant. | 18:53 |
| hub_cap | secgroup-add-rule Add a rule to a security group. | 18:53 |
| vipul | secgroup-add-rule Add a rule to a security group. | 18:53 |
| vipul | woah | 18:53 |
| hub_cap | do u want me to change? | 18:53 |
| vipul | yea, let's be consistent | 18:53 |
| hub_cap | k | 18:53 |
| datsun180b | so i understand we support selecting response types by appending a file ending to a request url, like .json or .xml or .baconcheeseburger or .pdf | 18:54 |
| datsun180b | is there an inked list somewhere of the suffixes we accept, or is it just json and xml for now | 18:54 |
| openstackgerrit | Robert Myers proposed a change to openstack/trove: Fixing restart tests https://review.openstack.org/50971 | 18:55 |
| datsun180b | this factors into some changes "a friend" is making to trove to deal with a url encoding issue "my friend" is trying to deal with | 18:56 |
| hub_cap | UNO MAS | 18:56 |
| hub_cap | https://gist.github.com/hub-cap/6923565 | 18:56 |
| hub_cap | datsun180b: prolly those for now, maybe .atom too | 18:57 |
| hub_cap | tell youre friend to keep it internal or ill give him a -2 | 18:57 |
| hub_cap | *your | 18:57 |
| hub_cap | and your friends review will go to die | 18:57 |
| datsun180b | hub_cap: but what if "my friend" can extinguish all your fears about restrictions | 18:57 |
| hub_cap | i doubt he can | 18:58 |
| hub_cap | but if he can and its not the worlds ugliest hack | 18:58 |
| hub_cap | https://gist.github.com/hub-cap/6923565 | 18:58 |
| hub_cap | eyeballs plz | 18:58 |
| *** SnowDust has joined #openstack-trove | 18:58 | |
| hub_cap | im going to update the docs too | 18:58 |
| datsun180b | well "he"'ll have to work on it later | 18:58 |
| hub_cap | ;) | 18:58 |
| datsun180b | i wonder if your subcommands would benefit from being listed in the same order, maybe show/list/create/delete/update | 18:59 |
| datsun180b | even if that isn't alphabetical | 18:59 |
| datsun180b | oh i'll fork that gist | 19:00 |
| *** adrian_otto has quit IRC | 19:02 | |
| datsun180b | hub_cap: https://gist.github.com/ed-/6923630/revisions eh | 19:03 |
| vipul | you can fok gists? | 19:06 |
| vipul | fork :p | 19:06 |
| vipul | instaed of alphabetical | 19:06 |
| vipul | why not keep the isntance operations at the top | 19:06 |
| vipul | the ones not 'prefixed' by anything | 19:06 |
| hub_cap | there is no way to fix that datsun180b | 19:07 |
| *** SnowDust has quit IRC | 19:11 | |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 19:11 |
| hub_cap | ^ ^ fixied | 19:12 |
| hub_cap | also added a readme that doesnt make us look like shmucks on pypi | 19:12 |
| datsun180b | oh right, help is generated dynamically isn't it | 19:12 |
| hub_cap | yup datsun180b | 19:12 |
| hub_cap | arg missed 2 things | 19:12 |
| datsun180b | wait there may be a way, if this client's using argparse subcommands | 19:13 |
| datsun180b | er subparsers | 19:13 |
| datsun180b | also argument groups | 19:14 |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 19:14 |
| hub_cap | uno mas | 19:14 |
| hub_cap | datsun180b: then we will be nonstandard | 19:14 |
| datsun180b | that is the worst excuse | 19:14 |
| datsun180b | but it's also your best excuse | 19:15 |
| hub_cap | yes but its the right excuse | 19:15 |
| hub_cap | :) | 19:15 |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 19:16 |
| hub_cap | uno uno mas | 19:16 |
| hub_cap | had some ugly red spaces in the readme | 19:16 |
| hub_cap | ok lets get these merged today, so plz review | 19:16 |
| ashestakov | hi all | 19:21 |
| ashestakov | during tox tests, from where it get troveclient? pypy? | 19:21 |
| yidclare | ah hub_cap, I was going to do my first commit fixing https://bugs.launchpad.net/python-troveclient/+bug/1238183 | 19:22 |
| hub_cap | heh i think i fixed that ya yidclare? | 19:22 |
| hub_cap | ashestakov: tarballs.o.o | 19:22 |
| yidclare | you did not :) | 19:23 |
| yidclare | and the first line says Cinder | 19:23 |
| ashestakov | hub_cap: i just thought, to pass tests in my change for service types, troveclient should be modified, so tarball should exist in repo | 19:24 |
| yidclare | need to figure out how to review and −1 you :) | 19:24 |
| ashestakov | do all changes build tarball and uploads to mirror? | 19:24 |
| hub_cap | hehe yidclare | 19:24 |
| hub_cap | tarballs are pushed _after_ they are commited ashestakov | 19:24 |
| hub_cap | so your uncommited work wont be in tarballs | 19:25 |
| ashestakov | hub_cap: yep, but once changes commited to master, it builds tarball? | 19:25 |
| hub_cap | ashestakov: correct | 19:25 |
| ashestakov | cool, thanks | 19:26 |
| hub_cap | yidclare: i dont have that anymore (delete backup error) | 19:26 |
| hub_cap | yidclare: https://review.openstack.org/#/c/51015/3/troveclient/v1/shell.py | 19:26 |
| hub_cap | vipul: ill fix those, good catch | 19:26 |
| vipul | hub_cap: actually you do the docstring on 'do_backup_create' -- you could leave it for yidclare :P | 19:27 |
| hub_cap | ohh icic | 19:27 |
| hub_cap | ok thats fine | 19:27 |
| hub_cap | more contributors = win | 19:27 |
| openstackgerrit | Michael Basnight proposed a change to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 19:28 |
| *** jasonb365 has quit IRC | 19:29 | |
| hub_cap | vipul: ^ ^ | 19:29 |
| vipul | hub_cap: So i don't understand the OS_AUTH_URL twice | 19:30 |
| yidclare | oh nice, thank hub_cap - you can ignore my comment | 19:31 |
| hub_cap | what vipul is that in ther 2x? | 19:31 |
| yidclare | and I'll make the fix :) | 19:31 |
| vipul | yes | 19:31 |
| hub_cap | i think thats a typo in the cinder readme! | 19:31 |
| vipul | so what's the way to overrride SERVICE_URL | 19:32 |
| vipul | i.e. I don't want to parse from catalog | 19:32 |
| hub_cap | ya but the clients dont even support that lol | 19:33 |
| hub_cap | https://pypi.python.org/pypi/python-novaclient | 19:33 |
| hub_cap | https://pypi.python.org/pypi/python-cinderclient/1.0.6 | 19:33 |
| hub_cap | both of them have that bug | 19:33 |
| hub_cap | it should say TROVE_URL for sure | 19:34 |
| hub_cap | oh it does | 19:34 |
| hub_cap | ill set that export TROVE_URL | 19:35 |
| hub_cap | mabye thatll explain it a bit better.... but both of those other clients have that issue | 19:35 |
| hub_cap | yidclare: u dont wanna fix that? | 19:35 |
| hub_cap | i can leave it, or i can clean it up | 19:35 |
| hub_cap | the delete/create docstring issue yidclare | 19:35 |
| vipul | i hear her working with esp on figuring how to rebase to your change.. she's gonna fix | 19:35 |
| hub_cap | ok sweet | 19:36 |
| *** adrian_otto has joined #openstack-trove | 19:36 | |
| vipul | hub_cap: Do we have code that looks at TROVE_URL | 19:36 |
| hub_cap | yes | 19:36 |
| hub_cap | the OS_AUTH_... uses TROVE_URL as the default | 19:36 |
| hub_cap | i think that means "bypass auth" maybe? | 19:37 |
| vipul | hmm | 19:37 |
| vipul | i wanna auth still though | 19:37 |
| * hub_cap defers to all the other clients | 19:38 | |
| hub_cap | Another BART strike may be imminent, and BART service could stop as early as October 11, 2013. We urge you to plan ahead and prepare to use other transportation options. | 19:38 |
| vipul | https://gist.github.com/vipulsabhaya/6924232 | 19:38 |
| hub_cap | CMON!!!!!!!!!!!!!!!!! | 19:38 |
| vipul | why is that twice | 19:38 |
| vipul | oh backwards compatible | 19:38 |
| * hub_cap shrugs | 19:38 | |
| hub_cap | vipul: https://review.openstack.org/#/c/50994 | 19:39 |
| hub_cap | ^ ^ thats the review that removes the httplib2 / lxml from the requirements | 19:40 |
| vipul | k | 19:40 |
| vipul | --bypass-url <bypass-url> | 19:40 |
| vipul | Use this API endpoint instead of the Service Catalog | 19:40 |
| vipul | hub_cap: ^^ nova supports | 19:40 |
| hub_cap | oh cool | 19:41 |
| hub_cap | we should add that | 19:41 |
| hub_cap | blueprint it up ;) | 19:41 |
| vipul | :p | 19:41 |
| hub_cap | i can spend some time on it today | 19:41 |
| hub_cap | once we get thru this mess | 19:41 |
| vipul | sure, not super urgent | 19:42 |
| hub_cap | ya | 19:42 |
| *** ashestakov has quit IRC | 19:42 | |
| hub_cap | id like to limit the # of downloads from pypi | 19:42 |
| hub_cap | vipul: im confused by your comment | 19:46 |
| hub_cap | on that review | 19:46 |
| hub_cap | the new stuff all has tests | 19:46 |
| vipul | so we have a compat/test/test_common | 19:46 |
| vipul | and we have test/tests_common | 19:46 |
| vipul | i am assuming new stuff uses tests/test_common | 19:47 |
| hub_cap | yes compat/test/test_common tests compat.common | 19:47 |
| hub_cap | tests/test_common exercises common.py | 19:47 |
| *** adrian_otto has quit IRC | 19:47 | |
| hub_cap | common.py now only has like 4 methods in it | 19:47 |
| hub_cap | and i kept the code that was testing that in tests/test_common | 19:48 |
| hub_cap | and moved the old junk into compat/ | 19:48 |
| vipul | oh shit | 19:48 |
| vipul | i'm stupid | 19:48 |
| *** adrian_otto has joined #openstack-trove | 19:48 | |
| vipul | you removed stuff | 19:48 |
| vipul | i'm good with it | 19:49 |
| vipul | actually i cna just merge it | 19:49 |
| hub_cap | ++ | 19:50 |
| hub_cap | merge it up vipul | 19:50 |
| hub_cap | also the old shit depended on httplib2 and lxml | 19:50 |
| *** jasonb365 has joined #openstack-trove | 19:50 | |
| *** dmakogon_home has joined #openstack-trove | 19:57 | |
| dmakogon_home | what's up. guys ! | 19:59 |
| hub_cap | hey dmakogon_home | 19:59 |
| hub_cap | vipul: can u review this too? https://review.openstack.org/#/c/51015/ | 19:59 |
| hub_cap | datsun180b: kevinconway ^ ^ | 19:59 |
| hub_cap | plz review this so we can kick it out to pypi | 20:00 |
| dmakogon_home | hub_cap: have you seen errors that i've sent to you ? | 20:00 |
| hub_cap | dmakogon_home: did u email me? | 20:00 |
| dmakogon_home | to | 20:00 |
| dmakogon_home | irc | 20:00 |
| *** jasonb365 has quit IRC | 20:01 | |
| hub_cap | oh can u resend? im in the middle of something, an ditll take me more time to go thru my irssi proxy history | 20:01 |
| dmakogon_home | one of the builds failed while kickstart mysql | 20:01 |
| hub_cap | huuuuuh? | 20:01 |
| hub_cap | thats odd | 20:01 |
| hub_cap | we should just try to retrigger it | 20:01 |
| hub_cap | it could mean many things | 20:01 |
| hub_cap | pypi couldve blipped | 20:01 |
| datsun180b | LGTM | 20:02 |
| dmakogon_home | hub_cap: https://rdjenkins.dyndns.org/job/Trove-Gate/1223/console | 20:02 |
| dmakogon_home | hub_cap: seems like mysql-server package error at this build | 20:03 |
| *** jasonb365 has joined #openstack-trove | 20:04 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Removing lxml/httplib2 depends https://review.openstack.org/50994 | 20:04 |
| dmakogon_home | https://review.openstack.org/#/c/45723/ ||||| https://review.openstack.org/#/c/45723/ ||||| https://review.openstack.org/#/c/50686/ | 20:04 |
| dmakogon_home | id like everyone to review this | 20:05 |
| openstackgerrit | Clare Springer proposed a change to openstack/python-troveclient: Fixes README https://review.openstack.org/51021 | 20:10 |
| *** Barker has quit IRC | 20:11 | |
| yidclare | GUYS! GUYS! I have something to be reviewed! | 20:11 |
| *** Barker has joined #openstack-trove | 20:12 | |
| yidclare | it's my first (pending) commit, so exciting | 20:13 |
| dmakogon_home | yidclare, congratulations ! | 20:14 |
| yidclare | (must thank esp and sliknik for helping me get there) | 20:15 |
| hub_cap | yidclare: i -1d u ;) | 20:15 |
| yidclare | tears | 20:15 |
| hub_cap | and grats | 20:15 |
| *** jmontemayor has quit IRC | 20:16 | |
| dmakogon_home | hub_cap: yeah, -1 him, let him know that this would isn't a place with unicorns and rainbows, and? also, +2 on every 1-line reviews | 20:16 |
| *** esp has left #openstack-trove | 20:16 | |
| * yidclare wipes the blood off | 20:17 | |
| dmakogon_home | hah | 20:17 |
| dmakogon_home | #action lmfao | 20:17 |
| * dmakogon_home wondering about imsplitbit clustering api | 20:18 | |
| hub_cap | dmakogon_home he sent a link out | 20:18 |
| hub_cap | did u see? | 20:18 |
| hub_cap | imsplitbit: re-fire the link | 20:18 |
| dmakogon_home | hub_cap: it's on github ? | 20:19 |
| hub_cap | naw its a wiki page | 20:20 |
| hub_cap | if u had a bouncer / proxy u could see the link ;) ;) | 20:20 |
| hub_cap | https://wiki.openstack.org/wiki/Trove-Replication-And-Clustering-API | 20:21 |
| *** demorris has quit IRC | 20:22 | |
| dmakogon_home | hub_cap: https://bugs.launchpad.net/trove/+bug/1237910 | 20:22 |
| hub_cap | what about it dmakogon_home? | 20:22 |
| dmakogon_home | about missing possibilities in sample configs | 20:23 |
| dmakogon_home | they are not full | 20:23 |
| hub_cap | yidclare: https://review.openstack.org/#/c/51015/4/troveclient/v1/shell.py <-- line 249, fix tht too :) | 20:24 |
| dmakogon_home | my thoughs are if there are several possibilities to configure each trove component, this possibility should be reflected to sample configs | 20:25 |
| dmakogon_home | even with # at line head | 20:25 |
| hub_cap | dmakogon_home yes that makes sense | 20:25 |
| dmakogon_home | i would ask ikhudoshyn to go through samples and update them with missing parameters | 20:26 |
| hub_cap | vipul SlickNik: https://review.openstack.org/#/c/51015/ | 20:26 |
| hub_cap | dmakogon_home: feel free to add that to the review | 20:26 |
| dmakogon_home | oh, yes, i would | 20:26 |
| openstackgerrit | Jenkins proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/51042 | 20:27 |
| dmakogon_home | hub_cap: i've got question to you about current heat template | 20:27 |
| hub_cap | im in a meeting rignt no | 20:28 |
| hub_cap | now | 20:28 |
| *** jasonb365 has quit IRC | 20:28 | |
| yidclare | hub_cap - ok to make your change the parent? | 20:30 |
| *** Barker has quit IRC | 20:32 | |
| *** rnirmal has quit IRC | 20:34 | |
| hub_cap | naw make it to your review | 20:37 |
| hub_cap | yidclare: ^ ^ | 20:37 |
| *** jasonb365 has joined #openstack-trove | 20:37 | |
| *** Barker has joined #openstack-trove | 20:38 | |
| imsplitbit | hub_cap: oh ok | 20:39 |
| imsplitbit | sec | 20:39 |
| imsplitbit | https://wiki.openstack.org/wiki/Trove-Replication-And-Clustering-API | 20:40 |
| *** radez is now known as radez_g0n3 | 20:41 | |
| imsplitbit | there are examples of replication shown with multiple trove instances and a singular trove instance | 20:41 |
| imsplitbit | I still don't prefer the single trove instance view because it implies exclusivity | 20:41 |
| dmakogon_home | imsplitbit: seems reasonable | 20:42 |
| imsplitbit | a cluster's nodes are exclusive to the cluster where a set of instnaces with a replication contract are not necessarily exclusive to each other | 20:43 |
| imsplitbit | the replication contract only implies that the data of a master may exist in part or in whole on a slave of that master | 20:44 |
| dmakogon_home | yeah, so, i think we should separate this entities as imsplitbit saud | 20:45 |
| dmakogon_home | *said | 20:45 |
| dmakogon_home | exclusivity is, currently, best way to keep things clear and understandable | 20:46 |
| openstackgerrit | Clare Springer proposed a change to openstack/python-troveclient: Fixes README and shell https://review.openstack.org/51021 | 20:49 |
| yidclare | oh oops. sorry hub_cap | 20:50 |
| yidclare | esp and I were thinking to keep yours as the parent so they would merge in order | 20:51 |
| hub_cap | yes | 20:52 |
| hub_cap | u did right | 20:52 |
| yidclare | yay | 20:52 |
| *** adrian_otto has quit IRC | 20:55 | |
| hub_cap | vipul: SlickNik can i plz get some eyes on https://review.openstack.org/#/c/51015/ | 21:04 |
| hub_cap | id like to get it pushed to pypi today | 21:04 |
| *** yogeshmehra has quit IRC | 21:04 | |
| SlickNik | looking hub_cap | 21:04 |
| *** yogesh has joined #openstack-trove | 21:05 | |
| hub_cap | <3 | 21:06 |
| *** pdmars has quit IRC | 21:08 | |
| *** yogesh has quit IRC | 21:09 | |
| SlickNik | hub_cap: I'm confused about the rst. | 21:11 |
| SlickNik | What's the OS_AUTH_URL vs the OS_TROVE_URL? | 21:11 |
| hub_cap | scroll up | 21:12 |
| hub_cap | vipul: and i talked about it | 21:12 |
| hub_cap | all 3 clients nova cinder us all asay it | 21:12 |
| vipul | hub_cap: I think the wording is confusing | 21:12 |
| vipul | basically they are interchangeable | 21:12 |
| vipul | OS_TROVE_URL and OS_AUTH_URL = same thing | 21:12 |
| vipul | parser.add_argument('--os-auth-url', | 21:13 |
| vipul | metavar='<auth-url>', | 21:13 |
| vipul | default=utils.env('OS_AUTH_URL', | 21:13 |
| vipul | 'TROVE_URL'), | 21:13 |
| SlickNik | They shouldn't be though, right? | 21:15 |
| SlickNik | (might be another bug we need to fix) | 21:15 |
| SlickNik | Shouldn't TROVE URL be the service_url (if you want to override the catalog)? | 21:15 |
| SlickNik | And OS_AUTH_URL be the keystone URL for auth (for example?) | 21:16 |
| SlickNik | hub_cap: not saying we need to fix it as part of this changeset, just trying to clarify... | 21:16 |
| vipul | I think it's legacy : https://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L301 | 21:16 |
| vipul | SlickNik: what we really need is https://github.com/openstack/python-novaclient/blob/master/novaclient/shell.py#L377 | 21:17 |
| SlickNik | I see. | 21:17 |
| SlickNik | yup, we definitely need that. | 21:18 |
| SlickNik | Gonna open a LP bug for that. | 21:18 |
| hub_cap | plz do | 21:20 |
| hub_cap | and review this code ;) | 21:20 |
| SlickNik | yeah. I'm thinking maybe we should remove the TROVE_URL pieces since we don't have any legacy part of trove that does AUTH. (would it even work?) | 21:22 |
| hub_cap | well that does legacy auth | 21:22 |
| hub_cap | aka, against old keystone | 21:22 |
| hub_cap | trove does no auth | 21:22 |
| amcrn | i only used service_url for trove debugging purposes (set auth_url + service_url w/ token) | 21:25 |
| hub_cap | ok great so u didnt do user/pass ? | 21:25 |
| hub_cap | amcrn: ^ ^ | 21:26 |
| SlickNik | I think legacy auth means auth against old compute (legacy nova auth), and we don't have that in trove. So we can omit that piece entirely. | 21:26 |
| hub_cap | srsly? | 21:26 |
| amcrn | hub_cap: nope | 21:26 |
| hub_cap | ok well lets fix it, but after we push 1.0.1 | 21:26 |
| hub_cap | amcrn: wonderful | 21:26 |
| hub_cap | i didnt test that, just figured it worked lol | 21:27 |
| hub_cap | mmmm day 3 of curry chicken :) | 21:29 |
| openstackgerrit | SlickNik proposed a change to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 21:33 |
| SlickNik | ^^Just removed the AUTH_URL env vars from the rst so as to avoid confusion. Will open up an LP bug to clean up the env vars later. | 21:34 |
| vipul | SlickNik: thanks - i'm good with this | 21:35 |
| hub_cap | vipul whats up w your recent +1s | 21:37 |
| hub_cap | you dont like the +2 anymore? :) | 21:37 |
| vipul | i'm toning it down | 21:37 |
| vipul | :D | 21:37 |
| hub_cap | lulz | 21:37 |
| vipul | i'll let Slick click the Approve button | 21:38 |
| datsun180b | just in case it's a trap or something | 21:38 |
| vipul | ¯\_(ツ)_/¯ | 21:39 |
| SlickNik | btw, the LP bug for TROVE_URL is https://bugs.launchpad.net/python-troveclient/+bug/1238282 | 21:39 |
| vipul | stolen from russellb in nova room | 21:39 |
| SlickNik | lol, nice | 21:39 |
| SlickNik | I was wondering how you typed it out that fast :P | 21:39 |
| datsun180b | yeah i have that bookmarked too, along with ¯\(º_o)/¯ just in case | 21:39 |
| russellb | vipul: \o/ | 21:40 |
| hub_cap | (/ﾟДﾟ)/ | 21:40 |
| datsun180b | <amusing table-flip emoji> | 21:40 |
| vipul | is it a car.. is it a wave.. | 21:40 |
| hub_cap | =’①。①’= | 21:41 |
| hub_cap | (ノ‥)ノ | 21:41 |
| hub_cap | (づ￣ ³￣)づ | 21:42 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Fixing the backwards verb/noun cli names https://review.openstack.org/51015 | 21:42 |
| hub_cap | lets see if magically yidclare's get merged | 21:43 |
| hub_cap | cuz hers was already +2appr | 21:43 |
| hub_cap | ok im tired of typing approved | 21:43 |
| hub_cap | im shortening it to +2A | 21:43 |
| hub_cap | yidclare: i think u will have to rebase https://review.openstack.org/#/c/51021/ | 21:44 |
| hub_cap | （´＿｀） | 21:44 |
| dmakogon_home | hub_cap: have you retrigged this one ? https://review.openstack.org/#/c/50944/ | 21:44 |
| hub_cap | .・゜゜・（／。＼）・゜゜・． | 21:44 |
| hub_cap | sure dmakogon_home | 21:44 |
| hub_cap | dmakogon_home https://rdjenkins.dyndns.org/job/Trove-Gate/1228/ | 21:45 |
| hub_cap | u know what i just found out? linkinus likes emacs cmds | 21:45 |
| hub_cap | its like im learning a language that everything linux/macos understands! | 21:46 |
| yidclare | hup_cap WHAT | 21:46 |
| hub_cap | blame SlickNik vipul they updated my patch | 21:47 |
| yidclare | my changes are stranded... | 21:47 |
| hub_cap | yidclare: your review says it depends on "Fixing the backwards verb/noun cli names (MERGED) [OUTDATED]" | 21:47 |
| hub_cap | u just have to rebase your change on top of mine | 21:47 |
| hub_cap | or on top of the actual master repo now | 21:47 |
| hub_cap | it wont be too painful | 21:47 |
| *** esp has joined #openstack-trove | 21:48 | |
| *** esp has left #openstack-trove | 21:49 | |
| *** esp has joined #openstack-trove | 21:49 | |
| hub_cap | yidclare: do u think u can get to it today? cuz id like to make it part of 1.0.1 as well on pypi | 21:49 |
| yidclare | yes, esp is here to help me | 21:50 |
| dmakogon_home | vipul: SlickNik: hab_cap: would you mind finally review this important review ? https://review.openstack.org/#/c/45723/ | 21:50 |
| openstackgerrit | Clare Springer proposed a change to openstack/python-troveclient: Fixes README and shell https://review.openstack.org/51021 | 21:50 |
| vipul | dmakogon_home: i guess you're home :D | 21:50 |
| yidclare | lol - the rebase button worked! | 21:51 |
| dmakogon_home | vipul: how did you found out that ??? | 21:51 |
| vipul | i'm sneaky like that | 21:51 |
| dmakogon_home | vipul: magician ? | 21:51 |
| SlickNik | what's linkinus? | 21:52 |
| vipul | you know what is sneaky.. pull down the Notifications in iOS7.. at the top.. it tells you how long it takes you to get home... in the morning how long it takes you to get to work | 21:52 |
| hub_cap | wonderful yidclare | 21:52 |
| SlickNik | apparently this http://www.conceitedsoftware.com/linkinus | 21:52 |
| hub_cap | SlickNik: http://www.conceitedsoftware.com/linkinus | 21:52 |
| hub_cap | BEEEEET U | 21:52 |
| SlickNik | heh | 21:53 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Fixes README and shell https://review.openstack.org/51021 | 21:53 |
| hub_cap | yidclare: ^ ^ automagic | 21:54 |
| dmakogon_home | vipul: i did not have any device running on iOS7, only old iPod Touch 3G (iOS 5.0.1) | 21:55 |
| yidclare | hub_cap: yep, totally painless. I take back all the complaints I was queuing up ;) | 21:55 |
| hub_cap | i just use irssi 1/2 the time tho | 21:55 |
| *** jcru has quit IRC | 21:57 | |
| vipul | dmakogon_home: so on your patch, i think amcrn had some comments regarding changing update_status_on_fail to be Plural | 21:59 |
| vipul | dmakogon_home: you plan to address that still? | 21:59 |
| hub_cap | so shall i push a 1.0.1 ? | 21:59 |
| vipul | hub_cap: Yes | 22:00 |
| dmakogon_home | vipul: comments explains what it means | 22:00 |
| dmakogon_home | vipul: if you ok with that, we could leave it | 22:00 |
| dmakogon_home | vipul: if we take into account instance provisioining workflow, reason why is that parameter is not plural is that we return to user only one status, not both (instance service status, task status) | 22:04 |
| *** dafter has quit IRC | 22:05 | |
| *** dafter has joined #openstack-trove | 22:05 | |
| dmakogon_home | vipul: from user point, task status is a under-the-hood dependency for correct representation of trove-instance-inner state | 22:07 |
| vipul | dmakogon_home: Ok, i'm fine either way regarding the cfg name | 22:07 |
| vipul | The only concern i have is not addressed, which is also SlickNik's comment.. why return things if they don't get used | 22:07 |
| dmakogon_home | vipul: i'm glad, thanks | 22:07 |
| vipul | If it's just for the purpose of tests.. than that's not a good reason | 22:08 |
| dmakogon_home | vipul: returns are very useful for tests | 22:08 |
| dmakogon_home | vipul: output very useful for black/white-box testing | 22:09 |
| hub_cap | https://pypi.python.org/pypi/python-troveclient/ | 22:09 |
| *** dafter has quit IRC | 22:10 | |
| hub_cap | ^^ README.. you are so beautiful, READ-ME | 22:10 |
| vipul | you can write the test such that the Object being modified (in this case InstanceServiceStatus) can be asserted instead of asserting the function return values | 22:10 |
| hub_cap | get it?? sing it like joe cocker | 22:10 |
| *** Barker has quit IRC | 22:12 | |
| dmakogon_home | vipul: if it so, than it doesn't need to be tested because it uses only models things | 22:12 |
| vipul | You want to test whether calling that method updates some status... Have the test go look at that Status object | 22:13 |
| hub_cap | yidclare: https://pypi.python.org/pypi/python-troveclient/ <-- u can see your work there | 22:13 |
| vipul | are you serious? 716 downloads in the last day | 22:16 |
| *** ikhudosh_ has quit IRC | 22:16 | |
| vipul | or are those just all our Jenkins runs :p | 22:16 |
| dmakogon_home | vipul: ok, i'll take a look and i would try to do that test | 22:16 |
| hub_cap | vipul: it must be automation | 22:17 |
| hub_cap | in some fashion | 22:17 |
| *** datsun180b has quit IRC | 22:17 | |
| *** jasonb365 has quit IRC | 22:24 | |
| *** robertmyers has quit IRC | 22:25 | |
| dmakogon_home | vipul: still here ? | 22:28 |
| vipul | yes | 22:29 |
| dmakogon_home | vipul: what if i would use outputed statuses for loggin process ? | 22:29 |
| dmakogon_home | and method would only update statuses in database | 22:29 |
| dmakogon_home | current issues are covered - return used, method covered with test | 22:30 |
| vipul | I think what you have to do is pretty easy.. you are mocking the find_by() to a fake Status | 22:30 |
| vipul | You keep a handle on the mocked Status that the method will obtain | 22:31 |
| vipul | and check it at the end | 22:31 |
| dmakogon_home | vipul: i know | 22:31 |
| vipul | I just don't see a purpose to returning things only tests care about | 22:32 |
| dmakogon_home | but current test seems more reasonable than even if it uses output | 22:32 |
| dmakogon_home | to make this func works without output and than cover it with tests, i should make service and db_info class attrs | 22:36 |
| hub_cap | | (• ◡•)| (❍ᴥ❍ʋ) <--- finn and jake | 22:37 |
| dmakogon_home | hub_cap: what do you think about using output for tests reasons ? | 22:39 |
| hub_cap | im not sure what you mean by that | 22:39 |
| dmakogon_home | https://review.openstack.org/#/c/45723/24/trove/tests/unittests/taskmanager/test_models.py | 22:40 |
| dmakogon_home | https://review.openstack.org/#/c/45723/24/trove/taskmanager/models.py | 22:40 |
| hub_cap | ᕙ(⇀‸↼‶)ᕗ | 22:41 |
| dmakogon_home | i've got func that does return, in "real life" it's ignored | 22:41 |
| dmakogon_home | but used in tests | 22:41 |
| hub_cap | remember we talked about this | 22:41 |
| dmakogon_home | yes | 22:41 |
| dmakogon_home | i remember | 22:41 |
| hub_cap | i do not like it | 22:41 |
| hub_cap | but if its 100000% necessary and the only way to test it | 22:41 |
| hub_cap | if you can find other ways to test i dont see the need for it | 22:41 |
| hub_cap | in general | 22:42 |
| dmakogon_home | i heard what i want | 22:42 |
| dmakogon_home | thanks | 22:42 |
| hub_cap | you should not change code to make tests behave | 22:42 |
| dmakogon_home | going to re-do it, again =/ | 22:42 |
| hub_cap | you sohudl change tests to make code behave | 22:42 |
| hub_cap | dmakogon_home: (╯°□°)╯︵ ┻━┻ | 22:43 |
| dmakogon_home | i've got an idea how to make FreshInstanceTask class to use single instance of each status | 22:43 |
| dmakogon_home | it would be easy enough | 22:44 |
| hub_cap | k | 22:44 |
| dmakogon_home | hub_cap: vipul: thanks for help, not even once))) | 22:51 |
| dmakogon_home | it would be greate if all of you take a look at othre reviews) | 22:51 |
| esp | hub_cap: nice work on the trove-cli | 22:51 |
| hub_cap | <3 | 22:55 |
| *** amytron has quit IRC | 22:55 | |
| hub_cap | im leaving soon dmakogon_home no reviews for me today | 22:55 |
| hub_cap | im triaging bugs now | 22:55 |
| hub_cap | its my wife and my 10th anniversary | 22:55 |
| hub_cap | we are gonna go eat some raw chicken | 22:56 |
| hub_cap | http://www.yelp.com/biz_photos/ippuku-berkeley-2?select=UWgPK2I3gdh4dUrxVXRzTA#CKJ-XuDD6AnVasAZR2Lbzg | 22:56 |
| *** paul_lodronio has left #openstack-trove | 22:57 | |
| *** grapex has quit IRC | 22:59 | |
| *** grapex has joined #openstack-trove | 23:00 | |
| esp | hub_cap: don't eat raw chicken! | 23:00 |
| vipul | yummy | 23:00 |
| esp | hmm.. that does look good tho | 23:00 |
| hub_cap | mmmm hmmmm | 23:02 |
| hub_cap | look @ the other pics | 23:02 |
| dmakogon_home | hub_cap: have a good day))) | 23:02 |
| hub_cap | it looks crazy good | 23:02 |
| hub_cap | and its <1 mi from my house, totally walking to it | 23:02 |
| * dmakogon_home left chat for nap | 23:02 | |
| hub_cap | thx dmakogon_home, have a good evening! | 23:02 |
| vipul | hub_cap: what's the plan for the mgmt client | 23:04 |
| vipul | Should it just be part of the main client? like nova tends to do | 23:04 |
| *** grapex has quit IRC | 23:04 | |
| jdbarry | hub_cap closing my auth bug for cli: ++ | 23:07 |
| *** dmakogon_home has quit IRC | 23:07 | |
| hub_cap | vipul: correct | 23:09 |
| hub_cap | jdbarry: :) | 23:09 |
| hub_cap | itll just be extra cmds that a admin user can execute | 23:09 |
| hub_cap | jdbarry: give the new cli a spin | 23:09 |
| hub_cap | u will like | 23:10 |
| hub_cap | i just cleaned up all the old bugs in troveclient | 23:10 |
| hub_cap | we now have 15 "actual" bugs | 23:10 |
| hub_cap | dmakogon: https://bugs.launchpad.net/trove/+bug/1219522 | 23:11 |
| hub_cap | i need to go thru /trove tomorrow | 23:12 |
| hub_cap | ok now 9 bugs! | 23:14 |
| hub_cap | woot | 23:14 |
| hub_cap | adn 4 of those are incomplete | 23:15 |
| jdbarry | hub_cap: cli has been spun | 23:20 |
| jdbarry | very nice | 23:20 |
| hub_cap | :) | 23:20 |
| jdbarry | i will file a new bug for secgroup-list. maybe i will try to fix it on the weekend. dust off the coding skillz. | 23:21 |
| jdbarry | something is up w/ formatting | 23:21 |
| hub_cap | there is some formatting problems w/ the "nested" lists | 23:23 |
| hub_cap | plz do jdbarry | 23:23 |
| jdbarry | is the intention to list not only all groups with this call but also all rules within all groups? | 23:24 |
| hub_cap | it is the intention to get the cli updated | 23:25 |
| hub_cap | after that all intention is thrown out the door | 23:25 |
| hub_cap | so, we are open to any ideas at this point | 23:25 |
| jdbarry | nova cli is a good place to start. i'll check it out. | 23:25 |
| hub_cap | def | 23:26 |
| hub_cap | juice: did u see that i reopened https://bugs.launchpad.net/trove/+bug/1236620 | 23:26 |
| hub_cap | i hit it 2x last night trying to merge code | 23:26 |
| juice | i did - has it been failing often in trove gate? | 23:26 |
| *** krow1 has quit IRC | 23:27 | |
| hub_cap | i saw it 2x on the same bug | 23:28 |
| hub_cap | err | 23:28 |
| hub_cap | review | 23:28 |
| hub_cap | ok im not going to muck w trove/+bugs today | 23:28 |
| hub_cap | i was gonna | 23:28 |
| hub_cap | then i was like no | 23:28 |
| hub_cap | too much thinking | 23:28 |
| hub_cap | imma drink a rodenbach and chill | 23:28 |
| hub_cap | http://beeradvocate.com/beer/profile/216/1882 | 23:29 |
| hub_cap | peace all | 23:29 |
| jdbarry | happy anniversary | 23:29 |
| *** krow has joined #openstack-trove | 23:29 | |
| hub_cap | thx! | 23:30 |
| *** radez_g0n3 is now known as radez | 23:49 | |
| radez | hub_cap: you around by chance? | 23:50 |
| radez | cp16net: you around? | 23:50 |
| radez | incase someone's watching that can help, I'm registering trove manually with keystone and I'm not sure what values I shoudl use for the service name and type | 23:51 |
| radez | I keep getting the error that an suitable endpoint can't be found. | 23:51 |
| hub_cap | radez: hi | 23:55 |
| hub_cap | i shoudlnt be but i am | 23:55 |
| hub_cap | are you using local keystone? | 23:55 |
| radez | :) I'll understand if you put me off till tomrrow | 23:55 |
| radez | yes | 23:55 |
| hub_cap | service_type=database | 23:55 |
| hub_cap | service_name ummm trove? | 23:56 |
| radez | hm, that's what I tried at first | 23:56 |
| hub_cap | lemme c what devstack is setting | 23:56 |
| radez | kk | 23:56 |
| radez | and endpoints should all be http://host_ipaddress:8779/ right? | 23:56 |
| radez | btw, I've finally circled back to this. I hope to plow through getting the package req into fedora sometime next week | 23:57 |
| hub_cap | https://github.com/openstack-dev/devstack/blob/master/lib/trove#L57 | 23:57 |
| hub_cap | v1.0 | 23:57 |
| hub_cap | --publicurl "http://$SERVICE_HOST:8779/v1.0/\$(tenant_id)s" \ | 23:57 |
| hub_cap | id just run the cmds that devstack is runnin (copy-pasta ftw) | 23:58 |
| * radez nods | 23:58 | |
| radez | kk, I'll look through this... is there a problem with me putting the trove user in the services tennant? | 23:58 |
| radez | I've got user: trove, tenant services role admin | 23:58 |
| amcrn | radez: i've only gotten that error if the region is off, are you sure it's RegionOne? | 23:59 |
| radez | amcrn: AH! now that you say that... the installer I use uses a different case... regionOne | 23:59 |
| radez | I bet that's it | 23:59 |
| amcrn | :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!