-
Notifications
You must be signed in to change notification settings - Fork 5
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: upgrade to TX EDC 0.7 #92
feat: upgrade to TX EDC 0.7 #92
Conversation
Bumps [docker/login-action](https://github.com/docker/login-action) from 2 to 3. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](docker/login-action@v2...v3) --- updated-dependencies: - dependency-name: docker/login-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
…n the EDC dataaddress interface.
…ion test dependency.
Bumps [helm/chart-testing-action](https://github.com/helm/chart-testing-action) from 2.4.0 to 2.6.1. - [Release notes](https://github.com/helm/chart-testing-action/releases) - [Commits](helm/chart-testing-action@v2.4.0...v2.6.1) --- updated-dependencies: - dependency-name: helm/chart-testing-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [org.junit:junit-bom](https://github.com/junit-team/junit5) from 5.9.3 to 5.10.1. - [Release notes](https://github.com/junit-team/junit5/releases) - [Commits](junit-team/junit5@r5.9.3...r5.10.1) --- updated-dependencies: - dependency-name: org.junit:junit-bom dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]>
…-junit-bom-5.10.1' into feature/upgrade-tx-edc-060
…helm/chart-testing-action-2.6.1' into feature/upgrade-tx-edc-060
…docker/login-action-3' into feature/upgrade-tx-edc-060
…ote service addressing.
@almadigabor The chart linting actually should go through, see https://github.com/catenax-ng/tx-knowledge-agents-edc/actions/runs/9035264222 (where we have already integrated this PR) - sometimes helm does not "know" the standard repos as it seems when doing "cross-repo" actions. I tried to add the repos now explicitely. However, the dependencies check will show the need to issue an IP check for the new versions of the depending modules, as Apache Jena https://github.com/catenax-ng/tx-knowledge-agents-edc/actions/runs/9035264223 maven/mavencentral/org.apache.httpcomponents/httpcore/4.4.16, Apache-2.0, approved, CQ23528 It would be great if you could issue that IP check and review/commit this until the 15th (such that SAST can run until the 17th where the Q-Gates are). Thx much! |
…testing and that referrred components are up-to-date.
Hi @drcgjung! I've ran the dash tool and got the following output:
Let's wait until the afternoon for these to be resolved then I can review and merge your PR. You might need a DEPENDENCIES file update once the IP issues are closed. |
Hey @drcgjung, can you update the dependencies file? If I see it correctly, all issues have been approved. |
@almadigabor done and successful. thx. |
I opened an IP review issue due to the large number of changes. Please track the status here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @drcgjung, see my comments below.
Again, the chart testing process somehow stuck but should be successful, if its been restarted: see https://github.com/catenax-ng/tx-knowledge-agents-edc/actions/runs/9138105951 |
The IP check issue have been approved. |
WHAT
Upgrades Agent Plane(s) to TX EDC 0.7.0 (and Apache Jena 4.9)
Switches from full connector charts to data/agent plane only chart.
WHY
TX EDC 0.7.0 is the breaking-change release as of CX 24.05
Apache Jena has provided a new release which also updates quite some dependencies.
Having agent-plane only charts makes it easier for sidecar deployments with EDC.
Furthermore we can thus successfully test the data plane chart alone which should be now compliant to the TRGHs.
This PR has been aligned to the provider-part of the 24.05 E2E test
FURTHER NOTES
Closes #94
Closes #179
Closes #129
Closes #98
Closes #97
Closes #61
Closes #31
Closes #30
Closes #26
Closes #185