Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-10
11:42:05 kashyap "If SEV is supported, MSR 0xc0010131 (MSR_AMD64_SEV) can be used to determine if SEV is active: [...]"
11:42:16 aspiers Yeah, I've already used that in the past
11:42:49 kashyap Ah, then disregard me.
11:42:57 aspiers No, the reminder is appreciated
11:43:39 kashyap aspiers: Your goal is to check if the instance (in the upstream CI) booted has indeed with SEV, yeah?
11:43:44 aspiers right
11:44:02 aspiers I guess the JeOS image will need to include cpuid
11:44:32 kashyap Isn't the JeOS (Just Enough OS, I presume) CirrOS in this case?
11:44:55 aspiers It's whatever tempest is configured with
11:45:07 kashyap (Nod)
11:47:13 sean-k-mooney aspiers: has one of the upstream ci provered provided you with a lable that will run on SEV hardware
11:47:41 sean-k-mooney aspiers: because 90% of the upstream ci cloud are proably runing intel x86_64
11:47:57 sean-k-mooney although rackspace did run power for a while
11:48:19 aspiers sean-k-mooney: SUSE has our own SEV boxes which can run 3rd party CI
11:48:42 sean-k-mooney ah so its for the suse thrid party ci not for upstream
11:48:55 aspiers well if upstream has SEV hardware then great, but I was not expecting that any time soon
11:49:10 sean-k-mooney aspiers: i dont think it currely does
11:49:17 sean-k-mooney or at least not in a way you can target
11:49:49 sean-k-mooney im sure some of the clould proably have at least a small amd eypc inventory if for nothing else but there own internal validation
11:50:50 aspiers yeah
12:00:03 donnyd sean-k-mooney: have the numa jobs been running?
12:02:57 donnyd I saw last night it was having some inbound ssh timeout issues.
12:04:09 sean-k-mooney i have not checked this morning but i think clarkb kicked off a job around 4/5 am so ill see if that passed
12:04:51 sean-k-mooney donnyd: so that one failed at 04:54
12:05:01 sean-k-mooney donnyd: was that after your l3 agent restart
12:05:22 donnyd i didn't restart till well after you were talking about it on infra last night
12:06:31 sean-k-mooney donnyd: ya looking at the infra scroll back clarkb kicked that off after you did the restart
12:06:35 openstackgerrit Chris Dent proposed openstack/os-resource-classes master: Update api-ref link to canonical location https://review.opendev.org/681235
12:06:46 donnyd oh yea i see
12:07:31 donnyd It really just makes no sense though
12:07:42 donnyd all the rest of the labels seem to work without issue
12:07:57 donnyd mostly without issue... not anymore than any other provider at least
12:09:54 openstackgerrit Chris Dent proposed openstack/os-traits master: Update README to be a bit more clear https://review.opendev.org/681237
12:09:57 donnyd So the old label we have setup for numa, does that still work?
12:10:07 donnyd I haven't set it back on my end
12:12:44 openstackgerrit Balazs Gibizer proposed openstack/nova master: Shrink the race window in confirm resize func test https://review.opendev.org/681238
12:25:02 kashyap aspiers: Is there a way to detect from _inside_ the guest that it has indeed booted with SEV?
12:25:20 kashyap aspiers: (From the host, we can use the CPUID bit or other ways)
12:26:51 kashyap aspiers: Meanwhile, I learnt that if SEV isn't provided by the host, an SEV-enabled kernel should fail to boot.
12:27:38 openstackgerrit Alexandra Settle proposed openstack/nova master: Fixing broken links https://review.opendev.org/681206
12:29:31 openstackgerrit Chris Dent proposed openstack/os-traits master: Update README to be a bit more clear https://review.opendev.org/681237
12:30:13 gibi efried, aspiers: Is there anything I can do regarding the SEV series?
12:30:15 stephenfin aspiers: Random question: why can we not add the iommu attribute for all those device types?
12:30:30 stephenfin gibi: I'm reviewing the last patch now. Think you've already hit it though
12:30:36 stephenfin by which I mean the first one
12:30:41 stephenfin since the other two are +2
12:30:49 stephenfin +W
12:30:51 gibi stephenfin: cool. Yeah I tried to find somebody how can look at the first
12:31:05 gibi but then it is in good hands now
12:33:40 kashyap aspiers: Also, when you're about, see my response to your comment: https://review.opendev.org/#/c/348394/10
12:34:33 kashyap aspiers: If you can confirm that /usr/share/qemu/ovmf-x86_64-suse-code.bin is indeed the binary built with Secure Boot, then we can fix it
12:34:36 luyao Hi, everyone, I have a question, when an instance is in rebuild, how many allocations will it have? Both new and old ?
12:43:00 brinzhang efried: Could you please review this patch https://review.opendev.org/#/c/681151/, it changes the novalient version to 15.1.0, needed by https://review.opendev.org/#/c/673725/
12:44:12 stephenfin aspiers: Is that follow-up for https://review.opendev.org/#/c/666616/ around yet?
12:47:35 gibi stephenfin: I did not find the followup either so I guess it isn't
12:58:43 openstackgerrit Stephen Finucane proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
12:58:44 openstackgerrit Stephen Finucane proposed openstack/nova master: Reject live migration and suspend on SEV guests https://review.opendev.org/680158
12:58:44 openstackgerrit Stephen Finucane proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
12:58:45 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
12:58:45 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693
12:58:46 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'InstanceNUMATopology.cpu_pinning' property https://review.opendev.org/680106
12:58:46 openstackgerrit Stephen Finucane proposed openstack/nova master: Validate CPU config options against running instances https://review.opendev.org/680107
12:58:47 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Use sane indent https://review.opendev.org/680229
12:58:47 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108
12:58:48 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
12:58:48 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571
12:58:49 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for translating CPU policy extra specs, image meta https://review.opendev.org/671801
12:58:49 openstackgerrit Stephen Finucane proposed openstack/nova master: fakelibvirt: Make 'Connection.getHostname' unique https://review.opendev.org/681060
12:58:50 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Mock 'libvirt_utils.file_open' properly https://review.opendev.org/681061
12:58:50 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895
12:58:55 bauzas gibi: add a question for you https://review.opendev.org/#/c/676140/
12:58:59 bauzas hadù
12:59:07 gibi bauzas: looking
13:00:00 openstackgerrit Stephen Finucane proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
13:00:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Reject live migration and suspend on SEV guests https://review.opendev.org/680158
13:00:01 openstackgerrit Stephen Finucane proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
13:00:02 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting PCPU inventory to placement https://review.opendev.org/671793
13:00:02 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: '_get_(v|p)cpu_total' to '_get_(v|p)cpu_available' https://review.opendev.org/672693
13:00:03 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'InstanceNUMATopology.cpu_pinning' property https://review.opendev.org/680106
13:00:03 openstackgerrit Stephen Finucane proposed openstack/nova master: Validate CPU config options against running instances https://review.opendev.org/680107
13:00:04 openstackgerrit Stephen Finucane proposed openstack/nova master: trivial: Use sane indent https://review.opendev.org/680229
13:00:04 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Add 'NUMACell.pcpuset' field https://review.opendev.org/680108
13:00:05 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Differentiate between shared and dedicated CPUs https://review.opendev.org/671800
13:00:05 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Start reporting 'HW_CPU_HYPERTHREADING' trait https://review.opendev.org/675571
13:00:06 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for translating CPU policy extra specs, image meta https://review.opendev.org/671801
13:00:06 openstackgerrit Stephen Finucane proposed openstack/nova master: fakelibvirt: Make 'Connection.getHostname' unique https://review.opendev.org/681060
13:00:07 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Mock 'libvirt_utils.file_open' properly https://review.opendev.org/681061
13:00:07 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895
13:04:55 stephenfin bauzas, alex_xu: I've merged in those follow-ups to https://review.opendev.org/#/c/671793/ and the next two patches, per dansmith's request, if you fancy re +2ing
13:05:18 bauzas ack
13:07:05 gibi bauzas: replied in https://review.opendev.org/#/c/676140
13:07:13 bauzas gibi: cool thanks
13:07:59 luyao efried: Hi efried, are you around?
13:08:13 mriedem gibi: do you have a bug reported for https://954d3ddb67e757934983-a9cc155153d08dd30dfffbbf1d71d234.ssl.cf5.rackcdn.com/676138/16/gate/nova-tox-functional-py36/fb5d235/testr_results.html.gz yet?
13:08:47 gibi mriedem: not yet. I have a patch that shrinks the window
13:08:58 gibi https://review.opendev.org/#/c/681238/
13:09:28 efried luyao: o/
13:10:09 bauzas gibi: mriedem: just an unrelated thought, should we somehow persist VGPU resource request to the RequestSpec requested_resources field ?
13:10:33 luyao efried: I would like another patch to refactor this later if necessary. Current appproach works well and it will be great for me to change after merged. https://review.opendev.org/#/c/678452/22/nova/compute/resource_tracker.py@1168
13:10:34 gibi requested_resources field is intentionally not persistes in the db

Earlier   Later