| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-10-11 | |||
| 19:12:07 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: Add EndpointData.__str__ for debugging https://review.openstack.org/489413 | |
| 19:13:58 | openstackgerrit | Eric Fried proposed openstack/keystoneauth master: WIP: Return the endpoint_override from EndpointData https://review.openstack.org/491947 | |
| 19:56:29 | dtroyer | hyang: yes, you can put it in ~/.config/openstack/clouds.yaml | |
| 20:14:07 | enis_ | I'm trying to retrieve details about a specific project using it's name but `conn.identity.projects(name='pr-name')` returns all the projects even though a project by the specified name exists. Is there a different parameter I should be using? | |
| 20:15:48 | enis_ | Using conn.keystone.projects.list(name='pr-name') works but I'd like to use single sdk. | |
| #openstack-sdks - 2017-10-12 | |||
| 12:31:09 | elmiko | cdent: were you referring to the openstack-sigs ml? | |
| 12:31:15 | cdent | ya | |
| 12:31:20 | elmiko | ack | |
| 12:33:18 | cdent | elmiko: is not your bad, nor anybody’s bad | |
| 12:33:26 | cdent | just occured to me that we’ve got that list, may as well use it | |
| 12:37:23 | elmiko | you are correct though, we should be talking about it on that list | |
| 12:37:40 | elmiko | i certainly don't want to contribute to anything that would exclude apac folks | |
| 12:37:58 | elmiko | i can see that having the discussion at our meeting doesn't really help them =( | |
| 12:38:24 | elmiko | part of this too is that i have some baggage about the last apac meetings we held, i need to drop that | |
| 14:21:00 | elmiko | =) | |
| 14:40:42 | openstackgerrit | Merged openstack/keystoneauth master: Add EndpointData.__str__ for debugging https://review.openstack.org/489413 | |
| 16:27:35 | cdent | elmiko, edleafe : ping https://etherpad.openstack.org/p/api-sig-newsletter | |
| 16:27:52 | edleafe | looking... | |
| 16:29:31 | cdent | I’m not going to encourage anyone to get in a fight with elmiko, he looks dangerous | |
| 16:30:07 | edleafe | ok, lgtm | |
| 16:30:12 | cdent | thanks | |
| 16:31:05 | elmiko | lgtm | |
| 16:31:31 | elmiko | although i don't want to fight either, seems counterproductive | |
| 16:32:03 | elmiko | i really need to start pinging cdent about the lack of folksy dialog in those newsletters XD | |
| 16:32:12 | cdent | I deliberately did a bit of os-sigs advertising | |
| 16:32:21 | cdent | yeah, I’m not sure what’s up with that | |
| 16:32:49 | elmiko | seems appropriate to me (re: advertising) | |
| 18:07:43 | openstackgerrit | Ed Leafe proposed openstack/api-wg master: Change API-WG to API-SIG https://review.openstack.org/508242 | |
| 21:26:47 | thingee | dtroyer: do we have any documents on the osc user studies? I was noticing the ux wiki makes mention of it https://wiki.openstack.org/wiki/UX/ProgramProposal#User_Research | |
| 21:33:58 | dtroyer | I'm not sure if Piet sent the final docs to me, I know I probably have Google docs links to the drafts somewhere, if the docs still exist… will look | |
| 21:49:42 | dtroyer | thingee: found a doc from Barcelona, sent to gmail | |
| 22:14:25 | thingee | dtroyer: thanks. should I post it to the wiki for archive purposes? | |
| 22:15:58 | dtroyer | thingee: sounds good | |
| #openstack-sdks - 2017-10-13 | |||
| 02:44:21 | openstackgerrit | Lin Yang proposed openstack/python-openstackclient master: Add python-rsdclient into plugin list https://review.openstack.org/511683 | |
| 03:14:38 | openstackgerrit | Lin Yang proposed openstack/python-openstackclient master: Add python-rsdclient into plugin list https://review.openstack.org/511683 | |
| 09:28:30 | openstackgerrit | Duong Ha-Quang proposed openstack/cliff master: Move legacy jobs to project https://review.openstack.org/510418 | |
| 12:34:50 | kuzko | good day y'all | |
| 12:35:59 | kuzko | just a heads up mordred : delete_server asks for a name and ID and doesn't allow for passing python server objects, is it intended | |
| 12:42:21 | mordred | kuzko: hrm. that's strange - nope, it's not intended | |
| 12:44:03 | mordred | kuzko: I'll look in to that- btw - 1.24.0 should be released today with the other fixes in it | |
| 14:23:54 | openstackgerrit | Merged openstack/python-openstackclient master: Avoid default mutable values in arguments https://review.openstack.org/510876 | |
| 14:42:24 | openstackgerrit | Merged openstack/cliff master: Generate demoapp CLI refernece https://review.openstack.org/483134 | |
| 18:25:46 | openstackgerrit | Merged openstack/python-openstackclient master: Optimize getting endpoint list https://review.openstack.org/507265 | |
| 20:48:49 | mordred | dtroyer: I just found the funnest thing in the WORLD in glance REST api ... | |
| 20:49:18 | dtroyer | oh I need a laugh | |
| 20:49:29 | mordred | dtroyer: in pagination, next links include the API version - so you get "next": "/v2/images?marker=bb15732f-e016-4544-a7e9-c73db58ce3e5" | |
| 20:49:58 | dtroyer | wheeee!!!!!!!!! | |
| 20:50:02 | mordred | dtroyer: which would be fine - except that the base catalog url is usually 'https://images.example.com/v2' | |
| 20:50:14 | dtroyer | to be fair, pagination is a general mess almost everywhere | |
| 20:50:27 | mordred | so if youhave an Adapter on https://images.example.com/v2 and you follow the next link ... you get https://images.example.com/v2/v2/images?marker=bb15732f-e016-4544-a7e9-c73db58ce3e5 | |
| 20:50:30 | mordred | \o/ | |
| 20:50:35 | mordred | dtroyer: indeed | |
| 20:51:00 | dtroyer | but that one is special… | |
| 20:51:24 | mordred | yah. it's even internally inconsistent - that's what I really like about it | |
| 20:52:48 | mordred | I believe part of the issue is that glanceclient strips the version suffix from the catalog and stores the url internally as https://images.example.com - then makes all of its REST calls as 'GET /v2/images' ... so for glanceclient the dissonance doesn't surface | |
| 20:53:06 | mordred | dtroyer: anywho - I thought you'd enjoy that one :) | |
| 20:54:31 | dtroyer | glanceclient is, uh, well you know… | |
| 20:56:27 | cdent | mordred: is that a problem with the next link or the adapter? | |
| 20:56:42 | cdent | presumably the adapter should see an absolute link as an absolute link? | |
| 20:57:02 | cdent | I have a similar problem when dealing prefixes in gabbi, it’s a pain in the ass | |
| 20:57:05 | mordred | cdent: well... it's not always that simple | |
| 20:57:06 | cdent | there’s no great solution | |
| 20:57:12 | mordred | cdent: bcause of suburl services | |
| 20:57:41 | cdent | right, I know, that’s what I’m saying: adapters (not just ksa) need to have more brains | |
| 20:57:48 | cdent | which is a pain | |
| 20:57:50 | mordred | yah | |
| 20:58:09 | mordred | I'd say the bug is in the next link - because it should EITHER be 'images?marker=bb15732f-e016-4544-a7e9-c73db58ce3e5' | |
| 20:58:19 | mordred | in which case the correct behavior is easy to figure out | |
| 20:58:32 | mordred | or it shold be 'https://images.example.com/v2/images?marker=bb15732f-e016-4544-a7e9-c73db58ce3e5' | |
| 20:58:36 | mordred | in which case same thing | |
| 20:58:46 | cdent | would you accept ///images? | |
| 20:58:54 | cdent | or is it // (can’t remember) | |
| 20:59:25 | mordred | yah - ksa adapter would do the right thing with that ... but i'm not sure it's correct for glance to return that - because glance might or might not be deployed at a url root | |
| 20:59:45 | cdent | sorry, that wasn’t I mean | |
| 20:59:46 | cdent | t | |
| 21:00:22 | cdent | /v2/images[…] or ///v2/images[…] (whichever is actually the magic fully qualify me as in the current context) | |
| 21:00:51 | mordred | yah- so ... we currently treat /v2/images as 'fully qualify me in the current context' ... | |
| 21:01:13 | cdent | I think the relative is probably better in the long run, but *shrug* | |
| 21:01:15 | mordred | it's just that the current context in this case is https://images.example.com/v2 - which is why it appends like that | |
| 21:03:44 | cdent | sorry, I’m talking a browser thing where // (or /// can’t remember) means off root of whatever the scheme and host is of “this page” | |
| 21:03:49 | mordred | so - technically I think we're doing that join wrong | |
| 21:03:54 | cdent | yes | |
| 21:04:12 | mordred | >>> urllib.parse.urljoin('https://images.example.com/images/v2', '/v2/images') | |
| 21:04:14 | mordred | 'https://images.example.com/v2/images' | |
| 21:04:16 | mordred | >>> urllib.parse.urljoin('https://images.example.com/images/v2', '//v2/images') | |
| 21:04:18 | mordred | 'https://v2/images' | |
| 21:04:36 | cdent | yeah, I suspect urllib doesn’t speak browser magic | |
| 21:05:11 | mordred | yah - mainly - next links returning rooted/absolute urls is a thing we shold not do | |
| 21:05:19 | cdent | yah | |
| 21:05:42 | mordred | since that makes suburl deployments and version discovery roots both very ... confusing ... to deal with | |
| 21:06:04 | mordred | whereas if it was just 'next': 'images?marker=bb15732f-e016-4544-a7e9-c73db58ce3e5' | |
| 21:06:21 | mordred | then the joins will all work properly no matter what the setup is - and it's not unclear what it's relative to | |
| 21:06:30 | cdent | yah | |
| 21:07:34 | cdent | such typing. wow. | |
| 21:08:41 | cdent | I think I should exit this establishment in favor of some relaxation. Good night all. | |
| 22:10:52 | openstackgerrit | Dean Troyer proposed openstack/cliff master: Add a --errexit option to exit the comamnd loop on command errors https://review.openstack.org/511971 | |
| 22:19:53 | openstackgerrit | Merged openstack/cliff master: doc: Cleanup of demoapp doc https://review.openstack.org/510802 | |
| #openstack-sdks - 2017-10-15 | |||
| 20:43:22 | openstackgerrit | Daniel Speichert proposed openstack/python-openstackclient master: Allow creating security rules without protocol https://review.openstack.org/512113 | |
| 23:49:01 | openstackgerrit | Ian Wienand proposed openstack/python-openstackclient master: Switch to $USER in post_test_hooks https://review.openstack.org/512125 | |
| 23:50:31 | openstackgerrit | Ian Wienand proposed openstack/python-openstackclient master: Switch to $USER in post_test_hooks https://review.openstack.org/512125 | |