| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-13 | |||
| 17:01:03 | ankur-gupta-f4 | +1 | |
| 17:01:48 | reedip | ok, gotta go now .. | |
| 17:03:14 | rabel | thanks | |
| 17:29:25 | cdent | edleafe, elmiko did either of you have thoughts on this message http://lists.openstack.org/pipermail/openstack-dev/2017-March/113689.html ? I haven't had a chance to really think about it yet, but figure it deserves a response from one of us | |
| 17:34:26 | edleafe | cdent: I think Nova can do something like this, although I'm not sure how far along it is. sdague and alex_xu would be better resources | |
| 17:36:36 | sdague | I'm... kind of confused what is being asked for | |
| 17:37:15 | cdent | sdague: yes, me too, but I was struggling to tease out enough to even start asking questions | |
| 17:37:27 | cdent | I'll give it a shot a bit later in the day. | |
| 17:37:47 | sdague | yeh, I feel like this might be the "I wish this was wsdl" comment | |
| 17:38:07 | cdent | I just wanted to see if made sense to anyone. Since it appears not to, then I'll respond with a "huh" and some clarifying questions | |
| 17:38:19 | edleafe | sdague: if I understand correctly, they want a way for every API to generate its schema, including available resources, body schemas, etc | |
| 17:38:28 | sdague | edleafe: then what? | |
| 17:38:36 | edleafe | sdague: yeh, pretty much wsdl | |
| 17:38:40 | sdague | a schema is just an interchange format | |
| 17:38:53 | cdent | sdague: then profit, of course | |
| 17:38:55 | sdague | so until/unless you have an equally complicated set of tools consuming it | |
| 17:39:20 | sdague | it does nothing but makes you look important | |
| 17:39:20 | edleafe | sdague: exactly. WSDL lives! | |
| 17:39:40 | sdague | so, it's fine if people would rather that the OpenStack API was WSDL, or gRPC | |
| 17:39:44 | sdague | but, it's not | |
| 17:40:10 | sdague | and trying to bolt that into the existing REST stuff is just going to be being completely custom ecosystem with no tooling | |
| 17:41:20 | sdague | which was actually the same issue around OpenAPI / Swagger. It's all well and good if you stay 100% inside the ecosystem tooling, but the moment you diverge, all you've done is signed up for writing a schema -> code compiler in 10 different languages | |
| 17:41:50 | sdague | and, given that we don't even have current 80% client API implementations in that many languages, building a compiler for them all seems... unlikely | |
| 17:42:21 | sdague | anyway... /rant | |
| 17:43:47 | cdent | edleafe: if you're done with that gold star I gave you earlier, you might give it to sdague now | |
| 17:43:53 | cdent | but only if you are done | |
| 17:44:00 | edleafe | never! | |
| 17:45:19 | cdent | fair | |
| 18:01:48 | openstackgerrit | Sean Dague proposed openstack/api-wg master: WIP: microversion architecture archival doc https://review.openstack.org/444892 | |
| 18:37:02 | openstackgerrit | Clark Boylan proposed openstack/os-client-config master: OVH supports qcow2 https://review.openstack.org/445042 | |
| 18:39:12 | openstackgerrit | Dean Troyer proposed openstack/cliff master: Fix broken test with cmd2 0.7.0 https://review.openstack.org/444951 | |
| 18:39:44 | dtroyer | stevemar: ^^^ will unblock cliff's queue if you have a minute | |
| 20:40:08 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Add --network and --port to server create https://review.openstack.org/444924 | |
| 20:50:19 | openstackgerrit | Merged openstack/python-openstacksdk master: Add image download example https://review.openstack.org/444940 | |
| 21:05:40 | openstackgerrit | Merged openstack/os-client-config master: OVH supports qcow2 https://review.openstack.org/445042 | |
| 21:06:01 | openstackgerrit | Merged openstack/cliff master: Fix broken test with cmd2 0.7.0 https://review.openstack.org/444951 | |
| 21:08:34 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add new parameter "is_default" to Network QoS policy. https://review.openstack.org/432260 | |
| 21:09:38 | openstackgerrit | Steve Martinelli proposed openstack/cliff master: Remove support for py34 https://review.openstack.org/430577 | |
| 23:26:44 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Non-Admin can't list own projects https://review.openstack.org/376056 | |
| 23:39:40 | reedip | o/ | |
| 23:48:30 | openstackgerrit | Merged openstack/cliff master: Remove support for py34 https://review.openstack.org/430577 | |
| #openstack-sdks - 2017-03-14 | |||
| 00:14:41 | openstackgerrit | Merged openstack/cliff master: Updated from global requirements https://review.openstack.org/440033 | |
| 00:41:30 | openstackgerrit | Colleen Murphy proposed openstack/python-openstackclient master: Narrow expected responses for CheckUserInGroup https://review.openstack.org/445257 | |
| 01:08:58 | stevemar | dtroyer: glance tests are borked | |
| 01:15:21 | stevemar | dtroyer: looks like we're setting IMAGE_API to 1: https://github.com/openstack/python-openstackclient/blob/master/openstackclient/tests/functional/image/v1/test_image.py#L29 | |
| 01:15:31 | stevemar | 21:10 stevemar: sample failure: http://logs.openstack.org/20/437320/2/gate/gate-osc-dsvm-functional-ubuntu-xenial/8c6e99b/testr_results.html.gz | |
| 01:15:31 | stevemar | 21:11 stevemar: Command 'openstack image create 2c10175aea9041aaabd5b2de90285e76 -f value -c name' returned non-zero exit status 1. | |
| 01:15:31 | stevemar | 21:11 stevemar: HTTPMultipleChoices (HTTP 300) Requested version of OpenStack Images API is not available. | |
| 01:16:00 | stevemar | dtroyer: ah its all mriedem's fault | |
| 01:16:02 | stevemar | https://github.com/openstack-dev/devstack/commit/80e82eac4cd127a68fceea37270d09f9cbd71c75 | |
| 01:29:36 | ankur-gupta-f4 | stevemar: Why keep v1 around if deprecated since Newton? | |
| 01:39:42 | stevemar | ankur-gupta-f4: you shouldn't be removing APIs, like ever | |
| 01:40:19 | stevemar | ankur-gupta-f4: the change in devstack made it so image v1 isn't in the pipeline by default (because it is, as you said, deprecated) | |
| 01:40:31 | stevemar | ankur-gupta-f4: our tests assume its there, we'll have to change that | |
| 01:41:43 | openstackgerrit | Lv Jiawei proposed openstack/python-openstackclient master: Add extra filtering options to qos policy list https://review.openstack.org/401131 | |
| 01:43:40 | ankur-gupta-f4 | stevemar: i meant the tests :| | |
| 01:46:06 | ankur-gupta-f4 | Would you change the tests or would you enable glance v1 in the test env? | |
| 02:16:40 | reedip | stevemar : hi , any comments on https://review.openstack.org/#/c/383025/ ? | |
| 02:16:56 | reedip | ankur-gupta-f4 : https://review.openstack.org/#/c/376311/ ???? | |
| 02:20:29 | openstackgerrit | jiahui.qiang proposed openstack/python-openstackclient master: Functional test for subnet_pool https://review.openstack.org/440150 | |
| 02:44:38 | stevemar | ankur-gupta-f4: probably just skip the tests IMO | |
| 02:45:03 | stevemar | ankur-gupta-f4: it would be nice to check for that exception and skip the test | |
| 02:47:39 | reedip | stevemar : hello :) | |
| 02:47:50 | stevemar | reedip: mr reedip | |
| 02:47:58 | reedip | ?? | |
| 02:50:01 | openstackgerrit | luqitao proposed openstack/osc-lib master: Replace six.iteritems() with .items() https://review.openstack.org/445273 | |
| 02:51:31 | reedip | ankur-gupta-f4, stevemar : How does OpenstackClient search for new Commands | |
| 02:51:45 | reedip | ankur-gupta-f4, stevemar : even if they are registered in tox.ini | |
| 02:57:34 | stevemar | reedip: you mean setup.cfg :) | |
| 02:57:44 | reedip | sorry , yeah stevemar | |
| 02:57:48 | reedip | except setup.cfg | |
| 02:57:59 | stevemar | reedip: you have to add it to setup.cfg and 'install' it | |
| 02:58:14 | stevemar | i normally create a new virtualenv by doing $ virtualenv testcmd | |
| 02:58:26 | stevemar | source it, $ source testcmd/bin/activate | |
| 02:58:34 | stevemar | go to my OSC directory | |
| 02:58:41 | stevemar | edit setup.cfg for the new command | |
| 02:58:52 | stevemar | and 'install' osc locally in my virtualenv | |
| 02:59:02 | stevemar | $ pip install -e . | |
| 02:59:10 | stevemar | ^ from OSC root directory | |
| 02:59:30 | stevemar | when you run $ openstack --version you'll see a .dev version installed | |
| 02:59:40 | stevemar | that should have the new command | |
| 02:59:50 | stevemar | you can run $ which openstack and see where it points also | |
| 03:00:04 | stevemar | when you're done with the virtualenv just run, $ deactivate | |
| 03:00:15 | stevemar | you can then rm -rf the virtualenv directory | |
| 03:01:42 | reedip | stevemar : the command is not in OSC , its in a separate repo , which extends OSC plugins | |
| 03:02:16 | stevemar | reedip: same principals i believe, install OSC in the virtualenv then the other repo | |
| 03:03:28 | reedip | lemme try | |
| 03:06:29 | reedip | stevemar : what if requirements are not met :) | |
| 03:09:37 | reedip | stevemar : getting an error | |
| 03:09:42 | reedip | in requirements | |
| 04:13:23 | ankur-gupta-f4 | reedip: it should work in the same way | |
| 04:13:37 | ankur-gupta-f4 | R u talking about the TaaS commands? | |
| 04:23:47 | rabel__b1 | can someone help me with https://review.openstack.org/#/c/444924/ ? two functional tests in the gate job are failing, but it seems to have nothing to do with the change. one of the test i also saw failing for other changes that seem to be unrelated. | |
| 04:24:31 | ankur-gupta-f4 | rabel__b1: the image tests are temporarily broken. | |
| 04:25:00 | ankur-gupta-f4 | Till its resolved avoid rechecking | |
| 04:36:49 | rabel__b1 | ankur-gupta-f4: ok, thank you. | |
| 04:52:14 | reedip | ankur-gupta-f4 : all working :) dont worry .. I worked around stevemar's solution :D | |
| 04:52:53 | ankur-gupta-f4 | K | |
| 05:04:59 | openstackgerrit | Lv Jiawei proposed openstack/python-openstackclient master: Add extra filtering options to qos policy list https://review.openstack.org/401131 | |
| 08:23:29 | openstackgerrit | Colleen Murphy proposed openstack/python-openstackclient master: Narrow expected responses for CheckUserInGroup https://review.openstack.org/445257 | |