| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-31 | |||
| 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 | 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:17 | jeremyfreudberg | ah, dtroyer, very good examples | |
| 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 | |
| 15:22:00 | mordred | by doing that, if we then called get_endpoint_data things would not work | |
| 15:22:11 | mordred | (that's easy to fix in shade and I've got that fix coming) | |
| 15:22:38 | efried | mordred Does this have impact on https://review.openstack.org/#/c/488137/13/nova/virt/ironic/client_wrapper.py ? | |
| 15:22:51 | mordred | which is all to say - I think, when we get around to it -it's not urgent - we should add support for that version of a versioned discovery document - just so that users don't do a thing and then get confused | |
| 15:23:22 | mordred | efried: no - I don't think it does | |
| 15:23:29 | efried | okay. | |
| 15:23:52 | mordred | efried: we might want to document that if people want to pass an ironic.endpoint_override in their config that it should be the unversioned, not the versioned endpoint | |
| 15:23:56 | mordred | I think that's what they'd do anyway | |
| 15:24:18 | mordred | but while other services would work with either, ksa will in fact blow up in the ironic case if a versioned endpoint is given | |
| 15:25:04 | efried | mordred So like here https://review.openstack.org/#/c/488137/13/nova/conf/ironic.py@43 the deprecation warning could mention that, but once we get rid of api_endpoint, how would we document it just for ironic? | |