| Shawn O. Pearce | e31d02c | 2009-12-08 12:21:37 -0800 | [diff] [blame] | 1 | Gerrit Code Review - System Design |
| 2 | ================================== |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 3 | |
| 4 | Objective |
| 5 | --------- |
| 6 | |
| 7 | Gerrit is a web based code review system, facilitating online code |
| 8 | reviews for projects using the Git version control system. |
| 9 | |
| 10 | Gerrit makes reviews easier by showing changes in a side-by-side |
| 11 | display, and allowing inline comments to be added by any reviewer. |
| 12 | |
| 13 | Gerrit simplifies Git based project maintainership by permitting |
| 14 | any authorized user to submit changes to the master Git repository, |
| 15 | rather than requiring all approved changes to be merged in by |
| 16 | hand by the project maintainer. This functionality enables a more |
| 17 | centralized usage of Git. |
| 18 | |
| 19 | |
| 20 | Background |
| 21 | ---------- |
| 22 | |
| 23 | Google developed Mondrian, a Perforce based code review tool to |
| 24 | facilitate peer-review of changes prior to submission to the central |
| 25 | code repository. Mondrian is not open source, as it is tied to the |
| 26 | use of Perforce and to many Google-only services, such as Bigtable. |
| 27 | Google employees have often described how useful Mondrian and its |
| 28 | peer-review process is to their day-to-day work. |
| 29 | |
| 30 | Guido van Rossum open sourced portions of Mondrian within Rietveld, |
| 31 | a similar code review tool running on Google App Engine, but for |
| 32 | use with Subversion rather than Perforce. Rietveld is in common |
| 33 | use by many open source projects, facilitating their peer reviews |
| 34 | much as Mondrian does for Google employees. Unlike Mondrian and |
| 35 | the Google Perforce triggers, Rietveld is strictly advisory and |
| 36 | does not enforce peer-review prior to submission. |
| 37 | |
| 38 | Git is a distributed version control system, wherein each repository |
| 39 | is assumed to be owned/maintained by a single user. There are no |
| 40 | inherit security controls built into Git, so the ability to read |
| 41 | from or write to a repository is controlled entirely by the host's |
| 42 | filesystem access controls. When multiple maintainers collaborate |
| 43 | on a single shared repository a high degree of trust is required, |
| 44 | as any collaborator with write access can alter the repository. |
| 45 | |
| 46 | Gitosis provides tools to secure centralized Git repositories, |
| 47 | permitting multiple maintainers to manage the same project at once, |
| 48 | by restricting the access to only over a secure network protocol, |
| 49 | much like Perforce secures a repository by only permitting access |
| 50 | over its network port. |
| 51 | |
| 52 | The Android Open Source Project (AOSP) was founded by Google by the |
| 53 | open source releasing of the Android operating system. AOSP has |
| 54 | selected Git as its primary version control tool. As many of the |
| 55 | engineers have a background of working with Mondrian at Google, |
| 56 | there is a strong desire to have the same (or better) feature set |
| 57 | available for Git and AOSP. |
| 58 | |
| Shawn O. Pearce | 4a7f6fa | 2009-02-17 17:14:56 -0800 | [diff] [blame] | 59 | Gerrit Code Review started as a simple set of patches to Rietveld, |
| 60 | and was originally built to service AOSP. This quickly turned |
| 61 | into a fork as we added access control features that Guido van |
| 62 | Rossum did not want to see complicating the Rietveld code base. As |
| 63 | the functionality and code were starting to become drastically |
| 64 | different, a different name was needed. Gerrit calls back to the |
| 65 | original namesake of Rietveld, Gerrit Rietveld, a Dutch architect. |
| 66 | |
| Shawn O. Pearce | e31d02c | 2009-12-08 12:21:37 -0800 | [diff] [blame] | 67 | Gerrit 2.x is a complete rewrite of the Gerrit fork, completely |
| 68 | changing the implementation from Python on Google App Engine, to Java |
| 69 | on a J2EE servlet container and a SQL database. |
| Shawn O. Pearce | 4a7f6fa | 2009-02-17 17:14:56 -0800 | [diff] [blame] | 70 | |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 71 | * link:http://video.google.com/videoplay?docid=-8502904076440714866[Mondrian Code Review On The Web] |
| 72 | * link:http://code.google.com/p/rietveld/[Rietveld - Code Review for Subversion] |
| 73 | * link:http://eagain.net/gitweb/?p=gitosis.git;a=blob;f=README.rst;hb=HEAD[Gitosis README] |
| 74 | * link:http://source.android.com/[Android Open Source Project] |
| 75 | |
| 76 | |
| 77 | Overview |
| 78 | -------- |
| 79 | |
| 80 | Developers create one or more changes on their local desktop system, |
| 81 | then upload them for review to Gerrit using the standard `git push` |
| 82 | command line program, or any GUI which can invoke `git push` on |
| 83 | behalf of the user. Authentication and data transfer are handled |
| 84 | through SSH. Users are authenticated by username and public/private |
| 85 | key pair, and all data transfer is protected by the SSH connection |
| 86 | and Git's own data integrity checks. |
| 87 | |
| 88 | Each Git commit created on the client desktop system is converted |
| 89 | into a unique change record which can be reviewed independently. |
| Chad Horohoe | e9855b8 | 2012-05-06 22:28:45 -0400 | [diff] [blame] | 90 | Change records are stored in a database: PostgreSQL, MySQL, or the |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 91 | built-in H2, where they can be queried to present customized user |
| 92 | dashboards, enumerating any pending changes. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 93 | |
| 94 | A summary of each newly uploaded change is automatically emailed |
| 95 | to reviewers, so they receive a direct hyperlink to review the |
| 96 | change on the web. Reviewer email addresses can be specified on the |
| 97 | `git push` command line, but typically reviewers are automatically |
| 98 | selected by Gerrit by identifying users who have change approval |
| 99 | permissions in the project. |
| 100 | |
| 101 | Reviewers use the web interface to read the side-by-side or unified |
| 102 | diff of a change, and insert draft inline comments where appropriate. |
| 103 | A draft comment is visible only to the reviewer, until they publish |
| 104 | those comments. Published comments are automatically emailed to |
| 105 | the change author by Gerrit, and are CC'd to all other reviewers |
| 106 | who have already commented on the change. |
| 107 | |
| 108 | When publishing comments reviewers are also given the opportunity |
| 109 | to score the change, indicating whether they feel the change is |
| 110 | ready for inclusion in the project, needs more work, or should be |
| 111 | rejected outright. These scores provide direct feedback to Gerrit's |
| 112 | change submit function. |
| 113 | |
| 114 | After a change has been scored positively by reviewers, Gerrit |
| 115 | enables a submit button on the web interface. Authorized users |
| 116 | can push the submit button to have the change enter the project |
| Edwin Kempin | f1acbb8 | 2011-09-15 12:49:42 +0200 | [diff] [blame] | 117 | repository. The equivalent in Subversion or Perforce would be |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 118 | that Gerrit is invoking `svn commit` or `p4 submit` on behalf of |
| 119 | the web user pressing the button. Due to the way Git audit trails |
| 120 | are maintained, the user pressing the submit button does not need |
| 121 | to be the author of the change. |
| 122 | |
| 123 | |
| 124 | Infrastructure |
| 125 | -------------- |
| 126 | |
| 127 | End-user web browsers make HTTP requests directly to Gerrit's |
| 128 | HTTP server. As nearly all of the user interface is implemented |
| 129 | through Google Web Toolkit (GWT), the majority of these requests |
| 130 | are transmitting compressed JSON payloads, with all HTML being |
| 131 | generated within the browser. Most responses are under 1 KB. |
| 132 | |
| 133 | Gerrit's HTTP server side component is implemented as a standard |
| 134 | Java servlet, and thus runs within any J2EE servlet container. |
| 135 | Popular choices for deployments would be Tomcat or Jetty, as these |
| 136 | are high-quality open-source servlet containers that are readily |
| 137 | available for download. |
| 138 | |
| 139 | End-user uploads are performed over SSH, so Gerrit's servlets also |
| 140 | start up a background thread to receive SSH connections through |
| 141 | an independent SSH port. SSH clients communicate directly with |
| 142 | this port, bypassing the HTTP server used by browsers. |
| 143 | |
| 144 | Server side data storage for Gerrit is broken down into two different |
| 145 | categories: |
| 146 | |
| 147 | * Git repository data |
| 148 | * Gerrit metadata |
| 149 | |
| 150 | The Git repository data is the Git object database used to store |
| 151 | already submitted revisions, as well as all uploaded (proposed) |
| 152 | changes. Gerrit uses the standard Git repository format, and |
| 153 | therefore requires direct filesystem access to the repositories. |
| 154 | All repository data is stored in the filesystem and accessed through |
| 155 | the JGit library. Repository data can be stored on remote servers |
| 156 | accessible through NFS or SMB, but the remote directory must |
| 157 | be mounted on the Gerrit server as part of the local filesystem |
| 158 | namespace. Remote filesystems are likely to perform worse than |
| 159 | local ones, due to Git disk IO behavior not being optimized for |
| 160 | remote access. |
| 161 | |
| 162 | The Gerrit metadata contains a summary of the available changes, |
| 163 | all comments (published and drafts), and individual user account |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 164 | information. The metadata is mostly housed in the database (*1), |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 165 | which can be located either on the same server as Gerrit, or on |
| 166 | a different (but nearby) server. Most installations would opt to |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 167 | install both Gerrit and the metadata database on the same server, |
| 168 | to reduce administration overheads. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 169 | |
| 170 | User authentication is handled by OpenID, and therefore Gerrit |
| 171 | requires that the OpenID provider selected by a user must be |
| 172 | online and operating in order to authenticate that user. |
| 173 | |
| 174 | * link:http://code.google.com/webtoolkit/[Google Web Toolkit (GWT)] |
| 175 | * link:http://www.kernel.org/pub/software/scm/git/docs/gitrepository-layout.html[Git Repository Format] |
| 176 | * link:http://www.postgresql.org/about/[About PostgreSQL] |
| 177 | * link:http://openid.net/developers/specs/[OpenID Specifications] |
| 178 | |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 179 | *1 Although an effort is underway to eliminate the use of the |
| 180 | database altogether, and to store all the metadata directly in |
| 181 | the git repositories themselves. So far, as of Gerrit 2.2.1, of |
| 182 | all Gerrit's metadata, only the project configuration metadata |
| 183 | has been migrated out of the database and into the git |
| 184 | repositories for each project. |
| 185 | |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 186 | |
| 187 | Project Information |
| 188 | ------------------- |
| 189 | |
| 190 | Gerrit is developed as a self-hosting open source project: |
| 191 | |
| 192 | * link:http://code.google.com/p/gerrit/[Project Homepage] |
| 193 | * link:http://code.google.com/p/gerrit/downloads/list[Release Versions] |
| Colby Ranger | bbc9de5 | 2012-04-26 09:01:10 -0700 | [diff] [blame] | 194 | * link:http://code.google.com/p/gerrit/source/checkout[Source] |
| Shawn O. Pearce | aa5b83b | 2009-12-02 08:10:24 -0800 | [diff] [blame] | 195 | * link:http://code.google.com/p/gerrit/issues/list[Issue Tracking] |
| 196 | * link:https://review.source.android.com/[Change Review] |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 197 | |
| 198 | |
| 199 | Internationalization and Localization |
| 200 | ------------------------------------- |
| 201 | |
| 202 | As a source code review system for open source projects, where the |
| 203 | commonly preferred language for communication is typically English, |
| 204 | Gerrit does not make internationalization or localization a priority. |
| 205 | |
| 206 | The majority of Gerrit's users will be writing change descriptions |
| 207 | and comments in English, and therefore an English user interface |
| 208 | is usable by the target user base. |
| 209 | |
| 210 | Gerrit uses GWT's i18n support to externalize all constant strings |
| 211 | and messages shown to the user, so that in the future someone who |
| 212 | really needed a translated version of the UI could contribute new |
| 213 | string files for their locale(s). |
| 214 | |
| 215 | Right-to-left (RTL) support is only barely considered within the |
| 216 | Gerrit code base. Some portions of the code have tried to take |
| 217 | RTL into consideration, while others probably need to be modified |
| 218 | before translating the UI to an RTL language. |
| 219 | |
| 220 | * link:i18n-readme.html[Gerrit's i18n Support] |
| 221 | |
| 222 | |
| 223 | Accessibility Considerations |
| 224 | ---------------------------- |
| 225 | |
| 226 | Whenever possible Gerrit displays raw text rather than image icons, |
| 227 | so screen readers should still be able to provide useful information |
| 228 | to blind persons accessing Gerrit sites. |
| 229 | |
| 230 | Standard HTML hyperlinks are used rather than HTML div or span tags |
| 231 | with click listeners. This provides two benefits to the end-user. |
| 232 | The first benefit is that screen readers are optimized to locating |
| 233 | standard hyperlink anchors and presenting them to the end-user as |
| 234 | a navigation action. The second benefit is that users can use |
| 235 | the 'open in new tab/window' feature of their browser whenever |
| 236 | they choose. |
| 237 | |
| 238 | When possible, Gerrit uses the ARIA properties on DOM widgets to |
| 239 | provide hints to screen readers. |
| 240 | |
| 241 | |
| 242 | Browser Compatibility |
| 243 | --------------------- |
| 244 | |
| 245 | Supporting non-JavaScript enabled browsers is a non-goal for Gerrit. |
| 246 | |
| 247 | As Gerrit is a pure-GWT application with no server side rendering |
| 248 | fallbacks, the browser must support modern JavaScript semantics in |
| 249 | order to access the Gerrit web application. Dumb clients such as |
| 250 | `lynx`, `wget`, `curl`, or even many search engine spiders are not |
| 251 | able to access Gerrit content. |
| 252 | |
| 253 | As Google Web Toolkit (GWT) is used to generate the browser |
| 254 | specific versions of the client-side JavaScript code, Gerrit works |
| 255 | on any JavaScript enabled browser which GWT can produce code for. |
| 256 | This covers the majority of the popular browsers. |
| 257 | |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 258 | The Gerrit project does not have the development resources necessary |
| 259 | to support two parallel UI implementations (GWT based JavaScript |
| 260 | and server-side rendering). Consequently only one is implemented. |
| 261 | |
| 262 | There are number of web browsers available with full JavaScript |
| 263 | support, and nearly every operating system (including any PDA-like |
| 264 | mobile phone) comes with one standard. Users who are committed |
| 265 | to developing changes for a Gerrit managed project can be expected |
| 266 | to be able to run a JavaScript enabled browser, as they also would |
| 267 | need to be running Git in order to contribute. |
| 268 | |
| 269 | There are a number of open source browsers available, including |
| 270 | Firefox and Chromium. Users have some degree of choice in their |
| 271 | browser selection, including being able to build and audit their |
| 272 | browser from source. |
| 273 | |
| 274 | The majority of the content stored within Gerrit is also available |
| 275 | through other means, such as gitweb or the `git://` protocol. |
| 276 | Any existing search engine spider can crawl the server-side HTML |
| 277 | produced by gitweb, and thus can index the majority of the changes |
| 278 | which might appear in Gerrit. Some engines may even choose to |
| 279 | crawl the native version control database, such as ohloh.net does. |
| 280 | Therefore the lack of support for most search engine spiders is a |
| 281 | non-issue for most Gerrit deployments. |
| 282 | |
| 283 | |
| 284 | Product Integration |
| 285 | ------------------- |
| 286 | |
| 287 | Gerrit integrates with an existing gitweb installation by optionally |
| 288 | creating hyperlinks to reference changes on the gitweb server. |
| 289 | |
| 290 | Gerrit integrates with an existing git-daemon installation by |
| 291 | optionally displaying `git://` URLs for users to download a |
| Shawn O. Pearce | d607846 | 2009-11-02 10:37:01 -0800 | [diff] [blame] | 292 | change through the native Git protocol. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 293 | |
| 294 | Gerrit integrates with any OpenID provider for user authentication, |
| 295 | making it easier for users to join a Gerrit site and manage their |
| 296 | authentication credentials to it. To make use of Google Accounts |
| 297 | as an OpenID provider easier, Gerrit has a shorthand "Sign in with |
| 298 | a Google Account" link on its sign-in screen. Gerrit also supports |
| 299 | a shorthand sign in link for Yahoo!. Other providers may also be |
| 300 | supported more directly in the future. |
| 301 | |
| Shawn O. Pearce | 142385d | 2009-03-01 11:09:05 -0800 | [diff] [blame] | 302 | Site administrators may limit the range of OpenID providers to |
| 303 | a subset of "reliable providers". Users may continue to use |
| 304 | any OpenID provider to publish comments, but granted privileges |
| 305 | are only available to a user if the only entry point to their |
| 306 | account is through the defined set of "reliable OpenID providers". |
| 307 | This permits site administrators to require HTTPS for OpenID, |
| 308 | and to use only large main-stream providers that are trustworthy, |
| 309 | or to require users to only use a custom OpenID provider installed |
| 310 | alongside Gerrit Code Review. |
| 311 | |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 312 | Gerrit integrates with some types of corporate single-sign-on (SSO) |
| 313 | solutions, typically by having the SSO authentication be performed |
| 314 | in a reverse proxy web server and then blindly trusting that all |
| 315 | incoming connections have been authenticated by that reverse proxy. |
| 316 | When configured to use this form of authentication, Gerrit does |
| 317 | not integrate with OpenID providers. |
| 318 | |
| 319 | When installing Gerrit, administrators may optionally include an |
| 320 | HTML header or footer snippet which may include user tracking code, |
| 321 | such as that used by Google Analytics. This is a per-instance |
| 322 | configuration that must be done by hand, and is not supported |
| 323 | out of the box. Other site trackers instead of Google Analytics |
| 324 | can be used, as the administrator can supply any HTML/JavaScript |
| 325 | they choose. |
| 326 | |
| 327 | Gerrit does not integrate with any Google service, or any other |
| 328 | services other than those listed above. |
| 329 | |
| 330 | |
| 331 | Standards / Developer APIs |
| 332 | -------------------------- |
| 333 | |
| 334 | Gerrit uses an XSRF protected variant of JSON-RPC 1.1 to communicate |
| 335 | between the browser client and the server. |
| 336 | |
| 337 | As the protocol is not the GWT-RPC protocol, but is instead a |
| 338 | self-describing standard JSON format it is easily implemented by |
| 339 | any 3rd party client application, provided the client has a JSON |
| 340 | parser and HTTP client library available. |
| 341 | |
| 342 | As the entire command set necessary for the standard web browser |
| 343 | based UI is exposed through JSON-RPC over HTTP, there are no other |
| 344 | data feeds or command interfaces to the server. |
| 345 | |
| 346 | Commands requiring user authentication may require the user agent to |
| 347 | complete a sign-in cycle through the user's OpenID provider in order |
| 348 | to establish the HTTP cookie Gerrit uses to track user identity. |
| 349 | Automating this sign-in process for non-web browser agents is |
| 350 | outside of the scope of Gerrit, as each OpenID provider uses its own |
| 351 | sign-in sequence. Use of OpenID providers which have difficult to |
| 352 | automate interfaces may make it impossible for non-browser agents |
| 353 | to be used with the JSON-RPC interface. |
| 354 | |
| 355 | * link:http://json-rpc.org/wd/JSON-RPC-1-1-WD-20060807.html[JSON-RPC 1.1] |
| Augie Fackler | ec2bb9e | 2011-10-27 13:26:15 -0500 | [diff] [blame] | 356 | * link:http://code.google.com/p/gerrit/source/browse/README?repo=gwtjsonrpc&name=master[XSRF JSON-RPC] |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 357 | |
| 358 | |
| 359 | Privacy Considerations |
| 360 | ---------------------- |
| 361 | |
| 362 | Gerrit stores the following information per user account: |
| 363 | |
| 364 | * Full Name |
| 365 | * Preferred Email Address |
| Shawn O. Pearce | aa8b3d4 | 2009-03-01 11:10:55 -0800 | [diff] [blame] | 366 | * Mailing Address '(Optional, Encrypted)' |
| 367 | * Country '(Optional, Encrypted)' |
| 368 | * Phone Number '(Optional, Encrypted)' |
| 369 | * Fax Number '(Optional, Encrypted)' |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 370 | |
| 371 | The full name and preferred email address fields are shown to any |
| 372 | site visitor viewing a page containing a change uploaded by the |
| 373 | account owner, or containing a published comment written by the |
| 374 | account owner. |
| 375 | |
| 376 | Showing the full name and preferred email is approximately the same |
| 377 | risk as the `From` header of an email posted to a public mailing |
| 378 | list that maintains archives, and Gerrit treats these fields in |
| 379 | much the same way that a mailing list archive might handle them. |
| 380 | Users who don't want to expose this information should either not |
| 381 | participate in a Gerrit based online community, or open a new email |
| 382 | address dedicated for this use. |
| 383 | |
| 384 | As the Gerrit UI data is only available through XSRF protected |
| 385 | JSON-RPC calls, "screen-scraping" for email addresses is difficult, |
| 386 | but not impossible. It is unlikely a spammer will go through the |
| 387 | effort required to code a custom scraping application necessary |
| 388 | to cull email addresses from published Gerrit comments. In most |
| 389 | cases these same addresses would be more easily obtained from the |
| 390 | project's mailing list archives. |
| 391 | |
| Shawn O. Pearce | aa8b3d4 | 2009-03-01 11:10:55 -0800 | [diff] [blame] | 392 | The user's name and email address is stored unencrypted in the |
| 393 | Gerrit metadata store, typically a PostgreSQL database. |
| 394 | |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 395 | The snail-mail mailing address, country, and phone and fax numbers |
| 396 | are gathered to help project leads contact the user should there |
| 397 | be a legal question regarding any change they have uploaded. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 398 | |
| Shawn O. Pearce | aa8b3d4 | 2009-03-01 11:10:55 -0800 | [diff] [blame] | 399 | These sensitive fields are immediately encrypted upon receipt with |
| 400 | a GnuPG public key, and stored "off site" in another data store, |
| 401 | isolated from the main Gerrit change data. Gerrit does not have |
| 402 | access to the matching private key, and as such cannot decrypt the |
| 403 | information. Therefore these fields are write-once in Gerrit, as not |
| 404 | even the account owner can recover the values they previously stored. |
| 405 | |
| 406 | It is expected that the address information would only need to be |
| 407 | decrypted and revealed with a valid court subpoena, but this is |
| 408 | really left to the discretion of the Gerrit site administrator as |
| 409 | to when it is reasonable to reveal this information to a 3rd party. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 410 | |
| 411 | |
| 412 | Spam and Abuse Considerations |
| 413 | ----------------------------- |
| 414 | |
| 415 | Gerrit makes no attempt to detect spam changes or comments. The |
| 416 | somewhat high barrier to entry makes it unlikely that a spammer |
| 417 | will target Gerrit. |
| 418 | |
| 419 | To upload a change, the client must speak the native Git protocol |
| 420 | embedded in SSH, with some custom Gerrit semantics added on top. |
| 421 | The client must have their public key already stored in the Gerrit |
| 422 | database, which can only be done through the XSRF protected |
| 423 | JSON-RPC interface. The level of effort required to construct |
| 424 | the necessary tools to upload a well-formatted change that isn't |
| 425 | rejected outright by the Git and Gerrit checksum validations is |
| 426 | too high to for a spammer to get any meaningful return. |
| 427 | |
| 428 | To post and publish a comment a client must sign in with an OpenID |
| 429 | provider and then use the XSRF protected JSON-RPC interface to |
| 430 | publish the draft on an existing change record. Again, the level of |
| 431 | effort required to implement the Gerrit specific XSRF protections |
| 432 | and the JSON-RPC payload format necessary to post a draft and then |
| 433 | publish that draft is simply too high for a spammer to bother with. |
| 434 | |
| 435 | Both of these assumptions are also based upon the idea that Gerrit |
| 436 | will be a lot less popular than blog software, and thus will be |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 437 | running on a lot fewer websites. Spammers therefore have very little |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 438 | returned benefit for getting over the protocol hurdles. |
| 439 | |
| 440 | These assumptions may need to be revisited in the future if any |
| 441 | public Gerrit site actually notices spam. |
| 442 | |
| 443 | |
| 444 | Latency |
| 445 | ------- |
| 446 | |
| 447 | Gerrit targets for sub-250 ms per page request, mostly by using |
| 448 | very compact JSON payloads bewteen client and server. However, as |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 449 | most of the serving stack (network, hardware, metadata |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 450 | database) is out of control of the Gerrit developers, no real |
| 451 | guarantees can be made about latency. |
| 452 | |
| 453 | |
| 454 | Scalability |
| 455 | ----------- |
| 456 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 457 | Gerrit is designed for a very large scale open source project, or |
| 458 | large commerical development project. Roughly this amounts to |
| 459 | parameters such as the following: |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 460 | |
| 461 | .Design Parameters |
| Karsten Dambekalns | a7f72a2 | 2011-03-25 14:21:59 +0100 | [diff] [blame] | 462 | [options="header"] |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 463 | |====================================================== |
| 464 | |Parameter | Default Maximum | Estimated Maximum |
| 465 | |Projects | 1,000 | 10,000 |
| 466 | |Contributors | 1,000 | 50,000 |
| 467 | |Changes/Day | 100 | 2,000 |
| 468 | |Revisions/Change | 20 | 20 |
| 469 | |Files/Change | 50 | 16,000 |
| 470 | |Comments/File | 100 | 100 |
| 471 | |Reviewers/Change | 8 | 8 |
| 472 | |====================================================== |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 473 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 474 | Out of the box, Gerrit will handle the "Default Maximum". Site |
| 475 | administrators may reconfigure their servers by editing gerrit.config |
| 476 | to run closer to the estimated maximum if sufficient memory is made |
| 477 | avaliable to the JVM and the relevant cache.*.memoryLimit variables |
| 478 | are increased from their defaults. |
| 479 | |
| 480 | Discussion |
| 481 | ~~~~~~~~~~ |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 482 | |
| 483 | Very few, if any open source projects have more than a handful of |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 484 | Git repositories associated with them. Since Gerrit treats each |
| 485 | Git repository as a project, an upper limit of 10,000 projects |
| 486 | is reasonable. If a site has more than 1,000 projects, administrators |
| 487 | should increase |
| 488 | link:config-gerrit.html#cache.name.memoryLimit[`cache.projects.memoryLimit`] |
| 489 | to match. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 490 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 491 | Almost no open source project has 1,000 contributors over all time, |
| 492 | let alone on a daily basis. This default figure of 1,000 was WAG'd by |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 493 | looking at PR statements published by cell phone companies picking |
| 494 | up the Android operating system. If all of the stated employees in |
| 495 | those PR statements were working on *only* the open source Android |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 496 | repositories, we might reach the 1,000 estimate listed here. Knowing |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 497 | these companies as being very closed-source minded in the past, it |
| 498 | is very unlikely all of their Android engineers will be working on |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 499 | the open source repository, and thus 1,000 is a very high estimate. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 500 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 501 | The upper maximum of 50,000 contributors is based on existing |
| 502 | installations that are already handling quite a bit more than the |
| 503 | default maximum of 1,000 contributors. Given how the user data is |
| 504 | stored and indexed, supporting 50,000 contributor accounts (or more) |
| 505 | is easily possible for a server. If a server has more than 1,000 |
| 506 | *active* contributors, |
| 507 | link:config-gerrit.html#cache.name.memoryLimit[`cache.accounts.memoryLimit`] |
| 508 | should be increased by the site administrator, if sufficient RAM |
| 509 | is available to the host JVM. |
| 510 | |
| 511 | The estimate of 100 changes per day was WAG'd off some estimates |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 512 | originally obtained from Android's development history. Writing a |
| 513 | good change that will be accepted through a peer-review process |
| 514 | takes time. The average engineer may need 4-6 hours per change just |
| 515 | to write the code and unit tests. Proper design consideration and |
| 516 | additional but equally important tasks such as meetings, interviews, |
| 517 | training, and eating lunch will often pad the engineer's day out |
| 518 | such that suitable changes are only posted once a day, or once |
| 519 | every other day. For reference, the entire Linux kernel has an |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 520 | average of only 79 changes/day. If more than 100 changes are active |
| 521 | per day, site administrators should consider increasing the |
| 522 | link:config-gerrit.html#cache.name.memoryLimit[`cache.diff.memoryLimit`] |
| 523 | and `cache.diff_intraline.memoryLimit`. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 524 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 525 | On average any given change will need to be modified once to address |
| 526 | peer review comments before the final revision can be accepted by the |
| 527 | project. Executing these revisions also eats into the contributor's |
| 528 | time, and is another factor limiting the number of changes/day |
| 529 | accepted by the Gerrit instance. However, even though this implies |
| 530 | only 2 revisions/change, many existing Gerrit installations have seen |
| 531 | 20 or more revisions/change, when new contributors are learning the |
| 532 | project's style and conventions. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 533 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 534 | On average, each change will have 2 reviewers, a human and an |
| 535 | automated test bed system. Usually this would be the project lead, or |
| 536 | someone who is familiar with the code being modified. The time |
| 537 | required to comment further reduces the time available for writing |
| 538 | one's own changes. However, existing Gerrit installations have seen 8 |
| 539 | or more reviewers frequently show up on changes that impact many |
| 540 | functional areas, and therefore it is reasonable to expect 8 or more |
| 541 | reviewers to be able to work together on a single change. |
| 542 | |
| 543 | Existing installations have successfully processed change reviews with |
| 544 | more than 16,000 files per change. However, since 16,000 modified/new |
| 545 | files is a massive amount of code to review, it is more typical to see |
| 546 | less than 10 files modified in any single change. Changes larger than |
| 547 | 10 files are typically merges, for example integrating the latest |
| 548 | version of an upstream library, where the reviewer has little to do |
| 549 | beyond verifying the project compiles and passes a test suite. |
| 550 | |
| 551 | CPU Usage - Web UI |
| 552 | ~~~~~~~~~~~~~~~~~~ |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 553 | |
| 554 | Gerrit's web UI would require on average `4+F+F*C` HTTP requests to |
| 555 | review a change and post comments. Here `F` is the number of files |
| 556 | modified by the change, and `C` is the number of inline comments left |
| 557 | by the reviewer per file. The constant 4 accounts for the request |
| 558 | to load the reviewer's dashboard, to load the change detail page, |
| 559 | to publish the review comments, and to reload the change detail |
| 560 | page after comments are published. |
| 561 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 562 | This WAG'd estimate boils down to 216,000 HTTP requests per day |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 563 | (QPD). Assuming these are evenly distributed over an 8 hour work day |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 564 | in a single time zone, we are looking at approximately 7.5 queries |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 565 | per second (QPS). |
| 566 | |
| 567 | ---- |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 568 | QPD = Changes_Day * Revisions_Change * Reviewers_Change * (4 + F + F * C) |
| 569 | = 2,000 * 2 * 1 * (4 + 10 + 10 * 4) |
| 570 | = 216,000 |
| Shawn O. Pearce | 57c4ba8 | 2009-12-22 08:10:50 -0800 | [diff] [blame] | 571 | QPS = QPD / 8_Hours / 60_Minutes / 60_Seconds |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 572 | = 7.5 |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 573 | ---- |
| 574 | |
| 575 | Gerrit serves most requests in under 60 ms when using the loopback |
| 576 | interface and a single processor. On a single CPU system there is |
| 577 | sufficient capacity for 16 QPS. A dual processor system should be |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 578 | more than sufficient for a site with the estimated load described above. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 579 | |
| 580 | Given a more realistic estimate of 79 changes per day (from the |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 581 | Linux kernel) suggests only 8,532 queries per day, and a much lower |
| 582 | 0.29 QPS when spread out over an 8 hour work day. |
| 583 | |
| 584 | CPU Usage - Git over SSH/HTTP |
| 585 | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ |
| 586 | |
| 587 | A 24 core server is able to handle ~25 concurrent `git fetch` |
| 588 | operations per second. The issue here is each concurrent operation |
| 589 | demands one full core, as the computation is almost entirely server |
| 590 | side CPU bound. 25 concurrent operations is known to be sufficient to |
| 591 | support hundreds of active developers and 50 automated build servers |
| 592 | polling for updates and building every change. (This data was derived |
| 593 | from an actual installation's performance.) |
| 594 | |
| 595 | Because of the distributed nature of Git, end-users don't need to |
| 596 | contact the central Gerrit Code Review server very often. For `git |
| 597 | fetch` traffic, link:pgm-daemon.html[slave mode] is known to be an |
| 598 | effective way to offload traffic from the main server, permitting it |
| 599 | to scale to a large user base without needing an excessive number of |
| 600 | cores in a single system. |
| 601 | |
| 602 | Clients on very slow network connections (for example home office |
| 603 | users on VPN over home DSL) may be network bound rather than server |
| 604 | side CPU bound, in which case a core may be effectively shared with |
| 605 | another user. Possible core sharing due to network bottlenecks |
| 606 | generally holds true for network connections running below 10 MiB/sec. |
| 607 | |
| 608 | If the server's own network interface is 1 Gib/sec (Gigabit Ethernet), |
| 609 | the system can really only serve about 10 concurrent clients at the |
| 610 | 10 MiB/sec speed, no matter how many cores it has. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 611 | |
| 612 | Disk Usage |
| 613 | ~~~~~~~~~~ |
| 614 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 615 | The average size of a revision in the Linux kernel once compressed by |
| 616 | Git is 2,327 bytes, or roughly 2 KiB. Over the course of a year a |
| 617 | Gerrit server running with the estimated maxium parameters above might |
| 618 | see an introduction of 1.4 GiB over the total set of 10,000 projects |
| 619 | hosted in that server. This figure assumes the majority of the content |
| 620 | is human written source code, and not large binary blobs such as disk |
| 621 | images or media files. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 622 | |
| Shawn O. Pearce | 0825581 | 2011-04-12 00:02:38 -0400 | [diff] [blame] | 623 | Production Gerrit installations have been tested, and are known to |
| 624 | handle Git repositories in the multigigabyte range, storing binary |
| 625 | files, ranging in size from a few kilobytes (for example compressed |
| 626 | icons) to 800+ megabytes (firmware images, large uncompressed original |
| 627 | artwork files). Best practices encourage breaking very large binary |
| 628 | files into their Git repositories based on access, to prevent desktop |
| 629 | clients from needing to clone unnecessary materials (for example a C |
| 630 | developer does not need every 800+ megabyte firmware image created by |
| 631 | the product's quality assurance team). |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 632 | |
| 633 | Redundancy & Reliability |
| 634 | ------------------------ |
| 635 | |
| 636 | Gerrit largely assumes that the local filesystem where Git repository |
| 637 | data is stored is always available. Important data written to disk |
| 638 | is also forced to the platter with an `fsync()` once it has been |
| 639 | fully written. If the local filesystem fails to respond to reads |
| 640 | or becomes corrupt, Gerrit has no provisions to fallback or retry |
| 641 | and errors will be returned to clients. |
| 642 | |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 643 | Gerrit largely assumes that the metadata database is online and |
| 644 | answering both read and write queries. Query failures immediately |
| 645 | result in the operation aborting and errors being returned to the |
| 646 | client, with no retry or fallback provisions. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 647 | |
| 648 | Due to the relatively small scale described above, it is very likely |
| Martin Fick | b026ca3 | 2011-07-27 18:23:20 -0600 | [diff] [blame] | 649 | that the Git filesystem and metadata database are all housed on the |
| 650 | same server that is running Gerrit. If any failure arises in one of |
| 651 | these components, it is likely to manifest in the others too. It is |
| 652 | also likely that the administrator cannot be bothered to deploy a |
| 653 | cluster of load-balanced server hardware, as the scale and expected |
| 654 | load does not justify the hardware or management costs. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 655 | |
| 656 | Most deployments caring about reliability will setup a warm-spare |
| 657 | standby system and use a manual fail-over process to switch from the |
| 658 | failed system to the warm-spare. |
| 659 | |
| 660 | As Git is a distributed version control system, and open source |
| 661 | projects tend to have contributors from all over the world, most |
| 662 | contributors will be able to tolerate a Gerrit down time of several |
| 663 | hours while the administrator is notified, signs on, and brings the |
| 664 | warm-spare up. Pending changes are likely to need at least 24 hours |
| 665 | of time on the Gerrit site anyway in order to ensure any interested |
| 666 | parties around the world have had a chance to comment. This expected |
| 667 | lag largely allows for some downtime in a disaster scenario. |
| 668 | |
| 669 | Backups |
| 670 | ~~~~~~~ |
| 671 | |
| Chad Horohoe | e9855b8 | 2012-05-06 22:28:45 -0400 | [diff] [blame] | 672 | PostgreSQL and MySQL can be configured to replicate their data to |
| 673 | other systems, where they are applied to a warm-standby backup in |
| 674 | real time. Gerrit instances which care about reduduncy will setup |
| 675 | this feature of PostgreSQL or MySQL to ensure the warm-standby is |
| 676 | reasonably current should the master go offline. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 677 | |
| Shawn O. Pearce | 7d2cb04 | 2012-05-10 19:12:09 -0700 | [diff] [blame^] | 678 | Using the standard replication plugin, Gerrit can be configured |
| 679 | to replicate changes made to the local Git repositories over any |
| 680 | standard Git transports. After the plugin is installed, remote |
| 681 | destinations can be configured in `'$site_path'/etc/replication.conf` |
| 682 | to send copies of all changes over SSH to other servers, or to the |
| 683 | Amazon S3 blob storage service. |
| Shawn O. Pearce | c4bcc09 | 2009-02-06 12:32:57 -0800 | [diff] [blame] | 684 | |
| 685 | |
| 686 | Logging Plan |
| 687 | ------------ |
| 688 | |
| 689 | Gerrit does not maintain logs on its own. |
| 690 | |
| 691 | Published comments contain a publication date, so users can judge |
| 692 | when the comment was posted and decide if it was "recent" or not. |
| 693 | Only the timestamp is stored in the database, the IP address of |
| 694 | the comment author is not stored. |
| 695 | |
| 696 | Changes uploaded over the SSH daemon from `git push` have the |
| 697 | standard Git reflog updated with the date and time that the upload |
| 698 | occurred, and the Gerrit account identity of who did the upload. |
| 699 | Changes submitted and merged into a branch also update the |
| 700 | Git reflog. These logs are available only to the Gerrit site |
| 701 | administrator, and they are not replicated through the automatic |
| 702 | replication noted earlier. These logs are primarly recorded for an |
| 703 | "oh s**t" moment where the administrator has to rewind data. In most |
| 704 | installations they are a waste of disk space. Future versions of |
| 705 | JGit may allow disabling these logs, and Gerrit may take advantage |
| 706 | of that feature to stop writing these logs. |
| 707 | |
| 708 | A web server positioned in front of Gerrit (such as a reverse proxy) |
| 709 | or the hosting servlet container may record access logs, and these |
| 710 | logs may be mined for usage information. This is outside of the |
| 711 | scope of Gerrit. |
| 712 | |
| 713 | |
| 714 | Testing Plan |
| 715 | ------------ |
| 716 | |
| 717 | Gerrit is currently manually tested through its web UI. |
| 718 | |
| 719 | JGit has a fairly extensive automated unit test suite. Most new |
| 720 | changes to JGit are rejected unless corresponding automated unit |
| 721 | tests are included. |
| 722 | |
| 723 | |
| 724 | Caveats |
| 725 | ------- |
| 726 | |
| 727 | Reitveld can't be used as it does not provide the "submit over the |
| 728 | web" feature that Gerrit provides for Git. |
| 729 | |
| 730 | Gitosis can't be used as it does not provide any code review |
| 731 | features, but it does provide basic access controls. |
| 732 | |
| 733 | Email based code review does not scale to a project as large and |
| 734 | complex as Android. Most contributors at least need some sort of |
| 735 | dashboard to keep track of any pending reviews, and some way to |
| 736 | correlate updated revisions back to the comments written on prior |
| 737 | revisions of the same logical change. |
| Shawn O. Pearce | 5500e69 | 2009-05-28 15:55:01 -0700 | [diff] [blame] | 738 | |
| 739 | GERRIT |
| 740 | ------ |
| 741 | Part of link:index.html[Gerrit Code Review] |