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

[flaky] test/e2e.TestBrokerTrigger/2 #127

Closed
knative-test-reporter-robot opened this issue Aug 26, 2020 · 6 comments
Closed

[flaky] test/e2e.TestBrokerTrigger/2 #127

knative-test-reporter-robot opened this issue Aug 26, 2020 · 6 comments

Comments

@knative-test-reporter-robot

Auto-generated issue tracking flakiness of test

  • Test name: test/e2e.TestBrokerTrigger/2
  • Repository name: eventing-kafka-broker
@knative-test-reporter-robot
Copy link
Author

knative-test-reporter-robot commented Aug 26, 2020

Latest result for this test: Flaky
Last build start time: 2020-09-13 10:59:30 +0000 UTC
Failed 1 times out of 10 runs. Failed runs: 1305083780309979136

2020-09-13 10:59:30 +0000 UTC:
2020-08-29 11:00:01 +0000 UTC:
2020-08-27 11:00:31 +0000 UTC:
2020-08-26 11:00:00 +0000 UTC:

✔ Passed ✖ Failed ◻ Skipped

@pierDipi
Copy link
Member

    creation.go:283: Creating v1beta1 trigger trigger
    tracker.go:152: Waiting for all KResources to become ready
=== CONT  TestBrokerTrigger/2
    operation.go:94: Failed to get all KResources ready failed waiting for {TypeMeta:{Kind:Trigger APIVersion:eventing.knative.dev/v1beta1} ObjectMeta:{Name:trigger GenerateName: Namespace:test-broker-trigger-2-ntsbg SelfLink: UID: ResourceVersion: Generation:0 CreationTimestamp:0001-01-01 00:00:00 +0000 UTC DeletionTimestamp:<nil> DeletionGracePeriodSeconds:<nil> Labels:map[] Annotations:map[] OwnerReferences:[] Finalizers:[] ClusterName: ManagedFields:[]}} to become ready: timed out waiting for the condition
    operation.go:115: Failed to get all test resources ready: failed waiting for {TypeMeta:{Kind:Trigger APIVersion:eventing.knative.dev/v1beta1} ObjectMeta:{Name:trigger GenerateName: Namespace:test-broker-trigger-2-ntsbg SelfLink: UID: ResourceVersion: Generation:0 CreationTimestamp:0001-01-01 00:00:00 +0000 UTC DeletionTimestamp:<nil> DeletionGracePeriodSeconds:<nil> Labels:map[] Annotations:map[] OwnerReferences:[] Finalizers:[] ClusterName: ManagedFields:[]}} to become ready: timed out waiting for the condition
        knative.dev/eventing/test/lib.(*Client).WaitForAllTestResourcesReadyOrFail
        	/home/prow/go/src/knative.dev/eventing-kafka-broker/vendor/knative.dev/eventing/test/lib/operation.go:115
        knative.dev/eventing-kafka-broker/test/e2e.TestBrokerTrigger.func1
        	/home/prow/go/src/knative.dev/eventing-kafka-broker/test/e2e/broker_trigger_test.go:93
        testing.tRunner
        	/root/.gvm/gos/go1.14.6/src/testing/testing.go:1039
        runtime.goexit
        	/root/.gvm/gos/go1.14.6/src/runtime/asm_amd64.s:1373

// ...
        {
          "apiVersion": "eventing.knative.dev/v1beta1",
          "kind": "Trigger",
          "metadata": {
            "annotations": {
              "eventing.knative.dev/creator": "prow-job@knative-tests.iam.gserviceaccount.com",
              "eventing.knative.dev/lastModifier": "prow-job@knative-tests.iam.gserviceaccount.com"
            },
            "creationTimestamp": "2020-08-26T10:27:27Z",
            "finalizers": [
              "kafka.triggers.eventing.knative.dev"
            ],
            "generation": 1,
            "labels": {
              "eventing.knative.dev/broker": "broker"
            },
            "name": "trigger",
            "namespace": "test-broker-trigger-2-ntsbg",
            "resourceVersion": "11347",
            "selfLink": "/apis/eventing.knative.dev/v1beta1/namespaces/test-broker-trigger-2-ntsbg/triggers/trigger",
            "uid": "ffb57f43-e6ee-4bd7-92cf-d620fb0c0f11"
          },
          "spec": {
            "broker": "broker",
            "filter": {
              "attributes": {
                "ext1": "value1",
                "source": "source1",
                "type": ""
              }
            },
            "subscriber": {
              "ref": {
                "apiVersion": "v1",
                "kind": "Service",
                "name": "subscriber",
                "namespace": "test-broker-trigger-2-ntsbg"
              }
            }
          },
          "status": {}
        }

@knative-test-reporter-robot
Copy link
Author

Closing issue: this test has passed in latest 2 scans

@knative-test-reporter-robot
Copy link
Author

Reopening issue: this test is flaky

@pierDipi
Copy link
Member

=== Failed
=== FAIL: test/e2e TestBrokerTrigger/3 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/4 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/0 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/1 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/2 (unknown)

gotestyourself/gotestsum#141

/close

@knative-prow-robot
Copy link
Contributor

@pierDipi: Closing this issue.

In response to this:

=== Failed
=== FAIL: test/e2e TestBrokerTrigger/3 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/4 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/0 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/1 (unknown)
=== FAIL: test/e2e TestBrokerTrigger/2 (unknown)

gotestyourself/gotestsum#141

/close

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/test-infra repository.

matzew pushed a commit to matzew/eventing-kafka-broker that referenced this issue Mar 17, 2022
When we probe for resource status, we want to avoid probing the
pod when the status is in our expected status.
Before this patch, we had an hard-coded expected status of
`StatusReady`, which is wrong in the `FinalizeKind` case.

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants