Skip to content
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

Testing: Add DUNE VO tests #6383 #6563

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jamesp-epcc
Copy link
Contributor

This is a first attempt at adding some VO tests for DUNE. It tests all of the DUNE-specific functionality in our policy package and performs a complete replication workflow from deterministic to non-deterministic storage. Because we need an environment with multiple containers so that we can include a fake MetaCat server, this is modeled more on the integration tests than on the other VO tests.

I am open to suggestions on how to integrate this better with the other VO tests. For now I have added a separate dune-tests job to the top level workflow, because creating a full docker-compose environment doesn't fit well with the existing ATLAS and BelleII tests that just use a single container. However there might be a better way, especially if other VOs want to do something similar to DUNE in the future.

These tests rely on the MetaCat container, which is on my Docker account, and the DUNE policy package, which is on my GitHub account. But it might be possible to move these to more official homes, such as the official DUNE GitHub account, if you would prefer not to have the tests rely on code from personal accounts.

@maany
Copy link
Member

maany commented Apr 10, 2024

I'm working on #6667 . Once merged, James can rebase this PR and adapt the DUNE VOTests to use the framework.

@bari12
Copy link
Member

bari12 commented Jun 4, 2024

@maany is there an update on this?

@maany
Copy link
Member

maany commented Jun 4, 2024

@bari12 this fell off the radar. I can get something working for @jamesp-epcc by Monday

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants