Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Capio v2 #113

Draft
wants to merge 86 commits into
base: master
Choose a base branch
from
Draft

Capio v2 #113

wants to merge 86 commits into from

Conversation

marcoSanti
Copy link
Collaborator

This pull request is aimed to activate the github actions for the v2 of CAPIO.

Marco Edoardo Santimaria and others added 25 commits October 2, 2024 10:04
Added optimization in read operations
Added file system monitor thread
WARNING: no mutually exclusive access is guaranteed and probably race
conditions exsists in the FileSystemMonitor class, but it is good enough
for now
Added doxygen documentation to the CAPIO codebase for most of the
server component and for some of the posix components.
Added a script to be called on the root directory to build and generate
the capio html documentation inside the doxy/ folder.
@marcoSanti marcoSanti marked this pull request as draft October 7, 2024 12:10
Copy link

codecov bot commented Oct 7, 2024

Codecov Report

Attention: Patch coverage is 50.44444% with 892 lines in your changes missing coverage. Please review.

Project coverage is 55.57%. Comparing base (ed9902d) to head (55de1b6).

Files with missing lines Patch % Lines
src/server/capio-cl-engine/json_parser.hpp 2.15% 180 Missing and 138 partials ⚠️
src/server/capio-cl-engine/capio_cl_engine.hpp 40.83% 74 Missing and 81 partials ⚠️
src/server/file-manager/file_manager_impl.hpp 19.44% 52 Missing and 93 partials ⚠️
src/server/utils/distributed_semaphore.hpp 0.00% 23 Missing and 12 partials ⚠️
src/posix/handlers/open.hpp 55.71% 9 Missing and 22 partials ⚠️
src/server/client-manager/handlers/consent.hpp 34.61% 3 Missing and 14 partials ⚠️
src/posix/utils/env.hpp 73.58% 6 Missing and 8 partials ⚠️
src/server/client-manager/handlers/read.hpp 38.09% 2 Missing and 11 partials ⚠️
...c/server/client-manager/request_handler_engine.hpp 78.33% 2 Missing and 11 partials ⚠️
src/common/capio/semaphore.hpp 45.45% 10 Missing and 2 partials ⚠️
... and 32 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   53.57%   55.57%   +1.99%     
==========================================
  Files          74       66       -8     
  Lines        3675     2834     -841     
  Branches     2265     1603     -662     
==========================================
- Hits         1969     1575     -394     
+ Misses        789      548     -241     
+ Partials      917      711     -206     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Marco Edoardo Santimaria and others added 13 commits October 18, 2024 07:13
This commit fixes the streaming functionality which has been broken by a previus commit. It also updates the simdjson dependency to the latest available stable release.
Fixed behavior on consent request for files not present in configuration file. Removed useless copy in Dockerfile that would break the docker container compilation stage
- Added class to provide storage functionality to CAPIO server component.
- Added  Unit tests for CapioMemoryFile
- Removed old tests that are not compatible with the current CAPIO implementation and cannot be ported

---------

Co-authored-by: Marco Edoardo Santimaria <[email protected]>
- Added class to provide storage functionality to CAPIO server component.
- Added  Unit tests for CapioMemoryFile
- Removed old tests that are not compatible with the current CAPIO implementation and cannot be ported

---------

Co-authored-by: Marco Edoardo Santimaria <[email protected]>
Marco Edoardo Santimaria and others added 8 commits December 11, 2024 15:20
- Added storage section in capio-cl json parser
- Added SPSCQueue initialization
- Capio memory file (#119)
- Added class to provide storage functionality to CAPIO server component.
- Added  Unit tests for CapioMemoryFile
- Removed old tests that are not compatible with the current CAPIO implementation and cannot be ported

---------

Co-authored-by: Marco Edoardo Santimaria <[email protected]>
- Changed order of CI tests
- Unit tests fir storage service
- General bug fixed and updates to CI/CD test order
- Fixed issue in handshake
- Cache fetching mechanism
- Send and recive files to store in memory
- Fixed potential shared mem res allocation
- Updated .gitub actions
- Fixed self interception error on shared memory unlink phased @ exit_handler
- Fixed issues on resource cleanup at termination
- Fixed minor typo
- Added tests on SPSCqueue and caches integration
- Added tests for data movement from posix to server
- Test with request
- Added seek tests on spscqueue and files
- Began work on Read cache for in-memory read
- Refactor of posix caches structure
- Work on ReadRequestCacheMem
- Began work on tests for ReadCacheMem
- Completed posix cache for memory file reads
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant