Skip to content

Commit 4013666

Browse files
committed
Semantic fix
Signed-off-by: Artem Glazychev <[email protected]>
1 parent 2da7f8c commit 4013666

File tree

4 files changed

+20
-15
lines changed

4 files changed

+20
-15
lines changed

.github/workflows/ci.yaml

+6-6
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,12 @@ jobs:
6666
git diff --name-only --exit-code go.sum || ( echo "Run go tidy" && false )
6767
6868
packet:
69-
name: packet (Calico ${{ matrix.calico }})
69+
name: packet (CNI ${{ matrix.calico }})
7070
runs-on: ubuntu-latest
7171
strategy:
7272
fail-fast: false
7373
matrix:
74-
calico: ["off", "on"]
74+
CNI: ["default", "calico"]
7575
steps:
7676
- name: Set up /bin permissions # 1. Set up /bin permissions
7777
run: |
@@ -94,11 +94,11 @@ jobs:
9494
- name: Compute suffix # 5. Compute suffix for cloudtest input and output paths
9595
id: suffix
9696
run: |
97-
if [[ "${CALICO}" == "on" ]]; then
97+
if [[ "${CNI}" == "calico" ]]; then
9898
echo ::set-output name=val::_calico
9999
fi
100100
env:
101-
CALICO: ${{ matrix.calico }}
101+
CNI: ${{ matrix.CNI }}
102102
- name: Checkout files # 6. Checkout files
103103
uses: actions/checkout@v2
104104
with:
@@ -111,7 +111,7 @@ jobs:
111111
PACKET_AUTH_TOKEN: ${{ secrets.PACKET_AUTH_TOKEN }}
112112
PACKET_PROJECT_ID: 383890d0-f5d1-4de1-881a-4d1ede549d18
113113
KUBERNETES_VERSION: ${{ secrets.NSM_KUBERNETES_VERSION }}
114-
CALICO: ${{ matrix.calico }}
114+
CNI: ${{ matrix.CNI }}
115115
suffix: ${{ steps.suffix.outputs.val }}
116116
- name: Publish test report # 8. Publish test report
117117
uses: mikepenz/[email protected]
@@ -120,7 +120,7 @@ jobs:
120120
report_paths: "**/cloud_test${{ steps.suffix.outputs.val }}/results/junit.xml"
121121
suite_regex: "Test*"
122122
github_token: ${{ secrets.GITHUB_TOKEN }}
123-
check_name: "JUnit Test Report (Calico ${{ matrix.calico }})"
123+
check_name: "JUnit Test Report (CNI ${{ matrix.CNI }})"
124124
- name: Upload logs # 9. Upload logs
125125
uses: actions/upload-artifact@v2
126126
if: ${{ always() }}

main_test.go

+5
Original file line numberDiff line numberDiff line change
@@ -29,21 +29,26 @@ import (
2929
)
3030

3131
func TestMemory(t *testing.T) {
32+
t.Skip()
3233
suite.Run(t, new(memory.Suite))
3334
}
3435

3536
func TestSRIOV(t *testing.T) {
37+
t.Skip()
3638
suite.Run(t, new(sriov.Suite))
3739
}
3840

3941
func TestMultiForwarder(t *testing.T) {
42+
t.Skip()
4043
suite.Run(t, new(multiforwarder.Suite))
4144
}
4245

4346
func TestHeal(t *testing.T) {
47+
t.Skip()
4448
suite.Run(t, new(heal.Suite))
4549
}
4650

4751
func TestRunObservabilitySuite(t *testing.T) {
52+
t.Skip()
4853
suite.Run(t, new(observability.Suite))
4954
}

scripts/create-kubernetes-cluster.sh

+4-4
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ sshkey=$3
88
SSH_CONFIG="ssh_config"
99
SSH_OPTS="-F ${SSH_CONFIG} -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -i ${sshkey}"
1010

11-
if [[ "$CALICO" == "on" ]]; then # calico
11+
if [[ "$CNI" == "calico" ]]; then # calico
1212
# Use a new 10.0.0.${base_ip}/30 subnet to prevent IP addresses collisions
1313
# ${base_ip} should be <= 248, because 10.0.0.252/30 subnet is reserved for manual testing
1414
base_ip=$(( GITHUB_RUN_NUMBER % 63 * 4 ))
@@ -19,7 +19,7 @@ if [[ "$CALICO" == "on" ]]; then # calico
1919
CALICO_INTERFACE="eno2"
2020
fi
2121

22-
ENVS="KUBERNETES_VERSION CALICO"
22+
ENVS="KUBERNETES_VERSION CNI"
2323

2424
# wait_pids pid_1 ... pid_n
2525
source scripts/include/wait-pids.sh
@@ -52,7 +52,7 @@ pids=""
5252
pids+=" $!"
5353
wait_pids "${pids}" "SR-IOV config failed" || exit 21
5454

55-
if [[ "$CALICO" == "on" ]]; then # calico
55+
if [[ "$CNI" == "calico" ]]; then # calico
5656
# 3. Create Calico scripts directory on nodes.
5757
ssh ${SSH_OPTS} root@${master_ip} mkdir -p calico || exit 31
5858
ssh ${SSH_OPTS} root@${worker_ip} mkdir -p calico || exit 32
@@ -123,7 +123,7 @@ wait_pids "${pids}" "worker join failed" || exit 94
123123
# 10. Save KUBECONFIG to file.
124124
scp ${SSH_OPTS} root@${master_ip}:.kube/config ${KUBECONFIG} || exit 101
125125

126-
if [[ "$CALICO" == "on" ]]; then # calico
126+
if [[ "$CNI" == "calico" ]]; then # calico
127127
# 11. Setup cluster nodes IPs.
128128
scp ${SSH_OPTS} scripts/calico/setup-node-ip.sh root@${master_ip}:calico/setup-node-ip.sh || exit 111
129129
scp ${SSH_OPTS} scripts/calico/setup-node-ip.sh root@${worker_ip}:calico/setup-node-ip.sh || exit 112

scripts/k8s/start-master.sh

+5-5
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ calico_ip="$2"
88

99
K8S_DIR=$(dirname "$0")
1010

11-
if [[ "$CALICO" != "on" ]]; then # not calico
12-
ip="${public_ip}"
13-
else
11+
if [[ "$CNI" == "calico" ]]; then # calico
1412
ip="${calico_ip}"
13+
else
14+
ip="${public_ip}"
1515
fi
1616

1717
kubeadm init \
@@ -24,13 +24,13 @@ mkdir -p ~/.kube
2424
cp -f /etc/kubernetes/admin.conf ~/.kube/config
2525
chown "$(id -u):$(id -g)" ~/.kube/config
2626

27-
if [[ "$CALICO" != "on" ]]; then # not calico
27+
if [[ "$CNI" == "default" ]]; then
2828
kubectl apply -f "https://cloud.weave.works/k8s/net?k8s-version=$(kubectl version | base64 | tr -d '\n')&env.IPALLOC_RANGE=192.168.0.0/16"
2929
fi
3030

3131
kubectl taint nodes --all node-role.kubernetes.io/master-
3232

33-
if [[ "$CALICO" == "on" ]]; then # calico
33+
if [[ "$CNI" == "calico" ]]; then # calico
3434
kubectl -n kube-system get configmap kubeadm-config -o jsonpath='{.data.ClusterConfiguration}' > kubeadm.yaml
3535
sed -i "/^apiServer:$/a \ \ certSANs:\n - \"${public_ip}\"\n - \"${calico_ip}\"" kubeadm.yaml
3636

0 commit comments

Comments
 (0)