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

Introduce Tomcat Source-to-Image (S2I) #188

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

Conversation

maxime-beck
Copy link
Contributor

Tomcat - Source-to-Image (S2I)

This resource provides a CentOS-based Docker image that enables Source-to-Image building for Tomcat. It builds the sources of a webapp and deploys it to a fully functional containerized Tomcat Server. The generated image can then easily be run locally or deployed to a Kubernetes-based Server.

Usage

  1. Build an image:
$ s2i build [SOURCE_URL] tomcat-s2i my-webapp-image

Where [SOURCE_URL] is either the URL to a Git repository or a path to local sources.

  1. Run it:
$ docker run -p 8080:8080 my-webapp-image

@michael-o
Copy link
Member

I am against pulling this one for several reasons:

  • This is CentOS-specific and OpenShift
  • Someone has to maintain it, likely none of us won't
  • It duplicates several configuration files

This is best handled by the community or by Red Hat ($$$)

@jfclere
Copy link
Contributor

jfclere commented Aug 13, 2019

Actually using Alpine like in res/tomcat-maven/Dockerfile might be a good idea.
The tool to build the image is independent of Openshift and the image will run in Kubernetes too.

@maxime-beck
Copy link
Contributor Author

Yes, switching to Alpine is definitely possible. I can update the PR if needed.

@ChristopherSchultz
Copy link
Contributor

I'm not sure this is something appropriate for the Tomcat team to adopt. Sounds like a great community-led project to exist outside of Tomcat, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants