Skip to content

Issue 1204 Fix Docker build dependency conflict in requirements.txt#1205

Merged
rlundeen2 merged 4 commits into
microsoft:mainfrom
EHO-makai:docker-fix-notebook
Dec 2, 2025
Merged

Issue 1204 Fix Docker build dependency conflict in requirements.txt#1205
rlundeen2 merged 4 commits into
microsoft:mainfrom
EHO-makai:docker-fix-notebook

Conversation

@EHO-makai

@EHO-makai EHO-makai commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

FIX: Issue 1204 - Relax Docker requirements.txt constraints to resolve dependency conflict

Description

Fixes dependency conflict in Docker build that prevented container creation. The issue was caused by overly restrictive version constraints in docker/requirements.txt:

  • notebook==7.3.2 requires jupyterlab<4.4 and >=4.3.4
  • pip attempted to install jupyterlab==4.4.8, causing a ResolutionImpossible error

Changes made:

  • Relaxed jupyterlab constraint from =4.4.8 to >=4.3.4,<4.5 (more permissive range)

This allows pip to automatically resolve compatible versions of both packages without manual intervention.

Testing:

  • Successfully built Docker image with docker-compose up -d
  • Container starts without errors
  • Verified pip resolves to compatible jupyterlab and notebook versions

Fixes #[1204]

Tests and Documentation

No new tests or documentation required for this change. This is a maintenance fix to the Docker environment configuration.

Docker build verification:

  • Ran docker-compose up -d successfully
  • Confirmed container pyrit-jupyter is running
  • Verified JupyterLab is accessible on port 8888

JupyText execution not applicable - this change only affects Docker dependency resolution and does not modify PyRIT code or documentation notebooks.

Comment thread docker/requirements.txt
@rlundeen2
rlundeen2 merged commit e4b9ea9 into microsoft:main Dec 2, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants