| *** imandhan has joined #openstack-trove | 00:01 | |
| *** akanksha_ has quit IRC | 00:08 | |
| *** amrith is now known as _amrith_ | 00:09 | |
| *** _amrith_ is now known as amrith | 00:27 | |
| *** ashleighfarnham has quit IRC | 00:28 | |
| dougshelley66 | SlickNik yt? | 00:45 |
|---|---|---|
| *** amit213 has quit IRC | 00:45 | |
| SlickNik | dougshelley66: What's up? | 00:45 |
| dougshelley66 | so i've been struggling with the exist notifier | 00:50 |
| dougshelley66 | i believe it is busted in liberty | 00:50 |
| dougshelley66 | i was looking at your single_tenant_remote.py change and it seems like to properly instantiate the nova client you need both the tenant id and name | 00:50 |
| dougshelley66 | our current create_admin_nova_client method doesn't have both so it is generating a nova client that can't authenticate | 00:51 |
| *** shakamunyi has joined #openstack-trove | 00:55 | |
| openstackgerrit | DongCan proposed openstack/trove: Fix the bug of "Fix spelling typo in trove" https://review.openstack.org/227718 | 00:59 |
| cp16net | dougshelley66: i'm here | 01:18 |
| dougshelley66 | ok so i'm trying to figure out https://bugs.launchpad.net/trove/+bug/1473550 | 01:19 |
| openstack | Launchpad bug 1473550 in Trove "enabling exist notifications cause auth errors" [Undecided,New] | 01:19 |
| dougshelley66 | from what i can tell, the "project_id" in novaclient is supposed to be a tenant name | 01:19 |
| dougshelley66 | not a UUID | 01:19 |
| dougshelley66 | which is clear from the single_tenant_remote.py that got merged recently | 01:20 |
| dougshelley66 | it seems like we need to change the base remote.py to be similar | 01:20 |
| cp16net | dougshelley66: yes exactly | 01:20 |
| dougshelley66 | ok | 01:20 |
| cp16net | i actaully figured out how to get the remote.py to work from the single_tenant_remote.py we used | 01:21 |
| cp16net | it caused much strife | 01:21 |
| dougshelley66 | well i think we need to add a "tenant_name" to TroveContext | 01:21 |
| cp16net | thats why i opened that bug | 01:21 |
| dougshelley66 | for the admin context | 01:21 |
| cp16net | hmmm | 01:21 |
| dougshelley66 | so my big question | 01:21 |
| cp16net | umm well i think there was another way... | 01:21 |
| dougshelley66 | well i'm all ears because this is totally broken | 01:22 |
| cp16net | i think overriding the management_url was the solution | 01:22 |
| cp16net | now its coming back to me | 01:22 |
| dougshelley66 | i can't get that to work with nova v2 api | 01:22 |
| cp16net | so the client to authentitcate needs the name... | 01:22 |
| dougshelley66 | the single tenant thing is specifically using nova v1.1 | 01:22 |
| cp16net | but the url needs the id | 01:22 |
| dougshelley66 | yes | 01:22 |
| dougshelley66 | i think we need both | 01:22 |
| dougshelley66 | like the single tenant thing | 01:22 |
| cp16net | so thats why you see the tenant_id ="xxxx" | 01:23 |
| cp16net | # this is terrible | 01:23 |
| cp16net | with that comment | 01:23 |
| dougshelley66 | i saw that | 01:23 |
| cp16net | but it was required in my setup | 01:23 |
| dougshelley66 | so we put back the nova_proxy_tenant_name with the single tenant comit | 01:23 |
| cp16net | so i might see a few ways around this... | 01:23 |
| dougshelley66 | so for the admin context both are now available | 01:23 |
| cp16net | when you create the client with the tenant_name you might be able to pull out the tenant_id | 01:24 |
| cp16net | (might be hacky tho) | 01:24 |
| cp16net | otherwise have a config option for the admin tenant_id | 01:25 |
| cp16net | like you are saying | 01:25 |
| cp16net | so dougshelley66 are you working on this bug? | 01:27 |
| dougshelley66 | yes | 01:27 |
| cp16net | nice :) | 01:27 |
| dougshelley66 | just wanted to get some thoughts before i went down the path | 01:27 |
| cp16net | i didnt have time because i was working on the clustering stuff when i peeked into this stuff | 01:27 |
| cp16net | no worries | 01:27 |
| dougshelley66 | thanks - i will put something together and send it up. | 01:28 |
| cp16net | sweet! thanks! | 01:28 |
| *** ViswaV_ has quit IRC | 01:33 | |
| *** tomblank has joined #openstack-trove | 01:48 | |
| *** imandhan has quit IRC | 01:57 | |
| *** imandhan has joined #openstack-trove | 01:58 | |
| *** amrith is now known as _amrith_ | 01:59 | |
| *** imandhan has quit IRC | 02:02 | |
| *** sabeen has joined #openstack-trove | 02:18 | |
| *** pmackinn has quit IRC | 02:45 | |
| openstackgerrit | Doug Shelley proposed openstack/trove: Fix publish_exists_event authentication exception https://review.openstack.org/228696 | 02:46 |
| dougshelley66 | cp16net ^^^ let me know what you think | 02:46 |
| *** haomaiwang has joined #openstack-trove | 02:51 | |
| cp16net | dougshelley66: does it work removing the management_url? | 03:00 |
| *** haomaiwang has quit IRC | 03:01 | |
| cp16net | i thought that was nesseary... | 03:01 |
| *** haomaiwang has joined #openstack-trove | 03:01 | |
| cp16net | or the bypass_url was nesseary | 03:02 |
| *** imandhan has joined #openstack-trove | 03:04 | |
| cp16net | i'd like to test that out again before i ok the change. | 03:04 |
| *** sabeen has quit IRC | 03:07 | |
| cp16net | dougshelley66: it might be impossible or not worth it... but could we write a unit test for the create_admin_nova_client to be sure we have the fields set correctly in the client? | 03:08 |
| *** sabeen has joined #openstack-trove | 03:14 | |
| *** sabeen has quit IRC | 03:17 | |
| *** khushbu_ has joined #openstack-trove | 03:32 | |
| *** sabeen1 has joined #openstack-trove | 03:35 | |
| *** links has joined #openstack-trove | 03:36 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-trove | 04:01 | |
| *** khushbu_ has quit IRC | 04:38 | |
| *** sgotliv has joined #openstack-trove | 04:59 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-trove | 05:01 | |
| *** sgotliv has quit IRC | 05:04 | |
| *** tomblank has quit IRC | 05:34 | |
| SlickNik | dougshelley66 / cp16net: I'm reading scrollback. | 05:42 |
| cp16net | SlickNik: just talking about the exist events | 05:44 |
| SlickNik | I believe you don't have to have that management_url set if you set the project_id in the nova_client to (correctly) be the tenant name. | 05:45 |
| SlickNik | cp16net: overriding the mgmt url looks like a hack in the current remote.py, and dougshelley66's fix looks like it's doing the right thing — haven't verified it though, just from eyeballing the review. | 05:48 |
| cp16net | yeah i'd like to verify it first | 05:48 |
| cp16net | i wasnt sure | 05:49 |
| *** k4n0 has joined #openstack-trove | 05:50 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-trove | 06:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/trove: Imported Translations from Zanata https://review.openstack.org/226827 | 06:12 |
| *** openstackgerrit has quit IRC | 06:16 | |
| *** openstackgerrit has joined #openstack-trove | 06:16 | |
| *** sabeen1 has quit IRC | 06:26 | |
| *** imandhan has quit IRC | 06:42 | |
| *** imandhan has joined #openstack-trove | 06:43 | |
| *** imandhan has quit IRC | 06:44 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-trove | 07:01 | |
| *** khushbu_ has joined #openstack-trove | 07:07 | |
| *** jengeb has joined #openstack-trove | 07:35 | |
| *** exploreshaifali has joined #openstack-trove | 07:40 | |
| *** _amrith_ is now known as amrith | 07:50 | |
| *** jengeb has quit IRC | 07:57 | |
| *** amrith is now known as _amrith_ | 07:59 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-trove | 08:01 | |
| *** sgotliv has joined #openstack-trove | 08:30 | |
| openstackgerrit | Masaki Matsushita proposed openstack/trove: Add scheduler_hints attributes to create_instance https://review.openstack.org/228784 | 08:30 |
| *** exploreshaifali has quit IRC | 08:45 | |
| *** tosky has joined #openstack-trove | 08:55 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-trove | 09:01 | |
| *** davideagnello has quit IRC | 09:12 | |
| *** davideagnello has joined #openstack-trove | 09:13 | |
| *** jengeb has joined #openstack-trove | 09:15 | |
| *** jengeb has quit IRC | 09:34 | |
| *** PeterS has quit IRC | 09:38 | |
| *** PeterS has joined #openstack-trove | 09:50 | |
| *** khushbu_ has quit IRC | 09:54 | |
| *** kbyrne has quit IRC | 09:58 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwa_ has joined #openstack-trove | 10:01 | |
| *** khushbu_ has joined #openstack-trove | 10:01 | |
| *** kbyrne has joined #openstack-trove | 10:03 | |
| *** IanGovett has joined #openstack-trove | 10:04 | |
| *** haomaiwa_ has quit IRC | 10:07 | |
| *** jengeb1 has joined #openstack-trove | 10:28 | |
| dougshelley66 | cp16net, just catching up (turned in after I posted the change last night) | 10:37 |
| dougshelley66 | cp16net the bypass_url is being set in create_nova_client and if you look in the novaclient code, management_url is set to bypass_url | 10:38 |
| dougshelley66 | cp16net certainly more testing of the change would be good - I ran some tests to confirm that the exists event now fires without error | 10:39 |
| dougshelley66 | cp16net it looks like there are some existing unit tests for this, I will see what enhancements can/should be made to account for this | 10:41 |
| *** 18VAARFFB has joined #openstack-trove | 10:56 | |
| *** 18VAARFFB has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-trove | 11:01 | |
| *** exploreshaifali has joined #openstack-trove | 11:03 | |
| *** jengeb1 has quit IRC | 11:04 | |
| *** jengeb has joined #openstack-trove | 11:07 | |
| *** haomaiwang has quit IRC | 11:11 | |
| openstackgerrit | Doug Shelley proposed openstack/trove: Fix publish_exists_event authentication exception https://review.openstack.org/228696 | 11:15 |
| *** khushbu_ has quit IRC | 11:15 | |
| dougshelley66 | cp16net I added a unit test to that review ^^^ let me know what you think | 11:16 |
| *** jengeb has quit IRC | 11:16 | |
| *** exploreshaifali has quit IRC | 11:32 | |
| *** _amrith_ is now known as amrith | 11:49 | |
| *** exploreshaifali has joined #openstack-trove | 12:07 | |
| *** sgotliv_ has joined #openstack-trove | 12:08 | |
| *** sgotliv has quit IRC | 12:11 | |
| *** exploreshaifali has quit IRC | 12:40 | |
| *** amrith is now known as _amrith_ | 12:45 | |
| *** _crobertsrh is now known as crobertsrh | 12:48 | |
| *** daemontool_ has joined #openstack-trove | 13:10 | |
| *** k4n0 has quit IRC | 13:18 | |
| *** links has quit IRC | 13:22 | |
| *** khushbu_ has joined #openstack-trove | 13:27 | |
| *** haomaiwang has joined #openstack-trove | 13:36 | |
| *** jwiles_ has joined #openstack-trove | 13:37 | |
| *** haomaiwang has quit IRC | 13:40 | |
| *** haomaiwa_ has joined #openstack-trove | 13:40 | |
| *** pmackinn has joined #openstack-trove | 13:44 | |
| *** tomblank has joined #openstack-trove | 13:53 | |
| *** khushbu_ has quit IRC | 13:57 | |
| *** sgotliv_ has quit IRC | 14:00 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwa_ has joined #openstack-trove | 14:01 | |
| *** _amrith_ is now known as amrith | 14:05 | |
| *** jwiles_ has quit IRC | 14:13 | |
| *** jengeb has joined #openstack-trove | 14:29 | |
| *** khushbu has joined #openstack-trove | 14:32 | |
| *** khushbu has quit IRC | 14:37 | |
| dougshelley66 | cp16net SlickNik on that publish_exists_event change - I believe i figured out how to do it such that we don't need the nova_proxy_admin_tenant_name | 14:49 |
| *** PeterSchwaller has joined #openstack-trove | 14:53 | |
| *** PeterS has quit IRC | 14:55 | |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-trove | 15:01 | |
| *** tomblank has quit IRC | 15:03 | |
| *** tomblank has joined #openstack-trove | 15:05 | |
| *** akanksha_ has joined #openstack-trove | 15:09 | |
| *** csoukup has joined #openstack-trove | 15:10 | |
| *** haomai___ has joined #openstack-trove | 15:16 | |
| openstackgerrit | Doug Shelley proposed openstack/trove: Fix publish_exists_event authentication exception https://review.openstack.org/228696 | 15:16 |
| *** haomaiwang has quit IRC | 15:18 | |
| *** georgelorch has joined #openstack-trove | 15:19 | |
| *** sabeen1 has joined #openstack-trove | 15:20 | |
| *** sabeen1 has quit IRC | 15:27 | |
| *** sabeen has joined #openstack-trove | 15:28 | |
| *** haomai___ has quit IRC | 15:35 | |
| *** tomblank has quit IRC | 15:55 | |
| *** tomblank has joined #openstack-trove | 15:57 | |
| *** jengeb has quit IRC | 15:59 | |
| *** jengeb has joined #openstack-trove | 16:01 | |
| *** ashleighfarnham has joined #openstack-trove | 16:24 | |
| *** tosky has quit IRC | 16:27 | |
| *** sabeen has quit IRC | 16:27 | |
| *** thanu has joined #openstack-trove | 16:27 | |
| *** ViswaV has joined #openstack-trove | 16:29 | |
| *** ViswaV has quit IRC | 16:33 | |
| *** ViswaV has joined #openstack-trove | 16:33 | |
| *** imandhan has joined #openstack-trove | 16:42 | |
| *** sgotliv_ has joined #openstack-trove | 17:29 | |
| *** sgotliv_ has quit IRC | 17:30 | |
| *** sgotliv__ has joined #openstack-trove | 17:30 | |
| *** coolsvap has quit IRC | 17:33 | |
| *** coolsvap has joined #openstack-trove | 17:37 | |
| *** ViswaV has quit IRC | 17:38 | |
| *** ViswaV has joined #openstack-trove | 17:39 | |
| *** exploreshaifali has joined #openstack-trove | 17:40 | |
| *** exploreshaifali has quit IRC | 17:41 | |
| *** ViswaV_ has joined #openstack-trove | 17:41 | |
| *** ViswaV has quit IRC | 17:43 | |
| *** sgotliv__ has quit IRC | 17:51 | |
| *** esmute has joined #openstack-trove | 17:59 | |
| *** tomblank has quit IRC | 18:01 | |
| *** tomblank has joined #openstack-trove | 18:03 | |
| *** sabeen1 has joined #openstack-trove | 18:11 | |
| *** sabeen1 has quit IRC | 18:15 | |
| *** akanksha_ has quit IRC | 18:28 | |
| *** jengeb has quit IRC | 18:36 | |
| *** ViswaV_ has quit IRC | 18:56 | |
| *** imandhan has quit IRC | 19:03 | |
| *** imandhan has joined #openstack-trove | 19:04 | |
| *** imandhan has quit IRC | 19:08 | |
| *** davideagnello has quit IRC | 19:09 | |
| dougshelley66 | cp16net SlickNik yt? | 19:09 |
| cp16net | hello | 19:10 |
| dougshelley66 | i took another crack at that admin client. Current fix is much simpler | 19:14 |
| dougshelley66 | i don't think we need tenant_name...turns out that if you pass context.tenant (which is a UUID) into tenant_id on the novaclient api, it works | 19:15 |
| dougshelley66 | I also added a unit test | 19:15 |
| cp16net | awesome dougshelley66 i'll take a loook | 19:15 |
| dougshelley66 | thx | 19:16 |
| vkmc | ~o~ | 19:16 |
| *** davideagnello has joined #openstack-trove | 19:20 | |
| cp16net | i think that looks good i'll try it out | 19:37 |
| dougshelley66 | cp16net great thanks! | 19:43 |
| dougshelley66 | no there are just 2 other "related" bugs to fix | 19:43 |
| dougshelley66 | https://bugs.launchpad.net/trove/+bug/1402055 | 19:43 |
| openstack | Launchpad bug 1402055 in puppet-trove "Trove-taskmanager fails to start if urls obtained from Keystone" [Undecided,Triaged] | 19:43 |
| dougshelley66 | https://bugs.launchpad.net/trove/+bug/1493576 | 19:44 |
| openstack | Launchpad bug 1493576 in Trove "Incorrect usage of python-novaclient" [Undecided,New] | 19:44 |
| dougshelley66 | (that should have been: "NOW there are just 2 other related bugs to fix" | 19:45 |
| *** clduser_ has joined #openstack-trove | 19:46 | |
| *** clduser has quit IRC | 19:46 | |
| *** csoukup has quit IRC | 20:00 | |
| *** amrith is now known as _amrith_ | 20:09 | |
| *** sabeen has joined #openstack-trove | 20:34 | |
| *** ViswaV has joined #openstack-trove | 20:42 | |
| *** tomblank has quit IRC | 20:44 | |
| *** ViswaV has quit IRC | 20:46 | |
| *** ViswaV has joined #openstack-trove | 20:50 | |
| *** robcresswell has left #openstack-trove | 20:52 | |
| *** jengeb has joined #openstack-trove | 21:04 | |
| *** tomblank has joined #openstack-trove | 21:04 | |
| *** arist has quit IRC | 21:07 | |
| *** arist has joined #openstack-trove | 21:07 | |
| *** tomblank has quit IRC | 21:08 | |
| *** ViswaV has quit IRC | 21:34 | |
| *** devlaps has joined #openstack-trove | 21:52 | |
| *** ViswaV has joined #openstack-trove | 22:24 | |
| *** ViswaV_ has joined #openstack-trove | 22:25 | |
| *** ViswaV has quit IRC | 22:29 | |
| *** sabeen has quit IRC | 22:52 | |
| *** thanu has quit IRC | 23:10 | |
| *** thanu has joined #openstack-trove | 23:14 | |
| *** ashleighfarnham has quit IRC | 23:15 | |
| *** ashleighfarnham has joined #openstack-trove | 23:15 | |
| *** ashleighfarnham has quit IRC | 23:19 | |
| *** thanu has quit IRC | 23:22 | |
| *** ViswaV_ has quit IRC | 23:24 | |
| *** jengeb has quit IRC | 23:25 | |
| *** imandhan has joined #openstack-trove | 23:26 | |
| *** kanzaros_ has joined #openstack-trove | 23:32 | |
| *** kanzaros has quit IRC | 23:32 | |
| *** tristanC has quit IRC | 23:32 | |
| *** kanzaros_ is now known as kanzaros | 23:32 | |
| *** tristanC has joined #openstack-trove | 23:33 | |
| *** albertom has quit IRC | 23:33 | |
| *** albertom has joined #openstack-trove | 23:33 | |
| *** esmute has quit IRC | 23:53 | |
| openstackgerrit | Merged openstack/trove: Use IPOpt to validate IPAddress https://review.openstack.org/228357 | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!