Skip to content
This repository has been archived by the owner on Jun 1, 2024. It is now read-only.

Create tox / venv builder for all builders #223

Merged
merged 2 commits into from
Sep 8, 2016
Merged

Conversation

adiroiban
Copy link
Member

@adiroiban adiroiban commented Aug 23, 2016

This requires Twisted updates https://twistedmatrix.com/trac/ticket/8544 so that we have tox env for everything

This creates tox/venv builders for nomodules and bdist ... and windows builders

After this merged, #220 should be easy to fix

windows builders are stuck at cleaning/recreating the venv

@glyph
Copy link
Contributor

glyph commented Aug 29, 2016

This looks fine to deploy, just eyeballing it, but is there any way to verify the buildmaster configuration on the dev VM, or do we just have to deploy and hope?

👍 , such as it is on this repo :)

@adiroiban
Copy link
Member Author

The nomodules can be tested in the dev VM, as it is running Linux.
The bdist and windows with or without coverage can't be tested using braid... but you can set up a Win VM, start a slave on it and trigger a build

Installing buildslave on windows is documented here https://twistedmatrix.com/trac/wiki/ContinuousIntegration/Buildslaves#BuildslaveonWindows

But, no, we don't have a vagrant config form creating a Windows VM and putting it all together so that you can have the windows slave up and running just by using a command.

I read the docs about getting a Win guest with Vagrant (http://huestones.co.uk/node/305) but for our needs and due to licensing issues, using Vagrant looks more complicated than doing a manual Win slave.


The description was wrong.
The related ticket is https://twistedmatrix.com/trac/ticket/8544

so you would help to review 8544 to see if the tox env names are ok as otherwise we will need

@hawkowl hawkowl merged commit b57a7c1 into master Sep 8, 2016
@hawkowl hawkowl deleted the 212-venv-tox-everywhere branch September 8, 2016 14:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants