Skip to content

Commit

Permalink
[consensus] remove narwhal (#20617)
Browse files Browse the repository at this point in the history
## Description 

Goodbye Narwhal! 

There are some small cleanups that still need to be done in follow up
PRs
  • Loading branch information
arun-koshy authored Dec 13, 2024
1 parent 8a992f0 commit 5603060
Show file tree
Hide file tree
Showing 155 changed files with 66 additions and 33,737 deletions.
13 changes: 0 additions & 13 deletions .config/nextest.toml
Original file line number Diff line number Diff line change
Expand Up @@ -10,19 +10,6 @@ retries = 3
# Timeout tests after 4 minutes
slow-timeout = { period = "60s", terminate-after = 5 }

[profile.narwhalnightly]
# Print out output for failing tests as soon as they fail, and also at the end
# of the run (for easy scrollability).
failure-output = "immediate-final"
# Show skipped tests in the CI output.
status-level = "skip"
# Do not cancel the test run on the first failure.
fail-fast = false
# Do not retry failing tests
retries = 0
# Mark tests as slow after 7 hours, kill them right after
slow-timeout = { period = "7h", terminate-after = 1 }

[profile.simtestnightly]
# Print out output for failing tests as soon as they fail, and also at the end
# of the run (for easy scrollability).
Expand Down
12 changes: 4 additions & 8 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,10 @@ CHANGELOG.md
/crates/sui-graphql-rpc/ @oxade @amnn @wlmyng @stefan-mysten @emmazzz @suiwombat
/crates/sui-graphql-e2e-tests/ @oxade @amnn @wlmyng @stefan-mysten @emmazzz @suiwombat


# Notify Narwhal changes to interested folks.
/narwhal/crypto/ @joyqvq @kchalkias
/narwhal/executor/ @asonnino @akichidis @laura-makdah
/narwhal/network/ @bmwill
/narwhal/storage/ @akichidis

/crates/sui-core/src/consensus_adapter.rs @laura-makdah
# Consensus working group
/consensus/ @MystenLabs/consensus
/crates/sui-core/src/consensus_handler.rs @MystenLabs/consensus
/crates/sui-core/src/consensus_adapter.rs @MystenLabs/consensus

apps/ @MystenLabs/fe-and-apps
sdk/ @MystenLabs/sdk-reviewers
Expand Down
2 changes: 0 additions & 2 deletions .github/actions/diffs/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ runs:
- 'consensus/**'
- 'crates/**'
- 'external-crates/**'
- 'narwhal/**'
- 'sui-execution/**'
- '.github/workflows/cargo-llvm-cov.yml'
- '.github/workflows/rust.yml'
Expand Down Expand Up @@ -77,7 +76,6 @@ runs:
- 'docker/**'
- 'external-crates/**'
- 'kiosk/**'
- 'narwhal/**'
- 'nre/**'
- 'sui-execution/**'
isMoveAutoFormatter:
Expand Down
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ build
storage
!consensus/core/src/storage
!crates/sui-types/src/storage
!narwhal/storage

# Move-related files
Move.lock
Expand Down
Loading

0 comments on commit 5603060

Please sign in to comment.