nickwallen commented on a change in pull request #1261: METRON-1860 new developer option for ansible in docker to deploy to vagrant URL: https://github.com/apache/metron/pull/1261#discussion_r279561072
########## File path: metron-deployment/development/centos6/docker_run_ansible.sh ########## @@ -14,15 +16,25 @@ # See the License for the specific language governing permissions and # limitations under the License. # -[defaults] -host_key_checking = False -library = ../extra_modules -roles_path = ../roles -pipelining = True -remote_user = centos -forks = 20 -log_path = ./ansible.log - -# fix for "ssh throws 'unix domain socket too long' " problem -[ssh_connection] -control_path = ~/.ssh/ansible-ssh-%%h-%%r + + +# +# This script runs IN the docker container +# + +cd /root/metron || exit 1 + +# make sure we have the right c++ tools + +#shellcheck disable=SC1091 +source /opt/rh/devtoolset-6/enable + +sed -i -e '/^node1.*/d' ~/.ssh/known_hosts Review comment: It my test run, I am getting this error. ``` sed: can't read /root/.ssh/known_hosts: No such file or directory ``` I understand this won't be a problem on the second run, but maybe we can guard this with a file existence check. ---------------------------------------------------------------- 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] With regards, Apache Git Services
