diff --git a/inventory/service/group_vars/zuul.yaml b/inventory/service/group_vars/zuul.yaml index ec9c290ac5..7bc0b6e680 100644 --- a/inventory/service/group_vars/zuul.yaml +++ b/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' diff --git a/playbooks/roles/gerrit/defaults/main.yaml b/playbooks/roles/gerrit/defaults/main.yaml index a05f919a32..36859e2dd5 100644 --- a/playbooks/roles/gerrit/defaults/main.yaml +++ b/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 diff --git a/playbooks/roles/gerrit/templates/gerrit.config b/playbooks/roles/gerrit/templates/gerrit.config index 21b715b99a..df942244c8 100644 --- a/playbooks/roles/gerrit/templates/gerrit.config +++ b/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 }} diff --git a/playbooks/roles/gerrit/templates/gerrit.vhost.j2 b/playbooks/roles/gerrit/templates/gerrit.vhost.j2 index 623945b175..c90d551197 100644 --- a/playbooks/roles/gerrit/templates/gerrit.vhost.j2 +++ b/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] - Require all granted Order allow,deny