| gmann | melwitt: sean-k-mooney artom gibi, if we return 409 then it need microversion. Only way to avoid microversion is if we are returning existing error code (400, 403, 404 and 415 are always return from all APIs). I think returning 400 here as gibi mentioned is good candidate. | 00:51 |
|---|---|---|
| melwitt | thanks gmann for confirming | 01:13 |
| opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add flavor scanning to migrate_to_unified_limits https://review.opendev.org/c/openstack/nova/+/924110 | 03:10 |
| opendevreview | Merged openstack/nova-specs master: Repropose spice-direct console support. https://review.opendev.org/c/openstack/nova-specs/+/932387 | 08:45 |
| stephenfin | gmann: sean-k-mooney: I wonder if you'd be able to take another look through the openapi series today/early next week. It starts here (Sean has already reviewed the first few) https://review.opendev.org/c/openstack/nova/+/915742 | 11:20 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Update Nova bdm with updated swap info https://review.opendev.org/c/openstack/nova/+/929858 | 11:30 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.opendev.org/c/openstack/nova/+/873901 | 13:21 |
| gibi | I feel sad about https://bugs.launchpad.net/nova/+bug/1854844/comments/7 | 15:08 |
| artom | Yeah, that's not great UX | 15:17 |
| artom | That being said, I think having an expectation that you can just plop a patch into a comment and have it magically fixed and merged by someone else is misplaced as well | 15:17 |
| dansmith | yep | 15:18 |
| dansmith | definitely unfortunate, but... | 15:18 |
| gibi | sure they shown goodwill by sharing the patch but we shown goodwill to keeping nova alive | 15:18 |
| gibi | so if they can have a limit the we can have it too | 15:20 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: Handle MAC dash-addresses during migration https://review.opendev.org/c/openstack/nova/+/919760 | 16:05 |
| gmann | stephenfin: ack, will check today or early next week | 17:29 |
| opendevreview | melanie witt proposed openstack/nova master: nova-manage: Add flavor scanning to migrate_to_unified_limits https://review.opendev.org/c/openstack/nova/+/924110 | 18:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!