Merge "Omnibus Gerrit 3.2 changes"

inventory/service/group_vars/zuul.yaml

@@ -21,7 +21,6 @@ zuul_connections:
     user: 'zuul'
     sshkey: '/var/lib/zuul/ssh/id_rsa'
     gitweb_url_template: 'https://opendev.org/{project.name}/commit/{sha}'
-    auth_type: 'digest'
 
   - name: 'opendaylight'
     driver: 'gerrit'

playbooks/roles/gerrit/defaults/main.yaml

@@ -5,7 +5,7 @@ gerrit_site_dir: "{{ gerrit_home_dir }}/review_site"
 gerrit_run_compose_up: false
 gerrit_run_init: false
 gerrit_packed_git_open_files: 4096
-gerrit_container_image: docker.io/opendevorg/gerrit:2.13
+gerrit_container_image: docker.io/opendevorg/gerrit:3.2
 gerrit_container_volumes:
   - /home/gerrit2/review_site/cache:/var/gerrit/cache
   - /home/gerrit2/review_site/etc:/var/gerrit/etc

playbooks/roles/gerrit/templates/gerrit.config

@@ -39,6 +39,9 @@
 {% if gerrit_heap_limit is defined %}
 	heapLimit = {{ gerrit_heap_limit }}
 {% endif %}
+	javaHome = /usr/local/openjdk-8/jre
+	javaOptions = "-Dflogger.backend_factory=com.google.common.flogger.backend.log4j.Log4jBackendFactory#getInstance"
+	javaOptions = "-Dflogger.logging_context=com.google.gerrit.server.logging.LoggingContext#getInstance"
 [gc]
 [core]
 	packedGitOpenFiles = {{ gerrit_packed_git_open_files }}

playbooks/roles/gerrit/templates/gerrit.vhost.j2

@@ -61,9 +61,6 @@
   #RewriteCond %{REQUEST_URI} !^/maintenance.html$
   #RewriteRule ^/(.*) /maintenance.html [last,redirect=temporary]
 
-  # TODO remove this when upgrading to Gerrit 2.16. /p/ is used
-  # for project dashboards.
-  ProxyPassMatch ^/p/ !
   ProxyPassMatch ^/robots.txt$ !
   ProxyPassMatch ^/server-status !
   # Comment out these two lines if the maintenance message above is in use
@@ -72,10 +69,6 @@
 
   Alias /robots.txt /home/gerrit2/review_site/static/robots.txt
 
-  # TODO remove this when upgrading to Gerrit 2.16. /p/ is used
-  # for project dashboards.
-  RewriteRule "^/p/.*" "-" [F,L]
-
   <Directory /home/gerrit2/review_site/git/>
     Require all granted
     Order allow,deny