Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-07-31
18:16:25 mriedem e.g. 2 nodes on a single host, and show that compute_node.stats.num_instances > 1 for the 2nd node processed
18:16:31 mriedem b/c that should never be true with ironic
18:17:03 dansmith tripleo gate jobs have that filter enabled
18:17:24 mriedem do their flavors have any capabilities extra specs?
18:17:25 dansmith they also have their own filter in front of it
18:18:01 dansmith idk, looking for evidence
18:19:22 dansmith not sure I'll be able to find it really
18:23:02 mgagne mriedem: yes, heavily iirc http://lists.openstack.org/pipermail/openstack-dev/2018-April/130022.html
18:24:06 mriedem mgagne: ok, is your baremetal deployment running >=ocata?
18:24:08 mriedem or still mitaka?
18:24:56 mgagne nova and ironic are mitaka
18:25:07 mriedem ok, so you wouldn't have seen this issue yet
18:25:22 mriedem only other group i know to ask is cern
18:28:25 openstackgerrit Merged openstack/nova stable/queens: Fix TypeError in prep_resize allocation cleanup https://review.openstack.org/581741
18:28:31 openstackgerrit Merged openstack/nova master: [placement] Retry allocation writes server side https://review.openstack.org/586048
18:28:40 openstackgerrit Merged openstack/nova master: Remove redundant join in _anchors_for_sharing_providers https://review.openstack.org/587377
18:28:47 openstackgerrit Merged openstack/nova stable/queens: Handle HostMappingNotFound when deleting a compute service https://review.openstack.org/583566
18:28:55 openstackgerrit Merged openstack/nova master: Don't poison Host._init_events if it's already mocked https://review.openstack.org/585748
18:30:51 mgagne mriedem: we do have a patch for _node_resource
18:32:03 mgagne mriedem: we use it to include more properties from the Ironic node to the resource stats
18:32:47 mriedem ok. my worry is that a change in ocata makes the stats reported per compute node no longer per compute node, i.e. makes it cumulative across all nodes managed by a given host
18:33:04 mriedem working on a functional test wrinkle to show it
18:33:07 mgagne hmm
18:33:12 mgagne our use case: https://gist.github.com/mgagne/078859de7bd8e3e8e93e64f9cf756ad0
18:34:10 mgagne where inheritable_host_properties is a list of props we wish to inherit from the Ironic node, currently some project/user reservation IDs and supported network models which are all scoped per Ironic node.
18:43:28 sean-k-mooney melwitt: thanks for taking a look at the disk_bus bug. ill create some better test cases to reproduce the bug and then assert that this fixes it.
18:45:55 melwitt sean-k-mooney: cool, thanks
18:53:05 openstackgerrit Merged openstack/nova stable/queens: unquiesce instance after quiesce failure https://review.openstack.org/581451
19:19:26 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add functional scale test for Stats.__deepcopy__ time https://review.openstack.org/587606
19:19:47 mriedem efried: heh ^
19:20:13 efried ...
19:22:30 mriedem "Took 4.21 with stats deepcopy, took 3.85 without stats deepcopy" for 100 instances
19:22:39 mriedem but that's totally variable at that scale
19:22:43 mriedem and has failed a couple of times
19:23:37 efried mriedem: Is it also going to matter that we're running "locally"?
19:24:43 efried um
19:24:43 mriedem locally?
19:24:56 efried yeah, meaning no remote-ness from conductor to compute and all that.
19:25:32 mriedem yeah for all the db stuf
19:25:47 mriedem obviously would be better done in a devstack env
19:55:26 mriedem i've got a recreate on that leaked node.stats thing
20:00:06 efried woot
20:04:17 openstack bug 1784705 in OpenStack Compute (nova) "ResourceTracker.stats can leak across multiple ironic nodes" [High,Triaged] https://launchpad.net/bugs/1784705
20:04:17 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add recreate test for bug 1784705 https://review.openstack.org/587614
20:04:18 mriedem efried: jaypipes: dansmith: ^
20:41:35 openstackgerrit sean mooney proposed openstack/nova master: libvirt: fix disk_bus handling for root disk https://review.openstack.org/584999
20:41:37 openstack bug 1759420 in OpenStack Compute (nova) "nova does not correctly support HW_DISK_BUS=sata or usb for kvm/qemu" [Medium,In progress] https://launchpad.net/bugs/1759420 - Assigned to sean mooney (sean-k-mooney)
20:41:37 openstackgerrit sean mooney proposed openstack/nova master: add regression test for bug 1759420 https://review.openstack.org/587615
21:03:36 mriedem efried: so i think not only will i fix the leaked stats issue, but also remove the need for those deepcopy clals
21:03:37 mriedem *calls
21:03:40 mriedem which fixes gary's bug
21:04:07 mriedem let the champagne flow
21:13:20 mriedem if only i can fix these 200 or so unit tests
21:16:10 efried mriedem: /me gets bubbles up nose, sprays
21:31:41 openstack bug 1784705 in OpenStack Compute (nova) queens "ResourceTracker.stats can leak across multiple ironic nodes" [High,Triaged] https://launchpad.net/bugs/1784705
21:31:41 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add recreate test for bug 1784705 https://review.openstack.org/587614
21:31:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636
22:10:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Make ResourceTracker.stats node-specific https://review.openstack.org/587636
22:10:15 openstack bug 1784705 in OpenStack Compute (nova) "ResourceTracker.stats can leak across multiple ironic nodes" [High,In progress] https://launchpad.net/bugs/1784705 - Assigned to Matt Riedemann (mriedem)
22:10:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Add recreate test for RT.stats bug 1784705 https://review.openstack.org/587614
22:34:12 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Compute: Handle reshaped provider trees https://review.openstack.org/576236
23:00:35 openstackgerrit Merged openstack/nova master: Remove unnecessary PlacementFixture setups https://review.openstack.org/587490
#openstack-nova - 2018-08-01
01:22:23 openstackgerrit Merged openstack/nova master: Delete checking a bool opt of None condition https://review.openstack.org/390705
01:27:17 openstackgerrit Merged openstack/nova master: libvirt: Remove usage of migrateToURI{2} APIs https://review.openstack.org/567258
01:41:41 openstackgerrit Merged openstack/nova master: Delete orphan compute nodes before updating resources https://review.openstack.org/579922
01:42:47 openstackgerrit Merged openstack/nova stable/queens: Fix server_group_members quota check https://review.openstack.org/581846
04:04:21 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679
04:49:43 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
05:02:55 openstackgerrit Merged openstack/nova master: Extract _update_to_placement method in resource tracker https://review.openstack.org/587171
05:12:42 gmann alex_xu: hi
05:15:42 alex_xu gmann: hi
05:16:40 gmann alex_xu: today i do not have much for API office hour. if you have anything we can have otherwise i would like to skip today
05:16:58 alex_xu gmann: nothing special from me
05:17:24 gmann alex_xu: ok, let we will cancel and have next week
05:17:32 alex_xu gmann: for the keypair quota usage, the conclude is just updating the doc I guess?
05:18:07 gmann alex_xu: i have not checked the detail yet but i will do check tomorrow
05:18:21 alex_xu gmann: ok, thanks
05:18:48 gmann alex_xu: thanks
06:22:50 openstackgerrit Jake Yip proposed openstack/nova master: Add --before to nova-manage db archive_deleted_rows https://review.openstack.org/556751
07:08:46 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Ensure the order of AllocationRequestResources https://review.openstack.org/587700
07:22:06 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704
08:35:50 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix comments in _anchors_for_sharing_providers and related test https://review.openstack.org/587679
08:37:35 yikun tetsuro ^ updated, :)
08:39:31 tetsuro yikun: Looks good to me. Thanks!
08:39:50 yikun thanks~
08:41:25 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Expand aggregate_metadata table index to cover 'value' as well https://review.openstack.org/587704
09:15:50 openstackgerrit Pranab proposed openstack/os-vif master: Support for OVS DB TCP socket communication. https://review.openstack.org/587378
09:30:10 openstackgerrit Merged openstack/nova master: compute node local_gb_used include swap disks https://review.openstack.org/585928
09:33:52 openstackgerrit Merged openstack/nova master: Define common variables for irrelevant-files https://review.openstack.org/578882
09:53:04 openstackgerrit Merged openstack/nova master: Change deprecated policies to policy https://review.openstack.org/583434
09:59:05 kosamara Hi bauzas, I was checking your spec numa-topology-with-rps. What's the status on this, are you actively reworking it?
09:59:36 bauzas kosamara: yup, I'm planning to rework on it at the end of August
10:00:20 bauzas I'll be on PTO for 3 weeks starting next Wed
10:00:31 bauzas gibi: stephenfin: cdent: ^
10:00:57 cdent ✔
10:05:29 kosamara bauzas: you mention RPs for PCI devices (L139 https://review.openstack.org/#/c/552924/9/specs/rocky/approved/numa-topology-with-rps.rst ). This is what I'm interested in. Nova currently doesn't create them, so I think it makes sense to do so either as part of this or as a new spec (maybe better, since your spec is already complex?)
10:05:56 gibi bauzas: have a nice time off!
10:09:51 bauzas kosamara: the spec is about discussing the topology, not really how to provide those devices
10:10:10 bauzas kosamara: so, yeah maybe a new spec
10:10:30 bauzas kosamara: at least about the upgrade between the existing and what you want

Earlier   Later