Uploaded image for project: 'Knative Serving'
  1. Knative Serving
  2. SRVKS-157

knative route is Ready before the knative-openshift-ingress OpenShift Route starts routing requests

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • None
    • v0.6.0
    • None
    • Serverless Sprint 171, Serverless Sprint 176, Serverless Sprint 177

      As can be shown e.g. on TestRouteAndConfigGenerateName run with --resolvabledomain on OpenShift 4.1 on AWS,

      The test waits for the knative Route state to be Ready before starting requests, but still gets 503 from the OpenShift Router:

       

       --- FAIL: TestRouteAndConfigGenerateName (12.05s)
       generatename_test.go:160: Creating new configuration with generateName route-and-config-generate-name-twphbwqp-
       util.go:34: resource {<nil> <*>{&TypeMeta{Kind:,APIVersion:,} &ObjectMeta{Name:,GenerateName:route-and-config-generate-name-twphbwqp-,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[],Finalizers:[],ClusterName:,Initializers:nil,} {0 <nil> <nil> <*>&ObjectMeta{Name:,GenerateName:,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[],Finalizers:[],ClusterName:,Initializers:nil,}} {{0 <nil>} { }}} <nil> <nil>}
       generatename_test.go:168: The configuration will be updated with the name of the associated Revision once it is created.
       generatename_test.go:175: When the configuration is created, the name is generated using the provided generateName
       generatename_test.go:181: Create new Route with generateName route-and-config-generate-name-twphbwqp-
       util.go:34: resource {<*>{&TypeMeta{Kind:,APIVersion:,} &ObjectMeta{Name:,GenerateName:route-and-config-generate-name-twphbwqp-,Namespace:,SelfLink:,UID:,ResourceVersion:,Generation:0,CreationTimestamp:0001-01-01 00:00:00 +0000 UTC,DeletionTimestamp:<nil>,DeletionGracePeriodSeconds:nil,Labels:map[string]string{},Annotations:map[string]string{},OwnerReferences:[],Finalizers:[],ClusterName:,Initializers:nil,} {0 [{ { route-and-config-generate-name-twphbwqp-thrjk <nil> 100 <nil>}}]} {{0 <nil>} {<nil> <nil> <nil>}}} <nil> <nil> <nil>}
       generatename_test.go:188: When the route is created, it will become ready
       generatename_test.go:194: When the route is created, the name is generated using the provided generateName
       generatename_test.go:78: Route route-and-config-generate-name-twphbwqp-rb4tn has a domain set in its status
       generatename_test.go:93: Route route-and-config-generate-name-twphbwqp-rb4tn can serve the expected data at the endpoint
       generatename_test.go:201: Configuration route-and-config-generate-name-twphbwqp-thrjk with Route route-and-config-generate-name-twphbwqp-rb4tn could not serve requests: the endpoint for Route route-and-config-generate-name-twphbwqp-rb4tn at domain route-and-config-generate-name-twphbwqp-rb4tn.serving-tests.apps.maschmid1.openshift-aws.rhocf-dev.com didn't serve the expected text "Hello World! How about some tasty noodles?": response: status: 503
      

      Adding a delay after test.WaitForRouteState , e.g.

       

      time.Sleep(30 * time.Second) 

      makes the test pass.

       

       

              Unassigned Unassigned
              maschmid@redhat.com Marek Schmidt
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Created:
                Updated:
                Resolved: