Browse Source

Remove redundant toolset environment flag (#4518)

* Remove reduntant toolset environment flag

* fix travis.yml

* update benchmark_config
Nate 6 years ago
parent
commit
2a9b8ee02e
4 changed files with 2 additions and 7 deletions
  1. 1 1
      .travis.yml
  2. 0 4
      toolset/run-tests.py
  3. 0 1
      toolset/utils/benchmark_config.py
  4. 1 1
      toolset/utils/docker_helper.py

+ 1 - 1
.travis.yml

@@ -130,4 +130,4 @@ script:
   # run-ci.py runs the diffing to see if travis needs to test this framework. Ideally/eventually,
   # we'd like to try and do the diffing before travis_clean & setup.
   # This will run the tests exactly as you would in your own vm:
-  - if [ "$RUN_TESTS" ]; then docker network create tfb > /dev/null 2>&1 && docker run --network=tfb -v /var/run/docker.sock:/var/run/docker.sock --mount type=bind,source=`pwd`,target=/FrameworkBenchmarks techempower/tfb --mode verify --test-dir $RUN_TESTS --benchmark-env travis; else echo 'Skipping test verification.'; fi
+  - if [ "$RUN_TESTS" ]; then docker network create tfb > /dev/null 2>&1 && docker run --network=tfb -v /var/run/docker.sock:/var/run/docker.sock --mount type=bind,source=`pwd`,target=/FrameworkBenchmarks techempower/tfb --mode verify --test-dir $RUN_TESTS --results-environment Travis; else echo 'Skipping test verification.'; fi

+ 0 - 4
toolset/run-tests.py

@@ -182,10 +182,6 @@ def main(argv=None):
         nargs='+',
         default=[1, 10, 20, 50, 100],
         help='List of cached query levels to benchmark')
-    parser.add_argument(
-        '--benchmark-env',
-        default='none',
-        help='Hostname/IP for database server')
 
     # Network options
     parser.add_argument(

+ 0 - 1
toolset/utils/benchmark_config.py

@@ -43,7 +43,6 @@ class BenchmarkConfig:
         self.cached_query_levels = args.cached_query_levels
         self.pipeline_concurrency_levels = args.pipeline_concurrency_levels
         self.query_levels = args.query_levels
-        self.benchmark_env = args.benchmark_env
         self.parse = args.parse
         self.results_environment = args.results_environment
         self.results_name = args.results_name

+ 1 - 1
toolset/utils/docker_helper.py

@@ -45,7 +45,7 @@ class DockerHelper:
                     timeout=3600,
                     pull=True,
                     buildargs=({
-                      'BENCHMARK_ENV': self.benchmarker.config.benchmark_env
+                      'BENCHMARK_ENV': self.benchmarker.config.results_environment
                     })
                 )
                 buffer = ""