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

Migrate go-block-format #58

Merged
merged 54 commits into from
Jan 26, 2023
Merged

Migrate go-block-format #58

merged 54 commits into from
Jan 26, 2023

Conversation

Jorropo
Copy link
Contributor

@Jorropo Jorropo commented Jan 26, 2023

Needed for ipfs/go-block-format#45

Todo:

jbenet and others added 30 commits June 26, 2014 01:14
This commit was moved from ipfs/go-block-format@c41fc61
This commit was moved from ipfs/go-block-format@1832342
link errors left:
- protocol buffers output is not lint-friendly


This commit was moved from ipfs/go-block-format@19745ca
dependencies are vendored into Godeps/_workspace and commit versions are
recorded in Godeps.json

update datastore to e89f0511
update go.crypto


This commit was moved from ipfs/go-block-format@058fa43
the u.Hash error can be safely ignored (panic) because multihash
only fails from the selection of hash function. If the fn + length
are valid, it won't error.

cc @whyrusleeping


This commit was moved from ipfs/go-block-format@a83b903
This commit was moved from ipfs/go-block-format@d1dd0be
- Modified Godeps/Godeps.json by hand
- [TEST] Updated welcome docs hash to sharness
- [TEST] Updated contact doc
- [TEST] disabled breaking test (t0080-repo refs local)


This commit was moved from ipfs/go-block-format@f3800ad
For the rest of the packages in util, move them to thirdparty
and update the references. util is gone!

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>


This commit was moved from ipfs/go-block-format@f079d17
…ore_log

Add log events when blocks are added/removed to/from the blockstore

This commit was moved from ipfs/go-block-format@3eec265
License: MIT
Signed-off-by: Kevin Atkinson <k@kevina.org>


This commit was moved from ipfs/go-block-format@92cc3f1
License: MIT
Signed-off-by: Kevin Atkinson <k@kevina.org>


This commit was moved from ipfs/go-block-format@1e01b02
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>


This commit was moved from ipfs/go-block-format@e6e5593
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@ipfs.io>


This commit was moved from ipfs/go-block-format@7614cda
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>


This commit was moved from ipfs/go-block-format@bf963fc
License: MIT
Signed-off-by: Jakub Sztandera <kubuxu@protonmail.ch>


This commit was moved from ipfs/go-block-format@4588bc6
…r-blocks

test: 82% coverage on blocks

This commit was moved from ipfs/go-block-format@3dc57fc
License: MIT
Signed-off-by: Jeromy <why@ipfs.io>


This commit was moved from ipfs/go-block-format@22ebb79
Rework go-ipfs to use content IDs version 0

This commit was moved from ipfs/go-block-format@4aeb40b
License: MIT
Signed-off-by: George Antoniadis <george@noodles.gr>


This commit was moved from ipfs/go-block-format@ea54261
whyrusleeping and others added 4 commits April 20, 2017 20:28
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>


This commit was moved from ipfs/go-block-format@57a3a5d
License: MIT
Signed-off-by: Zach Ramsay <zach.ramsay@gmail.com>


This commit was moved from ipfs/go-block-format@558aaac
License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>


This commit was moved from ipfs/go-block-format@7faeb7b
@Jorropo Jorropo self-assigned this Jan 26, 2023
@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Merging #58 (274ea52) into main (7a8eed3) will increase coverage by 0.00%.
The diff coverage is 64.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      ipfs/boxo#58   +/-   ##
=======================================
  Coverage   63.75%   63.75%           
=======================================
  Files          32       33    +1     
  Lines        1694     1719   +25     
=======================================
+ Hits         1080     1096   +16     
- Misses        477      485    +8     
- Partials      137      138    +1     
Impacted Files Coverage Δ
blocks/blocks.go 64.00% <64.00%> (ø)

Copy link
Member

@hacdias hacdias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All for it!

@Jorropo Jorropo merged commit 38b7ae8 into main Jan 26, 2023
@github-actions
Copy link

Suggested version: v0.3.0
Comparing to: v0.2.0 (diff)

Changes in go.mod file(s):

diff --git a/go.mod b/go.mod
index 7f40a640..946a7c14 100644
--- a/go.mod
+++ b/go.mod
@@ -7,6 +7,7 @@ require (
 	github.com/crackcomm/go-gitignore v0.0.0-20170627025303-887ab5e44cc3
 	github.com/gorilla/mux v1.8.0
 	github.com/ipfs/go-cid v0.3.2
+	github.com/ipfs/go-ipfs-util v0.0.2
 	github.com/ipfs/go-ipns v0.3.0
 	github.com/ipfs/go-log/v2 v2.5.1
 	github.com/libp2p/go-libp2p v0.23.4
@@ -24,7 +25,6 @@ require (
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/decred/dcrd/dcrec/secp256k1/v4 v4.1.0 // indirect
 	github.com/gogo/protobuf v1.3.2 // indirect
-	github.com/ipfs/go-ipfs-util v0.0.2 // indirect
 	github.com/ipld/go-ipld-prime v0.9.0 // indirect
 	github.com/klauspost/cpuid/v2 v2.1.1 // indirect
 	github.com/libp2p/go-buffer-pool v0.1.0 // indirect

gorelease says:

# github.com/ipfs/go-libipfs/blocks
## compatible changes
package added

# summary
Cannot suggest a release version.
Can only suggest a release version when compared against the most recent version of this major: v0.3.0.

gocompat says:

(empty)

@Jorropo Jorropo deleted the migrate-block-format branch January 26, 2023 15:12
Jorropo added a commit to ipfs/kubo that referenced this pull request Jan 26, 2023
Jorropo added a commit to ipfs/kubo that referenced this pull request Jan 26, 2023
Jorropo added a commit to Jorropo/lotus that referenced this pull request Jan 26, 2023
Jorropo added a commit to Jorropo/lotus that referenced this pull request Jan 26, 2023
Jorropo added a commit to ipfs/kubo that referenced this pull request Jan 26, 2023
Jorropo added a commit to Jorropo/lotus that referenced this pull request Jan 26, 2023
hacdias pushed a commit that referenced this pull request Jan 27, 2023
adlrocha added a commit to consensus-shipyard/lotus that referenced this pull request Feb 16, 2023
* Test and fix eth_FeeHistory

* Add test for eth_getStorageAt

* Add test for eth_call

* Change EthBlock.Extradata type to EthBytes

* fix: EthAPI: Drop hack in GetStorageAt

* Eth JSON-RPC: implement web3_clientVersion

* fix lint

* add genesis for ipc gateway

* Remove debug logging

* chore: cli

Standarize cli/code functions similar to: filecoin-project/lotus#9317

- cctx.NArg() instead of cctx.Args().xxx
- Add check for args and print help on functions that did not have it

* fix: rpcenc: Don't hang when source dies

* make docsgen-cli

make docsgen-cli

* fix: EthAPI: Handle EthCall return correctly

* feat: vm: Assert empty object CID when dumping state

* Update codeql to v2

* Enable code ql for release branches

* less strict ArgsCheck

Less strict ArgsCheck

* feat: wdpost: Emit more detailed errors

* Fix panic in EthGetCode

* Update codeql to v2

* Enable code ql for release branches

* Add actors to circ-supply

- Consider funds to EAM as locked
- Add evm, placeholder and ethaccount to circ supply

* devnets starts with nv17

* Use new kit.DisableEthRPC method in test

* make gen

* init git submodules in build-docker CI

* Remove old Snapcraft and AppDir folders

* Always include checked in files in docker context

The .dockerignore file is symlinked to the .gitignore file, and checked
in files should not be removed from our docker context otherwise they
result in dirty git state when we build our images.

* Check git state in Dockerfile to catch .dockerignore bugs

* make jen

- Add builtin.EthereumAddressManagerActorAddr to builtin.go.template and make jen
- Rename to EthereumAddressManagerActorAddr to match pattern of other actors (CronActorAddr/etc)

* chore: all: bump go-libipfs to replace go-block-format

Includes changes from:
- ipfs/go-block-format#37
- ipfs/boxo#58

* init git submodules in build-docker CI

* Remove old Snapcraft and AppDir folders

* Always include checked in files in docker context

The .dockerignore file is symlinked to the .gitignore file, and checked
in files should not be removed from our docker context otherwise they
result in dirty git state when we build our images.

* Check git state in Dockerfile to catch .dockerignore bugs

* Fix lint errors

* Skip some rpc conformance tests

* make gen

* Changed skip message

* fix: don't replace the method in EthSendRawTransaction (#10129)

This will just cause signature validation to fail.

* Remove stm: #integration comment

* Fix comment

* fix: devnets: default to starting from nv17

* Refactor to ensure conformance test can run in circleci

* go fmt

* Standardize path variable

* Update to go-state-types v0.10.0-alpha-11

* chore: node: migrate go-bitswap to go-libipfs/bitswap

This was migrated in ipfs/boxo#63.

* simplify transaction hash db queries, prevent fd leaks

* Refactor: Unify EthTx to FilecoinMessage methods v2 (#10095)

* Refactor: Unify EthTx to FilecoinMessage methods

* Filecoin messages can again be converted to Eth Txs

* All BLS messages should calculated tx hash with unsigned message

* Refactor newEthTxReceipt

* fill in from and to for non-eth transactions

* Hoist nil check out of newEthTxFromMessageLookup

---------

Co-authored-by: Aayush <arajasek94@gmail.com>
Co-authored-by: Raúl Kripalani <raul@protocol.ai>

* Populate transaction hash database if the database has not been set up before

* rpc: Switch eth_subscribe to reverse calls

* gateway: eth_subscribe support

* itests: Fix TestEthSubscribeLogs

* ethtypes: Serialize EthFilterID/EthSubscriptionID correctly

* eth rpc: Params are optional in eth_subscribe

* fix: cli: add ArgsUsage

fix: cli: add ArgsUsage

* make libp2p user agent overridable

* move UserAgentOption upwards

* Eth JSON-RPC: from in eth_getTransactionByHash is not correctly populated #1614

* Move assigning `from` address to the API level

* Improve the error message

* Add test for EthGetTransactionByHash

* fix: should not serve non v0 api in v1

* Fix: typo

Fix: typo

* Fix: typos

Fix: typos

* Fix: typo

Fix: typo

* Fix: typo

Fix: typo

* Fix: typo

Fix: typo

* Fix: typos

Fix: typos

* Fix: typos

Fix: typos

* test: fevm: add in tests for deploying, destroying contracts, recursive calls, sending value (#10082)

adds the following tests to itests/fevm_test.go:
 - recursive tests
 - delegate call tests
 - delegate call recursive tests
 - revert tests
 - destruct tests
 - contract deploy address tests
 - send value to contracts
 - gas limit on value transfer tests
 - sending value to destroyed contracts
adds the test to itests/fevm_address_test.go:
 - deploy contract and confirm address is different second deploy

* merge

* tests that use create2 and destroy to validate evm state

* wip: ipc_gateway in genesis

* deps: Update go-jsonrpc to v0.2.1

* Test: assert all fields returned from EthGetTransactionByHash have expected values

Related to  filecoin-project/lotus#10151 (comment)

* Fix merge conflicts

* Allow f4 address to send to all address types if ID address exists on chain

* fix: extend LOTUS_CHAIN_BADGERSTORE_DISABLE_FSYNC to the markset

Without doing this walking a badger markset on a non-nvme knocks the
node hopelessly out of sync during a compaction.

* review fixes

* gateway: Support all EthModule methods

* node builder: Use gateway eth module in lite mode

* feat: ethrpc: Support filtering by address in subscribe

* itests: Fix TestEthFilterAPIDisabledViaConfig

* Check decoding params for new methods

* Review fixes

* feat: evm: align events implementation with FIP-0049 and FIP-0054. (#10152)

- Event keys are now t1, t2, t3, t4 for topics; and d for data.
- ref-fvm no longer stores events in the blockstore for us. It just
  returns events to the client, who is now responsible for handling
  them as it wishes / according to its configuration.
- Add a flag to VMOpts to have the events AMT be written in the blockstore.
- Add a flag to the ChainStore to advertise to the rest of the system
  if the ChainStore is storing events.
- Enable that flag if the EthRPC is enabled (can also add an explicit
  configuration flag if wanted).

* cli: fix extend cmd to get the right sector number

* todo: rebase master to include release/v1.20.0

* feat: compute a better gas limit for recursive external contract calls

* make gen

* fix: ethtypes: Correct 'no uncles' hash in NewEthBlock

* LOTUS_FEVM_ENABLEETHRPC: Fix env variable name in error

* fix: ethtypes: Correct 'no transactions' hash in NewEthBlock

* fix: null rounds: pass correct timestamp to the FVM.

* itest: fix FEVM tests for upstream changes

* itest: fix: test comment

* itest: fix remaining fevm failures

* improve evm error handling in itests (#10161)

* use WithValue language for test

* clean up test for recursive delegate call count. improved readability (#10195)

* chain: explicitly check that gasLimit is above zero

* add bundle git tag from pack.sh into builtin_actors_gen

* fix: worker: add all task type flag

Add all flag for the `lotus-worker tasks enable/disable` cmd

* Update cmd/lotus-worker/tasks.go

Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>

* Update cmd/lotus-worker/tasks.go

Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>

* Update cmd/lotus-worker/tasks.go

Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>

* make docsgen-cli

make docsgen-cli

* retry make docsgen-cli

retry make docsgen-cli with the new usage

* remove bundle-gen from make gen - not actually useful today anyway

* fvm: chore: update FVM

This:

1. Updates the builtin actors bundle (for actors v10).
2. Updates the event entry type to include the codec.
3. Removes the cbor encoding and zero trimming from event data.

I've chose to:

1. _Not_ add codec handling to the event filtering system for now.
2. _Skip_ events with unexpected codecs.

We don't actually _allow_ these events in the FVM right now, and it
simplifies the implementation.

However, I _am_ recording the codecs in the database so we don't have to
migrate it later.

* Option to set-seal-delay seconds

Add the option to set-seal-delay in seconds

* make docsgen-cli

make docsgen-cli

* Error if backup file exists

Error out if backup file already exists

* fix: ethrpc: Don't send sub notifs in array

* Update eth_subscribe itests to work with correct responses

* test: build: ignore git tags in embedded metadata

* fix: build: use actual v9 bundles for butterfly

* fix: ethrpc: emit correct bloom filters (#10194)

* skip problematic test vectors.

* feat: actors: Add bytecode getter

* itests: Test EVM bytecode getters, check bytecode hash

* ethrpc: Add missing fields to EthTx

* Update cli/backup.go

Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>

* feat: eth cli: Strip out empty spaces around contract bytes

* fix: chain: make sure the head is empty, not the code

* bump test vectors.

* remove test vectors bleeding edge job.

We no longer need it because specs-actors is deprecated.
v7 vectors have been merged to master.

* fix: Don't call WalletExport in msg signing flows

* updates butterflynet reset artifacts

* bootstrap node multiaddr
* new genesis file

these files are necessary for others to build lotus and join the new butterflynet, reset on February 08, 2023

* fix: gas: update ffi & correct the message inclusion cost in nv18 (#10228)

Co-authored-by: Raúl Kripalani <raul@protocol.ai>

* refactor: use EthHash for event topics

This ensures they're always 32 bytes and padded, as required.

* fix: eth: strict event parsing

We now enforce the following rules:

1. No duplicate topics or data.
2. Topics must have 32 byte keys.
3. Topics may not be skipped. (e.g., no t1 & t3 without a t2).
4. Raw codecs.

We _don't_ require that topics/data be emitted in any specific order.

We _skip_ events with unknown keys. We _drop_ events that violate the
above rules.

* Apply suggestions from code review

Co-authored-by: raulk <raul@protocol.ai>

* fix: eth: log on unexpected events

We can remove these later as we add more event types, but this will aid
in debugging.

* fix: make gen

* fix config.yml

* implement itest and handle optional params

* fix typo

* fix: stmgr: make the tipset and height agree when estimating gas (#10216)

* fix: stmgr: make the tipset and height agree when estimating gas

Specifically re-execute all messages in the current tipset, tacking the new
message onto the end. That way, the epoch is the epoch of the current tipset.

We could try to "make" a fake block and use that, but that's unlikely to
work well.

* fix: stmgr: only apply tipset messages for CallWithGas

* fix: itest: window post dispute

* eth_feeHistory: parse block param correctly.

* add todo

* Update to FFI v1.20.0-rc1

* fix bad test.

* improve TODO.

* Update to go-state-types v0.10.0-rc2

* disable adding git tags to bundle metadata

* Update actors to v10.0.0-rc.1

* eth: FIP-0055: implement final version of transitory delegated signature. (#10239)

* make lint happy and re-generate devgen.car

* setup ipc gateway in genesis and assign initial balance

* override with ipc-actors type to load ipc bundle automatically

* set balance of reward actor according to network name and minor fix

* linter fixes

* make linter happy :)

* deploy ipc subnet actor

* fix config.yml

---------

Co-authored-by: Ian Davis <nospam@iandavis.com>
Co-authored-by: Aayush <arajasek94@gmail.com>
Co-authored-by: ychiao <ychiaoli18@users.noreply.github.com>
Co-authored-by: cryptoAtwill <willes.lau@protocol.ai>
Co-authored-by: Phi <orjan.roren@gmail.com>
Co-authored-by: Łukasz Magiera <magik6k@gmail.com>
Co-authored-by: Geoff Stuart <geoff.vball@gmail.com>
Co-authored-by: Jiaying Wang <42981373+jennijuju@users.noreply.github.com>
Co-authored-by: Geoff Stuart <geoffrey.stuart@protocol.ai>
Co-authored-by: Łukasz Magiera <magik6k@users.noreply.github.com>
Co-authored-by: Jennifer Wang <jiayingw703@gmail.com>
Co-authored-by: Ian Davis <jungziege@gmail.com>
Co-authored-by: Jorropo <jorropo.pgm@gmail.com>
Co-authored-by: Steven Allen <steven@stebalien.com>
Co-authored-by: Raúl Kripalani <raul@protocol.ai>
Co-authored-by: Richard Guan <richard.guan@protocol.ai>
Co-authored-by: Maciej Witowski <maciej.witowski@protocol.ai>
Co-authored-by: omahs <73983677+omahs@users.noreply.github.com>
Co-authored-by: snissn <snissn@gmail.com>
Co-authored-by: Peter Rabbitson <ribasushi@protocol.ai>
Co-authored-by: 0x5459 <0x5459@protonmail.com>
Co-authored-by: Travis Person <travisperson@users.noreply.github.com>
Co-authored-by: Anton Evangelatov <anton.evangelatov@gmail.com>
Co-authored-by: ognots <ognots@protonmail.com>
Jorropo added a commit to Jorropo/lotus that referenced this pull request Feb 17, 2023
hannahhoward pushed a commit to filecoin-project/kubo-api-client that referenced this pull request Jun 19, 2023
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.

None yet