-
Notifications
You must be signed in to change notification settings - Fork 579
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
operator/tests: waiting for deployment in each step is not needed #714
Conversation
the diff is really weird but I really just renamed all the file decrementing the steps numbers so 01 -> 00 etc. |
Couldn't we start our tests from |
@RafalKorepta yeah we could but why? Only to reduce diff on this PR? :) Since I already changed it we could do the "proper" indexing from 0 :) 🤷♀️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/gcbrun |
this is because we're already waiting for webhook to be ready during test suite setup and deployment and webhook run in the same process.
54f5931
to
04b4eaa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
this is because we're already waiting for webhook to be ready during test suite setup and deployment and webhook run in the same process.
fixes #655
Checklist
When referencing a related issue, remember to migrate duplicate stories from the
external tracker. This is not relevant for most users.