Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update export.Dockerfile #90

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update export.Dockerfile #90

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3aef654
Update export.Dockerfile
EliseCastle23 Jan 15, 2025
a446dee
Update import.Dockerfile
EliseCastle23 Jan 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion export.Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

USER gen3
# Install Poetry via pipx
RUN pipx install poetry
RUN pipx install 'poetry<2.0'
ENV PATH="/home/gen3/.local/bin:${PATH}"
USER root

Expand Down Expand Up @@ -81,7 +81,7 @@

RUN wget ${POSTGRES_JAR_URL} -O ${POSTGRES_JAR_PATH}

ENV HADOOP_CONF_DIR="$HADOOP_HOME/etc/hadoop" \

Check warning on line 84 in export.Dockerfile

View workflow job for this annotation

GitHub Actions / Build and Push Pelican Export Image / Build Image and Push

Variables should be defined before their use 

UndefinedVar: Usage of undefined variable '$LD_LIBRARY_PATH'
More info: https://docs.docker.com/go/dockerfile/rule/undefined-var/
HADOOP_MAPRED_HOME="${HADOOP_HOME}" \
HADOOP_COMMON_HOME="${HADOOP_HOME}" \
HADOOP_HDFS_HOME="${HADOOP_HOME}" \
Expand All @@ -105,4 +105,4 @@

ENV PYTHONUNBUFFERED=1

ENTRYPOINT poetry run python job_export.py

Check warning on line 108 in export.Dockerfile

View workflow job for this annotation

GitHub Actions / Build and Push Pelican Export Image / Build Image and Push

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals 

JSONArgsRecommended: JSON arguments recommended for ENTRYPOINT to prevent unintended behavior related to OS signals
More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
2 changes: 1 addition & 1 deletion import.Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

USER gen3
# Install Poetry via pipx
RUN pipx install poetry
RUN pipx install 'poetry<2.0'
ENV PATH="/home/gen3/.local/bin:${PATH}"
USER root

Expand Down Expand Up @@ -81,7 +81,7 @@

RUN wget ${POSTGRES_JAR_URL} -O ${POSTGRES_JAR_PATH}

ENV HADOOP_CONF_DIR="$HADOOP_HOME/etc/hadoop" \

Check warning on line 84 in import.Dockerfile

View workflow job for this annotation

GitHub Actions / Build and Push Pelican Import Image / Build Image and Push

Variables should be defined before their use 

UndefinedVar: Usage of undefined variable '$LD_LIBRARY_PATH'
More info: https://docs.docker.com/go/dockerfile/rule/undefined-var/
HADOOP_MAPRED_HOME="${HADOOP_HOME}" \
HADOOP_COMMON_HOME="${HADOOP_HOME}" \
HADOOP_HDFS_HOME="${HADOOP_HOME}" \
Expand All @@ -105,4 +105,4 @@

ENV PYTHONUNBUFFERED=1

ENTRYPOINT poetry run python job_import.py

Check warning on line 108 in import.Dockerfile

View workflow job for this annotation

GitHub Actions / Build and Push Pelican Import Image / Build Image and Push

JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals 

JSONArgsRecommended: JSON arguments recommended for ENTRYPOINT to prevent unintended behavior related to OS signals
More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
Loading