ferencerdei commented on PR #6075:
URL: https://github.com/apache/nifi/pull/6075#issuecomment-1154777327

   > Thansk @ferencerdei. I've verified the SIGTERM fix works as expected.
   > 
   > Regarding the logging output: I'm comparing the output of the docker image 
from this PR to the [latest release of nifi-minifi 
](https://hub.docker.com/r/apache/nifi-minifi/tags), and I get the full output 
(more than just the bootstrap process) when I run this:
   > 
   > ```
   > docker run apache/nifi-minifi:latest
   > ```
   > 
   > compared to this (after building this PR branch), which only has the 
bootstrap output:
   > 
   > ```
   > docker run apacheminifi:latest
   > ```
   > 
   > I could be missing something, but I would expect those to more or less be 
the same in docker container output behavior?
   
   Did you build the new version with the dockerhub dockerfile version? It has 
an additional "tail -F "${MINIFI_HOME}/logs/minifi-app.log" &" part in the 
startup script compared to the dockermaven. We didn't touch that part, so it 
should work the same.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to