| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-10-16 | |||
| 10:26:12 | openstackgerrit | Andrew Bonney proposed openstack/nova master: Handle disabled CPU features to fix live migration failures https://review.opendev.org/757577 | |
| 11:11:41 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore PCI devices with 32bit domain https://review.opendev.org/756697 | |
| 11:19:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore PCI devices with 32bit domain https://review.opendev.org/756697 | |
| 11:20:13 | gibi | sean-k-mooney, melwitt: I tried to follow the suggestion from the discussion you had yesterday about ^^. Let me know how do you like this | |
| 11:30:59 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Set instance host and drop migration under lock https://review.opendev.org/754815 | |
| 11:42:50 | frickler | gibi: I think you can be added to the chanserv config, but I need to verify the details | |
| 11:43:11 | gibi | frickler: OK, let me know if I need to anything extra | |
| 11:43:50 | gibi | dansmith: I think https://review.opendev.org/#/c/738482/ is ready for your eyes, again | |
| 11:52:48 | frickler | gibi: o.k., added you to chanserv, you should now be able to issue "/msg chanserv topic #openstack-nova topic <new topic line>". the bot also has some extensive help feature if needed | |
| 11:53:03 | gibi | frickler: thanks! | |
| 11:56:43 | gibi | frickler: works. thanks again | |
| 12:00:37 | frickler | gibi: seems I there was on "topic" too much in the command I gave you, need to drop the second one. also you dropped the "Current runways:" part, not sure if that was intentional | |
| 12:00:46 | frickler | other than that, welcome ;) | |
| 12:01:05 | gibi | good point | |
| 12:01:07 | gibi | fixing it | |
| 12:49:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Ignore PCI devices with 32bit domain https://review.opendev.org/756697 | |
| 14:25:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use _evacuate_server helper in func test https://review.opendev.org/758210 | |
| 19:18:04 | ganso | melwitt, elod: thanks a lot for the reviews yesterday! If you could please spare 1 more minute, this is the last one, to stable/stein, the conflict is due to a trivial change not being there: https://review.opendev.org/#/c/757886/ | |
| #openstack-nova - 2020-10-17 | |||
| 06:22:55 | openstackgerrit | Merged openstack/nova master: bindep: Install python3 and python3-devel on CentOS 8 and Fedora https://review.opendev.org/755928 | |
| 07:53:09 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Add placeholder migrations for Victoria backports https://review.opendev.org/754396 | |
| 07:53:36 | openstackgerrit | Takashi Natsume proposed openstack/nova master: Update contributor guide for Wallaby https://review.opendev.org/754427 | |
| 08:27:37 | admin0 | hi all .. i am getting resource provider 'xxxx' not found .. i think the compute node was renamed .. the name was set back to the same as before, but this log still persists | |
| 08:28:10 | admin0 | i want to check if there is a way that exists that fixes this .. or is it something that is fixed from the database | |
| 09:45:25 | openstackgerrit | Merged openstack/nova master: Use cell targeted context to query BDMs for metadata https://review.opendev.org/752459 | |
| 09:45:34 | openstackgerrit | Merged openstack/nova master: Clean up the DynamicVendorData constructor https://review.opendev.org/752718 | |
| 09:54:24 | openstackgerrit | Merged openstack/nova master: compute: Lock by instance.uuid lock during swap_volume https://review.opendev.org/754695 | |
| 12:02:17 | openstackgerrit | Merged openstack/nova stable/train: Removes the delta file once image is extracted https://review.opendev.org/748035 | |
| #openstack-nova - 2020-10-18 | |||
| 04:48:06 | openstackgerrit | Merged openstack/nova master: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/757656 | |
| #openstack-nova - 2020-10-19 | |||
| 02:58:39 | openstackgerrit | wangzhh proposed openstack/nova master: Ignore InstanceNotFound exception when host rejoin cluster https://review.opendev.org/757053 | |
| 07:27:11 | bauzas | good morning Nova | |
| 07:47:47 | xinranwang | gibi: Hi, Gibi. Could you and others please review the sriov nic support spec before PTG so we can discuss it during the meeting? https://review.opendev.org/#/c/742785/ | |
| 07:48:25 | xinranwang | Please feel free to leave the comment if any questions or suggestions. Thank you all in advance :) | |
| 07:54:43 | bauzas | review day for me indeed | |
| 08:10:31 | gibi | xinranwang: hi! I did review it two weeks ago and left comments | |
| 08:10:47 | gibi | bauzas: o/ | |
| 08:11:34 | xinranwang | gibi: oops, sorry for miss your comments. I will look into it soon. :) | |
| 08:30:20 | gibi | xinranwang: no worries | |
| 08:31:38 | stephenfin | bauzas: Seeing as I wasn't there Thursday afternoon, the suggestions was that we could use alembic for everything >= Train, allowing us to drop the unsupported/unloved sqlalchemy-migrate. However, that's a follow-up and probably needs a spec, for sure | |
| 08:32:13 | stephenfin | i.e. I wasn't suggesting introducing alembic as part of the compaction, just so we're clear :) | |
| 08:32:45 | bauzas | all good then | |
| 08:32:51 | bauzas | we can discuss it during next week | |
| 08:32:56 | stephenfin | yup | |
| 08:33:51 | gibi | stephenfin: I've approved the bp as per the meeting discussion | |
| 08:34:00 | stephenfin | gibi: Great! Thanks :) | |
| 08:34:03 | gibi | stephenfin: but left two comments in the bp whiteboard as requested | |
| 08:34:17 | stephenfin | Now to start generating schema diffs | |
| 08:34:24 | gibi | cool | |
| 08:34:50 | gibi | stephenfin: I spent some time on mypy and o.vo it is a tricky busines | |
| 08:35:10 | stephenfin | with the pyi approach? | |
| 08:35:13 | gibi | stephenfin: do you know a way to tell mypy that we have stub for a type but not for a whole file | |
| 08:35:16 | gibi | stephenfin: yes | |
| 08:35:31 | stephenfin | not off the top of my head, no | |
| 08:35:36 | gibi | stephenfin: for me it seems mypy handles pyi as per file :/ | |
| 08:36:09 | gibi | so if we generate pyi for the Instance o.vo in nova then mypy will not load the py file and does not check the member functions on Instance o.vo | |
| 08:37:10 | gibi | so now I looking at writing a plugin for mypy using the get_class_decorator_hook triggering on the oslo register decorator | |
| 08:37:53 | gibi | anyhow, it is interesting enough to spend some time on it :) | |
| 08:38:36 | stephenfin | I guess that's kind of greenfield? :) | |
| 08:38:46 | gibi | totally green :D | |
| 08:39:49 | gibi | this is how mypy sees an ovo definition http://paste.openstack.org/show/799163/ | |
| 08:40:51 | kashyap | stephenfin: Morning, what's the current Ubuntu LTS release, again? | |
| 08:41:02 | stephenfin | kashyap: In the gate? | |
| 08:41:08 | kashyap | stephenfin: Yeah, indeed | |
| 08:41:17 | kashyap | stephenfin: I'm trying to see what libvirt/QEMU version we test in the Gate | |
| 08:42:17 | stephenfin | I'm not sure if we're on Focal (20.04) yet. lyarwood is probably the guy to ask. We were on the version from the Bionic UCA (Cloud Archive) | |
| 08:42:37 | kashyap | (Okay, "Bionic Focal" is the latest LTS) | |
| 08:43:13 | stephenfin | Nope, Focal | |
| 08:43:15 | stephenfin | 20.04 | |
| 08:43:19 | stephenfin | Bionic is 18.04 | |
| 08:43:33 | stephenfin | (Focal Fossa and Bionic Beaver, fwiw) | |
| 08:43:45 | lyarwood | yup we should be using Focal in everything but the intergrated-compute job iirc | |
| 08:44:00 | lyarwood | https://review.opendev.org/#/c/755525/ will move that once the retry fix lands on stable/victoria | |
| 08:44:29 | kashyap | stephenfin: Err, I mean Bionic Beaver I meant. Sorry | |
| 08:44:45 | kashyap | Thanks, both. So whatever is in Focal, right? | |
| 08:44:53 | stephenfin | gibi: looks about right; I was attempting to work around it by doing something like https://review.opendev.org/#/c/738020/6/nova/objects/agent.py but that's just a hack, really | |
| 08:44:57 | lyarwood | https://review.opendev.org/#/c/757305/ - elod, bauzas, would you mind kicking this today? | |
| 08:45:13 | lyarwood | kashyap: yeah | |
| 08:45:46 | bauzas | lyarwood: can look, but I'd like to prioritize specs reviews | |
| 08:46:38 | lyarwood | kashyap: libvirt 6.0.0 and QEMU 4.2-3 | |
| 08:46:45 | lyarwood | bauzas: ack | |
| 08:47:03 | kashyap | lyarwood: Thank you; someone from the upstream libvirt/QEMU was asking the versions | |
| 08:47:32 | kashyap | lyarwood: And general "do you (OpenStack) folks do any grab-the-latest-release-of-libvirt/QEMU-to-catch-regressions kind of testing"? | |
| 08:47:37 | gibi | stephenfin: doesn't that field definitions conflict with the generated fields runtime? | |
| 08:47:55 | kashyap | lyarwood: The answer to the above is: we've got 'virt-preview' at most CI, but not testing from latest upstream releases, right? | |
| 08:48:10 | gibi | stephenfin: or these are no-op runtime? | |
| 08:48:25 | kashyap | (Although, I remember, some 4 years ago we wanted to allow testing tar releases of libvirt/QEMU in DevStack) | |
| 08:50:27 | lyarwood | kashyap: our virt-preview jobs are in the experimental queue at the moment, we might move them to the main queue as non-voting at some point this cycle if they continue to be stable enough | |
| 08:50:40 | kashyap | (Oh, right, experimental queue) | |
| 08:51:00 | lyarwood | kashyap: between them and our fedora jobs we should get enough coverage | |
| 08:51:02 | kashyap | lyarwood: What's the diff. between 'non-voting' and 'experimental'? One would think they're the same | |
| 08:51:12 | lyarwood | kashyap: we've already caught a number of issues early | |
| 08:51:18 | kashyap | (Right) | |
| 08:51:38 | lyarwood | kashyap: experimental is only used when someone leaves a `check experimental` comment | |
| 08:51:48 | lyarwood | kashyap: non-voting always runs automatically | |
| 08:51:53 | lyarwood | kashyap: but doesn't vote | |
| 08:51:58 | kashyap | lyarwood: Ah, thank you | |
| 08:57:02 | CeeMac | lyarwood: regarding the fix that merged recently https://review.opendev.org/#/c/754695/ | |
| 08:57:24 | CeeMac | is this compatible with rocky as is, or would it need a backport? | |
| 09:01:53 | lyarwood | CeeMac: I'm not sure if the cherry pick is clean back to stable/rocky directly from master | |
| 09:02:00 | lyarwood | CeeMac: I'll post stable backports now | |
| 09:02:08 | lyarwood | just sorting some admin stuff out downstream | |