Commit Graph

33 Commits

Author SHA1 Message Date
e7bc93852d Add latest changes from gitlab-org/gitlab@master 2020-05-21 21:08:31 +00:00
7b52c7cb63 Add latest changes from gitlab-org/gitlab@master 2020-03-02 18:07:42 +00:00
33795139ea Add latest changes from gitlab-org/gitlab@master 2020-02-19 18:09:10 +00:00
411cc77938 Add latest changes from gitlab-org/gitlab@master 2020-01-24 18:09:00 +00:00
3fe9588b1c Add latest changes from gitlab-org/gitlab@master 2019-10-30 15:14:17 +00:00
25989ab7ef Add latest changes from gitlab-org/gitlab@master 2019-10-18 11:11:44 +00:00
0ab89d8e36 Add a rubocop for Rails.logger
Suggests to use a JSON structured log instead

Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/54102
2019-07-10 19:26:47 +00:00
ae1487a8c8 Externalize strings in app/workers
- email_receiver_worker
- object_storage/migrate_uploads_worker
- Update PO file
2019-04-10 13:30:09 +00:00
6fbdc5ed52 Apply patches when creating MR via email
This allows users to add patches as attachments to merge request
created via email.

When an email to create a merge request is sent, all the attachments
ending in `.patch` will be applied to the branch specified in the
subject of the email. If the branch did not exist, it will be created
from the HEAD of the repository.

When the patches could not be applied, the error message will be
replied to the user.

The patches can have a maximum combined size of 2MB for now.
2018-11-07 16:27:55 +01:00
3bfe306624 Resolve Naming/UncommunicativeMethod 2018-07-09 21:13:08 +08:00
dfbe5ce435 Enable frozen string literals for app/workers/*.rb 2018-06-27 07:23:28 +00:00
29be9c1acc Merge branch 'dm-application-worker' into 'master'
Add ApplicationWorker and make every worker include it

See merge request gitlab-org/gitlab-ce!15632
2017-12-05 13:06:22 +00:00
0b15570e49 Add ApplicationWorker and make every worker include it 2017-12-05 11:59:39 +01:00
8cce70730c Create merge request from email
* new merge request can be created by sending an email to the specific
email address (similar to creating issues by email)
* for the first iteration, source branch must be specified in the mail
subject, other merge request parameters can not be set yet
* user should enable "Receive notifications about your own activity" in
user settings to receive a notification about created merge request

Part of #32878
2017-12-05 08:41:58 +01:00
f097e4dbcd Don't send rejection mails for all auto-generated mails
Also make it easier to have mailer helper
2017-08-03 00:40:10 +08:00
97731760d7 Re-organize queues to use for Sidekiq
Dumping too many jobs in the same queue (e.g. the "default" queue) is a
dangerous setup. Jobs that take a long time to process can effectively
block any other work from being performed given there are enough of
these jobs.

Furthermore it becomes harder to monitor the jobs as a single queue
could contain jobs for different workers. In such a setup the only
reliable way of getting counts per job is to iterate over all jobs in a
queue, which is a rather time consuming process.

By using separate queues for various workers we have better control over
throughput, we can add weight to queues, and we can monitor queues
better. Some workers still use the same queue whenever their work is
related. For example, the various CI pipeline workers use the same
"pipeline" queue.

This commit includes a Rails migration that moves Sidekiq jobs from the
old queues to the new ones. This migration also takes care of doing the
inverse if ever needed. This does require downtime as otherwise new jobs
could be scheduled in the old queues after this migration completes.

This commit also includes an RSpec test that blacklists the use of the
"default" queue and ensures cron workers use the "cronjob" queue.

Fixes gitlab-org/gitlab-ce#23370
2016-10-21 18:17:07 +02:00
c55eebb283 An instruction for what to do 2016-06-15 15:49:01 +08:00
cc69bd07e7 This is easier to read for me. No early return
76dbafba86 (note_4575102)
2016-05-24 15:16:16 +08:00
1f5d55907a Merge the places where exceptions could be raised 2016-05-24 01:23:07 +08:00
c337e748d3 so we use separate classes to handle different tasks 2016-05-18 17:25:45 -05:00
8156475ea5 Report better errors. TODO: Enable skipped test 2016-05-16 21:27:16 +00:00
eac3583340 Update wording because we're overloading UserNotAuthorizedError 2016-05-16 21:27:16 +00:00
034e752aa4 Handle InvalidIssueError as InvalidNoteError 2016-05-16 21:27:16 +00:00
e92ceb7b57 fix specs 2015-11-30 16:12:31 +02:00
7f214cee74 Migrate mailers to ActiveJob 2015-11-26 17:03:43 +02:00
ee028d9d60 Rename reply_by_email to incoming_email to prepare for the future. 2015-09-21 10:35:37 +02:00
2088880383 Ignore empty incoming messages. 2015-08-24 10:57:35 -07:00
69708dab9f Block blocked users from replying to threads by email. 2015-08-21 10:14:45 -07:00
2f78b5e8af Make error class names more consistent. 2015-08-20 11:33:18 -07:00
e9972efc2f Extract ReplyParser and AttachmentUploader from Receiver. 2015-08-20 11:05:06 -07:00
76dbafba86 Send a rejection email when the incoming email couldn't be processed. 2015-08-19 11:10:21 -07:00
8906cabae7 Changes and stuff. 2015-08-18 17:02:26 -07:00
f76eac56b9 Reply by email POC 2015-08-18 15:46:36 -07:00