Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-14
13:39:52 dustinc And by spec I meant blueprint so I could use it to find the change so all is good. Thanks!
13:40:36 efried dustinc: You can use some git tricks to find the change, lmk if you want to go over that.
13:40:56 stephenfin """Some string.
13:40:57 stephenfin
13:41:00 stephenfin More stuff.
13:41:03 stephenfin """ # noqa
13:41:08 stephenfin ^ efried (I think)
13:41:11 efried ah, /me tries...
13:42:06 gtema efried, dustinc, do you need placement support in sdk?
13:42:07 efried stephenfin: nope :(
13:42:20 efried gtema: It's already there, proxy-wise.
13:42:39 stephenfin Then I have no idea, I'm afraid. I assume it's the line length thing that's causing issues?
13:42:39 efried gtema: Other than something broken in the dep chain, this works: https://review.opendev.org/656023
13:42:55 efried stephenfin: Yes, I have a hyperlink in the docstring and the hyperlink is >80c by itself.
13:43:13 stephenfin You could manually wrap inside the docstring. It would make for uglier code but keep flake8 happy
13:43:19 stephenfin Otherwise, use a URL shortner?
13:43:22 efried backslash?
13:43:23 gtema efried: I mean in a normal sdk way (with resources and refular proxy methods)
13:43:34 gtema s/refular/regular/
13:43:53 efried gtema: I think we've talked about it before and decided it's not worth it, at least for now.
13:44:04 gtema ok, thks
13:44:17 efried gtema: It could be kinda neat, but the straight up APIs are pretty easy to use as is.
13:44:40 gtema that's good
13:44:59 efried Feel like if people have that kind of time, there's more important things to be done :)
13:45:13 efried like figuring out how to put long hyperlinks into the docstrings
13:45:15 gtema :)
13:46:54 efried stephenfin: backslash-wrapping works, and isn't *too* hideous. Thanks.
13:48:15 efried ahcrap. The pep8 error I was getting, it wasn't even in the docstring I was messing with.
13:48:40 mordred efried: BOW DOWN TO THE PEP8 GODS
13:51:36 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Link to baremetal API reference from patch_node https://review.opendev.org/659068
13:51:45 efried mordred, gtema, dtantsur: Toe in the water ^
13:52:03 efried if that flies, more can be done
13:54:18 mordred efried: ++
13:54:57 gtema efried: ++ from me as well
13:55:00 efried stephenfin: there was no pep8 error on long link in docstring. I was seeing things.
13:55:11 efried and it's not because we're ignoring H4 either.
13:55:31 efried Perhaps it's just smart enough to realize that a link doesn't count.
16:49:39 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
20:42:03 cdent mordred: 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.
#openstack-sdks - 2019-05-15
04:17:30 adriant mordred: when are we likely to get a 0.28.0 SDK release?
08:36:14 dtantsur adriant: it has been requested, waiting for the release team to catch up
09:01:45 mordred adriant: what dtantsur said
09:28:36 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
12:14:05 openstackgerrit Natal Ngétal proposed openstack/python-openstackclient master: Update sphinx requirement. https://review.opendev.org/659279
16:07:01 mordred cdent: did you get my tweet answer to your question?
16:08:00 cdent mordred: yes, thanks for that. I had also figured it out in the interim by doing some searching of my sent email. the initial struggle was not being sure how/what to search that wasn't going to return many results. In the end it was found by searching on the person it was sent to.
16:08:35 mordred \o/
16:08:41 mordred humans are the problem - and the solution
16:10:58 cdent amazing that...
16:20:21 openstackgerrit Merged openstack/openstacksdk master: Link to baremetal API reference from patch_node https://review.opendev.org/659068
17:23:00 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
19:26:02 openstackgerrit Raimund Hook (stingrayza) proposed openstack/openstacksdk master: Adding dns_domain parameter into create_network https://review.opendev.org/659366
20:55:35 mriedem dtroyer: how does one go about deprecation an option in osc? are there examples? is it just something in the help text of the option and dump something to the console if it's used?
21:02:04 dtroyer mriedem: deprecating options is mostly a manual bit, here is an example from volume transfer request: https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/volume_transfer_request.py#L69
21:03:10 dtroyer (for completeness) deprecating commands is more built-in, example from the old volume backup: https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/volume/v2/backup.py#L162
21:03:47 dtroyer That one specifically is how we renamed a command
21:14:29 mriedem ok i'll crank this out and see what i can do
22:04:02 openstackgerrit Matt Riedemann proposed openstack/python-openstackclient master: WIP: Deprecate openstack server migrate --host option https://review.opendev.org/659382
22:04:40 mriedem dtroyer: stephenfin: mordred: ok ^ is my stab at a fix for the live migration kerfuffle - needs tests but that's my idea anyway
22:18:23 mriedem i think https://review.opendev.org/#/c/460059/ can be abandoned now
22:49:05 openstackgerrit Merged openstack/python-openstackclient master: Stop leaving temp files after unit test runs https://review.opendev.org/658190
#openstack-sdks - 2019-05-16
00:35:30 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Volume https://review.opendev.org/659399
00:35:31 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Network https://review.opendev.org/659400
01:23:52 openstackgerrit Xuan Yandong proposed openstack-dev/os-http master: Change the author-email openstack-dev to openstack-discuss https://review.opendev.org/659403
01:43:45 openstackgerrit ZhouHeng proposed openstack/python-openstackclient master: openstack port create support --extra-dhcp-option https://review.opendev.org/657519
01:53:22 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Volume https://review.opendev.org/659399
01:53:22 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Network https://review.opendev.org/659400
01:53:23 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Compute https://review.opendev.org/659416
01:55:57 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Network https://review.opendev.org/659400
01:55:57 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Compute https://review.opendev.org/659416
01:55:58 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Volume https://review.opendev.org/659399
03:51:42 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Network https://review.opendev.org/659400
03:51:43 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Identity https://review.opendev.org/659430
03:51:43 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Volume https://review.opendev.org/659399
03:51:44 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Image https://review.opendev.org/659431
06:01:00 openstackgerrit zhangboye proposed openstack/openstacksdk master: Cap sphinx for py2 to match global requirements https://review.opendev.org/659442
11:09:23 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Image https://review.opendev.org/659431
11:09:23 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Identity https://review.opendev.org/659430
11:09:24 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Volume https://review.opendev.org/659399
11:09:24 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove deprecated stuff - Network https://review.opendev.org/659400
11:44:37 stingrayza hey folks. is anybody available to take a look at https://review.opendev.org/659366 ? I'd appreciate it greatly - would help me to get unstuck...
12:44:45 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Aggregate functional test tweak https://review.opendev.org/659552
16:00:06 edleafe The API-SIG Office Hour has begun. C'mon in, the water's fine!
16:00:12 elmiko \o/
16:01:23 elmiko lol
16:01:52 gtema well, it's end of Thursday here for me
16:08:41 dtantsur o/
16:08:58 dtantsur what about we merge some guidelines? ;)
16:09:12 elmiko sounds good to me, what did you have in mind?
16:09:24 dtantsur the two that we have frozen: version discovery and mDNS
16:09:32 gtema yeah, EACH SERVICE MUST DELIVER VER DISCOVERY DOCUMENT :D
16:09:47 dtantsur I haven't checked the comments on https://review.opendev.org/#/c/459405/ in depth though
16:10:15 dtantsur https://review.opendev.org/#/c/651222/ did not raise any objections in a month
16:10:51 elmiko the second one looks like it's ready for merge
16:11:01 edleafe I think we can merge https://review.opendev.org/#/c/651222/
16:11:08 edleafe Looks unanimous
16:11:08 elmiko although the first has no objections, seems like it would be easy to add morgan's suggestions
16:11:13 elmiko ++
16:11:20 edleafe Done
16:12:09 edleafe Yeah, we should do one more revision on https://review.opendev.org/#/c/459405. Once that's pushed, I can re-freeze it

Earlier   Later