Skip to content

Conversation

sayakpaul
Copy link
Member

@sayakpaul sayakpaul commented Oct 3, 2025

What does this PR do?

Docker build is broken from the past two days:
https://github.com/huggingface/diffusers/actions/runs/18208988147/job/51845669366#step:5:1173

Only changes the Dockerfiles that we use in .github/workflows/build_docker_images.yml.

@sayakpaul
Copy link
Member Author

sayakpaul commented Oct 3, 2025

@DN6 could you give this a quick check if the changes are alright? I have also gone ahead and included the "test" related installations in the Dockerfiles themselves so that we don't have install them during CI (those changes will come in a separate PR).

torchaudio \
--extra-index-url https://download.pytorch.org/whl/cpu

RUN uv pip install --no-cache-dir "git+https://github.com/huggingface/diffusers.git@main#egg=diffusers[test]"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed? We already install test deps in the workflow file?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah so in a future PR (immediately after this one), I will remove that step so that we can save some time skipping those installs.

@sayakpaul sayakpaul requested a review from DN6 October 3, 2025 12:42
@sayakpaul sayakpaul marked this pull request as ready for review October 3, 2025 12:42
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.

2 participants