Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-20
10:36:20 stephenfin yaawang: Sure thing
10:36:40 aspiers stephenfin: about who caring?
10:36:51 stephenfin Windriver
10:36:54 aspiers oh
10:37:04 aspiers SUSE used to use pg for OpenStack, but ditched it quite a while ago
10:37:06 stephenfin *Wind River?
10:37:22 stephenfin Yeah, I've no idea if it even works nowadays
10:37:24 sean-k-mooney cervigni: give me a sec and ill see if i can get the command
10:37:26 aspiers Yeah X-D Windriver makes me think of DLLs *shudder*
10:38:00 aspiers stephenfin: seems dumb that I have to install pg_config to run unit tests
10:38:13 stephenfin Or build docs
10:38:23 stephenfin or do anything, really
10:38:35 cervigni thanks
10:38:37 stephenfin tbh, we shouldn't even need the MySQL dependencies for that
10:39:09 aspiers true
10:39:36 sean-k-mooney cervigni: this is not the exact command but more or less does what you want
10:40:59 sean-k-mooney http://paste.openstack.org/show/760302/
10:41:18 sean-k-mooney you will want to pipe that to less and check iommu group 81
10:42:05 cervigni that is the same of what I pasted earlier, for IOMMU group 81 (where the problem comes from)
10:42:10 sean-k-mooney you can also just do ls /sys/kernel/iommu_groups/81/devices/
10:42:21 cervigni it reports only a lspci: -s: Invalid slot number
10:42:38 sean-k-mooney yes -s take a pci address not iommu group number
10:42:45 sean-k-mooney they are totally different things
10:42:51 cervigni ok
10:43:16 sean-k-mooney what do you get for "ls /sys/kernel/iommu_groups/81/devices/"
10:43:35 cervigni [root@compute-r6-h28 ~]# ls /sys/kernel/iommu_groups/81/devices/a0543cde-3934-460f-8994-6be8c80fe417/
10:43:35 cervigni driver iommu_group mdev_type nvidia power remove subsystem uevent
10:56:40 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Note use of 'nova-manage db sync --config-file' https://review.opendev.org/671298
10:56:40 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
10:56:41 openstackgerrit Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
11:34:50 openstackgerrit Stephen Finucane proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.opendev.org/639282
11:34:51 openstackgerrit Stephen Finucane proposed openstack/nova master: Move setting of device trust to privsep. https://review.opendev.org/639283
11:34:51 openstackgerrit Stephen Finucane proposed openstack/nova master: Move final bridge commands to privsep. https://review.opendev.org/639580
11:34:52 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.opendev.org/639581
11:34:52 openstackgerrit Stephen Finucane proposed openstack/nova master: We no longer need rootwrap. https://review.opendev.org/554438
11:34:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.opendev.org/639826
11:34:53 openstackgerrit Stephen Finucane proposed openstack/nova master: Address nits from privsep series https://review.opendev.org/677210
11:34:54 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP! Stop passing 'root_helper' argument to os-brick https://review.opendev.org/677451
12:43:26 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
12:46:36 openstackgerrit Adam Spiers proposed openstack/nova master: Track libvirt host/domain capabilities for multiple machine types https://review.opendev.org/673151
12:50:41 openstackgerrit Adam Spiers proposed openstack/nova master: Extract new base class for provider usage functional tests https://review.opendev.org/676964
12:50:41 openstackgerrit Adam Spiers proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported https://review.opendev.org/638680
12:51:33 openstackgerrit Alexandra Settle proposed openstack/nova stable/rocky: Fixing broken links https://review.opendev.org/677460
12:53:31 alex_xu aspiers: the line 697-701 of https://review.opendev.org/#/c/673151/11/nova/virt/libvirt/host.py is about the why we need that set of machine type?
12:54:10 aspiers alex_xu: We need to cover the main variants
12:54:39 alex_xu aspiers: ok, so the pc and q35 can cover all the cases, right?
12:54:43 aspiers yes
12:55:03 aspiers this is mentioned in the commit message
12:55:06 alex_xu aspiers: thanks, i see now
12:55:15 alex_xu aspiers: is it too arch specific?
12:55:31 alex_xu although I didn't see other arch need this method
12:55:32 aspiers pc / q35 are not hardcoded
12:56:18 aspiers alex_xu: they come from utils.get_default_machine_type() and libvirt's getCapabilities
12:56:28 alex_xu I mean in other arch, it is also about default and canonical covering all the cases
12:56:30 aspiers so other architectures will get a different set
12:56:36 aspiers yes, correct
12:57:03 alex_xu ok, I'm just going to trust the libvirt and platform experts
12:57:34 aspiers that's why I had to remove 'bamboo' from the fixture
12:58:15 sean-k-mooney bamboo as a fake arch or machine type :)
12:58:27 alex_xu hah
12:58:29 aspiers because the static fixtures for getDomainCapabilities on ppc did not return correct values for bamboo
12:58:30 sean-k-mooney i can see both working
12:58:44 aspiers bamboo is a real machine type for ppc
12:58:50 sean-k-mooney really ok
12:59:02 sean-k-mooney but ya its not in the static data we are using right
12:59:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Update api-ref for 2.75 to add config_drive in server update response https://review.opendev.org/677324
12:59:03 alex_xu aspiers: so the only complain is no unittest for host.py change, but I'm ok with a followup.
12:59:04 aspiers https://review.opendev.org/#/c/673151/13/nova/tests/unit/virt/libvirt/fakelibvirt.py@1673
13:01:20 openstackgerrit Mohammed Naser proposed openstack/nova master: requirements: move to using psycopg2-binary https://review.opendev.org/676684
13:01:49 openstackgerrit Mohammed Naser proposed openstack/nova master: requirements: move to using psycopg2-binary https://review.opendev.org/676684
13:02:52 aspiers alex_xu: hmm that's a good point
13:08:18 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.75 - Multiple API cleanup changes https://review.opendev.org/676275
13:10:16 mriedem final 2 patches for 2.75 https://review.opendev.org/#/q/topic:bp/api-consistency-cleanup+status:open
13:11:12 alex_xu aspiers: the spec said the trait "a) it cannot be used yet, only in the future when SEV support is fully implemented,"...that confuse me
13:11:30 alex_xu is there any update for the exist spec, we need that trait again?
13:12:29 sean-k-mooney alex_xu: we will need that trait only when intels mktme feature is supported
13:12:47 aspiers right
13:13:26 sean-k-mooney need is also a strong word there in that it is only needed if you specifica want SEV
13:13:52 sean-k-mooney if you jsut want encyrpetd memory you use the resouce class on its own
13:13:57 alex_xu sean-k-mooney: they share same resource class?
13:14:10 aspiers MKTME is not implemented yet
13:14:15 aspiers but we expect it to use the same resource class
13:14:21 alex_xu ok, i see now
13:14:32 aspiers the MKTME spec is still in draft
13:14:52 alex_xu yea, I didn't follow that spec
13:15:00 aspiers but the resource class is not used directly by users/operators, only internally
13:15:03 alex_xu but I "should"
13:15:17 aspiers externally they use hw:mem_encryption extra spec and hw_mem_encryption image property
13:15:40 aspiers and that gets translated in the RequestSpec to resources:MEM_ENCRYPTION_CONTEXT=1
13:15:54 sean-k-mooney given that one is an intel only tech and the other is an amd only tech we do not expec a singel RP to have inventoies fo both so 1 resouce class and 2 traits for what is effectvly the same thing form a guest point of view
13:16:08 aspiers right
13:16:38 alex_xu got it
13:17:54 openstackgerrit Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420
13:18:33 sean-k-mooney by the way these keystone auth issues "keystoneauth1.exceptions.auth_plugins.MissingAuthPlugin: An auth plugin is required to determine endpoint URL" if i rebase my patch these should go away right
13:21:09 sean-k-mooney they are unrelated to my code in anycase so ill assume its as a result of keystone/sdk changes
13:22:25 sean-k-mooney aspiers: by the way https://review.opendev.org/#/c/666914/7..8/nova/tests/unit/test_utils.py resolved my python 3.7 failure just in case you hit similar issues.
13:23:04 sean-k-mooney mock.patch.dict works slightly different then some of the other patch function so you might not have issues
13:23:05 aspiers sean-k-mooney: nice, thanks.
13:23:21 aspiers this should work for my patch_{open,exists} too
13:24:29 sean-k-mooney yep i commented on that

Earlier   Later