Skip to content

Merge persistent_sessions and persistent_state#3630

Merged
roman-khimov merged 2 commits intomasterfrom
3425-merge-persistent_sessions-and-persistent_state
Oct 21, 2025
Merged

Merge persistent_sessions and persistent_state#3630
roman-khimov merged 2 commits intomasterfrom
3425-merge-persistent_sessions-and-persistent_state

Conversation

@End-rey
Copy link
Contributor

@End-rey End-rey commented Oct 10, 2025

Closes #3425.

@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 51.91257% with 88 lines in your changes missing coverage. Please review.
✅ Project coverage is 26.81%. Comparing base (168b71b) to head (b3d2df9).
⚠️ Report is 9 commits behind head on master.

Files with missing lines Patch % Lines
pkg/util/state/token.go 55.84% 26 Missing and 8 partials ⚠️
cmd/neofs-node/config.go 0.00% 21 Missing ⚠️
pkg/util/state/migratation.go 53.33% 14 Missing and 7 partials ⚠️
pkg/util/state/session/temporary/storage.go 0.00% 4 Missing ⚠️
pkg/util/state/storage.go 73.33% 3 Missing and 1 partial ⚠️
cmd/neofs-node/main.go 0.00% 1 Missing ⚠️
cmd/neofs-node/session.go 0.00% 1 Missing ⚠️
pkg/innerring/state.go 0.00% 1 Missing ⚠️
pkg/util/state/session/temporary/executor.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3630      +/-   ##
==========================================
+ Coverage   26.78%   26.81%   +0.02%     
==========================================
  Files         653      654       +1     
  Lines       51111    51148      +37     
==========================================
+ Hits        13692    13716      +24     
- Misses      36366    36372       +6     
- Partials     1053     1060       +7     

☔ 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.

@End-rey End-rey force-pushed the 3425-merge-persistent_sessions-and-persistent_state branch from 44e674a to 66d0296 Compare October 10, 2025 15:39
@End-rey End-rey force-pushed the 3425-merge-persistent_sessions-and-persistent_state branch 2 times, most recently from 7fc2527 to b3d2df9 Compare October 17, 2025 10:41
Move `persistent_sessions` data to `persistent_state`, use single DB for local
state. Store sessions in `node.persistent_state.path`. Keep compatibility,
migrate existing DBs, config is to be deleted in subsequent versions.

Closes #3425.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
Accidentally deleted in the 35f7629 info about 0.49.1.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
@End-rey End-rey requested a review from roman-khimov October 17, 2025 12:47
@roman-khimov roman-khimov merged commit 86161a2 into master Oct 21, 2025
22 checks passed
@roman-khimov roman-khimov deleted the 3425-merge-persistent_sessions-and-persistent_state branch October 21, 2025 11:46
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.

Merge persistent_sessions and persistent_state

2 participants