Skip to content

Commit 65c6360

Browse files
committed
unify trigger_noconda.py and trigger.py
1 parent cd21029 commit 65c6360

File tree

10 files changed

+31
-98
lines changed

10 files changed

+31
-98
lines changed

Images/qp-deblur/qp-deblur.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ RUN sed -i "s|/opt/conda/envs/deblur/share/fragment-insertion/sepp/.sepp/bundled
105105

106106
# install tornado based trigger layer in base environment
107107
RUN pip install -U --no-cache-dir tornado pip-system-certs
108-
COPY trigger_noconda.py /trigger.py
108+
COPY trigger.py /trigger.py
109109
# ^^ 848 MB
110110

111111
WORKDIR /

Images/qp-target-gene/qp-target-gene.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ RUN mkdir -p /unshared_plugins
112112
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
113113

114114
RUN pip3 install tornado
115-
COPY trigger_noconda.py /trigger.py
115+
COPY trigger.py /trigger.py
116116

117117
# copy sortmerna binaries
118118
COPY --from=builder /sortmerna-2.0/sortmerna /usr/local/bin/sortmerna

Images/qtp-biom/qtp-biom.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ RUN ln -s /usr/local/lib/python3.8/site-packages/scikit_learn.libs/libgomp-a34b3
135135

136136
# install tornado based trigger layer in base environment
137137
#RUN pip install -U --no-cache-dir tornado
138-
COPY trigger_noconda.py /trigger.py
138+
COPY trigger.py /trigger.py
139139

140140
WORKDIR /
141141

Images/qtp-diversity/qtp-diversity.dockerfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ RUN mkdir -p /unshared_plugins
119119
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
120120

121121
# install tornado based trigger layer in base environment
122-
COPY trigger_noconda.py /trigger.py
122+
COPY trigger.py /trigger.py
123123

124124
#COPY --from=builder /opt/conda/envs/dependencies/bin/* /opt/conda/bin/
125125
#COPY --from=builder /opt/conda/envs/dependencies/sbin/* /opt/conda/sbin/
@@ -156,7 +156,7 @@ RUN for f in `echo "libssu.so libhdf5_cpp.so.103 liblapacke.so.3 libcblas.so.3 l
156156
# fix an pandas deprecation issue, i.e. patch q2templates code
157157
RUN sed -i "s/'display.max_colwidth', -1/'display.max_colwidth', None/" /usr/local/lib/python3.8/site-packages/q2templates/util.py
158158

159-
# COPY trigger_noconda.py /trigger.py
159+
# COPY trigger.py /trigger.py
160160
ENV LC_ALL=C.UTF-8
161161
ENV LANG=C.UTF-8
162162

Images/qtp-job-output-folder/qtp-job-output-folder.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ COPY --from=builder /wheels /wheels
7474
RUN pip install --no-cache-dir /wheels/* \
7575
&& rm -rf rm -rf `find /usr/local/lib/python3.6/site-packages -type d -name "tests" | grep -v numpy`
7676

77-
COPY trigger_noconda.py /trigger.py
77+
COPY trigger.py /trigger.py
7878

7979
COPY start_plugin.sh .
8080
RUN chmod 755 start_plugin.sh

Images/qtp-sequencing/qtp-sequencing.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ COPY --from=builder /opt/conda/envs/qtp-sequencing/lib/libdeflate.so.0 /lib/x86_
7878
# "install" pigz
7979
COPY --from=builder /opt/conda/envs/qtp-sequencing/bin/pigz /usr/local/bin/
8080

81-
COPY trigger_noconda.py /trigger.py
81+
COPY trigger.py /trigger.py
8282

8383
# link to quast program
8484
RUN ln -s /usr/local/bin/quast.py /usr/local/bin/quast

Images/qtp-visualization/qtp-visualization.dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ RUN chmod 755 start_plugin.sh
102102
RUN mkdir -p /unshared_plugins
103103
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
104104

105-
COPY trigger_noconda.py /trigger.py
105+
COPY trigger.py /trigger.py
106106

107107
## Export cert and config filepaths
108108
COPY qiita_server_certificates/qiita_server_certificates.pem /qiita_server_certificates/qiita_server_certificates.pem

Images/trigger.py

Lines changed: 22 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@
66
import sys
77
import traceback
88
import asyncio
9+
import sys
10+
import os
911

1012
conda_env_name = None
1113
plugin_start_script = None
@@ -28,7 +30,11 @@ async def post(self):
2830
return
2931

3032
# Systembefehl ausfuehren
31-
cmd = 'source /opt/conda/etc/profile.d/conda.sh; conda activate /opt/conda/envs/%s; %s/scripts/%s %s %s %s' % (conda_env_name, plugin_src_dir, plugin_start_script, qiita_worker_url, job_id, output_dir)
33+
cmd = ""
34+
if conda_env_name is not None:
35+
cmd = 'source /opt/conda/etc/profile.d/conda.sh; conda activate /opt/conda/envs/%s; %s/scripts/%s %s %s %s' % (conda_env_name, plugin_src_dir, plugin_start_script, qiita_worker_url, job_id, output_dir)
36+
else:
37+
cmd = '%s %s %s %s' % (plugin_start_script, qiita_worker_url, job_id, output_dir)
3238
# Asynchronen Subprozess starten
3339
proc = await asyncio.create_subprocess_shell(
3440
cmd,
@@ -54,9 +60,12 @@ async def post(self):
5460
self.set_status(500)
5561
# a hack to learn which docker service I am in
5662
plugin_name = "unknown"
57-
for f in glob('/start_*.sh'):
58-
plugin_name = f.split('_')[-1].replace('.sh', '')
59-
break
63+
if conda_env_name is not None:
64+
for f in glob('/start_*.sh'):
65+
plugin_name = f.split('_')[-1].replace('.sh', '')
66+
break
67+
else:
68+
plugin_name = os.path.basename(plugin_start_script).replace('start_', '')
6069
print("Error in service '%s': %s" % (plugin_name, str(e)), file=sys.stderr)
6170
traceback.print_exc()
6271
self.write({"error": str(e)})
@@ -78,9 +87,15 @@ def make_app():
7887
])
7988

8089
if __name__ == "__main__":
81-
conda_env_name = sys.argv[1]
82-
plugin_start_script = sys.argv[2]
83-
plugin_src_dir = sys.argv[3]
90+
if len(sys.argv) == 1:
91+
plugin_start_script = sys.argv[1]
92+
elif len(sys.argv) == 3:
93+
conda_env_name = sys.argv[1]
94+
plugin_start_script = sys.argv[2]
95+
plugin_src_dir = sys.argv[3]
96+
else:
97+
print("Incorrect number of arguments provided\nUsage: trigger.py <conda_env_name> <plugin_start_script> <plugin_src_dir> <-- for use with conda\n or trigger.py <plugin_start_script> <-- for use without conda\n", file=sys.stderr)
98+
exit(1)
8499

85100
app = make_app()
86101
app.listen(5000) # Server auf Port 5000 starten

Images/trigger_noconda.py

Lines changed: 0 additions & 82 deletions
This file was deleted.

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ $(DIR_REFERENCES)/qiita_server_certificates: Images/plugin_collector/stefan_csr.
3131
# === end: create own certificates ===
3232

3333
# a general target, executed for each plugin
34-
plugin: Images/trigger.py Images/trigger_noconda.py Images/start_plugin.sh $(DIR_REFERENCES)/qiita_server_certificates Images/test_plugin.sh
34+
plugin: Images/trigger.py Images/start_plugin.sh $(DIR_REFERENCES)/qiita_server_certificates Images/test_plugin.sh
3535
cp -r $^ $(tmpdir)/
3636

3737
.built_image_qtp-biom: Images/qtp-biom/qtp-biom.dockerfile src/qiita-files/ src/qtp-biom/ Images/qtp-biom/requirements.txt

0 commit comments

Comments
 (0)