Skip to content

Commit

Permalink
Replace busybox:ubuntu-14.04 image with busybox:1.36.1-glibc
Browse files Browse the repository at this point in the history
  • Loading branch information
mvdbeek committed Jun 21, 2024
1 parent f5610e7 commit 98608cc
Show file tree
Hide file tree
Showing 15 changed files with 286 additions and 275 deletions.
4 changes: 2 additions & 2 deletions config/job_conf.yml.interactivetools
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ execution:

#docker_cmd: /usr/local/custom_docker/docker
#docker_host:
#docker_container_id_override: busybox:ubuntu-14.04
#docker_default_container_id: busybox:ubuntu-14.04
#docker_container_id_override: busybox:1.36.1-glibc
#docker_default_container_id: busybox:1.36.1-glibc
#require_container: true
#container_monitor: true
#container_monitor_result: file
Expand Down
2 changes: 1 addition & 1 deletion doc/source/admin/jobs.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ execution:
k8s_namespace: default
runner: pulsar_k8s
docker_enabled: true
docker_default_container_id: busybox:ubuntu-14.04
docker_default_container_id: busybox:1.36.1-glibc
pulsar_app_config:
message_queue_url: 'amqp://guest:[email protected]:5672//'
local_environment:
Expand Down
4 changes: 2 additions & 2 deletions doc/source/admin/special_topics/interactivetools.rst
Original file line number Diff line number Diff line change
Expand Up @@ -172,8 +172,8 @@ An example ``job_conf.yml`` file as seen in ``config/job_conf.yml.interactivetoo
#docker_cmd: /usr/local/custom_docker/docker
#docker_host:
#docker_container_id_override: busybox:ubuntu-14.04
#docker_default_container_id: busybox:ubuntu-14.04
#docker_container_id_override: busybox:1.36.1-glibc
#docker_default_container_id: busybox:1.36.1-glibc
#require_container: true
#container_monitor: true
#container_monitor_result: file
Expand Down
2 changes: 1 addition & 1 deletion doc/source/admin/special_topics/mulled_containers.rst
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ requirement field.
<requirements>
<!-- Container based dependency handling -->
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
<!-- Conda based dependency handling -->
<requirement type="package" version="8.22">gnu_coreutils</requirement>
</requirements>
Expand Down
8 changes: 4 additions & 4 deletions lib/galaxy/config/sample/job_conf.sample.yml
Original file line number Diff line number Diff line change
Expand Up @@ -513,13 +513,13 @@ execution:
# trust tool's specified container - a destination wide override
# can be set. This will cause all jobs on this destination to use
# that docker image.
#docker_container_id_override: busybox:ubuntu-14.04
#docker_container_id_override: busybox:1.36.1-glibc

# Likewise, if deployer wants to use docker for isolation and
# does trust tool's specified container - but also wants tool's not
# configured to run in a container the following option can provide
# a fallback. -->
#docker_default_container_id: busybox:ubuntu-14.04
#docker_default_container_id: busybox:1.36.1-glibc

# If the destination should be secured to only allow containerized jobs
# the following parameter may be set for the job destination. Some tools
Expand Down Expand Up @@ -633,7 +633,7 @@ execution:
- type: docker
shell: '/bin/sh'
resolve_dependencies: false
identifier: 'busybox:ubuntu-14.04'
identifier: 'busybox:1.36.1-glibc'
- type: singularity
shell: '/bin/sh'
resolve_dependencies: false
Expand All @@ -642,7 +642,7 @@ execution:
- type: docker
shell: '/bin/sh'
resolve_dependencies: false
identifier: 'busybox:ubuntu-14.04'
identifier: 'busybox:1.36.1-glibc'
- type: singularity
shell: '/bin/sh'
resolve_dependencies: false
Expand Down
515 changes: 263 additions & 252 deletions lib/galaxy/config/sample/job_conf.xml.sample_advanced

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion test/functional/tools/catDocker.xml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<tool id="catdc" name="Concatenate datasets (in docker)" version="1.0.0">
<description>tail-to-head</description>
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command><![CDATA[
echo "Galaxy slots passed through contain as \$GALAXY_SLOTS" &&
Expand Down
2 changes: 1 addition & 1 deletion test/functional/tools/job_environment_default.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<tool id="job_environment_default" name="job_environment_default" version="0.1.0" profile="18.01">
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command detect_errors="exit_code"><![CDATA[
mktemp -d &&
Expand Down
2 changes: 1 addition & 1 deletion test/functional/tools/job_environment_default_legacy.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<tool id="job_environment_default_legacy" name="job_environment_default_legacy" version="0.1.0">
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command detect_errors="exit_code"><![CDATA[
mktemp -d &&
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<tool id="job_environment_explicit_isolated_home" name="job_environment_explicit_isolated_home" version="0.1.0">
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command use_shared_home="false"><![CDATA[
mktemp -d &&
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<tool id="job_environment_explicit_shared_home" name="job_environment_explicit_shared_home" version="0.1.0" profile="18.01">
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command use_shared_home="true"><![CDATA[
mktemp -d &&
Expand Down
2 changes: 1 addition & 1 deletion test/functional/tools/tool_directory_docker.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<tool id="tool_directory_docker" name="tool_directory_docker" version="1.0.0">
<requirements>
<container type="docker">busybox:ubuntu-14.04</container>
<container type="docker">busybox:1.36.1-glibc</container>
</requirements>
<command><![CDATA[
cp '$__tool_directory__/tool_directory.xml' output1
Expand Down
6 changes: 3 additions & 3 deletions test/integration/test_coexecution.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@
k8s_namespace: ${k8s_namespace}
runner: pulsar_k8s
docker_enabled: true
docker_default_container_id: busybox:ubuntu-14.04
docker_default_container_id: busybox:1.36.1-glibc
pulsar_app_config:
message_queue_url: '${container_amqp_url}'
env:
Expand Down Expand Up @@ -145,7 +145,7 @@ def job_config(template_str: str, jobs_directory: str) -> str:
runner: pulsar_tes
tes_url: ${tes_url}
docker_enabled: true
docker_default_container_id: busybox:ubuntu-14.04
docker_default_container_id: busybox:1.36.1-glibc
pulsar_app_config:
message_queue_url: '${container_amqp_url}'
env:
Expand Down Expand Up @@ -204,7 +204,7 @@ def job_config(template_str: str, jobs_directory: str) -> str:
runner: pulsar_tes
tes_url: ${tes_url}
docker_enabled: true
docker_default_container_id: busybox:ubuntu-14.04
docker_default_container_id: busybox:1.36.1-glibc
pulsar_app_config:
message_queue_url: '${container_amqp_url}'
env:
Expand Down
6 changes: 3 additions & 3 deletions test/integration/test_kubernetes_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -133,21 +133,21 @@ def job_config(jobs_directory: str) -> Config:
<param id="limits_cpu">1.1</param>
<param id="limits_memory">100M</param>
<param id="docker_enabled">true</param>
<param id="docker_default_container_id">busybox:ubuntu-14.04</param>
<param id="docker_default_container_id">busybox:1.36.1-glibc</param>
<env id="SOME_ENV_VAR">42</env>
</destination>
<destination id="k8s_destination_walltime_short" runner="k8s_walltime_short">
<param id="limits_cpu">1.1</param>
<param id="limits_memory">100M</param>
<param id="docker_enabled">true</param>
<param id="docker_default_container_id">busybox:ubuntu-14.04</param>
<param id="docker_default_container_id">busybox:1.36.1-glibc</param>
<env id="SOME_ENV_VAR">42</env>
</destination>
<destination id="k8s_destination_no_cleanup" runner="k8s_no_cleanup">
<param id="limits_cpu">1.1</param>
<param id="limits_memory">100M</param>
<param id="docker_enabled">true</param>
<param id="docker_default_container_id">busybox:ubuntu-14.04</param>
<param id="docker_default_container_id">busybox:1.36.1-glibc</param>
<env id="SOME_ENV_VAR">42</env>
</destination>
<destination id="local_dest" runner="local">
Expand Down
2 changes: 1 addition & 1 deletion test/unit/app/jobs/test_job_configuration.py
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ def test_container_tag_in_destination(self):
assert len(container) == 2
container0 = container[0]
assert container0["type"] == "docker"
assert container0["identifier"] == "busybox:ubuntu-14.04"
assert container0["identifier"] == "busybox:1.36.1-glibc"

container_override = container_dest.params["container_override"]
assert len(container_override) == 2
Expand Down

0 comments on commit 98608cc

Please sign in to comment.