Skip to content

Commit

Permalink
[CI] Fix image names in hw-offload CI
Browse files Browse the repository at this point in the history
Fix the wrong image names in the hw-offload CI.
  • Loading branch information
abdallahyas committed Apr 12, 2021
1 parent 51ab7ea commit 9864712
Show file tree
Hide file tree
Showing 2 changed files with 47 additions and 6 deletions.
35 changes: 34 additions & 1 deletion ci/jenkins/mellanox/common/common_functions.sh
Original file line number Diff line number Diff line change
Expand Up @@ -344,7 +344,7 @@ function change_k8s_resource {
name=$(yq r -d "$doc_num" $resource_file metadata.name)
if [[ "$name" == "$resource_name" ]];then
echo "changing $resource_key to $resource_new_value"
yq w -i -d "$doc_num" "$resource_file" "$resource_key" "$resource_new_value"
yq w -i -d "$doc_num" "$resource_file" "$resource_key" -- "$resource_new_value"
changed="true"
break
fi
Expand All @@ -360,6 +360,39 @@ function change_k8s_resource {
return 0
}

function get_daemonset_container_index_from_container_name {
local daemonset_name="$1"
local desired_container_name="$2"
local file="$3"

let doc_num=0
changed="false"
for kind in $(yq r -d "*" $file kind);do
if [[ "$kind" == "DaemonSet" ]];then
name=$(yq r -d "$doc_num" $file metadata.name)
if [[ "$name" == "$daemonset_name" ]];then
let container_index=0
for container_name in $(yq r -d "$doc_num" $file spec.template.spec.containers[*].name );do
if [[ "$container_name" == "$desired_container_name" ]];then
echo "$container_index"
changed="true"
break
fi
let container_index=$container_index+1
done
fi
fi
let doc_num=$doc_num+1
done

if [[ "$changed" == "false" ]];then
return 1
fi

return 0
}


function deploy_sriov_device_plugin {
build_github_project "sriov-network-device-plugin" \
"sed -i 's;^TAG=.*;TAG=$SRIOV_NETWORK_DEVICE_PLUGIN_HARBOR_IMAGE;' Makefile && make image"
Expand Down
18 changes: 13 additions & 5 deletions ci/jenkins/mellanox/scripts/start_ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -94,10 +94,18 @@ EOF
if [[ -z "${ANTREA_CNI_PR}${ANTREA_CNI_BRANCH}" ]];then
change_image_name $ANTREA_CNI_HARBOR_IMAGE antrea/antrea-ubuntu
fi

if [[ -z "$(grep hw-offload $WORKSPACE/antrea-cni/build/yamls/antrea.yml)" ]];then
sed -i '/start_ovs/a\ - --hw-offload' $WORKSPACE/antrea-cni/build/yamls/antrea.yml
fi

change_image_name antrea/antrea-ubuntu antrea/antrea-ubuntu:ci

change_k8s_resource "DaemonSet" "antrea-agent" spec.template.spec.containers[*].image "antrea/antrea-ubuntu:ci" "$WORKSPACE/antrea-cni/build/yamls/antrea.yml"
change_k8s_resource "DaemonSet" "antrea-agent" spec.template.spec.initContainers[*].image "antrea/antrea-ubuntu:ci" "$WORKSPACE/antrea-cni/build/yamls/antrea.yml"

change_k8s_resource "Deployment" "antrea-controller" spec.template.spec.containers[*].image "antrea/antrea-ubuntu:ci" "$WORKSPACE/antrea-cni/build/yamls/antrea.yml"

local antrea_ovs_container_index=$(get_daemonset_container_index_from_container_name "antrea-agent" "antrea-ovs" "$WORKSPACE/antrea-cni/build/yamls/antrea.yml")
change_k8s_resource "DaemonSet" "antrea-agent" spec.template.spec.containers["$antrea_ovs_container_index"].command[+] "--hw-offload" "$WORKSPACE/antrea-cni/build/yamls/antrea.yml"

cp $WORKSPACE/antrea-cni/build/yamls/antrea.yml $ARTIFACTS/antrea.yml

cat > $ARTIFACTS/antrea-net.yaml <<EOF
apiVersion: "k8s.cni.cncf.io/v1"
Expand Down Expand Up @@ -158,7 +166,7 @@ kubectl create -f $ARTIFACTS/antrea-net.yaml
kubectl create -f $ARTIFACTS/configMap.yaml
kubectl create -f $(ls -l $WORKSPACE/sriov-network-device-plugin/deployments/*/sriovdp-daemonset.yaml|tail -n1|awk '{print $NF}')

kubectl create -f $WORKSPACE/antrea-cni/build/yamls/antrea.yml
kubectl create -f $ARTIFACTS/antrea.yml

cp $ARTIFACTS/antrea-net.yaml $(ls -l $WORKSPACE/sriov-network-device-plugin/deployments/*/sriovdp-daemonset.yaml|tail -n1|awk '{print $NF}') $ARTIFACTS/
echo "All code in $WORKSPACE"
Expand Down

0 comments on commit 9864712

Please sign in to comment.