| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-02-26 | |||
| 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 | |
| 16:46:59 | dtroyer | the dependencies that changed in there do not change between test runs | |
| 16:47:08 | dtroyer | (I am removing the venv each tme so it is always refreshed) | |
| 16:48:32 | dtroyer | that cleanup is certainly worth doing no matter what, thanks | |
| 16:49:31 | mtreinish | yeah, parsing the giant blob in the logs I see the entrypoint failure now | |
| 16:49:39 | mtreinish | and I agree that's likely the issue | |
| 16:50:41 | mtreinish | I just looked at the pip freeze logs from the run and it seems like there are some weird issues there: http://logs.openstack.org/78/538478/5/check/osc-functional-devstack-tips/356323d/tox/functional-5.log | |
| 17:11:51 | openstackgerrit | Merged openstack/osc-lib master: Add project lookup utils https://review.openstack.org/522861 | |
| 17:27:24 | openstackgerrit | OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672 | |
| 17:39:11 | tobixen | dtroyer: thanks, clean.sh did the trick. Just verified that I managed getting some tests failing with a fresh devstack, and getting all tests passing by disabling tempest and doing a clean.sh | |
| 19:57:52 | frickler | mordred: this one looks fun, too ;) https://launchpad.net/bugs/1751822 | |
| 19:57:54 | openstack | Launchpad bug 1751822 in cliff "EXCEPTION of type 'AttributeError' when using 'help' from openstack-cli" [Undecided,In progress] - Assigned to Corey Bryant (corey.bryant) | |
| 20:12:44 | openstackgerrit | Corey Bryant proposed openstack/cliff master: Align parsed() call with cmd2 versions >= 0.7.3 https://review.openstack.org/548625 | |
| 20:16:08 | openstackgerrit | Corey Bryant proposed openstack/cliff master: Align parsed() call with cmd2 versions >= 0.7.3 https://review.openstack.org/548625 | |
| 22:16:27 | openstackgerrit | Tobias Brox proposed openstack-infra/shade master: Simple fix making the TestNetwork functional test cleanup more robust and a bit faster, by passing network id instead of network name. Task: 6594 Story: 2001617 https://review.openstack.org/548714 | |