avoid ./ in docker container - use better /app (#88)

* avoid ./ in docker container - use better /app

* fix local_config

* impact on container name change

Author: echambla

.github/workflows/ci_cd.yml

@@ -92,8 +92,8 @@ jobs:
           docker-compose up -d
           cd tests  # so we're testing the install, not local
           pytest -vx
-          docker kill pyoptics_speos-rpc 
-          docker rm pyoptics_speos-rpc
+          docker kill optics_speos-rpc
+          docker rm optics_speos-rpc
         env:
           LICENSE_SERVER: ${{ format('1055@{0}', secrets.LICENSE_SERVER) }}
 

README.rst

@@ -120,7 +120,7 @@ And test configuration file `<tests/local_config.json>`_ must be updated to use
    
    {
       "SpeosServerOnDocker": false,
-      "SpeosContainerName" : "pyoptics_speos-rpc",
+      "SpeosContainerName" : "optics_speos-rpc",
       "SpeosServerPort": 50051
    }
 

docker-compose.yml

@@ -1,14 +1,14 @@
 version: "3.4"
 services:
-  pyoptics_speos-rpc:
+  optics_speos-rpc:
     image: ghcr.io/pyansys/pyoptics/speos-rpc:2023.2.0.305
-    container_name: pyoptics_speos-rpc
+    container_name: optics_speos-rpc
     ports:
       - "50051:50051"
     environment:
       - SPEOS_LOG_LEVEL=2
       - ANSYSLMD_LICENSE_FILE=$LICENSE_SERVER
-    entrypoint: ./SpeosRPC_Server.x
+    entrypoint: /app/SpeosRPC_Server.x
     volumes:
       - type: bind
         source: ./tests/assets

tests/local_config.json

@@ -1,5 +1,5 @@
 {
   "SpeosServerOnDocker": true,
-  "SpeosContainerName" : "pyoptics_speos-rpc",
+  "SpeosContainerName" : "optics_speos-rpc",
   "SpeosServerPort": 50051
 }