-
Notifications
You must be signed in to change notification settings - Fork 183
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: upgrade MVD with DCP and Management Domains #270
Conversation
reformat some files, update seeding script, use DID as identifier everywhere
prepared distinction between DID documents used for running locally, vs running on a cluster
There is a new module "inmem-seeding" which creates the super-user. Exclude this module if database persistence is used to avoid conflicts.
...-resolver/src/main/java/org/eclipse/edc/demo/participants/ParticipantsResolverExtension.java
Fixed
Show fixed
Hide fixed
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.
New files should have an updated header: 2023 --> 2024
1de7714
to
8c6ad8a
Compare
8c6ad8a
to
68ee636
Compare
those files have been there for quite some time, only in a different branch. but technically, they were in the repo, since the branch was in the upstream. |
69e98df
to
f49c3a8
Compare
f49c3a8
to
b9ce172
Compare
369f2b9
to
6175851
Compare
f2f0a11
to
d652e37
Compare
I tried running the demo via For
For
For
|
@harnvo something is fishy there. The catalog server and the connectors should not need the In any event, I could not reproduce the errors you describe. If you're on Windows, be aware that paths may be a problem, and we don't officially support running this natively on Windows. We typically recommend using WSL. |
@paullatzelsperger I am using wsl2. I later tried reproducing the problem in my Debian PC but everything works fine. It seems something's wrong on the WSL's side. |
this is just the run configurations, so nothing critical, but it still detracts from developer experience. if you are open to investigating platform-independent run configs, we'd welcome a PR |
I would take a look when I am available. The two PCs I have uses different network proxies, so it is also possible that one of the maven mirror did not update the package in-time. Just curious: have eclipse-edc updated some packages recently? In this case waiting for a few days should solve the problem. |
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.
Tested with Intellij
we publish snapshot builds on every PR merge, but you can force the refresh by either appending |
Update: I eventually find out what happened. Somehow Intellij did not read the configuration in The problem would presist if I re-clone this repo. But I would stop the investigation since this is something wrong with |
@harnvo thanks a lot for investigating! |
What this PR changes/adds
This PR brings a completely new iteration of the MinimumViableDataspace project by adding the following high-level features:
Why it does that
Keep up with current developments, improve the developer experience
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.