Friday, 2019-08-30

*** avolkov has quit IRC00:01
*** threestrands has quit IRC00:03
*** slaweq has joined #openstack-nova00:11
*** slaweq has quit IRC00:15
*** markvoelker has joined #openstack-nova00:15
*** markvoelker has quit IRC00:20
*** dpawlik has joined #openstack-nova00:20
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for node.list  https://review.opendev.org/65602700:24
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for validating instance and node  https://review.opendev.org/65602800:24
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for setting instance id  https://review.opendev.org/65969000:24
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for add/remove instance info from node  https://review.opendev.org/65969100:24
openstackgerritDustin Cowles proposed openstack/nova master: Use SDK for getting network metadata from node  https://review.opendev.org/67021300:24
*** dpawlik has quit IRC00:24
dustinc🤞00:24
dustinc"all tests pass on my machine"00:25
*** tbachman has quit IRC00:33
*** gbarros has quit IRC00:34
*** tbachman has joined #openstack-nova00:34
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Create specs directory for Ussuri  https://review.opendev.org/67936400:35
artommriedem, ack - thanks for spending time on this00:35
*** mlavalle has quit IRC00:44
*** hongbin has quit IRC00:44
*** tbachman_ has joined #openstack-nova00:45
*** mriedem has quit IRC00:45
*** tbachman has quit IRC00:45
*** tbachman_ is now known as tbachman00:45
*** brinzhang_ has joined #openstack-nova01:01
*** bhagyashris has joined #openstack-nova01:03
*** brinzhang has quit IRC01:04
*** licanwei has quit IRC01:12
*** itlinux-away is now known as itlinux01:39
*** itlinux is now known as itlinux-away01:56
*** markvoelker has joined #openstack-nova02:05
*** markvoelker has quit IRC02:10
*** slaweq has joined #openstack-nova02:11
openstackgerritFan Zhang proposed openstack/nova master: Clear RequestSpec requested_destination and ignore_hosts in confirm/revert resize.  https://review.opendev.org/67921202:14
*** ricolin has joined #openstack-nova02:16
*** slaweq has quit IRC02:16
openstackgerritYongli He proposed openstack/nova master: Add server sub-resource topology API  https://review.opendev.org/62147602:17
*** tetsuro has joined #openstack-nova02:20
*** dpawlik has joined #openstack-nova02:21
*** brinzhang has joined #openstack-nova02:22
*** dpawlik has quit IRC02:25
*** brinzhang_ has quit IRC02:26
*** nicolasbock has quit IRC02:52
*** hongbin has joined #openstack-nova03:02
*** psachin has joined #openstack-nova03:17
*** itlinux-away is now known as itlinux03:31
openstackgerritMerged openstack/nova master: Add FUP unit test for port heal allocations  https://review.opendev.org/67214203:50
*** mkrai has joined #openstack-nova03:52
*** udesale has joined #openstack-nova03:52
*** mkrai has quit IRC03:55
*** mkrai has joined #openstack-nova03:58
*** dklyle has quit IRC04:00
*** david-lyle has joined #openstack-nova04:00
*** slaweq has joined #openstack-nova04:03
*** slaweq has quit IRC04:08
*** mkrai has quit IRC04:09
*** mkrai has joined #openstack-nova04:11
*** dave-mccowan has quit IRC04:14
*** hongbin has quit IRC04:19
*** jhesketh has quit IRC04:20
*** dpawlik has joined #openstack-nova04:22
*** mkrai has quit IRC04:22
*** mkrai has joined #openstack-nova04:23
*** dpawlik has quit IRC04:26
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Create specs directory for Ussuri  https://review.opendev.org/67936404:43
*** ratailor has joined #openstack-nova04:48
*** ratailor has quit IRC04:50
*** ratailor has joined #openstack-nova04:50
*** larainema has joined #openstack-nova04:54
*** ccamacho has joined #openstack-nova05:00
*** markvoelker has joined #openstack-nova05:00
*** bhagyashris has quit IRC05:00
*** tkajinam has quit IRC05:05
*** markvoelker has quit IRC05:06
*** slaweq has joined #openstack-nova05:11
*** tetsuro has quit IRC05:12
*** slaweq has quit IRC05:15
*** hamzy has quit IRC05:16
*** tetsuro has joined #openstack-nova05:18
*** stakeda has quit IRC05:20
*** jaosorior has joined #openstack-nova05:27
*** dpawlik has joined #openstack-nova05:30
openstackgerritMerged openstack/nova master: Move live_migration test hooks under gate/  https://review.opendev.org/67925105:37
*** tkajinam has joined #openstack-nova05:41
*** igordc has quit IRC05:46
*** slaweq has joined #openstack-nova06:08
*** sapd1 has joined #openstack-nova06:08
*** slaweq has quit IRC06:13
*** xek has joined #openstack-nova06:18
*** luksky has joined #openstack-nova06:28
*** brault has joined #openstack-nova06:30
*** brault has quit IRC06:35
*** avolkov has joined #openstack-nova06:35
*** rcernin has quit IRC06:48
*** aojea has joined #openstack-nova06:49
*** sapd1 has quit IRC06:49
*** slaweq has joined #openstack-nova06:49
*** slaweq has quit IRC06:54
openstackgerritya.wang proposed openstack/nova master: vCPU model selection  https://review.opendev.org/67029806:59
openstackgerritya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags  https://review.opendev.org/67029906:59
openstackgerritya.wang proposed openstack/nova master: Support reporting multi CPU model traits  https://review.opendev.org/67030006:59
openstackgerritLEIJIE SHA proposed openstack/os-vif master: Fix code bug in document  https://review.opendev.org/67938207:04
*** slaweq has joined #openstack-nova07:05
*** trident has quit IRC07:10
*** itlinux is now known as itlinux-away07:11
*** trident has joined #openstack-nova07:18
*** trident has quit IRC07:22
*** trident has joined #openstack-nova07:30
*** lpetrut has joined #openstack-nova07:30
*** jhesketh has joined #openstack-nova07:38
*** cdent has joined #openstack-nova07:39
*** tetsuro has quit IRC07:46
*** dtantsur|afk is now known as dtantsur07:46
*** ivve has joined #openstack-nova07:50
*** shilpasd has joined #openstack-nova08:01
*** cdent has quit IRC08:06
*** dougsz has joined #openstack-nova08:07
*** jaosorior has quit IRC08:07
*** brault has joined #openstack-nova08:12
*** mdbooth has joined #openstack-nova08:12
*** dpawlik has quit IRC08:12
*** markvoelker has joined #openstack-nova08:21
*** ralonsoh has joined #openstack-nova08:26
*** markvoelker has quit IRC08:27
*** tkajinam has quit IRC08:27
*** derekh has joined #openstack-nova08:35
*** dpawlik has joined #openstack-nova08:39
*** cdent has joined #openstack-nova08:41
*** dpawlik has quit IRC08:43
stephenfinbauzas, sean-k-mooney, alex_xu: You'll probably be interested in Bhagyashri's comments here https://review.opendev.org/#/c/674895/08:50
* bauzas clicks08:50
stephenfintl;dr: It works as expected but we do need to tweak that config option as noted08:50
sean-k-mooney e.g. default it to true08:51
bauzasyup, makes sense08:51
bauzaswhat ? I'm not always saying terrible ideas ?08:52
bauzasso sad08:52
bauzas:p08:52
*** dpawlik has joined #openstack-nova08:52
bauzasstephenfin: FWIW, I'll start looking over the rest of the series later in the afternoon08:54
sean-k-mooneyby the way thanks to donnyd we now have a multi numa lable in the gate with nested vert adn teh ablity to use hugepages,dpdk,and pinning testing in the gate... os i have a ci job to go right08:55
sean-k-mooney*write08:55
bauzascool08:56
sean-k-mooneyoh by the way i mentioned this to artom alreay but i also figured out how to do nested pci passthough over the weekend08:57
sean-k-mooneye.g. pci passthough from a host all the way to an l2 guest08:57
sean-k-mooneyand i might be able to use virtual device in the l1 guest as pci deivce to the l2 guest also.08:58
sean-k-mooneynova cant do it yet due to how we currently generate teh xml but i might be able to use that to test sriov or pci passthough at some point.08:59
kashyapsean-k-mooney: On nested PCI passthrough, have you seen this past blog from DanPB: https://www.berrange.com/posts/2017/02/16/setting-up-a-nested-kvm-guest-for-developing-testing-pci-device-assignment-with-numa/08:59
stephenfinbauzas: Not always terrible. Just mostly terrible ;)09:00
bauzaslike proposing to ski over a weekend ?09:00
*** dougsz has quit IRC09:00
bauzasI can assume that09:00
stephenfinthat was one of the non-terrible ones09:01
sean-k-mooney kashyap i think a few years ago09:01
sean-k-mooneykashyap: once we got a virtula iommu and q35 we could technically do it09:01
sean-k-mooneybut the issue is getin ghte right pci tree such  that the passthough devices are in there own iommu group09:02
sean-k-mooneyand ya i have been advocating creating a pci root per numa node for years at this point09:03
kashyapsean-k-mooney: On virtual IOMMU, there was a talk on its implementation earlier this year at FOSDEM:09:03
kashyaphttps://archive.fosdem.org/2019/schedule/event/vai_iommu_implementation_using_hw_nested_paging/09:03
kashyap(It's a different approach to existing impl.)09:03
sean-k-mooneyok09:04
sean-k-mooneyat this point if qemu supported a virtual device that had sriov support we would have everything we need to be able to test sriov in the gate09:08
sean-k-mooneywell assuming we lanned all the reuqired changes in openstack and a cloud provier or two ran that version and hadn the correct flavors09:08
kashyapsean-k-mooney: QEMU supporting an emulated SRIOV device is the key bit -- needs someone familiar with a card to implement it.09:10
kashyapI won't count on it, though09:10
*** jaosorior has joined #openstack-nova09:10
sean-k-mooneythere were patches for it before09:10
sean-k-mooneythey just never landed09:10
*** hamzy has joined #openstack-nova09:10
sean-k-mooneykashyap: https://github.com/knuto/qemu09:11
kashyapsean-k-mooney: Yes, I know the author.  He works for Oracle09:11
kashyapLast I checked he had a student working on it...but the student left09:12
kashyapKnut ('knuto') introduced the supporting infrastructure for adding an emulated SRIOV card09:12
*** xek has quit IRC09:20
openstackgerritLuyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs  https://review.opendev.org/67844809:22
openstackgerritLuyao Zhong proposed openstack/nova master: Add resources dict into _Provider  https://review.opendev.org/67844909:22
openstackgerritLuyao Zhong proposed openstack/nova master: Retrive the allocations early  https://review.opendev.org/67845009:22
openstackgerritLuyao Zhong proposed openstack/nova master: Claim resources in resource tracker  https://review.opendev.org/67845209:22
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces  https://review.opendev.org/67845309:22
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree  https://review.opendev.org/67845409:22
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup  https://review.opendev.org/67845509:22
openstackgerritLuyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec  https://review.opendev.org/67845609:22
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847009:22
openstackgerritLuyao Zhong proposed openstack/nova master: Retrive the allocations early  https://review.opendev.org/67845009:26
openstackgerritLuyao Zhong proposed openstack/nova master: Claim resources in resource tracker  https://review.opendev.org/67845209:26
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces  https://review.opendev.org/67845309:26
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree  https://review.opendev.org/67845409:26
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup  https://review.opendev.org/67845509:26
openstackgerritLuyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec  https://review.opendev.org/67845609:26
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847009:26
openstackgerritBalazs Gibizer proposed openstack/nova master: Add cold migrate and resize to nova-grenade-multinode  https://review.opendev.org/67921009:30
openstackgerritStephen Finucane proposed openstack/nova master: Provide HW_CPU_X86_AMD_SEV trait when SEV is supported  https://review.opendev.org/63868009:37
*** mkrai has quit IRC09:39
*** mkrai has joined #openstack-nova09:39
sean-k-mooneyjust tested the v22 iteration of artoms patches and its still working correctly09:39
*** dpawlik has quit IRC09:46
openstackgerritBrin Zhang proposed openstack/nova master: Add operator user_id/project_id to the migrations  https://review.opendev.org/67941309:49
*** mkrai has quit IRC09:49
*** luksky has quit IRC09:52
sean-k-mooneyfor those not on the neutron channel i just confrimed that cold migration and live migration with vnic_type=direct-physical is broken10:08
sean-k-mooneythere are at least two related issues, 1 we do not update the mac on migration and second we do not correctly find the device to detach it on live migration10:09
sean-k-mooneylive migration also like has the first issue but its failing before it gets that far10:09
*** icarusfactor has quit IRC10:11
*** icarusfactor has joined #openstack-nova10:11
*** dpawlik has joined #openstack-nova10:12
*** icarusfactor has quit IRC10:12
sean-k-mooneyit looks like detach works correctly for suspend and the live migration code should be useing the same fucntion i think but maybe its not10:12
*** icarusfactor has joined #openstack-nova10:13
sean-k-mooneyim going to try and fix this next week and ill file the upstream bug shortly. i think the cold migration behavior is broken all the way back to queens. the live migration behavior is obviouly new since we added that this cycle10:14
*** factor__ has joined #openstack-nova10:17
*** icarusfactor has quit IRC10:20
*** brault has quit IRC10:23
*** trident has quit IRC10:23
*** aojea has quit IRC10:23
*** udesale has quit IRC10:23
*** brinzhang has quit IRC10:23
*** tbachman has quit IRC10:23
*** cgoncalves has quit IRC10:23
*** francoisp has quit IRC10:23
*** bnemec has quit IRC10:23
*** owalsh|away has quit IRC10:23
*** frickler has quit IRC10:23
*** NostawRm has quit IRC10:23
*** tobias-urdin has quit IRC10:23
*** mloza has quit IRC10:24
*** lyarwood has quit IRC10:24
*** efried has quit IRC10:24
*** kevinbenton has quit IRC10:24
*** dtruong has quit IRC10:24
*** jbernard has quit IRC10:24
*** edleafe has quit IRC10:24
*** mandre has quit IRC10:24
*** ebbex has quit IRC10:24
*** eandersson has quit IRC10:24
*** evrardjp has quit IRC10:24
*** amorin has quit IRC10:24
*** lifeless has quit IRC10:24
*** weshay_pto has quit IRC10:24
*** ociuhandu has joined #openstack-nova10:24
*** NostawRm has joined #openstack-nova10:24
*** trident has joined #openstack-nova10:24
*** frickler has joined #openstack-nova10:24
stephenfinaspiers: Hit that whole SEV series. Couple of comments left, but ping me when you've tackled them and I'll re-review10:25
*** irclogbot_3 has quit IRC10:25
*** lbragstad has quit IRC10:25
*** brinzhang has joined #openstack-nova10:26
*** jbernard has joined #openstack-nova10:26
*** cgoncalves has joined #openstack-nova10:26
*** brinzhang has quit IRC10:26
*** aojea has joined #openstack-nova10:26
*** efried has joined #openstack-nova10:27
*** kevinbenton has joined #openstack-nova10:27
*** evrardjp has joined #openstack-nova10:27
*** lbragstad has joined #openstack-nova10:27
*** brinzhang has joined #openstack-nova10:27
*** tbachman has joined #openstack-nova10:27
*** dtruong has joined #openstack-nova10:27
*** owalsh has joined #openstack-nova10:27
*** irclogbot_3 has joined #openstack-nova10:28
*** bnemec has joined #openstack-nova10:28
*** lifeless has joined #openstack-nova10:29
*** ebbex has joined #openstack-nova10:29
*** amorin has joined #openstack-nova10:33
*** mandre has joined #openstack-nova10:33
artomsean-k-mooney, hey, so how feasible would be to hack together a tempest job that runs https://review.rdoproject.org/r/#/c/18832/ on that new FN flavor?10:36
artom('morning!)10:36
sean-k-mooneyartom: well my nfv test job did actully work a few months ago so i could make it work on FN pretty quick10:37
sean-k-mooneyas in today10:37
sean-k-mooneybut im currently reworking functional test10:37
sean-k-mooneywant me to give it ago when im done10:38
*** mkrai has joined #openstack-nova10:38
artomsean-k-mooney, that'd be awesome :) whitebox needs a bunch of configs to be set, I can help with those10:38
artomhttps://review.opendev.org/#/c/678887/ failed with KVM errors, so I'm trying think of other possibilities10:39
sean-k-mooneyya well first thing would be to just get it stacking multinode with a few smoketests. then we can add the plugin10:39
artomRight, baby steps :)10:40
sean-k-mooneyartom: i dont see errors in the n-cpu log10:40
sean-k-mooneyoh multinode10:41
sean-k-mooneystill no erros on the other node10:41
artomsean-k-mooney, no, they're in the qemu logs10:42
artomex: https://zuul.opendev.org/t/openstack/build/905b52e0b1e84532944f0606b5455381/log/logs/subnode-2/libvirt/qemu/instance-00000005.txt.gz10:42
*** ociuhandu has quit IRC10:42
sean-k-mooneyoh thats the kernel bug i mentioned to donnyd10:42
sean-k-mooneyactully may be not10:42
sean-k-mooneyit could be but that looks like an issue with the cpu model10:43
donnydi am using host-passthrough on nova10:43
*** jaosorior has quit IRC10:43
sean-k-mooneyits checking for svm with is amds version of vt-x10:43
sean-k-mooneydonnyd: ya this is on vexhost10:43
sean-k-mooneybut i think this is a different issue10:44
donnydoh10:44
sean-k-mooneyya so10:44
sean-k-mooney[libvirt]10:44
sean-k-mooneylive_migration_uri = qemu+ssh://stack@%s/system10:44
sean-k-mooneycpu_mode = none10:44
sean-k-mooneyvirt_type = kvm10:44
sean-k-mooneywe should be setting cpu_mode=custom and cpu_model=kvm64 in devstack10:45
sean-k-mooneywith cpu_mode=none we default to host-model10:45
sean-k-mooneybut host-model does not always detect the capablity proably espcailly with nested virt10:45
openstackgerritBrin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id  https://review.opendev.org/67424310:45
artomsean-k-mooney, I'll just add that to Matt's devstack nuke10:46
sean-k-mooneyso this host whihc is likel intel x86 was detected as support svm which from the kvm error it is not10:46
sean-k-mooneycool10:46
artomBut coffee first10:46
sean-k-mooneykvm64 is the most portable cpu model that work with kvm10:46
* artom is running on sleep fumes this week.10:46
artomDaughter started school. I'm guess I'm just excited/stressed for her.10:47
sean-k-mooneyfirst time10:47
artomGym attendance took a hit as well10:47
artomYeah10:47
kashyapsean-k-mooney: IIRC, there was a reason we went with existing CPU model 'qemu64'10:47
kashyapIn DevStack10:48
sean-k-mooneythat can be stressful for both her and you. im sure it will be fine10:48
kashyapBecause not all Gate hosts support KVM10:48
sean-k-mooneykashyap: yes but we are specific testing someing with nested vert10:48
sean-k-mooneye.g we are hardcodeing stuff and targeting 1 cloud that does10:48
sean-k-mooneyso we are trying to undo all the other suff that hard codes it to qemu10:48
*** tbachman has quit IRC10:49
kashyapsean-k-mooney: Ah, for that context, sure10:49
alex_xustephenfin: emm...all are expected, since he still using vcpu_pin_set, that means no PCPU10:49
openstackgerritBrin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id  https://review.opendev.org/67424310:50
sean-k-mooneykashyap: ill be writing a sperate permenet job later but this is a quick hack just to do some non me testing of artoms code10:50
kashyapsean-k-mooney: Sure.  I didn't realize the full context.10:50
openstackgerritBrin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id  https://review.opendev.org/67424310:50
sean-k-mooneykashyap: its kind of a libvirt/qemu bug.10:51
sean-k-mooneywith host model it should not be trying to use svm for the l2 guest on a l1 guest that does not have it10:52
sean-k-mooneybut if we use kvm64 it wont do that so its all good10:52
sean-k-mooneyi would not run real workload with kvm64 but its perfect for testing10:52
*** luksky has joined #openstack-nova10:53
kashyapsean-k-mooney: What precisely is a libvirt/QEMU bug?  /me reads on10:53
openstackgerritLuyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs  https://review.opendev.org/67844810:53
openstackgerritLuyao Zhong proposed openstack/nova master: Add resources dict into _Provider  https://review.opendev.org/67844910:53
openstackgerritLuyao Zhong proposed openstack/nova master: Retrive the allocations early  https://review.opendev.org/67845010:53
openstackgerritLuyao Zhong proposed openstack/nova master: Claim resources in resource tracker  https://review.opendev.org/67845210:53
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces  https://review.opendev.org/67845310:53
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree  https://review.opendev.org/67845410:53
openstackgerritLuyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup  https://review.opendev.org/67845510:53
openstackgerritLuyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec  https://review.opendev.org/67845610:53
openstackgerritLuyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory  https://review.opendev.org/67847010:53
sean-k-mooneykashyap: we are default to host-model because cpu_mode is none10:54
sean-k-mooneybut https://zuul.opendev.org/t/openstack/build/905b52e0b1e84532944f0606b5455381/log/logs/subnode-2/libvirt/qemu/instance-00000005.txt.gz10:54
kashyapsean-k-mooney: Right10:54
sean-k-mooneybut qemu expose an cpu flag to the l2 guest that the l1 guest does not suport10:55
kashyapsean-k-mooney: Hmm.  (Also, yes, agreed - on kvm64 for testing-only)10:56
sean-k-mooneyits deffintly a qemu bug since we are not passing the -cpu flag at all in this case10:56
sean-k-mooneyso its up to qemu to figure out what cpu model to use and it selected an invalid one for the host10:56
kashyapsean-k-mooney: File a bug, please, if you're confident.  I'm struggling to multi-task10:57
kashyapI'll be interested to see a simple reproducer.10:57
openstackgerritBrin Zhang proposed openstack/nova master: Add operator user_id/project_id to the migrations  https://review.opendev.org/67941310:57
sean-k-mooneyi have hit this many many times10:57
sean-k-mooneyits been a thing in qemu for quite a while10:58
sean-k-mooneybut yes i should file a bug10:58
openstackgerritShilpa Devharakar proposed openstack/nova master: Nova object changes for forbidden aggregates request filter  https://review.opendev.org/67107210:58
openstackgerritShilpa Devharakar proposed openstack/nova master: DB API changes to get non-matching aggregates from metadata  https://review.opendev.org/67107410:58
openstackgerritShilpa Devharakar proposed openstack/nova master: Add a new request filter to isolate aggregates  https://review.opendev.org/67107510:58
openstackgerritShilpa Devharakar proposed openstack/nova master: Enable request filter isolate_aggregates  https://review.opendev.org/66795210:58
*** artom has quit IRC10:58
*** artom has joined #openstack-nova10:59
sean-k-mooneythe reproduce is basicaly boot a qemu l2 gues with kvm enabled in a l1 guest where the host is intel x8610:59
sean-k-mooneyit picks up support for vmx in the l1 guest but enable svm in the l2 guest10:59
donnydsean-k-mooney: is the numa thing working10:59
sean-k-mooneyi have seen other similar case for other instruction too10:59
*** jaosorior has joined #openstack-nova10:59
sean-k-mooneydonnyd: havent tried it yet11:00
donnydoh ok11:00
sean-k-mooneybut im 99% sure it will11:00
sean-k-mooneyyou have configured the flavor more or less how i do for dev11:00
*** artom has quit IRC11:00
*** artom has joined #openstack-nova11:01
*** artom has joined #openstack-nova11:01
*** ociuhandu has joined #openstack-nova11:02
stephenfinkashyap: Is the secure boot spec punted til U?11:02
stephenfin*to11:02
stephenfin(i.e. https://review.opendev.org/#/c/506720/)11:02
kashyapstephenfin: No, it's not.  It's the CPU APIs spec that's punted to U11:03
kashyapSB spec aiming for Train, still.  I'm optimistic like that11:03
*** nicolasbock has joined #openstack-nova11:05
openstackgerritBrin Zhang proposed openstack/nova master: Filter migrations by user_id/project_id  https://review.opendev.org/67424311:05
* aspiers is back11:10
*** brinzhang has quit IRC11:13
*** ociuhandu has quit IRC11:14
openstackgerritArtom Lifshitz proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888711:21
*** artom has joined #openstack-nova11:22
*** slaweq has quit IRC11:26
openstackgerritMerged openstack/nova master: Add functional test for AggregateMultiTenancyIsolation + migrate  https://review.opendev.org/57126511:27
*** cdent has quit IRC11:30
*** ricolin has quit IRC11:35
*** dpawlik has quit IRC11:39
*** tesseract has joined #openstack-nova11:42
*** aojea has quit IRC11:46
*** ratailor has quit IRC11:48
*** jawad_axd has joined #openstack-nova11:48
*** ociuhandu has joined #openstack-nova11:50
*** markvoelker has joined #openstack-nova11:51
*** cdent has joined #openstack-nova11:52
openstackgerritAdam Spiers proposed openstack/nova master: libvirt/host.py: remove unnecessary temporary variable  https://review.opendev.org/67934011:56
openstackgerritAdam Spiers proposed openstack/nova master: libvirt/host.py: remove unnecessary temporary variable  https://review.opendev.org/67934011:57
openstackgerritAdam Spiers proposed openstack/nova master: Indent fake libvirt host capabilities fixtures more nicely  https://review.opendev.org/67933911:57
*** ociuhandu has quit IRC11:57
*** lee1 has joined #openstack-nova11:59
*** lee1 is now known as lyarwood11:59
*** dave-mccowan has joined #openstack-nova12:02
*** tbachman has joined #openstack-nova12:06
*** dpawlik has joined #openstack-nova12:12
openstackgerritsean mooney proposed openstack/nova master: make config drives sticky bug 1835822  https://review.opendev.org/66973812:15
openstackbug 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
openstackgerritsean mooney proposed openstack/nova master: Whether to use config drive depends on instance.configdrive  https://review.opendev.org/48593012:15
sean-k-mooney... i forgot to update the commit message12:16
sean-k-mooneyactully no never mind i di12:16
sean-k-mooneydidnt need too12:16
*** dpawlik has quit IRC12:16
lyarwoodurgh, 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
sean-k-mooneyyep12:19
sean-k-mooneyso that does not work proably12:19
sean-k-mooneyyou have to use _host i think12:19
sean-k-mooneyits sometimes works proably and sometiems does not dependin on your other mocking12:20
lyarwoodhttps://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1595962.py - yeah I'm basically following the approach taken in here12:21
lyarwoodbut using https://github.com/openstack/nova/blob/744615559eb575bb0cf0a5ddc91acf255ebc3825/nova/tests/functional/libvirt/base.py#L30-L93 as the base class12:22
lyarwoodaaaaaaah that's already mocking it in setup12:22
lyarwoodmy bad12:22
sean-k-mooney this12:23
sean-k-mooney self.useFixture(fixtures.MonkeyPatch(12:23
sean-k-mooney           'nova.virt.libvirt.host.libvirt',12:23
sean-k-mooneyfakelibvirt))12:23
sean-k-mooneywhen you do12:24
sean-k-mooneyhttps://github.com/openstack/nova/blob/master/nova/tests/functional/regressions/test_bug_1595962.py#L4112:24
lyarwoodNo, lower12:24
lyarwood        # Mock the 'get_connection' function, as we're going to need to provide12:24
lyarwood        # custom capabilities for each test12:24
lyarwood        _p = mock.patch('nova.virt.libvirt.host.Host.get_connection')12:24
lyarwood        self.mock_conn = _p.start()12:24
lyarwood        self.addCleanup(_p.stop)12:24
*** dpawlik has joined #openstack-nova12:25
sean-k-mooney i dont see that in the file you linked but sounds like you have figured it out12:25
lyarwoodsean-k-mooney: sorry that's in the base class I'm using in the second link12:27
lyarwoodsean-k-mooney: and yeah it's working now12:27
*** ociuhandu has joined #openstack-nova12:27
*** derekh has quit IRC12:27
*** francoisp has joined #openstack-nova12:28
*** xek has joined #openstack-nova12:31
*** brault has joined #openstack-nova12:33
*** ociuhandu has quit IRC12:33
openstackgerritFrançois Palin proposed openstack/nova master: Add retry when attachment_delete or terminate_connection fails  https://review.opendev.org/66967412:41
*** larainema has quit IRC12:43
openstackgerritsean mooney proposed openstack/nova master: fix lxml compatibility issues  https://review.opendev.org/67896412:45
sean-k-mooneyprometheanfire: efried ^ now using self.assertXmlEqual12:45
aspierssean-k-mooney: \o/12:46
aspiersglad I invested some time into that function12:46
sean-k-mooneyi was doing it the old way12:46
sean-k-mooneythen erric remided me it existited12:46
sean-k-mooneyi dont know why testtoold decied that matches shoudl return nothing for sucess12:47
sean-k-mooneybut i hate that api so im glad we have a wrapper12:47
sean-k-mooneyprometheanfire: actully we need to invert the depends on relationship12:48
sean-k-mooneyyour upperconstaits bump needs to depend on the nova change12:48
openstackgerritsean mooney proposed openstack/nova master: fix lxml compatibility issues  https://review.opendev.org/67896412:49
efriedsean-k-mooney: +2, thank you12:49
efriedrace condition12:49
sean-k-mooneyefried: hehe care to +2 again12:49
efrieddone12:49
sean-k-mooneythanks12:49
sean-k-mooneyprometheanfire: do you mind if i update your patch to depend on mine12:50
efriedsince you were updating the commit message, you could have s/XMLMatcher/assertXmlEqaual/12:50
efriedbut technically still true, so meh12:50
sean-k-mooneyoh i can if you want12:50
efriednah12:50
sean-k-mooneyok12:50
*** mriedem has joined #openstack-nova12:51
luyaodansmith:  Are you around?12:51
*** xek has quit IRC12:52
mriedemluyao: he's out today12:56
luyaomriedem: okey, thanks for telling me this12:58
sean-k-mooneymriedem: i finally got around to updating https://review.opendev.org/#/c/669738/. the functional tests are now in there own module as you asked. sorry for the delay.12:58
gibimriedem: hi! thanks for the reviews on the nova-grenade-multinode patches. It seems that GRENADE_PHASE var is not defined even in a grenade job. but GRENADE_NEW_BRANCH is, so I'm pushing a new patch set soon12:59
*** derekh has joined #openstack-nova12:59
sean-k-mooneyoh i added a shelve/unshelve test too12:59
mriedemgibi: hmm, when i looked at the job results yesterday i thought i looked for that, but can't remember the details now12:59
gibimriedem: here is a run with env dumped https://b8c08d7c14d41636ca2c-db936121e3703832159c63a0fb95ab4b.ssl.cf5.rackcdn.com/679210/5/check/nova-grenade-multinode/2489dbd/job-output.txt13:00
openstackgerritBalazs Gibizer proposed openstack/nova master: Add cold migrate and resize to nova-grenade-multinode  https://review.opendev.org/67921013:00
mriedemgibi: GRENADE_(NEW|OLD)_BRANCH should work yeah, we've used that before https://github.com/openstack/nova/blob/stable/pike/nova/tests/live_migration/hooks/run_tests.sh#L5213:00
luyaomriedem: Welcome to review this patch series if you would like, I'll appreciate that. Current design is NOT virt driver specific. https://review.opendev.org/#/q/topic:bp/virtual-persistent-memory+(status:open)13:00
mriedemgibi: ok you're going to drop that env dump yeah?13:02
mriedemthat's just WIP?13:02
gibimriedem: yes, I just noticed it13:02
gibimriedem: new PS is coming13:02
*** jawad_axd has quit IRC13:02
mriedemluyao: i'll be honest with you, that series is likely not something i'm going to be focusing on in the next 2 weeks to feature freeze, i'm already committed to reviewing some other series that have been around longer13:02
openstackgerritBalazs Gibizer proposed openstack/nova master: Add cold migrate and resize to nova-grenade-multinode  https://review.opendev.org/67921013:03
mriedemsean-k-mooney: btw, totally unrelated, i realize our docs for AggregateInstanceExtraSpecsFilter kind of suck https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html?highlight=aggregate_instance_extra_specs#aggregateinstanceextraspecsfilter13:03
mriedemmeaning, what the hell is aggregate_instance_extra_specs supposed to mean to a user trying to configure flavors or aggregates for that filter?13:04
mriedemlooking at the code it doesn't even really make sense to me13:04
sean-k-mooneyi know i hate that filter and tried to replaces it13:04
mriedemshould at least have an example of how to use it13:04
sean-k-mooneyyes13:05
mriedemit came up while triaging this bug https://bugs.launchpad.net/nova/+bug/184193213:05
openstackLaunchpad bug 1841932 in OpenStack Compute (nova) "hide_hypervisor_id extra_specs in nova flavor cannot pass AggregateInstanceExtraSpecsFilter" [Undecided,Incomplete]13:05
sean-k-mooneyi can proably add one if you want13:05
luyaomriedem: Yes, I fully understand you, but any comments are welcome, even small and short. :)13:05
sean-k-mooneymriedem: it bassicaly asserts that that every extraspec that is in the fiter (optionally that starts with aggregate_instance_extra_specs:) is set in at least 1 aggreate the current host is a member of13:07
openstackgerritAdam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption  https://review.opendev.org/66442013:07
aspiersalex_xu, stephenfin: Your feedback addressed ^^^13:08
sean-k-mooneymriedem: looking at the bug report its working as intended13:09
sean-k-mooneywell actuly13:10
*** edleafe has joined #openstack-nova13:10
sean-k-mooneyit might not handel un namepaced extra specs properly ill check13:11
*** nweinber has joined #openstack-nova13:11
*** cdent has quit IRC13:11
sean-k-mooneymriedem: no its working as indended13:12
sean-k-mooneyits not working as they want in https://bugs.launchpad.net/nova/+bug/184193213:12
openstackLaunchpad bug 1841932 in OpenStack Compute (nova) "hide_hypervisor_id extra_specs in nova flavor cannot pass AggregateInstanceExtraSpecsFilter" [Undecided,Incomplete]13:12
sean-k-mooneybut that is worrking the way that filter has always worked13:12
mriedemsean-k-mooney: i've reported a docs bug if you you're keen https://bugs.launchpad.net/nova/+bug/184206113:12
openstackLaunchpad bug 1842061 in OpenStack Compute (nova) "Compute schedulers in nova - AggregateInstanceExtraSpecsFilter docs are not clear" [Undecided,New]13:12
sean-k-mooneyill see if i can add it to my list. but we create https://github.com/openstack/nfv-filters/blob/master/nfv_filters/nova/scheduler/filters/aggregate_instance_type_filter.py to replace it for several reason. also with docs https://github.com/openstack/nfv-filters/blob/master/doc/source/scheduler_filters/aggregate-instance-type-filter.rst13:15
aspiersstephenfin: I also split off those fixups13:15
*** KeithMnemonic has joined #openstack-nova13:21
*** mkrai has quit IRC13:22
efriedaspiers: why is textwrap.dedent necessary https://review.opendev.org/#/c/679339/3/nova/tests/unit/virt/libvirt/fake_libvirt_data.py ?13:29
openstackgerritMerged openstack/nova-specs master: Create specs directory for Ussuri  https://review.opendev.org/67936413:29
sean-k-mooneyefried: its not13:29
efriedThis is xml, not python - it doesn't care about whitespace internally, does it?13:29
sean-k-mooneyactully it does13:29
aspiersefried: it's necessary to keep stephenfin happy, and I kinda like it too13:29
sean-k-mooneybut we are using xml parsing libs and we dont care13:29
aspiersI could live without, but he suggested it and I like keeping things as readable as possible13:30
aspierssean-k-mooney: it's about readability not parsing13:30
efriedI'm not asking why you're indenting the lines with the xml on them. Visually prettier. I'm asking why textwrap.dedent is necessary.13:30
stephenfinhmm, yeah, we could forget about the textwrap piece, actually. It was the indentation of the XML itself that I cared about13:30
aspiersOh right13:30
sean-k-mooneyaspiers: yes but we dont read the xml we read the code13:30
stephenfinefried: I suggested that out of habit13:30
aspierssean-k-mooney: strongly disagree13:30
stephenfinYou're right, it's not needed in this case13:30
sean-k-mooneyand that makes the code mroe readable13:30
aspiersI can drop dedent if that's the consensus13:30
sean-k-mooneye.g. leaving it out13:30
stephenfinI'm happy with it. I'm happy without it. Just so long as I get my block indentation :)13:31
*** eharney has joined #openstack-nova13:31
sean-k-mooneyaspiers: i read the xml all the time just not that xml its alway after its been parsed or pretty printed13:31
*** edmondsw_ has joined #openstack-nova13:31
aspierssean-k-mooney: OK you meant "we don't read the XML as it is at *run-time*, we read it in the source"? I would mostly agree with that13:32
efriedaspiers: it's just entirely unnecessary. And in fact if you folded the result into the surrounding xml doc, it would look weird, because it's completely left-justifying13:32
sean-k-mooneyaspiers: yes13:32
aspiersefried: what's entirely unnecessary, the dedent or the whole fixup?13:33
sean-k-mooneyaspiers: if im reading it at runtime its in a debugger or a traceback13:33
aspiersanother classic bikeshed topic :)13:33
sean-k-mooneyobviously13:33
aspiersIs there a mandatory minimum of one per day? ;-p13:33
*** xek has joined #openstack-nova13:33
efriedwell, both aspiers, and if it weren't for the fact that all these lines already belong to two very recent commits, I would -1 it for the sake of git history. I'm tempted to do that anyway.13:34
efriedbut I have an actual problem with the dedent.13:34
aspiersefried: I only submitted this because stephenfin asked for it - please negotiate with him not me :)13:34
efriedit's unnecessary processing (test only, but still) and the result is actually technically too dedented.13:34
aspiersI really don't mind what you guys agree on13:34
efriedI hear ya.13:34
aspiersthanks :)13:35
stephenfinefried: straight up indentation fixes aren't an issue, tbh - git blame -w13:36
* efried negotiates on the patch13:36
sean-k-mooneymriedem: i may have ranted a bit but hopefully that helps. https://bugs.launchpad.net/nova/+bug/184206113:37
openstackLaunchpad bug 1842061 in OpenStack Compute (nova) "Compute schedulers in nova - AggregateInstanceExtraSpecsFilter docs are not clear" [Undecided,New]13:37
efriedstephenfin: where's the -w option in pycharm?13:37
sean-k-mooneyill try and come up with a doc friendly way to say that13:37
stephenfinefried: Google tells me it's on by default? https://stackoverflow.com/questions/37868225/ignore-whitespace-identation-in-annotate-mode-with-intelliji-and-git13:37
sean-k-mooneyefried: you can get the history of the lines you have slected and go back though each commit13:38
stephenfinAlso, am I the only sucker still working in Vim?13:38
sean-k-mooneyyes13:38
efriedsean-k-mooney: Yes, of course I can, but is that extra work worth it? IMO it is not.13:38
efriedespecially for this13:38
efriedstephenfin: I believe cdent is with you.13:39
sean-k-mooneyefried: sooy i was not following the converstion13:39
sean-k-mooneycan you do git blame -w with pycharm13:39
sean-k-mooneyand yes if you want too13:39
efriedwell, that's interesting, stephenfin you're right about -w being on by default. That takes some of the sting out.13:40
sean-k-mooneyhow did i miss a full sentence...13:40
efriedI'll just abstain.13:40
stephenfin\o/13:40
stephenfinthat's a sane default13:41
*** edmondsw_ has quit IRC13:41
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add useful error log when _determine_version_cap raises DBNotAllowed  https://review.opendev.org/67944913:41
* stephenfin wonders why git doesn't allow this mode to be configured13:41
sean-k-mooneylitally ment to type "all i saw was" before "can you do..."13:41
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add useful error log when _determine_version_cap raises DBNotAllowed  https://review.opendev.org/67944913:41
sean-k-mooneystephenfin: proably because white space in lanagues like python matters13:41
sean-k-mooneyso changing indention could change behavior13:42
sean-k-mooneyor no one added it13:42
stephenfinsean-k-mooney: Right, but make it toggleable13:42
stephenfinMaybe 'git blame -W' to override whatever and re-enable it13:42
stephenfinHmm, that would be a decent patch13:42
stephenfinPossible weekend project there13:42
sean-k-mooneystephenfin: as long as you dont change the default behavior13:42
* efried moves on to more important reviews13:43
stephenfinoh yeah, of cours13:43
stephenfin*course13:43
sean-k-mooneyefried: speaking of which13:43
sean-k-mooneyefried: care to take a look at |https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/image-metadata-prefiltering13:43
sean-k-mooneythere are more more important reviews but im hoping those merge soon13:44
efriedsean-k-mooney: been on the list, but got some stuff in front of it.13:44
sean-k-mooneycool13:44
sean-k-mooneystephenfin: by they way i tested and fixed https://review.opendev.org/#/c/675776/13:45
stephenfinthe revert?13:45
sean-k-mooneystephenfin: we do need that fix to avoid out issue with libvirt13:45
sean-k-mooneyyes13:45
stephenfinyeah13:45
sean-k-mooneyi guess we are still waiting for the melonox ci to run on it too13:47
mdboothmriedem dansmith: I don't think I'm going to get better confirmation of the fix in https://review.opendev.org/#/c/677736/ . It *has* run on a real system, though, and didn't break it.13:49
sean-k-mooneyartom:  https://review.opendev.org/#/c/678887/ failed because its using qemu again13:49
openstackgerritAdam Spiers proposed openstack/nova master: Add <launchSecurity> and <driver iommu='on' /> to config.py  https://review.opendev.org/63631813:50
aspiersalex_xu, stephenfin, efried: need your take on this debate https://review.opendev.org/#/c/636318/49/nova/virt/libvirt/config.py@270113:51
mriedemmdbooth: dan is out today,13:52
mriedembut are you going to address https://review.opendev.org/#/c/677736/3/nova/virt/libvirt/host.py@a236 ?13:52
mriedemotherwise i'll +2 it13:52
aspiersstephenfin, efried: if the question isn't easy to parse from those comments, I can help explain here13:53
efriedaspiers: Give me 10 mins13:53
aspiersefried: np13:53
sean-k-mooneymdbooth: you know the py37 hang i was seeing before when we tryied to boot vms could also be fixed by https://review.opendev.org/#/c/677736/313:53
sean-k-mooneyi had assumed when i saw it it was due to eventlest and python37 not playing nicely but that looks like it is more likely the actual cause13:54
* aspiers goes for late lunch13:55
*** cdent has joined #openstack-nova13:55
*** JamesBenson has joined #openstack-nova13:57
mdboothsean-k-mooney: Why would the behaviour be different on py37, though?13:57
mdboothBug call time13:58
*** ociuhandu has joined #openstack-nova14:01
sean-k-mooneywell i was febuary, eventlet did not offically test/support py37 and we have see issue with it on python3 in the pasted that looked like it14:02
sean-k-mooneybut honestly it was just a guess and it happend on python 3 but python 2.714:02
*** ociuhandu has quit IRC14:02
*** ratailor has joined #openstack-nova14:05
*** mloza has joined #openstack-nova14:07
efriedaspiers: I don't have the bandwidth to dig into this all the way right now, so the following is a bit from the hip:14:09
efriedIf it's possible for Thing A (via extra spec, image meta, config, etc) to ask for a switch to be flipped on, and Thing B (likewise) for it to be off, that's a conflict, and should be an error.14:09
efriedWe have a place where we validate flavor+image compatibility for things like this: nova.compute.api.API._validate_flavor_image14:09
efriedAs of yet, I think all of these ^ checks are just checking whether Thing A is on in flavor and off in image meta, kind of thing.14:09
efriedSo this might be the first time we are cross-checking Thing A and Thing B for compatibility.14:09
efriedBut that's fine - I believe that was one of the long-term goals of that infrastructure. stephenfin might be able to say more there.14:09
efriedNow, that's at the API layer. Once you get down to the compute, you can start vetting against things like config or whatever. Hopefully any conflicts there would have already been filtered out based on placement; but if not, late fail the spawn.14:09
efriedSo if all of that were in place, having a single function that checks sev *and anything else relevant* and either sets iommu on or raises an exception: sounds like the right plan.14:09
*** gbarros has joined #openstack-nova14:10
*** dpawlik has quit IRC14:11
mriedemartom: so https://review.opendev.org/#/c/678887/18 we're just waiting for another run after your rebase right?14:13
*** dougsz has joined #openstack-nova14:15
* mriedem dials into bluejeans14:16
*** dougsz has quit IRC14:17
aspiersefried: thanks, I think that's enough justification for me to start switching the driver iommu flags so they all get enabled from a single place14:21
aspiersefried: right now the conflict is only a theoretical possibility14:22
*** luksky has quit IRC14:22
aspiersefried: so we may never need to worry about it, but yeah it makes sense what you say about there being enough places for us to proactively catch and handle any such conflicts if they did arise in future14:22
*** itlinux-away is now known as itlinux14:24
aspiersefried: the only risk I can't see us easily avoiding is someone adding code which explicitly disables iommu but forgets to prevent the SEV code from overriding that. Moving the SEV iommu code away from each individual chunk responsible for building driver configuration means that it's "hidden" out of sight from that code, and therefore easily missed14:25
aspiersefried: but that seems like a small risk, and in return we probably get much cleaner code enabling iommu for SEV14:25
aspiersalex_xu: ^^^14:25
*** shilpasd has left #openstack-nova14:27
*** shilpasd has joined #openstack-nova14:29
mriedemefried: dustinc: couple of questions in the ironic get_node sdk change https://review.opendev.org/#/c/642899/44/nova/virt/ironic/driver.py14:29
dansmithartom: I think the version checking in here is wrong and that you'll fail spectacularly if you have an rpc version pin and have to roll back a migration: https://review.opendev.org/#/c/63460614:33
*** ratailor has quit IRC14:34
*** ociuhandu has joined #openstack-nova14:34
efriedmriedem: responded14:37
*** mkrai has joined #openstack-nova14:39
*** ociuhandu has quit IRC14:39
mriedemsean-k-mooney: this is all host agnostic right? https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L212-L22814:44
mriedemsean-k-mooney: meaning, if an instance has non-sriov pci requests, we're going to hit that same code for each alternate host during live migration and just fail on all of them since that validation doesn't involve the host14:44
mriedemseems that should be pulled up higher in the conductor task before we even ask the scheduler for dest hosts14:44
mriedemlike https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L8514:44
sean-k-mooneywe check for non neutron pci device requets in the conductor already14:45
*** Sundar has joined #openstack-nova14:45
sean-k-mooneymriedem: here https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L216-L22114:46
sean-k-mooneymriedem: dan wanted us to check again in the compute node to be extra safe14:46
sean-k-mooneymriedem: so if you have pci deivce requested by the flavor alias we never get to artoms code14:47
mriedemsean-k-mooney: i think you're missing my point14:48
mriedemlet's say i have conductor configured to retry up to 100 hosts for live mgiration and the scheduler gives me back 100 possible dests,14:49
mriedemif the instance has flavor-defined pci requests, i'm going to hit that same block of validation code 100 times14:49
mriedemwith the exact same results14:49
mriedemalso, did we check with any of the non-libvirt driver maintainers that flavor-defined pci requests aren't supported in their driver for live migration?14:50
mriedemi'm assuming they don't, hyper-v would be the only one remotely close probably, but wondering if anyone asked or just assumed libvirt is the only driver in nova14:50
stephenfinmriedem: No, we don't14:51
stephenfinWe assume libvirt is the only driver for a lot of these things14:51
*** ivve has quit IRC14:51
stephenfinWe've also done a bad job of documenting the features and config options that are libvirt only, fwiw14:52
mriedemhttps://bugs.launchpad.net/nova/+bug/184208714:53
openstackLaunchpad bug 1842087 in OpenStack Compute (nova) "_check_can_migrate_pci in the LiveMigrationTask has host agnostic validation that is redundant/expensive" [Low,Triaged]14:53
stephenfinmriedem: fwiw, I've suggested aspiers adds similar checks for the SEV feature (based on his docs, you can't live migrate those instances)14:54
mriedemefried: ok so https://review.opendev.org/#/c/642899/44 - looks like quite a few unrelated things in there, but i'm just grumbling i think b/c it seems to be that kind of morning14:57
mriedemefried: can you sniff my latest comments and if i'm not crazy then i'll +W14:57
shilpasdmriedem: Hi, i am in the process of understanding fixture client created at novaclient https://opendev.org/openstack/python-novaclient/src/branch/master/novaclient/tests/unit/fixture_data/client.py#L2114:58
shilpasdcan you help me how this COMPUTE_URL is derived (endpoint >> http://<host_ip>/compute/v2.1)14:59
mriedemshilpasd: i'm not sure what you're asking, but i guess i'd follow this code: https://opendev.org/openstack/python-novaclient/src/branch/master/novaclient/tests/unit/fixture_data/client.py#L39-L4015:02
artomsean-k-mooney, wat. OK, I removed some ansible lines because the devstack patch was doing it, looks like I'll have to re-add them15:03
artommriedem, what I said to sean-k-mooney ^^15:03
sean-k-mooneymriedem: i did think we check but i cant recal specificlly15:03
efriedmriedem: Yeah, it's not terribly surprising that15:04
shilpasdmriedem:COMPUTE_URL is used for creating endpoint in fixture, and if you check endpoint for compute is 'http://<host_ip>/compute/v2.1'15:04
sean-k-mooneye.g. with other dirvers15:04
efriedmriedem: ...that a six-part series with two authors and four dozen patch sets got a bit of intra-series bleed :(15:04
efriedmriedem: If you want we can get dustinc to ack those comments.15:04
shilpasdmriedem: so my question, how what's the relation COMPUTE_URL as endpoint here in Fixture15:06
artomdansmith, ah, I think I see what you mean. I do a 'if src_supports_numa_live_migration' check here: https://review.opendev.org/#/c/634606/62/nova/compute/manager.py@651115:07
*** ociuhandu has joined #openstack-nova15:07
dansmithartom: ...but...15:08
artomdansmith, so I should move that if into post_claim_migrate_data, right?15:09
artomTo make sure the new field got through RPC version pinning to the dest15:09
dansmiththe field will make it through regardless of the rpc pin15:09
dansmiththey're unrelated15:09
dansmithentirely15:09
dansmiththat's my point15:09
artomOh.15:10
dansmithyou can't use the presence of a field in an object to know what the version pin is15:10
artomWait, so how does the conductor know when to backlevel an object?15:10
dansmithartom: conductor backlevels an object when asked. when an object arrives at a compute node and that compute node sees it's too new, and asks conductor to backlevel it to a specific version (i.e. its latest for that object)15:11
artomdansmith, aha, so I had it completely wrong. OK, need to rethink and update then.15:12
dansmithbut compute->compute there is no conductor involvement for the actual rpc call, and the version pin is not related to object versions15:12
dansmithartom: yep15:12
*** shilpasd has quit IRC15:15
mriedemartom: left my comments in the claims patch as well https://review.opendev.org/#/c/635669/4715:16
*** mlavalle has joined #openstack-nova15:16
artommriedem, ack, much thanks.15:16
* artom wishes he had more sleep under his belt to think clearer.15:16
artomAnd we want to see https://review.opendev.org/#/c/678887/ passing before we start the +W Train (zing!) going, right?15:17
mriedemspeaking of that,15:18
artom(OK, train may have been optimistic - snail?)15:18
mriedemwhy did you pull it away from the nova series?15:18
mriedemit's a nova change that depends on a devstack change that depends on a nova change...but why?15:18
artomIt became a PITA to work on both at the same time in the same stack?15:18
openstackgerritAdam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption  https://review.opendev.org/66442015:18
artomSpecifically, it needed a rebase, but no the stack below it15:18
artom*not15:19
mriedemoh yeah ok15:19
dansmithartom: I would really like to see that working, unless there's some fundamental reason it can't15:19
dansmithartom: it looks like it's still not even able to start an instance though right?15:19
mriedemas for the +W train yeah ^ what dan said15:19
artomsean-k-mooney would know more about the fundamental reasons, so if thinks it can work, let's keep trying.15:19
artomI removed some lines I thought were no longer needed (beause of the devstack nuke) when I rebased, looks like I'll have to put them back int15:20
artomAnd put it on top again, why not, I'm respinning the series anyways15:20
mriedemyou don't have to put it back on top,15:21
mriedembut can if you want15:22
sean-k-mooneyartom: the temp job? we jsut don have the right setting in the config.15:22
sean-k-mooneydansmith: mriedem do ye just want to see a gate job using numa migrtaion15:23
sean-k-mooneyor does it have to be that gate job15:23
artom*snerk* let's try and hack something in grenade!15:23
dansmithwell, especially given the upgrade issues, it'd also be nice to run the grenade LM job against this, even though it probably wouldn't have covered the situation I found earlier15:24
sean-k-mooneybecause at this point if adding the options artom remvoed does not work it would be simpler form me to just make my nfv job work15:24
dansmithsean-k-mooney: if it runs some LMs then that's better than noting15:24
dansmithI'm also going to be tempted to hack the job to pin the rpc version at 5.1 to check his work15:24
sean-k-mooneydansmith: well i have my https://review.opendev.org/#/c/652197/ nfv job15:24
dansmithbecause breaking LM across versions is icky15:24
sean-k-mooneythat i was going to rework15:24
sean-k-mooneyso i was going to stop using fedora and jsut use ubunut with nested vert15:25
sean-k-mooneythe multinode version runs live migratio tests15:25
dansmiththe closer it is to a standard job, the easier it will be to verify, but anything is better than nothing15:26
sean-k-mooneysure15:26
sean-k-mooneywhcih is why im swapping ot ubuntu15:26
sean-k-mooneybut we also need to move the nova-migration job to non legacy at some point15:26
sean-k-mooneyanyway i was planning to finish up early today given how late i have been working lately15:27
dansmithunderstandable, and holiday weekend here for lots of people15:28
*** luksky has joined #openstack-nova15:28
sean-k-mooneydansmith: donnyd  provide a mulit numa flavor we can use on FN and we can also use vexhost. so ill create a job on monday15:28
sean-k-mooneyassuming ye dont get the other one working15:29
openstackgerritArtom Lifshitz proposed openstack/nova master: DNM: Run LM integration tests with NUMA flavor  https://review.opendev.org/67888715:29
*** eandersson has joined #openstack-nova15:30
*** jaosorior has quit IRC15:31
*** gyee has joined #openstack-nova15:33
*** macz has joined #openstack-nova15:33
openstackgerritMatt Riedemann proposed openstack/nova master: Rename Claims resources to compute_node  https://review.opendev.org/67947015:34
mriedemartom: i thought this was funny https://review.opendev.org/#/c/635669/47/nova/compute/resource_tracker.py@39915:37
mriedem"because the fake objects in the tests aren't correct, we can't do anything about the actual code looking weird"15:37
*** jaosorior has joined #openstack-nova15:37
artommriedem, heh - so, either we create a migration for the test that already has the accepted status set, which is close to real like but then...15:38
artomWell15:38
artomNo, it'd work15:38
artomAssert it's still accepted15:38
artomThanks!15:38
artom*real life15:38
mriedemyeah....15:38
mriedemi think you're overthinking all of this :)15:38
artomLike a lot of things in life :(15:39
*** jaosorior has quit IRC15:42
*** rmk has joined #openstack-nova15:50
*** brault has quit IRC15:51
*** tesseract has quit IRC15:52
mriedemefried: as for the ironic + sdk series, i'll wait for your ack on https://review.opendev.org/#/c/656027/ since you brought up CERN performance anxiety15:59
*** ivve has joined #openstack-nova16:03
*** brault has joined #openstack-nova16:03
*** jangutter has quit IRC16:04
*** nweinber has quit IRC16:04
*** nweinber has joined #openstack-nova16:04
*** ccamacho has quit IRC16:05
*** brault has quit IRC16:07
openstackgerritMerged openstack/nova master: Follow up for specifying az to unshelve  https://review.opendev.org/67916416:12
openstackgerritMatt Riedemann proposed openstack/nova master: Handle VirtDriverNotReady in _cleanup_running_deleted_instances  https://review.opendev.org/67947316:13
*** eharney has quit IRC16:13
artomdansmith, sanity check with you before I go off haxing - if in https://review.opendev.org/#/c/634606/62/nova/compute/manager.py@6481 I check for the presense of the migration kwarg, which is new in RPC 5.2 and is not None from the conductor, is that a good way to make sure we're not pinned to < 5.2?16:14
dustincmriedem, efried: thanks for help and reviews, checking out comments now16:14
dustinc(on ironic sdk series)16:14
mriedemartom: wouldn't https://review.opendev.org/#/c/634606/62/nova/compute/manager.py@6511 already be False in that case?16:15
dansmithartom: isn't that the destination not the source?16:16
* mriedem goes to lunch16:16
*** mriedem is now known as mriedem_afk16:16
artomDoh, right16:16
dansmithartom: I quickly looked to see if you had an easy indication on the source like that and didn't see one16:16
dansmithbut yeah, that's a little obscure, but as long as it's commented you should be able to use something like that16:17
dansmithbut, not sure there is one for the source :/16:17
artomRight, it's set by the libvirt driver during the source check: https://review.opendev.org/#/c/634606/62/nova/virt/libvirt/driver.py16:18
artomThat's the bit that has to be conditional on RPC 5.216:18
artomdansmith, feels ugly, but add a new sentinel param to check_can_live_migrate_source when it's called from check_can_live_migrate_destination?16:19
artomOr just have check_can_live_migrate_source manually check the RPC version16:19
dansmithit's always called from there no?16:20
artomYou mean _source from _destination? Yeah, I was saying it from context16:21
artom*for16:21
*** xek has quit IRC16:21
dansmithit's always called there, so you mean when it's called from a new-enough node to pass that flag, yeah?16:22
*** markvoelker has quit IRC16:22
artomdansmith, yeah. If the node has the new code, it'll set the sentinel, and if the RPC is new enough, it'll actually reach the source16:22
dansmithyeah, I don16:23
dansmithI don't love it, but...16:23
*** itlinux is now known as itlinux-away16:24
artomOr.16:25
artomWhen the destination gets the migrate_data back from the source, pop the src_supports_numa_live_migration sentinel if the destination did not receive a migration from the conductor16:26
dansmiththe migration from conductor doesn't tell the whole story, so I don't think it's legit to use that as your sentinel16:28
*** markvoelker has joined #openstack-nova16:28
dansmithconductor's rpc version may be pinned differently16:28
artomDifferently.... from the computes?16:29
artomI thought it was a deployment-wide thing16:29
dansmithdifferently from anything, sure16:29
dansmithconfig is per-node16:30
*** mrjk has quit IRC16:30
dansmithin general you'd kinda assume they're the same, but especially when doing the slide-puzzle migration approch,16:30
dansmithyou might need to pin some computes differently to be able to communicate while things are old16:30
artomHrmm, so in addition to this whole thing, I need to check that the migration from the conductor is not None16:31
dansmithregardless of the pinning of the computes you mean?16:31
artomYeah16:31
dansmithif there's something you're doing that assumes conductor passed an argument from compute 5.2, then for sure,16:31
dansmithbecause if it's pinned at 5.1 or "auto" and there are older computes present, then..yeah16:32
dansmithalso,16:32
dansmithif everyone is set to "auto", then the version they think is legit right now depends on when they started, or were restarted during the upgrade16:32
dansmithlike, as soon as the last compute restarts, its auto pin is set to current, but everyone else hasn't gotten their service restart kick yet, etc16:32
dansmithso assuming they're all atomically set identically is not legit, IMHO16:33
artomSo I think migration=None ends up creating a migration object in _move_claim, which is not what we want16:33
artomOK, I think I got a better handle on this, thanks!16:34
dansmithack, maybe comment that on the patch in the proper place for tracking16:34
*** dtantsur is now known as dtantsur|afk16:36
openstackgerritAdam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption  https://review.opendev.org/66661616:36
aspiersefried: need your input on this (sorry) https://review.opendev.org/#/c/666616/38/nova/conf/libvirt.py@84516:38
efriedaspiers: more than earlier?16:38
*** igordc has joined #openstack-nova16:39
aspiersefried: uhh, different topic I think16:39
efriedokay, hit me16:39
aspiersURL above ^^^16:39
efriedoh, sorry, stand by16:39
aspiersalex_xu and stephenfin want the default for num_memory_encrypted_guests changed from None to 016:40
aspiersI'm not convinced it's the best idea, but I guess I'd be OK with it. However I wouldn't be OK with ditching None altogether.16:40
openstackgerritAdam Spiers proposed openstack/nova master: Indent fake libvirt host capabilities fixtures more nicely  https://review.opendev.org/67933916:43
sean-k-mooneyaspiers: you could replace non with -1 to signify unlimited16:43
aspierssean-k-mooney: that's already mentioned in my comment16:43
sean-k-mooneyi didnt click16:44
aspiersefried, stephenfin: dedent removed ^^^16:44
sean-k-mooneyi just assumed that that would be reason for not using none16:44
aspiersNo, None currently means "try to auto-detect, but if you can't, unlimited"16:44
sean-k-mooneyah right yes16:44
aspiersAnother reasonable default would be "try to auto-detect, but if you can't, 0"16:45
aspiersSo that operators have to explicitly request unlimited16:45
sean-k-mooneyya16:45
*** Sundar has left #openstack-nova16:45
aspiersI'd be OK with that too, although it's getting a bit complicated16:45
sean-k-mooneywell that would be cloaset to that you want16:46
aspiersNot necessarily16:46
aspiersRead the comment ;-)16:46
*** itlinux-away is now known as itlinux16:46
sean-k-mooneyim more or less done for the day. just waiting for food to be delivers so im trying to avoid starting any more reviews or work.16:47
sean-k-mooneyanyway i was fine with what you had orginally proposed16:47
aspiersthanks16:47
openstackgerritAdam Spiers proposed openstack/nova master: Indent fake libvirt host capabilities fixtures more nicely  https://review.opendev.org/67933916:48
aspiersDon't you just love it when the final tempest-slow-py3 in the gate fails with some unrelated issue and means you have to re-run your whole damn CI twice16:49
aspiers*sigh*16:49
sean-k-mooneytwice your lucky its not more :)16:49
aspiershaha maybe it will be16:49
aspierstempest.lib.exceptions.SSHTimeout: Connection to the 172.24.5.222 via SSH timed out.16:49
aspiersAh that good old favourite16:50
aspiersstephenfin: you still around?16:51
aspiersIf I address two nits in https://review.opendev.org/#/c/638680/ I guess I lose the +2 W+1, right?16:52
sean-k-mooneyyou will and i think he is gong for the weekend16:52
sean-k-mooneyadd a followup patch16:52
efriedaspiers: responded16:52
*** bnemec is now known as beekneemech16:53
*** eharney has joined #openstack-nova16:58
*** derekh has quit IRC17:00
*** mkrai has quit IRC17:08
*** cdent has quit IRC17:15
*** ralonsoh has quit IRC17:19
*** ricolin has joined #openstack-nova17:28
aspiersefried: thanks17:38
aspierssean-k-mooney: ok17:38
*** cfriesen has joined #openstack-nova17:42
*** ociuhandu has quit IRC17:51
*** ociuhandu has joined #openstack-nova17:57
*** ricolin has quit IRC18:00
*** ociuhandu has quit IRC18:02
*** itlinux is now known as itlinux-away18:10
*** psachin has quit IRC18:12
artomdansmith, so actually, I could at least test pinning to 5.1 in the func tests18:14
artomJust self.flags(compute='5.1', group='upgrade_levels')18:14
dansmithartom: yeah, you should be able to test old conductor (call), new compute(s), mixed computes, etc I think18:16
artomNot sure about mixed because CONF is global18:16
dansmithin unit tests for sure,18:17
dansmithbut the compute manager should grab its rpcapi object on start, IIRC, which means you can set it, start one, set it again, start another, etc18:17
dansmithI think it doesn't work for auto, but for manual pins I think you can18:17
*** adriant has quit IRC18:17
*** adriant has joined #openstack-nova18:18
artomOh, fun.18:19
artomAfter lunch though, I put it off long enough18:19
artomI'll push what I have I guess18:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Introduce live_migration_claim()  https://review.opendev.org/63566918:20
openstackgerritArtom Lifshitz proposed openstack/nova master: New objects for NUMA live migration  https://review.opendev.org/63482718:20
openstackgerritArtom Lifshitz proposed openstack/nova master: LM: Use Claims to update numa-related XML on the source  https://review.opendev.org/63522918:20
openstackgerritArtom Lifshitz proposed openstack/nova master: NUMA live migration support  https://review.opendev.org/63460618:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration  https://review.opendev.org/64002118:20
openstackgerritArtom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration  https://review.opendev.org/67259518:20
*** gbarros has quit IRC18:26
*** itlinux-away is now known as itlinux18:35
*** bbowen has quit IRC18:36
mriedem_afksean-k-mooney: you have a pep8 failure here https://review.opendev.org/#/c/669738/18:38
*** bbowen has joined #openstack-nova18:39
*** mrjk has joined #openstack-nova18:47
*** bbowen has quit IRC18:51
*** bbowen has joined #openstack-nova18:52
*** tbachman has quit IRC18:57
*** dave-mccowan has quit IRC19:02
*** itlinux has quit IRC19:04
*** xek has joined #openstack-nova19:11
*** KeithMnemonic has quit IRC19:22
*** mriedem_afk is now known as mriedem19:24
mriedemefried: i think these are ready for gibi's series https://review.opendev.org/#/q/topic:nova-grenade-multinode+(status:open+OR+status:merged)19:24
*** slaweq has joined #openstack-nova19:32
openstackgerritNathan Kinder proposed openstack/nova master: Allow TLS ciphers to be configurable for console proxies  https://review.opendev.org/67950219:33
openstackgerritMerged openstack/nova master: Introduces SDK to IronicDriver and uses for node.get  https://review.opendev.org/64289919:42
openstackgerritMerged openstack/nova master: libvirt/host.py: remove unnecessary temporary variable  https://review.opendev.org/67934019:42
openstackgerritMerged openstack/nova master: fix lxml compatibility issues  https://review.opendev.org/67896419:42
mriedemneed someone to backport https://review.opendev.org/#/c/678964/ to stein.19:44
*** bbowen has quit IRC19:45
*** nweinber has quit IRC19:48
*** bbowen has joined #openstack-nova19:53
*** xek has quit IRC19:57
*** nweinber has joined #openstack-nova20:12
*** adriant has quit IRC20:17
*** nweinber has quit IRC20:17
*** adriant has joined #openstack-nova20:18
*** tbachman has joined #openstack-nova20:22
*** ociuhandu has joined #openstack-nova20:25
*** ociuhandu has quit IRC20:30
*** bbowen has quit IRC20:32
*** bbowen has joined #openstack-nova20:34
*** markvoelker has quit IRC20:51
*** bbowen has quit IRC20:56
*** eharney has quit IRC21:18
*** JamesBenson has quit IRC21:19
openstackgerritAdam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection  https://review.opendev.org/63633421:19
openstackgerritAdam Spiers proposed openstack/nova master: Add <launchSecurity> and <driver iommu='on' /> to config.py  https://review.opendev.org/63631821:21
openstackgerritAdam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required  https://review.opendev.org/64456521:26
openstackgerritAdam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption  https://review.opendev.org/66661621:30
*** markvoelker has joined #openstack-nova21:30
*** xek has joined #openstack-nova21:34
*** markvoelker has quit IRC21:35
*** xek has quit IRC21:43
openstackgerritMatt Riedemann proposed openstack/nova master: Rename the nova-grenade-live-migration job to nova-grenade-multinode  https://review.opendev.org/67884121:49
openstackgerritMatt Riedemann proposed openstack/nova master: Add cold migrate and resize to nova-grenade-multinode  https://review.opendev.org/67921021:49
openstackgerritMatt Riedemann proposed openstack/nova master: allow getting resource request of every bound ports of an instance  https://review.opendev.org/65511021:49
openstackgerritMatt Riedemann proposed openstack/nova master: Pass network API to the conducor's MigrationTask  https://review.opendev.org/65511121:49
openstackgerritMatt Riedemann proposed openstack/nova master: Add request_spec to server move RPC calls  https://review.opendev.org/65572121:49
openstackgerritMatt Riedemann proposed openstack/nova master: re-calculate provider mapping during migration  https://review.opendev.org/65511221:49
openstackgerritMatt Riedemann proposed openstack/nova master: update allocation in binding profile during migrate  https://review.opendev.org/65642221:49
openstackgerritMatt Riedemann proposed openstack/nova master: Extend NeutronFixture to handle migrations  https://review.opendev.org/65511421:49
openstackgerritMatt Riedemann proposed openstack/nova master: prepare func test env for moving servers with bandwidth  https://review.opendev.org/65510921:49
openstackgerritMatt Riedemann proposed openstack/nova master: Func test for migrate server with ports having resource request  https://review.opendev.org/65511321:49
openstackgerritMatt Riedemann proposed openstack/nova master: Make _rever_allocation nested allocation aware  https://review.opendev.org/67613821:49
openstackgerritMatt Riedemann proposed openstack/nova master: Support reverting migration / resize with bandwidth  https://review.opendev.org/67614021:49
openstackgerritMatt Riedemann proposed openstack/nova master: Func test for migrate re-schedule with bandwidth  https://review.opendev.org/67697221:49
openstackgerritMatt Riedemann proposed openstack/nova master: Support migrating SRIOV port with bandwidth  https://review.opendev.org/67698021:49
openstackgerritMatt Riedemann proposed openstack/nova master: Allow migrating server with port resource request  https://review.opendev.org/67149721:49
openstackgerritMatt Riedemann proposed openstack/nova master: Do not query allocations twice in finish_revert_resize  https://review.opendev.org/67882721:49
openstackgerritMatt Riedemann proposed openstack/nova master: Allow resizing server with port resource request  https://review.opendev.org/67901921:49
mriedemgibi: i'm out monday but rebased the series and addressed my small test comments on the bottom few changes so I'll +2 those ^21:49
*** slaweq has quit IRC21:53
openstackgerritMatt Riedemann proposed openstack/nova master: make config drives sticky bug 1835822  https://review.opendev.org/66973822:09
openstackbug 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)22:09
*** mriedem has quit IRC22:09
*** luksky has quit IRC22:11
openstackgerritMatt Riedemann proposed openstack/nova stable/stein: rt: soften warning case in _remove_deleted_instances_allocations  https://review.opendev.org/67951922:14
*** avolkov has quit IRC22:24
donnydmriedem remembers FN likes work to do, so he rebases fridays:)22:29
*** macz has quit IRC22:33
*** vesper has quit IRC22:46
*** vesper11 has joined #openstack-nova22:47
openstackgerritArtom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration  https://review.opendev.org/67259522:52
artom^^ found a bug :D22:53
artomwhich I'll fix Monday22:53
artomo/22:53
*** artom has quit IRC22:53
*** JamesBenson has joined #openstack-nova22:56
*** JamesBenson has quit IRC23:00
*** yaawang has quit IRC23:00
*** yaawang has joined #openstack-nova23:01
*** ivve has quit IRC23:10
*** jhesketh has quit IRC23:18
*** xek has joined #openstack-nova23:19
*** xek has quit IRC23:34
*** gyee has quit IRC23:37
*** N3l1x has quit IRC23:44
openstackgerritNathan Kinder proposed openstack/nova master: WIP - Allow TLS ciphers/protocols to be configurable for console proxies  https://review.opendev.org/67950223:52
*** jamiec has quit IRC23:52

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!