[issue 99] cosmos upgrade to v0.44 #157

andkononykhin · 2021-12-31T07:54:49Z

Done

core & cli:
- fully functional with all the legacy features
  - generated from the scratch using starport (layout, modules, types)
  - migrated all business valuable logic from legacy code: adjusted the modules, re-implemented some cases for dclauth, validator and genutil
tests:
- optimized test and build infra a bit (makefile, dockerfile, localnet -> .localnet)
- all integrations tests were migrated and passed
  - NOTE. currently these tests include not only CLI and REST but also gRPC ones which are similar to REST ones except the endpoints used to communicate with a pool
- unit tests:
  - migrated legacy handler tests
  - migrated some other legacy unit tests

Follow-ups - #175

review cosmos (other blockchains) Makefile
- automate update of third-parties proto definitions
review FIXME issue 99 comments
review TODO issue 99 comments
upgrade to latest cosmos and tendermint
review how cosmos (tendermint and starport) organizes tests (mocks, simulation, testutil) and update ours
(possible overhead) looks like we review validators update at end of each txns block
(possible overhead) missed blocks indexes is implemented as state entry with (addr, index) key but it is iterated for all possible indexes within SignedBlocksWindow
switch custom iterators to query.Paginate (as starport scaffolds by default)
ValidatorOwner shouldn't be exported/imported in Genesis, instead it should be filled basing on Validator data during the import

- show 404 in REST - show empty or nil value in CLI

[issue 99] base modules fixes

Model module upgrade to cosmos v0.44

…gbee-alliance/distributed-compliance-ledger into vendorinfo-migrate-1

Vendorinfo module migrate

…iance/distributed-compliance-ledger into cosmos-update-pki-2 � Conflicts: � docs/static/openapi.yml � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.ts

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

[issue 99] various-fixes

…iance/distributed-compliance-ledger into cosmos-update-pki-2

Improved queries response when key is absent

…iance/distributed-compliance-ledger into cosmos-update-pki-tests

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

…ignments [issue 99] auth and validator modules alignments

- Simplified BuildAndBroadcastTx function use. - Migrating gRPC/REST integration tests of model module.

…iance/distributed-compliance-ledger into model-grpc-rest-upgrade � Conflicts: � config.yml � docs/static/openapi.yml � vue/src/store/generated/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.ts

Migrating model gRPC/REST integration tests

…iance/distributed-compliance-ledger into cosmos-update-pki-tests � Conflicts: � docs/static/openapi.yml � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.dclauth/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.model/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.pki/module/index.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.vendorinfo/module/index.d.ts � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.vendorinfo/module/index.js � vue/src/store/generated/zigbee-alliance/distributed-compliance-ledger/zigbeealliance.distributedcomplianceledger.vendorinfo/module/index.ts

…iance/distributed-compliance-ledger into compliance-unit-tests-migration � Conflicts: � x/compliance/keeper/msg_server_test.go � x/compliance/types/message_provision_model.go

Migrate compliance Handler unit tests

Compliancetest unit tests

…gbee-alliance/distributed-compliance-ledger into compliance-tests

Compliance unit tests

…iance/distributed-compliance-ledger into evidence-module

- Corrected compliance logic for off-ledger certification approach. - Updated unit tests and documentation correspondingly.

Jail/Slash validator if malicious behaviour is detected by Tendermint

…iance/distributed-compliance-ledger into compliance-off-ledger-certification-correction

- Corrected integration tests according to changed compliance logic.

…ation-correction Fix compliance logic for off-ledger certification

ashcherbakov and others added 30 commits December 22, 2021 18:57

Improved queries response when key is absent

f94d060

- show 404 in REST - show empty or nil value in CLI

Revised gRPC queries in model module

5a350f6

Merge pull request #139 from andkononykhin/issue-99-base-modules-fixes

626c1a7

[issue 99] base modules fixes

Merge pull request #135 from spivachuk/model-upgrade-to-cosmos-v0.44

077e5c4

Model module upgrade to cosmos v0.44

changed uint64 to int32

9daf4a2

fixed errors

0996160

Merge branch 'issue-99-cosmos-upgrade-v0.44' of https://github.com/zi…

1f0f0db

…gbee-alliance/distributed-compliance-ledger into vendorinfo-migrate-1

Merge pull request #131 from DenisRybas/vendorinfo-migrate-1

3810488

Vendorinfo module migrate

Initial REST tests structure

ccf5ebd

Move CLI utils to a shared place

17442b6

fixes docker ctx

ea2f793

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

fixes cli flags

89a0c60

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

Merge pull request #140 from andkononykhin/various-fixes

88b207f

[issue 99] various-fixes

Merge branch 'issue-99-cosmos-upgrade-v0.44' of github.com:zigbee-all…

0c3dba9

…iance/distributed-compliance-ledger into cosmos-update-pki-2

Merge pull request #138 from zigbee-alliance/cosmos-update-pki-2

1d9e1bc

Improved queries response when key is absent

Merge branch 'issue-99-cosmos-upgrade-v0.44' of github.com:zigbee-all…

f740b30

…iance/distributed-compliance-ledger into cosmos-update-pki-tests

fix REST endpoints for PKI

d22021f

cleans up dclauth and validator rest endpoints

8841038

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

PKI Rest tests: support query requests

921f2e3

restores string keys for validator state data

c43e817

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

restores string keys for dclauth state data

481c14c

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

fixes previous changes

68c1101

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

regenerates proto based data

be41f31

Signed-off-by: Andrey Kononykhin <andkononykhin@gmail.com>

Merge pull request #141 from andkononykhin/issue-99-auth-validator-al…

aa7d62f

…ignments [issue 99] auth and validator modules alignments

Migrating integration tests of model module

87f418a

- Simplified BuildAndBroadcastTx function use. - Migrating gRPC/REST integration tests of model module.

PKI Rest tests: support write requests

f9728ca

Merge pull request #142 from spivachuk/model-grpc-rest-upgrade

9b88466

Migrating model gRPC/REST integration tests

spivachuk and others added 27 commits January 13, 2022 16:52

Merge branch 'issue-99-cosmos-upgrade-v0.44' of github.com:zigbee-all…

53a57c8

…iance/distributed-compliance-ledger into compliance-unit-tests-migration � Conflicts: � x/compliance/keeper/msg_server_test.go � x/compliance/types/message_provision_model.go

Resolved linter errors

d7f8c65

negative tests added

864ecb6

updated tests

5ed739a

tests added

9239564

certify model tests updated

03c81f8

Merge pull request #178 from spivachuk/compliance-unit-tests-migration

af1ce36

Migrate compliance Handler unit tests

Merge pull request #177 from DenisRybas/compliancetest-tests

8ee3f96

Compliancetest unit tests

provision and revoke tests added

635a506

Merge branch 'issue-99-cosmos-upgrade-v0.44' of https://github.com/zi…

db42960

…gbee-alliance/distributed-compliance-ledger into compliance-tests

tests updated

0f83e26

vendorinfo tests updated

7e1d004

vendorinfo tests updated

863156e

tests from msgs_tests added

779bc86

errors fixed

51f8e0d

Merge pull request #176 from DenisRybas/compliance-tests

77a865e

Compliance unit tests

tests added

85d541f

add tests for HandleDoubleSign

fb101dc

Merge pull request #179 from DenisRybas/compliance-tests

8124930

Compliance unit tests

add tests for ApplyAndReturnValidatorSetUpdates

49b4efe

Merge branch 'issue-99-cosmos-upgrade-v0.44' of github.com:zigbee-all…

e528726

…iance/distributed-compliance-ledger into evidence-module

Fix compliance logic for off-ledger certification

9db0b7d

- Corrected compliance logic for off-ledger certification approach. - Updated unit tests and documentation correspondingly.

fix linter errors

424323e

Merge pull request #180 from zigbee-alliance/evidence-module

96ce165

Jail/Slash validator if malicious behaviour is detected by Tendermint

Merge branch 'issue-99-cosmos-upgrade-v0.44' of github.com:zigbee-all…

cc6fddd

…iance/distributed-compliance-ledger into compliance-off-ledger-certification-correction

Corrected integration tests

3754057

- Corrected integration tests according to changed compliance logic.

Merge pull request #181 from spivachuk/compliance-off-ledger-certific…

14fe651

…ation-correction Fix compliance logic for off-ledger certification

ashcherbakov approved these changes Jan 14, 2022

View reviewed changes

ashcherbakov merged commit f3866fc into master Jan 14, 2022

ashcherbakov deleted the issue-99-cosmos-upgrade-v0.44 branch April 18, 2022 05:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[issue 99] cosmos upgrade to v0.44 #157

[issue 99] cosmos upgrade to v0.44 #157

andkononykhin commented Dec 31, 2021 •

edited by ashcherbakov

[issue 99] cosmos upgrade to v0.44 #157

[issue 99] cosmos upgrade to v0.44 #157

Conversation

andkononykhin commented Dec 31, 2021 • edited by ashcherbakov

Done

Follow-ups - #175

andkononykhin commented Dec 31, 2021 •

edited by ashcherbakov