Skip to content

feat: adapted construct-x connector helm chart configurations#5

Merged
sb-embeteco merged 17 commits into
developfrom
feat/conn-helm-chart
Jun 16, 2026
Merged

feat: adapted construct-x connector helm chart configurations#5
sb-embeteco merged 17 commits into
developfrom
feat/conn-helm-chart

Conversation

@saudkhan116

@saudkhan116 saudkhan116 commented Apr 8, 2026

Copy link
Copy Markdown

WHAT

Briefly describe what your PR changes, which features it adds/modifies.

  • Adapted the controlplane and dataplane templates
  • Added seed jobs to inject certificates to the vault
  • Updated helm values files; values-consumer.yaml and values-provider.yaml
  • The following docker images have been used:
    • Controlplane: registry.onstackit.cloud/constructx/con-x-controlplane-postgresql-hashicorp-vault
    • Dataplane: registry.onstackit.cloud/constructx/con-x-dataplane-postgresql-hashicorp-vault

WHY

Briefly state why the change was necessary.

To demonstrate data exchange using Construct-X connectors and ensure compatibility between IdentityHub, Issuer-service and portal registration service.

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.

Closes # <-- insert Issue number if one exists

@saudkhan116 saudkhan116 requested a review from mhellmeier April 8, 2026 15:34
@saudkhan116 saudkhan116 self-assigned this Apr 8, 2026
@saudkhan116 saudkhan116 added the feature A new feature request label Apr 8, 2026
@mhellmeier mhellmeier requested a review from eschrewe April 9, 2026 14:16
@mhellmeier mhellmeier requested a review from snoeffels April 30, 2026 06:47

@sb-embeteco sb-embeteco left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both deployment options (consumer and provider) worked on my cluster using the deployments from the Construct-X IdentityHub Helm chart PR; see the PR comment in the IdentityHub repository.

Questions:

  • Would the deployment using the legacy values file and Tractus-X container still work after the deployment files have been updated to use the Conststruct-X EDC Testbed Container? If not, we might want to consider opening a separate Chart for the Con-X Testbed and restore the Tractus-X Chart.

Notes:

  • Since I'm using Istio as the ingress controller on my cluster, I wasn't able to explicitly test the ingress block in the values files. However, I don't think the changes to the configuration will have a negative impact on the deployment.
  • The README can no longer be fully applied to the new deployment and should be updated to reflect using the new values files.

Comment thread charts/tractusx-connector/templates/post-install-vault-setup.yaml
Comment thread charts/tractusx-connector/templates/vault-edc-configmap.yaml
Comment thread charts/tractusx-connector/values-consumer.yaml Outdated
Comment thread charts/tractusx-connector/values-consumer.yaml Outdated
Comment thread charts/tractusx-connector/values-consumer.yaml Outdated
Comment thread charts/tractusx-connector/values-consumer.yaml Outdated
Comment thread charts/tractusx-connector/templates/vault-edc-configmap.yaml
Comment thread charts/tractusx-connector/templates/vault-edc-configmap.yaml
Comment thread charts/tractusx-connector/templates/vault-edc-configmap.yaml
Comment thread charts/tractusx-connector/values-consumer.yaml
Comment thread charts/tractusx-connector/values-provider.yaml
Comment thread charts/tractusx-connector/values.yaml
Comment thread charts/tractusx-connector/templates/deployment-controlplane.yaml
Comment thread charts/tractusx-connector/templates/deployment-dataplane.yaml
@mhellmeier mhellmeier removed their request for review June 9, 2026 09:29
@snoeffels snoeffels self-requested a review June 16, 2026 06:59
@sb-embeteco sb-embeteco merged commit ba70099 into develop Jun 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants