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.

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

feat: tracks block time using a gauge metric type #1090

Closed
wants to merge 22 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
f5f063c
build(deps): Bump github.com/celestiaorg/nmt from 0.16.0 to 0.17.0 (#…
dependabot[bot] Jun 28, 2023
67cc27b
feat: pass chain-id in prepareProposal (#1028)
evan-forbes Jul 4, 2023
fc8278c
fix: use same source of now (#1032)
cmwaters Jul 11, 2023
d24c81a
Revert "feat!: account for time already elapsed when waiting after th…
evan-forbes Jul 11, 2023
82c5411
docs: fix godoc for chain_id (#1034)
rootulp Jul 12, 2023
3e636bb
feat: add `Namespace` method to blob (#1030)
rootulp Jul 13, 2023
d5cd08c
feat: pass software version from abci to node info (#1036)
cmwaters Jul 14, 2023
c3ab251
feat: include software version as a label in all metrics (#1037)
cmwaters Jul 17, 2023
03b58f7
fix: networks/local/localnode/Dockerfile to reduce vulnerabilities (#…
MSevey Jul 24, 2023
8c9a9af
build(deps): Bump github.com/celestiaorg/nmt from 0.17.0 to 0.18.0 (#…
dependabot[bot] Jul 24, 2023
9dd1ac1
feat: remove square size from qgb commitment creation (#1047)
rach-id Jul 26, 2023
518e0e1
feat: initial pyroscope integration (#1006)
evan-forbes Aug 3, 2023
2f93fc8
feat: Add standard tracing points to portions of the mempool and cons…
evan-forbes Aug 8, 2023
61fb662
fix: pass configured tables to the trace client (#1057)
evan-forbes Aug 9, 2023
819b969
fix: end exclusive data commitment range fix (#1058)
rach-id Aug 10, 2023
f0446b2
chore(deps): bump pyroscope to v0.7.2 (#1061)
rach-id Aug 14, 2023
76145bc
feat: traces block height and timestamp for InfluxDB (#1062)
staheri14 Aug 17, 2023
9e02fe1
feat: add more block stats to the block trace table (#1064)
evan-forbes Aug 22, 2023
de9b4ec
build(deps): bump nmt to 0.19.0 (#1076)
rootulp Sep 6, 2023
074d981
feat: add time to `RequestPreparePropsosal` (#1081)
evan-forbes Sep 18, 2023
38e13ff
feat: adds channel ID label to the Prometheus message_receive_bytes_t…
staheri14 Sep 19, 2023
367caa3
chore: bump to v0.34.29 of comet (#1087)
cmwaters Sep 21, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
- `[node]` Close evidence.db OnStop ([cometbft/cometbft\#1210](https://github.com/cometbft/cometbft/pull/1210): @chillyvee)
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
- `[node]` Make handshake cancelable ([cometbft/cometbft\#857](https://github.com/cometbft/cometbft/pull/857))
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- `[state/kvindex]` Querying event attributes that are bigger than int64 is now
enabled. ([\#771](https://github.com/cometbft/cometbft/pull/771))
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
- `[pubsub]` Pubsub queries are now able to parse big integers (larger than
int64). Very big floats are also properly parsed into very big integers
instead of being truncated to int64.
([\#771](https://github.com/cometbft/cometbft/pull/771))
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- `[rpc]` Remove response data from response failure logs in order
to prevent large quantities of log data from being produced
([\#654](https://github.com/cometbft/cometbft/issues/654))
3 changes: 3 additions & 0 deletions .changelog/v0.34.29/security-fixes/788-rpc-client-pw.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- `[rpc/jsonrpc/client]` **Low severity** - Prevent RPC
client credentials from being inadvertently dumped to logs
([\#788](https://github.com/cometbft/cometbft/pull/788))
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- `[cmd/cometbft/commands/debug/kill]` **Low severity** - Fix unsafe int cast in
`debug kill` command ([\#794](https://github.com/cometbft/cometbft/pull/794))
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- `[consensus]` **Low severity** - Avoid recursive call after rename to
`(*PeerState).MarshalJSON`
([\#863](https://github.com/cometbft/cometbft/pull/863))
3 changes: 3 additions & 0 deletions .changelog/v0.34.29/security-fixes/890-mempool-fix-cache.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
- `[mempool/clist_mempool]` **Low severity** - Prevent a transaction from
appearing twice in the mempool
([\#890](https://github.com/cometbft/cometbft/pull/890): @otrack)
4 changes: 4 additions & 0 deletions .changelog/v0.34.29/summary.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
*June 14, 2023*

Provides several minor bug fixes, as well as fixes for several low-severity
security issues.
2 changes: 1 addition & 1 deletion .github/workflows/check-generated.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
with:
go-version: '1.19'

- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 1 # we need a .git directory to run git diff

Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/cometbft-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Prepare
id: prep
run: |
Expand All @@ -40,10 +40,10 @@ jobs:
platforms: all

- name: Set up Docker Build
uses: docker/setup-buildx-action@v2.5.0
uses: docker/setup-buildx-action@v3.0.0

- name: Build but do not Publish to Docker Hub
uses: docker/build-push-action@v3
uses: docker/build-push-action@v5
with:
context: .
file: ./DOCKER/Dockerfile
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
split-test-files:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Create a file with all the pkgs
run: go list ./... > pkgs.txt
- name: Split pkgs into 4 files
Expand Down Expand Up @@ -46,7 +46,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: "1.19"
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand All @@ -68,7 +68,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: "1.19"
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand All @@ -92,7 +92,7 @@ jobs:
runs-on: ubuntu-latest
needs: tests
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/e2e-manual.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
with:
go-version: '1.19'

- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Build
working-directory: test/e2e
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/e2e-nightly-34x.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
with:
go-version: '1.19'

- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
ref: 'v0.34.x-celestia'

Expand All @@ -49,7 +49,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Notify Slack on failure
uses: rtCamp/action-slack-notify@12e36fc18b0689399306c2e0b3e0f2978b7f1ee7
uses: rtCamp/action-slack-notify@b24d75fe0e728a4bf9fc42ee217caa686d141ee8
env:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
SLACK_CHANNEL: cometbft-engineering
Expand All @@ -65,7 +65,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Notify Slack on success
uses: rtCamp/action-slack-notify@12e36fc18b0689399306c2e0b3e0f2978b7f1ee7
uses: rtCamp/action-slack-notify@b24d75fe0e728a4bf9fc42ee217caa686d141ee8
env:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
SLACK_CHANNEL: cometbft-engineering
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: '1.19'
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/fuzz-nightly.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
with:
go-version: '1.19'

- uses: actions/checkout@v3
- uses: actions/checkout@v4

- name: Install go-fuzz
working-directory: test/fuzz
Expand Down Expand Up @@ -72,7 +72,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Notify Slack if any crashers
uses: rtCamp/action-slack-notify@12e36fc18b0689399306c2e0b3e0f2978b7f1ee7
uses: rtCamp/action-slack-notify@b24d75fe0e728a4bf9fc42ee217caa686d141ee8
env:
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
SLACK_CHANNEL: cometbft-engineering
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 8
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: actions/setup-go@v4
with:
go-version: '1.19'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/markdown-linter.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout Code
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Lint Code Base
uses: docker://github/super-linter:v4
env:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/pre-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
fetch-depth: 0

Expand Down Expand Up @@ -44,7 +44,7 @@ jobs:
echo "See the [CHANGELOG](${CHANGELOG_URL}) for changes available in this pre-release, but not yet officially released." > ../release_notes.md

- name: Release
uses: goreleaser/goreleaser-action@v4
uses: goreleaser/goreleaser-action@v5
with:
version: latest
args: release --clean --release-notes ../release_notes.md
Expand All @@ -57,7 +57,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Notify Slack upon pre-release
uses: slackapi/slack-github-action@v1.23.0
uses: slackapi/slack-github-action@v1.24.0
env:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
SLACK_WEBHOOK_TYPE: INCOMING_WEBHOOK
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/proto-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 5
steps:
- uses: actions/checkout@v3
- uses: bufbuild/buf-setup-action@v1.17.0
- uses: actions/checkout@v4
- uses: bufbuild/buf-setup-action@v1.26.1
- uses: bufbuild/buf-lint-action@v1
with:
input: 'proto'
2 changes: 1 addition & 1 deletion .github/workflows/release-version.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
check-version:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4

- uses: actions/setup-go@v4
with:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
fetch-depth: 0

Expand All @@ -26,7 +26,7 @@ jobs:
echo "See the [CHANGELOG](${CHANGELOG_URL}) for this release." > ../release_notes.md

- name: Release
uses: goreleaser/goreleaser-action@v4
uses: goreleaser/goreleaser-action@v5
with:
version: latest
args: release --clean --release-notes ../release_notes.md
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/testapp-docker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: Prepare
id: prep
run: |
Expand All @@ -42,17 +42,17 @@ jobs:
platforms: all

- name: Set up Docker Build
uses: docker/setup-buildx-action@v2.5.0
uses: docker/setup-buildx-action@v3.0.0

- name: Login to DockerHub
if: ${{ github.event_name != 'pull_request' }}
uses: docker/login-action@v2.1.0
uses: docker/login-action@v3.0.0
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Publish to Docker Hub
uses: docker/build-push-action@v4.0.0
uses: docker/build-push-action@v5.0.0
with:
context: .
file: ./test/e2e/docker/Dockerfile
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: "1.19"
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand Down Expand Up @@ -58,7 +58,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: "1.19"
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand Down Expand Up @@ -90,7 +90,7 @@ jobs:
- uses: actions/setup-go@v4
with:
go-version: "1.19"
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: technote-space/get-diff-action@v6
with:
PATTERNS: |
Expand Down
36 changes: 36 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,41 @@
# CHANGELOG

## v0.34.29

*June 14, 2023*

Provides several minor bug fixes, as well as fixes for several low-severity
security issues.

### BUG FIXES

- `[state/kvindex]` Querying event attributes that are bigger than int64 is now
enabled. ([\#771](https://github.com/cometbft/cometbft/pull/771))
- `[pubsub]` Pubsub queries are now able to parse big integers (larger than
int64). Very big floats are also properly parsed into very big integers
instead of being truncated to int64.
([\#771](https://github.com/cometbft/cometbft/pull/771))

### IMPROVEMENTS

- `[rpc]` Remove response data from response failure logs in order
to prevent large quantities of log data from being produced
([\#654](https://github.com/cometbft/cometbft/issues/654))

### SECURITY FIXES

- `[rpc/jsonrpc/client]` **Low severity** - Prevent RPC
client credentials from being inadvertently dumped to logs
([\#788](https://github.com/cometbft/cometbft/pull/788))
- `[cmd/cometbft/commands/debug/kill]` **Low severity** - Fix unsafe int cast in
`debug kill` command ([\#794](https://github.com/cometbft/cometbft/pull/794))
- `[consensus]` **Low severity** - Avoid recursive call after rename to
`(*PeerState).MarshalJSON`
([\#863](https://github.com/cometbft/cometbft/pull/863))
- `[mempool/clist_mempool]` **Low severity** - Prevent a transaction from
appearing twice in the mempool
([\#890](https://github.com/cometbft/cometbft/pull/890): @otrack)

## v0.34.28

*April 26, 2023*
Expand Down
2 changes: 1 addition & 1 deletion DOCKER/docker-entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ if [ ! -d "$CMTHOME/config" ]; then
-e "s/^proxy_app\s*=.*/proxy_app = \"$PROXY_APP\"/" \
-e "s/^moniker\s*=.*/moniker = \"$MONIKER\"/" \
-e 's/^addr_book_strict\s*=.*/addr_book_strict = false/' \
-e 's/^target_height_duration\s*=.*/target_height_duration = "1000ms"/' \
-e 's/^timeout_commit\s*=.*/timeout_commit = "500ms"/' \
-e 's/^index_all_tags\s*=.*/index_all_tags = true/' \
-e 's,^laddr = "tcp://127.0.0.1:26657",laddr = "tcp://0.0.0.0:26657",' \
-e 's/^prometheus\s*=.*/prometheus = true/' \
Expand Down
4 changes: 2 additions & 2 deletions abci/client/grpc_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import (
"context"

"google.golang.org/grpc"
"google.golang.org/grpc/credentials/insecure"

"github.com/tendermint/tendermint/abci/types"
cmtnet "github.com/tendermint/tendermint/libs/net"
Expand Down Expand Up @@ -88,8 +89,7 @@ func (cli *grpcClient) OnStart() error {

RETRY_LOOP:
for {
//nolint:staticcheck // SA1019 Existing use of deprecated but supported dial option.
conn, err := grpc.Dial(cli.addr, grpc.WithInsecure(), grpc.WithContextDialer(dialerFunc))
conn, err := grpc.Dial(cli.addr, grpc.WithTransportCredentials(insecure.NewCredentials()), grpc.WithContextDialer(dialerFunc))
if err != nil {
if cli.mustConnect {
return err
Expand Down
4 changes: 2 additions & 2 deletions abci/example/example_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import (
"github.com/stretchr/testify/require"

"google.golang.org/grpc"
"google.golang.org/grpc/credentials/insecure"

"golang.org/x/net/context"

Expand Down Expand Up @@ -148,8 +149,7 @@ func testGRPCSync(t *testing.T, app types.ABCIApplicationServer) {
})

// Connect to the socket
//nolint:staticcheck // SA1019 Existing use of deprecated but supported dial option.
conn, err := grpc.Dial(socket, grpc.WithInsecure(), grpc.WithContextDialer(dialerFunc))
conn, err := grpc.Dial(socket, grpc.WithTransportCredentials(insecure.NewCredentials()), grpc.WithContextDialer(dialerFunc))
if err != nil {
t.Fatalf("Error dialing GRPC server: %v", err.Error())
}
Expand Down
Loading
Loading