| *** ramishra has joined #openstack-trove | 00:04 | |
| *** grapex_ has joined #openstack-trove | 00:06 | |
| *** ramishra has quit IRC | 00:09 | |
| *** bwilhelmi has joined #openstack-trove | 00:13 | |
| bwilhelmi | First day on trove. Installed trove-integration. After $ ./redstack build-image mysql, what am I supposed to do? Where is the image that was built? What is it called? What are the legacy datastore and datastore versions for? | 00:16 |
|---|---|---|
| SlickNik | bwilhelmi: Did you go through ./redstack install already? | 00:17 |
| *** nehav has quit IRC | 00:18 | |
| bwilhelmi | Yes. I completed ./redstack install | 00:18 |
| SlickNik | bwilhelmi: Okay, after that you should run ./redstack kick-start mysql | 00:19 |
| SlickNik | THat will not only build the image (internally it calls build-image mysql) but will also set confs up, and upload the image to the glance catalog. | 00:19 |
| SlickNik | bwilhelmi: When you say legacy, do you mean inactive datastore? | 00:21 |
| bwilhelmi | I completed that. I see two images in the glance catalog--cirros and fedora | 00:21 |
| SlickNik | you should see a ubuntu_mysql image in the glance catalog if that completed successfully. | 00:21 |
| bwilhelmi | trove datastore-list returns a listing for "Legacy MySQL" and trove datastore-version-list returns "Unknown Legacy Version" | 00:22 |
| bwilhelmi | I do not see ubuntu_mysql | 00:22 |
| *** annashen_ has quit IRC | 00:22 | |
| SlickNik | btw, there is a bug with upstream diskimage-builder which is causing image builds to fail. I wonder if you're running into this issue: https://review.openstack.org/#/c/102623/ | 00:23 |
| SlickNik | If your image built successfully, it should be under ~/images/ubuntu_mysq/ubuntu_mysql.qcow2 | 00:24 |
| *** matsuhashi has joined #openstack-trove | 00:25 | |
| bwilhelmi | ~/image/ubuntu_mysql/ directory exists, but empty. Checking the kick-start output, I do see that DISTRO: unbound variable message, so it looks like I hit that defect | 00:27 |
| *** demorris has joined #openstack-trove | 00:30 | |
| openstackgerrit | Doug Shelley proposed a change to openstack/trove: Add datastore/version name into configurations response https://review.openstack.org/98985 | 00:35 |
| SlickNik | bwilhelmi: Looks like the fix for that issue just merged ~5 minutes ago. | 00:36 |
| SlickNik | bwilhelmi: if you cd to /opt/stack/tripleo-image-elements, fetch the latest changes from the git repo, and then try running "./redstack kick-start mysql" again, you should have better luck. | 00:38 |
| *** matsuhashi has quit IRC | 00:40 | |
| bwilhelmi | Just made the change manually and am running kick-start again | 00:40 |
| *** matsuhashi has joined #openstack-trove | 00:40 | |
| *** shayneburgess has joined #openstack-trove | 00:49 | |
| *** matsuhashi has quit IRC | 00:51 | |
| *** matsuhashi has joined #openstack-trove | 00:51 | |
| *** matsuhashi has quit IRC | 00:55 | |
| *** grapex_ has quit IRC | 01:03 | |
| *** ramishra has joined #openstack-trove | 01:05 | |
| *** openstackgerrit has quit IRC | 01:06 | |
| *** openstackgerrit has joined #openstack-trove | 01:06 | |
| *** ramishra has quit IRC | 01:09 | |
| *** pdmars has quit IRC | 01:11 | |
| *** grapex has joined #openstack-trove | 01:14 | |
| *** matsuhashi has joined #openstack-trove | 01:21 | |
| *** demorris has quit IRC | 01:25 | |
| *** shayneburgess has quit IRC | 01:26 | |
| *** shayneburgess has joined #openstack-trove | 01:27 | |
| *** matsuhashi has quit IRC | 01:27 | |
| *** annashen_ has joined #openstack-trove | 01:32 | |
| *** nosnos has joined #openstack-trove | 01:52 | |
| *** michael-yu has quit IRC | 01:52 | |
| *** shayneburgess has quit IRC | 01:53 | |
| *** shayneburgess has joined #openstack-trove | 02:03 | |
| *** grapex has quit IRC | 02:05 | |
| *** ramishra has joined #openstack-trove | 02:06 | |
| *** ramishra has quit IRC | 02:10 | |
| *** nehav has joined #openstack-trove | 02:27 | |
| *** grapex has joined #openstack-trove | 02:35 | |
| *** eghobo has quit IRC | 02:38 | |
| *** ViswaV has joined #openstack-trove | 02:48 | |
| *** ramishra has joined #openstack-trove | 02:49 | |
| *** ViswaV_ has joined #openstack-trove | 02:49 | |
| *** nehav has quit IRC | 02:49 | |
| *** nehav has joined #openstack-trove | 02:50 | |
| *** ViswaV has quit IRC | 02:53 | |
| *** demorris has joined #openstack-trove | 03:01 | |
| *** achampion has joined #openstack-trove | 03:08 | |
| *** shayneburgess has quit IRC | 03:12 | |
| *** shayneburgess has joined #openstack-trove | 03:17 | |
| *** demorris has quit IRC | 03:22 | |
| *** ramishra has quit IRC | 03:25 | |
| *** shayneburgess has quit IRC | 03:27 | |
| *** dkehn__ has joined #openstack-trove | 03:28 | |
| *** ViswaV_ has quit IRC | 03:28 | |
| *** dkehn_ has quit IRC | 03:31 | |
| *** shayneburgess has joined #openstack-trove | 03:33 | |
| *** shayneburgess has quit IRC | 03:33 | |
| *** ViswaV has joined #openstack-trove | 03:33 | |
| *** ramishra has joined #openstack-trove | 03:40 | |
| *** grapex_ has joined #openstack-trove | 03:42 | |
| *** grapex has quit IRC | 03:45 | |
| *** nosnos has quit IRC | 03:49 | |
| *** nosnos has joined #openstack-trove | 03:49 | |
| *** nosnos has quit IRC | 03:50 | |
| *** robertmyers has quit IRC | 03:50 | |
| *** nosnos has joined #openstack-trove | 03:50 | |
| *** nosnos has quit IRC | 03:55 | |
| *** eghobo has joined #openstack-trove | 03:55 | |
| *** nosnos has joined #openstack-trove | 04:13 | |
| *** eghobo has quit IRC | 04:16 | |
| *** mattgriffin has quit IRC | 04:30 | |
| *** sgotliv has joined #openstack-trove | 04:38 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add datastore/version name into configurations response https://review.openstack.org/98985 | 04:40 |
| *** eghobo has joined #openstack-trove | 04:46 | |
| *** ViswaV_ has joined #openstack-trove | 04:50 | |
| *** ViswaV has quit IRC | 04:50 | |
| *** ViswaV has joined #openstack-trove | 04:58 | |
| *** ViswaV_ has quit IRC | 04:58 | |
| *** michael-yu has joined #openstack-trove | 05:04 | |
| openstackgerrit | Steve Leon proposed a change to openstack/trove: Deleting incremental backup metadata after restore https://review.openstack.org/102585 | 05:06 |
| *** michael-yu has quit IRC | 05:07 | |
| *** sgotliv has quit IRC | 05:10 | |
| *** ramishra_ has joined #openstack-trove | 05:10 | |
| *** ramishra has quit IRC | 05:11 | |
| *** ajc_ has joined #openstack-trove | 05:15 | |
| *** nehav has quit IRC | 05:16 | |
| *** grapex_ has quit IRC | 05:19 | |
| *** coolsvap|afk is now known as coolsvap | 05:22 | |
| *** ramishra has joined #openstack-trove | 05:29 | |
| *** ramishra_ has quit IRC | 05:29 | |
| *** eghobo has quit IRC | 05:38 | |
| *** eghobo has joined #openstack-trove | 05:38 | |
| *** ViswaV has quit IRC | 05:58 | |
| *** eguz has joined #openstack-trove | 06:06 | |
| *** dkehn_ has joined #openstack-trove | 06:06 | |
| *** ViswaV has joined #openstack-trove | 06:08 | |
| *** dkehn_ has quit IRC | 06:08 | |
| *** dkehn_ has joined #openstack-trove | 06:09 | |
| *** dkehn__ has quit IRC | 06:09 | |
| *** eghobo has quit IRC | 06:10 | |
| *** grapex has joined #openstack-trove | 06:19 | |
| *** denis_makogon has joined #openstack-trove | 06:23 | |
| denis_makogon | SlickNik, hey, i update this patch https://review.openstack.org/#/c/96795/, could you please re-review it ? | 06:23 |
| *** grapex has quit IRC | 06:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements https://review.openstack.org/99095 | 06:37 |
| openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 06:37 |
| *** nosnos has quit IRC | 06:44 | |
| *** nosnos has joined #openstack-trove | 06:44 | |
| *** ViswaV has quit IRC | 06:46 | |
| *** rwsu has quit IRC | 06:53 | |
| *** flaper87 has quit IRC | 06:59 | |
| *** flaper87 has joined #openstack-trove | 06:59 | |
| *** ramishra has quit IRC | 07:01 | |
| *** ramishra has joined #openstack-trove | 07:03 | |
| *** annashen_ has quit IRC | 07:14 | |
| *** eguz has quit IRC | 07:15 | |
| *** nehav has joined #openstack-trove | 07:18 | |
| openstackgerrit | A change was merged to openstack/trove: Fix data too long for column 'task_description' https://review.openstack.org/102601 | 07:20 |
| *** Longgeek has joined #openstack-trove | 07:23 | |
| *** haomaiwa_ has joined #openstack-trove | 07:23 | |
| openstackgerrit | A change was merged to openstack/trove: Fix enable on boot when working with systemd and symlinked units https://review.openstack.org/100504 | 07:23 |
| openstackgerrit | A change was merged to openstack/trove: Fix typos in trove/instance/models.py https://review.openstack.org/102338 | 07:24 |
| *** nehav has quit IRC | 07:26 | |
| *** julienvey has joined #openstack-trove | 07:28 | |
| *** ramishra has quit IRC | 07:32 | |
| *** ramishra_ has joined #openstack-trove | 07:34 | |
| *** Longgeek has quit IRC | 07:36 | |
| *** Longgeek has joined #openstack-trove | 07:37 | |
| *** ashishsnigam has joined #openstack-trove | 07:53 | |
| *** boden has joined #openstack-trove | 07:59 | |
| ashishsnigam | Hi.. Can any one tell me how can i get default datastore in upstream tempest setup? | 08:04 |
| ashishsnigam | The effect which we get after "redstack kickstart mysql" workflow. | 08:04 |
| *** haomaiwa_ has quit IRC | 08:07 | |
| *** nosnos has quit IRC | 08:07 | |
| *** rwsu has joined #openstack-trove | 08:08 | |
| *** rwsu has quit IRC | 08:08 | |
| *** haomaiwang has joined #openstack-trove | 08:10 | |
| openstackgerrit | boden proposed a change to openstack/trove: Expose trove conductor manager class as conf property https://review.openstack.org/102526 | 08:16 |
| SlickNik | ashishsnigam: It will be setup for you after https://review.openstack.org/#/c/102455/ merges. | 08:18 |
| *** juantwo_ has quit IRC | 08:20 | |
| *** juantwo has joined #openstack-trove | 08:21 | |
| ashishsnigam | Thanks SlickNik | 08:23 |
| *** juantwo has quit IRC | 08:26 | |
| *** denis_makogon has quit IRC | 08:55 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Updated from global requirements https://review.openstack.org/99095 | 09:00 |
| openstackgerrit | A change was merged to openstack/trove: Updated from global requirements https://review.openstack.org/96275 | 09:00 |
| openstackgerrit | A change was merged to openstack/trove-integration: Change olso.config to be installed from pypi instead of tarballs.o.o https://review.openstack.org/102974 | 09:05 |
| *** SnowDust has joined #openstack-trove | 09:24 | |
| *** nosnos has joined #openstack-trove | 09:42 | |
| boden | SlickNik hi... in the code review what is the +1 for 'workflow' ? | 09:43 |
| SlickNik | boden: the Workflow +1 is what triggers zuul to run the gate jobs and attempt to merge the patch. | 09:44 |
| *** nosnos has quit IRC | 09:44 | |
| boden | SlickNik -- is there something else I need to get workflow +1 here: https://review.openstack.org/#/c/102526/ | 09:45 |
| *** nosnos has joined #openstack-trove | 09:45 | |
| SlickNik | The core reviewer who gives the second +2 on a patch usually also sets Workflow +1. | 09:45 |
| boden | SlickNik -- ah, gotcha | 09:45 |
| boden | thanks | 09:45 |
| SlickNik | Yup, patches in trove, like ones in other OpenStack projects, requires two +2s before a merge. | 09:47 |
| SlickNik | But you probably already knew that. | 09:47 |
| SlickNik | Not a problem. | 09:48 |
| *** nosnos has quit IRC | 09:49 | |
| flaper87 | What's Dan Nguyen nick? | 09:57 |
| flaper87 | https://blueprints.launchpad.net/trove/+spec/oslo-messaging <- he is assigned to this blueprint | 09:57 |
| flaper87 | oh nevermind | 09:58 |
| *** denis_makogon has joined #openstack-trove | 09:58 | |
| flaper87 | there's an IRC nick field in his LP profile | 09:58 |
| flaper87 | esp: hey, just wanted to ask if you could share the status of Trove's switch-over to oslo.messaging | 09:58 |
| flaper87 | (whenever you're on-line and have time) | 09:59 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Remove redundant heat templates from codebase https://review.openstack.org/99000 | 10:00 |
| SlickNik | flaper87: He had a review out for it that he's working through, IIRC | 10:00 |
| SlickNik | let me try to find it, one sec | 10:00 |
| flaper87 | SlickNik: awesome, he probably didn't mention the blueprint in the commit message since there's no link in launchpad | 10:01 |
| flaper87 | SlickNik: thank you | 10:01 |
| SlickNik | flaper87: https://review.openstack.org/#/c/94484/ | 10:01 |
| flaper87 | thanks, SlickNik | 10:02 |
| SlickNik | flaper87: Np. He has some info regarding it on the wiki page here - https://wiki.openstack.org/wiki/Trove-rpc-versioning as well. | 10:04 |
| * flaper87 clicks and reads | 10:06 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Sync gettextutils from oslo with new logging standards https://review.openstack.org/97472 | 10:09 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Fix updated timestamp for SecurityGroup model https://review.openstack.org/97510 | 10:10 |
| *** coolsvap is now known as coolsvap|afk | 10:28 | |
| *** ashishsnigam has quit IRC | 10:55 | |
| *** SnowDust has quit IRC | 11:11 | |
| *** haomaiwang has quit IRC | 11:12 | |
| *** haomaiwang has joined #openstack-trove | 11:12 | |
| *** haomaiw__ has joined #openstack-trove | 11:13 | |
| *** haomaiwang has quit IRC | 11:17 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.backup.* actions https://review.openstack.org/102206 | 11:26 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 11:38 |
| *** ramishra_ has quit IRC | 11:43 | |
| *** ramishra has joined #openstack-trove | 11:44 | |
| *** ramishra has quit IRC | 11:47 | |
| *** ramishra has joined #openstack-trove | 11:47 | |
| *** ajc_ has quit IRC | 12:03 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Adding command for copying backups https://review.openstack.org/89972 | 12:06 |
| *** haomaiw__ has quit IRC | 12:09 | |
| *** haomaiwa_ has joined #openstack-trove | 12:10 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Retrieve heat-integration permission from OS environment https://review.openstack.org/103111 | 12:24 |
| *** nehav has joined #openstack-trove | 12:33 | |
| *** nehav has quit IRC | 12:34 | |
| *** nehav has joined #openstack-trove | 12:34 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 12:35 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.backup.* actions https://review.openstack.org/102206 | 12:38 |
| *** ashestakov has quit IRC | 12:42 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Retrieve heat-integration permission from OS environment https://review.openstack.org/103111 | 12:49 |
| *** tvoran has joined #openstack-trove | 12:50 | |
| *** nehav has quit IRC | 12:55 | |
| *** achampion has quit IRC | 13:07 | |
| *** sgotliv has joined #openstack-trove | 13:13 | |
| *** radez_g0n3 is now known as radez | 13:16 | |
| *** Longgeek has quit IRC | 13:19 | |
| *** haomaiwa_ has quit IRC | 13:21 | |
| *** Longgeek has joined #openstack-trove | 13:21 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Retrieve heat-integration permission from OS environment https://review.openstack.org/103111 | 13:26 |
| *** robertmyers has joined #openstack-trove | 13:38 | |
| *** jcru has joined #openstack-trove | 13:43 | |
| openstackgerrit | A change was merged to openstack/python-troveclient: Add datatore/version name into configuration view https://review.openstack.org/98988 | 13:46 |
| openstackgerrit | A change was merged to openstack/python-troveclient: Fix shell view for secgroup-* commands https://review.openstack.org/85314 | 13:48 |
| *** jcru has quit IRC | 13:48 | |
| *** jcru has joined #openstack-trove | 13:49 | |
| *** mattgriffin has joined #openstack-trove | 13:53 | |
| *** achampion has joined #openstack-trove | 13:55 | |
| *** demorris has joined #openstack-trove | 13:58 | |
| *** jmontemayor has joined #openstack-trove | 14:01 | |
| *** demorris has quit IRC | 14:01 | |
| *** annashen_ has joined #openstack-trove | 14:03 | |
| *** jmontemayor_ has joined #openstack-trove | 14:03 | |
| *** jmontemayor has quit IRC | 14:07 | |
| *** pdmars has joined #openstack-trove | 14:07 | |
| *** annashen_ has quit IRC | 14:09 | |
| *** Robe has left #openstack-trove | 14:18 | |
| *** rhodgin has joined #openstack-trove | 14:18 | |
| *** amytron has joined #openstack-trove | 14:25 | |
| *** rwsu has joined #openstack-trove | 14:27 | |
| *** grapex has joined #openstack-trove | 14:27 | |
| *** grapex_ has joined #openstack-trove | 14:28 | |
| *** kevinconway has joined #openstack-trove | 14:29 | |
| *** pdmars has quit IRC | 14:30 | |
| *** pdmars has joined #openstack-trove | 14:31 | |
| *** grapex has quit IRC | 14:32 | |
| bwilhelmi | Hi All. Just got trove-integration installed and running successfully for the first time this week. However, I cannot successfully create an instance. Nova list shows ACTIVE, but trove list shows BUILD. Cannot delete the trove instance--error says instance is not ready. How do I troubleshoot this? | 14:44 |
| *** ramishra has quit IRC | 14:45 | |
| *** demorris has joined #openstack-trove | 14:45 | |
| glucas | bwilhelmi: Can you SSH to the nova instance? If something goes wrong while the guest agent is starting on the guest instance you can get situations where your trove instance gets stuck in BUILD. | 14:46 |
| glucas | bwilhelmi: if you SSH in you can check /tmp/logfile.txt (the guest agent log). If the log isn't there the issue was earlier, you can check in /var/log/upstart as well. | 14:47 |
| bwilhelmi | I will check. Thanks | 14:48 |
| glucas | bwilhelmi: FYI, there is a timeout such that an instance stuck in BUILD will be moved to ERROR (and can then be deleted). That timeout is disabled by default, see update_status_on_fail in the trove taskmanager conf. | 14:50 |
| glucas | most likely there is a conf issue here preventing the guest from starting or from communicating out. | 14:51 |
| *** shayneburgess has joined #openstack-trove | 14:55 | |
| bwilhelmi | Thanks--I'll see if I can log in. | 14:56 |
| *** bwilhelmi_ has joined #openstack-trove | 15:06 | |
| bwilhelmi_ | ERROR trove.common.utils [req-288b346f-dd05-4bda-85e8-94006a521168 0a755b6240e34f7385a45254ca3d2817 e40ed6915e7543ddb082f19 262ec6913] Time out after waiting 30 seconds when running proc: ("sudo sed -i '/^manual$/d' /etc/init/mysql.conf",) {'shell': True} | 15:09 |
| bwilhelmi_ | ERROR trove.openstack.common.rpc.amqp [req-288b346f-dd05-4bda-85e8-94006a521168 0a755b6240e34f7385a45254ca3d2817 e40ed6915e7543ddb082f19262ec6913] Exception during message handling | 15:10 |
| *** ramishra has joined #openstack-trove | 15:16 | |
| *** nehav1 has joined #openstack-trove | 15:18 | |
| *** thedodd has joined #openstack-trove | 15:19 | |
| *** yidclare has joined #openstack-trove | 15:20 | |
| *** ramishra has quit IRC | 15:20 | |
| *** openstackgerrit has quit IRC | 15:21 | |
| *** nehav1 has quit IRC | 15:27 | |
| *** nehav1 has joined #openstack-trove | 15:27 | |
| glucas | bwilhelmi: OK, not immediately obvious to me what the issue is.. :-) That timeout seems odd. | 15:32 |
| *** shayneburgess has quit IRC | 15:35 | |
| bwilhelmi | This is in a laptop & virtualbox. Upon deploying trove instance, everything got rather sluggish | 15:35 |
| bwilhelmi | Is there a way to restart the initialization within the instance? | 15:36 |
| *** openstackgerrit has joined #openstack-trove | 15:44 | |
| denis_makogon | SlickNik, hey, need your eyes on this patch https://review.openstack.org/#/c/103172/ | 15:46 |
| denis_makogon | bwilhelmi, glucas, there's timeout, but setting an ERROR state should be enabled in taskmanager.conf | 15:47 |
| *** michael-yu has joined #openstack-trove | 15:47 | |
| *** michael-yu has quit IRC | 15:47 | |
| denis_makogon | bwilhelmi, glucas https://github.com/openstack/trove/blob/master/trove/common/cfg.py#L50-L55 | 15:47 |
| *** ViswaV has joined #openstack-trove | 15:48 | |
| glucas | denis_makogon: Yes, I mentioned the update_status_on_fail earlier. | 15:50 |
| denis_makogon | glucas, sorry, there're alot of text =) | 15:51 |
| glucas | Actually.. that default seems backward to me. | 15:51 |
| glucas | denis_makogon: :-) yeah, can be hard to keep track | 15:51 |
| *** dkehn_ is now known as dkehnx | 15:51 | |
| denis_makogon | glucas, being an admin gives you an ability to reset task for given instance | 15:52 |
| *** doddstack has joined #openstack-trove | 15:52 | |
| glucas | denis_makogon: Did you see the error bwilhelmi pasted above? The guest seems to have timed out waiting for a sed command, which seems odd to me. | 15:53 |
| denis_makogon | glucas, default 30sec for exec | 15:53 |
| denis_makogon | glucas, i bet he's testing environment is too slow | 15:54 |
| denis_makogon | glucas, devstack/redstack needs at least 4GB RAM | 15:54 |
| *** thedodd has quit IRC | 15:55 | |
| bwilhelmi | I'll try on a bigger machine | 15:56 |
| *** esp has left #openstack-trove | 15:57 | |
| *** michael-yu has joined #openstack-trove | 15:57 | |
| *** ViswaV has quit IRC | 16:00 | |
| *** haomaiwang has joined #openstack-trove | 16:01 | |
| *** eghobo has joined #openstack-trove | 16:04 | |
| *** eghobo has quit IRC | 16:04 | |
| *** eghobo has joined #openstack-trove | 16:05 | |
| *** michael-yu has quit IRC | 16:16 | |
| denis_makogon | amrith, ping | 16:19 |
| amrith | denis_makogon, ping | 16:19 |
| denis_makogon | amrith, https://review.openstack.org/#/c/103172 there's no bug and there's no valid place to file it | 16:20 |
| denis_makogon | amrith, if you take a look at whole infra/config dashboard you'll see that there's no need to file somewhere any bug or something else | 16:21 |
| denis_makogon | amrith, and if you want to be on the same page with me, take a look at https://github.com/openstack/trove-integration/blob/master/tests/integration/int_tests.py#L222-L248 | 16:22 |
| denis_makogon | amrith, there you'll find that every registered datastore requires special test suit to be launched | 16:22 |
| denis_makogon | amrith, --group=cassandra | 16:22 |
| denis_makogon | etc | 16:22 |
| amrith | denis_makogon, my comment was this "Is there a bug? Not saying there has to be, but if there is one, please reference it." | 16:23 |
| amrith | so let me repeat, "Not saying there has to be, but if there is one, please reference it." | 16:23 |
| amrith | OK? | 16:23 |
| denis_makogon | amrith, commit message clearly states what kind of issue is being fixed. | 16:24 |
| amrith | denis_makogon, the comment was "Not saying there has to be, but if there is one, please reference it." | 16:24 |
| denis_makogon | amrith, i get that, no need to repeat multiply =) | 16:25 |
| amrith | so, I don't understand your point. there's no bug, over. finished. | 16:25 |
| *** michael-yu has joined #openstack-trove | 16:25 | |
| denis_makogon | amrith, there's a bug | 16:25 |
| denis_makogon | amrith, and it's related to int-tests launch process | 16:26 |
| amrith | what's the number? | 16:26 |
| denis_makogon | amrith, there's no filed bug, because there's no valid place to file it | 16:26 |
| amrith | ok, let me clarify my comments ... "Is there a filed bug? Not saying there has to be, but if there is one, please reference it." | 16:27 |
| *** dkehn_ has joined #openstack-trove | 16:28 | |
| denis_makogon | amrith, no bug report was filed, nothing to reference to | 16:28 |
| amrith | denis_makogon, very good. Now put DATASTORE_TYPE within {} or put a $ before it and your code looks OK. | 16:28 |
| *** achampion has quit IRC | 16:29 | |
| *** michael-yu_ has joined #openstack-trove | 16:29 | |
| *** dkehn_ has quit IRC | 16:30 | |
| *** dkehn_ has joined #openstack-trove | 16:30 | |
| *** dkehnx has quit IRC | 16:30 | |
| *** dkehn_ is now known as dkehnx | 16:30 | |
| *** michael-yu has quit IRC | 16:31 | |
| *** michael-yu_ is now known as michael-yu | 16:31 | |
| openstackgerrit | Joe Cruz proposed a change to openstack/trove: Make configuration tests configurable https://review.openstack.org/102972 | 16:33 |
| *** esp has joined #openstack-trove | 16:36 | |
| *** esp has left #openstack-trove | 16:36 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Mark all running backups as FAILED when instance was deleted https://review.openstack.org/102838 | 16:37 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Update backup state when guest error occures https://review.openstack.org/102262 | 16:37 |
| denis_makogon | jcru, ping | 16:37 |
| jcru | denis_makogon: yo | 16:38 |
| amrith | thanks denis_makogon I see patch set 2. I'll look at that. | 16:38 |
| denis_makogon | jcru, your last patch doesn't contain required changed to test config | 16:38 |
| denis_makogon | jcru, https://github.com/openstack/trove/blob/master/trove/tests/config.py#L53-L158 | 16:39 |
| denis_makogon | jcru, this what i asked for at previous patchsets | 16:39 |
| denis_makogon | jcru, you cannot just ask test config for missing attribute that's why int-tests were failing, and fake tests were passing (configuration API tests are not enabled there) | 16:40 |
| *** michael-yu has quit IRC | 16:40 | |
| jcru | denis_makogon: k, yeah I can add those configs there | 16:42 |
| jcru | denis_makogon: I had included those configs in the cdb-integration test_begin file and I believe install/kick-start sets up the test config | 16:43 |
| jcru | which is used by the int tests | 16:43 |
| denis_makogon | jcru, not just "can", you "should" =) | 16:43 |
| denis_makogon | jcru, they would not be pulled from test_begin.conf since there's not such option | 16:43 |
| jcru | denis_makogon: k, didn't know, will do | 16:43 |
| denis_makogon | jcru, int-tests (fake and real) are dependent on TestConfig class (see link above), that describes which configuration options are can be used =) | 16:45 |
| jcru | denis_makogon: oh I see, its just making sure defaults exist | 16:45 |
| denis_makogon | jcru, there's no defaults in TestConfig class | 16:46 |
| *** esp has joined #openstack-trove | 16:47 | |
| *** jcru has quit IRC | 16:55 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Add notifications for trove.backup.* actions https://review.openstack.org/102206 | 16:55 |
| *** jcru has joined #openstack-trove | 16:56 | |
| *** michael-yu has joined #openstack-trove | 16:57 | |
| *** dkehn_ has joined #openstack-trove | 17:00 | |
| *** sgotliv has quit IRC | 17:00 | |
| *** michael-yu has quit IRC | 17:01 | |
| *** michael-yu has joined #openstack-trove | 17:03 | |
| *** dkehnx has quit IRC | 17:03 | |
| amrith | denis_makogon, you there? | 17:07 |
| *** demorris has quit IRC | 17:12 | |
| *** ramishra has joined #openstack-trove | 17:17 | |
| openstackgerrit | Mat Lowery proposed a change to openstack/python-troveclient: Change .gitreview https://review.openstack.org/103197 | 17:21 |
| *** ramishra has quit IRC | 17:21 | |
| denis_makogon | amrith, yes | 17:24 |
| *** bwilhelmi_ has quit IRC | 17:25 | |
| denis_makogon | amrith, i reverted to previous(almost) patch since each job/template is being parsed as XML layout, and i used simple $XXXXX format | 17:25 |
| denis_makogon | amrith, and there's no test group named as 'mysql' | 17:26 |
| denis_makogon | amrith, to be honest mysql == huge set of test groups | 17:27 |
| *** julienvey has quit IRC | 17:28 | |
| amrith | denis_makogon when you run without --group=xxxx it will run what? | 17:28 |
| denis_makogon | amrith, it'll run given group if it was registered | 17:28 |
| denis_makogon | amrith, "--group=blackbox" | 17:29 |
| amrith | if I just ran ./redstack int-tests | 17:30 |
| amrith | then what group would it run? | 17:30 |
| denis_makogon | amrith, blackbox | 17:30 |
| denis_makogon | amrith, https://github.com/openstack/trove-integration/blob/master/tests/integration/int_tests.py#L164-L189 | 17:30 |
| *** doddstack has quit IRC | 17:31 | |
| amrith | so it would to me that the easier change would be to just create a group called mysql and point it at blackbox | 17:31 |
| *** annashen_ has joined #openstack-trove | 17:31 | |
| *** yidclare has quit IRC | 17:33 | |
| *** ramishra has joined #openstack-trove | 17:33 | |
| denis_makogon | amrith, it'll take too much effort | 17:33 |
| amrith | denis_makogon, fair point | 17:34 |
| denis_makogon | amrith, since gate already broken for everything except mysql, and taking into account pace of merging - it'll take too long to fix it | 17:34 |
| denis_makogon | amrith, i will fix gate first and then i'll fix trove-integration and then simplify gate jobs | 17:35 |
| denis_makogon | amrith, any objections? | 17:35 |
| *** yidclare has joined #openstack-trove | 17:36 | |
| amrith | denis_makogon, you don't want to file a bug in openstack-ci? I believe that's the place for it, no? | 17:36 |
| denis_makogon | amrith, do you think that it's a appropriate place to file a bug ? | 17:37 |
| denis_makogon | amrith, what about hudson-openstack ? | 17:37 |
| *** yogeshmehra has joined #openstack-trove | 17:37 | |
| denis_makogon | amrith, no need to follow common guideline, at least in infra | 17:37 |
| denis_makogon | amrith, nobody does it | 17:37 |
| amrith | other changes to this file and other files in this directory had bugs in openstack-ci, so I suggested openstack-ci. Why not hudson-openstack, I don't know? maybe a good place, maybe not. | 17:38 |
| denis_makogon | amrith, lets just skip this topic, only for infra | 17:39 |
| amrith | denis_makogon, that's fine with me. just seems like an exception to the rule. | 17:40 |
| *** ViswaV has joined #openstack-trove | 17:40 | |
| denis_makogon | amrith, infra is interesting place among all openstack =) | 17:40 |
| *** ViswaV_ has joined #openstack-trove | 17:42 | |
| amrith | denis_makogon, to your comment on https://review.openstack.org/#/c/97472/3 | 17:42 |
| amrith | you write: Amrith, before -1 this patch, you should've take check oslo repository and this patch https://review.openstack.org/#/c/91068/4/trove/openstack/common/__init__.py,cm. Oslo auto-sync substitutes sync'ed modules. So, your -1 has no reasons, just small misunderstanding for oslo-sync process. | 17:42 |
| denis_makogon | amrith, yes | 17:42 |
| amrith | As you probably don't understand, I'm not -1'ing for no reason | 17:42 |
| amrith | The reason is here, in the comments | 17:42 |
| amrith | https://wiki.openstack.org/wiki/LegalIssuesFAQ#Copyright_Headers | 17:42 |
| denis_makogon | amrith, header was dropped from oslo-incubator | 17:43 |
| amrith | the change proposed violates the stated policy in wiki | 17:43 |
| denis_makogon | amrith, n | 17:43 |
| denis_makogon | amrith, no | 17:43 |
| amrith | It doesn't? | 17:43 |
| denis_makogon | amrith, please learn process of auto-sync, and if you don't trust me - verify it by yourself | 17:43 |
| amrith | so tell me what a reviewer is to do when something changed happens to be contrary to the policy on wiki | 17:44 |
| amrith | I don't know | 17:44 |
| amrith | so tell me | 17:44 |
| amrith | You're suggesting that I should just accept all changes | 17:44 |
| amrith | realize that the discussion on OSLO that you referenced | 17:45 |
| *** ViswaV has quit IRC | 17:45 | |
| amrith | also had some conversation about the change and they thought the copyright made no sense | 17:45 |
| amrith | that's fine, the comments talk about lawyers and stuff | 17:46 |
| amrith | but it was still (IMHO) not consistent with the wiki and the policy stated therein | 17:46 |
| amrith | so, I'm just a reviewer. | 17:46 |
| amrith | someone on core can come along and opine that it is fine | 17:46 |
| amrith | that's fine. | 17:46 |
| denis_makogon | amrith, we are all familiar with rules that that blocks from changing oslo-incubator code inside any project, so, i'd never did that | 17:47 |
| denis_makogon | *do | 17:47 |
| amrith | OK, good, I'll learn the rules. thanks! | 17:48 |
| *** achampion has joined #openstack-trove | 17:50 | |
| denis_makogon | amrith, thanks | 17:52 |
| openstackgerrit | Michael Yu proposed a change to openstack/trove: Adds Backup/Restore support for Couchbase https://review.openstack.org/86731 | 17:52 |
| amrith | denis_makogon, I'm sure that if I don't you'll teach me. | 17:53 |
| denis_makogon | amrith, i'd be glad to help | 17:53 |
| openstackgerrit | Denis M. proposed a change to openstack/trove-integration: Fix signature for launching int-tests https://review.openstack.org/103201 | 17:53 |
| * amrith grins | 17:54 | |
| denis_makogon | cp16net, hey, i updated patch according to your suggestions, could you please re-review it https://review.openstack.org/#/c/99000/? | 17:55 |
| cp16net | denis_makogon: sure | 17:56 |
| denis_makogon | SlickNik, hey, i fixed SG's registration according to SG's flags https://review.openstack.org/#/c/96795/ | 17:57 |
| denis_makogon | cp16net, thanks man =) | 17:57 |
| cp16net | remove commented lines | 17:58 |
| *** julienvey has joined #openstack-trove | 17:58 | |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Retrieve heat-integration permission from OS environment https://review.openstack.org/103111 | 17:59 |
| *** julienvey has quit IRC | 18:00 | |
| *** julienvey has joined #openstack-trove | 18:00 | |
| denis_makogon | peterstac, hey, could you please not change commit message at patches that you're not own. It overrides commiter, just ping me next time. Thanks. | 18:00 |
| peterstac | denis_makogon: no problem, just thought I was helping | 18:02 |
| *** yidclare has quit IRC | 18:02 | |
| *** thedodd has joined #openstack-trove | 18:02 | |
| *** ViswaV has joined #openstack-trove | 18:02 | |
| *** ViswaV_ has quit IRC | 18:02 | |
| denis_makogon | peterstac, you helped me, i appreciate that. | 18:02 |
| denis_makogon | cp16net, done =) | 18:03 |
| openstackgerrit | Denis M. proposed a change to openstack/trove: Remove redundant heat templates from codebase https://review.openstack.org/99000 | 18:03 |
| denis_makogon | guys, i'm going to home, will be reachable in 2-3h, have a good day | 18:05 |
| *** julienvey has quit IRC | 18:05 | |
| *** denis_makogon has quit IRC | 18:05 | |
| *** yidclare has joined #openstack-trove | 18:13 | |
| *** mriedem has joined #openstack-trove | 18:16 | |
| mriedem | is anyone here aware of any in progress oslo sync reviews? | 18:17 |
| amrith | mriedem you must be joking ;) | 18:17 |
| mriedem | i must be missing the joke | 18:17 |
| amrith | oh, ok. | 18:18 |
| mriedem | does trove have a weird oslo sync policy or something? | 18:18 |
| amrith | mriedem, denis and I have just been going back and forth on two of the oslo sync reviews ;) | 18:18 |
| amrith | I don't know, can't speak to that at all | 18:18 |
| mriedem | just joined the channel | 18:18 |
| mriedem | links? | 18:18 |
| mriedem | amrith: oslo sync guildelines are here https://wiki.openstack.org/wiki/ReviewChecklist#Oslo_Syncing_Checklist | 18:21 |
| *** demorris has joined #openstack-trove | 18:21 | |
| mriedem | i'm trying to get caught up on why __init__.py needed to be changed when gettextutils.py is the only thing synced in | 18:21 |
| amrith | you bring up an issue I didn't even know of, thanks for the pointer to the sync checklist | 18:22 |
| mriedem | so i'd -1 for not listing the commits being brought in | 18:23 |
| mriedem | for example https://review.openstack.org/#/c/103205/ | 18:23 |
| mriedem | general policy is to list the commits sync'ed in so the reviewers can compare | 18:23 |
| mriedem | rather than guess | 18:23 |
| amrith | very good point | 18:23 |
| amrith | and can you guess what the oslo commit would have been for this, I'm looking as well ... | 18:24 |
| amrith | mriedem, what say you about https://review.openstack.org/#/c/65581/ | 18:25 |
| mriedem | amrith: i would have -1'ed for the same reason, list the commits | 18:25 |
| *** jcru_ has joined #openstack-trove | 18:26 | |
| amrith | i'm looking for a commit that lists the oslo commit | 18:26 |
| mriedem | some projects don't care about that as much, it's really up to the core team on what they are willing to accept for reviewng these types of changes | 18:26 |
| mriedem | i.e. "jenkins passes, lgtm" | 18:26 |
| amrith | https://review.openstack.org/#/c/75258/ | 18:26 |
| mriedem | amrith: see my comment here https://review.openstack.org/#/c/97472/ | 18:26 |
| amrith | looking now ... | 18:27 |
| *** jcru has quit IRC | 18:27 | |
| *** Longgeek has quit IRC | 18:28 | |
| mriedem | so -1 for two reasons: 1. list the commits and 2. drop the __init__.py change since it looks like it's only needed for oslo unit tests to work | 18:31 |
| amrith | got it! thanks ... | 18:34 |
| mriedem | \o/ | 18:34 |
| amrith | thanks for the pointer to tips on syncing | 18:34 |
| mriedem | yw | 18:34 |
| amrith | mriedem, you should also take a look at https://review.openstack.org/#/c/91068 | 18:36 |
| amrith | I think you'll see the same two issues there as well | 18:36 |
| amrith | at least the one ... | 18:36 |
| amrith | sorry, it lists the oslo incubator change | 18:36 |
| openstackgerrit | Joe Cruz proposed a change to openstack/trove: Make configuration tests configurable https://review.openstack.org/102972 | 18:37 |
| *** jcru_ has quit IRC | 18:37 | |
| mriedem | fun, competing syncs | 18:38 |
| *** jcru has joined #openstack-trove | 18:38 | |
| mriedem | i've seen ihar's sync in glance and cinder already | 18:38 |
| amrith | which reference each other as a rationale for existance no less ... | 18:38 |
| mriedem | ihar's is for a py26 jsonutils performance fix | 18:38 |
| mriedem | denis' is for new logging standards | 18:38 |
| amrith | I mean the references to why they are removing __init__.py | 18:39 |
| amrith | they're both attempting to change trove/openstack/common/__init__.py | 18:39 |
| amrith | on master | 18:40 |
| amrith | in the same way ;) | 18:40 |
| amrith | fun fun | 18:40 |
| amrith | oops, my fun-meter for the day just hit 100% | 18:40 |
| mriedem | well so i'd -2 https://review.openstack.org/#/c/97472/ because https://review.openstack.org/#/c/91068/ came first and has the gettextutils sync | 18:40 |
| mriedem | core team around? | 18:41 |
| amrith | some for sure | 18:41 |
| openstackgerrit | Simon Chang proposed a change to openstack/python-troveclient: Add calls to the datastore capabilities API https://review.openstack.org/103211 | 18:41 |
| amrith | I'll let you do that. | 18:41 |
| mriedem | amrith: in https://review.openstack.org/#/c/91068/, if he sync'ed using the openstack-common.conf file, that pulled in the __init__.py chnage | 18:42 |
| mriedem | so i'd be fine with that | 18:42 |
| amrith | the commit id in the commit message (0f4586c0076183c6356eec682c8a593648125abd) is kaka | 18:42 |
| amrith | it was cut-and-pasted sans 1 digit/letter | 18:43 |
| amrith | maybe | 18:43 |
| *** rueben has joined #openstack-trove | 18:44 | |
| mriedem | amrith: it's based on commit 18f2bc1bf080b41a22c70842e7c127da21c63b8b | 18:45 |
| mriedem | so yeah the commit message has the wrong ref | 18:45 |
| amrith | mriedem, how did you find that? | 18:46 |
| mriedem | compared the changes in the review against the git history in the same file in oslo-incubator | 18:46 |
| mriedem | plus based on when it was last updated | 18:46 |
| amrith | ok, the hard way ;) | 18:47 |
| mriedem | when the committer doesn't list the commits, that's what you have to do, | 18:47 |
| mriedem | which is why -1 for not listing them | 18:48 |
| mriedem | like a gentleman | 18:48 |
| amrith | doesn't look like the same change | 18:50 |
| mriedem | amrith: well the sync is cumulative | 18:50 |
| amrith | the jsonutils.py appears to be different | 18:51 |
| amrith | non-overlapping | 18:51 |
| mriedem | it's behind by 1 commit from master in o-i | 18:51 |
| amrith | i see ... the merge into trove is a collection of changes from o-i | 18:52 |
| mriedem | yes | 18:52 |
| mriedem | amrith: https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator | 18:52 |
| mriedem | amrith: it's basically just a script that copies code from oslo to the target project | 18:53 |
| mriedem | so honestly, i think i'm going to just rebase on top of https://review.openstack.org/#/c/91068/ and clean it up (the commit message), then do the sync i need to do on top of that (to get processutils synced over) | 18:57 |
| mriedem | and the other sync for gettextutils will be left out to dry | 18:58 |
| openstackgerrit | amrith proposed a change to openstack/trove: Logging audit for trove/common module. https://review.openstack.org/103217 | 18:59 |
| *** julienvey has joined #openstack-trove | 19:01 | |
| *** yidclare has quit IRC | 19:01 | |
| *** julienvey has quit IRC | 19:06 | |
| *** demorris_ has joined #openstack-trove | 19:07 | |
| *** ramishra has quit IRC | 19:09 | |
| amrith | mriedem, thanks much for your help, this has been most useful | 19:09 |
| *** demorris has quit IRC | 19:09 | |
| *** demorris_ is now known as demorris | 19:09 | |
| mriedem | amrith: np | 19:09 |
| openstackgerrit | Simon Chang proposed a change to openstack/trove: Implement datastore capabilities API https://review.openstack.org/103219 | 19:12 |
| *** openstackgerrit has quit IRC | 19:20 | |
| *** openstackgerrit has joined #openstack-trove | 19:20 | |
| bwilhelmi | Trove Question: Got trove-integration working on Ubuntu 14.04 VM with 8 GB memory. Trove instance deploys successfully, but takes a looooooong time to achieve Active status. Is this normal? | 19:23 |
| abramley | bwilhelmi - is your ubuntu the base OS or running in a vm ? | 19:24 |
| openstackgerrit | Simon Chang proposed a change to openstack/trove: Implement datastore capabilities API https://review.openstack.org/103219 | 19:24 |
| bwilhelmi | Running in a VM | 19:25 |
| amrith | bwilhelmi, does it eventually get to active | 19:26 |
| amrith | and is it the trove vm that takes a long time? what about a nova vm with ubuntu | 19:26 |
| abramley | bwilhelmi - so you should check to verify that your ubuntu vm is capable of running hardware acceleration - by running kvm-ok | 19:26 |
| abramley | are you using vmware / virtualbox / other ? | 19:26 |
| *** demorris has quit IRC | 19:28 | |
| abramley | You want it to say: KVM acceleration can be used | 19:28 |
| bwilhelmi | I'm using HP Cloud VM. kvm-ok returns that KVM acceleration cannot be used | 19:29 |
| abramley | Check out https://github.com/openstack/trove-integration - the section on VMWare Fusion speed improvements | 19:29 |
| bwilhelmi | OK | 19:29 |
| *** grapex_ has quit IRC | 19:29 | |
| bwilhelmi | Trove Question: trove-integration uses localrc. Can I use local.conf instead? | 19:29 |
| *** jcru has quit IRC | 19:30 | |
| amrith | bwilhelmi, if KVM can't be used that's a definite issue ... | 19:30 |
| bwilhelmi | Where are the trove server requirements specified? I don't recall seeing that anywhere. | 19:31 |
| *** robertmyers has quit IRC | 19:31 | |
| *** juantwo has joined #openstack-trove | 19:32 | |
| *** demorris has joined #openstack-trove | 19:32 | |
| SlickNik | bwilhelmi: It's not a requirements issue but a perf issue. Nova, and consequently trove, will run virt–in-virt without hardware extensions enabled, but it's going to be slow — as you found out. | 19:34 |
| *** yidclare has joined #openstack-trove | 19:34 | |
| amrith | Actually SlickNik it will be SSSSSSLLLLLLLLLOOOOOOOOOOOOOOOOOOOOOOOOOOWWW | 19:35 |
| bwilhelmi | Thanks NIK. Even with a VM that has lots of RAM and vCPUs, very sloow | 19:35 |
| *** pdmars has quit IRC | 19:35 | |
| amrith | abramley, good catch! | 19:36 |
| mriedem | bwilhelmi: 250cd88 Fixed a new pep8 error and a small typo | 19:36 |
| mriedem | 71208fe Fix spelling errors in comments | 19:36 |
| mriedem | 7013471 Fix typo in parameter documentation (timeutils) | 19:36 |
| mriedem | ae931a9 Avoid TypeError in is_older_than, is_newer_than | 19:36 |
| mriedem | 12bcdb7 Remove vim header | 19:36 |
| mriedem | f3b5f17 Add helper method total_seconds in timeutils.py | 19:36 |
| mriedem | 53ebd30 python3: use six.text_types for unicode() | 19:36 |
| mriedem | 3bc6f79 Fix timeutils.set_override_time not defaulting to current wall time | 19:36 |
| mriedem | af76064 Optimize timeutils.utcnow_ts() | 19:36 |
| mriedem | oops, sorry! | 19:36 |
| mriedem | bwilhelmi: http://docs.openstack.org/trunk/config-reference/content/qemu.html | 19:36 |
| amrith | thanks Matt, time to review some code ;) | 19:36 |
| mriedem | doc on nested virt with nova | 19:36 |
| mriedem | amrith: running tox locally now | 19:36 |
| amrith | mriedem, thanks. I'll look for the change when it comes up | 19:38 |
| amrith | I'd like to see how it is really done ... | 19:38 |
| bwilhelmi | What about the devstack envronment--once installed, can you use unstack/stack, or is there equivalent trove-integration commands? | 19:38 |
| mriedem | my god the trove tests run quickly | 19:38 |
| bwilhelmi | mriedem: thanks for the link | 19:39 |
| amrith | mriedem, you'd asked earlier if there were any cores around. Yes SlickNik is here and you can thank him for the test speedup ;) | 19:39 |
| mriedem | amrith: well it doesn't hurt that there aren't 10K unit tests | 19:39 |
| amrith | oh, you said tox, maybe that too but I was thinking of somethign else | 19:39 |
| mriedem | i'm used to going for coffee when i rebuild tox for nova tests | 19:39 |
| SlickNik | bwilhelmi: Also on the local.conf front, the install pieces in trove-integration haven't been updated since devstack started allowing the local.conf format. Since devstack still supports the localrc format it hasn't been high on our list of priorities, either. | 19:40 |
| *** PeterSchwaller has joined #openstack-trove | 19:40 | |
| *** michael-yu has quit IRC | 19:40 | |
| SlickNik | bwilhelmi: You should be able to run "~/devstack/unstack.sh", followed by "./redstack install" and "./redstack kick-start mysql" to reset your environment. | 19:42 |
| *** sgotliv has joined #openstack-trove | 19:42 | |
| *** PeterSchwaller_ has quit IRC | 19:43 | |
| *** mordred has quit IRC | 19:43 | |
| bwilhelmi | Thanks Nik. Does trove-integration care if I enable Neutron? | 19:43 |
| *** mordred has joined #openstack-trove | 19:43 | |
| SlickNik | bwilhelmi: There's currently a bunch of workarounds needed to enable neutron. There's a patch in the works to address this, so once that merges these workarounds won't be required. | 19:45 |
| SlickNik | bwilhelmi: https://review.openstack.org/#/c/78123/ | 19:45 |
| *** yidclare has quit IRC | 19:46 | |
| openstackgerrit | Simon Chang proposed a change to openstack/trove: Implement datastore capabilities API tests https://review.openstack.org/103228 | 19:48 |
| bwilhelmi | Thanks Nik, mriedem, and others. Y'all are very helpful! | 19:52 |
| schang | SkickNik: I have a question about commit dependancies that spans multiple projects. I'm working on the capailities API bp and the API test in trove depends on my other change in python-troveclient. I'm leaving the API test commit with workflow -1 because it will break without the python-troveclient changes. Can you take a look at my commits and advise at what point I should do with the API test commit? | 20:01 |
| schang | SlickNik: I have a question about commit dependancies that spans multiple projects. I'm working on the capailities API bp and the API test in trove depends on my other change in python-troveclient. I'm leaving the API test commit with workflow -1 because it will break without the python-troveclient changes. Can you take a look at my commits and advise at what point I should do with the API test commit? | 20:01 |
| *** julienvey has joined #openstack-trove | 20:02 | |
| schang | SlickNik: My commits are as follows: API Implementation: | 20:02 |
| schang | https://review.openstack.org/#/c/103219/ | 20:02 |
| schang | API Test: | 20:02 |
| schang | https://review.openstack.org/#/c/103228/ | 20:02 |
| schang | Trove client | 20:02 |
| schang | https://review.openstack.org/#/c/103211/ | 20:02 |
| *** rueben has quit IRC | 20:03 | |
| *** thedodd has quit IRC | 20:05 | |
| *** julienvey has quit IRC | 20:07 | |
| openstackgerrit | Matt Riedemann proposed a change to openstack/trove: Sync jsonutils (and dependencies) from oslo-incubator https://review.openstack.org/91068 | 20:11 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/trove: Sync processutils from oslo with deps https://review.openstack.org/103232 | 20:11 |
| mriedem | amrith: ^ that's how you do it | 20:11 |
| mriedem | i still have to make the change that sits on top of that series | 20:12 |
| *** boden has quit IRC | 20:12 | |
| *** flaper87 has quit IRC | 20:18 | |
| *** flaper87 has joined #openstack-trove | 20:18 | |
| *** fifieldt has quit IRC | 20:19 | |
| *** ViswaV has quit IRC | 20:19 | |
| SlickNik | schang: Thanks, will take a look when I get to reviews. | 20:28 |
| SlickNik | schang: btw, did you see https://review.openstack.org/#/c/102450/? | 20:28 |
| schang | SlickNik: I saw that review and talked to Rueben yesterday about it. | 20:29 |
| SlickNik | schang: I haven't gotten to reviewing it yet, but a cursory glance suggests that that patch also involves capabilities. | 20:29 |
| SlickNik | schang: cool, what did you guys discuss / decide? | 20:30 |
| SlickNik | schang: Just want to make sure Rueben and you are aware and aren't duplicating effort :) | 20:30 |
| schang | SlickNik: It does, that commit implement capabilities for mgt.cmd, not the api. I've confirmed with Rueben that he won't be doing the api part. | 20:31 |
| *** fifieldt has joined #openstack-trove | 20:31 | |
| SlickNik | schang: Awesome; thanks for following up on that! | 20:32 |
| schang | SlickNik: His change however, do involve a change in datastore/models.py, which later on my api stuff may use. | 20:32 |
| *** sgotliv has quit IRC | 20:32 | |
| *** yidclare has joined #openstack-trove | 20:36 | |
| bwilhelmi | $ trove metadata-list b173d0d9-c1f3-4c27-8f7f-ae6866556c6e / ERROR: The resource could not be found. (HTTP 404) -- does this indicate that there is no meta-data? If so, wouldn't a blank table be better? | 20:36 |
| *** thedodd has joined #openstack-trove | 20:45 | |
| SlickNik | bwilhelmi: The metadata feature hasn't been implemented yet. It's sill in progress. That's probably why you're seeing that error message. | 20:47 |
| openstackgerrit | Matt Riedemann proposed a change to openstack/trove: Use (# of CPUs) api/conductor workers by default https://review.openstack.org/103239 | 20:48 |
| SlickNik | bwilhelmi: https://review.openstack.org/#/c/82123/ | 20:48 |
| bwilhelmi | ok. thanks nik | 20:50 |
| *** thedodd has quit IRC | 20:50 | |
| *** coolsvapl has joined #openstack-trove | 20:51 | |
| *** coolsvap|afk has quit IRC | 20:52 | |
| *** iccha1 has quit IRC | 20:52 | |
| *** iccha1 has joined #openstack-trove | 20:57 | |
| *** mattgriffin has quit IRC | 20:58 | |
| openstackgerrit | Dan Nguyen proposed a change to openstack/python-troveclient: Adds support for Keystone v3 API https://review.openstack.org/102315 | 21:02 |
| *** julienvey has joined #openstack-trove | 21:02 | |
| *** ViswaV has joined #openstack-trove | 21:06 | |
| *** julienvey has quit IRC | 21:07 | |
| *** demorris has quit IRC | 21:08 | |
| *** sgotliv has joined #openstack-trove | 21:15 | |
| *** ViswaV has quit IRC | 21:16 | |
| *** ViswaV has joined #openstack-trove | 21:19 | |
| *** ViswaV has quit IRC | 21:24 | |
| *** vgnbkr_ has joined #openstack-trove | 21:24 | |
| *** parstac_pete_ has joined #openstack-trove | 21:24 | |
| *** zigo_ has joined #openstack-trove | 21:24 | |
| *** demorris has joined #openstack-trove | 21:24 | |
| *** mayfield_ has joined #openstack-trove | 21:25 | |
| *** sgotliv has quit IRC | 21:30 | |
| *** nehav1 has quit IRC | 21:31 | |
| *** mayfield has quit IRC | 21:31 | |
| *** peterstac has quit IRC | 21:31 | |
| *** zacksh has quit IRC | 21:31 | |
| *** zigo has quit IRC | 21:31 | |
| *** erik_x1 has quit IRC | 21:31 | |
| *** vgnbkr has quit IRC | 21:31 | |
| *** k-pom has quit IRC | 21:31 | |
| *** abramley has quit IRC | 21:31 | |
| *** vgnbkr_ is now known as vgnbkr | 21:31 | |
| *** parstac_pete_ is now known as peterstac | 21:31 | |
| *** abramley has joined #openstack-trove | 21:32 | |
| *** radez is now known as radez_g0n3 | 21:32 | |
| *** ViswaV has joined #openstack-trove | 21:37 | |
| *** ViswaV has quit IRC | 21:38 | |
| *** julienvey has joined #openstack-trove | 21:39 | |
| *** julienvey has quit IRC | 21:41 | |
| *** demorris has quit IRC | 21:41 | |
| *** julienvey has joined #openstack-trove | 21:41 | |
| *** demorris has joined #openstack-trove | 21:43 | |
| *** flaper87 is now known as flaper87|afk | 21:46 | |
| *** julienvey has quit IRC | 21:46 | |
| *** yogeshmehra has quit IRC | 21:49 | |
| *** rwsu has quit IRC | 21:49 | |
| *** mriedem has left #openstack-trove | 22:04 | |
| *** mriedem has joined #openstack-trove | 22:17 | |
| *** yidclare has quit IRC | 22:37 | |
| *** mriedem has left #openstack-trove | 22:38 | |
| *** julienvey has joined #openstack-trove | 22:42 | |
| *** julienvey has quit IRC | 22:47 | |
| *** achampion has quit IRC | 22:49 | |
| *** rhodgin has quit IRC | 22:51 | |
| *** demorris has quit IRC | 22:55 | |
| *** juantwo has quit IRC | 22:59 | |
| *** juantwo has joined #openstack-trove | 23:00 | |
| openstackgerrit | Shayne Burgess proposed a change to openstack/trove: Fix for unreferences variable https://review.openstack.org/103265 | 23:01 |
| openstackgerrit | Shayne Burgess proposed a change to openstack/trove: Fix for unreferenced variable https://review.openstack.org/103265 | 23:02 |
| *** ViswaV has joined #openstack-trove | 23:31 | |
| *** jmontemayor_ has quit IRC | 23:34 | |
| *** openstackgerrit has quit IRC | 23:37 | |
| *** julienvey has joined #openstack-trove | 23:42 | |
| *** ViswaV_ has joined #openstack-trove | 23:46 | |
| *** yidclare has joined #openstack-trove | 23:47 | |
| *** julienvey has quit IRC | 23:47 | |
| *** ViswaV has quit IRC | 23:48 | |
| *** rwsu has joined #openstack-trove | 23:54 | |
| *** yidclare has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!