| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-02-25 | |||
| 21:29:47 | adriant | SamYaple: Yeah, I think it needs to be done. I'd put my hand up to do it, but I doubt I'd get the time to do it any time soon :( | |
| 21:43:26 | SamYaple | i definetely cant lead the work at this time, but someone could put me down to do some of it if we wanted to split it up | |
| 22:25:45 | adriant | SamYaple: I'm at least going to try and do Nova in the next week or two. | |
| #openstack-sdks - 2018-02-26 | |||
| 08:58:13 | mordred | adriant: heya - sorry, was travelling - reading scrollback now | |
| 08:58:30 | adriant | mordred: you didn't miss much | |
| 09:00:45 | mordred | ah - yes - *definitely* need more query params from api-ref | |
| 09:01:00 | mordred | the ruby folks have been actually parsing the api-ref and generating code from it ... | |
| 09:01:19 | mordred | perhaps we should write a script or something to do a similar thing and update all of the resources ... | |
| 09:01:45 | mordred | adriant: "extra query values on list and get" - what's what? | |
| 09:04:10 | adriant | mordred: could do, just a tool to parse and compare what we have an what is expected. | |
| 09:04:28 | adriant | mordred: also, not quite parsing your last comment there :P What do you mean? | |
| 09:10:04 | mordred | adriant: oh, you said "oh, ok, so yes those servers and search paths for hypervisors are deprecated, and now you can do that with extra query values on list and get. which we don't have in the SDK yet." | |
| 09:10:44 | adriant | oh, shit ok let me now parse what the hell I was saying with that | |
| 09:10:53 | mordred | adriant: OH! - wait | |
| 09:10:55 | mordred | I think I get it | |
| 09:11:49 | adriant | me too... yeah, if you do /hypervisors?servers=true you get the servers on each hypervisor | |
| 09:11:52 | mordred | there was /os-hypervisors/{hypervisor}/servers is deprecated and | |
| 09:11:54 | mordred | yah | |
| 09:12:12 | mordred | this is actually a good example of a microversion difference | |
| 09:12:17 | adriant | rather than /hypervisor/<id>/servers or whatever | |
| 09:12:37 | mordred | for us to figure out how to express in a way that isn't *super* crazy | |
| 09:14:58 | adriant | does the sdk do any microversion stuff? | |
| 09:15:05 | adriant | I didn't think it did yet | |
| 09:15:17 | adriant | I think the nova client kind of does | |
| 09:20:50 | mordred | oh - novaclient definitely does - it defaults to latest microversion when you use nova cli | |
| 09:21:32 | mordred | sdk doesn't do them yet on any proxy calls - but the underlying Proxy/Adapter class supports them, so we can pass them in impl code when we need to | |
| 09:25:44 | mordred | this thing seems like a good first candidate to add - that and I wanna add 'get-me-a-network' support (in get-me-a-network I think sdk should emulate it for older clouds that donm' thave that microversion | |
| 09:37:31 | mordred | adriant: can you pass ?hypervisor_hostname_pattern and ?with_servers before 2.53 and 2.53 just deprecates the /hypervisor/<id>/servers ? or it's two different ways to do the same thing I'm guessing | |
| 09:39:27 | adriant | mordred: I'm not really sure, I know dug into that because some code I had to rewrite was doing the older calls (against an old version of nova) and I had to make it marginally less awful | |
| 09:39:46 | adriant | I now* dug into | |
| 09:40:03 | adriant | it looks like different ways of doing the same thing | |
| 09:40:52 | mordred | nod | |
| 09:45:19 | mordred | adriant: maybe we should add a list_hypervisors_by_server or something, so we can do "if microversion_2.53_available: return self.get('/os-hypervisors/foo/servers') else: return self.hypervisors(with_servers=foo) | |
| 09:45:39 | adriant | mordred: that would work | |
| 09:46:12 | mordred | adriant: alternately, we could just look to see if hypervisor_hostname_pattern and with_servers have been given to hypervisors() and do the logic there | |
| 09:46:59 | adriant | mordred: https://github.com/openstack/python-novaclient/blob/master/novaclient/v2/hypervisors.py#L86 | |
| 09:47:28 | adriant | ^ your first suggestion is pretty much how novaclient does it | |
| 09:47:41 | mordred | that might be better ... since that would be more in keeping with the shape of the 'modern' version of the API - with calling the | |
| 09:47:48 | mordred | adriant: cool | |
| 09:48:46 | adriant | also, sorry if distracted, in the middle of rewriting an random fork of some library, and it's late (why the hell am I still working!?) | |
| 09:50:39 | mordred | adriant: that'sa question I ask myself all the time ... | |
| 14:29:33 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: [WIP] Network: Add tag support for QoS policy https://review.openstack.org/547810 | |
| 14:35:28 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: Add tag support for security group https://review.openstack.org/547369 | |
| 16:38:15 | openstackgerrit | Masayuki Igawa proposed openstack/cliff master: Fix cmd2 doc URL https://review.openstack.org/548063 | |
| #openstack-sdks - 2018-02-27 | |||
| 01:38:19 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for QoS policy https://review.openstack.org/547810 | |
| 02:54:21 | openstackgerrit | wangqi proposed openstack/keystoneauth master: fix minor typos in tox_install.sh https://review.openstack.org/548128 | |
| 05:38:15 | openstackgerrit | melissaml proposed openstack/python-openstacksdk master: Update the invalid url in pages https://review.openstack.org/548149 | |
| 05:55:19 | openstackgerrit | melissaml proposed openstack-infra/shade master: Update the invalid url in pages https://review.openstack.org/548152 | |
| 07:52:14 | openstackgerrit | Nguyen Hung Phuong proposed openstack-infra/shade master: Fix coverage running https://review.openstack.org/548174 | |
| 09:56:37 | openstackgerrit | Merged openstack-infra/shade master: Update the invalid url in pages https://review.openstack.org/548152 | |
| 10:08:33 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: overwrite set_tags for QoS policy https://review.openstack.org/548207 | |
| 10:34:50 | openstackgerrit | Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for security group https://review.openstack.org/547733 | |
| 10:38:27 | openstackgerrit | shogoadachi proposed openstack/python-openstackclient master: Remove --volume and its relating options from image set command https://review.openstack.org/548212 | |
| 12:27:06 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: overwrite set_tags for QoS policy https://review.openstack.org/548207 | |
| 16:17:37 | Bar_ | dtroyer, hey, Do you have an idea when is osc-lib 1.10 expected to release? | |
| 16:18:53 | dtroyer | I don't have a schedule…and IIRC you are waiting on it, right? | |
| 16:20:15 | Bar_ | dtroyer, yup, as I said before, it isn't urgent by any means, but I would like to release that patch | |
| 16:20:45 | dtroyer | thanks for the reminder, I'll push one out this week or next | |
| 16:20:59 | Bar_ | dtroyer, thank you | |
| 18:11:20 | openstackgerrit | Harald Jensås proposed openstack/python-openstackclient master: Allow setting network-segment on subnet update https://review.openstack.org/523974 | |
| 19:26:57 | openstackgerrit | Lucas Meneghel Rodrigues proposed openstack/cliff master: Fix help for subcommands on interactive shell https://review.openstack.org/548397 | |
| 21:03:41 | openstackgerrit | Slawek Kaplonski proposed openstack/python-openstacksdk master: Functional test for set_tags on Neutron resources https://review.openstack.org/548405 | |
| #openstack-sdks - 2018-02-28 | |||
| 02:01:54 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: Add tag support for QoS policy https://review.openstack.org/547372 | |
| 02:09:15 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: Add tag support for security group https://review.openstack.org/547369 | |
| 02:13:17 | openstackgerrit | Dongcan Ye proposed openstack/python-openstacksdk master: Network: Add tag support for floating ip https://review.openstack.org/547366 | |
| 07:52:25 | tobixen | I've set up a devstack, and I'm trying to run functional tests towards it. Set up a simple clouds.yaml to access devcloud, but I'm constantly getting 400s from the server, "Expecting to find domain in project" | |
| 07:52:29 | tobixen | what am I missing? | |
| 07:53:14 | tobixen | trying to run functional tests for shade, that is. | |
| 08:19:39 | frickler | tobixen: devstack should already create a working /etc/openstack/clouds.yaml, maybe use that one? or compare to yours to see where the issue might be? | |
| 08:38:54 | tobixen | frickler: thanks, that seems to help | |
| 08:39:41 | tobixen | actually, already got things working by trial and error, adding "project_domain_name: default" and "domain_name: default" to my self-created clouds.yaml | |
| 09:34:24 | tobixen | still 6 tests failing for me when running functional tests on shade, using the master branch | |
| 09:37:37 | mordred | tobixen: hrm... I'd expect them to work, so that's not happy-making - which tests? | |
| 09:42:10 | tobixen | let's see ... https://ipfs.io/ipfs/QmU86ETBVHoTaiA4CTcJmPQTFoBevytsMyf6hDDVsbprpV | |
| 09:43:48 | slaweq | mordred: hi | |
| 09:44:16 | slaweq | I know that You are busy this week but if You would have a minute, please take a look at https://review.openstack.org/#/c/548405/ - it's only some additional functional tests | |
| 09:44:20 | slaweq | thx in advance | |
| 09:49:54 | tobixen | mordred: test logs at https://ipfs.io/ipfs/QmU86ETBVHoTaiA4CTcJmPQTFoBevytsMyf6hDDVsbprpV that is. Rolled out a devstack VM on our openstack, and ran shade functional tests from the same VM | |
| 10:00:27 | mordred | tobixen: awesome - I'll check that in just a few minutes ... | |
| 10:00:29 | mordred | slaweq: will do | |
| 10:02:25 | slaweq | mordred: thx | |
| 10:12:26 | frickler | tobixen: mordred: I'm getting a similar set of failures when I run that locally, wondering what the gate sets up differently for devstack. looking at a log, "disable_service tempest" may have some effect | |
| 10:22:27 | openstackgerrit | Merged openstack/python-openstacksdk master: modify typos of word password https://review.openstack.org/542635 | |
| 11:12:24 | tobixen | I added "disable_service tempest" and upgraded devstack to newest master, and ... tests passed | |
| 11:13:42 | tobixen | frickler: mordred: ^ | |
| 11:15:47 | frickler | tobixen: thanks for confirming my guess :) | |
| 13:25:26 | tobixen | frickler: still doing research on this one. It may seem that things aren't well-enough cleaned up when doing "unstack ; stack", perhaps the trick to get tests going is to start devstack completely from scratch. | |
| 13:40:57 | dtroyer | tobixen: have you tried using clean.sh rather than unstack.sh? Clean gets more aggressive at removing things, unstack was meant for a quick(-ish) restart. | |
| 13:52:05 | mordred | tobixen, frickler: oh, fun! | |
| 13:54:15 | mordred | tobixen, frickler: looking at the traceback (sorry, jst got to it) I think even though there's a workaround, we could/should also fix the conflicts | |
| 14:13:25 | openstackgerrit | Thomas Herve proposed openstack/python-openstacksdk master: Allow not resolving outputs on get stacks https://review.openstack.org/548595 | |
| 14:46:19 | openstackgerrit | Merged openstack/python-openstacksdk master: Functional test for set_tags on Neutron resources https://review.openstack.org/548405 | |
| 15:15:34 | openstackgerrit | Merged openstack/osc-lib master: Updated from global requirements https://review.openstack.org/538478 | |
| 15:53:06 | openstackgerrit | Corey Bryant proposed openstack/cliff master: Align parsed() call with cmd2 versions >= 0.7.3 https://review.openstack.org/548625 | |
| 16:39:28 | openstackgerrit | Matthew Treinish proposed openstack/python-openstackclient master: Cleanup error messages on failure https://review.openstack.org/548656 | |
| 16:42:32 | openstackgerrit | Matthew Treinish proposed openstack/python-openstackclient master: Cleanup tox.ini for functional tests https://review.openstack.org/548657 | |
| 16:45:12 | mtreinish | dtroyer: ^^^ still reading things, but that first patch might help | |
| 16:45:23 | mtreinish | I'm having a hard time reading the giant blob of text | |
| 16:45:57 | dtroyer | I think the entry point failures is the key, that they are there at all, not which ones specifically | |
| 16:46:39 | dtroyer | I've narrowed it down to working fine locally without https://review.openstack.org/#/c/538478/ and failing with it, and that commit is the only difference in the venv | |