| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-11-05 | |||
| 20:25:19 | mordred | that way we get the extra coverage because senlin is an sdk consumer - not only the coverage of testing using sdk to talk to senlin | |
| 22:53:21 | openstackgerrit | Merged openstack/openstacksdk master: Document "insecure" flag https://review.openstack.org/615512 | |
| 23:14:44 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| #openstack-sdks - 2018-11-06 | |||
| 00:58:06 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add wait functions to orchestration proxy https://review.openstack.org/615702 | |
| 01:00:54 | dtruong | mordred: ^ This change adds back the functions that senlin is calling. | |
| 02:27:10 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 06:58:51 | openstackgerrit | Shilpa Devharakar proposed openstack/openstacksdk master: Add functional tests for masakari https://review.openstack.org/615744 | |
| 08:53:33 | openstackgerrit | Nguyen Hai Truong proposed openstack-infra/shade master: Add python 3.6 unit test job https://review.openstack.org/615783 | |
| 11:57:27 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: Add py36 tox environment https://review.openstack.org/615845 | |
| 12:23:55 | openstackgerrit | Sven Wegener proposed openstack/python-openstackclient master: image/v2: support multiple property filters https://review.openstack.org/615850 | |
| 13:27:11 | dtantsur | mordred: hi! how do I use baremetal_endpoint_override in clouds.yaml? a naive approach does not seem to work.. | |
| 13:35:42 | dtantsur | nvm, I'm probably being stupid | |
| 13:38:52 | mordred | dtantsur: you put it in the clouds.yaml :) | |
| 13:39:13 | dtantsur | right, I probably screwed up a thing in metalsmith | |
| 13:41:17 | mordred | it's also entirely possible there is a bug | |
| 13:41:25 | mordred | dtruong: awesome - and yay, a test failure! :) | |
| 13:48:07 | dtantsur | mordred: in case you're curious: https://review.openstack.org/#/c/615878/ | |
| 13:51:02 | mordred | I'm always curious | |
| 13:51:56 | mordred | ah - interesting | |
| 13:53:08 | mordred | dtantsur: out of more curiousity - why does _os_api.API take both a session/adapter and a Connection ... is that just hysterical raisins? | |
| 13:53:58 | mordred | ah ... because it also uses ironic_client | |
| 13:54:49 | mordred | I'm glad passing that adapter to the ironic client works! | |
| 14:06:59 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 14:09:54 | dtantsur | mordred: yep, it's to make ironicclient happy (I cannot find time to get rid of it) | |
| 14:10:53 | mordred | dtantsur: eventually | |
| 14:12:08 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 14:28:46 | dtantsur | mordred: just learned: https://github.com/openstack/openstacksdk/blob/master/openstack/service_description.py#L154 can return None, causing the following code to fail | |
| 14:30:40 | mordred | dtantsur: oh - right. poo. I think just line 161 should be if data and (data.catalog_url != data.service_url): ? | |
| 14:31:17 | mordred | hrm. no - I htink you're right - more understanding is needed | |
| 14:31:28 | dtantsur | mordred: in my case it happened when ironic was down and baremetal_endpoint_override was set | |
| 14:31:57 | mordred | ah - yeah - so in that case, the None is an indication that no service was found | |
| 14:33:04 | Shrews | such a common bug | |
| 14:33:12 | mordred | so - I think the above if data would then result in an adapter with the endpoint_override - which would then throw a network connection error on use? | |
| 14:33:30 | mordred | what a great edge case! | |
| 14:33:41 | dtantsur | \o/ | |
| 14:34:40 | mordred | dtantsur: ++ | |
| 14:34:49 | Shrews | swift actually handles this very well, too | |
| 14:35:17 | Shrews | yeah yeah, i know | |
| 14:35:27 | Shrews | i was curious | |
| 14:35:49 | Shrews | and pleasantly surprised, tbh | |
| 14:36:44 | mordred | weird | |
| 14:37:33 | Shrews | "swift" being the language, not the object store thingy (if that wasn't clear) | |
| 14:38:28 | mordred | yeah. I mean - I'm sure swift the object store would also handle this well | |
| 14:38:49 | mordred | but that's largely just becaues timburke finds all the bugs | |
| 14:53:52 | Shrews | mordred: this is unrelated to your change, yeah? http://logs.openstack.org/32/614832/3/check/osc-functional-devstack-tips/aa1ce44/job-output.txt.gz#_2018-11-04_15_03_56_197747 | |
| 14:54:12 | Shrews | i've never seen that failure before | |
| 15:58:50 | mordred | Shrews: yeah - I think that's a rando | |
| 16:01:57 | Shrews | mordred: k k | |
| 16:45:37 | mordred | Shrews: replied on https://review.openstack.org/#/c/614834 - good point about testing though - I'll go lift that from nodepool | |
| 16:46:58 | Shrews | mordred: gotcha. can live w/o the depends | |
| 16:47:10 | mordred | Shrews: I could also totes go the other way :) | |
| 16:49:08 | Shrews | mordred: let's not add it until we *need* to | |
| 16:49:15 | mordred | ++ | |
| 17:27:43 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 20:18:23 | openstackgerrit | melanie witt proposed openstack/python-openstackclient master: Handle multiple ports in AddFloatingIP https://review.openstack.org/615353 | |
| 21:21:11 | openstack | Launchpad bug 1802001 in Zun "please add documentation and example for python-zunclient API" [Undecided,New] | |
| 21:21:11 | hongbin | hi, i am traiging this bug: https://bugs.launchpad.net/zun/+bug/1802001 , and wondering why the user is trying to consume zun-api via openstacksdk, does openstacksdk supports zun? | |
| 21:35:10 | mordred | hongbin: we don't REALLY support it - you can get a configured REST client for any openstack service in openstacksdk | |
| 21:35:43 | mordred | hongbin: so conn.application_container.get('/containers') should work and return a requests.response.Response | |
| 21:36:05 | mordred | hongbin: but we do not currently have any support for zun in the object layer | |
| 21:36:41 | mordred | hongbin: of course, support for zun is totaly _welcome_ :) | |
| 21:36:55 | hongbin | mordred: i see, then to be exact, what is the difference between the confiugred REST client and the support in object layer? | |
| 21:37:31 | hongbin | any pointer to a docs would be enough :) | |
| 21:38:30 | mordred | hongbin: the wrapper layer makes higher level objects - so for example, for neutron, there is http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/_proxy.py | |
| 21:39:09 | mordred | hongbin: which provides things liek "create_network()" and "networks()" ... and then each resource that is managed gets a Resouce object | |
| 21:39:18 | mordred | like http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/network.py | |
| 21:39:39 | hongbin | i see | |
| 21:40:11 | mordred | https://docs.openstack.org/openstacksdk/latest/user/guides/network.html has some examples of using that | |
| 21:40:42 | hongbin | mordred: everything is clear now, thank for the information | |
| 21:41:03 | mordred | sure thing! let me know if I can be helpful with anything | |
| 21:41:59 | hongbin | thanks, will ping you know if anything else comes up | |
| #openstack-sdks - 2018-11-07 | |||
| 05:06:09 | openstackgerrit | Ian Wienand proposed openstack/openstacksdk master: Add a __main__ handler, version command https://review.openstack.org/616070 | |
| 13:41:06 | openstackgerrit | Merged openstack/openstacksdk master: Test python2 with py27 https://review.openstack.org/615543 | |
| 13:56:53 | openstackgerrit | Sven Wegener proposed openstack/python-openstackclient master: image/v2: support multiple property filters https://review.openstack.org/615850 | |
| 14:33:50 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add wait functions to orchestration proxy https://review.openstack.org/615702 | |
| 15:41:07 | openstackgerrit | Merged openstack/openstacksdk master: [Trivial Fix] Correct spelling error of "bandwidth" https://review.openstack.org/615421 | |
| 16:03:07 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Slightly refactor vendor profile loading https://review.openstack.org/616227 | |
| 16:03:08 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Support remote vendor profiles https://review.openstack.org/616228 | |
| 16:03:21 | mordred | mnaser: ^^ there's a first-stab at that idea | |
| 16:07:51 | mordred | mnaser: it works for me against vexxhost in local testing, so I think it's ok. needs tests and docs | |
| 17:27:05 | openstackgerrit | Nguyen Hai Truong proposed openstack/openstacksdk master: Add coverage job https://review.openstack.org/616257 | |
| 17:38:27 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Slightly refactor vendor profile loading https://review.openstack.org/616227 | |
| 17:38:28 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: WIP Support remote vendor profiles https://review.openstack.org/616228 | |
| 18:29:11 | samueldmq | oops | |
| 19:19:20 | mordred | cdent, edleafe, elmiko: "{scheme}://{netloc}/.well-known/openstack/client" ... in terms of a well-known entry (like from the mailing list) ... do we like that? or would .well-known/openstack/api (or something else) be better? | |
| 19:21:54 | cdent | mordred: I fergit, was client a directory or a filename? I'd be vaguely inclined in the api direction either way if only because I think "client" implies a certain style of talking to the api | |
| 19:23:52 | cdent | and i'd be inclined in not-a-directory | |
| 20:05:32 | mordred | cdent: it's a filename | |
| 20:05:56 | mordred | cdent: and yeah - I think that's what I'm thinking re: api vs. client | |
| 20:06:12 | mordred | the json contents returned from that url contain informatoin _about_ the openstack api | |
| 20:06:41 | cdent | yes | |
| 20:08:07 | edleafe | I concur with api | |
| 20:09:43 | mordred | mnaser: if you get a sec, would you mind moving that test file to .well-known/openstack/api instead of .well-known/openstack/client ? | |
| 20:10:19 | mordred | mnaser: also - thank you for putting it there - it made verifying the new consumption code very nice | |
| 20:20:48 | openstackgerrit | Merged openstack/openstacksdk master: Shift swift segment async code out of adapter https://review.openstack.org/614832 | |
| 21:00:29 | openstackgerrit | Merged openstack/openstacksdk master: Remove unused Task classes https://review.openstack.org/614833 | |
| 21:12:09 | elmiko | mordred: sorry, just saw the ping | |
| 21:12:28 | elmiko | i agree with edleafe and cdent though (no surprise) | |
| 21:16:05 | mordred | elmiko: \o/ | |
| 21:16:33 | cdent | elmiko: I'm sorry to report we _still_ haven't used that syrup. We keep saying we'll make pancakes and then forget. But we are very pleased to see it in the cupboard every time we look. | |