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
feat: add state machine to DataPlaneManager
#3420
feat: add state machine to DataPlaneManager
#3420
Conversation
...in/java/org/eclipse/edc/connector/dataplane/framework/DataPlaneDefaultServicesExtension.java
Fixed
Show fixed
Hide fixed
Codecov ReportPatch coverage:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #3420 +/- ##
==========================================
- Coverage 72.21% 71.96% -0.25%
==========================================
Files 835 837 +2
Lines 16838 17012 +174
Branches 953 948 -5
==========================================
+ Hits 12159 12243 +84
- Misses 4271 4364 +93
+ Partials 408 405 -3
☔ View full report in Codecov by Sentry. |
8891f15
to
50f45f5
Compare
50f45f5
to
82bca8a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
4308c5a
to
289f4ec
Compare
289f4ec
to
1334bc8
Compare
* feat: add state machine to DataPlaneManager * Trigger build * create databases before tests
What this PR changes/adds
Make
DataPlaneManager
work withStateMachineManager
.This PR is quite big, but it's mostly adaptation to an already used pattern (like in
TransferProcessManager
)Why it does that
Better state handling for data-plane
Further notes
data-plane-framework
module as it was only used bydata-plane-core
Linked Issue(s)
Closes #3401
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.