Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-16
14:44:25 sean-k-mooney or at least efried weekend
14:46:25 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
14:46:26 openstackgerrit Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964
14:47:25 aspiers stephenfin: ^^^
14:49:11 efried We got the right sdk, I think https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/job-output.txt#924
14:49:14 sean-k-mooney has the battle for 2.76 concluded by the way or is the power-update patch still under review
14:52:30 efried It looks like we're now raising VirtDriverNotReady properly when we get the failure to connect to ironic ++
14:52:38 efried https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/controller/logs/screen-n-cpu.txt.gz?severity=1#935
14:54:48 efried ...but then we stop getting that error, which implies that we got the connection good, which is what we expect --- but we still get ComputeHostNotFound
14:55:00 efried dustinc: It's possible that this ^ is an actual bug on the nova side
14:57:56 mordred efried: yay! that does seem like the sequence of actions we want
14:58:40 efried Reckon a little extra logging would help out here.
15:08:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for migrate server with ports having resource request https://review.opendev.org/655113
15:08:35 openstackgerrit Balazs Gibizer proposed openstack/nova master: Make _rever_allocation nested allocation aware https://review.opendev.org/676138
15:08:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support reverting migration / resize with bandwidth https://review.opendev.org/676140
15:15:14 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
15:15:15 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972
15:22:33 openstackgerrit Chris Dent proposed openstack/nova stable/stein: Add an issue releasenote for placement eventlet stall https://review.opendev.org/676973
15:23:00 cdent efried, belmoreira : first draft ^
15:33:56 melwitt cdent: fyi I had this similar reno going, is it for the same thing or no? https://review.opendev.org/662095
15:34:35 cdent same issue, different context, so maybe worth having both?
15:34:50 cdent Not sure whether combining they will be useful or not
15:34:53 melwitt ok. I'm on a call right now but skimmed over
15:35:27 melwitt if they are different then that's fine. wasn't sure at a glance
15:35:28 openstackgerrit sean mooney proposed openstack/nova master: Deprecate Aggregate[Core|Ram|Disk]Filters https://review.opendev.org/673496
15:55:47 openstackgerrit Merged openstack/nova stable/rocky: Add 'path' query parameter to console access url https://review.opendev.org/674916
15:55:54 openstackgerrit Merged openstack/nova master: Avoid timeout from service update api unit tests https://review.opendev.org/676923
15:56:01 efried mordred, dustinc: I can't figure it out so far.
15:56:01 efried I'm not very experienced looking at ironic logs (maybe we can bring in a ringer) but afaict we make requests for /nodes that return nothing (I assume 13 bytes is "no results") for a while, and then start returning stuff, whereupon we actually start requesting and receiving real data.
15:56:01 efried And yet, the nova side never seems to "get there".
15:56:01 efried The devstack times out waiting for n-api to respond, which is weird, because n-api looks active https://zuul.opendev.org/t/openstack/build/8b81e6788f21493e861948ce60687e7b/log/controller/logs/screen-n-api.txt.gz
15:56:01 efried Not sure if that /flavors/baremetal business is "normal" or not.
15:56:01 efried Could it really be that we got a spurious failure? That would seem a terrible coincidence. I'll recheck just in case.
15:56:01 efried I need to run for ~1h. donnyd is spinning us up an env where we can play with this more realtime-ish.
16:03:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
16:10:18 dustinc efried: awesome, thanks
16:40:44 donnyd I should have something to use in a bit, maybe an hour or so
17:12:01 stephenfin yonglihe: Sorry, missed the ping. I'll take a look at that Monday, sure :)
17:23:00 openstackgerrit ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
17:23:00 openstackgerrit ya.wang proposed openstack/nova master: vCPU model selection https://review.opendev.org/670298
17:23:01 openstackgerrit ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
17:42:15 sean-k-mooney hum look like gerrit is down
17:42:24 sean-k-mooney i think that is a sign
17:42:41 mnaser sean-k-mooney: wfm
17:43:15 sean-k-mooney nope cant open any of the links above
17:43:24 mnaser i can so many its your side :)
17:43:27 mnaser and also a sign :)
17:43:46 sean-k-mooney yep its friday and 6:43
17:43:53 sean-k-mooney time to not be working
18:07:38 openstackgerrit Merged openstack/nova master: libvirt: Mock libvirt'y things in setUp https://review.opendev.org/676727
18:52:06 efried_afk alex_xu: I had to give up on https://review.opendev.org/#/c/673151/ -- would you please review it when you get a chance?
19:12:12 openstackgerrit Merged openstack/nova master: Allow assertXmlEqual() to pass options to matchers.XMLMatches https://review.opendev.org/674628
19:12:19 openstackgerrit Merged openstack/nova master: Fix libvirt driver tests to use LibvirtConfigCapsGuest instances https://review.opendev.org/674629
19:12:52 artom dansmith, thanks for all the reviews - I won't get to them all this afternoon, obviously, but I'm aiming for Monday
19:13:01 dansmith ack
19:16:12 efried aspiers: https://review.opendev.org/#/c/638680/ qq if you're still around.
20:16:40 aspiers efried: yup?
20:16:59 efried aspiers: see inline, want to make sure caching-only-False is intentional
20:17:03 aspiers ok
20:22:49 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
20:23:44 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
20:24:38 aspiers efried: are there questions I can answer which would make reviewing https://review.opendev.org/#/c/673151/ a possibility for you?
20:25:02 aspiers efried: I really doubt I know significantly more than you in this area - you're probably just missing one or two key facts
20:25:42 efried aspiers: I am missing domain (so to speak) knowledge of libvirt+xml
20:25:47 efried like, the whole thing.
20:25:54 efried and machine types
20:25:57 efried and architectures
20:26:04 efried It's just all an enormous hole in my brain.
20:26:10 aspiers haha
20:26:20 aspiers it's really not that hard, honest :)
20:26:27 efried be like asking me to review, I dunno, a Windows kernel patch.
20:26:33 aspiers LOL
20:26:49 aspiers I doubt it
20:27:02 efried aspiers: But it's this one that I was asking about https://review.opendev.org/#/c/638680/41
20:27:08 efried specifically https://review.opendev.org/#/c/638680/41/nova/virt/libvirt/host.py@1186
20:27:45 aspiers oh, looks like I screwed it up
20:31:48 openstackgerrit James E. Blair proposed openstack/nova master: WIP: sphinx latex build tests https://review.opendev.org/677028
20:33:14 efried okay
20:44:04 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
20:44:21 efried dustinc, mordred: Okay, donnyd set up us the cloud, and I've got it configured to reproduce the failure
20:44:21 efried ...which afaict is, we're talking to ironic okay, but for some reason the node.list is coming back with 0 nodes.
20:44:34 efried so dustinc this might be where you need to jump in and debug.
20:49:17 openstackgerrit Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964
20:49:18 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
20:54:03 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
20:54:29 aspiers efried: what's the world record for the most patchsets in a nova review? :-o
20:54:40 aspiers am I even close yet?
20:54:58 aspiers seems like a very dubious achievement
20:55:16 efried you are nowhere close.
20:55:20 efried Well into three figures
20:55:22 aspiers hah, phew
20:55:24 aspiers wow
21:12:15 openstackgerrit Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964
21:12:15 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
21:14:36 aspiers Damn, I intended to not work at all today, and here I am still going ... time for late dinner
21:14:49 aspiers Just desperate to get this damn series all to +2
21:16:56 efried aspiers: https://review.opendev.org/#/c/676964/3/nova/tests/functional/libvirt/test_report_cpu_traits.py show me where STUB_INIT_HOST is being used?
21:19:36 aspiers efried: ah sorry, it *is* in the next patch
21:19:54 efried It's not worth respinning stuff for, just want to make sure I'm not blind.
21:19:59 aspiers yeah

Earlier   Later