| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-01-11 | |||
| 18:30:03 | ankur-gupta-f | dtroyer: here? | |
| 18:30:55 | dtroyer | ankur-gupta-f: yo | |
| 18:31:06 | ankur-gupta-f | Client functional tests :D | |
| 18:31:14 | ankur-gupta-f | so we tried the regexes | |
| 18:31:16 | ankur-gupta-f | now JSON | |
| 18:31:28 | ankur-gupta-f | but for existing patches can we leave regexes and follow up with JSON | |
| 18:32:05 | dtroyer | yes. it may still evolve more, I don't want you guys to chase too much of a moving target, OSC does that all the time anyway | |
| 18:32:18 | dtroyer | ther are a couple of things I'd like to make sure get in theough, | |
| 18:32:25 | dtroyer | using addCleanup() is one of those | |
| 18:32:49 | dtroyer | that is more than a cosmentic/developer preference thing | |
| 18:33:05 | ankur-gupta-f | for everything created. | |
| 18:33:32 | ankur-gupta-f | I do like the change that every test does the creation, <list, show, etc..> deletion | |
| 18:34:24 | ankur-gupta-f | Okay thats all. Thanks | |
| 18:40:56 | dtroyer | stevemar: I ran the -efunctional-tips locally, no tests ran…do we need to include the ostestr command in that section? | |
| 18:52:59 | cdent | elmiko: that link you tweeted a while ago to radanalytics wants auth? | |
| 18:56:22 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: add support for running tests with master dependencies https://review.openstack.org/419032 | |
| 19:00:06 | stevemar | dtroyer: thats a good question, the [fucntional] target doesn't have any ostestr stuff | |
| 19:00:53 | stevemar | dtroyer: do you know why we have to add ostestr to -tips but not the other? | |
| 19:01:13 | openstackgerrit | Merged openstack/python-openstackclient: Fix filter error in os volume list https://review.openstack.org/416852 | |
| 19:01:24 | dtroyer | I just added it to the command list. I think tox inherits from [env] if there isn't a command= | |
| 19:01:55 | thingee | dtroyer seems to be working now | |
| 19:12:53 | thingee | sorry I missed that | |
| 19:18:55 | dtroyer | heh, I did too until I actually tried it :) | |
| 19:20:30 | stevemar | dtroyer: ahh, but since there is a command= there, it doesn't inherit from env | |
| 19:29:17 | openstackgerrit | Diana Clarke proposed openstack/python-openstackclient: WIP - Add nova microversion 2.40 support https://review.openstack.org/419108 | |
| 19:40:17 | openstackgerrit | Merged openstack/cliff: Add Constraints support https://review.openstack.org/412742 | |
| 19:49:58 | openstackgerrit | Ankur proposed openstack/python-openstackclient: Add meter rule to OSC https://review.openstack.org/378058 | |
| 20:27:44 | stevemar | dtroyer: have you been keeping track of what networking commands broke and are fixed in osc? | |
| 20:39:26 | dtroyer | I don't have an exhaustive list… AFAIK only the qos rule type is left though? | |
| 20:39:37 | dtroyer | well, I haven't gone looking for the remaining skips yet | |
| 20:40:21 | dtroyer | erg, there are still 4 | |
| 20:41:19 | dtroyer | stevemar: do you recall how many tests were being skipped before this mess started? | |
| 20:41:40 | stevemar | only 4 (they were conditional on a networking extension) | |
| 20:41:58 | stevemar | we had no skipped tests otherwise | |
| 20:42:04 | dtroyer | ok then, your 418650 has them all then | |
| 20:46:17 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstackclient: Introduce Neutron DHCP agent commands to OSC. https://review.openstack.org/387611 | |
| 20:47:08 | openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: unskip network qos rule type functional test https://review.openstack.org/419129 | |
| 20:47:27 | stevemar | dtroyer: ^ i think that one worked, just a bad test | |
| 20:48:17 | dtroyer | I think it is more than a bad test, I get a 404 when I try the command | |
| 20:53:43 | dtroyer | #$^%&%$#$!@#$%^*%&$#@#$%^*&^&%^%$@#^$&%&^*&%^$%#$@#!@#$^%&^ | |
| 20:54:01 | dtroyer | Why did we let an extension command into the main repo?????????????????? | |
| 20:55:51 | dtroyer | so nevermind my 404, I don't have the (*&^%$#$%^&*( extension enabled | |
| 20:56:32 | dtroyer | we gotta handle that better… and by better I mean at all | |
| 20:56:45 | dtroyer | Is ther any way to make the tests discover and auto-skip this? | |
| 20:57:38 | stevemar | dtroyer: i thought there was a discover and skip mechanism | |
| 20:57:46 | stevemar | i guess not, since it didn't skip for you | |
| 20:58:33 | stevemar | dtroyer: that test and command was created by rich back in the early networking days, before we had a fleshed out plan for extensions and had the neutron team on board | |
| 21:00:09 | dtroyer | hmmm… git sez it was committed on Aug 12 | |
| 21:00:27 | dtroyer | by https://github.com/ralonsoh | |
| 21:01:06 | dtroyer | well, that was an auto-complete | |
| 21:01:17 | dtroyer | ralonsoh, may or may not be the same person | |
| 21:02:09 | stevemar | dtroyer: i was referring to https://github.com/openstack/python-openstackclient/commit/4f23a77de04bfdfafd4bf8f16b0365df7663e9e5 | |
| 21:02:51 | stevemar | dtroyer: skippage is here: https://github.com/openstack/python-openstackclient/blob/2026dee17e7b6b8d738345fd27dc905efb6561f1/openstackclient/tests/functional/network/v2/test_network_segment.py#L54-L66 | |
| 21:02:54 | dtroyer | oh, I thought we were talking about qos rule type still | |
| 21:05:43 | stevemar | dtroyer: i thought we were tlaking about skipped tests -- 418650 unskips all the skipped ones, except those network segment ones | |
| 21:06:05 | stevemar | i have no idea what qos rule type is | |
| 21:06:24 | dtroyer | that's the one test that failed in your 418650 test with sdk 0.9.12 | |
| 21:06:35 | dtroyer | I can't tell (yet) if it ever worked | |
| 21:07:02 | dtroyer | have to restart my devstack with q-qos running, and that takes a long time these days | |
| 21:07:30 | stevemar | dtroyer: wel https://review.openstack.org/#/c/419129/ fixes it, i think | |
| 21:07:31 | dtroyer | I believe this is the last blocker for sdk 0.9.12 | |
| 21:07:33 | stevemar | let me run it for you | |
| 21:08:11 | dtroyer | so is that a fix or just not testing for the missing piece? | |
| 21:08:30 | dtroyer | ie, is something still broken since minimum_bandwidth was expected to be ther at one time? | |
| 21:12:19 | stevemar | dtroyer: i'm wondering that myself, looking at the neutron APIs while I'm stacking with qos enabled | |
| 21:12:56 | dtroyer | I don't see it in http://developer.openstack.org/api-ref/networking/v2/ | |
| 21:13:54 | stevemar | dtroyer: looking at http://developer.openstack.org/api-ref/networking/v2/?expanded=#quality-of-service | |
| 21:14:07 | stevemar | theres definitely bandwidth_limit_rules and dscp_marking_rules | |
| 21:14:14 | stevemar | no idea where minimum bandwidth came from | |
| 21:14:30 | dtroyer | ok, that's good enough for moi | |
| 21:14:49 | stevemar | also, the functional command returned only the two, i added that to the commit message | |
| 21:15:07 | stevemar | dtroyer: did you want me to punt some things through while we're short on time? | |
| 21:16:30 | dtroyer | http://docs.openstack.org/developer/neutron/devref/quality_of_service.html#qos-versioned-objects lists a QosMinimumBandwidth object | |
| 21:19:05 | stevemar | dtroyer: *shrug* | |
| 21:20:12 | dtroyer | I guess at this point… if that passes, let's roll with it | |
| 21:21:37 | dtroyer | I just +W the hook_tips.sh review | |
| 21:23:18 | dtroyer | stevemar: if you have a minute, I think https://review.openstack.org/#/c/352477/ is ready | |
| 21:23:52 | dtroyer | and lookie there, it includes minimum-bandwidth | |
| 21:39:52 | openstackgerrit | Abhishek Raut proposed openstack/python-openstackclient: SDK refactor: Prepare security group commands https://review.openstack.org/418183 | |
| 21:43:01 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Add one option to "volume host set" command https://review.openstack.org/411259 | |
| 22:12:58 | stevemar | dtroyer: ahh theres the bugger | |
| 22:21:42 | stevemar | dtroyer: punted | |
| 22:22:33 | stevemar | dtroyer: that should make https://review.openstack.org/#/c/419129/ unnecessary | |
| 22:23:38 | stevemar | dtroyer: so i'm not sure if briancurtin will release a new SDK in the next week or two, how do we want to handle the SDK version in our requirements file? | |
| 22:23:53 | stevemar | should we revert the SDK blacklist? | |
| 22:24:18 | stevemar | hmm... that'll cause gate issues with rally and others | |
| 22:24:34 | dtroyer | yeah, not until after we release, at least... | |
| 22:24:54 | stevemar | dtroyer: we may have to release osc 3.7.0, revert the sdk blacklist, and do a quick release with SDK 0.9.12 included? | |
| 22:25:19 | stevemar | it would be wonky to do all this compatability work and not have it work with sdk 0.9.12 | |
| 22:25:38 | dtroyer | That's partly why I was wanting to do two releases before freeze, the second, without the blacklists, will become stable and we don't live with that for a long time | |
| 22:25:55 | dtroyer | also, there's stuff that wants sdk 0.9.11 as min anyway | |
| 22:26:21 | dtroyer | so maybe that's what we set when we remove the blacklist? | |
| 22:27:26 | openstackgerrit | Merged openstack/python-openstackclient: add support for running tests with master dependencies https://review.openstack.org/419032 | |
| 22:28:06 | openstackgerrit | Merged openstack/python-openstackclient: Add support for Glance 'update image members' feature https://review.openstack.org/410848 | |
| 22:28:19 | openstackgerrit | Merged openstack/python-openstackclient: Error handling for delete commands in identity https://review.openstack.org/415799 | |
| 22:28:53 | stevemar | ok with me | |
| 22:30:05 | openstackgerrit | Steve Martinelli proposed openstack/python-openstackclient: Add unit test for multi volume types delete https://review.openstack.org/405947 | |
| 23:04:47 | openstackgerrit | Ankur proposed openstack/python-openstackclient: Add meter rule to OSC https://review.openstack.org/378058 | |
| 23:12:38 | openstackgerrit | Merged openstack/python-openstackclient: Add two consistency group commands https://review.openstack.org/407983 | |
| 23:37:01 | openstackgerrit | Merged openstack/python-openstackclient: Add support for Network QoS rule commands https://review.openstack.org/352477 | |