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

[devel] Collect batch system info #222

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

aldbr
Copy link
Contributor

@aldbr aldbr commented Nov 10, 2023

Aims at solving DIRACGrid/DIRAC#7177
Associated with DIRACGrid/DIRAC#7289

@aldbr aldbr changed the title [devel] fix collect batch system info [devel] Collect batch system info Nov 10, 2023
@aldbr aldbr force-pushed the devel_FIX_collect-batch-system-info branch 5 times, most recently from 37f542a to 2913aea Compare November 10, 2023 15:38
Pilot/pilotTools.py Show resolved Hide resolved
pilotReference = "sshoar://" + ceName + "/" + os.environ["OAR_JOBID"]
batchSystemType = "OAR"
batchSystemJobID = os.environ["OAR_JOBID"]

Copy link
Contributor

Choose a reason for hiding this comment

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

batchSystemParameters["BinaryPath"] and batchSystemParameters["Queue"] for this case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is no OARResourceUsage module + I don't know how to get the BinaryPath and the Queue from the environment variables in this case.

Copy link
Contributor

Choose a reason for hiding this comment

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

Mmm maybe @atsareg knows what was this for. The feeling is that it could be dropped...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

From what I have seen, the project is not very known but still exists and has active developers working on it.

Pilot/pilotTools.py Outdated Show resolved Hide resolved
Pilot/pilotTools.py Outdated Show resolved Hide resolved
Pilot/pilotTools.py Show resolved Hide resolved
@aldbr aldbr force-pushed the devel_FIX_collect-batch-system-info branch 3 times, most recently from f5be1a6 to 5e0884a Compare November 20, 2023 13:26
@aldbr aldbr force-pushed the devel_FIX_collect-batch-system-info branch from 5e0884a to dd3f8d2 Compare November 23, 2023 12:15
@aldbr aldbr requested a review from fstagni November 23, 2023 16:31
@aldbr aldbr force-pushed the devel_FIX_collect-batch-system-info branch from dd3f8d2 to a3dbabf Compare December 11, 2023 07:19
@aldbr aldbr force-pushed the devel_FIX_collect-batch-system-info branch from a3dbabf to bc604ec Compare December 11, 2023 08:02
@fstagni fstagni merged commit 8f2617f into DIRACGrid:devel Jan 17, 2024
11 checks passed
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.

2 participants