Skip to content

feat(helm): add connector, wallet, issuer-wallet charts and constructx-edc PR#9 - #4

Open
simbrgr wants to merge 3 commits into
mainfrom
feat(helm)/add-connector-and-wallet
Open

feat(helm): add connector, wallet, issuer-wallet charts and constructx-edc PR#9#4
simbrgr wants to merge 3 commits into
mainfrom
feat(helm)/add-connector-and-wallet

Conversation

@simbrgr

@simbrgr simbrgr commented Sep 2, 2026

Copy link
Copy Markdown

WHAT

Changes from constructx-edc/PR#9

  • Adds vault initialization script job to generate and push AES or RSA keys into the vault on install
  • Removes static demonstration keypairs from templates
  • Fixes missing assignment of iatp default scope environment variables
  • Changes PostgreSQL Dependency from legacy Bitnami to CloudPirates (also see Tractus-X migration Guide)
  • Changes default connector, wallet and issuer name values to better align with Construct-X Wallet values
  • Removes duplicate assignments of certain environment variables

WHY

  • This Repository will keep all deployment related files to keep them separated from source code

Reason for constructx-edc/PR#9

  • The current EDC Helm deployment uses static RSA keypairs stored in a ConfigMap and pushes them into the vault, which is only recommended in demonstration scenarios.
  • The transaction flow will fail due to the missing iatp default scope environment variables

FURTHER NOTES

What has been tested for constructx-edc/PR#9

  • helm lint --strict passes without errors
  • helm template renders without errors
  • Deployed to cluster using Construct-X Wallet and Issuer
    • Note: with Istio as Ingresscontroller instead of nginx Ingresscontroller
  • Vault-init job completed successfully
  • Controlplane and Dataplane pods reached Running state with no error messages
  • Endpoints reachable
  • Paricipant creation successful via super-user extension
  • Transcation flow between namespace-separated consumer- and provider-edc was tested and passed using Construct-X Bruno Collection

Linked Issue(s)

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.

1 participant