Change Details

This task is to upgrade to 3.10.x series Release notes: https://www.gerritcodereview.com/3.10.html **Notes** Gerrit 3.10 drops support for Java 11, we have already switched to Java 17 as per T364342. H2 cache pruning is configurable! Filed as T367504 Native log deletions with [[ https://gerrit-review.googlesource.com/Documentation/config-gerrit.html | log.timeToKeep ]]. Filed as T367505 **Important notes** https://www.gerritcodereview.com/3.10.html#important-notes > If you’re using the `change_notes` cache, the file needs to be deleted. It will be automatically regenerate// ``` Name |Entries | AvgGet |Hit Ratio| | Mem Disk Space| |Mem Disk| --------------------------------+---------------------+---------+---------+ change_notes | 161 | 1.6ms | 55% | ``` We only have an in-memory version of it, nothing on disk. > The schema version hasn’t been updated, however, because Lucene was bumped 2 versions from 7.x to 9.8 the upgrade is supported only from Gerrit v3.9, as Lucene libraries do not support more than one version upgrade at a time. We are already running Gerrit 3.9 and can thus do the online upgrade/reindexing **Breaking changes** https://www.gerritcodereview.com/3.10.html#breaking-changes I haven't spotted anything concerning or ruled out the ones that could be suspicious such as: * //Deprecate review command without project argument//: Zuul has a single call to it and it has `--project %s` * //PluginCommandModule requires the plugin name in its constructor, hence all plugins that expose SSH commands need amending with an explicit constructor//. The only plugin I had in mind is zuul and it does not offers a PluginCommandModule.

This task is to upgrade to 3.10.x series Release notes: https://www.gerritcodereview.com/3.10.html **Notes** Gerrit 3.10 drops support for Java 11, we have already switched to Java 17 as per T364342. H2 cache pruning is configurable! Filed as T367504 Native log deletions with [[ https://gerrit-review.googlesource.com/Documentation/config-gerrit.html | log.timeToKeep ]]. Filed as T367505 **Important notes** https://www.gerritcodereview.com/3.10.html#important-notes > If you’re using the `change_notes` cache, the file needs to be deleted. It will be automatically regenerate// ``` Name |Entries | AvgGet |Hit Ratio| | Mem Disk Space| |Mem Disk| --------------------------------+---------------------+---------+---------+ change_notes | 161 | 1.6ms | 55% | ``` We only have an in-memory version of it, nothing on disk. > The schema version hasn’t been updated, however, because Lucene was bumped 2 versions from 7.x to 9.8 the upgrade is supported only from Gerrit v3.9, as Lucene libraries do not support more than one version upgrade at a time. We are already running Gerrit 3.9 and can thus do the online upgrade/reindexing **Breaking changes** https://www.gerritcodereview.com/3.10.html#breaking-changes I haven't spotted anything concerning or ruled out the ones that could be suspicious such as: > Deprecate review command without project argument Zuul has a single call to it and it has `--project %s` > PluginCommandModule requires the plugin name in its constructor, hence all plugins that expose SSH commands need amending with an explicit constructor The only plugin I had in mind is zuul and it does not offers a PluginCommandModule. LFS broke though with stable-3.10: [x] [[ https://gerrit-review.googlesource.com/c/plugins/lfs/+/430164/ | 430164 - Remove ambiguous Truth8.assertThat ]] [x] [[ https://gerrit-review.googlesource.com/c/plugins/lfs/+/430179/ | 430179 - Add plugin name to PluginCommandModule constructor ]]

This task is to upgrade to 3.10.x series Release notes: https://www.gerritcodereview.com/3.10.html **Notes** Gerrit 3.10 drops support for Java 11, we have already switched to Java 17 as per T364342. H2 cache pruning is configurable! Filed as T367504 Native log deletions with [[ https://gerrit-review.googlesource.com/Documentation/config-gerrit.html | log.timeToKeep ]]. Filed as T367505 **Important notes** https://www.gerritcodereview.com/3.10.html#important-notes > If you’re using the `change_notes` cache, the file needs to be deleted. It will be automatically regenerate// ``` Name |Entries | AvgGet |Hit Ratio| | Mem Disk Space| |Mem Disk| --------------------------------+---------------------+---------+---------+ change_notes | 161 | 1.6ms | 55% | ``` We only have an in-memory version of it, nothing on disk. > The schema version hasn’t been updated, however, because Lucene was bumped 2 versions from 7.x to 9.8 the upgrade is supported only from Gerrit v3.9, as Lucene libraries do not support more than one version upgrade at a time. We are already running Gerrit 3.9 and can thus do the online upgrade/reindexing **Breaking changes** https://www.gerritcodereview.com/3.10.html#breaking-changes I haven't spotted anything concerning or ruled out the ones that could be suspicious such as: > Deprecate review command without project argument Zuul has a single call to it and it has `--project %s` > PluginCommandModule requires the plugin name in its constructor, hence all plugins that expose SSH commands need amending with an explicit constructor The only plugin I had in mind is zuul and it does not offers a PluginCommandModule. * //Deprecate review command wiLFS broke thout project argument//: Zuul has a single call to it and it has `--project %s`gh with stable-3.10: * //PluginCommandModule requires the plugin name in its constructor, hence all plugins that expose SSH commands need amending with an explicit constructor//. The only plugin I had in mind is zuul and it does not offers a PluginCommandModule. [x] [[ https://gerrit-review.googlesource.com/c/plugins/lfs/+/430164/ | 430164 - Remove ambiguous Truth8.assertThat ]] [x] [[ https://gerrit-review.googlesource.com/c/plugins/lfs/+/430179/ | 430179 - Add plugin name to PluginCommandModule constructor ]]