- Notifications
You must be signed in to change notification settings - Fork1.1k
chore: add prebuilds system user#16916
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Changes from1 commit
Commits
Show all changes
28 commits Select commitHold shift + click to select a range
300e80f add prebuilds system user database changes and associated changes
SasSwartb788237 optionally prevent system users from counting to user count
dannykopping8122595 appease the linter
dannykoppingbfb7c28 add unit test for system user behaviour
dannykopping6639167 reverting RBAC changes; not relevant here
dannykopping769ae1d removing unnecessary changes
dannykoppinge7e9c27 exclude system user db tests from non-linux OSs
dannykopping3936047 Rename prebuild system user reference
SasSwart8bdcafb ensure that users.IsSystem is not nullable
SasSwart324fde2 Fixes
dannykopping81d9dfa Merge remote-tracking branch 'origin/main' into prebuilds-system-user
SasSwart896c881 renumber migrations
SasSwartde4fb8a ensure that system users are filtered and returned consistently
SasSwart2751d5b make -B lint
SasSwart1042c39 rewrite prebuilds system user tests in our usual style
SasSwartf9e9d11 add support for prebuilds user to dbmem
SasSwart7492965 appease the linter
SasSwart29e2020 add support for the prebuilds system user to dbmem
SasSwart8c51585 linter
SasSwartcdc5c71 fix dbmem tests
SasSwart0d4813a remove restriction on modifying system users for now
SasSwart95d70a3 remove system user index
SasSwart8f1d71c Merge remote-tracking branch 'origin/main' into prebuilds-system-user
SasSwart7e009e5 invert tests that check for system user update protection
SasSwartaddd7c6 lint
SasSwart7a4ef24 Allow TestUpdateSystemUser to run against dbmem
SasSwartf30ce72 Merge remote-tracking branch 'origin/main' into prebuilds-system-user
SasSwart5f0ae5e Renumber migrations
SasSwartFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
Allow TestUpdateSystemUser to run against dbmem
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit7a4ef24f62270d39c97e816e60577baed40272c8
There are no files selected for viewing
7 changes: 1 addition & 6 deletionscoderd/database/querier_test.go
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
Oops, something went wrong.
Uh oh!
There was an error while loading.Please reload this page.
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.