| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-09 | |||
| 15:42:21 | efried | cmurphy Oh, okay, this has nothing to do with what I saw yesterday. But yeah, this looks like RBB to me. | |
| 16:40:12 | mordred | cmurphy: yay nerdsniping | |
| 16:42:41 | mordred | cmurphy: oh - fun. so - I haven't started digging in to the code yet, but there's an additional weirdness in there, which is OS_IDENTITY_API_VERSION and how that's getting set or not via python-openstackclient | |
| 16:45:19 | mordred | cmurphy: I asked for some additional informatoin in the bug | |
| 18:14:16 | cdent | mordred: you might know this, but anyone else feel free to chime in: Is there a canonical doc somewhere on what the service catalog is and is for? | |
| 18:15:46 | cdent | there’s plenty of stuff near to that, but what I’ve been able to find so far glances off being what I’m looking for | |
| 18:54:37 | mordred | cdent: maybe? can you say that same thing maybe in different words though, I may misunderstand the question | |
| 18:54:57 | cdent | mordred: I can link you to why it matters, one moment | |
| 18:55:17 | cdent | mordred: see the discussion with edleafe here: https://review.openstack.org/#/c/491611/ | |
| 18:57:02 | mordred | cdent: hrm. this is deeper and more philosophical | |
| 18:57:09 | mordred | cdent: (and important, obvs) | |
| 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 | efried | phew, I say again. | |
| 21:24:13 | mordred | totally just dumb PEBKAC on my part | |
| 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 | edleafe | IOW, API-WG -> API SIG or API-SIG? | |
| 17:21:42 | elmiko | uh oh, THIS WASN'T MENTIONED IN THE VOTE! | |
| 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 | cdent | yeah, I already changed it and sent it | |
| 17:28:27 | elmiko | +1 | |
| 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 | |