Skip to content
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.

Sign up for GitHub

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

Jump to bottom

build(deps): bump cosmos-sdk to v047 #690

Merged
merged 27 commits into from
Mar 13, 2024
Merged

build(deps): bump cosmos-sdk to v047 #690

merged 27 commits into from
Mar 13, 2024

Conversation

atheeshp
Copy link
Collaborator

@atheeshp atheeshp commented Feb 7, 2024

Closes: #686

@atheeshp atheeshp changed the base branch from main to develop February 7, 2024 10:26
@atheeshp atheeshp changed the base branch from develop to prop31 February 8, 2024 05:33
akhilkumarpilli
akhilkumarpilli reviewed Feb 13, 2024
View reviewed changes
x/did/module.go Outdated Show resolved Hide resolved
akhilkumarpilli
akhilkumarpilli reviewed Feb 13, 2024
View reviewed changes
Copy link
Collaborator

@akhilkumarpilli akhilkumarpilli left a comment

Choose a reason for hiding this comment

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

Remove commented code if not needed

@akhilkumarpilli
Copy link
Collaborator

May be we need to add this code to provide newer gRPC services, have a look on this: https://github.com/cosmos/cosmos-sdk/blob/v0.47.8/UPGRADING.md#app-wiring

akhilkumarpilli
akhilkumarpilli reviewed Feb 13, 2024
View reviewed changes
app/export.go Outdated Show resolved Hide resolved
atheeshp and others added 7 commits February 14, 2024 15:51
@atheeshp
fixes
3132693
@atheeshp
review changes
0d98129
@atheeshp
review changes
02ede70
@atheeshp
chore: delete simapp package
662a796
@deepan95dev @atheeshp @akhilkumarpilli
feat!: x/did and x/resource module changes according to sdk v0.47 (…
8d48896 
…#691)

* chore: did module upgrade changes

* fix lint

* go mod tidy

* remove commented code

* remove commented line

* update golang version

* fix lint

* fix lint

* try fix lint

* fix lint

* fix proto-gen

* add logs

* add log

* update

* fix lint

* fix proto-gen

* add perms

* fix build

* lint fix

* lint:gofmt enabled in ci

* try fixing lint

* try fix lint

* reveert

* try lint

* try fix lint

* fix lint

* lint fixes

* fix permissions

* fix releases

* revert

* review changes

* try fixing jobs

* try fixing jobs

* add logs

* add change perms

* revert

* chore: appmodule missing methods added

* try fix build

* add perms

* chore: missing method with attrition variable

* Update .github/linters/.golangci.yaml

Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>

* Update scripts/protocgen.sh

Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>

* review changes

---------

Co-authored-by: atheesh <atheesh@vitwit.com>
Co-authored-by: atheeshp <59333759+atheeshp@users.noreply.github.com>
Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>
@atheeshp
proto-gen
fb98ead
@deepan95dev @atheeshp @akhilkumarpilli
ci: Go releaser fix (#689) (#693)
c7c4554 
* fix: release permission check

* try fix go releaser

* touch

* touch

* release check

* try fix

* release check

* fix swagger

* fix swagger gen

* fix

* release check

* update goreleaser config

* revert changes

* fix: ante handler ibc fix

* review changes

---------

Co-authored-by: atheesh <atheesh@vitwit.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Eengineer1
Eengineer1 requested changes Feb 26, 2024
View reviewed changes
.github/linters/.golangci.yaml Outdated Show resolved Hide resolved
.github/linters/.golangci.yaml Outdated Show resolved Hide resolved
app/app.go Show resolved Hide resolved
app/app.go Show resolved Hide resolved
app/app.go Show resolved Hide resolved
tests/upgrade/integration/v1/cli/tx_did.go Outdated Show resolved Hide resolved
tests/upgrade/integration/v2/loader.go Outdated Show resolved Hide resolved
x/did/types/tx_msg_create_diddoc_payload.go Outdated Show resolved Hide resolved
x/resource/keeper/querier.go Outdated Show resolved Hide resolved
x/resource/keeper/querier_collection_resources.go Outdated Show resolved Hide resolved
@Eengineer1 Eengineer1 mentioned this pull request Feb 26, 2024
Merged
This was referenced Feb 27, 2024
Closed
Closed
Closed
Closed
Closed
Closed
@ankurdotb ankurdotb changed the title build(deps)!: bump cosmos-sdk to v047 build(deps): bump cosmos-sdk to v047 Feb 27, 2024
atheeshp and others added 3 commits February 28, 2024 01:10
@atheeshp @deepan95dev
@akhilkumarpilli @ankurdotb @Eengineer1
fix: failing tests (#694)
357ba2a 
* fix: release permission check

* try fix go releaser

* touch

* touch

* release check

* try fix

* release check

* fix swagger

* fix swagger gen

* fix

* release check

* update goreleaser config

* revert changes

* fix: ante handler ibc fix

* fix tests

* remove test.sh

* fix: integration test check

* keys check

* try fix start script

* fix jq

* remove logs

* fix import keys

* revert

* fix: tx msgServer fixes

* fix: keys check

* fix import keys

* revert

* fix tests

* fix panic

* fix build

* add wait timer

* reduce timer

* increase time

* fix: upgrade test fixes for old binary

* fix: old binary setup.sh

* fix: localnet dir path fix

* fix: upgrade test fix

* add missing post hand;er

* fix tests

* add timer

* add post handler

* add handler

* fix test

* add timer

* try increasing the time

* decrease the timer

* fix tests

* fix lint

* fix lint

* fix lint

* fix tests

* fix

* increase time

* fix: upgrade test moduleVersion fix

* fix: upgrade test fix

* fix: test docker ps

* fix: checking docker before post test

* fix: parse fix

* fix: docker check

* fix: docker keys check

* revert: entrypoint revert

* fix: docker ps check with logs

* fix: upgrade handler with consensus, crisis

* test: docker ps logs check

* test: SendEnabled param fix

* lint: gofumpt fix

* chore: consensus module included

* chore: bank subspace fix

* lint: golang ci fix

* test: feeparam register fix

* fix: bankType param fix

* ci: Update CI workflows (#695)

* update package lock

* Update .releaserc.json

* Bump setup-go

* Update dependabot.yml

* Update pull-request.yml

* Update release workflows

* Uploaded download/upload artifact

* Ignore twitter links

* archives.rlcp is deprecated

* Standardise quotes

* ci: Resolved Integration, Upgrade test suites (#696)

* Debug

* Switch to param key table

* Skip deprecation staticcheck

* Added crisis param key table

* Added distribution parameter key table

* Added gov parameter key table

* Added mint parameter key table

* Added slashing parameter key table

* Added staking parameter key table

* Removed exclusion

* Added auth parameter key table

* Added consensus module expected definition

* Added legacy gov router registration

* Added artificial idle time against sequence mismatch

* Additional idle time

* Run sequential

* Switch to utilise other op acc

* Additional overhead

* Re-order against sequence mismatch

* Removed idle time

* Added polling for optimistic tx submission

* Additional overhead

---------

Co-authored-by: deepan95dev <deepan@vitwit.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Co-authored-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>
@deepan95dev
chore: removed commented lines
b1c641d
@deepan95dev
deps: updated bufbuild version + idle time extended in integration test
1bc8b74
@deepan95dev deepan95dev mentioned this pull request Feb 28, 2024
Closed
atheeshp and others added 2 commits March 6, 2024 09:57
@atheeshp
fix: ibc defi tests (#697)
e575046 
* fix ibc tests

* fix ibc tests

* add keys to gitignore

* remove unwanted line

* update ibc handler

* add comments
@deepan95dev
fix: state sync fixes using upgrade handler (#708)
90ad109 
* test: state-sync upgrade handler fix

* fix: state sync param fix
@Eengineer1 Eengineer1 marked this pull request as ready for review March 11, 2024 22:45
@Eengineer1 Eengineer1 self-requested a review March 11, 2024 22:46
@Eengineer1 Eengineer1 merged commit 71e574b into prop31 Mar 13, 2024
17 checks passed
@Eengineer1 Eengineer1 deleted the v047_upgrade branch March 13, 2024 19:38
Eengineer1 added a commit that referenced this pull request Mar 13, 2024
@Eengineer1 @ankurdotb
@whalelephant @j0nl1 @atheeshp @deepan95dev @akhilkumarpilli
feat!: Nymlab (AVIDA Protocol) Prop31 Stage 1 implementation(#628) (#682
be1135a 
)

* feat!: Nymlab Prop31 (#628)

* fix: LD_FLAGS -> LDFLAGS

* feat/resource: pump consensus version to v3, update ibcModule, types + keeper

* tests/resource/ibc: OnChanOpen* tests

* chore: bump x/resource pb version 2->3

* wip: comment out migration to test build

* wip test

* chore: store resource v2 types

* tests/resource: ibc on recv packets

* feat/resource: add more detailed errors

* feat: starship test base

* feat/resources: resourcePortId as const /proto revert

* chore: remove deadcode

* feat: upgrade and migration

* fix: update app migrator for keeper

* wip: conflict

* chore: bump version and remove test in go.work

* chore: make sure ibc-app-version aligns with consensus

* chore: move interchain tests out of repo

* fix: ibc connection

---------

Co-authored-by: bwty <whalelephant@users.noreply.github.com>
Co-authored-by: j0nl1 <jonlitech@gmail.com>
Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>
Co-authored-by: Belsy <email-me@belsy.space>

* chore: Align with overall codebase + cleanups

* chore: Removed incompatible cache based setup

* chore: Resolved lint issues

* test: Bump upgrade test suite in prep for v2 major (#683)

* test: Transition to v2 + relevant indicative upgrade tests

* Overall format

* Added distinction on build flags + binary paths

* Switched to ordered container

* Updated relative path references

* Switched to url-safe

* Added padding where relevant

* Cleanup

* Switch to legacy literals

* Removed optional prop info

* Added sample realistic upgrade info

* Added checksum indication

* Removed legacy querier

* Bumped extra blocks

* Streamline expected payloads as proto JSON

* Cleanup

* Added intermittent EOL

* Cleanup

* fix: Add msg signature normalisation for deactivate did doc msg (#685)

* Add msg signature normalization for deactivate did doc msg

* Update MD rules

* Pin GoReleaser

* Pin GoReleaser v1.22.1

* Removed pin

---------

Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>

* build(deps): bump cosmos-sdk to v047 (#690)

* bump cosmos sdk

* code clean up

* clean

* fix proto-gen

* fix build

* !build(deps): fix build (#692)

* code clean up

* clean

* fix proto-gen

* fix build

* fix build

* fix build

* fix makefile

* fix chain

* fix app.go

* protocgen

* fix ci job

* fix build

* fixes

* review changes

* review changes

* chore: delete simapp package

* feat!: `x/did` and `x/resource`  module changes according to sdk v0.47 (#691)

* chore: did module upgrade changes

* fix lint

* go mod tidy

* remove commented code

* remove commented line

* update golang version

* fix lint

* fix lint

* try fix lint

* fix lint

* fix proto-gen

* add logs

* add log

* update

* fix lint

* fix proto-gen

* add perms

* fix build

* lint fix

* lint:gofmt enabled in ci

* try fixing lint

* try fix lint

* reveert

* try lint

* try fix lint

* fix lint

* lint fixes

* fix permissions

* fix releases

* revert

* review changes

* try fixing jobs

* try fixing jobs

* add logs

* add change perms

* revert

* chore: appmodule missing methods added

* try fix build

* add perms

* chore: missing method with attrition variable

* Update .github/linters/.golangci.yaml

Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>

* Update scripts/protocgen.sh

Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>

* review changes

---------

Co-authored-by: atheesh <atheesh@vitwit.com>
Co-authored-by: atheeshp <59333759+atheeshp@users.noreply.github.com>
Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>

* proto-gen

* ci: Go releaser fix (#689) (#693)

* fix: release permission check

* try fix go releaser

* touch

* touch

* release check

* try fix

* release check

* fix swagger

* fix swagger gen

* fix

* release check

* update goreleaser config

* revert changes

* fix: ante handler ibc fix

* review changes

---------

Co-authored-by: atheesh <atheesh@vitwit.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>

* fix: failing tests (#694)

* fix: release permission check

* try fix go releaser

* touch

* touch

* release check

* try fix

* release check

* fix swagger

* fix swagger gen

* fix

* release check

* update goreleaser config

* revert changes

* fix: ante handler ibc fix

* fix tests

* remove test.sh

* fix: integration test check

* keys check

* try fix start script

* fix jq

* remove logs

* fix import keys

* revert

* fix: tx msgServer fixes

* fix: keys check

* fix import keys

* revert

* fix tests

* fix panic

* fix build

* add wait timer

* reduce timer

* increase time

* fix: upgrade test fixes for old binary

* fix: old binary setup.sh

* fix: localnet dir path fix

* fix: upgrade test fix

* add missing post hand;er

* fix tests

* add timer

* add post handler

* add handler

* fix test

* add timer

* try increasing the time

* decrease the timer

* fix tests

* fix lint

* fix lint

* fix lint

* fix tests

* fix

* increase time

* fix: upgrade test moduleVersion fix

* fix: upgrade test fix

* fix: test docker ps

* fix: checking docker before post test

* fix: parse fix

* fix: docker check

* fix: docker keys check

* revert: entrypoint revert

* fix: docker ps check with logs

* fix: upgrade handler with consensus, crisis

* test: docker ps logs check

* test: SendEnabled param fix

* lint: gofumpt fix

* chore: consensus module included

* chore: bank subspace fix

* lint: golang ci fix

* test: feeparam register fix

* fix: bankType param fix

* ci: Update CI workflows (#695)

* update package lock

* Update .releaserc.json

* Bump setup-go

* Update dependabot.yml

* Update pull-request.yml

* Update release workflows

* Uploaded download/upload artifact

* Ignore twitter links

* archives.rlcp is deprecated

* Standardise quotes

* ci: Resolved Integration, Upgrade test suites (#696)

* Debug

* Switch to param key table

* Skip deprecation staticcheck

* Added crisis param key table

* Added distribution parameter key table

* Added gov parameter key table

* Added mint parameter key table

* Added slashing parameter key table

* Added staking parameter key table

* Removed exclusion

* Added auth parameter key table

* Added consensus module expected definition

* Added legacy gov router registration

* Added artificial idle time against sequence mismatch

* Additional idle time

* Run sequential

* Switch to utilise other op acc

* Additional overhead

* Re-order against sequence mismatch

* Removed idle time

* Added polling for optimistic tx submission

* Additional overhead

---------

Co-authored-by: deepan95dev <deepan@vitwit.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Co-authored-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>

* chore: removed commented lines

* deps: updated bufbuild version + idle time extended in integration test

* fix: ibc defi tests (#697)

* fix ibc tests

* fix ibc tests

* add keys to gitignore

* remove unwanted line

* update ibc handler

* add comments

* fix: state sync fixes using upgrade handler (#708)

* test: state-sync upgrade handler fix

* fix: state sync param fix

---------

Co-authored-by: Deepan kumar <150890063+deepan95dev@users.noreply.github.com>
Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Co-authored-by: deepan95dev <deepan@vitwit.com>
Co-authored-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>

---------

Co-authored-by: Ankur Banerjee <ankurdotb@users.noreply.github.com>
Co-authored-by: bwty <whalelephant@users.noreply.github.com>
Co-authored-by: j0nl1 <jonlitech@gmail.com>
Co-authored-by: Belsy <email-me@belsy.space>
Co-authored-by: Andrew Nikitin <andrew.nikitin@cheqd.io>
Co-authored-by: atheeshp <59333759+atheeshp@users.noreply.github.com>
Co-authored-by: Deepan kumar <150890063+deepan95dev@users.noreply.github.com>
Co-authored-by: Akhil Kumar P <36399231+akhilkumarpilli@users.noreply.github.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Co-authored-by: deepan95dev <deepan@vitwit.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deepan95dev deepan95dev deepan95dev left review comments

@akhilkumarpilli akhilkumarpilli akhilkumarpilli left review comments

@Eengineer1 Eengineer1 Eengineer1 approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bump cosmos sdk version to v047
4 participants
@atheeshp @akhilkumarpilli @Eengineer1 @deepan95dev