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

Discussing adding separate system tests for REST and gRPC. #2658

Closed
daspecster opened this issue Nov 1, 2016 · 11 comments
Closed

Discussing adding separate system tests for REST and gRPC. #2658

daspecster opened this issue Nov 1, 2016 · 11 comments
Assignees
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. testing

Comments

@daspecster
Copy link
Contributor

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.

@tseaver
Copy link
Contributor

tseaver commented Nov 1, 2016

The Travis cron jobs docs say:

Cron jobs can run daily, weekly or monthly, which in practice means up to an hour after the selected time span, and they can also be skipped. Cron jobs cannot be set to run at specific times.

They also say:

Please note that cron jobs are not enabled by default. Set “Build pushes” to on in your settings, then ask us to unlock this feature for your repository: support@travis-ci.com.

Detecting a cron-triggered build:

To check whether a build was triggered by cron, examine the TRAVIS_EVENT_TYPE environment variable to see if it has the value cron.

@daspecster
Copy link
Contributor Author

daspecster commented Nov 1, 2016

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

@tseaver
Copy link
Contributor

tseaver commented Nov 1, 2016

@daspecster Looking at the dialog for adding a cron job. I don't see any knob for tweaking time-of-day.

@dhermes
Copy link
Contributor

dhermes commented Nov 1, 2016

Why would we need to run more than once a day?

@daspecster
Copy link
Contributor Author

@tseaver maybe not, I'm not sure what's in the options drop down there. The way it's worded makes it sound like you may be able to select some portion of the day.

@dhermes not sure that we do, but it would be nice to have it run at night or early in the morning.

@lukesneeringer lukesneeringer added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 19, 2017
@lukesneeringer lukesneeringer removed the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 19, 2017
@dhermes
Copy link
Contributor

dhermes commented Apr 19, 2017

@lukesneeringer This is actually important.

@lukesneeringer
Copy link
Contributor

I actually thought this was done.

@dhermes
Copy link
Contributor

dhermes commented Apr 19, 2017

@lukesneeringer ISTM that the "correct" way to proceed is to

  • Do both HTTP and gRPC in nightlies
  • One of
    • Never do HTTP in "push" builds
    • Randomly pick one of HTTP / gRPC in "push" builds based on e.g. build_number % 2

@lukesneeringer lukesneeringer added the priority: p2 Moderately-important priority. Fix may not be included in next release. label Apr 19, 2017
@lukesneeringer
Copy link
Contributor

At this point I am closing this:

  • It is not doing us any good multiple pages down.
  • This kind of thing will be solved by the CI rehash when we split the repos and use Google's internal Jenkins magic thing that I-do-not-know-if-the-codename-is-public.

@theacodes
Copy link
Contributor

theacodes commented Aug 8, 2017 via email

@lukesneeringer
Copy link
Contributor

lukesneeringer commented Aug 8, 2017

Haha. In fairness, see #3479 (specifically comment). Which, it should be noted, I got to later, and was searching for this ticket to link there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. testing
Projects
None yet
Development

No branches or pull requests

5 participants