forked from maticnetwork/bor
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Bor v0.2.17 version bump: matic -> blocknative merge #22
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
Set default user to ubuntu in bor.service
…icnetwork#517) (maticnetwork#523) * rpc: add finalized and safe block types * ethclient: honour safe and finalized block requests * fix: remove BlockByNumberWithoutTx function
…acos disable macos tests
* added go and bash script to get config out of start.sh and updated flagset.go * changed 'requiredblocks' flag back to 'eth.requiredblocks' * updated script * changed 'requiredblocks' flag back to 'eth.requiredblocks' * updated tests, and removed requiredblocks from json and hcl * addressed comments
chg : generalised state-sync integration test
* remove setting maxpeers to 0 for nodiscover flag * set default prometheus and open-collector endpoint * skip building grpc address from pprof address and port * fix: linters * fix and improve tests * use loopback address for prometheus and open-collector endpoint * add logs for prometheus and open-collector setup * updated the script to handle prometheus-addr * updated builder/files/config.toml Co-authored-by: Pratik Patil <pratikspatil024@gmail.com>
Failing geth integration tests
Change heimdall branch to develop in CI
* base setup for miner opentel * version change * modify ctx passing * add attributes * update fill txs span attributes * fix: use common attributes for remote and local txs * pass context in seal * fix * fix * add traces to finalize and assemble * fix * use task ctx in result loop * only start parent span if no error * send nil tracer from Finalize * clean up * add sub function timings in span attribute * modify span attributes * set time attribute to milliseconds * linters fix * fix linters for consensus * add nolint to worker * fix testcase * Added fillTransactions subTraces * add traces in intermediate root hash function * add traces in WriteBlockAndSetHead function, fix linters * fix: linting errors * fix: test cases * fix: go.mod * fix: testcase * extract tracing package * linters fix * debug * Revert "debug" This reverts commit 2d68b7c. * fix: panic in NewTransactionsByPriceAndNonce iteration * change heimdall version to develop * miner/worker: fix duplicate call to tx ordering * miner/worker: refactor tracing * consensus/bor: use tracing package * tracing: add more abstraction for spans * tracing: set all attributes at once * remove nested tracing from blockchain.WriteBlockAndSetHead function * remove nested tracing from statedb.IntermediateRoot function * handle end span in bor.Seal function * fix: typo * minor fixes * fix: linters * fix: remove nolint * go mod tidy Co-authored-by: Manav Darji <manavdarji.india@gmail.com> Co-authored-by: Shivam Sharma <shivam691999@gmail.com> Co-authored-by: Evgeny Danienko <6655321@bk.ru>
* dev: add: pull request template
Fix Benchmark test
* dev: fix: ci * dev: fix: add default block configs * dev: fix: add sprintSize * dev: fix: revert * dev: fix: go-versions
* metrics: handle metrics flag from config in metrics.init() * internal/cli/server: log for metrics enabling and misconfiguration
…etwork#555) * changed default of maxpeers from 200 to 50 * docs: update additional notes for new-cli * docs: update additional notes for new-cli * small bug fix in the script to fix shopt issue Co-authored-by: Manav Darji <manavdarji.india@gmail.com>
jessenia
changed the title
Bor v0.2.17: matic -> blocknative merge
Bor v0.2.17 version bump: matic -> blocknative merge
Oct 21, 2022
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.
Prevents the nodes from accidentally migrating to snap sync mode. Bor does not support snap sync mode, yet and hence it’s hard to recover the nodes if they go into this mode. (eth, cli: prevent snap sync mode migration - v0.2.x maticnetwork/bor#495)
Regression fix for the whitelist/eth.requiredblocksflag and some nomenclature changes according to the latest geth release. (eth: fix whitelist/requiredblocks regression - v0.2.x maticnetwork/bor#497, cmd: rename requiredblocks flag maticnetwork/bor#505)