-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
pbTests: Allow use of default values for make-adopt-build-farm.sh #1956
Conversation
Looks like the build PRs blocking it are merged now, so I'll test this :-) |
b9b7a08
to
6a3e754
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.
Otherwise, LGTM - I'm assuming this has already been put through VPC?
EDIT: The VPC runs you linked above failed, as did the most recent run(s) of VPC that are testing your branch (1072 and 1073). I can see the errors are unbounded variables, which I assume are fixed with the addition of :-}
to the debug output?
b12a4cd
to
933180c
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.
LGTM
Signed-off-by: Stewart X Addison <[email protected]>
https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1078/OS=Ubuntu1804,label=vagrant/console has passed so taking this out of draft. |
Not a clue if I've stripped too much out or not, but might as well give it a shot ... Now possible thanks to adoptium/temurin-build#2167 adoptium/temurin-build#2479 adoptium/temurin-build#2482 but requires adoptium/temurin-build#2478 but weill require adoptium/temurin-build#2478 to be merged before testing
Signed-off-by: Stewart X Addison [email protected]
Checklist