Tuesday, 2019-05-14

*** zbr_ has joined #openstack-sdks00:13
*** zbr has quit IRC00:14
*** ttsiouts has joined #openstack-sdks01:22
openstackgerritcaoyuan proposed openstack/keystoneauth master: Replace git.openstack.org URLs with opendev.org URLs  https://review.opendev.org/65501601:22
*** ttsiouts has quit IRC01:54
*** whoami-rajat has joined #openstack-sdks02:59
*** ttsiouts has joined #openstack-sdks03:03
*** gkadam has joined #openstack-sdks03:28
*** ttsiouts has quit IRC03:35
*** ricolin has joined #openstack-sdks03:58
openstackgerritqian wang proposed openstack/microversion-parse master: Add .idea/* in .gitignore  https://review.opendev.org/65895604:02
*** gtema has joined #openstack-sdks04:47
*** gtema has quit IRC04:52
*** ttsiouts has joined #openstack-sdks05:33
*** gtema has joined #openstack-sdks05:36
*** gtema has quit IRC05:53
*** Luzi has joined #openstack-sdks06:02
*** ttsiouts has quit IRC06:06
*** holser_ has joined #openstack-sdks06:18
*** gtema has joined #openstack-sdks06:43
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add image.stage methods  https://review.opendev.org/65298106:49
*** slaweq has joined #openstack-sdks06:51
*** slaweq has quit IRC07:03
*** oyrogerg has joined #openstack-sdks07:14
*** ttsiouts has joined #openstack-sdks07:31
*** jpich has joined #openstack-sdks07:46
*** jangutter has joined #openstack-sdks07:50
*** dtantsur|afk is now known as dtantsur07:54
*** ralonsoh has joined #openstack-sdks07:58
*** jpena|off is now known as jpena07:59
*** ttsiouts has quit IRC08:34
*** oyrogerg has quit IRC09:09
*** ttsiouts has joined #openstack-sdks09:10
*** e0ne has joined #openstack-sdks09:13
*** ricolin has quit IRC09:14
*** tssurya has joined #openstack-sdks09:42
*** ttsiouts has quit IRC09:44
*** yolanda_ has quit IRC10:06
*** yolanda_ has joined #openstack-sdks10:08
openstackgerritMerged openstack/openstacksdk master: Extract image download method into a mixin  https://review.opendev.org/65834610:10
*** gtema has quit IRC10:18
*** jangutter has quit IRC10:56
*** jangutter has joined #openstack-sdks11:31
*** e0ne has quit IRC11:31
*** e0ne has joined #openstack-sdks11:31
*** jpena is now known as jpena|lunch11:41
*** slaweq has joined #openstack-sdks11:53
*** e0ne has quit IRC12:02
*** slaweq has quit IRC12:21
*** jpena|lunch is now known as jpena12:32
*** slaweq has joined #openstack-sdks12:37
*** gkadam has quit IRC12:40
*** slaweq has quit IRC12:42
*** e0ne has joined #openstack-sdks12:50
*** gtema has joined #openstack-sdks12:52
efrieddtantsur: see question from dustinc about 15h ago12:52
*** yolanda_ has quit IRC12:57
*** ttsiouts has joined #openstack-sdks13:02
*** ttsiouts has quit IRC13:07
mordredefried, dustinc: what do you mean by "spec" in this context?13:13
openstackgerritMerged openstack/openstacksdk master: Replace use of log.warn with log.warning  https://review.opendev.org/65852913:14
efriedmordred: FYI dustinc is on Pacific time, so it may be a couple hours before he gets back to you. (And I don't know what he meant by 'spec')13:18
mordredefried: neat! I'm also in and out today - so we may or may not manage to connect with each other :)13:20
efriedmordred: In general, what's the process when someone wants to enhance one of these proxies? Is a spec (as in rst document describing the change) required?13:21
mordrednope - one just dives in. if it's a big change, checking in with other folks first is a good idea - but we don't have nearly enough active devs to warrant the overhead of specs atm13:22
gtemamordred, why do you say so, a 3 of us is not enough? ;-)13:23
efrieddustinc: FYI the change appears to have been delivered via https://review.opendev.org/#/c/647730/13:23
*** dmellado has quit IRC13:24
*** dmellado has joined #openstack-sdks13:24
*** ttsiouts has joined #openstack-sdks13:26
efrieddustinc: If you're looking for more information on what the json payload should look like, I would start with the corresponding baremetal API: https://developer.openstack.org/api-ref/baremetal/?expanded=update-node-detail#update-node13:26
mordredgtema: 3 of us is perfect :)13:27
gtemawe are nearly almighty13:27
efried(which, mordred/dtantsur/gtema, would be a good thing to link from any of these proxy methods)13:27
efried(especially the ones that have **kwargs)13:27
gtemafor baremetal its dtantsur13:27
mordredefried: I agree - that's not a ba idea at all13:27
* efried wonders if there's a way to autodoc that13:28
efriedprolly not13:29
efriedsince it's in another repo13:29
efriedI was thinking of the :oslo.config:`group.option` role13:29
efriedbut that only works in same repo13:30
*** ttsiouts has quit IRC13:30
mordredyeah - there's ways to do fancy sphinx stuff with multi-repo external links and stuff - and connecting that with zuul for multi-repo source might be neat - but it's also probably easier to just put the links in for now13:31
mordredI could see us easily getting so clever we don't understand how to write docs anymore13:31
efriedI thought we had already passed that point.13:33
efriedonly stephenfin knows how to write docs13:33
efriedjob security13:33
stephenfin /o/13:33
stephenfin \o\13:33
mordredefried: does that mean stephenfin will write all my docs for me? neat!13:34
stephenfinefried: It could be done by building on top of intersphinx13:34
stephenfinbut whether it's important enough is another matter13:35
efriedconsidering we should ideally do it for every proxy method, and probably a bunch of the others, it would be nice if it happened automatically without any input. Like introspected something about the method. Like if we (already) had decorators on the methods indicating which operation they invoke. But we don't. So we'd have to add something to each method anyway. Or something more convoluted like a centralized mapping.13:37
efriedso if we're going to do something manual to each method anyway, might as well be adding the actual link.13:37
dustincefried mordred thanks I’m in and out next couple hours but will check that out.13:37
efriedstephenfin: how do I noqa a docstring?13:37
dustincAnd by spec I meant blueprint so I could use it to find the change so all is good. Thanks!13:39
efrieddustinc: You can use some git tricks to find the change, lmk if you want to go over that.13:40
stephenfin"""Some string.13:40
stephenfinMore stuff.13:41
stephenfin"""  # noqa13:41
stephenfin^ efried (I think)13:41
efriedah, /me tries...13:41
gtemaefried, dustinc, do you need placement support in sdk?13:42
efriedstephenfin: nope :(13:42
efriedgtema: It's already there, proxy-wise.13:42
efriedgtema: Other than something broken in the dep chain, this works: https://review.opendev.org/65602313:42
stephenfinThen I have no idea, I'm afraid. I assume it's the line length thing that's causing issues?13:42
efriedstephenfin: Yes, I have a hyperlink in the docstring and the hyperlink is >80c by itself.13:42
stephenfinYou could manually wrap inside the docstring. It would make for uglier code but keep flake8 happy13:43
stephenfinOtherwise, use a URL shortner?13:43
gtemaefried: I mean in a normal sdk way (with resources and refular proxy methods)13:43
efriedgtema: I think we've talked about it before and decided it's not worth it, at least for now.13:43
gtemaok, thks13:44
efriedgtema: It could be kinda neat, but the straight up APIs are pretty easy to use as is.13:44
gtemathat's good13:44
efriedFeel like if people have that kind of time, there's more important things to be done :)13:44
efriedlike figuring out how to put long hyperlinks into the docstrings13:45
efriedstephenfin: backslash-wrapping works, and isn't *too* hideous. Thanks.13:46
efriedahcrap. The pep8 error I was getting, it wasn't even in the docstring I was messing with.13:48
mordredefried: BOW DOWN TO THE PEP8 GODS13:48
openstackgerritEric Fried proposed openstack/openstacksdk master: Link to baremetal API reference from patch_node  https://review.opendev.org/65906813:51
efriedmordred, gtema, dtantsur: Toe in the water ^13:51
efriedif that flies, more can be done13:52
mordredefried: ++13:54
gtemaefried: ++ from me as well13:54
efriedstephenfin: there was no pep8 error on long link in docstring. I was seeing things.13:55
efriedand it's not because we're ignoring H4 either.13:55
efriedPerhaps it's just smart enough to realize that a link doesn't count.13:55
*** Luzi has quit IRC14:18
*** gtema has quit IRC14:58
*** ttsiouts has joined #openstack-sdks15:27
*** openstackgerrit has quit IRC15:54
*** e0ne has quit IRC15:57
*** ttsiouts has quit IRC16:01
*** jpich has quit IRC16:31
*** jpich has joined #openstack-sdks16:31
*** holser_ has quit IRC16:35
*** openstackgerrit has joined #openstack-sdks16:49
openstackgerritSurya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock  https://review.opendev.org/65912416:49
*** dtantsur is now known as dtantsur|afk16:51
*** tssurya has quit IRC17:06
*** slaweq has joined #openstack-sdks17:06
*** jpich has quit IRC17:10
*** slaweq has quit IRC17:14
*** ralonsoh has quit IRC17:25
*** slaweq has joined #openstack-sdks17:31
*** slaweq has quit IRC17:35
*** jpena is now known as jpena|off17:41
*** e0ne has joined #openstack-sdks17:57
*** ttsiouts has joined #openstack-sdks17:58
*** e0ne has quit IRC18:05
*** ttsiouts has quit IRC18:31
*** e0ne has joined #openstack-sdks18:38
*** slaweq has joined #openstack-sdks19:21
*** e0ne has quit IRC20:20
*** ttsiouts has joined #openstack-sdks20:28
*** cdent has joined #openstack-sdks20:36
cdentmordred: we talked about this for and I've managed to forget: you added a command somewhere to list the versions of all the apis configured in the catalog. I've forgotten where you put it.20:42
*** slaweq has quit IRC20:44
*** cdent has quit IRC20:58
*** ttsiouts has quit IRC21:01
*** whoami-rajat has quit IRC22:18
*** ttsiouts has joined #openstack-sdks22:58
*** ttsiouts has quit IRC23:32
*** irclogbot_0 has quit IRC23:45
*** irclogbot_1 has joined #openstack-sdks23:48

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!