Earlier  
Posted Nick Remark
#openstack-nova - 2020-10-09
16:57:18 sean-k-mooney you would need to update the instnace.hostname filed in the db
16:57:21 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L130
18:00:08 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:00:08 openstack bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:00:08 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:03:20 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:03:20 openstack bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:03:21 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:03:43 melwitt omg uuughh
18:05:27 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Add functional test for bug 1731668 https://review.opendev.org/756636
18:05:27 openstack bug 1731668 in OpenStack Compute (nova) queens "placement: claim allocations fails with IndexError in _ensure_lookup_table_entry" [Low,In progress] https://launchpad.net/bugs/1731668 - Assigned to melanie witt (melwitt)
18:05:28 openstackgerrit melanie witt proposed openstack/nova stable/queens: [stable-only] Use a separate transaction for reading after race https://review.opendev.org/756637
18:06:19 melwitt sorry for the spam
19:05:31 openstackgerrit melanie witt proposed openstack/nova stable/train: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/757177
19:47:44 melwitt mnaser: is ^ something you would find useful/appropriate for sta
19:47:55 melwitt *stable/train or nah?
#openstack-nova - 2020-10-12
01:00:45 openstackgerrit Jorhson Deng proposed openstack/nova master: add task state for attach volume and detach volume https://review.opendev.org/754258
05:56:04 openstackgerrit MaAoyu proposed openstack/os-traits master: bump py37 to py38 in tox.ini https://review.opendev.org/757432
06:27:41 openstackgerrit zhanghao proposed openstack/nova master: Fix virsh domifstat to get vhostuser vif statistics https://review.opendev.org/757448
07:00:36 gibi good morning nova
07:26:51 openstackgerrit Merged openstack/nova master: Follow up for cherry-pick check for merge patch https://review.opendev.org/756639
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_

Earlier   Later