-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Move migrating repository from frontend to backend #6200
Merged
Merged
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
b8d2762
move migrating to backend
lunny b8d9e8c
add loading image when migrating and fix tests
lunny 258afa4
fix format
lunny 58d35b0
fix lint
lunny 7771acb
add redis task queue support and improve docs
lunny ac357dc
add redis vendor
lunny dff614f
fix vet
lunny 75f2aa3
add database migrations and fix app.ini sample
lunny 585fa50
add comments for task section on app.ini.sample
lunny fffcc0e
Update models/migrations/v84.go
lafriks 8a29d8a
Update models/repo.go
lafriks 40a6d64
move migrating to backend
lunny 2bba6a9
add loading image when migrating and fix tests
lunny 2b1626d
fix fmt
lunny 6b6ece7
add redis task queue support and improve docs
lunny cf24a79
fix fixtures
lunny 2643529
fix fixtures
lunny e018d40
fix duplicate function on index.js
lunny fe4557b
fix tests
lunny 8187434
rename repository statuses
lunny 10caeeb
check if repository is being create when SSH request
lunny 8abee04
fix lint
lunny a1b539c
fix template
lunny dd0550e
some improvements
lunny 65c3582
fix template
lunny 26d82d3
unified migrate options
lunny a1b1744
fix lint
lunny 5384a12
fix loading page
lunny 19d10ee
refactor
lunny 38b6726
When gitea restart, don't restart the running tasks because we may ha…
lunny 237c73a
fix js
lunny 4ef33d6
fix tests
lunny e4adc77
Update models/repo.go
lunny 6c6bde7
Update docs/content/doc/advanced/config-cheat-sheet.en-us.md
lunny 220de6f
rename ErrTaskIsNotExist to ErrTaskDoesNotExist
lunny a1a1358
delete release after add one on tests to make it run happy
lunny 45ab0cd
fix tests
lunny 5e1888f
fix tests
lunny fb4e7f3
improve codes
lunny 80c9e8d
fix lint
lunny 8edb889
fix lint
lunny badbf9e
fix migrations
lunny 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 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -77,3 +77,4 @@ prime/ | |
*.snap | ||
*.snap-build | ||
*_source.tar.bz2 | ||
.DS_Store |
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we add a queue that long as default I'd say we'll need a queue management page. That would be overkill for this PR, so I'd reduce this
QUEUE_LENGTH
to a much lower value (20?).There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@guillep2k all other queue_length is also default
1000
. i.e. webhook's . A task/queue management will be next PRs.