-
Notifications
You must be signed in to change notification settings - Fork 30
garbage collector system overhaul and extras #1724
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
Merged
GitHK
merged 68 commits into
ITISFoundation:master
from
GitHK:gc-remove-guests-with-no-projects
Sep 15, 2020
Merged
Changes from 1 commit
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
fcd3ca9
guests without projects are now removed
1efac17
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 0afaefa
makes no senso for GUEST users
4620bd8
Merge remote-tracking branch 'upstream/master' into gc-remove-guests-…
ae7e470
garbage collection should result more descirptive
d5e33f8
added more notes
3a2f83a
added more notes
9f15f2c
minor refactor
6f2aaf5
fixed error
7dfb928
functions have been split into smaller self contained entities
dbbbb48
removed outdated description
7a16518
moved code blocks up
bba42b6
removed unused function
53415cb
moved function position
076f238
Merge remote-tracking branch 'upstream/master' into gc-remove-guests-…
67be62d
changed import name
57b5092
gc takes into consideration access rights when removing users
575e036
fixed pylint issues
519646b
removed unused code
59a457f
refactored code
1259919
updated changelog
2101786
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK e47fb02
added link in changelog
a3bfc58
PRs are now linked in the changelog
55ce9e7
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 685c5f2
marking container nvidia-smi for autoremove
3e7578a
during testing aiohttp logs are displayed
6e78b49
removed unencessary comments
caa3f6b
fixed issue which caused silent 401
3e748ea
replaced with a more updated version
ce07201
added empty_project_data and trapped a new error
5eeeb76
updated dependencies
d635e70
removed unused import
8660d17
added tests for garbage collection
e94a793
fixed some dependencies
8fa678c
conftest refactoring
a3e1b30
now creates a template db to be used between tests
3151e22
tests correctly run in integration testing
717e2a9
refactored tests
d4e5179
fixed an issue with project ownership detection
a9d8124
implemented t6
673a7ff
implemented t7
8d80661
implemented t8
6f15ca1
implemented t9
6764f59
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 5a086e4
updated changelog
36bbae6
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 49b8a1c
removed usless test module
d33f963
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 1d9b59a
@sanderegg suggestion
081b052
@sanderegg should adds clarity to the reasoning
cfe17a2
@sanderegg renaming function to something more clear
8d7889a
removed typo
acb935d
@sanderegg removed redundant comments
1a77741
@sanderegg refactored fixture into a function
07793aa
fixing out more typos
cb03419
minor refactor and cleanup
3853210
introduce tests to handle multiple user deletion
28c1d9d
removed uneeded comment
345e41e
used @pcrespov suggestion and refactored
f070c1f
reverting depenencies for fugure upgrade @pcrespov
cca0789
moved in a more adequate palce
fc10058
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 1e2007a
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK e7458e4
changed outdated pytest fixture
59686c6
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 5bb56b9
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK 7d5a2a3
Merge branch 'master' into gc-remove-guests-with-no-projects
GitHK File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
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.
Uh oh!
There was an error while loading. Please reload this page.