Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-11-18
10:15:31 stephenfin the first one is the aggregate, of course :)
10:15:38 gtema can you add multiple FIPs in same command?
10:16:12 stephenfin no, because the FIPs have an optional attribute
10:16:20 stephenfin (fixed IP address)
10:16:38 gtema aggregate add host also doesn'T allow multiple hosts
10:16:53 stephenfin <stephenfin> And personally I'd like to expand (in a follow-up, of course) some of the ones that only accept a single child resource to accept multiple child resources, e.g. 'aggregate add host'
10:17:03 gtema :D
10:17:19 stephenfin and again, 'object create' does, so we have prior art here
10:17:28 gtema I started SDK change for adding host support and was pulling hears from my head yesterday
10:18:17 stephenfin :-D OSC and openstacksdk are both opinionated libraries. It's fair to expect strong opinions ;)
10:18:33 gtema that's true
10:18:42 stephenfin Also, to my credit, I think the rest of my reviews have been fair and I've caught a good few bugs
10:19:11 gtema let me perhaps say like that: you come from nova, so I expect you more to be responsible for the nova CLIs
10:20:15 gtema to your credit - sure. Great work
10:20:32 stephenfin If the concern is purely with the fact that you can specify multiple images, then I'd be okay with only accepting a single image, i.e. 'aggregate cache image AGGREGATE IMAGE'
10:20:42 stephenfin People can always invoke it multiple times
10:21:10 stephenfin I really do think positional is the way to go though, and the fact that mriedem said the same (and dansmith didn't argue with him) helps my case
10:21:54 gtema yeah yeah, doing that
10:25:53 gtema nice hint with another patch - haven't seen that
10:26:28 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server * -f yaml' output https://review.opendev.org/761205
10:26:28 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'usage * -f yaml' output https://review.opendev.org/761595
10:26:29 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'server group * -f yaml' output https://review.opendev.org/761596
10:26:30 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix 'hypervisor show -f yaml' output https://review.opendev.org/763004
10:26:30 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'server group create --rule' parameter https://review.opendev.org/761597
10:26:30 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: trivial: Use plural for appended parameters https://review.opendev.org/761598
10:26:42 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: Switch compute aggregate functions to SDK https://review.opendev.org/762134
10:26:51 gtema here you go
10:27:00 stephenfin thanks :)
10:27:52 openstackgerrit Ananya proposed openstack/ansible-collections-openstack master: migrating routers_info from AnsibleModule to OpenStackModule https://review.opendev.org/763149
10:29:33 gtema stephenfin, do you think we need to cover reboot/shutdown/start host stuff in SDK?
10:29:48 gtema I think OSC currently doesn't support those
10:30:26 stephenfin The SDK is supposed to be complete, but that API (os-hosts) is deprecated and the things we said to use instead (os-hypervisors, os-services) are present in 2.1 iirc
10:30:32 gtema investing time in deprecated/dropped functionality is hard
10:30:49 stephenfin So I think it's exceptionally low priority, IMO. I wouldn't bother, personally
10:31:05 gtema pretty much same for me
10:31:22 gtema my plan was to get basic host ops covered in SDK to be able to use them from OSC
10:32:26 stephenfin we already have e.g. 'hypervisors list'
10:32:39 stephenfin so I think you're covered there?
10:32:48 stephenfin that'll give you host information
10:33:04 gtema that's true, but if you want to fill 'openstack aggregate add host' with host check you need that
10:33:19 gtema or you mean hypervisors are really giving same info back?
10:33:42 stephenfin the latter; it's the same information exposed by both afaik
10:33:55 gtema hm, interesting
10:35:12 stephenfin To be clear though, that idea was also a nice-to-have UX improvement. Converting more of the other commands to SDK like you've been doing is maybe more helpful?
10:35:27 stephenfin I should start prefixing those comments with 'idea:' or something to distinguish them
10:36:11 gtema perfectly fine with that. I would be perhaps better if I work on SDK switch changes (SDK and OSC) and you improve UX
10:36:24 stephenfin wfm :)
10:36:48 gtema hypervisor vs host are 2 totally different set of APIs (facepalm)
10:37:01 gtema making smooth transition is not easy
10:37:33 gtema especially getting host details is so bad, while getting hypervisor details is ok
10:39:03 gtema you will participate tomorrow in a meeting, right?
10:39:11 stephenfin gtema: We could just deprecate the 'host *' commands
10:39:19 gtema I wanted to agree on the "order" of next switches
10:39:20 stephenfin rather than providing a shim
10:39:54 gtema interesting idea. This would however mean we can't drop novaclient from deps in a long run
10:40:00 stephenfin it's not like we'll break older clouds; those have os-hypervisors and os-services too
10:40:02 gtema we did that with glance
10:40:20 stephenfin we could if deprecate them for removal
10:40:31 stephenfin i.e. we'll drop novaclient in the next major version bump
10:40:47 stephenfin fwiw, we have the same issue with commands like 'server add floating ip'
10:41:18 gtema I was personally thinking to do a major bump with nova switch, so we can take it with us
10:41:40 stephenfin those use deprecated "orchestration" APIs; I've seen patches to replace some of those with client side equivalents but I'm not sure if that's better than asking people to do this stuff manually
10:41:56 stephenfin as for the meeting, that's a 14:00 UTC, yeah? I can be there
10:42:13 gtema so you suggest not to deal with host (SDK/OSC) as of now at all?
10:42:44 stephenfin unless you really want to
10:42:52 stephenfin just deprecate the commands and remove them in a future major version bump
10:43:06 gtema surely not. As I said - my hears were flying around yesterday
10:43:33 stephenfin then let's do that; if someone complains, they can do the SDK work :)
10:43:39 gtema perhaps I will then finish SDK patch as is to get some sort of support
10:44:54 gtema oki, then I will start with hypervisors next
10:47:04 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add support for deprecated nova hosts https://review.opendev.org/763081
11:15:07 openstackgerrit Ananya proposed openstack/ansible-collections-openstack master: Migrating routers_info from AnsibleModule to OpenStackModule https://review.opendev.org/763162
11:28:18 openstackgerrit Ananya proposed openstack/ansible-collections-openstack master: Migrating routers_info from AnsibleModule to OpenStackModule https://review.opendev.org/763162
11:36:51 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'server shelve --offload', 'server shelve --wait' options https://review.opendev.org/762396
11:36:51 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'server unshelve --wait' option https://review.opendev.org/763166
11:53:35 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Add 'server show --topology' option https://review.opendev.org/680928
12:07:06 openstackgerrit Merged openstack/openstacksdk master: Add update_flavor method https://review.opendev.org/762989
12:08:03 openstackgerrit Merged openstack/openstacksdk master: Complete compute aggregate functions https://review.opendev.org/762131
12:08:04 openstackgerrit Merged openstack/openstacksdk stable/victoria: Don't set list_type to dict for server groups. https://review.opendev.org/763119
13:26:51 stephenfin gtema: FYI https://review.opendev.org/#/c/763178/
13:27:08 stephenfin that should unblock your aggregate and flavor OSC changes
13:27:41 gtema hmm, I wanted to go with hypervisors further, but ok
13:28:18 gtema do not really want to make release each week ;-), but is still required
13:28:44 stephenfin meh, releases are cheap :)
13:28:56 openstackgerrit Merged openstack/python-openstackclient master: Add a few selectable fields to the "openstack server list" output https://review.opendev.org/741181
13:28:57 openstackgerrit Merged openstack/openstacksdk master: Update TOX_CONSTRAINTS_FILE https://review.opendev.org/761055
13:28:59 openstackgerrit Merged openstack/python-openstackclient master: update lower-constraints.txt https://review.opendev.org/760270
13:29:03 stephenfin but we can wait of course, if you'd prefer
13:29:06 gtema yes, they are
13:29:14 gtema already +1-ed
13:30:54 stephenfin sweet
13:32:09 gtema hypervisors API is also not so straight forward
13:32:16 gtema but not that bad as host
14:16:48 openstackgerrit Gregory Thiemonge proposed openstack/openstacksdk master: Add tls_enabled param for Octavia Pools https://review.opendev.org/763189
14:17:36 openstackgerrit Gregory Thiemonge proposed openstack/openstacksdk master: Add tls_enabled param for Octavia Pools https://review.opendev.org/763189
15:26:17 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Complete compute.hypervisor functions https://review.opendev.org/763202
17:42:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListPort query https://review.opendev.org/754117
18:06:07 openstackgerrit Stephen Finucane proposed openstack/cliff master: trivial: Remove references to Python 2.7 https://review.opendev.org/760334
19:56:57 openstackgerrit Merged openstack/cliff master: Remove six https://review.opendev.org/754683
21:08:27 openstackgerrit Merged openstack/cliff master: trivial: Remove references to Python 2.7 https://review.opendev.org/760334

Earlier   Later