Bug 169084 – Add a script for automatically running the buildbot configurations on a test environment.

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Carlos Alberto Lopez Perez]

I have been working on a script that does the following:

 * Creates a temporal workingdir
 * Installs buildbot and its dependencies there (very same version used by the webkit.org buildbot master https://build.webkit.org/about ). This is important as there are problems that that may not be detected if you don't test with the same version that will be used on production.
 * Copies the buildmaster config files to the temporal workingdir.
 * Performs the required steps to start a buildmaster.
 * Creates the required config for each slave and starts all of them.


Then you can use your browser to log into the test buildmaster at http://localhost:8710/ and test to trigger manual builds, etc.

So you can actually test any configuration changes on the buildmaster on a test environment that is pretty much identical to the production one.

When you finish testing, simply press CTRL+C and all the temporal files will be cleaned. Note that if you actually want to trigger builds, the default path for the temporal workingdir on Linux (/tmp/) may have not enough free space. You can specify another temporal directory with the --base-temp-dir argument.

[Carlos Alberto Lopez Perez]

Created attachment 303216 [details]
Patch

[WebKit Commit Bot]

Attachment 303216 [details] did not pass style-queue:


ERROR: Tools/BuildSlaveSupport/build.webkit.org-config/run-buildbot-test.py:213:  [start_worker] Instance of 'Popen' has no 'kill' member  [pylint/E1101] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Alexey Proskuryakov]

Interesting!

Aakash and Dean, did you have some automation for setting up test buildbot instances? What do you think?

[Carlos Alberto Lopez Perez]

Created attachment 303246 [details]
Patch

Fix some typos and improve the wording on the help strings for the arguments

[WebKit Commit Bot]

Attachment 303246 [details] did not pass style-queue:


ERROR: Tools/BuildSlaveSupport/build.webkit.org-config/run-buildbot-test.py:213:  [start_worker] Instance of 'Popen' has no 'kill' member  [pylint/E1101] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Carlos Alberto Lopez Perez]

ping reviewers?

[Michael Catanzaro]

Comment on attachment 303246 [details]
Patch

I have not actually reviewed this script. Maybe that's bad of me. :) But you can commit it.

[Aakash Jain]

Thanks for adding the script. I have verified that the script works fine.

Let me review it today. I might have few comments.

[Michael Catanzaro]

Comment on attachment 303246 [details]
Patch

Sure, I was just approving it because I figured nobody would want to review it.

[Michael Catanzaro]

(In reply to Aakash Jain from comment #8)
> Thanks for adding the script. I have verified that the script works fine.
> 
> Let me review it today. I might have few comments.

Aakash, are you still planning to review this?

[Michael Catanzaro]

Comment on attachment 303246 [details]
Patch

(Carlos Lopez, you can commit this, barring future negative feedback.)

[Aakash Jain]

Yeah, please go ahead. The script works. We can make the improvements on ongoing basis.

[Aakash Jain]

Carlos, can you please commit it?

[WebKit Commit Bot]

Comment on attachment 303246 [details]
Patch

Clearing flags on attachment: 303246

Committed r225336: <https://trac.webkit.org/changeset/225336>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/35777704>