- 17 Apr, 2019 32 commits
-
-
Douwe Maan authored
Bring back Rugged implementation of ListCommitsByOid See merge request gitlab-org/gitlab-ce!27441
-
Sean McGivern authored
Add backtrace to Gitaly performance bar Closes #60092 See merge request gitlab-org/gitlab-ce!27345
-
James Lopez authored
Include link to raw job log in plain-text emails Closes #56849 See merge request gitlab-org/gitlab-ce!27409
-
Stan Hu authored
This brings back changes in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20432. For users using Gitaly on top of NFS, accessing the Git data directly via Rugged may be faster than going through than Gitaly. This merge request introduces the feature flag `rugged_list_commits_by_oid` to activate the Rugged method. For one customer, we saw that ListCommitsByOid was the second highest used endpoint that may be causing increased load.
-
Sean McGivern authored
Revert "Merge branch 'boards-stylelint' into 'master'" Closes gitlab-ee#11200 See merge request gitlab-org/gitlab-ce!27452
-
Stan Hu authored
This adds the backtrace to a table to show exactly where the Gitaly call was made to make it easier to understand where the call originated. This change also collapses the details in the same row to improve the usability when there is a backtrace.
-
Toon Claes authored
People who prefer plain text email also might prefer the raw job log files. So include the link directly to there. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/56849
-
Lin Jen-Shin authored
Don't fail the job if the Review App doesn't deploy properly See merge request gitlab-org/gitlab-ce!27223
-
Rémy Coutable authored
This reverts merge request !27313
-
Rémy Coutable authored
Restore "download repository path" feature, now with fixes and a feature flag Closes gitlab-workhorse#218 and #24704 See merge request gitlab-org/gitlab-ce!27275
-
Nick Thomas authored
Rewrite "Review turnaround time" section in light of Global Optimization value See merge request gitlab-org/gitlab-ce!27411
-
Sanad Liaquat authored
Resolve "Add QA tests for the performance bar" Closes #60356 See merge request gitlab-org/gitlab-ce!27305
-
Sean McGivern authored
Fix extra emails for custom notifications Closes #56861 See merge request gitlab-org/gitlab-ce!25607
-
Patrick Derichs authored
Using custom_action and recipient filtering Add more generic filtering to user_ids_notifiable_on Add changelog entry Remove commented class Method is no longer needed Overloading no longer required Filter by action just in case of custom notification level Add participant check Fix unexpected extra notification mails Using custom_action and recipient filtering Add more generic filtering to user_ids_notifiable_on Add changelog entry Remove commented class Method is no longer needed Overloading no longer required Filter by action just in case of custom notification level Fix comment Add repond_to? checks Reverted custom_action filtering Enhanced output of should_email helper Changed :watch to :participating for custom notifiable users Change spec variable name Enhanced participating check These conditions are no longer needed Fix custom notification handling for participating type Participating level should include maintainers Fixed add_guest notification Fix successful pipeline notification Refactoring: Use maintainer? method on team instead Add spec for new_issue: true for a custom group setting which should have lower prio than an available project setting Clean up specs
-
Achilleas Pipinellis authored
Add note about MR Pipeline Runner version Closes gitlab-ee#11122 See merge request gitlab-org/gitlab-ce!27330
-
Achilleas Pipinellis authored
Docs: Fix typo on DB migration See merge request gitlab-org/gitlab-ce!27435
-
Achilleas Pipinellis authored
Update external prometheus config See merge request gitlab-org/gitlab-ce!27439
-
Filipa Lacerda authored
Removes EE diffs for dropdown_value_collapsed.vue Closes gitlab-ee#10083 See merge request gitlab-org/gitlab-ce!27367
-
Filipa Lacerda authored
Backport of "Resolves EE differences for project_new.js" See merge request gitlab-org/gitlab-ce!27285
-
Donald Cook authored
-
Achilleas Pipinellis authored
Fix typo See merge request gitlab-org/gitlab-ce!27443
-
Ken McKnight authored
-
Filipa Lacerda authored
Fix stylelint warnings in related_items_list.scss See merge request gitlab-org/gitlab-ce!27432
-
Filipa Lacerda authored
Fix stylelint rules in boards.scss Closes #59895 See merge request gitlab-org/gitlab-ce!27313
-
Rémy Coutable authored
It seems the deploy function causes the job to fail if it doesn't succeed. That wasn't the intent as we want to curl the Review App after the deploy finished (even if it failed) because sometimes the Review App is just a bit long to be ready. This change wraps the Review App deployment with "set +e"/"set -e" to ensure that the job doesn't fail right away if the deploy fails. Signed-off-by: Rémy Coutable <remy@rymai.me>
-
Phil Hughes authored
Removes ifEE example for JS code. See merge request gitlab-org/gitlab-ce!27408
-
Phil Hughes authored
Add help texts to K8 form fields Closes #59275 See merge request gitlab-org/gitlab-ce!27274
-
Enrique Alcántara authored
-
Lin Jen-Shin authored
Fix logic in Api::Internal test See merge request gitlab-org/gitlab-ce!27193
-
Steve Azzopardi authored
-
Grzegorz Bizon authored
Allow extra args for helm in Auto-DevOps See merge request gitlab-org/gitlab-ce!26838
-
Michael Kozono authored
[CE backport] Fix no assignee text in sidebar_spec See merge request gitlab-org/gitlab-ce!27444
-
- 16 Apr, 2019 8 commits
-
-
Luke Bennett authored
-
Luke Duncalfe authored
The intention of this test is to ensure that the service class MergeRequests::PushOptionsHandlerService does not run when the :mr_push_options feature flag is disabled. This test was passing, however was not testing what it was supposed to be! For one, setting Feature.disable(:feature) in the test does not disable the feature, as rspec config in spec_helper stubs Feature to make all features enabled: https://gitlab.com/gitlab-org/gitlab-ce/commit/3ee48e422defaedd69946c607bd8d3672e510375 So the feature was still enabled in the test. But this test wasn't failing because unfortunately I had put: ``` expect(MergeRequests::PushOptionsHandlerService).to receive(:new) ``` instead of not_to! This meant that the `.new` method was being stubbed, so the service class did not create a MergeRequest, which satisfied the second expectation. ``` expect(MergeRequests::PushOptionsHandlerService).to receive(:new) ```
-
Cindy Pallares authored
Updated to include missing metrics: * GitLab Workhorse * Nginx * Sidekiq
-
Stan Hu authored
Set correct timestamps when creating past issues Closes #58361 See merge request gitlab-org/gitlab-ce!27406
-
James Fargher authored
-
Annabel Dunstone Gray authored
`None` for assignees Closes #60599 See merge request gitlab-org/gitlab-ce!27419
-
Victor Wu authored
Update assignees_spec.js Update sidebar_spec.rb Update issues_spec.rb
-
Douglas Barbosa Alexandre authored
Test using Git 2.21 See merge request gitlab-org/gitlab-ce!27418
-