This is an automated email from the ASF dual-hosted git repository.

tvb pushed a commit to branch tristan/remote-execution-testing
in repository https://gitbox.apache.org/repos/asf/buildstream.git

commit 517e7b35014058d08ea0f5de9b0555e2ef79b0c7
Author: Tristan van Berkom <[email protected]>
AuthorDate: Wed Jan 20 16:06:21 2021 +0900

    .github/compose/ci.docker-compose.yml: Minor refactoring here
    
      * Comment the tests-template a bit more clearly, explain what is going on
    
      * Define TOXENV using variables from common.env
    
      * Remove redundant definition of BST_PLUGINS_EXPERIMENTAL_VERSION, this
        is already defined in tox.ini and overridden to `master` in one of
        the supported tests here.
    
        No need to have to update this in more than one place when it changes.
---
 .github/compose/ci.docker-compose.yml | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/.github/compose/ci.docker-compose.yml 
b/.github/compose/ci.docker-compose.yml
index 8d476a2..0a0da86 100644
--- a/.github/compose/ci.docker-compose.yml
+++ b/.github/compose/ci.docker-compose.yml
@@ -1,14 +1,20 @@
 version: '3.4'
 
 x-tests-template: &tests-template
+    image: 
registry.gitlab.com/buildstream/buildstream-docker-images/testsuite-fedora:32-${CI_IMAGE_VERSION:-latest}
     command: tox -vvvvv -- --color=yes --integration -n 4
-    devices:
-      - /dev/fuse:/dev/fuse
     environment:
-      BST_PLUGINS_EXPERIMENTAL_VERSION: 1.93.4
-      TOXENV: 
py36,py37,py38-nocover,py36-plugins,py37-plugins,py38-plugins-nocover
-    image: 
registry.gitlab.com/buildstream/buildstream-docker-images/testsuite-fedora:32-${CI_IMAGE_VERSION:-latest}
+      TOXENV: ${CI_TOXENV_ALL}
+
+    # Enable privileges to run the sandbox
+    #
     privileged: true
+    devices:
+      - /dev/fuse:/dev/fuse
+
+    # Mount the local directory and set the working directory
+    # to run the tests from.
+    #
     volumes:
       - ../..:/home/testuser/buildstream
     working_dir: /home/testuser/buildstream

Reply via email to