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

Last changes from NeonLabs #909

Merged
merged 22 commits into from
Oct 27, 2023
Merged

Conversation

afalaleev
Copy link
Contributor

No description provided.

andreisilviudragnea and others added 22 commits October 27, 2023 07:47
* Remove #[allow(dead_code)]

* NDEV-2184: Report error when requested too early slot
Remove emulate-hash, trace-hash and trace-next-block.
Delete everything related to accessing indexer database.
* NDEV-2183: Implement get_sync_status for eth_syncing
* added pr_url as param

* fix param name

* Update pipeline.yml
The main purpose of this PR is to remove usages of [`block_in_place`](https://docs.rs/tokio/latest/tokio/task/fn.block_in_place.html) and to re-use the same code in both an async (`neon-api`) and non-async (`evm-loader`) context.

- Replaced `#[cfg(feature = "tracing")]` with `#[cfg(target_os = "solana")]` and extended its usage to:
  - code from `evm-loader` crate used only as a library (`#[cfg(not(target_os = "solana")]`);
  - code from `evm-loader` crate used only as part of a Solana program (`#[cfg(target_os = "solana")]`).
- Replaced [`axum`](https://github.com/tokio-rs/axum) with [`actix-web`](https://github.com/actix/actix-web), because most of the `evm-loader` crate is based on non-`Send` types and [`actix-web`](https://github.com/actix/actix-web) is a web framework which does not require `Future`s to be `Send`. Inspired from #107.
- Used [`maybe_async`](https://github.com/fMeow/maybe-async-rs) crate to re-write common parts of `evm-loader` crate used both as an async library and as a non-async Solana program. This also removed all usages of [`block_in_place`](https://docs.rs/tokio/latest/tokio/task/fn.block_in_place.html).
- Simplified all types related to multi-threading in the context of non-`Send` `Future`s: less `Arc`s, less `RwLock`s.
* move tests from evm_loader folder

* chmod
* NDEV-2252: Use preserve_order serde_json feature

* NDEV-2252: Make StructLog and StructLoggerResult Geth-compatible

NDEV-2252: Fix StructLog field order

NDEV-2252: Fix StructLoggerResult field order

NDEV-2252: Output storage only on SLOAD and SSTORE opcodes

NDEV-2252: Output storage in hex format without 0x prefix

NDEV-2252: Output memory in hex format without 0x prefix

NDEV-2252: Add StructLoggerResult serialization tests

NDEV-2252: Fix StructLog format

NDEV-2252: Add github link

* NDEV-2252: Make tracer_config optional

* NDEV-2252: Remove unused trace Events

* NDEV-2252: Add more links to Geth structs
* Add 'write version' to EmulateRequestModel
* DOPS-420 image name variable added

* DOPS-420 workflow changed

- added variable with githab organisation and repo name for run link
- hardcoded proxy repo url changed to variable

* DOPS-420 github api script fix

added os lib
@afalaleev afalaleev self-assigned this Oct 27, 2023
@afalaleev afalaleev merged commit 68c0871 into neonevm:develop Oct 27, 2023
3 of 4 checks passed
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.

7 participants