camunda / camunda-bpm-platform

Flexible framework for workflow and decision automation with BPMN and DMN. Integration with Quarkus, Spring, Spring Boot, CDI.
https://camunda.com/
Apache License 2.0
4.1k stars 1.55k forks source link

Add Tomcat 10 Support to Docker image #4389

Closed psavidis closed 2 months ago

psavidis commented 4 months ago

Related-to: https://github.com/camunda/camunda-bpm-platform/issues/4386

psavidis commented 3 months ago

The Dockerfile downloads the latest Tomcat version from the SNAPSHOT artifact.

This task can be conveniently tested when the Tomcat 10@7.22 issue is merged to master.

psavidis commented 2 months ago

Progress Update

psavidis commented 2 months ago

Hint for the Reviewer

No code changes are needed for this task. The docker images can be tested using the following commands (using example A for demonstration purposes)

$container_id: The identifier that is returned from the run command.

psavidis commented 2 months ago

Assigning to @danielkelemen for double checking the testing. After the review, this task can be closed since there are no code changes.

danielkelemen commented 2 months ago

Since there are no code changes to review, this could be a better fit for QA to test. Anyways:

👍 Looks good. Latest docker snapshot image is using the Apache Tomcat/10.1.25 version.

psavidis commented 2 months ago

Testing is done. Closing this task.