| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-02-08 | |||
| 17:02:14 | bauzas | DHilsbos: then you'll need a blueprint | |
| 17:02:30 | bauzas | DHilsbos: you can start writing this blueprint | |
| 17:02:40 | bauzas | and then working on patches | |
| 17:02:58 | DHilsbos | Ok, that's what I trhought. Are there blueprint forms? | |
| 17:03:09 | bauzas | but we won't look at both patches and the blueprint until RC1 which is when Z becomes a thing | |
| 17:03:27 | dansmith | I'm sure anything with that description requires a spec and not just a blueprint | |
| 17:03:29 | dansmith | and yes, there's a spec template :) | |
| 17:03:30 | bauzas | DHilsbos: there is even a button | |
| 17:03:48 | bauzas | dansmith: don't disagreed, just oversimplified the explanation | |
| 17:04:10 | DHilsbos | I'm not suprised by the need for the spec. | |
| 17:04:25 | bauzas | https://docs.openstack.org/nova/latest/contributor/process.html#when-do-i-need-a-blueprint-vs-a-spec | |
| 17:04:25 | DHilsbos | So; Blueprint, spec, patches, yeah? | |
| 17:04:42 | bauzas | blueprint is the first procedural step | |
| 17:05:35 | DHilsbos | Ok. | |
| 17:05:45 | bauzas | then you'd have to upload a spec if you really care in advance but given the directory name isn't created yet due to legal naming implications ( :) ) you'd have to either refrain to push or fake a name when you uploda | |
| 17:06:04 | DHilsbos | Anyone on that is famialir with the Hyper-V driver code? | |
| 17:06:12 | bauzas | and then rebase your spec change with the proper dirname once known and created | |
| 17:06:57 | bauzas | DHilsbos: I guess you'll find plenty of people who are familiar with the process that aren't familiar with Hyper-V | |
| 17:07:15 | bauzas | the contrary seems difficult to me | |
| 17:08:11 | bauzas | DHilsbos: but yeah, if you really care about a feature, just start writing a few things in a blueprint and try to grab me or someone else once we're free | |
| 17:08:11 | DHilsbos | I need to know what the hw:virtual_displays flavor extra_spec does within Hyper-V | |
| 17:08:22 | opendevreview | Merged openstack/os-resource-classes master: Test all supported python version https://review.opendev.org/c/openstack/os-resource-classes/+/819207 | |
| 17:08:27 | bauzas | gmann: ^ | |
| 17:08:39 | gmann | bauzas: thanks | |
| 17:09:08 | DHilsbos | It can wait for Z naming to be addressed. I'll start writing the blueprint. | |
| 17:09:42 | bauzas | DHilsbos: https://docs.openstack.org/nova/latest/configuration/extra-specs.html#hw | |
| 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: Move keypair quota error message into exception https://review.opendev.org/c/openstack/nova/+/828186 | |
| 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: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: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: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:06 | opendevreview | Dan Smith proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/c/openstack/nova/+/712142 | |
| 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 limit APIs https://review.opendev.org/c/openstack/nova/+/712707 | |
| 18:57:08 | opendevreview | Dan Smith proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/c/openstack/nova/+/712749 | |
| 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:10 | opendevreview | Dan Smith proposed openstack/nova master: Enforce resource limits using oslo.limit https://review.opendev.org/c/openstack/nova/+/615180 | |
| 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: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:14 | opendevreview | Dan Smith proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 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: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: Introduce remote_managed tag for PCI devs https://review.opendev.org/c/openstack/nova/+/824834 | |
| 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:50 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: Add supports_remote_managed_ports capability https://review.opendev.org/c/openstack/nova/+/827839 | |
| 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:51 | opendevreview | Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_REMOTE_MANAGED https://review.opendev.org/c/openstack/nova/+/824835 | |
| 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:56:49 | opendevreview | Merged openstack/nova master: neutron: Rework how we check for extensions https://review.opendev.org/c/openstack/nova/+/705792 | |