Replace DAPS with SSI in Helm charts #511
Conversation
paullatzelsperger
changed the title
paullatzelsperger
force-pushed
the
feature/add_helmchart_for_ssi
branch
5 times, most recently
from
e7957c7 to
6abbe96
Compare
paullatzelsperger
force-pushed
the
feature/add_helmchart_for_ssi
branch
from
6abbe96 to
fe33478
Compare
|
\cc @jimmarino |
| | web.http.validation.path | X | /validation | | | ||
| | web.http.control.port | X | 9999 | | | ||
| | web.http.control.path | X | /api/controlplane/control | | | ||
| | web.http.ids.port | X | 8282 | | |
There was a problem hiding this comment.
Are these ids. properties used anymore?
There was a problem hiding this comment.
i don't think they are, but the template still exposes them apparently. the README's are regenerated when we publish the helm chart. it's possible that the regenerated version never found it's way back into main. I regenerated them and committed them now.
There was a problem hiding this comment.
also, this is only in the legacy chart, which will go away eventually
There was a problem hiding this comment.
LGTM, just a minor nit on example dataspace README.
I've tested the chart on a local machine with a real MIW and Keycloak running.
Tested with MIW revision catenax-ng/tx-managed-identity-wallets@bed8bd6
Co-authored-by: Enrico Risa <[email protected]>
Thanks @wolf4ood for doing this 🤟 I think we're on a good track then! |
|
Kudos, SonarCloud Quality Gate passed!
|
WHAT
Changes all Helm charts to use SSI/MiW instead of DAPS.
Adds a
-legacyhelm chart that still uses DAPS, intended as a fallback.WHY
Replace DAPS with SSI
FURTHER NOTES
tractusx-connector-legacywill be left in place that still contains DAPS for fallback purposes. However, this is discontinued and will not be supported anymore. There is no deployment test for it, and it's scheduled for removal in upcoming releases.docs/samples/example-dataspaceMockWebServerdidn't get re-instantiated on every testCloses # <-- insert Issue number if one exists