From 3cd7125d60ebc87fb74c9e136ca891851d80d538 Mon Sep 17 00:00:00 2001 From: Todd Morse Date: Thu, 3 Feb 2022 12:50:58 -0500 Subject: [PATCH] fix lambda signatures (#43) --- .github/workflows/sfn-wdl-ci.yml | 1 + .../modules/sfn-io-helper-lambdas/app/app.py | 8 ++++---- .../modules/sfn-io-helper-lambdas/deployment.zip | Bin 5560 -> 0 bytes version | 2 +- 4 files changed, 6 insertions(+), 5 deletions(-) delete mode 100644 terraform/modules/sfn-io-helper-lambdas/deployment.zip diff --git a/.github/workflows/sfn-wdl-ci.yml b/.github/workflows/sfn-wdl-ci.yml index 02ef15b1..8cdfb01e 100644 --- a/.github/workflows/sfn-wdl-ci.yml +++ b/.github/workflows/sfn-wdl-ci.yml @@ -43,6 +43,7 @@ jobs: docker build --cache-from ghcr.io/chanzuckerberg/swipe:latest -t ghcr.io/chanzuckerberg/swipe:$(cat version) . docker-compose pull docker-compose up & + sleep 1 make deploy-mock test terraform_format: diff --git a/terraform/modules/sfn-io-helper-lambdas/app/app.py b/terraform/modules/sfn-io-helper-lambdas/app/app.py index 05095d56..c3d2380b 100644 --- a/terraform/modules/sfn-io-helper-lambdas/app/app.py +++ b/terraform/modules/sfn-io-helper-lambdas/app/app.py @@ -79,11 +79,11 @@ def handle_failure(sfn_data, _): raise failure_type(cause) -def process_batch_event(event): +def process_batch_event(event, _): reporting.emit_batch_metric_values(event) -def process_sfn_event(event): +def process_sfn_event(event, _): execution_arn = event.detail["executionArn"] if os.environ["APP_NAME"] in execution_arn: batch_events.archive_sfn_history(execution_arn) @@ -91,9 +91,9 @@ def process_sfn_event(event): reporting.emit_sfn_metric_values(event) -def report_metrics(event): +def report_metrics(_, __): reporting.emit_periodic_metrics() -def report_spot_interruption(event): +def report_spot_interruption(event, _): reporting.emit_spot_interruption_metric(event) diff --git a/terraform/modules/sfn-io-helper-lambdas/deployment.zip b/terraform/modules/sfn-io-helper-lambdas/deployment.zip deleted file mode 100644 index 1b1b737017f90f0e547636d129def00e1bf22ace..0000000000000000000000000000000000000000 GIT binary patch literal 0 HcmV?d00001 literal 5560 zcmaJ_bySpJ*Bxr4OB$r4TUt;MkQ}Z@uriYd!1!bN71gJ!hT0_fZF-p_2eG?-l^S5OeqWzX0CtEZp39-28I%Ok9_S ziGucOC~A3$-&Byb@>Zcd6>VOw=&6ez1k>V%Q6|Z_--o2Hm?+J4^V)hVP9%QI{}@H|bO`g(7c+n}}2R?9s3iSBSHWStY$PAvQWb}2nu zJjQ1?gx3R=f@7k<2wJ`?{7I?UxiEGV2GK3@RZ-9NC;l&i=SmJ)4WO5fqo{`UC3Pz1 z8d+_^0VOSA@;*sS-5-Z#oZiWT@KeCzyfldTnlCN2BH!c)T*|Bj@^}wmoYBTJiUsJ z5IvS+j`qh-fbbCmuy+jm>3u#ylnXKWNHdeeb$m69!G{sSU z?1f8p{cyeCCWfY@x{2!kl=pvIIfdsyAl`^ABJ`e;nwCPDIv+qXta*Bb=Bx;{nM3n_ zHSvLR{=RnW!)VLrO-=@#3hXhF_&*!ExmF(`!pD>3bWL&m{PVxr>qociXyf9l&>PWR)!}GKzjMS+T&;CV z;txN%Ubd%7q)*FbGEqfjmwqW5U8=6V=vFltioeFy@I#|U^~bZmVfyx3=5gK4BPTQ= z(D>&^GvbxvhbvKfi~_sFpZEKknBe|sjSo+9i)XbMRE(y|KJw7%*48M@yyl{}Uj$7g z?N*2Ls`>b&=Y!LO(pSDVmOHtbn9A-`zG}OOiJvC9mqP+7&dM%5dD7_+H#7_3h&l#6 zo+qpj^mIm94cF}34A={N#y!Xm>IOpX@mn$)z=4=O?O5JLot2 zhBdP<3j)DE3B6C*@9{D73TXUVNQxZp(S&Xn$1>ug9WhNeWEUvwT|<+utVh0FK7NsD zKP0wOd_ABQYswCLea<;9=L+zVBUPtnwdt*ViIPTh(ESqygSK^di$beVvK71qsSwr_ z%8Adny|uQ>DMp4{v!SN&>#y7f0FT~QRP4b0{ho$pkIZr-1C4`BP3^AK5*pAy=(qmx zMDDoz1xchoCRtVlM~roTz6mSQ?1X%^V4(_lp-N67JT9VlxP}cuh`a+L4B7X(&tf=c zIq_nPnj)b?GN?K$5j$R8JhsMM0gV_VpgAfWrE}Hi(HXhhLIt~9)fk`e)#j$v3@D1L z`5si13UxqDkO%DoZAjzn6Ws%sUfG`@+^wpv!G#X*%yaG~)63KxwvLXO-`MW$b8EelEvGMnTvFGvw(y^AW zo>{ouA}q6^JeEU&3|u|WUr|zg+9|$ts129(&2qQ0IlUr)0{q;3Y`><-dCrT!OZ`bC z1ChTb(D$pz;%YqiX&AzWmpS_MIHOOK>mdal4GVZ2hmF%48ewIKeuP6U(PP!5_rj*B zQ6lCcp`e@+m>a@KKl5zy3gK{zN#=Ktk+&tsF!L>BfuP+;@BoHx0Uy>J*Whu4Q!9QI zM)sM9*d*UFxG$rxUj)VXOLbLf2oC-zuo3>TW&b0G=P^Zf^H4oOpm45K0Yowri0(>2 zF2kzysWvg_QF`_j#Yv3XO!|Qb2V>AK^>CWypp*f{kDRc)hY@d>MYf)?!22B}6Vj%i z|4cZLi0SuU1hQR<+Yh{p)9qz6*4r(R%bVG_m_Af6OEq79vPz#n zGZGbbd4uh1O?$`%P2mq-6M|z^XqbCnr zu$hCanZ1p(n+=55%*+Ao;Av)d$5<(e6R#Kf2%})ZF-dB|F)u$aCyF*Y5~OmJ-N*+# zV}m54bPO#f+2L-U=()dmWx}&9c=n@fVyn)zlK?j_yfr1&&J0rugKI#HKtUpsDz#YJ z_C@tUrToEhiF*U~VEy-MDePKbtE%pj0};s`6s%FP|Uhv%)&&wP)qwFS`HPf13SynE0E*i&@UOO3PY*1^@t;2pCD>tm=$9GX%Go zKem)e7n3d5ZyBU)^o`ufI2IpK8n*#PtOgo5Yw^$ztiOo)VZ`_u^DywK>Eb3rOwsk* z9Hm_9s-9_Z(4e_(Pac29*y4=3N$E0zj}LS1<2|PC_q0WNPw4wzk8BVTFlnW;9LLbA zkk8EaV)o|El1v2L$ISNR&{99d6*da0q3mkttERNseEK^X6D4~Uawq`6KI(r<#)E$( z!_vaj%HGVz+Xn3E@hc%(6EQi;gx%jsB3O6Mp?L4PeHG}VK1d5YgL~<#(iY{~mA$;Y zS=H#;X5CpDaE53i>#J^VIgQi=>HrJ&RX~t!h5VtIOWTPCPCa4#-fEWeNye1Is+fD6 zuwy1m;!LH=J*O{W&BWZi@RLeS%_~a%kMF%o$yEAn&DYYk@$^9?evZ~`!gzjBaKiCq z!I(mz5k*rfU#<%5A>aia{sVt$=5hb|Fa9#DMWVuP6!>S)X7^Ym zq(Qi7_6>vM6AeSpX7vb+E5D-YsD&2jznXq=wq2@tHnc}bmr!v7bQx>w!C)I*QpvBe zowabdlYr3cj&kH%C)HzoW4EF!Hh~=LAvi`-<4F5?8nobG$I7lCx#C#>i z*#uF%n~mELbc7iF+-eB^RO$eXvZBg00RM2It}oPAa-j-T^wA)H$BSZSw}p(+x!yqC z_*oc<{Py8B=ulYVG)DKDy1nMz=!?_?zV{>M&!Ls02YpK8k-aRWUpjwAm$;}bIB|Y@ zoBABePMSSiz6Vb-3qEh3J^v)}JygXVkHl?o=(sqC(&!ddwzUqSP+s?OoMFz^oA^6L zSNPn(Bqd$%1>!Zc^|^s!K0&bx1S3nz+337mK31SLEpfL2y8Te-EhC`_IE#F96lU^X z1Di8v*WS^QPmz4;_NLP$6Q=IH*0RvKfuy7H9qjg8H(K-jD*n6JT3oS*8Se^>1>?UJ z8`VFG4PxWw3h{IR+uc=Khwhr|k}zTLm6`jxX|ak|?o+ZE0IPQ-NqlZ;OX&kwNtA3u%?YPLnDGK-tvciS@O0rM4a0A5)o(pX(zb|5=fuAzt_a_EUrXsc4;_11ybD!Qc zF;v10{nd%If2b;`ic-b%k4kY6BeX0J1!b*Jp(CFNS#j6m0RRdsJq$JXiWpp>ZKW`& zOand|CBeuwhD22SkU9M#filO22t1LDP&qAf^i*BpwnU$}I`C#rY~kFsBul@h`lhdv z5)=3Ib||?}7JyD&6FoWIo+Hn)a+G2xY*MLu{^F3sK$&;Xx$~YVFUH5rk{beQ{6NvI zS~pRz03LhZ_f?mYDxFWva9OKKn5BswP4t@Ss3>YiClZK!0mQp@k}t=A(o~!LvZBi3 znAOtU+APyRD3)<`I=$@puqvot$M6X!C9ITf@ERJe)-yHI!Hz`(Il~g*J0NRldxkT? zpq@Yoi}d80wVK4=Yb_PL0H~)+!+Wl3Ln(e*jx04aAOeN+>oDgpF1{R5cR$HD4a}IB zUmHI&by2SOg9JBSiYLZCQ@}>)YS>DE;SU7cD9EQxH{n<+URlv^gFm2s+1e5>E?&?S zGpEebj>TiSa)gS~_4Y*4ea95Zlvs>esTgzhZdzxqznmxhVoi(T61LJwbG^u_lN?^@ z-r`WVV5|}+KiRuCJ^nIeOynC(zmTc-}Sbfhsj)J0L5N zS3m<(CJm#v2Xj$+>toLfVw#i?=5(Ng|5F&M{g8E(ST)MU%4U%V_pBcKbk??ys+YY^ zs3Qo66*|O3<5XFBjB&wbUA?)bDjv9O^Ry>Bcr{-P4P}Ko5zO)ZL4Yw1jn|$doeiGk zCpUhG>IrUNEyN_?V3phBkq6^-7juFv-+g8!D@ob9mY;DeFkOnRgpD>637S+_*BdKz zg61rq$vS#26~dbX!p9kHxc)5h@JNOqxX_?Wl}A>lI&2N*t#3F#vnz@16@EDGbSc0D zm%PakV&$SLZU_pLWrbn|uDxw_ITY%+WjvgmsY6aCRAepIDD_8+TXsTDZG}+HhU4xh zWGl&pL|wvslfco<;wd0B!JfjfX;VKf}zWwVm z$9}AZHddN}EWXdo6`>bMUD~w0Sihs&S79!5W9CwjLKw(N)3OdeZEcK?3GrR6NHq#E%{q1go>%`02T%e0^a^Hvt z!9>ub4sbC){Uuii`|&EciDV^!S&x%Px540@H6F;pnt@bWzmvD5+(XI<7wFJZxV8iF zi;c(}<3i`i@X?PKPC2k=wQO7>HM%VHm6A~S_N-o;!A2v8pNpxN?G(LfZej|Tp!ZO;&B=Zj+@tMyy+6*v7lL z)P_nLX;AP)0QX1*k>mxNLL%d^Y5x(|9saAbFpAQ6XU5I_z~m8+_9voyoULrR&s{!g1A}BO|_EGeG^A0}Y+eq~oNIcpH zEKMn+cqW{w)1zY08!i`WIXt<9Y;6%z5h6v!4&UtD9x|nO=FkshtrRJ z+j4Be?XTu5K7g0IO`xreJivC#ZtVOAzzg{OVo;gJ6;U7&w>wBfnt+Jg=fDGMOmmQfBZy zr!HbLNMAEPufMtuyyY22;NIa2#h4}~&rSq=M|=R4sXr(E5?0+rPws7+$o1MnB1+^b zyPEx+h*a*4*vc>mp{uZIR*zpl|TbOU#&boT$ z@lS;H0psq{lIYS)blowHeIS~bAZ|Mzj@=|{cHfo2q8tYbw~>yy+0KS?kibQ!JFm@4 zX=9H9JW>ofT6Prbj_8f>{X*8n2TUiO;vIe2&Xc*fZ(9~x2`p`_X6dZEXKopD!QXqr zt~h8r^~T;dcz1q5Qj6_(q2AawZMMhDC5K|9ET$N1>Nh*pq!!PB&3s=Cko+&stJA zBbd8c7u=SKZci^RkJNQGlx9WS>ng3nlZ1qvGGJn(RojeID9ms8W>5Gvt*LL#SQU z)t!G+2ce*np#A?i@lPlBYn=EynnQo>{xw|u`^=w)<5x%i9nGQncM$*4@%;_+rFZJ&?JH!7Q=MUyl2Vwm0 S+hG7O06KS0Ooimv5Bv{my#P}H diff --git a/version b/version index a60b32a0..622ec544 100644 --- a/version +++ b/version @@ -1 +1 @@ -v0.14.0-beta +v0.15.0-beta