| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-12 | |||
| 14:55:04 | openstackgerrit | Merged openstack/nova stable/pike: Init HostState.failed_builds https://review.opendev.org/669538 | |
| #openstack-nova - 2020-07-13 | |||
| 08:37:17 | openstack | bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 08:37:17 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Reproduce bug 1862633 https://review.opendev.org/740634 | |
| 08:38:29 | openstack | bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 08:38:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Reproduce bug 1862633 https://review.opendev.org/740634 | |
| 08:40:54 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/740636 | |
| 08:41:50 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/740636 | |
| 10:29:58 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Track blockjob status in DEBUG when polling for completion https://review.opendev.org/729595 | |
| 10:30:03 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove blockjob.end == 0 workaround resolved in libvirt v2.3.0 https://review.opendev.org/729596 | |
| 10:40:51 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962 | |
| 10:46:34 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove use of pkg_resources https://review.opendev.org/740661 | |
| 11:08:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/ussuri: Correct reported system memory https://review.opendev.org/740666 | |
| 11:08:45 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/train: Correct reported system memory https://review.opendev.org/740667 | |
| 11:15:46 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Correct typo https://review.opendev.org/740670 | |
| 11:16:44 | stephenfin | giblet: addressed your comments on https://review.opendev.org/#/c/461456/ with a follow-up | |
| 11:17:12 | gibi | stephenfin: hi! will look | |
| 11:26:01 | huaqiang | stephenfin: there is another change related to user/flavors.rst:https://review.opendev.org/#/c/740176/. If you have time please review as well | |
| 11:26:02 | sean-k-mooney | stephenfin: did you grep for any other checks for linux2 by the way | |
| 11:26:17 | stephenfin | huaqiang: will do | |
| 11:26:25 | huaqiang | thanks | |
| 11:26:25 | stephenfin | sean-k-mooney: I did. Didn't spot anything else | |
| 11:26:57 | sean-k-mooney | stephenfin: cool | |
| 11:29:27 | sean-k-mooney | stephenfin: you dont happen to have an upstream bug for pci assignment affintiy when using the prefer policy do you. e.g. the fact we dont sort the numa nodes to prefer the numa nodes with pci devices, we just prefer a host where affinity can be achived. | |
| 11:29:47 | sean-k-mooney | i want to file a related bug for some other non optimal behavior | |
| 11:30:23 | sean-k-mooney | namely that we dont loadbalnace accross numa nodes based on avaiable pcpus or memory | |
| 11:31:01 | sean-k-mooney | i just noticed all my vms on my home openstack are running on numa node 0 since i have not ran out of hugepages yet | |
| 11:31:30 | sean-k-mooney | nova really should be loadbalanceing them between both host numa nodes and it pretty trival to do | |
| 11:32:15 | sean-k-mooney | so i would like to fix that and backport it as a bug before tracking numa in placment. | |
| 11:33:54 | sean-k-mooney | ya i dont se an upstream bug ill file one to cover both | |
| 11:52:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/740636 | |
| 11:52:11 | openstack | bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 11:52:11 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Reproduce bug 1862633 https://review.opendev.org/740634 | |
| 12:03:50 | openstack | Launchpad bug 1887377 in OpenStack Compute (nova) "nova does not loadbalnce asignmnet of resouce on a host based on avaiablity of pci device, hugepages or pcpus." [Undecided,New] - Assigned to sean mooney (sean-k-mooney) | |
| 12:03:50 | sean-k-mooney | stephenfin: filed https://bugs.launchpad.net/nova/+bug/1887377 | |
| 13:46:24 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Bring back allowed_direct_url_schemes in support of RBD image download https://review.opendev.org/728095 | |
| 13:55:20 | openstackgerrit | Jiri Suchomel proposed openstack/nova master: Move image verification and writing out of download method https://review.opendev.org/738738 | |
| 14:30:48 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 15:21:20 | openstackgerrit | Merged openstack/nova master: hardware: Allow 'hw:cpu_realtime_mask' to be omitted https://review.opendev.org/737026 | |
| 15:21:28 | openstackgerrit | Merged openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456 | |
| 15:21:40 | openstackgerrit | Merged openstack/nova master: test_hardware: code formating for multiline construct https://review.opendev.org/728793 | |
| 16:04:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/740636 | |
| 16:14:15 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 16:39:18 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: objects: Introduce 'pcpuset' field for InstanceNUMACell https://review.opendev.org/714658 | |
| 16:41:07 | huaqiang | stephenfin: Addressed your comments on https://review.opendev.org/#/c/714658/ | |
| 18:40:56 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make our ceph job test with glance in multistore mode https://review.opendev.org/734184 | |
| #openstack-nova - 2020-07-14 | |||
| 03:43:05 | openstackgerrit | zhangboye proposed openstack/nova-specs master: migrate testing to ubuntu focal https://review.opendev.org/740863 | |
| 05:33:37 | openstackgerrit | Takashi Kajinami proposed openstack/nova master: Change default num_retries for glance to 3 https://review.opendev.org/740389 | |
| 08:02:01 | Habib-A | Hello all please help :D https://ask.openstack.org/en/question/128468/windows-server-2019-with-ovs-and-nova-compute/ | |
| 08:26:59 | gibi | kashyap: hi! you might have knowledge that I could use. I see a strange behavior from libvirt / qemu . with libvirt 4.0.0 and qemu 2.11 I can only hotplug an SRIOV VF to a guest if I detach the VF first from the host even if I specify managed=yes in the xml. The domain create works, libvirt automatically detach the device from the host. | |
| 08:27:18 | kashyap | gibi: Mornin | |
| 08:27:35 | gibi | kashyap: then I upgraded to libvirt 6.0.0 and qemu 4.2 and now hotplug also works without a manual detach | |
| 08:27:56 | gibi | I looked at the changelog of libvirt but nothing pops up | |
| 08:28:35 | kashyap | gibi: So a "feature was introduced silently"? :-) | |
| 08:28:58 | gibi | I think it is more like bugfix than a feature | |
| 08:29:18 | kashyap | gibi: A question: is the libvirt-6.0.0 & qemu-4.2 behaviour desirable to you, or not? (Let's keep the surprise part aside :)) | |
| 08:29:27 | gibi | desirable | |
| 08:29:27 | kashyap | Right, I first wrote 'bug-fix' indeed | |
| 08:29:35 | gibi | very mutch | |
| 08:29:55 | gibi | I'm working on nova to support sriov attach | |
| 08:30:10 | gibi | and for that this behavior is needed | |
| 08:30:25 | kashyap | gibi: So you want to know whether this is intended or not? | |
| 08:30:45 | gibi | I want to know what is the minimum libvirt / qemu version that this behavior exists | |
| 08:31:56 | kashyap | gibi: Ah, good one; I can check w/ one of the upstream libvirt folks | |
| 08:32:04 | gibi | kashyap: thanks a lot! | |
| 08:32:26 | kashyap | gibi: I'll get back to you in a bit. Today also I need to finish some administrative paperwork (deadline is 16 Jul; dammit). But it's on my to-do | |
| 08:32:41 | gibi | kashyap: thanks. it is not urgent | |
| 08:33:10 | kashyap | gibi: But yeah; silently introducing such a user-visible bug fix without even a changelog is not nice | |
| 08:38:30 | gibi | I'm glad they fixed it :) | |
| 09:09:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove support for Intel CMT events https://review.opendev.org/740922 | |
| 09:09:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Use constants for libvirt version checks https://review.opendev.org/740921 | |
| 09:33:17 | stephenfin | gibi: You're working on SR-IOV attach, you say? | |
| 09:33:50 | stephenfin | That's interesting, because that's on the backlogs of sean-k-mooney and I for quite some time now, and we simply hadn't gotten to it yet | |
| 09:37:41 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | |
| 09:37:41 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267 | |
| 09:40:23 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor the code for CPU policy sanity check https://review.opendev.org/740925 | |
| 09:40:51 | stephenfin | gibi: I parroted your question back to danpb, given kashyap was occupied - as far as he's concerned, the behaviour you saw on the older version is bogus and reflects a bug | |
| 09:41:04 | stephenfin | "as managed=yes stuff has been around forever and never required a manual detach" | |
| 09:41:35 | kashyap | sfinucan: Thanks; you checked off a TODO on my list :) | |
| 10:12:49 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: [DNM] bump nova-compute version for mixed instance in API https://review.opendev.org/740933 | |
| 10:14:23 | huaqiang | stephenfin: I have sent out another ongoing change as a possible alternative of https://review.opendev.org/716271 | |
| 10:14:51 | huaqiang | https://review.opendev.org/716271 checks the nova-compute service version in conductor layer | |
| 10:15:17 | huaqiang | while https://review.opendev.org/740933 is doing this in API layer | |
| 10:15:26 | huaqiang | just for comparison | |
| 10:19:32 | gibi | stephenfin: yeah, I did not assigned the bp or file a spec for it as I did not know how much time I will have during this cycle. So basically I spent yesterday putting a PoC together that started working today morning. I will do the paperwork at some point before M2 | |
| 10:20:22 | gibi | regarding the libvirt bug, it would be nice to know what is the minimum libvirt version that works for documentation purposes | |
| 10:28:29 | stephenfin | huaqiang: ack, will look | |
| 10:29:11 | stephenfin | gibi: Okay, cool. I know Sean and I would be keen to take a look at that, so if/when you do lemme know | |
| 10:30:40 | stephenfin | I can't help with testing at the moment since my SR-IOV machines are at home (I'm staying with my folks in the countryside since March), but hopefully it was just a single bad version of libvirt. Can probably test by building various versions from source | |
| 10:30:59 | stephenfin | Once I get access to those machines, I could do that testing | |
| 10:39:01 | gibi | stephenfin: OK, thanks. I have access to an SRIOV capable machine so I can try different libvirt version too | |
| 10:50:22 | gibi | I think I will be able to post some WIP code this week | |
| 11:20:19 | stephenfin | huaqiang: Okay, I prefer https://review.opendev.org/#/c/740933/ | |
| 11:21:09 | stephenfin | May be wise to get someone that knows RPC better than I do (dansmith or gibi, perhaps?) to weigh in | |
| 11:25:53 | stephenfin | lyarwood: I had my message asking you to look at the vTPM work already typed out before I checked Gerrit 😅 Thanks for the reviews | |
| 11:25:53 | stephenfin | lyarwood: I had my message asking you to look at the vTPM work already typed out before I checked Gerrit 😅 Thanks for the reviews | |
| 11:32:46 | gibi | stephenfin, huaqiang: looking at the alternatives I think I would go with the check in the api service | |
| 11:33:33 | stephenfin | gibi: so you mean prefer https://review.opendev.org/#/c/740933/ over https://review.opendev.org/716271 ? | |
| 11:34:05 | gibi | yes I prefer https://review.opendev.org/#/c/740933/ | |
| 11:37:08 | gibi | it is a bit more explicit what operations we block if the check is in the API | |
| 11:38:08 | gibi | and also understanding the code is easier for me if the check is not burried down in a long call path | |