12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409141014111412141314141415141614171418141914201421142214231424142514261427142814291430143114321433143414351436143714381439144014411442144314441445144614471448144914501451145214531454145514561457145814591460146114621463146414651466146714681469147014711472147314741475147614771478147914801481148214831484148514861487148814891490149114921493149414951496149714981499150015011502150315041505150615071508150915101511151215131514151515161517151815191520152115221523152415251526152715281529153015311532153315341535153615371538153915401541154215431544154515461547154815491550155115521553155415551556155715581559156015611562156315641565156615671568156915701571157215731574157515761577157815791580158115821583158415851586158715881589159015911592159315941595159615971598159916001601160216031604160516061607160816091610161116121613161416151616161716181619162016211622162316241625162616271628162916301631163216331634163516361637163816391640164116421643164416451646164716481649165016511652165316541655165616571658165916601661166216631664166516661667166816691670167116721673167416751676167716781679168016811682168316841685168616871688168916901691169216931694169516961697169816991700170117021703170417051706170717081709171017111712171317141715171617171718171917201721172217231724172517261727172817291730173117321733173417351736173717381739174017411742174317441745174617471748174917501751175217531754175517561757175817591760176117621763176417651766176717681769177017711772177317741775177617771778177917801781178217831784178517861787178817891790179117921793179417951796179717981799180018011802180318041805180618071808180918101811181218131814181518161817181818191820182118221823182418251826182718281829183018311832183318341835183618371838183918401841184218431844184518461847184818491850185118521853185418551856185718581859186018611862186318641865186618671868186918701871187218731874187518761877187818791880188118821883188418851886188718881889189018911892189318941895189618971898189919001901190219031904190519061907190819091910191119121913191419151916191719181919192019211922192319241925192619271928192919301931193219331934193519361937193819391940194119421943194419451946194719481949195019511952195319541955195619571958195919601961196219631964196519661967196819691970197119721973197419751976197719781979198019811982198319841985198619871988198919901991199219931994199519961997199819992000200120022003200420052006200720082009201020112012201320142015201620172018201920202021202220232024202520262027202820292030203120322033203420352036203720382039204020412042204320442045204620472048204920502051205220532054205520562057205820592060206120622063206420652066206720682069207020712072207320742075207620772078207920802081208220832084208520862087208820892090209120922093209420952096209720982099210021012102210321042105210621072108210921102111211221132114211521162117211821192120212121222123212421252126212721282129213021312132213321342135213621372138213921402141214221432144214521462147214821492150215121522153215421552156215721582159216021612162216321642165216621672168216921702171217221732174217521762177217821792180218121822183218421852186218721882189219021912192219321942195219621972198219922002201220222032204220522062207220822092210221122122213221422152216221722182219222022212222222322242225222622272228222922302231223222332234223522362237223822392240224122422243224422452246224722482249225022512252225322542255225622572258225922602261226222632264226522662267226822692270227122722273227422752276227722782279228022812282228322842285228622872288228922902291229222932294229522962297229822992300230123022303230423052306230723082309231023112312231323142315231623172318231923202321232223232324232523262327232823292330233123322333233423352336233723382339234023412342234323442345234623472348234923502351235223532354235523562357235823592360236123622363236423652366236723682369237023712372237323742375237623772378237923802381238223832384238523862387238823892390239123922393239423952396239723982399240024012402240324042405240624072408240924102411241224132414241524162417241824192420242124222423242424252426242724282429243024312432243324342435243624372438243924402441244224432444244524462447244824492450245124522453245424552456245724582459246024612462246324642465246624672468246924702471247224732474247524762477247824792480248124822483248424852486248724882489249024912492249324942495249624972498249925002501250225032504250525062507250825092510251125122513251425152516251725182519252025212522252325242525252625272528252925302531253225332534253525362537253825392540254125422543254425452546254725482549255025512552255325542555255625572558255925602561256225632564256525662567256825692570257125722573257425752576257725782579258025812582258325842585258625872588258925902591259225932594259525962597259825992600260126022603260426052606260726082609261026112612261326142615261626172618261926202621262226232624262526262627262826292630263126322633263426352636263726382639264026412642264326442645264626472648264926502651265226532654265526562657265826592660266126622663266426652666266726682669267026712672267326742675267626772678267926802681268226832684268526862687268826892690269126922693269426952696269726982699270027012702270327042705270627072708270927102711271227132714271527162717271827192720272127222723272427252726272727282729273027312732273327342735273627372738273927402741274227432744274527462747274827492750275127522753275427552756275727582759276027612762276327642765276627672768276927702771277227732774277527762777277827792780278127822783278427852786278727882789279027912792279327942795279627972798279928002801280228032804280528062807280828092810281128122813281428152816281728182819282028212822282328242825282628272828282928302831283228332834283528362837283828392840284128422843284428452846284728482849285028512852285328542855285628572858285928602861286228632864286528662867286828692870287128722873287428752876287728782879288028812882288328842885288628872888288928902891289228932894289528962897289828992900290129022903290429052906290729082909291029112912291329142915291629172918291929202921292229232924292529262927292829292930293129322933293429352936293729382939294029412942294329442945294629472948294929502951295229532954295529562957295829592960296129622963296429652966296729682969297029712972297329742975297629772978297929802981298229832984298529862987298829892990299129922993299429952996299729982999300030013002300330043005300630073008300930103011301230133014301530163017301830193020302130223023302430253026302730283029303030313032303330343035303630373038303930403041304230433044304530463047304830493050305130523053305430553056305730583059306030613062306330643065306630673068306930703071307230733074307530763077307830793080308130823083308430853086308730883089309030913092309330943095309630973098309931003101310231033104310531063107310831093110311131123113311431153116311731183119312031213122312331243125312631273128312931303131313231333134313531363137313831393140314131423143314431453146314731483149315031513152315331543155315631573158315931603161316231633164316531663167316831693170317131723173317431753176317731783179318031813182318331843185318631873188318931903191319231933194319531963197319831993200320132023203320432053206320732083209321032113212321332143215321632173218321932203221322232233224322532263227322832293230323132323233323432353236323732383239324032413242324332443245324632473248324932503251325232533254325532563257325832593260326132623263326432653266326732683269327032713272327332743275327632773278327932803281328232833284328532863287328832893290329132923293329432953296329732983299330033013302330333043305330633073308330933103311331233133314331533163317331833193320332133223323332433253326332733283329333033313332333333343335333633373338333933403341334233433344334533463347334833493350335133523353335433553356335733583359336033613362336333643365336633673368336933703371337233733374337533763377337833793380338133823383338433853386338733883389339033913392339333943395339633973398339934003401340234033404340534063407340834093410341134123413341434153416341734183419342034213422342334243425342634273428342934303431343234333434343534363437343834393440344134423443344434453446344734483449345034513452345334543455345634573458345934603461346234633464346534663467346834693470347134723473347434753476347734783479348034813482348334843485348634873488348934903491349234933494349534963497349834993500350135023503350435053506350735083509351035113512351335143515351635173518351935203521352235233524352535263527352835293530353135323533353435353536353735383539354035413542354335443545354635473548354935503551355235533554355535563557355835593560356135623563356435653566356735683569357035713572357335743575357635773578357935803581358235833584358535863587358835893590359135923593359435953596359735983599360036013602360336043605360636073608360936103611361236133614361536163617361836193620362136223623362436253626362736283629363036313632363336343635363636373638363936403641364236433644364536463647364836493650365136523653365436553656365736583659366036613662366336643665366636673668366936703671367236733674367536763677367836793680368136823683368436853686368736883689369036913692369336943695369636973698369937003701370237033704370537063707370837093710371137123713371437153716371737183719372037213722372337243725372637273728372937303731373237333734373537363737373837393740374137423743374437453746374737483749375037513752375337543755375637573758375937603761376237633764376537663767376837693770377137723773377437753776377737783779378037813782378337843785378637873788378937903791379237933794379537963797379837993800380138023803380438053806380738083809381038113812381338143815381638173818381938203821382238233824382538263827382838293830383138323833383438353836383738383839384038413842384338443845384638473848384938503851385238533854385538563857385838593860386138623863386438653866386738683869387038713872387338743875387638773878387938803881388238833884388538863887388838893890389138923893389438953896389738983899390039013902390339043905390639073908390939103911391239133914391539163917391839193920392139223923392439253926392739283929393039313932393339343935393639373938393939403941394239433944394539463947394839493950395139523953395439553956395739583959396039613962396339643965396639673968396939703971397239733974397539763977397839793980398139823983398439853986398739883989399039913992399339943995399639973998399940004001400240034004400540064007400840094010401140124013401440154016401740184019402040214022402340244025402640274028402940304031403240334034403540364037403840394040404140424043404440454046404740484049405040514052405340544055405640574058405940604061406240634064406540664067406840694070407140724073407440754076407740784079408040814082408340844085408640874088408940904091409240934094409540964097409840994100410141024103410441054106410741084109411041114112411341144115411641174118411941204121412241234124412541264127412841294130413141324133413441354136413741384139414041414142414341444145414641474148414941504151415241534154415541564157415841594160416141624163416441654166416741684169417041714172417341744175417641774178417941804181418241834184418541864187418841894190419141924193419441954196419741984199420042014202420342044205420642074208420942104211421242134214421542164217421842194220422142224223422442254226422742284229423042314232423342344235423642374238423942404241424242434244424542464247424842494250425142524253425442554256425742584259426042614262426342644265426642674268426942704271427242734274427542764277427842794280428142824283428442854286428742884289429042914292429342944295429642974298429943004301430243034304430543064307430843094310431143124313431443154316431743184319432043214322432343244325432643274328432943304331433243334334433543364337433843394340434143424343434443454346434743484349435043514352435343544355435643574358435943604361436243634364436543664367436843694370437143724373437443754376437743784379438043814382438343844385438643874388438943904391439243934394439543964397439843994400440144024403440444054406440744084409441044114412441344144415441644174418441944204421442244234424442544264427442844294430443144324433443444354436443744384439444044414442444344444445444644474448444944504451445244534454445544564457445844594460446144624463446444654466446744684469447044714472447344744475447644774478447944804481448244834484448544864487448844894490449144924493449444954496449744984499450045014502450345044505450645074508450945104511451245134514451545164517451845194520452145224523452445254526452745284529453045314532453345344535453645374538453945404541454245434544454545464547454845494550455145524553455445554556455745584559456045614562456345644565456645674568456945704571457245734574457545764577457845794580458145824583458445854586458745884589459045914592459345944595459645974598459946004601460246034604460546064607460846094610461146124613461446154616461746184619462046214622462346244625462646274628462946304631463246334634463546364637463846394640464146424643464446454646464746484649465046514652465346544655465646574658465946604661466246634664466546664667466846694670467146724673467446754676467746784679468046814682468346844685468646874688468946904691469246934694469546964697469846994700470147024703470447054706470747084709471047114712471347144715471647174718471947204721472247234724472547264727472847294730473147324733473447354736473747384739474047414742474347444745474647474748474947504751475247534754475547564757475847594760476147624763476447654766476747684769477047714772477347744775477647774778477947804781478247834784478547864787478847894790479147924793479447954796479747984799480048014802480348044805480648074808480948104811481248134814481548164817481848194820482148224823482448254826482748284829483048314832483348344835483648374838483948404841484248434844484548464847484848494850485148524853485448554856485748584859486048614862486348644865486648674868486948704871487248734874487548764877487848794880488148824883488448854886488748884889489048914892489348944895489648974898489949004901490249034904490549064907490849094910491149124913491449154916491749184919492049214922492349244925492649274928492949304931493249334934493549364937493849394940494149424943494449454946494749484949495049514952495349544955495649574958495949604961496249634964496549664967496849694970497149724973497449754976497749784979498049814982498349844985498649874988498949904991499249934994499549964997499849995000500150025003500450055006 |
- /*
- Copyright 2014 The Kubernetes Authors.
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
- http://www.apache.org/licenses/LICENSE-2.0
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
- package predicates
- import (
- "fmt"
- "os"
- "reflect"
- "strconv"
- "strings"
- "testing"
- "k8s.io/api/core/v1"
- storagev1 "k8s.io/api/storage/v1"
- "k8s.io/apimachinery/pkg/api/resource"
- metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
- "k8s.io/apimachinery/pkg/util/sets"
- v1helper "k8s.io/kubernetes/pkg/apis/core/v1/helper"
- schedulerapi "k8s.io/kubernetes/pkg/scheduler/api"
- schedulernodeinfo "k8s.io/kubernetes/pkg/scheduler/nodeinfo"
- schedulertesting "k8s.io/kubernetes/pkg/scheduler/testing"
- )
- var (
- extendedResourceA = v1.ResourceName("example.com/aaa")
- extendedResourceB = v1.ResourceName("example.com/bbb")
- kubernetesIOResourceA = v1.ResourceName("kubernetes.io/something")
- kubernetesIOResourceB = v1.ResourceName("subdomain.kubernetes.io/something")
- hugePageResourceA = v1helper.HugePageResourceName(resource.MustParse("2Mi"))
- )
- func makeResources(milliCPU, memory, pods, extendedA, storage, hugePageA int64) v1.NodeResources {
- return v1.NodeResources{
- Capacity: v1.ResourceList{
- v1.ResourceCPU: *resource.NewMilliQuantity(milliCPU, resource.DecimalSI),
- v1.ResourceMemory: *resource.NewQuantity(memory, resource.BinarySI),
- v1.ResourcePods: *resource.NewQuantity(pods, resource.DecimalSI),
- extendedResourceA: *resource.NewQuantity(extendedA, resource.DecimalSI),
- v1.ResourceEphemeralStorage: *resource.NewQuantity(storage, resource.BinarySI),
- hugePageResourceA: *resource.NewQuantity(hugePageA, resource.BinarySI),
- },
- }
- }
- func makeAllocatableResources(milliCPU, memory, pods, extendedA, storage, hugePageA int64) v1.ResourceList {
- return v1.ResourceList{
- v1.ResourceCPU: *resource.NewMilliQuantity(milliCPU, resource.DecimalSI),
- v1.ResourceMemory: *resource.NewQuantity(memory, resource.BinarySI),
- v1.ResourcePods: *resource.NewQuantity(pods, resource.DecimalSI),
- extendedResourceA: *resource.NewQuantity(extendedA, resource.DecimalSI),
- v1.ResourceEphemeralStorage: *resource.NewQuantity(storage, resource.BinarySI),
- hugePageResourceA: *resource.NewQuantity(hugePageA, resource.BinarySI),
- }
- }
- func newResourcePod(usage ...schedulernodeinfo.Resource) *v1.Pod {
- containers := []v1.Container{}
- for _, req := range usage {
- containers = append(containers, v1.Container{
- Resources: v1.ResourceRequirements{Requests: req.ResourceList()},
- })
- }
- return &v1.Pod{
- Spec: v1.PodSpec{
- Containers: containers,
- },
- }
- }
- func newResourceInitPod(pod *v1.Pod, usage ...schedulernodeinfo.Resource) *v1.Pod {
- pod.Spec.InitContainers = newResourcePod(usage...).Spec.Containers
- return pod
- }
- func GetPredicateMetadata(p *v1.Pod, nodeInfo map[string]*schedulernodeinfo.NodeInfo) PredicateMetadata {
- pm := PredicateMetadataFactory{schedulertesting.FakePodLister{p}}
- return pm.GetMetadata(p, nodeInfo)
- }
- func TestPodFitsResources(t *testing.T) {
- enoughPodsTests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- reasons []PredicateFailureReason
- ignoredExtendedResources sets.String
- }{
- {
- pod: &v1.Pod{},
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 10, Memory: 20})),
- fits: true,
- name: "no resources requested always fits",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 10, Memory: 20})),
- fits: false,
- name: "too many resources fails",
- reasons: []PredicateFailureReason{
- NewInsufficientResourceError(v1.ResourceCPU, 1, 10, 10),
- NewInsufficientResourceError(v1.ResourceMemory, 1, 20, 20),
- },
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 3, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 8, Memory: 19})),
- fits: false,
- name: "too many resources fails due to init container cpu",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceCPU, 3, 8, 10)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 3, Memory: 1}, schedulernodeinfo.Resource{MilliCPU: 2, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 8, Memory: 19})),
- fits: false,
- name: "too many resources fails due to highest init container cpu",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceCPU, 3, 8, 10)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 1, Memory: 3}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 19})),
- fits: false,
- name: "too many resources fails due to init container memory",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceMemory, 3, 19, 20)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 1, Memory: 3}, schedulernodeinfo.Resource{MilliCPU: 1, Memory: 2}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 19})),
- fits: false,
- name: "too many resources fails due to highest init container memory",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceMemory, 3, 19, 20)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 19})),
- fits: true,
- name: "init container fits because it's the max, not sum, of containers and init containers",
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}, schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 19})),
- fits: true,
- name: "multiple init containers fit because it's the max, not sum, of containers and init containers",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 5})),
- fits: true,
- name: "both resources fit",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 2, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 5})),
- fits: false,
- name: "one resource memory fits",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceCPU, 2, 9, 10)},
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 2}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- fits: false,
- name: "one resource cpu fits",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourceMemory, 2, 19, 20)},
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- fits: true,
- name: "equal edge case",
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 4, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 5, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- fits: true,
- name: "equal edge case for init container",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(newResourcePod(schedulernodeinfo.Resource{})),
- fits: true,
- name: "extended resource fits",
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}), schedulernodeinfo.Resource{ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(newResourcePod(schedulernodeinfo.Resource{})),
- fits: true,
- name: "extended resource fits for init container",
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 0}})),
- fits: false,
- name: "extended resource capacity enforced",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 10, 0, 5)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 0}})),
- fits: false,
- name: "extended resource capacity enforced for init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 10, 0, 5)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 5}})),
- fits: false,
- name: "extended resource allocatable enforced",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 1, 5, 5)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 5}})),
- fits: false,
- name: "extended resource allocatable enforced for init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 1, 5, 5)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 3}},
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 3}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 2}})),
- fits: false,
- name: "extended resource allocatable enforced for multiple containers",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 6, 2, 5)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 3}},
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 3}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 2}})),
- fits: true,
- name: "extended resource allocatable admits multiple init containers",
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 6}},
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 3}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{extendedResourceA: 2}})),
- fits: false,
- name: "extended resource allocatable enforced for multiple init containers",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceA, 6, 2, 5)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceB: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0})),
- fits: false,
- name: "extended resource allocatable enforced for unknown resource",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceB, 1, 0, 0)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceB: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0})),
- fits: false,
- name: "extended resource allocatable enforced for unknown resource for init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(extendedResourceB, 1, 0, 0)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{kubernetesIOResourceA: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0})),
- fits: false,
- name: "kubernetes.io resource capacity enforced",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(kubernetesIOResourceA, 10, 0, 0)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{kubernetesIOResourceB: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0})),
- fits: false,
- name: "kubernetes.io resource capacity enforced for init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(kubernetesIOResourceB, 10, 0, 0)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 0}})),
- fits: false,
- name: "hugepages resource capacity enforced",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(hugePageResourceA, 10, 0, 5)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{}),
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 10}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 0}})),
- fits: false,
- name: "hugepages resource capacity enforced for init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(hugePageResourceA, 10, 0, 5)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 3}},
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 3}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0, ScalarResources: map[v1.ResourceName]int64{hugePageResourceA: 2}})),
- fits: false,
- name: "hugepages resource allocatable enforced for multiple containers",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(hugePageResourceA, 6, 2, 5)},
- },
- {
- pod: newResourcePod(
- schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1, ScalarResources: map[v1.ResourceName]int64{extendedResourceB: 1}}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 0, Memory: 0})),
- fits: true,
- ignoredExtendedResources: sets.NewString(string(extendedResourceB)),
- name: "skip checking ignored extended resource",
- },
- }
- for _, test := range enoughPodsTests {
- t.Run(test.name, func(t *testing.T) {
- node := v1.Node{Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 5, 20, 5).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 5, 20, 5)}}
- test.nodeInfo.SetNode(&node)
- RegisterPredicateMetadataProducerWithExtendedResourceOptions(test.ignoredExtendedResources)
- meta := GetPredicateMetadata(test.pod, nil)
- fits, reasons, err := PodFitsResources(test.pod, meta, test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, test.reasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, test.reasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- notEnoughPodsTests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- reasons []PredicateFailureReason
- }{
- {
- pod: &v1.Pod{},
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 10, Memory: 20})),
- fits: false,
- name: "even without specified resources predicate fails when there's no space for additional pod",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourcePods, 1, 1, 1)},
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 5})),
- fits: false,
- name: "even if both resources fit predicate fails when there's no space for additional pod",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourcePods, 1, 1, 1)},
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- fits: false,
- name: "even for equal edge case predicate fails when there's no space for additional pod",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourcePods, 1, 1, 1)},
- },
- {
- pod: newResourceInitPod(newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 1}), schedulernodeinfo.Resource{MilliCPU: 5, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- fits: false,
- name: "even for equal edge case predicate fails when there's no space for additional pod due to init container",
- reasons: []PredicateFailureReason{NewInsufficientResourceError(v1.ResourcePods, 1, 1, 1)},
- },
- }
- for _, test := range notEnoughPodsTests {
- t.Run(test.name, func(t *testing.T) {
- node := v1.Node{Status: v1.NodeStatus{Capacity: v1.ResourceList{}, Allocatable: makeAllocatableResources(10, 20, 1, 0, 0, 0)}}
- test.nodeInfo.SetNode(&node)
- fits, reasons, err := PodFitsResources(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, test.reasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, test.reasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- storagePodsTests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- reasons []PredicateFailureReason
- }{
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 10, Memory: 10})),
- fits: false,
- name: "due to container scratch disk",
- reasons: []PredicateFailureReason{
- NewInsufficientResourceError(v1.ResourceCPU, 1, 10, 10),
- },
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 1, Memory: 1}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 2, Memory: 10})),
- fits: true,
- name: "pod fit",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{EphemeralStorage: 25}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 2, Memory: 2})),
- fits: false,
- name: "storage ephemeral local storage request exceeds allocatable",
- reasons: []PredicateFailureReason{
- NewInsufficientResourceError(v1.ResourceEphemeralStorage, 25, 0, 20),
- },
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{EphemeralStorage: 10}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 2, Memory: 2})),
- fits: true,
- name: "pod fits",
- },
- }
- for _, test := range storagePodsTests {
- t.Run(test.name, func(t *testing.T) {
- node := v1.Node{Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 5, 20, 5).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 5, 20, 5)}}
- test.nodeInfo.SetNode(&node)
- fits, reasons, err := PodFitsResources(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, test.reasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, test.reasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestPodFitsHost(t *testing.T) {
- tests := []struct {
- pod *v1.Pod
- node *v1.Node
- fits bool
- name string
- }{
- {
- pod: &v1.Pod{},
- node: &v1.Node{},
- fits: true,
- name: "no host specified",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeName: "foo",
- },
- },
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "foo",
- },
- },
- fits: true,
- name: "host matches",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeName: "bar",
- },
- },
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "foo",
- },
- },
- fits: false,
- name: "host doesn't match",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrPodNotMatchHostName}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(test.node)
- fits, reasons, err := PodFitsHost(test.pod, GetPredicateMetadata(test.pod, nil), nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("unexpected difference: expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- func newPod(host string, hostPortInfos ...string) *v1.Pod {
- networkPorts := []v1.ContainerPort{}
- for _, portInfo := range hostPortInfos {
- splited := strings.Split(portInfo, "/")
- hostPort, _ := strconv.Atoi(splited[2])
- networkPorts = append(networkPorts, v1.ContainerPort{
- HostIP: splited[1],
- HostPort: int32(hostPort),
- Protocol: v1.Protocol(splited[0]),
- })
- }
- return &v1.Pod{
- Spec: v1.PodSpec{
- NodeName: host,
- Containers: []v1.Container{
- {
- Ports: networkPorts,
- },
- },
- },
- }
- }
- func TestPodFitsHostPorts(t *testing.T) {
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- }{
- {
- pod: &v1.Pod{},
- nodeInfo: schedulernodeinfo.NewNodeInfo(),
- fits: true,
- name: "nothing running",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "UDP/127.0.0.1/9090")),
- fits: true,
- name: "other port",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "UDP/127.0.0.1/8080")),
- fits: false,
- name: "same udp port",
- },
- {
- pod: newPod("m1", "TCP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.1/8080")),
- fits: false,
- name: "same tcp port",
- },
- {
- pod: newPod("m1", "TCP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.2/8080")),
- fits: true,
- name: "different host ip",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.1/8080")),
- fits: true,
- name: "different protocol",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8000", "UDP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "UDP/127.0.0.1/8080")),
- fits: false,
- name: "second udp port conflict",
- },
- {
- pod: newPod("m1", "TCP/127.0.0.1/8001", "UDP/127.0.0.1/8080"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.1/8001", "UDP/127.0.0.1/8081")),
- fits: false,
- name: "first tcp port conflict",
- },
- {
- pod: newPod("m1", "TCP/0.0.0.0/8001"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.1/8001")),
- fits: false,
- name: "first tcp port conflict due to 0.0.0.0 hostIP",
- },
- {
- pod: newPod("m1", "TCP/10.0.10.10/8001", "TCP/0.0.0.0/8001"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/127.0.0.1/8001")),
- fits: false,
- name: "TCP hostPort conflict due to 0.0.0.0 hostIP",
- },
- {
- pod: newPod("m1", "TCP/127.0.0.1/8001"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/0.0.0.0/8001")),
- fits: false,
- name: "second tcp port conflict to 0.0.0.0 hostIP",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8001"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/0.0.0.0/8001")),
- fits: true,
- name: "second different protocol",
- },
- {
- pod: newPod("m1", "UDP/127.0.0.1/8001"),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newPod("m1", "TCP/0.0.0.0/8001", "UDP/0.0.0.0/8001")),
- fits: false,
- name: "UDP hostPort conflict due to 0.0.0.0 hostIP",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrPodNotFitsHostPorts}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fits, reasons, err := PodFitsHostPorts(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if test.fits != fits {
- t.Errorf("expected %v, saw %v", test.fits, fits)
- }
- })
- }
- }
- func TestGCEDiskConflicts(t *testing.T) {
- volState := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- GCEPersistentDisk: &v1.GCEPersistentDiskVolumeSource{
- PDName: "foo",
- },
- },
- },
- },
- }
- volState2 := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- GCEPersistentDisk: &v1.GCEPersistentDiskVolumeSource{
- PDName: "bar",
- },
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- isOk bool
- name string
- }{
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(), true, "nothing"},
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "one state"},
- {&v1.Pod{Spec: volState}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), false, "same state"},
- {&v1.Pod{Spec: volState2}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "different state"},
- }
- expectedFailureReasons := []PredicateFailureReason{ErrDiskConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- ok, reasons, err := NoDiskConflict(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !ok && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if test.isOk && !ok {
- t.Errorf("expected ok, got none. %v %s", test.pod, test.nodeInfo)
- }
- if !test.isOk && ok {
- t.Errorf("expected no ok, got one. %v %s", test.pod, test.nodeInfo)
- }
- })
- }
- }
- func TestAWSDiskConflicts(t *testing.T) {
- volState := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- AWSElasticBlockStore: &v1.AWSElasticBlockStoreVolumeSource{
- VolumeID: "foo",
- },
- },
- },
- },
- }
- volState2 := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- AWSElasticBlockStore: &v1.AWSElasticBlockStoreVolumeSource{
- VolumeID: "bar",
- },
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- isOk bool
- name string
- }{
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(), true, "nothing"},
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "one state"},
- {&v1.Pod{Spec: volState}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), false, "same state"},
- {&v1.Pod{Spec: volState2}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "different state"},
- }
- expectedFailureReasons := []PredicateFailureReason{ErrDiskConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- ok, reasons, err := NoDiskConflict(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !ok && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if test.isOk && !ok {
- t.Errorf("expected ok, got none. %v %s", test.pod, test.nodeInfo)
- }
- if !test.isOk && ok {
- t.Errorf("expected no ok, got one. %v %s", test.pod, test.nodeInfo)
- }
- })
- }
- }
- func TestRBDDiskConflicts(t *testing.T) {
- volState := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- RBD: &v1.RBDVolumeSource{
- CephMonitors: []string{"a", "b"},
- RBDPool: "foo",
- RBDImage: "bar",
- FSType: "ext4",
- },
- },
- },
- },
- }
- volState2 := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- RBD: &v1.RBDVolumeSource{
- CephMonitors: []string{"c", "d"},
- RBDPool: "foo",
- RBDImage: "bar",
- FSType: "ext4",
- },
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- isOk bool
- name string
- }{
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(), true, "nothing"},
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "one state"},
- {&v1.Pod{Spec: volState}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), false, "same state"},
- {&v1.Pod{Spec: volState2}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "different state"},
- }
- expectedFailureReasons := []PredicateFailureReason{ErrDiskConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- ok, reasons, err := NoDiskConflict(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !ok && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if test.isOk && !ok {
- t.Errorf("expected ok, got none. %v %s", test.pod, test.nodeInfo)
- }
- if !test.isOk && ok {
- t.Errorf("expected no ok, got one. %v %s", test.pod, test.nodeInfo)
- }
- })
- }
- }
- func TestISCSIDiskConflicts(t *testing.T) {
- volState := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- ISCSI: &v1.ISCSIVolumeSource{
- TargetPortal: "127.0.0.1:3260",
- IQN: "iqn.2016-12.server:storage.target01",
- FSType: "ext4",
- Lun: 0,
- },
- },
- },
- },
- }
- volState2 := v1.PodSpec{
- Volumes: []v1.Volume{
- {
- VolumeSource: v1.VolumeSource{
- ISCSI: &v1.ISCSIVolumeSource{
- TargetPortal: "127.0.0.1:3260",
- IQN: "iqn.2017-12.server:storage.target01",
- FSType: "ext4",
- Lun: 0,
- },
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- isOk bool
- name string
- }{
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(), true, "nothing"},
- {&v1.Pod{}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "one state"},
- {&v1.Pod{Spec: volState}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), false, "same state"},
- {&v1.Pod{Spec: volState2}, schedulernodeinfo.NewNodeInfo(&v1.Pod{Spec: volState}), true, "different state"},
- }
- expectedFailureReasons := []PredicateFailureReason{ErrDiskConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- ok, reasons, err := NoDiskConflict(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !ok && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if test.isOk && !ok {
- t.Errorf("expected ok, got none. %v %s", test.pod, test.nodeInfo)
- }
- if !test.isOk && ok {
- t.Errorf("expected no ok, got one. %v %s", test.pod, test.nodeInfo)
- }
- })
- }
- }
- // TODO: Add test case for RequiredDuringSchedulingRequiredDuringExecution after it's implemented.
- func TestPodFitsSelector(t *testing.T) {
- tests := []struct {
- pod *v1.Pod
- labels map[string]string
- nodeName string
- fits bool
- name string
- }{
- {
- pod: &v1.Pod{},
- fits: true,
- name: "no selector",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- },
- },
- },
- fits: false,
- name: "missing labels",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "same labels",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- "baz": "blah",
- },
- fits: true,
- name: "node labels are superset",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- "baz": "blah",
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "node labels are subset",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"bar", "value2"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "Pod with matchExpressions using In operator that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "kernel-version",
- Operator: v1.NodeSelectorOpGt,
- Values: []string{"0204"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- // We use two digit to denote major version and two digit for minor version.
- "kernel-version": "0206",
- },
- fits: true,
- name: "Pod with matchExpressions using Gt operator that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "mem-type",
- Operator: v1.NodeSelectorOpNotIn,
- Values: []string{"DDR", "DDR2"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "mem-type": "DDR3",
- },
- fits: true,
- name: "Pod with matchExpressions using NotIn operator that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "GPU",
- Operator: v1.NodeSelectorOpExists,
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "GPU": "NVIDIA-GRID-K1",
- },
- fits: true,
- name: "Pod with matchExpressions using Exists operator that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"value1", "value2"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "Pod with affinity that don't match node's labels won't schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: nil,
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "Pod with a nil []NodeSelectorTerm in affinity, can't match the node's labels and won't schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{},
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "Pod with an empty []NodeSelectorTerm in affinity, can't match the node's labels and won't schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{},
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "Pod with empty MatchExpressions is not a valid value will match no objects and won't schedule onto the node",
- },
- {
- pod: &v1.Pod{},
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "Pod with no Affinity will schedule onto a node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: nil,
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "Pod with Affinity but nil NodeSelector will schedule onto a node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "GPU",
- Operator: v1.NodeSelectorOpExists,
- }, {
- Key: "GPU",
- Operator: v1.NodeSelectorOpNotIn,
- Values: []string{"AMD", "INTER"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "GPU": "NVIDIA-GRID-K1",
- },
- fits: true,
- name: "Pod with multiple matchExpressions ANDed that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "GPU",
- Operator: v1.NodeSelectorOpExists,
- }, {
- Key: "GPU",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"AMD", "INTER"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "GPU": "NVIDIA-GRID-K1",
- },
- fits: false,
- name: "Pod with multiple matchExpressions ANDed that doesn't match the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"bar", "value2"},
- },
- },
- },
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "diffkey",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"wrong", "value2"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "Pod with multiple NodeSelectorTerms ORed in affinity, matches the node's labels and will schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- },
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpExists,
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: true,
- name: "Pod with an Affinity and a PodSpec.NodeSelector(the old thing that we are deprecating) " +
- "both are satisfied, will schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeSelector: map[string]string{
- "foo": "bar",
- },
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpExists,
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "barrrrrr",
- },
- fits: false,
- name: "Pod with an Affinity matches node's labels but the PodSpec.NodeSelector(the old thing that we are deprecating) " +
- "is not satisfied, won't schedule onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpNotIn,
- Values: []string{"invalid value: ___@#$%^"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- labels: map[string]string{
- "foo": "bar",
- },
- fits: false,
- name: "Pod with an invalid value in Affinity term won't be scheduled onto the node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_1",
- fits: true,
- name: "Pod with matchFields using In operator that matches the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_2",
- fits: false,
- name: "Pod with matchFields using In operator that does not match the existing node",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- },
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_2",
- labels: map[string]string{"foo": "bar"},
- fits: true,
- name: "Pod with two terms: matchFields does not match, but matchExpressions matches",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_2",
- labels: map[string]string{"foo": "bar"},
- fits: false,
- name: "Pod with one term: matchFields does not match, but matchExpressions matches",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_1",
- labels: map[string]string{"foo": "bar"},
- fits: true,
- name: "Pod with one term: both matchFields and matchExpressions match",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchFields: []v1.NodeSelectorRequirement{
- {
- Key: schedulerapi.NodeFieldSelectorKeyNodeName,
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"node_1"},
- },
- },
- },
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "foo",
- Operator: v1.NodeSelectorOpIn,
- Values: []string{"not-match-to-bar"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- nodeName: "node_2",
- labels: map[string]string{"foo": "bar"},
- fits: false,
- name: "Pod with two terms: both matchFields and matchExpressions do not match",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrNodeSelectorNotMatch}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- node := v1.Node{ObjectMeta: metav1.ObjectMeta{
- Name: test.nodeName,
- Labels: test.labels,
- }}
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(&node)
- fits, reasons, err := PodMatchNodeSelector(test.pod, GetPredicateMetadata(test.pod, nil), nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestNodeLabelPresence(t *testing.T) {
- label := map[string]string{"foo": "bar", "bar": "foo"}
- tests := []struct {
- pod *v1.Pod
- labels []string
- presence bool
- fits bool
- name string
- }{
- {
- labels: []string{"baz"},
- presence: true,
- fits: false,
- name: "label does not match, presence true",
- },
- {
- labels: []string{"baz"},
- presence: false,
- fits: true,
- name: "label does not match, presence false",
- },
- {
- labels: []string{"foo", "baz"},
- presence: true,
- fits: false,
- name: "one label matches, presence true",
- },
- {
- labels: []string{"foo", "baz"},
- presence: false,
- fits: false,
- name: "one label matches, presence false",
- },
- {
- labels: []string{"foo", "bar"},
- presence: true,
- fits: true,
- name: "all labels match, presence true",
- },
- {
- labels: []string{"foo", "bar"},
- presence: false,
- fits: false,
- name: "all labels match, presence false",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrNodeLabelPresenceViolated}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- node := v1.Node{ObjectMeta: metav1.ObjectMeta{Labels: label}}
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(&node)
- labelChecker := NodeLabelChecker{test.labels, test.presence}
- fits, reasons, err := labelChecker.CheckNodeLabelPresence(test.pod, GetPredicateMetadata(test.pod, nil), nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestServiceAffinity(t *testing.T) {
- selector := map[string]string{"foo": "bar"}
- labels1 := map[string]string{
- "region": "r1",
- "zone": "z11",
- }
- labels2 := map[string]string{
- "region": "r1",
- "zone": "z12",
- }
- labels3 := map[string]string{
- "region": "r2",
- "zone": "z21",
- }
- labels4 := map[string]string{
- "region": "r2",
- "zone": "z22",
- }
- node1 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine1", Labels: labels1}}
- node2 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine2", Labels: labels2}}
- node3 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine3", Labels: labels3}}
- node4 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine4", Labels: labels4}}
- node5 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine5", Labels: labels4}}
- tests := []struct {
- pod *v1.Pod
- pods []*v1.Pod
- services []*v1.Service
- node *v1.Node
- labels []string
- fits bool
- name string
- }{
- {
- pod: new(v1.Pod),
- node: &node1,
- fits: true,
- labels: []string{"region"},
- name: "nothing scheduled",
- },
- {
- pod: &v1.Pod{Spec: v1.PodSpec{NodeSelector: map[string]string{"region": "r1"}}},
- node: &node1,
- fits: true,
- labels: []string{"region"},
- name: "pod with region label match",
- },
- {
- pod: &v1.Pod{Spec: v1.PodSpec{NodeSelector: map[string]string{"region": "r2"}}},
- node: &node1,
- fits: false,
- labels: []string{"region"},
- name: "pod with region label mismatch",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: selector}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}}},
- fits: true,
- labels: []string{"region"},
- name: "service pod on same node",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine2"}, ObjectMeta: metav1.ObjectMeta{Labels: selector}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}}},
- fits: true,
- labels: []string{"region"},
- name: "service pod on different node, region match",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine3"}, ObjectMeta: metav1.ObjectMeta{Labels: selector}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}}},
- fits: false,
- labels: []string{"region"},
- name: "service pod on different node, region mismatch",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns1"}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine3"}, ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns1"}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}, ObjectMeta: metav1.ObjectMeta{Namespace: "ns2"}}},
- fits: true,
- labels: []string{"region"},
- name: "service in different namespace, region mismatch",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns1"}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine3"}, ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns2"}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}, ObjectMeta: metav1.ObjectMeta{Namespace: "ns1"}}},
- fits: true,
- labels: []string{"region"},
- name: "pod in different namespace, region mismatch",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns1"}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine3"}, ObjectMeta: metav1.ObjectMeta{Labels: selector, Namespace: "ns1"}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}, ObjectMeta: metav1.ObjectMeta{Namespace: "ns1"}}},
- fits: false,
- labels: []string{"region"},
- name: "service and pod in same namespace, region mismatch",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine2"}, ObjectMeta: metav1.ObjectMeta{Labels: selector}}},
- node: &node1,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}}},
- fits: false,
- labels: []string{"region", "zone"},
- name: "service pod on different node, multiple labels, not all match",
- },
- {
- pod: &v1.Pod{ObjectMeta: metav1.ObjectMeta{Labels: selector}},
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine5"}, ObjectMeta: metav1.ObjectMeta{Labels: selector}}},
- node: &node4,
- services: []*v1.Service{{Spec: v1.ServiceSpec{Selector: selector}}},
- fits: true,
- labels: []string{"region", "zone"},
- name: "service pod on different node, multiple labels, all match",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrServiceAffinityViolated}
- for _, test := range tests {
- testIt := func(skipPrecompute bool) {
- t.Run(fmt.Sprintf("%v/skipPrecompute/%v", test.name, skipPrecompute), func(t *testing.T) {
- nodes := []v1.Node{node1, node2, node3, node4, node5}
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(test.node)
- nodeInfoMap := map[string]*schedulernodeinfo.NodeInfo{test.node.Name: nodeInfo}
- // Reimplementing the logic that the scheduler implements: Any time it makes a predicate, it registers any precomputations.
- predicate, precompute := NewServiceAffinityPredicate(schedulertesting.FakePodLister(test.pods), schedulertesting.FakeServiceLister(test.services), FakeNodeListInfo(nodes), test.labels)
- // Register a precomputation or Rewrite the precomputation to a no-op, depending on the state we want to test.
- RegisterPredicateMetadataProducer("ServiceAffinityMetaProducer", func(pm *predicateMetadata) {
- if !skipPrecompute {
- precompute(pm)
- }
- })
- if pmeta, ok := (GetPredicateMetadata(test.pod, nodeInfoMap)).(*predicateMetadata); ok {
- fits, reasons, err := predicate(test.pod, pmeta, nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- } else {
- t.Errorf("Error casting.")
- }
- })
- }
- testIt(false) // Confirm that the predicate works without precomputed data (resilience)
- testIt(true) // Confirm that the predicate works with the precomputed data (better performance)
- }
- }
- func newPodWithPort(hostPorts ...int) *v1.Pod {
- networkPorts := []v1.ContainerPort{}
- for _, port := range hostPorts {
- networkPorts = append(networkPorts, v1.ContainerPort{HostPort: int32(port)})
- }
- return &v1.Pod{
- Spec: v1.PodSpec{
- Containers: []v1.Container{
- {
- Ports: networkPorts,
- },
- },
- },
- }
- }
- func TestRunGeneralPredicates(t *testing.T) {
- resourceTests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- node *v1.Node
- fits bool
- name string
- wErr error
- reasons []PredicateFailureReason
- }{
- {
- pod: &v1.Pod{},
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 9, Memory: 19})),
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{Name: "machine1"},
- Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 0, 0, 0).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 0, 0, 0)},
- },
- fits: true,
- wErr: nil,
- name: "no resources/port/host requested always fits",
- },
- {
- pod: newResourcePod(schedulernodeinfo.Resource{MilliCPU: 8, Memory: 10}),
- nodeInfo: schedulernodeinfo.NewNodeInfo(
- newResourcePod(schedulernodeinfo.Resource{MilliCPU: 5, Memory: 19})),
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{Name: "machine1"},
- Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 0, 0, 0).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 0, 0, 0)},
- },
- fits: false,
- wErr: nil,
- reasons: []PredicateFailureReason{
- NewInsufficientResourceError(v1.ResourceCPU, 8, 5, 10),
- NewInsufficientResourceError(v1.ResourceMemory, 10, 19, 20),
- },
- name: "not enough cpu and memory resource",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- NodeName: "machine2",
- },
- },
- nodeInfo: schedulernodeinfo.NewNodeInfo(),
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{Name: "machine1"},
- Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 0, 0, 0).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 0, 0, 0)},
- },
- fits: false,
- wErr: nil,
- reasons: []PredicateFailureReason{ErrPodNotMatchHostName},
- name: "host not match",
- },
- {
- pod: newPodWithPort(123),
- nodeInfo: schedulernodeinfo.NewNodeInfo(newPodWithPort(123)),
- node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{Name: "machine1"},
- Status: v1.NodeStatus{Capacity: makeResources(10, 20, 32, 0, 0, 0).Capacity, Allocatable: makeAllocatableResources(10, 20, 32, 0, 0, 0)},
- },
- fits: false,
- wErr: nil,
- reasons: []PredicateFailureReason{ErrPodNotFitsHostPorts},
- name: "hostport conflict",
- },
- }
- for _, test := range resourceTests {
- t.Run(test.name, func(t *testing.T) {
- test.nodeInfo.SetNode(test.node)
- fits, reasons, err := GeneralPredicates(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, test.reasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, test.reasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- // TODO: Add test case for RequiredDuringSchedulingRequiredDuringExecution after it's implemented.
- func TestInterPodAffinity(t *testing.T) {
- podLabel := map[string]string{"service": "securityscan"}
- labels1 := map[string]string{
- "region": "r1",
- "zone": "z11",
- }
- podLabel2 := map[string]string{"security": "S1"}
- node1 := v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "machine1", Labels: labels1}}
- tests := []struct {
- pod *v1.Pod
- pods []*v1.Pod
- node *v1.Node
- fits bool
- name string
- expectFailureReasons []PredicateFailureReason
- }{
- {
- pod: new(v1.Pod),
- node: &node1,
- fits: true,
- name: "A pod that has no required pod affinity scheduling rules can schedule onto a node with no existing pods",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: true,
- name: "satisfies with requiredDuringSchedulingIgnoredDuringExecution in PodAffinity using In operator that matches the existing pod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpNotIn,
- Values: []string{"securityscan3", "value3"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: true,
- name: "satisfies the pod with requiredDuringSchedulingIgnoredDuringExecution in PodAffinity using not in operator in labelSelector that matches the existing pod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- Namespaces: []string{"DiffNameSpace"},
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel, Namespace: "ns"}}},
- node: &node1,
- fits: false,
- name: "Does not satisfy the PodAffinity with labelSelector because of diff Namespace",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"antivirusscan", "value2"},
- },
- },
- },
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: false,
- name: "Doesn't satisfy the PodAffinity because of unmatching labelSelector with the existing pod",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpExists,
- }, {
- Key: "wrongkey",
- Operator: metav1.LabelSelectorOpDoesNotExist,
- },
- },
- },
- TopologyKey: "region",
- }, {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan"},
- }, {
- Key: "service",
- Operator: metav1.LabelSelectorOpNotIn,
- Values: []string{"WrongValue"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: true,
- name: "satisfies the PodAffinity with different label Operators in multiple RequiredDuringSchedulingIgnoredDuringExecution ",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpExists,
- }, {
- Key: "wrongkey",
- Operator: metav1.LabelSelectorOpDoesNotExist,
- },
- },
- },
- TopologyKey: "region",
- }, {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan2"},
- }, {
- Key: "service",
- Operator: metav1.LabelSelectorOpNotIn,
- Values: []string{"WrongValue"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: false,
- name: "The labelSelector requirements(items of matchExpressions) are ANDed, the pod cannot schedule onto the node because one of the matchExpression item don't match.",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"antivirusscan", "value2"},
- },
- },
- },
- TopologyKey: "node",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: true,
- name: "satisfies the PodAffinity and PodAntiAffinity with the existing pod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"antivirusscan", "value2"},
- },
- },
- },
- TopologyKey: "node",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"antivirusscan", "value2"},
- },
- },
- },
- TopologyKey: "node",
- },
- },
- },
- },
- },
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- },
- },
- node: &node1,
- fits: true,
- name: "satisfies the PodAffinity and PodAntiAffinity and PodAntiAffinity symmetry with the existing pod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel2,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: false,
- name: "satisfies the PodAffinity but doesn't satisfy the PodAntiAffinity with the existing pod",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"antivirusscan", "value2"},
- },
- },
- },
- TopologyKey: "node",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- },
- },
- node: &node1,
- fits: false,
- name: "satisfies the PodAffinity and PodAntiAffinity but doesn't satisfy PodAntiAffinity symmetry with the existing pod",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpNotIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "machine2"}, ObjectMeta: metav1.ObjectMeta{Labels: podLabel}}},
- node: &node1,
- fits: false,
- name: "pod matches its own Label in PodAffinity and that matches the existing pod Labels",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- },
- },
- node: &node1,
- fits: false,
- name: "verify that PodAntiAffinity from existing pod is respected when pod has no AntiAffinity constraints. doesn't satisfy PodAntiAffinity symmetry with the existing pod",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpNotIn,
- Values: []string{"securityscan", "value2"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- },
- },
- node: &node1,
- fits: true,
- name: "verify that PodAntiAffinity from existing pod is respected when pod has no AntiAffinity constraints. satisfy PodAntiAffinity symmetry with the existing pod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: podLabel,
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel2},
- Spec: v1.PodSpec{NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- node: &node1,
- fits: false,
- name: "satisfies the PodAntiAffinity with existing pod but doesn't satisfy PodAntiAffinity symmetry with incoming pod",
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel2},
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- node: &node1,
- fits: false,
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- name: "PodAntiAffinity symmetry check a1: incoming pod and existing pod partially match each other on AffinityTerms",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel2},
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: podLabel},
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "security",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- node: &node1,
- fits: false,
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- name: "PodAntiAffinity symmetry check a2: incoming pod and existing pod partially match each other on AffinityTerms",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"abc": "", "xyz": ""}},
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "abc",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "def",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"def": "", "xyz": ""}},
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "abc",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "def",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- node: &node1,
- fits: false,
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- name: "PodAntiAffinity symmetry check b1: incoming pod and existing pod partially match each other on AffinityTerms",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"def": "", "xyz": ""}},
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "abc",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "def",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"abc": "", "xyz": ""}},
- Spec: v1.PodSpec{
- NodeName: "machine1",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "abc",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "def",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- node: &node1,
- fits: false,
- expectFailureReasons: []PredicateFailureReason{ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- name: "PodAntiAffinity symmetry check b2: incoming pod and existing pod partially match each other on AffinityTerms",
- },
- }
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- node := test.node
- var podsOnNode []*v1.Pod
- for _, pod := range test.pods {
- if pod.Spec.NodeName == node.Name {
- podsOnNode = append(podsOnNode, pod)
- }
- }
- fit := PodAffinityChecker{
- info: FakeNodeInfo(*node),
- podLister: schedulertesting.FakePodLister(test.pods),
- }
- nodeInfo := schedulernodeinfo.NewNodeInfo(podsOnNode...)
- nodeInfo.SetNode(test.node)
- nodeInfoMap := map[string]*schedulernodeinfo.NodeInfo{test.node.Name: nodeInfo}
- fits, reasons, _ := fit.InterPodAffinityMatches(test.pod, GetPredicateMetadata(test.pod, nodeInfoMap), nodeInfo)
- if !fits && !reflect.DeepEqual(reasons, test.expectFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, test.expectFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestInterPodAffinityWithMultipleNodes(t *testing.T) {
- podLabelA := map[string]string{
- "foo": "bar",
- }
- labelRgChina := map[string]string{
- "region": "China",
- }
- labelRgChinaAzAz1 := map[string]string{
- "region": "China",
- "az": "az1",
- }
- labelRgIndia := map[string]string{
- "region": "India",
- }
- labelRgUS := map[string]string{
- "region": "US",
- }
- tests := []struct {
- pod *v1.Pod
- pods []*v1.Pod
- nodes []v1.Node
- nodesExpectAffinityFailureReasons [][]PredicateFailureReason
- fits map[string]bool
- name string
- nometa bool
- }{
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "machine1"}, ObjectMeta: metav1.ObjectMeta{Name: "p1", Labels: podLabelA}},
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "machine1", Labels: labelRgChina}},
- {ObjectMeta: metav1.ObjectMeta{Name: "machine2", Labels: labelRgChinaAzAz1}},
- {ObjectMeta: metav1.ObjectMeta{Name: "machine3", Labels: labelRgIndia}},
- },
- fits: map[string]bool{
- "machine1": true,
- "machine2": true,
- "machine3": false,
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{nil, nil, {ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch}},
- name: "A pod can be scheduled onto all the nodes that have the same topology key & label value with one of them has an existing pod that matches the affinity rules",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- NodeAffinity: &v1.NodeAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: &v1.NodeSelector{
- NodeSelectorTerms: []v1.NodeSelectorTerm{
- {
- MatchExpressions: []v1.NodeSelectorRequirement{
- {
- Key: "hostname",
- Operator: v1.NodeSelectorOpNotIn,
- Values: []string{"h1"},
- },
- },
- },
- },
- },
- },
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"abc"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "abc"}}},
- {Spec: v1.PodSpec{NodeName: "nodeB"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "def"}}},
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "hostname": "h1"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "hostname": "h2"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{nil, nil},
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": true,
- },
- name: "NodeA and nodeB have same topologyKey and label value. NodeA does not satisfy node affinity rule, but has an existing pod that matches the inter pod affinity rule. The pod can be scheduled onto nodeB.",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: map[string]string{
- "foo": "bar",
- "service": "securityscan",
- },
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{{Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Name: "p1", Labels: map[string]string{"foo": "bar"}}}},
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"zone": "az1", "hostname": "h1"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"zone": "az2", "hostname": "h2"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{nil, nil},
- fits: map[string]bool{
- "nodeA": true,
- "nodeB": true,
- },
- name: "The affinity rule is to schedule all of the pods of this collection to the same zone. The first pod of the collection " +
- "should not be blocked from being scheduled onto any node, even there's no existing pod that matches the rule anywhere.",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"abc"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "abc"}}},
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{{ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch}, {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch}},
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "NodeA and nodeB have same topologyKey and label value. NodeA has an existing pod that matches the inter pod affinity rule. The pod can not be scheduled onto nodeA and nodeB.",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"abc"},
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "service",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"securityscan"},
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "abc", "service": "securityscan"}}},
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "This test ensures that anti-affinity matches a pod when any term of the anti-affinity rule matches a pod.",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"abc"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "abc"}}},
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: labelRgChina}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: labelRgChinaAzAz1}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeC", Labels: labelRgIndia}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{{ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch}, {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch}, nil},
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- "nodeC": true,
- },
- name: "NodeA and nodeB have same topologyKey and label value. NodeA has an existing pod that matches the inter pod affinity rule. The pod can not be scheduled onto nodeA and nodeB but can be scheduled onto nodeC",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "123"}},
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {Spec: v1.PodSpec{NodeName: "nodeA"}, ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "bar"}}},
- {
- Spec: v1.PodSpec{
- NodeName: "nodeC",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"123"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: labelRgChina}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: labelRgChinaAzAz1}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeC", Labels: labelRgIndia}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeD", Labels: labelRgUS}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- nil,
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- "nodeC": false,
- "nodeD": true,
- },
- name: "NodeA and nodeB have same topologyKey and label value. NodeA has an existing pod that matches the inter pod affinity rule. NodeC has an existing pod that match the inter pod affinity rule. The pod can not be scheduled onto nodeA, nodeB and nodeC but can be schedulerd onto nodeD",
- nometa: true,
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Labels: map[string]string{"foo": "123"},
- Namespace: "NS1",
- },
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"bar"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{
- Labels: map[string]string{"foo": "bar"},
- Namespace: "NS1",
- },
- Spec: v1.PodSpec{NodeName: "nodeA"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Namespace: "NS2"},
- Spec: v1.PodSpec{
- NodeName: "nodeC",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpIn,
- Values: []string{"123"},
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: labelRgChina}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: labelRgChinaAzAz1}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeC", Labels: labelRgIndia}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- nil,
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- "nodeC": true,
- },
- name: "NodeA and nodeB have same topologyKey and label value. NodeA has an existing pod that matches the inter pod affinity rule. The pod can not be scheduled onto nodeA, nodeB, but can be scheduled onto nodeC (NodeC has an existing pod that match the inter pod affinity rule but in different namespace)",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": ""}},
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "invalid-node-label",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{},
- fits: map[string]bool{
- "nodeA": true,
- "nodeB": true,
- },
- name: "Test existing pod's anti-affinity: if an existing pod has a term with invalid topologyKey, labelSelector of the term is firstly checked, and then topologyKey of the term is also checked",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "invalid-node-label",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{},
- fits: map[string]bool{
- "nodeA": true,
- "nodeB": true,
- },
- name: "Test incoming pod's anti-affinity: even if labelSelector matches, we still check if topologyKey matches",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "", "bar": ""}},
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "pod1"},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "pod2"},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "Test existing pod's anti-affinity: incoming pod wouldn't considered as a fit as it violates each existingPod's terms on all nodes",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"bar": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeB",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "Test incoming pod's anti-affinity: incoming pod wouldn't considered as a fit as it at least violates one anti-affinity rule of existingPod",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "", "bar": ""}},
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "invalid-node-label",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": true,
- },
- name: "Test existing pod's anti-affinity: only when labelSelector and topologyKey both match, it's counted as a single term match - case when one term has invalid topologyKey",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "invalid-node-label",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "podA", Labels: map[string]string{"foo": "", "bar": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": true,
- },
- name: "Test incoming pod's anti-affinity: only when labelSelector and topologyKey both match, it's counted as a single term match - case when one term has invalid topologyKey",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "", "bar": ""}},
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "Test existing pod's anti-affinity: only when labelSelector and topologyKey both match, it's counted as a single term match - case when all terms have valid topologyKey",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "", "bar": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "Test incoming pod's anti-affinity: only when labelSelector and topologyKey both match, it's counted as a single term match - case when all terms have valid topologyKey",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Labels: map[string]string{"foo": "", "bar": ""}},
- },
- pods: []*v1.Pod{
- {
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "labelA",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- {
- Spec: v1.PodSpec{
- NodeName: "nodeB",
- Affinity: &v1.Affinity{
- PodAntiAffinity: &v1.PodAntiAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "labelB",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeC", Labels: map[string]string{"region": "r1", "zone": "z3", "hostname": "nodeC"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrExistingPodsAntiAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- "nodeC": true,
- },
- name: "Test existing pod's anti-affinity: existingPod on nodeA and nodeB has at least one anti-affinity term matches incoming pod, so incoming pod can only be scheduled to nodeC",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "pod1", Labels: map[string]string{"foo": "", "bar": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {},
- {ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": true,
- "nodeB": true,
- },
- name: "Test incoming pod's affinity: firstly check if all affinityTerms match, and then check if all topologyKeys match",
- },
- {
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Affinity: &v1.Affinity{
- PodAffinity: &v1.PodAffinity{
- RequiredDuringSchedulingIgnoredDuringExecution: []v1.PodAffinityTerm{
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "foo",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "region",
- },
- {
- LabelSelector: &metav1.LabelSelector{
- MatchExpressions: []metav1.LabelSelectorRequirement{
- {
- Key: "bar",
- Operator: metav1.LabelSelectorOpExists,
- },
- },
- },
- TopologyKey: "zone",
- },
- },
- },
- },
- },
- },
- pods: []*v1.Pod{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "pod1", Labels: map[string]string{"foo": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeA",
- },
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "pod2", Labels: map[string]string{"bar": ""}},
- Spec: v1.PodSpec{
- NodeName: "nodeB",
- },
- },
- },
- nodes: []v1.Node{
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeA", Labels: map[string]string{"region": "r1", "zone": "z1", "hostname": "nodeA"}}},
- {ObjectMeta: metav1.ObjectMeta{Name: "nodeB", Labels: map[string]string{"region": "r1", "zone": "z2", "hostname": "nodeB"}}},
- },
- nodesExpectAffinityFailureReasons: [][]PredicateFailureReason{
- {ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- {ErrPodAffinityNotMatch, ErrPodAffinityRulesNotMatch},
- },
- fits: map[string]bool{
- "nodeA": false,
- "nodeB": false,
- },
- name: "Test incoming pod's affinity: firstly check if all affinityTerms match, and then check if all topologyKeys match, and the match logic should be satified on the same pod",
- },
- }
- selectorExpectedFailureReasons := []PredicateFailureReason{ErrNodeSelectorNotMatch}
- for indexTest, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- nodeListInfo := FakeNodeListInfo(test.nodes)
- nodeInfoMap := make(map[string]*schedulernodeinfo.NodeInfo)
- for i, node := range test.nodes {
- var podsOnNode []*v1.Pod
- for _, pod := range test.pods {
- if pod.Spec.NodeName == node.Name {
- podsOnNode = append(podsOnNode, pod)
- }
- }
- nodeInfo := schedulernodeinfo.NewNodeInfo(podsOnNode...)
- nodeInfo.SetNode(&test.nodes[i])
- nodeInfoMap[node.Name] = nodeInfo
- }
- for indexNode, node := range test.nodes {
- testFit := PodAffinityChecker{
- info: nodeListInfo,
- podLister: schedulertesting.FakePodLister(test.pods),
- }
- var meta PredicateMetadata
- if !test.nometa {
- meta = GetPredicateMetadata(test.pod, nodeInfoMap)
- }
- fits, reasons, _ := testFit.InterPodAffinityMatches(test.pod, meta, nodeInfoMap[node.Name])
- if !fits && !reflect.DeepEqual(reasons, test.nodesExpectAffinityFailureReasons[indexNode]) {
- t.Errorf("index: %d unexpected failure reasons: %v expect: %v", indexTest, reasons, test.nodesExpectAffinityFailureReasons[indexNode])
- }
- affinity := test.pod.Spec.Affinity
- if affinity != nil && affinity.NodeAffinity != nil {
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(&node)
- nodeInfoMap := map[string]*schedulernodeinfo.NodeInfo{node.Name: nodeInfo}
- fits2, reasons, err := PodMatchNodeSelector(test.pod, GetPredicateMetadata(test.pod, nodeInfoMap), nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits2 && !reflect.DeepEqual(reasons, selectorExpectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, selectorExpectedFailureReasons)
- }
- fits = fits && fits2
- }
- if fits != test.fits[node.Name] {
- t.Errorf("expected %v for %s got %v", test.fits[node.Name], node.Name, fits)
- }
- }
- })
- }
- }
- func TestPodToleratesTaints(t *testing.T) {
- podTolerateTaintsTests := []struct {
- pod *v1.Pod
- node v1.Node
- fits bool
- name string
- }{
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod0",
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{{Key: "dedicated", Value: "user1", Effect: "NoSchedule"}},
- },
- },
- fits: false,
- name: "A pod having no tolerations can't be scheduled onto a node with nonempty taints",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod1",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod1:V1"}},
- Tolerations: []v1.Toleration{{Key: "dedicated", Value: "user1", Effect: "NoSchedule"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{{Key: "dedicated", Value: "user1", Effect: "NoSchedule"}},
- },
- },
- fits: true,
- name: "A pod which can be scheduled on a dedicated node assigned to user1 with effect NoSchedule",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{{Key: "dedicated", Operator: "Equal", Value: "user2", Effect: "NoSchedule"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{{Key: "dedicated", Value: "user1", Effect: "NoSchedule"}},
- },
- },
- fits: false,
- name: "A pod which can't be scheduled on a dedicated node assigned to user2 with effect NoSchedule",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{{Key: "foo", Operator: "Exists", Effect: "NoSchedule"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{{Key: "foo", Value: "bar", Effect: "NoSchedule"}},
- },
- },
- fits: true,
- name: "A pod can be scheduled onto the node, with a toleration uses operator Exists that tolerates the taints on the node",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{
- {Key: "dedicated", Operator: "Equal", Value: "user2", Effect: "NoSchedule"},
- {Key: "foo", Operator: "Exists", Effect: "NoSchedule"},
- },
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{
- {Key: "dedicated", Value: "user2", Effect: "NoSchedule"},
- {Key: "foo", Value: "bar", Effect: "NoSchedule"},
- },
- },
- },
- fits: true,
- name: "A pod has multiple tolerations, node has multiple taints, all the taints are tolerated, pod can be scheduled onto the node",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{{Key: "foo", Operator: "Equal", Value: "bar", Effect: "PreferNoSchedule"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{
- {Key: "foo", Value: "bar", Effect: "NoSchedule"},
- },
- },
- },
- fits: false,
- name: "A pod has a toleration that keys and values match the taint on the node, but (non-empty) effect doesn't match, " +
- "can't be scheduled onto the node",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{{Key: "foo", Operator: "Equal", Value: "bar"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{
- {Key: "foo", Value: "bar", Effect: "NoSchedule"},
- },
- },
- },
- fits: true,
- name: "The pod has a toleration that keys and values match the taint on the node, the effect of toleration is empty, " +
- "and the effect of taint is NoSchedule. Pod can be scheduled onto the node",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- Tolerations: []v1.Toleration{{Key: "dedicated", Operator: "Equal", Value: "user2", Effect: "NoSchedule"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{
- {Key: "dedicated", Value: "user1", Effect: "PreferNoSchedule"},
- },
- },
- },
- fits: true,
- name: "The pod has a toleration that key and value don't match the taint on the node, " +
- "but the effect of taint on node is PreferNochedule. Pod can be scheduled onto the node",
- },
- {
- pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{
- Name: "pod2",
- },
- Spec: v1.PodSpec{
- Containers: []v1.Container{{Image: "pod2:V1"}},
- },
- },
- node: v1.Node{
- Spec: v1.NodeSpec{
- Taints: []v1.Taint{
- {Key: "dedicated", Value: "user1", Effect: "PreferNoSchedule"},
- },
- },
- },
- fits: true,
- name: "The pod has no toleration, " +
- "but the effect of taint on node is PreferNochedule. Pod can be scheduled onto the node",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrTaintsTolerationsNotMatch}
- for _, test := range podTolerateTaintsTests {
- t.Run(test.name, func(t *testing.T) {
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(&test.node)
- fits, reasons, err := PodToleratesNodeTaints(test.pod, GetPredicateMetadata(test.pod, nil), nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reason: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected: %v got %v", test.fits, fits)
- }
- })
- }
- }
- func makeEmptyNodeInfo(node *v1.Node) *schedulernodeinfo.NodeInfo {
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(node)
- return nodeInfo
- }
- func TestPodSchedulesOnNodeWithMemoryPressureCondition(t *testing.T) {
- // specify best-effort pod
- bestEffortPod := &v1.Pod{
- Spec: v1.PodSpec{
- Containers: []v1.Container{
- {
- Name: "container",
- Image: "image",
- ImagePullPolicy: "Always",
- // no requirements -> best effort pod
- Resources: v1.ResourceRequirements{},
- },
- },
- },
- }
- // specify non-best-effort pod
- nonBestEffortPod := &v1.Pod{
- Spec: v1.PodSpec{
- Containers: []v1.Container{
- {
- Name: "container",
- Image: "image",
- ImagePullPolicy: "Always",
- // at least one requirement -> burstable pod
- Resources: v1.ResourceRequirements{
- Requests: makeAllocatableResources(100, 100, 100, 0, 0, 0),
- },
- },
- },
- },
- }
- // specify a node with no memory pressure condition on
- noMemoryPressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: "Ready",
- Status: "True",
- },
- },
- },
- }
- // specify a node with memory pressure condition on
- memoryPressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: "MemoryPressure",
- Status: "True",
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- }{
- {
- pod: bestEffortPod,
- nodeInfo: makeEmptyNodeInfo(noMemoryPressureNode),
- fits: true,
- name: "best-effort pod schedulable on node without memory pressure condition on",
- },
- {
- pod: bestEffortPod,
- nodeInfo: makeEmptyNodeInfo(memoryPressureNode),
- fits: false,
- name: "best-effort pod not schedulable on node with memory pressure condition on",
- },
- {
- pod: nonBestEffortPod,
- nodeInfo: makeEmptyNodeInfo(memoryPressureNode),
- fits: true,
- name: "non best-effort pod schedulable on node with memory pressure condition on",
- },
- {
- pod: nonBestEffortPod,
- nodeInfo: makeEmptyNodeInfo(noMemoryPressureNode),
- fits: true,
- name: "non best-effort pod schedulable on node without memory pressure condition on",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrNodeUnderMemoryPressure}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fits, reasons, err := CheckNodeMemoryPressurePredicate(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestPodSchedulesOnNodeWithDiskPressureCondition(t *testing.T) {
- pod := &v1.Pod{
- Spec: v1.PodSpec{
- Containers: []v1.Container{
- {
- Name: "container",
- Image: "image",
- ImagePullPolicy: "Always",
- },
- },
- },
- }
- // specify a node with no disk pressure condition on
- noPressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: "Ready",
- Status: "True",
- },
- },
- },
- }
- // specify a node with pressure condition on
- pressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: "DiskPressure",
- Status: "True",
- },
- },
- },
- }
- tests := []struct {
- pod *v1.Pod
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- }{
- {
- pod: pod,
- nodeInfo: makeEmptyNodeInfo(noPressureNode),
- fits: true,
- name: "pod schedulable on node without pressure condition on",
- },
- {
- pod: pod,
- nodeInfo: makeEmptyNodeInfo(pressureNode),
- fits: false,
- name: "pod not schedulable on node with pressure condition on",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrNodeUnderDiskPressure}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fits, reasons, err := CheckNodeDiskPressurePredicate(test.pod, GetPredicateMetadata(test.pod, nil), test.nodeInfo)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestPodSchedulesOnNodeWithPIDPressureCondition(t *testing.T) {
- // specify a node with no pid pressure condition on
- noPressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: v1.NodeReady,
- Status: v1.ConditionTrue,
- },
- },
- },
- }
- // specify a node with pressure condition on
- pressureNode := &v1.Node{
- Status: v1.NodeStatus{
- Conditions: []v1.NodeCondition{
- {
- Type: v1.NodePIDPressure,
- Status: v1.ConditionTrue,
- },
- },
- },
- }
- tests := []struct {
- nodeInfo *schedulernodeinfo.NodeInfo
- fits bool
- name string
- }{
- {
- nodeInfo: makeEmptyNodeInfo(noPressureNode),
- fits: true,
- name: "pod schedulable on node without pressure condition on",
- },
- {
- nodeInfo: makeEmptyNodeInfo(pressureNode),
- fits: false,
- name: "pod not schedulable on node with pressure condition on",
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrNodeUnderPIDPressure}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fits, reasons, err := CheckNodePIDPressurePredicate(&v1.Pod{}, GetPredicateMetadata(&v1.Pod{}, nil), test.nodeInfo)
- if err != nil {
- t.Fatalf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.fits {
- t.Errorf("expected %v got %v", test.fits, fits)
- }
- })
- }
- }
- func TestNodeConditionPredicate(t *testing.T) {
- tests := []struct {
- name string
- node *v1.Node
- schedulable bool
- }{
- {
- name: "node1 considered",
- node: &v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "node1"}, Status: v1.NodeStatus{Conditions: []v1.NodeCondition{{Type: v1.NodeReady, Status: v1.ConditionTrue}}}},
- schedulable: true,
- },
- {
- name: "node2 ignored - node not Ready",
- node: &v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "node2"}, Status: v1.NodeStatus{Conditions: []v1.NodeCondition{{Type: v1.NodeReady, Status: v1.ConditionFalse}}}},
- schedulable: false,
- },
- {
- name: "node3 ignored - node unschedulable",
- node: &v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "node9"}, Spec: v1.NodeSpec{Unschedulable: true}},
- schedulable: false,
- },
- {
- name: "node4 considered",
- node: &v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "node10"}, Spec: v1.NodeSpec{Unschedulable: false}},
- schedulable: true,
- },
- {
- name: "node5 considered",
- node: &v1.Node{ObjectMeta: metav1.ObjectMeta{Name: "node11"}},
- schedulable: true,
- },
- }
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- nodeInfo := makeEmptyNodeInfo(test.node)
- if fit, reasons, err := CheckNodeConditionPredicate(nil, nil, nodeInfo); fit != test.schedulable {
- t.Errorf("%s: expected: %t, got %t; %+v, %v",
- test.node.Name, test.schedulable, fit, reasons, err)
- }
- })
- }
- }
- func createPodWithVolume(pod, pv, pvc string) *v1.Pod {
- return &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Name: pod, Namespace: "default"},
- Spec: v1.PodSpec{
- Volumes: []v1.Volume{
- {
- Name: pv,
- VolumeSource: v1.VolumeSource{
- PersistentVolumeClaim: &v1.PersistentVolumeClaimVolumeSource{
- ClaimName: pvc,
- },
- },
- },
- },
- },
- }
- }
- func TestVolumeZonePredicate(t *testing.T) {
- pvInfo := FakePersistentVolumeInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_1", Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a"}},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_2", Labels: map[string]string{v1.LabelZoneRegion: "us-west1-b", "uselessLabel": "none"}},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_3", Labels: map[string]string{v1.LabelZoneRegion: "us-west1-c"}},
- },
- }
- pvcInfo := FakePersistentVolumeClaimInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_1", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_1"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_2", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_2"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_3", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_3"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_4", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_not_exist"},
- },
- }
- tests := []struct {
- name string
- Pod *v1.Pod
- Fits bool
- Node *v1.Node
- }{
- {
- name: "pod without volume",
- Pod: &v1.Pod{
- ObjectMeta: metav1.ObjectMeta{Name: "pod_1", Namespace: "default"},
- },
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a"},
- },
- },
- Fits: true,
- },
- {
- name: "node without labels",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_1"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- },
- },
- Fits: true,
- },
- {
- name: "label zone failure domain matched",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_1"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a", "uselessLabel": "none"},
- },
- },
- Fits: true,
- },
- {
- name: "label zone region matched",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_2"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneRegion: "us-west1-b", "uselessLabel": "none"},
- },
- },
- Fits: true,
- },
- {
- name: "label zone region failed match",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_2"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneRegion: "no_us-west1-b", "uselessLabel": "none"},
- },
- },
- Fits: false,
- },
- {
- name: "label zone failure domain failed match",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_1"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "no_us-west1-a", "uselessLabel": "none"},
- },
- },
- Fits: false,
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrVolumeZoneConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fit := NewVolumeZonePredicate(pvInfo, pvcInfo, nil)
- node := &schedulernodeinfo.NodeInfo{}
- node.SetNode(test.Node)
- fits, reasons, err := fit(test.Pod, nil, node)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.Fits {
- t.Errorf("expected %v got %v", test.Fits, fits)
- }
- })
- }
- }
- func TestVolumeZonePredicateMultiZone(t *testing.T) {
- pvInfo := FakePersistentVolumeInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_1", Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a"}},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_2", Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-b", "uselessLabel": "none"}},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_3", Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-c__us-west1-a"}},
- },
- }
- pvcInfo := FakePersistentVolumeClaimInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_1", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_1"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_2", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_2"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_3", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_3"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_4", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_not_exist"},
- },
- }
- tests := []struct {
- name string
- Pod *v1.Pod
- Fits bool
- Node *v1.Node
- }{
- {
- name: "node without labels",
- Pod: createPodWithVolume("pod_1", "Vol_3", "PVC_3"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- },
- },
- Fits: true,
- },
- {
- name: "label zone failure domain matched",
- Pod: createPodWithVolume("pod_1", "Vol_3", "PVC_3"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a", "uselessLabel": "none"},
- },
- },
- Fits: true,
- },
- {
- name: "label zone failure domain failed match",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_1"),
- Node: &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-b", "uselessLabel": "none"},
- },
- },
- Fits: false,
- },
- }
- expectedFailureReasons := []PredicateFailureReason{ErrVolumeZoneConflict}
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fit := NewVolumeZonePredicate(pvInfo, pvcInfo, nil)
- node := &schedulernodeinfo.NodeInfo{}
- node.SetNode(test.Node)
- fits, reasons, err := fit(test.Pod, nil, node)
- if err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if !fits && !reflect.DeepEqual(reasons, expectedFailureReasons) {
- t.Errorf("unexpected failure reasons: %v, want: %v", reasons, expectedFailureReasons)
- }
- if fits != test.Fits {
- t.Errorf("expected %v got %v", test.Fits, fits)
- }
- })
- }
- }
- func TestVolumeZonePredicateWithVolumeBinding(t *testing.T) {
- var (
- modeWait = storagev1.VolumeBindingWaitForFirstConsumer
- class0 = "Class_0"
- classWait = "Class_Wait"
- classImmediate = "Class_Immediate"
- )
- classInfo := FakeStorageClassInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: classImmediate},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: classWait},
- VolumeBindingMode: &modeWait,
- },
- }
- pvInfo := FakePersistentVolumeInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "Vol_1", Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a"}},
- },
- }
- pvcInfo := FakePersistentVolumeClaimInfo{
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_1", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{VolumeName: "Vol_1"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_NoSC", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{StorageClassName: &class0},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_EmptySC", Namespace: "default"},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_WaitSC", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{StorageClassName: &classWait},
- },
- {
- ObjectMeta: metav1.ObjectMeta{Name: "PVC_ImmediateSC", Namespace: "default"},
- Spec: v1.PersistentVolumeClaimSpec{StorageClassName: &classImmediate},
- },
- }
- testNode := &v1.Node{
- ObjectMeta: metav1.ObjectMeta{
- Name: "host1",
- Labels: map[string]string{v1.LabelZoneFailureDomain: "us-west1-a", "uselessLabel": "none"},
- },
- }
- tests := []struct {
- name string
- Pod *v1.Pod
- Fits bool
- Node *v1.Node
- ExpectFailure bool
- }{
- {
- name: "label zone failure domain matched",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_1"),
- Node: testNode,
- Fits: true,
- },
- {
- name: "unbound volume empty storage class",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_EmptySC"),
- Node: testNode,
- Fits: false,
- ExpectFailure: true,
- },
- {
- name: "unbound volume no storage class",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_NoSC"),
- Node: testNode,
- Fits: false,
- ExpectFailure: true,
- },
- {
- name: "unbound volume immediate binding mode",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_ImmediateSC"),
- Node: testNode,
- Fits: false,
- ExpectFailure: true,
- },
- {
- name: "unbound volume wait binding mode",
- Pod: createPodWithVolume("pod_1", "vol_1", "PVC_WaitSC"),
- Node: testNode,
- Fits: true,
- },
- }
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- fit := NewVolumeZonePredicate(pvInfo, pvcInfo, classInfo)
- node := &schedulernodeinfo.NodeInfo{}
- node.SetNode(test.Node)
- fits, _, err := fit(test.Pod, nil, node)
- if !test.ExpectFailure && err != nil {
- t.Errorf("unexpected error: %v", err)
- }
- if test.ExpectFailure && err == nil {
- t.Errorf("expected error, got success")
- }
- if fits != test.Fits {
- t.Errorf("expected %v got %v", test.Fits, fits)
- }
- })
- }
- }
- func TestGetMaxVols(t *testing.T) {
- previousValue := os.Getenv(KubeMaxPDVols)
- tests := []struct {
- rawMaxVols string
- expected int
- name string
- }{
- {
- rawMaxVols: "invalid",
- expected: -1,
- name: "Unable to parse maximum PD volumes value, using default value",
- },
- {
- rawMaxVols: "-2",
- expected: -1,
- name: "Maximum PD volumes must be a positive value, using default value",
- },
- {
- rawMaxVols: "40",
- expected: 40,
- name: "Parse maximum PD volumes value from env",
- },
- }
- for _, test := range tests {
- t.Run(test.name, func(t *testing.T) {
- os.Setenv(KubeMaxPDVols, test.rawMaxVols)
- result := getMaxVolLimitFromEnv()
- if result != test.expected {
- t.Errorf("expected %v got %v", test.expected, result)
- }
- })
- }
- os.Unsetenv(KubeMaxPDVols)
- if previousValue != "" {
- os.Setenv(KubeMaxPDVols, previousValue)
- }
- }
- func TestCheckNodeUnschedulablePredicate(t *testing.T) {
- testCases := []struct {
- name string
- pod *v1.Pod
- node *v1.Node
- fit bool
- }{
- {
- name: "Does not schedule pod to unschedulable node (node.Spec.Unschedulable==true)",
- pod: &v1.Pod{},
- node: &v1.Node{
- Spec: v1.NodeSpec{
- Unschedulable: true,
- },
- },
- fit: false,
- },
- {
- name: "Schedule pod to normal node",
- pod: &v1.Pod{},
- node: &v1.Node{
- Spec: v1.NodeSpec{
- Unschedulable: false,
- },
- },
- fit: true,
- },
- {
- name: "Schedule pod with toleration to unschedulable node (node.Spec.Unschedulable==true)",
- pod: &v1.Pod{
- Spec: v1.PodSpec{
- Tolerations: []v1.Toleration{
- {
- Key: schedulerapi.TaintNodeUnschedulable,
- Effect: v1.TaintEffectNoSchedule,
- },
- },
- },
- },
- node: &v1.Node{
- Spec: v1.NodeSpec{
- Unschedulable: true,
- },
- },
- fit: true,
- },
- }
- for _, test := range testCases {
- nodeInfo := schedulernodeinfo.NewNodeInfo()
- nodeInfo.SetNode(test.node)
- fit, _, err := CheckNodeUnschedulablePredicate(test.pod, nil, nodeInfo)
- if err != nil {
- t.Fatalf("Failed to check node unschedulable: %v", err)
- }
- if fit != test.fit {
- t.Errorf("Unexpected fit: expected %v, got %v", test.fit, fit)
- }
- }
- }
|