New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Git Artifact does not support relative submodules (fixed in go-git v5.3.0) #7141
Comments
Would you like to upgrade it and then verify that fixes the issue? |
@terrytangyuan I can give it a shot. Assuming I manage to get a successful build with that version of go-git, what's the best way to test that ? Is there maybe a way I can specify an alternative argoexec image for the init? |
You'll want to rebuild the images and then test locally: https://github.com/argoproj/argo-workflows/blob/master/docs/running-locally.md You can specify a new argoexec image in your controller deployment. |
Upgrades src-d/go-git v4 to go-git/go-git v5.3.0. See go-git/go-git#195. Signed-off-by: valorl <11498571+valorl@users.noreply.github.com>
Thanks. I can confirm that upgrading go-git to v5.3.0 fixes this. See #7162, which is now ready for review. |
Upgrades src-d/go-git v4 to go-git/go-git v5.3.0. See go-git/go-git#195. Signed-off-by: valorl <11498571+valorl@users.noreply.github.com>
Upgrades src-d/go-git v4 to go-git/go-git v5.3.0. See go-git/go-git#195. Signed-off-by: valorl <11498571+valorl@users.noreply.github.com>
Summary
Using GitArtifact to clone a Git repository with a submodule specified as a relative path, argoexec init fails with
executor error: repository not found
.The error comes from go-git (my guess would be during the initial clone). See
As mentioned there, it has been fixed in v5.3.0.
Diagnostics
You can use the following workflow to reproduce. It tries to clone this example repo to which I added a submodule with a relative path (
../argo-workflows
).For me on 3.0.7, this results in the following error in init log:
What Kubernetes provider are you using?
EKS
What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary
Docker
Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.
The text was updated successfully, but these errors were encountered: