feat: tracks block time using a gauge metric type #1090
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…1026) Bumps [github.com/celestiaorg/nmt](https://github.com/celestiaorg/nmt) from 0.16.0 to 0.17.0. - [Release notes](https://github.com/celestiaorg/nmt/releases) - [Commits](celestiaorg/nmt@v0.16.0...v0.17.0) --- updated-dependencies: - dependency-name: github.com/celestiaorg/nmt 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>
* feat: pass chain-id in prepareProposal * chore: proto-gen --------- Co-authored-by: rachid <chamirachid1@gmail.com>
* pass software version from abci to node info * Update consensus/replay_test.go Co-authored-by: Rootul P <rootulp@gmail.com>
…1042) The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-ALPINE37-MUSL-458286 - https://snyk.io/vuln/SNYK-ALPINE37-MUSL-458286 Co-authored-by: snyk-bot <snyk-bot@snyk.io>
…1039) Bumps [github.com/celestiaorg/nmt](https://github.com/celestiaorg/nmt) from 0.17.0 to 0.18.0. - [Release notes](https://github.com/celestiaorg/nmt/releases) - [Commits](celestiaorg/nmt@v0.17.0...v0.18.0) --- updated-dependencies: - dependency-name: github.com/celestiaorg/nmt 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> Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
## Description Closes #1040 After merging this one, I will cherry-pick this change for main. Then, I will open a separate PR for main to remove the square size and the data hash from the `Data` struct. --- #### PR checklist - [ ] Tests written/updated - [ ] Changelog entry added in `.changelog` (we use [unclog](https://github.com/informalsystems/unclog) to manage our changelog) - [ ] Updated relevant documentation (`docs/` or `spec/`) and code comments
## Description leaving as a draft and WIP until I test this out more. not ready for review closes #1004 --------- Co-authored-by: Rootul P <rootulp@gmail.com>
…ensus (#1055) ## Description This PR updates the trace client with a slight refactor (renaming to be more consistent) and then adds a few standard tables to store traced information. We are also able to select which tables we wish to update (and therefore which information to trace) in the config. closes #978 --------- Co-authored-by: Rootul P <rootulp@gmail.com>
## Description since its difficult to test the tracer irl, we missed this silly bug :facepalm:
## Description Fixes celestiaorg/orchestrator-relayer#432 #### PR checklist - [ ] Tests written/updated - [ ] Changelog entry added in `.changelog` (we use [unclog](https://github.com/informalsystems/unclog) to manage our changelog) - [ ] Updated relevant documentation (`docs/` or `spec/`) and code comments --------- Co-authored-by: Rootul P <rootulp@gmail.com>
## Description Fixes: celestiaorg/celestia-app#2249 #### PR checklist - [ ] Tests written/updated - [ ] Changelog entry added in `.changelog` (we use [unclog](https://github.com/informalsystems/unclog) to manage our changelog) - [ ] Updated relevant documentation (`docs/` or `spec/`) and code comments
## Description To accurately gauge block time, it's essential to track the timestamps of committed blocks. The modifications made in this PR incorporate this feature. Incremental work toward #1056
## Description adds more things to the block table, see #1062 (comment) for further context. part of #1056 --------- Co-authored-by: Sanaz Taheri <35961250+staheri14@users.noreply.github.com>
## Motivation #1073 (comment)
## Description Adds the last commit median time to `RequestPreparePropsosal`. This allows the application to accurately calculate the spendable funds of vesting accounts, since this is the time used when executing the transactions and when evaluating the propsoal during `ProcessProposal`. The height was also added, but it doesn't have to be since the application can calculate that value. That value is also passed in other versions of comet, and we'll have to add it we want to unfork from the sdk so I figured why not. Happy to remove it as well.
Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Thane Thomson <connect@thanethomson.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: Sergio Mena <sergio@informal.systems> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com> Co-authored-by: Lasaro <lasaro@informal.systems> Co-authored-by: Thane Thomson <connect@thanethomson.com> Co-authored-by: mmsqe <mavis@crypto.com> Co-authored-by: yihuang <huang@crypto.com> Co-authored-by: Steven Ferrer <steven.r.ferrer@gmail.com> Co-authored-by: Chill Validation <92176880+chillyvee@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.