summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Respect needs for artifacts65483-add-a-resend-confirmation-link-to-the-need-to-confirm-alertKamil Trzciński2019-08-022-12/+21
| | | | | | When `needs:` is defined, the value of it is not respected when returning a list of artifacts to the runner from the job.
* Prevent empty classes in ansi2html conversionLukas '+ alert('Eipi') + ' Eipert2019-08-027-44/+54
| | | | | | | 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!
* Use stable image version for dindLukas Eipert2019-08-022-2/+2
| | | | | | See https://gitlab.com/gitlab-com/gl-infra/production/issues/1023 This also has been suggested by Tomasz in the MR
* Syncs the vue test utils helpersSam Beckham2019-08-023-36/+20
|
* Fix performance bar warnings when Rugged is disabledStan Hu2019-08-021-1/+1
| | | | | | | | `detailed_metric.vue` was attempting to read the `duration` field even when the metric did not exist in the JSON payload. We now check for the existence of the metrics before attempting to display the value. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65491
* Improve test for .with_needsKamil Trzciński2019-08-021-2/+3
|
* Update e2e style guide with new guidelineWalmyr Lima2019-08-021-0/+44
|
* Update sections titles with right headingWalmyr Lima2019-08-021-2/+2
|
* Clean up job_app_specPaul Slaughter2019-08-021-396/+287
| | | | | - Remove setTimeout calls - Add helper function to encapsulate setup
* Call `GC::Profiler.clear` only in one placeAleksei Lipniagov2019-08-026-53/+44
| | | | | | | | | | | | | | | 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.
* Make `bin/web_puma` consider RAILS_ENVAleksei Lipniagov2019-08-022-1/+6
|
* Use Mermaid Markdown for testing guide docsStan Hu2019-08-024-28/+19
| | | | | | | * Remove PNGs in favor of dynamically rendered graphs * Eliminate <b> styling because this is no longer supported * Change <br /> to <br> because the latter is only supported * Add quotes around subgraph titles
* Custom project templates also on GitLab.comEvan Read2019-08-021-2/+2
|
* Resolve Incorrect empty state message on "Explore projects"Dennis Tang2019-08-025-4/+11
|
* Update ldap#security sectionGeorge Koltsov2019-08-021-7/+10
|
* Update regex for docs branches in ciMarcel Amirault2019-08-027-11/+12
| | | | Changes * to + to be more precise with acceptable branch names
* Document required quotes in Mermaid subgraph titlesStan Hu2019-08-021-0/+43
| | | | | | | | | | Mermaid v8.2.3 added support for styling in subgraphs, but if there are multiple words in a title, quotes are required. * https://github.com/knsv/mermaid/pull/845 * https://github.com/knsv/mermaid/issues/391#issuecomment-495602059 Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/65338
* Add rate limit docsMichael Kozono2019-08-028-12/+212
|
* Only render job sidebar block if it has contentMiranda Fluharty2019-08-023-5/+34
| | | | | | Change sidebar's renderBlock() to return true only if any of the block's <detail-row>s will be rendered Change block to render only if renderBlock() returns true
* Corrected name of Jira groupRussell Dickenson2019-08-021-2/+2
|
* Update global dashboard shortcutsNishad Sohoni2019-08-021-5/+5
| | | | Needs shift keypress
* Backport from EEjboyson12019-08-022-0/+4
|
* Make diff_refs_match_commit validation reusablePatrick Bajao2019-08-024-7/+64
| | | | | Move it to DiffPositionableNote concern which will be re-used in EE in DraftNote model.
* Add commit_id to draft_notes tablePatrick Bajao2019-08-023-1/+13
| | | | CE port of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/14520
* Quarantine folder containing only a folder specSean McGivern2019-08-021-1/+2
|
* Remove snippet breadcrumb specSean McGivern2019-08-021-19/+0
| | | | We removed the breadcrumbs in 0def36777e3f9e5f57fd0e1ebd023b43229622fc.
* Fix project nav specsSean McGivern2019-08-021-2/+2
|
* Remove mail_google_schema_whitelisting specSean McGivern2019-08-021-27/+0
| | | | | | This task was removed in 2014! The spec wasn't, probably because it didn't fail as it was never executed. See 0fc5c80207a1ebcfbdae1d0d16b9aeb984bac271.
* Fix milestone view specsSean McGivern2019-08-023-2/+3
|
* Fix labels issuable link specsSean McGivern2019-08-023-15/+40
|
* Rename broken spec filesSean McGivern2019-08-0215-0/+0
| | | | RSpec only executes files ending in _spec.rb.
* Add RSpec/TopLevelDescribePath copSean McGivern2019-08-023-0/+103
| | | | | | | | The RSpec/FilePath cop checks that a spec file's path is correct, but only if the file describes a constant. We want to check, for any file with a top-level `describe`, whether the file path ends in `_spec.rb`. We don't care what comes before that; just that it will be executed by RSpec at all.
* Add documentation for Security Approvals for MRsLucas Charles2019-08-022-0/+42
| | | | Documents how to enable security approvals
* Remove local echarts dependencyMartin Wortschack2019-08-022-9/+8
| | | | - Update package.json and yarn.lock
* Revert "Merge branch 'mh/editor-indents' into 'master'"Paul Slaughter2019-08-0216-1287/+39
| | | | | This reverts commit 28f2225bdfee4d353f07a766c6c8b29ba6181397, reversing changes made to 96ae5bd83da31350e9856a290127d7aa1469710a.
* Backport of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3809Valery Sizov2019-08-029-3/+57
| | | | Introducing Docker Registry replication
* Add can_merge option to autocomplete responseIgor2019-08-024-1/+62
| | | | It's used only if merge_request_iid param is provided
* Skip specs for default relative positionsBob Van Landuyt2019-08-021-1/+6
| | | | | If a model has a default relative position we don't need to test that it is moved to the end, it will instead get the default value assigned.
* Document the current state of the Review App clustersRémy Coutable2019-08-021-9/+22
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Change block parameter name to paramsPatrick Derichs2019-08-022-3/+3
|
* Use NotesFinder in IssuableActions modulePatrick Derichs2019-08-0215-78/+316
| | | | | | | | | | Remove project from NotesFinder constructor Add project parameter to specs Also look for methods in private scope Fix specs to match new NotesFinder constructor
* Hardcodes namespaces lease timing value to 1.5hMayra Cabrera2019-08-022-45/+3
| | | | | | | | | After measuring the response and performance of the workers on https://gitlab.com/gitlab-org/gitlab-ce/issues/64092, we've decided 1.5 is fine for now. The time can be decreased later in the future when enforcing the quota limits Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/64092
* Add exclusive lease to mergeability check processOswaldo Ferreira2019-08-027-27/+128
| | | | | | | | 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.
* Jupyter fix hostname editing and remove extra load balancerAmit Rathi2019-08-023-0/+10
|
* Add support for DAGKamil Trzciński2019-08-0224-74/+567
| | | | | | This implements the support for `needs:` keyword as part of GitLab CI. That makes some of the jobs to be run out of order.
* Ensure review-stop can run even when the branch has been deletedRémy Coutable2019-08-021-3/+7
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix bug when moving batches of items to the endHeinrich Lee Yu2019-08-023-10/+13
| | | | | | | Starts from START_POSITION when there are no existing positions. Also improves the test to actually test the behavior
* Fix push file size testMark Lapierre2019-08-021-6/+28
| | | | And remove it from quarantine
* Portray better use of types of variables in .gitlab-ci.ymlVlad Stoianovici2019-08-021-2/+5
|
* Implement review suggestionsMark Lapierre2019-08-021-4/+16
| | | | | | | Make package-and-qa manual so the default `on_success` doesn't need to be set on package-and-qa-always. Remove unnecessary `except: []`