| *** matsuhashi has joined #openstack-trove | 00:27 | |
| openstackgerrit | Dan Nguyen proposed a change to openstack/trove: Add a new column and indexes to agent_heartbeats https://review.openstack.org/81682 | 00:27 |
|---|---|---|
| *** eguz has quit IRC | 00:28 | |
| *** michael-yu has quit IRC | 00:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 00:40 |
| *** yidclare has quit IRC | 00:48 | |
| *** haomaiwang has joined #openstack-trove | 01:10 | |
| *** cweid has quit IRC | 01:10 | |
| *** haomaiw__ has joined #openstack-trove | 01:12 | |
| *** haomaiwang has quit IRC | 01:15 | |
| *** ramashri has quit IRC | 01:22 | |
| *** shakayumi has joined #openstack-trove | 01:30 | |
| *** NehaV has joined #openstack-trove | 01:31 | |
| *** nosnos has joined #openstack-trove | 01:33 | |
| *** sbfox1 has quit IRC | 01:37 | |
| *** annashe__ has quit IRC | 01:38 | |
| *** annashe__ has joined #openstack-trove | 01:38 | |
| *** annashe__ has quit IRC | 01:42 | |
| *** saurabhs has left #openstack-trove | 01:50 | |
| *** ViswaV_ has joined #openstack-trove | 01:54 | |
| *** ViswaV has quit IRC | 01:54 | |
| *** amcrn has quit IRC | 02:08 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Add instance_metadata functionality to the trove python library https://review.openstack.org/82124 | 02:23 |
| *** ramashri has joined #openstack-trove | 02:24 | |
| *** ramashri has quit IRC | 02:45 | |
| *** rwsu has quit IRC | 02:52 | |
| *** NehaV has quit IRC | 02:57 | |
| *** NehaV has joined #openstack-trove | 02:57 | |
| *** tvoran has joined #openstack-trove | 03:00 | |
| *** tvoran has quit IRC | 03:01 | |
| *** tvoran has joined #openstack-trove | 03:01 | |
| *** grapex has joined #openstack-trove | 03:07 | |
| *** grapex has joined #openstack-trove | 03:08 | |
| *** ramishra has joined #openstack-trove | 03:10 | |
| *** michael-yu has joined #openstack-trove | 03:13 | |
| *** michael-yu has quit IRC | 03:15 | |
| *** haomaiw__ has quit IRC | 03:27 | |
| *** haomaiwang has joined #openstack-trove | 03:27 | |
| *** NehaV has quit IRC | 03:28 | |
| *** tvoran_ has joined #openstack-trove | 03:31 | |
| *** tvoran_ has quit IRC | 03:33 | |
| *** tvoran_ has joined #openstack-trove | 03:34 | |
| *** tvoran has quit IRC | 03:35 | |
| *** nosnos has quit IRC | 03:43 | |
| *** annashe__ has joined #openstack-trove | 03:47 | |
| *** grapex has quit IRC | 03:51 | |
| *** Longgeek has joined #openstack-trove | 03:52 | |
| *** eghobo has joined #openstack-trove | 03:55 | |
| *** eghobo has quit IRC | 03:55 | |
| *** eghobo has joined #openstack-trove | 03:56 | |
| *** coolsvap|afk is now known as coolsvap | 03:57 | |
| *** Longgeek has quit IRC | 03:57 | |
| *** tvoran_ has quit IRC | 04:01 | |
| *** ramishra has quit IRC | 04:17 | |
| *** ramishra has joined #openstack-trove | 04:21 | |
| *** nosnos has joined #openstack-trove | 04:24 | |
| *** setho has joined #openstack-trove | 04:24 | |
| *** setho2 has quit IRC | 04:26 | |
| *** eghobo has quit IRC | 04:36 | |
| *** eghobo has joined #openstack-trove | 04:50 | |
| *** eghobo has quit IRC | 04:54 | |
| *** eghobo has joined #openstack-trove | 04:54 | |
| *** esp has left #openstack-trove | 04:55 | |
| *** michael-yu has joined #openstack-trove | 05:04 | |
| *** sbfox has joined #openstack-trove | 05:07 | |
| *** SushillKM has joined #openstack-trove | 05:32 | |
| *** michael-yu has quit IRC | 05:45 | |
| *** Longgeek has joined #openstack-trove | 05:49 | |
| *** Longgeek has quit IRC | 05:49 | |
| *** eguz has joined #openstack-trove | 05:50 | |
| *** eghobo has quit IRC | 05:50 | |
| *** Longgeek has joined #openstack-trove | 05:50 | |
| *** Longgeek has quit IRC | 05:51 | |
| *** Longgeek has joined #openstack-trove | 05:51 | |
| *** achampion has joined #openstack-trove | 05:53 | |
| *** SushilKM has joined #openstack-trove | 05:55 | |
| *** SushillKM has quit IRC | 05:57 | |
| *** michael-yu has joined #openstack-trove | 06:00 | |
| *** annashe__ has quit IRC | 06:02 | |
| *** SushilKM has quit IRC | 06:03 | |
| *** Longgeek has quit IRC | 06:04 | |
| *** Longgeek has joined #openstack-trove | 06:04 | |
| *** Longgeek has quit IRC | 06:04 | |
| *** Longgeek has joined #openstack-trove | 06:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex https://review.openstack.org/97008 | 06:06 |
| *** Longgeek has quit IRC | 06:06 | |
| *** Longgeek_ has joined #openstack-trove | 06:06 | |
| *** ramishra has quit IRC | 06:08 | |
| *** ramishra has joined #openstack-trove | 06:08 | |
| *** SushilKM has joined #openstack-trove | 06:39 | |
| *** eguz has quit IRC | 06:41 | |
| *** SushilKM has quit IRC | 06:50 | |
| *** michael-yu has quit IRC | 06:52 | |
| *** SushilKM has joined #openstack-trove | 06:52 | |
| *** r0j4z0 has quit IRC | 06:56 | |
| *** nosnos has quit IRC | 06:59 | |
| *** matsuhashi has quit IRC | 07:01 | |
| *** matsuhas_ has joined #openstack-trove | 07:01 | |
| *** nosnos has joined #openstack-trove | 07:05 | |
| *** flaper87|afk is now known as flaper87 | 07:06 | |
| *** sgotliv has joined #openstack-trove | 07:30 | |
| *** achampion has quit IRC | 07:33 | |
| *** sbfox has quit IRC | 08:03 | |
| *** julienvey has joined #openstack-trove | 08:07 | |
| *** ramishra has quit IRC | 08:12 | |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/python-troveclient: Remove RAX-specific auth in troveclient https://review.openstack.org/69081 | 08:14 |
| *** ramishra has joined #openstack-trove | 08:19 | |
| *** matsuhas_ has quit IRC | 08:38 | |
| *** SushilKM has quit IRC | 08:38 | |
| *** nosnos has quit IRC | 08:40 | |
| *** nosnos has joined #openstack-trove | 08:43 | |
| *** nosnos has quit IRC | 08:44 | |
| *** matsuhashi has joined #openstack-trove | 08:44 | |
| *** ramishra has quit IRC | 08:47 | |
| *** sgotliv has quit IRC | 08:51 | |
| *** nosnos has joined #openstack-trove | 09:03 | |
| *** denis_makogon has joined #openstack-trove | 09:03 | |
| *** ramishra has joined #openstack-trove | 09:06 | |
| *** sgotliv has joined #openstack-trove | 09:07 | |
| *** denis_makogon has quit IRC | 09:10 | |
| *** isviridov_ is now known as isviridov | 09:12 | |
| *** denis_makogon has joined #openstack-trove | 09:13 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add datatore version name into configuration response https://review.openstack.org/98985 | 09:30 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add datatore and version name into configuration response https://review.openstack.org/98985 | 09:31 |
| openstackgerrit | Denis M. proposed a change to openstack/python-troveclient: Add datatore and version name into configuration shell view https://review.openstack.org/98988 | 09:36 |
| *** ramishra has quit IRC | 09:36 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Support full backup/restore for Cassandra https://review.openstack.org/77551 | 09:40 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add backup size validation at restoring https://review.openstack.org/86300 | 09:43 |
| *** sgotliv has quit IRC | 09:44 | |
| *** sgotliv has joined #openstack-trove | 09:57 | |
| *** julienvey has quit IRC | 09:59 | |
| openstackgerrit | Shalini khandelwal proposed a change to openstack/trove: Initial support for Vertica Datastore in trove https://review.openstack.org/91761 | 10:17 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Remove redundant heat templates from codebase https://review.openstack.org/99000 | 10:22 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for cached guest-images https://review.openstack.org/95304 | 10:23 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images https://review.openstack.org/91487 | 10:23 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for cached guest-images https://review.openstack.org/95304 | 10:27 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images https://review.openstack.org/91487 | 10:27 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Refactor taskmanager. Split heat/native/migration code path https://review.openstack.org/98404 | 10:29 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add stack_id to base instance model https://review.openstack.org/98403 | 10:29 |
| denis_makogon | SlickNik, hey, could you please take a look https://review.openstack.org/#/c/98403/, it was proposed for J1 | 10:30 |
| openstackgerrit | Shuichiro MAKIGAKI proposed a change to openstack/trove: Add warnings when a path for api_extensions_path does not exist https://review.openstack.org/99001 | 10:31 |
| openstackgerrit | Shalini khandelwal proposed a change to openstack/trove-integration: Initial support for Vertica Datastore in trove https://review.openstack.org/89804 | 10:35 |
| openstackgerrit | Shuichiro MAKIGAKI proposed a change to openstack/trove: Add warnings when a path for api_extensions_path does not exist https://review.openstack.org/99001 | 10:40 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add backup size validation at restoring https://review.openstack.org/86300 | 10:47 |
| *** sgotliv has quit IRC | 10:49 | |
| *** sgotliv has joined #openstack-trove | 10:57 | |
| *** julienvey has joined #openstack-trove | 10:59 | |
| *** SushilKM has joined #openstack-trove | 11:02 | |
| *** coolsvap is now known as coolsvap|afk | 11:04 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add datatore and version name into configuration response https://review.openstack.org/98985 | 11:21 |
| openstackgerrit | Shuichiro MAKIGAKI proposed a change to openstack/trove: Add warnings when a path for api_extensions_path does not exist https://review.openstack.org/99001 | 11:25 |
| *** grapex has joined #openstack-trove | 11:25 | |
| *** grapex has quit IRC | 11:30 | |
| openstackgerrit | Shalini khandelwal proposed a change to openstack/trove: Initial support for Vertica Datastore in trove https://review.openstack.org/91761 | 11:35 |
| *** miqui has quit IRC | 11:38 | |
| *** radez_g0n3 is now known as radez | 12:05 | |
| *** grapex has joined #openstack-trove | 12:09 | |
| *** grapex has quit IRC | 12:13 | |
| *** sgotliv has quit IRC | 12:15 | |
| *** sgotliv has joined #openstack-trove | 12:15 | |
| *** Longgeek_ has quit IRC | 12:25 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Add datastore filter to backup-list https://review.openstack.org/90462 | 12:26 |
| *** Longgeek has joined #openstack-trove | 12:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 12:33 |
| *** julienvey has quit IRC | 12:35 | |
| openstackgerrit | Greg Lucas proposed a change to openstack/python-troveclient: Add 'slave_of' option for enabling replication https://review.openstack.org/97840 | 12:39 |
| openstackgerrit | Greg Lucas proposed a change to openstack/trove: Add slave_of_id to instance model https://review.openstack.org/97846 | 12:42 |
| *** pdmars has joined #openstack-trove | 12:44 | |
| *** pdmars has quit IRC | 12:44 | |
| *** SushilKM has quit IRC | 12:44 | |
| *** pdmars has joined #openstack-trove | 12:45 | |
| *** julienvey has joined #openstack-trove | 12:49 | |
| *** julienvey has quit IRC | 13:04 | |
| *** miqui has joined #openstack-trove | 13:05 | |
| *** julienvey has joined #openstack-trove | 13:06 | |
| *** julienvey has quit IRC | 13:11 | |
| *** Barker has joined #openstack-trove | 13:26 | |
| *** tvoran has joined #openstack-trove | 13:27 | |
| *** julienvey has joined #openstack-trove | 13:35 | |
| *** matsuhashi has quit IRC | 13:39 | |
| *** julienvey has quit IRC | 13:40 | |
| *** Longgeek has quit IRC | 13:42 | |
| *** Longgeek has joined #openstack-trove | 13:42 | |
| *** julienvey has joined #openstack-trove | 13:43 | |
| *** rueben has joined #openstack-trove | 13:45 | |
| *** kevinconway has joined #openstack-trove | 13:45 | |
| *** rueben has quit IRC | 13:45 | |
| *** nosnos has quit IRC | 13:50 | |
| *** rueben has joined #openstack-trove | 13:51 | |
| *** rueben has quit IRC | 13:51 | |
| *** grapex has joined #openstack-trove | 13:54 | |
| *** grapex has quit IRC | 13:54 | |
| *** achampion has joined #openstack-trove | 13:55 | |
| *** grapex has joined #openstack-trove | 13:55 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Remove redundant heat templates from codebase https://review.openstack.org/99000 | 13:56 |
| *** rueben has joined #openstack-trove | 13:57 | |
| *** rueben has quit IRC | 13:58 | |
| *** rueben has joined #openstack-trove | 13:59 | |
| *** amytron has joined #openstack-trove | 14:01 | |
| *** iccha has joined #openstack-trove | 14:01 | |
| *** sgotliv_ has joined #openstack-trove | 14:02 | |
| *** NehaV has joined #openstack-trove | 14:02 | |
| *** sgotliv has quit IRC | 14:03 | |
| denis_makogon | SlickNik, grapex, vipul, guys, please take a look at https://review.openstack.org/#/c/84964/, it blocks us from adding datastore gates | 14:11 |
| *** jcru has joined #openstack-trove | 14:11 | |
| *** NehaV1 has joined #openstack-trove | 14:14 | |
| *** NehaV has quit IRC | 14:16 | |
| *** miqui has quit IRC | 14:23 | |
| *** miqui has joined #openstack-trove | 14:23 | |
| *** miqui has quit IRC | 14:23 | |
| *** miqui has joined #openstack-trove | 14:23 | |
| *** isviridov is now known as isviridov|away | 14:24 | |
| *** jcru has quit IRC | 14:27 | |
| *** jcru has joined #openstack-trove | 14:28 | |
| *** jasonb365 has joined #openstack-trove | 14:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements https://review.openstack.org/99095 | 14:40 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 14:40 |
| *** thedodd has joined #openstack-trove | 14:45 | |
| *** jmontemayor has joined #openstack-trove | 14:48 | |
| *** jmontemayor has quit IRC | 14:48 | |
| *** jmontemayor has joined #openstack-trove | 14:50 | |
| *** isviridov|away is now known as isviridov | 14:56 | |
| *** NehaV1 has left #openstack-trove | 15:02 | |
| *** rueben has quit IRC | 15:02 | |
| *** sbfox has joined #openstack-trove | 15:02 | |
| *** rueben has joined #openstack-trove | 15:03 | |
| *** Gordon has joined #openstack-trove | 15:07 | |
| *** Gordon is now known as Guest72113 | 15:07 | |
| *** ViswaV has joined #openstack-trove | 15:12 | |
| *** ViswaV_ has quit IRC | 15:14 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Fix int-tests for different datastores https://review.openstack.org/99126 | 15:14 |
| *** Barker has quit IRC | 15:16 | |
| *** jasonb365 has quit IRC | 15:16 | |
| *** jasonb365 has joined #openstack-trove | 15:17 | |
| *** NehaV has joined #openstack-trove | 15:22 | |
| *** SushilKM has joined #openstack-trove | 15:23 | |
| *** Barker has joined #openstack-trove | 15:27 | |
| *** thedodd has quit IRC | 15:32 | |
| *** imsplitbit has joined #openstack-trove | 15:33 | |
| *** thedodd has joined #openstack-trove | 15:34 | |
| *** sgotliv_ has quit IRC | 15:35 | |
| *** Longgeek has quit IRC | 15:39 | |
| *** edmund has joined #openstack-trove | 15:39 | |
| *** SushilKM has quit IRC | 15:40 | |
| *** SushilKM has joined #openstack-trove | 15:40 | |
| *** robertmyers has joined #openstack-trove | 15:41 | |
| *** cweid has joined #openstack-trove | 15:49 | |
| *** sbfox has quit IRC | 15:57 | |
| *** esp has joined #openstack-trove | 15:57 | |
| amrith | hi vipul, yt? | 15:58 |
| *** demorris has joined #openstack-trove | 16:00 | |
| openstackgerrit | Tim Simpson proposed a change to openstack/trove: Minor test tweak to run without an instance https://review.openstack.org/98543 | 16:01 |
| *** coolsvap|afk is now known as coolsvap | 16:01 | |
| *** sbfox has joined #openstack-trove | 16:02 | |
| *** cp16net_ has joined #openstack-trove | 16:02 | |
| *** sbfox has quit IRC | 16:02 | |
| amrith | SushilKM, you there? | 16:02 |
| *** Barker has quit IRC | 16:03 | |
| *** sbfox has joined #openstack-trove | 16:03 | |
| SushilKM | yeah @amrith | 16:03 |
| *** iccha has quit IRC | 16:03 | |
| amrith | hi SushilKM ... | 16:04 |
| amrith | re: https://review.openstack.org/#/c/95334/1 | 16:04 |
| amrith | I don't understand why this fix works. That (by itself) isn't a reason to -1 a change. But, thanks to your comment on patch set 1, you point me to https://review.openstack.org/#/c/75516/2. | 16:04 |
| amrith | I notice in the review there that it wasn't clear why the fix worked either (see exchange between vipul and amcrn in that review). | 16:04 |
| *** rueben has quit IRC | 16:04 | |
| amrith | do you know why this fix works? | 16:04 |
| SushilKM | let me chk | 16:05 |
| amrith | thx | 16:05 |
| SushilKM | ok this one | 16:05 |
| *** Barker has joined #openstack-trove | 16:05 | |
| SushilKM | yeah reason i m also not aware ... just had a guess | 16:05 |
| amrith | no guess | 16:05 |
| SushilKM | :( | 16:05 |
| SushilKM | something was distorting the results | 16:06 |
| *** cp16net_ has quit IRC | 16:06 | |
| amrith | I thought it was that there was a call to del links but _print_instance has one of those ... | 16:06 |
| amrith | so why this works beats me | 16:06 |
| SushilKM | i chkd it a number of times thinking different theories but everytime i thought one was perfect it came null .... :( | 16:07 |
| amrith | ok, thx! | 16:07 |
| SushilKM | once u figure it out let me know too :) | 16:08 |
| *** iccha has joined #openstack-trove | 16:08 | |
| *** ramishra has joined #openstack-trove | 16:09 | |
| *** julienvey has quit IRC | 16:14 | |
| *** rueben has joined #openstack-trove | 16:16 | |
| *** denis_makogon_ has joined #openstack-trove | 16:27 | |
| *** NehaV has left #openstack-trove | 16:28 | |
| *** PeterSchwaller_ has joined #openstack-trove | 16:28 | |
| *** PeterSchwaller has quit IRC | 16:30 | |
| denis_makogon_ | SlickNik, vipul, grapex, hub_cap, cp16net, hey, guys, please take a look at this patch, it blocks adding 3d-party testing gates for other datastores(except mysql-server-5.X) https://review.openstack.org/99126 | 16:32 |
| SlickNik | denis_makogon: This is a duplicate of https://review.openstack.org/#/c/84964/ which you have already commented on. | 16:38 |
| SlickNik | denis_makogon: What's the purpose of adding a competing patchset? | 16:38 |
| *** iccha has quit IRC | 16:38 | |
| *** isviridov is now known as isviridov|away | 16:40 | |
| *** demorris has quit IRC | 16:40 | |
| *** SushilKM has quit IRC | 16:41 | |
| *** michael-yu has joined #openstack-trove | 16:41 | |
| imsplitbit | whats up party people | 16:43 |
| *** Barker has quit IRC | 16:44 | |
| *** Barker has joined #openstack-trove | 16:45 | |
| *** Barker has quit IRC | 16:45 | |
| *** Barker has joined #openstack-trove | 16:46 | |
| *** ramashri has joined #openstack-trove | 16:46 | |
| *** Barker has quit IRC | 16:49 | |
| *** Barker has joined #openstack-trove | 16:50 | |
| *** yogeshmehra has joined #openstack-trove | 16:51 | |
| *** denis_makogon has quit IRC | 17:00 | |
| *** denis_makogon_ is now known as denis_makogon | 17:00 | |
| *** dmakogon_ has joined #openstack-trove | 17:00 | |
| *** sbfox has quit IRC | 17:02 | |
| denis_makogon | SlickNik, it was just a test to see if it works for all datastore and then suggest to Shalini how to deal with it in a best way. | 17:04 |
| *** sbfox has joined #openstack-trove | 17:07 | |
| *** michael-yu has quit IRC | 17:09 | |
| openstackgerrit | Steve Leon proposed a change to openstack/trove: Add datastore version to backups https://review.openstack.org/89474 | 17:09 |
| *** yidclare has joined #openstack-trove | 17:10 | |
| *** sbfox1 has joined #openstack-trove | 17:11 | |
| denis_makogon | SlickNik, could you take a look at https://review.openstack.org/#/c/98403/ ? | 17:12 |
| *** sbfox1 has quit IRC | 17:14 | |
| *** sbfox1 has joined #openstack-trove | 17:14 | |
| *** michael-yu has joined #openstack-trove | 17:14 | |
| *** sbfox has quit IRC | 17:15 | |
| glucas | denis_makogon: question about your comments on this review - https://review.openstack.org/#/c/98147/1/trove/guestagent/strategies/replication/base.py | 17:16 |
| glucas | denis_makogon: you said 'class should be inherited from Strategy with ABC metaclass' but it looks like what vgnbkr has done here is consistent with the other strategies. | 17:17 |
| denis_makogon | glucas, yes, but if you're proposing interface, whole class should come with metaclass defined as ABCMeta | 17:19 |
| denis_makogon | glucas, it is 100% pythonic way to work with interfaces, since there are no strict types | 17:20 |
| denis_makogon | http://stackoverflow.com/questions/4382945/abstract-methods-in-python | 17:21 |
| denis_makogon | https://docs.python.org/3/library/abc.html | 17:21 |
| glucas | denis_makogon: thx for the refs, just comparing to e.g. strategies/storage/base.py and seems consistent.. The implication is the same should have been done there as well, I guess. | 17:22 |
| denis_makogon | glucas, agreed | 17:22 |
| openstackgerrit | Dan Nguyen proposed a change to openstack/trove: Updates RPC API to use oslo.messaging for versions https://review.openstack.org/94484 | 17:22 |
| glucas | denis_makogon: thx | 17:23 |
| denis_makogon | glucas, thanks to you too | 17:23 |
| grapex | esmute: I responded to your most recent PRs: https://review.openstack.org/#/c/89474/14 | 17:26 |
| grapex | The view still doesn't have the datastore ID... :( | 17:26 |
| esmute | ahhh sorry i read your review comment fast | 17:27 |
| esmute | i thought i only had the version but not the type | 17:28 |
| esmute | so you want me to add the datastore version id as well in the view? | 17:28 |
| esmute | grapex ^ | 17:28 |
| esmute | why do you think we should have the version ID when the name is already there? | 17:29 |
| grapex | esmute: because there will be some situation where some poor person will need the ID for whatever reason | 17:30 |
| grapex | and they'll have to search the version's to find it, which will be a drag. | 17:31 |
| esmute | "version_id": self.backup.datastore_version.id | 17:31 |
| esmute | good with that? ^ | 17:32 |
| grapex | esmute: Looks good | 17:32 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for cached guest-images https://review.openstack.org/95304 | 17:32 |
| openstackgerrit | Dan Nguyen proposed a change to openstack/trove: Partially implements guest agent upgrade strategy https://review.openstack.org/85225 | 17:32 |
| grapex | esmute: also "type" and "datastore" in the view are the same thing- are you going to lose "datastore"? | 17:32 |
| *** michael-yu has quit IRC | 17:33 | |
| esmute | my bad, i glanced at your last comment and thought i was missing the datastore name instead.. so i added it | 17:33 |
| esmute | grapex: yeah.. that will go away... | 17:33 |
| grapex | esmute: cool | 17:33 |
| grapex | esmute: Thanks | 17:34 |
| *** michael-yu has joined #openstack-trove | 17:34 | |
| *** denis_makogon has quit IRC | 17:37 | |
| michael-yu | robertmyers - ping | 17:38 |
| robertmyers | michael-yu: hello | 17:40 |
| michael-yu | hi robert | 17:40 |
| michael-yu | https://review.openstack.org/#/c/75269/ | 17:40 |
| michael-yu | single drop down using optgroup | 17:40 |
| robertmyers | nice | 17:40 |
| michael-yu | oh it's not done | 17:40 |
| michael-yu | :) | 17:40 |
| michael-yu | one question | 17:40 |
| robertmyers | oh | 17:40 |
| openstackgerrit | Dan Nguyen proposed a change to openstack/trove: Partially implements guest agent upgrade strategy https://review.openstack.org/85225 | 17:40 |
| michael-yu | almost there | 17:40 |
| michael-yu | if there's more than 1 datastore… should the first be auto selected? | 17:41 |
| michael-yu | or, if there's more than 1 datastore… should we add an "empty" option to enforce the user to choose one? | 17:42 |
| michael-yu | thoughts? | 17:42 |
| *** thedodd has quit IRC | 17:42 | |
| robertmyers | hmm, I'm guessing we should make an empty choice | 17:42 |
| michael-yu | that's what i was thinking too | 17:43 |
| robertmyers | that way the user has to decide which one they want | 17:43 |
| robertmyers | rather than opps I got mysql | 17:43 |
| michael-yu | yup | 17:43 |
| michael-yu | cool. thanks. | 17:43 |
| robertmyers | np | 17:43 |
| grapex | esp: Let me know if you're ready for me to review that. | 17:44 |
| esp | hey grapex: I was just about to ping you | 17:44 |
| esp | grapex: so I made the changes we talked about last wrt to moving the versioning out to be configurable | 17:45 |
| *** ViswaV_ has joined #openstack-trove | 17:45 | |
| esp | and extracting the Heartbeat logic into it’s own class. | 17:45 |
| esp | the thing that is a little bit different is that I didn’t move the Heartbeat class method call into each manager. | 17:46 |
| esp | This was to be more conservative with the change | 17:47 |
| *** sgotliv_ has joined #openstack-trove | 17:47 | |
| esp | if you would still rather I move it let me know | 17:47 |
| grapex | esp: ok. I'm pretty busy today but I'll try to get to it as soon as I can | 17:48 |
| grapex | esp: Thanks. | 17:48 |
| *** sbfox1 has quit IRC | 17:48 | |
| *** sbfox has joined #openstack-trove | 17:48 | |
| esp | grapex: np, thx for touching base! | 17:48 |
| *** ViswaV has quit IRC | 17:48 | |
| openstackgerrit | Steve Leon proposed a change to openstack/trove: Add datastore version to backups https://review.openstack.org/89474 | 17:51 |
| *** haomaiwang has quit IRC | 17:52 | |
| *** SushilKM has joined #openstack-trove | 17:52 | |
| *** ViswaV has joined #openstack-trove | 17:54 | |
| *** ViswaV_ has quit IRC | 17:56 | |
| *** sbfox has quit IRC | 17:58 | |
| *** rueben has quit IRC | 18:00 | |
| *** Barker has quit IRC | 18:01 | |
| *** rwsu has joined #openstack-trove | 18:02 | |
| *** Barker has joined #openstack-trove | 18:02 | |
| *** rueben has joined #openstack-trove | 18:02 | |
| *** sbfox has joined #openstack-trove | 18:04 | |
| *** jmontemayor has quit IRC | 18:08 | |
| *** jmontemayor has joined #openstack-trove | 18:10 | |
| *** jmontemayor has quit IRC | 18:11 | |
| *** jmontemayor has joined #openstack-trove | 18:12 | |
| *** jmontemayor has quit IRC | 18:12 | |
| *** jmontemayor has joined #openstack-trove | 18:13 | |
| *** coolsvap is now known as coolsvap|afk | 18:17 | |
| *** saurabhs has joined #openstack-trove | 18:24 | |
| *** SushilKM has quit IRC | 18:31 | |
| *** thedodd has joined #openstack-trove | 18:46 | |
| *** yogeshmehra has quit IRC | 18:58 | |
| *** agingh1pster has quit IRC | 19:00 | |
| *** spiffxp_ has quit IRC | 19:01 | |
| *** SlickN1k has quit IRC | 19:01 | |
| *** Barker has quit IRC | 19:02 | |
| *** Barker has joined #openstack-trove | 19:03 | |
| *** thedodd has quit IRC | 19:04 | |
| *** thedodd has joined #openstack-trove | 19:05 | |
| *** michael-yu has quit IRC | 19:08 | |
| *** michael-yu has joined #openstack-trove | 19:12 | |
| *** sbfox has quit IRC | 19:13 | |
| *** iccha has joined #openstack-trove | 19:14 | |
| *** yogeshmehra has joined #openstack-trove | 19:15 | |
| *** iccha has quit IRC | 19:19 | |
| *** cp16net__ has joined #openstack-trove | 19:23 | |
| *** iccha has joined #openstack-trove | 19:28 | |
| *** ramishra has quit IRC | 19:28 | |
| *** amcrn has joined #openstack-trove | 19:31 | |
| *** pdmars_ has joined #openstack-trove | 19:33 | |
| *** ramashri has quit IRC | 19:35 | |
| *** pdmars has quit IRC | 19:35 | |
| *** jmontemayor has quit IRC | 19:36 | |
| *** michael-yu has quit IRC | 19:43 | |
| openstackgerrit | A change was merged to openstack/trove: Fix calls for proboscis methods https://review.openstack.org/97478 | 19:46 |
| *** yidclare has quit IRC | 19:47 | |
| *** thedodd has quit IRC | 19:47 | |
| *** jmontemayor has joined #openstack-trove | 19:48 | |
| *** thedodd has joined #openstack-trove | 19:50 | |
| *** michael-yu has joined #openstack-trove | 19:56 | |
| *** ViswaV has quit IRC | 19:58 | |
| *** jasonb365 has quit IRC | 20:02 | |
| *** sbfox has joined #openstack-trove | 20:06 | |
| *** sbfox has quit IRC | 20:06 | |
| *** jasonb365 has joined #openstack-trove | 20:06 | |
| *** sbfox has joined #openstack-trove | 20:06 | |
| *** Barker has quit IRC | 20:10 | |
| *** Barker has joined #openstack-trove | 20:12 | |
| *** sbfox has quit IRC | 20:12 | |
| *** denis_makogon has joined #openstack-trove | 20:15 | |
| *** ViswaV has joined #openstack-trove | 20:16 | |
| *** miqui has quit IRC | 20:20 | |
| *** saurabhs has quit IRC | 20:22 | |
| denis_makogon | yogeshmehra, ping | 20:27 |
| *** demorris has joined #openstack-trove | 20:29 | |
| *** Barker has quit IRC | 20:33 | |
| *** esmute has quit IRC | 20:33 | |
| *** esmute has joined #openstack-trove | 20:33 | |
| peterstac | anyone know what's happening with trove-gate? | 20:41 |
| peterstac | #link https://rdjenkins.dyndns.org/job/Trove-Gate/ | 20:41 |
| *** esp_ has quit IRC | 20:42 | |
| SlickNik | peterstac: What's up with it? | 20:44 |
| peterstac | SlickNik: seems to be backed up - just looking at some of the logs now | 20:45 |
| SlickNik | Yes, lots of patches coming in since last night. | 20:45 |
| SlickNik | I've upped the workers to 6, but it's still chugging away. | 20:46 |
| SlickNik | Also looks like there's still a timing issue with one of the tests. | 20:46 |
| *** yidclare has joined #openstack-trove | 20:46 | |
| SlickNik | https://review.openstack.org/99000 | 20:47 |
| *** saurabhs has joined #openstack-trove | 20:48 | |
| SlickNik | oops, I meant to post the gate log for that review: https://rdjenkins.dyndns.org/job/Trove-Gate/4049/console | 20:48 |
| SlickNik | ^^ esp: We bumped the timeout for this; do we need to bump it more? | 20:49 |
| * esp looking | 20:49 | |
| esp | SlickNik: maybe dude. | 20:50 |
| denis_makogon | SlickNik, yes, it seems that timeout should be increased | 20:50 |
| denis_makogon | up to 2 time | 20:50 |
| esp | SlickNik: I was thinking maybe we should drop that @time_out all together | 20:50 |
| esp | because it’s kinda an artificial thing that seems like it’s just there to try to make the tests run faster | 20:50 |
| denis_makogon | but it also seems that it happed because all 6 workers were bussy and there were not enough resources to be on time | 20:51 |
| esp | denis_makogon: yeah, could be | 20:51 |
| peterstac | esp, SlickNik: Or make it something a fair bit larger - re-running the tests takes time too ;) | 20:51 |
| esp | so far we’ve gone from 10-20s and 20-30s | 20:51 |
| denis_makogon | esp, we can fell into infinity loop | 20:51 |
| denis_makogon | esp, i'd suggest to have like 5 mins timeout | 20:52 |
| denis_makogon | that's gonna be enough tho | 20:52 |
| *** jasonb365 has quit IRC | 20:52 | |
| *** demorris has quit IRC | 20:52 | |
| esp | peterstac: yeah, but seems like it should just time out when the API times out | 20:52 |
| peterstac | esp, denis_makogon: I was thinking 2 min, but 5 may be better | 20:52 |
| SlickNik | We've already gone from 20-30? I thought we were at 20? | 20:52 |
| * SlickNik needs to check the code | 20:52 | |
| esp | SlickNik: 20s didn’t cut it :) | 20:52 |
| denis_makogon | SlickNik, but seems we need more =) | 20:53 |
| esp | SlickNik: what’s the default request time out on the API? | 20:53 |
| * esp looknig too | 20:53 | |
| peterstac | isn't that dictated by the web server? I think nginx has a 60 sec default | 20:53 |
| SlickNik | denis_makogon: contention between jobs shouldn't be an issue since each job is running on it's own VM and not on a shared Jenkins host. | 20:54 |
| denis_makogon | esp, do we have any timeouts on external API ? | 20:54 |
| *** amcrn has quit IRC | 20:54 | |
| *** iccha has quit IRC | 20:54 | |
| *** coolsvap|afk has quit IRC | 20:54 | |
| *** openstackgerrit has quit IRC | 20:54 | |
| esp | denis_makogon: yeah we should dude | 20:54 |
| denis_makogon | SlickNik, i meant same hardware | 20:54 |
| denis_makogon | same AZ and Region | 20:54 |
| esp | peterstac: we’ll it’d be what ever trove-integration is running, I think it’s just a python process if I remember correctly | 20:55 |
| denis_makogon | esp, why? | 20:55 |
| *** julienvey has joined #openstack-trove | 20:55 | |
| esp | denis_makogon: because you wouldn’t want your request to hang forever | 20:55 |
| denis_makogon | esp, i do agree on RPC API, but HTTP - i've never seen such thing across all OS projects | 20:55 |
| esp | denis_makogon: they might leave it up to the deployer as peterstac eluded to. | 20:56 |
| esp | denis_makogon: so if you deploy your stuff with nginx you set your own time out | 20:56 |
| denis_makogon | esp, from your words, it's not a problem of ReST framework, it's up to deployer | 20:57 |
| denis_makogon | from the scratch each API service is independent from any web-server/servlet container | 20:58 |
| esp | denis_makogon: yep that’s correct as far as I can tell | 20:58 |
| denis_makogon | esp, does AWS has timeouts? | 20:59 |
| esp | denis_makogon: so I’m saying what ever we have iin code should be the default | 20:59 |
| esp | but what we have in test_code is artificially setting a different timeout | 20:59 |
| denis_makogon | esp, in this case HTTP polling inside API service is not a good idea | 21:00 |
| denis_makogon | even more, in this case all timeouts across services should be chained/dependent | 21:00 |
| *** jcru has quit IRC | 21:02 | |
| *** pdmars_ has quit IRC | 21:03 | |
| denis_makogon | esp, i guess, for given issue, we're gonna say "Eh, what's the hell? Let's increase timeouts" | 21:03 |
| denis_makogon | and it'll fix all our issues | 21:03 |
| esp | denis_makogon: hehe, yeah. I’m just wondering if we will have too keep doing it again... | 21:03 |
| *** michael-yu has quit IRC | 21:04 | |
| esp | so, remove the @time_out would basically allow for the true max timeout | 21:04 |
| esp | I *think* | 21:04 |
| denis_makogon | esp, if we would use instance creation timeout - we would forget about those timing issues at all | 21:04 |
| esp | not sure I understand denis_makogon | 21:05 |
| denis_makogon | esp, we have 32*60 sec for instance creation | 21:05 |
| denis_makogon | lets use it | 21:05 |
| denis_makogon | it is ok | 21:05 |
| esp | ah I see. you mean use the same value | 21:06 |
| denis_makogon | yes | 21:06 |
| esp | yeah maybe it would be ok | 21:06 |
| denis_makogon | esp, i a bit disagee, we cannot predict how fast task will accomplish, even on the same HW and Operating system | 21:06 |
| esp | let’s see what SlickNik finds first, maybe there is something more | 21:06 |
| esp | denis_makogon: yes. and rdjenkins is hard to predict. | 21:07 |
| denis_makogon | esp, that's why we're going to infra =) | 21:07 |
| *** sgotliv_ has quit IRC | 21:07 | |
| esp | denis_makogon: :) | 21:08 |
| *** amcrn has joined #openstack-trove | 21:08 | |
| *** iccha has joined #openstack-trove | 21:08 | |
| *** coolsvap|afk has joined #openstack-trove | 21:08 | |
| *** openstackgerrit has joined #openstack-trove | 21:08 | |
| denis_makogon | esp, infra has huge army to help with possible gate issues | 21:09 |
| denis_makogon | so, i'd like to keep @time_out 's | 21:10 |
| *** michael-yu has joined #openstack-trove | 21:10 | |
| *** jasonb365 has joined #openstack-trove | 21:10 | |
| esp | k, lets see how it goes | 21:10 |
| denis_makogon | esp, ok, thanks for chat =) | 21:10 |
| esp | np, good talking to you all | 21:11 |
| *** michael-yu has quit IRC | 21:14 | |
| *** yogeshmehra has quit IRC | 21:14 | |
| *** jasonb365 has quit IRC | 21:15 | |
| *** edmund1 has joined #openstack-trove | 21:19 | |
| *** michael-yu has joined #openstack-trove | 21:19 | |
| *** edmund has quit IRC | 21:20 | |
| *** cp16net__ has quit IRC | 21:21 | |
| *** annashe__ has joined #openstack-trove | 21:22 | |
| *** ViswaV_ has joined #openstack-trove | 21:31 | |
| *** pdmars has joined #openstack-trove | 21:31 | |
| *** amcrn has quit IRC | 21:31 | |
| *** flaper87 is now known as flaper87|afk | 21:32 | |
| *** kevinconway has quit IRC | 21:32 | |
| *** ViswaV has quit IRC | 21:34 | |
| *** ViswaV_ has quit IRC | 21:36 | |
| *** pdmars has quit IRC | 21:36 | |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for cached guest-images https://review.openstack.org/95304 | 21:38 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images https://review.openstack.org/91487 | 21:38 |
| *** sbfox has joined #openstack-trove | 21:45 | |
| *** robertmyers has quit IRC | 21:49 | |
| *** ramashri has joined #openstack-trove | 21:50 | |
| *** ViswaV has joined #openstack-trove | 21:52 | |
| *** michael-yu has quit IRC | 21:52 | |
| *** michael-yu has joined #openstack-trove | 21:54 | |
| *** ramashri_ has joined #openstack-trove | 21:55 | |
| *** ramashri has quit IRC | 21:55 | |
| *** ViswaV has quit IRC | 21:56 | |
| *** michael-yu has quit IRC | 21:56 | |
| *** michael-yu has joined #openstack-trove | 22:01 | |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for cached guest-images https://review.openstack.org/95304 | 22:01 |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images https://review.openstack.org/91487 | 22:02 |
| *** grapex has quit IRC | 22:04 | |
| *** amytron has quit IRC | 22:04 | |
| *** ViswaV has joined #openstack-trove | 22:10 | |
| *** amytron has joined #openstack-trove | 22:12 | |
| *** grapex has joined #openstack-trove | 22:12 | |
| *** grapex has quit IRC | 22:13 | |
| *** grapex has joined #openstack-trove | 22:13 | |
| *** ViswaV has quit IRC | 22:14 | |
| *** ViswaV has joined #openstack-trove | 22:15 | |
| *** denis_makogon has quit IRC | 22:19 | |
| *** grapex has quit IRC | 22:21 | |
| *** amytron has quit IRC | 22:22 | |
| *** Gordon has joined #openstack-trove | 22:25 | |
| *** Gordon is now known as Guest8131 | 22:25 | |
| *** Guest72113 has quit IRC | 22:28 | |
| *** Guest8131 has quit IRC | 22:30 | |
| *** achampion has quit IRC | 22:36 | |
| *** amytron has joined #openstack-trove | 22:38 | |
| *** rueben_ has joined #openstack-trove | 22:42 | |
| *** rueben_ has quit IRC | 22:42 | |
| *** thedodd has quit IRC | 22:43 | |
| *** rueben has quit IRC | 22:46 | |
| *** amytron has quit IRC | 22:51 | |
| *** amytron has joined #openstack-trove | 22:52 | |
| *** amytron has quit IRC | 22:54 | |
| *** annashe__ has quit IRC | 22:57 | |
| *** annashe__ has joined #openstack-trove | 23:00 | |
| openstackgerrit | Nikhil Manchanda proposed a change to openstack/python-troveclient: Remove RAX-specific auth in troveclient https://review.openstack.org/69081 | 23:04 |
| *** radez is now known as radez_g0n3 | 23:04 | |
| *** michael-yu has quit IRC | 23:05 | |
| *** yidclare has quit IRC | 23:05 | |
| *** julienvey has quit IRC | 23:07 | |
| *** michael-yu has joined #openstack-trove | 23:11 | |
| *** yidclare has joined #openstack-trove | 23:11 | |
| *** edmund1 has quit IRC | 23:12 | |
| *** tvoran has quit IRC | 23:21 | |
| *** sbfox has quit IRC | 23:25 | |
| *** ViswaV_ has joined #openstack-trove | 23:38 | |
| *** jmontemayor has quit IRC | 23:39 | |
| *** ViswaV has quit IRC | 23:42 | |
| *** saurabhs has left #openstack-trove | 23:42 | |
| *** ViswaV_ has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!