| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-03-12 | |||
| 11:44:24 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query https://review.opendev.org/710825 | |
| 13:18:53 | BakaKuna | I'm probably missing something in the documentation, but maybe someone can help. When I try to commit a change on dhcp_agent I get the error that 'AttributeError: 'Session' object has no attribute 'default_microversion'. I have set the microversion in my env variables as OS_NEUTRON_DEFAULT_MICROVERSION=2, but that is not used apparently. | |
| 13:18:53 | BakaKuna | ||
| 13:46:19 | openstackgerrit | Jose Castro Leon proposed openstack/python-openstackclient master: Allow os quota list query to filter by project https://review.opendev.org/712678 | |
| 14:07:41 | bverschueren | gate tests failed for https://review.opendev.org/#/c/688353/ due to the aggregate functional test, but it is rebased now after the fix and should be fine | |
| 14:07:58 | bverschueren | if anyone could have a look ? | |
| 14:40:29 | mordred | if anyone sees BabaKuna, the env var woudl bs OS_NETWORK_DEFAULT_MICROVERSION | |
| 14:42:47 | BakaKuna | mordred: thanks! I'll try that. Have to go work at home now. | |
| 14:43:10 | mordred | BakaKuna: cool - let me know if it still doens;'t work | |
| 14:43:28 | mordred | oh - also - "2" isn't a microversion | |
| 14:45:05 | mordred | slaweq: does neutron have microversions now? | |
| 14:48:23 | bverschueren | thanks mordred smcginnis ! | |
| 14:54:23 | mordred | dtantsur|brb: if we were to do an SDK v2 (hahaha assuming we ever tag a v1) - I'd totally have methods return a Future if wait wasn't true | |
| 14:55:02 | mordred | dtantsur|brb: or even just have two methods - one that is wait=False that returns a future and one that is wait=True which calls the first and does the wait and returns the object | |
| 15:07:16 | openstackgerrit | Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'subnetpool' type support to rbac commands https://review.opendev.org/712705 | |
| 15:39:13 | slaweq | mordred: hi, nope | |
| 15:39:22 | slaweq | mordred: we still have only those API extensions | |
| 15:39:56 | slaweq | amotoki was going to propose some spec about microversions for extensions but I don't think he worked on it recently | |
| 15:42:28 | mordred | slaweq: ok. so does BakaKuna's question above make any sense? | |
| 15:46:28 | slaweq | mordred: TBH it looks more like some bug in SDK code | |
| 15:47:18 | mordred | yay | |
| 15:50:58 | noonedeadpunk | hi everyone. Need some help to understand how CLI commands are "routed". Like `openstack image add project` | |
| 15:51:41 | noonedeadpunk | I kinda feel that there's some "bug" there but can't really find "entry point" | |
| 15:52:32 | noonedeadpunk | It looks like https://github.com/openstack/python-glanceclient/blob/master/glanceclient/v2/image_members.py#L55-L60 but can't find how it's called | |
| 15:53:12 | noonedeadpunk | The thing is that I suggest, that while searching for member_id there's exception handling missing | |
| 15:55:46 | noonedeadpunk | so while doing `openstack project list` error handling is preset like http://paste.openstack.org/show/790616/ | |
| 15:57:10 | noonedeadpunk | but `openstack image add project` ends up that way: http://paste.openstack.org/show/790617/ | |
| 15:57:46 | dtantsur | mordred: good idea. let's live to see v1 first :) | |
| 16:00:20 | elmiko | API SIG office hour now open \o/ | |
| 16:00:23 | mordred | noonedeadpunk: look in setup.cfg at the entrypoints | |
| 16:00:35 | mordred | noonedeadpunk: you'll see one called "image" - which will point you in the right direction to start tracing | |
| 16:01:10 | mordred | noonedeadpunk: I'm sorry - you'll find one called "image_add_project" | |
| 16:02:03 | noonedeadpunk | mordred: oh, nice! thanks | |
| 16:02:44 | mordred | noonedeadpunk: we just recently landed the patch to replace the calls to glanceclient with calls to sdk - but I don't think I've cut a release with that code yet | |
| 16:03:09 | mordred | noonedeadpunk: so you might want to start with trying to see how it goes with the code in master since it's RADICALLY different for image | |
| 16:03:29 | noonedeadpunk | mordred: so do you suggest trying out master version before futher investigation? | |
| 16:03:33 | mordred | yup | |
| 16:03:36 | noonedeadpunk | yeah, ok | |
| 16:04:27 | dtantsur | o/ | |
| 16:05:12 | mordred | oh - actually | |
| 16:05:35 | noonedeadpunk | tbh, image_add_project & image_member_list & image_remove_project pretty confising, because they point to the same api path, but are so different for CLI.... is there alias support? Like, can be `image_member_add` safely added for deprecating older one later? | |
| 16:05:46 | mordred | noonedeadpunk: those aren't landed yet - https://review.opendev.org/#/c/699416/ ... you might want to grab that change and try with that | |
| 16:06:06 | noonedeadpunk | mordred: oh, thanks ffor providing patch:) | |
| 16:11:00 | noonedeadpunk | patch do not resolve issue, infortunatelly | |
| 16:11:17 | mordred | ok. well - good to know at least | |
| 16:11:43 | mordred | noonedeadpunk: if you find a fix - do it on top of that stack - just so we don't hit weird merge conflicts | |
| 16:12:33 | noonedeadpunk | yes, sure:) | |
| 16:21:15 | openstackgerrit | Merged openstack/openstacksdk master: Add support for additional volume backup options https://review.opendev.org/688353 | |
| 16:22:41 | elmiko | dtantsur: given the consolidation efforts that have been discussed recently, and the low traffic of sig-api acitivity. i wonder if it's time that we re-evaluate ourselves a little? or perhaps consider a next phase for the sig, what do you think? | |
| 16:22:59 | dtantsur | elmiko: I was actually about to propose us to merge with someone.. SDK? | |
| 16:23:17 | dtantsur | because realistically, we're not doing anything except for trolling each other for quite a while :) | |
| 16:25:26 | elmiko | right | |
| 16:25:40 | elmiko | i think merging into sdk would be great | |
| 16:26:13 | elmiko | is there a chair or something we should talk to in the sdk group? | |
| 16:26:31 | elmiko | or maybe i should just make an email to discuss ml about this? | |
| 16:28:32 | dtantsur | elmiko: well, mordred? | |
| 16:29:36 | elmiko | i should probably send an email as well, just to help promote transparency and collaboration for the decision | |
| 16:30:06 | dtantsur | yup | |
| 16:30:42 | dtantsur | mordred: we want to grant you an unlimited access to elmiko, are you happy? | |
| 16:30:49 | mordred | YES GOD YES | |
| 16:32:02 | mordred | we could fold the api sig docs into the SDK/CLI team- I thnk they're all super useful - but I also agree, the main driving humans of the sig are busy with other parts of their life at this point | |
| 16:32:22 | mordred | tl;dr - happy to ingest the stuff so it has a place to exist - also happy to not do that | |
| 16:33:00 | elmiko | lol | |
| 16:33:44 | elmiko | mordred: i'm happy to help how i can as well, but if my recent track record is any evidence my openstack time is super limited | |
| 16:33:56 | elmiko | at the least, i will kick off an email to get this discussion more in the open | |
| 16:34:01 | mordred | kk | |
| 16:34:12 | dtantsur | in my brain SDK and CLI at least have stakes in the API SIG work | |
| 16:34:21 | dtantsur | email++ | |
| 16:34:38 | dtantsur | now that we've ensured that mordred is not going to run away screaming | |
| 16:34:40 | elmiko | agreed dtantsur, and we have been welcome here for awhile so it seems like a natural merge to me | |
| 16:34:44 | elmiko | LOL | |
| 16:34:46 | dtantsur | yup | |
| 16:35:03 | mordred | basically - this channel is all one big (or maybe not _big_) happy family | |
| 16:35:09 | elmiko | \o/ | |
| 16:35:28 | mordred | elmiko: just be warned, this is going to mean you're going to become core on sdk and osc as a result | |
| 16:35:30 | dtantsur | smaller than it should be, but very happy and welcoming | |
| 16:35:48 | dtantsur | here goes The Evin Plan (tm) | |
| 16:36:00 | elmiko | mordred: ack, thanks for the warn^H^H^H^H heads up | |
| 16:37:10 | amotoki | slaweq: mordred: as slaweq mentioned neutron does not support micro-versioning. I explored a way to microversion in neutron, but per PTG discussion perhaps in Denver we agreed that API extension versioning would be simple and reasonable as microversion in neutron may potentially lead to regression in current neutron behaviors. | |
| 16:37:50 | amotoki | I hope I can restore the work on API extension versioning but the horizon PTL and less contributors in horizon steal my time..... :( | |
| 16:38:08 | mordred | amotoki: I understand these issues | |
| 16:58:32 | elmiko | dtantsur, mordred, just sent an email. hopefully it helps get the ball rolling =) | |
| 16:58:55 | dtantsur | thanks! | |
| 17:04:52 | noonedeadpunk | Am I right, that https://docs.openstack.org/api-ref/identity/v3/?expanded=list-projects-detail,list-projects-for-user-detail#list-projects-for-user is just not present in openstackclient? | |
| 17:06:52 | noonedeadpunk | ah, ok, it's used with flag --user | |
| 17:37:50 | noonedeadpunk | them most sad part with glance memer is that glanceclient itself works:( http://paste.openstack.org/show/790619/ | |
| 17:39:50 | noonedeadpunk | and things are so because of the way how find_resource is designed.... | |
| 18:36:00 | openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Fix excpected exception for find_identity_resource https://review.opendev.org/712764 | |
| 20:14:08 | openstackgerrit | Merged openstack/openstacksdk stable/train: Fix microversion negotiation in some bare metal node call https://review.opendev.org/712536 | |
| 20:15:59 | mordred | noonedeadpunk: I think we can fix that | |
| 21:06:04 | umbSublime | Hey I'm trying to see if maybe the osc-lib or sdk has something equivalent to novaclient.utils.flatten_dict or keystone.common.utils.flatten_dict I can't find a method with name flatten_dict, but maybe the name was just refactored | |
| 21:15:18 | openstackgerrit | Merged openstack/openstacksdk master: Rationalize examples and functional extra config loading https://review.opendev.org/712538 | |
| 21:29:31 | umbSublime | I know it's only like 10L of code, but it's quite useful, but maybe it would have it's place in cliff instead of osc/sdk | |
| 22:28:38 | mordred | umbSublime: sorry - don't know of anything like it | |
| 22:28:44 | mordred | noonedeadpunk: oh - wow. that's ... | |
| 22:28:46 | mordred | what? | |
| 22:29:08 | mordred | noonedeadpunk: your fix is showing a slightly deeper issue | |
| 22:42:27 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810 | |
| 22:42:54 | mordred | noonedeadpunk, mnaser: ^^ try that | |
| 22:43:26 | mordred | the other thing is a rabbit hole - and one that we need to go down - but it'll take me a couple of seconds to even write down what is broken there and what needs to be fixed | |