| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-21 | |||
| 10:47:09 | sean-k-mooney | gibi: i dont have it deploy but i have worked with dvd on https://bugs.launchpad.net/nova/+bug/1851545 downstream https://bugzilla.redhat.com/show_bug.cgi?id=1767797 | |
| 10:47:09 | openstack | Launchpad bug 1851545 in OpenStack Compute (nova) "Port update exception on nova unshelve for instance with PCI devices (part 2)" [Undecided,New] | |
| 10:47:10 | openstack | bugzilla.redhat.com bug 1767797 in openstack-nova "When unshelving an SR-IOV instance, the binding profile isn't reclaimed or rescheduled, and this might cause PCI-PT conflicts" [High,New] - Assigned to nova-maint | |
| 10:48:06 | sean-k-mooney | gibi: it looks like all my comments downstream are private but yes this is broken on queens i memtioned this seperatly before. we are not updating the port binding profile before generating the xml on unshelve | |
| 10:48:33 | sean-k-mooney | im not sure if it affect all release but its broken on queens | |
| 10:50:57 | sean-k-mooney | gibi: ill traige the upstream bug. i can proably try and replicate it later i just have not had time to work on it since it was first reported but now might be as good a time as any one i finish rebasing my current patches | |
| 11:11:07 | gibi | sean-k-mooney: thanks a lot! | |
| 11:29:57 | openstackgerrit | Merged openstack/nova master: Add missing white spaces between words in log messages https://review.opendev.org/720112 | |
| 12:03:16 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Revert "Temporarily skip TestNovaMigrationsMySQL" https://review.opendev.org/718630 | |
| 12:15:21 | gibi | bauzas and others: I have to jump on a meeting marathon for the next 2 and a half hours | |
| 12:19:47 | openstackgerrit | Merged openstack/nova-specs master: Adds spec infrastructure for Victoria https://review.opendev.org/710023 | |
| 12:48:35 | bauzas | gibi: ack thanks, and I'll review https://review.opendev.org/#/c/719163/ | |
| 13:05:10 | openstackgerrit | Merged openstack/nova master: docs: Add stable device rescue docs https://review.opendev.org/700837 | |
| 13:16:31 | kashyap | lyarwood: Since I've written it partially, and been vacillating on this for a while, I'll repropose this spec for V, as that is on TODO "forever" anyway. I know you've already got plenty on your plate | |
| 13:17:01 | kashyap | lyarwood: And I'll add you to the 'collaborators' list, that sounds okay? | |
| 13:17:32 | lyarwood | ack sure | |
| 13:20:01 | kashyap | lyarwood: Appreciate the virtual kick in the *** | |
| 13:20:38 | kashyap | A real one will also be accepted when circumstances allow in-person gathering ;-) | |
| 13:22:44 | gmann | gibi: stephenfin thanks. let me squash both policy doc patches. | |
| 13:43:37 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type https://review.opendev.org/708701 | |
| 13:45:25 | lyarwood | kashyap: ^ added a virt-preview job, I think swap volume might only work once we switch to block copy | |
| 13:45:32 | lyarwood | kashyap: with q35 | |
| 13:46:59 | kashyap | lyarwood: Ah, yes. Indeed that's true | |
| 13:49:05 | lyarwood | that said I can't reproduce this outside of the tempest test | |
| 13:55:26 | lyarwood | yup np | |
| 14:20:26 | openstackgerrit | Mark Goddard proposed openstack/nova master: Add functional regression test for bug 1853009 https://review.opendev.org/695012 | |
| 14:20:26 | openstack | bug 1853009 in OpenStack Compute (nova) ussuri "Ironic node rebalance race can lead to missing compute nodes in DB" [High,In progress] https://launchpad.net/bugs/1853009 - Assigned to Mark Goddard (mgoddard) | |
| 14:20:26 | openstackgerrit | Mark Goddard proposed openstack/nova master: Clear rebalanced compute nodes from resource tracker https://review.opendev.org/695187 | |
| 14:20:26 | openstackgerrit | Mark Goddard proposed openstack/nova master: Invalidate provider tree when compute node disappears https://review.opendev.org/695188 | |
| 14:20:27 | openstackgerrit | Mark Goddard proposed openstack/nova master: Prevent deletion of a compute node belonging to another host https://review.opendev.org/694802 | |
| 14:20:27 | openstackgerrit | Mark Goddard proposed openstack/nova master: Fix inactive session error in compute node creation https://review.opendev.org/695189 | |
| 14:20:39 | mgoddard | bauzas: ^ | |
| 14:21:34 | mgoddard | unit test fix was simple, missing a fair=True argument to the synchronized decorator in RT, added upstream | |
| 14:25:21 | bauzas | mgoddard: ack, will review it later today | |
| 14:25:28 | mgoddard | thanks! | |
| 14:26:49 | bauzas | stephenfin: can you please help me understand what you mean by https://review.opendev.org/#/c/721548/1/releasenotes/notes/ussuri-prelude-4b96f1244cefcdf4.yaml@12 ? | |
| 14:27:07 | bauzas | stephenfin: looking at https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html#hyperlinks I can't find what you mean | |
| 14:27:15 | stephenfin | bauzas: | |
| 14:27:17 | stephenfin | `Hello, world`__ | |
| 14:27:18 | stephenfin | ||
| 14:27:26 | stephenfin | __ https://example.com | |
| 14:27:52 | stephenfin | Or grep for '__' in releasenotes/notes | |
| 14:27:54 | bauzas | okay, but that's not something related to sphinx rST right? | |
| 14:28:13 | stephenfin | It's a general rST thing, not specific to Sphinx | |
| 14:28:46 | stephenfin | bauzas: https://docutils.sourceforge.io/docs/user/rst/quickref.html#indirect-hyperlink-targets | |
| 14:28:51 | bauzas | ok, /me goes looking at https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html | |
| 14:29:01 | bauzas | heh, jinxed | |
| 14:29:10 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type https://review.opendev.org/708701 | |
| 14:30:37 | bauzas | ah, OK, I understand better : "The reference name of the reference is not used to match the reference to its target. Instead, the order of anonymous hyperlink references and targets within the document is significant: the first anonymous reference will link to the first anonymous target. The number of anonymous hyperlink references in a document must match the number of anonymous targets. " | |
| 14:31:42 | stephenfin | Yup. Named links must be unique in their document so they chance of having conflicts in release notes is reasonably high, given the volume of them we have | |
| 14:32:07 | stephenfin | *so the chance | |
| 14:47:37 | stephenfin | dansmith: Am I wrong, or should this not happen? https://bugs.launchpad.net/nova/+bug/1868033 | |
| 14:47:37 | openstack | Launchpad bug 1868033 in OpenStack Compute (nova) "Booting instance with pci_device fails during rocky->stein live upgrade" [High,Confirmed] - Assigned to Stephen Finucane (stephenfinucane) | |
| 14:47:38 | stephenfin | Because of https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L555-L564 | |
| 14:50:03 | dansmith | stephenfin: well, he's saying that it _is_ trying to backport, but it's not backporting far enough, which makes it sound like the manifest being sent it wrong or something | |
| 14:50:08 | dansmith | which really shouldn't happen | |
| 14:50:31 | dansmith | he's wrong to equate the upgrade_level to anything with objects though -- they are unrelated | |
| 14:50:55 | stephenfin | yeah, that patch isn't a suitable fix | |
| 14:53:21 | dansmith | stephenfin: I wonder if it's because that object isn't calling super() | |
| 14:53:56 | dansmith | and its obj_make_compat() is only handling the sub-object backporting, which shouldn't be needed anymore anyway, so maybe this one got skipped when those were culled? | |
| 14:55:23 | stephenfin | Ah, good spot. That's likely it | |
| 14:55:53 | stephenfin | the superclass handles ListOfObjectField and ObjectField fields explicitly | |
| 14:56:04 | dansmith | right, I'm having to reboot my dev machine, | |
| 14:56:18 | dansmith | but I think you'll find most list objects don't have an explicit backport routine anymore | |
| 14:57:33 | dansmith | yeah, confirmed, but let me look up the culling patch | |
| 14:59:29 | dansmith | ah, I think I see | |
| 15:00:12 | dansmith | we used to use that obj_relationships mapping, and obj_make_compatible was still centralized based on that map for lists | |
| 15:00:35 | dansmith | but this one didn't use that, so it was overriding from the beginning and thus didn't get caught in the grep | |
| 15:02:21 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Ussuri 21.0.0 prelude section https://review.opendev.org/721548 | |
| 15:02:25 | dansmith | stephenfin: I think we need to get sam to test the proper fix..happy to do that unless you've already started | |
| 15:02:43 | stephenfin | dansmith: Nope, go for it | |
| 15:05:40 | bauzas | gibi: stephenfin: if you don't mind, as I said in a comment, I don't want to depend on https://review.opendev.org/#/c/720129/ | |
| 15:06:07 | bauzas | we had a terrible experience once in Newton when we merged the prelude reno file *after* we branched to Ocata (IIRC) | |
| 15:06:28 | bauzas | because the prelude wasn't seen in 14.0.0 then | |
| 15:07:31 | bauzas | yeah, I remembered correctly : https://docs.openstack.org/releasenotes/nova/newton.html#release-notes-14-0-1-stable-newton | |
| 15:08:52 | bauzas | to avoid this, you need to first merge a prelude reno file in the master, then once you branch, you can backport a fix for the file | |
| 15:13:51 | gibi | bauzas: OK, then lets have a preliminary prelude merged soon, and fix it when the doc is available | |
| 15:20:13 | openstackgerrit | Dan Smith proposed openstack/nova master: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/721667 | |
| 15:27:09 | gibi | bauzas: I still not get where we use aggregate metadata for image pre-chacing feature. I think we simply use aggregates but not aggregate metadata | |
| 15:27:27 | bauzas | gibi: okay, that's maybe me | |
| 15:27:46 | bauzas | I may have misunderstood the API version | |
| 15:30:51 | bauzas | gibi: OK, you're right, will respin | |
| 15:31:02 | bauzas | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/approved/image-precache-support.html#os-aggregates | |
| 15:33:27 | gibi | thanks | |
| 15:36:32 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Ussuri 21.0.0 prelude section https://review.opendev.org/721548 | |
| 15:39:07 | gibi | bauzas: +2 ^^ | |
| 15:39:21 | bauzas | ack, thanks | |
| 15:39:34 | stephenfin | bauzas: think you missed something | |
| 15:39:55 | bauzas | we should hold a bit until a handful of cores and contributors have reviewed it I think | |
| 15:39:59 | bauzas | ahah, see | |
| 15:40:07 | bauzas | stephenfin: probably, tell me what :) | |
| 15:40:16 | stephenfin | I left a comment :) | |
| 15:41:04 | gibi | OK I will only vote +1 until most of the cores leave some feedback on it | |
| 15:41:09 | stephenfin | bauzas: fwiw, because you're not using Sphinx'isms there, you can use a tool like rst.ninjs.org for rendering | |
| 15:41:42 | stephenfin | (so long as you remove the yaml stuff and dedent things, obviously) | |
| 15:44:03 | bauzas | stephenfin: you can test locally with the releasenotes target | |
| 15:44:10 | bauzas | stephenfin: that's what I did and it worked | |
| 15:44:22 | bauzas | stephenfin: also replied | |
| 15:44:37 | bauzas | with the releasenotes tox* target, my bad | |
| 15:45:08 | bauzas | (but remember, you need to commit first your changes in order for reno to work | |