From f0b51f9f066ff53ae9d0250422eb77768a4baa21 Mon Sep 17 00:00:00 2001 From: andyzhangx Date: Fri, 21 Jan 2022 13:57:17 +0000 Subject: [PATCH] feat: upgrade to csi-provisioner v3.1.0 fix lint --- .github/workflows/static.yaml | 4 ++-- README.md | 2 +- charts/README.md | 5 ++--- charts/latest/csi-driver-nfs-v3.2.0.tgz | Bin 3603 -> 3603 bytes charts/latest/csi-driver-nfs/values.yaml | 4 ++-- deploy/csi-nfs-controller.yaml | 2 +- deploy/csi-nfs-driverinfo.yaml | 1 + test/external-e2e/run.sh | 3 +-- 8 files changed, 10 insertions(+), 11 deletions(-) diff --git a/.github/workflows/static.yaml b/.github/workflows/static.yaml index 9efaaa44..45c0f6aa 100644 --- a/.github/workflows/static.yaml +++ b/.github/workflows/static.yaml @@ -12,8 +12,8 @@ jobs: - name: Run linter uses: golangci/golangci-lint-action@v2 with: - version: v1.31 - args: -E=gofmt,deadcode,unused,varcheck,ineffassign,golint,misspell --timeout=30m0s + version: v1.43 + args: -E=gofmt,deadcode,unused,varcheck,ineffassign,revive,misspell,exportloopref,asciicheck,bodyclose,contextcheck --timeout=30m0s verify-helm: name: Verify Helm runs-on: ubuntu-latest diff --git a/README.md b/README.md index 719b518e..1ca67242 100644 --- a/README.md +++ b/README.md @@ -10,7 +10,7 @@ This is a repository for [NFS](https://en.wikipedia.org/wiki/Network_File_System ### Container Images & Kubernetes Compatibility: |driver version | supported k8s version | status | |----------------|-----------------------|--------| -|master branch | 1.19+ | beta | +|master branch | 1.20+ | beta | |v3.1.0 | 1.19+ | beta | |v3.0.0 | 1.19+ | beta | |v2.0.0 | 1.14+ | alpha | diff --git a/charts/README.md b/charts/README.md index d68d3858..1eb8fed7 100644 --- a/charts/README.md +++ b/charts/README.md @@ -6,7 +6,6 @@ ### Tips - make controller only run on master node: `--set controller.runOnMaster=true` - set replica of controller as `1`: `--set controller.replicas=1` - - enable `fsGroupPolicy` on a k8s 1.20+ cluster (this feature is in beta, check details [here](../deploy/example/fsgroup)): `--set feature.enableFSGroupPolicy=true` ### install a specific version ```console @@ -39,13 +38,13 @@ The following table lists the configurable parameters of the latest NFS CSI Driv |---------------------------------------------------|------------------------------------------------------------|-------------------------------------------------------------------| | `driver.name` | alternative driver name | `nfs.csi.k8s.io` | | `driver.mountPermissions` | mounted folder permissions name | `0777` -| `feature.enableFSGroupPolicy` | enable `fsGroupPolicy` on a k8s 1.20+ cluster | `false` | +| `feature.enableFSGroupPolicy` | enable `fsGroupPolicy` on a k8s 1.20+ cluster | `true` | | `feature.enableInlineVolume` | enable inline volume | `false` | | `image.nfs.repository` | csi-driver-nfs image | `mcr.microsoft.com/k8s/csi/nfs-csi` | | `image.nfs.tag` | csi-driver-nfs image tag | `latest` | | `image.nfs.pullPolicy` | csi-driver-nfs image pull policy | `IfNotPresent` | | `image.csiProvisioner.repository` | csi-provisioner docker image | `k8s.gcr.io/sig-storage/csi-provisioner` | -| `image.csiProvisioner.tag` | csi-provisioner docker image tag | `v2.0.4` | +| `image.csiProvisioner.tag` | csi-provisioner docker image tag | `v3.1.0` | | `image.csiProvisioner.pullPolicy` | csi-provisioner image pull policy | `IfNotPresent` | | `image.livenessProbe.repository` | liveness-probe docker image | `k8s.gcr.io/sig-storage/livenessprobe` | | `image.livenessProbe.tag` | liveness-probe docker image tag | `v2.5.0` | diff --git a/charts/latest/csi-driver-nfs-v3.2.0.tgz b/charts/latest/csi-driver-nfs-v3.2.0.tgz index 884c816d67e164a54c2fb62d9f5b11d7fe2e89a7..432c07d3650a9ab4bc0aeddc8b08646d1add08d6 100644 GIT binary patch delta 3555 zcmV<94IJ{59FrW7J%4ZGwzA)6eTunGak1UIlK&>@HIRpGlI;oF%?5G02Srf?v^2JP zLXkQo<-|>MKl=qq$(Cd}cK&Q~7BwFdTO^0W;mmOO8_J=gekAD>rO!vIc{m|b2Qw10 zH_umEtyXKV*R%gytycAatG(BG)9$tRJGr2J6Pc`r@IbH9;4OH#(p z1f$`s4=1Bjq0b~L)PHjvQ>7(&xFigX+>m8XWr?NU zCM=rQ-X#fM% zpaqNwQwVfq9>O)@_F<~fe3u}ud0@s9TuZ?iO1GU@J)1xC6@eW%B{6~+!64%xW7m)? z3@@Z19)EeB9Fnj)6%W8|S!SQ|ob>v|he4=lV5Clk2M`m*?$SZD2Gcst+CWE#;s2 zC!(~GlOt8D&J!d|PtJtYedzq}-mdA(#P}DSB7f_{t_N@}4~zEK&*YIX!tt@ANK_(~A!bvm_d<-uR2N)gjIp)CUaR#fcX<;3ouVqX3X`-C z?SEFh)UA{Es`WRk25UcYu!2)xHALqLLwIJDmUY%VH3ma&e&u9JC^MShhxU@*ZJ?~( zdL^t~^2)lttfW_a@mE~j?o$?bX9v!4gc5m(#;*#F#-YYSA}9rxN`{MAqpM%275AJ>L@Z6G^8; zW4^!5n%>TiXRFRq#s-d|KK%ZNw*!8rk&UdYq2gF>Y)>GCQi=aH#)*yJnx~&Hjt7Ch z(N7;^-TA-UsmA}iyS?r<{(p%A7Za?W>ZyV;YHh4I!i0$#M#ju-fEKdUZ0~{RU4IA& zCm3E;dnY1Npfu5`dJhRhqR<2?ea{0h)*osHzHh8kB_zay+gk|E(Y6at^QUlkXLxYL zd|x5MPz!m`kgnxYFSxsFfbU-u9Zp_brr%OoY5$iKWC=aY zF=N~A_ST08N0cKp%<;)IYk$G;ey{-R$ ziQ@adcSU*Bha(&6#mKc+On(ZIBvH+2+lx_?h-k7n>>G!XdGCUfaOY04RyLS(P3vMj z@Aeib4_O*j9CHAF0p$^L4IR@?C1}yR1^!h(dS351$iqf=;Ehde_{-een?jOg{XBA; z$f(TaumSITj=}>51IVXpit)8{H1AP8>n^A-oq`Q_TKq<0wz~J6&3_*jgmUjbe=reB zpQ64N@+!lBVW)u40%1x3M~o~KU?EcABm2cctq{>1h>p0*G0{VJj`&EW(*LA-@((vm z*078~pH|haTwM_4f^9A$9F&&dl=8+YR38}S>CL>pBJZMNEuhxIl9uU7!LSbw|16P2 z9?d%?k_t)n?2r+q+<(l}Olge$>}chOl4=?fW_Q|aswZ{)6cDk+q)RYaK>;c&y|l)2 zw50$I`2|R?F3pnZToy_cRV*mOslg(H0e5%(b*(iSXCcenZsfSYm*cJBPY*hcrH8)Z zmPC<6r4GJ(`{C&5{CF_x1v!9u7_}k57-zKA)Uk)HQX-4YS5YZrsy65~}HDHPKHN>DQXx z)eLPtr7YQ*QH-cuA(cu}v_fpTu$z3PA=b#T#G*|b%_*Ogo5?q;_$3cx8jP1 z6%e5;m3T3cs3w9%F0Z&K6Rne~|_6;XZX^^Q6bkybRiL8 zRnvLND8n2f>Nf}{qnb!{rD)6%#w8UJZ-^M5Z=icn8>pDerZ)SR|6Waw~k;&OZUgFxjSd=A+V zt$#^4ImdseRGMfizo+Ev_!m0Da27KBBqE$&?l?0nDeW%ERSxCZZL47UI#uQ(d^k=f z7^5W2E1$pbrBu4KWt)#w=^EEZ%CJQ6wb$-B%3Aw30ml>X1K)4|?RKl%t>pjqJA0k2 z{l7%nfr|-M@Y{z^zu4?w9_=ZpUM7YAhJR4e?6Jky5|)lBhYx2bAcp_KP%Cc-^n_?2 z5`klgN)b5&EkLH+-V-Qv8wqKTcp3UkFcy4(j{u*ohJIbxIZYy>aiAs9IG#B#WkMFp z$G_UDJhZ6Zc&oBmuFr!S7h;?j2|fnBt_b35;ntO5z5&E~2ZW?!$_ca3bSOA?F@G*x ziNkOh1|leBGpA2pxj-Uhho?dL!g(iUMuAnLwWzED4iAZ-x8m*Gk-KPckjyv z_hEI*cVyRMcQsv>73dy-0B;uFDKDXjW5T0lz{Q6R-lixjM0RYPw=!nwo1^8ycW;l* zPre+VU!EWTbUFBZ`19r2hl_uFUw=y_77-rQld9O`BO!?0wZ>M~?1viQQ^c?mNUQ6( zBKfj(inlSwJc@N+SF!I#`}x-&k1s8DclzPe@r$8%8{oV(7Vx5;WYM{_y!VqRHhrvD zs+su%vj*P)#{aB24P)D|4)}Mw-rocA-=g#n3v#yj{C)WR(~@#u**U*fYJcEiF8$%S zeyva*hUwQu1NVXapXH&wyY%`whR4kFGJip8qcLFB-u~Lq)V91pS&9G3AqoBGxdgl> z|I_VstNEXHue<&Imlr7&ysmn)(%s`4Q0h_x)fy!#?2|^Z@|dsu zQvWl9w}#`&nY#(k@AT0@z=2N z%9UwAu6zw6k@SO0-+X0RaP=L3I#1Xy0$fZ61 z>-zvcb2hWTF_JizAVCR?atyrSXmB~uLZY_=ha!#zhcAZ%h&!lMUNEM*Y5zL*y{=)zO delta 3555 zcmV<94IJ{59FrW7J%3Z(=btvGa!jdsD>&3(OvBcYq@7oZ8yjq7=K6 z=paizk{rmg+|PbXlAYL&L&BedGquqJB$m`_wYpoaU&{#<^&?3qD1AOu&BHN~I+&7} zy?eRRYPDLs-LC!LYPG8WTkYM>yLPv|*V^6N-QL@K*J^iL-G7~T(0V~0mX=g$BHy*1 zJyvyc-$)?|{en_a!Fw=idn8F_Um4M;v=^ZYB~7$_cGy3G(@%XkvMK{ZA>rqAfRdv| z<#7^Y56UX#O=je`gHF)$-g?HHQY;nwpAeR!dSM5!!v4G4yE|3;-)?s|_Wv4%#$<#& z4*-AXL5@nDOEJJlv5@ynOwR=dc7oP=3~&*lbAPB8IHF2R@_0!Y8o43MoXP@A z-E~;<6v_z=@gNLE%C%u7lt?u8K+60J6=?u{ z)Sv|n2~!AkXdc21;r3yw(0rF5Zg^nE6I@Hd7)rODSUsCP^A&+DI3+QH7(zeeAY<2% zD-18BA%7luo*a;{Iu#GVZCPfY@|^T~#fL$tXkesHg$EED`sXNPs*KL59<=uM_B?Nh zM5ofK4stSJ_^JPs6ls!6*G_p>dBPdx_(ibP4C3m%!h~nzFHYRE>cD77R1b`H%aot- z&qQe>sC<|jeI6rWdVDUV?m_2&cXv!*CPu&D1bDklZQ;CdDU7}bm|KaEV$L}GdiL-F{QDy{Mg91bzmJ!qVY zemKS`W!PAYekPBE5sptCMWPa^3^AKhy&GaQrn=x0V~njCc3Z8_xzCgM?-W(BRhXnb zXn(ijg>Id^SFOWYHdyq>gL8-K&a-G0vEZf(H@4pAZx(KuGY(RkEYNCbtUM6EJ^Yn&|Hn0>+Q zE0mAGMi$DDkT~qYSnEXfn$5B;MuT>rL<*E9I#nMbVMr93IHl)#0LJ)3&A|7KajJxbxPNyC!3ElO!D;>!?(Yo` zj+pN&WEg58_Z!l+T<8V&_YLs<8=}MUTg&uYDogGEYK$yFsRErm7YbNm|LxXZ_58ov z?QYKhuTt*rnjc_7;~pr~Fr*CiG=IVU*oY+@V-G$w@9#a&Haose2#-LI(eAD0@MS)S z7BjZ(?(Teua7Z~q!yKPXvj+I{8j>;%fkTuL z`12=u&%f+x(|F^?|1_X+)v&Y*!hdV!-IQmSrT%{reRwh|yu$zQ?Q|>gU#GRxZEgJj zYZTx2y=%&&9vsHH}%zGD5(erx8K^```^KNWn!(V3J-V~A~tLKs1 zL`G#Mhc$TLbrc>k7(hN%Q;e^r!&QFl&#;Sg-N)8aQ0v*o>KY=8bRCzN~l*@LlA z`V{qzkk=Xh3tI(z76?-UIAUax01J@kr-zIv<$q?Trb=V%XGbePlvLA@FuT*4i0( zfh`4O$S)vzb!nDN=dw_usA54GO%3K547k7Vt!k~wC<|F;b|Xgxz8r52f3n|cEIjlL zw=ypU^IC4hNerszBXCN^VW0r~}TxXc96W5-8@L&v@03$;JDMt~w-Y8AEOF-nd# zYgl0y{sNxqR=ZPVI2T$D!TRMS*#qiR0&Q1mPr31W%2QU8Weew8TCROfV-%_0-&P9^ z99r!sLhAi`=YL*iagJ`6m)B97>~|ieJ7wBWQJn{r!<6Nk^#s)^al2A;_)x8*E@tap zp(LisqG~7%Q|pNZ*LbVTf=MKuP=+InW~st?2}wc*lu=EwB2NGkMLqbv(f_S~b#V0g zDQXx z)eLPtr7YK(QH-cuB9%%`v_fp@HjzoGZkzZdMO;-FfFVT|UEpxBCaZJyX9B~(tc8gh zYDX?gzJGp~N;$plHJi(Pp7CM}dNYd-TuSbFPA=b#T#G*|b%_*Ogo5?p^6{iRN2&+gtx}t8 zvzS%4%hd@=Dx&(%s~vSBA~jds2b}eU$R?u$n}2U(sGFu^ZF4CSGRk~V<3}1%=|UpH zs;2XrQHD7})Nc?@Mm3S@Qqh5hh@@P*%^)e~^H-ChRW{)ktmaue0IUJmyfEfG>L#@0m&|{*3 zNCb`{Dn;ZJv;diMdrzRyZ78HY;$`SF!C3GUw=y_77^~(ld9O`6CsG5mBv=q?8h45bHuO|NUQ6( zB>A#%inlSwEQ)nsm$C0g`}x-&kFP9tcY5&o_|?$6HE`Y<3wTveGVfek-iJvPn?BVm z)y({nS%Yr?w@;VJXH%HNM#YYbSnx4$(swJEPqmg2v1KtlgzE&;E| z|7>@*tNEXHcYE{uFRxN6cwO~orMtoD-`{(_?|YRWi*d+pztp7$sx?Yf*e8u*br3)+597%*gn#%qu2VD`GP%Bjh^k+T*dBTB6~&o$?S6My9T36jfZ^AQ@! zvf)-wD^%1(P%k>6XAumAkP+oY3#c2z+U?g7TJ7q)1VP<+e|YnE916z$Bjpk0qpxA( zl`GT!ARB7!tW)Z;Y0k)Nz3e<}dyIEo&%YXPl$~=IUE_FM0be}__u z|AH~Hn2xxR_;O>cx_|$()7h>5{#R#blmCB}vIXZv8paAa^i9>pekxP62 z*Y^ROaW=I-F_JizAVCR?atyrSsDIVhLZY_?ha!#zhcAbHh&!lMUNEA%Y5zL*y