openstackgerrit | James E. Blair proposed openstack/python-troveclient master: Zuul: add file extension to playbook path https://review.openstack.org/523612 | 00:31 |
---|---|---|
openstackgerrit | James E. Blair proposed openstack/python-troveclient master: Zuul: add file extension to playbook path https://review.openstack.org/523612 | 00:40 |
*** kumarmn has joined #openstack-trove | 00:42 | |
amrith | your irc viewer/tool should show my name with a different icon next to it; the same icon as say ChanServ and openstack | 01:24 |
amrith | oops | 01:24 |
*** kumarmn has quit IRC | 01:27 | |
*** crazydude has joined #openstack-trove | 01:56 | |
*** spotz has joined #openstack-trove | 01:57 | |
crazydude | I say annoying things | 01:58 |
*** amrith sets mode: +b *!47e9386b@gateway/web/freenode/ip.71.233.56.107 | 01:58 | |
*** kumarmn has joined #openstack-trove | 02:17 | |
*** zhaochao has quit IRC | 02:20 | |
*** kumarmn has quit IRC | 02:23 | |
*** daidv has joined #openstack-trove | 02:33 | |
*** zhaochao has joined #openstack-trove | 02:34 | |
*** crazydude has quit IRC | 02:36 | |
*** kei_yama has quit IRC | 03:39 | |
*** links has joined #openstack-trove | 03:42 | |
*** lijinhui has joined #openstack-trove | 03:43 | |
*** kumarmn has joined #openstack-trove | 03:51 | |
*** kei_yama has joined #openstack-trove | 04:02 | |
*** kumarmn has quit IRC | 04:09 | |
*** kumarmn has joined #openstack-trove | 04:11 | |
*** kumarmn has quit IRC | 04:16 | |
openstackgerrit | Andy Botting proposed openstack/trove master: List flavors based on datastore/version associations https://review.openstack.org/523295 | 04:16 |
*** lijinhui has quit IRC | 04:18 | |
*** kumarmn has joined #openstack-trove | 04:37 | |
*** kumarmn has quit IRC | 04:41 | |
*** gouthamr has quit IRC | 04:43 | |
*** gouthamr has joined #openstack-trove | 05:46 | |
*** gouthamr has quit IRC | 06:04 | |
openstackgerrit | Andy Botting proposed openstack/trove master: List flavors based on datastore/version associations https://review.openstack.org/523295 | 06:08 |
openstackgerrit | Merged openstack/trove master: Change RPC dispatcher access_policy to DefaultRPCAccessPolicy https://review.openstack.org/517285 | 06:46 |
*** flwang1 has quit IRC | 06:53 | |
*** flwang1 has joined #openstack-trove | 07:34 | |
*** maciejjozefczyk has quit IRC | 07:41 | |
*** rcernin has quit IRC | 07:43 | |
*** pcaruana has joined #openstack-trove | 08:04 | |
*** maciejjozefczyk has joined #openstack-trove | 08:22 | |
*** b_bezak has joined #openstack-trove | 08:27 | |
*** b_bezak has quit IRC | 08:29 | |
*** kei_yama has quit IRC | 08:35 | |
fanzhang | Just a kind reminder, would someone help review this bp https://review.openstack.org/#/c/522696/, really appreciate :) | 08:38 |
*** rcernin has joined #openstack-trove | 08:40 | |
*** spectr has joined #openstack-trove | 09:03 | |
*** kumarmn has joined #openstack-trove | 09:10 | |
*** kumarmn has quit IRC | 09:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/trove master: Updated from global requirements https://review.openstack.org/523808 | 09:25 |
openstackgerrit | Andy Botting proposed openstack/trove master: List flavors based on datastore/version associations https://review.openstack.org/523295 | 09:38 |
*** gcb has joined #openstack-trove | 09:39 | |
*** tosky has joined #openstack-trove | 10:24 | |
*** spectr has quit IRC | 10:33 | |
*** daidv has quit IRC | 10:39 | |
*** gcb has quit IRC | 11:05 | |
openstackgerrit | Yao Wang proposed openstack/python-troveclient master: Fix datastore abnormal display with trove backup-show https://review.openstack.org/523851 | 11:55 |
*** smatzek has joined #openstack-trove | 12:11 | |
*** b_bezak has joined #openstack-trove | 12:13 | |
openstackgerrit | Merged openstack/trove-dashboard master: Imported Translations from Zanata https://review.openstack.org/523310 | 12:50 |
openstackgerrit | Fan Zhang proposed openstack/trove master: Implementation of root-enable, root-disable in redis. https://review.openstack.org/522696 | 13:10 |
*** kumarmn has joined #openstack-trove | 13:11 | |
*** kumarmn has quit IRC | 13:15 | |
*** kumarmn has joined #openstack-trove | 13:20 | |
*** links has quit IRC | 13:21 | |
*** kumarmn has quit IRC | 13:25 | |
*** zhaochao has quit IRC | 13:38 | |
*** zhurong has joined #openstack-trove | 13:39 | |
*** openstackstatus has quit IRC | 13:43 | |
*** openstack has joined #openstack-trove | 13:43 | |
*** ChanServ sets mode: +o openstack | 13:43 | |
*** openstackstatus has joined #openstack-trove | 13:45 | |
*** ChanServ sets mode: +v openstackstatus | 13:45 | |
*** zhurong has quit IRC | 13:56 | |
*** zhurong has joined #openstack-trove | 13:58 | |
*** dasTor_ has joined #openstack-trove | 14:10 | |
*** zhaochao has joined #openstack-trove | 14:11 | |
*** dasTor has quit IRC | 14:13 | |
*** kumarmn has joined #openstack-trove | 14:20 | |
*** rcernin has quit IRC | 14:27 | |
*** smatzek has quit IRC | 14:29 | |
*** smatzek has joined #openstack-trove | 14:30 | |
*** smatzek has quit IRC | 14:34 | |
*** zhurong has quit IRC | 14:46 | |
kumarmn | Reminder that the weekly meeting will start on openstack-meeting-alt in 15 minutes | 14:46 |
*** openstack has quit IRC | 14:46 | |
*** openstack has joined #openstack-trove | 14:47 | |
*** ChanServ sets mode: +o openstack | 14:47 | |
*** Yao has joined #openstack-trove | 14:47 | |
*** zhurong has quit IRC | 14:50 | |
*** songjian has joined #openstack-trove | 14:50 | |
openstackgerrit | Merged openstack/python-troveclient master: Zuul: add file extension to playbook path https://review.openstack.org/523612 | 14:52 |
*** dasTor_ has quit IRC | 14:54 | |
*** dasTor has joined #openstack-trove | 14:54 | |
*** vserrao_ has joined #openstack-trove | 15:02 | |
*** smatzek has joined #openstack-trove | 15:12 | |
*** songjian has quit IRC | 15:16 | |
*** songjian has joined #openstack-trove | 15:17 | |
*** gouthamr has joined #openstack-trove | 15:22 | |
*** songjian has quit IRC | 15:53 | |
*** Yao has quit IRC | 15:59 | |
*** b_bezak has quit IRC | 16:00 | |
openstackgerrit | Yao Wang proposed openstack/python-troveclient master: Add show to OSC https://review.openstack.org/523314 | 16:14 |
*** vserrao_ has left #openstack-trove | 16:35 | |
*** qgrolleau has joined #openstack-trove | 16:53 | |
*** tosky has quit IRC | 18:24 | |
*** tosky has joined #openstack-trove | 19:18 | |
*** smatzek has quit IRC | 19:32 | |
*** smatzek has joined #openstack-trove | 19:38 | |
*** smatzek_ has joined #openstack-trove | 19:41 | |
*** smatzek has quit IRC | 19:43 | |
*** smatzek_ has quit IRC | 19:46 | |
*** smatzek has joined #openstack-trove | 19:46 | |
*** smatzek_ has joined #openstack-trove | 19:51 | |
*** smatzek has quit IRC | 19:51 | |
*** flwang1 has quit IRC | 20:09 | |
*** spectr has joined #openstack-trove | 20:18 | |
*** spectr has quit IRC | 20:20 | |
*** smatzek_ has quit IRC | 20:21 | |
*** smatzek has joined #openstack-trove | 20:22 | |
*** smatzek has quit IRC | 20:24 | |
*** smatzek has joined #openstack-trove | 20:26 | |
*** smatzek has quit IRC | 20:28 | |
*** smatzek has joined #openstack-trove | 20:29 | |
*** smatzek has quit IRC | 20:30 | |
*** smatzek_ has joined #openstack-trove | 20:33 | |
*** smatzek__ has joined #openstack-trove | 20:35 | |
*** smatzek__ has quit IRC | 20:35 | |
*** smatzek__ has joined #openstack-trove | 20:35 | |
*** smatzek_ has quit IRC | 20:37 | |
*** pcaruana has quit IRC | 21:00 | |
*** smatzek__ has quit IRC | 21:18 | |
*** andybotting has joined #openstack-trove | 22:01 | |
*** rcernin has joined #openstack-trove | 22:03 | |
andybotting | amrith: Wanted to follow up on your comment in https://review.openstack.org/#/c/523295/ if you have a sec | 22:03 |
amrith | i have a second | 22:51 |
andybotting | Thanks | 22:54 |
andybotting | You mentioned that change breaks the API rules - are those rules documented anywhere? | 22:55 |
amrith | not that I know of, but currently the API requires those two parameters, datastore version and name. making them not required is now an extension that is inconsistent as the past behavior was to produce an error. | 22:55 |
amrith | Traditionally the way we've done this is make a new API call because we still don't version. | 22:56 |
andybotting | so the intention of the change is to limit the flavors returned to be only those which are available for trove across all datastores/versions, so is this something we can get in without it being an API change as such? | 22:57 |
amrith | I wouldn't given that we are changing the semantics of the api call. | 22:58 |
amrith | I would recommend that we treat it as a breaking change | 22:58 |
amrith | but since this is a gray line, I didn't score my review with either a -1 or -2. I'll leave it to someone else to concur and either +2 or -1 it ... | 22:59 |
andybotting | ok sure | 22:59 |
andybotting | thanks | 23:00 |
amrith | andy, also please remember that we don't have robust tempest tests in trove. So breaking API changes get caught only through review/inspection (early discovery) of bug reports (late discovery) | 23:02 |
andybotting | yeah, understand | 23:06 |
andybotting | we | 23:06 |
andybotting | we're running something similar in our prod environment now | 23:06 |
andybotting | we're upgrading to Pike now, but we'd like to get as much of our custom stuff upstream as possible | 23:07 |
andybotting | the patch has changed a bit from what we have now on ocata, but if you think the patch is generally OK, I'll backport it to Pike and get it going with some decent testing in our test environment first | 23:08 |
*** hoonetorg has joined #openstack-trove | 23:10 | |
*** kei_yama has joined #openstack-trove | 23:31 | |
*** kumarmn has quit IRC | 23:41 | |
*** tesseract has joined #openstack-trove | 23:48 | |
*** tesseract has quit IRC | 23:49 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!