| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-08-31 | |||
| 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 | |
| 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? | |
| 15:26:25 | efried | I suppose we could update the document for the imported ksa opts - because in any case I imagine you would recommend using an unversioned endpoint_override and specifying the version via the `version` conf opt. | |
| 15:27:34 | openstackgerrit | Merged openstack/keystoneauth master: Make discover._version_between more consistent https://review.openstack.org/485827 | |
| 15:30:56 | mordred | efried: yes - I mean, that's what I'm actually going to do in shade now - just do the darned discovery | |
| 15:31:37 | mordred | efried: however, there is only one major version of ironic, so some folks might desire to put in an override in their config of the versioned endpoint to cause the discovery step to be skipped | |