Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-08-09
18:58:52 mordred cdent: I do not believe there is a document that explains service catalog intent at a high level
18:59:02 mordred cdent: but I do believe having such a document would be helpful
19:00:26 cdent “service catalog intent” is the right phrase
19:01:21 mordred yah. I mean, it has a very specific purpose in openstack, but is also completely posible for a deployer to stick other things in it too.sticking other things in it wn't neccessarily mean any tools will know what to do with those things
19:01:32 mordred cdent: also, I'd suggest that we write https://review.openstack.org/#/c/491611/ as more of a sliding scale of recommendations to accomodate things that provide capabilities discovery ... but maybe I should, you know, leave a review comment
19:02:33 cdent a) maybe, b) yes, c) I really wanted to keep this thing as simple as possible and while we don’t currently have a standard for capabilities the “safe” strategy is NO
19:11:13 mordred cdent: totally - and I agree with that broadly - I mostly worry that if we published an API-WG recommendation that is wildly contrary to widespread practice without including accomodation in some manner for how people are doing things today that we run the risk of being disregarded which would be bad
19:11:40 mordred aw. cdent missed my response
19:59:50 openstackgerrit Merged openstack/keystoneauth master: Parameter to tune mutual authentication in kerberos https://review.openstack.org/455330
21:22:48 mordred efried: I was working a patch a few seconds ago using the new ksa stuff - and the Adapter _would_not_ actually consume the version arg I was passing it and I was starting to freak out that we'd released a completely broken thing ...
21:23:07 mordred efried: turns out I was overwriting the object due to copy-pasta acouple of lines below. WHOOPS
21:23:28 efried mordred Phew.
21:23:34 efried Wait.
21:23:37 mordred right?
21:23:59 efried Not phew, you're saying you overwrote the object in your consuming patch, or you found a real bug in ksa?
21:24:05 mordred no - in my consuming patch
21:24:13 mordred totally just dumb PEBKAC on my part
21:24:13 efried phew, I say again.
21:24:20 mordred but WOW did I have print statements EVERYWHERE
21:24:26 efried hahaha
21:24:31 mordred *why is this not using the right url????*
21:24:46 efried I'm experiencing an interesting oslo_config possible-bug.
21:25:02 efried As I'm deprecating old opts with new adapter ones...
21:25:07 mordred ooh. that sounds like fun
21:25:15 efried The option comes through to the real code correctly, but in UT when we do CONF.set_override, it doesn't.
21:25:39 mordred I love it when there's a bug in unittests that is just a bug in unittests
21:25:59 efried That's not very clear. Here: (dhellmann this might interest you)
21:26:50 efried I've got an option 'old'. I deprecate it in favor of 'new' by registering a DeprecatedOpt. I set up my conf to contain old = 'foo'. In my actual code, I can see CONF.old == 'foo' and CONF.new == 'foo', which is how deprecation aliases are s'posed to work.
21:26:52 efried BUT
21:27:17 efried When I say CONF.set_override(old='bar')
21:27:31 efried ...now CONF.old == 'bar' but CONF.new still == 'foo'.
21:47:45 dhellmann efried : that sounds like a bug. The original version of the deprecation stuff didn't support using the old name in code, because you were supposed to update all of those references at the same time. When mtreinish added that feature for tempest, we missed some access paths.
21:48:45 dhellmann efried : my own position is that I would expect all references to the old option name to be updated when the deprecated option is registered, so that the only place one should see that old name is in an old configuration file
21:59:04 efried dhellmann That makes sense to me. Ergo I wouldn't expect someone to use CONF.set_override('old', ...) at all... except in unit tests to prove that stuff still works if the deprecated conf option is used ;-)
21:59:19 efried (in the conf)
22:41:59 efried dhellmann (mordred) https://bugs.launchpad.net/oslo.config/+bug/1709728
22:42:00 openstack Launchpad bug 1709728 in oslo.config "CONF.set_override doesn't alias deprecated opts" [Undecided,New]
23:03:30 efried mordred Does endpoint_filter={'interface': ['internal', 'public']} work as an arg to Session.request??
23:03:48 efried Cause doesn't that endpoint_filter wind up in requests.Session?
23:11:52 mordred efried: no - requests doesn't know anything about endpoint_filter
23:12:11 efried mordred I must not have traced it through right, then.
23:12:16 mordred efried: session.request(endpoint_filter={'interface': ['internal', 'public']}) SHOULD work and if it doesn't somethig is broke
23:12:43 efried mordred We'll find out in https://review.openstack.org/492247
23:14:46 mordred efried: https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L599
23:16:04 efried mordred Beaut. But https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L703 will still send the endpoint filter through, right? It'll just get ignored?
#openstack-sdks - 2017-08-10
00:52:10 mordred efried: yah
11:45:37 openstackgerrit Colleen Murphy proposed openstack/keystoneauth master: Allow discovery URLs to have trailing slashes https://review.openstack.org/492484
13:19:43 openstackgerrit Colleen Murphy proposed openstack/keystoneauth master: Allow discovery URLs to have trailing slashes https://review.openstack.org/492484
14:28:44 openstackgerrit XueFeng Liu proposed openstack/service-types-authority master: It's better to use 'cluster' as senlin's service_type https://review.openstack.org/484011
15:27:54 openstackgerrit Reedip proposed openstack/python-openstackclient master: Allow PD as Subnetpool during Subnet creations https://review.openstack.org/491664
17:19:16 cdent elmiko, edleafe https://etherpad.openstack.org/p/api-wg-newsletter
17:19:59 elmiko ack
17:20:58 edleafe nice touch on the 308
17:21:23 elmiko yeah, love the 308
17:21:26 edleafe wait - do we lose the hyphen?
17:21:42 elmiko uh oh, THIS WASN'T MENTIONED IN THE VOTE!
17:21:42 edleafe IOW, API-WG -> API SIG or API-SIG?
17:23:16 elmiko fwiw, i'm good with that copy cdent
17:23:57 cdent hmmm, yes, to hyphen or not to hyphen
17:24:13 cdent let’s hyphen?
17:24:20 elmiko i had assumed hyphen, but i'm not sure what the idea from the tc was
17:24:27 edleafe I like hyphens
17:24:53 edleafe "You can take my hyphen when you pry it from my cold dead hands!"
17:26:01 edleafe Like all things, the particulars aren't important, as long as we're consistent
17:26:58 cdent hobgoblins
17:28:02 edleafe Seeing as you have "Location: /api-sig/news", let's keep the hyphen
17:28:27 elmiko +1
17:28:27 cdent yeah, I already changed it and sent it
17:29:07 elmiko \o/
17:30:40 edleafe kewl
18:20:20 openstackgerrit Brian Curtin proposed openstack/python-openstacksdk master: Introduce Member for Octavia (load balancing) https://review.openstack.org/451488
19:31:09 openstackgerrit OpenStack Release Bot proposed openstack/python-openstackclient master: Update reno for stable/pike https://review.openstack.org/492660
21:15:09 openstackgerrit Merged openstack/python-openstacksdk master: Update load_balancer for v2 API https://review.openstack.org/454410
21:30:01 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Allow PD as Subnetpool during Subnet creations https://review.openstack.org/491664
21:37:29 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Update release/stable branch table in docs https://review.openstack.org/492699
#openstack-sdks - 2017-08-11
02:50:29 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
07:24:40 openstackgerrit Rui Chen proposed openstack/python-openstackclient master: [Compute]Make column content readable for both human and machine https://review.openstack.org/445794
07:59:08 openstackgerrit Zhaokun Fu proposed openstack/python-openstackclient master: add --password option for openstack server create Closes-Bug:1708570 https://review.openstack.org/490718
09:30:30 openstackgerrit yfzhao proposed openstack/os-client-config master: Fix wrong links https://review.openstack.org/492924
09:42:57 openstackgerrit sudhir agarwal proposed openstack/python-openstackclient master: Replace six.iteritems() with .items() https://review.openstack.org/492929
10:26:37 openstackgerrit Merged openstack/keystoneauth master: Enable some off-by-default checks https://review.openstack.org/477775
12:05:01 _heena_py__ Hi
12:05:25 _heena_py__ I dont know if this is the right IRC but I needed to know a way to fetch cinder volume attachments in python
13:12:55 openstackgerrit Sean Handley proposed openstack/os-client-config master: DataCentred supports Keystone V3 and Glance V2. https://review.openstack.org/493002
14:13:40 openstackgerrit Merged openstack/python-openstackclient master: Update reno for stable/pike https://review.openstack.org/492660
14:36:24 openstackgerrit Merged openstack/python-openstackclient master: Update release/stable branch table in docs https://review.openstack.org/492699
15:43:56 openstackgerrit Merged openstack/python-openstackclient master: Add .htaccess for docs migration https://review.openstack.org/491631
17:26:56 openstackgerrit Merged openstack/osc-lib feature/osc4: Refactor initialize_app() to allow overriding https://review.openstack.org/491846
18:18:14 openstackgerrit Merged openstack/cliff master: Make openstackdocstheme an optional doc dependency https://review.openstack.org/491378
18:31:15 openstackgerrit Merged openstack/os-client-config master: Update the documentation link for doc migration https://review.openstack.org/491374
22:22:17 fried_rice mordred How do you feel about returning a list from ServiceTypes.get_service_data_for_project?
22:50:55 openstackgerrit Eric Fried proposed openstack/os-service-types master: Non-unique project names: by_project as list https://review.openstack.org/493254
22:50:59 fried_rice mordred ^
22:54:49 openstackgerrit Eric Fried proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140
22:56:08 fried_rice Made ^ depend on -^
#openstack-sdks - 2017-08-12
11:41:12 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/493298
11:50:14 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/491294
13:14:53 mordred fried_rice: what if instead we don't do project: nova for placement? I think by_project's value is that it can answer the question "what is the service-type that people mean when they say {name}?" - placement is NEVER the answer to that question for name==nova - it just happens to live in the nova repo right now
18:31:47 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Support secondary services https://review.openstack.org/493325
18:34:26 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140

Earlier   Later