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

Clean up task log entries by removing some configuration paths #1317

Open
prioux opened this issue Mar 30, 2023 · 0 comments
Open

Clean up task log entries by removing some configuration paths #1317

prioux opened this issue Mar 30, 2023 · 0 comments
Labels
Admin Features or bugs related to administrative features Priority: Low To implement when someone actually requests it!

Comments

@prioux
Copy link
Member

prioux commented Mar 30, 2023

Whenever anything is logged in a task's log, we should remove the constant paths that match configuration settings, such as the gridshare path or the DP cache path.

So, a task log entry that now looks like:

[2023-03-27 18:42:27 EDT] make_cluster_workdir() Trying to create workdir '/scratch/prioux/BourreauBeluga/GridShare/224/06/57/blah-FMRIprepBidsSubject-T2240657'.

would instead be logged as:

[2023-03-27 18:42:27 EDT] make_cluster_workdir() Trying to create workdir '224/06/57/blah-FMRIprepBidsSubject-T2240657'.

Given the CbrainTask class already has an overriden version of hte method 'addlog', it should be an easy fix to zap these things before actually logging:

def addlog(message,options={})

@prioux prioux added Priority: Low To implement when someone actually requests it! Admin Features or bugs related to administrative features labels Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Features or bugs related to administrative features Priority: Low To implement when someone actually requests it!
Projects
None yet
Development

No branches or pull requests

1 participant