Merge branch 'git_data_dir' into 'master'

Git Data Dir

Merge branch 'git_data_dir' into 'master'
Git Data Dir
Jacob Vosmaer
2 parents 33d1c979 d0cdcf74
Showing 4 changed files with 37 additions and 13 deletions Show diff stats
README.md
files/gitlab-cookbooks/gitlab/attributes/default.rb
files/gitlab-cookbooks/gitlab/libraries/gitlab.rb
files/gitlab-cookbooks/gitlab/recipes/gitlab-shell.rb
@@ -98,9 +98,20 @@ Omnibus-gitlab uses four different directories.
 - `/var/log/gitlab` contains all log data generated by components of
   omnibus-gitlab.
+### Storing Git data in an alternative directory
+
+By default, omnibus-gitlab stores Git repository data in `/var/opt/gitlab/git-data`.
+You can change this location by adding the following line to `/etc/gitlab/gitlab.rb`.
+
+```ruby
+git_data_dir "/mnt/nas/git-data"
+```
+
+Run `sudo gitlab-ctl reconfigure` for the change to take effect.
+
 ## Building your own package
 See [the separate build documentation](doc/build.md).
 [downloads]: https://www.gitlab.com/downloads
-[CE README]: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/README.md
 \ No newline at end of file
+[CE README]: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/README.md
@@ -86,10 +86,10 @@ default[&#39;gitlab&#39;][&#39;gitlab-rails&#39;][&#39;ldap_method&#39;] = &quot;ssl&quot;
 default['gitlab']['gitlab-rails']['ldap_bind_dn'] = "_the_full_dn_of_the_user_you_will_bind_with"
 default['gitlab']['gitlab-rails']['ldap_password'] = "_the_password_of_the_bind_user"
 default['gitlab']['gitlab-rails']['ldap_allow_username_or_email_login'] = true
-default['gitlab']['gitlab-rails']['satellites_path'] = "/var/opt/gitlab/gitlab-satellites"
+default['gitlab']['gitlab-rails']['satellites_path'] = "/var/opt/gitlab/git-data/gitlab-satellites"
 default['gitlab']['gitlab-rails']['backup_path'] = "tmp/backups"
 default['gitlab']['gitlab-rails']['gitlab_shell_path'] = "/opt/gitlab/embedded/service/gitlab-shell/"
-default['gitlab']['gitlab-rails']['gitlab_shell_repos_path'] = "/var/opt/gitlab/repositories"
+default['gitlab']['gitlab-rails']['gitlab_shell_repos_path'] = "/var/opt/gitlab/git-data/repositories"
 default['gitlab']['gitlab-rails']['gitlab_shell_hooks_path'] = "/opt/gitlab/embedded/service/gitlab-shell/hooks/"
 default['gitlab']['gitlab-rails']['gitlab_shell_upload_pack'] = true
 default['gitlab']['gitlab-rails']['gitlab_shell_receive_pack'] = true
@@ -125,6 +125,7 @@ default[&#39;gitlab&#39;][&#39;sidekiq&#39;][&#39;log_directory&#39;] = &quot;/var/log/gitlab/sidekiq&quot;
 # gitlab-shell
 ###
 default['gitlab']['gitlab-shell']['log_directory'] = "/var/log/gitlab/gitlab-shell/"
+default['gitlab']['gitlab-shell']['git_data_directory'] = "/var/opt/gitlab/git-data"
 ###
@@ -31,11 +31,13 @@ module Gitlab
   postgresql Mash.new
   redis Mash.new
   gitlab_rails Mash.new
+  gitlab_shell Mash.new
   unicorn Mash.new
   sidekiq Mash.new
   nginx Mash.new
   node nil
   external_url nil
+  git_data_dir nil
   class << self
@@ -103,6 +105,14 @@ module Gitlab
       Gitlab['gitlab_rails']['gitlab_port'] = uri.port
     end
+    def parse_git_data_dir
+      return unless git_data_dir
+
+      Gitlab['gitlab_shell']['git_data_directory'] ||= git_data_dir
+      Gitlab['gitlab_rails']['gitlab_shell_repos_path'] ||= File.join(git_data_dir, "repositories")
+      Gitlab['gitlab_rails']['satellites_path'] ||= File.join(git_data_dir, "gitlab-satellites")
+    end
+
     def generate_hash
       results = { "gitlab" => {} }
       [
@@ -110,6 +120,7 @@ module Gitlab
         "user",
         "redis",
         "gitlab_rails",
+        "gitlab_shell",
         "unicorn",
         "sidekiq",
         "nginx",
@@ -125,6 +136,7 @@ module Gitlab
     def generate_config(node_name)
       generate_secrets(node_name)
       parse_external_url
+      parse_git_data_dir
       generate_hash
     end
   end
@@ -44,16 +44,16 @@ execute &quot;chcon -t ssh_home_t #{ssh_dir}&quot; do
   only_if "id -Z"
 end
-directory log_directory do
-  owner git_user
-  mode "0700"
-  recursive true
-end
-
-directory gitlab_shell_var_dir do
-  owner git_user
-  mode '0700'
-  recursive true
+[
+  log_directory,
+  gitlab_shell_var_dir,
+  node['gitlab']['gitlab-shell']['git_data_directory']
+].each do |dir|
+  directory dir do
+    owner git_user
+    mode "0700"
+    recursive true
+  end
 end
 template_symlink File.join(gitlab_shell_var_dir, "config.yml") do
	@@ -98,9 +98,20 @@ Omnibus-gitlab uses four different directories.		@@ -98,9 +98,20 @@ Omnibus-gitlab uses four different directories.
98	- `/var/log/gitlab` contains all log data generated by components of	98	- `/var/log/gitlab` contains all log data generated by components of
99	omnibus-gitlab.	99	omnibus-gitlab.
100		100
		101	+### Storing Git data in an alternative directory
		102	+
		103	+By default, omnibus-gitlab stores Git repository data in `/var/opt/gitlab/git-data`.
		104	+You can change this location by adding the following line to `/etc/gitlab/gitlab.rb`.
		105	+
		106	+```ruby
		107	+git_data_dir "/mnt/nas/git-data"
		108	+```
		109	+
		110	+Run `sudo gitlab-ctl reconfigure` for the change to take effect.
		111	+
101	## Building your own package	112	## Building your own package
102		113
103	See [the separate build documentation](doc/build.md).	114	See [the separate build documentation](doc/build.md).
104		115
105	[downloads]: https://www.gitlab.com/downloads	116	[downloads]: https://www.gitlab.com/downloads
106	-[CE README]: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/README.md
107	\ No newline at end of file	117	\ No newline at end of file
		118	+[CE README]: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/README.md