Skip to content

Commit bc5ea0c

Browse files
committed
clusterversion: remove obs TODO_Delete_V25_2 versions
Resolves: #147776 Release note: None
1 parent 2f72cf4 commit bc5ea0c

File tree

8 files changed

+3
-174
lines changed

8 files changed

+3
-174
lines changed

pkg/clusterversion/cockroach_versions.go

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -205,10 +205,6 @@ const (
205205

206206
TODO_Delete_V25_2_Start
207207

208-
TODO_Delete_V25_2_AddSqlActivityFlushJob
209-
210-
TODO_Delete_V25_2_SetUiDefaultTimezoneSetting
211-
212208
// V25_2 is CockroachDB v25.2. It's used for all v25.2.x patch releases.
213209
V25_2
214210

@@ -277,9 +273,7 @@ var versionTable = [numKeys]roachpb.Version{
277273
V25_1: {Major: 25, Minor: 1, Internal: 0},
278274

279275
// v25.2 versions. Internal versions must be even.
280-
TODO_Delete_V25_2_Start: {Major: 25, Minor: 1, Internal: 2},
281-
TODO_Delete_V25_2_AddSqlActivityFlushJob: {Major: 25, Minor: 1, Internal: 4},
282-
TODO_Delete_V25_2_SetUiDefaultTimezoneSetting: {Major: 25, Minor: 1, Internal: 6},
276+
TODO_Delete_V25_2_Start: {Major: 25, Minor: 1, Internal: 2},
283277

284278
V25_2: {Major: 25, Minor: 2, Internal: 0},
285279

pkg/ui/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ go_library(
2121
"//pkg/server/serverpb",
2222
"//pkg/settings",
2323
"//pkg/settings/cluster",
24+
"//pkg/ui/settings",
2425
"//pkg/util/httputil",
2526
"//pkg/util/log",
2627
"//pkg/util/timeutil",

pkg/ui/ui.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ import (
2525
"github.com/cockroachdb/cockroach/pkg/server/serverpb"
2626
"github.com/cockroachdb/cockroach/pkg/settings"
2727
"github.com/cockroachdb/cockroach/pkg/settings/cluster"
28+
_ "github.com/cockroachdb/cockroach/pkg/ui/settings" // Import the settings package to register UI-related settings for doc generation.
2829
"github.com/cockroachdb/cockroach/pkg/util/httputil"
2930
"github.com/cockroachdb/cockroach/pkg/util/log"
3031
"github.com/cockroachdb/cockroach/pkg/util/timeutil"

pkg/upgrade/upgrades/BUILD.bazel

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,6 @@ go_library(
1717
"schema_changes.go",
1818
"upgrades.go",
1919
"v25_1_prepared_transactions_table.go",
20-
"v25_2_add_sql_activity_flush_job.go",
21-
"v25_2_set_ui_default_timezone.go",
2220
"v25_3_add_event_log_column_and_index.go",
2321
"v25_3_add_hot_range_logger_job.go",
2422
"v25_3_add_users_last_login_time_column.go",
@@ -55,7 +53,6 @@ go_library(
5553
"//pkg/sql/sessiondata",
5654
"//pkg/sql/sqlstats/sqlactivityjob",
5755
"//pkg/sql/tablemetadatacache",
58-
"//pkg/ui/settings",
5956
"//pkg/upgrade",
6057
"//pkg/upgrade/upgradebase",
6158
"//pkg/util/envutil",
@@ -87,7 +84,6 @@ go_test(
8784
"schema_changes_helpers_test.go",
8885
"upgrades_test.go",
8986
"v25_1_prepared_transactions_table_test.go",
90-
"v25_2_set_ui_default_timezone_test.go",
9187
"v25_3_add_event_log_column_and_index_test.go",
9288
"v25_3_add_hot_range_logger_job_test.go",
9389
"v25_3_add_users_last_login_time_column_test.go",
@@ -135,7 +131,6 @@ go_test(
135131
"//pkg/testutils/skip",
136132
"//pkg/testutils/sqlutils",
137133
"//pkg/testutils/testcluster",
138-
"//pkg/ui/settings",
139134
"//pkg/upgrade",
140135
"//pkg/upgrade/upgradebase",
141136
"//pkg/util/ctxgroup",
@@ -148,6 +143,5 @@ go_test(
148143
"@com_github_cockroachdb_errors//:errors",
149144
"@com_github_stretchr_testify//assert",
150145
"@com_github_stretchr_testify//require",
151-
"@org_golang_x_exp//maps",
152146
],
153147
)

pkg/upgrade/upgrades/upgrades.go

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -71,22 +71,6 @@ var upgrades = []upgradebase.Upgrade{
7171

7272
newFirstUpgrade(clusterversion.TODO_Delete_V25_2_Start.Version()),
7373

74-
upgrade.NewTenantUpgrade(
75-
"add new sql activity flush job",
76-
clusterversion.TODO_Delete_V25_2_AddSqlActivityFlushJob.Version(),
77-
upgrade.NoPrecondition,
78-
addSqlActivityFlushJob,
79-
upgrade.RestoreActionNotRequired("cluster restore does not restore this job"),
80-
),
81-
82-
upgrade.NewTenantUpgrade(
83-
"set new ui.default_timezone setting to ui.display_timezone value",
84-
clusterversion.TODO_Delete_V25_2_SetUiDefaultTimezoneSetting.Version(),
85-
upgrade.NoPrecondition,
86-
setUiDefaultTimezone,
87-
upgrade.RestoreActionNotRequired("cluster restore does not restore this setting"),
88-
),
89-
9074
newFirstUpgrade(clusterversion.V25_3_Start.Version()),
9175

9276
upgrade.NewTenantUpgrade(

pkg/upgrade/upgrades/v25_2_add_sql_activity_flush_job.go

Lines changed: 0 additions & 23 deletions
This file was deleted.

pkg/upgrade/upgrades/v25_2_set_ui_default_timezone.go

Lines changed: 0 additions & 40 deletions
This file was deleted.

pkg/upgrade/upgrades/v25_2_set_ui_default_timezone_test.go

Lines changed: 0 additions & 82 deletions
This file was deleted.

0 commit comments

Comments
 (0)