| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-08 | |||
| 20:57:33 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Adapter.get_conf_options(deprecated_opts) https://review.openstack.org/490895 | |
| 21:32:07 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Protect against missing interface attribute https://review.openstack.org/488568 | |
| 21:48:35 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Protect against missing interface attribute https://review.openstack.org/488568 | |
| 21:48:36 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Adapter.get_conf_options(deprecated_opts) https://review.openstack.org/490895 | |
| 22:43:30 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: WIP: Return the endpoint_override from EndpointData https://review.openstack.org/491947 | |
| #openstack-sdks - 2017-08-09 | |||
| 01:22:09 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/491294 | |
| 04:21:13 | openstackgerrit | Artur Basiak proposed openstack/service-types-authority master: Add monasca-events-api project https://review.openstack.org/490750 | |
| 04:50:52 | openstackgerrit | Merged openstack/python-openstackclient master: Add new commands for karbor osc plugin https://review.openstack.org/491416 | |
| 14:09:36 | openstackgerrit | liyi proposed openstack/python-openstacksdk master: Add profile type ops cli https://review.openstack.org/492048 | |
| 14:22:42 | openstackgerrit | Eric Fried proposed openstack/service-types-authority master: Change default swift name to object-storage https://review.openstack.org/462138 | |
| 14:25:24 | openstackgerrit | Merged openstack/service-types-authority master: Add and rename monitoring services https://review.openstack.org/487484 | |
| 14:25:48 | openstackgerrit | Merged openstack/service-types-authority master: Add monasca-events-api project https://review.openstack.org/490750 | |
| 14:38:02 | openstackgerrit | Eric Fried proposed openstack/service-types-authority master: Add placement service https://review.openstack.org/462140 | |
| 15:40:06 | cmurphy | mordred: efried I had a customer report https://bugs.launchpad.net/keystoneauth/+bug/1709658 to me, it seems like there was some surprising behavior in ksa 2.x that was fixed in a backwards incompatible way | |
| 15:40:07 | openstack | Launchpad bug 1709658 in keystoneauth ""Could not find requested endpoint in Service Catalog" when requesting unavailable identity endpoint" [Undecided,New] | |
| 15:41:06 | cmurphy | I will try to look into it later tonight or tomorrow morning if it doesn't nerdsnipe you sooner | |
| 15:41:08 | efried | cmurphy Looking. But given what I was playing around with yesterday, I'm probably not going to be surprised... | |
| 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 | 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 | |