Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-10-12
08:23:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Revert "Temporarily disable parts of heal port allocation test" https://review.opendev.org/755995
08:25:25 openstackgerrit Balazs Gibizer proposed openstack/nova master: Use cell targeted context to query BDMs for metadata https://review.opendev.org/752459
08:25:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Clean up the DynamicVendorData constructor https://review.opendev.org/752718
08:26:53 gibi stephenfin: hi! if you want to start your week with a simple patch then I suggest https://review.opendev.org/#/c/754124/ :)
08:36:11 openstackgerrit Balazs Gibizer proposed openstack/nova master: Move query param generation to RequestGroup o.vo https://review.opendev.org/756894
08:36:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/756895
08:36:12 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow extending an existing instance allocation https://review.opendev.org/757109
08:37:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make remove allocation symmetric with add allocation https://review.opendev.org/757110
08:38:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out port resource deallocation https://review.opendev.org/757111
08:43:08 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/756530
09:00:16 stephenfin gibi: sure, done
09:09:55 openstackgerrit Merged openstack/nova-specs master: Move implemented specs https://review.opendev.org/754124
09:16:18 gibi stephenfin: thanks
09:29:03 openstackgerrit Wenping Song proposed openstack/python-novaclient master: Cleanup py27 support https://review.opendev.org/757513
09:40:22 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Fix issue in exception type https://review.opendev.org/756069
09:40:22 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add tests for mixed CPU policy https://review.opendev.org/755852
09:40:23 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add test for #1898272 https://review.opendev.org/756100
09:40:23 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Check inventory of shared CPUs for 'mixed' policy https://review.opendev.org/756101
09:40:24 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'InstanceNUMACell.min_threads' https://review.opendev.org/756070
09:42:35 stephenfin lyarwood, gibi: What do I do about https://review.opendev.org/#/c/631038/ now? The issue is still there for libvirt < 4.3.0, but our min supported version on master is now 5.0.0
09:43:07 stephenfin lyarwood, gibi: i.e. should I propose as stable only or simply drop it?
09:44:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware https://review.opendev.org/631053
09:46:45 stephenfin lyarwood, gibi: Also, could I get that related fix merged? ^ I'm going to rework that function (again) but want to get that and https://review.opendev.org/#/c/756101/ in first so I can backport
09:47:19 gibi stephenfin: https://review.opendev.org/#/c/631038/ is a bugfix that is not effecting master. It think it is totally OK to propose it to stable branches as stable-only fix
09:48:10 stephenfin great; coming right up
09:58:02 openstackgerrit Stephen Finucane proposed openstack/nova stable/ussuri: [Stable Only] Handle libvirt reporting incorrect 4k page quantities https://review.opendev.org/757524
09:58:19 stephenfin gibi, lyarwood: One stable-only patch awaits ^ :)
09:58:32 gibi stephenfin: thanks
10:38:08 gibi stephenfin: I have question in https://review.opendev.org/#/c/631053/9//COMMIT_MSG@18
10:38:25 stephenfin looking
10:43:29 lyarwood stephenfin: ack sorry stuck downstream with something today
10:48:35 stephenfin gibi: replied; tl;dr: on x86 hosts with no explicit mempage requests in the guest, nova will always use standard (4k) pages
10:48:54 stephenfin i.e. it'll never consume 2M or 1G pages unless the guest explicitly requests them
10:49:11 sean-k-mooney also replied
10:49:42 sean-k-mooney we do have a hw:mem_page_size=any that was ment to allow nova to use any page size but we did nto fully implement it
10:50:38 sean-k-mooney not specifying hw:mem_page_size is different then setting hw:mem_page_size=small or hw:mem_page_size=4k explcitly
10:50:51 gibi OK, thanks this helps
10:51:00 sean-k-mooney when unset nova will ignore numa affintiy and allow over commit
10:51:21 sean-k-mooney if set to any value including 4k or small over commit is disabeld and numa affinity is enforced
10:52:12 sean-k-mooney hw:mem_page_size=any today means use 4k small pages unless the image asks for a page size then do what the image says
10:52:45 sean-k-mooney the orginal intent of hw:mem_page_size=any was to use the smallest page size that would fit
10:53:04 sean-k-mooney unless the image requested one
10:53:16 sean-k-mooney thats the bit we never implemented
10:55:01 sean-k-mooney stephenfin: by the way this brings me back to the topic of defualting to hw:mem_page_size=small when ever we have a numa toplogoy and it is not set
10:55:39 sean-k-mooney we actully proably would want to default to hw:mem_page_size=any
10:56:12 sean-k-mooney but ya i need to go work on that stable abi spec
11:27:09 openstackgerrit zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
11:42:24 openstackgerrit Wenping Song proposed openstack/python-novaclient master: Remove Babel requirement https://review.opendev.org/757541
12:33:44 openstackgerrit Merged openstack/python-novaclient master: Cleanup py27 support https://review.opendev.org/757513
12:50:23 openstackgerrit Wenping Song proposed openstack/python-novaclient master: Remove Babel from lower-constraints.txt https://review.opendev.org/757541
13:20:32 gibi dansmith, melwitt, gmann: responded in https://review.opendev.org/#/c/731396
14:01:52 openstackgerrit Andrew Bonney proposed openstack/nova master: Handle disabled CPU features to fix live migration failures https://review.opendev.org/757577
14:02:23 sean-k-mooney does anyone remember why we use psycopg2 and not psycopg2-binary in our test requirements
14:03:06 sean-k-mooney its come up a few times before but it would be nice if we could swap to the binary wheel instead of having to instlal postgress dev libs to build the python bindingsx
15:46:18 openstackgerrit Merged openstack/python-novaclient master: Remove Babel from lower-constraints.txt https://review.opendev.org/757541
15:53:52 openstackgerrit Mark Goddard proposed openstack/nova master: Fix PCI passthrough race on reschedule (claims) https://review.opendev.org/710847
16:13:26 openstackgerrit melanie witt proposed openstack/nova stable/victoria: Follow up for cherry-pick check for merge patch https://review.opendev.org/757613
16:14:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor _claim_pci_device_for_interface_attach to prepare for qos https://review.opendev.org/756895
16:14:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow extending an existing instance allocation https://review.opendev.org/757109
16:15:30 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make remove allocation symmetric with add allocation https://review.opendev.org/757110
16:15:31 openstackgerrit Balazs Gibizer proposed openstack/nova master: Factor out port resource deallocation https://review.opendev.org/757111
16:15:31 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/756530
16:15:32 openstackgerrit Balazs Gibizer proposed openstack/nova master: Refactor update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/757614
16:15:32 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused Instance method https://review.opendev.org/757615
16:16:54 openstackgerrit Stephen Finucane proposed openstack/nova stable/victoria: libvirt: 'video.vram' property must be an integer https://review.opendev.org/757616
16:17:46 openstackgerrit Stephen Finucane proposed openstack/nova stable/ussuri: libvirt: 'video.vram' property must be an integer https://review.opendev.org/757617
16:18:24 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: libvirt: 'video.vram' property must be an integer https://review.opendev.org/757618
16:59:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with qos ports https://review.opendev.org/756530
20:31:29 openstackgerrit melanie witt proposed openstack/nova master: Add note about setting CONF.compute.max_disk_devices_to_attach = 0 https://review.opendev.org/755360
20:48:42 openstackgerrit melanie witt proposed openstack/nova master: Disallow CONF.compute.max_disk_devices_to_attach = 0 https://review.opendev.org/755360
22:47:09 openstackgerrit melanie witt proposed openstack/nova master: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/757656
22:53:52 openstackgerrit melanie witt proposed openstack/nova master: Prevent archiving of pci_devices records because of 'instance_uuid' https://review.opendev.org/757656
#openstack-nova - 2020-10-13
01:02:20 openstackgerrit zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
07:18:05 openstackgerrit zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
08:30:43 openstackgerrit Jorhson Deng proposed openstack/nova master: add task state for attach volume and detach volume https://review.opendev.org/754258
09:37:31 lyarwood this might be the 4am baby feed but shouldn't we be able to run individual func tests using `tox -e functional $path.to.test.file` ?
09:37:45 lyarwood ERROR: InvocationError for command /home/lyarwood/redhat/devel/src/openstack/nova/.tox/functional/bin/stestr --test-path=./nova/tests/functional run nova.tests.functional.regressions.bug_az_vol
09:38:01 lyarwood is what I'm getting, did I miss a memo or is there likely something wrong in the test itself?
09:38:32 lyarwood ah wait, the filename needs to be test_
09:39:37 gibi lyarwood: don't be too hard on yourself (and the keyboard)
09:40:16 lyarwood gibi: ^_^
14:06:43 openstackgerrit Rodrigo Barbieri proposed openstack/nova stable/train: Allow tap interface with multiqueue https://review.opendev.org/757870
14:32:23 mnaser sean-k-mooney: have you played around with sriov + mlx cards?
14:32:33 mnaser i'm having instance creation failures with errors like this: `group 13 is not viable\nPlease ensure all devices within the iommu_group are bound to their vfio bus driver.`
14:32:45 sean-k-mooney mnaser: stephenfin has they are more work then intel
14:33:11 sean-k-mooney mnaser: both prots on the same physical card i think are in the same iommue group
14:33:28 sean-k-mooney you can check that with sysfs
14:33:39 sean-k-mooney mnaser:you need to have both port bound to the same driver
14:35:56 mnaser sean-k-mooney: not sure i follow, but, it seems like they are all in group 13 according to `/sys/kernel/iommu_groups/`
14:36:50 mnaser qemu-system-x86_64: -device vfio-pci,host=0000:41:00.5,id=hostdev0,bus=pci.0,addr=0x4: vfio 0000:41:00.5: group 13 is not viable\nPlease ensure all devices within the iommu_group are bound to their vfio bus driver.
14:37:07 mnaser i wonder if i need to setup vfio stuff, i've mostly only done that for pci passthrough but not sriov
14:58:05 openstackgerrit Rodrigo Barbieri proposed openstack/nova stable/stein: Allow tap interface with multiqueue https://review.opendev.org/757886
14:59:15 sean-k-mooney mnaser: if its a two port card both port will need to be bound to vfio-pci
14:59:55 sean-k-mooney libvirt can somethimes handel this for you but it gets complicated when you have multipel ports on teh same card
15:00:19 sean-k-mooney in somecase each physical port will get its own iommu group
15:00:26 sean-k-mooney on oder hardware it does not
15:00:35 sean-k-mooney older
15:14:03 mnaser sean-k-mooney: i think my issue was thta i should have been on https://docs.openstack.org/neutron/ussuri/admin/config-ovs-offload.html
15:21:28 sean-k-mooney are you using hardware offloaed ovs
15:21:34 sean-k-mooney or just sriov with the sriov nic agent

Earlier   Later