Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

start jenkins using CMD #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

heiderich
Copy link
Contributor

I think the main purpose of the docker container is to run jenkins. So I think we should start it by default. There are other options for debugging.

@rbehrends
Copy link
Collaborator

I see where you are going with this, but right now it is possible to just add jenkins to the run command, e.g.

docker run -it -v jenkins_home:/var/jenkins_home -p 8080:8080 oscar-ci jenkins

This is not too onerous (and why I put jenkins in the path in the first place) and that is actually what our server script is currently doing. While the main reason why we currently type in the command manually is to test the system.

What I'd really like instead is to have two docker images (as they're based on the same initial layers, that wouldn't take up a lot more space), one of which is designed to just run Jenkins, while the other is for testing and debugging purposes. This would also mean that the server version could directly pull Jenkinsfile and the meta repo from GitHub, while the other allows for local modification.

@rbehrends rbehrends force-pushed the master branch 2 times, most recently from 3e29c75 to 7437530 Compare March 5, 2020 14:36
@rbehrends rbehrends force-pushed the master branch 3 times, most recently from 714cd59 to 8dd3c15 Compare March 24, 2020 12:15
@rbehrends rbehrends force-pushed the master branch 3 times, most recently from c3391bd to c227233 Compare April 10, 2020 23:13
@rbehrends rbehrends force-pushed the master branch 2 times, most recently from 45b8823 to a83da77 Compare April 22, 2020 18:00
@rbehrends rbehrends force-pushed the master branch 3 times, most recently from 63eb841 to d0486ed Compare May 12, 2020 07:26
@rbehrends rbehrends force-pushed the master branch 2 times, most recently from e3e7505 to f635185 Compare May 20, 2020 17:29
@rbehrends rbehrends force-pushed the master branch 3 times, most recently from 2a1f195 to c8f147b Compare June 23, 2020 11:18
@rbehrends rbehrends force-pushed the master branch 2 times, most recently from 2ccc962 to 6ba8274 Compare July 23, 2020 08:41
@rbehrends rbehrends force-pushed the master branch 7 times, most recently from aeae581 to 4fee37a Compare September 4, 2020 05:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants