Avoid pulling all VSC accounts in slurm sync #157

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.

Sign up for GitHub

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

Jump to bottom

Open

wpoely86 wants to merge 6 commits into hpcugent:master from wpoely86:active_accounts

Collaborator

wpoely86 commented Aug 19, 2025

I think this should do it. @itkovian what do you think?

wpoely86 added 2 commits

August 19, 2025 08:16


          Let VO fetch directly filter on active accounts for slurm sync

4ed5591


          fix tests + bug fix

83b11b4

stdweird reviewed

View reviewed changes

bin/sync_slurm_acct.py Outdated Show resolved Hide resolved


          fix syntax

6f8fe4b

stdweird reviewed

View reviewed changes

test/slurm_sync.py Show resolved Hide resolved

stdweird reviewed

View reviewed changes

lib/vsc/administration/slurm/sync.py Outdated Show resolved Hide resolved

stdweird reviewed

View reviewed changes

lib/vsc/administration/slurm/sync.py Outdated Show resolved Hide resolved

stdweird reviewed

View reviewed changes

lib/vsc/administration/slurm/sync.py Outdated Show resolved Hide resolved

stdweird reviewed

View reviewed changes

lib/vsc/administration/slurm/sync.py Outdated Show resolved Hide resolved

wpoely86 added 3 commits

August 19, 2025 10:06


          correct parameter name

63d8ae9


          use consts for arguments


          fix remarks

82665fe

Collaborator Author

wpoely86 commented Aug 19, 2025

This is now running on our side without issue so far.

Collaborator Author

wpoely86 commented Sep 4, 2025

@stdweird or @itkovian can this get merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet