diff options
author | 2015-10-15 16:54:52 -0700 | |
---|---|---|
committer | 2015-10-15 16:54:52 -0700 | |
commit | 22b6bed4835ebc044c6b7ce415ea4c62196dd9d6 (patch) | |
tree | 87bbcffdfe3c21cd024f9793c582761a18b70405 | |
parent | 5e75116b9f8e0c60e3a3705aac132ee5fd2ac491 (diff) |
revert global_env usage for build_interop_image
-rwxr-xr-x | tools/run_tests/run_interop_tests.py | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/tools/run_tests/run_interop_tests.py b/tools/run_tests/run_interop_tests.py index a4b6a747ed..fe8050d1bc 100755 --- a/tools/run_tests/run_interop_tests.py +++ b/tools/run_tests/run_interop_tests.py @@ -230,10 +230,7 @@ class PHPLanguage: return _SSL_CERT_ENV def global_env(self): - # need to manually copy to each jenkins machine if we run into github - # rate limit when running `composer install` - return {"BUILD_INTEROP_DOCKER_EXTRA_ARGS": - "-v /var/local/.composer/auth.json:/root/.composer/auth.json:ro"} + return {} def __str__(self): return 'php' @@ -475,16 +472,21 @@ def server_jobspec(language, docker_image): def build_interop_image_jobspec(language, tag=None): """Creates jobspec for building interop docker image for a language""" - environ = language.global_env() if not tag: tag = 'grpc_interop_%s:%s' % (language.safename, uuid.uuid4()) - environ['INTEROP_IMAGE'] = tag - environ['BASE_NAME'] = 'grpc_interop_%s' % language.safename + env = {'INTEROP_IMAGE': tag, + 'BASE_NAME': 'grpc_interop_%s' % language.safename} if not args.travis: - environ['TTY_FLAG'] = '-t' + env['TTY_FLAG'] = '-t' + # This env variable is used to get around the github rate limit + # error when running the PHP `composer install` command + # TODO(stanleycheung): find a more elegant way to do this + if language.safename == 'php': + env['BUILD_INTEROP_DOCKER_EXTRA_ARGS'] = \ + "-v /var/local/.composer/auth.json:/root/.composer/auth.json:ro" build_job = jobset.JobSpec( cmdline=['tools/jenkins/build_interop_image.sh'], - environ=environ, + environ=env, shortname="build_docker_%s" % (language), timeout_seconds=30*60) build_job.tag = tag |