summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Fix error on project nameGosia Ksionek2019-08-058-21/+28
| | | | Add project path to sql query to build proper path
* Add group level container repository endpointsSteve Abrams2019-08-054-14/+59
| | | | | | | API endpoints for requesting container repositories and container repositories with their tag information are enabled for users that want to specify the group containing the repository rather than the specific project.
* Merge branch 'sh-disable-redis-peek' into 'master'Douglas Barbosa Alexandre2019-08-051-0/+5
|\ | | | | | | | | Only track Redis calls if Peek is enabled See merge request gitlab-org/gitlab-ce!31438
| * Only track Redis calls if Peek is enabledsh-disable-redis-peekStan Hu2019-08-021-0/+5
| | | | | | | | | | | | | | | | In dev environments, Sidekiq was encountering the message: Circular dependency detected while autoloading constant Gitlab::Profiler This saves some overhead during normal usage.
* | Merge branch 'add-missing-timezone-legacy-artifacts' into 'master'Stan Hu2019-08-051-6/+12
|\ \ | | | | | | | | | | | | Add missing timezone when copying legacy artifacts (ci_builds) See merge request gitlab-org/gitlab-ce!31447
| * | Add missing timezone to legacy artifacts (ci_builds)David H. Wilkins2019-08-041-6/+12
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | - ci_builds.artifacts_expire_at are copied to ci_job_artifacts.expire_at with incorrect timestamps when the database timezone is NOT utc - ci_builds.artifacts_expire_at is `timestamp without time zone` and ci_job_artifacts.expire_at is `timestamp with time zone` on postgresql - Tests fail locally for `rspec ./spec/lib/gitlab/import_export/import_export_spec.rb` without this change
* | Merge branch 'georgekoltsov/55474-outbound-setting-system-hooks' into 'master'Rémy Coutable2019-08-055-4/+12
|\ \ | | | | | | | | | | | | Add outbound setting for system hooks See merge request gitlab-org/gitlab-ce!31177
| * | Add backwards compatibility for legacy settingGeorge Koltsov2019-08-022-0/+8
| | | | | | | | | | | | | | | `allow_local_requests_for_hooks_and_services` was renamed to `allow_local_requests_for_web_hooks_and_services`.
| * | Refactor SystemHookUrlValidator and specsGeorge Koltsov2019-08-021-1/+1
| | | | | | | | | | | | | | | | | | Simplify SystemHookUrlValidator to inherit from PublicUrlValidator Refactor specs to move out shared examples to be used in both system hooks and public url validators.
| * | Add outbound requests setting for system hooksGeorge Koltsov2019-08-022-3/+3
| | | | | | | | | | | | | | | | | | | | | This MR adds new application setting to network section `allow_local_requests_from_system_hooks`. Prior to this change system hooks were allowed to do local network requests by default and we are adding an ability for admins to control it.
* | | Merge branch 'GL-12757' into 'master'Sean McGivern2019-08-051-2/+2
|\ \ \ | | | | | | | | | | | | | | | | Increase clair scanner from v11 to v12 See merge request gitlab-org/gitlab-ce!30809
| * | | increase clair local scan versionmo2019-08-021-1/+1
| | | |
| * | | increase clair from v11 to v12mo2019-08-021-1/+1
| | |/ | |/|
* | | Set DOCKER_TLS_CERTDIR in CI job templatesCameron Boulton2019-08-051-0/+1
|/ /
* | Implement Helm ResetCommand for removing TillerDylan Griffith2019-08-022-11/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | Also creates specs Only allow Helm to be uninstalled if it's the only app - Remove Tiller leftovers after reser command - Fixes specs and offenses Adds changelog file Fix reset_command specs
* | Merge branch 'make-needs-strong-connection' into 'master'Sean McGivern2019-08-022-12/+38
|\ \ | | | | | | | | | | | | | | | | | | Make `needs:` to require a strong reference Closes #65512 See merge request gitlab-org/gitlab-ce!31419
| * | Make needs: to require previous jobsmake-needs-strong-connectionKamil Trzciński2019-08-022-12/+38
| |/ | | | | | | | | | | | | | | | | This changes `needs:` from weak reference to have a strong reference. This means that job will not be created unless all needs are present as part of a pipeline.
* | Merge branch '17276-breakage-in-displaying-svg-in-the-same-repository' into ↵Robert Speicher2019-08-021-1/+1
|\ \ | |/ |/| | | | | | | | | 'master' Resolve "Breakage in displaying SVG in the same repository" See merge request gitlab-org/gitlab-ce!31352
| * Fix inline rendering of SVGs from current repo17276-breakage-in-displaying-svg-in-the-same-repositoryHeinrich Lee Yu2019-08-021-1/+1
| | | | | | | | Changes generated URL to raw instead of blob
* | Prevent empty classes in ansi2html conversionLukas '+ alert('Eipi') + ' Eipert2019-08-021-10/+17
| | | | | | | | | | | | | | Currently we write out empty CSS classes (`class=""`) every time we create a new tag. This adds 9 unnecessary bytes per span element. In a recent trace, I have counted 11950 span elements. So we transported 105 unnecessary kilobytes!
* | Call `GC::Profiler.clear` only in one placeAleksei Lipniagov2019-08-022-28/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, both InfluxSampler and RubySampler were relying on the `GC::Profiler.total_time` data which is the sum over the list of captured GC events. Also, both samplers asynchronously called `GC::Profiler.clear` which led to incorrect metric data because each sampler has the wrong assumption it is the only object who calls `GC::Profiler.clear` and thus could rely on the gathered results between such calls. We should ensure that `GC::Profiler.total_time` is called only in one place making it possible to rely on accumulated data between such wipes. Also, we need to track the amount of profiler reports we lost.
* | Update regex for docs branches in ciMarcel Amirault2019-08-021-1/+1
|/ | | | Changes * to + to be more precise with acceptable branch names
* Merge branch '52123-issuable-actions-notesfinder-pderichs' into 'master'Stan Hu2019-08-012-2/+2
|\ | | | | | | | | | | | | Use NotesFinder to fetch notes on API and Controllers Closes #52123 See merge request gitlab-org/gitlab-ce!31300
| * Use NotesFinder in IssuableActions modulePatrick Derichs2019-08-012-2/+2
| | | | | | | | | | | | | | | | | | | | Remove project from NotesFinder constructor Add project parameter to specs Also look for methods in private scope Fix specs to match new NotesFinder constructor
* | Merge branch 'osw-avoid-errors-due-to-concurrent-calls' into 'master'Douwe Maan2019-08-011-2/+3
|\ \ | | | | | | | | | | | | Add exclusive lease to mergeability check process See merge request gitlab-org/gitlab-ce!31082
| * | Add exclusive lease to mergeability check processosw-avoid-errors-due-to-concurrent-callsOswaldo Ferreira2019-07-311-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | Concurrent calls to UserMergeToRef RPC updating a single ref can lead to an opaque fail that is being rescued at Gitaly. So this commit adds an exclusive lease to the mergeability check process with the key as the current MR ID.
* | | Add support for DAGKamil Trzciński2019-08-013-35/+67
| | | | | | | | | | | | | | | | | | This implements the support for `needs:` keyword as part of GitLab CI. That makes some of the jobs to be run out of order.
* | | Merge branch 'sh-backtrace-clean-sidekiq' into 'master'Sean McGivern2019-08-011-0/+3
|\ \ \ | | | | | | | | | | | | | | | | Add additional paths to clean from backtrace See merge request gitlab-org/gitlab-ce!31363
| * | | Add additional paths to clean from backtracesh-backtrace-clean-sidekiqStan Hu2019-07-311-0/+3
| | | | | | | | | | | | | | | | | | | | `Gitlab::Profiler` can also be used to profile Sidekiq jobs. Add some call traces that can be omitted from the backtrace.
* | | | Merge branch 'sh-fix-infite-loop-peek-bar' into 'master'Sean McGivern2019-08-011-2/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix SystemStackError when Peek bar is active with Rugged calls Closes #65404 See merge request gitlab-org/gitlab-ce!31357
| * | | | Fix SystemStackError when Peek bar is active with Rugged callssh-fix-infite-loop-peek-barStan Hu2019-07-311-2/+6
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Peek attempts to serialize results with `to_json`, which calls `ActiveSupport::JSON`. If an object is passed to `to_json` that contains instance variables, `ActiveSupport` will attempt to recursively traverse all variables. The problem is that we can get into an infinite loop if the instance references to an instance that references to something else that points back to the same instance. To avoid this mess, we just call `to_s` on the object. It appears only `Gitlab::Git::Repository` and `::Repository` are the culprits here. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65404
* | | | Merge branch '65363-add-feature-remove' into 'master'Stan Hu2019-08-011-0/+7
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | Add Feature.remove Closes #65363 See merge request gitlab-org/gitlab-ce!31315
| * | | Add new Feature.remove method65363-add-feature-removeAsh McKenzie2019-08-011-0/+7
| | |/ | |/|
* | | Merge branch 'add-project-scope-to-live-trace-feature-flag' into 'master'Lin Jen-Shin2019-07-311-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Add project scope to live trace feature flag See merge request gitlab-org/gitlab-ce!31325
| * | | Add project scope to live trace feature flagadd-project-scope-to-live-trace-feature-flagShinya Maeda2019-07-311-1/+1
| |/ / | | | | | | | | | Add project scope
* | | Merge branch 'fix-sidekiq-memory-killer-warn-message' into 'master'Stan Hu2019-07-311-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix sidekiq memory killer warning message Closes gitlab-com/gl-infra/infrastructure#7371 See merge request gitlab-org/gitlab-ce!31264
| * | | Fix sidekiq memory killer warning messagefix-sidekiq-memory-killer-warn-messageShinya Maeda2019-07-301-1/+1
| | | |
* | | | Adds Sidekiq scheduling latency structured logging fieldAndrew Newdigate2019-07-311-0/+6
| | | |
* | | | Default dependency job stage index to Infinity, and correctly report it as ↵drew2019-07-311-1/+3
| | | | | | | | | | | | | | | | undefined in prior stages
* | | | Allow knative do be uninstalled:João Cunha2019-07-312-12/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - After uninstalling the knative helm chart it's necessary to also remove some leftover resources to allow the cluster to be clean and knative to be reinstalleable. - Adds knative uninstall disclaimer - Uninstall ksvc before uninstalling knative Make list of Knative and Ingres resources explicit - To avoid deleting unwanted resources we are listing exact which resources will be deleted rather than simply deleting any resource that contains istio or knative words.
* | | | Merge branch '65317-adapt-tae-role-name-change' into 'master'Sean McGivern2019-07-311-1/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | Fix Danger finding a test reviewer Closes #65317 See merge request gitlab-org/gitlab-ce!31292
| * | | Update regular expression to extract stage nameLin Jen-Shin2019-07-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since now the role name can be: "<a href=\"/job-families/engineering/test-automation-engineer/\">Senior Test Automation Engineer</a>, Create:Source Code" We need to cope with </a> in the middle.
* | | | Add support page link in help menuDiego Louzán2019-07-301-1/+1
| |_|/ |/| | | | | | | | Creates a new item in help dropdown to show configured support page link
* | | Merge branch 'sh-fix-gitaly-access-control' into 'master'Douglas Barbosa Alexandre2019-07-301-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Fix exception handling in Gitaly autodetection Closes #65328 See merge request gitlab-org/gitlab-ce!31285
| * | Fix exception handling in Gitaly autodetectionStan Hu2019-07-301-1/+1
| | | | | | | | | | | | | | | | | | | | | In SELinux, the file cannot be written, and `Errno::EACCES`, not `Errno::ACCESS` is thrown. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65328
* | | Fix broken update_project_templates rake taskHordur Freyr Yngvason2019-07-302-20/+52
|/ / | | | | | | | | | | This rake task had been broken for a while. This fixes the breakages, adds a test to help avoid future breakages, and adds a few ergonomic improvements to the task itself.
* | Remove line profiler from performance barSean McGivern2019-07-302-125/+1
|/ | | | | | | | | 1. The output isn't great. It can be hard to find hotspots and, even when you do find them, to find why those are hotspots. 2. It uses some jQuery-specific frontend code which we can remove now that we don't have this any more. 3. It's only possible to profile the initial request, not any subsequent AJAX requests.
* Merge branch 'determinstic_dns_specs' into 'master'Douglas Barbosa Alexandre2019-07-301-1/+4
|\ | | | | | | | | Have deterministic DNS for specs See merge request gitlab-org/gitlab-ce!31086
| * Explicitly reject non http(s) schemesThong Kuah2019-07-291-1/+4
| | | | | | | | Rather than relying on NoMethodError deep inside faraday
* | Merge branch ↵Robert Speicher2019-07-291-0/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | '63547-add-system-notes-for-when-a-zoom-call-was-added-removed-from-an-issue' into 'master' Resolve "Add system notes for when a zoom call was added/removed from an issue" Closes #63547 See merge request gitlab-org/gitlab-ce!30857