Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 1 | = Release notes for Gerrit 2.1.3 |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 2 | |
| 3 | Gerrit 2.1.3 is now available in the usual location: |
| 4 | |
Shawn Pearce | 6d7ebc6 | 2015-06-12 16:34:42 -0700 | [diff] [blame] | 5 | link:https://www.gerritcodereview.com/download/index.html[https://www.gerritcodereview.com/download/index.html] |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 6 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 7 | == Schema Change |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 8 | |
| 9 | *WARNING* This release contains multiple schema changes. To upgrade: |
| 10 | ---- |
| 11 | java -jar gerrit.war init -d site_path |
| 12 | ---- |
| 13 | |
| 14 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 15 | == New Features |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 16 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 17 | === Web UI |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 18 | |
| 19 | * issue 289 Remove reviewers (or self) from a change |
| 20 | + |
| 21 | Project and change owners can now remove any reviewer from a change |
| 22 | by clicking an "X" next to their name in the approval table. |
| 23 | Individual users can also remove themselves from any change. |
| 24 | This feature permits users to stop getting notified about a change |
| 25 | they no longer have an interest in, but had commented on previously. |
| 26 | |
| 27 | * issue 124 Index changes by external issue tracking id numbers |
| 28 | + |
| 29 | Changes can be searched for by an external issue tracking system's |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 30 | id numbers. Site administrators can configure trackingid sections in |
| 31 | gerrit.config to parse and extract issue tracking links from a commit |
| 32 | message's footer, and have them indexed by Gerrit. Users can search |
| 33 | for relevant changes using the search operator `tr:` or `bug:`, |
| 34 | for example `tr:432181` or `bug:JIRA-42`. Administrators can index |
| 35 | existing change records using the ScanTrackingIds program. |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 36 | |
| 37 | * List branches/tags containing a merged change |
| 38 | + |
| 39 | Merged change pages now display a new expandable section, 'Included |
| 40 | In', listing all branches and tags that contain the change. |
| 41 | |
| 42 | * issue 391 Reduce clicks need to approve and submit |
| 43 | + |
| 44 | Users who have Submit +1 permission for a change can now click |
| 45 | 'Publish Comments and Submit' on the publish comments screen, |
| 46 | combining the 'Publish Comments' and 'Submit Patch Set n' actions |
| 47 | into a single click. |
| 48 | |
| 49 | * Simplify setup of non-range access such as Submit |
| 50 | + |
| 51 | If an access control doesn't really make sense as a range of values, |
| 52 | Gerrit now displays only one box to select the maximum permitted |
| 53 | value from, rather than two boxes to set the min/max. |
| 54 | |
| 55 | * Make Admin > Projects UI accessible to all users |
| 56 | + |
| 57 | All projects that are visible to the current user are now listed |
| 58 | in the Admin > Projects page, as are the project's Branches and |
| 59 | Access tabs. Editing is obviously disabled, unless the user has |
| 60 | owner level access to the project, or one of its branches. |
| 61 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 62 | === Access Controls |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 63 | |
| 64 | * Branch-level read access is now supported |
| 65 | + |
| 66 | Project owners/administrators can now use the access tab to |
| 67 | control which groups can read certain branches, enabling hidden |
| 68 | branches within a more widely visible project. Additionally, |
| 69 | replication.config honors these settings through the authGroup |
| 70 | variable, allowing a server administrator to limit which branches |
| 71 | are replicated to certain mirrors. |
| 72 | |
| 73 | * issue 273 Inherit project permissions from more than just All Projects |
| 74 | + |
| 75 | Projects can now be organized into an inheritance hierarchy, allowing |
| 76 | administrators to cluster common access rules for different groups |
| 77 | of projects. The create-project command learned a new \--parent |
| 78 | option to set the hierarchy immediately. |
| 79 | |
| 80 | * auth.allowedOpenID can limit which providers can be used |
| 81 | + |
| 82 | Administrators can now set auth.allowedOpenID in gerrit.config |
| 83 | to restrict which OpenID provider(s) a user can use to register |
| 84 | for an account. This may be useful to restrict login to only the |
| 85 | organization's local provider, or a single trusted 3rd party. |
| 86 | |
| 87 | * Branch-level access control is now inherited by default |
| 88 | + |
| 89 | Previously branch level access controls were exclusive, locking out |
| 90 | all other groups that may have been inherited from All Projects, |
| 91 | or through a wildcard like 'refs/heads/*'. Branch access is now |
| 92 | inherited by default, but the old exclusive behavior can be obtained |
| 93 | by prefixing the reference with '-'. |
| 94 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 95 | === SSH Commands |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 96 | |
| 97 | * create-account: Permit creation of batch user accounts over SSH |
| 98 | * issue 269 Enable create-project for non-Administrators |
| 99 | |
| 100 | * ls-projects: New -b option displays the sha1 of each branch |
| 101 | * ls-projects: New -t option shows the project hierarchy |
| 102 | |
| 103 | * gerrit show-queue is now accessible to all users |
| 104 | + |
| 105 | Results are filtered to display only queue entries that are operating |
| 106 | on projects the user is permitted to see. Replication URLs are |
| 107 | masked for non-admin users, and instead display the remote name |
| 108 | from the replication.config file. |
| 109 | |
| 110 | * issue 310 review \--submit: Submit a change over SSH |
| 111 | + |
| 112 | Changes can now be submitted over SSH by using the new \--submit |
| 113 | command line flag to gerrit review. |
| 114 | |
| 115 | * gerrit approve deprecated |
| 116 | + |
| 117 | To support the new \--submit flag, gerrit approve has been renamed |
| 118 | to gerrit review, better matching the web UI name for the concept. |
Shawn O. Pearce | 4a5afa6 | 2010-06-16 10:12:45 -0700 | [diff] [blame] | 119 | The old `gerrit approve` name will be kept around as an alias to |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 120 | provide time to migrate hooks/scripts/etc. |
| 121 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 122 | === Hooks / Stream Events |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 123 | |
| 124 | * \--change-url parameter passed to hooks |
| 125 | + |
| 126 | The change URL was supplied in the stream-events feed, but was |
| 127 | not passed into hooks, making it difficult for a hook to send a |
| 128 | notification email with a link back to Gerrit. Fixed by adding |
| 129 | the parameter. |
| 130 | |
| 131 | * Patch set uploader passed to hooks |
| 132 | + |
| 133 | The identity of the user who uploaded a patch set was added as both |
| 134 | a parameter to patchset-created hook, and to the patch set entity |
| 135 | sent through stream-events. |
| 136 | |
| 137 | * issue 506 stream-events: Include the ref in patch sets |
| 138 | + |
| 139 | The reference (e.g. 'refs/changes/12/812/2') to download a patch |
| 140 | set is now included in the stream-events record, making it possible |
| 141 | for a monitor to easily pull down a patch set and compile it. |
| 142 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 143 | === Contrib |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 144 | |
| 145 | * Example hook to auto-re-approve a trivial rebase |
| 146 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 147 | === Misc. |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 148 | |
| 149 | * transfer.timeout: Support configurable timeouts for dead clients |
| 150 | + |
| 151 | Sometimes `repo sync` can leave dead connections open to Gerrit Code |
| 152 | Review, resulting in worker threads that are tied up indefinitely, |
| 153 | waiting for client IO that will never occur. Administrators may set |
| 154 | transfer.timeout to place an upper bound on how long the server will |
| 155 | wait for the client before aborting the connection and releasing |
| 156 | the worker thread back into the pool. |
| 157 | |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 158 | * container.slave: Automatically enable --slave |
| 159 | + |
| 160 | Adminstrators can now add `container.slave = true` to their slave's |
| 161 | gerrit.config file, avoiding the need to make sure they always |
| 162 | pass the --slave flag on the command line when starting their |
| 163 | slave server. |
| 164 | |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 165 | * Add separate task queue for non-interactive users |
| 166 | + |
| 167 | Users who are a member of the special 'Non Interactive Users' group |
| 168 | can now have all of their SSH commands scheduled onto a different |
| 169 | thread pool than everyone else. If enabled, this feature can help |
| 170 | ensure quick response time for normal users when the system is |
| 171 | heavily loaded by batch tasks. |
| 172 | |
| 173 | * Explain a remote rejection of a non-fast-forward |
| 174 | + |
| 175 | If the remote peer rejected a non-fast-forward replication, make |
| 176 | it clear that it was the remote that rejected the push, and not |
| 177 | Gerrit Code Review's client logic. The error is often caused by |
| 178 | the remote repository having receive.denyNonFastForwards being set |
| 179 | to true in $GIT_DIR/config. Gerrit's error log message now hints |
| 180 | at checking this setting on the remote repository. |
| 181 | |
| 182 | * Internal dependencies updated |
| 183 | + |
Shawn O. Pearce | cc81809 | 2010-06-16 16:28:10 -0700 | [diff] [blame] | 184 | Updated JGit to 0.8.4, Jetty to 7.0.2.v20100331, H2 database to |
| 185 | 1.2.134, Apache Commons Codec to 1.4, Apache Commons Net to 2.1, |
| 186 | Apache Commons DBCP to 1.4. |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 187 | |
| 188 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 189 | == Bug Fixes |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 190 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 191 | === Web UI |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 192 | |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 193 | * issue 396 Prevent 'no-score' approvals from being recorded |
| 194 | + |
| 195 | Change messages no longer say 'No score; no score' when the user |
| 196 | has not selected a particular approval setting. |
| 197 | |
| 198 | * issue 396 Summarize the number of inline comments |
| 199 | + |
| 200 | A change message is now always recorded at the top level of a change |
| 201 | anytime inline comments are published, even if no score change |
| 202 | took place, and no cover letter was supplied by the user. The |
| 203 | auto-generated message is a one line summary indicating how many |
| 204 | inline comments were published at that time. This makes it easier |
| 205 | to see what has occurred on the change. |
| 206 | |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 207 | * issue 461 Space out Review and Submit Patch Set buttons |
| 208 | + |
| 209 | The risk of clicking 'Submit Patch Set n' when the user meant to |
| 210 | click 'Review' has been reduced by spacing the buttons further apart. |
| 211 | |
| 212 | * issue 587 Fix user site header/footer preference |
| 213 | + |
| 214 | The user preference to hide the site header/footer wasn't always |
| 215 | being applied. Fixed. |
| 216 | |
| 217 | * issue 575 Require branches to always start from commits |
| 218 | + |
| 219 | Branches could be created starting from annotated tags, resulting |
| 220 | in crashes when a change gets submitted to the branch. Fixed by |
| 221 | ensuring branches always start from commits. |
| 222 | |
| 223 | * issue 574 Add Cancel button to Register New Email dialog |
| 224 | + |
| 225 | Users couldn't (easily) get out of the dialog popped up by the |
| 226 | 'Register New Email...' button. A cancel button was added to |
| 227 | close the dialog. |
| 228 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 229 | === Server Programs |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 230 | |
| 231 | * init: Import non-standardly named Git repositories |
| 232 | + |
| 233 | When scanning for projects, any directory that is a valid Git |
| 234 | repository is now imported, even if its name does not end with |
| 235 | the standard '.git' suffix. |
| 236 | |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 237 | * issue 460 gerrit.sh: Request at least 1024 file descriptors |
| 238 | + |
| 239 | In the default configuration, Gerrit Code Review started with a |
| 240 | hard limit of 256 file descriptors, which is too small for any site. |
| 241 | This caused a number of failures, and a number of bugs were filed. |
| 242 | The default has been raised to 1024. |
| 243 | |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 244 | * issue 578 Improve schema version update by avoiding early pruning |
| 245 | + |
| 246 | Previously init kept trying to remove unused tables or columns |
| 247 | during each schema upgrade step. These removes are now deferred |
| 248 | until the last step. |
| 249 | |
| 250 | * review: Actually log an internal server error's root cause |
| 251 | + |
| 252 | Internal server failures (such as database connectivity errors) |
| 253 | were not properly logged by `gerrit approve` (now gerrit review). |
| 254 | Fixed by logging the root cause of the failure. |
| 255 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 256 | === Configuration |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 257 | |
| 258 | * Display error when HTTP authentication isn't configured |
| 259 | + |
| 260 | Error reporting for a failed login attempt when auth.type is HTTP |
| 261 | and the HTTP server isn't supplying the expected header is now more |
| 262 | explicit about describing the problem. This helps new site setups, |
| 263 | but doesn't have any impact on an existing site. |
| 264 | |
| 265 | * Fix javax.naming.PartialResultException: Unprocessed Continuation |
| 266 | + |
| 267 | LDAP directory trees that require following a referral in order |
| 268 | to lookup a name usually failed with the above Java exception |
| 269 | during sign-in. Administrators can enable following by adding |
| 270 | `ldap.referral = follow` to their gerrit.config file. |
| 271 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 272 | === Documentation |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 273 | |
| 274 | * documentation: Clarified the ownership of '\-- All Projects \--' |
| 275 | + |
| 276 | The magic project All Projects isn't allowed to have ownership |
| 277 | delegated, and the documentation wasn't clear why. Fixed by |
| 278 | explaining the rationale in more detail. |
| 279 | |
| 280 | * issue 533 Fix JAR versions in other container installation |
| 281 | + |
| 282 | The installation process for putting Gerrit Code Review under a |
| 283 | 3rd party servlet container was out of date, as some JARs had |
| 284 | the wrong versions listed. Fixed. |
| 285 | |
| 286 | * suexec: Document the suexec command |
| 287 | + |
| 288 | The suexec command introduced in 2.1.2 was never documented. Fixed. |
| 289 | |
| 290 | * Corrected Eclipse documentation on importing Maven projects |
| 291 | + |
| 292 | The Maven plugin changed some of its user interface, resulting in |
| 293 | our step-by-step documentation being out of date. Fixed to match |
| 294 | the current stable version of the Maven plugin. |
| 295 | |
| 296 | |
Yuxuan 'fishy' Wang | 4f5ad9d | 2016-05-03 16:18:58 -0700 | [diff] [blame] | 297 | == Version |
Shawn O. Pearce | c6b9bae | 2010-06-15 19:45:18 -0700 | [diff] [blame] | 298 | |
Shawn O. Pearce | 74b40b2 | 2010-06-17 13:02:10 -0700 | [diff] [blame] | 299 | e8fd49f5f7481e2f916cb0d8cfbada79309562b4 |