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

Merge/foundation release/1.10.26 #510

Conversation

ziogaschr and others added 30 commits August 25, 2022 16:43
Error is "The ipc endpoint is longer than %d characters."
If IPC paths are too long, Unix systems will complain
because there are limits, albeit inconsistent ones.
So we try to keep IPC path names short (ie <100 chars).

I removed all instances where flags were used
in test names because
- they cause issues with apple silicon M1,
- and they are pointless

I also removed the redundant subtest naming scheme
which included the test name. Go includes the
test name by default anyway.

Date: 2022-08-29 09:21:05-05:00
Signed-off-by: meows <b5c6@protonmail.com>
Bumps dep to a tag version equivalent to the
previously referenced, untagged version.

Date: 2022-08-29 09:22:26-05:00
Signed-off-by: meows <b5c6@protonmail.com>
…s too

Date: 2022-08-31 19:55:29-07:00
Signed-off-by: meows <b5c6@protonmail.com>
…0.21-tests-generate

Merge/foundation release/1.10.21 tests generate
…0.21-tests-generate-etchash-mordor-pr-2

fix Etchash cache at Mordor
* graphql: fix tx logs

* minor

* Use optimized search for selecting tx logs
* graphql: fix tx logs

* graphql: refactor test service setup

* graphql: add test for tx logs
* params: set TerminalTotalDifficultyPassed to true

* Update params/config.go

Co-authored-by: Martin Holst Swende <martin@swende.se>
Bumps [joblib](https://github.com/joblib/joblib) from 1.0.0 to 1.2.0.
- [Release notes](https://github.com/joblib/joblib/releases)
- [Changelog](https://github.com/joblib/joblib/blob/master/CHANGES.rst)
- [Commits](joblib/joblib@1.0.0...1.2.0)

---
updated-dependencies:
- dependency-name: joblib
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
build(deps): bump joblib from 1.0.0 to 1.2.0
Signed-off-by: meows <b5c6@protonmail.com>
Date: 2022-10-25 12:22:04-07:00
Signed-off-by: meows <b5c6@protonmail.com>
Date: 2022-10-28 09:22:06-07:00
Signed-off-by: meows <b5c6@protonmail.com>
…5666)

* eth/protocols/snap: throttle trie heal requests when peers DoS us

* eth/protocols/snap: lower heal throttle log to debug

Co-authored-by: Martin Holst Swende <martin@swende.se>

* eth/protocols/snap: fix comment

Co-authored-by: Martin Holst Swende <martin@swende.se>
This PR changes the block field in the filter to be a pointer, to disambiguate between empty hash and no hash
Date: 2022-11-07 10:03:55-08:00
Signed-off-by: meows <b5c6@protonmail.com>
… interpreter when set

Date: 2022-11-07 10:04:36-08:00
Signed-off-by: meows <b5c6@protonmail.com>
@meowsbits
Copy link
Member

meowsbits commented Jan 4, 2023

  • gencodec w/r/t Genesis, GenesisSetup, genesis_account.go
  • EVMC; either keep the new (upgraded v10) feature, or pull the EVMC upgrade out and handle separately

…dd config type switch)

Date: 2023-01-10 08:11:37-08:00
Signed-off-by: meows <b5c6@protonmail.com>
… 1s sleep

They depend on the output of a geth instance.
Geth can take a while to start up, and assertions
about geth logs should normally use some kind of a hook
to decide when its ok to check the logs (eg startup complete).

Date: 2023-01-10 09:48:43-08:00
Signed-off-by: meows <b5c6@protonmail.com>
meowsbits and others added 7 commits January 10, 2023 10:36
Revering the EVMCv10 upgrade.
I can't get all tests to pass with EVM One.

Date: 2023-01-10 10:36:57-08:00
Signed-off-by: meows <b5c6@protonmail.com>
Date: 2023-01-10 10:38:13-08:00
Signed-off-by: meows <b5c6@protonmail.com>
Date: 2023-01-10 10:40:11-08:00
Signed-off-by: meows <b5c6@protonmail.com>
These tests keep failing... sometimes.
This is a shot in the dark.

Date: 2023-01-10 11:54:56-08:00
Signed-off-by: meows <b5c6@protonmail.com>
… in go-generate-check

We currently have a modified generated file which will cause this check to always file, until we actually fix the generated code, we decided to whitelist this file.

The file is: `params/types/genesisT/gen_genesis.go`
- skip protobuf installation as trezor code generation requires special handling
- whitelist `params/types/genesisT/gen_genesis.go` which is custom modified
@ziogaschr ziogaschr force-pushed the merge/foundation-release/1.10.26 branch from 00f182b to c4b7520 Compare January 11, 2023 15:19
ziogaschr and others added 5 commits January 11, 2023 18:32
I think this helps with readability.

Date: 2023-01-11 09:03:55-08:00
Signed-off-by: meows <b5c6@protonmail.com>
…om:etclabscore/core-geth into merge/foundation-release/1.10.26
…0.26-isaac-no-evmc

Merge/foundation release/1.10.26 <- Remove EVMCv10 upgrade, minor fixes
@meowsbits meowsbits force-pushed the merge/foundation-release/1.10.26 branch from adb6c00 to 5164696 Compare January 12, 2023 15:10
@ziogaschr ziogaschr force-pushed the merge/foundation-release/1.10.26 branch 2 times, most recently from 594483f to 8bb00d0 Compare January 13, 2023 21:10
…flows: go-generate-check changed logic for whitelisting changes in generated files directly

The intention of this commit is to not break/change the logic for developers when coding.
This script serves as a warning notification if a developer forgets to generate those files using gencodec.

On specific cases where generated files are modified, we revert the modifications and compare the files so as they will show up any changes in the original files.

The developer then will find out the correct way to resolve generating the new files and keeping the modifications intact if needed.
@ziogaschr ziogaschr force-pushed the merge/foundation-release/1.10.26 branch from 8bb00d0 to 9a48b9c Compare January 13, 2023 21:13
@ziogaschr
Copy link
Member Author

I suggest finishing our syncing tests and merge this back to master as it has been a long time here. What do you think @meowsbits?

@meowsbits
Copy link
Member

Yes, LGTM.

@ziogaschr
Copy link
Member Author

LGTM too, we can merge it.

@meowsbits meowsbits merged commit 92d5b25 into merge/foundation-release/1.10.23-tests-generate-fixes-statefix-difficulty-test-v2 Jan 19, 2023
@meowsbits meowsbits deleted the merge/foundation-release/1.10.26 branch January 19, 2023 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants