Skip to content

Commit

Permalink
Merge #64413
Browse files Browse the repository at this point in the history
64413: clusterversion: Move JoinTokensTable,AcquisitionTypeInLeaseHistory to 21.2 r=dt a=itsbilal

First commit is #64391.

This change moves the cluster versions JoinTokensTable and
AcquisitionTypeInLeaseHistory to sort after 21.1, as neither
of these changes will be backported to 21.1. The JoinTokensTable
migration is idempotent so it's okay if it runs twice on a
small set of nodes.

Release note: None.

Co-authored-by: Bilal Akhtar <bilal@cockroachlabs.com>
  • Loading branch information
craig[bot] and itsbilal committed May 1, 2021
2 parents 3fd6e98 + 5907930 commit 684e753
Show file tree
Hide file tree
Showing 4 changed files with 23 additions and 23 deletions.
2 changes: 1 addition & 1 deletion docs/generated/settings/settings-for-tenants.txt
Original file line number Diff line number Diff line change
Expand Up @@ -105,4 +105,4 @@ trace.datadog.project string CockroachDB the project under which traces will be
trace.debug.enable boolean false if set, traces for recent requests can be seen at https://<ui>/debug/requests
trace.lightstep.token string if set, traces go to Lightstep using this token
trace.zipkin.collector string if set, traces go to the given Zipkin instance (example: '127.0.0.1:9411'). Only one tracer can be configured at a time.
version version 21.1-2 set the active cluster version in the format '<major>.<minor>'
version version 21.1-6 set the active cluster version in the format '<major>.<minor>'
2 changes: 1 addition & 1 deletion docs/generated/settings/settings.html
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,6 @@
<tr><td><code>trace.debug.enable</code></td><td>boolean</td><td><code>false</code></td><td>if set, traces for recent requests can be seen at https://<ui>/debug/requests</td></tr>
<tr><td><code>trace.lightstep.token</code></td><td>string</td><td><code></code></td><td>if set, traces go to Lightstep using this token</td></tr>
<tr><td><code>trace.zipkin.collector</code></td><td>string</td><td><code></code></td><td>if set, traces go to the given Zipkin instance (example: '127.0.0.1:9411'). Only one tracer can be configured at a time.</td></tr>
<tr><td><code>version</code></td><td>version</td><td><code>21.1-2</code></td><td>set the active cluster version in the format '<major>.<minor>'</td></tr>
<tr><td><code>version</code></td><td>version</td><td><code>21.1-6</code></td><td>set the active cluster version in the format '<major>.<minor>'</td></tr>
</tbody>
</table>
30 changes: 15 additions & 15 deletions pkg/clusterversion/cockroach_versions.go
Original file line number Diff line number Diff line change
Expand Up @@ -317,20 +317,20 @@ const (
// ProtectedTsMetaPrivilegesMigration is for the migration which fixes the
// privileges of the protected_ts_meta system table.
ProtectedTsMetaPrivilegesMigration
// JoinTokensTable adds the system table for storing ephemeral generated
// join tokens.
JoinTokensTable
// AcquisitionTypeInLeaseHistory augments the per-replica lease history to
// include the type of lease acquisition event that resulted in that replica's
// current lease.
AcquisitionTypeInLeaseHistory
// V21_1 is CockroachDB v21.1. It's used for all v21.1.x patch releases.
V21_1

// v21.2 versions.
//
// Start21_2 demarcates work towards CockroachDB v21.2.
Start21_2
// JoinTokensTable adds the system table for storing ephemeral generated
// join tokens.
JoinTokensTable
// AcquisitionTypeInLeaseHistory augments the per-replica lease history to
// include the type of lease acquisition event that resulted in that replica's
// current lease.
AcquisitionTypeInLeaseHistory

// Step (1): Add new versions here.
)
Expand Down Expand Up @@ -533,14 +533,6 @@ var versionsSingleton = keyedVersions([]keyedVersion{
Key: ProtectedTsMetaPrivilegesMigration,
Version: roachpb.Version{Major: 20, Minor: 2, Internal: 48},
},
{
Key: JoinTokensTable,
Version: roachpb.Version{Major: 20, Minor: 2, Internal: 50},
},
{
Key: AcquisitionTypeInLeaseHistory,
Version: roachpb.Version{Major: 20, Minor: 2, Internal: 52},
},
{
// V21_1 is CockroachDB v21.1. It's used for all v21.1.x patch releases.
Key: V21_1,
Expand All @@ -552,6 +544,14 @@ var versionsSingleton = keyedVersions([]keyedVersion{
Key: Start21_2,
Version: roachpb.Version{Major: 21, Minor: 1, Internal: 2},
},
{
Key: JoinTokensTable,
Version: roachpb.Version{Major: 21, Minor: 1, Internal: 4},
},
{
Key: AcquisitionTypeInLeaseHistory,
Version: roachpb.Version{Major: 21, Minor: 1, Internal: 6},
},

// Step (2): Add new versions here.
})
Expand Down
12 changes: 6 additions & 6 deletions pkg/clusterversion/key_string.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 684e753

Please sign in to comment.