adoroszlai commented on pull request #1667: URL: https://github.com/apache/ozone/pull/1667#issuecomment-746560661
Thanks @fapifta for taking another look, and also your initial review and nudging me to fix it properly. > Now I am guessing, but based on your change envtoconf.py is using transformation.py and with adding it to the docker-compose.yaml file, we effectively overriding the original transformation.py that is backed into the image(?). Exactly. This change is necessary only for `apache/ozone` and `apache/hadoop` images, since `apache/ozone-runner` requires Ozone binaries to be mounted anyway. We can remove these mounts if/when the images are rebuilt with fixed `envtoconf`. Answered the other part of your question about `transformation.py` in the separate PR where this is being fixed (should be merged before this one). ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
