Skip to content
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

set core.longpaths=true in all the windows containers used for builds #2847

Closed
jtnord opened this issue Mar 23, 2022 · 6 comments
Closed

set core.longpaths=true in all the windows containers used for builds #2847

jtnord opened this issue Mar 23, 2022 · 6 comments

Comments

@jtnord
Copy link

jtnord commented Mar 23, 2022

Service(s)

ci.jenkins.io

Summary

git on windows will barf if a file has a path length > 260 characters. Why this is the default do not ask me but still.

This causes some problems for plugins where files can not be checkedout (may be during some IT tests where the java.io.tmpdir is used whcih results in some longer paths than normal).
or it could be there is just a long path/filename in the git repo.

As such the containers should set on windows core.longpaths to true in the containers.

ref: https://github.com/git-for-windows/build-extra/blob/main/ReleaseNotes.md#:~:text=Many%20Windows%20programs,core.longPaths%3Dtrue%20....

(additionally if the user running the container supports it symlinks core.symlinks shoulld be set)

Reproduction steps

No response

@jtnord jtnord added the triage Incoming issues that need review label Mar 23, 2022
@dduportal
Copy link
Contributor

dduportal commented Mar 23, 2022

Related pipeline that triggered the issue: https://github.com/jenkinsci/pipeline-model-definition-plugin/blob/master/Jenkinsfile (there might be others)

ToDo list for the person of the infra team that would take this issue (can be me, can be someone else of the team):

@dduportal dduportal added this to the infra-team-sync-2022-03-29 milestone Mar 23, 2022
@dduportal dduportal removed the triage Incoming issues that need review label Mar 23, 2022
@timja
Copy link
Member

timja commented Mar 23, 2022

Set core.longpaths on the Windows VM containers (hard)

i would just set it tbh

@dduportal
Copy link
Contributor

Set core.longpaths on the Windows VM containers (hard)

i would just set it tbh

Same here. I've mentioned that it's "hard" because it requires a contribution out of the scope of pure jenkins-infra. It's easy technically though, and coul be fast if it is merged quickly on the jenkinsci/docker-agents repo

@lemeurherve lemeurherve self-assigned this Mar 23, 2022
@MarkEWaite
Copy link

jenkinsci/git-plugin#856 attempted to add git plugin configuration for the core.longpaths setting . It was closed because there is an open issue on git for windows that prevents the setting from having the desired result in all cases.

I don't object to setting the global configuration value core.longpaths = true, but the comments in git-for-windows/git#2576 indicate that there will be places where that is not enough.

lemeurherve pushed a commit to lemeurherve/pipeline-library that referenced this issue Mar 24, 2022
lemeurherve pushed a commit to lemeurherve/pipeline-library that referenced this issue Mar 24, 2022
lemeurherve pushed a commit to lemeurherve/pipeline-library that referenced this issue Mar 24, 2022
lemeurherve added a commit to jenkins-infra/pipeline-library that referenced this issue Mar 24, 2022
…s agents (#334)

Ref: jenkins-infra/helpdesk#2847
Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
Co-authored-by: James Nord <jtnord@users.noreply.github.com>
lemeurherve pushed a commit to lemeurherve/docker-agent that referenced this issue Mar 24, 2022
lemeurherve pushed a commit to lemeurherve/docker-agent that referenced this issue Mar 24, 2022
@dduportal
Copy link
Contributor

It looks like that it is resolved through the pipeline library: is that correct @jtnord?

If correct, we can close.

For information: #2873 was opened to allow the jenkisn infra team to build their own image: that would ease such issues in the future.

@lemeurherve
Copy link
Member

Confirmed on a private channel, we can close this issue. Thanks for the new one!

lemeurherve added a commit to lemeurherve/pipeline-library that referenced this issue Jun 1, 2022
…s agents (jenkins-infra#334)

Ref: jenkins-infra/helpdesk#2847
Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
Co-authored-by: James Nord <jtnord@users.noreply.github.com>
smerle33 pushed a commit to smerle33/pipeline-library that referenced this issue Jan 16, 2024
…s agents (jenkins-infra#334)

Ref: jenkins-infra/helpdesk#2847
Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
Co-authored-by: James Nord <jtnord@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants