-
Notifications
You must be signed in to change notification settings - Fork 370
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release merge #342
Merged
Merged
Release merge #342
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…#312) - do not do any view changes since we're dealing with non-master instance and may have not order all requests if view is changed
* Log error when the file is corrupted. Raise appropriate exception. Shows traceback * Log error when the file is corrupted. Raise appropriate exception. Shows traceback. * finished changes * Log error when the file is corrupted. Raise appropriate exception. Shows traceback * Log error when the file is corrupted. Raise appropriate exception. Shows traceback. * finished changes * Changed JsonDecodeError to a more generic error (ValueError). MsgPack decode errors need to be supported * Created helper function to make parseLedgerForHaAndKeys more readable. Fixed my tests to check for a more descriptive message. Changed names of the tests and fixtures in that test module. * fix error where test function was using the old fixture name. * Fixed formatting in test file.
…into terminology
* Reject messages exceeding size limit * Do not send messages exceeding limit * Add tests for send and recv huge data * Fix review notes * Fix names to snake-case
* fix testPrimaryRecvs3PhaseMessageOutsideWatermarks test: - do not do any view changes since we're dealing with non-master instance and may have not order all requests if view is changed * fix testPrimaryRecvs3PhaseMessageOutsideWatermarks test: - do not do any view changes since we're dealing with non-master instance and may have not order all requests if view is changed - delay for all nodes (both primary and non-primary), since this is delay for receiving, not sending. * fix testPrimaryRecvs3PhaseMessageOutsideWatermarks test
…ledger#317) * add a minimum timeout for catchup during view change One of the conditions to finish catch-up during view change is to have MAX_CATCHUPS_DONE_DURING_VIEW_CHANGE rounds of catch-up without any new transactions caught up. But this should not finish very quickly. So, we should try to catch-up if no new txns caught up until MIN_TIMEOUT_CATCHUPS_DONE_DURING_VIEW_CHANGE in addition to the max number of rounds. * abstract the catch-up limit logic in a method * set info log level for a stress test producing lots of debug/trace output
Merge M1 into master
Modify/Update remaining terminology changes
…ger#320) * Fix crash with new version of promt-toolkit on ubuntu 17.04 * Use local var for token reassign * Move test to separate file
Fully reproducible build, INDY-346
* Adhere pep8 for state * Fix some pep8 errors for ledger * Fix some pep8 errors for stp_core
* Provide error from transport level to upper level * Add config to Batched initialization * Check tests to handle multiple return values from send func * Fix fix node tests * Add test to be sure that transport error passed to the upper levels * Add import of conftest
* Use GroupedLoadScheduling for testing * Switch on test jenkins-shared * Install xdist * Speed up logcapture * Turn asyncio looper debug off * Use PYTHONASYNCIODEBUG environment variable * Slice the plenum tests on 3 parts * Write the test results in separate files * Fix running test as a module test_node_key_changed has to be the last test in pool_transactions module because the test changes a node's key so it breaks other tests * Skip test_slow_nodes_catchup_before_selecting_primary_in_new_view * Enable view-change after test_disable_view_change
* add todo about removal of error.py * remove empty looper.py file * create Replicas class for replica management * use Replicas instead of list of Replica in Node * move logic of adding of new replica to Replicas * move logic of replica removal from Node to Replicas * move creation of replica instance from Node to Replicas * move checking of whethere there is primary on some replica to Replicas * implement removal of instances from monitoring * annotate getlogger with type to enable autocomplete on it * move logic of handling replica messages to Replicas * replace typing.Deque by collections.deque in type hint because interpreter cannot find it * pass full node to Replicas instead of just name because Replica need it * move adding of new ledger to Replicas * remove all_instances_have_primary from Node * create and use has_master_primary instead if manual checking * remove sync3PhaseState from Node since it is not used and has invalid implementation * update docstrings for hasPrimary and has_master_primary * allow accessing master replica directly but leave todo that it should be prohibited * fix mismatches in usage of replicas * make repr of Stats return string * update monitor in replicas when it is updated it tests * fix forwarding of messages to replicas * fix stashing replica outbox messages in tests * clean imports in Replicas * rewrite pass_message in a simpler way * fix log message in get_output * use explicit comparison instead of range in Instances.remove
* PEP8 fixes for stp_raet, stp_zmq, INDY-493 * Remove importing of a fixture It causes problems in case using 'optimise imports' feature
The package already exists in the canonical archive
…ica (hyperledger#329) * fix bug where after suspension node gets an ordered request from replica * correct expression
* change ledger serialization - re-factore storage hierarchy - use leveldb by default as a ledhger storage - use msgpack serializer by default - separate initialization from genesis txn logic * fix tests * fix tests * fix tests * Merge branches 'ledger-serialization' and 'master' of github.com:hyperledger/indy-plenum into ledger-serialization # Conflicts: # ledger/test/test_ledger.py * separate txn and hash serializers in ledger * fix tests * fix order in msgpack serialization * fix tests; support proper opening/closinf of ledger ons start/stop of the node * fix tests * Merge branches 'ledger-serialization' and 'master' of github.com:hyperledger/indy-plenum into ledger-serialization # Conflicts: # ledger/ledger.py # plenum/client/client.py * Merge branches 'ledger-serialization' and 'master' of github.com:hyperledger/indy-plenum into ledger-serialization # Conflicts: # ledger/test/test_ledger.py * fix tests * fix tests * fix tests * do not serialize keys * Merge branches 'ledger-serialization' and 'm1' of github.com:hyperledger/indy-plenum into ledger-serialization # Conflicts: # plenum/test/common/test_parse_ledger.py * fixed client transaction log. * use json for state; add TODOs * separate client req-rep store serialization * fix GET_TXN call and tests * re-factor genesis txn naming * fix import * fix tests * fix tests * fix tests * fix method * use leveldb hash-store * add comments * add build dependency * rename sample genesis txn files * rename sample genesis txn files * get rid of creation unnecessary files in .sovrin when CLI is initiated * improve dealing with non-existing genesis file; cleanups * use msgpack==0.4.6; cleanup * cleanup * fix tests
The delay has to be increased because of the performance fix in the 'logcapture' function (decreased number of re.search). The tests became faster so delay=3 was not enough.
* INDY-484: Cleaned up INFO log level * INDY-484: Cleaned up INFO log level * INDY-484: Reverted levels of 2 log messages Reverted the levels of the log messages about listening for other nodes when ZMQ/RAET stack is starting. * INDY-484: Processed code review findings * INDY-484: Reverted formatting * INDY-484: Added prefixes to important log messages * INDY-484: Corrected tests
ashcherbakov
approved these changes
Aug 16, 2017
Toktar
pushed a commit
to Toktar/indy-plenum
that referenced
this pull request
Mar 1, 2021
Release merge Signed-off-by: Alexander Shcherbakov <[email protected]> Sign-off-executed-by: toktar <[email protected]> Approved-at: h-master
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.