The review-dev service playbook should do everything now that
the puppet did. Update how we're running things.
Change-Id: I70303c48328ea6713c24bf9c6f63d4808d30b95c
When a job is killed by zuul due to failure like DISK_FULL, a different
message ends up in as a comment.
<li>job-name
finger://ze09.openstack.org/8b6d...6f : DISK_FULL in 2h 59m 50s</li>
This adds another pattern that recognize these messages as failures,
regardess the case (DISK_FULL in this case).
Change-Id: Ib17f05a043430362b02a2826d69572f6b2dbd64a
Needed-By: https://review.opendev.org/#/c/631509/
This enables automatic reload of the replication configuration for
review-dev.
Depends-On: https://review.openstack.org/650049
Change-Id: I3be630339870d527bedcfbd84b8dc8084dc10f4b
This war is based on merged code and is hosted permanently (unlike the
war hosted on our logs server that we were testing with earlier). If
this ends up being a happy war we'll deploy it to production.
Change-Id: I7a0ca93f0cd7a9eed19172d315000c3b7e4464b7
This updates the gerrit isntall on review-dev to 2.13.12 + our 8 local
patches. Will need to restart gerrit and make sur ethat things are
happy.
In particular this update comes with jgit updates.
Change-Id: Iebd5617a9e516381100d424ce5749b4cbf866ab2
Add a commentlink entry which matches task footers in commit
messages and links them to the corresponding task URL on
storyboard-dev. This has already been tested manually on review-dev,
and seems to work without breaking the separate its-storyboard entry
which handles task updating automation.
Change-Id: Iea9bde67432249e7314917ff63d6ee5e16a80f25
The current amount of backslashes is not good enough for gerrit, and it
rejects the config file. Probably related to futureparser? In any case,
we need doubled-backslashes in the generated file, so 4 backslashes does
the right thing in the puppet.
Change-Id: I950b8efbcb876b2d1309f1117626a41ef22025b0
Now that we've got base server stuff rewritten in ansible, remove the
old puppet versions.
Depends-On: https://review.openstack.org/588326
Change-Id: I5c82fe6fd25b9ddaa77747db377ffa7e8bf23c7b
Configure the its-storyboard Gerrit plugin to leave comments with
hyperlinks on stories mentioned in a change's commit message footer
whenever that change gets a new patchset, or is abandoned or merged.
This now must be explicitly configured for the version of the plugin
used with newer Gerrit (circa 2.13).
Change-Id: I14dea61a906c4b267145dce6911a10d8075b4ba9
This will allow us to bootstrap a server with gerrit users, then
attach the volumes with hold the git repos for gerrit, then we can
remove this flag and properly puppet the rest of the server.
Change-Id: I9159c941ece4f6928204601b9933d7a953baa2dd
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
Remove the sources.list
From review.pp, don't set java_home. The puppet-gerrit module figures
it out automatically.
For review-dev.pp, it seems java paths are used to import keys for
storyboard. Add matching logic for that.
Remove precise only workaround in status.o.o
[1] https://git.openstack.org/cgit/openstack-infra/puppet-gerrit/tree/manifests/init.pp#n274
Change-Id: I31c9196a32febf4760b897a3110150fcd581e173
Migrate backups to new backup01.ord.rax.ci.openstack.org
We decided to start fresh backups on the new server, so this is ready
to go. I have performed an initial backup on each server so it has
accepted the host key of the new server and been tested (I also fixed
up review-dev.o.o, which was rebuilt but keys not updated ... todo:
add this to puppet, but since it changes so infrequently not high
priority).
Change-Id: I0872f9fcf4a334d32f632b3cb04801deefab4fd1
As part of the "Gerrit ContactStore Removal" specification, now that
owners.py is checking foundation membership when generating voter
rolls it's fine for us to stop using requireContactInformation on
the ICLA and rip out our accompanying configuration.
Note that this should not be merged until the All-Projects config on
review.openstack.org and review-dev.openstack.org has been manually
adjusted to remove the "requireContactInformation = true" line from
the [contributor-agreement "ICLA"] section therein. Further, this
must be done by pushing the necessary edit directly into
refs/meta/config as there is no WebUI control nor API method to
alter contributor-agreement options.
Change-Id: I8c39a6bf43f5b12db3e8aab18bedbf7e1a0f0b7e
Story: #2001094
Task: #4867
Testing on review-dev has shown that gerrit won't honor utf8 characters
without explicitly enabling utf8 on the jdbc url string. Add utf8
parameters to the jdbc connection url for the accountPatchReviewDb on
review_dev.
Change-Id: Ib24a7e53e290cc32e9842d524d51a381aace181a
This updates the gerrit war, plugin versions, and sets the new
accountPatchReviewDb url value to use mysql. Also uses updated Gerrit
query api paths for changeid and sha1 lookups in commentlinks.
Depends-On: Ifc04395f076200a68e6398190a0712c80e5278c6
Change-Id: Ic1a116b1608eb9134d18dad9463b9b9b142700fa
Setup known_hosts file on review.o.o and review-dev.o.o so that
known services connecting to it won't be blocked.
Change-Id: If0b082867df73322a9115b94ea5be18d0631acc0
depends-on: I0bb6638f5362696cd9a9a7401cb2e96ab69eea06
upgrade Gerrit's javamelody plugin[1] on review-dev.o.o
This brings in an upstream change[2] to update javamelody to version
1.60.0
NOTE: Approving this change will kill Gerrit due to a bug, the change
to fix[3] this bug is uncommited atm. The way to avoid the bug is
to stop Gerrit, approve this change, allow puppet to propogate
the update then start Gerrit.
[1] http://tarballs.openstack.org/ci/gerrit/plugins/javamelody/javamelody-1de5d37.jar
[2] 1de5d373af
[3] https://review.openstack.org/#/c/268689/
Change-Id: I33102ce76521c5ffc25f83d3d2022f87f9732f18
Refactor and rename our Launchpad "sync" credentials (they haven't
been used to synchronize group membership from LP to Gerrit for
nearly 4 years now) which currently perform bug updates for
new/merged changes, so that they can be reused in the future to
update bugs as a part of release automation jobs.
Change-Id: Icd08dffef88fc8e99683d991ac6ac88b93bcb3e6
Now that we're running with a backport of
https://gerrit-review.googlesource.com/80320 from upstream, the
hyperlinks in comments include a rel="nofollow" parameter. Our
regular expression for these entries needs to take that into
account, so give it a try on review-dev.
Change-Id: Ida090f45cfdfb41c43c4ba1157f0478fc0936a20
The change-restored event isn't getting process by the its-storyboard
plugin due to spaces in the config string. This is a bug in plugin
and should be fixed. Until then this change is a workaround so that the
change-restored event will get processed.
Change-Id: Ic4489d029ce8409520f16c23a676125c7b3ab164
This upgrade does not require a reindex. It will pull in the following
changes:
277538 Append submitted info to the change REST endpoint.
267927 Remove required access permission to list account emails
340157 Update Google gson library to 2.3.1
345540 Add rel=nofollow parameter to links in comments
311903 Use maven repos for bouncy castle resources
260324 Add syntax highlighting support for puppet
Change-Id: I81553738f4cbbcdc3db9bd02325f0e254d9f4289
Update its-storyboard actions on review-dev.o.o to be same as on
review.o.o so it will provide more prod like testing environment.
This is basically turning off commenting to storyboard and instead
just enables task transitions.
Change-Id: I1b5a7109d90f0b3ad67afe41b43d8cbe5de471d9
Install Gerrit its-storyboard plugin[1] to support integration between
gerrit and storyboard. Install the plugin from tarballs.o.o[2] to
review-dev.o.o for testing.
Additional requirement for this change to take effect:
1. A parameter needs to be set to enable its-storyboard plugin. This
is a per project setting in refs/meta/config (project.config file)
If we want to enable for all project then set this param in
All-Projects:
[plugin "its-storyboard"]
enabled = true
[1] https://gerrit.googlesource.com/plugins/its-storyboard
[2] http://tarballs.openstack.org/ci/gerrit/plugins/its-storyboard
Change-Id: Ida9c7ece6b53df10da1a95291ec2b2de1adb7fbd
Add configurations for the its-storyboard plugin:
1. Move java_home variable to a higher level module so it can be used
to install certificate on review-dev.o.o
2. Configure its-storyboard for review.o.o and review-dev.o.o
3. Associate review.o.o with storyboard.o.o and review-dev.o.o with
storyboard-dev.o.o
4. Import ssl certificate to java on review-dev.o.o so that the plugin
can POST updates to storyboard-dev.o.o using storyboard REST APIs.
5. Configure rules (or conditions) telling its-plugin to update
storyboard tasks status on specific gerrit change updates.
Gerrit change to storyboard task transition mapping:
change abandoned -> SB task set to 'todo'
change createed or change restored -> SB task set to 'review'
change merged -> SB task set to 'merged'
Change-Id: Id6ec16e267fca5fbbc42b1d3547fc5d2fa4c671b
depends-on: I9f47a2ed88ffbe827e494a478c0dc89a08bbe370
depends-on: I5e817fab8a8973b688fd44dd819e3616df171321
Refactor the commentlinks to allow review.o.o and review-dev.o.o to
define seperate comment links. We essentially want to point
review.o.o story links to storyboard.o.o and review-dev.o.o links to
storyboard-dev.o.o
Change-Id: I70e5791a76ca97756613c393e598978ec13c8271
The generated cgit links in the Gerrit change screen contain "%2F" in URL
instead of "/" character. Add the urlEncode setting to make Gerrit
generate links with "/" character instead.
Change-Id: I07d1b3175c039bba0d8b7392c756fa6a5b2e74e9
depends-on: I0ed9f2e872617d601120a5ab15b512e30aae51b5
Change Ieecb4f03b72 added a job to build javamelody from stable-2.10
branch [1]. Since review-dev.o.o is now running Gerrit 2.10 we need
to update the javamelody plugin.
[1] https://gerrit.googlesource.com/plugins/javamelody/+/stable-2.10
Change-Id: I1c5977ab09f8689ee27d743725191ef131da0fcb
Change Ia931d053cbf setup a build for javamelody with Gerrit-2.9.x
This change Updates the javamelody plugin version on review-dev.o.o to the
new javamelody build.
The file is at http://tarballs.openstack.org/ci/gerrit/plugins/javamelody/
Change-Id: I431043bec2cd0815334cde24480307200166bf32