| *** tqtran_ has quit IRC | 00:11 | |
| *** tqtran has joined #openstack-trove | 00:24 | |
| amrith | peterstac, I'm reviewing that change now. | 00:27 |
|---|---|---|
| *** tqtran has quit IRC | 00:57 | |
| *** cheneydc has joined #openstack-trove | 01:05 | |
| *** tqtran has joined #openstack-trove | 01:08 | |
| *** chlong has joined #openstack-trove | 01:18 | |
| openstackgerrit | gengchc2 proposed openstack/trove: Translate info-level log messages for LOG.info https://review.openstack.org/360437 | 01:35 |
| *** tqtran has quit IRC | 01:39 | |
| openstackgerrit | gengchc2 proposed openstack/trove: Translate info-level log messages for LOG.error https://review.openstack.org/360449 | 01:43 |
| openstackgerrit | gengchc2 proposed openstack/trove: Translate info-level log messages for LOG.error https://review.openstack.org/360449 | 01:52 |
| *** pmackinn has quit IRC | 01:59 | |
| *** cheneydc has quit IRC | 02:04 | |
| openstackgerrit | jian.song proposed openstack/python-troveclient: Display flavor-ephemeral in trove flavor-list command https://review.openstack.org/359579 | 02:16 |
| *** songjian has joined #openstack-trove | 02:21 | |
| songjian | amrith,I have rebase to the latest version, please help me to review,thx... | 02:22 |
| openstackgerrit | jian.song proposed openstack/trove-dashboard: Remove white space between print and () https://review.openstack.org/360867 | 03:21 |
| openstackgerrit | jian.song proposed openstack/trove-dashboard: Remove white space between print and () https://review.openstack.org/360867 | 03:24 |
| *** chlong has quit IRC | 03:37 | |
| *** links has joined #openstack-trove | 04:07 | |
| *** chlong has joined #openstack-trove | 04:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Updated from global requirements https://review.openstack.org/360899 | 05:15 |
| *** chlong has quit IRC | 05:21 | |
| *** ishashero has quit IRC | 06:01 | |
| openstackgerrit | gengchc2 proposed openstack/trove: Translate info-level log messages for LOG.info https://review.openstack.org/360437 | 06:13 |
| *** rcernin has joined #openstack-trove | 06:26 | |
| *** tesseract- has joined #openstack-trove | 06:44 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove-dashboard: Imported Translations from Zanata https://review.openstack.org/360954 | 07:54 |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** openstackgerrit has joined #openstack-trove | 08:18 | |
| *** tqtran has joined #openstack-trove | 08:39 | |
| *** tqtran has quit IRC | 08:43 | |
| openstackgerrit | Andreas Jaeger proposed openstack/trove-image-builder: Cleanup tox.ini https://review.openstack.org/360992 | 08:54 |
| *** khushbu_ has joined #openstack-trove | 09:12 | |
| kmARC | hi all, I'm having problems with trove-taskmanager connecting to nova. I double (triple) checked credentials, yet I get back the following errors: | 09:41 |
| kmARC | http://paste.openstack.org/show/563823/ | 09:41 |
| kmARC | creating an openrc.sh with the same API endpoints and credentials works, even with `nova list --all-tenants` | 09:42 |
| *** tosky has joined #openstack-trove | 09:51 | |
| dougshelley66 | kmARC - that code needs a admin user,pwd and tenant set in CONF (nova_proxy_admin_user/pass/tenant_id) - are these all set to valid values and the user has admin access to the tenant? | 10:58 |
| kmARC | hmmm | 11:03 |
| kmARC | i have the tenant specified not the id | 11:03 |
| kmARC | let me check | 11:03 |
| kmARC | thanks for pointing me to there | 11:03 |
| dougshelley66 | np | 11:04 |
| dougshelley66 | btw, your other option is to disable that exists_event | 11:04 |
| dougshelley66 | you likely don't need it | 11:04 |
| kmARC | yepp, it solved the issue, thanks dougshelley66 | 11:05 |
| dougshelley66 | glad to hear it | 11:06 |
| *** tosky has quit IRC | 11:13 | |
| *** tosky has joined #openstack-trove | 11:15 | |
| *** songjian has quit IRC | 11:45 | |
| *** songjian has joined #openstack-trove | 11:46 | |
| *** pmackinn has joined #openstack-trove | 12:12 | |
| *** johnma has quit IRC | 12:16 | |
| *** tqtran has joined #openstack-trove | 12:41 | |
| *** tqtran has quit IRC | 12:45 | |
| *** khushbu_ has quit IRC | 12:46 | |
| *** songjian has quit IRC | 12:55 | |
| openstackgerrit | Merged openstack/trove: Updated from global requirements https://review.openstack.org/360899 | 13:03 |
| *** johnma has joined #openstack-trove | 13:24 | |
| *** links has quit IRC | 13:45 | |
| *** ShaikApsar has joined #openstack-trove | 14:12 | |
| *** chlong has joined #openstack-trove | 14:32 | |
| amrith | ShaikApsar, yt? | 14:32 |
| ShaikApsar | amrith: yes | 14:32 |
| amrith | yes, so I'm trying to figure out what that change is doing | 14:33 |
| amrith | and I can't see what exactly is being accomplished | 14:33 |
| amrith | with all the twists and turns on that bug | 14:33 |
| amrith | is it the empty catalog issue? | 14:33 |
| amrith | or something else | 14:33 |
| amrith | so I'll work my way through it | 14:33 |
| ShaikApsar | amrith: it;s not the empty catalog issue | 14:34 |
| ShaikApsar | amrith: AttributeError: 'Client' object has no attribute 'tenants' errors in trove-taskmanager.log | 14:34 |
| ShaikApsar | amrith: https://github.com/openstack/trove/blob/master/trove/taskmanager/manager.py#L421 | 14:35 |
| ShaikApsar | amrith: code here is trying to get the tenant list from novaclient and it;s failing | 14:35 |
| ShaikApsar | amrith: we should use keystoneclient to get the tenant list | 14:36 |
| amrith | thank you, yes, I see what you are changing | 14:37 |
| amrith | leaving aside the fact that the code isn't getting run for me (and that's my problem to figure out) I'm not sure how these notifications are even useful | 14:37 |
| amrith | because as best as I can tell, the payload doesn't tell who the tenant is | 14:37 |
| amrith | notification.py@266 | 14:38 |
| amrith | the payload does not include the tenant | 14:38 |
| amrith | therefore, as best as I can tell, you have prevented an error but not really generated a notification that was of any value. | 14:42 |
| amrith | that's the reason why I'm very interested to actually test the code | 14:42 |
| amrith | and figure out what exactly is going on | 14:42 |
| amrith | My bet is that if I have multiple tenants | 14:42 |
| amrith | who each have trove instances | 14:42 |
| amrith | you will get notifications for each tenant's quota usages without actually knowing who that tenant is | 14:43 |
| amrith | and that (IMHO) is rather pointless | 14:43 |
| amrith | but that is my hypothesis | 14:43 |
| amrith | which I would like to either prove or disprove | 14:43 |
| amrith | hence I am NOT comfortable with a review of this code purely on the merit of the fact that there's no method in nova to get all tenants. | 14:43 |
| amrith | furthermore, I see that the notification method also receives a context which is in the notification itself | 14:44 |
| amrith | I have no idea what the security implications of that are | 14:44 |
| ShaikApsar | amrith: publish_quota_notifications is trying to update tenant quota information | 14:45 |
| amrith | you are effectively sending a notification with a context object | 14:45 |
| ShaikApsar | amrith: in database | 14:45 |
| amrith | so how is it doing that? | 14:45 |
| amrith | clearly you are saying that this didn't happen in the past | 14:45 |
| amrith | but I know that I've had cases where I have run out of trove quotas | 14:45 |
| amrith | so something was tracking quotas. | 14:45 |
| amrith | ShaikApsar, without your change, I get the following error every 30s in my log file | 14:51 |
| amrith | 2016-08-26 10:50:35.215 DEBUG oslo_service.periodic_task [-] Running periodic task Manager.publish_quota_notifications from (pid=103884) run_periodic_tasks /usr/local/lib/python2.7/dist-packages/oslo_service/periodic_task.py:215 | 14:51 |
| amrith | 2016-08-26 10:50:35.216 ERROR oslo_service.periodic_task [-] Error during Manager.publish_quota_notifications | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task Traceback (most recent call last): | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task File "/usr/local/lib/python2.7/dist-packages/oslo_service/periodic_task.py", line 220, in run_periodic_tasks | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task task(self, context) | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task File "/opt/stack/trove/trove/taskmanager/manager.py", line 420, in publish_quota_notifications | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task nova_client = remote.create_nova_client(self.admin_context) | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task File "/opt/stack/trove/trove/common/remote.py", line 99, in nova_client | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task endpoint_type=CONF.nova_compute_endpoint_type) | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task File "/opt/stack/trove/trove/common/remote.py", line 58, in get_endpoint | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task raise exception.EmptyCatalog() | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task EmptyCatalog: Empty catalog. | 14:51 |
| amrith | 2016-08-26 10:50:35.216 TRACE oslo_service.periodic_task | 14:51 |
| amrith | this is the Empty Catalog error | 14:51 |
| amrith | I don't get a message about no method etc.,etc., | 14:51 |
| ShaikApsar | amrith: without my changes, Code is trying to prepare the novaclient and in devstack setup we didn't provide the nova_url | 14:53 |
| ShaikApsar | amrith: that's the reason it's failing with Empty Catalog error | 14:53 |
| ShaikApsar | amrith: If we use keystone client to get the tenant list then you won't see Empty Catalog error | 14:54 |
| ShaikApsar | amrith: that was the reason i created the defect for Empty Catalog error | 14:58 |
| amrith | ok, so I reverted the code to master and created two instances with tenant1 and two instances with tenant2 | 14:58 |
| amrith | the quota was properly reflected in the db | 14:58 |
| amrith | then I deleted the instances | 14:58 |
| amrith | and quotas came back to 0 | 14:58 |
| amrith | and all along, I got the empty catalog errro | 14:58 |
| amrith | so the code that you are adding (notifications) isn't the thing that is updating the database. | 14:58 |
| amrith | i'll contine to debug, now I have to go attend another meeting. | 14:59 |
| *** drifterza has joined #openstack-trove | 15:02 | |
| *** spilla has quit IRC | 15:02 | |
| *** rcernin has quit IRC | 15:04 | |
| ShaikApsar | amrith: https://github.com/openstack/trove/blob/3196d347f00363f84c8986a959ff8c49f104a262/trove/taskmanager/manager.py#L51 | 15:08 |
| ShaikApsar | amrith: it;s setting nova_admin password as token | 15:09 |
| ShaikApsar | it will be security issue | 15:10 |
| ShaikApsar | that has to be fixed | 15:10 |
| *** ShaikApsar has quit IRC | 15:15 | |
| *** drifterza has quit IRC | 15:30 | |
| *** trevormc has joined #openstack-trove | 15:31 | |
| *** pcaruana has quit IRC | 15:33 | |
| *** gouthamr has joined #openstack-trove | 15:38 | |
| *** tesseract- has quit IRC | 15:38 | |
| *** tqtran has joined #openstack-trove | 16:07 | |
| *** chlong has quit IRC | 16:07 | |
| *** tqtran has quit IRC | 16:11 | |
| *** itisha has joined #openstack-trove | 16:18 | |
| *** chlong has joined #openstack-trove | 16:19 | |
| *** david-lyle has joined #openstack-trove | 16:36 | |
| openstackgerrit | chen.xing proposed openstack/trove: [install] Use one command to create endpoint https://review.openstack.org/361359 | 16:43 |
| *** links has joined #openstack-trove | 16:44 | |
| *** rcernin has joined #openstack-trove | 16:44 | |
| *** chlong has quit IRC | 17:02 | |
| *** khushbu has joined #openstack-trove | 17:14 | |
| *** rcernin has quit IRC | 17:22 | |
| *** tosky has quit IRC | 17:28 | |
| openstackgerrit | amrith proposed openstack/trove: Update api-ref documentation https://review.openstack.org/357792 | 17:35 |
| *** tqtran has joined #openstack-trove | 18:04 | |
| amrith | peterstac, ping ... https://review.openstack.org/#/c/288297/ | 18:30 |
| peterstac | amrith, I meant to look at that (since I think it's a valid bug) but haven't got around to it | 18:31 |
| amrith | thx peterstac ... I didn't want to update the review and reset the timer on it | 18:33 |
| amrith | fyi, see above conversation with ShaikApsar. | 18:33 |
| amrith | I don't understand that patch or what it is doing completely | 18:33 |
| peterstac | right - I'll see if I can squeeze it in next week amidst the end-of-release crush ;) | 18:33 |
| *** khushbu has quit IRC | 18:33 | |
| amrith | no worries, it isn't crucial for next week | 18:34 |
| amrith | it is a 'sometime' thing ... | 18:34 |
| *** tqtran has quit IRC | 18:56 | |
| *** tqtran has joined #openstack-trove | 18:57 | |
| trevormc | hey pmalik, I'm trying to test this patch but I can't seem to get my flavor-list output to change after I checkout the patch. https://review.openstack.org/#/c/352786/ Is the python-troveclient being used instead or am I doing something wrong? | 19:33 |
| *** links has quit IRC | 19:33 | |
| trevormc | I use the default install from redstack | 19:34 |
| *** tqtran has quit IRC | 19:42 | |
| *** tqtran has joined #openstack-trove | 19:42 | |
| *** gouthamr has quit IRC | 20:01 | |
| *** zhiyan has quit IRC | 20:03 | |
| *** ctracey has quit IRC | 20:04 | |
| *** serverascode has quit IRC | 20:04 | |
| *** johnma has quit IRC | 20:08 | |
| *** gouthamr has joined #openstack-trove | 20:23 | |
| *** dkehn_ has quit IRC | 20:34 | |
| *** ctracey has joined #openstack-trove | 20:40 | |
| *** serverascode has joined #openstack-trove | 20:43 | |
| *** zhiyan has joined #openstack-trove | 20:45 | |
| *** markstur has joined #openstack-trove | 21:00 | |
| *** xb_ has quit IRC | 21:08 | |
| *** tqtran has quit IRC | 21:14 | |
| *** xb_ has joined #openstack-trove | 21:15 | |
| *** tqtran has joined #openstack-trove | 21:26 | |
| *** gouthamr_ has joined #openstack-trove | 21:33 | |
| openstackgerrit | Petr Malik proposed openstack/trove: Fix race in Postgres user-list https://review.openstack.org/361477 | 21:36 |
| *** gouthamr has quit IRC | 21:36 | |
| *** trevormc has quit IRC | 21:53 | |
| *** gouthamr_ is now known as gouthamr | 21:57 | |
| openstackgerrit | Petr Malik proposed openstack/trove: Tests verify cluster data via every node https://review.openstack.org/355043 | 22:03 |
| openstackgerrit | Petr Malik proposed openstack/trove: Add Couchbase helper client methods https://review.openstack.org/356698 | 22:20 |
| *** tqtran has quit IRC | 22:30 | |
| *** tqtran has joined #openstack-trove | 22:34 | |
| *** itisha has quit IRC | 22:40 | |
| *** serverascode has quit IRC | 23:01 | |
| *** ctracey has quit IRC | 23:01 | |
| *** zhiyan has quit IRC | 23:02 | |
| *** bapalm_ has quit IRC | 23:06 | |
| openstackgerrit | Merged openstack/trove: Couchbase run backup as root https://review.openstack.org/356752 | 23:09 |
| *** bapalm has joined #openstack-trove | 23:11 | |
| *** ctracey has joined #openstack-trove | 23:15 | |
| *** serverascode has joined #openstack-trove | 23:20 | |
| *** zhiyan has joined #openstack-trove | 23:28 | |
| *** amit213 has quit IRC | 23:37 | |
| *** amit213 has joined #openstack-trove | 23:40 | |
| *** ctracey has quit IRC | 23:46 | |
| *** zhiyan has quit IRC | 23:46 | |
| *** serverascode has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!