Skip to content

Commit 1b6b1cf

Browse files
nammnfealebenpae
authored andcommitted
om7 - only run upgrade test for major version changes (#4300)
# Summary - we have been testing updating between patch versions, but the test was there to test major version upgrades. - Since we've removed 5.x support, we don't need that test for 6.x ## Proof of Work - all upgrade tests will be green ## Checklist - [ ] Have you linked a jira ticket and/or is the ticket in the title? - [ ] Have you checked whether your jira ticket required DOCSP changes? - [ ] Have you checked for release_note changes? ## Reminder (Please remove this when merging) - Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible - Our Short Guide for PRs: [Link](REDACTED) - Remember the following Communication Standards - use comment prefixes for clarity: * **blocking**: Must be addressed before approval. * **follow-up**: Can be addressed in a later PR or ticket. * **q**: Clarifying question. * **nit**: Non-blocking suggestions. * **note**: Side-note, non-actionable. Example: Praise * --> no prefix is considered a question
1 parent 3cd35b2 commit 1b6b1cf

14 files changed

+30
-20
lines changed

.evergreen.yml

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1002,7 +1002,6 @@ task_groups:
10021002
- e2e_om_ops_manager_https_enabled_internet_mode
10031003
- e2e_om_ops_manager_https_enabled_prefix
10041004
- e2e_om_ops_manager_scale
1005-
- e2e_om_ops_manager_upgrade
10061005
- e2e_om_validation_webhook
10071006
- e2e_om_ops_manager_secure_config
10081007
- e2e_om_update_before_reconciliation
@@ -1039,7 +1038,6 @@ task_groups:
10391038
- e2e_om_ops_manager_backup_kmip
10401039
- e2e_om_ops_manager_pod_spec
10411040
- e2e_om_ops_manager_scale
1042-
- e2e_om_ops_manager_upgrade
10431041
- e2e_om_validation_webhook
10441042
- e2e_om_ops_manager_secure_config
10451043
- e2e_om_update_before_reconciliation
@@ -1084,6 +1082,15 @@ task_groups:
10841082
- e2e_om_ops_manager_prometheus
10851083
<<: *teardown_group
10861084

1085+
# Tests features only supported on OM70 and OM80, its only upgrade test as we test upgrading from 6 to 7 or 7 to 8
1086+
- name: e2e_ops_manager_upgrade_only_task_group
1087+
max_hosts: -1
1088+
<<: *setup_group
1089+
<<: *setup_and_teardown_task
1090+
tasks:
1091+
- e2e_om_ops_manager_upgrade
1092+
<<: *teardown_group
1093+
10871094
# Tests features only supported on OM60
10881095
- name: e2e_static_ops_manager_kind_6_0_only_task_group
10891096
max_hosts: -1
@@ -1253,6 +1260,7 @@ buildvariants:
12531260
- name: e2e_ops_manager_kind_only_task_group
12541261
- name: e2e_ops_manager_kind_5_0_only_task_group
12551262
- name: e2e_ops_manager_kind_6_0_only_task_group
1263+
- name: e2e_ops_manager_upgrade_only_task_group
12561264

12571265
- name: e2e_static_om70_kind_ubi
12581266
display_name: e2e_static_om70_kind_ubi
@@ -1263,6 +1271,7 @@ buildvariants:
12631271
tasks:
12641272
- name: e2e_static_ops_manager_kind_only_task_group
12651273
- name: e2e_static_ops_manager_kind_6_0_only_task_group
1274+
- name: e2e_ops_manager_upgrade_only_task_group
12661275

12671276
- name: e2e_om80_kind_ubi
12681277
display_name: e2e_om80_kind_ubi
@@ -1274,6 +1283,7 @@ buildvariants:
12741283
- name: e2e_ops_manager_kind_only_task_group
12751284
- name: e2e_ops_manager_kind_5_0_only_task_group
12761285
- name: e2e_ops_manager_kind_6_0_only_task_group
1286+
- name: e2e_ops_manager_upgrade_only_task_group
12771287

12781288
- name: e2e_static_om80_kind_ubi
12791289
display_name: e2e_static_om80_kind_ubi
@@ -1284,6 +1294,7 @@ buildvariants:
12841294
tasks:
12851295
- name: e2e_static_ops_manager_kind_only_task_group
12861296
- name: e2e_static_ops_manager_kind_6_0_only_task_group
1297+
- name: e2e_ops_manager_upgrade_only_task_group
12871298

12881299
- name: e2e_operator_race_ubi_with_telemetry
12891300
display_name: e2e_operator_race_ubi_with_telemetry

scripts/dev/contexts/e2e_kind_olm_ubi

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=kind

scripts/dev/contexts/e2e_multi_cluster_kind

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

scripts/dev/contexts/e2e_multi_cluster_om_appdb

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,12 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
10+
11+
# TODO Remove this once the startup script for OM can handle skipping preflight checks.
12+
# As it stands now, OM 7.0.13 will fail the preflight checks in a disaster recovery scenario.
13+
# https://jira.mongodb.org/browse/CLOUDP-297377
14+
export CUSTOM_OM_VERSION=7.0.12
1015

1116
export KUBE_ENVIRONMENT_NAME=multi
1217
export CLUSTER_NAME="kind-e2e-cluster-1"

scripts/dev/contexts/e2e_multi_cluster_om_operator_not_in_mesh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

scripts/dev/contexts/e2e_operator_no_webhook_roles_cloudqa

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export ops_manager_version="cloud_qa"
1212

scripts/dev/contexts/e2e_operator_race_ubi_with_telemetry

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

scripts/dev/contexts/e2e_smoke

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export DATABASE_REGISTRY="${QUAY_REGISTRY}"
1212
export APPDB_REGISTRY="${QUAY_REGISTRY}"

scripts/dev/contexts/e2e_static_multi_cluster_kind

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

scripts/dev/contexts/e2e_static_multi_cluster_om_appdb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,6 @@ script_dir=$(dirname "${script_name}")
88
source "${script_dir}/root-context"
99
source "${script_dir}/variables/om70"
1010

11-
# The earliest version supporting Static Containers for OM
12-
# We started supporting Static Containers in 6.0.21, except for OM
13-
# This CUSTOM version is required for upgrade tests to work
14-
# shellcheck disable=SC2034
15-
export CUSTOM_OM_PREV_VERSION=6.0.27
16-
1711
# TODO Remove this once the startup script for OM can handle skipping preflight checks.
1812
# As it stands now, OM 7.0.13 will fail the preflight checks in a disaster recovery scenario.
1913
# https://jira.mongodb.org/browse/CLOUDP-297377

scripts/dev/contexts/e2e_static_operator_kind_ubi_cloudqa

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export ops_manager_version="cloud_qa"
1212
export MDB_DEFAULT_ARCHITECTURE=static

scripts/dev/contexts/e2e_static_smoke

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export DATABASE_REGISTRY="${QUAY_REGISTRY}"
1212
export APPDB_REGISTRY="${QUAY_REGISTRY}"

scripts/dev/contexts/manual_telemetry_multi_cluster_dev

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

scripts/dev/contexts/manual_telemetry_multi_cluster_prod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ script_name=$(readlink -f "${BASH_SOURCE[0]}")
66
script_dir=$(dirname "${script_name}")
77

88
source "${script_dir}/root-context"
9-
source "${script_dir}/variables/om60"
9+
source "${script_dir}/variables/om70"
1010

1111
export KUBE_ENVIRONMENT_NAME=multi
1212
export CLUSTER_NAME="kind-e2e-operator"

0 commit comments

Comments
 (0)