Merged
Conversation
This commit includes the following changes and progress:
1. **Build System Setup & Dependency Installation:**
* Successfully configured the CMake build environment.
* Installed missing system dependencies: `libfuse3-dev`, `libboost-dev`, `libboost-system-dev`, `libgrpc++-dev`, `protobuf-compiler-grpc`.
2. **Libsodium Build Fix:**
* Modified the root `CMakeLists.txt` to correctly build the `libsodium` library, which is fetched using `FetchContent`. This involved adding `execute_process` calls to run `configure`, `make`, and `make install` for libsodium.
* Corrected the `make` command for libsodium to properly use the number of available processors (`nproc`).
* Updated `tests/CMakeLists.txt` to ensure that FUSE test executables correctly include `SODIUM_INCLUDE_DIRS`.
3. **Project Build:**
* The project now builds successfully after the above fixes.
4. **Test Execution Issue (Unresolved):**
* Encountered a persistent issue where running any Google Test executable (either via CTest or directly) results in a timeout.
* This hang occurs even with a minimal GTest setup (`MinimalTest`) that includes only GTest itself and no project-specific code.
* Troubleshooting steps taken:
* Simplified `tests/tests_main.cpp` by commenting out custom initializations (Logger, KeyManager).
* Downgraded Google Test from v1.14.0 to release-1.10.0.
* Ensured the minimal test was correctly added to CMake and CTest.
* The root cause of the GTest hang could not be determined within the available time and tools, suggesting a deeper incompatibility with the testing environment or toolchain.
Due to the inability to run tests and gather feedback, further debugging of application-specific logic based on test failures was not possible. The primary focus shifted to resolving build issues and attempting to get the test framework operational.
…e-for-tests Fix var_dir path configuration
…th-node-name Fix node file collisions in tests
…1.47-and-reintroduce-features Restore AGENTS test guidelines
…es-for-metaserver-and-nodes feat: periodic state saves
…ion-failure Fix decryption crash in file verifier
…test Fix fuse random write file open
…glethreadtest-failure Fix fuse concurrency wrapper
…n-fuse-tests Fix SIMPLIDFS_CLUSTER_KEY handling in FUSE wrapper
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.