Tuesday, 2018-07-31

*** Kevin_Zheng has joined #openstack-sdks00:20
*** edmondsw has joined #openstack-sdks00:42
*** edmondsw has quit IRC00:48
*** chenyb4 has joined #openstack-sdks00:53
openstackgerritFan Zhang proposed openstack/python-openstackclient master: Remove volume replication v1 support.  https://review.openstack.org/58700801:15
*** mhen has quit IRC01:32
*** mhen has joined #openstack-sdks01:33
dtroyerthis ^^^ and https://review.openstack.org/#/c/587005/ will likely need to be merged to pass tests01:38
*** bobh has joined #openstack-sdks01:38
*** annp has joined #openstack-sdks01:50
openstackgerrityanpuqing proposed openstack/openstacksdk master: Add an error message when router add subnet  https://review.openstack.org/58699602:05
openstackgerritFan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs.  https://review.openstack.org/58700502:24
*** bobh has quit IRC02:29
*** gildub has joined #openstack-sdks02:49
openstackgerritMerged openstack/openstacksdk master: Add some backoff to find_best_address  https://review.openstack.org/57558002:56
*** dtroyer has quit IRC03:40
*** dtroyer has joined #openstack-sdks03:52
*** gkadam has joined #openstack-sdks04:05
*** samP has quit IRC04:08
*** yolanda has quit IRC04:09
*** samP has joined #openstack-sdks04:14
*** yolanda has joined #openstack-sdks04:24
*** pooja_jadhav has joined #openstack-sdks04:30
*** gildub has quit IRC04:46
*** mriedem has quit IRC04:49
*** e0ne has joined #openstack-sdks05:59
openstackgerritFan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs.  https://review.openstack.org/58700506:03
*** gkadam has quit IRC06:04
*** gkadam has joined #openstack-sdks06:05
openstackgerritFan Zhang proposed openstack/python-openstackclient master: Fix broken gate jobs.  https://review.openstack.org/58700506:08
*** madorn has quit IRC06:15
*** madorn has joined #openstack-sdks06:17
*** e0ne has quit IRC06:51
*** Luzi has joined #openstack-sdks07:04
*** peereb has joined #openstack-sdks07:10
*** gildub has joined #openstack-sdks07:25
*** fanzhang has joined #openstack-sdks07:30
*** ttsiouts has joined #openstack-sdks07:48
openstackgerritFan Zhang proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs.  https://review.openstack.org/58700507:48
*** jpena|off is now known as jpena07:53
*** cdent has joined #openstack-sdks07:55
*** ttsiouts has quit IRC07:59
*** jpich has joined #openstack-sdks08:10
*** fanzhang has quit IRC08:12
*** ttsiouts has joined #openstack-sdks08:26
*** gkadam has quit IRC08:31
*** gkadam has joined #openstack-sdks08:37
*** fanzhang has joined #openstack-sdks08:39
*** gkadam has quit IRC08:41
*** e0ne has joined #openstack-sdks09:13
*** pfo has joined #openstack-sdks09:45
*** gildub has quit IRC09:49
*** gildub has joined #openstack-sdks09:53
*** ttsiouts has quit IRC10:14
*** ttsiouts has joined #openstack-sdks10:19
*** ttsiouts has quit IRC10:24
*** gildub has quit IRC10:37
*** chenyb4 has quit IRC10:48
*** annp has quit IRC10:49
*** ttsiouts has joined #openstack-sdks10:59
openstackgerritMerged openstack/openstacksdk master: Adds toggle port security on network create  https://review.openstack.org/58716611:02
*** annp has joined #openstack-sdks11:03
openstackgerritMerged openstack/openstacksdk master: python-shade expose MTU setting.  https://review.openstack.org/58716711:06
*** gildub has joined #openstack-sdks11:31
*** jpena is now known as jpena|lunch11:46
*** gildub has quit IRC12:00
*** gildub has joined #openstack-sdks12:02
*** rcarrillocruz has quit IRC12:04
*** edmondsw has joined #openstack-sdks12:24
*** jpena|lunch is now known as jpena12:48
*** bobh has joined #openstack-sdks12:48
*** bobh has quit IRC12:48
*** bobh has joined #openstack-sdks12:48
*** bobh has quit IRC12:53
*** crunchengine has joined #openstack-sdks12:57
*** ttsiouts has quit IRC13:01
*** ttsiouts has joined #openstack-sdks13:01
*** ttsiouts has quit IRC13:04
*** ttsiouts has joined #openstack-sdks13:04
*** mriedem has joined #openstack-sdks13:05
mordreddtroyer: so we should squash 587008 and 587005 ?13:12
mordredfanzhang: looks like you're on top of it - let  me know if you'd like me to help in any way13:15
openstackgerritMonty Taylor proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs.  https://review.openstack.org/58700513:32
mordredfanzhang: ^^ pushed up an update13:32
*** thrash|g0ne is now known as thrash13:33
*** Luzi has quit IRC13:36
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Pass microversion info through from Profile  https://review.openstack.org/58743513:37
*** gildub has quit IRC14:00
*** pfo has quit IRC14:07
mordredcdent: youknow all the things, right?14:11
cdentmordred: I know everything except what you want to know.14:11
mordredsweet. perfect14:11
mordredcdent: do you happen to know what response bodies from nova errors look like? they don't seem to be documented in the api docs14:12
mordredcdent: I have just learned that neutron sends a json dict {'NeutronError': 'some string'}14:12
mordredbecause of course14:12
cdentthe nova api is murky to me, but I should be able to find something clearly relatively quickly14:13
mordredcool - thanks! I *think* it might be a list of dicts with a message key - since we've got sdk code to handle that case14:14
cdentmordred: looks like a dict keys on the fault name: https://github.com/openstack/nova/blob/03ccb1f9333bcfbdb2caf8a8c6e98622a63088cc/nova/api/openstack/wsgi.py#L1080-L108314:16
mordredcdent: ah - cool. that would explain the code in sdk currently14:17
mordredI will add a comment14:17
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add support for neutron error messages  https://review.openstack.org/58748314:22
mordredcdent: ^^ just in case you are curious.14:34
cdentI'm sure we made this http://specs.openstack.org/openstack/api-wg/guidelines/errors.html for some reason but I can't remember why14:42
openstackgerritMerged openstack/openstacksdk master: Support for microversions in baremetal resources  https://review.openstack.org/58353215:01
openstackgerritMerged openstack/openstacksdk master: Add a simple baremetal functional job  https://review.openstack.org/58657415:01
*** pfo has joined #openstack-sdks15:03
*** peereb has quit IRC15:06
*** pfo has quit IRC15:13
*** e0ne has quit IRC15:23
*** pfo has joined #openstack-sdks15:30
*** bobh has joined #openstack-sdks16:22
*** bobh has quit IRC16:22
*** zhongjun__ has quit IRC16:25
*** jpich has quit IRC16:35
*** dtruong2 has joined #openstack-sdks16:36
*** ttsiouts has quit IRC16:48
*** ttsiouts has joined #openstack-sdks16:48
*** ttsiouts has quit IRC16:53
*** jpena is now known as jpena|off17:05
*** slaweq has quit IRC17:05
*** pfo has quit IRC17:44
*** e0ne has joined #openstack-sdks17:51
*** slaweq has joined #openstack-sdks17:55
*** e0ne has quit IRC19:06
*** cdent has quit IRC19:14
*** slaweq has quit IRC19:47
*** e0ne has joined #openstack-sdks19:48
*** slaweq has joined #openstack-sdks19:55
openstackgerritDean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs.  https://review.openstack.org/58700520:13
openstackgerritDean Troyer proposed openstack/python-openstackclient master: [WIP] Fix broken gate jobs.  https://review.openstack.org/58700520:19
dtroyermordred: ^^^ should get us really close… it needs a way to detect Image API v1 yet20:23
mordreddtroyer: like - needs to be able to detect whether the cloud has v1?20:24
dtroyeryes20:24
mordredhttps://review.openstack.org/#/c/585029/ <--20:24
mordreddtroyer: that is what you need - unfortunately I did not manage to get it in to keystoneauth before the feature freeze20:25
dtroyerI've hard-coded it 'not there' in that rev just to get this through… ah, nice20:25
mordreddtroyer: 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 it20:25
dtroyerI'll think about this harder after the 30,000 foot fatigue is less20:26
dtroyerI'm fine, this is better than ripping it all out20:26
mordreddtroyer: 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 pleasant20:26
mordredcool20:26
dtroyerhmmm… yeah, I was thinking something like that, it's a one-time deal so we can live with it20:27
mordred++20:27
mordredand 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 IRC20:32
*** e0ne has quit IRC20:51
samueldmqhi. I am not getting why we need ansible tests for the upper abstraction level/shade20:58
samueldmqas opposed to having just the unit and functional tests...20:58
samueldmqseems like those ansible tests should belong somewhere in the shade plugin to ansible (or something like that)?20:59
*** edmondsw has quit IRC21:00
*** edmondsw has joined #openstack-sdks21:01
mordredsamueldmq: yes - they should indeed be there21:01
*** edmondsw has quit IRC21:01
mordredsamueldmq: they were in the shade repo for hysterical raisins21:01
*** edmondsw has joined #openstack-sdks21:01
mordredsamueldmq: mostly that we didn't have a way to run them if we put them into the ansible repo21:01
mordredsamueldmq: but now that we can (and do) trigger jobs in Infra on PRs to ansible ...21:02
mordredsamueldmq: we should TOTALLY move them to ansible/ansible and get them integrated with ansible-test21:02
samueldmqkk should be up to them to run the tests?21:02
samueldmqor to us on our infra (seems to be what you're suggesting)21:03
samueldmqmordred: ^21:03
samueldmqahh I got it ... we can add a check on sdks to run ansible tests against proposed shade changes21:05
samueldmqjust to make sure we're not breaking'em on our side21:05
mordredthat's right21:06
mordredwe actually already run the shade-ansible test job on ansible/ansible PRs that touch openstack bits21:06
mordredas well as on both shade and openstacksdk21:06
samueldmqok it's just a matter of putting things in their right place21:07
samueldmqfor running against arbitrary cloud ... do you think I should work on shade/functional tests side only?21:07
samueldmqshade should be able to run its tests against arbitrary clouds, regardless being useful to ansible21:08
mordredyes - I agree21:08
mordred(although do it in openstacksdk and not shade- but yes)21:08
mordredwe also have WAY more coverage in the openstacksdk functional tests than we do in the ansible tests21:08
mordredso we get a much nicer win21:09
samueldmq++21:09
samueldmqyes I meant openstacksdk ... just not got used to the change yet (since I had disapeared by the time of the merge) :-)21:09
samueldmqdidnt get*21:10
samueldmqkk I'll take a look there .. but I'm a bit worried about the policy bits...21:10
samueldmqsince different users will have different permissions on different clouds21:10
samueldmqthat's a bit different than running just on devstack (opinionated).  may require some work but should not be terrible'21:11
openstackgerritDoug Hellmann proposed openstack/openstacksdk master: fix 2 typos in documentation  https://review.openstack.org/58762421:16
*** pfo has joined #openstack-sdks21:16
*** pfo has quit IRC21:21
*** yolanda has quit IRC21:25
*** bobh has joined #openstack-sdks21:26
mordredsamueldmq: yes - I totally agree (and same here - I still say shade a lot)21:28
*** pfo has joined #openstack-sdks21:47
*** bobh has quit IRC22:03
*** mriedem has quit IRC22:14
*** slaweq has quit IRC22:16
*** edmondsw has quit IRC22:23
*** edmondsw has joined #openstack-sdks22:23
*** gildub has joined #openstack-sdks22:28
*** edmondsw has quit IRC22:30
*** edmondsw has joined #openstack-sdks22:31
*** edmondsw has quit IRC23:11
*** pfo has quit IRC23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!