Gerrit supports turning off the automatic full replication on startup.
The default config value is on, so set the default here to on as well
so that this is not a behavior change. OpenDev will want to set this
to false.
Change-Id: Ibb16d212ace90750ced51ff43e714532144f577c
The default is now 90 seconds and we run into that and timeout which
confuses systemd. To avoid confusing systemd increase the timeout to its
old value of 300 seconds which worked back with older Gerrit. Hopefully
this doesn't confuse systemd anymore.
Change-Id: I8476c832f950e18035e11f225202eff8572f5ce8
We need to set type = gitweb. Also, the gitweb script itself
is in /usr/share from the git package.
Change-Id: I7e73a0ba721ed93e4acf308821ad3efd3d5eb4c8
If we want to link directly to opendev urls, we need gerrit to
understand what the link structure looks like. Add support.
Change-Id: I8efefe365f3b9ebe97c8c2ce322fa8c6f3b70b3a
Sometimes one would like to only replicate a subset of the refs in
a git repo, such as not replicating refs/changes.
Change-Id: Ibf6c02d59c6901f82be6a96b676b1bd28c4b61f7
Gerrit allows for multiple footers in a single trackingid. If a
footers list is provided, iterate through all its entries; if not
though, we should continue supporting the old footer string
as a fallback.
Change-Id: I99e16246e81b7ebc50e96054a2e3790aed0089bc
Gerrit, as of 2.12, no longer implements the Contact Store
subfeature for their contributor agreement implementation.
OpenStack's Gerrit deployment no longer relies on this, and it's
likely they were the only actual consumer to start with so probably
safe to clean up at this point.
Change-Id: I50572d436b9d59a6184576adb5c27a3fc0fbec04
Depends-On: I8c39a6bf43f5b12db3e8aab18bedbf7e1a0f0b7e
Story: #2001094
Task: #4869
Testing on review-dev has shown that newer gerrit won't support utf8
properly unless we explicitly enable utf8 on the mysql jdbc string.
Without this non ascii characters get entered into the db as literal
"?". Go ahead and add explicit utf8 parameters to the database url in
this change.
Note that this should be safe on older gerrit too as gerrit should
ignore any config options it doesn't know about.
Change-Id: Ib097f93951904951a285ec40a21096988fb13562
Add accountPatchReviewDb.url as a configurable option to the secure
config file (we put it in secure config as the url likely contains a
database password in it). This is a new database in Gerrit 2.13 which
will default to using an H2 db in review_site/db if a url is not
specified.
Note that this database must have a different name than the review
database (called 'reviewdb' by this module) and if they are the same
that database will be dropped and you will have a sad afternoon.
Change-Id: Ifc04395f076200a68e6398190a0712c80e5278c6
We serve robots.txt out of the gerrit static dir, but did not allow
apache to access it. This corrects that.
Change-Id: Ibf7338641247bb5f66e3a9d7be92ce45a72da1fd
The 'event-type' reference should be 'event_type' instead.
Also fixed a bug in the documentation and subsequent beaker test.
Change-Id: I4802f49e1fb1d3210aa49b4fe12a6e33a9f3bd42
This change will allow puppet to configure the its-storyboard plugin.
Settings in gerrit.config and secure.config is to associate gerrit
with an its plugin and settings in actions.config is to tell the
plugin to do something when a gerrit change gets updated. These
conditions are provided by the its-base plugin[1].
[1] https://gerrit.googlesource.com/plugins/its-base/+/stable-2.11/src/main/resources/Documentation/config-common.md
Change-Id: I9f47a2ed88ffbe827e494a478c0dc89a08bbe370
The java_home is needed in review.pp and review-dev.pp (in
system-config) so instead of defining it in here allow a parent
module to pass in the value.
Change-Id: I5e817fab8a8973b688fd44dd819e3616df171321
The non-default safe mimetype in gerrit exists purely to proect installs
against setting this - when set, downloading a text/html file will
result in that file being rendered.
Change-Id: I648ada0b26aaf35ece9ef57f609b46f23f6e422a
The previous value, 2m, actually means 2 minutes, which is not in line
with the stated desires in the patch. After discussion in IRC, it seems
that people are less comfortable with 2 months. Let's try 7 days.
Change-Id: I95c20373d572b061e870103a813fef1d25de4b23
Gerrit 2.11 has a gerrit.canonicalWebUrl[1] option that will display
an alternative git repo location to users using the git protocol.
[1] https://review.openstack.org/Documentation/
config-gerrit.html#gerrit.canonicalGitUrl
Change-Id: I5bd57d91b4d4a685328efe4d589a191debaa0fe5
This allows folk to go on holiday or whatever and not have to deal
with Gerrits rough handling of session expiry.
Change-Id: I39aff380aa6a41ba61efebfb4691bc6727d453ce
Gerrit 2.11 has a sshd.batchThreads[1] option that will create
a seperate queue for ssh requests from non-inertactive users.
[1] https://gerrit-documentation.storage.googleapis.com/Documentation/
2.11/config-gerrit.html#sshd.batchThreads
Change-Id: I1dd229dec0aa64616e620caf51105a77c9190ece
Allow puppet to set Gerrit's sshd.idleTimeout[1] and
sshd.maxConnectionsPerUser options to help control the
number of incoming ssh connections.
[1] https://gerrit-documentation.storage.googleapis.com/Documentation/
2.11/config-gerrit.html#sshd.idleTimeout
Change-Id: I34e5fd4ea332779c4fd97f8bf5c5abf5053320c4
Gerrit has options to configure the diff and intraline timeouts[1],
setup puppet to allow overriding those configs.
[1] https://gerrit-documentation.storage.googleapis.com/Documentation/
2.11/config-gerrit.html#cache.diff.timeout
Change-Id: I038211be70d25baf8fade55fd42bcefe77409204
Gerrit 2.11 has a new sendmail.includeDiff[1] option that when set to true
will add the unified diff info of that change in the notification email.
This change allows puppet to configure this setting.
[1] https://gerrit-documentation.storage.googleapis.com/Documentation/
2.11/config-gerrit.html#sendemail.includeDiff
Change-Id: I28043548e5ec9b00e9446445efb0299792ce5540
Monty Taylor