search

cloudfoundry / cf-mysql-release

Cloud Foundry MySQL Release
Apache License 2.0
58 stars 106 forks source link

Tighten up IPsec monit check #221

Closed xtreme-nitin-ravindran closed 5 years ago

xtreme-nitin-ravindran commented 5 years ago

Thanks for opening a PR. Please make sure you've read and followed the Contributing guide, including signing the Contributor License Agreement.

This PR fixes an issue where the mariadb_ctl script incorrectly exits with "Exiting for restart because IPsec is present but not yet running" message if the vm hostname contain "ipsec"

Command to check IPsec job is /var/vcap/bosh/bin/monit summary | grep ipsec is too permissive. In our case it captured System 'system_vm-b7770176-d990-4543-64af-3f9439760ba8.c.cf-xxx-xxx-ipsec.internal' running

cf-gitbot commented 5 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/162668900

The labels on this github issue will be updated when the story is started.

cfdreddbot commented 5 years ago

:white_check_mark: Hey xtreme-nitin-ravindran! The commit authors and yourself have already signed the CLA.

ctaymor commented 5 years ago

Hi @xtreme-nitin-ravindran, Just giving an update: ee are trying to give this a try with ipsec, so that we can get it merged in. We're happy to accept the change, we've just been trying to figure out the proper approach to testing it.