📦 feat(inspect): Add docker build #3136

ashwinvaidya17 · 2025-11-21T09:30:10Z

📝 Description

Add CPU build

TODO

Add GPU build
Add XPU build

✨ Changes

Select what type of change your PR is:

✅ Checklist

Before you submit your pull request, please make sure you have completed the following steps:

📚 I have made the necessary updates to the documentation (if applicable).
🧪 I have written tests that support my changes and prove that my fix is effective or my feature works (if applicable).
🏷️ My PR title follows conventional commit format.

For more information about code review checklists, see the Code Review Checklist.

Mainly based on the UI files and architecture from [Geti Tune](https://github.com/open-edge-platform/training_extensions/tree/develop/ui)

* initial backend commit Signed-off-by: Ma, Xiangxiang <[email protected]> * app -> src Signed-off-by: Ma, Xiangxiang <[email protected]> * Remove empty file Signed-off-by: Ma, Xiangxiang <[email protected]> * move code Signed-off-by: Ma, Xiangxiang <[email protected]> * fix style backend Signed-off-by: Ma, Xiangxiang <[email protected]> * rename media endpoint Signed-off-by: Ma, Xiangxiang <[email protected]> --------- Signed-off-by: Ma, Xiangxiang <[email protected]>

…#2942) add unit tests for endpoints Signed-off-by: Ma, Xiangxiang <[email protected]>

These won't be needed yet

…tform#2948) * Use src folder inside run.sh * Set openapi_url path * Update to react 19 * Specify bash language in readme

…edge-platform#2945) * add training + inference endpoint Signed-off-by: Ma, Xiangxiang <[email protected]> * remove model api Signed-off-by: Ma, Xiangxiang <[email protected]> * cleanup code Signed-off-by: Ma, Xiangxiang <[email protected]> * update async execution Signed-off-by: Ma, Xiangxiang <[email protected]> * improve training worker loop and predict endpoint Signed-off-by: Ma, Xiangxiang <[email protected]> * fix style Signed-off-by: Ma, Xiangxiang <[email protected]> * fix style to use python3.10 generics Signed-off-by: Ma, Xiangxiang <[email protected]> * add tests for services Signed-off-by: Ma, Xiangxiang <[email protected]> * style fix Signed-off-by: Ma, Xiangxiang <[email protected]> * style fix Signed-off-by: Ma, Xiangxiang <[email protected]> * style fix Signed-off-by: Ma, Xiangxiang <[email protected]> * style fix Signed-off-by: Ma, Xiangxiang <[email protected]> * style fix Signed-off-by: Ma, Xiangxiang <[email protected]> --------- Signed-off-by: Ma, Xiangxiang <[email protected]>

…edge-platform#2961) * Add github actions for ui and server of geti inspect * Exclude UI from pre-commit prettier configuration The UI uses a different prettier configuration that does not seem to be picked up by pre-commit. * Add newline to .prettierignore * Apply prettier to `geti-inspect.yaml` * Generate OpenAPI spec before running UI checks * Checkout with lfs * Fix lint issues by removing wip components * Try installing git lfs in the playwright docker image * Fix unused noqa

…edge-platform#2963) * Add OpenAPI route * Remove MSW browser worker * Rename infernece to inspect * Update navbar title * Rename infernece to inspect

* rename models/ to pydantic_models/ Signed-off-by: Ma, Xiangxiang <[email protected]> * switch to use async session context manager Signed-off-by: Ma, Xiangxiang <[email protected]> * fix unit tests Signed-off-by: Ma, Xiangxiang <[email protected]> * add pipeline endpoints Signed-off-by: Ma, Xiangxiang <[email protected]> * fix frame aquisition worker and rename pipiline endpoints Signed-off-by: Ma, Xiangxiang <[email protected]> * add sources and sinks endpoints Signed-off-by: Ma, Xiangxiang <[email protected]> * stream loading working Signed-off-by: Ma, Xiangxiang <[email protected]> * add webrtc endpoints Signed-off-by: Ma, Xiangxiang <[email protected]> * fix workers: stream loading + inference + dispatcher Signed-off-by: Ma, Xiangxiang <[email protected]> * fix sinks Signed-off-by: Ma, Xiangxiang <[email protected]> * style Signed-off-by: Ma, Xiangxiang <[email protected]> * add unit tests and address comments Signed-off-by: Ma, Xiangxiang <[email protected]> * add tests Signed-off-by: Ma, Xiangxiang <[email protected]> * fix example schema Signed-off-by: Ma, Xiangxiang <[email protected]> * add todo Signed-off-by: Ma, Xiangxiang <[email protected]> --------- Signed-off-by: Ma, Xiangxiang <[email protected]>

…2970) * chore: Add path alias to icons * feat: Add sidebar with dataset, models and stats

* feat: Display placeholders for images * refactor: Fix scrollbar * chore: Remove learn more

* chore(inspect): Update UI scripts * chore: Update github actions * chore: Update port to 8000

…en-edge-platform#2978) chore(inspect): Configure project(s) mocks for playwright

* chore: Add project route * feat: Add project management * revert ui lock change * chore: Remove not needed code for project management

…ts from the UI (open-edge-platform#2980) chore(inspect): Update allowed origins

… training progress (open-edge-platform#2984) * feat: Allow user to upload images * feat: List uploaded images * chore: Extract components to separate files and add ready to train and training progress * chore: comment thumbnail url generation

chore(inspect): Update openapi page title

…en-edge-platform#2989) * chore(inspect): Renamed app to application * chore(inspect): Rename app to application in github actions

…of email (open-edge-platform#2990) refactor: Update photo placeholder to use indicator instead of email

…dge-platform#2991) Improve error and suspense handling in router By moving all of the routes into a single root route we can make sure that all routes are rendered inside of an layout that has a suspense and error boundary.

Signed-off-by: Ashwin Vaidya <[email protected]>

…open-edge-platform#2992) refactor: Improvements to the jobs management and training

…platform#2994) chore(inspect): Add more models

🐛 fix(inspect): Fix unable to start train job

chore(inspect): Update uv.lock

* add trainable models endpoint * fix Signed-off-by: Ma, Xiangxiang <[email protected]> * add test Signed-off-by: Ma, Xiangxiang <[email protected]> * add copyright Signed-off-by: Ma, Xiangxiang <[email protected]> --------- Signed-off-by: Ma, Xiangxiang <[email protected]>

…rm#3004) * add thumbnails endpoint Signed-off-by: Ma, Xiangxiang <[email protected]> * add tests Signed-off-by: Ma, Xiangxiang <[email protected]> * generate thumbnails as background task Signed-off-by: Ma, Xiangxiang <[email protected]> * update docstring Signed-off-by: Ma, Xiangxiang <[email protected]> --------- Signed-off-by: Ma, Xiangxiang <[email protected]>

* Replace "pre-commit" with "prek" in both root and application/backend pyproject.toml files. * Remove dependency for "geti-inspect" in the root pyproject.toml. Signed-off-by: Ashwin Vaidya <[email protected]>

Signed-off-by: Ashwin Vaidya <[email protected]>

Copilot

Pull request overview

Copilot reviewed 10 out of 13 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-03T13:17:18Z

application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


Allowing all origins with allow_origins=['*'] creates a security vulnerability by permitting cross-origin requests from any domain. This bypasses CORS protection and could enable malicious sites to access the API. Restrict allow_origins to specific trusted domains or use environment variables to configure allowed origins.

Copilot · 2025-12-03T13:17:18Z

application/docker/README.md

+
+```bash
+cd application/.packaging/docker
+AI_DEVICE=cuda docker compose up


The device name 'cuda' is inconsistent with the docker-compose.yml and Dockerfile which use 'cu124'. Either update the example to use 'cu124' or ensure all references use 'cuda' consistently.

Suggested change

AI_DEVICE=cuda docker compose up

AI_DEVICE=cu124 docker compose up

Copilot · 2025-12-03T13:17:18Z

application/backend/pyproject.toml

    "paho-mqtt~=2.1.0",
 ]

+# Explicit versions are needed as we can't propogate extra dependencies to anomalib


Corrected spelling of 'propogate' to 'propagate'.

Suggested change

# Explicit versions are needed as we can't propogate extra dependencies to anomalib

# Explicit versions are needed as we can't propagate extra dependencies to anomalib

Signed-off-by: Ashwin Vaidya <[email protected]>

Copilot

Pull request overview

Copilot reviewed 10 out of 13 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-03T13:26:03Z

application/docker/docker-compose.yml

+    # Map all host devices to provide access to webcams and other attached devices
+    privileged: true
+    devices:
+      - /dev:/dev


Running the container in privileged mode with full access to /dev is a significant security risk that grants the container nearly unrestricted access to the host system. Consider mapping only specific required devices (e.g., /dev/video0 for webcams) and removing privileged: true unless absolutely necessary for the application's core functionality.

Suggested change

# Map all host devices to provide access to webcams and other attached devices

privileged: true

devices:

- /dev:/dev

# Map only required host devices (e.g., webcam) to the container for security.

# privileged: true # Removed for security; only enable if absolutely necessary.

# devices:

# - /dev/video0:/dev/video0 # Example: map only webcam device if needed.

warrkan · 2025-12-03T13:21:29Z

application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


I'd rather implement it via env variable: https://github.com/open-edge-platform/geti-prompt/blob/main/application/backend/app/main.py#L73

warrkan · 2025-12-03T13:22:25Z

application/backend/src/settings.py

    # Alembic
-    alembic_config_path: str = "src/alembic.ini"
-    alembic_script_location: str = "src/alembic"
+    alembic_config_path: str = str(_MODULE_DIR / "alembic.ini")


Should be adjusted further when we introduce Pyinstaller

warrkan · 2025-12-03T13:49:28Z

application/docker/nginx.conf

@@ -0,0 +1,52 @@
+# PID file in a location non-root user can write to


Can we omit having nginx by serving UI static using FastAPI?

maxxgx · 2025-12-04T16:35:29Z

application/docker/docker-compose.yml

Just a question: why put everything in a parent folder .packaging/?

Could we move it into application/docker (without .packaging?) no need to hide it imho. This would also align well with the other applications.

maxxgx · 2025-12-04T16:36:26Z

application/.packaging/docker/Dockerfile

+
+EXPOSE 80
+
+CMD ["sh", "-c", "nginx && exec uv run src/main.py"]


Should we use backend/run.sh?

maxxgx · 2025-12-04T16:38:30Z

application/docker/README.md

Could you add example of running the container with camera passthrough? I guess would be a common usecase

maxxgx · 2025-12-04T16:43:08Z

application/docker/Dockerfile

I've tried to build it and it works. However, couldn't get camera passthrough to work on mac. Have you tested it on linux?

Signed-off-by: Ashwin Vaidya <[email protected]>

Copilot

Pull request overview

Copilot reviewed 11 out of 14 changed files in this pull request and generated 5 comments.

Comments suppressed due to low confidence (1)

application/docker/Dockerfile:1

The dockerfile path references application/.packaging/docker/Dockerfile, but based on the file structure, the Dockerfile is located at application/docker/Dockerfile. This mismatch will cause the build to fail.

#------------------------------------------

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-06T05:18:30Z

application/docker/README.md

+## To create CPU build
+
+```bash
+cd application/.packaging/docker


The directory path in the instructions is incorrect. The actual docker files are in application/docker/, not application/.packaging/docker/. This inconsistency appears in all three build instruction sections and will cause users to encounter errors when following the documentation.

Copilot · 2025-12-06T05:18:30Z

application/backend/src/api/endpoints/webui_endpoints.py

+
+
+@webui_router.get("/", include_in_schema=False)
+async def get_webui(full_path: str = "") -> FileResponse:  # noqa: ARG001


The function parameter full_path is declared but never used, and there's no logic to prevent potential path traversal attacks or handle different routes. If this endpoint is meant to handle multiple paths, the implementation should use full_path to serve the appropriate files. If it only serves index.html, the parameter should be removed.

Suggested change

async def get_webui(full_path: str = "") -> FileResponse: # noqa: ARG001

async def get_webui() -> FileResponse:

Copilot · 2025-12-06T05:18:30Z

application/backend/src/api/endpoints/webui_endpoints.py

+@webui_router.get("/", include_in_schema=False)
+async def get_webui(full_path: str = "") -> FileResponse:  # noqa: ARG001
+    """Get the webui index.html file."""
+    if settings.static_files_dir and not (file_path := Path(settings.static_files_dir) / "index.html").exists():


If settings.static_files_dir is None or empty, file_path will not be defined, causing an UnboundLocalError on line 20. The condition should ensure file_path is always defined before the return statement, or raise an appropriate error when static_files_dir is not configured.

Suggested change

if settings.static_files_dir and not (file_path := Path(settings.static_files_dir) / "index.html").exists():

if not settings.static_files_dir:

raise HTTPException(status_code=500, detail="Static files directory is not configured")

file_path = Path(settings.static_files_dir) / "index.html"

if not file_path.exists():

Copilot · 2025-12-06T05:18:31Z

application/backend/src/main.py

-        "http://localhost:9000",
-        "http://127.0.0.1:9000",
-    ],
+    allow_origins=["*"],


Allowing all origins with allow_origins=[\"*\"] is a security risk in production environments as it permits any domain to make requests to the API. Consider making this configurable through settings and restricting it to specific trusted origins in production.

Copilot · 2025-12-06T05:18:31Z

application/docker/Dockerfile

+ && echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/ubuntu jammy unified" | \
+    tee /etc/apt/sources.list.d/intel-gpu-jammy.list \


The XPU build stage is adding Ubuntu 'jammy' repositories to a Debian-based image (python:3.13-slim is based on Debian). This repository mismatch may cause package installation issues or incompatibilities. Use the appropriate Debian-compatible Intel GPU repository instead.

Suggested change

&& echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/ubuntu jammy unified" | \

tee /etc/apt/sources.list.d/intel-gpu-jammy.list \

&& echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/debian bookworm unified" | \

tee /etc/apt/sources.list.d/intel-gpu-bookworm.list \

application/docker/docker-compose.yml

+      # Persist logs
+      - backend-logs:/app/logs
+    ports:
+      - "8000:8000"


Signed-off-by: Ashwin Vaidya <[email protected]>

application/docker/Dockerfile

+
+EXPOSE 8000
+
+CMD ["sh", "-c", "exec ./run.sh"]


application/docker/Dockerfile

+
+EXPOSE 8000
+
+CMD ["sh", "-c", "exec ./run.sh"]


application/docker/Dockerfile

+
+EXPOSE 8000
+
+CMD ["sh", "-c", "exec ./run.sh"]


Copilot

Pull request overview

Copilot reviewed 10 out of 13 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2025-12-10T12:37:56Z

application/docker/docker-compose.yml

+    #   - "${RENDER_GROUP_ID}" # This is needed to allow access to the 'render' group for torch XPU support
+    volumes:
+      # Persist database and uploaded data
+      - /backend/data:/app/data


The volume mount source path /backend/data is an absolute host path that likely doesn't exist. This should either be a relative path like ./data:/app/data or use a named volume. The same issue exists for the logs volume on line 33.

Copilot · 2025-12-10T12:37:57Z

application/backend/src/settings.py

    def database_url(self) -> str:
        """Get database URL"""
-        return f"sqlite+aiosqlite:///./{self.data_dir / self.database_file}?journal_mode=WAL"
+        return f"sqlite+aiosqlite:///{self.data_dir / self.database_file}?journal_mode=WAL"


The change from sqlite+aiosqlite:///./{path} to sqlite+aiosqlite:///{path} removes the relative path prefix. While this might work for absolute paths, it could break existing deployments using relative paths. Consider maintaining backward compatibility or documenting this breaking change.

Copilot · 2025-12-10T12:37:57Z

application/docker/Dockerfile

+ && echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/ubuntu jammy unified" | \
+    tee /etc/apt/sources.list.d/intel-gpu-jammy.list \


The XPU build stage is based on Debian 12 (bookworm) as indicated by the base image python:3.13-slim, but this configures Ubuntu Jammy (22.04) repositories. This architecture mismatch will likely cause package installation failures or runtime incompatibilities.

Suggested change

&& echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/ubuntu jammy unified" | \

tee /etc/apt/sources.list.d/intel-gpu-jammy.list \

&& echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/debian bookworm unified" | \

tee /etc/apt/sources.list.d/intel-gpu-bookworm.list \

Signed-off-by: Ashwin Vaidya <[email protected]>

MarkRedeman · 2025-12-11T13:17:58Z

application/backend/src/api/endpoints/webui_endpoints.py

+webui_router = APIRouter(tags=["Webui"])
+
+
+@webui_router.get("/", include_in_schema=False)


Suggested change

@webui_router.get("/", include_in_schema=False)

@webui_router.get("/", include_in_schema=False)

@webui_router.get("/{full_path:path}", include_in_schema=False)

With the previous implementation if the user is on a route (say /projects) and then refresh the browser page then they would see a 404 page.
This is because the UI uses client side routing, so what we need the server to do is serve our index.html file for any page that the user might access.

MarkRedeman · 2025-12-11T13:21:20Z

application/docker/docker-compose.yml

+    group_add:
+    #   - "${RENDER_GROUP_ID}" # This is needed to allow access to the 'render' group for torch XPU support
+      - "${VIDEO_GROUP_ID}"  # This is needed to allow access to video devices (webcams)


I didn't need this setting while testing on a lunar lake laptop. Though I guess this could differ based on how docker was installed?

MarkRedeman · 2025-12-11T13:23:41Z

application/docker/docker-compose.yml

+    restart: unless-stopped
+    build:
+      # Use root directory as context to build the image
+      context: ../..


Should this not be ../? (i.e. the application being the context excluding any library files?)

MarkRedeman · 2025-12-11T13:24:45Z

application/docker/Dockerfile

+ARG PUBLIC_API_BASE_URL=""
+ENV PUBLIC_API_BASE_URL=${PUBLIC_API_BASE_URL}


I think we could omit this, or set PUBLIC_API_BASE_URL="".
When deploying Inspect the UI and backend should be expected to be served on the same hosts, this allows us to avoid security policies like CORS and (later?) have a better CSP configuration.

MarkRedeman · 2025-12-11T13:26:12Z

application/docker/Dockerfile

+
+COPY --link application/ui/package.json ./package.json
+COPY --link application/ui/package-lock.json ./package-lock.json
+COPY --link application/ui/packages ./packages


I think this should be omitted. The npm install script should clone and install the geti packages with the npm ci command.
If we copy from the host then we risk the docker image containing outdated packages (or at least packages that are not in sync with what's in the anomalib repo).

MarkRedeman · 2025-12-11T13:27:28Z

application/docker/README.md

+
+```bash
+cd application/docker
+RENDER_GROUP_ID=$(getent group render | cut -d: -f3) AI_DEVICE=xpu docker compose up


For me just running,

AI_DEVICE=xpu docker compose up

was working. (same for build command)

MarkRedeman · 2025-12-11T13:28:42Z

application/docker/README.md

@@ -0,0 +1,28 @@
+# Docker distribution for Geti Inspect
+
+## To create CPU build


The docs here mention creating builds, but the commands show up being used, is that intended?

MarkRedeman · 2025-12-11T13:29:34Z

application/docker/README.md

Perhaps one other thing we could look into is having separate docker-compose.xpu.yaml, docker-compsoe.nvidia.yaml files and tell the user to use these (instead of them uncommenting stuff).

See for instance https://github.com/open-webui/open-webui as an example of a repo using this.

MarkRedeman · 2025-12-11T13:30:15Z

.dockerignore

I would move this file into the application folder, since I think we shouldn't expect users to use docker for the library?

MarkRedeman and others added 30 commits October 8, 2025 13:52

Initial UI setup for Geti Inspect

5bdd704

Mainly based on the UI files and architecture from [Geti Tune](https://github.com/open-edge-platform/training_extensions/tree/develop/ui)

🧪 test(get-inspect): add unit tests for endpoints (open-edge-platform…

82dd6b0

…#2942) add unit tests for endpoints Signed-off-by: Ma, Xiangxiang <[email protected]>

Add openapi spec to gitignore

1588d72

Use error page

22f8f8f

Delete nginx and Dockerfile

978a895

These won't be needed yet

🔧 chore(geti-inspect): Update Geti Inspect to react 19 (open-edge-pla…

cbb0a46

…tform#2948) * Use src folder inside run.sh * Set openapi_url path * Update to react 19 * Specify bash language in readme

🔧 chore(inspect): Update route structure and add openapi route (open-…

0614756

…edge-platform#2963) * Add OpenAPI route * Remove MSW browser worker * Rename infernece to inspect * Update navbar title * Rename infernece to inspect

chore: Update package lock (open-edge-platform#2967)

e07a955

feat: Add sidebar with dataset, models and stats (open-edge-platform#…

e04ef66

…2970) * chore: Add path alias to icons * feat: Add sidebar with dataset, models and stats

🚀 feat(inspect): Add dataset list (open-edge-platform#2971)

3489422

* feat: Display placeholders for images * refactor: Fix scrollbar * chore: Remove learn more

🔧 chore(inspect): Update UI scripts (open-edge-platform#2974)

4e160a1

* chore(inspect): Update UI scripts * chore: Update github actions * chore: Update port to 8000

🔧 chore(inspect): Configure project(s) mocks for playwright tests (op…

7437855

…en-edge-platform#2978) chore(inspect): Configure project(s) mocks for playwright

🚀 feat(inspect): Add project management (open-edge-platform#2977)

590ece3

* chore: Add project route * feat: Add project management * revert ui lock change * chore: Remove not needed code for project management

🔧 chore(inspect): Update allowed origins so server can receive reques…

d9a613d

…ts from the UI (open-edge-platform#2980) chore(inspect): Update allowed origins

🔧 chore(inspect): Update openapi page (open-edge-platform#2985)

1baa2eb

chore(inspect): Update openapi page title

🔧 chore(inspect): Rename folder structure from app to application (op…

0a2af70

…en-edge-platform#2989) * chore(inspect): Renamed app to application * chore(inspect): Rename app to application in github actions

refactor(inspect): Update photo placeholder to use indicator instead …

8f1ef19

…of email (open-edge-platform#2990) refactor: Update photo placeholder to use indicator instead of email

Fix path (open-edge-platform#2993)

237731d

Signed-off-by: Ashwin Vaidya <[email protected]>

🔄 refactor(inspect): Improvements to the jobs management and training (…

2a4f257

…open-edge-platform#2992) refactor: Improvements to the jobs management and training

🔧 chore(inspect): Add more models to be used for training (open-edge-…

7653915

…platform#2994) chore(inspect): Add more models

🐛 fix(inspect): Use null pool for async engine (open-edge-platform#2998)

f0a7643

🐛 fix(inspect): Fix unable to start train job

🔧 chore(inspect): Update uv.lock (open-edge-platform#2995)

df8eb42

chore(inspect): Update uv.lock

ashwinvaidya17 added 6 commits December 2, 2025 15:37

chore: update dependencies in pyproject.toml

5cf0ad6

* Replace "pre-commit" with "prek" in both root and application/backend pyproject.toml files. * Remove dependency for "geti-inspect" in the root pyproject.toml. Signed-off-by: Ashwin Vaidya <[email protected]>

Update prek references

83a7c16

Signed-off-by: Ashwin Vaidya <[email protected]>

Update uv.lock

85d4860

Signed-off-by: Ashwin Vaidya <[email protected]>

update dockerfile

54501fb

Signed-off-by: Ashwin Vaidya <[email protected]>

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

5d3a63c

Revert formatting

bdac68a

Signed-off-by: Ashwin Vaidya <[email protected]>

Copilot AI review requested due to automatic review settings December 3, 2025 13:16

Copilot AI reviewed Dec 3, 2025

View reviewed changes

ashwinvaidya17 added 3 commits December 3, 2025 14:17

Revert formatting

3af24ca

Signed-off-by: Ashwin Vaidya <[email protected]>

Update actions

0e450d1

Signed-off-by: Ashwin Vaidya <[email protected]>

fix typo

86e137a

Signed-off-by: Ashwin Vaidya <[email protected]>

ashwinvaidya17 requested review from AlexanderBarabanov and mramotowski as code owners December 3, 2025 13:24

Copilot AI review requested due to automatic review settings December 3, 2025 13:25

Copilot AI reviewed Dec 3, 2025

View reviewed changes

warrkan reviewed Dec 3, 2025

View reviewed changes

maxxgx reviewed Dec 4, 2025

View reviewed changes

ashwinvaidya17 added 3 commits December 6, 2025 06:08

Serve UI from fastapi in prod

0740a18

Signed-off-by: Ashwin Vaidya <[email protected]>

move dir

be206fe

Signed-off-by: Ashwin Vaidya <[email protected]>

Merge branch 'feature/geti-inspect' into ashwin/feature/dockerfile_2

9ae1a10

Copilot AI review requested due to automatic review settings December 6, 2025 05:17

Copilot AI reviewed Dec 6, 2025

View reviewed changes

github-advanced-security bot found potential problems Dec 6, 2025

View reviewed changes

application/docker/docker-compose.yml

# Persist logs

- backend-logs:/app/logs

ports:

- "8000:8000"

Check warning

Code scanning / Semgrep OSS

Semgrep Finding: trailofbits.yaml.docker-compose.port-all-interfaces.port-all-interfaces Warning

Service port is exposed on all interfaces

ashwinvaidya17 and others added 2 commits December 8, 2025 10:28

fix context path

bada298

Signed-off-by: Ashwin Vaidya <[email protected]>

remove non-root

419b839

Signed-off-by: Ashwin Vaidya <[email protected]>

Copilot AI review requested due to automatic review settings December 10, 2025 12:36

github-advanced-security bot found potential problems Dec 10, 2025

View reviewed changes

Copilot AI reviewed Dec 10, 2025

View reviewed changes

add video group

f1e4943

Signed-off-by: Ashwin Vaidya <[email protected]>

MarkRedeman reviewed Dec 11, 2025

View reviewed changes

	AI_DEVICE=cuda docker compose up
	AI_DEVICE=cu124 docker compose up

	# Explicit versions are needed as we can't propogate extra dependencies to anomalib
	# Explicit versions are needed as we can't propagate extra dependencies to anomalib

		@@ -0,0 +1,52 @@
		# PID file in a location non-root user can write to


		EXPOSE 80

		CMD ["sh", "-c", "nginx && exec uv run src/main.py"]



		@webui_router.get("/", include_in_schema=False)
		async def get_webui(full_path: str = "") -> FileResponse: # noqa: ARG001

	async def get_webui(full_path: str = "") -> FileResponse: # noqa: ARG001
	async def get_webui() -> FileResponse:

-    if settings.static_files_dir and not (file_path := Path(settings.static_files_dir) / "index.html").exists():
+    if not settings.static_files_dir:
+        raise HTTPException(status_code=500, detail="Static files directory is not configured")
+    file_path = Path(settings.static_files_dir) / "index.html"
+    if not file_path.exists():

		&& echo "deb [arch=amd64,i386 signed-by=/usr/share/keyrings/intel-graphics.gpg] https://repositories.intel.com/gpu/ubuntu jammy unified" \| \
		tee /etc/apt/sources.list.d/intel-gpu-jammy.list \

		webui_router = APIRouter(tags=["Webui"])


		@webui_router.get("/", include_in_schema=False)

	@webui_router.get("/", include_in_schema=False)
	@webui_router.get("/", include_in_schema=False)
	@webui_router.get("/{full_path:path}", include_in_schema=False)

		ARG PUBLIC_API_BASE_URL=""
		ENV PUBLIC_API_BASE_URL=${PUBLIC_API_BASE_URL}

		@@ -0,0 +1,28 @@
		# Docker distribution for Geti Inspect

		## To create CPU build

📦 feat(inspect): Add docker build #3136

Are you sure you want to change the base?

📦 feat(inspect): Add docker build #3136

Uh oh!

Conversation

ashwinvaidya17 commented Nov 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Description

TODO

✨ Changes

✅ Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 6, 2025

Choose a reason for hiding this comment

Uh oh!

Check warning

Check failure

Check failure

Check failure

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 10, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 10, 2025

Choose a reason for hiding this comment

ashwinvaidya17 commented Nov 21, 2025 •

edited

Loading