Test Fixes and CI - Githubissues

key4hep / k4LCIOReader

Generate EDM4hep collections from LCIO format data

Apache License 2.0

2 stars 11 forks source link

Test Fixes and CI #16

Closed vvolkl closed 2 years ago

vvolkl commented 2 years ago

BEGINRELEASENOTES

BUILD_TESTING is a CTest builtin, no need to declare it as an option
use test file from key4hep www space
rename read_lcio exe to k4lcioreader_read-lcio - better in case it will be installed in the future
simple cli for the read-lcio test (filename)

ENDRELEASENOTES

mirguest commented 2 years ago

Hi @vvolkl, After I merge this PR, the CI runs with following errros:

Run docker run -it --name CI_container -v ${GITHUB_WORKSPACE}:/Package -v /cvmfs:/cvmfs:shared -d clicdp/cc7-lcg /bin/bash
Unable to find image 'clicdp/cc7-lcg:latest' locally
docker: Error response from daemon: pull access denied for clicdp/cc7-lcg, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.

Could you help us solve this problem? Thank you! Tao

andresailer commented 2 years ago

Should be fixed by #18 . I closed the dockerhub account because of changing terms of service.