Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-13
21:08:19 sdague https://github.com/openstack/keystoneauth/blob/9e010b995ba2f19fcc63b14cf5048cedd5ff1e36/keystoneauth1/session.py#L511-L513
21:09:39 mriedem i'm not following what that has to do with 'internal' as a default
21:10:06 sdague mriedem: I'm trying to figure out if you send down None what it is going to do
21:10:29 sdague I guess it gets converted to public here - https://github.com/openstack/keystoneauth/blob/9e010b995ba2f19fcc63b14cf5048cedd5ff1e36/keystoneauth1/identity/base.py#L209-L210
21:10:39 mriedem yeah that's what i pointed out above
21:11:08 sdague yeh, sorry, I was thinking that it jumped into service_catalog.py get_endpoints directly
21:11:20 mriedem well, there are certainly plenty of spaghetti hoops in here
21:11:24 mriedem and plugins and such
21:11:24 sdague given that there are similar named functions in a bunch of modules here
21:11:28 mriedem right
21:11:33 mriedem i could have totally broken everything
21:11:40 mriedem i was hoping devstack would tell me
21:15:12 mriedem cdent: also on that unit test thing, it constructs a new object each time, so i'm not sure why i had to move it into __init__ but i blame python
21:16:47 cdent mriedem: probably the joys of global conf getting your face
21:17:39 cdent but yeah, *shrug*
21:24:16 openstackgerrit STEW TY proposed openstack/nova master: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606
21:30:16 cdent sigh, I'm clearly in the wrong time zone, miss all the cool shit. oh well.
21:37:35 openstackgerrit STEW TY proposed openstack/nova master: Complete verification of servers-action-fixed-ip.inc https://review.openstack.org/338606
21:45:09 knikolla mriedem: hi. would it be possible to backport the fix for this to newton? https://bugs.launchpad.net/nova/+bug/1635008
21:45:09 openstack Launchpad bug 1635008 in OpenStack Compute (nova) "rbd credentials in nova.conf overwrite connection_info from cinder" [Low,Fix released] - Assigned to Matt Riedemann (mriedem)
21:45:45 mriedem knikolla: no because it's a behavior change
21:46:23 knikolla mriedem: i see.
21:46:28 knikolla thanks
21:46:32 mriedem sorry, np
21:51:38 jamielennox mriedem, sdague: figure out your ksa issue?
21:52:12 sdague jamielennox: yeh, it was mostly just making sure we didn't encode too much keystone schema into nova.
21:53:05 jamielennox great
21:53:17 jamielennox although out of interest what is keystone schema in this cnotext?
21:53:37 jamielennox from highlights it seems to be mostly interface related
21:53:54 mriedem jamielennox: yeah,
21:54:05 sdague jamielennox: yeh, the interface list
21:54:09 mriedem there was a change to encode the admin/internal/public endpoint interfaces in nova.conf
21:54:16 mriedem ksa also allows 'auth',
21:54:30 mriedem and keystone v2 doesn't restrict the interface it looks like, but the v3 api does
21:54:51 sdague especially because I know when we were talking about service catalog improvements that list was very much an oddity for a lot of ops
21:55:17 sdague so I didn't want to force encode that beyond keystone, which would prevent it evolving in the future
21:55:55 jamielennox i *think* keystone v2 restricts that interface because in v2 'publicURL', 'adminURL' etc are actual fields where in v3 it's just like {'url': ..., 'interface': ...}
21:56:03 jamielennox but i would want to double check that
21:56:15 jamielennox but there's no validation of that in ksa
21:56:53 jamielennox the way cinder and others do this is with a : seperated string, which isn't great but more flexibility than most provide
21:57:41 jamielennox mriedem: i'm not sure what you mean with ksa allows 'auth' - auth is a plugin so something that will be loaded at a very different time
21:58:36 mriedem jamielennox: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/identity/base.py#L198
21:59:11 mriedem oh i thought that meant it could be "auth"
21:59:15 jamielennox oh, right
21:59:19 mriedem which meant lookup the endpoint via auth_url config
21:59:33 mriedem it's not really doc'ed from what i can tell
21:59:43 jamielennox that's mostly a special case when you want to look back to the auth_url provided to the plugin and used for some discovery
21:59:57 jamielennox from nova and services perspective you would always expect to be actually looking at the catalog
22:00:40 jamielennox ie, i can't see a reason nova config would want to set that
22:00:59 felipemonteiro_ Is anyone familiar with the consoles API? When I create a console on a running server, then list all the consoles running on that server, I get an empty set.
22:08:14 openstackgerrit Dan Smith proposed openstack/nova master: Make server_external_events cells-aware https://review.openstack.org/433298
22:08:23 dansmith mriedem: melwitt: with ^ I can _create_ an instance in another cell
22:10:19 mriedem dansmith: should we start a bp for tracking just random api compat things in pike?
22:10:30 mriedem multicell support or whatever
22:10:40 dansmith mriedem: well, I think we'll need a blueprint for all the "make XXX cells aware" stuff yeah
22:10:45 dansmith we know we have a bunch to do
22:10:57 dansmith it'll be a slow background burning type situation
22:10:58 mriedem yeah so let's group that into a bp
22:10:59 mriedem sure
22:11:05 mriedem you want to create that?
22:11:14 dansmith like when you pee and you know something's wrong but you avoid going to the doc for a while
22:11:21 dansmith sure
22:11:37 mriedem cranberry juice will help with that
22:11:47 dansmith won't help with cells unfortunately
22:11:51 dansmith beer might though
22:11:58 mriedem we don't have a cells-sized condom?
22:12:19 mriedem you don't have to answer that
22:13:10 dansmith https://blueprints.launchpad.net/nova/+spec/cells-aware-api
22:13:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Cleanup some issues with CONF.placement.os_interface https://review.openstack.org/433257
22:23:34 openstackgerrit Pushkar Umaranikar proposed openstack/nova master: Placement api: set custom json_error_formatter in root https://review.openstack.org/433301
22:39:37 openstackgerrit Anusha Unnam proposed openstack/nova master: Remove mox from nova/tests/unit/virt/vmwareapi/test_configdrive.py https://review.openstack.org/433315
22:51:49 mriedem https://etherpad.openstack.org/p/nova-ptg-pike is relatively organized now
23:21:31 openstackgerrit Anusha Unnam proposed openstack/nova master: Remove mox from nova/tests/unit/virt/vmwareapi/test_configdrive.py https://review.openstack.org/433315
23:55:24 tonyb mriedem_afk: Should I revert https://review.openstack.org/#/c/430498/ ? as it has no trusted nova +1 ?
#openstack-nova - 2017-02-14
00:13:46 openstackgerrit Pushkar Umaranikar proposed openstack/nova master: Expand feature matrix with server admin actions https://review.openstack.org/362408
00:42:45 openstackgerrit Dan Smith proposed openstack/nova master: Make server_external_events cells-aware https://review.openstack.org/433298
00:42:46 openstackgerrit Dan Smith proposed openstack/nova master: Make start and stop admin actions cells-aware https://review.openstack.org/433362
00:58:35 gcb bauzas: I didn't do that work, just suggest we remove mox/mox3 then we can retire mox3 from oslo side
02:26:04 openstackgerrit huangtianhua proposed openstack/nova master: Should raise correct error instead of class exist in Placement API https://review.openstack.org/428612
02:33:07 openstackgerrit Eric Xie proposed openstack/nova master: Fix improper prompt when update RC with existed one's name. https://review.openstack.org/431392
02:54:22 openstackgerrit huangtianhua proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568
03:02:14 openstackgerrit Wang Qing wu proposed openstack/nova master: PowerVM Driver: localdisk https://review.openstack.org/409401
03:36:49 openstackgerrit LisaLi proposed openstack/nova master: Attach/detach encryptors during swap_volume https://review.openstack.org/252809
04:08:22 openstackgerrit YAMAMOTO Takashi proposed openstack/os-vif master: Correct object path in comments https://review.openstack.org/433433
04:15:43 openstackgerrit YAMAMOTO Takashi proposed openstack/os-vif master: Correct object path in comments https://review.openstack.org/433433
04:29:26 openstackgerrit YAMAMOTO Takashi proposed openstack/nova master: os-vif-util: Support VIF_TYPE_MIDONET https://review.openstack.org/433436
04:51:45 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix doc generation warnings https://review.openstack.org/425549
04:52:02 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882
04:52:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608
06:04:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix misuse of assertTrue https://review.openstack.org/433454
06:20:05 openstackgerrit Wang Qing wu proposed openstack/nova master: PowerVM Driver: localdisk https://review.openstack.org/409401
06:56:55 Dinesh_Bhor Hi all, is someone getting this error: "(ValueError): dictionary update sequence element #0 has length 1; 2 is required " from novaclient while booting an instance with metadata?
07:01:54 openstackgerrit huangtianhua proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568
07:33:33 openstackgerrit Xia Yan proposed openstack/nova master: PowerVM Driver: console https://review.openstack.org/409402
07:37:18 openstackgerrit jichenjc proposed openstack/nova master: Check instance existing before check in mapping https://review.openstack.org/433470
07:40:04 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955
07:40:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964
07:54:55 openstackgerrit jichenjc proposed openstack/nova master: Prevent delete cell0 in nova-manage command https://review.openstack.org/433476
08:32:51 openstackgerrit ChangBo Guo(gcb) proposed openstack/nova master: Remove usage of config option verbose https://review.openstack.org/432271
08:38:26 kashyap mriedem_afk: Hey there, sorry, I was on a train last evening, so wasn't looking at the computer.

Earlier   Later