| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-18 | |||
| 10:26:02 | gibi | stephenfin: so have an explicit well named anchor in the nova doc and update the novaclient reno to refer to the well named anchor? | |
| 10:26:14 | stephenfin | I think that would be a better approach, yes | |
| 10:26:20 | stephenfin | Personally | |
| 10:26:47 | gibi | OK lets do that | |
| 10:26:57 | gibi | I can propose the fix | |
| 10:27:05 | stephenfin | Sounds good. Happy to review | |
| 10:27:09 | gibi | cool | |
| 11:10:30 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Reject requests to commit intermediary snapshot of an inactive instance https://review.opendev.org/c/openstack/nova/+/781138 | |
| 11:12:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432 | |
| 11:20:23 | lyarwood | gibi: mind if I add a cinder cross project session in the PTG pad, have a topic. | |
| 11:20:38 | gibi | lyarwood: please do | |
| 11:28:10 | lyarwood | done thanks | |
| 11:50:07 | lyarwood | elod: https://review.opendev.org/c/openstack/nova/+/758733/ is ready for review btw | |
| 12:31:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add a resource limits guide https://review.opendev.org/c/openstack/nova/+/781433 | |
| 12:31:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove duplicate TPM extra spec info https://review.opendev.org/c/openstack/nova/+/781435 | |
| 12:31:09 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add a real-time guide https://review.opendev.org/c/openstack/nova/+/781434 | |
| 12:31:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add SEV guide https://review.opendev.org/c/openstack/nova/+/781437 | |
| 12:31:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Remove duplicated PCI passthrough extra spec info https://review.opendev.org/c/openstack/nova/+/781436 | |
| 12:31:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Change formatting of hypervisor config guides https://review.opendev.org/c/openstack/nova/+/781439 | |
| 12:31:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add CPU models guide https://review.opendev.org/c/openstack/nova/+/781438 | |
| 12:31:12 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add libvirt misc doc https://review.opendev.org/c/openstack/nova/+/781440 | |
| 12:31:29 | stephenfin | Some good docs there. I was holding off until after feature freeze ^ | |
| 12:36:45 | kashyap | stephenfin: Cool; a separate CPU models guide is good; instead of that info buried elsewhere | |
| 12:37:02 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add image metadata property reference guide https://review.opendev.org/c/openstack/nova/+/756867 | |
| 12:37:10 | kashyap | stephenfin: We could also link docs like these into it: something like "See also: https://docs.openstack.org/nova/latest/admin/mitigation-for-Intel-MDS-security-flaws.html" | |
| 12:37:31 | kashyap | If only to make it more discoverable; /me adds a note in the review | |
| 12:37:40 | stephenfin | I'll probably move that doc into the cpu-models one (with a redirect) at some point | |
| 12:37:59 | stephenfin | It would fit nicely in there as a concrete use case | |
| 12:39:31 | kashyap | Yeah. Idd | |
| 12:41:24 | kashyap | stephenfin: There's additional things we can mention on 'host-model' that can be very useful for people to know _why_ we default to it | |
| 12:49:19 | elod | lyarwood: +2+W'd | |
| 13:02:39 | lyarwood | elod: thanks | |
| 13:10:27 | kashyap | stephenfin: Sorry for the wall o' text here, but I hope some useful crumbs in there: https://review.opendev.org/c/openstack/nova/+/781438 | |
| 13:13:20 | kashyap | s/hope/hope there are/ | |
| 13:23:26 | dansmith | lyarwood: ugh, yeah sorry.. fix a bug, get punished I guess | |
| 13:24:13 | dansmith | lyarwood: assuming that's going to affect all older branches, we may want to make tempest only rely on that behavior for new enough glance | |
| 13:28:09 | gibi | stephenfin: it seems '.. _2.88 microversion:' does not generate an anchor but '.. _microversion 2.88:' does generate 'microversion-2-88' as an anchor | |
| 13:28:53 | gibi | so I'm wondering how our other anchors starting with numbers in the microversion history document ever worked | |
| 13:28:59 | lyarwood | dansmith: no issues, no idea about the glance jobs but Nova only started testing that in stable/victoria so the backports should be enough for us | |
| 13:29:40 | dansmith | lyarwood: ahh, right | |
| 13:31:21 | dansmith | man gerrit is s l o w today | |
| 13:56:52 | openstackgerrit | Balazs Gibizer proposed openstack/python-novaclient master: Use well named anchor into the microversion history https://review.opendev.org/c/openstack/python-novaclient/+/781484 | |
| 14:03:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: doc: mark the max microversion for wallaby https://review.opendev.org/c/openstack/nova/+/780401 | |
| 14:04:51 | gibi | stephenfin: ^^ your turn | |
| 14:07:28 | mnaser | weirdest thing ever | |
| 14:07:46 | mnaser | we changed the db for a system (literally, just mysqldump | mysql) | |
| 14:07:59 | mnaser | and now, limit= from nova api is always returning exactly half of what we are supposed to get | |
| 14:08:16 | mnaser | https://www.irccloud.com/pastebin/BEMJghhM/ | |
| 14:08:20 | mnaser | anyone seen something like this? o_O | |
| 14:18:12 | openstackgerrit | Lee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent https://review.opendev.org/c/openstack/nova/+/720769 | |
| 14:25:54 | gmann | lyarwood: gibi stephenfin IMO, 400 is right one and if we change existing 409/403 to 400 it does not require microversion bump. | |
| 14:44:18 | lyarwood | mnaser: odd running master? | |
| 14:44:47 | lyarwood | stephenfin: ^ are you cool with gmann's suggestion? | |
| 14:57:07 | dansmith | elod: [07:53:00] <abhishekk> lyarwood, dansmith may be Elod will approve the patches while I am gone | |
| 14:57:16 | elod | lyarwood: I think https://review.opendev.org/781493 needs to be squashed with https://review.opendev.org/781426/ otherwise gate won't pass, am I right? | |
| 14:57:52 | elod | dansmith: sure | |
| 14:57:55 | abhishekk | elod, right | |
| 14:58:34 | lyarwood | jebus | |
| 14:58:36 | lyarwood | okay sorry | |
| 14:58:36 | abhishekk | dansmith, so to avoid all this dance, can we just drop lower constraints job? | |
| 14:59:01 | dansmith | abhishekk: yeah, for sure, I was just trying to avoid the dance since it was supposed to all be headed to the gate | |
| 14:59:13 | dansmith | abhishekk: but if you want to push that in underneath (or get lyarwood to do it) that's fine with me | |
| 14:59:16 | abhishekk | and sqash the dropping of nova-ceph job with action wrapper | |
| 14:59:39 | dansmith | I wasn't 100% sure we had settled on that as a wider community, as was asserted in the meeting, but I was definitely not following that whole thread | |
| 14:59:58 | abhishekk | ok, so lets keep it as it is at the moment | |
| 15:00:14 | lyarwood | I'll squash things now | |
| 15:00:25 | abhishekk | lyarwood, yep, go ahead | |
| 15:00:31 | dansmith | abhishekk: to be clear, I'm saying at this point, we're not headed for gate, so we can do whatever you want :) | |
| 15:00:51 | dansmith | lyarwood: has nova dropped the lower-constraints job(s)? | |
| 15:01:06 | abhishekk | dansmith, yes, lets merge that dropping lower constraint patch differently | |
| 15:01:16 | dansmith | abhishekk: ack | |
| 15:01:22 | abhishekk | after this is done | |
| 15:01:35 | elod | no, lc jobs are still there (only disabled on rocky(?)) | |
| 15:01:42 | lyarwood | dansmith: no | |
| 15:01:44 | dansmith | ack, okay | |
| 15:01:47 | gmann | dansmith: abhishekk we still have not concluded that what to do on l-c job | |
| 15:01:59 | gmann | discussing in today TC meeting | |
| 15:02:03 | abhishekk | gmann, ack, thanks | |
| 15:02:03 | dansmith | gmann: okay I thought it wasn't quite decided yet, but others seemed to think it was | |
| 15:43:30 | bauzas | gibi: dansmith: fwiw, I removed my -2 from https://review.opendev.org/c/openstack/nova/+/761452 as we're after the FF | |
| 15:43:32 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Use proper headers in nova-manage man page https://review.opendev.org/c/openstack/nova/+/726191 | |
| 15:43:33 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage db' docs to use proper directives https://review.opendev.org/c/openstack/nova/+/677509 | |
| 15:43:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage placement' docs https://review.opendev.org/c/openstack/nova/+/726193 | |
| 15:43:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage api_db', 'cell_v2' docs https://review.opendev.org/c/openstack/nova/+/726192 | |
| 15:43:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Populate "See Also" section with meaningful links https://review.opendev.org/c/openstack/nova/+/729917 | |
| 15:43:35 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document options for remaining nova-* commands https://review.opendev.org/c/openstack/nova/+/726195 | |
| 15:43:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite 'nova-manage libvirt' docs https://review.opendev.org/c/openstack/nova/+/781504 | |
| 15:43:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add man pages for 'nova-policy' https://review.opendev.org/c/openstack/nova/+/729918 | |
| 15:44:04 | bauzas | dansmith: still working on removing 5.0 but given we only have a week, maybe we should just merge the above ? ^ | |
| 15:45:29 | dansmith | bauzas: ack, I'm out tomorrow and won't get to revisiting that today I expect, so whatever gibi wants to do | |
| 15:45:57 | bauzas | dansmith: given we will remove the V5proxy in Xena, the 5.0 support would only be in the Wallaby release | |
| 15:46:17 | gibi | dansmith bauzas: if we land what we have today, does that pose any serious limitation? | |
| 15:46:45 | bauzas | gibi: mmmm, I don't know | |
| 15:47:07 | dansmith | gibi: well, I haven't reviewed the latest rev, but no, it's just more than we need and also it lies a bit about older versions we say we can support but can't really, but in practice it's probably not a big deal | |
| 15:47:11 | bauzas | we have a lof of merge conflicts that said => https://review.opendev.org/c/openstack/nova/+/761452 | |
| 15:47:51 | gibi | dansmith: OK, so it is safe to merge, and if time allows we can improve. But considering RC1 next week I don't think we will improve much | |
| 15:48:23 | dansmith | again, haven't reviewed the latest one, but the approach yeah | |
| 15:50:33 | gibi | I reviewed the changes between the version you reviewed and the latest and nothing major there. So I ugrade my vote to +2 | |
| 15:51:22 | gibi | btw, nova meeting starts in 10 minutes in #openstack-meeting-3 | |
| 15:55:24 | elod | lyarwood: about https://review.opendev.org/c/openstack/nova/+/758733 | |