Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

add DB guard check for nodeos_under_min_avail_ram.py #7291

Merged
merged 7 commits into from
May 9, 2019

Conversation

taokayan
Copy link
Contributor

@taokayan taokayan commented May 7, 2019

Change Description

Added DB guard check for existing test nodeos_under_min_avail_ram.py #7214

Consensus Changes

N/A

API Changes

N/A

Documentation Additions

N/A

@taokayan taokayan changed the title Kayan db guard test add DB guard check for nodeos_under_min_avail_ram.py May 7, 2019
tests/nodeos_under_min_avail_ram.py Show resolved Hide resolved
Copy link
Contributor

@heifner heifner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should target develop. I see no reason to add it to the 1.8.x release.

Copy link
Contributor

@brianjohnson5972 brianjohnson5972 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changing to not approved, so we get the merge problem, pointed out by Kevin, taken care of

@brianjohnson5972 brianjohnson5972 dismissed their stale review May 7, 2019 14:26

Changing to not approved, so we get the merge problem, pointed out by Kevin, taken care of

@taokayan taokayan changed the base branch from release/1.8.x to develop May 8, 2019 03:16
@taokayan
Copy link
Contributor Author

taokayan commented May 8, 2019

hey @heifner @brianjohnson5972 I've re-based please check it again.

@taokayan
Copy link
Contributor Author

taokayan commented May 9, 2019

test case nodeos_under_min_avail_ram_lr_test passed on build servers, now it is set back to "long running test".

Test project /data/job/build
      Start 47: nodeos_sanity_test
 1/13 Test #47: nodeos_sanity_test ...................   Passed    5.20 sec
      Start 48: nodeos_run_test
 2/13 Test #48: nodeos_run_test ......................   Passed   41.20 sec
      Start 49: p2p_dawn515_test
 3/13 Test #49: p2p_dawn515_test .....................   Passed    8.06 sec
      Start 50: nodeos_run_test-mongodb
 4/13 Test #50: nodeos_run_test-mongodb ..............   Passed   92.20 sec
      Start 51: producer-preactivate-feature-test
 5/13 Test #51: producer-preactivate-feature-test ....   Passed   45.57 sec
      Start 52: nodeos_protocol_feature_test
 6/13 Test #52: nodeos_protocol_feature_test .........   Passed    6.17 sec
      Start 53: distributed-transactions-test
 7/13 Test #53: distributed-transactions-test ........   Passed   77.67 sec
      Start 54: restart-scenarios-test-resync
 8/13 Test #54: restart-scenarios-test-resync ........   Passed   82.97 sec
      Start 55: restart-scenarios-test-hard_replay
 9/13 Test #55: restart-scenarios-test-hard_replay ...   Passed   78.71 sec
      Start 56: restart-scenarios-test-none
10/13 Test #56: restart-scenarios-test-none ..........   Passed   78.92 sec
      Start 57: validate_dirty_db_test
11/13 Test #57: validate_dirty_db_test ...............   Passed    6.54 sec
      Start 58: launcher_test
12/13 Test #58: launcher_test ........................   Passed   96.26 sec
      Start 65: nodeos_under_min_avail_ram_lr_test
13/13 Test #65: nodeos_under_min_avail_ram_lr_test ...   Passed  259.10 sec

100% tests passed, 0 tests failed out of 13

Label Time Summary:
nonparallelizable_tests    = 878.56 sec*proc (13 tests)

@heifner heifner merged commit 2d97df0 into EOSIO:develop May 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants