Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-15
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
16:12:46 dtantsur I can do it.. one day..
16:12:54 dtantsur if somebody has free time (LOL), go ahead with the updates
16:13:15 edleafe dtantsur: Weren't you just saying you wanted something to do with all your free time?
16:13:38 dtantsur I'm enjoying fixing ironic CI 24/7 :(
16:14:27 openstackgerrit Colleen Murphy proposed openstack/api-sig master: Update liaison for keystone https://review.opendev.org/659613
16:15:57 elmiko ouch
16:19:28 dtantsur speaking of which, I'm exhausted, so I'll call it a day a bit earlier.. see you
16:19:43 elmiko rest easy dmitry o/
16:20:19 dtantsur|afk I have to move some furniture today, otherwise I will :D
16:22:57 elmiko yikes!
16:25:40 edleafe I would update, but I really don't know the answer to the formatting question on https://review.opendev.org/#/c/459405/19/guidelines/consuming-catalog.rst@293
16:26:01 edleafe too many levels :)
16:27:46 elmiko hmm, that's a good question and i'm not sure the correct answer
16:31:18 edleafe We can wait for when dtantsur|afk has copious free time.
16:31:40 elmiko i wonder if mordred might comment there too, he certainly understands the intent better than most
16:32:03 edleafe I was going to suggest that, but was afraid to summon the spirit of mordred
16:32:14 elmiko i mean, by the reading of the rule it seems like morgan's assumption is correct re the v3 v3.6 example
16:32:17 elmiko haha, fair
16:33:07 edleafe Yeah, but I don't trust my reading of that rule. Maybe that's a sign that a bit more explanation is needed
16:33:28 elmiko i share your concerns, i felt very similar
16:34:14 elmiko i guess i will add that to the comments, at the least it helps expose an area where more clarity is needed
16:58:53 mordred elmiko: what did I do?
17:01:13 elmiko mordred: oh nothing in specific, morgan had brought up a question on the version discovery pr and we thought you might be able to shed some light
17:01:21 mordred oh jeez. what a fun edgecase
17:01:27 elmiko exactly
17:01:35 mordred my first response is "don't do that"
17:01:42 elmiko i just don't know the problem space well enough to provide a solid answer
17:01:45 elmiko lol
17:02:08 mordred I don't know that I've ever encountered a vX.Y url with a service that doesn't have a working discovery document
17:02:36 elmiko any guidance, or opinion, you could share on the pr would help =)
17:02:53 mordred so while we do account for crazy things - most of them are tied to crazy things people have actually done - not crazy things people might have done
17:02:58 mordred I will leave a comment
17:03:15 elmiko yeah, and the speculation business is a dangerous one
17:03:17 elmiko thanks
17:03:20 mordred ++
17:11:40 dustinc dtantsur|afk or anyone else able to answer a question about node.patch for baremetal...the ironicclient equivalent would raise an exception if a node is locked when you try to patch it, but the docstring for it does not how any exceptions. I dug in a bit and did not see any possible exception sources (other than parent class, but only if patch is not enabled, so not relevant for node) can someone confirm?
17:11:52 dustinc implemented here: https://review.opendev.org/#/c/647730/
17:12:13 dustinc **the docstring for sdk**
19:09:56 openstackgerrit Merged openstack/python-openstackclient master: Update sphinx requirement. https://review.opendev.org/659279
#openstack-sdks - 2019-05-17
00:20:52 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: Remove code migrated to osc-lib long ago https://review.opendev.org/659693
04:35:35 openstackgerrit Merged openstack/python-openstackclient master: Aggregate functional test tweak https://review.opendev.org/659552

Earlier   Later