*** Kevin_Zheng has joined #openstack-sdks | 00:20 | |
*** edmondsw has joined #openstack-sdks | 00:42 | |
*** edmondsw has quit IRC | 00:48 | |
*** chenyb4 has joined #openstack-sdks | 00:53 | |
openstackgerrit | Fan Zhang proposed openstack/python-openstackclient master: Remove volume replication v1 support. https://review.openstack.org/587008 | 01:15 |
---|---|---|
*** mhen has quit IRC | 01:32 | |
*** mhen has joined #openstack-sdks | 01:33 | |
dtroyer | this ^^^ and https://review.openstack.org/#/c/587005/ will likely need to be merged to pass tests | 01:38 |
*** bobh has joined #openstack-sdks | 01:38 | |
*** annp has joined #openstack-sdks | 01:50 | |
openstackgerrit | yanpuqing proposed openstack/openstacksdk master: Add an error message when router add subnet https://review.openstack.org/586996 | 02:05 |
openstackgerrit | Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005 | 02:24 |
*** bobh has quit IRC | 02:29 | |
*** gildub has joined #openstack-sdks | 02:49 | |
openstackgerrit | Merged openstack/openstacksdk master: Add some backoff to find_best_address https://review.openstack.org/575580 | 02:56 |
*** dtroyer has quit IRC | 03:40 | |
*** dtroyer has joined #openstack-sdks | 03:52 | |
*** gkadam has joined #openstack-sdks | 04:05 | |
*** samP has quit IRC | 04:08 | |
*** yolanda has quit IRC | 04:09 | |
*** samP has joined #openstack-sdks | 04:14 | |
*** yolanda has joined #openstack-sdks | 04:24 | |
*** pooja_jadhav has joined #openstack-sdks | 04:30 | |
*** gildub has quit IRC | 04:46 | |
*** mriedem has quit IRC | 04:49 | |
*** e0ne has joined #openstack-sdks | 05:59 | |
openstackgerrit | Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005 | 06:03 |
*** gkadam has quit IRC | 06:04 | |
*** gkadam has joined #openstack-sdks | 06:05 | |
openstackgerrit | Fan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs. https://review.openstack.org/587005 | 06:08 |
*** madorn has quit IRC | 06:15 | |
*** madorn has joined #openstack-sdks | 06:17 | |
*** e0ne has quit IRC | 06:51 | |
*** Luzi has joined #openstack-sdks | 07:04 | |
*** peereb has joined #openstack-sdks | 07:10 | |
*** gildub has joined #openstack-sdks | 07:25 | |
*** fanzhang has joined #openstack-sdks | 07:30 | |
*** ttsiouts has joined #openstack-sdks | 07:48 | |
openstackgerrit | Fan Zhang proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005 | 07:48 |
*** jpena|off is now known as jpena | 07:53 | |
*** cdent has joined #openstack-sdks | 07:55 | |
*** ttsiouts has quit IRC | 07:59 | |
*** jpich has joined #openstack-sdks | 08:10 | |
*** fanzhang has quit IRC | 08:12 | |
*** ttsiouts has joined #openstack-sdks | 08:26 | |
*** gkadam has quit IRC | 08:31 | |
*** gkadam has joined #openstack-sdks | 08:37 | |
*** fanzhang has joined #openstack-sdks | 08:39 | |
*** gkadam has quit IRC | 08:41 | |
*** e0ne has joined #openstack-sdks | 09:13 | |
*** pfo has joined #openstack-sdks | 09:45 | |
*** gildub has quit IRC | 09:49 | |
*** gildub has joined #openstack-sdks | 09:53 | |
*** ttsiouts has quit IRC | 10:14 | |
*** ttsiouts has joined #openstack-sdks | 10:19 | |
*** ttsiouts has quit IRC | 10:24 | |
*** gildub has quit IRC | 10:37 | |
*** chenyb4 has quit IRC | 10:48 | |
*** annp has quit IRC | 10:49 | |
*** ttsiouts has joined #openstack-sdks | 10:59 | |
openstackgerrit | Merged openstack/openstacksdk master: Adds toggle port security on network create https://review.openstack.org/587166 | 11:02 |
*** annp has joined #openstack-sdks | 11:03 | |
openstackgerrit | Merged openstack/openstacksdk master: python-shade expose MTU setting. https://review.openstack.org/587167 | 11:06 |
*** gildub has joined #openstack-sdks | 11:31 | |
*** jpena is now known as jpena|lunch | 11:46 | |
*** gildub has quit IRC | 12:00 | |
*** gildub has joined #openstack-sdks | 12:02 | |
*** rcarrillocruz has quit IRC | 12:04 | |
*** edmondsw has joined #openstack-sdks | 12:24 | |
*** jpena|lunch is now known as jpena | 12:48 | |
*** bobh has joined #openstack-sdks | 12:48 | |
*** bobh has quit IRC | 12:48 | |
*** bobh has joined #openstack-sdks | 12:48 | |
*** bobh has quit IRC | 12:53 | |
*** crunchengine has joined #openstack-sdks | 12:57 | |
*** ttsiouts has quit IRC | 13:01 | |
*** ttsiouts has joined #openstack-sdks | 13:01 | |
*** ttsiouts has quit IRC | 13:04 | |
*** ttsiouts has joined #openstack-sdks | 13:04 | |
*** mriedem has joined #openstack-sdks | 13:05 | |
mordred | dtroyer: so we should squash 587008 and 587005 ? | 13:12 |
mordred | fanzhang: looks like you're on top of it - let me know if you'd like me to help in any way | 13:15 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005 | 13:32 |
mordred | fanzhang: ^^ pushed up an update | 13:32 |
*** thrash|g0ne is now known as thrash | 13:33 | |
*** Luzi has quit IRC | 13:36 | |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Pass microversion info through from Profile https://review.openstack.org/587435 | 13:37 |
*** gildub has quit IRC | 14:00 | |
*** pfo has quit IRC | 14:07 | |
mordred | cdent: youknow all the things, right? | 14:11 |
cdent | mordred: I know everything except what you want to know. | 14:11 |
mordred | sweet. perfect | 14:11 |
mordred | cdent: do you happen to know what response bodies from nova errors look like? they don't seem to be documented in the api docs | 14:12 |
mordred | cdent: I have just learned that neutron sends a json dict {'NeutronError': 'some string'} | 14:12 |
mordred | because of course | 14:12 |
cdent | the nova api is murky to me, but I should be able to find something clearly relatively quickly | 14:13 |
mordred | cool - thanks! I *think* it might be a list of dicts with a message key - since we've got sdk code to handle that case | 14:14 |
cdent | mordred: looks like a dict keys on the fault name: https://github.com/openstack/nova/blob/03ccb1f9333bcfbdb2caf8a8c6e98622a63088cc/nova/api/openstack/wsgi.py#L1080-L1083 | 14:16 |
mordred | cdent: ah - cool. that would explain the code in sdk currently | 14:17 |
mordred | I will add a comment | 14:17 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add support for neutron error messages https://review.openstack.org/587483 | 14:22 |
mordred | cdent: ^^ just in case you are curious. | 14:34 |
cdent | I'm sure we made this http://specs.openstack.org/openstack/api-wg/guidelines/errors.html for some reason but I can't remember why | 14:42 |
openstackgerrit | Merged openstack/openstacksdk master: Support for microversions in baremetal resources https://review.openstack.org/583532 | 15:01 |
openstackgerrit | Merged openstack/openstacksdk master: Add a simple baremetal functional job https://review.openstack.org/586574 | 15:01 |
*** pfo has joined #openstack-sdks | 15:03 | |
*** peereb has quit IRC | 15:06 | |
*** pfo has quit IRC | 15:13 | |
*** e0ne has quit IRC | 15:23 | |
*** pfo has joined #openstack-sdks | 15:30 | |
*** bobh has joined #openstack-sdks | 16:22 | |
*** bobh has quit IRC | 16:22 | |
*** zhongjun__ has quit IRC | 16:25 | |
*** jpich has quit IRC | 16:35 | |
*** dtruong2 has joined #openstack-sdks | 16:36 | |
*** ttsiouts has quit IRC | 16:48 | |
*** ttsiouts has joined #openstack-sdks | 16:48 | |
*** ttsiouts has quit IRC | 16:53 | |
*** jpena is now known as jpena|off | 17:05 | |
*** slaweq has quit IRC | 17:05 | |
*** pfo has quit IRC | 17:44 | |
*** e0ne has joined #openstack-sdks | 17:51 | |
*** slaweq has joined #openstack-sdks | 17:55 | |
*** e0ne has quit IRC | 19:06 | |
*** cdent has quit IRC | 19:14 | |
*** slaweq has quit IRC | 19:47 | |
*** e0ne has joined #openstack-sdks | 19:48 | |
*** slaweq has joined #openstack-sdks | 19:55 | |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005 | 20:13 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs. https://review.openstack.org/587005 | 20:19 |
dtroyer | mordred: ^^^ should get us really close… it needs a way to detect Image API v1 yet | 20:23 |
mordred | dtroyer: like - needs to be able to detect whether the cloud has v1? | 20:24 |
dtroyer | yes | 20:24 |
mordred | https://review.openstack.org/#/c/585029/ <-- | 20:24 |
mordred | dtroyer: that is what you need - unfortunately I did not manage to get it in to keystoneauth before the feature freeze | 20:25 |
dtroyer | I've hard-coded it 'not there' in that rev just to get this through… ah, nice | 20:25 |
mordred | dtroyer: yah - if you're ok with the test suite being a little wonk until we can get another ksa release cut - there's a path to it | 20:25 |
dtroyer | I'll think about this harder after the 30,000 foot fatigue is less | 20:26 |
dtroyer | I'm fine, this is better than ripping it all out | 20:26 |
mordred | dtroyer: alternately, the code in the versions show command that does it client-side would also work - it'll just be more expensive than would normally be pleasant | 20:26 |
mordred | cool | 20:26 |
dtroyer | hmmm… yeah, I was thinking something like that, it's a one-time deal so we can live with it | 20:27 |
mordred | ++ | 20:27 |
mordred | and it should all be really cheap local calls in functional tests - not quite like doing the same thing on an actual public cloud :) | 20:28 |
*** crunchengine has quit IRC | 20:32 | |
*** e0ne has quit IRC | 20:51 | |
samueldmq | hi. I am not getting why we need ansible tests for the upper abstraction level/shade | 20:58 |
samueldmq | as opposed to having just the unit and functional tests... | 20:58 |
samueldmq | seems like those ansible tests should belong somewhere in the shade plugin to ansible (or something like that)? | 20:59 |
*** edmondsw has quit IRC | 21:00 | |
*** edmondsw has joined #openstack-sdks | 21:01 | |
mordred | samueldmq: yes - they should indeed be there | 21:01 |
*** edmondsw has quit IRC | 21:01 | |
mordred | samueldmq: they were in the shade repo for hysterical raisins | 21:01 |
*** edmondsw has joined #openstack-sdks | 21:01 | |
mordred | samueldmq: mostly that we didn't have a way to run them if we put them into the ansible repo | 21:01 |
mordred | samueldmq: but now that we can (and do) trigger jobs in Infra on PRs to ansible ... | 21:02 |
mordred | samueldmq: we should TOTALLY move them to ansible/ansible and get them integrated with ansible-test | 21:02 |
samueldmq | kk should be up to them to run the tests? | 21:02 |
samueldmq | or to us on our infra (seems to be what you're suggesting) | 21:03 |
samueldmq | mordred: ^ | 21:03 |
samueldmq | ahh I got it ... we can add a check on sdks to run ansible tests against proposed shade changes | 21:05 |
samueldmq | just to make sure we're not breaking'em on our side | 21:05 |
mordred | that's right | 21:06 |
mordred | we actually already run the shade-ansible test job on ansible/ansible PRs that touch openstack bits | 21:06 |
mordred | as well as on both shade and openstacksdk | 21:06 |
samueldmq | ok it's just a matter of putting things in their right place | 21:07 |
samueldmq | for running against arbitrary cloud ... do you think I should work on shade/functional tests side only? | 21:07 |
samueldmq | shade should be able to run its tests against arbitrary clouds, regardless being useful to ansible | 21:08 |
mordred | yes - I agree | 21:08 |
mordred | (although do it in openstacksdk and not shade- but yes) | 21:08 |
mordred | we also have WAY more coverage in the openstacksdk functional tests than we do in the ansible tests | 21:08 |
mordred | so we get a much nicer win | 21:09 |
samueldmq | ++ | 21:09 |
samueldmq | yes I meant openstacksdk ... just not got used to the change yet (since I had disapeared by the time of the merge) :-) | 21:09 |
samueldmq | didnt get* | 21:10 |
samueldmq | kk I'll take a look there .. but I'm a bit worried about the policy bits... | 21:10 |
samueldmq | since different users will have different permissions on different clouds | 21:10 |
samueldmq | that's a bit different than running just on devstack (opinionated). may require some work but should not be terrible' | 21:11 |
openstackgerrit | Doug Hellmann proposed openstack/openstacksdk master: fix 2 typos in documentation https://review.openstack.org/587624 | 21:16 |
*** pfo has joined #openstack-sdks | 21:16 | |
*** pfo has quit IRC | 21:21 | |
*** yolanda has quit IRC | 21:25 | |
*** bobh has joined #openstack-sdks | 21:26 | |
mordred | samueldmq: yes - I totally agree (and same here - I still say shade a lot) | 21:28 |
*** pfo has joined #openstack-sdks | 21:47 | |
*** bobh has quit IRC | 22:03 | |
*** mriedem has quit IRC | 22:14 | |
*** slaweq has quit IRC | 22:16 | |
*** edmondsw has quit IRC | 22:23 | |
*** edmondsw has joined #openstack-sdks | 22:23 | |
*** gildub has joined #openstack-sdks | 22:28 | |
*** edmondsw has quit IRC | 22:30 | |
*** edmondsw has joined #openstack-sdks | 22:31 | |
*** edmondsw has quit IRC | 23:11 | |
*** pfo has quit IRC | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!