Friday, 2019-07-05

*** alex_xu has quit IRC00:10
*** slaweq has joined #openstack-nova00:16
*** frank_wang has quit IRC00:19
*** takashin has joined #openstack-nova00:23
*** slaweq has quit IRC00:24
*** ricolin has joined #openstack-nova00:56
openstackgerritMerged openstack/nova stable/stein: Ignore hw_vif_type for direct, direct-physical vNIC types  https://review.opendev.org/66735401:01
*** tetsuro has joined #openstack-nova01:02
*** imacdonn has quit IRC01:11
*** imacdonn has joined #openstack-nova01:12
*** slaweq has joined #openstack-nova01:14
*** spatel has joined #openstack-nova01:16
*** _alastor_ has joined #openstack-nova01:16
*** spatel has quit IRC01:20
*** slaweq has quit IRC01:25
*** adriant has joined #openstack-nova01:49
*** austinsun has joined #openstack-nova01:57
*** ricolin_ has joined #openstack-nova02:08
*** ricolin has quit IRC02:11
*** slaweq has joined #openstack-nova02:15
openstackgerritBoxiang Zhu proposed openstack/nova master: Make evacuation respects anti-affinity rule  https://review.opendev.org/64996302:18
*** ricolin__ has joined #openstack-nova02:18
*** ricolin_ has quit IRC02:21
*** spatel has joined #openstack-nova02:22
*** slaweq has quit IRC02:24
*** alex_xu has joined #openstack-nova02:26
*** whoami-rajat has joined #openstack-nova02:26
*** spatel has quit IRC02:27
*** tetsuro has quit IRC03:03
*** tetsuro has joined #openstack-nova03:04
*** lewe has joined #openstack-nova03:12
leweHi, Maybe someone can help me, I am trying to cold migrate an instance but it does not seems to work right. The instance was newly created on the destination hosts rather than copying the original disk from the instance original host. Thank you.03:13
*** slaweq has joined #openstack-nova03:14
*** tetsuro has quit IRC03:20
*** slaweq has quit IRC03:25
openstackgerritwangwei1 proposed openstack/nova master:   fix spelling error in nova/api/validation/__init__.py  https://review.opendev.org/66924403:36
*** BjoernT has joined #openstack-nova03:37
*** BjoernT_ has joined #openstack-nova03:40
*** BjoernT has quit IRC03:42
*** tetsuro has joined #openstack-nova03:55
*** slaweq has joined #openstack-nova04:16
*** rcernin has quit IRC04:17
*** lewe has quit IRC04:20
*** _alastor_ has quit IRC04:24
*** slaweq has quit IRC04:24
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Fix a broken link  https://review.opendev.org/66924804:26
*** abhishekk has joined #openstack-nova04:36
openstackgerritSeyeong Kim proposed openstack/nova stable/queens: Share snapshot image membership with instance owner  https://review.opendev.org/66166704:45
*** udesale has joined #openstack-nova04:49
*** BjoernT_ has quit IRC04:53
*** tkajinam has quit IRC04:54
*** tkajinam has joined #openstack-nova04:55
*** tetsuro has quit IRC04:57
*** tetsuro has joined #openstack-nova05:04
*** rcernin has joined #openstack-nova05:10
*** slaweq has joined #openstack-nova05:16
*** slaweq has quit IRC05:24
*** tkajinam_ has joined #openstack-nova05:25
*** tkajinam has quit IRC05:27
*** Luzi has joined #openstack-nova05:27
*** ricolin_ has joined #openstack-nova05:32
*** ricolin__ has quit IRC05:34
*** rcernin has quit IRC05:41
*** ratailor has joined #openstack-nova05:42
*** rcernin has joined #openstack-nova05:58
openstackgerritwangwei1 proposed openstack/nova stable/stein: fix spelling error in nova/api/validation/__init__.py  https://review.opendev.org/66925106:00
*** shilpasd has quit IRC06:00
*** luksky11 has joined #openstack-nova06:06
*** slaweq has joined #openstack-nova06:12
*** slaweq has quit IRC06:17
*** slaweq has joined #openstack-nova06:27
*** ivve has joined #openstack-nova06:31
openstackgerritGhanshyam Mann proposed openstack/nova-specs master: Fix the cross_az_attach value in AZ-unshelved spec  https://review.opendev.org/66926106:32
*** slaweq has quit IRC06:35
openstackgerritArtom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration  https://review.opendev.org/64002106:39
*** slaweq has joined #openstack-nova06:46
*** maciejjozefczyk has joined #openstack-nova06:54
*** boxiang has joined #openstack-nova06:59
*** jaosorior has joined #openstack-nova07:02
*** ivve has quit IRC07:06
*** rpittau|afk is now known as rpittau07:06
*** xek has joined #openstack-nova07:06
*** ricolin__ has joined #openstack-nova07:10
*** ricolin_ has quit IRC07:13
*** damien_r has joined #openstack-nova07:14
*** hoonetorg has quit IRC07:16
openstackgerritYongli He proposed openstack/nova master: Clean up orphan instances virt driver  https://review.opendev.org/64891207:18
openstackgerritYongli He proposed openstack/nova master: clean up orphan instances  https://review.opendev.org/62776507:18
*** tesseract has joined #openstack-nova07:21
*** ccamacho has joined #openstack-nova07:23
openstackgerritYongli He proposed openstack/nova master: clean up orphan instances  https://review.opendev.org/62776507:24
*** helenafm has joined #openstack-nova07:27
*** tssurya has joined #openstack-nova07:27
*** hoonetorg has joined #openstack-nova07:29
*** ricolin__ is now known as ricolin07:44
*** rcernin has quit IRC07:45
*** ttsiouts has joined #openstack-nova07:55
*** ralonsoh has joined #openstack-nova08:00
*** rcernin has joined #openstack-nova08:02
*** ivve has joined #openstack-nova08:06
*** ttsiouts has quit IRC08:06
*** ttsiouts has joined #openstack-nova08:07
*** xek has quit IRC08:10
*** ttsiouts has quit IRC08:11
*** tkajinam_ has quit IRC08:11
openstackgerritKashyap Chamarthy proposed openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec  https://review.opendev.org/66928408:12
kashyapstephenfin: johnthetubaguy: A trivial cosmetic fix --^ (rST syntax related)08:13
*** lpetrut has joined #openstack-nova08:23
*** lpetrut has quit IRC08:24
*** lpetrut has joined #openstack-nova08:25
openstackgerritLee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__()  https://review.opendev.org/66856408:25
*** ttsiouts has joined #openstack-nova08:26
*** tetsuro has quit IRC08:29
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Remove native LUKS compat code  https://review.opendev.org/66912108:35
*** tetsuro has joined #openstack-nova08:38
*** tetsuro has quit IRC08:40
*** rcernin has quit IRC08:49
*** ociuhandu has joined #openstack-nova09:04
*** takashin has left #openstack-nova09:05
*** rcernin has joined #openstack-nova09:05
*** jistr has quit IRC09:07
*** ricolin has quit IRC09:08
*** belmoreira has joined #openstack-nova09:12
openstackgerritStephen Finucane proposed openstack/nova master: Fix spelling error in 'nova/api/validation/__init__.py'  https://review.opendev.org/66924409:12
openstackgerritStephen Finucane proposed openstack/nova stable/stein: Fix spelling error in 'nova/api/validation/__init__.py'  https://review.opendev.org/66925109:14
*** jistr has joined #openstack-nova09:14
kashyapstephenfin: Hope I answered your question there: https://review.opendev.org/#/c/669129/09:15
kashyap(Not an entirely satisfactory one; but it's the truth :-))09:16
kashyapNew rev upcoming.09:16
*** rcernin has quit IRC09:16
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_PERF_VERSION  https://review.opendev.org/66912809:17
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: Remove support for Intel CMT `perf` events  https://review.opendev.org/66912909:17
openstackgerritKashyap Chamarthy proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN  https://review.opendev.org/66917609:17
openstackgerritMerged openstack/nova-specs master: Fix indentation in the "Secure Boot for KVM ... guests" spec  https://review.opendev.org/66928409:25
*** abhishekk is now known as abhishekk|interv09:34
*** jaosorior has quit IRC09:36
*** belmoreira has quit IRC09:52
*** stakeda has quit IRC09:57
*** ttsiouts has quit IRC10:08
*** ttsiouts has joined #openstack-nova10:08
*** abhishekk|interv is now known as abhishekk10:10
*** ttsiouts has quit IRC10:13
*** abhishekk has quit IRC10:25
*** brault has joined #openstack-nova10:32
*** luksky11 has quit IRC10:32
*** xek has joined #openstack-nova10:36
*** brault has quit IRC10:37
openstackgerritBoxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously  https://review.opendev.org/65196910:38
bauzasgibi: remember me, do we document in the API docs something like https://review.opendev.org/#/c/630723/4/nova/api/openstack/compute/migrate_server.py ?10:40
bauzasnot the API guide I mean10:40
bauzasI can read https://docs.openstack.org/neutron/latest/admin/config-qos-min-bw.html tho10:41
bauzasand https://developer.openstack.org/api-guide/compute/port_with_resource_request.html document what's missing10:41
bauzasbut I wonder if within the API docs, we said that too10:42
bauzasI mean api-ref10:42
bauzaslemme check actually10:42
bauzasnope, we don't (eg. https://developer.openstack.org/api-ref/compute/?expanded=resize-server-resize-action-detail#resize-server-resize-action )10:44
bauzasanyway, not a big deal, it's just for internal docs10:44
bauzasgibi: FWIW, ping me if you have any changes about ^ btw.10:44
*** boxiang has quit IRC10:50
*** maciejjozefczyk has quit IRC11:00
*** maciejjozefczyk has joined #openstack-nova11:01
*** luksky11 has joined #openstack-nova11:02
mdbooth>>> x.config_drive = False11:04
mdbooth>>> x.config_drive11:04
mdboothu'False'11:04
mdbooth>>> x.config_drive = True11:04
mdboothDon't you just love python?11:04
mdboothAnd that field has the wrong type, of course11:05
mdbooth>>> x.config_drive = False11:06
mdbooth>>> bool(x.config_drive)11:06
mdboothTrue11:06
*** whoami-rajat has quit IRC11:06
openstackgerritMerged openstack/nova master: api-ref: Fix a broken link  https://review.opendev.org/66924811:06
*** trident has quit IRC11:10
*** belmoreira has joined #openstack-nova11:13
*** trident has joined #openstack-nova11:14
*** brault has joined #openstack-nova11:14
*** cdent has joined #openstack-nova11:16
*** mrch_ has joined #openstack-nova11:25
*** ratailor has quit IRC11:28
*** tbachman has quit IRC11:37
*** brault has quit IRC11:46
*** brault has joined #openstack-nova11:47
stephenfinkashyap: Replied to https://review.opendev.org/#/c/669129/11:47
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN  https://review.opendev.org/66917611:48
*** ttsiouts has joined #openstack-nova11:49
gibibauzas: we have the limitation separately, not in the api-ref. I think it was intentional as it is expected to be a temporary limitation11:51
gibibauzas: the actual implementation of server move operation still needs my time. it is started up here https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-move-ops-with-qos-ports11:51
gibibauzas: but I was busy with https://review.opendev.org/#/c/637955/11:52
*** yaawang has quit IRC12:00
*** yaawang has joined #openstack-nova12:02
*** openstackgerrit has quit IRC12:04
*** tbachman has joined #openstack-nova12:05
*** whoami-rajat has joined #openstack-nova12:19
*** belmoreira has quit IRC12:21
*** xek has quit IRC12:26
*** xek_ has joined #openstack-nova12:26
kashyapstephenfin: Will respond (saddled with a couple off meetings); thanks for the thoughtful reply.12:29
*** psachin has joined #openstack-nova12:33
*** belmoreira has joined #openstack-nova12:34
*** openstackgerrit has joined #openstack-nova12:41
openstackgerritLee Yarwood proposed openstack/nova master: Get rid of args to RBDDriver.__init__()  https://review.opendev.org/66856412:41
*** eharney has joined #openstack-nova12:47
bauzasgibi: ack ok12:47
*** udesale has quit IRC12:51
*** udesale has joined #openstack-nova12:51
*** tesseract has quit IRC12:52
*** shilpasd has joined #openstack-nova12:52
*** belmoreira has quit IRC12:53
*** belmorei_ has joined #openstack-nova12:53
*** tesseract has joined #openstack-nova12:54
*** cdent has quit IRC13:03
openstackgerritBalazs Gibizer proposed openstack/nova master: Error out interrupted builds  https://review.opendev.org/66685713:05
*** jangutter has quit IRC13:13
*** jangutter has joined #openstack-nova13:14
*** mrch_ has quit IRC13:24
*** tbachman has quit IRC13:30
*** tbachman has joined #openstack-nova13:33
*** cdent has joined #openstack-nova13:33
*** mriedem has joined #openstack-nova13:36
*** trident has quit IRC13:41
*** Luzi has quit IRC13:50
*** tbachman has quit IRC13:53
*** ratailor has joined #openstack-nova14:00
*** tbachman has joined #openstack-nova14:03
*** belmorei_ has quit IRC14:09
*** cdent has quit IRC14:09
*** _alastor_ has joined #openstack-nova14:12
mriedemlyarwood: have you actually hit a recreate for this? https://review.opendev.org/#/q/I15a7c13edf78884ec223fd531a78a341106b41b814:12
lyarwoodmriedem: did you mean reason?14:15
mriedemno,14:15
mriedemwhen that was reported there was no recreate in the bug for it,14:15
mriedemand i was unable to recreate it in a functional test14:15
mriedemso i'm hesitant to backport that without knowing how to recreate it14:15
lyarwoodmriedem: ah reproducer14:15
mriedemyeah14:16
lyarwoodmriedem: no, that's fair it just looked like a trivial thing to change14:16
mriedemok i'd rather not backport it unless we have someone else reporting the problem14:16
lyarwoodmriedem: ack that's fine, I'll take a look next week at reproducing, if I can't I'll close14:16
mriedembtw it occurs to me now we maybe shouldn't have merged this https://review.opendev.org/#/c/663887/14:18
mriedemb/c i can't run fast8/pep8 on my bionic vm (py27 and py36) on rocky changes now14:19
mriedemstephenfin: ^14:19
stephenfinBecause bionic doesn't support py3.5?14:21
stephenfinIt's a tough one. Without it, I can't run the same on my Fedora 30 machine for the reasons in the commit message14:22
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for regression bug 1825537  https://review.opendev.org/66936114:23
openstackbug 1825537 in OpenStack Compute (nova) rocky "finish_resize failures incorrectly revert allocations" [Medium,Confirmed] https://launchpad.net/bugs/182553714:23
mriedemrocky doesn't support py37 at all though right?14:23
mriedemrocky tox.ini at least has py36 targets14:23
sean-k-mooneystephenfin: it use py3614:23
mriedemiow, if you're doing rocky stuff and only have 3.7, you're going to need another vm14:23
mriedembionic has 2.7 and 3.614:24
stephenfinIt doesn't, no, but the use of python3 means "give me any version of Python 3.x"14:24
sean-k-mooneystephenfin: we dont support py3.5 anymore14:24
sean-k-mooneyits not part of the train PTI14:24
stephenfinWe did in rocky though14:24
stephenfinThat's a stable-only patch14:24
sean-k-mooneyrocky proably yes14:24
stephenfinmriedem: Could we use basepython = python3.6 instead, assuming that's available on both Bionic and Xenial (or whatever 16.04 is?)14:25
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Revert "tox: Use basepython of python3.5"  https://review.opendev.org/66936214:25
mriedemstephenfin: in rocky i don't think so,14:25
mriedembecause rocky supported (in the gate and the tox default env list) 27 and 3514:26
mriedembut has a 36 target14:26
mriedemthere is no official 37 test targeting in rocky though14:26
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova-specs master: Boot a VM with an unaddressed port  https://review.opendev.org/64167014:27
mriedemso like i said, i think if you're doing rocky changes and only have 37, you'll just need another dev vm for working on those older branches (which is what i have to do for ocata changes at this point)14:27
mriedemi have a xenial vm just for doing ocata stuff now14:27
mriedemxenial less for the python thing but more b/c of native packages like postgresql14:27
*** zul has quit IRC14:29
*** ttsiouts has quit IRC14:30
*** ttsiouts has joined #openstack-nova14:31
*** ttsiouts has quit IRC14:36
*** ttsiouts has joined #openstack-nova14:37
*** belmoreira has joined #openstack-nova14:58
openstackgerritStephen Finucane proposed openstack/nova master: Remove nova-consoleauth  https://review.opendev.org/65296915:04
stephenfinlyarwood: The stable/stein variant of this has merged now. Could you tackle this one? https://review.opendev.org/#/c/667355/15:07
*** tesseract-RH has joined #openstack-nova15:08
*** tbachman has quit IRC15:08
*** xek_ has quit IRC15:09
*** tesseract has quit IRC15:10
*** rpittau is now known as rpittau|afk15:12
*** cdent has joined #openstack-nova15:13
*** tbachman has joined #openstack-nova15:14
dansmithsean-k-mooney: mriedem: Artom made several changes to the revert resize patch yesterday but didn't respond to comments and also rebased it so there's a lot of change.. did I miss any discussion about that?15:20
dansmithlooks like he removed the try..except around the signature change which was my deal, but otherwise I either need to eye-diff it or pull it down15:21
sean-k-mooneydansmith: he change it to the db lookup15:21
*** lpetrut has quit IRC15:21
sean-k-mooneyright15:21
sean-k-mooneyi havent looked  at it this mornig15:22
dansmithalright15:22
*** damien_r has quit IRC15:22
mriedemi was going to have to eye-diff anyway15:23
*** gyee has joined #openstack-nova15:28
*** tssurya has quit IRC15:32
sean-k-mooneyartom is on pto while he is flying home to canada today but if there are small things like mising test exctra ill be around for an hour or two to fix them15:34
sean-k-mooneyi think artom is back on monday but if not ill continue looking at this until he is back.15:34
*** belmoreira has quit IRC15:36
*** luksky11 has quit IRC15:38
*** helenafm has quit IRC15:42
*** shilpasd has quit IRC15:46
*** maciejjozefczyk has quit IRC15:48
*** tbachman has quit IRC15:51
*** tbachman has joined #openstack-nova15:54
dansmithsean-k-mooney: I don't see the "will wait for bind time events" string in the logs15:55
dansmithon the hybrid plug test in experimental15:55
dansmithmaybe that test is single-host which means we're not running any of this new stuff anyway?15:55
sean-k-mooneyit is15:56
*** ratailor_ has joined #openstack-nova15:56
*** ratailor_ has quit IRC15:56
sean-k-mooneyit is testign the resize to same host case15:56
dansmithah yeah I see the same host debug message instead15:56
sean-k-mooneyhttps://review.opendev.org/#/c/664442/14 is testing resize to different host15:56
*** ratailor has quit IRC15:56
dansmithah, right forgot that was on top and was thinking we'd get testing from mriedem's patch, I see15:57
sean-k-mooneyya this is the message form the dnm change http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_49_57_37176115:58
*** mdbooth_ has joined #openstack-nova15:58
dansmithyeah, but lower we receive and discard that event: http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_49_58_00338415:59
dansmithsee the "no event matching" above that15:59
sean-k-mooneyya i think this might be the souce node i need to check the other node15:59
sean-k-mooneyand confim this is the right vm15:59
dansmithwe shouldn't emit the "will wait..." message on the wrong node right?15:59
sean-k-mooneywe should not e might get thos events on both nodes16:00
dansmithreceive, but not waiting16:00
dansmithactually, that's the unplug event16:00
dansmithwe get the plug event shortly after that:16:00
dansmithhttp://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_50_00_98316416:01
*** mdbooth has quit IRC16:02
sean-k-mooney i dont kno if we ever actully wait for unplugged events do we16:02
*** mdbooth_ has quit IRC16:02
sean-k-mooneyit looks like libvirt died oh that vm breilfy http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/screen-n-cpu.txt.gz#_Jul_04_19_59_12_86259916:08
sean-k-mooneywhich is unrelated but annoying16:08
sean-k-mooneyactully that is part of the testing i think16:08
sean-k-mooneyi think we are killing it so we can do evacuate testing16:09
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: WIP: Drop source node allocations if finish_resize fails  https://review.opendev.org/66939316:15
mriedemthe nova-live-migration job, renamed to nova-multinode in 664442, yes does evacuate testing and kills libvirtd on the controller node16:17
sean-k-mooneyin any case we start waiting for the event here http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_04_19_49_48_754192 and we recive it here http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_04_19_49_51_90579216:19
sean-k-mooneythere is a vif unplugged and plugged event recrved just before we wait that im not sure where they came form16:20
sean-k-mooneyhttp://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_04_19_49_40_441961 and http://logs.openstack.org/42/664442/14/check/nova-multinode/7c532c4/logs/subnode-2/screen-n-cpu.txt.gz#_Jul_04_19_49_45_25862916:20
*** ttsiouts has quit IRC16:21
sean-k-mooneybut it does look like we do wait as expected16:21
*** ttsiouts has joined #openstack-nova16:21
*** jangutter has quit IRC16:22
*** ttsiouts has quit IRC16:26
*** udesale has quit IRC16:28
openstackgerritMatt Riedemann proposed openstack/nova stable/rocky: Drop source node allocations if finish_resize fails  https://review.opendev.org/66939316:31
mriedemlyarwood: dansmith: ^ this backport got a bit hairy because before stein we had to deal with non-migration-based allocations,16:31
mriedemso i've documentd the difference in the commit message and code, and added a functional test for the non-migration-based allocations legacy wrinkle16:31
mriedemjust fyi so it doesn't get fast-approved as a stable core16:31
*** cdent has quit IRC16:32
dansmithugh16:35
*** slaweq has quit IRC16:52
*** tesseract-RH has quit IRC17:01
*** factor has joined #openstack-nova17:03
*** whoami-rajat has quit IRC17:06
*** factor has quit IRC17:08
*** luksky11 has joined #openstack-nova17:14
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: [WIP] Allow to spawn VMs with port without IP address  https://review.opendev.org/66941117:18
redkriegI'm having an interesting issue with a large flavor not being deployable on a compute host but two flavors of half the size working fine.  the host has just enough disk and ram for the flavor, plus disk_allocation_ratio and ram_allocation_ratio are > 1.  Nova scheduler logs with debug enabled don't even show the host in question during filtering, so I think it's being removed earlier somehow.  I17:22
redkriegsee accurate information in the "Update host state from compute node" ComputeNode output just minutes before launching an instance.  Can anyone suggest a reason it wouldn't even be considered for filtering?17:22
*** ralonsoh has quit IRC17:23
redkriegI've narrowed it down to a ram issue by dropping my flavor's ram requirement to 30000 from 32768.  the logs show that the hypervisor is reporting 32141 but even 32000 doesn't work.  Is there some hard limit I don't know about that bypasses the allocation ratio settings?17:37
*** psachin has quit IRC17:42
*** factor has joined #openstack-nova17:48
*** ociuhandu_ has joined #openstack-nova17:54
*** slaweq has joined #openstack-nova17:55
*** ociuhandu_ has quit IRC17:55
*** ociuhandu has quit IRC17:57
openstackgerritCorey Bryant proposed openstack/nova master: Add Python 3 Train unit tests  https://review.opendev.org/66943717:58
openstackgerritCorey Bryant proposed openstack/os-vif master: Add Python 3 Train unit tests  https://review.opendev.org/66943818:00
openstackgerritCorey Bryant proposed openstack/python-novaclient master: Add Python 3 Train unit tests  https://review.opendev.org/66943918:01
*** slaweq has quit IRC18:05
mriedemredkrieg: which release? placement could be filtering it out - enable debug logging in the placement logs and you should see the allocation candidate filtering messages (on newer releases), there is the reserved_host_ram_mb config option on the compute that will be reporting reserved MEMORY_MB to placement18:08
mriedemwhich is part of the overall capacity calculation18:08
Nick_Amriedem - rocky, phys_ram=32141MB used_ram=0MB, reserved_host_memory_mb = 0 in nova.conf18:12
Nick_Awe found https://docs.openstack.org/arch-design/design-compute/design-compute-overcommit.html which explains why 32768 doesn't work, but we're not seeing why 32000 is also failing18:12
*** ociuhandu has joined #openstack-nova18:13
*** ociuhandu has quit IRC18:17
mriedemare you using the RamFilter or AggregateRamFilter?18:17
*** factor has quit IRC18:20
redkriegyes, we do have the AggregateRamFilter active.  I don't see this host being filtered by it though.18:21
openstackgerritMerged openstack/nova master: libvirt: Remove MIN_{QEMU,LIBVIRT}_LUKS_VERSION  https://review.opendev.org/66892418:25
openstackgerritMerged openstack/nova-specs master: Fix the cross_az_attach value in AZ-unshelved spec  https://review.opendev.org/66926118:31
mriedemthe reason i ask is because those filters return a "limit" from the scheduler which gets passed down to the compute which is part of the resource claim,18:34
*** factor has joined #openstack-nova18:34
mriedemand if the scheduler is passing down a limit for something that you don't expect, that might be the cause18:35
mriedemhttps://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#allocation-ratios18:35
mriedemiow, what you think and what placement is using might be different allocation ratios18:37
mriedemhopefully that doc gives some things to look at18:37
*** factor has quit IRC18:40
*** factor has joined #openstack-nova18:40
*** factor has quit IRC18:47
Nick_AIt actually looks like we're dealing with some sort of qemu/kernel bug... It successfully attempts a claim on compute if we set the flavor RAM less than the physical RAM, but libvirt errors at certain amounts with "qemu-system-x86_64: cannot set up guest memory 'pc.ram': Cannot allocate memory."18:48
*** tbachman has quit IRC18:56
dansmithNick_A: using anything that requires pinned memory? cpu pinning, huge pages, etc?19:15
dansmithalso, even if you have 32g of memory, you can't necessarily expect to land a 30g instance there if you have no swap and have a large enough footprint of other stuff19:16
*** factor has joined #openstack-nova19:16
dansmithbut, if you're getting to libvirt, then you're out of the realm of it being a nova thing19:17
Nick_Awe are doing cpu pinning on here - yeah, I think the only nova issue was the hard limit based on physical ram19:17
Nick_AI am curious why that's the case if you happen to know19:17
Nick_Awe have non-openstack centos libvirt nodes that are happy to create KVMs with over 32GB on the same hardware19:18
dansmithdo you also have a numa topology specified?19:18
dansmiththere's no 32g limit specifically of course19:18
Nick_A"Regardless of the overcommit ratio, an instance can not be placed on any physical node with fewer raw (pre-overcommit) resources than the instance flavor requires." Am I misunderstanding that?19:19
*** tbachman has joined #openstack-nova19:19
Nick_Are: numa - we're using openstack-ansible defaults, not sure19:19
Nick_AI got that from https://docs.openstack.org/arch-design/design-compute/design-compute-overcommit.html19:21
*** slaweq has joined #openstack-nova19:21
dansmithno, I'm just saying there's no specific 32g limit. nova's overcommit is for landing multiple instances on a node, however ill-advised19:21
openstackgerritMatt Riedemann proposed openstack/nova master: Remove old TODO about forced_host policy check  https://review.opendev.org/66947419:21
dansmitha single instance larger than physical memory is silly, so yeah, no support for it19:22
Nick_Aright, but there is a limit based on the physical RAM preventing a single instance from being allocated above that amount. I'm curious why it allows multiple instances to do so but not a single one.19:22
dansmithit allows it because it was designed that way for earlier cloud providers that did it for cost scale, but it's never a good idea for performance, obviously, as you're just using host disk for the overage19:23
dansmitha single instance that is larger than the host doesn't make sense, you might as well run a smaller guest and let it do its own swapping19:24
dansmithanyway, if you're doing cpu pinning you might be running up against something that requires page pinning which is getting you to the "cannot allocate memory" thing19:25
dansmithI doubt it's a kernel/qemu bug19:25
dansmithbest bet would be to ask around monday morning when more of the folks that are familiar with that stuff are around19:25
Nick_Athank you19:25
dansmithsean, sfinucan, etc19:25
*** wwriverrat has quit IRC19:28
openstackgerritCorey Bryant proposed openstack/os-resource-classes master: Add Python 3 Train unit tests  https://review.opendev.org/66947919:43
openstackgerritCorey Bryant proposed openstack/os-traits master: Add Python 3 Train unit tests  https://review.opendev.org/66948019:44
mriedembauzas: how is your audit command coming? it's been a couple of weeks and this came up in the ML today http://lists.openstack.org/pipermail/openstack-discuss/2019-July/007534.html20:15
openstackgerritMatt Riedemann proposed openstack/nova master: Remove Newton-era min compute checks for server create with device tags  https://review.opendev.org/66952320:35
mriedemdansmith: is https://review.opendev.org/#/c/667177/ just waiting for my eye-ball diff and if we're good, fix the thing you noted and then we can move forward?20:37
dansmithmriedem: yeah20:37
*** rdopiera has quit IRC21:13
mriedemok comments in there now21:16
mriedemalso - https://review.opendev.org/#/c/668631/ - it's a good thing we didn't do the interface change in the same patch...21:17
*** maciejjozefczyk has joined #openstack-nova21:17
*** ivve has quit IRC21:25
*** maciejjozefczyk has quit IRC21:25
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Init HostState.failed_builds  https://review.opendev.org/66953821:31
openstackgerritMatt Riedemann proposed openstack/nova master: Remove Rocky-era min compute trusted certs compat check  https://review.opendev.org/66953921:37
*** tbachman has quit IRC22:01
sean-k-mooneyNick_A: im not sure if this is the issue but i do not belive we allow vms to oversubsribe resouces against itself.22:02
sean-k-mooneyin placment the max_unit for MEMORY_MB should be set to the total capasity of the host22:02
sean-k-mooneyso we will not allow vms that request more ram then the host has to be schduled to a node22:03
sean-k-mooneythe same is true from ram and disk22:03
sean-k-mooneyi havent read the full scrollback but that is the first limitation that came to mind22:03
sean-k-mooneyreading back22:05
sean-k-mooneywe reserve 512MB of ram for the host by default22:05
openstackgerritMatt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing  https://review.opendev.org/64345122:08
openstackgerritMatt Riedemann proposed openstack/nova master: Refresh instance in MigrationTask.execute Exception handler  https://review.opendev.org/66901222:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing  https://review.opendev.org/66901322:08
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add confirm_snapshot_based_resize_at_source  https://review.opendev.org/63705822:08
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add ConfirmResizeTask  https://review.opendev.org/63707022:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method  https://review.opendev.org/63707522:08
openstackgerritMatt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API  https://review.opendev.org/63731622:08
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add revert_snapshot_based_resize_at_dest compute method  https://review.opendev.org/63763022:08
openstackgerritMatt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations  https://review.opendev.org/63945322:08
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add finish_revert_snapshot_based_resize_at_source compute method  https://review.opendev.org/63764722:08
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask  https://review.opendev.org/63804622:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method  https://review.opendev.org/63804722:08
openstackgerritMatt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API  https://review.opendev.org/63804822:08
openstackgerritMatt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server  https://review.opendev.org/63826822:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add archive_deleted_rows wrinkle to cross-cell functional test  https://review.opendev.org/65165022:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add CrossCellWeigher  https://review.opendev.org/61435322:08
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional test for anti-affinity cross-cell migration  https://review.opendev.org/66185922:08
openstackgerritMatt Riedemann proposed openstack/nova master: Support cross-cell moves in external_instance_event  https://review.opendev.org/65847822:08
Nick_Asean-k-mooney thanks22:10
sean-k-mooneyi would have to think about it some more but im guessing you are hiting one of the limit we have to make sure a single vm does not exceed the total amount of one of the resouce we have on the host22:11
*** jistr has quit IRC22:24
*** jistr has joined #openstack-nova22:24
Nick_Ayeah that's clearly why 32768 wasn't working. We were just surprised it had a limit of that sort when the RAM allocated to multiple smaller instances could still cross that threshold.22:30
*** luksky11 has quit IRC22:33
sean-k-mooneyi think the qemu error is proably a similar similar safte check22:36
*** dklyle has quit IRC22:36
sean-k-mooneythe os and qemu both need some ram to work properly. so there is some overhad per host that should not be allocated to a guest unless you have enough swap space22:37
sean-k-mooneybut even then i generally do not recommend ram over commit at all22:37
sean-k-mooneyit causes more issues the in sovles as the kernel OOM killer will reap the vms first if it need to allocate memory and it can swap the vm memory to make room22:38
*** tbachman has joined #openstack-nova22:39
*** tbachman_ has joined #openstack-nova22:40
*** tbachman has quit IRC22:43
*** tbachman_ is now known as tbachman22:43
*** mriedem has quit IRC22:47
openstackgerritMerged openstack/nova master: libvirt: Remove MIN_LIBVIRT_PERF_VERSION  https://review.opendev.org/66912822:54
openstackgerritMerged openstack/nova master: libvirt: vif: Remove MIN_LIBVIRT_MACVTAP_PASSTHROUGH_VLAN  https://review.opendev.org/66917623:00
*** sapd1 has quit IRC23:05

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