Commit Graph

140 Commits

Author SHA1 Message Date
James E. Blair 19fd812c7a Document gerrit comment deletion procedure
Change-Id: Iedc9446ea12415c8f9ba49b7f8390442045762fa
2024-02-07 10:57:23 -08:00
Clark Boylan 5c02d66231 Document gerrit openid login failure debugging
This tries to cover a few common cases of Gerrit login failures.
Breadcrumbs for the future to make potential debugging easier.

Change-Id: If935601d4b102d41c186e43bf4c4b77054e48bba
2024-01-24 09:30:20 -08:00
Jeremy Stanley d3224e71b4 Enable receive.rejectImplicitMerges for Gerrit
Document our intent to enforce the rejectImplicitMerges option for
receipt of changes, in order to avoid silently merging the target
branch to include the change's history from another branch where
those commits are already present.

Change-Id: I7478825ed58c5a4e7cf29d06a770e70c7b5a7759
2023-06-05 22:16:19 +00:00
Clark Boylan 64957e9e0d Fix boolean operator in submittableIf rules
Gerrit 3.6 and older do not support 'and' as a boolean operator. They
only support 'AND'. For maximum compatibility (we are running 3.6.4
currently) convert 'and' to 'AND' in Gerrit submittableIf rules.

Change-Id: Iac5e2cda4a245f99e98a1354ad4107da07e1f60d
2023-03-09 08:33:22 -08:00
Ian Wienand b37ec6632a
doc/gerrit : update to submit-requirements
Update the MaxWithBlock function in the default labels to a new-format
submit-requirement.

Change-Id: I9887c4949fe375db0397bf273820e57a9d539749
2023-03-08 10:24:54 +11:00
Ian Wienand 22506e5682
doc/gerrit : update copyCondition
Update the deprecated copy flags to copyCondition in the boostrap
documentation.

The verified and workflow labels don't ever copy their approvals, so
it is really only code-review.

Change-Id: I3563a7a394ae7d96af9e27b10dc18ba5c459ba82
2023-03-07 10:21:43 +11:00
Ian Wienand 4f3be693e5 gerrit docs: add note that duplicate user may have email addresses to remove
Change-Id: I54e935704d38dd60cdf75415dd8c61b06cae7b11
2022-06-24 15:37:54 +10:00
Ian Wienand 3a09bf7e8a gerrit docs: cleanup and use shell-session
A few formatting fixes

* try to more consistently use shell-session formatting for shell
  sessions (makes it easier to copy-paste).
* fix up and use more `` around verbatim/code things.

Fixes:

 * Gerrit Configuration : there's no db to set the ICLA fields in now,
   remove
 * Duplicate Accounts : add required arg "origin" to git fetch command
 * Deactivating account : can not delete comments via sql query,
   remove

Change-Id: Ia481750aa59fc88bef5c00bb0fd9e6f9e23b2777
2022-06-24 15:37:52 +10:00
Jeremy Stanley 15e07d8a37 Fix a trivial typo for repo renaming playbook path
Change-Id: I3cb5c42df9cc4e9c308da592353b7867391211a3
2022-04-15 14:49:00 +00:00
Clark Boylan bdeef4e4ce Add info on running jstack against gerrit to docs
This is meant to help debug gerrit in some circumstances particularly
now that the Java Melody plugin is not installed.

Change-Id: Ifedb7abd08c7fe1281ac510c6872fe8d9fe700a1
2022-01-31 12:06:05 -08:00
Clark Boylan dce8b2b4a6 Rename Gerrit Non-Interactive Users to Service Users
This is just a documentation update but reflect the change upstream
Gerrit made in versions 3.3 renaming this group.

Change-Id: I5458afd2683c2a7c4616f4894884e3d3ce03bbaf
2021-10-11 08:53:59 -07:00
Clark Boylan 87a67c7656 Update our project rename docs
This update captures that we host projects outside of openstack and
intend for projects like openstack or others to do some steps on their
own. We also update this to reflect chagnes in the configuration
management and deployment tooling that we use today.

Change-Id: I0bc0ce335fd90e6187253e18007361a133a8f30c
2021-08-13 13:33:48 -07:00
Ian Wienand d4ba30251b gerrit docs : remove old database docs/update duplicate account info
A lot of the current sections here talk about modify the Gerrit
database that no longer exists.  Remove these.

Update the section on duplicate accounts to handle removing the second
account via NoteDB and the API.

Change-Id: I2139ff33d87bf42e4453f6e7252fcc427594967a
2021-08-12 08:37:34 +10:00
Ian Wienand e79e3a2f04 Remove review01 references
This server is no longer in production, so remove the various
references to it.

Change-Id: I2cdd8052c48713e9ba648be20ccad5069d5fe40e
2021-07-20 11:57:10 +10:00
Jeremy Stanley b87e938a13 Clean up Gerrit global config documentation
Recent work has concluded adding OpenStack Release Manager
permissions explicitly to all openstack/ namespace projects with the
addition of inheritance from openstack/meta-config in their
individual ACLs. This made the earlier Release Manager permissions
in our global configuration redundant, so it's being removed. The
cleanup is done by hand due to how global configuration is managed
in Gerrit's All-Projects metaproject, but we're updating our
documentation to reflect it.

While here, clean up obsolete references to API-Projects inheritance
and stable/.* branch permissions which we've not applied for some
years now.

Change-Id: Ib9314f7a1deb3d343eb2d9b476064de41186f57a
2021-05-03 16:36:48 +00:00
Ian Wienand 9f11fc5c75 Remove references to review-dev
With our increased ability to test in the gate, there's not much use
for review-dev any more.  Remove references.

Change-Id: I97e9865e0b655cd157acf9ffa7d067b150e6fc72
2021-03-24 11:40:31 +11:00
Jeremy Stanley 9bac4659e2 Correct "ansbile" typos
Two mistypings of the string "ansible" (case insensitive) as
"ansbile" appear in our documentation. One of them is in a sample
command, which is particularly dangerous. Correct both.

Change-Id: Ib644f57060f467d4bfd70be60225e39385d38737
2020-06-12 22:41:54 +00:00
Monty Taylor 5468f49254 Remove unused gerrit puppet things
We ain't using em.

Change-Id: I4ce9188a6b6a7e6a670e61bb17ab07e890faebcf
2020-04-19 10:59:25 -05:00
Zuul e71221ea33 Merge "Add a note about rename files to project renames doc" 2020-04-09 14:30:24 +00:00
Andreas Jaeger 2c0b82e5e8 Update infra-manual location
The infra-manual now lives on docs.opendev.org, update links.

New location is: https://docs.opendev.org/opendev/infra-manual/latest

Change-Id: I7716c68cbff4f3a640d7161f59cfc034a7ccca52
2020-03-20 22:03:09 +01:00
James E. Blair fc2a742b24 Add a note about rename files to project renames doc
We keep track of these files now in the opendev/project-config repo,
so make sure that they are committed there.

Change-Id: Icf4b4e32ac4f209811ba8361bbb9d8458c79251a
2020-03-20 07:09:56 -07:00
Monty Taylor d500651367 Rename cgit_file to git_file
In sphinx, we have a :cgit_file: directive that makes links to files.
Thing is - we're not using cgit anymore. So just rename it to git_file.

Change-Id: I80aca5fb3cc84281e29843944fea33e6f4d9fe6f
2019-04-22 11:47:11 +00:00
Monty Taylor eaa74543de Finish updating docs for opendev
The zuul and zuulv3 docs need to be merged, but that seemed like
too much for this. Also, the 3rd party CI doc is out of date, but
in this patch only removed sections that linked to docs or files
that don't exist anymore.

Change-Id: Ie5497edd762d2146165608f3227b0bac88a913df
2019-04-20 18:25:37 +00:00
Monty Taylor e01ed4f066 Update some docs for opendev
There's a lot of these, so doing them in chunks. This fixes
the custom roles.

Remove the git and jjb docs, since we don't use them anymore.

Change-Id: I0c5b74f7b73315dac93bce6be0d920cddb94fb58
2019-04-20 09:41:45 -07:00
Zuul d3b9dfb04a Merge "Ectomy some Jenkins out of the docs" 2018-12-12 11:53:39 +00:00
Monty Taylor 55ddc7bf88
Ectomy some Jenkins out of the docs
There are a bunch of places where the narrative text say things like
"jenkins runs jobs". This hasn't been true for a while and it's getting
less true.

Left third-party and running-your-own alone because those are
instructions for other people to do things - and we are not yet at the
point where we are suggesting anyone do their things like we do our
things.

The devstack-gate document describes an old deprecated system, but the
system is still in use (sadly) so it was mostly left intact. A warning
was added so that people would be clear that it was deprecated.

Also removed the logstash client config file. It's the only change in
here that actually affects running code and became unnecessary when it
was switched to geard with Ie3f814e6d3278d87f2a20a72e40b6b92217684fc

Change-Id: Iaf2128c3f953976180c71cb599fcbff7bc06c28a
2018-12-10 17:55:04 +00:00
Zuul dad0716592 Merge "Fix enumerated list in gerrit.rst" 2018-12-10 01:47:35 +00:00
Xing Zhang 2d049f4524 Fix enumerated list in gerrit.rst
Remove incorrect whitespace

Change-Id: I45203210da5288c14f1f273513b20160add73227
Ref: docs.o.o/infra/system-config/gerrit.html#troubleshooting
2018-12-10 11:23:27 +11:00
François Magimel 46260a79ee Fix spelling mistakes and reST typos in the doc
Change-Id: I61d9780f3f1937c6e8d326a670c40fb6a931dbce
2018-12-08 19:13:53 +01:00
Monty Taylor 1a8c2f66da
Move /opt/system-config/production to /opt/system-config
The production directory is a relic from the puppet environment concept,
which we do not use. Remove it.

The puppet apply tests run puppet locally, where the production
environment is still needed, so don't update the paths in the
tools/prep-apply.sh.

Depends-On: https://review.openstack.org/592946
Change-Id: I82572cc616e3c994eab38b0de8c3c72cb5ec5413
2018-08-17 09:41:02 -05:00
Zuul 04aac06820 Merge "Update Gerrit project renaming for Zuul v3" 2018-08-01 16:45:10 +00:00
Ian Wienand 882b730fdf Update to openstackdocstheme
This modernises the openstack-infra documentation by switching to
openstackdocstheme.  Update dependencies as required.

To remove non-relevant stuff from conf.py, I have just taken the demo
file from openstackdocstheme and lightly modified it.

It seems later sphinx has included it's own ":file:" role which now
conflicts.  Change it it ":cgit_file:" in our documentation.  Remove
the custom header template which no longer applies.  Add the
post-2.0-pbr sphinx-based warning-as-error, which fixes the original
problem that I actually noticed that errors could slip through the
gate tests :)

Change-Id: Ic7bec57b971bb4c75fc839e7269d1f69a576b85c
2018-06-25 11:19:43 +10:00
Jeremy Stanley cbbceb2330 Update Gerrit project renaming for Zuul v3
With the switch to Zuul v3, we need to resolve some configuration
catch-22s where project names and related in-repository job
definitions can't happen without a complex multi-stage removal and
reintroduction process to get it through speculative testing
successfully. For now, just punt and use monolithic changes
bypassing CI in code review. As an up side, the Ansible automation
of this process coupled with Zuul v3's increased resilience to
on-the-fly configuration changes means we can skip stopping/starting
it now and significantly simplify the process.

Since we're here, correct the section heading level for
"Force-Merging a Change" in the sysadmin document.

Change-Id: I335c23abd0b5706f43bbea2dd8cfffa4280dd5db
2018-03-19 15:26:58 +00:00
Ian Wienand 2c52fa6cbe Add notes on Release Managers abandon permissions
We are granting abandon permissions for release managers so that
changes on EOL branches can be removed.

Change-Id: I7486fead3d1066e0c45cd3a9c2ff1545849bd7e3
2017-10-25 10:47:36 +11:00
Jeremy Stanley d5dad4fd09 Stop using the Gerrit Contact Store
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
2017-08-04 20:24:37 +00:00
Monty Taylor a8f6b554c6
Document Zuul v3 GitHub integration
We have GitHub integration for Zuul v3, so we should have documentation
of that integration.

Change-Id: Ibbd41a43338f1d439d47aa13ced60ed477a6bbef
2017-07-27 11:18:16 -05:00
Paul Belanger b1b5f7cc4c Additional updates for 'Duplicate Accounts in Gerrit'
Make sure you set the old account to inactive to complete the process.

Change-Id: I2dd3d639967ff8164beb0e9c07fdfb2520b84b45
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-03-02 11:52:48 -05:00
Paul Belanger 47d5314b47 Create Continuous Integration Tools Development group
We'd like to have zuulv3-dev.o.o be able to vote -1/+1 on all
projects. For that, create the Continuous Integration Tools
Development group in gerrit global ACLS.

Change-Id: I0190be91feb0198c147b96628cfb02c5abc626fe
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2017-02-21 14:54:35 -05:00
Ian Wienand def483ab74 Updates for rename instructions
Small updates: show an example of multiple projects in the yaml file,
and warn about when to restart puppet

Change-Id: I77c07035f0df775558d82fa54b968f1acdbfb713
2017-01-13 08:03:45 +11:00
Jenkins 5039ba4ebb Merge "Update project rename instructions" 2016-11-22 23:53:11 +00:00
Jeremy Stanley 65ed837e31 Update project rename instructions
When I919ba42b0d22126719daa7ad308f75ce021720b7 merged, it introduced
a few regressions into our process:

  * Github renaming/transferring was dropped

  * Switched to a very slow (for our environment) Zuul stopping
    method

  * it advocated for composing a rename parameters file very late
    in the process

This change fixes the above issues. It also updates the
documentation to note that Puppet should be stopped well in advance
of the maintenance window, and updates the playbook to no longer run
an offline Gerrit reindex (since online reindexing is now
supported).

Change-Id: Ie249214c0d1b1df6c66d4910002e35d8c17c3b69
2016-11-22 13:41:56 -08:00
Khai Do 54b9af4786 doc: add Storyboard integration info
Change-Id: I17f35142ea29f36efdb8810e35ca898f6dc1218b
2016-08-26 13:09:14 -07:00
Jeremy Stanley 7cee62f7d9 Document use of its-storyboard in Gerrit config
Update Gerrit deployment documentation to cover the its-storyboard
plugin addition in our configuration.

Change-Id: Iaa84d50756b00a0c83b087a0f4ad7ccd38e6b293
2016-08-20 15:37:33 +00:00
Jeremy Stanley ab7c9c449d Project renames may need governance updates
Change-Id: Id0fe4db26ab30de1b93f9937d5b17cb640714677
2016-06-15 20:23:10 +00:00
Jeremy Stanley 1138d74913 Check openstack/openstack in rename process
In the project rename process documentation, amend the list of
reminders to include checking .gitmodules in openstack/openstack as
we've missed it more than once in the past.

Change-Id: I47b09ed317a14222088097c84a5bab157e3d5b47
2016-05-19 13:30:33 +00:00
James E. Blair 43c3a31674 Remove proposed/ acls
I am reliably informed these are not used anymore.  Note, this
change updates All-Projects ACLs, so on approval, we should make
the corresponding change in Gerrit.

Change-Id: I1073663b8f80de8639a75b4b155258c1abed9df9
2016-04-21 07:58:42 -07:00
Jenkins 89eddd7cc7 Merge "Add troubleshooting section to github" 2016-03-02 15:25:05 +00:00
Yolanda Robla 0471bfb40d Add troubleshooting section to github
Create a section about how to fix the replication problems
to github, when gerrit team member needs to be added to the
project.

Change-Id: Ic3354b7c1d49ebdd6294698ab32062421f3c5569
2016-02-18 15:10:58 +01:00
Yolanda Robla 74e36c01f4 Add instructions on how to disable a third party ci account
Change-Id: Idb04a31a6b3b4697d2199e601b0a1b4d4bff0c85
2016-02-02 16:53:00 +01:00
Jenkins abec9d984a Merge "Add documentation about how to reload Gerrit HTML" 2016-01-30 07:53:16 +00:00