Monday, 2020-05-18

*** brinzhang has joined #openstack-nova00:08
*** tetsuro has joined #openstack-nova00:16
*** ociuhandu has joined #openstack-nova00:22
*** hongbin has quit IRC00:29
*** ociuhandu has quit IRC00:29
*** threestrands has joined #openstack-nova00:43
*** hongbin has joined #openstack-nova00:43
*** brinzhang_ has joined #openstack-nova00:57
*** brinzhang has quit IRC01:01
*** Liang__ has joined #openstack-nova01:02
*** brinzhang_ has quit IRC01:22
*** lbragstad has quit IRC01:48
*** xiaolin has quit IRC02:38
*** sapd__x has joined #openstack-nova03:03
*** Liang__ has quit IRC03:12
*** sapd__x has quit IRC03:31
*** sapd1 has joined #openstack-nova03:31
*** psachin has joined #openstack-nova03:39
*** spatel has quit IRC03:40
*** ociuhandu has joined #openstack-nova03:49
*** udesale has joined #openstack-nova03:58
*** ociuhandu has quit IRC03:59
*** ociuhandu has joined #openstack-nova04:01
*** ociuhandu has quit IRC04:06
*** ratailor has joined #openstack-nova04:17
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-nova04:33
openstackgerritWenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider  https://review.opendev.org/71916304:36
*** hongbin has quit IRC04:53
*** tetsuro has quit IRC05:04
*** tetsuro has joined #openstack-nova05:14
*** ociuhandu has joined #openstack-nova05:17
*** ociuhandu has quit IRC05:23
*** links has joined #openstack-nova05:47
*** dpawlik has joined #openstack-nova05:53
*** xiaolin has joined #openstack-nova05:55
*** dpawlik has quit IRC05:57
*** tetsuro has quit IRC06:03
*** maciejjozefczyk has joined #openstack-nova06:06
*** jsuchome has joined #openstack-nova06:10
*** dpawlik has joined #openstack-nova06:13
*** ccamacho has joined #openstack-nova06:13
*** damien_r has joined #openstack-nova06:29
*** damien_r has quit IRC06:36
*** jraju__ has joined #openstack-nova06:41
*** links has quit IRC06:45
*** ttsiouts has joined #openstack-nova06:46
*** ccstone has quit IRC06:50
*** eandersson has quit IRC06:50
*** eandersson has joined #openstack-nova06:50
*** ccstone has joined #openstack-nova06:50
openstackgerritHarshavardhan Metla proposed openstack/nova master: [Nova] Add reference to Placement installation guide  https://review.opendev.org/72693606:52
*** slaweq has joined #openstack-nova06:53
*** belmoreira has joined #openstack-nova06:58
*** larainema has joined #openstack-nova07:01
*** ttsiouts has quit IRC07:02
*** tesseract has joined #openstack-nova07:10
*** ttsiouts has joined #openstack-nova07:11
*** iurygregory has joined #openstack-nova07:17
openstackgerritWenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider  https://review.opendev.org/71916307:19
*** tosky has joined #openstack-nova07:22
*** jraju__ has quit IRC07:27
*** evrardjp has quit IRC07:28
*** rpittau|afk is now known as rpittau07:29
*** avolkov has joined #openstack-nova07:29
*** tetsuro has joined #openstack-nova07:30
*** evrardjp has joined #openstack-nova07:30
*** tetsuro has quit IRC07:32
bauzasgood morning Nova07:34
*** ralonsoh has joined #openstack-nova07:42
openstackgerritvinay harsha mitta proposed openstack/nova stable/queens: For Installing and Configure Controller Node for openSUSE and SUSE Linux Enterprise. Modified the Placement API Service Endpoint port from "8780" to "8778".  https://review.opendev.org/72876607:43
*** jraju__ has joined #openstack-nova07:52
*** tetsuro has joined #openstack-nova08:01
*** jraju__ has quit IRC08:08
gibigood morning08:09
*** threestrands has quit IRC08:13
*** jraju__ has joined #openstack-nova08:13
*** xek__ has joined #openstack-nova08:15
* kashyap waves08:17
*** Yumeng has joined #openstack-nova08:19
openstackgerritBalazs Gibizer proposed openstack/nova master: Reserve DISK_GB resource for the image cache  https://review.opendev.org/72843908:19
openstackgerritLiang Fang proposed openstack/nova-specs master: Support volume local cache  https://review.opendev.org/72877808:28
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1869050  https://review.opendev.org/72878108:36
openstackbug 1869050 in OpenStack Compute (nova) train "migration of anti-affinity server fails due to stale scheduler instance info" [Low,Triaged] https://launchpad.net/bugs/186905008:36
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Update scheduler instance info at confirm resize  https://review.opendev.org/72878208:36
*** jraju__ is now known as links08:42
openstackgerritJiri Suchomel proposed openstack/nova master: Bring back allowed_direct_url_schemes in support of RBD image download  https://review.opendev.org/72809508:43
openstackgerritHuaqiang Wang proposed openstack/nova master: test_hardware: code formating for multiline construct  https://review.opendev.org/72879308:51
*** ttsiouts has quit IRC09:02
*** martinkennelly has joined #openstack-nova09:04
openstackgerritBalazs Gibizer proposed openstack/nova stable/rocky: Clean up allocation if unshelve fails due to neutron  https://review.opendev.org/71319609:09
*** takamatsu has quit IRC09:10
*** vishalmanchanda has joined #openstack-nova09:13
*** jraju__ has joined #openstack-nova09:17
*** ttsiouts has joined #openstack-nova09:17
*** links has quit IRC09:17
*** klindgren_ has quit IRC09:21
*** klindgren has joined #openstack-nova09:21
*** ttsiouts has quit IRC09:33
*** tetsuro has quit IRC09:35
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Bump hacking min version to 3.0.1  https://review.opendev.org/72880309:48
gibipep8 is broken on stable/ussuri I'm trying ^^ to fix it09:49
gibielod, lyarwood, bauzas: ^^^09:49
bauzasoh shit09:50
bauzasok thanks for working on it09:50
bauzasthe flake8 story I guess ?09:50
openstackgerritvinay harsha mitta proposed openstack/nova stable/queens: For Installing and Configure Controller Node for openSUSE and SUSE Linux Enterprise. Modified the Placement API Service Endpoint port from "8780" to "8778".  https://review.opendev.org/72876609:50
gibibauzas: yes, hacking version09:50
elodgibi: thx, looking09:51
bauzasgibi: I see09:51
lyarwoodgibi: ack thanks was waiting to see some CI results this morning for that, I need to drop for a few hours, will review later today once I'm back if it's still open.09:52
gibithanks09:53
bauzasI just said +2, let's see whether it fixes09:54
elodI think 'hacking>=2.0,<3.1.0 would be better.09:56
elodso just keep the lower constraint and just raise the upper09:56
gibielod: 3.0.0 is broken09:57
gibifor us09:57
gibibut I guess some of the 2.x as well borken09:58
gibichecking ...09:58
gibihm09:59
gibielod: https://zuul.opendev.org/t/openstack/build/70853c9681d448f890da99bfe6ca310b/log/job-output.txt#54109:59
gibithis is the run of https://review.opendev.org/#/c/728781/09:59
gibifor some reason it picked up 3.1.0 from hacking10:00
elodweird10:00
elodactually I don't even know how it was working before, because as far as I know this should not work: 'hacking>=2.0<2.1' because of the missing ','10:02
gibibased on that run I expect that the change https://review.opendev.org/728803 will fail too10:02
bauzasthat's becoming a saga10:03
gibielod: that missing coma coma could explain why pip picked up hacking 3.1.010:03
elodso maybe it's enough to just add the comma, instead of the new upper constrain? let me check.10:06
*** rpittau is now known as rpittau|bbl10:06
gibielod: could be10:06
gibielod: let me push a parallel patch for that10:06
gibielod: if that works the I agree that we should merge the smaller change10:07
*** tetsuro has joined #openstack-nova10:08
elodgibi: +110:08
openstackgerritWenping Song proposed openstack/nova master: delete sub resource provider when delete resource provider  https://review.opendev.org/71916310:09
*** ttsiouts has joined #openstack-nova10:10
gibielod: with hacking 2.0.0 installed I still get flake8==3.8.110:13
gibielod: so the coma does not help10:14
gibielod: so we need the more complex bump10:15
*** ttsiouts has quit IRC10:15
*** tkajinam has quit IRC10:15
elodgibi: thanks, just tested locally and got the same result10:17
gibimeanwhile zuul is +1 on the backported fix https://review.opendev.org/#/c/72880310:18
openstackgerritNalini Varshney proposed openstack/nova master: Add migration to make key field type VARBINARY in aggregate_metadata table,  https://review.opendev.org/72552210:19
elod+W'd, so it's on the way10:23
gibielod, bauzas: thanks10:24
*** ttsiouts has joined #openstack-nova10:44
openstackgerritMerged openstack/nova stable/ussuri: Bump hacking min version to 3.0.1  https://review.opendev.org/72880310:52
*** ttsiouts has quit IRC10:56
*** ttsiouts has joined #openstack-nova10:56
*** ociuhandu has joined #openstack-nova11:28
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Reproduce bug 1869050  https://review.opendev.org/72878111:29
openstackbug 1869050 in OpenStack Compute (nova) ussuri "migration of anti-affinity server fails due to stale scheduler instance info" [Low,In progress] https://launchpad.net/bugs/1869050 - Assigned to Balazs Gibizer (balazs-gibizer)11:29
openstackgerritBalazs Gibizer proposed openstack/nova stable/ussuri: Update scheduler instance info at confirm resize  https://review.opendev.org/72878211:29
*** udesale_ has joined #openstack-nova11:40
*** ociuhandu has quit IRC11:40
*** udesale has quit IRC11:43
gibistephenfin: hi! I dont get your comment in https://review.opendev.org/#/c/725801/ . What do you mean by " I'm not actually sure if we can approve stuff rn"11:54
gibi?11:54
*** ociuhandu has joined #openstack-nova11:59
*** nweinber has joined #openstack-nova12:00
kashyapgibi: "rn" == right now12:01
kashyapProabably there are some other issues?12:01
gibihm, could be12:01
kashyapNot sure, I'm not him, but just guessing :-)12:01
*** tetsuro has quit IRC12:01
gibicould be that he hold off +W as we are in the middle of the Ussuri release12:02
gibiI mean we were12:02
kashyapstephenfin is one of the fine folks who freely uses two-letter acronyms :D12:02
kashyapgibi: Nod; I'll let him chime in...12:02
* kashyap bbiab12:02
*** ociuhandu has quit IRC12:08
*** ttsiouts has quit IRC12:09
*** rpittau|bbl is now known as rpittau12:12
*** sapd1 has quit IRC12:20
*** raildo has joined #openstack-nova12:24
openstackgerritNalini Varshney proposed openstack/nova master: Add migration to make key field type BINARY in aggregate_metadata table,  https://review.opendev.org/72552212:30
*** ttsiouts has joined #openstack-nova12:32
stephenfingibi: Ah, sorry, yes - I meant right now12:32
gibistephenfin: I think we can approve such re-propose now12:33
gibistephenfin: do you have any technical issues with it or it was just procedural?12:33
*** Luzi has joined #openstack-nova12:33
stephenfinJust procedural12:33
stephenfinCool, approved12:34
gibistephenfin: thanks!12:36
*** evrardjp has quit IRC12:36
openstackgerritMerged openstack/nova-specs master: Use PCPU and VCPU in one instance  https://review.opendev.org/72580112:42
openstackgerritAndrey Volkov proposed openstack/nova master: [DNM] Implementing graceful shutdown.  https://review.opendev.org/72886012:46
openstackgerritLee Yarwood proposed openstack/nova stable/ussuri: Silence amqp heartbeat warning  https://review.opendev.org/72805412:48
*** evrardjp has joined #openstack-nova12:49
*** ociuhandu has joined #openstack-nova12:53
*** ociuhandu has quit IRC12:58
*** martinkennelly has quit IRC13:01
*** ociuhandu has joined #openstack-nova13:03
*** eharney has joined #openstack-nova13:04
*** ociuhandu has quit IRC13:08
*** jhesketh has quit IRC13:15
*** ttsiouts has quit IRC13:15
*** JamesBenson has joined #openstack-nova13:16
*** lbragstad has joined #openstack-nova13:18
*** jhesketh has joined #openstack-nova13:22
lyarwoodhttps://review.opendev.org/#/c/728011/ - trival bugfix if anyone has the time this afternoon btw13:22
*** martinkennelly has joined #openstack-nova13:32
lyarwoodbauzas: https://review.opendev.org/728413 - you might be interested in this btw13:36
* bauzas looks13:36
lyarwoodhttps://bit.ly/2ThOLW3 for example13:37
bauzaslyarwood: excellent idea13:37
*** ratailor has quit IRC13:40
*** jhesketh has quit IRC13:41
*** Luzi has quit IRC13:43
*** ttsiouts has joined #openstack-nova13:50
*** ociuhandu has joined #openstack-nova13:52
*** ociuhandu has quit IRC13:56
*** spatel has joined #openstack-nova13:56
kashyaplyarwood: Hi, when you get a sec, I'm looking at the Q35 failures from the 'virt-preview' job13:59
kashyapFrom here: https://ca77fe97db31348aa486-4e7b2e69a2914bf326001f07747bad73.ssl.cf2.rackcdn.com/708701/7/check/devstack-platform-fedora-latest-virt-preview-q35/8011a4c/testr_results.html13:59
kashyapEspecially the test_snapshot_..._delete_online failure:14:00
*** ociuhandu has joined #openstack-nova14:00
kashyaptempest.lib.exceptions.TimeoutException: Request timed out14:00
kashyapDetails: volume f8764f61-35ee-4e1a-90e3-105c8536aa04 failed to reach available status (current in-use) within the required time (196 s).14:00
*** ttsiouts has quit IRC14:00
kashyaplyarwood: "Nothing to do with Q35" - is it?14:00
kashyapDo you know top off your head what is causing the in-use + time out thing?14:01
kashyapAlso, I recall you saying that you have a local Ubuntu repro - do you still have it?14:01
kashyap(If so, mind paste-binning it somewhere?)14:01
openstackgerritMerged openstack/nova master: tox: Integrate mypy  https://review.opendev.org/67620814:01
lyarwoodkashyap: iirc things were failing when we used -drive instead of -blockdev14:01
*** ttsiouts has joined #openstack-nova14:01
kashyaplyarwood: I've now lost the thread :-(  So if you have a prior + current failure link, I'm all eyes for it14:02
lyarwoodkashyap: lets rerun the job and debug things with fresh logs14:02
* lyarwood rechecks14:02
kashyapRight, I thought I did it just now ...14:02
kashyapIf I haven't, please do14:02
kashyapI just want 'something', 'anything' that is caused by Q35, that will _rouse_ me to chase it down to the root cause :D14:03
*** ccamacho has quit IRC14:05
openstackgerritLee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type  https://review.opendev.org/70870114:05
lyarwoodkashyap: ^ there you go14:06
kashyapA rebase?  Thx14:06
*** sapd1 has joined #openstack-nova14:08
*** sapd1_y has joined #openstack-nova14:09
*** ttsiouts has quit IRC14:11
*** ttsiouts has joined #openstack-nova14:12
*** sapd1_x has quit IRC14:13
*** mriedem has joined #openstack-nova14:19
*** KeithMnemonic has joined #openstack-nova14:25
*** ttsiouts has quit IRC14:26
*** mlavalle has joined #openstack-nova14:26
openstackgerritLee Yarwood proposed openstack/nova master: Add functional test for bug 1550919  https://review.opendev.org/63129414:31
openstackbug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood)14:31
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Don't delete disks on shared storage during evacuate  https://review.opendev.org/57884614:31
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Make the use of ImageBackendFixture optional within ServersTestBase  https://review.opendev.org/72891314:31
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Mock the connector within ServersTestBase  https://review.opendev.org/72891414:31
openstackgerritLee Yarwood proposed openstack/nova master: DNM / WIP libvirt: rework _LibvirtEvacuateTest to use ServersTestBase  https://review.opendev.org/72891514:31
*** ociuhandu has quit IRC14:43
*** ociuhandu has joined #openstack-nova14:45
*** psachin has quit IRC14:46
*** links has joined #openstack-nova14:47
*** jraju__ has quit IRC14:48
*** ociuhandu has quit IRC14:51
gibigmann, beekneemech, bauzas: we have to oslo topics (policy, and healthcheck) on the nova PTG etherpad. Is it OK to talk about those on the oslo side of the PTG? (Jun 1, 15:00 - 17:00 UTC)14:53
*** beekneemech is now known as bnemec14:53
bauzasI'm not opinionated about a bit for healthchecks :)14:54
bauzasbut sure we can do what you said14:55
gibibauzas: your name is on the etherpad for that topic hence my ping :)14:55
gmanngibi: policy stuff is ok but healthcheck things can be more nova things to decide first and then discuss with oslo.14:55
bauzasoh ok :shame:14:55
gmanngibi: bauzas i added bauzas as he was reviewing that patch :)14:56
gibigmann: could you start an ML thread about the healtheck topic? Let's see if we can create some common understanding before the PTG14:56
bnemecgibi: Should be fine. We don't have a ton of topics for Oslo so I expect we'll have time.14:56
gmanngibi: ok, sure, i have pending poc to do and then send it over ML.14:57
gibigmann: PoC is even better :)14:57
gibibnemec: thanks14:57
*** dklyle has joined #openstack-nova14:57
openstackgerritJiri Suchomel proposed openstack/nova-specs master: Add spec for downloading images via RBD  https://review.opendev.org/57280514:58
*** ociuhandu has joined #openstack-nova14:59
*** READ10 has joined #openstack-nova15:11
*** links has quit IRC15:16
*** ociuhandu has quit IRC15:25
*** dpawlik has quit IRC15:31
*** ociuhandu has joined #openstack-nova15:36
*** eharney has quit IRC15:36
*** gyee has joined #openstack-nova15:42
*** eharney has joined #openstack-nova16:01
*** rpittau is now known as rpittau|afk16:06
openstackgerritMerged openstack/nova-specs master: Add spec for downloading images via RBD  https://review.opendev.org/57280516:06
openstackgerritAndreas Jaeger proposed openstack/nova master: Add link to PDF document  https://review.opendev.org/72893916:17
*** ociuhandu_ has joined #openstack-nova16:18
openstackgerritAndreas Jaeger proposed openstack/python-novaclient master: Add link to PDF document  https://review.opendev.org/72894116:20
*** ociuhandu has quit IRC16:22
*** ociuhandu_ has quit IRC16:23
*** udesale_ has quit IRC16:41
*** avolkov has quit IRC16:49
*** mlavalle has quit IRC17:08
*** mlavalle has joined #openstack-nova17:09
openstackgerritMerged openstack/nova stable/ussuri: Reproduce bug 1869050  https://review.opendev.org/72878117:42
openstackbug 1869050 in OpenStack Compute (nova) ussuri "migration of anti-affinity server fails due to stale scheduler instance info" [Low,In progress] https://launchpad.net/bugs/1869050 - Assigned to Balazs Gibizer (balazs-gibizer)17:42
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove remaining '_LE' instances  https://review.opendev.org/72896317:51
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove remaining '_LW' instances  https://review.opendev.org/72896417:51
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove remaining '_LI' instances  https://review.opendev.org/72896517:51
openstackgerritStephen Finucane proposed openstack/nova master: trivial: Remove log translations  https://review.opendev.org/72896617:51
openstackgerritStephen Finucane proposed openstack/nova master: hacking: Modify checks for translated logs  https://review.opendev.org/72896717:51
openstackgerritStephen Finucane proposed openstack/nova master: Remove hooks  https://review.opendev.org/72896817:51
*** sapd1 has quit IRC17:53
*** ralonsoh has quit IRC17:54
openstackgerritGhanshyam Mann proposed openstack/python-novaclient stable/ussuri: DNM: testing  https://review.opendev.org/72897518:18
*** tesseract has quit IRC18:22
*** josephillips has joined #openstack-nova18:26
*** ociuhandu has joined #openstack-nova18:28
*** ociuhandu has quit IRC18:32
josephillipshi18:37
josephillipsim trying to perform a Host aggregate with some spec18:37
josephillipsi put on the flavor and work great18:37
josephillipsbut all another flavors can choose the server that is part of extra spec18:37
josephillipsthis is normal18:38
josephillips?18:38
sean-k-mooneyyou are trying to require that a specific flavor extra spec must be requested to have the schduler select a host in a given aggreate correct18:38
josephillipsyes18:39
josephillipsis workign with the flavor with the extra spec18:39
josephillipsmy problem is all the other flavors without the extra specs can choose the server that is on host aggregate18:39
sean-k-mooneyunfortunetly that AggregateInstanceExtraSpecsFilter does not support that usecase18:40
sean-k-mooneynor does plamement today18:40
josephillipsto be more specific18:40
josephillipsm2.flavor can choose m2.server only18:40
josephillipsat this point im right18:40
josephillipsbut using m1.flavor is choosing m1.server and m2.server too18:41
sean-k-mooneyyes18:41
josephillipsi want m1.flavor just choose m1 flavors18:41
josephillipsm1.servers18:41
sean-k-mooneyso i dont know if this filter still works but https://opendev.org/x/nfv-filters18:41
sean-k-mooneycontains a filter that will support the behavior you want18:41
sean-k-mooneyspecificaly the AggregateInstanceTypeFilter18:41
josephillipsbut using m1.flavor is choosing m1.server and m2.server too (im not looking this behaivor)18:42
openstackgerritAndreas Jaeger proposed openstack/nova master: Add link to PDF document  https://review.opendev.org/72893918:42
josephillipsim looking m1.flavor only choose m1 servers18:42
josephillipsand m2 choose m2 servers18:42
josephillipsto be more simple18:43
sean-k-mooneyjosephillips: so you could try using the AggregateTypeAffinityFilter18:43
sean-k-mooneythat will allow you to map flavor to aggreates by name18:43
sean-k-mooneythat does not look at the extra spec so you have to manually map each flavor to a host aggreate18:44
josephillipsand all another flavors without extra specs use the all servers wihtout m2 servers18:44
josephillipswithout extra specs18:44
sean-k-mooneyjosephillips: yes as i said the intree filter dont support that behavior18:44
*** AJaeger has joined #openstack-nova18:45
AJaegermelwitt: https://review.opendev.org/#/c/728941 is correct, test places PDF file differently than we do in production.18:45
* AJaeger has updated his commit message template - do you want me to update that for the change as well?18:45
sean-k-mooneyyou can use the AggregateTypeAffinityFilter to map flavor by name to speficic host aggreates or you can use the out of tree AggregateInstanceTypeFilter to get the ablity to require specific extra spec to be requested to land on a host aggreate18:45
melwittAJaeger: no it's ok, I will change my vote18:46
josephillipssean-k-mooney: ok so the correct way is create host-aggregates for all the nodes?18:46
josephillipsor use another filter18:47
sean-k-mooneyjosephillips: yes so if you can change your workflow slightly to be based on flavor name you dont need to create aggreates for all host. you just need to create aggretes for the hosts that need special handeling18:47
sean-k-mooneythe same is true for the our of tree filter18:48
AJaegerthanks, melwitt18:48
AJaegermelwitt: and thanks for double-checking!18:48
sean-k-mooneyyou just need aggreate for the special hosts18:48
*** AJaeger has left #openstack-nova18:49
josephillipsusing this filter sean-k-mooney right AggregateTypeAffinityFilter?18:50
sean-k-mooneyjosephillips: once hack you can do is to add a custom trait to all flavor with a specific value e.g. CUSTOM_AGGREATE=General or CUSTOM_AGGREATE=M218:50
sean-k-mooneyif you use the AggregateTypeAffinityFilter you do not need to modfiy all your flavors only the one that need specail handeling18:51
sean-k-mooneyusing AggregateTypeAffinityFilter you basicaly list what flavor may land on a give agggreate using aggreate metadata18:52
josephillipsok let me try with this filter18:52
josephillipsAggregateTypeAffinityFilter18:52
sean-k-mooneyif you use AggregateInstanceExtraSpecsFilter then you need to add  aggregate_instance_extra_specs:CUSTOM_AGGREATE=General to all your normal flavor and aggregate_instance_extra_specs:CUSTOM_AGGREATE=MY_SPECIAL_AGGREATE to your specil flavor18:54
*** belmoreira has quit IRC18:54
sean-k-mooneyso the AggregateInstanceExtraSpecsFilter requires you to modify all flavor to get teh behavior you want18:54
josephillipsand is requiered for the host aggregate have the extra filtes for ram allocation cpu allocation?18:57
josephillipsor just with the normal filters this functionality works too18:57
josephillips?18:57
sean-k-mooneyjosephillips: we do not support setting it via the aggreate anymore18:58
sean-k-mooneywe removed that a few release ago18:58
sean-k-mooneyjosephillips: http://lists.openstack.org/pipermail/openstack-discuss/2019-July/008118.html18:59
openstackgerritAndreas Jaeger proposed openstack/os-vif master: Switch to newer openstackdocstheme and reno versions  https://review.opendev.org/72861218:59
sean-k-mooneyjosephillips: actully i ment to link http://lists.openstack.org/pipermail/openstack-dev/2018-January/126283.html18:59
josephillipsgreat18:59
sean-k-mooneywe deprecated the Aggregate[Core|Ram|Disk]Filter in ocata and remvoed them in train i think19:00
sean-k-mooneyjosephillips: what release of openstack are you using19:00
josephillipstrain19:00
sean-k-mooneyjosephillips: ok in train you have two options. you can manage this via config file by setting the allcaotion ratios in the compute node config or via the api via placment19:01
sean-k-mooneyjosephillips:  this is the spec that cover why we made this change https://specs.openstack.org/openstack/nova-specs/specs/stein/implemented/initial-allocation-ratios.html let me see if i can get the docs link19:02
melwittnote that the --aggregate and --amend options were added to osc-placement cli to help with this https://docs.openstack.org/osc-placement/latest/cli/index.html#cmdoption-openstack-resource-provider-inventory-set-aggregate19:02
sean-k-mooneyjosephillips: https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html#scheduling-considerations19:03
sean-k-mooneymelwitt: the were added in stien right19:03
openstackgerritMerged openstack/nova master: compute: Allow snapshots to be created from PAUSED volume backed instances  https://review.opendev.org/72801119:03
sean-k-mooneyso it should be avilable in train19:03
melwittlooks like train https://docs.openstack.org/releasenotes/osc-placement/train.html#relnotes-1-7-0-stable-train19:04
openstackgerritMerged openstack/nova stable/ussuri: Update scheduler instance info at confirm resize  https://review.opendev.org/72878219:06
openstackgerritMerged openstack/nova master: compute: Remove snapshot quiesce tests for STOPPED and SUSPENDED instances  https://review.opendev.org/72801219:06
josephillipssean-k-mooney: looks like work with the filterAggregateTypeAffinityFilter19:17
josephillipsso for live migrations this host aggregate respect this?19:17
sean-k-mooneyyes it will19:17
josephillipsso i can move only on nodes that meets the host aggregate specs19:18
sean-k-mooneyi cant recall if we support --force on live migrtion if we do that might bypass it19:18
josephillipsi will try on my lab19:18
sean-k-mooneyso yes and no19:18
sean-k-mooneythe limitation of the AggregateTypeAffinityFilter is that if flavor X is the only flavor that you allow for host aggreate X the Flavor y can land in host aggreate X19:19
josephillipsnow im just have to check how ubuntu openstack is handling the ratio of memory or cpu if is on placement19:20
sean-k-mooneybut flavor X can still land on a host not in host aggreate X19:20
sean-k-mooneyjosephillips: that is not disto specfic19:20
sean-k-mooneyalthough the charms may or may not configure the plament aggreats for you i have not really looked at how they work in a few years19:21
openstackgerritElod Illes proposed openstack/nova stable/train: [train-only] Remove pycodestyle from test requirements  https://review.opendev.org/72900419:21
sean-k-mooneyjosephillips: i would expect that you can either configure it in the compute node cofnig per host using the juju chram for nova or you have to do it manually via placement after the fact19:22
josephillipssean-k-mooney: i lost in your explanation of the AggregateTypeAffinityFilter limitations19:22
josephillipsif i have the flavor m1 for the m1 servers they dont have any host aggregate19:23
josephillipsm2 flavors only for m2 servers with host aggregate19:23
josephillipsm2 flavors can land on m1 servers?19:23
sean-k-mooneyyes19:23
josephillipsunder what circustances?19:23
sean-k-mooneybut m1 instance cannot land on m2 servers19:23
sean-k-mooneyjosephillips: it depend on the weighers19:24
josephillipsso the correct way is be specific with nodes m1 and nodes m219:24
sean-k-mooneyif the weigher consider the m1 server to be a better choice because they have more free ram,disk ectra then it will select them19:24
sean-k-mooneyjosephillips: yes19:24
sean-k-mooneyif you want to force it to only land on the specifc set you need to create a general host aggreate with all the m1 host and an m2 host aggreate with just the m2 hosts19:25
sean-k-mooneythen in the genral host aggreate list all the flavor excpet the m2 flavor and in the m2 host aggreate only list the m2 flavor19:25
sean-k-mooneyjosephillips: tl;dr the AggregateTypeAffinityFilter prevent flavor that are not in the list form landing in the aggreate19:26
sean-k-mooneyjosephillips: it does not force flavor to land on an aggreate19:27
sean-k-mooneythree is one thing you could do since your on train19:28
sean-k-mooneyto make it slightly simpler.19:28
*** maciejjozefczyk has quit IRC19:28
sean-k-mooneyyou could create only the aggreate for the special m2 hosts and use teh AggregateTypeAffinityFilter as you are doing now19:29
sean-k-mooneybut you caould also add a custom trait to all those hosts in plamcnet and then add trait:CUSTOM_MY_TRAIT=required19:29
sean-k-mooneyto the flavor19:30
sean-k-mooneythe custom required trait will force plamcent to only slect the hosts where you applied trait19:30
sean-k-mooneyand the filter will prevent instnace that dont have the correct flavor form landing there19:30
sean-k-mooneyif you put those two things togeter it will do what you want19:30
sean-k-mooneyyour m2 instance will only land on the m2 host and the m1 instance cant land on the m2 hosts19:31
sean-k-mooneyjosephillips: im about to leave for the night but one question before i do. your are not trying to use this for tenant isolation are you?19:32
sean-k-mooneyjosephillips: this feature https://docs.openstack.org/nova/latest/reference/isolate-aggregates.html might be of use to you19:33
sean-k-mooneyjosephillips: as might https://docs.openstack.org/nova/latest/admin/aggregates.html#tenant-isolation-with-placement if you are interested in the teant isolation case19:35
sean-k-mooneythe isolate aggreates feature is what was descibing as the hybrid approch19:35
sean-k-mooneythe isolate-aggregates featuer which is new in train might be the only thing you require to achive your goal. it was intended for more or less the same usecase but im not sure as you did not clearly state what you were trying to achive.19:40
josephillipsno19:41
josephillipssean-k-mooney: im just looking use a second family of procesors and19:41
josephillipsdedicate some nodes for SaaS applicatios19:41
sean-k-mooneyhttps://docs.openstack.org/nova/latest/reference/isolate-aggregates.html shoudl allow you to do that19:42
sean-k-mooneyopenstack --os-placement-api-version 1.6 trait create CUSTOM_SAS_HOSTS19:42
sean-k-mooneythen you can add CUSTOM_SAS_HOSTS to each host that you want to use19:43
sean-k-mooneyfor sas applcaipton19:43
josephillipsand i add these meta to the flavor19:43
josephillips?19:43
josephillipsCUSTOM_SAS_HOSTS19:44
sean-k-mooneyin the flavor you set trait:CUSTOM_SAS_HOSTS=required19:44
sean-k-mooneyand you set the same in a host aggreate19:44
sean-k-mooneyopenstack --os-compute-api-version 2.53 aggregate set --property trait:CUSTOM_SAS_HOSTS=required MY_SAS_HOST_AGGREATE19:44
josephillipsso im still have to use host aggregate19:45
josephillips?19:45
sean-k-mooneyyes19:45
josephillipswithout any filter?19:45
sean-k-mooneyyes19:45
josephillipsmhmmm. i will try this19:45
sean-k-mooneyso you enable the featrure in the scudler config  scheduler.enable_isolated_aggregate_filtering=true19:45
josephillipsjust for ask and sorry for ask this but what is exactly the difference doing the filtering by nova or by placement19:46
sean-k-mooneyplacemnt is faster and the way we want to do this going forward19:47
sean-k-mooneyplamcent returns a subset of host which is then processed by the filters in nova19:47
sean-k-mooneyso if you do it in placment the filter have much less work to do19:47
josephillipsok19:47
josephillipsone las question what is the recommendation19:48
josephillipsif i will add more nodes with m2 processors19:48
josephillipsfor general use19:48
josephillipsi create another trait:M2_hosts19:48
josephillipsand the flavor19:48
josephillipsfor these hosts and thats it? is the correct way when you adding new servers with different specs19:48
sean-k-mooneyif you take the https://docs.openstack.org/nova/latest/reference/isolate-aggregates.html apporch you just dont add teh custom trait to the m2 hosts you want for general use19:49
sean-k-mooneyjosephillips: what i would suggest it take a few minutes to read teh spec https://specs.openstack.org/openstack/nova-specs/specs/train/implemented/placement-req-filter-forbidden-aggregates.html and understand teh feature an see if your probelme is adress by it. i think it is and then feel free to come back and ask some quetions about it after you play awound with it a bit in a day or two19:51
josephillipsthank you so much19:52
sean-k-mooneythere are a few way with verinly level of complexity to achive what you want and different level of operation overhead so really you need to make a value judgement on what is best for you19:52
josephillipsyou19:52
josephillipsyou have been very helpful sean-k-mooney  at least now i have ideas of how to solve it19:53
sean-k-mooneyglad i could help o/19:53
openstackgerritElod Illes proposed openstack/nova stable/train: [train-only] Cap pycodestyle to < 2.6.0  https://review.opendev.org/72900420:06
*** nweinber has quit IRC20:17
*** jsuchome has quit IRC20:22
openstackgerritGage Hugo proposed openstack/nova stable/train: DNM - testing numa image  https://review.opendev.org/72903020:26
*** READ10 has quit IRC20:34
openstackgerritmelanie witt proposed openstack/nova master: Add packages required for pdf-docs run to bindep.txt  https://review.opendev.org/72789820:42
*** efried has quit IRC20:46
*** francoisp has quit IRC20:52
*** efried has joined #openstack-nova20:53
*** raildo has quit IRC20:56
*** slaweq has quit IRC21:05
openstackgerritMerged openstack/nova master: Make quotas respect instance_list_per_project_cells  https://review.opendev.org/72857521:14
*** xek__ has quit IRC21:16
*** slaweq has joined #openstack-nova21:16
*** slaweq has quit IRC21:21
*** martinkennelly has quit IRC21:26
*** slaweq has joined #openstack-nova21:28
openstackgerritMerged openstack/nova master: Add link to PDF document  https://review.opendev.org/72893921:34
*** JamesBenson has quit IRC21:46
*** threestrands has joined #openstack-nova22:06
*** spatel has quit IRC22:07
*** JamesBenson has joined #openstack-nova22:24
*** slaweq has quit IRC22:51
*** tkajinam has joined #openstack-nova22:53
*** slaweq has joined #openstack-nova22:58
*** mriedem has left #openstack-nova22:59
*** vishalmanchanda has quit IRC23:22
*** tosky has quit IRC23:39
*** spatel has joined #openstack-nova23:41
*** tetsuro has joined #openstack-nova23:52
*** amotoki has quit IRC23:53
*** amotoki has joined #openstack-nova23:53

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!