Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-30
10:46:10 sean-k-mooney so this host whihc is likel intel x86 was detected as support svm which from the kvm error it is not
10:46:22 sean-k-mooney cool
10:46:32 artom But coffee first
10:46:43 sean-k-mooney kvm64 is the most portable cpu model that work with kvm
10:47:02 artom Daughter started school. I'm guess I'm just excited/stressed for her.
10:47:20 sean-k-mooney first time
10:47:24 artom Gym attendance took a hit as well
10:47:26 artom Yeah
10:47:58 kashyap sean-k-mooney: IIRC, there was a reason we went with existing CPU model 'qemu64'
10:48:02 kashyap In DevStack
10:48:04 sean-k-mooney that can be stressful for both her and you. im sure it will be fine
10:48:09 kashyap Because not all Gate hosts support KVM
10:48:25 sean-k-mooney kashyap: yes but we are specific testing someing with nested vert
10:48:39 sean-k-mooney e.g we are hardcodeing stuff and targeting 1 cloud that does
10:48:58 sean-k-mooney so we are trying to undo all the other suff that hard codes it to qemu
10:49:42 kashyap sean-k-mooney: Ah, for that context, sure
10:49:57 alex_xu stephenfin: emm...all are expected, since he still using vcpu_pin_set, that means no PCPU
10:50:02 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
10:50:09 sean-k-mooney kashyap: ill be writing a sperate permenet job later but this is a quick hack just to do some non me testing of artoms code
10:50:34 kashyap sean-k-mooney: Sure. I didn't realize the full context.
10:50:59 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
10:51:29 sean-k-mooney kashyap: its kind of a libvirt/qemu bug.
10:52:04 sean-k-mooney with host model it should not be trying to use svm for the l2 guest on a l1 guest that does not have it
10:52:21 sean-k-mooney but if we use kvm64 it wont do that so its all good
10:52:57 sean-k-mooney i would not run real workload with kvm64 but its perfect for testing
10:53:26 kashyap sean-k-mooney: What precisely is a libvirt/QEMU bug? /me reads on
10:53:31 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs https://review.opendev.org/678448
10:53:32 openstackgerrit Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449
10:53:32 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrive the allocations early https://review.opendev.org/678450
10:53:33 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
10:53:33 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/678453
10:53:34 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
10:53:34 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
10:53:35 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
10:53:35 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
10:54:14 sean-k-mooney kashyap: we are default to host-model because cpu_mode is none
10:54:17 sean-k-mooney but https://zuul.opendev.org/t/openstack/build/905b52e0b1e84532944f0606b5455381/log/logs/subnode-2/libvirt/qemu/instance-00000005.txt.gz
10:54:57 kashyap sean-k-mooney: Right
10:55:14 sean-k-mooney but qemu expose an cpu flag to the l2 guest that the l1 guest does not suport
10:56:08 kashyap sean-k-mooney: Hmm. (Also, yes, agreed - on kvm64 for testing-only)
10:56:32 sean-k-mooney its deffintly a qemu bug since we are not passing the -cpu flag at all in this case
10:56:53 sean-k-mooney so its up to qemu to figure out what cpu model to use and it selected an invalid one for the host
10:57:20 kashyap sean-k-mooney: File a bug, please, if you're confident. I'm struggling to multi-task
10:57:34 kashyap I'll be interested to see a simple reproducer.
10:57:41 openstackgerrit Brin Zhang proposed openstack/nova master: Add operator user_id/project_id to the migrations https://review.opendev.org/679413
10:57:54 sean-k-mooney i have hit this many many times
10:58:04 sean-k-mooney its been a thing in qemu for quite a while
10:58:09 sean-k-mooney but yes i should file a bug
10:58:11 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter https://review.opendev.org/671072
10:58:12 openstackgerrit Shilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata https://review.opendev.org/671074
10:58:12 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Add a new request filter to isolate aggregates https://review.opendev.org/671075
10:58:13 openstackgerrit Shilpa Devharakar proposed openstack/nova master: Enable request filter isolate_aggregates https://review.opendev.org/667952
10:59:10 sean-k-mooney the reproduce is basicaly boot a qemu l2 gues with kvm enabled in a l1 guest where the host is intel x86
10:59:35 sean-k-mooney it picks up support for vmx in the l1 guest but enable svm in the l2 guest
10:59:49 donnyd sean-k-mooney: is the numa thing working
10:59:51 sean-k-mooney i have seen other similar case for other instruction too
11:00:00 sean-k-mooney donnyd: havent tried it yet
11:00:06 donnyd oh ok
11:00:11 sean-k-mooney but im 99% sure it will
11:00:27 sean-k-mooney you have configured the flavor more or less how i do for dev
11:02:38 stephenfin kashyap: Is the secure boot spec punted til U?
11:02:42 stephenfin *to
11:02:50 stephenfin (i.e. https://review.opendev.org/#/c/506720/)
11:03:33 kashyap stephenfin: No, it's not. It's the CPU APIs spec that's punted to U
11:03:54 kashyap SB spec aiming for Train, still. I'm optimistic like that
11:05:28 openstackgerrit Brin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id https://review.opendev.org/674243
11:21:20 openstackgerrit Artom Lifshitz proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor https://review.opendev.org/678887
11:27:42 openstackgerrit Merged openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate https://review.opendev.org/571265
11:56:32 openstackgerrit Adam Spiers proposed openstack/nova master: libvirt/host.py: remove unnecessary temporary variable https://review.opendev.org/679340
11:57:05 openstackgerrit Adam Spiers proposed openstack/nova master: libvirt/host.py: remove unnecessary temporary variable https://review.opendev.org/679340
11:57:21 openstackgerrit Adam Spiers proposed openstack/nova master: Indent fake libvirt host capabilities fixtures more nicely https://review.opendev.org/679339
12:15:52 openstackgerrit sean mooney proposed openstack/nova master: make config drives sticky bug 1835822 https://review.opendev.org/669738
12:15:53 openstack bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,In progress] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney)
12:15:53 openstackgerrit sean mooney proposed openstack/nova master: Whether to use config drive depends on instance.configdrive https://review.opendev.org/485930
12:16:19 sean-k-mooney ... i forgot to update the commit message
12:16:33 sean-k-mooney actully no never mind i di
12:16:41 sean-k-mooney didnt need too
12:19:10 lyarwood urgh, stupid question time. I'm using functional.libvirt.base.ServersTestBase and mocking calls to nova.virt.libvirt.host.Host.get_connection but my functional test is still barfing http://paste.openstack.org/show/767633/
12:19:27 sean-k-mooney yep
12:19:41 sean-k-mooney so that does not work proably
12:19:50 sean-k-mooney you have to use _host i think
12:20:04 sean-k-mooney its sometimes works proably and sometiems does not dependin on your other mocking
12:21:36 lyarwood https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1595962.py - yeah I'm basically following the approach taken in here
12:22:05 lyarwood but using https://github.com/openstack/nova/blob/744615559eb575bb0cf0a5ddc91acf255ebc3825/nova/tests/functional/libvirt/base.py#L30-L93 as the base class
12:22:44 lyarwood aaaaaaah that's already mocking it in setup
12:22:47 lyarwood my bad
12:23:19 sean-k-mooney this
12:23:21 sean-k-mooney self.useFixture(fixtures.MonkeyPatch(
12:23:23 sean-k-mooney 'nova.virt.libvirt.host.libvirt',
12:23:25 sean-k-mooney fakelibvirt))
12:24:02 sean-k-mooney when you do
12:24:03 sean-k-mooney https://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1595962.py#L41
12:24:17 lyarwood No, lower
12:24:23 lyarwood # Mock the 'get_connection' function, as we're going to need to provide
12:24:24 lyarwood # custom capabilities for each test
12:24:26 lyarwood _p = mock.patch('nova.virt.libvirt.host.Host.get_connection')
12:24:28 lyarwood self.mock_conn = _p.start()
12:24:30 lyarwood self.addCleanup(_p.stop)
12:25:29 sean-k-mooney i dont see that in the file you linked but sounds like you have figured it out
12:27:04 lyarwood sean-k-mooney: sorry that's in the base class I'm using in the second link

Earlier   Later