Monday, 2020-06-08

« Sunday, 2020-06-07 Index Tuesday, 2020-06-09 »
*** eandersson has quit IRC00:35
*** eandersson has joined #openstack-sdks00:36
openstackgerritMonty Taylor proposed openstack/osc-lib master: Remove commandmanager subclass  https://review.opendev.org/73400001:55
*** gtema has joined #openstack-sdks02:45
*** gtema has quit IRC02:49
*** gtema has joined #openstack-sdks03:47
*** gtema has quit IRC03:52
*** slaweq has joined #openstack-sdks04:11
*** ricolin has joined #openstack-sdks04:14
*** slaweq has quit IRC04:20
*** gtema has joined #openstack-sdks04:59
*** gtema has quit IRC05:04
*** factor has quit IRC06:04
*** jawad_axd has joined #openstack-sdks06:07
*** jawad_axd has quit IRC06:12
*** gtema has joined #openstack-sdks06:35
*** ttsiouts has joined #openstack-sdks06:42
*** yolanda has joined #openstack-sdks06:43
*** gtema_ has joined #openstack-sdks06:45
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Wait for the project cleanup to complete  https://review.opendev.org/73401606:47
*** gtema has quit IRC06:49
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Do not clean keypairs in the project cleanup  https://review.opendev.org/73401906:51
*** gtema has joined #openstack-sdks07:02
*** stingrayza has joined #openstack-sdks07:03
*** also_stingrayza has quit IRC07:05
*** gtema_ has quit IRC07:06
*** ttsiouts has quit IRC07:08
*** iurygregory has joined #openstack-sdks07:10
*** rpittau|afk is now known as rpittau07:14
*** ralonsoh has joined #openstack-sdks07:15
*** slaweq has joined #openstack-sdks07:16
*** jawad_axd has joined #openstack-sdks07:34
*** tosky has joined #openstack-sdks07:39
*** ttsiouts has joined #openstack-sdks07:39
*** ttsiouts has quit IRC07:44
*** jpich has joined #openstack-sdks07:54
*** jpena|off is now known as jpena07:56
*** tkajinam has quit IRC08:07
*** nightmare_unreal has joined #openstack-sdks08:08
*** ttsiouts has joined #openstack-sdks08:19
*** sshnaidm|afk is now known as sshnaidm08:44
*** ttsiouts has quit IRC08:52
sshnaidmcores, please take a look in your time https://review.opendev.org/#/c/733991/09:00
*** ricolin has quit IRC09:18
*** jistr has quit IRC09:50
*** jistr has joined #openstack-sdks09:51
*** iokiwi3 has quit IRC09:56
*** adriant has quit IRC09:56
*** adriant has joined #openstack-sdks09:57
*** slaweq has quit IRC09:57
*** slaweq has joined #openstack-sdks09:58
*** rpittau is now known as rpittau|bbl10:15
*** ttsiouts has joined #openstack-sdks10:27
*** nightmare_unreal has quit IRC10:33
*** ttsiouts has quit IRC11:01
*** gtema has quit IRC11:19
*** nightmare_unreal has joined #openstack-sdks11:32
*** jpena is now known as jpena|lunch11:32
*** ttsiouts has joined #openstack-sdks11:47
*** gtema has joined #openstack-sdks11:51
*** gtema has quit IRC11:51
*** gtema has joined #openstack-sdks11:51
*** ttsiouts has quit IRC11:52
*** Luzi has joined #openstack-sdks11:53
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Do not clean keypairs in the project cleanup  https://review.opendev.org/73401911:56
*** rpittau|bbl is now known as rpittau12:13
*** enriquetaso has joined #openstack-sdks12:19
*** gtema_ has joined #openstack-sdks12:21
*** gtema has quit IRC12:25
*** ttsiouts has joined #openstack-sdks12:44
*** gtema has joined #openstack-sdks12:47
*** mgariepy has quit IRC12:48
openstackgerritSean McGinnis proposed openstack/keystoneauth master: Remove mox3 from local upper-constraints  https://review.opendev.org/73407112:50
*** gtema_ has quit IRC12:51
*** mgariepy has joined #openstack-sdks13:03
*** jpena|lunch is now known as jpena13:03
*** gtema_ has joined #openstack-sdks13:03
*** lbragstad has joined #openstack-sdks13:06
*** gtema has quit IRC13:07
openstackgerritMerged openstack/openstacksdk master: Make optional name and admin password  https://review.opendev.org/73399113:13
*** ttsiouts has quit IRC13:18
*** iurygregory is now known as iurygregory|afk13:20
*** gtema has joined #openstack-sdks13:22
*** gtema_ has quit IRC13:26
*** gtema_ has joined #openstack-sdks13:34
*** lbragstad has quit IRC13:34
*** lbragstad has joined #openstack-sdks13:37
*** gtema has quit IRC13:38
*** Luzi has quit IRC13:41
*** gtema has joined #openstack-sdks13:50
*** gtema__ has joined #openstack-sdks13:53
*** gtema_ has quit IRC13:54
*** gtema has quit IRC13:57
*** gtema has joined #openstack-sdks14:14
mordredgtema__: if you get a sec, I started hacking on plugin loading in osc: https://review.opendev.org/#/q/topic:osc-plugin-optimize - right now it's just about unwinding the deps a little bit and optimizing a little bit - next I'll work on the client creation steps14:15
mordredI think it shouldn't take too long to get those bits done14:15
mordredhowever, looking at the osc-lib patch, I'm not convinced osc-functional-devstack and osc-functional-devstack-tips are doing the right things - I'd expect tips to pass and non-tips to fail there14:15
gtemayeah, I'm fighting with the project cleanup, which seems also not that easy wrt filters (dry_run with filters is pretty much useless in the network area)14:16
gtemayou can't easily estimate, whether ports would be present in the real run or not (re-evaluate whether dropping resource will result in dropping port or not)14:17
*** gtema__ has quit IRC14:17
gtemamordred: what is your intention is "black"?14:19
mordredgtema: dunno. that was just me playing14:20
gtemaokay14:20
mordredgtema: good point re: ports - thats "fun" :(14:20
gtemasome things are indeed interesting, but replacing ' to " seems really mad14:21
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add cliff to libs-from-git for devstack functional tips  https://review.opendev.org/73409514:21
gtemaI think we also need to have one src job in the osc with all it deps14:21
mordredgtema: yeah. that one is a bit annoying. I like the indentation choice - I think it's good for making refactoring patches cleaner14:21
gtemaagree14:22
nightmare_unrealfrickler:  hey , if you get time. Can you read my comment here : https://review.opendev.org/#/c/733573/ . Basically it's regarding that it's cliff that handles the output formatting . So the problem is not in osc-client per se ( I think)14:22
mordredgtema: and yes - the osc patch there updates the tips job to include cliff14:22
gtemawe need same for sdk, osc-lib and cliff14:23
gtemawe will have pretty much changes soon and it becomes madness14:24
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add cliff to libs-from-git for devstack functional tips  https://review.opendev.org/73409514:26
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Remove os-client-config references  https://review.opendev.org/73409714:26
mordredgtema: when you say "need same" - what do you mean? you mean updating the jobs there to make sure it's all good?14:27
mordredalso - there's a "fun" thing about osc and devstack jobs that I don't think we've fully captured in our test jobs yet14:27
gtemaoh, I missed definition of the functional-devstack-tips. I thought it was not taking libs from git, but only release ones14:28
mordredwe have "release vs src" - and we have that for both devstack/openstack itself, as well as the client stuff installed into the tox virtualenv14:28
gtemayes, now I see14:28
*** jawad_axd has quit IRC14:31
*** iurygregory|afk is now known as iurygregory14:36
fricklernightmare_unreal: well if you think you can fix this in cliff, go ahead.14:44
dtantsurmordred: hey, do you remember from the top of your head, if OS_AUTH_URL has /v3/ in the end, do we trim it in keystoneauth?14:50
dtantsurthis is re https://github.com/dtantsur/rust-openstack/issues/11414:50
gtemain my case it has v314:51
mordreddtantsur: we don't trim it if we're looking for version 314:51
mordreddtantsur: I think the only time we trim it and use unversioned is if someone requests v214:52
dtantsurgotcha14:52
mordredbecause /v3 should also contain a versioned discovery doc14:52
dtantsurhttps://github.com/dtantsur/rust-osauth/blob/master/src/identity.rs#L226 doesn't account for a trailing slash #facepalm14:53
mordreddtantsur: hah14:54
dtantsurthis project has a loong way ahead in terms of robustness14:54
dtantsuron the other hand, people are using it \o/14:54
mordreddtantsur: this comes back to the ptg discussion about trailing vs non-trailing in catalogs14:54
mordreddtantsur: \o/14:54
mordredI pinged sdague about that btw and he did not remember14:54
dtantsurtoo much time has passed14:55
*** gtema_ has joined #openstack-sdks14:55
mordredyah. I remember having a summit discussion at some summit. I mostly remember that the room was downstairs14:55
mordredbut I don't remember which city ... maybe it was atlanta?14:55
mordredmight have been paris14:56
mordredor hong kong14:56
gtema_:D14:56
*** gtema has quit IRC14:59
*** gtema has joined #openstack-sdks15:01
*** gtema_ has quit IRC15:05
*** ttsiouts has joined #openstack-sdks15:15
*** iurygregory has quit IRC15:29
*** iurygregory has joined #openstack-sdks15:29
openstackgerritMerged openstack/ansible-collections-openstack master: Add setup.py for install with pip  https://review.opendev.org/73040015:33
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396115:34
openstackgerritMonty Taylor proposed openstack/osc-lib master: Remove commandmanager subclass  https://review.opendev.org/73400015:36
mordredgtema: sorry - I missed this one in the topic: https://review.opendev.org/#/c/733968/115:38
gtemadone15:39
gtemafeel free to set +A yourself15:39
mordredgtema: thansk!15:40
mordredgtema: so - with the cliff patch, the idea is that in osc we'll have a file that has a ton of "commandmanager.add_command" calls pointing to the classes in-tree rather than registering our own plugins as entrypoints15:41
mordredthen, since cliff will first attempt to resolve those from what it knows before scanning entrypoints15:42
mordredfor the common case we can avoid the entrypoint cost- and only take that cost if someone asks for a command tha we don't know15:42
gtemayupp, makes sense15:42
*** ttsiouts has quit IRC15:48
*** gtema has quit IRC16:03
*** rpittau is now known as rpittau|afk16:08
*** dtantsur is now known as dtantsur|afk16:10
openstackgerritMonty Taylor proposed openstack/cliff master: Add load_plugins option  https://review.opendev.org/73396116:20
*** jpich has quit IRC16:29
*** sshnaidm is now known as sshnaidm|afk16:34
*** gtema has joined #openstack-sdks16:36
smcginnismordred: Hey there. Doing a search on requirements, I can't find any repos that have shade declaring it as a dependency in their requirements files. Does that seem right?16:40
*** gtema has quit IRC16:40
mordredsmcginnis: yeah - I don't think anything should be depending on shade at this point16:40
mordredsmcginnis: everything that used to should be depending on openstacksdk at this point16:40
mordredsmcginnis: same with os-client-config16:41
smcginnisOK, great. Another one I can clean out of global reqs then.16:41
mordredsmcginnis: we're probably in a good place to change the release model for them now too16:41
mordredsmcginnis: maybe just to independent?16:41
smcginnismordred: If you think it will only need occasional releases from time to time, that may be better.16:42
*** nightmare_unreal has quit IRC16:43
mordredsmcginnis: yeah- the're mostly on life-support for backwards compat16:44
mordredthey definitely don't need cycle based releases16:44
smcginnismordred: cinder16:46
smcginnisDarn clipboard.16:46
smcginnismordred: https://review.opendev.org/73414916:47
*** AJaeger has left #openstack-sdks16:58
*** jawad_axd has joined #openstack-sdks17:05
openstackgerritMerged openstack/cliff master: Remove python3.5  https://review.opendev.org/73396817:18
openstackgerritMerged openstack/cliff master: Migrate to stestr  https://review.opendev.org/73396917:18
openstackgerritMerged openstack/cliff master: Remove unneeded tests  https://review.opendev.org/73397017:18
*** jpena is now known as jpena|off17:20
*** ttsiouts has joined #openstack-sdks17:25
*** ttsiouts has quit IRC17:29
*** jawad_axd has quit IRC18:02
*** ralonsoh has quit IRC18:15
*** dustinc has joined #openstack-sdks18:16
*** gtema has joined #openstack-sdks18:37
*** gtema has quit IRC18:42
mordredmnaser, dtantsur|afk, smcginnis: if you get a sec, mind reviewing https://review.opendev.org/#/c/733964 ? It's mostly just moving code from osc-lib to cliff so that we can remove it from osc-lib and then do a followup which will let us optimize osc by skipping entrypoints loading for common cases18:44
*** dmsimard has joined #openstack-sdks19:01
dmsimardhey o/ maybe I'm blind but I can't find an openstacksdk method that maps to keystone's /v3/users/{user_id}/projects ? https://docs.openstack.org/api-ref/identity/v3/?expanded=list-users-detail,list-projects-for-user-detail#list-projects-for-user19:04
dmsimardlooking at the openstacksdk docs from https://docs.openstack.org/openstacksdk/latest/user/proxies/identity_v3.html#user-operations19:05
dmsimardI guess I found https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/identity/v3/_proxy.py#L507-L520 but not seeing in the docs :p19:11
*** jawad_axd has joined #openstack-sdks19:19
*** gtema has joined #openstack-sdks19:23
*** jawad_axd has quit IRC19:27
*** gtema has quit IRC19:28
mordreddmsimard: we might just not have it linked in the docs :(:19:33
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add user_projects method to docs index  https://review.opendev.org/73418319:36
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add user_projects method to docs index  https://review.opendev.org/73418319:37
mordreddmsimard: ^^ thanks!19:37
dmsimard\o/19:38
*** gtema has joined #openstack-sdks20:01
*** jawad_axd has joined #openstack-sdks20:01
*** gtema has quit IRC20:09
*** enriquetaso has quit IRC20:24
*** jawad_axd has quit IRC20:31
*** enriquetaso has joined #openstack-sdks20:53
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Add cliff to libs-from-git for devstack functional tips  https://review.opendev.org/73409521:18
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: Remove os-client-config references  https://review.opendev.org/73409721:20
*** enriquetaso has quit IRC21:26
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: identity: autogenerate docs  https://review.opendev.org/69293121:34
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: image: autogenerate docs  https://review.opendev.org/69302521:34
*** slaweq has quit IRC21:45
*** slaweq has joined #openstack-sdks21:47
mordredamotoki: if you get some time, would you mind reviewing https://review.opendev.org/#/c/733964/ ? the more important bits are the followup change which won't start working until https://review.opendev.org/#/c/734000/ is landed, but that won't start working until we release osc-lib with the first change :)21:49
mordredthen once all of that is in and happy, we can make an osc patch to register the in-tree plugins with add_command and skip the entrypoint scan cost for most of the commands21:50
*** gtema has joined #openstack-sdks22:05
openstackgerritHervé Beraud proposed openstack/cliff master: drop mock from lower-constraints and requirements  https://review.opendev.org/73432622:06
*** efried has quit IRC22:07
*** gtema has quit IRC22:10
*** tkajinam has joined #openstack-sdks22:49
openstackgerritMerged openstack/osc-lib master: Add get_osc_show_columns_for_sdk_resource function  https://review.opendev.org/69887422:58
*** dmellado_ has joined #openstack-sdks23:10
*** dmellado has quit IRC23:11
*** slaweq has quit IRC23:11
*** dmellado_ is now known as dmellado23:11
*** tosky has quit IRC23:13
*** slaweq has joined #openstack-sdks23:17
openstackgerritMerged openstack/python-openstackclient master: Add OpenStack Client for Manila docs  https://review.opendev.org/70625223:17
openstackgerritHervé Beraud proposed openstack/keystoneauth master: drop mock from lower-constraints  https://review.opendev.org/73435923:18
openstackgerritMerged openstack/openstacksdk master: Extend statistics reporting  https://review.opendev.org/72709723:40
« Sunday, 2020-06-07 Index Tuesday, 2020-06-09 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!