| openstackgerrit | Adam Harwell proposed openstack/openstacksdk master: Show owner_id AND owner as properties https://review.opendev.org/726562 | 00:27 |
|---|---|---|
| openstackgerrit | Adam Harwell proposed openstack/python-openstackclient master: Ignore `self` property if passed on image create https://review.opendev.org/726563 | 00:30 |
| rm_work | mordred: ^^ fixed. | 00:33 |
| rm_work | any idea when https://review.opendev.org/#/c/725427/ might merge? Are you the only core reviewer really merging things at this point? T_T | 00:33 |
| openstackgerrit | Adam Harwell proposed openstack/openstacksdk master: Show owner_id AND owner as properties https://review.opendev.org/726562 | 01:04 |
| *** dayou has quit IRC | 01:24 | |
| *** dayou has joined #openstack-sdks | 01:44 | |
| openstackgerrit | Adam Harwell proposed openstack/python-openstackclient master: Ignore `self` property if passed on image create https://review.opendev.org/726563 | 02:16 |
| *** dayou has quit IRC | 04:15 | |
| *** evrardjp has quit IRC | 04:36 | |
| *** evrardjp has joined #openstack-sdks | 04:36 | |
| *** dayou has joined #openstack-sdks | 04:36 | |
| *** slaweq has joined #openstack-sdks | 06:22 | |
| rm_work | mordred: so i don't really understand why the one tox-py36-tips test would have the failures it does | 07:06 |
| rm_work | in https://review.opendev.org/#/c/726562/ | 07:06 |
| *** slaweq has quit IRC | 07:53 | |
| *** tosky has joined #openstack-sdks | 08:43 | |
| sshnaidm|off | need an opinion of SDK devs - are there possible problems when using latest SDK to manage openstack old release, like Queens for example? | 11:03 |
| *** slaweq has joined #openstack-sdks | 11:21 | |
| *** slaweq has quit IRC | 11:51 | |
| *** slaweq has joined #openstack-sdks | 11:56 | |
| *** slaweq has quit IRC | 12:00 | |
| mordred | sshnaidm|off: no, it should 100% work and is explicitly supported | 12:24 |
| mordred | rm_work: looking at it | 12:44 |
| openstackgerrit | Victor Coutellier proposed openstack/openstacksdk master: Add name query filter to keystone service https://review.opendev.org/726621 | 12:51 |
| sshnaidm|off | mordred, great, so modules should work too | 12:54 |
| mordred | sshnaidm|off: yes. if they don't it's a bug | 13:01 |
| mordred | sshnaidm|off: imo, there is no reason to ever use any release of openstacksdk other than the most recent one | 13:02 |
| mordred | rm_work: ok - looking in to this - I think this is a slightly different/worse issue :( | 13:03 |
| mordred | rm_work: "owner_id" should never have existed - that's an sdk-side API break. the glance value is owner, the sdk value has been owner for a very long time, and it looks like the osc value was owner before the update | 13:03 |
| mordred | rm_work: I'm going to see if I can't figure out how to fix it without things completely sucking | 13:04 |
| openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Ensure modules work with queens devstack https://review.opendev.org/726421 | 13:07 |
| mordred | rm_work: also, we're supposed to be stripping self from the image object in sdk - but we're apparently only doing it in list() | 13:20 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Strip self from incoming glance properties https://review.opendev.org/726626 | 13:21 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix image owner field https://review.opendev.org/726627 | 13:21 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove some unneeded things from test-requirements https://review.opendev.org/715707 | 13:27 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove uses of from six.moves https://review.opendev.org/715712 | 13:27 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Remove use of six https://review.opendev.org/715713 | 13:29 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Stop subclassing object https://review.opendev.org/715714 | 13:29 |
| *** cmorpheus is now known as cmurphy | 13:44 | |
| *** dayou has quit IRC | 13:56 | |
| *** dayou has joined #openstack-sdks | 13:58 | |
| *** adriant has quit IRC | 13:58 | |
| *** adriant has joined #openstack-sdks | 13:59 | |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: Updated allocation pool checks https://review.opendev.org/723064 | 14:21 |
| openstackgerrit | Dean Troyer proposed openstack/openstacksdk master: Set BaseImageProxy.create_image validate_checksum default to False https://review.opendev.org/726465 | 15:18 |
| openstackgerrit | Merged openstack/openstacksdk master: Add "id" to Port's query parameters https://review.opendev.org/726050 | 15:43 |
| *** adriant has quit IRC | 15:47 | |
| *** evrardjp has quit IRC | 16:36 | |
| *** evrardjp has joined #openstack-sdks | 16:36 | |
| openstackgerrit | Dean Troyer proposed openstack/openstacksdk master: Set BaseImageProxy.create_image validate_checksum default to False https://review.opendev.org/726465 | 17:03 |
| dtroyer | ^^^ That should pass functional tests now | 17:04 |
| *** tkajinam has joined #openstack-sdks | 22:46 | |
| *** adriant has joined #openstack-sdks | 22:59 | |
| rm_work | mordred: ok so we can do things in a slightly less silly way then, that's actually good? | 23:02 |
| rm_work | I was afraid we COULDNT strip "self" because it might be relied on? | 23:02 |
| rm_work | It's technically been published in a release where it does exist in the Show, are you sure we CAN remove it now? | 23:03 |
| rm_work | And owner_id was shown in the OSC for a long time I thought, not just after the switch to SDK, so same deal | 23:04 |
| rm_work | What I was seeing was "pre-update" owner_id in properties | 23:04 |
| rm_work | I literally just have an older client (4.x series) and the new client (5.x series) installed side by side and was plugging json output from an "image show" into a jsondiff utility | 23:05 |
| rm_work | And worked on the new one until the output was identical | 23:05 |
| *** tosky has quit IRC | 23:13 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!