Run tests in Slurm test env #88
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Adds CI for running tests * Fix CI typo * Adds setup step for python * Adds minimal pyproject.toml file * Update pyproject.toml * Adds missing --with directive * Adds missing description field * Adds missing docstring [skip ci] * Updates the way coverage is run * Expand .gitignore
* Moves exception pattern checking into dedicated method * Update top of file comments for clarity * Deletes test placeholder * Fix boolean typo
* Switch to builtin logging * Explicitly set logging level * Moves logging to /var * Log directly to syslog * Add missing import
* Replace accumulator patterns with single iteration * Fix typo in list unpacking
* Drop support for regex ignore patterns * Increase number/detail of log messages * Drop Python2 support and impliment Python3 features * Delete test_check_ignore_node.py * Simplify set logic
* Adds basic whitelist * Replaces usernames with uid * Updates docs and comments * Ignore uid from 0 to 15000
* Move source into a package * Add GPL3 license * Update pyproject.toml * Removes 'if __name__' conditional * Drop /home/djperrefort/GitHub/pitt-crc/shinigami reference in testing CI * Update test suite structure to match package structure
* Updates README * Updates package docstring
* Introduces settings module * Configure settings to load from disk * Add dedicated method for loading settings from disk * Adds ability to skip FileNotFoundError error
* Initial commit of cli module * Fixes outdated import signature * Mves main function into Application class * Mid-flight cleanup pass * Updates import signatures in test suite
* Execute SSH commands asynchronously * Run async functins using asyncio.gather * Fix typo in SSH connection * Run all remote commands through SSH object * Imposes limit on max SSH connections * PEP8 * Drops old tests * Adds dummy test coverage for ci
* Adds tests for parser debug option * Adds tests for Settings class * Adds tests for id_in_whitelist function
* Adds configurable logging * Adds verbosity argument * Adds tests for logging configuration
There was a typo in the cron job example.
* Makes Application.settings private * Runs AI linter
* Drops GID whitelist * Drops GID related tests
* Move logic for loading settings into Settings class * Revert changes to parser class * Raise error on file not found * Revert last commit
* Terminate orphaned processes in userlist * Rename whitelist to blacklist * Kill processes using group ID * Fix bug in fetching of remote process data * updates tests * Lower pandas requirement
* Adds test coverage for settings file parsing * PEP8
* Outline CLI options to replace settings module * Print help text on error * Parse UID list as json string * Dynamically determine valid arguments from func signature * Add log message when there are no processes to terminate * Add nargs='+' to uid-whitelist arg * Updates docstring * Adds missing type hints * Introduce dedicat ssh argument group * Abstract away SSH options in function signatures * Adds tests for argument parsing * Drops application tests * Makes logging setup private * Shorten line lengths
* Bump actions/checkout from 3 to 4 (#81) Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Updates CI (#82) * Fix termination signal (#84) --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the CI/CD workflows to run the test suite in an environment where Slurm is installed.