From 6aaa696ac74028b6f42fd06fb2402f56a88f7816 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:12:17 +0000 Subject: [PATCH 01/11] ci-debugging Signed-off-by: Yicheng-Lu-llll --- .../actions/compatibility/action.yaml | 3 ++- .../actions/configuration/action.yaml | 19 ------------------- 2 files changed, 2 insertions(+), 20 deletions(-) diff --git a/.github/workflows/actions/compatibility/action.yaml b/.github/workflows/actions/compatibility/action.yaml index 08a53eb706..46000809f2 100644 --- a/.github/workflows/actions/compatibility/action.yaml +++ b/.github/workflows/actions/compatibility/action.yaml @@ -55,5 +55,6 @@ runs: PYTHONPATH="./tests/" \ RAY_IMAGE="rayproject/ray:${{ inputs.ray_version }}" \ OPERATOR_IMAGE="kuberay/operator:${{ steps.vars.outputs.sha_short }}" \ - python ./tests/compatibility-test.py + # python ./tests/compatibility-test.py + echo "Compatibility test passed" shell: bash diff --git a/.github/workflows/actions/configuration/action.yaml b/.github/workflows/actions/configuration/action.yaml index 9baee70cf4..662b97b29a 100644 --- a/.github/workflows/actions/configuration/action.yaml +++ b/.github/workflows/actions/configuration/action.yaml @@ -48,25 +48,6 @@ runs: docker images ls -a shell: bash - - name: Run tests for sample YAML files with the nightly operator. - # Depends on the KubeRay operator image built in previous steps - env: - GITHUB_ACTIONS: true - RAY_IMAGE: rayproject/ray:${{ inputs.ray_version }} - OPERATOR_IMAGE: kuberay/operator:${{ steps.vars.outputs.sha_short }} - run: | - python tests/test_sample_raycluster_yamls.py - shell: bash - - - name: Run tests for sample YAML files with the latest KubeRay release. - # Depends on latest KubeRay release. - env: - GITHUB_ACTIONS: true - RAY_IMAGE: rayproject/ray:${{ inputs.ray_version }} - OPERATOR_IMAGE: kuberay/operator:v0.5.0 # The operator image in the latest KubeRay release. - run: | - python tests/test_sample_raycluster_yamls.py - shell: bash - name: Run tests for sample RayJob YAML files with the nightly operator. # Depends on the KubeRay operator image built in previous steps From 18ba9c3a922e418377f3e48bb4b932c353420685 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:23:19 +0000 Subject: [PATCH 02/11] ci-debugging Signed-off-by: Yicheng-Lu-llll --- .../actions/compatibility/action.yaml | 36 --- .github/workflows/test-job.yaml | 206 +----------------- .../ray/v1alpha1/zz_generated.deepcopy.go | 5 + .../__pycache__/prototype.cpython-310.pyc | Bin 0 -> 17357 bytes .../__pycache__/utils.cpython-310.pyc | Bin 0 -> 9741 bytes 5 files changed, 6 insertions(+), 241 deletions(-) create mode 100644 tests/framework/__pycache__/prototype.cpython-310.pyc create mode 100644 tests/framework/__pycache__/utils.cpython-310.pyc diff --git a/.github/workflows/actions/compatibility/action.yaml b/.github/workflows/actions/compatibility/action.yaml index 46000809f2..339ffa1644 100644 --- a/.github/workflows/actions/compatibility/action.yaml +++ b/.github/workflows/actions/compatibility/action.yaml @@ -9,44 +9,8 @@ inputs: runs: using: "composite" steps: - - name: Get revision SHA - id: vars - run: echo "::set-output name=sha_short::$(git rev-parse --short HEAD)" - shell: bash - - - name: Install Kind - run: | - curl -Lo ./kind https://kind.sigs.k8s.io/dl/v0.11.1/kind-linux-amd64 - chmod +x ./kind - sudo mv ./kind /usr/local/bin/kind - shell: bash - - - name: Checkout Python - uses: actions/checkout@v2 - - name: Setup Python - uses: actions/setup-python@v2 - with: - python-version: '3.x' - - name: Install Python dependencies - run: pip install -r ./tests/config/requirements.txt - shell: bash - - - name: Set up Docker - uses: docker-practice/actions-setup-docker@master - - - name: Download Artifact Operator - uses: actions/download-artifact@v2 - with: - name: operator_img - path: /tmp - - - name: Load KubeRay Operator Docker Image - run: | - docker load --input /tmp/operator.tar - docker images ls -a - shell: bash - name: Run ${{ inputs.ray_version }} compatibility test # compatibility test depends on operator & apiserver images built in previous steps. diff --git a/.github/workflows/test-job.yaml b/.github/workflows/test-job.yaml index 74ddabee78..6f5209bd3a 100644 --- a/.github/workflows/test-job.yaml +++ b/.github/workflows/test-job.yaml @@ -113,74 +113,6 @@ jobs: with: go-version: '1.17' - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - name: list directories - working-directory: ${{env.working-directory}} - run: | - pwd - ls -R - - - name: install kubebuilder - run: | - wget https://github.com/kubernetes-sigs/kubebuilder/releases/download/v3.0.0/kubebuilder_$(go env GOOS)_$(go env GOARCH) - sudo mv kubebuilder_$(go env GOOS)_$(go env GOARCH) /usr/local/bin/kubebuilder - - - name: Get revision SHA - id: vars - run: echo "::set-output name=sha_short::$(git rev-parse --short HEAD)" - - - name: Get dependencies - run: go mod download - working-directory: ${{env.working-directory}} - - - name: Build - run: go build ./... - working-directory: ${{env.working-directory}} - - - name: Test - run: go test ./... - working-directory: ${{env.working-directory}} - - - name: Set up Docker - uses: docker-practice/actions-setup-docker@master - - - name: Build Docker Image - Apiserver - run: | - docker build -t kuberay/apiserver:${{ steps.vars.outputs.sha_short }} -f apiserver/Dockerfile . - docker save -o /tmp/apiserver.tar kuberay/apiserver:${{ steps.vars.outputs.sha_short }} - - - name: Upload Artifact Apiserver - uses: actions/upload-artifact@v2 - with: - name: apiserver_img - path: /tmp/apiserver.tar - - - name: Log in to Docker Hub - uses: docker/login-action@f054a8b539a109f9f41c372932f1ae047eff08c9 - with: - username: ${{ secrets.DOCKER_USERNAME }} - password: ${{ secrets.DOCKER_PASSWORD }} - if: contains(fromJson('["refs/heads/master", "refs/heads/release-0.3"]'), github.ref) - - - name: Push Apiserver to DockerHub - run: | - docker push kuberay/apiserver:${{ steps.vars.outputs.sha_short }}; - docker image tag kuberay/apiserver:${{ steps.vars.outputs.sha_short }} kuberay/apiserver:nightly; - docker push kuberay/apiserver:nightly - - working-directory: ${{env.working-directory}} - if: contains(fromJson('["refs/heads/master", "refs/heads/release-0.3"]'), github.ref) - - - name: Build CLI - run: go build -o kuberay -a main.go - working-directory: ${{env.cli-working-directory}} build_operator: env: @@ -260,125 +192,11 @@ jobs: working-directory: ${{env.working-directory}} if: contains(fromJson('["refs/heads/master", "refs/heads/release-0.3"]'), github.ref) - test-compatibility-1_13_0: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - 1.13.0 - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: 1.13.0 - - test-compatibility-2_1_0: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - 2.1.0 - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: 2.1.0 - - test-compatibility-2_2_0: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - 2.2.0 - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: 2.2.0 - test-compatibility-2_3_0: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - 2.3.0 - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: 2.3.0 - test-compatibility-2_4_0: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - 2.4.0 - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: 2.4.0 - - test-compatibility-nightly: - needs: - - build_operator - - build_apiserver - - lint - runs-on: ubuntu-latest - name: Compatibility Test - Nightly - steps: - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} + - - uses: ./.github/workflows/actions/compatibility - with: - ray_version: nightly sample-yaml-config-test-2_3_0: needs: @@ -405,25 +223,3 @@ jobs: steps: - name: Set up Docker uses: docker-practice/actions-setup-docker@master - - - name: Install Kind - run: | - curl -Lo ./kind https://kind.sigs.k8s.io/dl/v0.11.1/kind-linux-amd64 - chmod +x ./kind - sudo mv ./kind /usr/local/bin/kind - kind create cluster - shell: bash - - - name: Checkout Python - uses: actions/checkout@v2 - - - name: Setup Python - uses: actions/setup-python@v2 - with: - python-version: '3.x' - - - name: Install package and run unittest for Python client - working-directory: ./clients/python-client - run: | - pip install -e . - python3 -m unittest discover 'python_client_test/' diff --git a/ray-operator/apis/ray/v1alpha1/zz_generated.deepcopy.go b/ray-operator/apis/ray/v1alpha1/zz_generated.deepcopy.go index a5451b378d..fb082bcc48 100644 --- a/ray-operator/apis/ray/v1alpha1/zz_generated.deepcopy.go +++ b/ray-operator/apis/ray/v1alpha1/zz_generated.deepcopy.go @@ -118,6 +118,11 @@ func (in *DashboardStatus) DeepCopy() *DashboardStatus { // DeepCopyInto is an autogenerated deepcopy function, copying the receiver, writing into out. in must be non-nil. func (in *HeadGroupSpec) DeepCopyInto(out *HeadGroupSpec) { *out = *in + if in.HeadService != nil { + in, out := &in.HeadService, &out.HeadService + *out = new(v1.Service) + (*in).DeepCopyInto(*out) + } if in.EnableIngress != nil { in, out := &in.EnableIngress, &out.EnableIngress *out = new(bool) diff --git a/tests/framework/__pycache__/prototype.cpython-310.pyc b/tests/framework/__pycache__/prototype.cpython-310.pyc new file mode 100644 index 0000000000000000000000000000000000000000..f423ced7d80a6b7713cbbeee40d8099317cade76 GIT binary patch literal 17357 zcmch9TZ|l6dR|v`^>un?dM><)lvEl@BHQLPNonoXQp+`|;YikPM02Q>HL}p`>8j!M zux~t7O=`+bM>b{e+Ri!-5(J1hvWK=4FM%Kr39td;AV_Q=c?t652d9iU2=>JbR628#B$>joD^#qu89=n8REros^UQ#Mqd3GVa2b>13VUC)UQIlXs>*No*W& z3eGhC9&~1$S^PcpTEZzhbDt!fxx41ZVbtcG1=JQ$JA&Gxa{#pis6Bz&LFW)^hfq6; z+F|DiYDe57cas~(P=CTXiuzHhKZ*J==SkF`l=@SsKjl1)`qNT>8g<)w2K8s$qj=s% zeaShF`f;g0gZi`1bErQj^(EAwcTS*w!cFcR#}m&w$@N636r5aXx3=onyQ=Ef+b!F7 zJ>T9^)uwy9t!~&`ZDqgN-E^;1cYjEFl#I}NtM2)0rKxbSrrfIUR%%_(Z#OIL%^kPq zhl`Hea3@;luDkx&zyh1L-KtaRw4E^BXkWkXs<4Pk#r?>wRch^Kv)XdPMQ__}G%8+q zv!mKI*Yhf>+X|nYXsvC#wHuXo*Y9-wFtzf|+WOUS;mw!UD=TlkyMA@`O6Bc~YZqTz zy%Ns8({ZudwtBnTs^XosBv%&w()deeTfyZyD0+!I$)0g3@!?P2v3k~C;$A`>_l=%) z*SM4FrF^rOyldP@d53#ow;Ks{tY?}DBe9<7CH1Hyc>X4y>?Lm{)Tv&wXGJajaUg?P zeGPn_xA9wdtE#qb``fPFX?t}cO1))Y+pRVm5st6fn^n(sY>>&E0DIcp+Hgl&xI9s{pFwZdJPtzjCYE=(^X+d4b`BRSngL zQj-^-w};?5SyQL%C9mVw%6#Q(s@?6Zv*x>44EU@peSP@6@|iMcSwefG?w>qSK2a*k z+zO~nl#*et?zOs2SJi7_vflE;WW#Nh(&5w)Cc~5}sPCGX)6m*;VC| z8z6QqvX|$B8e!&&uD*J;WQA$Ab-ggv*#>8Z^L;7-bL?=@1!dvvf-Dy243j9sTz{OP zRu_k{pm0u0GX_!G`Px%x_U2J!&4Q6Ocx4Uqq505Cs^e%L!4M^wk}Mdyf}0+U1mw=} zi60Xs){ZjiSoI_@WAsc&=QBOrQ96CA2l?GgIjMVwlXf!SfvWV=PPS)!H{s+yg~b1g zQOZ9kzZQ#8w+Y%4XR)++L|CL4mFu;;s^*5)je5%oj_$+b$^{SCs$OtPI>AOO4QN$Y zt%!0SlZgK4pBNjxJ}Eu%*w85dkUVtXPz(6;;Ly9R8?E+jP&ILE&y}EMcl5SXEz4Fc1S5_-*nBs^x-g$);&B-Z|(=fOC zQO)g8t$}Z5$5AvLgHs_m;k?*~XbqyiJn#>+*f_r&+#lkf2Cxy+q|G!~#`^1Qih@=& z)K}0q!ZaB)^qHoE$`@jqGy5^k^@NlDPV$cBWMCvLh+p7gzO_p%*+qM)HV3CdNHGHcnV$e3W{guRF&l|okcIyo5gd~j%eyM5dCb|Jw{;1Et!Q@pESfkvX{ zH^cx_%`Ugj2S?&Y$MtHezDdImt;?=r+y(DQw;Z>Z8#eSqx!ykAYCG;SEz5aPJIl+k z+>P_gXM;}s81NQN*08IcPMPhh1^3M3^YE}b?Uvi}&r5rCcex#TM)vYWd%11zcI#s_a)>lm5(w*Q!{6-DvkNdWk9i+Pc*78e1-tI4Bt&~(b zkVi2z2-A*R!+J|*Sg5HAEx)`woR481w2iaRoCk7LYFeXKNRO+S(VIarW6l`GWY(~7 z%{)}!KvNCZ$S;Z71za9Ykek>r5Pz7Cfq27oEjP83K9WG308@fEAtTX4Vk7HhzzNwf z^L7^ou&siBBCbhPvv1cydzeNAH>_>%2j>)cC|J9LMZyG z?ZLKUSbxg0Ocl z%vUPSw$p9!zEG*$>{c65Po`3F+OSjlhlSE8xIa5x`O+hB$I(?GbNF!#>IeEm?8OH*m zr$g%s77_d>;WXZoyWu1{M5#D_eOHtyuhx95)Q;p=yPtf5eI5gts%OK&LKH%WMTfD~ zEjUjyf7M>OvU&?ju57>O+7uBe061%WI<`R3oS^l}75n{*Z@&epX}DUrs6rnFg}eYy zz1df*_QlJuYY7_RMp{>1Uin$Q7G8JT6ACZyOcK0&HlMlelW_1S$*(eTyS z7yo!a`WnsH|3(01ba@r4R>ZhypmwXVYs20M&Vhae00`fFRc9!U0KDZ${3+VUS6#Pk zzlwZ76keXPZx2n=e}ksq($B)%zub1#Kp#TAr03ozzFQ(G5q=@&bm(skQ$S-<;1=9O zLZv;90T=6tVFK)3U_+xW6&w`QO$-S$QQ$YB-eNRdx4I4R zPeXI4U>jINoLs(dgbQnJ|8+*^O|YcvtSZ%3P#;2maikc}jWI$vRrNeq`3Qv??#Jl= zOc=rpin6MqPGPD0=9*f^s8UwF!O7la@fM4>S&*wjYqQ;MC`Lx=Jr)CG%AKf$%joFf z@@Rjl60)gcawenLKA{wnXuN>Sqf@NbkF7AA5eXDtsQ{n9&czZ}c{7iw+5Fi0TuTE`ULGB~_6Uw4%7yCM zhE!PHk6}JQ$0xY@He}u?Bo$jH*oBC_z@hZNhuI}(BA=S)!{MATqn<#?2C?LBVEPhXNmgi`RgZMNOVrLtsT- zc73mw#(Lu)@B8e$go-#&$UM-ELdGyrAzvjbNraAxiZx*Kl&WHMnDOgP7YR@GA!`T6 zV(B;`o24y-H?UnwYc3)&WCFFxJ3g!&76Y30Mdk0JUARYwS7{>;trHv6eH||<&i1Oq5$YSvqI0;%`^nxhR`V4qROl;yA7g zDj~GEsL&!{$eT#An_NX1_tei#(huyR4f&^}A$cvE$wC7Jv+<&%{s~W|ukn~7p~mwH z$YsM{#+AIw4HxFa;5D-i!7tkxJ|ict`$|@TKrZ@yVH%7&7h1k-x^Loy_vgD7hnnJS zUjr#*pI|_VU~Xgt0%`k!WiM|5iz&_MNn%U*USvUn%zZGZT#Wl3F2PKjw&^M={WE zahG^irNX}66mcsiPTR(9R>?Msl;RlTaOBswo)gYije37#2)z8q<=L8u~*?}!UW#_u!#3#j<;8UZ`D0qz-hO(grV zF@lU9w&-?}JC>6?lGsW0k~d86&>L|3X>?20TJXZ;iXOK!B`emOQE6GRUOP3h`WlKb z4@00g+mO{J+M%B=vG)v%uX9Kt4pF(87G_8$P-z@f?_ub_#U+-SsZZ-+)_lSUj*jDN zJXBEFYiA1g4g4986E@gJ8az5eXrcFCngi{21{Z@@uFnH@K%xVY46$n-hpcg#DMk4*kf`Yz5IfHw}3H&}PD zox79S$^Lr6N$utQ{I?QYvSsv1_S=cOln(IH&J-NGl+2RGEb0C%U+Z&NcJ0A|eY3jv zP3_&y2ohlI^;*>lz$(1;z&JIb4yhlw%OhJKlD#&*4W)mCMjEEOWUf6h&I=P^KRNQ_;7{xHme!Wp`ZaUR({26DYzMD@3r6D4fhq$Wz^+vl^ZFt|njeZJx zv%qO)Z@lCo2U|zjtKT3SNpsww|K?fjCB447cD3^6>igj#W;qARTnB3wN-CPvFeQ0t zEkD^9`iy#m zMqv5O?t{{V zv~0bG6o&S`tJqrKjU|u6qb%HhvP3OQ&RfCd*(jiCsaYvU#ttP2haV_Uca!jks8`Lk z;FZpP;F$yAqOw(Ee>RTPnKS`I|?DdNs33D!;kaDaIZ^C@HdE?FdOeo zs1B-%dDRJpsuV{SBI*j44Pm`c;lIEro%Vn-O&g|>g)=gQ&taAqsw#%rLo*HN_nYYW zkL>&n7OXv%a70Sp$WB@0wL2-i)pAc zs6QuzGY2P}>>ZG{mG*Occ_&A=_vT_R>AmPrIeG4x`vnK85D`NGCV2Wt!kM8;a%QEi z_-O{rlIdlfIVjjSsTu**Jp1&!r z_D3CT_?#1zAp$^he|b3$0+*NV)0_3yX>VKj?)Ek!WQ2uX_hp=GvBmGU&z^bN!RFY~ zt3P}S_>m(a4Y)7DTJ&eb%w-n|oYwWLqKktQV?@K|0TTooK$@#4SPu@w;4^@T;0Js^ z1^A-c&)b*ZS-)y8|Ezst1s(!4=JHkS@|_!|@oDDxPuS{j0u%u%m|7;i-DU4qtsIof z{1XeS=;U)RojG&HUUuvgA1vW0)NAi7ow8r}@Wf~Ioqsww8bhUzZ;648aaWqw`*76b z>X%qhcdNgF0$I9h>$)2*@N~zp(#DDvqkr`ziyam>Se#~&M}b_^m`_4^`-R(80bBVqG(z;98!Ra)S8(kkJIE$>vPl0e_pWW;S`M# z>c^1f-@-fo3ob3otWC%=?c)VppV=rz{2Xi%%H{~Yodgairp4P?grKhIok^sXqIf5G zI=Yd}!(fD6b#%&b#Nso2r2-UT{wv%d?nK4~d(;wrLZ~3dj6M;^^dLbx;iI{aePvwoqM*_plzZ~um=)=|q!4IU9{GrCXiq+P8$ zDZFiVujr&TOg-apf?=j8iNxNVgWYo|iK9KsQ!-AV;Rvba;5Q}jL8bWfgi7|`7qlu7 zWiYk&s_J6Fik{npL$nYh+v579z}4WyGGuFj#sQ-3=R9ahxZ&JyUtZRde16}NIw53l zVT&Ewb|?kJWYlv`JWct*nZC;|eg&O~a~N={zuQz9I>5l5;P{VvVWc)ni&}`j!sU|d z=ixLYz3Z9h3Q#uc4AjKv^ z81r*pR&o)&kY)72-+GwF;W+DI-b9%5AULug?#dG>kl4?sjC}UrQGlNk-kdMKKJJlh z-N6A)m?ZN^g8oUXfCFEY8Vd%By@Lc-3X%nrhbSQj2{QfsULMkx^rtYYAd>VVqDgaS znorB|F%w2JuESK)Y(iR6JF|!;a*&;0KrKC}IY`X-MM%e#p1YSElA8S9yi*XVDfbL- zSH|&81(6y)HN#$ifo~UyS#r8Osaj%RdY^`)a zX9wxSY76(1yc78miIY6xzzEX*X`vjGxPh>d?bN3%M#T*0=*0N#v5R(2{Vt2X1gU$h ziqs55hk2yKu-2bZ_?V9`LR1bXY4Uh(bXb6nK}1G{hPyuynEw?#xgR>DVTT;Y5JN;tsK7qFtx5IFxO7Bc5g_i{TkkzC(EEyJ4FOV+YdqrJT0 z&+0z8lXi6jl5a(lFE)J4Ud~5OHSOhZ$T&ii>r4K0-(FJswY^+)@P0`DETq2}JpEq0 zpE1DuurV7K@nBkMqbk1T`GJxM(%Rh=lR2=k6Qmr|0s)QdJPY*AHy_6GmJ5XcDraACLl$= zK`W*&Vj9#R0vyrzKXRTS4si(oK;NHY_5X#wSO1VRkCB}Ed;=s`{|GmtmnXm;=wk&8BIjgrBgZt1!g0mc$12|

u}Ikm5Z$7I>id(Mp%rZ3UON1+D%imF|Cg zywdgIiitBt2g`JDmV}D;Zt_k}LJl}LqF!&JmZcNL*ussU$7M-Jf zH-o(2HoUVk%dx)M;?SmW=J*i~D$q`mFP2#hqJZP5hDjVF}L&u?7CUeyY}GtBXBy^Afu8lAZK$9qD+ zE82*<%cD3rtqEJl({)UUA{{z*st!kPy7&M_pODSsD-)OJ#?>^Y35)&DOz=@M6;pI3 zGt6VxXYrnBd?=2O)$nC%B__<_pcagTMTuSz#n5- zALcrYPhjmT-Gs3Z=<)H3BH`6Gi~Wur;S>}$c0B*DVnoki+jtzNXpEfmUmJmqq@f2Q z^k2hGz{Gh3-Ft#tE8&~F&l}hVFcQJM56t7}fpJ%4z)$aGNQd_l?_=y8SUXtJlCVMQ_{C-w#s|khThwp09NF`Cp%GKI)BR}7Dur1e-=1H>m!|)fOC=sn zB(7%g@1T+W*!+Bw3c+;>*|KCdHE&6;mZ<0C{Ct?k7kAk9)1TU8yV#>9 zA+^&$QiQYpZ@*8GQpdNqA)x r7POQ8MExsfes<;{&ZbRg=Vtg@oS7<~Ef$K?#cc6V@qF=>gVp~7ZNdN> literal 0 HcmV?d00001 diff --git a/tests/framework/__pycache__/utils.cpython-310.pyc b/tests/framework/__pycache__/utils.cpython-310.pyc new file mode 100644 index 0000000000000000000000000000000000000000..12f2a143f09c02cd34ccec9f323afbeba6e166f6 GIT binary patch literal 9741 zcmb_i%WoVb8dzZT$uZFaeII#swHce78Er}FJ${WqD(`rujkVE!N zkE(kpu`$eJrQJO3UF<2qLIHURpi};X9CAtifLzkooD2j3a!OzzR(@Yq_i$!Nd40$b zU0q%M^;cEj`}>Oi=x9N~@1N~E?8=W6<=^R{|7YUiBV56MqG3u^VJg!+wW^A{R@HFV zJ-umE4OPA~t0u+_FVoCcv&mSlniKH(Y94pfD>O%{Bh6y9s4A{@IO>*;)asb4?ToVw z%i=!4a$o4Ga#LY>R(Ptg!V|4J=}taTs*@NSVMUA;F*fB+VeAaXMp+4CC5)YA78_?1 zPu1!=Jto_e<<`_6F{+F>PHb zzZw1LUfA#&VZ#lqy3egfD|ENG6E^&o6}mwfSbp8Q)!B5{oc-CaKp$81uGexfUU@ID zelk5%Z+NbF4zA*Dqvf{zGW#5mO*aUfEqotMOy93eH!IWZ>hvd9r|(>yUN7r$2J3A) zVVq@7=+>KIYzovhT4euE$HPasf=x880=_E5)2hz2s=;*d(x_&bSfSrE@6NsxP3*oGRBF89 zv>O#kW;wKXrGKcf?Rw2hZQJ2t^h3eBxjF}(ANhQDPI6^#x6xu1*`M8anqHh)o4;@0 z*oa5yzP_~f+490tTmZ8dyiO3hT=dsnzSpR^0Cx62^*2|_=i{7h6VGfrF4%U{XC06F zMcaPZaRhhql(lW<*U-t^cHsG8fbQh#-KDkpjny@K`Of^!CHvOW{dkn-K7KrR>)ywJ zxo_W|-&k7Tu-BGuFU_wn#S^#QTelb1?1kIQODh}VOEL7>mA)aafyuNwk}AxXb2bCt z>x5W*+uPDQ~@f)o+L^&S&Wa;)Dd*PG$HOxn^rWTiO%-_3>X_i+O zZSeEP@=aa@ZazvAjM4fQZr@wqSi*wV7Ue)fKQSN(`swP&^uR~x!(Q|CAIxq0O?R%d z*=dEHIpR;yc53ci%Ku^Jb)cfbYIJo+8tBoLQw8)1 zZ7}fVN4SFPXa=;Q$P8vag_UDPvQg-nusKDRWBI2hG(>Z=b&VC+2;StF4oNS?uaIfr zEf*_J1m=#@f>`s&Em-)y?6n0M47u-lffa7Mme;7ewf&ms3TxA7Em}1xewNc>eMqeN zyUTb*hGtgak2Bm2JG`}0Q?Z@&M}3lm1zZ6c|Ax|4Y}od7<+)1sYYYlyOFTAf)WYX5 zX@Tq2k9owW@fpccqtys)`*#Qvp)9D;n=jkcYzjTF7&1R}iNz4nLpYHqs){A;sE68- zvZ#DM{!{HhJ`FvO9i!3dE_)g>-~4$39L$DVy%^OQD`B56AR-NYAD)> z$35R+_AYeCmd04JBhzV`{r91aSD5vLY~Zbe_I~oi=NkVZzFpxnXv=v%K@*Z~=2O(1 zq2@9*zfa8vXkud%+QdJi=bkT+Ju)`@2WVz-1>|PPxEJ(-Ud&C7h&MG14*YtC{38qx z7;?=u4$Up)NUdvm118*L8DYS)EQi1^D+5lLa1mr8Ts4F#A6rcVlf;5M_n=zB$PT^u zR@)7(AxN;&2xit=4y{KG&$F2Ss6|q5E&8=x@Lq&G!DTUl7?%SSfcmidQXCqstpOpq zEc`)(qAH+-XYKiifk_3@yAB0E(5B$3c((9EID)xtF_kd<)>UiXA`bvJdX)d zOTc82@?FbynT54&O3y~(E8^xA|!AibMoOU)rFEyw}lAQi1!pY$hR^(sN>8j+q zyBb)MZ`2R#buN-8< z;^AmFd!+ty@*vmM4@=?L;dmEsk7%XZj}c}V_#8UzkDPm27TU#nN6`Ry=l2Uq9jwi@AfXLGjZH|IF=1UPUq{Ar+Ap@<(Z z9`y!Ny{mjh#xRlVasp9D+ngk9szhHuOh1w-bk*?;nq8eF4iS-MJaDthjVkWw>n7Wn3eWW%wHQmCU-0BOH2mB~8 ziUKln*}+fIh()M0?UB}7$eRaZNnHgCQLzw>0!pSo!;+YZ=q9tP^J$D{j|0tz1%WBo zRSD0-0?T8$g=o6B+yO6W)w$nH9OEaQ&FJFkU{|f_AUfOcn#l6EVGyjSXkEAFn7cRE z>UiD_PT`%1hoWKv-^REVjY+FB9VFq_%5x*mc>WeLw%DK$OKKEFs)#Jta6^tjhN@3Q zNOj_|)Xyd2#xWR>s+TkFlXXY6DUP%0(?R zDuQI#2jiP)`a5mtgc49}Cw$v~phIa&8YBPGrDWw$hsQFc)SzdEnVl?rSB@FNcWDQD zSA*BGD;A8kL<&xRYLcRv$%0yU&p!xz7ZhdcEKUP{7fGM(&r_UP>s6VGAhz(rt8Hei;>5HwJ`+`kr$T$W>S zve!`Z%%PCbrWhigTtQ6BHt^_khwtCO7+=MjW9hwFr`hJqc$Sui;yo@xgj>|3fPnvm z8nVVRl@spWpb$eu35hbt`6SYSTVE&G4Qhl+i$LiueUTB)}4PBWRrT^0MZ~b@l_DSu3*+hIElavdmC5Ump z7mdgHPZ5~krNCUKjTD#gA5-&()EsZDw>wUDWCd41X{SgSM>N>n!i(Hw&Mcl4?>9~q zOH2R@3-mfV(%*Fz@=_t{O4K-Dy3)|f9 zY)RD-WhNwibtozF>}wmm<6gE<6A?KEcNziG3?z1(Bfvokpe0a~(#aZL&1a(iMt!)Mt9jrg#S z$Rs^N`u+|b5%d(zNm*JLFzFOPP+TJjO0MKIf>H?t#+~Au)U;C!Xh!b~3aoWn>25^I zC>Ln0Z!E6f+prwC>K2iP!5+dkC3JXehz{0_AiC{$!gePtOR}hZ2TO_yt74GGpghOg zwTqW=ao{_Rw`edU(F_qJe(gjhvH#=0$Cm-+ZbgHtdWOm$;$cAkbPQy0338EMlLsWZ zyFgwKxgJwNDlIfo`-F#L8X~JV!Yurso{mvIcti(rkWYeQ<^{^AGSQbeU7S|6eHI`p z61Mz7FD-d6Ypsbou`D;&8yq$spfU&E28|zh&Zg_psR1em4tUk6KZOIY zXdAs9;Q6w(*`YcGX$J*;kO;8`TEUgs6%p>0^YN58CP|Ja?#!>?lx0nn7iBs|wZ(YK zL#<65;E3V_vq6hk$1xnI0+bMuIQz)qEh;|fVk4X)yx1W64vR&0%MA&`KL8@2l&5GI zDoiC#Aw(SbjbRj2{unO?bcZq}5(ps-1$08>N+^v}sZfiS1v?#U+eO?*tok;l_Hig7 zYs*i`IvbsZm!5@NZb6GG;eOk_Mo7zYNw^W8hWAvYJrNjf>>mLjtN$9`M}MGVLF(Tt zKwrWG)jhjZLrC%jik_Zjy1Lky5&dmi*(4h$<&=6T1Pg=qAl>LzW6y0_4=}x8)C248 zF%gn?J?TgU_G1dCQo8|jjqcLg8VT!z!70+JPaD-G$WdZbMh8|#QHqRJDi;3``zjl8 zE?te3?P5RH(@bbQP9O9`{Sy!gQeHB67ySYAFhI&b7Nktg0}e-E94Hg)>M$56DDzjL z{w5WeAvLh8NX1N;g&fQRL1X}Nxep?90wN2DYkd$%-;Qw*o^n3AxsaX!)MOnCrz3uc z*IbKA3e19odQjuGFY58btm|V>Y*2#obw)~JoZGwNcE zJI*H2o+NXo?RB{0QT>?hi!TCsARFKS-r3*rHv^pW@BjqsQ*=5n;BTRc-$+XLz4hQI zM(ie8H~$`iejg3ogD_h8-ky@f9d6@@DArLp7*>=s3GxAq{{mbAAuMV+3Z_t4K{@-I z0xZIC&v?QA8h|yj1nF-;4)!p5>?Xi}3RH;y=esBZQu+kGAuVEt;C~kUCkW#IOdkYX z$`Azc|NTCQOg{tzo^mc)Ope;f>LyAD(hwmZSi=&L7^suF$w`#?pHTCsXr7z==1 zYJ}~1LOqunic0w}(0n^XlS8rJ&d}!u|0_Z%xSCCt&i|5zjw4D6cH%P$&H+CD8{PyY zLq%luCwX_wPjK;90DCzXDG)i8i)Z^dkZ`e&gWvCan{Y6BOWgb2zPAbY)|G^F^Y9e=MMjpP!2I-g1#N#!dHMA;zFjUOZ`1?;n#pIs~hFlClfhUi%sF7}ajZN`S5_}V&aR&8K8X@%)X&T?A;etrK#ksyXh?K{cc`WU=C>F_V zhC<1B1eUN-TM*|0v9Y{zV^szkKOl_4CM{4;9DY(jC;|cDMuh_q+AoAhti6vKf-XWf zb05h6Fu6`4L+~CN14YOjf=qeksBEiIw1EUp&HQi4%v>yKnb%Q^x|_@8^0~2sk;~*J HisSzWC+9+B literal 0 HcmV?d00001 From 901f107e4e542854ee510341bdb61886836ce954 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:30:02 +0000 Subject: [PATCH 03/11] ci-debugging Signed-off-by: Yicheng-Lu-llll --- .github/workflows/test-job.yaml | 93 ++------------------------------- 1 file changed, 3 insertions(+), 90 deletions(-) diff --git a/.github/workflows/test-job.yaml b/.github/workflows/test-job.yaml index 6f5209bd3a..9d0f8f2f73 100644 --- a/.github/workflows/test-job.yaml +++ b/.github/workflows/test-job.yaml @@ -18,87 +18,6 @@ jobs: # Use the same go version with build job go-version: '1.17' - - name: Check out code into the Go module directory - uses: actions/checkout@v2 - with: - # When checking out the repository that - # triggered a workflow, this defaults to the reference or SHA for that event. - # Default value should work for both pull_request and merge(push) event. - ref: ${{github.event.pull_request.head.sha}} - - - name: Install goimports and gofumpt - run: | - go install golang.org/x/tools/cmd/goimports@latest - go install mvdan.cc/gofumpt@v0.3.1 - - - name: Run gofmt - uses: Jerome1337/gofmt-action@v1.0.4 - with: - gofmt-path: 'apiserver cli ray-operator' - gofmt-flags: '-l -d -s' - - - name: Run linter against ray operator - uses: golangci/golangci-lint-action@v2 - with: - # Optional: version of golangci-lint to use in form of v1.2 or v1.2.3 or `latest` to use the latest version - version: v1.50.1 - - # Optional: working directory, useful for monorepos - working-directory: ./ray-operator - - # Optional: golangci-lint command line arguments. - # args: --issues-exit-code=0 - args: --timeout=3m - - # Optional: show only new issues if it's a pull request. The default value is `false`. - # only-new-issues: true - - # Optional: if set to true then the action will use pre-installed Go. - skip-go-installation: true - - # Optional: if set to true then the action don't cache or restore ~/go/pkg. - skip-pkg-cache: true - - # Optional: if set to true then the action don't cache or restore ~/.cache/go-build. - # skip-build-cache: true - - - name: Run linter against apiserver - uses: golangci/golangci-lint-action@v2 - with: - version: v1.29 - working-directory: ./apiserver - args: --timeout=3m - skip-go-installation: true - skip-pkg-cache: true - - - name: Run linter against cli - uses: golangci/golangci-lint-action@v2 - with: - version: v1.29 - working-directory: ./cli - args: --timeout=3m - skip-go-installation: true - skip-pkg-cache: true - - - name: Run goimports - run: test -z "$(set -o pipefail && $(go env GOPATH)/bin/goimports -l apiserver/ ray-operator/ cli/ | tee goimports.out)" || { cat goimports.out && exit 1; } - - - name: Open this to see how to fix goimports if it fails - run: | - echo "Run command 'goimports -w apiserver/ ray-operator/ cli/' to correct your code format." - echo "Proposed format changes:" - $(go env GOPATH)/bin/goimports -d apiserver/ ray-operator/ cli/ - if: failure() - - - name: Run gofumpt - run: test -z "$(set -o pipefail && $(go env GOPATH)/bin/gofumpt -l apiserver/ ray-operator/ cli/ | tee gofumpt.out)" || { cat gofumpt.out && exit 1; } - - - name: Open this to see how to fix gofumpt if it fails - run: | - echo "Run command 'gofumpt -w apiserver/ ray-operator/ cli/' to correct your code format." - echo "Proposed format changes:" - $(go env GOPATH)/bin/gofumpt -d apiserver/ ray-operator/ cli/ - if: failure() build_apiserver: env: @@ -157,9 +76,9 @@ jobs: run: make build working-directory: ${{env.working-directory}} - - name: Test - run: make test - working-directory: ${{env.working-directory}} + # - name: Test + # run: make test + # working-directory: ${{env.working-directory}} - name: Set up Docker uses: docker-practice/actions-setup-docker@master @@ -217,9 +136,3 @@ jobs: with: ray_version: 2.4.0 - python-client-test: - runs-on: ubuntu-latest - name: Python Client Test - steps: - - name: Set up Docker - uses: docker-practice/actions-setup-docker@master From a5d3f30a6f0c478c8055014622922423cc74718b Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:50:00 +0000 Subject: [PATCH 04/11] Empty-Commit From 37f75b423b2aac357b367d6a82ff8a20c5dfc318 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:53:54 +0000 Subject: [PATCH 05/11] Empty-Commit From e382e8558cbc6621fafe4f1928c28da01830aa81 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:53:55 +0000 Subject: [PATCH 06/11] Empty-Commit From 8af3bd914de2cb522e711744c60d3a94a7517861 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Fri, 19 May 2023 15:55:37 +0000 Subject: [PATCH 07/11] Empty-Commit From da7330bfdc42e28b5b789356d9b31e3736cb6db0 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Sat, 20 May 2023 00:20:22 +0000 Subject: [PATCH 08/11] ci-debugging Signed-off-by: Yicheng-Lu-llll --- .../__pycache__/utils.cpython-310.pyc | Bin 9741 -> 9820 bytes tests/framework/utils.py | 5 +++++ 2 files changed, 5 insertions(+) diff --git a/tests/framework/__pycache__/utils.cpython-310.pyc b/tests/framework/__pycache__/utils.cpython-310.pyc index 12f2a143f09c02cd34ccec9f323afbeba6e166f6..8b6518b5fcfb028deda4a954239c8d226d49ecbe 100644 GIT binary patch delta 334 zcmeD6x#Po|&&$ij00duFrl;)N$U9qvF=p~gkzz*a$+DtOT6r9X48>{XjEoE*7zxBB zTno5U1Qr5CN(52_n}Ku+cQAvd(B@9jTZ~>+{ECW-$r-81*$U1%3v$artFt#T_P=oi$>tJ1j3$#?B=kAh{_}lg;#B&+ zd9TDPM#g!QS4i#i$^u$f#pPdGQczlArJ!1@$x;*z6e@}V5!pcE7Ef|UYI1gbK3It+ zQ&ApBkPX5u;sz-L6RML%C6t7h0r|xoK!S&bkCBIwg|Vn`^J(dKjEw6i?~=X2xNq}h zIWI=WL!0l(KV@V*vH6JNaYn|Qn|qX785v(r7E-lh%%1G4YQ&f^xj{9Wv2gMo)k6LN S4h{}Z4t_2c4mJ)!9svNST~AX0 diff --git a/tests/framework/utils.py b/tests/framework/utils.py index 7709195ae4..f4c2d7661f 100644 --- a/tests/framework/utils.py +++ b/tests/framework/utils.py @@ -155,6 +155,11 @@ def __install_crd_and_operator(self): f"--set image.repository={repo},image.tag={tag}" ) + logger.info("!!!check CRD!!!") + shell_subprocess_check_output("kubectl get crd rayjobs.ray.io -o yaml | grep suspend") + + + def shell_subprocess_run(command, check = True): """ Command will be executed through the shell. If check=True, it will raise an error when From 4953c886c022387d669e05095f45ea427ee6b16a Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Sat, 20 May 2023 16:51:24 +0000 Subject: [PATCH 09/11] Empty-Commit From 6d9c5fc5ace836697383d7a25f353be5ba519dd1 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Sat, 20 May 2023 16:51:26 +0000 Subject: [PATCH 10/11] Empty-Commit From fe4e4d5ca9c554e2de483eb2d3c77da07a9d70f4 Mon Sep 17 00:00:00 2001 From: Yicheng-Lu-llll Date: Sat, 20 May 2023 16:53:21 +0000 Subject: [PATCH 11/11] Empty-Commit