Skip to content

Data further refactor #10912

Data further refactor

Data further refactor #10912

Triggered via pull request December 30, 2024 22:51
Status Failure
Total duration 10m 11s
Artifacts 5

merge.yml

on: pull_request
Matrix: Network churning tests
Matrix: E2E tests
Matrix: Unit Tests
Matrix: Verify data location and Routing Table
Fit to window
Zoom out
Zoom in

Annotations

24 errors and 12 warnings
various checks
Process completed with exit code 1.
lint
You have commit messages with errors ⧗ input: refactor: update self_encryption to use streaming API - Replace DataMapLevel with direct streaming API usage - Update client utils to use streaming_decrypt_from_storage - Add detailed implementation and schedule documentation - Clean up unused imports and improve error handling - Update archive.rs to work with new self_encryption API ✖ header must not be longer than 100 characters, current length is 335 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor: remove WASM support - Remove WASM dependencies and crate-type - Remove WASM test file and documentation - Update README to remove WASM references ✖ header must not be longer than 100 characters, current length is 155 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: fix: improve local network testing - Require only one peer when in local mode instead of CLOSE_GROUP_SIZE - Remove global address check for local testing - Fix test configuration to properly use local feature - Clean up unused imports and variables ✖ header must not be longer than 100 characters, current length is 248 [header-max-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor(network): Simplify LocalNode implementation - Refactor LocalNode to use tokio async/await consistently - Add proper timeouts and cleanup in start/stop methods - Improve process handling with kill_on_drop - Remove unused peer discovery and event handling - Clean up imports and remove warnings - Add comprehensive test with proper cleanup ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor: make local mode runtime configurable - Remove local feature flag dependency, making it a runtime configuration - Ensure bootstrap cache is not written to in local mode - Fix builder pattern usage in NetworkBuilder - Add proper config handling in deprecated connect method - Update tests to work with runtime local mode configuration This change makes the local mode behavior more explicit and configurable at runtime, rather than being a compile-time feature flag. It also ensures that when in local mode, we don't pollute the bootstrap cache with local-only peers. ✖ body's lines must not be longer than 100 characters [body-max-line-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor: improve network interface detection and QUIC support - Update find_local_ip to use UDP sockets for QUIC protocol\n- Add comprehensive IP validation (non-loopback, private range, etc.)\n- Update multiaddr format to use QUIC instead of TCP\n- Fix error handling in network code and tests\n- Add test coverage for IP validation and QUIC multiaddr format\n\nThis change ensures we use the correct network interface for QUIC connections and validates the IP address meets all requirements for P2P networking. ✖ body's lines must not be longer than 100 characters [body-max-line-length] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor: Update local network to use find_local_ip() and QUIC ✖ subject must not be sentence-case, start-case, pascal-case, upper-case [subject-case] ✖ found 1 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: refactor: improve error handling and test infrastructure This commit enhances error handling and test infrastructure across the codebase: Error Handling: - Add Serialization variant to CostError for consistent error handling - Fix error conv
Unit Tests (macos-latest)
Process completed with exit code 101.
Unit Tests (windows-latest)
The job was canceled because "macos-latest" failed.
Unit Tests (windows-latest)
The operation was canceled.
Unit Tests (ubuntu-latest)
The job was canceled because "macos-latest" failed.
Unit Tests (ubuntu-latest)
The operation was canceled.
Verify data location and Routing Table (ubuntu-latest, /home/runner/.local/share/autonomi/node, /...
The job was canceled because "macos-latest__Users_runne" failed.
Unused dependency check
Process completed with exit code 1.
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
Process completed with exit code 101.
Verify data location and Routing Table (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\...
The job was canceled because "macos-latest__Users_runne" failed.
E2E tests (ubuntu-latest, /home/runner/.local/share/autonomi)
The job was canceled because "macos-latest__Users_runne" failed.
E2E tests (ubuntu-latest, /home/runner/.local/share/autonomi)
The operation was canceled.
E2E tests (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\\autonomi)
The job was canceled because "macos-latest__Users_runne" failed.
Network churning tests (macos-latest, /Users/runner/Library/Application Support/autonomi/node, /U...
The job was canceled because "ubuntu-latest__home_runne" failed.
Network churning tests (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\\autonomi\\node,...
The job was canceled because "ubuntu-latest__home_runne" failed.
Large file upload
Process completed with exit code 1.
Large file upload
Process completed with exit code 1.
various checks
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Verify data location and Routing Table (macos-latest, /Users/runner/Library/Application Support/a...
jq 1.7.1 is already installed and up-to-date. To reinstall 1.7.1, run: brew reinstall jq
Verify data location and Routing Table (macos-latest, /Users/runner/Library/Application Support/a...
jq 1.7.1 is already installed and up-to-date. To reinstall 1.7.1, run: brew reinstall jq
Verify data location and Routing Table (macos-latest, /Users/runner/Library/Application Support/a...
ripgrep 14.1.1 is already installed and up-to-date. To reinstall 14.1.1, run: brew reinstall ripgrep
Unused dependency check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
jq 1.7.1 is already installed and up-to-date. To reinstall 1.7.1, run: brew reinstall jq
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
jq 1.7.1 is already installed and up-to-date. To reinstall 1.7.1, run: brew reinstall jq
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
ripgrep 14.1.1 is already installed and up-to-date. To reinstall 14.1.1, run: brew reinstall ripgrep
Large file upload
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Artifacts

Produced during runtime
Name Size
safe_test_logs_churn_ubuntu-latest
495 KB
safe_test_logs_data_location_macos-latest
441 KB
safe_test_logs_e2e_macos-latest
655 KB
safe_test_logs_e2e_ubuntu-latest
578 KB
safe_test_logs_large_file_upload_no_ws_ubuntu-latest
3.79 MB