Enable Dynamic agent #2397
Enable Dynamic agent #2397
Conversation
|
@smlambert updated. We will also need to update azure VM test agent's label be |
|
re: #2397 (comment) azuretestagent is now updated with |
|
I will also note that this PR is using centos6_build_image as the test container. @sxa has been compiling a list of tests that do not run well in containers in #2138, and making updates to static container nodes he has created. I think it may be valuable to have a test container with any/all updates that Stewart has been making in his investigation and use it in place of the centos6_build_image. |
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
sophia-guo
force-pushed
the
azureVM
branch
2 times, most recently
from
689d3bf
to
984b10e
Compare
buildenv/jenkins/testJobTemplate
Outdated
| @@ -246,6 +245,7 @@ ARCH_OS_LIST.each { ARCH_OS -> | |||
| stringParam('UPSTREAM_TEST_JOB_NAME', "", "Auto-populated. Upstream test job name. It will be used together with PARALLEL=Dynamic") | |||
| stringParam('UPSTREAM_TEST_JOB_NUMBER', "", "Auto-populated. Upstream test job number. It will be used together with PARALLEL=Dynamic") | |||
| booleanParam('LIGHT_WEIGHT_CHECKOUT', LIGHT_WEIGHT_CHECKOUT.toBoolean(), "Optional. Default is false for Grinders") | |||
| stringParam('CLOUD_PROVIDER', CLOUD_PROVIDER, "available cloud provider") | |||
There was a problem hiding this comment.
Minor nitpick, capitalize Available to match the other param descriptors that are capitalized
There was a problem hiding this comment.
Since we are updating this line, could you also provide examples?
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
| } | ||
| } else { | ||
| boolean isNodeIdle = false | ||
| node('master') { |
There was a problem hiding this comment.
Eclipse OpenJ9 Jenkins does not have any executors on master so this won't work there.
Signed-off-by: Sophia Guo sophia.gwf@gmail.com