Skip to content

Commit

Permalink
Issue #1: pass env variables as docker parameters
Browse files Browse the repository at this point in the history
  • Loading branch information
SonOfLope committed Jun 21, 2024
1 parent 1002556 commit 91ebbfa
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/updatecsv.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
run: |
docker build -t csvscraper .
### Running the container
docker run -v ./data:/usr/src/app/data -p 8081:8081 csvscraper
docker run -v ./data:/usr/src/app/data -p 8081:8081 -e NEW_REPORT_URL=$NEW_REPORT_URL -e MASTERLIST_URL=$MASTERLIST_URL csvscraper
env:
NEW_REPORT_URL: ${{ secrets.NEW_REPORT_URL }}
MASTERLIST_URL: ${{ secrets.MASTERLIST_URL }}
Expand Down
5 changes: 5 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,11 @@ RUN apt-get update && \
# Set the working directory
WORKDIR /usr/src/app

ARG NEW_REPORT_URL
ENV NEW_REPORT_URL=$NEW_REPORT_URL
ARG MASTERLIST_URL
ENV MASTERLIST_URL=$MASTERLIST_URL

# Copy the requirements.txt file into the container
COPY requirements.txt ./

Expand Down

0 comments on commit 91ebbfa

Please sign in to comment.