Skip to content

Conversation

@mlohvynenko
Copy link
Member

No description provided.

@mlohvynenko mlohvynenko marked this pull request as draft June 27, 2025 09:21
@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch 6 times, most recently from acdf6ce to a5b28ab Compare June 30, 2025 12:05
@mlohvynenko mlohvynenko changed the title [WIP] cm storage state cm: storage state interfaces Jun 30, 2025
@mlohvynenko mlohvynenko marked this pull request as ready for review June 30, 2025 12:06
@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch 9 times, most recently from c763e62 to f3f4dbe Compare July 2, 2025 09:05
@codecov
Copy link

codecov bot commented Jul 2, 2025

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 82.30%. Comparing base (28210cd) to head (91838a8).
Report is 7 commits behind head on develop.

Files with missing lines Patch % Lines
src/common/crypto/mbedtls/cryptoprovider.cpp 50.00% 1 Missing ⚠️
tests/include/mocks/filesystemmock.hpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #334      +/-   ##
===========================================
- Coverage    82.31%   82.30%   -0.02%     
===========================================
  Files          171      171              
  Lines        17357    17360       +3     
  Branches      2331     2333       +2     
===========================================
  Hits         14288    14288              
- Misses        3069     3072       +3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch from f3f4dbe to ea117ed Compare July 2, 2025 09:25
Copy link
Contributor

@mykola-kobets-epam mykola-kobets-epam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Mykola Kobets <[email protected]>

@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch 8 times, most recently from d900fef to 783efc5 Compare July 3, 2025 13:17
@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch 2 times, most recently from 1a6d279 to 80a296c Compare July 3, 2025 15:55
Copy link
Collaborator

@al1img al1img left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Oleksandr Grytsov <[email protected]>

@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch 2 times, most recently from 2abf2cf to 9025a45 Compare July 8, 2025 14:37
Copy link

@MykolaSuperman MykolaSuperman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed-by: Mykola Solianko <[email protected]>

mlohvynenko and others added 7 commits July 8, 2025 19:17
This patch adds cloudprotocol state structures.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch adds storage state interfaces to the cm module.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch makes it possible to use instance ident
with STL containers that require a less operator.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch adds support for the SHA3-224 hash algorithm
in the crypto provider.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch enhances the platform filesystem interface
by adding a method to set user quota.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch removes the redundant definition of the SHA1 digest
size in the configuration header file.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
This patch adds a new communication interface to the cm module
with API needed for storage state module.

Signed-off-by: Mykhailo Lohvynenko <[email protected]>
Reviewed-by: Mykola Kobets <[email protected]>
Reviewed-by: Mykola Solianko <[email protected]>
Reviewed-by: Oleksandr Grytsov <[email protected]>
@mlohvynenko mlohvynenko force-pushed the feature_cm_storage_state branch from 9025a45 to 91838a8 Compare July 8, 2025 16:18
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jul 8, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
43.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@mlohvynenko mlohvynenko merged commit a5fd070 into aosedge:develop Jul 8, 2025
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants