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

Bump Go version to v1.18 #368

Conversation

vcastellm
Copy link
Member

@linear
Copy link

linear bot commented Apr 6, 2022

POS-296 Bump Go version in Bor and Heimdall

According to https://groups.google.com/g/golang-announce/c/vtbMjE04kPk/m/xE-FGxCXCAAJ?pli=1, #golang 1.18.1 and 1.17.9 will be released tomorrow, with security patches. It is likely that we will have to issue a new #geth release with these patches (but we have no inside info about the issues).

temaniarpit27
temaniarpit27 previously approved these changes Apr 7, 2022
manav2401
manav2401 previously approved these changes Apr 7, 2022
@vcastellm vcastellm dismissed stale reviews from manav2401 and temaniarpit27 via dd75f38 April 13, 2022 14:17
@vcastellm vcastellm changed the base branch from master to v0.2.16-candidate April 13, 2022 14:18
@vcastellm vcastellm changed the title Bump Go version to v1.18.1 Bump Go version to v1.18 Apr 13, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 13, 2022

Codecov Report

Merging #368 (f4118fe) into v0.2.16-candidate (d5b76d4) will increase coverage by 0.00%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           v0.2.16-candidate     #368   +/-   ##
==================================================
  Coverage              56.79%   56.80%           
==================================================
  Files                    578      578           
  Lines                  68305    68305           
==================================================
+ Hits                   38797    38800    +3     
+ Misses                 26163    26155    -8     
- Partials                3345     3350    +5     
Impacted Files Coverage Δ
core/state_prefetcher.go 85.18% <0.00%> (-7.41%) ⬇️
rlp/raw.go 79.37% <0.00%> (-5.63%) ⬇️
p2p/discover/table.go 82.48% <0.00%> (-1.83%) ⬇️
core/rawdb/chain_iterator.go 58.38% <0.00%> (-1.25%) ⬇️
eth/downloader/skeleton.go 76.45% <0.00%> (-0.76%) ⬇️
trie/trie.go 76.47% <0.00%> (-0.74%) ⬇️
trie/proof.go 75.00% <0.00%> (-0.71%) ⬇️
p2p/simulations/http.go 66.02% <0.00%> (-0.65%) ⬇️
les/costtracker.go 64.02% <0.00%> (-0.53%) ⬇️
p2p/discover/v5_udp.go 80.68% <0.00%> (-0.53%) ⬇️
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5b76d4...f4118fe. Read the comment docs.

@vcastellm vcastellm merged commit bf85e15 into v0.2.16-candidate Apr 19, 2022
@vcastellm vcastellm deleted the vcastellm/pos-296-bump-go-version-in-bor-and-heimdall branch April 19, 2022 08:40
vcastellm pushed a commit that referenced this pull request May 6, 2022
* merge geth v1.10.15

* fix: Removed FastSync from cli server

* fix: TestHeadersRLPStorage

* Added t.skip(ETH2 in bor)

* fix: flow in create consensus engine

* bumped version

* Fix typo

* increase block time

* remove file

* bumped version

* merge gethv1.10.17

* bumped version

* fix failing tests

* Bump Go version to v1.18 (#368)

* Bump Go version to v1.18.1

* Build using netgo tag

This will create a static build using Go native networking stack.

Checked and it works stable for all archs and distros.

* Fix meta

* Bump version

* Meta as stable

Co-authored-by: Shivam Sharma <shivam691999@gmail.com>
Co-authored-by: Manav Darji <manavdarji.india@gmail.com>
Co-authored-by: Sandeep Sreenath <sandeep.sreenath@gmail.com>
Co-authored-by: Victor Castell <victor@victorcastell.com>
JekaMas added a commit that referenced this pull request Jun 15, 2022
* merge geth v1.10.15

* fix: Removed FastSync from cli server

* fix: TestHeadersRLPStorage

* Added t.skip(ETH2 in bor)

* fix: flow in create consensus engine

* bumped version

* Fix typo

* increase block time

* remove file

* bumped version

* merge gethv1.10.17

* bumped version

* fix failing tests

* Bump Go version to v1.18 (#368)

* Bump Go version to v1.18.1

* Build using netgo tag

This will create a static build using Go native networking stack.

Checked and it works stable for all archs and distros.

* Fix meta

* initial implementation for common ancestor approach

* extract whitelist interface

* fix types

* fix tests and format

* add unit tests for IsValidChain function

* more tests

* wip

* test ErrCheckpointMismatch

* minor fixes

* fix test

* dont panic

* fmt

* Limit state sync by gas

* Added logging for state-sync total gas usage

* Added number of event-records in log

* Minor Changes

* Minor Fix

* Adding individual gasUsed

* Minor Fix

* fix: return value for no remote block

* handle all errors

* modularise fake chain validator in downloader

* add more tests

* fix tests

* Modifying miner.recommit flag and its adjustment function. (#370)

* changed min/max/current recommit values

* Remove Hardcoded min/max

* Code Sanitization

* Skipping tests for constant recommit interval

* Adding default miner.recommit value

* Minor Change

* Increased default value of rpc.txfeecap to 5

* add debug rpc endpoints for checkpoint whitelist service

* minor fixes and enhancements

* avoid capping warnings for gas set by internal system transactions

* use typed mocks

* fix

* fix

* fix

* fix close

* fix

* Create stale.yml

* Fix bor consensus checkpoint bug

Co-authored-by: Arpit Temani <temaniarpit27@gmail.com>
Co-authored-by: Shivam Sharma <shivam691999@gmail.com>
Co-authored-by: Manav Darji <manavdarji.india@gmail.com>
Co-authored-by: Sandeep Sreenath <sandeep.sreenath@gmail.com>
Co-authored-by: Victor Castell <victor@victorcastell.com>
Co-authored-by: Ferran <ferranbt@protonmail.com>
Co-authored-by: Krishna Upadhyaya <krishnau1604@gmail.com>
Co-authored-by: Karlo <karlonovak@gmail.com>
Co-authored-by: Sandeep Sreenath <ssandeep@users.noreply.github.com>
Co-authored-by: Jerry <jerrycgh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants