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

Add Windows slave hosted by Rackspace #215

Merged
merged 5 commits into from
Jul 5, 2016
Merged

Conversation

adiroiban
Copy link
Member

@adiroiban adiroiban commented Jul 3, 2016

Scope

In this way it should be easier to allow people to access it for maintenance

I saw that we are using only about $700 of rax invoice.

The windows slave was quite busy sometimes so a second slave should help

Changes

I have created the smallest win2012r2 windows vm on RAX

The private repo was already pushed.

I am documenting the steps required to have a windows slave here https://twistedmatrix.com/trac/wiki/ContinuousIntegration/Buildslaves#BuildslaveonWindows

For now the new slaves are in the unsuported category as I did not managed to start buildslave a a service.

I can start it in debug mode... but when I use the Windows service tool, it fails to start

The builddir option is no longer used as I think that it is automatically created by buildslave based on the builder name

To do

  • Start buildslave as a service

How to test

Changes are deployed in production

https://buildbot.twistedmatrix.com/builders/win-64-py2.7

https://buildbot.twistedmatrix.com/builders/win-64-py2.7-coverage

@adiroiban
Copy link
Member Author

adiroiban commented Jul 3, 2016

the bdist builder fails to upload the file... I am still investigating

please check that initial changes are ok.

@tomprince I followed the wiki page from here http://trac.buildbot.net/wiki/RunningBuildbotOnWindows but BuildBot service is still not working ... the registry are created and I can start the server in debug mode using buildbot_service debug, even when I am connected with as the dedicated service account

Thanks!

@@ -80,6 +80,8 @@ ubuntu16_04_slaves = ['rax-ubuntu1604-%d' % (i,) for i in range(1,4)]

allpy27_slaves = fedora22_slaves + fedora23_slaves + ubuntu12_04_slaves + ubuntu16_04_slaves

win_slaves = ['bot-glyph-6', 'rax-win2012r2-1']
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't look like this is used anywhere?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. I will rename it. It was there as a reminder of my intention to unify .

I will remove and use a separate win2012r2 builder

@hawkowl
Copy link
Member

hawkowl commented Jul 5, 2016

See comments, fix them up and then merge.

@adiroiban
Copy link
Member Author

Thanks. I am adding it as unsupported for now.

It can be used to send coverage reports for windows in case win7 is down :)

I hope that we can get some VMs from Azure and them move all Windows slaves there.

@adiroiban adiroiban merged commit 7e9205c into master Jul 5, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants