delta/gitlab/gitlab-ce.git/app/workers, branch fix/import-export-gitlab-errors gitlab.com: gitlab-org/gitlab-ce.git Merge branch 'repo-check-require-push-events' into 'master' 2016-06-17T19:14:43+00:00 Robert Speicher robert@gitlab.com 2016-06-17T19:14:43+00:00 1db4fd3ae7cd3a4fa2f356a4c252820e26783a27 Do not check repos without push events Reduce false positives from automatic repository checks by skipping projects without push events. See merge request !4684 

Do not check repos without push events

Reduce false positives from automatic repository checks by skipping
projects without push events.

See merge request !4684
 fixed merge conflicts on UI branch 2016-06-16T13:01:27+00:00 James Lopez james@jameslopez.es 2016-06-16T13:01:27+00:00 b3a0a1c3155702e2deddade0fea3e79087b40da4 






fixed merge conflicts
2016-06-16T12:07:49+00:00

James Lopez
james@jameslopez.es

2016-06-16T12:07:49+00:00

2a747d386dbdc05453fce6b8be3f483e8cd9e796












Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into feature/project-export
2016-06-16T11:04:00+00:00

James Lopez
james@jameslopez.es

2016-06-16T11:04:00+00:00

778d72664f386dfee45dab171f137395739958f6

# Conflicts:
#	app/models/ci/pipeline.rb





# Conflicts:
#	app/models/ci/pipeline.rb







Revert "squashed merge and fixed conflicts"
2016-06-16T10:59:07+00:00

James Lopez
james@jameslopez.es

2016-06-16T10:59:07+00:00

452c076a34cc11cc97f4b1c3113e86ce4367e055

This reverts commit 13e37a3ee5c943525a99481b855d654e97e8597c.





This reverts commit 13e37a3ee5c943525a99481b855d654e97e8597c.







squashed merge and fixed conflicts
2016-06-16T10:12:28+00:00

James Lopez
james@jameslopez.es

2016-06-16T10:12:27+00:00

13e37a3ee5c943525a99481b855d654e97e8597c












Do not check repos without push events
2016-06-15T17:11:25+00:00

Jacob Vosmaer
jacob@gitlab.com

2016-06-15T17:11:25+00:00

d9f6d5dd59281a8efcd6cb7b392b1ef3bafa778d












enable retry on worker
2016-06-15T07:06:10+00:00

James Lopez
james@jameslopez.es

2016-06-15T07:06:10+00:00

8922712693a57d6c65620d03599fe7c4b4f84119












some refactoring - renaming things, etc..
2016-06-15T07:03:54+00:00

James Lopez
james@jameslopez.es

2016-06-15T07:03:54+00:00

bacd2c4fa0253f7b7206dc949e5c2d8d132a8087












Merge branch 'dont-drop-stuck-builds' into 'master'

2016-06-15T02:54:19+00:00

Stan Hu
stanhu@gmail.com

2016-06-15T02:54:19+00:00

bf4455d14659f1fde6391164b38310d361bf407d


Don't fail builds for projects that are deleted when they are stuck

## What does this MR do?

Solves when dropping stuck connection.
```
Dropping stuck pending build 1545510 for runner 
NoMethodError: undefined method `origin_merge_requests' for nil:NilClass
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/base_service.rb:50:in `merge_request_from'
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/base_service.rb:57:in `each_merge_request'
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/add_todo_when_build_fails_service.rb:5:in `execute'
    from /opt/gitlab/embedded/service/gitlab-rails/lib/gitlab/metrics/instrumentation.rb:164:in `execute'
    from /opt/gitlab/embedded/service/gitlab-rails/app/models/commit_status.rb:51:in `block (2 levels) in <class:CommitStatus>'
    from /opt/gitlab/embedded/service/gem/ruby/2.1.0/gems/state_machines-0.4.0/lib/state_machines/eval_helpers.rb:79:in `call'
...
```

This happens, because that `default_scope` of `Projects` filters returns the projects that are not deleted, where `Ci::Build` doesn't take into account.


See merge request !4609





Don't fail builds for projects that are deleted when they are stuck

## What does this MR do?

Solves when dropping stuck connection.
```
Dropping stuck pending build 1545510 for runner 
NoMethodError: undefined method `origin_merge_requests' for nil:NilClass
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/base_service.rb:50:in `merge_request_from'
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/base_service.rb:57:in `each_merge_request'
    from /opt/gitlab/embedded/service/gitlab-rails/app/services/merge_requests/add_todo_when_build_fails_service.rb:5:in `execute'
    from /opt/gitlab/embedded/service/gitlab-rails/lib/gitlab/metrics/instrumentation.rb:164:in `execute'
    from /opt/gitlab/embedded/service/gitlab-rails/app/models/commit_status.rb:51:in `block (2 levels) in <class:CommitStatus>'
    from /opt/gitlab/embedded/service/gem/ruby/2.1.0/gems/state_machines-0.4.0/lib/state_machines/eval_helpers.rb:79:in `call'
...
```

This happens, because that `default_scope` of `Projects` filters returns the projects that are not deleted, where `Ci::Build` doesn't take into account.


See merge request !4609