*** Viswanath has joined #openstack-trove | 00:04 | |
*** Riddhi has joined #openstack-trove | 00:05 | |
*** Viswanath has quit IRC | 00:09 | |
*** X019 has quit IRC | 00:12 | |
openstackgerrit | Nikhil Manchanda proposed openstack/trove: Updated the dev install docs https://review.openstack.org/107278 | 00:24 |
---|---|---|
*** radez is now known as radez_g0n3 | 00:27 | |
*** edmund has joined #openstack-trove | 00:27 | |
*** david-lyle is now known as david-lyle_afk | 00:33 | |
*** _amrith_ is now known as amrith | 00:47 | |
*** sushilkm has quit IRC | 01:02 | |
*** Riddhi has quit IRC | 01:02 | |
*** Riddhi has joined #openstack-trove | 01:03 | |
*** sushilkm has joined #openstack-trove | 01:03 | |
*** sushilkm has quit IRC | 01:05 | |
*** mattgriffin has joined #openstack-trove | 01:11 | |
*** vkmc has quit IRC | 01:17 | |
*** sushilkm has joined #openstack-trove | 01:55 | |
*** mattgriffin has quit IRC | 01:55 | |
*** vkmc has joined #openstack-trove | 01:55 | |
*** vkmc has quit IRC | 01:55 | |
*** vkmc has joined #openstack-trove | 01:55 | |
*** newb has joined #openstack-trove | 01:56 | |
*** Riddhi_ has joined #openstack-trove | 02:02 | |
*** haomaiwang has joined #openstack-trove | 02:06 | |
*** Riddhi has quit IRC | 02:06 | |
*** X019 has joined #openstack-trove | 02:07 | |
*** nosnos has joined #openstack-trove | 02:12 | |
*** annashen has quit IRC | 02:16 | |
*** annashen has joined #openstack-trove | 02:17 | |
*** topshare has joined #openstack-trove | 02:18 | |
*** annashen has quit IRC | 02:21 | |
*** annashen has joined #openstack-trove | 02:26 | |
*** robertmyers has joined #openstack-trove | 02:31 | |
*** erkules_ has joined #openstack-trove | 02:34 | |
*** erkules has quit IRC | 02:37 | |
*** Longgeek has joined #openstack-trove | 02:37 | |
openstackgerrit | Merged openstack/trove: Updated from global requirements https://review.openstack.org/135995 | 02:40 |
*** Longgeek has quit IRC | 02:46 | |
*** Riddhi_ has quit IRC | 02:57 | |
*** Riddhi has joined #openstack-trove | 03:01 | |
openstackgerrit | Nikhil Manchanda proposed openstack/trove: Ensure Replication Tests do not use a stale token https://review.openstack.org/136219 | 03:05 |
openstackgerrit | Merged openstack/trove: Fix broken instance provisioning with disabled volume support https://review.openstack.org/136062 | 03:08 |
openstackgerrit | Merged openstack/trove: Deleting failed replication backup can hide error https://review.openstack.org/131593 | 03:08 |
*** amcrn has quit IRC | 03:17 | |
*** vkmc has quit IRC | 03:17 | |
*** nosnos has quit IRC | 03:24 | |
*** harlowja is now known as harlowja_away | 03:39 | |
*** Longgeek has joined #openstack-trove | 03:42 | |
*** Longgeek has quit IRC | 03:47 | |
*** ViswaV has joined #openstack-trove | 04:00 | |
*** johnma has quit IRC | 04:03 | |
*** Riddhi has quit IRC | 04:08 | |
*** Viswanath has joined #openstack-trove | 04:09 | |
*** Viswanath has quit IRC | 04:13 | |
*** mattgriffin has joined #openstack-trove | 04:20 | |
*** nosnos has joined #openstack-trove | 04:29 | |
*** sushilkm has quit IRC | 04:31 | |
*** edmund has quit IRC | 04:37 | |
*** rushiagr_away is now known as rushiagr | 04:43 | |
*** david-lyle_afk has quit IRC | 04:43 | |
*** Longgeek has joined #openstack-trove | 04:43 | |
*** Longgeek has quit IRC | 04:48 | |
*** ViswaV has quit IRC | 05:05 | |
*** nosnos has quit IRC | 05:14 | |
*** nosnos has joined #openstack-trove | 05:16 | |
*** erkules_ is now known as erkules | 05:25 | |
*** exploreshaifali has joined #openstack-trove | 05:28 | |
*** mattgriffin has quit IRC | 05:29 | |
*** nosnos has quit IRC | 05:59 | |
*** nosnos has joined #openstack-trove | 06:00 | |
*** ajayaa has joined #openstack-trove | 06:00 | |
*** Longgeek has joined #openstack-trove | 06:09 | |
*** k4n0 has joined #openstack-trove | 06:17 | |
*** tomblank has quit IRC | 06:50 | |
*** annashen has quit IRC | 06:53 | |
*** X019 has quit IRC | 07:33 | |
*** topshare has quit IRC | 07:34 | |
*** topshare has joined #openstack-trove | 07:43 | |
*** romainh has joined #openstack-trove | 08:11 | |
*** ajayaa has quit IRC | 08:21 | |
*** nosnos has quit IRC | 08:24 | |
*** nosnos has joined #openstack-trove | 08:24 | |
*** ajayaa has joined #openstack-trove | 08:33 | |
*** exploreshaifali has quit IRC | 08:45 | |
*** nosnos has quit IRC | 08:51 | |
*** nosnos has joined #openstack-trove | 08:51 | |
*** exploreshaifali has joined #openstack-trove | 08:54 | |
*** nosnos has quit IRC | 09:06 | |
*** boblebauce has joined #openstack-trove | 09:20 | |
*** exploreshaifali has quit IRC | 09:22 | |
*** nosnos has joined #openstack-trove | 09:34 | |
*** Longgeek has quit IRC | 09:51 | |
*** k4n0 has quit IRC | 09:53 | |
*** Longgeek has joined #openstack-trove | 10:04 | |
openstackgerrit | Li Ma proposed openstack/trove: Add nova_proxy_admin_tenant_id to make nova_compute_url effective https://review.openstack.org/111679 | 10:31 |
*** isviridov_away is now known as isvirido | 10:43 | |
*** isvirido is now known as isviridov | 10:43 | |
*** exploreshaifali has joined #openstack-trove | 10:47 | |
*** haomaiwang has quit IRC | 10:51 | |
*** topshare has quit IRC | 10:57 | |
*** exploreshaifali has quit IRC | 10:57 | |
*** ajayaa has quit IRC | 11:14 | |
*** ajayaa has joined #openstack-trove | 11:25 | |
*** vkmc has joined #openstack-trove | 11:44 | |
*** X019 has joined #openstack-trove | 11:58 | |
*** IanGovett has joined #openstack-trove | 12:19 | |
*** tomblank has joined #openstack-trove | 12:19 | |
*** ajayaa has quit IRC | 12:26 | |
*** pboros has joined #openstack-trove | 12:36 | |
*** ajayaa has joined #openstack-trove | 12:46 | |
*** IanGovett has quit IRC | 12:52 | |
*** IanGovett has joined #openstack-trove | 12:56 | |
*** X019 has quit IRC | 13:10 | |
zhiyan | hi folk, if/when you ok pls help reviews https://review.openstack.org/116653 (osprofiler integration stuff) ,thanks! | 13:12 |
*** nosnos has quit IRC | 13:16 | |
openstackgerrit | Denis M. proposed openstack/trove: Fix replica source state validation https://review.openstack.org/117222 | 13:24 |
openstackgerrit | Denis M. proposed openstack/trove: Fix replica source state validation https://review.openstack.org/117222 | 13:25 |
openstackgerrit | Denis M. proposed openstack/trove: Fix replica source state validation https://review.openstack.org/117222 | 13:26 |
*** dmakogon_ is now known as denis_makogon | 13:28 | |
denis_makogon | zhiyan, i will try it out during weekend (could you please provide some docs about how to combine it with python-troveclient and trove itself?). | 13:29 |
zhiyan | denis_makogon: nice! i explained some keys in the commit message of #116653 | 13:30 |
zhiyan | denis_makogon: do you think anything I need to supply here? | 13:30 |
amrith | denis_makogon, ping. | 13:35 |
denis_makogon | amrith, hi =) | 13:35 |
amrith | denis_makogon, I just saw https://review.openstack.org/#/c/117222/11 | 13:36 |
denis_makogon | zhiyan, will speak to you later | 13:36 |
amrith | and I'm confused. | 13:36 |
amrith | could you help me understand the change here please. | 13:36 |
amrith | please see my comment (I just posted) in the review | 13:36 |
amrith | does it make sense (it may not). | 13:36 |
zhiyan | amrith: ok, looking | 13:36 |
amrith | zhiyan, the question was to denis_makogon (but please feel free to look as well) ;) | 13:37 |
zhiyan | amrith: oops :) | 13:37 |
zhiyan | amrith: too positive for response in trove ..anyway, btw, amrith if you ok pls let me know your thoughts on 116653. thanks | 13:38 |
amrith | denis_makogon, we should also probably speak with SlickNik and understand what his thinking was. | 13:38 |
*** rushiagr is now known as rushiagr_away | 13:38 | |
amrith | zhiyan, it is on my list to review, this is osprofiler, yes? | 13:38 |
zhiyan | amrith: yep | 13:39 |
amrith | zhiyan, after we spoke in Paris, I wanted to actually try it out before just reading the code and attempting to review. that's why I haven't yet got to it. | 13:39 |
amrith | sorry. | 13:39 |
denis_makogon | amrith, yes, when i made #8 all those interactions were necessary due to need of them to call validate_can_perform_action | 13:39 |
amrith | denis_makogon, that was what I understood from reading _validate_can_perform_action() | 13:39 |
amrith | which seemed to be used by others as well | 13:40 |
amrith | but I felt that the checks were probably somewhat generic | 13:40 |
*** X019 has joined #openstack-trove | 13:40 | |
amrith | but it probably made sense to use it | 13:40 |
denis_makogon | for given bug we can actually use check that tries to see if instance status is really ACTIVE | 13:40 |
amrith | maybe that was a mistake on my part. | 13:40 |
amrith | denis_makogon, that's my point. | 13:40 |
zhiyan | amrith: ok, sensible. you can check the steps to try it out which i mentioned in commit msg. any question pls ping me feel free | 13:40 |
amrith | maybe in the other cases that use this validate_... just the check for active would suffice | 13:40 |
amrith | and in that case the better way to handle this would be to simplify the validate... code. | 13:41 |
denis_makogon | amrith, we need to lots of DB interactions in #8, but #11 really looks simpler | 13:41 |
amrith | but I also noticed that there are other validate functions as well. | 13:41 |
amrith | denis_makogon, no doubt, #11 looks simpler. | 13:41 |
denis_makogon | amrith, and it seems to do what we need - see if master is ACTIVE | 13:41 |
amrith | the question is (a) is it sufficient, and (b) is the heavier check from validate... required in the other cases. | 13:41 |
denis_makogon | we don't need to see if it's in BUILD/CRASHED or whatever can be | 13:42 |
amrith | well, not really right | 13:42 |
amrith | suppose a backup is in progress, can you create a replica at that time? | 13:42 |
denis_makogon | in which case we'd allow to create slave from instance that is not ACTIVE ? | 13:42 |
amrith | validate...() would check for that. | 13:42 |
amrith | agreed about CRASHED but what about BACKUP? | 13:43 |
denis_makogon | in terms of correct resource utilization i'd not allow to do this, because you can hit polling timeout because guest still doing backup | 13:44 |
denis_makogon | BACKUP state is a tricky thing with replication because you can hit race condition or not | 13:45 |
denis_makogon | for example if you have small set of data, and you'd be in time with doing backup and taking a snapshot for new slave | 13:46 |
amrith | that's a problem in that case, right? | 13:47 |
amrith | because the current check (#11) would allow you to go ahead | 13:47 |
amrith | service status would be "RUNNING" | 13:47 |
denis_makogon | from user experience point of view - it's so implicit because with the same instance i can create broken slave but during next attempt it'll be ACTIVE | 13:47 |
denis_makogon | with given code Trove will fail to create new instance from master that is not actually ACTIVE | 13:47 |
denis_makogon | so, as you can see, this code will fail with any status that differs from ACTIVE | 13:48 |
amrith | are you saying that in #11 (latest patch set) you would fail to create replica if a backup is in progress? | 13:49 |
amrith | currently you are checking for RUNNING | 13:49 |
amrith | service status RUNNING | 13:49 |
amrith | if (datastore_status.get_status() != | 13:49 |
amrith | tr_instance.ServiceStatuses.RUNNING): | 13:49 |
denis_makogon | let me try it out | 13:50 |
*** radez_g0n3 is now known as radez | 13:55 | |
openstackgerrit | Merged openstack/trove-specs: Enhance Mgmt-Show To Support Deleted Spec https://review.openstack.org/129752 | 13:59 |
openstackgerrit | Merged openstack/trove-specs: Make Rsync For Guest Optional Spec https://review.openstack.org/129740 | 13:59 |
denis_makogon | amrith, it seems like #11 would allow backuping | 14:02 |
denis_makogon | i'd like to fall back to #8 | 14:02 |
denis_makogon | because it looks more consistent, safer | 14:02 |
amrith | it seems that way to me. but before you do that, let's chat with nikhil | 14:05 |
amrith | no point in double and triple work. | 14:05 |
denis_makogon | amrith, sure, let's do this | 14:06 |
*** jcru has joined #openstack-trove | 14:09 | |
openstackgerrit | amrith proposed openstack/trove-specs: Switch Trove to use OSLO Messaging. https://review.openstack.org/136352 | 14:09 |
*** haomaiwang has joined #openstack-trove | 14:11 | |
openstackgerrit | Denis M. proposed openstack/trove: Forbid replica provisioning from replica site https://review.openstack.org/119771 | 14:11 |
denis_makogon | amrith, could you please take a look at https://review.openstack.org/119771 | 14:12 |
denis_makogon | amrith, and this one please, https://review.openstack.org/#/c/130519/ <--- we've talked at Paris about this one | 14:13 |
amrith | denis_makogon, I have to hop in a car now. will do it once I get to a place where I can. | 14:13 |
denis_makogon | amrith, appreciate it | 14:22 |
*** amrith is now known as _amrith_ | 14:24 | |
*** boblebauce has quit IRC | 14:25 | |
*** edmund has joined #openstack-trove | 14:25 | |
*** rushiagr_away is now known as rushiagr | 14:27 | |
*** topshare has joined #openstack-trove | 14:27 | |
*** topshare has quit IRC | 14:27 | |
*** boblebauce has joined #openstack-trove | 14:32 | |
*** newb has quit IRC | 14:36 | |
*** newb has joined #openstack-trove | 14:42 | |
*** mattgriffin has joined #openstack-trove | 15:11 | |
*** exploreshaifali has joined #openstack-trove | 15:14 | |
*** Barker has joined #openstack-trove | 15:21 | |
*** grapex has joined #openstack-trove | 15:31 | |
*** mattgriffin has quit IRC | 15:35 | |
*** mattgriffin has joined #openstack-trove | 15:38 | |
*** Riddhi has joined #openstack-trove | 15:45 | |
openstackgerrit | Monty Taylor proposed openstack/trove: Clean up github references from docs https://review.openstack.org/136400 | 15:54 |
*** Viswanath has joined #openstack-trove | 16:11 | |
*** Viswanath has quit IRC | 16:14 | |
denis_makogon | grapex, ping | 16:18 |
*** haomaiwang has quit IRC | 16:29 | |
*** haomaiwang has joined #openstack-trove | 16:39 | |
*** ajayaa has quit IRC | 16:52 | |
openstackgerrit | Doug Shelley proposed openstack/trove-specs: Add spec for add instance name to various commands in CLI https://review.openstack.org/136424 | 17:00 |
*** exploreshaifali has quit IRC | 17:07 | |
*** mattgriffin has quit IRC | 17:09 | |
*** grapex_ has joined #openstack-trove | 17:09 | |
*** grapex has quit IRC | 17:12 | |
*** romainh has left #openstack-trove | 17:13 | |
*** exploreshaifali has joined #openstack-trove | 17:21 | |
*** mattgriffin has joined #openstack-trove | 17:22 | |
*** Barker has quit IRC | 17:36 | |
openstackgerrit | Doug Shelley proposed openstack/trove-specs: Add spec for add instance name to various commands in CLI https://review.openstack.org/136424 | 17:37 |
*** boblebauce has quit IRC | 17:38 | |
*** dkehn has quit IRC | 17:39 | |
*** dkehnm has quit IRC | 17:42 | |
openstackgerrit | Riddhi Shah proposed openstack/trove: Forbid replica provisioning from replica site https://review.openstack.org/119771 | 17:44 |
*** sushilkm has joined #openstack-trove | 17:45 | |
*** X019 has quit IRC | 17:45 | |
*** X019 has joined #openstack-trove | 17:45 | |
*** dkehn has joined #openstack-trove | 17:46 | |
*** Longgeek has quit IRC | 17:49 | |
*** ajayaa has joined #openstack-trove | 17:50 | |
*** harlowja_away is now known as harlowja | 17:52 | |
*** annashen has joined #openstack-trove | 17:53 | |
*** mattgriffin has quit IRC | 17:56 | |
*** dkehn has quit IRC | 17:58 | |
*** dkehn has joined #openstack-trove | 17:59 | |
*** ViswaV has joined #openstack-trove | 18:02 | |
*** _amrith_ is now known as amrith | 18:06 | |
*** ViswaV_ has joined #openstack-trove | 18:06 | |
*** ViswaV has quit IRC | 18:07 | |
*** annashen has quit IRC | 18:17 | |
*** annashen has joined #openstack-trove | 18:17 | |
*** annashen has quit IRC | 18:21 | |
*** newb has quit IRC | 18:22 | |
*** exploreshaifali has quit IRC | 18:23 | |
*** Barker has joined #openstack-trove | 18:23 | |
*** amcrn has joined #openstack-trove | 18:25 | |
openstackgerrit | Auston McReynolds proposed openstack/trove: Config Group Load Fails If DS Version Inactive https://review.openstack.org/129145 | 18:25 |
*** ViswaV_ has quit IRC | 18:28 | |
*** ViswaV has joined #openstack-trove | 18:30 | |
*** ViswaV_ has joined #openstack-trove | 18:31 | |
*** ViswaV has quit IRC | 18:34 | |
*** mattgriffin has joined #openstack-trove | 18:37 | |
*** mattgriffin has quit IRC | 18:51 | |
*** mattgriffin has joined #openstack-trove | 18:51 | |
*** annashen has joined #openstack-trove | 19:11 | |
*** amcrn has quit IRC | 19:13 | |
*** amrith is now known as _amrith_ | 19:14 | |
*** annashen has quit IRC | 19:16 | |
*** radez is now known as radez_g0n3 | 19:21 | |
*** johnma has joined #openstack-trove | 19:24 | |
*** newb has joined #openstack-trove | 19:53 | |
*** mattgriffin has quit IRC | 20:16 | |
*** mattgriffin has joined #openstack-trove | 20:24 | |
*** agomez has joined #openstack-trove | 20:31 | |
*** _amrith_ is now known as amrith | 20:40 | |
openstackgerrit | Merged openstack/trove: Create example generator https://review.openstack.org/132367 | 20:43 |
*** rushiagr is now known as rushiagr_away | 20:50 | |
*** ajayaa has quit IRC | 20:56 | |
*** shakamunyi_ has joined #openstack-trove | 21:02 | |
*** shakamunyi_ has quit IRC | 21:26 | |
*** pboros has quit IRC | 21:44 | |
-openstackstatus- NOTICE: gating is going offline while we deal with a broken block device, eta unknown | 21:46 | |
*** ChanServ changes topic to "gating is going offline while we deal with a broken block device, eta unknown" | 21:46 | |
*** mattgriffin has quit IRC | 21:51 | |
*** jcru has quit IRC | 21:51 | |
*** grapex_ has quit IRC | 22:04 | |
sgotliv | amrith, ping | 22:08 |
*** Riddhi has quit IRC | 22:12 | |
*** annashen has joined #openstack-trove | 22:12 | |
*** annashen_ has joined #openstack-trove | 22:18 | |
*** annashen has quit IRC | 22:20 | |
*** X019 has quit IRC | 22:22 | |
*** Barker has quit IRC | 22:34 | |
*** annashen_ has quit IRC | 22:35 | |
*** mattgriffin has joined #openstack-trove | 22:36 | |
*** vkmc has quit IRC | 23:01 | |
*** Riddhi has joined #openstack-trove | 23:01 | |
*** mattgriffin has quit IRC | 23:05 | |
*** annashen has joined #openstack-trove | 23:07 | |
*** annashen has quit IRC | 23:08 | |
*** mattgriffin has joined #openstack-trove | 23:09 | |
*** mattgriffin has quit IRC | 23:21 | |
*** ci-testing has joined #openstack-trove | 23:24 | |
ci-testing | Hi, is there a problem with "tarballs.openstack.org" I can't access/download any files | 23:25 |
*** annashen has joined #openstack-trove | 23:27 | |
*** ci-testing has quit IRC | 23:30 | |
*** ci-testing has joined #openstack-trove | 23:31 | |
*** mattgriffin has joined #openstack-trove | 23:34 | |
*** mattgriffin has quit IRC | 23:38 | |
*** Barker has joined #openstack-trove | 23:42 | |
*** annashen has quit IRC | 23:48 | |
*** ci-testing has left #openstack-trove | 23:48 | |
*** Barker has quit IRC | 23:51 | |
*** annashen_ has joined #openstack-trove | 23:52 | |
*** IanGovett has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!