| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| | |
Unify spec/support/matchers/match_ids.rb
See merge request gitlab-org/gitlab-ce!20108
|
| | |
|
|/ |
|
|\
| |
| |
| |
| | |
[Rails5] Fix "'-1' is not a valid data_store"
See merge request gitlab-org/gitlab-ce!19917
|
| |
| |
| |
| | |
Part of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/19882
|
|\ \
| | |
| | |
| | |
| | | |
Disallow methods that copy data on large tables
See merge request gitlab-org/gitlab-ce!20021
|
| | |
| | |
| | |
| | |
| | |
| | | |
{change_column_type,rename_column}_concurrently both copy data from one column
to another during a migration, which should not be done on GitLab.com. Instead,
we should use background migrations.
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Fix ruby sampler metrics
See merge request gitlab-org/gitlab-ce!20039
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
* Use a simple counter for sampler duration instead of a histogram.
* Use a counter to collect GC time.
* Remove unused objects metric.
* Cleanup metric names to match Prometheus conventions.
* Prefix generic GC stats with `gc_stat`.
* Include worker label on memory and file descriptor metrics.
|
|\ \ \ \
| |/ / /
|/| | |
| | | |
| | | | |
Unify spec/features/issues/filtered_search/filter_issues_spec.rb
See merge request gitlab-org/gitlab-ce!20070
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Unify spec/support/redis/redis_shared_examples.rb
See merge request gitlab-org/gitlab-ce!20074
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Unify object_storage_spec.rb with EE
See merge request gitlab-org/gitlab-ce!20069
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
'master'
Unify spec/services/auth/container_registry_authentication_service_spec.rb
See merge request gitlab-org/gitlab-ce!20071
|
| | |_|/ / /
| |/| | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Use expect_next_instance_of to replace expect_any_instance_of
See merge request gitlab-org/gitlab-ce!20072
|
| | |_|/ / /
| |/| | | | |
|
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This will only be displayed if `X-GitLab-Show-Login-Captcha` is set as an HTTP
header.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Move boards modal EmptyState vue component
See merge request gitlab-org/gitlab-ce!20068
|
| | | | | | | |
|
| |/ / / / /
|/| | | | | |
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | |
| | | | | |
| | | | | | |
CE: Resolve "Extract EE specific files/lines for project_creator_spec"
See merge request gitlab-org/gitlab-ce!19850
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | | |
We need this because `expect_any_instance_of` doesn't work on
prepended models. Now we could use the same code between CE/EE
|
| |/ / /
|/| | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | | |
Add enabled prefix to feature keys to be consistent with other keys
See merge request gitlab-org/gitlab-ce!20043
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Properly detect label reference if followed by period or question mark
See merge request gitlab-org/gitlab-ce!19965
|
| | | | | | |
|
| | | | | | |
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Rails 5 fix Capybara::ElementNotFound: Unable to find visible css #modal-revert-commit
Closes #48140 et #48139
See merge request gitlab-org/gitlab-ce!20044
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
#modal-revert-commit and expected: "/bar" got: "/foo"
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Raw changes calculation is Gitaly only now
Closes gitaly#1138
See merge request gitlab-org/gitlab-ce!20057
|
| | |_|_|/ / /
| |/| | | | |
| | | | | | |
| | | | | | | |
Closes https://gitlab.com/gitlab-org/gitaly/issues/1138
|
|\ \ \ \ \ \ \
| |/ / / / / /
|/| | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
Add a cop for `FinderMethods`
Closes #42705
See merge request gitlab-org/gitlab-ce!19951
|
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
This notifies developers when calling `find(_by!)` chained on
`execute`. And suggests using the methods from `FinderMethods`. These
will perform the correct authorization checks on the resource when it
is found.
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fix N+1 queries in uploads check rake task
Closes #47924
See merge request gitlab-org/gitlab-ce!20012
|
| | |_|_|/ / /
| |/| | | | | |
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Fall back to the first sign-in tab if the local storage value is bad
See merge request gitlab-org/gitlab-ce!20050
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
If `current_signin_tab` is `#crowd`, then no tab will be active. Fix
this by using the first available tab.
Relates to https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/20049#note_82996324
|
|\ \ \ \ \ \ \ \
| |_|/ / / / / /
|/| | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
'master'
Notify only merge request unmergeable due to conflict
See merge request gitlab-org/gitlab-ce!19548
|
| | |_|/ / / /
| |/| | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | |
| | | | | | | |
There is still the edge case when 'no commits' changes to 'conflict'
would not trigger notification, which we ignore for now.
Calling can_be_merged? can cause exception (e.g. non-UTF8)
Ignore those by rescueing.
Remove unmergeable_reason as now only conflict is notified
Update spec
|
|\ \ \ \ \ \ \
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Make the stubbed methods for SAML authorize paths more specific
See merge request gitlab-org/gitlab-ce!20009
|
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | |
| | | | | | | | |
Globally stubbing any instance of `Object` seems like a very bad idea,
and it led to test failures here:
- https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/76025159
- https://gitlab.com/gitlab-org/gitlab-ee/-/jobs/76024972
|
|\ \ \ \ \ \ \ \
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Limit the action suffixes in transaction metrics
See merge request gitlab-org/gitlab-ce!20032
|
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
There seem to be a lot of cases where the suffix of an action (e.g.
".html") is set to bogus data, such as "*/*" or entire URLs. This can
increase cardinality of our metrics, and isn't very useful for
monitoring and filtering. To work around this, we enforce a whitelist
containing a few suffixes we actually care about. Suffixes not supported
will be grouped under the action without a suffix. This means that a
request to "FooController#bar.jpeg" will be assigned to
"FooController#bar".
|
|\ \ \ \ \ \ \ \ \
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | |
| | | | | | | | | | |
Remove remaining traces of the Allocations Gem
See merge request gitlab-org/gitlab-ce!20029
|
| |/ / / / / / / /
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | |
| | | | | | | | | |
In MR https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15834 we
removed use of the data produced by the Allocations Gem. However, we
never removed the code that just enables tracking of allocations. In
this commit we remove all remaining traces of this Gem.
|
|\ \ \ \ \ \ \ \ \
| |_|_|/ / / / / /
|/| | | | | | | |
| | | | | | | | |
| | | | | | | | | |
Use eslint to find unused directives
See merge request gitlab-org/gitlab-ce!19940
|