inspirehep / inspire-docker

Dockerfiles for inspirehep/inspire-next application
GNU General Public License v2.0
4 stars 13 forks source link

docker: added support for xrootd-python #45

Closed ammirate closed 7 years ago

jacquerie commented 7 years ago

Uhm, we're not going to need this for a while, or am I missing something in how close we are to have a working Jenkins infrastructure running on CERN's hardware, @david-caro ?

jacquerie commented 7 years ago

I'm not against this PR, eh, I was just surprised by it!

david-caro commented 7 years ago

In order to be able to install pyxrootd (on one of the other xrootd python packages) you need to have installed the python-xrootd library, that is not distributed through pypi. This work started wen the xrootd extension was being added, and was not an optional extra. I also know that there's some docker containers with eos running around cern CI that we can try to use on the jenkins at some point.

jacquerie commented 7 years ago

This work started wen the xrootd extension was being added, and was not an optional extra.

Ah, I see. Shall we merge this and revert https://github.com/inspirehep/inspire-next/commit/dce0e3b3fe7df3329667e36a948313ed4984ada5 then?

david-caro commented 7 years ago

That would be the way to go I think, yes, there's no hurry though.