Skip to content

Commit ce9716a

Browse files
authored
chore: small improvement installing dependencies for HF Spaces Dockerfile (#5651)
# Description Small improvement moving the install of some dependencies before cleaning up. **Type of change** - Improvement (change adding some improvement to an existing functionality) **How Has This Been Tested** - [ ] Deploying it to a new HF space. **Checklist** - I added relevant documentation - I followed the style guidelines of this project - I did a self-review of my code - I made corresponding changes to the documentation - I confirm My changes generate no new warnings - I have added tests that prove my fix is effective or that my feature works - I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)
1 parent c2dddc3 commit ce9716a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

argilla-server/docker/argilla-hf-spaces/Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,13 @@ RUN \
4040
chmod +x /home/argilla/start_argilla_server.sh && \
4141
# Give ownership of the data directory to the argilla user
4242
chown -R argilla:argilla /data && \
43+
# Install additional tools to be used in the start.sh script to get default username and generate passwords
44+
apt-get install -y curl jq pwgen && \
4345
# Clean up
4446
apt-get remove -y wget gnupg && \
4547
apt-get clean && \
4648
rm -rf /var/lib/apt/lists/* && \
47-
rm -rf /packages && \
48-
# Install pwgen curl and jq
49-
apt-get update && apt-get install -y curl jq pwgen
49+
rm -rf /packages
5050

5151
COPY config/elasticsearch.yml /etc/elasticsearch/elasticsearch.yml
5252

0 commit comments

Comments
 (0)