Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-02-07
18:12:45 johnsom dtroyer it's an element of the octavia driver that does the load balancing. Could be service vm, container, baremetal, etc.
18:13:50 johnsom dtroyer Yeah, it's a bit interesting. The need is there for admin commands for the plugin drivers.
18:14:11 dtroyer my first (sime-uninformed) reaction is to suggest just using "amphora" as the top-level resource. It is closer to a project name in style than a resource description, but might make snese in this case
18:14:23 dtroyer s/sime/semi/
18:14:31 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Cleanup the BaseProxy docs a bit https://review.openstack.org/539270
18:14:31 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Shift config exceptions to openstack.exceptions https://review.openstack.org/539396
18:14:32 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Generate proxy methods from resource objects https://review.openstack.org/540169
18:14:32 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename BaseProxy to Proxy https://review.openstack.org/539271
18:14:33 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use get_session_client in Connection https://review.openstack.org/541060
18:14:33 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Calculate name in CloudRegion https://review.openstack.org/541059
18:14:47 johnsom dtroyer This is why I was thinking making a namespace under the overall service, "openstack loadbalancer" might be the best idea. Then the plugins can just expand that as needed.
18:14:49 mordred rods_: ^^ rebased that stack on the OSC tips enablement patch
18:14:57 dtroyer johnsom: the big win with that is "amphora" is likely to be unique
18:15:24 dtroyer I really try to not use the word "namespace" as it is subtly different from the way OSC names resources
18:15:25 johnsom dtroyer The downside is only the "octavia" driver has "amphora". F5 doesn't for example.
18:15:35 rods_ mordred cool, I'll give a look at all of them in a few minutes
18:15:50 mordred dtroyer: yah - I think in this case it'sa driver-specific function for admins who have deployed that driver, so having them do 'openstack amphora blah' doesn't seem super bad to me
18:16:06 dtroyer johnsom: so maybe that is why we use it bare? it names the resource directly. Is there a parallel for f5 or other drivers?
18:16:56 johnsom dtroyer Speaking out of line since I don't work for F5, maybe "bigIP"????
18:17:07 johnsom dtroyer or "LTM"
18:18:03 dtroyer the point being that other lb drivers may have a similar concept, if they do, is using amphora for the octavia driver setting a good or bad precedent?
18:18:04 johnsom dtroyer So we would be ok with third party OSC plugins that add top level commands?
18:19:08 dtroyer johnsom: mostly, yes. we do ask for plugins to play nice and register resource names with OSC, but can not enforce that
18:19:59 dtroyer and when we get into areas where there are multiple implementations of a thing, we either need to hide that from the user, or if the user is expected to know and care about the difference, make it logical for them
18:20:50 johnsom dtroyer Ok, so "openstack octavia amphora failover" = "openstack <driver> <element><command>"
18:20:52 dtroyer is the use of LB drivers "one-per-cloud"? or could a deployment have multiple? it does sound like the user of this CLI will need to kow in any case
18:21:13 johnsom dtroyer "openstack amphora failover" = "openstack <element> <action>"
18:21:17 dtroyer think this way: openstack <resource-name> <action>
18:21:32 johnsom dtroyer most deployments use two or more load balancing drivers
18:21:36 dtroyer your suggestion is resource-name == "octavia amphora", mine is == "amphora"
18:23:37 dtroyer johnsom: ok, for the other drivers, how will users think about driver-specific stuff? I'm looking for the name of the driver-specific resources, they might begin with something parallel to amphora for octavia. this is a space I don't have much experience with
18:23:49 johnsom dtroyer Ok, I think we have good guidance to bring to our meeting today. The meeting is at 20:00 UTC today in #openstack-lbaas if you want to join the conversation.
18:24:37 johnsom dtroyer For non-admin commands, they have little knowledge of the driver. They simply do "openstack loadbalancer create --provider <driver name>"
18:25:38 dtroyer johnsom: I'll be on the phone most of the afternoon… ok, so using a name that identifies the driver as part of the admin driver-specific resource nmae is appropriate I think
18:28:23 dtroyer johnsom: I left a couple of comments in that etherpad
18:28:30 johnsom dtroyer Thank you
19:19:51 openstackgerrit Monty Taylor proposed openstack/keystoneauth master: WIP Handle unversioned endpoints with no discovery document https://review.openstack.org/541844
19:23:36 mordred dtroyer: https://review.openstack.org/#/c/540554/ <-- that's green now! dhellmann if you're around too
19:24:05 mordred dtroyer, dhellmann: this is the fix for the gate job issue that let us release sdk with changes that broke osc
19:25:01 mordred dtroyer, dhellmann: and https://review.openstack.org/#/c/540556 is the corresponding SDK change that's going through recheck right now
19:25:06 dhellmann dtroyer beat me to it
19:25:10 dtroyer mordred: +W and thank you!
19:25:50 mordred woot!
19:30:56 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Make Profile fallback go bye-bye https://review.openstack.org/540541
19:31:25 mordred dtroyer: there's the followup for post-queens to remove the profile fallback code
20:01:49 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Calculate name in CloudRegion https://review.openstack.org/541059
20:05:47 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use get_session_client in Connection https://review.openstack.org/541060
20:25:48 openstackgerrit Merged openstack-infra/shade master: Use openstacksdk for most transitive depends https://review.openstack.org/541796
20:26:17 openstackgerrit Monty Taylor proposed openstack/os-service-types master: Update from service-types-authority https://review.openstack.org/541864
20:51:02 EmilienM hi
20:54:19 mordred EmilienM: please don't tell me that the latest openstacksdk release has broken tripleo in new and creative ways
20:54:38 mordred EmilienM: I think the release team might kill me if I have to release another point release during freeze :)
20:54:58 mordred EmilienM: (I'm kidding, obviously tell us if something broke)
20:55:55 mordred dtroyer: if you have a sec, wanna pull the trigger on sdk gating on osc tips? https://review.openstack.org/#/c/540556/
20:56:37 stevebaker mordred: we're just talking optimal default behaviour when someone ctrl-c's a command
20:56:56 mordred oh good. that sounds like an osc question :)
20:57:39 dtroyer mordred: +W
20:57:56 mordred dtroyer: thanks! I'll be much happier with that landed
20:58:27 openstackgerrit Merged openstack/python-openstackclient master: Make osc-functional-devstack-tips actually use tips https://review.openstack.org/540554
20:59:15 mordred \o/
21:01:33 mordred dtroyer: tell me about it
21:17:16 openstackgerrit Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/534911
21:27:14 EmilienM mordred: which makes me think I always complain!
21:28:16 mordred EmilienM: :) nah - I just had a fun week last week of discovering sdk release breaks due to missing tests
21:28:46 EmilienM but if you ask, I can certainly find something that broke us recently :P
21:31:43 mordred EmilienM: hehe. I'm sure :)
21:34:53 openstackgerrit Merged openstack/python-openstacksdk master: fix misspelling of 'volume' https://review.openstack.org/541095
22:02:43 rods_ mordred probably not important but you have small typo here https://review.openstack.org/#/c/539271/6
22:25:34 openstackgerrit Merged openstack/python-openstacksdk master: Gate on osc-functional-tips https://review.openstack.org/540556
#openstack-sdks - 2018-02-08
00:07:03 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/541943
00:36:46 openstackgerrit zhangzhaoshan proposed openstack/keystoneauth master: Fix a spelling error https://review.openstack.org/541949
00:38:34 openstackgerrit Merged openstack/python-openstackclient master: Zuul: Remove project name https://review.openstack.org/541004
06:59:12 openstackgerrit Merged openstack/osc-lib master: Zuul: Remove project name https://review.openstack.org/540080
08:25:20 openstackgerrit Thomas Goirand proposed openstack/python-openstackclient master: Fix crashing "console log show" https://review.openstack.org/541609
15:48:23 cdent elmiko, edleafe can one of your run the api-sig meeting today? it's looking like tc office hours is going to run over
15:48:51 elmiko cdent: ack, i can do it
15:49:43 cdent thanks
15:50:16 edleafe I can do it, too, but will gracefully follow elmiko's lead
15:51:57 elmiko i'd say we can flip for it, but i'm not sure my knees can handle that ;)
15:52:27 edleafe heh
15:54:13 cdent that's a game I'd like to see
15:55:36 elmiko lol
16:17:00 openstackgerrit Merged openstack/python-openstacksdk master: Add shade jobs to openstacksdk zuul config https://review.openstack.org/541802
17:14:25 edleafe cdent, elmiko, dtantsur: have at it. https://etherpad.openstack.org/p/api-sig-newsletter
17:39:35 cdent edleafe: sorry for delay, workmen in house briefly. LGTM
17:40:09 edleafe wow, not even a typo or three?
17:40:37 cdent well, you know me and typos. they are invisible to me. I'd wait for a second opinion
17:41:32 elmiko edleafe: lgtm
17:41:48 cdent speaking of typos, I may need you to help me evaluate a blog post draft edleafe
17:41:52 cdent let me run it through my own proofread first though
17:44:37 edleafe cdent: ack
18:11:47 openstackgerrit Kam Nasim proposed openstack/keystoneauth master: Ignore unexpected arguments to token auth plugin https://review.openstack.org/542355
19:49:24 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Disallow setting default on internal network https://review.openstack.org/539303
20:00:15 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Add option '--mtu' to network create/set https://review.openstack.org/540530
#openstack-sdks - 2018-02-09
01:55:09 openstackgerrit Bailey Johnson proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537
06:49:48 openstackgerrit brandon zhao proposed openstack/python-openstacksdk master: modify typos of word password https://review.openstack.org/542635
08:53:51 openstackgerrit Huang Cheng proposed openstack/python-openstackclient master: Fix subnet host_routes error https://review.openstack.org/540568
14:18:21 openstackgerrit Merged openstack/cliff master: sphinxext: Warn if namespace or command pattern invalid https://review.openstack.org/510913
21:23:59 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Add option '--mtu' to network create/set https://review.openstack.org/540530
21:57:23 openstackgerrit Daniel Speichert proposed openstack/python-openstacksdk master: Strip API version from subsequent paginated links https://review.openstack.org/543035

Earlier   Later