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

feat: add BPDM chart to umbrella chart #108

Closed
wants to merge 1 commit into from

Conversation

nicoprow
Copy link
Contributor

@nicoprow nicoprow commented Jul 17, 2024

Description

This pull request adds the BPDM chart for Catena-X 24.05 to the umbrella chart.

The umbrella chart sets up the central Pool component and the necessary components for the golden record process. Also, it creates and configures a Gate for the Portal with which to share business partner data to the golden record process.

Please note that a hotfix was needed in order to use the BPDM chart as a dependency: 5.0.3.

To be honest the need for this hotfix still confuses me. The faulty configuration for 5.0.2 really only takes effect if you use this chart as a dependency but not stand-alone. Maybe someone with better insight into helm can exaplain this. In any case, the Chart is working now as a dependency.

The BPDM component will successfully install with the following command from the root directory:

helm install  \
 --set centralidp.enabled=true,bpdm.enabled=true \
 --namespace umbrella \
 --generate-name \
 --create-namespace \
 charts/umbrella

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

  • DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
  • Copyright and license header are present on all affected files
  • Create documentation for enabling this component
  • Create chart tests for BPDM

@nicoprow nicoprow force-pushed the feat/24.05/bpdm branch 2 times, most recently from 2b829a6 to 8ad2363 Compare July 18, 2024 03:49
@nicoprow nicoprow marked this pull request as ready for review July 18, 2024 04:04
@nicoprow nicoprow requested a review from evegufy July 18, 2024 04:04
Copy link
Contributor

@evegufy evegufy left a comment

Choose a reason for hiding this comment

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

Hi @nicoprow great that you opened the PR!

Does it contain the missing config discussed here? eclipse-tractusx/portal-backend#819 (comment)

I tested and encounter the following issues with the database connection:
logs-from-bpdm-gate-in-umbrella-bpdm-gate-85b9fb65bd-hqkqh.log
logs-from-bpdm-cleaning-service-dummy-in-umbrella-bpdm-cleaning-service-dummy-9969db5cb-qjb6f.log
logs-from-bpdm-pool-in-umbrella-bpdm-pool-f546b5f99-b9t8s.log

There's also the PR #106 from @gomezbc who did the awesome work of providing his configuration of BPDM 5.0.2 version and the database connection issue seems to already addressed there, but currently we're facing 401 errors there which might be due to this missing configuration...

Could you please align with @gomezbc to bring the best of both of your PRs together?

As the BPDM 5.0.3 version is for R24.05 and main is still set on R24.03 it would be great if you could redirect your PR to the upgrade branch for 24.05 https://github.com/eclipse-tractusx/tractus-x-umbrella/tree/upgrade/24.05

@evegufy evegufy mentioned this pull request Jul 18, 2024
2 tasks
@nicoprow
Copy link
Contributor Author

Hi @nicoprow great that you opened the PR!

Does it contain the missing config discussed here? eclipse-tractusx/portal-backend#819 (comment)

I tested and encounter the following issues with the database connection: logs-from-bpdm-gate-in-umbrella-bpdm-gate-85b9fb65bd-hqkqh.log logs-from-bpdm-cleaning-service-dummy-in-umbrella-bpdm-cleaning-service-dummy-9969db5cb-qjb6f.log logs-from-bpdm-pool-in-umbrella-bpdm-pool-f546b5f99-b9t8s.log

There's also the PR #106 from @gomezbc who did the awesome work of providing his configuration of BPDM 5.0.2 version and the database connection issue seems to already addressed there, but currently we're facing 401 errors there which might be due to this missing configuration...

Could you please align with @gomezbc to bring the best of both of your PRs together?

As the BPDM 5.0.3 version is for R24.05 and main is still set on R24.03 it would be great if you could redirect your PR to the upgrade branch for 24.05 https://github.com/eclipse-tractusx/tractus-x-umbrella/tree/upgrade/24.05

I guess these logs happened before the BPDM-Postgres was finished setting up. The helm install like stated above works. The services just need the centralidp and postgres to be running and after that they stop crashing.

I will close this and make the target the update 24.05 branch. I will also try to get in touch @gomezbc for aligning, as he seems to have better grasp of the portal config needed.

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.

2 participants