Cleaning up responsibilities of gitlab module

Removed the mysql class references, this forces the admin to explicitly install and configure mysql - this is mostly useful when the mysql db is going to be a remote system; doesn't make sense to force install on the app server.

Also fixed a puppet variable scope warning for ldap variables in server.pp