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.

Sign up for GitHub

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

Jump to bottom

e2e flake: service create - referenced revision not found #544

Closed
navidshaikh opened this issue Dec 10, 2019 · 6 comments
Closed

e2e flake: service create - referenced revision not found #544

navidshaikh opened this issue Dec 10, 2019 · 6 comments
Labels
area/e2e kind/bug Categorizes issue or PR as related to a bug.

Comments

@navidshaikh
Copy link
Collaborator

navidshaikh commented Dec 10, 2019 •
edited
Loading 

Failed to successfully execute 
'kn service create hello --image gcr.io/knative-samples/helloworld-go -n kne2etests5': 
Execution error: stderr: 'RevisionMissing: Revision "hello-bzbty-1" 
referenced in traffic not found.

ref
@navidshaikh navidshaikh added the kind/bug Categorizes issue or PR as related to a bug. label Dec 10, 2019
@maximilien
Copy link
Contributor

I've seen this (or very similar) with kn revision update .... I'll track it down and see if the resulting message is same or different and open an issue or report here appropriately.

@navidshaikh navidshaikh mentioned this issue Jan 10, 2020
Merged
@navidshaikh
Copy link
Collaborator Author

navidshaikh commented Jan 22, 2020 •
edited
Loading

Every now and then this error comes up, latest one against serving v0.12.0 release here

@navidshaikh
Copy link
Collaborator Author

++
logs

@rhuss rhuss mentioned this issue Jan 22, 2020
Closed
@navidshaikh
Copy link
Collaborator Author

++
logs

@navidshaikh navidshaikh mentioned this issue Feb 5, 2020
Merged
@rhuss rhuss added the area/e2e label Feb 5, 2020
rhuss added a commit to rhuss/knative-client that referenced this issue Feb 7, 2020
@rhuss
fix(serving): Add an error window for ReadyCondition
be0267c 
A default error window of 2 seconds (not sure if this should be made configurable
as it is hard to explain/document) cause return an error during
sync operations only, if a ReadyCondition == false stays for that
long in false. This is needed to compensate situation where the
condition is false when a race between revision and route creation
occurse during a service creation.

This should fix the flakes we have in our E2E tests as described in knative#544.
@rhuss rhuss mentioned this issue Feb 7, 2020
Merged
knative-prow-robot pushed a commit that referenced this issue Feb 7, 2020
@rhuss
fix(serving): Add an error window for ReadyCondition (#644)
f634ab9 
* fix(serving): Add an error window for ReadyCondition

A default error window of 2 seconds (not sure if this should be made configurable
as it is hard to explain/document) cause return an error during
sync operations only, if a ReadyCondition == false stays for that
long in false. This is needed to compensate situation where the
condition is false when a race between revision and route creation
occurse during a service creation.

This should fix the flakes we have in our E2E tests as described in #544.

* fix: words, words

* fix(serving): More words + a check
@rhuss rhuss mentioned this issue Feb 8, 2020
Closed
coryrc pushed a commit to coryrc/client that referenced this issue May 14, 2020
@srinivashegde86 @knative-prow-robot
Use the common monitoring component for prometheus (knative#544)
804daea
@navidshaikh
Copy link
Collaborator Author

/close

@knative-prow-robot
Copy link
Contributor

@navidshaikh: Closing this issue.

In response to this:

/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.

dsimansk pushed a commit to dsimansk/client that referenced this issue Dec 9, 2020
@navidshaikh
Update SPEC file for v0.18.4 (knative#544)
2dd4334
dsimansk pushed a commit to dsimansk/client that referenced this issue Dec 10, 2024
@serverless-qe
[release-v1.15] Add Konflux configurations (knative#544)
dc79e5c 
Add Konflux components and pipelines

Co-authored-by: serverless-qe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/e2e kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@maximilien @rhuss @navidshaikh @knative-prow-robot