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

Dictionary generation not working on batch jobs (condor) #485

Merged
merged 8 commits into from
Nov 17, 2023

Conversation

lobis
Copy link
Member

@lobis lobis commented Sep 26, 2023

lobis Ok: 61 Powered by Pull Request Badge

After this PR, the environment varible REST_HOME will be used to define the "rest home directory" (currently $HOME/.rest. If REST_HOME does not exist, it will default to the old behaviour.

This is because using home directory can lead to problems in some cases, expecially when running batch jobs, the gdml geometry is replicated multiple times for each job, which causes the disk to fill up quite fast and leads to errors.

With this feature the user could use the /tmp directory (for example) to store these files.

@lobis lobis linked an issue Sep 26, 2023 that may be closed by this pull request
@lobis lobis requested review from jgalan, nkx111 and juanangp September 26, 2023 10:19
@lobis lobis marked this pull request as ready for review September 26, 2023 10:39
@lobis lobis marked this pull request as draft September 26, 2023 10:40
@lobis lobis marked this pull request as ready for review September 26, 2023 11:30
Copy link
Member

@jgalan jgalan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve, it would be interesting that thisREST.sh defines a default REST_HOME?

@lobis
Copy link
Member Author

lobis commented Nov 17, 2023

I approve, it would be interesting that thisREST.sh defines a default REST_HOME?

Done in 9f916cf. With this change users should be aware that this env variable should be overrided after loading thisREST.sh (before this change it didn't matter).

@lobis lobis merged commit 3d3951f into master Nov 17, 2023
63 checks passed
@lobis lobis deleted the 481-error-running-restmanager-using-condor branch November 17, 2023 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error running restManager using condor
3 participants