| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| |
|
|
| |
fix spec and added changelog
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Reduce instrumentation overhead
## What does this MR do?
This MR reduces the overhead of instrumented methods. See the commit message of 905f8d763ab1184dc0b1e4bf6f18d7981753a860 for more information.
## Are there points in the code the reviewer needs to double check?
Not that I can think of.
## Why was this MR needed?
The overhead of method call instrumentation was too great.
## Does this MR meet the acceptance criteria?
- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [x] ~~[Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)~~
- [x] ~~API support added~~
- Tests
- [x] Added for this feature/bug
- [ ] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
See merge request !5550
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This reduces the overhead of the method instrumentation code primarily
by reducing the number of method calls. There are also some other small
optimisations such as not casting timing values to Floats (there's no
particular need for this), using Symbols for method call metric names,
and reducing the number of Hash lookups for instrumented methods.
The exact impact depends on the code being executed. For example, for a
method that's only called once the difference won't be very noticeable.
However, for methods that are called many times the difference can be
more significant.
For example, the loading time of a large commit
(nrclark/dummy_project@81ebdea5df2fb42e59257cb3eaad671a5c53ca36)
was reduced from around 19 seconds to around 15 seconds using these
changes.
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Add commit stats
## What does this MR do?
When getting commit by SHA in api it returns commit stats
## Are there points in the code the reviewer needs to double check?
Not sure about test I added do we need separate test or we can just shove stats assertion in test above
## Why was this MR needed?
So api users/clients can get statistics on single commit
## What are the relevant issue numbers?
#20307
## Screenshots (if relevant)
## Does this MR meet the acceptance criteria?
- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added
- [x] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md)
- [x] API support added
- Tests
- [x] Added for this feature/bug
- [x] All builds are passing
- [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides)
- [x] Branch has no merge conflicts with `master` (if you do - rebase it please)
- [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits)
See merge request !5517
|
| | |/ |
|
| |\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| | |
Added update guide for 8.11
## What does this MR do?
I need to add notes about Elastic on EE side. To not create ugly conflicts in the future and to not duplicate work I created it here first.
I followed this guide https://gitlab.com/gitlab-org/release-tools/blob/master/doc/release-candidates.md#gitlab-ce
See merge request !5545
|
| |/ |
|
| |\
| |
| |
| |
| |
| |
| | |
Replace reject_blocked with reject_blocked! in skipped callbacks.
`reject_blocked!` was introduced in c9def945, I can't find any references to `reject_blocked` ever existing.
See merge request !5532
|
| |/
|
|
|
|
|
|
|
|
| |
In Rails 4.2 and below, skipping callbacks (skip_before_action, skip_after_action, etc.) that use methods which do not exist will not throw any errors.
On the other hand, Rails 5 does. See https://github.com/rails/rails/pull/19029
After testing with Rails 5 I noticed there are some methods that don't actually exist (because they were renamed, usually), this fixes a few instances of those.
reject_blocked! was introduced in c9def945d4222eeb8026a0311495259bf99267a1, I can't find any references to reject_blocked ever existing.
|
| |\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Submit new issues created via the WebUI or API to Akismet for spam check on public projects.
## What does this MR do?
Submit new issues created via the WebUI by non project members to Akismet for spam check.
## Why was this MR needed?
Support for Akismet was added only to the API with !2266. This MR builds on that functionality to also check issues submitted via the WebUI for spam.
## What are the relevant issue numbers?
Related to:
- #5573
- #5932
- gitlab-com/infrastructure#14
- gitlab-com/support#61
- !2266
cc @stanhu @MrChrisW
See merge request !5333
|
| | |
| |
| |
| | |
legibility in `SpamCheckService`
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
for spam check.
|
| |\ \ |
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Update attr_encrypted from 3.0.1 to 3.0.2
This removes methods that were deprecated by Rails 5 so we won't have as many deprecation warnings when we update to Rails 5. Working toward #14286.
Changelog: https://github.com/attr-encrypted/attr_encrypted/blob/master/CHANGELOG.md#302
See merge request !5438
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This removes methods that were deprecated by Rails 5.
Changelog: https://github.com/attr-encrypted/attr_encrypted/blob/master/CHANGELOG.md#303
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Update rack-oauth2 from 1.2.1 to 1.2.3.
Removes methods deprecated in Rails 5. Working toward #14286.
Changelog: https://github.com/nov/rack-oauth2/compare/v1.2.1...v1.2.3
See merge request !5439
|
| | |/ / /
| | | |
| | | |
| | | |
| | | |
| | | | |
Removes methods deprecated in Rails 5.
https://github.com/nov/rack-oauth2/compare/v1.2.1...v1.2.3
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Upgrade Bullet from 5.0.0 to 5.2.0
Removes methods that were deprecated in Rails 5. Working toward #14286.
Changelog: https://github.com/flyerhzm/bullet/blob/master/CHANGELOG.md#520-07262016
See merge request !5443
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Removes methods that were deprecated in Rails 5.
Changelog: https://github.com/flyerhzm/bullet/blob/master/CHANGELOG.md#520
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Upgrade gon from 6.0.1 to 6.1.0.
Removes methods that were deprecated in Rails 5. Working toward #14286.
Changelog: https://github.com/gazay/gon/compare/3a2b56d...9d89bc9
See merge request !5442
|
| | |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Removes methods that were deprecated in Rails 5.
Changelog: https://github.com/gazay/gon/compare/3a2b56d...9d89bc9
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Upgrade default_value_for from 3.0.1 to 3.0.2.
Includes support for Rails 5. Working toward #14286.
Changelog: https://github.com/FooBarWidget/default_value_for/compare/release-3.0.1...release-3.0.2
See merge request !5441
|
| | | |/ / /
| |/| | |
| | | | |
| | | | |
| | | | |
| | | | | |
Includes support for Rails 5.
Changelog: https://github.com/FooBarWidget/default_value_for/compare/release-3.0.1...release-3.0.2
|
| |\ \ \ \ \
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Fix renaming repository when name contains invalid chars under settings
Fixes #19538
See merge request !5490
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| |\ \ \ \ \ \
| |/ / / / /
|/| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Generate coverage report from whole test suite
Extends our CI plan to merge all coverage results and generate HTML report.
See merge request !5018
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
| | | | | | | |
|
