f5df349cee
wip
2021-07-23 17:31:55 -04:00
c3ae6f5b58
p2p: add coverage for mConnConnection.TrySendMessage ( #6754 )
...
This change adds additional coverage to the `mConnConnection.TrySendMessage` code path. Adds test to ensure it returns `io.EOF` when closed.
Addresses: #6570
2021-07-23 17:29:19 +00:00
a393cf8bab
internal: update blockchain reactor godoc ( #6749 )
2021-07-23 08:15:57 -04:00
0e2752ae42
light: improve error handling and allow providers to be added ( #6733 )
2021-07-22 18:12:34 +02:00
97a8f125e0
e2e: allow for both v0 and v1 mempool implementations ( #6752 )
2021-07-22 17:59:02 +02:00
84c15857e4
mempool: return mempool errors to the abci client ( #6740 )
...
This changes adds an `MempoolError` field to the `ResponseCheckTx`. This will allow clients to understand that their transaction was rejected from the mempool despite passing the ABCI check.
This change also updates the code to make use of early returns to prevent highly nested code blocks. Namely, it returns when the type assertion fails at the beginning of the method, instead of wrapping the entire method in a large if statement. This has a somewhat large effect on the diff as rendered by github.
addresses: #3546
2021-07-22 14:52:29 +00:00
e70445f942
statesync/event: emit statesync start/end event ( #6700 )
2021-07-22 08:16:50 +02:00
478f5321ad
light: run examples as integration tests ( #6745 )
2021-07-21 09:54:14 -04:00
08e4e2ed3d
ignore issues for stale bot ( #6747 )
2021-07-21 10:39:16 +00:00
7d63e991c5
build(deps): Bump codecov/codecov-action from 1.5.2 to 2.0.1 ( #6739 )
...
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action ) from 1.5.2 to 2.0.1.
- [Release notes](https://github.com/codecov/codecov-action/releases )
- [Changelog](https://github.com/codecov/codecov-action/blob/master/CHANGELOG.md )
- [Commits](https://github.com/codecov/codecov-action/compare/v1.5.2...v2.0.1 )
---
updated-dependencies:
- dependency-name: codecov/codecov-action
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sam Kleinman <garen@tychoish.com >
2021-07-20 17:38:25 -04:00
7638235d33
e2e: longer test harness timeouts ( #6728 )
2021-07-20 15:57:48 -04:00
2abfe20114
e2e: prevent adding light clients as persistent peers ( #6743 )
2021-07-20 18:11:43 +00:00
0bf7813c4e
fix makefile test target to rely on test makefile ( #6746 )
...
The Makefile at the root of the repo [includes](https://github.com/tendermint/tendermint/blob/cd19ef244efda1d666fe4e73b045f94dd5f646ff/Makefile#L61 ) the Makefile under the `test` package. This fix removes the target defined in the root Makefile in favor of the included one.
2021-07-20 18:00:51 +00:00
ff9038e2ce
e2e: run tests in fewer groups ( #6742 )
2021-07-20 13:45:08 -04:00
00a40835a2
e2e: remove cartesian testing of ipv6 ( #6734 )
...
Having looked at our network address parsing and connection code, it
really looks like we're not doing anything on top of what the standard
library is doing (both in terms using `net.ParseIP` and also
`net.Dial`,) and I don't think we need to run the tests 2x the number
of times just to run through different areas of the standard
library. I think most of our users are going to be using IPv4, and
would be down to fully remove this dimension as well, if we find it's
making noise, but for now I think it's fine.
2021-07-20 15:47:35 +00:00
c4f77ab6d1
fastsync/event: emit fastsync status event when switching consensus/fastsync ( #6619 )
...
closes #2498
solves part of #3365
Note: difficult to test the event emit in SwitchToFastSync part, might need to change `stateSyncReactor` to an interface in the `nodeImpl` struct
2021-07-20 15:36:47 +00:00
2030875056
e2e: drop single node hybrid configurations ( #6737 )
2021-07-20 11:23:51 -04:00
639e145729
e2e: avoid systematic key-type variation ( #6736 )
2021-07-20 09:25:00 -04:00
68ffe8bc64
mempool: add TTL configuration to mempool ( #6715 )
2021-07-19 15:54:44 -04:00
21309ccb7b
clist: add a few basic clist tests ( #6727 )
2021-07-19 14:02:21 +00:00
f70396c6fd
add and run make target for generating existing mocks ( #6732 )
...
There are many `//go:generate mockery` lines in the source code.
This change adds a make target to invoke these mock generations.
This change also invokes the mock invocations and adds the resulting mocks to the repo.
Related to #5274
2021-07-18 00:46:04 +00:00
fdc246e4a8
libs/clist: revert clear and detach changes while debugging ( #6731 )
2021-07-17 12:10:53 -04:00
78a0a5fe73
blockchain: error on v2 selection ( #6730 )
...
## Description
Remove v2 flag from toml
2021-07-16 10:19:55 +00:00
4f885209aa
RPC: mark grpc as deprecated ( #6725 )
...
## Description
Mark gRPC as deprecated in the RPC layer.
closes #6718
2021-07-15 22:05:21 +00:00
6dd0cf92c8
router/statesync: add helpful log messages ( #6724 )
2021-07-15 19:26:35 +02:00
626d9b4fbe
build(deps): Bump actions/stale from 3.0.19 to 4 ( #6726 )
...
Bumps [actions/stale](https://github.com/actions/stale ) from 3.0.19 to 4.
- [Release notes](https://github.com/actions/stale/releases )
- [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md )
- [Commits](https://github.com/actions/stale/compare/v3.0.19...v4 )
---
updated-dependencies:
- dependency-name: actions/stale
dependency-type: direct:production
update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2021-07-15 10:17:51 -04:00
8addf99f90
e2e: tweak sleep for pertubations ( #6723 )
...
This tweaks sleeps around pertubations, based on a theory that our
tests with "kill" pertubations restart the nodes fast enough the peers
haven't marked it down when it tries to reconnect. In my local test
runs, this clears out *most* of the test failures that I've seen,
except for one evidence-related test-harness problem (which should be
handled separately.)
2021-07-14 21:07:25 +00:00
76c6c67734
docs: fix broken links ( #6719 )
...
## Description
Fix broken links
closes #6695
2021-07-14 14:31:08 +00:00
a46724e4f6
statesync: dispatcher test uses internal channel for timing ( #6713 )
...
This code change amends the dispatcher tests to read from the dispatcher's `requestCh`. This ensures that a request is waiting when the test calls `dispatcher.respond`.
addresses: #6711
2021-07-14 14:16:09 +00:00
40fba3960d
add missing context catch and tests ( #6701 )
2021-07-14 11:23:15 +02:00
36a859ae54
e2e: ensure evidence validator set matches nodes validator set ( #6712 )
2021-07-13 19:47:36 +02:00
ab5c63eff3
statesync: increase dispatcher timeout ( #6714 )
2021-07-13 13:04:18 -04:00
8228936155
e2e: extend timeouts in test harness ( #6694 )
2021-07-13 11:28:07 -04:00
a12e2bbb60
statesync: use initial height as a floor to backfilling ( #6709 )
2021-07-13 16:36:16 +02:00
11bebfb6a0
build(deps): Bump github.com/google/uuid from 1.2.0 to 1.3.0 ( #6708 )
...
Bumps [github.com/google/uuid](https://github.com/google/uuid ) from 1.2.0 to 1.3.0.
- [Release notes](https://github.com/google/uuid/releases )
- [Commits](https://github.com/google/uuid/compare/v1.2.0...v1.3.0 )
---
updated-dependencies:
- dependency-name: github.com/google/uuid
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2021-07-13 15:15:09 +02:00
4009102e2b
statesync: remove outgoingCalls race condition in dispatcher ( #6699 )
...
* statesync: remove outgoing calls race condition
2021-07-12 19:05:47 -04:00
cabd916517
Revert "statesync: keep peer despite lightblock query fail ( #6692 )" ( #6696 )
...
* Revert "statesync: keep peer despite lightblock query fail (#6692 )"
This reverts commit 50b00dff71
2021-07-12 15:20:02 -04:00
363ea56680
abci: remove counter app ( #6684 )
...
* remove counter app
* remove unneeeded ci
* lint fix
* modify tx sizes
* cleanup docs
* Update abci/cmd/abci-cli/abci-cli.go
Co-authored-by: Callum Waters <cmwaters19@gmail.com >
* Update docs/app-dev/getting-started.md
Co-authored-by: Callum Waters <cmwaters19@gmail.com >
* Update docs/app-dev/getting-started.md
Co-authored-by: Callum Waters <cmwaters19@gmail.com >
* bring back comment
* migrate to kvstore and not persistent
* remove unused func
* test persistent
Co-authored-by: Callum Waters <cmwaters19@gmail.com >
2021-07-12 14:55:32 +00:00
aa4854ff8f
docs: add docs file for the peer exchange ( #6665 )
2021-07-12 14:11:29 +02:00
581dd01d47
Update CODEOWNERS to include williambanfield ( #6683 )
2021-07-09 18:50:13 -04:00
50b00dff71
statesync: keep peer despite lightblock query fail ( #6692 )
...
When a peer responds with no lightblock for the height we queried, we call the [removePeer method](https://github.com/tendermint/tendermint/blob/master/internal/statesync/reactor.go#L339 ). This removes the peer from the [dispatcher's list of called peer's](https://github.com/tendermint/tendermint/blob/ad6588315256162a9ef799a5a77ab7237cd46f38/internal/statesync/dispatcher.go#L159 ). When the dispatcher then receives responses from the removed peer, it [drops their responses](https://github.com/tendermint/tendermint/blob/ad6588315256162a9ef799a5a77ab7237cd46f38/internal/statesync/dispatcher.go#L130 ). These responses may be meaningful or contain a block or data that will help statesync proceed.
[The logs](https://gist.github.com/tychoish/34a1f61eaae3c36c23efc7d0001e805c ), when this change is applied, show an additional 3 networking testnets passing.
addresses: #6691
2021-07-09 21:20:25 +00:00
051e127d38
light: correctly handle contexts ( #6687 )
2021-07-09 18:48:18 +02:00
5530726df8
tools: move tools.go to subdir ( #6689 )
...
## Description
Move tools to subdir to fix `go get`
2021-07-09 13:05:27 +00:00
decac693ab
p2p: remove annoying error log ( #6688 )
...
I put this error log in here because I thought it might be a helpful indicator to see when a reactor sends a message to a peer that doesn't have that channel open but it turns out this is happening all the time and it's kind of annoying
2021-07-09 12:48:33 +00:00
7ca0f24040
build(deps): Bump github.com/golangci/golangci-lint ( #6686 )
...
Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint ) from 1.38.0 to 1.41.1.
- [Release notes](https://github.com/golangci/golangci-lint/releases )
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md )
- [Commits](https://github.com/golangci/golangci-lint/compare/v1.38.0...v1.41.1 )
---
updated-dependencies:
- dependency-name: github.com/golangci/golangci-lint
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com >
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2021-07-09 13:47:01 +02:00
69848bef26
deps: run go mod tidy ( #6677 )
...
## Description
Run go mod tidy
2021-07-09 08:17:28 +00:00
2c14d491f6
fix leaking statesync test ( #6680 )
2021-07-08 15:26:35 +02:00
cd248576ea
e2e: remove colorized output from docker-compose ( #6670 )
2021-07-08 12:54:13 +00:00
c256edc622
fix evidence rpc test by extending wait time ( #6678 )
2021-07-08 14:43:41 +02:00
9d9360774f
adjust tx load ( #6681 )
2021-07-08 14:22:50 +02:00
William Banfield
William Banfield
Aleksandr Bezobchuk
Callum Waters
JayT106
Sam Kleinman
Marko
dependabot[bot]