Skip to content

Commit 5a9bf8a

Browse files
committed
feat: upgrade testing now starts in protocol version 10
- Configured the testnet to start in PV10 in node_upgrade_pytest.sh - Disabled tests not needed for PV10 in test_node_upgrade.py - Updated documentation for nightly system tests to reflect PV10
1 parent 3f1d418 commit 5a9bf8a

File tree

3 files changed

+5
-7
lines changed

3 files changed

+5
-7
lines changed

.github/node_upgrade_pytest.sh

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ if [ "$1" = "step1" ]; then
3333

3434
export UPGRADE_TESTS_STEP=1
3535
export ENABLE_LEGACY=1
36-
export PV9=1
3736

3837
if [ -n "${BASE_TAR_URL:-""}" ]; then
3938
# download and extract base revision binaries
@@ -107,7 +106,6 @@ elif [ "$1" = "step2" ]; then
107106

108107
export UPGRADE_TESTS_STEP=2
109108
export MIXED_P2P=1
110-
export PV9=1
111109
unset ENABLE_LEGACY
112110

113111
# Setup `cardano-cli` binary
@@ -233,7 +231,7 @@ elif [ "$1" = "step2" ]; then
233231
pytest cardano_node_tests/tests/test_node_upgrade.py -k test_ignore_log_errors
234232
err_retval="$?"
235233

236-
# Update PlutusV3 cost models.
234+
# Update Plutus cost models.
237235
pytest cardano_node_tests/tests/test_node_upgrade.py -k test_update_cost_models || exit 6
238236

239237
# run smoke tests
@@ -269,7 +267,7 @@ elif [ "$1" = "step3" ]; then
269267
printf "STEP3 start: %(%H:%M:%S)T\n" -1
270268

271269
export UPGRADE_TESTS_STEP=3
272-
unset ENABLE_LEGACY MIXED_P2P PV9
270+
unset ENABLE_LEGACY MIXED_P2P
273271

274272
# Setup `cardano-cli` binary
275273
if [ -n "${UPGRADE_CLI_REVISION:-""}" ]; then

cardano_node_tests/tests/test_node_upgrade.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ def test_ignore_log_errors(
9999
)
100100

101101
@allure.link(helpers.get_vcs_link())
102+
@pytest.mark.disabled(reason="The test is not needed when we are already in PV10 on mainnet")
102103
@pytest.mark.skipif(UPGRADE_TESTS_STEP != 2, reason="runs only on step 2 of upgrade testing")
103104
def test_update_cost_models(
104105
self,
@@ -196,6 +197,7 @@ def _check_models(cost_models: dict):
196197
_check_models(enact_gov_state["currentPParams"]["costModels"])
197198

198199
@allure.link(helpers.get_vcs_link())
200+
@pytest.mark.disabled(reason="The test is not needed when we are already in PV10 on mainnet")
199201
@pytest.mark.skipif(UPGRADE_TESTS_STEP != 3, reason="runs only on step 3 of upgrade testing")
200202
def test_hardfork(
201203
self,

src_docs/source/test_results/nightly_system_tests.rst

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,20 +47,18 @@ Nightly upgrade testing
4747
* `Step 1 <https://cardano-tests-reports-3-74-115-22.nip.io/cardano-node-tests-nightly-upgrade/step1/>`__: |nightly-upgrade-step1-badge|
4848
* use the `latest cardano-node release <https://github.com/IntersectMBO/cardano-node-tests/blob/master/.github/env_nightly_upgrade>`__ for Mainnet
4949
* network in Conway era
50-
* protocol version 9 (bootstrap phase)
50+
* protocol version 10
5151
* Constitutional Commitee has 5 members
5252
* default (legacy) network topology
5353
* smoke tests
5454
* governance info action test
5555
* `Step 2 <https://cardano-tests-reports-3-74-115-22.nip.io/cardano-node-tests-nightly-upgrade/step2/>`__: |nightly-upgrade-step2-badge|
5656
* upgrade all nodes except one to the latest cardano-node master
5757
* mixed network topology (half nodes P2P, half nodes legacy topology)
58-
* update PlutusV3 cost models
5958
* smoke tests
6059
* `Step 3 <https://cardano-tests-reports-3-74-115-22.nip.io/cardano-node-tests-nightly-upgrade/step3/>`__: |nightly-upgrade-step3-badge|
6160
* upgrade the last remaining node to latest cardano-node master
6261
* P2P network topology
63-
* hard fork to Conway protocol version 10
6462
* smoke tests
6563
* governance treasury withdrawal action test
6664

0 commit comments

Comments
 (0)