| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-08 | |||
| 17:10:49 | DHilsbos | hw:virtual_displays is not present on that page. | |
| 17:11:43 | bauzas | correct, because it's virt-specific | |
| 17:12:03 | bauzas | which we now are relunctant to add | |
| 17:13:03 | DHilsbos | Ah, ok. | |
| 17:13:31 | DHilsbos | So, looking at naming, I think I would need something like "hw_video:count"? | |
| 17:14:23 | DHilsbos | I think I understand what is needed, thank you. | |
| 17:14:29 | bauzas | spec it up, I guess :) | |
| 17:20:53 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 17:30:24 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 17:31:16 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 17:49:13 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Document remote-managed port usage considerations https://review.opendev.org/c/openstack/nova/+/827513 | |
| 17:53:05 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 17:58:15 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 18:01:45 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 18:03:47 | opendevreview | Pedro Monteiro Azevedo de Moura Almeida proposed openstack/nova master: Change to make live_migration_downtime clearer https://review.opendev.org/c/openstack/nova/+/828387 | |
| 18:57:03 | opendevreview | Dan Smith proposed openstack/nova master: Join quota exception family trees https://review.opendev.org/c/openstack/nova/+/828185 | |
| 18:57:04 | opendevreview | Dan Smith proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/c/openstack/nova/+/712137 | |
| 18:57:04 | opendevreview | Dan Smith proposed openstack/nova master: Move keypair quota error message into exception https://review.opendev.org/c/openstack/nova/+/828186 | |
| 18:57:05 | opendevreview | Dan Smith proposed openstack/nova master: Make unified limits APIs return reserved of 0 https://review.opendev.org/c/openstack/nova/+/712141 | |
| 18:57:05 | opendevreview | Dan Smith proposed openstack/nova master: Assert quota related API behavior when noop https://review.opendev.org/c/openstack/nova/+/712140 | |
| 18:57:06 | opendevreview | Dan Smith proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 18:57:06 | opendevreview | Dan Smith proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 18:57:07 | opendevreview | Dan Smith proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 18:57:08 | opendevreview | Dan Smith proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 18:57:08 | opendevreview | Dan Smith proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 18:57:10 | opendevreview | Dan Smith proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 18:57:10 | opendevreview | Dan Smith proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 18:57:12 | opendevreview | Dan Smith proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 18:57:12 | opendevreview | Dan Smith proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 18:57:14 | opendevreview | Dan Smith proposed openstack/nova master: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 18:57:14 | opendevreview | Dan Smith proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 18:57:16 | dansmith | melwitt: ^ | |
| 18:57:27 | dansmith | I think it turned out nice | |
| 18:57:50 | dansmith | there's one thing in the placement patch where it was raising quotaerror more like an actual error because of a placement failure | |
| 18:58:04 | dansmith | but it was raising quotaerror which was actually the over-quota case before, so I converted it as such, | |
| 18:58:28 | dansmith | but I think we should probably not just be copying the api/network/whatever failure into the OverQuota.message, for the same reason as the session init thing from earlier | |
| 18:59:09 | melwitt | dansmith: ok cool, thanks | |
| 19:16:57 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 19:27:55 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/828413 | |
| 19:27:56 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/828414 | |
| 19:28:01 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 19:32:40 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 19:40:13 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add nova-ovs-hybrid-plug job https://review.opendev.org/c/openstack/nova/+/828418 | |
| 19:40:14 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Revert "Revert resize: wait for events according to hybrid plug" https://review.opendev.org/c/openstack/nova/+/828419 | |
| 19:40:35 | artom | fsck me that was mostly clean to wallaby, and I don't think we need it further | |
| 19:41:49 | artom | Well, at least downstream we don't | |
| 19:42:55 | artom | ... and same for upstream \o/ | |
| 19:43:08 | artom | For some reason I was expecting stable/train levels of pain | |
| 19:46:04 | chateaulav | https://www.irccloud.com/pastebin/XwYHRJuH/ | |
| 19:46:19 | chateaulav | any advice is appreciated | |
| 19:51:32 | artom | chateaulav, you don't need Depends-on if it's in the same project (nova) | |
| 19:51:39 | artom | Just stack them on top of each other | |
| 19:52:03 | artom | IOW, locally `git commit` as many patches as makes sense, then `git review` them in one shot | |
| 19:53:33 | chateaulav | ok, so then to make sure i understand. remove the depends on, make all my commits and then when all the commits are staged do review and it will run all of them. | |
| 19:55:00 | artom | Yep | |
| 19:55:16 | artom | So if you have A depends on B depends on C | |
| 19:55:28 | artom | Commit C first, then B, then A, then submit for review | |
| 19:56:32 | artom | And if you have A depends on B depends on C but also D depends on C | |
| 19:56:50 | artom | Commit C first, then C and D in either order, then A | |
| 19:56:57 | artom | "linearize" it, so to speak | |
| 19:57:17 | artom | Sorry, then B and D in any order, then A | |
| 20:12:02 | chateaulav | artom: ok, makes sense | |
| 20:18:06 | opendevreview | Pedro Monteiro Azevedo de Moura Almeida proposed openstack/nova master: Update live_migration_downtime definition https://review.opendev.org/c/openstack/nova/+/828387 | |
| 20:25:37 | opendevreview | Jonathan Race proposed openstack/nova master: zuul-job for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828372 | |
| 20:26:08 | opendevreview | Jonathan Race proposed openstack/nova master: object/notification for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/828369 | |
| 20:26:19 | opendevreview | Jonathan Race proposed openstack/nova master: driver/secheduler/docs for Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/nova/+/822053 | |
| 20:54:08 | opendevreview | Pedro Monteiro Azevedo de Moura Almeida proposed openstack/nova master: Update live_migration_downtime definition https://review.opendev.org/c/openstack/nova/+/828387 | |
| 22:23:49 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675 | |
| 22:23:49 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Introduce remote_managed tag for PCI devs https://review.opendev.org/c/openstack/nova/+/824834 | |
| 22:23:50 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111 | |
| 22:23:50 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Add supports_remote_managed_ports capability https://review.opendev.org/c/openstack/nova/+/827839 | |
| 22:23:51 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Document remote-managed port usage considerations https://review.opendev.org/c/openstack/nova/+/827513 | |
| 22:23:51 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_REMOTE_MANAGED https://review.opendev.org/c/openstack/nova/+/824835 | |
| 22:56:49 | opendevreview | Merged openstack/nova master: neutron: Rework how we check for extensions https://review.opendev.org/c/openstack/nova/+/705792 | |
| 23:32:56 | opendevreview | melanie witt proposed openstack/placement master: Make perfload jobs fail if write allocation fails https://review.opendev.org/c/openstack/placement/+/828438 | |
| 23:58:31 | opendevreview | melanie witt proposed openstack/placement master: Make perfload jobs fail if write allocation fails https://review.opendev.org/c/openstack/placement/+/828438 | |
| #openstack-nova - 2022-02-09 | |||
| 06:38:10 | opendevreview | melanie witt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 06:38:10 | opendevreview | melanie witt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/c/openstack/nova/+/712139 | |
| 06:38:11 | opendevreview | melanie witt proposed openstack/nova master: Update limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 06:38:11 | opendevreview | melanie witt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/c/openstack/nova/+/712143 | |
| 06:38:12 | opendevreview | melanie witt proposed openstack/nova master: Tell oslo.limit how to count nova resources https://review.opendev.org/c/openstack/nova/+/713301 | |
| 06:38:12 | opendevreview | melanie witt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 06:38:13 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 06:38:13 | opendevreview | melanie witt proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 06:38:14 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 06:38:14 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 06:38:16 | opendevreview | melanie witt proposed openstack/nova master: Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 08:31:59 | gibi | melwitt: thanks a lot for the quick re-review on the any-traits series | |
| 10:50:52 | tobias-urdin | gibi: sorry for the delay but i've updated the bugfix as requested https://review.opendev.org/c/openstack/nova/+/827865 | |
| 10:58:29 | gibi | tobias-urdin: thanks. I'm +2 on the fix | |
| 11:01:39 | sean-k-mooney | looks good to me too | |
| 11:07:55 | opendevreview | Felix Huettner proposed openstack/nova master: Gracefull recovery when attaching volume fails https://review.opendev.org/c/openstack/nova/+/828516 | |
| 11:22:06 | tobias-urdin | thx, you guys ok with backporting with the rename of the internal function in libvirt driver? | |
| 11:22:36 | sean-k-mooney | its not part of the driver api so it should be fine | |
| 11:22:58 | tobias-urdin | ack | |
| 11:24:05 | gibi | yepp, feel free to propose the backports | |
| 11:24:54 | opendevreview | Tobias Urdin proposed openstack/nova stable/wallaby: libvirt: Add announce-self post live-migration workaround https://review.opendev.org/c/openstack/nova/+/825178 | |
| 11:56:38 | sean-k-mooney | gibi: dmitriis im +2 on all the off-path backend series bar the docs patch at the end | |
| 11:57:29 | dmitriis | sean-k-mooney: ack, I am finishing up some changes to the doc patch | |
| 11:57:56 | dmitriis | interestingly, the mlx5 driver doesn't allow "trusted" to be set on a VF so I'll document that as a limitation | |