Skip to content

Commit

Permalink
[vs-image-pr] Fix submodule checkout for VS image tests (sonic-net#199)
Browse files Browse the repository at this point in the history
We've started noticing strange errors when we try to checkout submodules during the Test stage of this Jenkins job, like this:

09:01:33 From https://github.com/Azure/sonic-telemetry
09:01:33 e54f9ca..d3336d1 master -> origin/master
09:01:33 fatal: remote error: upload-pack: not our ref ca07e68bb8bafc6e5aab341c5c8a35605044a7d3
09:01:33 fatal: the remote end hung up unexpectedly

We've noticed these for sairedis and telemetry. The Build stage is working consistently, and the only difference we can see between this stage and the Test stage is this recursiveSubmodules flag. This PR sets that flag for the Test stage as well in order to mitigate the impact on our PR builds.

Signed-off-by: Danny Allen <[email protected]>
  • Loading branch information
daall authored Dec 17, 2020
1 parent e7b159c commit 81be150
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion jenkins/vs/buildimage-vs-image-pr/Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ sudo cp ../target/sonic-vs.bin /nfs/jenkins/sonic-vs-${JOB_NAME##*/}.${BUILD_NUM
extensions: [[$class: 'SubmoduleOption',
disableSubmodules: false,
parentCredentials: false,
recursiveSubmodules: false,
recursiveSubmodules: true,
reference: '',
trackingSubmodules: false]],
userRemoteConfigs: [[url: 'http://github.com/Azure/sonic-buildimage',
Expand Down

0 comments on commit 81be150

Please sign in to comment.