| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-31 | |||
| 13:06:31 | edleafe | elmiko: I won't be around for today's API-SIG meeting. And cdent is in the air for much of today, so I don't know if he'll make it either. | |
| 13:30:47 | elmiko | edleafe: ack, i'll show up and see if we get any takers | |
| 15:48:11 | iwonka | hi guys, i'm an outreachy intern in sahara and as a part of my internship i've added new commands to openstack cli. the commands export some stuff to json, and there are doubts among sahara people whether the command that does it should print to stdout or take an argument with a name to write. thus question to you: what is the better option to do it? | |
| 16:06:45 | jeremyfreudberg | to build on what iwonka said, we have some API in sahara which returns some json, and that json doesn't just represent the result, the json **is** the result that the user wants to keep. therefore when creating an OSC command to use this API we don't use any standard osc formatter to represent the output. so we are wondering if there are any best practices regarding write to stdout, write to file, etc, in OSC | |
| 16:18:11 | elmiko | good question, maybe dtroyer has a quick answer ^^ | |
| 16:18:35 | elmiko | i mean, you can always redirect stdout =) | |
| 16:18:55 | elmiko | but having a --out option or something is nice also | |
| 16:19:20 | jeremyfreudberg | elmiko, sounds a lot like what we had been going back and forth about in the sahara channel | |
| 16:19:58 | jeremyfreudberg | if dtroyer could weigh in that would be nice, we know it's gonna break osc traditions no matter what we pick, so we just wanna make sure we do it right | |
| 16:20:45 | elmiko | +1 | |
| 16:20:48 | dtroyer | osc does have a -f json output format so we already have that, written to stdout. | |
| 16:21:02 | dtroyer | In this case, though, it doesn't sound like any other output formats make sense | |
| 16:21:39 | dtroyer | if the user is expecting JSON always, jsut write it to stdout. We do have a couple of comamnds that write output to files, like downloading objects or images | |
| 16:22:19 | dtroyer | image save —file <filename> | |
| 16:22:36 | dtroyer | (oops, there's an image nmae/id on there too) | |
| 16:23:15 | dtroyer | object save [—file <filename>] <object-name-or-id> | |
| 16:24:17 | jeremyfreudberg | ah, dtroyer, very good examples | |
| 16:24:17 | dtroyer | if you really want to have an option to specify a filename, I'd do it that way. OSC users will be used to redirecting stdout for a lot of things though, including those save sommands if that's where they want the images to go | |
| 16:24:54 | dtroyer | so to me, the inconsistency is in always outputting JSON, which isn't an issue if that is really the only thing the user wants to get out | |
| 16:25:45 | dtroyer | make sure including '-f json' is handled without error though | |
| 16:26:44 | jeremyfreudberg | dtroyer, if take_action returns None, that's ok right? | |
| 16:27:35 | dtroyer | yes, although it may depend on what kind of command class you inherit from to begin with | |
| 16:28:14 | jeremyfreudberg | osc_lib.command.command.Command | |
| 16:29:43 | dtroyer | there no return is expcted | |
| 16:30:03 | jeremyfreudberg | cool | |
| 16:30:39 | jeremyfreudberg | thanks for your help, i'll play around with it some... i'm leaning towards doing it like `openstack image save` with optional file that defaults to stdout | |
| 16:31:29 | dtroyer | what is the command verb? | |
| 16:32:04 | jeremyfreudberg | export | |
| 16:33:18 | dtroyer | ok, I think that makes sense. I wanted to make sure these semantics were not going to be surprising to users | |
| 16:34:14 | jeremyfreudberg | cool cool | |
| 16:34:31 | jeremyfreudberg | need to go afk now, if you have any sudden change of heart feel free to jump on the sahara channel later | |
| 16:34:34 | jeremyfreudberg | thanks a ton | |
| #openstack-sdks - 2017-09-01 | |||
| 03:16:47 | ldl___ | I can not install the openstacksdk in my Mac: https://stackoverflow.com/questions/45981213/error-no-available-formula-with-the-name-openstacksdk | |
| 03:17:09 | ldl___ | As you can? | |
| 12:44:44 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/500013 | |
| 12:44:50 | openstackgerrit | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/500014 | |
| 12:45:02 | openstackgerrit | OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/500015 | |
| 12:47:13 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/500030 | |
| 12:47:17 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/500031 | |
| 16:24:17 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Make discover._version_between more lenient https://review.openstack.org/485827 | |
| 16:32:01 | openstackgerrit | Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/500030 | |
| 16:40:57 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Make discover._version_between more consistent https://review.openstack.org/485827 | |
| 18:01:16 | cdent | elmiko, figleaf: did you guys skip the api-wg^sig meeting yesterday? | |
| 20:19:26 | openstackgerrit | Merged openstack/os-service-types master: Updated from global requirements https://review.openstack.org/500014 | |
| 20:38:24 | figleaf | cdent: we had a quick meeting. I made it back just in time | |
| 20:38:58 | cdent | figleaf: but not enough to warrant a newsletter, I guess? | |
| 20:39:52 | figleaf | cdent: you know you could read for yourself :) http://eavesdrop.openstack.org/meetings/api_sig/2017/api_sig.2017-08-31-16.03.log.html | |
| 20:40:14 | cdent | but then what would I talk to you about? | |
| 20:40:44 | cdent | figleaf: how’s the weather? wife? kids? | |
| 20:41:58 | figleaf | Why would you want to talk to me in the first place?? | |
| 20:42:17 | cdent | I hear you have the inside scoop on augmentation | |
| 20:43:51 | figleaf | heh - just got my bionic ears | |
| 20:44:00 | figleaf | everything is very LOUD | |
| 20:44:27 | cdent | do you have an app on your phone to control them? seems like that’s how it should be in these modern times | |
| 20:44:36 | cdent | IOT bionic ears | |
| 21:33:33 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Add tests for mutually exclusive [min|max]version https://review.openstack.org/500198 | |
| 21:55:24 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Sanely order assertion args in test_discovery https://review.openstack.org/500204 | |
| 22:01:04 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Extract iterables for test_keystone_version_data https://review.openstack.org/500206 | |
| 22:14:39 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Nits in using-sessions.rst https://review.openstack.org/500211 | |
| #openstack-sdks - 2017-09-02 | |||
| 12:12:58 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/500282 | |
| #openstack-sdks - 2017-09-03 | |||
| 09:22:35 | openstackgerrit | Merged openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/500031 | |
| 18:33:32 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: DNM Intentionally failing change https://review.openstack.org/500385 | |
| #openstack-sdks - 2017-09-04 | |||
| 04:16:35 | ldl | how to start develop a website to manage openstack like dashboard? | |
| 06:19:12 | openstackgerrit | Tytus Kurek proposed openstack/python-openstackclient master: Add support for "--dns-domain" argument https://review.openstack.org/500450 | |
| 07:08:37 | ldl | who can help me with this ? | |
| 07:08:38 | ldl | https://ask.openstack.org/en/question/110059/importerror-no-module-named-deprecation/ | |
| 07:14:14 | Dinesh_Bhor | ldl: You will need to install the required libraries. Please refer: https://github.com/openstack/python-openstacksdk/blob/master/requirements.txt | |
| 07:14:29 | ldl | okay, I will try | |
| 07:14:34 | ldl | thank you | |
| 07:15:54 | Dinesh_Bhor | ldl: Np | |
| 07:16:52 | ldl | do you mean use pip to install them on Mac? | |
| 07:17:58 | Dinesh_Bhor | ldl: not sure about Mac. Sorry | |
| 07:18:35 | ldl | and does the virtualenv helps? | |
| 07:19:31 | ldl | and windows ok? | |
| #openstack-sdks - 2017-09-05 | |||
| 02:55:23 | openstackgerrit | Reedip proposed openstack/python-openstacksdk master: Add support for dns-domain in Openstack SDK https://review.openstack.org/500660 | |
| 02:56:52 | reedip | stevemar: ^^ | |
| 03:00:54 | openstackgerrit | Reedip proposed openstack/python-openstacksdk master: Add support for dns-domain in Openstack SDK https://review.openstack.org/500660 | |
| 07:10:07 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: Introduce quota unset command https://review.openstack.org/376311 | |
| 08:21:49 | openstackgerrit | Tytus Kurek proposed openstack/python-openstackclient master: Add support for "--dns-domain" argument https://review.openstack.org/500450 | |
| 09:47:44 | openstackgerrit | Tytus Kurek proposed openstack/python-openstackclient master: Add support for "--dns-domain" argument https://review.openstack.org/500450 | |
| 12:54:31 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Add support for dns-domain https://review.openstack.org/500660 | |
| 13:29:53 | openstackgerrit | Merged openstack/service-types-authority master: Generate standard api_reference https://review.openstack.org/495887 | |
| 14:19:17 | efried | mordred What's the story with the automatic sta publisher? | |
| 15:00:30 | efried | mordred Seems to have kicked in now. | |
| 15:16:00 | mordred | efried: hey guess what? | |
| 15:16:25 | efried | mordred Tell me | |
| 15:16:28 | mordred | efried: this morning I have discovered that ironic version discovery returns something different and exciting on its versioned endpoint | |
| 15:16:59 | mordred | efried: please enjoy this: https://developer.openstack.org/api-ref/baremetal/#show-v1-api | |
| 15:17:22 | efried | ... | |
| 15:19:04 | efried | mordred I don't think I have an ironic service anywhere. Paste me? | |
| 15:19:22 | efried | oh | |
| 15:19:25 | mordred | efried: yah | |
| 15:19:43 | efried | well holy incompatibility, Batman. | |
| 15:20:01 | mordred | efried: I mean, the issue will never get hit in normal usage because people put the unversioned endpoint in the catalog | |
| 15:20:14 | efried | we hope | |
| 15:20:17 | mordred | and the unversioned document is compatible, so that's what ksa discovery will read and use | |
| 15:20:33 | mordred | efried: they have to - ironicclient hard-code appends a /v1 to what it gets from the catalog :) | |
| 15:20:55 | mordred | so any deployment with ironic not having unversioned endpoint in the catalog would not work with ironicclient | |
| 15:21:06 | efried | So.... what action needs to be taken here? And by whom? | |
| 15:21:41 | mordred | HOWEVER - we have code in shade to simulate what ironicclient does by doing get_endpoint() on our baremetal adapter, appending a '/v1' and then setting endpoint_override | |