Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Reenable podio tests in edm4hep workflows #359

Closed
tmadlener opened this issue Dec 16, 2022 · 1 comment
Closed

Reenable podio tests in edm4hep workflows #359

tmadlener opened this issue Dec 16, 2022 · 1 comment
Labels
CI issues with CI

Comments

@tmadlener
Copy link
Collaborator

    I have disabled the podio tests in the edm4hep workflows for the moment, because using `PODIO_SET_RPATH` does not seem to work in downstream edm4hep in this environment, even when also using `RPATH` to build there. I will open an issue to keep track of that, but in order to unblock this, I figured disabling the podio tests (which are run in any case in other workflows) is the easiest way forward.

Originally posted by @tmadlener in #333 (comment)

The main reason for this is that discovering the individual test cases from Catch2 runs an executable, which picks up the podio from the underlying system on ubuntu. This would be fixable by using RPATH during the build process, but that then breaks the edm4hep tests.

@tmadlener
Copy link
Collaborator Author

Given the large overlap between the stacks that we use for the edm4hep CI workflows and the ones that we use for building regular podio, there is no real benefit any longer from also running the podio tests for the workflows where we build and test EDM4hep. I am closing this.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CI issues with CI
Projects
None yet
Development

No branches or pull requests

1 participant