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

Think of faster build system #67

Closed
grzanka opened this issue Jun 15, 2016 · 6 comments
Closed

Think of faster build system #67

grzanka opened this issue Jun 15, 2016 · 6 comments
Milestone

Comments

@grzanka
Copy link
Collaborator

grzanka commented Jun 15, 2016

It looks that with current configuration travis is pretty slow.
To merge a PR we need to wait >30 mins.

We should think on how to make commit + PR integration loop faster.

One idea would be to:

  • have a more compact and fewer tests on travis
  • run more tests elsewhere
    • on dedicated jenkins (we can have VM in PLGRID running it)
    • make a mirror of this repo on gitlab. Gitlab has very advanced embedded CI solution (gitlab runners). We can run have own, docker based runners in PLGRID. Currently gitlab can sync with github every 60 min, but with some tricks this can be faster.

In this way:

  • short tests have passed and long ones are pending : admin decided whether to merge or not
  • short tests have failed - we don't merge
  • short and long tests passed - we probably merge

We can also add some bot and system of +1, -1 voting, like here: cms-sw/cmssw#14889

@grzanka
Copy link
Collaborator Author

grzanka commented Jun 15, 2016

It would be also good to know why the running whole test machine is slow.

@grzanka grzanka added this to the Next release milestone Jun 15, 2016
@antnieszka
Copy link
Contributor

antnieszka commented Jun 22, 2016

Have you investigated wercker?

@grzanka
Copy link
Collaborator Author

grzanka commented Jun 22, 2016

I've just added beprof to wercker (and made you admin). You can play around on https://github.com/DataMedSci/beprof/tree/feature/67-wercker-test

@antnieszka
Copy link
Contributor

antnieszka commented Jun 22, 2016

To avoid this travis thing I'm testing wercker on beprof fork: https://app.wercker.com/#applications/576a55b319a594010006c48e I've managed to create two-pipeline workflow with pytest.

@grzanka
Copy link
Collaborator Author

grzanka commented Jun 22, 2016

As you see builds are failing because wercker.yml is missing in MANIFEST file. Please add it to manifest.

@antnieszka
Copy link
Contributor

Do we still need this? Or just improve wercker config (there is another issue for that, I think)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants