asqasq commented on issue #76: Crail Kubernetes Deployment on POD Network with 
revised docker configuration to acomodate Node Discovery
URL: https://github.com/apache/incubator-crail/pull/76#issuecomment-500727408
 
 
   Hi Kaustav
   
   Actually I do understand the flow.
   
   This PR cannot be merged for several reasons:
   1. In my opinion it is not the right way to just copy files from other 
people and claim that
      they are youres (even more, you copied our files from our repositories)
   2. The Pocket files do not work with the latest Crail version. Pocket was a 
research project
       based on a older and especially a modified version of Crail.
   3. It seems to me that you don't understand what you are doing
   4. I doubt that you tested it, especially after your last change. Deleting 
few files
       could potentially mean that something does not work anymore (I doubt 
that it worked before).
   5. We cannot reference docker images from a non-trusted source in the yaml 
files.
   
   Therefore, we cannot merge this PR.
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to