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
Set up a post-commit bot to test workspace builds #155
Set up a post-commit bot to test workspace builds #155
Conversation
|
'clean-and-update-working-directory', | ||
'show-identifier', | ||
'kill-old-processes', | ||
'delete-WebKitBuild-directory', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we persist WebKitBuild between builds, like EWS does? One thing I'd like this bot to be testing is whether incremental builds are successful.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Though impossible to determine from this unit test, the 'delete-WebKitBuild-directory' step would only run if someone triggered a clean build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, nevermind then :)
@@ -292,6 +292,11 @@ | |||
"platform": "mac-bigsur", "configuration": "release", "architectures": ["x86_64", "arm64"], | |||
"workernames": ["bot184"] | |||
}, | |||
{ "name": "Apple-BigSur-XCBuild-Release-Build", "factory": "BuildFactory", "builddir": "bigsur-release", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like EWS caught a bug for you:
Configuration Errors:
duplicate builder builddir 'bigsur-release'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now I have to wonder why python3 runUnittests.py build-webkit-org
told me all tests passed!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was buildbot checkconfig
that actually flagged the failure, looking at the logs.
7680acd
to
bde66c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Naming and build arguments LGTM!
Testing here before landing: https://build.webkit-dev.org/#/builders/123/builds/1 |
bde66c3
to
168bc34
Compare
168bc34