-
Notifications
You must be signed in to change notification settings - Fork 11
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
Support local condor. #1139
base: master
Are you sure you want to change the base?
Support local condor. #1139
Conversation
local condor support was put in specifically for FNAL use and maintained by Eric Vaandering there. I will not risk breaking it by calling /cvmfs/cms.cern,ch/cmsser_default.sh w/o a request from them. Did you look into defining VO_CMS_SW_DIR on your local system ? |
Very interesting discussion/patch. I'm supporting local condor at Caltech too. And what I needed to do is to have a local CRAB copy, where I had to patch PRODCOMMON with a one-line patch, which inserts something into the Condor submit file :
So the user is free to set whatever works in his/her environment and Condor jobs will inherit the same supposedly-working settings. If the submit node is uniform with worker nodes. With that, it works fine for me. I guess that it would be less sensitive to other things. Concerning support at FNAL I'm forwarding an email to Stefano with a recent discussion. Might help deciding what to do here. The advantage of using that upstream is that users will be able to use CRAB straight out of CVMFS and have the same workflow in our T3, lxplus, anywhere CVMFS-enabled. So if you're interested I could provide the formal patch for that. |
Samir, so you do not need any patch. Will that have the needed flexibility to accomodate anybody ? Of course if @ericvaandering says that they are happy to use /cvmfs at fnal I have no problem in acceptin Geonmo's patch |
Stefano, It might work. I remember seeing it somewhere but never found it again. Thanks for the pointer. Even though the docs say "Works both for gLite and remoteGlidein" I think I've seen support in the Condor code too. Will try that. Thanks! |
ah.. correct. I think it will not work for local condor :-( On 07/21/2014 07:07 PM, samircury wrote:
Stefano Belforte - I.N.F.N. tel : +39 040 375-6261 (fax: 375-6258) |
Having a hard time imagining how things are passed along though. Does this look a correct configuration for what I want to achieve : [GRID] Should be easy to add for the people that know the code for long/wrote it :-) Any pointers are welcome. If I manage to include support to that will that be politically feasible to merge it? |
you need a semicolon at the end: If needed I'll look into adding it to local condor scheduler |
In principle, I see nothing wrong with @geonmo 's patch. But we would want to test it at FNAL first. I might be able to get to it later this week. |
Hello, everyone. |
Hello, everyone. |
I have not. Thanks for the reminder. |
I think at this late stage I'm not willing to add a whole section of code for condor and risk things at FNAL. You are welcome to create a patch and apply it to what you build, of course. This will not change anymore, so it is a single patch and a command to apply it. Let me tell you how we get around this at FNAL: we have people put this single line: I suspect your way is better, but at this point, it's not worth the effort and risk that something goes wrong at FNAL. Eric |
Dear admin,
I updated to use condor scheduler on CVMFS system.
I tested at local machine and checked cmssw.sh file is created correctly.
Could you merge this commit?