Skip to content

Commit cc419ef

Browse files
authored
patch validator continue for PoA (#3)
1 parent 158f146 commit cc419ef

File tree

3 files changed

+2
-129
lines changed

3 files changed

+2
-129
lines changed

.github/workflows/release.yml

-51
This file was deleted.

.github/workflows/test.yml

-77
Original file line numberDiff line numberDiff line change
@@ -144,52 +144,6 @@ jobs:
144144
name: "${{ github.sha }}-e2e-coverage"
145145
path: ./tests/e2e-profile.out
146146

147-
repo-analysis:
148-
runs-on: ubuntu-latest
149-
needs: [tests, test-integration, test-e2e]
150-
steps:
151-
- uses: actions/checkout@v3
152-
- uses: technote-space/[email protected]
153-
id: git_diff
154-
with:
155-
PATTERNS: |
156-
**/*.go
157-
go.mod
158-
go.sum
159-
**/go.mod
160-
**/go.sum
161-
- uses: actions/download-artifact@v4
162-
if: env.GIT_DIFF
163-
with:
164-
name: "${{ github.sha }}-00-coverage"
165-
- uses: actions/download-artifact@v4
166-
if: env.GIT_DIFF
167-
with:
168-
name: "${{ github.sha }}-01-coverage"
169-
- uses: actions/download-artifact@v4
170-
if: env.GIT_DIFF
171-
with:
172-
name: "${{ github.sha }}-02-coverage"
173-
- uses: actions/download-artifact@v4
174-
if: env.GIT_DIFF
175-
with:
176-
name: "${{ github.sha }}-03-coverage"
177-
- uses: actions/download-artifact@v4
178-
if: env.GIT_DIFF
179-
with:
180-
name: "${{ github.sha }}-integration-coverage"
181-
- uses: actions/download-artifact@v4
182-
if: env.GIT_DIFF
183-
with:
184-
name: "${{ github.sha }}-e2e-coverage"
185-
continue-on-error: true
186-
- name: sonarcloud
187-
if: ${{ env.GIT_DIFF && !github.event.pull_request.draft }}
188-
uses: SonarSource/sonarcloud-github-action@master
189-
env:
190-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
191-
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
192-
193147
liveness-test:
194148
runs-on: ubuntu-latest
195149
timeout-minutes: 15
@@ -497,37 +451,6 @@ jobs:
497451
with:
498452
projectBaseDir: orm/
499453

500-
test-cosmovisor:
501-
runs-on: ubuntu-latest
502-
steps:
503-
- uses: actions/checkout@v3
504-
- uses: actions/setup-go@v4
505-
with:
506-
go-version: "1.21"
507-
check-latest: true
508-
cache: true
509-
cache-dependency-path: tools/cosmovisor/go.sum
510-
- uses: technote-space/[email protected]
511-
id: git_diff
512-
with:
513-
PATTERNS: |
514-
tools/cosmovisor/**/*.go
515-
tools/cosmovisor/go.mod
516-
tools/cosmovisor/go.sum
517-
- name: tests
518-
if: env.GIT_DIFF
519-
run: |
520-
cd tools/cosmovisor
521-
go test -mod=readonly -timeout 30m -coverprofile=coverage.out -covermode=atomic -tags='norace ledger test_ledger_mock rocksdb_build' ./...
522-
- name: sonarcloud
523-
if: ${{ env.GIT_DIFF && !github.event.pull_request.draft && env.SONAR_TOKEN != null }}
524-
uses: SonarSource/sonarcloud-github-action@master
525-
env:
526-
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
527-
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
528-
with:
529-
projectBaseDir: tools/cosmovisor/
530-
531454
test-confix:
532455
runs-on: ubuntu-latest
533456
steps:

x/staking/keeper/val_state_change.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,8 @@ func (k Keeper) ApplyAndReturnValidatorSetUpdates(ctx context.Context) (updates
159159
validator := k.mustGetValidator(ctx, valAddr)
160160

161161
if validator.Jailed {
162-
panic("should never retrieve a jailed validator from the power store")
162+
// panic("should never retrieve a jailed validator from the power store")
163+
continue
163164
}
164165

165166
// if we get to a zero-power validator (which we don't bond),

0 commit comments

Comments
 (0)