| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-07 | |||
| 22:10:42 | openstackgerrit | Merged openstack/python-openstackclient master: Add dns-domain support to Network object https://review.openstack.org/516701 | |
| 23:05:53 | openstackgerrit | Merged openstack/openstacksdk master: Stop calling get_all_types when service-type is None https://review.openstack.org/589212 | |
| 23:54:48 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Stop using the -consumer devstack jobs https://review.openstack.org/589675 | |
| 23:56:49 | mordred | Shrews: ^^ I think openstacksdk is sufficiently integrated with devstack that we shoudl do that | |
| #openstack-sdks - 2018-08-08 | |||
| 01:26:35 | adriant | mordred: will look! | |
| 01:27:21 | adriant | mordred: sorry I haven't had much time to look at anything SDK related. Been deep within billing related stuff for the last while. | |
| 09:14:50 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Add error message when network resources use "set" "unset" command https://review.openstack.org/579515 | |
| 11:09:20 | openstackgerrit | Yushiro FURUKAWA proposed openstack/osc-lib master: Revert "Fixes a bug with issubclass() being called on an instance" https://review.openstack.org/586617 | |
| 12:58:05 | Shrews | mordred: any idea what these failures are about? http://logs.openstack.org/75/589675/1/check/osc-functional-devstack-tips/cc9ccf7/testr_results.html.gz | |
| 13:00:11 | mordred | Shrews: I do not - osc's gate tests were broken until yesterday | |
| 13:21:31 | mordred | Shrews: got a sec for this one? https://review.openstack.org/#/c/589675/ should reduce some node churn | |
| 13:22:23 | Shrews | mordred: well, the tests I asked about are for that one | |
| 13:24:32 | mordred | ah - yeah | |
| 14:33:06 | amotoki | dtroyer: could you look at https://review.openstack.org/#/c/586617/ ? it is a revert in osc-lib which breaks neutronclient OSC plugin | |
| 14:38:37 | dmsimard | mordred: if it's any consolation, after doing even a bit of AWS I find the OpenStack APIs and Ansible modules for OpenStack MUCH better (and especially more consistent) | |
| 14:39:30 | dmsimard | The AWS APIs and modules are very inconsistent and unpredictable, it's almost like if they had been developed by entirely different companies | |
| 14:39:34 | dmsimard | so... good job ? :D | |
| 14:39:48 | dtroyer | amotoki: +2+W | |
| 14:40:10 | mordred | dmsimard: yay! thanks - that's great to hear | |
| 14:40:34 | amotoki | dtroyer: thanks. | |
| 14:43:13 | openstackgerrit | Merged openstack/osc-lib master: Remove PyPI downloads https://review.openstack.org/589369 | |
| 14:46:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 14:46:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| 14:46:43 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | |
| 14:46:43 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 17:43:42 | openstackgerrit | Merged openstack/osc-lib master: Revert "Fixes a bug with issubclass() being called on an instance" https://review.openstack.org/586617 | |
| 18:31:25 | frickler | Shrews: mordred: I'd guess those two failures are just racy tests that have unstable results, the job did pass before the recheck | |
| 18:32:39 | mordred | frickler: you mean https://review.openstack.org/#/c/589675/ ? - oh,you're right | |
| 18:32:54 | frickler | mordred: yes | |
| 18:33:00 | mordred | ++ | |
| 18:33:16 | mordred | I am pretty sure they are unrelated, since that patch doesn't touch that job at all | |
| 18:33:38 | frickler | ack | |
| 18:34:20 | frickler | dtroyer: with the dependency merged, I hope https://review.openstack.org/558824 at last is now ready for some final reviews | |
| 18:34:27 | Shrews | are we comfortable approving it then? | |
| 18:35:29 | mordred | Shrews: yeah - I think it should be fine | |
| 20:03:59 | openstackgerrit | Merged openstack/openstacksdk master: Stop using the -consumer devstack jobs https://review.openstack.org/589675 | |
| 20:15:08 | openstackgerrit | Merged openstack/python-openstackclient master: Add DNS support to floating IP commands https://review.openstack.org/558824 | |
| #openstack-sdks - 2018-08-09 | |||
| 00:09:42 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 00:09:53 | mordred | Shrews: I found new terrible things I can't wait to tell you about | |
| 00:10:21 | mordred | adriant: no worries re: time - you've been busy! I mostly want to make sure that once this stack goes green, we make sure it doesn't break you before we land it | |
| 00:11:12 | adriant | mordred: it mostly looks like stuff that shouldn't affect me, but I won't get a chance to play with it until next week at the earliest | |
| 00:11:25 | mordred | adriant: kk. no rush | |
| 00:11:30 | adriant | it's just the changing of the Resource internals to make it dict based right? | |
| 00:11:56 | mordred | adriant: yah- and renaming two of the methods - but most things shoudl be touching those Resource bits via proxy methods | |
| 00:12:18 | adriant | Yeah, none of my code goes that deep into the Resource class | |
| 00:12:27 | mordred | woot. then you should be fine | |
| 00:12:44 | adriant | Most of the stuff that I always ran into was the session stuff you fixed :) | |
| 00:14:13 | adriant | mordred: oh and https://review.openstack.org/#/c/544544/ totally forgot about that patch, but the person who proposed it never responded | |
| 00:45:56 | samueldmq | mordred: Shrews: I liked having openstack cloud and operator cloud as separate things. having that boundary made sense from user's perspective | |
| 00:48:43 | samueldmq | for instance, say I want to validate the abstraction layer against all public clouds. that means I would only consider calls for users, excluding operator's calls (which were in a separate thing) | |
| 03:02:29 | openstackgerrit | Rui Chen proposed openstack/openstacksdk master: Add more options to enable ansible testing feature https://review.openstack.org/590078 | |
| 08:56:10 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 09:17:35 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 10:02:59 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 10:28:41 | gtema | does anyone experience broken osc interactive mode? I do always get "unrecognized arguments" for each command. Unfortunately same happend even with 3.14.2, so likely problem is elsewhere | |
| 10:36:01 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 10:40:49 | openstackgerrit | Yang JianFeng proposed openstack/openstacksdk master: Add support for octavia's resuorces quota https://review.openstack.org/590193 | |
| 12:13:17 | openstackgerrit | Lajos Katona proposed openstack/python-openstackclient master: Detailed help message for QoS max-burst-kbps value https://review.openstack.org/588168 | |
| 12:25:56 | dtroyer | gtema: check your version of cmd2 (0.8.8) and cliff (2.13.0). Those are the versions I have in my development venv | |
| 12:27:17 | gtema | dtroyer: If I do a fresh 'pip install python-openstackclient' I get cmd2-0.9.3 and cliff-2.13.0 | |
| 12:29:43 | gtema | dtroyer, if I manually downgrade cmd2 to 0.8.8 it works, so probably freeze the version if the fix is not fast? | |
| 12:36:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Rename Resource get and update to not clash with dict https://review.openstack.org/589193 | |
| 12:36:32 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add computed attribute type and location to base resource https://review.openstack.org/589194 | |
| 12:36:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove special handling of stacks https://review.openstack.org/589195 | |
| 12:36:33 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make resource a dict subclass usable by shade layer https://review.openstack.org/530835 | |
| 12:42:22 | Shrews | mordred: i'm giddy with anticipation | |
| 12:43:20 | dtroyer | gtema: cmd2 has been a barrel of fun lately… we do have it capped at <0.9.0 for python 2, python3 is fine above that | |
| 12:43:50 | gtema | dtroyer: I'm having this issue with python 3.6 | |
| 12:44:22 | mordred | Shrews: in https://review.openstack.org/530835 - please to enjoy the bits in image.py and test_image.py | |
| 12:45:12 | mordred | Shrews: where the test was testing setting a property on the remote image called "properties" with the value '{"description": "This is not an image"}' | |
| 12:51:54 | Shrews | mordred: i don't yet know what the fundamental issue is, but i find it humorous that someone thought making the dict a string was the proper way to send the value | |
| 12:53:46 | mordred | Shrews: I thought you'd enjoy that | |
| 12:54:18 | Shrews | or maybe it's sad that that is what the api actually expects?? | |
| 12:54:37 | Shrews | i put nothing past openstack apis anymore | |
| 12:54:52 | mordred | nope. the api expects extra key-value pairs to be sent in the main image dict (this is one of the differences with v1 of the glance api which has a properties field) | |
| 12:55:13 | mordred | Shrews: so the sdk in this case is actually not doing that - and is not actually supporting dealing with the extra key-value pairs | |
| 12:56:15 | Shrews | so glance broke their own api by removing properties? | |
| 12:57:12 | Shrews | i mean, i guess you can do that moving from v1 to v2... but that seems silly | |
| 12:57:24 | Shrews | meh | |
| 12:57:57 | gtema | dtroyer: even cmd2==0.9.0.1 has a problem already | |
| 12:58:35 | mordred | Shrews: no - glance didn't break their own api - the sdk code implementing it never implemented it - and the test which theoretically showed it worked wasn't actually testing it | |
| 12:58:39 | mordred | Shrews: but yeah | |
| 12:59:33 | Shrews | ah | |
| 13:02:22 | mordred | Shrews: I think the whole stack should be green now - I had to add a property back to the https://review.openstack.org/589193 to make osc happy | |
| 13:51:18 | dtroyer | gtema: hmmm, ok, maybe py3 should be capped too… | |
| 14:25:18 | mordred | Shrews: if you get a sec, https://review.openstack.org/#/c/586727 could use eyes - it's needed for https://review.openstack.org/#/c/586728/ to work | |
| 14:40:10 | mordred | (we'll also need to cut a release with that in it) | |
| 14:40:51 | Shrews | +Ad | |
| 14:42:26 | mordred | yay | |
| 14:50:54 | elmiko | cdent edleafe dtantsur|brb heads up, i have a conflict at the start of the meeting, will most likely be late | |
| 14:53:32 | edleafe | elmiko: good, we'll have some time to gossip about you | |
| 14:55:10 | elmiko | + | |
| 14:55:13 | elmiko | ++ even | |
| 15:07:28 | elmiko | i might be signed up for a phone call after the conflicting meeting, no guarantees that i will make our meeting | |
| 15:16:45 | cdent | something something sparkle motion | |
| 15:52:30 | openstackgerrit | Merged openstack/openstacksdk master: Update config doc url to point to openstacksdk https://review.openstack.org/586726 | |
| 15:58:26 | openstackgerrit | Merged openstack/openstacksdk master: Handle image and object key metadata for shade transition https://review.openstack.org/586727 | |
| 16:29:10 | cdent | edleafe: you wanna go ahead merge https://review.openstack.org/#/c/589130/ as it's just a naming thing? | |
| 16:30:25 | cdent | edleafe, dtantsur, elmiko : In fact I'm not sure _any_ of that stack warrants full guideline-style review | |
| 16:30:38 | cdent | it's basically reformatting and linking | |