Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-07-26
00:06:41 melwitt yeah, agreed
00:12:18 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:20:43 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
00:22:03 openstackgerrit melanie witt proposed openstack/nova master: Verify archive_deleted_rows --all-cells in post test hook https://review.opendev.org/672840
03:00:44 openstackgerrit karim proposed openstack/nova-specs master: Support VM memory encryption using Intel's MKTME https://review.opendev.org/666769
03:22:36 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:23:42 openstackgerrit pengyuesheng proposed openstack/os-vif master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/672856
03:25:24 eandersson sean-k-mooney, no clue if that is the fix, but looks reasonable to me ^
03:26:08 openstackgerrit pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:27:17 openstackgerrit pengyuesheng proposed openstack/os-vif master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/672857
03:46:35 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
03:49:14 openstackgerrit Erik Olof Gunnar Andersson proposed openstack/nova master: Always cleanup when hitting MaxRetriesExceeded https://review.opendev.org/672855
04:25:41 openstackgerrit Merged openstack/nova-specs master: Add user_id to the migrations https://review.opendev.org/667894
04:31:37 alex_xu sean-k-mooney: stephenfin the real_time_mask indeed neeed the '^', you can check my reply https://review.opendev.org/#/c/668656/4/specs/train/approved/use-pcpu-vcpu-in-one-instance.rst@84
04:49:20 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Functional test for NUMA live migration https://review.opendev.org/672595
07:02:41 openstackgerrit Huachang Wang proposed openstack/os-traits master: CPU: add a trait for AVX512-VNNI support https://review.opendev.org/672888
07:48:57 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
07:48:57 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
07:48:58 openstackgerrit Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334
07:48:58 openstackgerrit Adam Spiers proposed openstack/nova master: Add and to config.py https://review.opendev.org/636318
07:48:59 openstackgerrit Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565
07:48:59 openstackgerrit Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616
07:51:36 openstackgerrit Alex Xu proposed openstack/nova master: Add the virt driver interface for claim and unclaim the devices https://review.opendev.org/670782
07:51:37 openstackgerrit Alex Xu proposed openstack/nova master: Moves the allocation retrieving early https://review.opendev.org/670783
07:51:37 openstackgerrit Alex Xu proposed openstack/nova master: Calling the virt driver's claim/unclaim_for_instance in resource tracker https://review.opendev.org/670784
07:51:38 openstackgerrit Alex Xu proposed openstack/nova master: Add DeviceManager to the libvirt virt driver https://review.opendev.org/671388
07:51:38 openstackgerrit Alex Xu proposed openstack/nova master: Populates the existing mediated devices in the libvirt device manager https://review.opendev.org/670787
07:51:39 openstackgerrit Alex Xu proposed openstack/nova master: Using the claim/unclaim_for_instance for mdevs https://review.opendev.org/671222
07:51:39 openstackgerrit Alex Xu proposed openstack/nova master: Adds functional test for creating the instance with vgpus https://review.opendev.org/671398
10:11:55 aspiers sean-k-mooney, efried, bbobrov: exciting progress! http://paste.openstack.org/show/754889/
10:13:21 aspiers cdent: inventory stuff seems to be working nicely :) ^^^
10:14:01 cdent excellent, glad to hear it
10:42:27 sean-k-mooney aspiers: yep that does look like its working well
10:42:59 sean-k-mooney aspiers: you said you finally booted an sev instance with openstack yesterday right or is that the goal for today
10:43:35 sean-k-mooney aspiers: cdent actully should th max unit on that inventory not be 1
10:44:02 sean-k-mooney you can only have one mem_encryption_context per vm right
10:45:19 openstackgerrit sean mooney proposed openstack/os-vif master: only disable mac ageing for ovs hybrid plug https://review.opendev.org/672834
10:45:33 sean-k-mooney this a relitivly minor nit
10:55:12 aspiers sean-k-mooney: booting is the goal for today
10:56:34 aspiers sean-k-mooney: I don't get why you think max_unit should be 1? there are 16 slots configured on that machine
10:56:57 sean-k-mooney yes but max_unit is the maxium amount you can request in a singe request
10:56:57 aspiers ohhh, are you saying max_unit represents the maximum per VM, not the maximum per host?
10:57:03 sean-k-mooney yes
10:57:04 aspiers ok I didn't know that
10:57:10 aspiers in that case yeah it should be 1
10:57:12 aspiers good catch
10:57:14 sean-k-mooney total is the total capasity
10:57:21 aspiers right that never quite made sense before
10:57:23 aspiers easy to fix
10:57:39 sean-k-mooney ya as i said minor nit
10:57:46 aspiers minor but v important!
10:57:50 sean-k-mooney we will only ever ask for 1 anyway
10:57:56 aspiers true
10:57:59 aspiers well
10:58:14 aspiers yeah
10:58:32 aspiers unless some operator requested the resource directly via the flavor
10:58:37 sean-k-mooney still it does not hurt to be defensive
10:58:40 sean-k-mooney ya
10:59:48 aspiers sean-k-mooney: is min_unit=1 correct though?
11:00:11 sean-k-mooney can you ask for less then 1 allocation of something if you want a resouce
11:00:14 sean-k-mooney yes it is
11:00:45 sean-k-mooney the min unit only applie if you ask for a resouce of that type
11:01:22 sean-k-mooney and since we dont want people saying resouce:MEM_ENCRYPTION_CONTEXT=0 it is correct
11:01:28 aspiers got it
11:01:30 aspiers thanks
11:02:35 sean-k-mooney resouce:MEM_ENCRYPTION_CONTEXT=0 would actully for the vm to land on a host that supported sev but not consume sev resouces so it could be used incorrectly by opertors to stave those nodes or ram cpus and disk
11:03:02 sean-k-mooney /for/force
11:03:51 aspiers good point
11:28:25 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace https://review.opendev.org/634549
11:28:25 openstackgerrit Luyao Zhong proposed openstack/nova master: Report pmem namespaces resources by provider tree https://review.opendev.org/634551
11:28:26 openstackgerrit Luyao Zhong proposed openstack/nova master: Indroduce new LibvirtConfigGuestPMEM https://review.opendev.org/672956
11:28:26 openstackgerrit Luyao Zhong proposed openstack/nova master: Populates existing pmems in libvirt device manager https://review.opendev.org/672957
11:28:27 openstackgerrit Luyao Zhong proposed openstack/nova master: Support instance creation and resize with pmems https://review.opendev.org/672958
11:31:38 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.75: Add 'power-update' external event https://review.opendev.org/645611
11:39:58 efried mnaser: re openstacksdk:
11:39:58 efried spec: http://specs.openstack.org/openstack/nova-specs/specs/train/approved/openstacksdk-in-nova.html
11:39:58 efried code: https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/openstacksdk-in-nova
11:40:58 efried mnaser: note that we're not using clouds.yaml. We're using newly introduced functionality to consume ksa.loading oslo.config opts to build the Connection obj.
11:42:27 efried aspiers, sean-k-mooney: You can't ask placement for an allocation of 0.
11:43:13 sean-k-mooney efried: because we explcitly check or because we set min_unit to 1
11:43:15 efried If you want to land on a SEV host but not consume SEV resources (why??) you would have to require the SEV trait. Which of course wouldn't be generic for SEV/MKTME like the rc is.
11:43:30 sean-k-mooney efried: we dont
11:43:43 efried sean-k-mooney: I'm actually not sure. Because I'm pretty sure you can't set min_unit to 0 either.
11:43:59 sean-k-mooney efried: aspiers was just asking if min_unit=1 was correct
11:44:04 sean-k-mooney and i was saying yes it is
11:44:08 efried yeah, it certainly is.
11:44:17 efried and per the design, max_unit=1 is also correct.
11:44:21 efried as you said
11:48:06 efried alex_xu: yt?
11:50:04 efried sean-k-mooney, aspiers: Who among cores do y'all think would be appropriate reviewers for https://review.opendev.org/670189 (harden get_domain_capabilities)?
11:52:32 sean-k-mooney good question. am most of the redhat folk are familar with libvirt, matt knows all but is also always over subsribed, maybe alex_xu on the intel side
11:53:16 sean-k-mooney to be honest we added a tone of comments so most core should be able to review at this point
11:54:36 sean-k-mooney short list would proably be dansmith bauzas alex_xu or mriedem
12:00:19 sean-k-mooney stephenfin is on pto today and maybe monday hence why he is not on the list
12:04:19 sean-k-mooney johnthetubaguy: by the way regarding https://review.opendev.org/#/c/669190/
12:05:29 sean-k-mooney one i assume you still want to merge ^ but second i assume you are not against either reverting the feature entrily or making it work properly
12:24:39 openstackgerrit Merged openstack/os-vif master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/672856
12:57:37 aspiers efried: re. core reviewers, I can't add anything to sean-k-mooney's answer
12:57:52 aspiers I just found an issue with some of the SEV tests
12:58:11 aspiers turns out you can't use decorators on classes after all :-( or at least in some circumstances. digging more now

Earlier   Later