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

Integration test flake "service-account default not found" #329

Closed
rhuss opened this issue Aug 5, 2019 · 2 comments · Fixed by #348
Closed

Integration test flake "service-account default not found" #329

rhuss opened this issue Aug 5, 2019 · 2 comments · Fixed by #348
Labels
kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@rhuss
Copy link
Contributor

rhuss commented Aug 5, 2019

There is currently an integration test flaking popping up from time to time with an error:

Running 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests2'...
Failed to successfully execute 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests3': Execution error: stderr: 'RevisionFailed: Revision "hello-qj4nm" failed with message: Unable to fetch image "gcr.io/knative-samples/helloworld-go": serviceaccounts "default" not found.
' error: 'exit status 1'FAIL	github.com/knative/client/test/e2e	1.024s

See e.g. https://storage.googleapis.com/knative-prow/pr-logs/pull/knative_client/312/pull-knative-client-integration-tests/1157313391212105729/build-log.txt

It smells like an issue in the cluster setup but not so much like a race as it happens after some successful image pulls in those tests.

@adrcunha do you have any idea what the cause could be ?

@rhuss rhuss added the kind/bug Categorizes issue or PR as related to a bug. label Aug 5, 2019
@adrcunha
Copy link
Contributor

adrcunha commented Aug 8, 2019

This doesn't sound like a cluster setup to me. A few tests passed before the one failing, and the service account was (correctly) set before the tests. Since this is a flake, my guess is that one of the tests is doing some sort of teardown and thus causing TestRevision/create_hello_service_and_return_no_error to fail.

=== RUN   TestRoute/create_hello_service_and_return_no_error
Running 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests4'...
=== RUN   TestServiceOptions/create_and_validate_service_with_concurrency_options
Running 'kn service create svc1 --image gcr.io/knative-samples/helloworld-go --concurrency-limit 250 --concurrency-target 300 -n kne2etests1'...
=== RUN   TestBasicWorkflow/returns_no_service_before_running_tests
Running 'kn service list -n kne2etests0'...
=== RUN   TestService/create_hello_service_duplicate_and_get_service_already_exists_error
Running 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests3'...
=== RUN   TestBasicWorkflow/create_hello_service_and_return_no_error
Running 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests0'...
=== RUN   TestRevision/create_hello_service_and_return_no_error
Running 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests2'...
Failed to successfully execute 'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests3': Execution error: stderr: 'RevisionFailed: Revision "hello-qj4nm" failed with message: Unable to fetch image "gcr.io/knative-samples/helloworld-go": serviceaccounts "default" not found.
' error: 'exit status 1'FAIL	github.com/knative/client/test/e2e	1.024s

@rhuss
Copy link
Contributor Author

rhuss commented Aug 8, 2019

Yes, it looks like as every of those test setup a test namespace and tear it down after the test.

After its the first test of TestRevision, its the first test right after the namespace is setup. I guess this is just a race between the first test running and the namespace not fully setup. Let me add some seconds wait to this setup and see how it goes.

rhuss added a commit to rhuss/knative-client that referenced this issue Aug 8, 2019
knative-prow-robot pushed a commit that referenced this issue Aug 8, 2019
@navidshaikh navidshaikh added this to the v0.8.0 milestone Aug 29, 2019
navidshaikh added a commit to navidshaikh/client that referenced this issue Apr 14, 2020
* Update Dockerfiles

* Update ENTRYPOINT and CMD, add shebang to python script

* Remove 'image' from display name and description in labels
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
3 participants