-
Notifications
You must be signed in to change notification settings - Fork 4
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.
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
V2 #21
V2 #21
Changes from 95 commits
341d130
4d358ef
c630e12
1dc7199
7b48b61
2f58d4d
b444802
e5aa0f7
128f11b
1b48ecd
ecaff05
0518ecd
bc5cfe0
4dbd27e
cd3ae4b
da3f45c
5dad9a7
3a679a1
3a3ca0f
859a89a
258a48c
040c3a5
5c464ce
dc62afb
980be3b
ac17d34
1e094a7
87e1763
8274d4f
ff6381d
5ef941a
ce00863
8ab9245
18f0fe5
35f19a9
d0a7179
b83d46b
dbac99d
3be8136
776f175
6ef7a28
d1fc32b
15d804e
8037b40
c7ac71d
f473709
f122d14
5ad7e5a
cc490d4
45d0204
ff0e534
c4afd82
d1cea8c
713ec27
29422d5
9e66a9b
c7e0ffe
36b21c6
6a2ddc3
cbbb93f
bc4055a
a858069
8ec48c2
6826b58
7bb3a30
117e692
ece148c
14e2155
c71f3b1
9cff8f2
9fd9591
d83421e
f2ed65f
769c9fd
169323b
f9a3578
c7b2d38
93fe459
e191960
a2b4ba0
7645dd8
19e0436
d34209a
faf96a1
6914bbc
ea71075
d389e0e
a4f0589
ef79400
729b63e
48cd563
5dd0c44
d969ea5
0ee848c
f72b731
fe78543
21813ac
3c3b657
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -182,5 +182,9 @@ dmypy.json | |
# pytype static role analyzer | ||
.pytype/ | ||
|
||
# vscode | ||
.vscode/ | ||
|
||
## Misc | ||
sample_data/** | ||
/aws_config.json |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,36 +2,23 @@ FROM continuumio/miniconda3:latest | |
|
||
ENV LANG=C.UTF-8 LC_ALL=C.UTF-8 PATH=/opt/conda/envs/gsp_api/bin:$PATH API_PREFIX=/api | ||
|
||
# For Development On Analytics Only: | ||
# ENV AWS_ACCESS_KEY_ID= | ||
# ENV AWS_SECRET_ACCESS_KEY= | ||
# ENV AWS_LOG_GROUP_NAME= | ||
# ENV AWS_LOG_STREAM_NAME= | ||
# ENV AWS_REGION= | ||
|
||
RUN mkdir /var/uwsgi | ||
|
||
RUN apt-get update -qq && apt-get install -yqq supervisor vim | ||
|
||
COPY ./environment.yml ./startup.sh ./ | ||
COPY environment.yaml ./startup.sh ./ | ||
|
||
RUN conda config --set channel_priority strict && \ | ||
conda config --add channels conda-forge && \ | ||
conda env create -f environment.yml && \ | ||
echo "conda activate gsp_api" >> ~/.bashrc | ||
conda env create -f environment.yaml && \ | ||
echo "conda activate app-env" >> ~/.bashrc | ||
|
||
RUN mkdir -p /mnt/output/forecasts && \ | ||
mkdir -p /mnt/output/era-interim && \ | ||
mkdir -p /mnt/output/era-5 && \ | ||
mkdir -p /mnt/output/forecast-records | ||
|
||
# COPY ./sample_data/forecasts /mnt/output/forecasts | ||
# COPY ./sample_data/era-interim /mnt/output/era-interim | ||
# COPY ./sample_data/era-5 /mnt/output/era-5 | ||
# COPY ./sample_data/forecast-records /mnt/output/forecast-records | ||
|
||
# Copy API code | ||
COPY ./GSP_API /app/GSP_API/ | ||
COPY app /app | ||
Comment on lines
+8
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The updates to the Dockerfile correctly reflect the changes in environment file naming and application directory structure. Ensure that the |
||
COPY ./supervisord.conf /etc/supervisor/conf.d/uwsgi.conf | ||
|
||
# startup.sh is a helper script | ||
|
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
before_script
section for Docker authentication is crucial for security. Ensure that the credentials used ($CI_REGISTRY_USER
,$CI_REGISTRY_PASSWORD
,$ECCR_USER
,$ECCR_PASSWORD
) are securely managed and not exposed in logs or other insecure channels.