Skip to content

Data further refactor #10910

Data further refactor

Data further refactor #10910

Triggered via pull request December 30, 2024 20:11
Status Failure
Total duration 7m 11s
Artifacts

merge.yml

on: pull_request
build
3s
build
Unused dependency check
7m 0s
Unused dependency check
lint
23s
lint
various checks
7s
various checks
Large file upload
5m 16s
Large file upload
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

30 errors and 8 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 (ubuntu-latest)
Process completed with exit code 101.
Unit Tests (macos-latest)
The job was canceled because "ubuntu-latest" failed.
Unit Tests (macos-latest)
Process completed with exit code 101.
Unit Tests (windows-latest)
The job was canceled because "ubuntu-latest" failed.
Unit Tests (windows-latest)
The operation was canceled.
E2E tests (ubuntu-latest, /home/runner/.local/share/autonomi)
Process completed with exit code 127.
E2E tests (ubuntu-latest, /home/runner/.local/share/autonomi)
Process completed with exit code 101.
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
Process completed with exit code 127.
E2E tests (macos-latest, /Users/runner/Library/Application\ Support/autonomi)
Process completed with exit code 101.
Large file upload
Process completed with exit code 127.
Large file upload
Process completed with exit code 1.
Large file upload
Process completed with exit code 101.
Large file upload
Process completed with exit code 1.
Network churning tests (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\\autonomi\\node,...
The job was canceled because "macos-latest__Users_runne" failed.
E2E tests (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\\autonomi)
The job was canceled because "ubuntu-latest__home_runne" failed.
Network churning tests (ubuntu-latest, /home/runner/.local/share/autonomi/node, /home/runner/.loc...
The job was canceled because "macos-latest__Users_runne" failed.
Verify data location and Routing Table (windows-latest, C:\\Users\\runneradmin\\AppData\\Roaming\...
The job was canceled because "macos-latest__Users_runne" failed.
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 101.
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
various checks
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
Unit Tests (ubuntu-latest)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Network churning tests (macos-latest, /Users/runner/Library/Application Support/autonomi/node, /U...
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
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
Unused dependency check
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636