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

check user partitions and show custom error messages #4

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

brnovasco
Copy link
Collaborator

@brnovasco brnovasco commented Jul 25, 2024

Changes made only in the user-partitions.sh script

  • New feature for checking if user is registered in SLURM before running other commands.
  • Fix missing custom error_exit function for showing custom error messages and exit.

Checking user is supposed to be redundant on the app context and in our cluster's account management, but this addition should make the script more complete and predictable for other use cases.

@brnovasco brnovasco assigned brnovasco and matyson and unassigned brnovasco and matyson Jul 25, 2024
Copy link
Collaborator

@matyson matyson left a comment

Choose a reason for hiding this comment

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

ok.

@matyson matyson merged commit 2369ee3 into main Jul 26, 2024
3 checks passed
@matyson matyson deleted the check-user-and-error-messages branch July 26, 2024 12:45
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