-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Discussing adding separate system tests for REST and gRPC. #2658
Comments
The Travis cron jobs docs say:
They also say:
Detecting a cron-triggered build:
|
@tseaver I think we could make 4 cron jobs to get it to run 4 times a day. Just change the time span of when each one can run in. |
@daspecster Looking at the dialog for adding a cron job. I don't see any knob for tweaking time-of-day. |
Why would we need to run more than once a day? |
@lukesneeringer This is actually important. |
I actually thought this was done. |
@lukesneeringer ISTM that the "correct" way to proceed is to
|
At this point I am closing this:
|
We can call kokoro "kokoro". ;)
|
Per what we talked about a little bit today.
It would probably be a good idea to run both the REST and gRPC code in the system tests.
I think most services that could have both REST and gRCP tests could use
skipTest
to manage when it is appropriate to run them.(#2648 (comment))When to run them?
I think @dhermes brought up that we shouldn't run both of them all the time and that maybe we should use Travis-CI cron to run them once or twice a day. We could also just cron building master 1-4 times per 24 hours instead of on every merge.
The text was updated successfully, but these errors were encountered: