Tuesday, 2019-01-15

*** markvoelker has quit IRC00:06
*** mriedem_away has quit IRC00:07
*** ttsiouts has joined #openstack-nova00:08
*** _alastor_ has quit IRC00:14
*** wolverineav has quit IRC00:18
*** macza has quit IRC00:18
*** ttsiouts has quit IRC00:26
*** ttsiouts has joined #openstack-nova00:27
*** _alastor_ has joined #openstack-nova00:29
*** ttsiouts has quit IRC00:32
*** wolverineav has joined #openstack-nova00:35
*** _alastor_ has quit IRC00:39
*** _alastor_ has joined #openstack-nova00:40
*** _alastor_ has quit IRC00:45
*** wolverineav has quit IRC00:48
*** Kevin_Zheng has joined #openstack-nova00:48
*** wolverineav has joined #openstack-nova00:50
*** wolverineav has quit IRC00:55
*** whoami-rajat has joined #openstack-nova01:20
*** bhagyashris has joined #openstack-nova01:22
*** brinzhang has joined #openstack-nova01:29
*** fragatina has quit IRC01:34
*** markvoelker has joined #openstack-nova01:35
*** wolverineav has joined #openstack-nova01:35
*** wolverin_ has joined #openstack-nova01:38
*** wolverineav has quit IRC01:39
*** markvoelker has quit IRC01:40
*** wolverin_ has quit IRC01:43
openstackgerritDakshina Ilangovan proposed openstack/nova-specs master: Nova changes to support Local Resource Management that uses Resource Management Daemon  https://review.openstack.org/63081701:48
*** sapd1 has quit IRC01:50
*** hongbin has joined #openstack-nova01:57
*** hongbin has quit IRC01:59
*** Dinesh_Bhor has joined #openstack-nova02:03
*** hongbin has joined #openstack-nova02:04
*** macza has joined #openstack-nova02:06
*** tiendc has joined #openstack-nova02:07
*** Dinesh_Bhor has joined #openstack-nova02:09
*** macza has quit IRC02:11
*** erlon_ has quit IRC02:11
openstackgerritMerged openstack/nova master: Allow run metadata api per cell  https://review.openstack.org/62461202:20
*** sapd1_x has joined #openstack-nova02:21
*** brinzhang has quit IRC02:22
*** jangutter has joined #openstack-nova02:22
*** TxGirlGeek has quit IRC02:23
*** jangutter has quit IRC02:27
*** alex_xu has quit IRC02:37
*** alex_xu has joined #openstack-nova02:40
*** mhen has quit IRC02:53
*** mhen has joined #openstack-nova02:54
openstackgerritMerged openstack/nova master: Move some linux network helpers to use privsep.  https://review.openstack.org/62139803:04
*** TxGirlGeek has joined #openstack-nova03:19
*** wolverineav has joined #openstack-nova03:27
*** wolverineav has quit IRC03:32
*** Dinesh_Bhor has quit IRC03:38
*** Dinesh_Bhor has joined #openstack-nova03:45
*** TxGirlGeek has quit IRC03:51
*** Dinesh_Bhor has quit IRC03:55
*** udesale has joined #openstack-nova04:05
*** igordc has quit IRC04:05
*** _alastor_ has joined #openstack-nova04:11
*** dave-mccowan has quit IRC04:11
*** udesale has quit IRC04:15
*** _alastor_ has quit IRC04:16
*** udesale has joined #openstack-nova04:16
*** hongbin has quit IRC04:25
*** udesale has quit IRC04:26
*** diga has joined #openstack-nova04:33
*** abhishekk has joined #openstack-nova04:43
*** fragatina has joined #openstack-nova04:45
*** sorrison has joined #openstack-nova04:46
*** fragatina has quit IRC04:47
*** fragatina has joined #openstack-nova04:48
*** Dinesh_Bhor has joined #openstack-nova04:49
*** fragatina has quit IRC04:51
*** wolverineav has joined #openstack-nova04:51
*** fragatina has joined #openstack-nova04:51
*** _alastor_ has joined #openstack-nova04:52
*** fragatina has quit IRC04:54
*** _fragatina_ has joined #openstack-nova04:54
*** _fragatina_ has quit IRC04:56
*** fragatina has joined #openstack-nova04:56
*** _alastor_ has quit IRC04:57
*** fragatina has quit IRC04:58
*** fragatina has joined #openstack-nova04:58
*** wolverineav has quit IRC05:02
*** wolverineav has joined #openstack-nova05:03
*** fragatina has quit IRC05:03
*** fragatina has joined #openstack-nova05:04
*** fragatina has quit IRC05:11
*** fragatina has joined #openstack-nova05:11
*** fragatina has quit IRC05:13
*** fragatina has joined #openstack-nova05:13
*** abhishekk has quit IRC05:13
*** fragatina has quit IRC05:16
*** fragatina has joined #openstack-nova05:16
*** fragatina has quit IRC05:17
*** fragatina has joined #openstack-nova05:17
*** fragatina has quit IRC05:19
*** fragatina has joined #openstack-nova05:19
*** fragatina has quit IRC05:27
*** fragatina has joined #openstack-nova05:28
*** fragatina has quit IRC05:30
*** fragatina has joined #openstack-nova05:30
*** fragatina has quit IRC05:32
*** fragatina has joined #openstack-nova05:32
*** fragatina has quit IRC05:36
*** fragatina has joined #openstack-nova05:36
*** Bhujay has joined #openstack-nova05:43
*** Bhujay has quit IRC05:44
*** Bhujay has joined #openstack-nova05:45
*** fragatina has quit IRC05:45
*** Bhujay has quit IRC05:46
*** fragatina has joined #openstack-nova05:46
*** Bhujay has joined #openstack-nova05:46
*** markvoelker has joined #openstack-nova05:48
*** fragatina has quit IRC05:51
*** fragatina has joined #openstack-nova05:52
*** lbragstad has quit IRC05:56
*** fragatina has quit IRC05:59
*** markvoelker has quit IRC05:59
*** fragatina has joined #openstack-nova05:59
*** fragatina has quit IRC06:01
*** fragatina has joined #openstack-nova06:01
*** udesale has joined #openstack-nova06:05
*** brinzhang has joined #openstack-nova06:08
*** udesale has quit IRC06:12
*** udesale has joined #openstack-nova06:13
*** ratailor has joined #openstack-nova06:13
*** fragatina has quit IRC06:13
*** fragatina has joined #openstack-nova06:14
*** fragatina has quit IRC06:18
*** fragatina has joined #openstack-nova06:18
*** fragatina has quit IRC06:22
*** fragatina has joined #openstack-nova06:23
*** Dinesh_Bhor has quit IRC06:23
*** fragatina has quit IRC06:23
*** fragatina has joined #openstack-nova06:24
*** jangutter has joined #openstack-nova06:24
*** fragatina has quit IRC06:25
*** fragatina has joined #openstack-nova06:25
*** fragatina has quit IRC06:26
*** fragatina has joined #openstack-nova06:26
*** fragatina has quit IRC06:27
*** fragatina has joined #openstack-nova06:27
*** wolverineav has quit IRC06:28
*** itlinux_ has joined #openstack-nova06:28
*** wolverineav has joined #openstack-nova06:29
*** jangutter has quit IRC06:29
*** fragatina has quit IRC06:29
*** fragatina has joined #openstack-nova06:29
*** itlinux has quit IRC06:30
*** fragatina has quit IRC06:31
*** fragatina has joined #openstack-nova06:32
*** fragatina has quit IRC06:32
*** wolverineav has quit IRC06:33
*** fragatina has joined #openstack-nova06:33
*** udesale has quit IRC06:33
*** udesale has joined #openstack-nova06:33
*** brault has quit IRC06:33
*** _fragatina_ has joined #openstack-nova06:34
*** _fragatina_ has quit IRC06:37
*** _fragatina_ has joined #openstack-nova06:37
*** fragatina has quit IRC06:38
openstackgerritmelanie witt proposed openstack/nova master: Propagate exception message from _prep_block_device  https://review.openstack.org/62483206:42
openstackgerritmelanie witt proposed openstack/nova master: Add method to generate device names universally  https://review.openstack.org/57306606:42
openstackgerritmelanie witt proposed openstack/nova master: Add configuration of maximum disk devices to attach  https://review.openstack.org/61677706:42
*** _fragatina_ has quit IRC06:44
*** fragatina has joined #openstack-nova06:44
*** fragatina has quit IRC06:47
*** fragatina has joined #openstack-nova06:48
*** fragatina has quit IRC06:48
*** itlinux_ has quit IRC06:48
*** fragatina has joined #openstack-nova06:48
*** fragatina has quit IRC06:49
*** fragatina has joined #openstack-nova06:50
*** fragatina has quit IRC06:52
*** Dinesh_Bhor has joined #openstack-nova06:53
*** fragatina has joined #openstack-nova06:53
*** rcernin has quit IRC06:59
*** Bhujay has quit IRC07:01
*** fragatina has quit IRC07:07
*** fragatina has joined #openstack-nova07:08
*** moshele has joined #openstack-nova07:08
*** Bhujay has joined #openstack-nova07:10
*** Bhujay has quit IRC07:11
*** Bhujay has joined #openstack-nova07:11
*** tetsuro has joined #openstack-nova07:14
*** tetsuro has quit IRC07:16
*** fragatina has quit IRC07:18
*** fragatina has joined #openstack-nova07:19
*** fragatina has quit IRC07:25
*** fragatina has joined #openstack-nova07:26
*** fragatina has quit IRC07:31
*** fragatina has joined #openstack-nova07:31
*** dpawlik has joined #openstack-nova07:36
openstackgerritShilpa Devharakar proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API  https://review.openstack.org/61262607:39
*** wolverineav has joined #openstack-nova07:43
*** fragatina has quit IRC07:44
*** fragatina has joined #openstack-nova07:44
*** tetsuro has joined #openstack-nova07:47
*** wolverineav has quit IRC07:47
*** fragatina has quit IRC07:49
*** _fragatina_ has joined #openstack-nova07:49
*** tetsuro has quit IRC07:52
*** tetsuro has joined #openstack-nova07:54
*** tetsuro has quit IRC07:55
*** ccamacho has joined #openstack-nova07:56
*** _fragatina_ has quit IRC07:59
*** fragatina has joined #openstack-nova08:00
*** jangutter has joined #openstack-nova08:01
*** jangutter has quit IRC08:01
*** jangutter has joined #openstack-nova08:02
*** rpittau has joined #openstack-nova08:03
*** tetsuro has joined #openstack-nova08:06
*** tetsuro has quit IRC08:07
*** fragatina has quit IRC08:13
*** fragatina has joined #openstack-nova08:14
*** helenafm has joined #openstack-nova08:17
*** panda|off is now known as panda08:19
*** fragatina has quit IRC08:22
*** fragatina has joined #openstack-nova08:23
*** ccamacho has quit IRC08:25
*** fragatina has quit IRC08:26
*** fragatina has joined #openstack-nova08:27
*** yan0s has joined #openstack-nova08:28
*** mdbooth has joined #openstack-nova08:31
*** mdbooth_ has quit IRC08:33
*** ralonsoh has joined #openstack-nova08:36
openstackgerritJan Gutter proposed openstack/nova master: Convert vrouter legacy plugging to os-vif  https://review.openstack.org/57132508:42
*** fragatina has quit IRC08:45
*** fragatina has joined #openstack-nova08:46
*** fragatina has quit IRC08:46
*** fragatina has joined #openstack-nova08:46
*** _alastor_ has joined #openstack-nova08:53
*** dtantsur|afk is now known as dtantsur08:56
*** _alastor_ has quit IRC08:58
*** xek has joined #openstack-nova09:00
openstackgerritAdrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support  https://review.openstack.org/62011509:01
*** fragatina has quit IRC09:01
*** fragatina has joined #openstack-nova09:01
*** fragatina has quit IRC09:02
*** fragatina has joined #openstack-nova09:03
*** fragatina has quit IRC09:04
*** fragatina has joined #openstack-nova09:05
*** ccamacho has joined #openstack-nova09:08
*** alex_xu has quit IRC09:09
*** jangutter has quit IRC09:18
*** jangutter has joined #openstack-nova09:23
*** pcaruana has joined #openstack-nova09:30
brinzhang+lib/placement:start_placement_api:179     die 179 'placement-api did not start'09:30
brinzhangWhen I execute ./stack.sh report this error, anyone can give some help?09:31
brinzhangthis is the local.conf file on my devstack http://paste.openstack.org/show/742619/09:32
*** derekh has joined #openstack-nova09:33
*** ttsiouts has joined #openstack-nova09:36
*** wolverineav has joined #openstack-nova09:36
*** jungleboyj has quit IRC09:39
*** wolverineav has quit IRC09:41
*** jungleboyj has joined #openstack-nova09:42
*** bhagyashris has quit IRC09:52
*** ratailor_ has joined #openstack-nova09:52
*** ratailor has quit IRC09:55
*** Dinesh_Bhor has quit IRC09:56
*** Dinesh_Bhor has joined #openstack-nova10:05
*** Dinesh_Bhor has quit IRC10:06
*** jaosorior has joined #openstack-nova10:09
*** fragatina has quit IRC10:23
*** fragatina has joined #openstack-nova10:24
*** udesale has quit IRC10:25
*** fragatina has quit IRC10:28
*** fragatina has joined #openstack-nova10:29
*** diga has quit IRC10:34
*** udesale has joined #openstack-nova10:35
*** lpetrut has joined #openstack-nova10:39
*** moshele has quit IRC10:41
openstackgerritZhenyu Zheng proposed openstack/nova master: Per-instance serial number  https://review.openstack.org/61995310:45
Kevin_Zhengthe unstack script did not work correctly10:45
Kevin_Zhengbrinzhang10:45
Kevin_Zhengdelete all your devstack related files from site-enabled and site-availables10:46
Kevin_Zhengafter unstack10:46
Kevin_Zhengthen stack again10:46
*** macza has joined #openstack-nova10:50
brinzhangKevin_Zheng: I was unstack more than once, it always has this issue :(10:52
Kevin_Zhengyeah, you should clean up those files manually10:52
Kevin_Zhengbefore you call stack again10:52
Kevin_Zhengfiles in site-enabled and site-avialable10:53
brinzhangI don't particularly understand which files are included?10:53
*** brault has joined #openstack-nova10:54
brinzhangFor example the devstack folder?10:54
Kevin_Zhengand systemd files10:54
Kevin_Zhengcheck your apache2/site-enabled10:54
*** macza has quit IRC10:54
*** udesale has quit IRC11:01
*** cdent has joined #openstack-nova11:02
*** jangutter has quit IRC11:03
*** jangutter has joined #openstack-nova11:04
*** ratailor__ has joined #openstack-nova11:10
*** ratailor_ has quit IRC11:12
*** erlon_ has joined #openstack-nova11:15
*** sapd1_x has quit IRC11:16
*** kashyap has joined #openstack-nova11:16
*** openstackgerrit has quit IRC11:22
*** moshele has joined #openstack-nova11:24
kashyaptime tox -e docs11:26
*** jangutter has quit IRC11:26
kashyapOeps, false window11:26
*** jangutter has joined #openstack-nova11:27
kashyapstephenfin: Before I waste too much time on this, the `below <`Prerequisites`>__ hyperlink to a section in rST doesn't seem to fly :-(11:33
kashyapThe result: http://logs.openstack.org/27/629627/5/check/openstack-tox-docs/bde0494/html/admin/Prerequisites11:33
kashyapSource on line-19: https://review.openstack.org/#/c/629627/5/doc/source/admin/secure-live-migration-with-qemu-native-tls.rst@1911:33
stephenfinkashyap: Ah, whoops. `below <#Prerequisites>`__11:34
kashyapDarn, was just reading the man page.11:34
stephenfinkashyap: Alternatively, :ref:`below <Prerequisites>`11:34
kashyapYeah, read the :ref: too11:34
kashyapIt is a so subtly frustrating that it tests my stoic skills.11:34
kashyap:D11:34
kashyapThank you!11:35
*** udesale has joined #openstack-nova11:36
*** ttsiouts has quit IRC11:37
*** ttsiouts has joined #openstack-nova11:37
kashyapstephenfin: Strange, the former (with #) didn't work; now trying the latter.11:38
kashyap(By "didn't work", clicking on 'below' results in nothing.)11:39
*** ttsiouts has quit IRC11:42
*** tbachman has quit IRC11:51
*** fragatina has quit IRC11:54
*** jangutter has quit IRC11:54
*** jangutter has joined #openstack-nova11:55
*** rpittau is now known as rpittau|lunch11:59
*** rpittau has joined #openstack-nova12:02
*** rpittau is now known as rpittau|lunch12:05
kashyap(Yep, the latter works.  Most excellent.)12:09
cdentI'm going to write a book on RST. It's going to be a compelling tale of the master and his student, staring stephenfin and kashyap . It will sell millions12:13
kashyapHeh12:13
*** rpittau|lunch has quit IRC12:14
cdentit will be framed as a series of vignettes12:14
kashyapFunny you mention vingettes—12:15
*** rpittau|lunch has joined #openstack-nova12:15
kashyapThe other day when my Dutch language teacher asked for ideas for "different kinds of creative writing", I sent the idea of vignettes12:16
cdenta worth style12:16
cdentworthy12:16
kashyapWith an example pointing to the vignettes about "The House on the Mango Street" (the first story here: https://www.sausd.us/cms/lib/CA01000471/Centricity/Domain/457/The%20House%20on%20Mango%20Street.pdf)12:16
*** jangutter has quit IRC12:17
*** jangutter has joined #openstack-nova12:18
kashyapcdent: But we ran out of time, we had our last class (and exam) of "level-10" yesterday.  Next Monday are the results...12:18
*** ttsiouts has joined #openstack-nova12:18
kashyap(But the teacher said she's going to do that vignet exercise for the next set of students.)12:18
kashyapAlthough nothing is stopping me from practising it myself.12:18
cdentgood luck12:19
*** rpittau|lunch has quit IRC12:29
*** rpittau|lunch has joined #openstack-nova12:29
*** openstackgerrit has joined #openstack-nova12:30
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962712:30
brinzhangKevin_Zheng: Thanks for your help, my devstack12:31
brinzhangis ok now :)12:31
*** jangutter has quit IRC12:32
Kevin_Zhengyw12:36
*** rpittau|lunch has quit IRC12:39
*** ratailor__ has quit IRC12:40
*** Luzi has joined #openstack-nova12:41
*** _alastor_ has joined #openstack-nova12:43
*** panda is now known as panda|lunch12:47
*** brinzhang has quit IRC12:48
*** _alastor_ has quit IRC12:48
kashyapstephenfin: Squashed all the additional into the main doc: https://review.openstack.org/62962712:50
kashyapRendered version:12:50
kashyaphttp://logs.openstack.org/27/629627/6/check/openstack-tox-docs/39c0a0b/html/admin/secure-live-migration-with-qemu-native-tls.html12:50
kashyap(All the damn hyperlinks work. :D)12:50
dpawlikkashyap: Hi, does Openstack documentation have any information, which cpu_model_extra_flags are "recommended" to enable?12:51
*** tiendc has quit IRC12:51
kashyapdpawlik: Hi, I wrote some docuemntation for it, and linked to it as well12:51
kashyapLet me find12:51
dpawlikkashyap, thx12:52
kashyapdpawlik: A couple of URLs:12:53
*** gyee has joined #openstack-nova12:53
kashyap(1) See slides 35 (76 of 80) here: https://kashyapc.fedorapeople.org/Effective-Virtual-CPU-Configuration-in-Nova-Berlin2018.pdf12:53
kashyap(2) https://www.berrange.com/posts/2018/06/29/cpu-model-configuration-for-qemu-kvm-on-x86-hosts/12:53
kashyap(3) https://www.qemu.org/2018/02/14/qemu-2-11-1-and-spectre-update/12:53
dpawlikkashyap: thank you :)12:54
kashyap(4) `grep` for "cpu_model_extra_flags" here: https://docs.openstack.org/nova/rocky/configuration/config.html12:54
kashyapThat's it.12:54
dpawlikkashyap: I guess for now I will test just those PCID, VMX, pdpe1gb12:56
dpawlikI was thinking that there are more of them, that are recommended12:56
kashyapdpawlik: For Intel hosts, don't forget: "spec-ctrl", "ssbd" as well.12:57
amorinyes12:57
dpawlikkashyap: it will be nice to have "default suggested" features that should be enabled12:58
dpawlikright now I just add PCID and pdpe1gb12:59
dpawlikbut about "spec-ctrl", "ssbd" i didn't know that they should be also enabled12:59
kashyapdpawlik: That "suggestion" is not as easy as it sounds -- because that requires several checks to be done.12:59
kashyapE.g. what host microcode you have, what host/guest kernel, what version of libvirt and QEMU12:59
kashyapAnd _then_, some magic code among libvirt/QEMU should do what you have and _then_ suggest something.13:00
dpawlikwow13:00
kashyapWe had some discussion about this with the KVM/QEMU/libvirt folks at one of the recent confereces.  More details to emerge...13:00
kashyapUntil dpawlik Until then, I only have: "x86: Updating to patched vCPU models" section13:01
dpawlikit seems like table with "which features are available in which kernel version and qemu/libvirt"13:01
* kashyap wonders if he should convert that LaTex slides into a proper single PDF (LaTex, of course) document.13:01
kashyapBut that has to wait...13:01
dpawlikkashyap: ack.  Thanks a lot13:02
kashyapNP.13:02
*** rpittau has joined #openstack-nova13:03
*** jangutter has joined #openstack-nova13:08
*** wolverineav has joined #openstack-nova13:12
*** wolverineav has quit IRC13:16
*** tbachman has joined #openstack-nova13:19
*** tbachman has quit IRC13:23
*** eharney has joined #openstack-nova13:24
ShilpaSDmriedem: Hi, replied to your comment and uploaded patch https://review.openstack.org/#/c/612626/2, Please give your feedback on the latest patch.13:26
*** alex_xu has joined #openstack-nova13:27
*** dave-mccowan has joined #openstack-nova13:29
*** tbachman has joined #openstack-nova13:30
*** gyee has quit IRC13:34
*** udesale has quit IRC13:36
*** udesale has joined #openstack-nova13:36
*** asmita has quit IRC13:40
*** panda|lunch is now known as panda13:43
*** ade_lee has quit IRC13:46
openstackgerritJan Gutter proposed openstack/os-vif master: Clean up versioned object backlevelling code  https://review.openstack.org/63097613:48
openstackgerritMaria Luisa Arches proposed openstack/nova master: Update network info during confirmed cold migration  https://review.openstack.org/63097813:51
*** jangutter_ has joined #openstack-nova13:52
*** jangutter has quit IRC13:55
*** jangutter_ is now known as jangutter13:58
*** mvkr has quit IRC13:58
*** mlavalle has joined #openstack-nova13:58
alex_xuwill NUMACell.memory include the size of hugepages?14:03
*** efried1 has joined #openstack-nova14:06
stephenfinalex_xu: No, it's just a basic values, i.e. what you'd get from 'cat /sys/devices/system/node/nodeN/meminfo | grep MemTotal'14:08
stephenfin*value14:08
sean-k-mooneyactully i think it will14:08
sean-k-mooneythe memory field should be the total memory on that numa node14:08
sean-k-mooneymem total include hugepages14:08
stephenfinOh, I read that directly (as in would it have information about hugepages, which is actually reported via 'NUMACell.mempages')14:09
alex_xuI doesn't configure hugepage on my pc, so can't verify that...14:09
sean-k-mooneyone sec ill post a past bin of the output14:10
stephenfinalex_xu: I can give you info from one of my machine, if you'd like14:10
*** efried has quit IRC14:10
*** efried1 is now known as efried14:10
*** mriedem has joined #openstack-nova14:10
stephenfinOr, better yet, let sean-k-mooney do it :)14:10
alex_xuhah, I just want an answer :)14:10
sean-k-mooneyhttp://paste.openstack.org/show/742643/14:10
sean-k-mooneyso i have 64GB of ram on that numa node14:10
sean-k-mooneyNode 0 MemTotal:       65998248 kB14:11
alex_xujust want to ensure whether we have this problem https://review.openstack.org/#/c/629281/1/nova/virt/hardware.py@1050 or not14:11
sean-k-mooneyi have hugepages allocated too "HugePages_Total: 27136" these are 2MB huge pages14:11
stephenfinalex_xu: I saw the comment, but I'm not sure I understand the actual problem. What do you think could go wrong?14:12
stephenfinJust saw you replied. Oops14:12
*** mmethot has joined #openstack-nova14:12
sean-k-mooneyinstance_cell.memory is the amoung of guest ram14:13
stephenfinalex_xu: So I think you're saying that https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L982 could be wrong?14:13
sean-k-mooneyon that virtual numa node14:13
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS"  https://review.openstack.org/63098014:13
stephenfinalex_xu: Or rather that 'host_cell.avail_memory' includes memory that's not actually available because it's used for hugepages14:13
sean-k-mooneythis will be compared to the amount of ram available for the page size selected14:13
stephenfinalex_xu: Do I understand correctly? If so, I guess this would be easily checked. On a 64GB host, I create 34 1GB huge pages then try to boot a 32GB pinned instance. If it doesn't fail, we're likely overcommitting14:16
*** psachin has joined #openstack-nova14:16
stephenfinthough if that is the case, it's a different bug again that I think we should probably handle in a follow-up14:16
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS"  https://review.openstack.org/63098014:16
alex_xustephenfin: yes, that is the case I talk about14:17
sean-k-mooney host_cell.available is compareing free memory  not total but the page size check is done by https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1021-L1056 which is run after14:17
kashyapDamn it, one last respin14:17
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS"  https://review.openstack.org/63098014:17
kashyap(Sorry for the spam, folks.)14:18
stephenfinsean-k-mooney: Right, but I'm changing that here https://review.openstack.org/#/c/629281/1/nova/virt/hardware.py@105014:18
stephenfinsean-k-mooney: to allow overcommit if using the smallest pagesize14:18
sean-k-mooneyyes i think it si still correct i like the wrong peice of code one sec14:19
stephenfinsean-k-mooney: Ah, yes. https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L982 is available memory, as you say, not total memory14:19
stephenfinbut it's based on total memory (which could be include memory used for hugepages) minus memory consumed by instances14:21
stephenfinugh, I need to check this locally again14:22
sean-k-mooneyso you are changing _pack_instance_onto_cores to allow oversubsciption you are not modifying _numa_fit_instance_cell14:22
sean-k-mooney_pack_instance_onto_cores is called from _numa_fit_instance_cell_with_pinning which is called in _numa_fit_instance_cell after the check on line 104614:24
*** lbragstad has joined #openstack-nova14:24
sean-k-mooneysorry no ignore that14:24
sean-k-mooneygerrit is mseeing with me you are modifying _numa_fit_instance_cell14:25
stephenfinsean-k-mooney: I'm modifying _numa_cell_supports_pagesize_request which is called by _numa_fit_instance_cell14:26
sean-k-mooneystephenfin: in either case i think the code you have is correct14:27
sean-k-mooneycan_fit_pagesize will only consider teh total or availbe memory for the specified page size which will always be less or equal to the total or available ram for the numa node14:29
alex_xuemm...the problem is overcommit, not less or equal to the total or avail ram14:30
sean-k-mooneythis code is only ment to ensure the vm does not request more ram then is available for that pagesize on that numa node14:31
sean-k-mooneyover commit from multiple vms is intended to be allowed for the smallest page size as we allow ram allocation ratios > 114:31
sean-k-mooneyalex_xu: i take it that your are conserned that https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1085-L1106 will not be correct with this change14:34
alex_xuyes, but that patch allow the overcommit for each numa cell, but in the later, we found it is cpu pinned instance, we only ensure it isn't overcommit by the amount memory for all the numa cells14:35
alex_xusean-k-mooney: no, I concern this branch https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1076-L1083...14:35
sean-k-mooneyah well i just spotted that stpehn has not allowed oversubscition here also which shoudl be allowed https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L982-L99014:36
sean-k-mooneyor rather this should not be checking host_cell.avail_memory it should be checking host_cell.memory14:37
sean-k-mooneyor better it should be doing a page size aware check14:37
stephenfinI don't think I changed that14:37
sean-k-mooneystephenfin: you didnt that is my point14:37
sean-k-mooneyif you dont chenge that pinned instance can still not have over subsribed memory which is part of the original bug14:38
stephenfinsean-k-mooney: Yes, that's what alex_xu is saying, I think :)14:38
stephenfinHowever, I should point out that this is yet another corner case that I think I should fix in a follow-up14:38
alex_xusean-k-mooney: why that should check with host_cell.memory, sorry I didn't get the point14:39
sean-k-mooneystephenfin: well alex_xu is also makeing that such a check shoudl be page size aware14:39
sean-k-mooneyalex_xu: checking avilable memory is checking free memory so it decresase with each vm booted14:40
sean-k-mooneyso if i have 20 gigabes of memory and ram allocation raitio of 1.5 avilable will alow only 2 10G vms to both and total will allow 3 respecting the allocation ratio14:40
alex_xusean-k-mooney: yea, that is we said not allow overcommit by checking free memory, right?14:41
sean-k-mooneyalex_xu: over commit of ram is intended to be support for pinned instances14:41
alex_xusean-k-mooney: ah...why14:41
sean-k-mooneystephenfin: if you chage it to patial bug then it can be a follow up but if you want it to be closed bug then it needs to be in this change14:42
alex_xuthen I would say that isn't the problem of stephenfin's patch14:42
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Restore nova-consoleauth to install docs  https://review.openstack.org/63099714:42
stephenfinsean-k-mooney: You sure? It's very much a different bug14:42
sean-k-mooneyalex_xu: the cpu pinning specs never forbid over subsription of memory and it was a bevhioral change that was intoduced in error after the fact with no spec14:42
sean-k-mooneystephenfin: its not from my point of view its the exact same bug14:43
alex_xusean-k-mooney: ok, got it14:43
alex_xuso that is another thing now14:44
sean-k-mooneystephenfin: if you update the bug to specicialy state it dose not fix pinned instance and file a new bug then you could close the existing bug. i consider pinned instance to just be a special case of instnace with numa topologies14:45
alex_xuwhatever...close to 11pm, I need to go to bed now14:46
sean-k-mooneyalex_xu: good night o/14:47
maciejjozefczykstephenfin: hey, if you have a minute PTAL https://review.openstack.org/#/c/614167/ and https://review.openstack.org/#/c/591607. You have already reviewed that :) Thanks14:49
stephenfinmaciejjozefczyk: Just working through the above hugepage issues atm, but I'll stick it on my review queue :)14:50
*** mchlumsky has joined #openstack-nova14:50
maciejjozefczykstephenfin: good luck :)14:50
*** artom has quit IRC14:52
*** moshele has quit IRC14:53
sean-k-mooneystephenfin: ill reply to the bug and suggest coverign the pinning issue in a follwo up patch14:53
sean-k-mooneyas a seperate bug14:53
*** artom has joined #openstack-nova14:56
stephenfinsean-k-mooney: I'm rather confused now http://paste.openstack.org/show/742644/14:56
*** tbachman has quit IRC14:58
sean-k-mooneylooking14:58
*** artom has quit IRC15:00
*** mvkr has joined #openstack-nova15:00
sean-k-mooneyhost_cell.avail_memory include hugepages15:00
*** artom has joined #openstack-nova15:00
sean-k-mooneyso the first instace boots15:01
sean-k-mooneyand the second lets see15:01
stephenfinyup, I get that. What's up with the second one though15:01
*** tbachman has joined #openstack-nova15:01
stephenfinI figured this would prevent that from happening https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L1021-L103615:01
* stephenfin sticks in logs and tries again15:01
ShilpaSDmriedem: Hi, replied to your comment and uploaded patch https://review.openstack.org/#/c/612626/2, Please give your feedback on the latest patch.15:02
sean-k-mooneystephenfin: is this a dual numa system15:02
*** Bhujay has quit IRC15:02
stephenfinshouldn't matter. I've configured the same amount of hugepages for both, and both sockets appear to have ~ the same amount of memory15:02
sean-k-mooneyi see form the free you have 6.7G free but the cat command shows ~400MB free so im guessing its booting on numa node 115:03
sean-k-mooneystephenfin: you have configred 16GB of hugepages total15:03
stephenfinsean-k-mooney: Yup, 8GB per node15:03
sean-k-mooneyso there are 8G of small pages on each node some of which are used by the os15:04
sean-k-mooneyoh its a 10G vm not 1G15:04
stephenfinYup15:04
stephenfinSorry, should have highlighted that15:04
stephenfin16 GB per node - 8 GB 2M hugepages should leave only 8 GB usable small pages per node15:05
sean-k-mooneyno i just missed the 0 at the end15:05
sean-k-mooneyyep15:05
stephenfinSo a 10 GB instance should fall flat on its face15:05
sean-k-mooneyso rather then looking at this output its better to look at libvirts15:05
sean-k-mooneyalso what is your ram allocation ratio set to im assuming 1.515:06
stephenfin16.015:06
stephenfinmanually set15:06
stephenfinshouldn't matter for hugepages though15:06
sean-k-mooneyyour ram is set to 1615:06
sean-k-mooneytrue15:06
sean-k-mooneydo you have your patch applied or master15:07
stephenfinmy patch15:07
sean-k-mooneywith an explicit pagesize of small we expect https://github.com/openstack/nova/blob/89c2d1056b3f37bad243c7468d974ded71336a0f/nova/virt/hardware.py#L623-L649 to reject it correct15:09
stephenfinYes, that would be my expectation15:10
sean-k-mooneyso you have not modifed that in your patch so that is either broken and has always been or there is something else going on15:11
*** mchlumsky has quit IRC15:11
sean-k-mooneywell you changed the name of the fuction it calls but otherwise its the same15:12
*** tbachman has quit IRC15:12
*** mchlumsky has joined #openstack-nova15:12
*** jangutter has quit IRC15:18
*** tbachman has joined #openstack-nova15:18
*** jangutter_ has joined #openstack-nova15:19
*** ttsiouts has quit IRC15:19
*** ttsiouts has joined #openstack-nova15:19
*** ttsiouts has quit IRC15:24
*** efried has quit IRC15:24
*** efried has joined #openstack-nova15:25
openstackgerritMatt Riedemann proposed openstack/nova master: The field instance_name was added to InstanceCreatePayload  https://review.openstack.org/62796715:27
*** ttsiouts has joined #openstack-nova15:28
*** hongbin has joined #openstack-nova15:28
*** TxGirlGeek has joined #openstack-nova15:30
*** _alastor_ has joined #openstack-nova15:31
*** mrch_ has quit IRC15:35
openstackgerritJan Gutter proposed openstack/os-vif master: Clean up versioned object backlevelling code  https://review.openstack.org/63097615:36
*** tbachman has quit IRC15:36
*** efried has quit IRC15:36
*** efried has joined #openstack-nova15:37
sean-k-mooneystephenfin: looks like i was wrong about pinning and memory oversubsription. https://github.com/openstack/nova/commit/892e4032acea379bac6d1ae18fc5271ea17c0131 the intel implementation did not prevent it and the spec did not appove that behavior as a result we do not document it https://docs.openstack.org/nova/pike/admin/cpu-topologies.html#customizing-instance-cpu-pinning-policies. i consider that a15:40
sean-k-mooneylong term but which i think we should resolve seperatly15:40
*** efried has quit IRC15:40
*** tbachman has joined #openstack-nova15:41
*** jangutter_ is now known as jangutter15:44
*** Luzi has quit IRC15:44
*** fragatina has joined #openstack-nova15:46
mdboothcdent: Don't suppose you submitted that tox.ini patch, did you?15:49
*** efried has joined #openstack-nova15:49
mdboothcdent: NM, some gerrit-fu was a match to the task15:50
*** efried has quit IRC15:50
mriedemlyarwood: is the conflict in here still valid? https://review.openstack.org/#/c/627958/15:52
*** efried has joined #openstack-nova15:53
lyarwoodmriedem: yeah it's actually due to I217fba9138132b107e9d62895d699d238392e761 landing in stable/rocky after the initial PS I posted15:53
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: libvirt: Add workaround to cleanup instance dir when using rbd  https://review.openstack.org/62795815:53
*** tbachman has quit IRC15:53
mriedemlyarwood: yeah fixe15:54
mriedem*fixed15:54
lyarwoodhuh, I don't get it anymore, weird.15:55
lyarwoodokay well it wasn't clean just after that landed but is now, meh!15:55
*** tbachman has joined #openstack-nova16:01
*** eharney has quit IRC16:14
*** ccamacho has quit IRC16:16
*** tbachman has quit IRC16:18
openstackgerritKashyap Chamarthy proposed openstack/nova master: docs: Secure live migration with QEMU-native TLS  https://review.openstack.org/62962716:20
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: A few miscellaneous items related to "native TLS"  https://review.openstack.org/63098016:20
*** munimeha1 has joined #openstack-nova16:20
kashyap(Okay, let's put the bikeshed paint back in the cellar.)16:20
openstackgerritJan Gutter proposed openstack/os-vif master: Clean up versioned object backlevelling code  https://review.openstack.org/63097616:22
*** fragatina has quit IRC16:25
*** fragatina has joined #openstack-nova16:26
*** dklyle has joined #openstack-nova16:35
*** yan0s has quit IRC16:37
*** moshele has joined #openstack-nova16:40
*** ade_lee has joined #openstack-nova16:41
*** hamzy_ has quit IRC16:43
*** tbachman has joined #openstack-nova16:44
*** ttsiouts has quit IRC16:44
*** ttsiouts has joined #openstack-nova16:44
*** tbachman has quit IRC16:44
*** jangutter has quit IRC16:47
*** fragatina has quit IRC16:47
*** fragatina has joined #openstack-nova16:48
*** fragatina has quit IRC16:48
*** wolverineav has joined #openstack-nova16:49
*** fragatina has joined #openstack-nova16:49
*** rpittau has quit IRC16:49
*** ttsiouts has quit IRC16:50
*** fragatina has quit IRC16:53
*** wolverineav has quit IRC16:53
cdentmdbooth: did you find (and then try) that py37 tox fix? I was on the wrong machine to be able to give it a totally robust check16:53
*** fragatina has joined #openstack-nova16:53
*** tbachman has joined #openstack-nova16:54
*** mvkr has quit IRC16:56
mdboothcdent: Yes. Tested locally, all good.16:57
cdentmdbooth: great, thanks16:57
openstackgerritMatt Riedemann proposed openstack/nova master: Send context.global_id on neutron calls  https://review.openstack.org/62703416:58
mdboothcdent: I rebased my current work on top of it for convenience.16:58
*** fragatina has quit IRC16:58
*** fragatina has joined #openstack-nova16:59
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Send context.global_id on neutron calls  https://review.openstack.org/62760316:59
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Send context.global_id on neutron calls  https://review.openstack.org/62760417:01
mdboothcdent: It breaks my 'testone' script, though, by not using the 'py37' env any more :/ I note that's by design, though.17:03
cdenttis17:03
*** udesale has quit IRC17:04
*** moshele has quit IRC17:04
*** cfriesen has joined #openstack-nova17:04
*** helenafm has quit IRC17:05
*** fragatina has quit IRC17:06
cfriesenmriedem: so I guess the "specify timeout per live migration request" spec kind of died on the vine?17:06
*** macza has joined #openstack-nova17:08
*** eharney has joined #openstack-nova17:13
*** macza_ has joined #openstack-nova17:13
*** macza has quit IRC17:14
*** wwriverrat has joined #openstack-nova17:16
openstackgerritMatt Riedemann proposed openstack/python-novaclient master: Add a note in "nova service-delete" help about deleting computes  https://review.openstack.org/62763717:20
*** lbragstad has quit IRC17:22
*** derekh has quit IRC17:22
*** wwriverrat has quit IRC17:23
*** lbragstad has joined #openstack-nova17:26
*** dtantsur is now known as dtantsur|afk17:28
*** lbragstad has quit IRC17:33
*** lbragstad has joined #openstack-nova17:33
*** dtantsur|afk has quit IRC17:36
eanderssonIt's still not possible to make weights plugins right?17:39
mriedemstephenfin: does the latest version of https://review.openstack.org/#/c/623564/ fix your problem?17:42
stephenfinmriedem: Knee deep in hugepage issues and don't want to unload that context. I can check first thing tomorrow though and post my feedback on the review17:43
sean-k-mooneymriedem: that is in relate to sqlite17:44
sean-k-mooneymriedem: i can test that locally with a docker container and comment back17:44
stephenfinsean-k-mooney: *changes in sqlite highlighted by sqlalchemy-migrate17:44
sean-k-mooneyright so if i clone that change an pip install it into my test env i should be able to test it with sqlite 3.2617:46
sean-k-mooneyso i can repoduce the bug locally again and manully test this since your busy17:46
*** ttsiouts has joined #openstack-nova17:47
jaypipesgibi: sorry, when is the bw scheduling hangout?17:52
sean-k-mooneyjaypipes: ehterpad say jan 18th at 17:00 utc17:53
sean-k-mooneyhttps://etherpad.openstack.org/p/bandwidth-way-forward17:53
jaypipessean-k-mooney: danke17:53
sean-k-mooneyit was originally ment to be an hour ago but was moved17:53
*** fragatina has joined #openstack-nova17:54
jaypipesack, I remember it moved but couldn't remember the new time. now on my calendar. thx again17:54
*** wolverineav has joined #openstack-nova17:54
*** wolverineav has quit IRC17:57
*** wolverineav has joined #openstack-nova17:57
*** mdrabe has joined #openstack-nova17:57
*** lbragstad is now known as lbragstad_lunch17:58
*** cfriesen has quit IRC18:00
*** cfriesen has joined #openstack-nova18:01
*** pcaruana has quit IRC18:01
*** psachin has quit IRC18:03
*** TxGirlGeek has quit IRC18:04
*** igordc has joined #openstack-nova18:05
openstackgerritStephen Finucane proposed openstack/nova master: Fix overcommit for NUMA-based instances  https://review.openstack.org/62928118:06
openstackgerritStephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities  https://review.openstack.org/63103818:06
stephenfinsean-k-mooney: ^18:06
stephenfinalex_xu: too ^18:06
*** TxGirlGeek has joined #openstack-nova18:08
sean-k-mooneystephenfin: mriedem: commented on https://review.openstack.org/#/c/623564/7.18:09
mriedemthanks18:10
mriedemzzzeek: are you ok with https://review.openstack.org/#/c/623564/ now?18:10
sean-k-mooneystephenfin: looking. i assuem that with both applied the guest with explcit small pages now works?18:12
*** wwriverrat has joined #openstack-nova18:12
stephenfinsean-k-mooney: Yeah, the second test case from paste.openstack.org/show/742644/ now does what I'd expect18:13
stephenfintesting the first case now18:13
sean-k-mooneyack18:13
*** brault has quit IRC18:14
*** brault has joined #openstack-nova18:15
*** itlinux has joined #openstack-nova18:16
*** panda is now known as panda|off18:19
*** brault has quit IRC18:20
*** efried has quit IRC18:21
*** efried has joined #openstack-nova18:26
sean-k-mooneystephenfin: left a comment on https://review.openstack.org/#/c/631038/118:29
cdentthanks for the review on the gabbi-tempest stuff jaypipes . your comment is exactly why I like it too18:30
sean-k-mooneycdent: gabbi-tempest test for placemet?18:31
melwittmriedem: thanks for the comments on the potential to leak host-related info here https://review.openstack.org/#/c/624832/5/nova/compute/manager.py I think the approach similar to InvalidBDM sounds safer, so I'm gonna respin18:31
cdentsean-k-mooney: for nova: https://review.openstack.org/#/c/613386/618:31
jaypipescdent: I'm giddy to get me some good ol' gabbi.18:33
cdentyou've installed your alliterative poet guise today18:34
sean-k-mooneyah i see. can gabbi also be used to simulatate an api? e.g. can i write the gabbits locally without needing and openstack/tempest deployment18:34
melwittcdent: I was about the say the same thing. alliterations and rhymes today I see18:34
*** wolverineav has quit IRC18:34
*** wolverineav has joined #openstack-nova18:35
cdentsean-k-mooney: something needs to host the api. it can either be "live" like that tempest stuff, or with wsgi-intercept, as the placement functional tests do it18:35
sean-k-mooneycdent: ah yes that was basically my intendt e.g. could you test it with say a tox job or something18:36
sean-k-mooney*env18:36
sean-k-mooneybut ya that seams cool18:36
jaypipescdent: I blame efried for that.18:37
cdentsean-k-mooney: have a look at the placement functional test setup, in tox18:37
jaypipescdent: it was his surge to purge the scourge of safe_connect that pushed me over the verge.18:37
sean-k-mooneyin any case ya it will be nice to seperate out real api test form senario tests18:37
*** ralonsoh has quit IRC18:38
sean-k-mooneythe distiction in my head is if you need to ping or ssh to the vm to check something its a senario test. or in other words you should be able to execute the api tests using fake compute nodes but senario test need real instances18:39
*** wolverineav has quit IRC18:40
*** wolverineav has joined #openstack-nova18:43
sean-k-mooneycdent: is the intent to eventurlly remove the current compute api test form tempest and migrate them to this?18:43
cdentsean-k-mooney: I don't really have a plan or intent. I simply created that so people could see it and think about the options18:44
*** hamzy_ has joined #openstack-nova18:44
sean-k-mooneyok the yaml format certenly seams simpler then the tempest python api to do the same.18:45
sean-k-mooneythat said im not that familar with tempest eiter so for someone who is that may not hold true18:45
*** wolverineav has quit IRC18:48
stephenfinsean-k-mooney, alex_xu: Yup, there's an issue there. It's a different bug though. Would have happened with/without sahid's patch http://paste.openstack.org/show/742676/18:51
*** lbragstad_lunch is now known as lbragstad18:52
sean-k-mooneythe bug being https://bugs.launchpad.net/nova/+bug/181187018:53
openstackLaunchpad bug 1811870 in OpenStack Compute (nova) "libvirt reporting incorrect value of 4k (small) pages" [Undecided,New]18:53
stephenfinsean-k-mooney: Nah, I'm saying it's different to https://launchpad.net/bugs/181097718:53
openstackLaunchpad bug 1810977 in OpenStack Compute (nova) "Oversubscription broken for instances with NUMA topologies" [Medium,In progress] - Assigned to Stephen Finucane (stephenfinucane)18:53
stephenfinthough it's also nothing to do with https://bugs.launchpad.net/nova/+bug/181187018:53
stephenfinthree bugs in the same piece of the code. Fun!18:53
*** eharney has quit IRC18:54
sean-k-mooneywell the best things come in threes18:54
stephenfinthat's what they told me in Amsterdam18:55
* sean-k-mooney read you past again18:55
sean-k-mooney*read your paste... i cant type some times18:56
sean-k-mooneyagh reads....18:56
mriedemmelwitt: ack18:57
mriedemmelwitt: so are you just going to work that into https://review.openstack.org/#/c/616777/ then?18:57
mriedemwith the new TooManyDiskDevices exception?18:57
melwittmriedem: yeah, that's what I was realizing. the problem is that the "Failure prepping block device" squashes everything in instance fault message. because, if I back out that one commit (the propagate commit) then I get InvalidBDM, but I don't get that message in the instance fault (where the user can see it)18:59
*** hamzy_ has quit IRC18:59
*** hamzy_ has joined #openstack-nova19:00
melwittmriedem: yeah, sorry, I guess if I just catch TooManyDiskDevices specifically (in the same area as the Exception catch-all that squashes), then I could do it19:00
mriedemi left a comment in the top patch - if TooManyDiskDevices extends InvalidBDM then you get that handling for free19:01
mriedemeither works19:01
*** cfriesen has quit IRC19:01
melwittyeah, I see that now. thanks19:01
*** wolverineav has joined #openstack-nova19:02
*** eharney has joined #openstack-nova19:09
*** lbragstad has quit IRC19:10
*** lbragstad has joined #openstack-nova19:11
*** hamzy_ has quit IRC19:13
openstackgerritmelanie witt proposed openstack/nova master: Add method to generate device names universally  https://review.openstack.org/57306619:18
openstackgerritmelanie witt proposed openstack/nova master: Add configuration of maximum disk devices to attach  https://review.openstack.org/61677719:18
openstackgerritStephen Finucane proposed openstack/nova master: Handle libvirt reporting incorrect 4k page quantities  https://review.openstack.org/63103819:19
openstackgerritStephen Finucane proposed openstack/nova master: Fix overcommit for NUMA-based instances  https://review.openstack.org/62928119:19
openstackgerritStephen Finucane proposed openstack/nova master: Make overcommit check for pinned instance pagesize aware  https://review.openstack.org/63105319:19
stephenfinsean-k-mooney: Done19:19
stephenfinand now I'm off home to watch the UK strap itself even tighter to that stake -_-19:20
*** lbragstad has quit IRC19:20
sean-k-mooneystephenfin: ya instead of popcorn maybe lyarwood should have grabed wiskey19:21
*** lbragstad has joined #openstack-nova19:21
stephenfinsean-k-mooney: indeed :)19:23
stephenfinmeanwhile, mriedem: you'll hate this https://review.openstack.org/#/c/631038/ cfriesen would no doubt be interested too, if he were around19:23
mriedemcommented19:27
*** jackding has quit IRC19:28
*** hamzy_ has joined #openstack-nova19:31
*** cfriesen has joined #openstack-nova19:35
*** lpetrut has quit IRC19:38
openstackgerritMerged openstack/nova stable/rocky: Add links to summit videos in user/cells.rst  https://review.openstack.org/62701919:38
*** ttsiouts has quit IRC19:45
*** ttsiouts has joined #openstack-nova19:46
*** wolverineav has quit IRC19:48
*** wolverineav has joined #openstack-nova19:48
*** wolverineav has quit IRC19:51
*** ttsiouts has quit IRC19:51
*** wolverineav has joined #openstack-nova19:51
*** wolverineav has quit IRC19:52
*** wolverineav has joined #openstack-nova19:52
sean-k-mooneya nice simple change for people to review https://review.openstack.org/#/c/573066/1019:53
sean-k-mooneythat said i guess we are still waiting on zuul19:54
*** wolverineav has quit IRC19:58
*** whoami-rajat has quit IRC20:00
*** jackding has joined #openstack-nova20:02
*** erlon_ has quit IRC20:05
*** brault has joined #openstack-nova20:06
lyarwoodsean-k-mooney: all brexit related donations of whiskey welcome in the coming days ;)20:10
*** brault has quit IRC20:11
*** hamzy_ has quit IRC20:12
openstackgerritJack Ding proposed openstack/nova master: [WIP] Flavor extra spec and image properties validation  https://review.openstack.org/62070620:13
openstackgerritEric Fried proposed openstack/nova master: Reduce calls to placement from _ensure  https://review.openstack.org/61567720:21
openstackgerritEric Fried proposed openstack/nova master: Consolidate inventory refresh  https://review.openstack.org/61569520:21
openstackgerritEric Fried proposed openstack/nova master: Commonize _update code path  https://review.openstack.org/61570520:21
openstackgerritEric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient  https://review.openstack.org/61704220:21
openstackgerritEric Fried proposed openstack/nova master: Rip out the SchedulerClient  https://review.openstack.org/61704920:21
openstackgerritEric Fried proposed openstack/nova master: Use a static resource tracker in compute manager  https://review.openstack.org/62071120:21
efriedjaypipes, cdent, mriedem: Thanks for the reviews, these should be ready now ^^^20:21
jaypipesefried: np. will re-review tomorrow morn.20:23
*** mvkr has joined #openstack-nova20:32
*** itlinux_ has joined #openstack-nova20:36
efriedjaypipes: Thanks.20:36
*** itlinux has quit IRC20:37
*** wolverineav has joined #openstack-nova20:39
*** itlinux_ has quit IRC20:39
*** wolverineav has quit IRC20:43
cdentthanks efried I'll also look tomorrow20:46
efriedcdent: Are you on your way out, or can you help me understand https://review.openstack.org/#/c/630745/ quick?20:46
cdentI can look20:46
cdentefried: what's the confusion?20:47
efriedfirst, the commit message makes it sound like there would be a conflict between zuul and tox somehow. I thought each ignored the other.20:47
cdentthe tox-siblings functionality used by zuul only works if the tox name and the tox env have he same name, becuse it does some digging around in the tox directory20:48
cdentthat was discussed in the change which extracted placement:20:48
cdent(the functional test changes for that, I mean):20:48
efriedmethinks stephenfin ought to be a reviewer on this patch...20:48
cdenthttps://review.openstack.org/#/c/617941/20:49
cdentefried: it's literally: make functional-py37 the same as py3620:49
cdentpy36 already works20:49
cdentpy37 does not20:49
cdentpy37 was merged at the same time as the placement external fixture stuff20:49
cdentit's just a merge conflict that didn't get caught because we don't do functional-py37 in the gate20:50
efriedcdent: So yeah, that was #2: would the problem be exposed by changing .zuul.yaml to add the py37 job, then prove this "fixes" it by rebasing that change on this one?20:50
cdentI didn't want to add the functional-py37 job becuase there isn't consensus to have that in nova yet, and mdbooth already confirmed it by hand20:50
efriedOkay. #3, is there (or should there be) a bug about this?20:51
cdent#3: meh20:51
cdentisn't that just overhead, especially since functional-py37 is a nice to have, not a thing we do in the gate that is impacting many20:51
cdentit only showed up because mdbooth encountered some troubles20:52
cdentwhy not just fix it and get it over with?20:52
efriedwfm, just ticking the boxes.20:52
*** wolverineav has joined #openstack-nova20:55
*** wolverineav has quit IRC21:00
*** wolverineav has joined #openstack-nova21:00
*** mdbooth_ has joined #openstack-nova21:14
*** cfriesen has quit IRC21:14
*** eharney has quit IRC21:15
*** cfriesen has joined #openstack-nova21:17
*** mdbooth has quit IRC21:18
*** whoami-rajat has joined #openstack-nova21:18
*** hamzy_ has joined #openstack-nova21:21
*** xek has quit IRC21:24
melwittefried: came across this bug which has something to do with Power8. would you mind taking a look and commenting if any of it makes sense to you? https://bugs.launchpad.net/nova/+bug/177692921:28
openstackLaunchpad bug 1776929 in OpenStack Compute (nova) "Web Console for Power8 KVM guest sends mouse events to guest mouse instead of guest tablet?" [Undecided,New]21:28
efriedmelwitt: Neat. We should talk to somebody at IBM about that. mmedvede?21:28
efriedmelwitt: Even when I worked for IBM (lo those many days ago) I never knew squat about power *k*vm.21:29
*** hamzy_ has quit IRC21:29
mmedvedeefried: hehe, I'll take a look. I'll probably need to find someone who is more knowledgeable about this than me21:30
melwittok. I don't know anything about power or power kvm so just looking for someone who knows more than me to help triage21:30
efriedmmedvede: I'm counting on six degrees here. For sure you'd be one degree closer than me.21:30
melwittthanks mmedvede21:30
efriedmelwitt: you, me, mmedvede - three degrees to go.21:30
efried#save21:37
efrieddamn21:37
melwittmriedem: reading through your comments. when I said "returns 500" for the server create, I meant via the server fault message. correct that the API returns 20221:42
mriedemok i read that as 500 response21:44
melwittyeah, I know. it needs more context to be clear21:45
melwittmeaning the comment needs to be better or different21:45
mriedemsome of my other comments are musing on side effects of this that could break other things, like rebuild/evacuate/unshelve, but that doesn't necessarily mean you need to change anything21:46
mriedemif the operator f*s you by dropping the limit and then you rebuild and blow up, that's on the operator21:46
melwittyeah21:49
melwittI'll go through the comments in detail and add to the patch to have more information where you've pointed out21:51
mriedemmelwitt: were you going to abandon https://review.openstack.org/#/c/624832/ ?21:54
melwittmriedem: yeah. wanted to wait for feedback before doing. sounds like it's clear to abandon now21:55
mriedemyeah i think what you have in the last patch in the series would be sufficient21:55
mriedemthinking about that, you could work that as an assertion wrinkle into the functional test for bfv21:55
mriedemassert that too many devices is in the fault21:55
melwittI thought I did already. /me looks again21:56
mriedemyeah you did, nvm21:56
mriedemjust looked again21:56
melwittk21:57
*** wolverineav has quit IRC21:58
*** wolverineav has joined #openstack-nova21:58
*** imacdonn has quit IRC22:06
*** imacdonn has joined #openstack-nova22:07
*** owalsh_ has joined #openstack-nova22:07
*** owalsh has quit IRC22:10
*** owalsh_ is now known as owalsh22:13
*** cdent has quit IRC22:20
*** mhen has quit IRC22:21
*** mhen has joined #openstack-nova22:23
mriedemdansmith: random question: you know how the context loses it's cell targeting when it goes over rpc, and then in conductor methods like unshelve_instance we have the targets_cell decorator to re-target the context based on the instance - what happens with the instance._context that is also originally targeted to the cell when the instance is retrieved in the API and it goes over rpc?22:30
openstackgerritEric Fried proposed openstack/nova master: Reduce calls to placement from _ensure  https://review.openstack.org/61567722:30
openstackgerritEric Fried proposed openstack/nova master: Consolidate inventory refresh  https://review.openstack.org/61569522:30
openstackgerritEric Fried proposed openstack/nova master: Commonize _update code path  https://review.openstack.org/61570522:30
openstackgerritEric Fried proposed openstack/nova master: Rip the report client out of SchedulerClient  https://review.openstack.org/61704222:30
openstackgerritEric Fried proposed openstack/nova master: Rip out the SchedulerClient  https://review.openstack.org/61704922:30
openstackgerritEric Fried proposed openstack/nova master: Use a static resource tracker in compute manager  https://review.openstack.org/62071122:31
mriedemdoes that retain the targeting? because i don't see is re-targeting the instance._context, but yet instance.save() works22:31
mriedem*us22:31
* efried can't wait until he can tox locally22:31
dansmithmriedem: the _context for the objects doesn't go over rpc either, if that's what you're asking22:32
dansmithmriedem: you get a new context on de-serialize from whatever context was used for the rpc call22:32
*** lpetrut has joined #openstack-nova22:34
mriedemok i'm lost on where that happens22:34
mriedemfor the instance._context i mean22:34
dansmithwhere the _context gets rebuilt on deserialize?22:35
mriedemyeah22:35
dansmithit's deep, probably in ovo, but I can look22:35
dansmitheither way.. why?22:35
dansmithI fear these questions come from a readiness to abuse something :)22:35
mriedembecause i was rebasing and looking at this again https://review.openstack.org/#/c/627890/2/nova/compute/manager.py@5990 where i'm sending an instance and context, targeted at the source cell, to a host in the target cell22:36
mriedemand that won't work22:36
mriedemi know about targets_cell to retarget the context in the conductor methods, but am wondering where the instance._context is re-targeted so it can continue to do things like instance.save() in conductor22:36
mriedemand have those changes go back to the source cell db22:37
dansmithit won't be re-targeted, it will be un-targeted22:37
mriedembecause of the cheating serializer in our tests, chances are i'll be writing a lot of code before i really find out in test if i'm screwing something up fundamental here22:37
*** lpetrut has quit IRC22:38
dansmithmriedem: the rpc layer calls, via the serializer and _process_object(), the obj_from_primitive, which does: https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L39222:38
mriedemok so if instance._context is untargeted, and you unshelve, which is in superconductor and presumably is configured for [database]/connection = cell0, where are those instance.save() updates going?22:38
dansmithmriedem: they'd try to go to cell0 and just fail because save() can't find the instance to update22:39
mriedemi'm not sure how tempest isn't blowing up then22:39
dansmithsurely because we run through an @targets_cell somewhere yeah?22:40
mriedemComputeTaskManager.unshelve_instance in conductor has targets_cell yeah22:41
mriedemso that hits before we deserialize?22:41
dansmithafter22:41
mriedemand instance._context is getting targeted by reference?22:42
dansmithyeah because instance._context _is_ context22:42
mriedemyeah ok22:42
mriedemok you can go back to watching jeapordy! now. i'll ask you about this again in another couple of years. :P22:43
dansmithhah22:43
dansmithis that when you're targeting cross-cell resize?22:43
dansmith(to be done... in a couple years)22:44
mriedemumm22:44
dansmithjust poking22:44
mriedemi believe march 30 was my given deadline22:44
dansmithI totes believe it22:44
mriedemafter which point i will be fired and working at our local video rental place22:44
mriedemof which there is 122:44
*** brault has joined #openstack-nova22:45
mriedemor, the TSA, i hear things are going well for employees there22:45
dansmithat least you have options22:45
*** brault has quit IRC22:45
dansmithhaha22:45
*** rcernin has joined #openstack-nova22:46
*** tbachman has quit IRC22:58
*** tbachman has joined #openstack-nova22:58
*** brault has joined #openstack-nova23:00
*** burt has quit IRC23:00
*** hongbin has quit IRC23:01
*** brault has quit IRC23:05
*** efried1 has joined #openstack-nova23:06
*** efried has quit IRC23:06
*** efried1 is now known as efried23:06
*** munimeha1 has quit IRC23:14
*** ade_lee has quit IRC23:22
*** efried has quit IRC23:22
*** efried has joined #openstack-nova23:22
*** whoami-rajat has quit IRC23:47
*** wolverineav has quit IRC23:56
*** ade_lee has joined #openstack-nova23:59

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