Skip to content

Commit 5ed19da

Browse files
committed
Fix errors
Signed-off-by: Ye Cao <[email protected]>
1 parent eb5002a commit 5ed19da

File tree

4 files changed

+24
-63
lines changed

4 files changed

+24
-63
lines changed

.github/workflows/vineyard-operator.yaml

Lines changed: 21 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -123,15 +123,6 @@ jobs:
123123
username: ${{ github.actor }}
124124
password: ${{ secrets.GITHUB_TOKEN }}
125125

126-
- name: Build a kind cluster with local registry
127-
run: |
128-
touch /tmp/e2e-k8s.config
129-
export KUBECONFIG=/tmp/e2e-k8s.config
130-
bash k8s/test/hack/build-kind-cluster-with-local-registry.sh
131-
make -C k8s install-cert-manager
132-
make -C k8s install-vineyard-operator
133-
make -C k8s install-vineyard-cluster
134-
135126
- name: Build docker image
136127
run: |
137128
make -C k8s docker-build
@@ -158,67 +149,52 @@ jobs:
158149
run: |
159150
make -C docker build-python-dev
160151
161-
- name: Install e2e test dependencies
162-
if: ${{ matrix.job != 'release' }}
163-
run: |
164-
sudo bash k8s/test/hack/prepare-e2e.sh
165-
166152
- name: e2e-tests-assembly-local
167153
if: ${{ matrix.job == 'e2e-tests-assembly-local' }}
168-
uses: apache/skywalking-infra-e2e@main
169-
with:
170-
e2e-file: k8s/test/e2e/assembly/local-assembly-e2e.yaml
154+
run: |
155+
make -C k8s e2e-tests-assembly-local
171156
172157
- name: e2e-tests-assembly-distributed
173158
if: ${{ matrix.job == 'e2e-tests-assembly-distributed' }}
174-
uses: apache/skywalking-infra-e2e@main
175-
with:
176-
e2e-file: k8s/test/e2e/assembly/distributed-assembly-e2e.yaml
159+
run: |
160+
make -C k8s e2e-tests-assembly-distributed
177161
178162
- name: e2e-tests-autogenerated-helm-chart
179163
if: ${{ matrix.job == 'e2e-tests-autogenerated-helm-chart' }}
180-
uses: apache/skywalking-infra-e2e@main
181-
with:
182-
e2e-file: k8s/test/e2e/autogenerated-helm-chart/e2e.yaml
164+
run: |
165+
make -C k8s e2e-tests-autogenerated-helm-chart
183166
184167
- name: e2e-tests-failover
185168
if: ${{ matrix.job == 'e2e-tests-failover' }}
186-
uses: apache/skywalking-infra-e2e@main
187-
with:
188-
e2e-file: k8s/test/e2e/failover/e2e.yaml
169+
run: |
170+
make -C k8s e2e-tests-failover
189171
190172
- name: e2e-tests-repartition-dask
191173
if: ${{ matrix.job == 'e2e-tests-repartition-dask' }}
192-
uses: apache/skywalking-infra-e2e@main
193-
with:
194-
e2e-file: k8s/test/e2e/repartition/dask-repartition-e2e.yaml
174+
run: |
175+
make -C k8s e2e-tests-repartition-dask
195176
196177
- name: e2e-tests-scheduler-outside-cluster
197178
if: ${{ matrix.job == 'e2e-tests-scheduler-outside-cluster' }}
198-
uses: apache/skywalking-infra-e2e@main
199-
with:
200-
e2e-file: k8s/test/e2e/scheduling-outside-cluster/e2e.yaml
179+
run: |
180+
make -C k8s e2e-tests-scheduler-outside-cluster
201181
202182
- name: e2e-tests-serialize
203183
if: ${{ matrix.job == 'e2e-tests-serialize' }}
204-
uses: apache/skywalking-infra-e2e@main
205-
with:
206-
e2e-file: k8s/test/e2e/serialize/e2e.yaml
184+
run: |
185+
make -C k8s e2e-tests-serialize
207186
208187
- name: e2e-tests-sidecar
209188
if: ${{ matrix.job == 'e2e-tests-sidecar' }}
210-
uses: apache/skywalking-infra-e2e@main
211-
with:
212-
e2e-file: k8s/test/e2e/sidecar/e2e.yaml
189+
run: |
190+
make -C k8s e2e-tests-sidecar
213191
214192
- name: e2e-tests-spill
215-
if: false ## FIXME(tao) ${{ matrix.job == 'e2e-tests-spill' }}
216-
uses: apache/skywalking-infra-e2e@main
217-
with:
218-
e2e-file: k8s/test/e2e/spill/e2e.yaml
193+
if: ${{ matrix.job == 'e2e-tests-spill' }}
194+
run: |
195+
make -C k8s e2e-tests-spill
219196
220197
- name: e2e-tests-workflow
221198
if: ${{ matrix.job == 'e2e-tests-workflow' }}
222-
uses: apache/skywalking-infra-e2e@main
223-
with:
224-
e2e-file: k8s/test/e2e/workflow/e2e.yaml
199+
run: |
200+
make -C k8s e2e-tests-workflow

k8s/Makefile

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,8 @@ install-vineyard-cluster:
9898
@kubectl wait vineyardd/vineyardd-sample --for condition=Available -n vineyard-system --timeout=300s --kubeconfig=/tmp/e2e-k8s.config
9999
@echo "Vineyard cluster Ready"
100100

101+
install-vineyard: build-local-cluster install-cert-manager install-vineyard-operator install-vineyard-cluster
102+
sleep 6000
101103
e2e-tests-assembly-local: kustomize prepare-e2e-test build-local-cluster install-cert-manager install-vineyard-operator install-vineyard-cluster
102104
@echo "Running local assembly e2e test..."
103105
@cd .. && ${GOBIN}/e2e run --config=k8s/test/e2e/assembly/local-assembly-e2e.yaml

k8s/config/manager/kustomization.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,5 @@ apiVersion: kustomize.config.k8s.io/v1beta1
44
kind: Kustomization
55
images:
66
- name: controller
7-
newName: vineyardcloudnative/vineyard-operator
7+
newName: localhost:5001/vineyard-operator
88
newTag: latest

k8s/test/e2e/assembly/distributed-assembly-e2e.yaml

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -18,23 +18,6 @@ setup:
1818
env: kind
1919
kubeconfig: /tmp/e2e-k8s.config
2020
steps:
21-
- name: install cert-manager
22-
command: |
23-
kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/v1.9.1/cert-manager.yaml
24-
wait:
25-
- namespace: cert-manager
26-
resource: pod
27-
for: condition=Ready
28-
- name: install vineyard operator
29-
command: |
30-
docker tag vineyardcloudnative/vineyard-operator:latest localhost:5001/vineyard-operator:latest
31-
docker push localhost:5001/vineyard-operator:latest
32-
export IMG=localhost:5001/vineyard-operator:latest
33-
make -C k8s deploy
34-
wait:
35-
- namespace: vineyard-system
36-
resource: deployment/vineyard-controller-manager
37-
for: condition=Available
3821
- name: download all distributed assembly images into kind cluster
3922
command: |
4023
make -C k8s/test/e2e e2e-test-distributed-assembly REGISTRY=localhost:5001

0 commit comments

Comments
 (0)