Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
8 changes: 1 addition & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -233,22 +233,16 @@ jobs:
test:
runs-on: ubuntu-24.04
steps:
- name: Build something else
uses: docker/build-push-action@v6
with:
tags: localhost/fnndsc/pman:dev
load: true
- name: Run ChRIS backend integration tests
uses: FNNDSC/ChRIS_ultron_backEnd@master
# all inputs are optional
with:
engine: docker # or podman
command: test-integration # or test-unit, ...
# optionally change image used for pman, pfcon, or cube
# optionally change image used for pfcon, or cube
env:
CUBE_IMAGE: localhost/fnndsc/cube:dev
PFCON_IMAGE: localhost/fnndsc/pfcon:dev
PMAN_IMAGE: localhost/fnndsc/pman:dev
```

## Documentation
Expand Down
2 changes: 1 addition & 1 deletion chris_backend/plugininstances/services/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -369,7 +369,7 @@ def delete_plugin_instance_job_from_remote(self):
try:
self._delete_job(job_id)
except PfconRequestException as e:
if '404' in str(e):
if e.code == 404:
logger.error(f'Job {job_id} not found in remote fcon '
f'url -->{self.pfcon_client.url}<--, detail: {str(e)}')
if self.c_plugin_inst.error_code == 'CODE12':
Expand Down
2 changes: 1 addition & 1 deletion chris_backend/plugininstances/tests/test_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ def test_mananger_can_check_plugin_instance_app_exec_status(self):
# }
# post_mock.assert_called_with(msg)

@tag('integration', 'error-pman')
@tag('integration', 'error-pfcon')
def test_integration_mananger_can_run_and_check_plugin_instance_app_exec_status(self):
"""
Test whether the manager can check a plugin's app execution status.
Expand Down
2 changes: 1 addition & 1 deletion chris_backend/plugininstances/tests/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -672,7 +672,7 @@ def test_plugin_instance_detail_success(self):
# # check that the check_plugin_instance_exec_status task was called with appropriate args
# delay_mock.assert_called_with(self.pl_inst.id)

@tag('integration', 'error-pman')
@tag('integration', 'error-pfcon')
def test_integration_plugin_instance_detail_success(self):

# add an FS plugin to the system
Expand Down
28 changes: 7 additions & 21 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -124,46 +124,32 @@ services:
pfcon:
image: ${PFCON_IMAGE:-ghcr.io/fnndsc/pfcon:latest}
environment:
COMPUTE_SERVICE_URL: http://pman:5010/api/v1/
SECRET_KEY: secret
PFCON_USER: pfcon
PFCON_PASSWORD: pfcon1234
PFCON_INNETWORK: "true"
STORAGE_ENV: "fslink"
STOREBASE_MOUNT: /var/local/storeBase
CONTAINER_ENV: docker
CONTAINER_USER: "${UID}:${GID}"
ENABLE_HOME_WORKAROUND: "yes"
JOB_LABELS: "org.chrisproject.miniChRIS=plugininstance"
REMOVE_JOBS: "yes"
# port is hard-coded in chris_backend/config/settings/local.py
command: gunicorn -b 0.0.0.0:30005 -w 8 -t 120 pfcon.wsgi:application
ports:
- "30005:30005"
volumes:
- chris_files:/var/local/storeBase
user: ${UID}:${GID}
- "${DOCKER_SOCK:-/var/run/docker.sock}:/var/run/docker.sock"
networks:
local:
aliases:
- pfcon.remote # hard-coded in chris_backend/config/settings/local.py
userns_mode: "host"
labels:
org.chrisproject.role: "pfcon"

pman:
image: ${PMAN_IMAGE:-ghcr.io/fnndsc/pman:latest}
environment:
CONTAINER_ENV: docker
CONTAINER_USER: "${UID}:${GID}"
ENABLE_HOME_WORKAROUND: "yes"
JOB_LABELS: "org.chrisproject.miniChRIS=plugininstance"
SECRET_KEY: secret
REMOVE_JOBS: "yes"
volumes:
- "${DOCKER_SOCK:-/var/run/docker.sock}:/var/run/docker.sock"
depends_on:
- pfcon
ports:
- "5010:5010"
networks:
- local
userns_mode: "host"

nats:
image: docker.io/library/nats:2.10.20-alpine3.20
ports:
Expand Down
2 changes: 1 addition & 1 deletion justfile
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ nuke: reap-plugin-instances (docker-compose '--profile=cube --profile=tools down

# Remove all plugin instance containers.
[group('(2) shutdown')]
reap-plugin-instances: (docker-compose 'run --rm pman python -c' '''
reap-plugin-instances: (docker-compose 'run --rm pfcon python -c' '''
'
import os
import docker
Expand Down
2 changes: 1 addition & 1 deletion requirements/base.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ django-storage-swift==1.4.0
celery==5.5.2
django-celery-beat==2.8.1
python-chrisstoreclient==1.0.0
python-pfconclient==3.2.0
python-pfconclient==3.3.0
django-auth-ldap==5.2.0
PyYAML==6.0.2
whitenoise[brotli]==6.9.0
Expand Down
Loading