Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-11
17:43:40 artom The other idea was test automation stuff with tempest/whitebox, since the learning curbe isn't as steep as nova
17:43:45 artom *curve
17:43:47 stephenfin gotcha
17:43:53 stephenfin I can do that (y)
17:44:21 stephenfin alex_xu: Can I do it as a follow-up though?
17:44:29 stephenfin because that patch is already huge and it's a corner case
17:44:42 stephenfin I'll have it done first thing in the morning
17:45:03 alex_xu stephenfin: sure, or I can help that in my morning
17:45:51 stephenfin sweet
17:46:03 stephenfin I'm just moving your quotas patch further down the series
17:46:13 stephenfin should be done shortly then I'll push it up
17:46:37 alex_xu cool, I thought it should be adjust the order also
17:46:52 alex_xu anyway, see you guys tomorrow
17:46:56 stephenfin o/
17:47:37 efried stephenfin: did you see luyao's question about https://github.com/intel/Intel-OpenStack-CI-jobs/pull/2 ?
17:47:49 efried s,that link,https://review.opendev.org/#/c/678455/25/nova/virt/libvirt/driver.py@1297,
17:47:54 mriedem https://review.opendev.org/#/q/topic:bp/numa-aware-live-migration+status:open+label:Code-Review=2+project:openstack/nova
17:47:58 mriedem artom: dansmith: sean-k-mooney: ^
17:48:01 mriedem donnyd: ^
17:48:13 efried \o/
17:48:15 artom zomgies
17:48:46 mriedem success bot?
17:49:01 efried ffs wait until they merge
17:49:14 mriedem psh
17:49:22 dansmith mriedem: praise be
17:49:28 sean-k-mooney :)
17:49:59 mriedem gibi_bus: are you going to get https://review.opendev.org/#/c/676980/ updated today or should i work on that?
17:50:15 gibi_bus mriedem: I'm about to finish the rework
17:50:23 gibi_bus mriedem: I will ping you when it is up
17:51:17 mriedem ok, then i'm going to get lunch
17:52:20 gibi_bus mriedem: have a nice one!
18:03:34 efried o/ dolpher :)
18:07:19 dolpher efried: Hello, seems the filter is not working, no one is triggering the CI job now
18:07:39 dolpher Hi donnyd :)
18:19:30 donnyd sean-k-mooney: I didn't hear back on the labels.. but i hope this https://review.opendev.org/#/c/634827/ isn't waiting on jobs for FN
18:19:58 donnyd I submitted a bit more for resources, kinda need it right now
18:20:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
18:20:11 gibi_bus mriedem: ^^
18:21:20 gibi_bus bauzas: for tomorrow's coffee ^^
18:22:03 sean-k-mooney donnyd: no but the jobs got canceld
18:22:27 sean-k-mooney the 4 build i had in flight got killed because of a rebase/fix of the patches
18:22:51 sean-k-mooney i can start them again and see what happens
18:24:33 openstackgerrit sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656
18:24:48 openstackgerrit sean mooney proposed openstack/nova master: [DNM] testing with new lable. https://review.opendev.org/680738
18:25:44 sean-k-mooney donnyd: just rebased those via the gerrit ui that shoudl run with the old and new lable
18:26:04 donnyd sean-k-mooney: ok cool
18:39:00 efried dolpher: Can we kill the filter for now?
18:39:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
18:40:52 dolpher https://github.com/intel/Intel-OpenStack-CI-jobs/pull/4
18:41:26 dolpher efried: send the pull request to remove the filter, and I'll ping Matt to merge it.
18:41:34 efried okay
18:42:03 openstackgerrit Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827
18:44:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
18:52:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Extract pf$N literals as constants from func test https://review.opendev.org/680991
18:52:36 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450
18:52:37 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
18:52:37 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453
18:52:38 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
18:52:38 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
18:52:39 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
18:52:39 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640
18:52:40 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
18:52:40 openstackgerrit Luyao Zhong proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397
18:54:34 openstackgerrit Balazs Gibizer proposed openstack/nova master: Improve dest service level func tests https://review.opendev.org/680998
18:56:52 eandersson Having an odd bug - a VM can't be rebuilt with a new image. Pretty sure placement is filtering out all the potential matches.
18:57:02 eandersson Rebuilding it with the same image works fine.
18:57:05 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up for Ib50b6b02208f5bd2972de8a6f8f685c19745514c https://review.opendev.org/681490
18:57:41 mriedem eandersson: which release?
18:57:46 eandersson Running Rocky
18:57:58 eandersson Very latest stable/rocky
18:58:26 eandersson I see that the scheduler gets a single result, but it is failing on NUMA Topology.
18:58:32 mriedem we shouldn't be calling placement for rebuild https://github.com/openstack/nova/blob/stable/rocky/nova/scheduler/manager.py#L126
18:58:46 eandersson > 'NUMATopologyFilter: (start: 1, end: 0)'
18:58:51 eandersson Ah bad assumption
18:58:53 mriedem https://bugs.launchpad.net/nova/+bug/1763766 ?
18:58:54 openstack Launchpad bug 1763766 in OpenStack Compute (nova) "nova needs to disallow resource consumption changes on image rebuild" [Medium,Triaged]
18:59:14 mriedem or https://bugzilla.redhat.com/show_bug.cgi?id=1700412
18:59:15 openstack bugzilla.redhat.com bug 1700412 in openstack-nova "[RFE] Rebuild with a different image fails due to NUMATopologyFilter" [High,New] - Assigned to nova-maint
18:59:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: Skip querying resource request if no qos port https://review.opendev.org/681513
18:59:46 eandersson Yep looks like it
18:59:52 dansmith yeah if the image is what causes it trouble,
19:00:07 dansmith it's likely the new numa topo filtering out the one host you're on
19:00:37 mriedem sean-k-mooney has to fix it and get that backported to queens
19:02:19 artom mriedem, it's not getting backported, but one of the things we discussed was making that RUN_ON_REBUILD flag we added way back when for the rebuild CVE configurable in some way
19:02:26 artom We kinda dreaded coming to upstream with that
19:02:49 sean-k-mooney dansmith: for the downstream issue yes its just the numa toploy filter filtering it out beacue tehre is no space for the rebuild
19:03:08 dansmith sean-k-mooney: I'm talking about eandersson's case
19:03:25 mriedem artom: we don't claim on rebuild so if the numa topo changes in the new image we won't claim for it properly,
19:03:33 mriedem so you could just be kicking the error down to compute
19:03:33 sean-k-mooney ill read scollback
19:03:43 sean-k-mooney mriedem: correct
19:03:54 artom mriedem, yeah, it wouldn't be a correct fix
19:04:04 sean-k-mooney we rebuild with the new image but old toplogy
19:04:23 artom More of a candy we throw our customers on, like, 10 to placate them until they an upgrade to whatever release the un-backportable fix lands in
19:04:23 mriedem sean-k-mooney: you mean in some downstream-only patch?
19:04:52 sean-k-mooney mriedem: no updstream if you trun off the numa toplogy filter to get passed it
19:05:03 sean-k-mooney when we rebuild but the toplogy change via the image
19:05:09 mriedem oh
19:05:14 sean-k-mooney we end up with the old numa toplogy and new image
19:05:29 sean-k-mooney so i want to put an upstream check to prevent that

Earlier   Later