Skip to content
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

chore: rework chart testing and tx-data-provider #73

Merged
merged 7 commits into from
Apr 29, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 8 additions & 3 deletions .github/workflows/helm-checks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -140,10 +140,15 @@ jobs:
- name: Install chart and run tests (tx-data-provider)
run: ct install --charts charts/tx-data-provider --target-branch ${{ github.event.repository.default_branch }}

- name: Install chart and run tests (umbrella)
- name: Install chart for shared services (umbrella)
run: |
helm install umbrella charts/umbrella -f charts/values-test.yaml --namespace install --create-namespace --debug --timeout 10m
helm uninstall umbrella --namespace install
helm install umbrella charts/umbrella -f charts/values-test-shared-services.yaml -f charts/values-test-iam-init-container.yaml --namespace shared-services --create-namespace --debug --timeout 10m
helm uninstall umbrella --namespace shared-services

- name: Install chart for data exchange (umbrella)
run: |
helm install umbrella charts/umbrella -f charts/values-test-data-exchange.yaml -f charts/values-test-iam-init-container.yaml --namespace data-exchange --create-namespace --debug --timeout 10m
helm uninstall umbrella --namespace data-exchange

## Skip upgrade for now until a working chart is released
#- name: Run helm upgrade
Expand Down
5 changes: 2 additions & 3 deletions charts/umbrella/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ sources:
- https://github.com/eclipse-tractusx/tractus-x-umbrella

type: application
version: 0.11.2
version: 0.11.3

dependencies:
# portal
Expand Down Expand Up @@ -72,7 +72,6 @@ dependencies:
condition: dataconsumer.enabled
# TX Data Providers
- name: tx-data-provider
alias: dataprovider
version: 0.0.2
repository: https://eclipse-tractusx.github.io/charts/dev
condition: dataprovider.enabled
condition: tx-data-provider.enabled
28 changes: 18 additions & 10 deletions charts/umbrella/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -200,10 +200,18 @@ To set your own configuration and secret values, install the helm chart with you
helm install -f your-values.yaml umbrella tractusx-dev/umbrella --namespace umbrella
```

To install only the components currently in focus of the **E2E Adopter Journey**, install the helm chart with the values-adopter.yaml file:
To install only the components currently in focus of the **E2E Adopter Journey**, install the helm chart with the respective `values-adopter-xxx.yaml` file:

**Data Exchange**

```bash
helm install -f values-adopter-data-exchange.yaml umbrella tractusx-dev/umbrella --namespace umbrella
```

**Portal**

```bash
helm install -f values-adopter.yaml umbrella tractusx-dev/umbrella --namespace umbrella
helm install -f values-adopter-portal.yaml umbrella tractusx-dev/umbrella --namespace umbrella
```

> **Note**
Expand All @@ -214,6 +222,14 @@ helm install -f values-adopter.yaml umbrella tractusx-dev/umbrella --namespace u

### E2E Adopter Journeys

#### Data exchange

Involved components:

EDC, MIW, DTR, Vault (data provider and consumer in tx-data-provider), CentralIdP.

TBD.

#### Get to know the Portal

Perform first login and send out an invite to a company to join the network (SMTP account required to be configured in custom values.yaml file).
Expand Down Expand Up @@ -261,14 +277,6 @@ tractusx-umbr3lla!
linkStyle 0,1 stroke:lightblue
```

#### Data exchange

Involved components:

EDC, MIW, DTR, Vault (data provider and consumer in tx-data-provider), CentralIdP.

TBD.

### Uninstall

To teardown your setup, run:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@
---

portal:
enabled: true
enabled: false

centralidp:
enabled: true

sharedidp:
enabled: true
enabled: false

bpndiscovery:
enabled: false
Expand All @@ -40,7 +40,7 @@ managed-identity-wallet:
enabled: true

dataconsumer:
enabled: false
enabled: true

dataprovider:
enabled: false
tx-data-provider:
enabled: true
46 changes: 46 additions & 0 deletions charts/umbrella/values-adopter-portal.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,46 @@
# #############################################################################
# Copyright (c) 2024 Contributors to the Eclipse Foundation
#
# See the NOTICE file(s) distributed with this work for additional
# information regarding copyright ownership.
#
# This program and the accompanying materials are made available under the
# terms of the Apache License, Version 2.0 which is available at
# https://www.apache.org/licenses/LICENSE-2.0.
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
#
# SPDX-License-Identifier: Apache-2.0
# #############################################################################
---

portal:
enabled: true

centralidp:
enabled: true

sharedidp:
enabled: true

bpndiscovery:
enabled: false

discoveryfinder:
enabled: false

sdfactory:
enabled: false

managed-identity-wallet:
enabled: true

dataconsumer:
enabled: false

tx-data-provider:
enabled: false
31 changes: 15 additions & 16 deletions charts/umbrella/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
# #############################################################################
---
portal:
enabled: true
enabled: false
replicaCount: 1
postgresql:
postgresql:
Expand Down Expand Up @@ -159,7 +159,7 @@ portal:
port: 8080

centralidp:
enabled: true
enabled: false
keycloak:
nameOverride: "centralidp"
replicaCount: 1
Expand All @@ -170,7 +170,7 @@ centralidp:
initContainers:
- name: realm-import
image: docker.io/tractusx/umbrella-init-container:0.0.1-init
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command:
- sh
args:
Expand All @@ -183,7 +183,7 @@ centralidp:
mountPath: "/realms"
- name: theme-import
image: docker.io/tractusx/portal-iam:v2.1.0
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command:
- sh
args:
Expand All @@ -196,7 +196,7 @@ centralidp:
mountPath: "/themes"
- name: init-certs
image: docker.io/bitnami/keycloak:22.0.3-debian-11-r14
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command: ["/bin/bash"]
args:
- -ec
Expand Down Expand Up @@ -272,7 +272,7 @@ centralidp:
truststorePassword: ""

sharedidp:
enabled: true
enabled: false
keycloak:
nameOverride: "sharedidp"
replicaCount: 1
Expand All @@ -283,7 +283,7 @@ sharedidp:
initContainers:
- name: realm-import
image: docker.io/tractusx/umbrella-init-container:0.0.1-init
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command:
- sh
args:
Expand All @@ -296,7 +296,7 @@ sharedidp:
mountPath: "/realms"
- name: theme-import
image: docker.io/tractusx/portal-iam:v2.1.0
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command:
- sh
args:
Expand All @@ -313,7 +313,7 @@ sharedidp:
mountPath: "/themes-catenax-shared-portal"
- name: init-certs
image: docker.io/bitnami/keycloak:22.0.3-debian-11-r14
imagePullPolicy: Always
imagePullPolicy: IfNotPresent
command: ["/bin/bash"]
args:
- -ec
Expand Down Expand Up @@ -391,7 +391,7 @@ sharedidp:
truststorePassword: ""

bpndiscovery:
enabled: true
enabled: false
enablePostgres: true
bpndiscovery:
authentication: false
Expand All @@ -413,7 +413,7 @@ bpndiscovery:
size: 8Gi

discoveryfinder:
enabled: true
enabled: false
enablePostgres: true
discoveryfinder:
authentication: true
Expand Down Expand Up @@ -455,7 +455,7 @@ discoveryfinder:
size: 8Gi

sdfactory:
enabled: true
enabled: false
secret:
# -- JWK Set URI
jwkSetUri: "https://centralidp.example.org/auth/realms/CX-Central/protocol/openid-connect/certs"
Expand Down Expand Up @@ -483,7 +483,7 @@ sdfactory:
clearingHouseClientSecret: ""

managed-identity-wallet:
enabled: true
enabled: false
## TODO figure out tls setup with centralidp keycloak
miw:
host: "managed-identity-wallets.example.org"
Expand Down Expand Up @@ -532,7 +532,6 @@ managed-identity-wallet:
nginx.ingress.kubernetes.io/enable-cors: "true"

dataconsumer:
# Disable until install works
enabled: false
seedTestdata: false
nameOverride: dataconsumer
Expand Down Expand Up @@ -584,8 +583,7 @@ dataconsumer:
simple-data-backend:
enabled: false

dataprovider:
# Disable until install works
tx-data-provider:
enabled: false
seedTestdata: true
backendUrl: http://{{ .Release.Name }}-dataprovider-submodelserver:8080
Expand Down Expand Up @@ -641,6 +639,7 @@ dataprovider:
postgresql:
nameOverride: dataprovider-dtr-db
auth:
password: "dtrdataproviderpw"
existingSecret: dataprovider-secret-dtr-postgres-init
registry:
host: dataprovider-dtr.test
Expand Down
45 changes: 45 additions & 0 deletions charts/values-test-data-exchange.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
###############################################################
# Copyright (c) 2024 Contributors to the Eclipse Foundation
#
# See the NOTICE file(s) distributed with this work for additional
# information regarding copyright ownership.
#
# This program and the accompanying materials are made available under the
# terms of the Apache License, Version 2.0 which is available at
# https://www.apache.org/licenses/LICENSE-2.0.
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
#
# SPDX-License-Identifier: Apache-2.0
###############################################################

portal:
enabled: false

centralidp:
enabled: true

sharedidp:
enabled: false

bpndiscovery:
enabled: false

discoveryfinder:
enabled: false

sdfactory:
enabled: false

managed-identity-wallet:
enabled: true

dataconsumer:
enabled: true

tx-data-provider:
enabled: true
File renamed without changes.
45 changes: 45 additions & 0 deletions charts/values-test-shared-services.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
###############################################################
# Copyright (c) 2024 Contributors to the Eclipse Foundation
#
# See the NOTICE file(s) distributed with this work for additional
# information regarding copyright ownership.
#
# This program and the accompanying materials are made available under the
# terms of the Apache License, Version 2.0 which is available at
# https://www.apache.org/licenses/LICENSE-2.0.
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
#
# SPDX-License-Identifier: Apache-2.0
###############################################################

portal:
enabled: true

centralidp:
enabled: true

sharedidp:
enabled: true

bpndiscovery:
enabled: true

discoveryfinder:
enabled: true

sdfactory:
enabled: true

managed-identity-wallet:
enabled: true

dataconsumer:
enabled: false

tx-data-provider:
enabled: false