| opendevreview | 신민경 proposed openstack/python-openstackclient master: Add functional tests for image info commands https://review.opendev.org/c/openstack/python-openstackclient/+/958908 | 10:36 |
|---|---|---|
| opendevreview | 신민경 proposed openstack/python-openstackclient master: Add functional tests for image info commands https://review.opendev.org/c/openstack/python-openstackclient/+/958908 | 12:13 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Implement conflict resolution https://review.opendev.org/c/openstack/python-openstackclient/+/968178 | 12:28 |
| opendevreview | 신민경 proposed openstack/python-openstackclient master: Add functional tests for image info commands https://review.opendev.org/c/openstack/python-openstackclient/+/958908 | 13:10 |
| greatgatsby | Good day. After upgrading from the python openstack SDK library version 4.7.1 to 4.8.0, it seems that I can no longer do a simple conn.dns.recordsets(all_projects=True, status="PENDING"). This now throws an KeyError for zone_id. I'm digging through the code now, but I'm hoping there's a workaround so I don't have to iterate over each tenant/zone, as this was way too slow when we first tested it | 13:51 |
| greatgatsby | that way. | 13:51 |
| lutimura | hey stephenfin, i'm trying to get https://review.opendev.org/c/openstack/python-openstackclient/+/958806 in, but, for some reason, the zuul jobs are still failing despite openstacksdk 4.8.0 has already been released (with commit 839e3ccc6). what am i missing here? :( | 15:00 |
| tkajinam | lutimura, I guess you have to revert https://review.opendev.org/c/openstack/python-openstackclient/+/958872 ? | 15:37 |
| tkajinam | greatgatsby, if you could report a bug in https://bugs.launchpad.net/openstacksdk with some more details (like traceback) that would be helpful | 15:40 |
| opendevreview | Merged openstack/os-api-ref master: Remove reference to tag framework https://review.opendev.org/c/openstack/os-api-ref/+/968120 | 16:16 |
| opendevreview | cid proposed openstack/openstacksdk master: Add Node.instance_name support https://review.opendev.org/c/openstack/openstacksdk/+/955103 | 16:17 |
| lutimura | tkajinam: oh, you're right! i totally forgot about that one heh | 16:29 |
| tkajinam | lutimura, I guess it's better to bump min version of sdk, too, but I'm not 100% familiar with the existing workflow so will defer it to stephenfin (or other sdk cores) | 16:30 |
| lutimura | tkajinam: do you mean in openstack/requirements? i.e. https://review.opendev.org/c/openstack/requirements/+/967863 | 16:32 |
| lutimura | i'm thinking about reverting the change you mentioned in https://review.opendev.org/c/openstack/python-openstackclient/+/958806 (i guess it has to be done this way...) | 16:34 |
| tkajinam | lutimura, no I mean the line in requirements.txt in python-openstackclient | 16:36 |
| tkajinam | hmm. seems we should do it actually. | 16:38 |
| tkajinam | at least for test code, but I guess it's still useful for functional code | 16:38 |
| tkajinam | lutimura, yeah I agree with doing that revert WITHIN your change | 16:40 |
| lutimura | ack! about requirements.txt, we wait for stephenfin (or other sdk cores) and handle it on a separate change? | 16:43 |
| tkajinam | maybe or within yours if they want to bump it now | 16:46 |
| tkajinam | update the change with the required revert now, and hear any feedback | 16:47 |
| opendevreview | Luan Utimura proposed openstack/python-openstackclient master: volume: Add missing backup_id field in tests https://review.opendev.org/c/openstack/python-openstackclient/+/958806 | 16:52 |
| opendevreview | Merged openstack/python-openstackclient master: volume: Add missing backup_id field in tests https://review.opendev.org/c/openstack/python-openstackclient/+/958806 | 19:48 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!