| opendevreview | Gaudenz Steinlin proposed openstack/openstacksdk master: Use 2.87 Nova microversion to fetch hypervisors https://review.opendev.org/c/openstack/openstacksdk/+/925168 | 07:57 |
|---|---|---|
| stephenfin | gtema: I have a large number of purely typing-related reviews for SDK starting here https://review.opendev.org/c/openstack/openstacksdk/+/924847. Are you interested in reviewing them or will I just self-approve the lot? | 10:35 |
| gtema | I'll look over | 10:35 |
| stephenfin | ack | 10:36 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: [WIP]: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | 11:05 |
| frickler | gtema: stephenfin: can you decide on either https://review.opendev.org/c/openstack/openstacksdk/+/923671 or https://review.opendev.org/c/openstack/openstacksdk/+/922390 ? I'm fine with either choice but I really hate to regularly see failing, non-voting jobs being run in gate | 11:46 |
| gtema | fricker: ack, looking | 11:49 |
| gtema | You are right, non voting in gate is pointless | 11:50 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: [WIP]: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | 11:58 |
| opendevreview | Merged openstack/openstacksdk master: Added missing stack documentation and rewrite stack proxy document https://review.opendev.org/c/openstack/openstacksdk/+/921863 | 12:01 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Replace reference to removed variable https://review.opendev.org/c/openstack/openstacksdk/+/924847 | 12:11 |
| opendevreview | Merged openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/923221 | 12:11 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Remove check for nova extensions https://review.opendev.org/c/openstack/openstacksdk/+/923213 | 12:13 |
| opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | 12:40 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/923216 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (2/2) https://review.opendev.org/c/openstack/openstacksdk/+/923217 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Misc type fixes https://review.opendev.org/c/openstack/openstacksdk/+/923218 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Combine networking-related classes https://review.opendev.org/c/openstack/openstacksdk/+/923220 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.cloud https://review.opendev.org/c/openstack/openstacksdk/+/923219 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove TestClustering functional tests https://review.opendev.org/c/openstack/openstacksdk/+/923223 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.tests.functional https://review.opendev.org/c/openstack/openstacksdk/+/923224 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable warn_return_any https://review.opendev.org/c/openstack/openstacksdk/+/923226 | 13:05 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: mypy: Enable stricter type checking https://review.opendev.org/c/openstack/openstacksdk/+/925199 | 13:05 |
| ganso | stephenfin: hi! just following up on our previous discussion, would it make sense to merge https://review.opendev.org/c/openstack/openstacksdk/+/922965 as-is while the large rework you're working on is still far from merging? | 13:41 |
| stephenfin | sure | 13:44 |
| opendevreview | Merged openstack/openstacksdk master: zuul: Use more stable dib job https://review.opendev.org/c/openstack/openstacksdk/+/923671 | 14:57 |
| opendevreview | Merged openstack/openstacksdk master: block_storage: Add support for project_id in Limits https://review.opendev.org/c/openstack/openstacksdk/+/922965 | 15:39 |
| gibi | gtema: thanks for the hints about instantiating a sdk connection based on a user context. This seems to work how I expect https://review.opendev.org/c/openstack/nova/+/925277/1/nova/utils.py#1003 If you have any comments about it then feel free to leave it on the review | 16:34 |
| gtema | gibi: perfect | 16:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!