Merge pull request containers#126 from lmilbaum/chatbot

Spin Model Server in chatbot recipe

.github/workflows/chatbot.yaml

@@ -4,15 +4,9 @@ on:
   pull_request:
     branches:
       - main
-    paths:
-      - ./recipes/natural_language_processing/chatbot/**
-      - .github/workflows/chatbot.yaml
   push:
     branches:
       - main
-    paths:
-      - ./recipes/natural_language_processing/chatbot/**
-      - .github/workflows/chatbot.yaml
 
 env:
   REGISTRY: ghcr.io
@@ -49,11 +43,14 @@ jobs:
 
       - name: Set up Python
         uses: actions/[email protected]
+        with:
+          python-version: '3.11'
 
-      - name: Install python dependencies
+      - name: Install dependencies
         working-directory: ./recipes/natural_language_processing/chatbot
         run: make install
 
       - name: Run tests
         working-directory: ./recipes/natural_language_processing/chatbot
         run: make test
+

.github/workflows/model_servers.yaml

@@ -4,15 +4,9 @@ on:
   pull_request:
     branches:
       - main
-    paths:
-      - ./model_servers/llamacpp_python/**
-      - .github/workflows/model_servers.yaml
   push:
     branches:
       - main
-    paths:
-      - ./model_servers/llamacpp_python/**
-      - .github/workflows/model_servers.yaml
 
 env:
   REGISTRY: ghcr.io

recipes/natural_language_processing/chatbot/Makefile

@@ -29,6 +29,13 @@ quadlet:
 
 .PHONY: install
 install:
+	# wget -qO- https://github.com/mozilla/geckodriver/releases/download/v0.34.0/geckodriver-v0.34.0-linux64.tar.gz | tar xvz
+	# sudo apt update
+	# sudo apt upgrade
+	wget https://www.slimjetbrowser.com/chrome/files/103.0.5060.53/google-chrome-stable_current_amd64.deb
+	sudo dpkg -i google-chrome-stable_current_amd64.deb
+	wget https://chromedriver.storage.googleapis.com/103.0.5060.53/chromedriver_linux64.zip
+	unzip chromedriver_linux64.zip
 	pip install -r tests/requirements-test.txt
 
 .PHONY: run
@@ -37,4 +44,4 @@ run:
 
 .PHONY: test
 test:
-	pytest --log-cli-level NOTSET
+	python3 -m pytest -vvv --driver=Chrome --driver-path=./chromedriver tests

recipes/natural_language_processing/chatbot/tests/conftest.py

@@ -1,7 +1,32 @@
 import pytest_container
 import os
+import pytest
+from selenium import webdriver
 
 
+MS = pytest_container.Container(
+        url=f"containers-storage:{os.environ['REGISTRY']}/model_servers",
+        volume_mounts=[
+            pytest_container.container.BindMount(
+                container_path="/locallm/models",
+                host_path="./",
+                flags=["ro"]
+            )
+        ],
+        extra_environment_variables={
+            "MODEL_PATH": "models/mistral-7b-instruct-v0.1.Q4_K_M.gguf",
+            "HOST": "0.0.0.0",
+            "PORT": "8001"
+        },
+        forwarded_ports=[
+            pytest_container.PortForwarding(
+                container_port=8001,
+                host_port=8001
+            )
+        ],
+        extra_launch_args=["--net=host"]
+    )
+
 CB = pytest_container.Container(
         url=f"containers-storage:{os.environ['REGISTRY']}/{os.environ['IMAGE_NAME']}",
         extra_environment_variables={
@@ -21,3 +46,20 @@ def pytest_generate_tests(metafunc):
 
 def pytest_addoption(parser):
     pytest_container.add_logging_level_options(parser)
+
+@pytest.fixture
+def firefox_options(firefox_options):
+    firefox_options.headless = True
+    firefox_options.set_preference("webdriver.log.file", "/tmp/driver.log")
+    return firefox_options
+
+@pytest.fixture
+def custom_selenium(selenium, firefox_options):
+    selenium.webdriver = webdriver.Firefox(firefox_options=firefox_options)
+    yield selenium
+    selenium.webdriver.quit()
+
+@pytest.fixture
+def chrome_options(chrome_options):
+    chrome_options.add_argument("--headless")
+    return chrome_options

recipes/natural_language_processing/chatbot/tests/test_alive.py → recipes/natural_language_processing/chatbot/tests/test_app.py

@@ -10,3 +10,8 @@ def test_etc_os_release_present(auto_container: pytest_container.container.Conta
 @tenacity.retry(stop=tenacity.stop_after_attempt(5), wait=tenacity.wait_exponential())
 def test_alive(auto_container: pytest_container.container.ContainerData, host):
     host.run_expect([0],f"curl http://localhost:{auto_container.forwarded_ports[0].host_port}",).stdout.strip()
+
+def test_title(auto_container: pytest_container.container.ContainerData, selenium):
+    selenium.get(f"http://localhost:{auto_container.forwarded_ports[0].host_port}")
+    assert selenium.title == "Streamlit"
+

requirements-test.txt

@@ -1,6 +1,8 @@
 pip==24.0
 pytest-container==0.4.0
+pytest-selenium==4.1.0
 pytest-testinfra==10.1.0
 pytest==8.1.1
 requests==2.31.0
+selenium==4.19.0
 tenacity==8.2.3