Skip to content

Commit 8b5c075

Browse files
authored
Merge pull request #58 from jlab/unify_startplugin
replace individual plugin start bash scripts with ONE general one, wh…
2 parents a3fe620 + ab0ea34 commit 8b5c075

21 files changed

+111
-148
lines changed

.github/workflows/buildContainer.yaml

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,9 @@ jobs:
4949
run: |
5050
VERSION=$(head -n 1 Images/${{ matrix.container }}/${{ matrix.container }}.dockerfile | cut -d ":" -f 2- | tr -d " ")
5151
echo "IMAGE_TAG=$VERSION" >> $GITHUB_ENV
52-
cp Images/${{ matrix.container }}/start_${{ matrix.container }}.sh Images/test_plugin.sh Images/nginx/nginx_qiita.conf Images/${{ matrix.container }}/requirements.txt .
52+
cp Images/start_plugin.sh Images/test_plugin.sh Images/nginx/nginx_qiita.conf Images/${{ matrix.container }}/requirements.txt Images/nginx/start_nginx.sh .
5353
cp Images/qiita/config_portal.cfg Images/qiita/config_qiita_oidc.cfg Images/qiita/drop_workflows.py Images/qiita/start_plugin.py Images/qiita/start_qiita-initDB.sh Images/qiita/start_qiita.sh .
54-
cp Images/plugin_collector/collect_configs.py Images/plugin_collector/fix_test_db.py Images/plugin_collector/stefan_cert.conf Images/plugin_collector/stefan_csr.conf .
54+
cp Images/plugin_collector/collect_configs.py Images/plugin_collector/fix_test_db.py Images/plugin_collector/stefan_cert.conf Images/plugin_collector/stefan_csr.conf Images/plugin_collector/start_plugin_collector.sh .
5555
5656
- name: Download certificates from job build_main
5757
uses: actions/download-artifact@v4
@@ -118,7 +118,7 @@ jobs:
118118
run: |
119119
VERSION=$(head -n 1 Images/${{ matrix.plugin }}/${{ matrix.plugin }}.dockerfile | cut -d ":" -f 2- | tr -d " ")
120120
echo "IMAGE_TAG=$VERSION" >> $GITHUB_ENV
121-
cp Images/${{ matrix.plugin }}/start_${{ matrix.plugin }}.sh Images/test_plugin.sh Images/${{ matrix.plugin }}/requirements.txt Images/trigger_noconda.py Images/trigger.py .
121+
cp Images/start_plugin.sh Images/test_plugin.sh Images/${{ matrix.plugin }}/requirements.txt Images/trigger_noconda.py Images/trigger.py .
122122
123123
- name: Download certificates from job build_main
124124
uses: actions/download-artifact@v4
@@ -193,7 +193,7 @@ jobs:
193193
run: |
194194
VERSION=$(head -n 1 Images/${{ matrix.multiplugin }}/${{ matrix.multiplugin }}.dockerfile | cut -d ":" -f 2- | tr -d " ")
195195
echo "IMAGE_TAG=$VERSION" >> $GITHUB_ENV
196-
cp Images/${{ matrix.multiplugin }}/start_${{ matrix.multiplugin }}.sh Images/test_plugin.sh Images/${{ matrix.multiplugin }}/requirements.txt Images/trigger_noconda.py Images/trigger.py .
196+
cp Images/start_plugin.sh Images/test_plugin.sh Images/${{ matrix.multiplugin }}/requirements.txt Images/trigger_noconda.py Images/trigger.py .
197197
198198
- name: Download certificates from job build_main
199199
uses: actions/download-artifact@v4
@@ -223,6 +223,12 @@ jobs:
223223
cache-from: type=gha,scope=tinqiita-${{ github.ref_name }}
224224
cache-to: type=gha,scope=tinqiita-${{ github.ref_name }},mode=max
225225

226+
- name: Prune build cache after build (saving disc space)
227+
run: |
228+
df -h /
229+
docker buildx prune -af
230+
df -h /
231+
226232
- name: adapt compose file to select specific plugin
227233
run: |
228234
if [[ "${{ matrix.multiplugin }}" == "qp-qiime2" ]]; then sed -i 's|- QIITA_PLUGINS="MATRIXPLUGIN:"|- QIITA_PLUGINS="${{ matrix.multiplugin }}:qtp-diversity:qtp-visualization:"|' compose_github.yaml; sed -i 's|MATRIXPLUGIN:|${{ matrix.multiplugin }}:\n condition: service_started\n qtp-diversity:\n condition: service_started\n qtp-visualization:|' compose_github.yaml; fi;
@@ -241,12 +247,8 @@ jobs:
241247
242248
- name: Push image to ghcr (only if tests passed)
243249
if: success()
244-
uses: docker/build-push-action@v6
245-
with:
246-
context: .
247-
push: true
248-
file: Images/${{ matrix.multiplugin }}/${{ matrix.multiplugin }}.dockerfile
249-
tags: ghcr.io/${{ github.repository }}/${{ matrix.multiplugin }}:testcandidate
250+
run: |
251+
docker push ghcr.io/${{ github.repository }}/${{ matrix.multiplugin }}:testcandidate
250252
251253
publish_images:
252254
needs:

Images/qp-deblur/qp-deblur.dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,8 @@ COPY trigger_noconda.py /trigger.py
110110

111111
WORKDIR /
112112

113-
COPY start_qp-deblur.sh .
114-
RUN chmod 755 start_qp-deblur.sh
113+
COPY start_plugin.sh .
114+
RUN chmod 755 start_plugin.sh
115115

116116
RUN mkdir -p /unshared_plugins
117117
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
@@ -139,5 +139,5 @@ RUN sed -i "s|source /opt/conda/etc/profile.d/conda.sh; conda activate /opt/cond
139139
# for testing
140140
COPY test_plugin.sh /test_plugin.sh
141141

142-
CMD ["./start_qp-deblur.sh"]
142+
CMD ["./start_plugin.sh"]
143143
# ^^ 848 MB

Images/qp-deblur/start_qp-deblur.sh

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

Images/qp-qiime2/qp-qiime2.dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,8 @@ RUN dpkg-reconfigure -f noninteractive tzdata
8686

8787
WORKDIR /
8888

89-
COPY start_qp-qiime2.sh .
90-
RUN chmod 755 start_qp-qiime2.sh
89+
COPY start_plugin.sh .
90+
RUN chmod 755 start_plugin.sh
9191

9292
RUN mkdir -p /unshared_plugins
9393
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
@@ -108,4 +108,4 @@ RUN sed -i -E "s/^START_SCRIPT = .+/START_SCRIPT = python \/start_plugin.py qp-q
108108
# for testing
109109
COPY test_plugin.sh /test_plugin.sh
110110

111-
CMD ["./start_qp-qiime2.sh"]
111+
CMD ["./start_plugin.sh"]

Images/qp-qiime2/start_qp-qiime2.sh

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

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ COPY --from=builder /opt/conda/envs/qp-target-gene/lib/libpython2.7.so.1.0 /usr/
105105
# "install" pigz
106106
COPY --from=builder /opt/conda/envs/qp-target-gene/bin/pigz /usr/local/bin/
107107

108-
COPY start_qp-target-gene.sh .
109-
RUN chmod 755 start_qp-target-gene.sh
108+
COPY start_plugin.sh .
109+
RUN chmod 755 start_plugin.sh
110110

111111
RUN mkdir -p /unshared_plugins
112112
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
@@ -133,4 +133,4 @@ RUN sed -i -E "s/^START_SCRIPT = .+/START_SCRIPT = python \/start_plugin.py qp-t
133133
# for testing
134134
COPY test_plugin.sh /test_plugin.sh
135135

136-
CMD ["./start_qp-target-gene.sh"]
136+
CMD ["./start_plugin.sh"]

Images/qp-target-gene/start_qp-target-gene.sh

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

Images/qtp-biom/qtp-biom.dockerfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ RUN conda install cython
113113
RUN pip wheel --no-cache-dir --wheel-dir /wheels -r requirements.txt
114114
RUN pip install iow
115115

116-
CMD ["./start_qtp-biom.sh"]
116+
CMD ["./start_plugin.sh"]
117117

118118
# ==========================
119119
# Stage 2: Runtime
@@ -139,8 +139,8 @@ COPY trigger_noconda.py /trigger.py
139139

140140
WORKDIR /
141141

142-
COPY start_qtp-biom.sh .
143-
RUN chmod 755 start_qtp-biom.sh
142+
COPY start_plugin.sh .
143+
RUN chmod 755 start_plugin.sh
144144

145145
RUN mkdir -p /unshared_plugins
146146
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
@@ -173,6 +173,6 @@ RUN sed -i "s/'display.max_colwidth', -1/'display.max_colwidth', None/" /usr/loc
173173
# for testing
174174
COPY test_plugin.sh /test_plugin.sh
175175

176-
CMD ["./start_qtp-biom.sh"]
176+
CMD ["./start_plugin.sh"]
177177

178178
# python -c "import qiime2.plugins.feature_table"

Images/qtp-biom/start_qtp-biom.sh

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

Images/qtp-diversity/qtp-diversity.dockerfile

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,8 @@ COPY --from=builder /wheels /wheels
112112
RUN pip install --no-cache-dir /wheels/* \
113113
&& rm -rf rm -rf `find /usr/local/lib/python3.8/site-packages -type d -name "tests" | grep -v numpy`
114114

115-
COPY start_qtp-diversity.sh .
116-
RUN chmod 755 start_qtp-diversity.sh
115+
COPY start_plugin.sh .
116+
RUN chmod 755 start_plugin.sh
117117

118118
RUN mkdir -p /unshared_plugins
119119
ENV QIITA_PLUGINS_DIR=/unshared_plugins/
@@ -144,11 +144,11 @@ COPY --from=builder /opt/conda/envs/qiime2/lib/libhdf5_cpp.so.103.2.0 /usr/lo
144144
COPY --from=builder /opt/conda/envs/qiime2/lib/libhdf5_hl_cpp.so.100.1.4 /usr/local/lib/libhdf5_hl_cpp.so.100
145145
COPY --from=builder /opt/conda/envs/qiime2/lib/libhdf5_hl.so.100.1.3 /usr/local/lib/libhdf5_hl.so.100
146146
COPY --from=builder /opt/conda/envs/qiime2/lib/libhdf5.so.103.2.0 /usr/local/lib/libhdf5.so.103
147-
COPY --from=builder /opt/conda/envs/qiime2/lib/libcrypto.so.1.1 /usr/local/lib/libcrypto.so.1.1
147+
COPY --from=builder /opt/conda/envs/qiime2/lib/libcrypto.so* /usr/local/lib/
148148
COPY --from=builder /opt/conda/envs/qiime2/lib/libcurl.so.4.8.0 /usr/local/lib/libcurl.so.4
149-
COPY --from=builder /opt/conda/envs/qiime2/lib/libnghttp2.so.14.24.1 /usr/local/lib/libnghttp2.so.14
149+
COPY --from=builder /opt/conda/envs/qiime2/lib/libnghttp2.so.* /usr/local/lib/
150150
COPY --from=builder /opt/conda/envs/qiime2/lib/libssh2.so.1.0.1 /usr/local/lib/libssh2.so.1
151-
COPY --from=builder /opt/conda/envs/qiime2/lib/libssl.so.1.1 /usr/local/lib/libssl.so.1.1
151+
COPY --from=builder /opt/conda/envs/qiime2/lib/libssl.so* /usr/local/lib/
152152
RUN ln -s /usr/local/lib/libopenblasp-r0.3.25.so /usr/local/lib/libcblas.so.3
153153
RUN ln -s /usr/local/lib/libopenblasp-r0.3.25.so /usr/local/lib/liblapacke.so.3
154154
RUN for f in `echo "libssu.so libhdf5_cpp.so.103 liblapacke.so.3 libcblas.so.3 libhdf5_hl_cpp.so.100 libhdf5_hl.so.100 libhdf5.so.103 libcrypto.so.1.1 libcurl.so.4 libgfortran.so.5 libnghttp2.so.14 libssh2.so.1 libssl.so.1.1 libquadmath.so.0"`; do ln -s /usr/local/lib/$f /lib/x86_64-linux-gnu/$f; done
@@ -180,4 +180,4 @@ RUN sed -i -E "s/^START_SCRIPT = .+/START_SCRIPT = python \/start_plugin.py qtp-
180180
# for testing
181181
COPY test_plugin.sh /test_plugin.sh
182182

183-
CMD ["./start_qtp-diversity.sh"]
183+
CMD ["./start_plugin.sh"]

0 commit comments

Comments
 (0)