chore(ci): enable workflow Helm test on PRs

[fty4]

I'd love if the Helm test workflow is running on PRs.

This would help to verify if a PR breaks the Helm chart.
Currently it is only triggered via manual workflow_dispatch.
Other projects in eclipse-tractusx test it the same way.

---

Marco Lecheler [email protected] Mercedes-Benz Tech Innovation GmbH (ProviderInformation)

[fty4]

current issues:

• daps test will use wrong endpoint resolving in HTTP 404 and always fail
  (see fix(helm): use openid-configuration path for Helm test daps-helm-chart#38)

• FailedMount for volumes from script from e.g. job-daps certconsumer

https://github.com/eclipse-tractusx/e2e-testing/blob/6c332561980ba508d2a36c9c84b0958f353c2d99/charts/umbrella/charts/certs/templates/job-daps.yaml#L54-L63

Fixes should happen in other PR (IMHO)

[almadigabor]

Looks good!

[almadigabor]

Probably need to include the fixed chart dependencies in this PR for the helm test to pass.

[fty4]

Probably need to include the fixed chart dependencies in this PR for the helm test to pass.

You are right - the mentioned issues above should be the problems. But I think these changes should be in another PR.

[almadigabor]

Probably need to include the fixed chart dependencies in this PR for the helm test to pass.

You are right - the mentioned issues above should be the problems. But I think these changes should be in another PR.

That is true but until we have the fixes in this branch, the test workflow will fail and PR can't be merged.

[fty4]

I just tested it with the changes from #22.
The test was successfully.

Therefore I will wait until this mentioned PR is merged.

Helm test

 helm test e2e                                                                       
NAME: e2e
LAST DEPLOYED: Wed Jun 21 09:57:44 2023
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE:     e2e-server-test
Last Started:   Wed Jun 21 10:01:56 2023
Last Completed: Wed Jun 21 10:02:00 2023
Phase:          Succeeded
TEST SUITE:     e2e-edcconsumertest-controlplane-readiness
Last Started:   Wed Jun 21 10:01:37 2023
Last Completed: Wed Jun 21 10:01:42 2023
Phase:          Succeeded
TEST SUITE:     e2e-edcconsumertest-dataplane-readiness
Last Started:   Wed Jun 21 10:01:42 2023
Last Completed: Wed Jun 21 10:01:47 2023
Phase:          Succeeded
TEST SUITE:     e2e-edcprovidertest-controlplane-readiness
Last Started:   Wed Jun 21 10:01:47 2023
Last Completed: Wed Jun 21 10:01:52 2023
Phase:          Succeeded
TEST SUITE:     e2e-edcprovidertest-dataplane-readiness
Last Started:   Wed Jun 21 10:01:52 2023
Last Completed: Wed Jun 21 10:01:56 2023
Phase:          Succeeded

[fty4]

As assumed - after merging #22 the test will succeed.

[almadigabor]

As assumed - after merging #22 the test will succeed.

Great!:) Can I merge this commit now?

[fty4]

yes would be great if this will be merged