Skip to content

feat(devnet-sdk): Add kurtosis data fetcher for externally deployed enclaves#14446

Merged
janjakubnanista merged 7 commits intodevelopfrom
jan/kt-native-data-fetcher
Feb 24, 2025
Merged

feat(devnet-sdk): Add kurtosis data fetcher for externally deployed enclaves#14446
janjakubnanista merged 7 commits intodevelopfrom
jan/kt-native-data-fetcher

Conversation

@janjakubnanista
Copy link
Copy Markdown
Contributor

Description

  • Adds a ktnative URL schema for devnet data fetching directly from an externally deployed enclave

Tests

Additional context

Metadata

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 42.00%. Comparing base (a4e716c) to head (6af44ee).
Report is 2 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (a4e716c) and HEAD (6af44ee). Click for more details.

HEAD has 11 uploads less than BASE
Flag BASE (a4e716c) HEAD (6af44ee)
2 1
contracts-bedrock-tests 2 0
cannon-go-tests-32 4 0
cannon-go-tests-64 4 0
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14446       +/-   ##
============================================
- Coverage    78.11%   42.00%   -36.12%     
============================================
  Files          178      859      +681     
  Lines        10667    78476    +67809     
============================================
+ Hits          8333    32966    +24633     
- Misses        2143    42615    +40472     
- Partials       191     2895     +2704     
Flag Coverage Δ
cannon-go-tests-32 ?
cannon-go-tests-64 ?
contracts-bedrock-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1029 files with indirect coverage changes

@janjakubnanista janjakubnanista self-assigned this Feb 20, 2025
@janjakubnanista janjakubnanista added A-devnet-sdk Area: devnet-sdk A-kt-devnet Area: kurtosis devnet labels Feb 20, 2025
@janjakubnanista janjakubnanista marked this pull request as ready for review February 20, 2025 21:47
@janjakubnanista janjakubnanista requested a review from a team as a code owner February 20, 2025 21:47
@janjakubnanista janjakubnanista added this pull request to the merge queue Feb 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 24, 2025
@janjakubnanista janjakubnanista added this pull request to the merge queue Feb 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 24, 2025
@janjakubnanista janjakubnanista force-pushed the jan/kt-native-data-fetcher branch from 4b1b09d to f089563 Compare February 24, 2025 16:28
@janjakubnanista janjakubnanista added this pull request to the merge queue Feb 24, 2025
Merged via the queue into develop with commit b1ff39c Feb 24, 2025
46 checks passed
@janjakubnanista janjakubnanista deleted the jan/kt-native-data-fetcher branch February 24, 2025 16:55
Rjected pushed a commit to paradigmxyz/optimism that referenced this pull request Feb 25, 2025
…nclaves (ethereum-optimism#14446)

* wip: ktnative data fetcher

* wip: ktnative data fetcher

* fix: Only accept absolute paths

* chore: Add tests

* chore: inject dependencies instead of using globals

* chore: Comment fixes

* fix: Lint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-devnet-sdk Area: devnet-sdk A-kt-devnet Area: kurtosis devnet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants