tigerquoll commented on a change in pull request #1503: METRON-2247 - rpm-docker: Provide an option to bypass running rpmlint URL: https://github.com/apache/metron/pull/1503#discussion_r324021975
########## File path: metron-deployment/development/centos7/ansible.cfg ########## @@ -21,6 +21,7 @@ roles_path = ../../ansible/roles pipelining = True log_path = ./ansible.log callback_plugins = ../../ansible/callback_plugins +callback_whitelist = profile_tasks Review comment: Ahh.. not directly related to the PR, it was more for situational awareness of the ansible steps. It prints out a breakdown of timings of the various ansible steps once ansible has finished running. It is how I identified rpmlint as a candidate to fix up. I think the devs should keep good awareness of the time taken for everything leading up to dev deployment so that any changes in deployment time are immediately noticed. Easy enough to remove if you don't think it should be there. ---------------------------------------------------------------- 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