integration tests added

[ahmetcihank]

E2E auth test added.

[knative-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ahmetcihank
Once this PR has been reviewed and has the lgtm label, please assign pierdipi for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[pierDipi]

I believe this is a good start!

To enable and run tests, additionally, we need to:

• add a test/e2e-tests.sh script, similar to:
  • https://github.com/knative/eventing/blob/main/test/e2e-tests.sh
  • that uses https://github.com/knative/eventing/blob/main/test/e2e-common.sh
  • Documentation for this and the various "well-known functions" can be found here https://github.com/knative/hack/blob/9c9eed6f6679d33ae7db78d0cfc6f03f7c2aee83/README.md#using-the-e2e-testssh-helper-script
• Once the PR is merged and the test/e2e-tests.sh script is present, we can configure Prow to run those jobs as presubmit jobs, an example can be found here: https://github.com/knative/infra/blob/34cd766e4e17f6c5f5c751a99fbcb40b4997a8a1/prow/jobs_config/knative-extensions/eventing-istio.yaml#L7-L20 and those configurations need to be added here https://github.com/knative/infra/blob/main/prow/jobs_config/knative-extensions/backstage-plugins.yaml

[pierDipi]

For this one, I would create a ServiceAccount (SA) with the expected permissions (ClusterRole and ClusterRoleBinding) in test/config/ and apply it as part of the knative_setup (see my top-level review comment) and then this TokenRequest will use that SA to get an associated token and pass it to eventshub.

[aliok]

Related PRs:

• e2e test infrastructure #82
• Backend controller improvements #84
• E2E tests for backend security #85
• Update Eventing manifest nightlies for Backstage plugins repo knobots#409

[knative-prow-robot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[aliok]

/close

Closing in favor of #85

[knative-prow]

@aliok: Closed this PR.

In response to this:

/close

Closing in favor of #85

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.