Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-12
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
22:43:46 mordred the good news is that I dont think fixing it is going to be super HARD - it just needs to be done in a few places all at once
22:48:10 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813
#openstack-sdks - 2020-03-13
00:20:38 openstackgerrit Merged openstack/openstacksdk stable/train: Consistent normalization of Machine objects in the cloud layer https://review.opendev.org/712295
10:08:36 BakaKuna Hi, I was trying to change the admin state of a dhcp agent yesterday. Mordred gave me a pointer, but I think I just don't understand the flow yet. Should I be able to change the admin state of a DHCP agent (or any other attribut)? And if so, could anyone tell me how that should be done?
10:10:49 BakaKuna I have looked through the documentation, but I can't seem to find a place where an attribute is changed without using a dedicated method.
10:29:13 BakaKuna Found it, network.update_agent() did the trick. This is amazing! How can I only find out about this whole project now? I have been immersed in OpenStack literature the last 6 months since I started a new job :D
10:39:59 openstackgerrit Alfredo Moralejo proposed openstack/osc-lib master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/712904
10:42:33 noonedeadpunk mordred: yeah, I tried to do kinda the same as https://review.opendev.org/#/c/712810/1 but also had local tests failing...
10:43:10 noonedeadpunk so ended up in just fixing exception....
10:49:11 noonedeadpunk also there are the way more usages _find_identity_resource....
10:49:46 noonedeadpunk actually I'm really wondering why osc thtows the same error for so different exceptions...
11:50:42 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_nova_host_aggregate: Fixup sanity test failures https://review.opendev.org/712917
13:53:35 amoralej hi, wrt https://review.opendev.org/#/c/712904/, is osc-lib gate broken?
13:54:09 amoralej ERROR: Package 'neutron' requires a different Python: 2.7.17 not in '>=3.6'
14:06:22 mordred noonedeadpunk: yeah - that's the larger mess I was talking about - the same exception multiple causes thing
14:06:57 mordred BakaKuna: \o/
14:09:54 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
14:10:01 mordred noonedeadpunk: try that one
14:11:18 mordred amoralej|lunch: might be - let me look

Earlier   Later