-
-
Notifications
You must be signed in to change notification settings - Fork 230
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
curl is not installed in jnlp agent based image #153
Comments
The line was added 4 years ago by @carlossg when alpine agents were enabled. I believe the general pattern is to include only the required components in an agent image. Fewer components means fewer chances for issues with components. Command line git is bundled with the agent images because it is the preferred SCM system for the majority of Jenkins users. |
I hope you reconsider to install curl back to the image. Our Jenkins library uses curl a lot to integrate job states to the git. If curl installation is not possible, then I ask you to provide workaround to install it. |
Create your own Docker image that is derived from this image. Install the tools that you need in the derived docker image. Use the Docker image you created. |
In the jenkins documentation it is suggested to use I would say extending a docker container is much more complex (because of maintenance overhead) then writing some "complex" groovy code. |
I would suggest not to remove curl in the agent |
This essentially breaks custom tool installation/download for us for tools that are NOT hosted as archive but static binaries (like many releases on github). |
Pls add necessary command line tool curl |
Any news on this ? |
We don't plan to add curl to the agent image. If you need curl installed on your agent image, you should create a custom agent image with the tools that you need (including curl). See my comment at #153 (comment) and see the link from @timja at #253 (comment) |
If you need help creating and hosting your docker agent-based image containing curl, we could try to pair one of these days. |
Reopening as discussed by maintainers in IRC: the value of adding Besides, Please note that |
(Wait for the release to be availble: I'll comment here) |
Will this work on fargate? |
Hi all,
could you please explain why did you delete curl from based image:
https://github.com/jenkinsci/docker-agent/blob/master/8/alpine/Dockerfile#L45
It's better to have git/curl in jnlp images.
The text was updated successfully, but these errors were encountered: