search

knative / eventing

Event-driven application platform for Kubernetes
https://knative.dev/docs/eventing
Apache License 2.0
1.41k stars 588 forks source link

E2E tests are failing for Knative-eventing #4626

Closed VIjayHP closed 3 years ago

VIjayHP commented 3 years ago

Following pods are running under knative-eventing namespace: eventing-controller-66c877b879-7ffzz 1/1 Running 0 61m eventing-webhook-644c5c7667-d5qzg 1/1 Running 0 61m imc-controller-587f98f97d-68smx 1/1 Running 0 59m imc-dispatcher-6db95d7857-c5j2z 1/1 Running 0 59m mt-broker-controller-76b65f7c96-lxsvm 1/1 Running 0 59m mt-broker-filter-6bd64f8c65-d5jf7 1/1 Running 0 59m mt-broker-ingress-7d8595d747-mp768 1/1 Running 0 59m

Following Configmaps are also already configured : config-br-default-channel 1 61m config-br-defaults 1 61m config-imc-event-dispatcher 2 60m config-leader-election 1 61m config-logging 3 61m config-observability 1 61m config-tracing 1 61m default-ch-webhook 1 61m

But still E2E tests are failing after firing the below command : go test -v -timeout=0m -tags=e2e -count=1 ./test/e2e --dockerrepo {private-docker-repo}

Error: TestDefaultBrokerWithManyTriggers/test_default_broker_with_many_deprecated_triggers: test_runner.go:153: namespace is : "test-default-broker-with-many-triggers-test-default-broker5nkzz" TestDefaultBrokerWithManyTriggers/test_default_broker_with_many_deprecated_triggers: test_runner.go:160: Couldn't initialize clients: error while retrieving the config-tracing config map: configmaps "config-tracing" not found.

VIjayHP commented 3 years ago

Could someone help me with the above failures or any additional setup/procedure I need to follow for running e2e tests for eventing?

lberk commented 3 years ago

You might need to export the envvar SYSTEM_NAMESPACE="knative-eventing"

VIjayHP commented 3 years ago

@lberk : Also the yaml file from the documentation is not working $kubectl apply --filename https://github.com/knative/eventing/releases/download/v0.19.0/eventing-core.yaml After applying this webhook pod fails issue is mentioned here: https://github.com/knative/eventing/issues/4629

VIjayHP commented 3 years ago

$go test -v -timeout=0m -tags=e2e -count=1 ./test/e2e --dockerrepo registry-ip:port/local-images

Error: Failed to get test-default-broker-with-many-triggers-test-default-brokerffn6k/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-brokerdxmkd/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-brokerdxps5/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-brokerx278b/default: brokers.eventing.knative.dev "default" not foundFailed

VIjayHP commented 3 years ago

SYSTEM_NAMESPACE @lberk After exporting this environmental variable only i am getting the issue: Failed to get test-default-broker-with-many-triggers-test-default-brokergslcs/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-broker2zcpd/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-broker8lr8c/default: brokers.eventing.knative.dev "default" not foundFailed to get test-default-broker-with-many-triggers-test-default-brokerqqqqb/default

pierDipi commented 3 years ago

You need to install the sugar controller for running E2E tests. Also, it's recommended to use the latest patch release v0.19.2.

pierDipi commented 3 years ago

Also, there are test-specific configurations to apply: https://github.com/knative/eventing/tree/master/test/config.

You can also try to follow these steps (from a clean cluster):

  1. export KO_DOCKER_REPO=<your_container_registry>
  2. source test/e2e-common.sh
  3. knative_setup
  4. test_setup
  5. Run tests: 
    go test -tags=e2e -timeout=30m -parallel=20 ./test/e2e \
-brokerclass=MTChannelBasedBroker \
-channels=messaging.knative.dev/v1beta1:Channel,messaging.knative.dev/v1beta1:InMemoryChannel,messaging.knative.dev/v1:Channel,messaging.knative.dev/v1:InMemoryChannel \
-sources=sources.knative.dev/v1beta1:ApiServerSource,sources.knative.dev/v1alpha2:ContainerSource,sources.knative.dev/v1beta1:PingSource,sources.knative.dev/v1beta2:PingSource,sources.knative.dev/v1:ApiServerSource,sources.knative.dev/v1:ContainerSource
VIjayHP commented 3 years ago

It worked after applying above changes, Thanks @pierDipi

lionelvillard commented 3 years ago

FWIW source test/e2e-common.sh does not work on v0.19.2 since $0 is bound to where bash is located.

VIjayHP commented 3 years ago

@lionelvillard : I have used v.0.19.0 in this case