Thursday, 2020-03-12

openstackgerritFelix Yan proposed openstack/cliff master: Fix compatibility with new cmd2
*** openstackstatus has joined #openstack-sdks00:44
*** ChanServ sets mode: +v openstackstatus00:44
*** diablo_rojo has quit IRC02:13
*** KeithMnemonic1 has joined #openstack-sdks03:28
*** KeithMnemonic has quit IRC03:31
*** ricolin_ has joined #openstack-sdks03:31
*** ricolin_ has quit IRC03:33
*** ricolin_ has joined #openstack-sdks03:34
*** dave-mccowan has quit IRC03:58
*** KeithMnemonic1 has quit IRC04:06
*** tobiash has quit IRC04:16
*** tobiash has joined #openstack-sdks04:17
*** factor has joined #openstack-sdks04:34
*** tobiash has quit IRC04:43
*** tobiash has joined #openstack-sdks04:46
*** evrardjp has quit IRC05:35
*** evrardjp has joined #openstack-sdks05:36
*** Luzi has joined #openstack-sdks06:02
*** smcginnis has quit IRC07:03
*** smcginnis has joined #openstack-sdks07:04
*** rpittau|afk is now known as rpittau07:17
*** slaweq has joined #openstack-sdks08:02
*** tosky has joined #openstack-sdks08:18
*** jpena|off is now known as jpena08:45
*** jpich has joined #openstack-sdks08:47
*** rpittau is now known as rpittau|bbl08:52
*** ralonsoh has joined #openstack-sdks08:58
*** dtantsur|afk is now known as dtantsur09:13
*** sshnaidm|afk is now known as sshnaidm09:39
*** gtema has joined #openstack-sdks09:49
dtantsurmordred: morning! when you have a minute, this backport will likely make sshnaidm a bit happier09:54
sshnaidmdtantsur, thanks!09:55
sshnaidmdtantsur, btw take a look
dtantsursshnaidm: are you aware of this call?
dtantsuryou seem to be duplicating it09:59
dtantsurif you miss something from it, let's update the SDK rather then duplicating the functionality10:00
sshnaidmdtantsur, yeah I know this call, but it's blocking, in the module I run all nodes provision in parallel10:01
dtantsursshnaidm: I mean, it's a direct replacement for lines 354-40010:02
dtantsurwhere you're looping over nodes waiting for them to be finished10:02
dtantsurwait_for_nodes_provision_state does exactly that10:02
dtantsurI don't suggest you use wait=True on set_node_provision_state10:02
sshnaidmdtantsur, a-ha, seems like I missed it can be done for multiple nodes10:03
dtantsuryep, I implemented it this way specially for cases like yours :)10:03
dtantsurlemme know if anything is missing there, we can update it10:03
sshnaidmdtantsur, and in case of failure, does it still return nodes that were provisioned successfully?10:04
dtantsursshnaidm: I'm not sure, but we can implement that10:04
* dtantsur checks the source code10:04
dtantsursshnaidm: it raises an exception by default. do you need this functionality?10:06
sshnaidmdtantsur, yeah, maybe an option to not raise, but return passed/failed10:06
sshnaidmdtantsur, not too critical though10:07
dtantsursshnaidm: I'll look into that. I'd suggest you start without it to avoid these 50 lines of code.10:07
sshnaidmdtantsur, yeah, will replace it, thanks10:08
sshnaidmdtantsur, also need to figure out if we need one module for "set state" for nodes, right now it's one for provide, and one for cleaning10:09
sshnaidmand they have dups of course10:09
dtantsura good question. at the very least, they have different parameters10:10
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: [WIP] baremetal: fail-less mode for wait_for_nodes_provision_state
dtantsursshnaidm: wdyt about this design ^^^?10:18
openstackgerritIgor Malinovskiy proposed openstack/python-openstackclient master: Add 'address_scope' type support to network rbac commands
sshnaidmdtantsur, perfect10:24
dtantsurokie, will add tests now10:24
sshnaidmdtantsur, cool, thanks10:25
sshnaidmdtantsur, after I'm done, we'll need to talk about how to optimize the modules10:25
dtantsursshnaidm: do we need to schedule a virtual meetup for that? given how less and less likely the PTG is...10:26
sshnaidmdtantsur, yeah, I think it's good to do10:26
sshnaidmdtantsur, tbh I don't believe ptg will happen in June10:26
sshnaidmmaybe only virtual one10:27
*** BakaKuna has joined #openstack-sdks10:36
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: baremetal: fail-less mode for wait_for_nodes_provision_state
dtantsursshnaidm: the final one (testing welcome) ^^^11:04
sshnaidmdtantsur, ack, will try to test it today11:06
*** sshnaidm has quit IRC11:18
*** sshnaidm has joined #openstack-sdks11:19
*** sshnaidm has quit IRC11:22
*** sshnaidm has joined #openstack-sdks11:30
*** sshnaidm has quit IRC11:30
*** gtema has quit IRC11:31
openstackgerritRodolfo Alonso Hernandez proposed openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query
*** jpena is now known as jpena|lunch11:49
*** sshnaidm has joined #openstack-sdks11:52
*** sshnaidm has quit IRC11:53
*** sshnaidm has joined #openstack-sdks11:54
*** enriquetaso has joined #openstack-sdks12:09
*** Luzi has quit IRC12:10
*** lastmikoi has quit IRC12:20
*** lastmikoi has joined #openstack-sdks12:25
*** tkajinam has quit IRC12:27
*** jpena|lunch is now known as jpena12:32
*** rpittau|bbl is now known as rpittau13:02
BakaKunaI'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
*** jpich has quit IRC13:29
*** jpich has joined #openstack-sdks13:30
*** dtantsur is now known as dtantsur|brb13:36
openstackgerritJose Castro Leon proposed openstack/python-openstackclient master: Allow os quota list query to filter by project
*** jistr is now known as jistr|mtg13:59
*** dave-mccowan has joined #openstack-sdks14:02
bverschuerengate tests failed for due to the aggregate functional test, but it is rebased now after the fix and should be fine14:07
bverschuerenif anyone could have a look ?14:07
*** dave-mccowan has quit IRC14:25
*** dave-mccowan has joined #openstack-sdks14:29
mordredif anyone sees BabaKuna, the env var woudl bs OS_NETWORK_DEFAULT_MICROVERSION14:40
*** ricolin_ has quit IRC14:41
*** ricolin_ has joined #openstack-sdks14:41
BakaKunamordred: thanks! I'll try that. Have to go work at home now.14:42
mordredBakaKuna: cool - let me know if it still doens;'t work14:43
mordredoh - also - "2" isn't a microversion14:43
mordredslaweq: does neutron have microversions now?14:45
*** ricolin_ has quit IRC14:48
*** BakaKuna has quit IRC14:48
bverschuerenthanks mordred smcginnis !14:48
mordreddtantsur|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 true14:54
mordreddtantsur|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 object14:55
*** bverschueren has quit IRC14:58
*** enriquetaso has quit IRC15:02
*** enriquetaso has joined #openstack-sdks15:02
*** bverschueren has joined #openstack-sdks15:03
openstackgerritIgor Malinovskiy proposed openstack/python-openstackclient master: Add 'subnetpool' type support to rbac commands
*** jistr|mtg is now known as jistr15:33
slaweqmordred: hi, nope15:39
slaweqmordred: we still have only those API extensions15:39
slaweqamotoki was going to propose some spec about microversions for extensions but I don't think he worked on it recently15:39
mordredslaweq: ok. so does BakaKuna's question above make any sense?15:42
slaweqmordred: TBH it looks more like some bug in SDK code15:46
noonedeadpunkhi everyone. Need some help to understand how CLI commands are "routed". Like `openstack image add project`15:50
noonedeadpunkI kinda feel that there's some "bug" there but can't really find "entry point"15:51
noonedeadpunkIt looks like but can't find how it's called15:52
noonedeadpunkThe thing is that I suggest, that while searching for member_id there's exception handling missing15:53
noonedeadpunkso while doing `openstack project list` error handling is preset like
*** dtantsur|brb is now known as dtantsur15:56
noonedeadpunkbut `openstack image add project` ends up that way:
dtantsurmordred: good idea. let's live to see v1 first :)15:57
*** diablo_rojo has joined #openstack-sdks15:58
elmikoAPI SIG office hour now open \o/16:00
mordrednoonedeadpunk: look in setup.cfg at the entrypoints16:00
mordrednoonedeadpunk: you'll see one called "image" - which will point you in the right direction to start tracing16:00
mordrednoonedeadpunk: I'm sorry - you'll find one called "image_add_project"16:01
noonedeadpunkmordred: oh, nice! thanks16:02
mordrednoonedeadpunk: 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 yet16:02
mordrednoonedeadpunk: so you might want to start with trying to see how it goes with the code in master since it's RADICALLY different for image16:03
noonedeadpunkmordred: so do you suggest trying out master version before futher investigation?16:03
noonedeadpunkyeah, ok16:03
mordredoh - actually16:05
noonedeadpunktbh, 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
mordrednoonedeadpunk: those aren't landed yet - ... you might want to grab that change and try with that16:05
noonedeadpunkmordred: oh, thanks ffor providing patch:)16:06
*** diablo_rojo has quit IRC16:06
*** diablo_rojo has joined #openstack-sdks16:10
noonedeadpunkpatch do not resolve issue, infortunatelly16:11
mordredok. well - good to know at least16:11
mordrednoonedeadpunk: if you find a fix - do it on top of that stack - just so we don't hit weird merge conflicts16:11
noonedeadpunkyes, sure:)16:12
openstackgerritMerged openstack/openstacksdk master: Add support for additional volume backup options
elmikodtantsur: 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
dtantsurelmiko: I was actually about to propose us to merge with someone.. SDK?16:22
dtantsurbecause realistically, we're not doing anything except for trolling each other for quite a while :)16:23
elmikoi think merging into sdk would be great16:25
elmikois there a chair or something we should talk to in the sdk group?16:26
elmikoor maybe i should just make an email to discuss ml about this?16:26
dtantsurelmiko: well, mordred?16:28
elmikoi should probably send an email as well, just to help promote transparency and collaboration for the decision16:29
* mordred waves to elmiko16:30
dtantsurmordred: we want to grant you an unlimited access to elmiko, are you happy?16:30
mordredYES GOD YES16:30
*** Blinkiz has quit IRC16:30
* dtantsur has found a truly horrible way to convey the information and is very proud of it16:30
*** Blinkiz has joined #openstack-sdks16:31
mordredwe 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 point16:32
mordredtl;dr - happy to ingest the stuff so it has a place to exist - also happy to not do that16:32
elmikomordred: i'm happy to help how i can as well, but if my recent track record is any evidence my openstack time is super limited16:33
elmikoat the least, i will kick off an email to get this discussion more in the open16:33
dtantsurin my brain SDK and CLI at least have stakes in the API SIG work16:34
dtantsurnow that we've ensured that mordred is not going to run away screaming16:34
elmikoagreed dtantsur, and we have been welcome here for awhile so it seems like a natural merge to me16:34
mordredbasically - this channel is all one big (or maybe not _big_) happy family16:35
mordredelmiko: just be warned, this is going to mean you're going to become core on sdk and osc as a result16:35
dtantsursmaller than it should be, but very happy and welcoming16:35
dtantsurhere goes The Evin Plan (tm)16:35
elmikomordred: ack, thanks for the warn^H^H^H^H heads up16:36
amotokislaweq: 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
amotokiI hope I can restore the work on API extension versioning but the horizon PTL and less contributors in horizon steal my time..... :(16:37
*** enriquetaso has quit IRC16:38
mordredamotoki: I understand these issues16:38
elmikodtantsur, mordred, just sent an email. hopefully it helps get the ball rolling =)16:58
noonedeadpunkAm I right, that,list-projects-for-user-detail#list-projects-for-user is just not present in openstackclient?17:04
noonedeadpunkah, ok, it's used with flag --user17:06
*** sshnaidm is now known as sshnaidm|afk17:07
*** rpittau is now known as rpittau|afk17:27
*** jpich has quit IRC17:28
*** dtantsur is now known as dtantsur|afk17:34
*** evrardjp has quit IRC17:35
*** evrardjp has joined #openstack-sdks17:36
noonedeadpunkthem most sad part with glance memer is that glanceclient itself works:(
noonedeadpunkand things are so because of the way how find_resource is designed....17:39
*** ralonsoh has quit IRC18:29
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Fix excpected exception for find_identity_resource
*** LarsErikP has left #openstack-sdks18:37
*** jpena is now known as jpena|off18:51
*** gmann is now known as gmann_lunch19:00
*** BakaKuna has joined #openstack-sdks19:37
*** gmann_lunch is now known as gmann20:01
openstackgerritMerged openstack/openstacksdk stable/train: Fix microversion negotiation in some bare metal node call
mordrednoonedeadpunk: I think we can fix that20:15
umbSublimeHey 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 refactored21:06
openstackgerritMerged openstack/openstacksdk master: Rationalize examples and functional extra config loading
*** enriquetaso has joined #openstack-sdks21:28
umbSublimeI 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/sdk21:29
*** BakaKuna has quit IRC21:32
*** enriquetaso has quit IRC21:32
*** senrique_ has joined #openstack-sdks21:32
*** senrique_ has quit IRC21:38
*** factor has quit IRC22:27
*** slaweq has quit IRC22:28
mordredumbSublime: sorry - don't know of anything like it22:28
mordrednoonedeadpunk: oh - wow. that's ...22:28
mordrednoonedeadpunk: your fix is showing a slightly deeper issue22:29
*** slaweq has joined #openstack-sdks22:40
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id
mordrednoonedeadpunk, mnaser: ^^ try that22:42
mordredthe 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 fixed22:43
mordredthe 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 once22:43
*** slaweq has quit IRC22:44
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more
*** dave-mccowan has quit IRC22:50
*** tkajinam has joined #openstack-sdks22:56
*** dave-mccowan has joined #openstack-sdks23:10

Generated by 2.15.3 by Marius Gedminas - find it at!