[GitHub] [nifi] ferencerdei commented on pull request #6075: NIFI-9666 Implement base C2 client functionality

[GitBox <gi...@apache.org>]

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

   > It appears something has changed with the dockermaven Docker image for MiNIFi that makes it so it does not respond to interrupt signals.
   > 
   > When running the docker image in the foreground, ctrl-c (^C), should send an interrupt to stop the container and have it shut down, like so:
   > 
   > ```
   > Received trapped signal, beginning shutdown...
   > ```
   > 
   > This does not work for me on this PR branch (it does for the latest stable release version of MiNiFi Java).
   > 
   > Also, possibly relatedly, but I only see the bootstrap logs in the Docker output for this branch, whereas we should see bootstrap and app log output, correct? Again, my expectation is that this PR should not change that behavior for the minifi Docker image. I think there may have been some unintentional regressions introduced.
   
   Thanks for checking it. I've fixed the startup shell script so now the bootstrap process receives the SIGTERM signal.
   
   Regarding the logging: the output is configurable via the logback.xml, if you want to display both bootstrap and minifi process logs, you need to add the console appender to both and mount the file to the config dir. There was no change around this, it works the same with the legacy minifi.


-- 
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: issues-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org