rebase: workaround missing redirect #1370
Conversation
|
There were the following issues with this Pull Request
You may need to change the commit messages to comply with the repository contributing guidelines. 🤖 This comment was generated by commitlint[bot]. Please report issues here. Happy coding! |
Madhu-1
changed the title
|
/test ci/centos/containerized-tests |
1 similar comment
|
/test ci/centos/containerized-tests |
|
/test all |
|
/test ci/centos/containerized-tests |
|
/retest ci/centos/containerized-tests |
|
E2E is catching the issue which is corrected by #1368 ... Does the PR need rebasing? |
|
@Mergifyio rebase |
This fix is needed because we were hitting: ``` vbom.ml/[email protected]: unrecognized import path "vbom.ml/util": reading https://vbom.ml/util?go-get=1: 503 Service Unavailable ``` Source for the workaround: golang/dep#1169 The fix consists in simply enforcing what the redirect was supposed to do. Signed-off-by: Madhu Rajanna <[email protected]>
|
Command
|
Madhu-1
requested review from
nixpanic and
humblec
and removed request for
nixpanic
running: go mod verify https://jenkins-ceph-csi.apps.ocp.ci.centos.org/job/containerized-tests/324/console |
|
/retest all |
No it doesn't include any file changes in vendor directory , can you please test it out locally if am missing anything |
In that case I expect ci to fail |
This fix is needed because we were hitting:
Source for the workaround:
golang/dep#1169
The fix consists in simply enforcing what the redirect was supposed to do.
Signed-off-by: Madhu Rajanna [email protected]