Skip to content

Commit

Permalink
feat: use upstream's cosmovisor (#792)
Browse files Browse the repository at this point in the history
* Remove cosmovisor

* Add scripts for testing cosmovisor

* Enable cosmovisor test in CI

* Update CHANGELOG.md

* Add a manual download test case

* Separate installations

* Remove unused variables

* Use any for the test
  • Loading branch information
0Tech committed Nov 17, 2022
1 parent 451b622 commit 81f2ed4
Show file tree
Hide file tree
Showing 36 changed files with 211 additions and 1,662 deletions.
43 changes: 21 additions & 22 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,28 +60,27 @@ jobs:
- name: Build
run: GOOS=linux CGO_ENABLED=1 GOARCH=${{ matrix.goarch }} CC=${{ matrix.gcc }} LEDGER_ENABLED=false make build

# TODO: disable test-cosmovisor; this test uses uploaded binary(cosmos-sdk)
# test-cosmovisor:
# runs-on: ubuntu-latest
# steps:
# - uses: actions/checkout@v3
# - uses: actions/setup-go@v3.3.1
# with:
# go-version: 1.18
# - name: Display go version
# run: go version
# - uses: technote-space/get-diff-action@v6.1.1
# id: git_diff
# with:
# PREFIX_FILTER: |
# cosmovisor
# PATTERNS: |
# **/**.go
# go.mod
# go.sum
# - name: Run cosmovisor tests
# run: cd cosmovisor; make
# if: env.GIT_DIFF

test-cosmovisor:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/setup-go@v3.3.1
with:
go-version: 1.18
- name: Display go version
run: go version
- uses: technote-space/get-diff-action@v6.1.1
id: git_diff
with:
PATTERNS: |
**/**.go
go.mod
go.sum
tools/cosmovisor/**
- name: Run cosmovisor tests
run: make cosmovisor
if: env.GIT_DIFF

split-test-files:
runs-on: ubuntu-latest
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
* (store,x/wasm) [\#742](https://github.com/line/lbm-sdk/pull/742) fix to add error message in GetByteCode()
* (amino) [\#745](https://github.com/line/lbm-sdk/pull/745) apply the missing amino codec of `x/token`, `x/collection`, `x/wasm` and `x/foundation`
* (x/foundation) [\#757](https://github.com/line/lbm-sdk/pull/757) remove redundant granter from x/foundation events
* (cosmovisor) [\#792](https://github.com/line/lbm-sdk/pull/792) Use upstream's cosmovisor

### Bug Fixes
* (x/wasm) [\#453](https://github.com/line/lbm-sdk/pull/453) modify wasm grpc query api path
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ $(BUILDDIR)/:
mkdir -p $(BUILDDIR)/

cosmovisor:
$(MAKE) -C cosmovisor cosmovisor
$(MAKE) -C tools/cosmovisor test

.PHONY: build build-linux cosmovisor

Expand Down
12 changes: 0 additions & 12 deletions cosmovisor/Makefile

This file was deleted.

236 changes: 0 additions & 236 deletions cosmovisor/README.md

This file was deleted.

Loading

0 comments on commit 81f2ed4

Please sign in to comment.