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