1. 02 Jun, 2016 11 commits
    • Jacob Vosmaer's avatar
      Merge branch 'master' into git-http-controller · 8299fc27
      Jacob Vosmaer authored
      Conflicts:
      	config/routes.rb
      8299fc27
    • Achilleas Pipinellis's avatar
      Merge branch 'minor_doc_updates' into 'master' · acfbeced
      Achilleas Pipinellis authored
      Minor doc updates
      
      ## What does this MR do?
      
      Adds to and clarifies some documentation. Specifically, when using `gdb` to debug you need to detach and exit when done. Next, this clarifies the type of file locking that we require.
      
      Thanks to @northrup for the details about mandatory vs. advisory file locking.
      
      Note: These were two really small changes so I combined in to one merge request. However, I left the commits separate for clarity.
      
      See merge request !4421
      acfbeced
    • Robert Speicher's avatar
      Merge branch 'cs-unnecessary-assets' into 'master' · 30524901
      Robert Speicher authored
      Load d3 only when necessary
      
      Now d3 is only loaded on the users' show page and the graphs pages, where it's necessary.
      
      Redo of !3993.
      
      This decreases page size from 4.6MB to 4.3MB for a given Activity page (before compression), or a savings of 300KB. This is essentially a proof of concept for removing larger dependencies such as the Ace Editor, Chart.js, and Raphael, which will likely provide another 1MB of savings on most pages (again, before compression).
      
      See merge request !4394
      30524901
    • Robert Speicher's avatar
      Merge branch 'cleanup-readme' into 'master' · 34861c5f
      Robert Speicher authored
      Remove Coveralls and Semaphore badges. [ci skip]
      
      Remove Coveralls and Semaphore badges from the README. They're outdated,
      unused, and not GitLab CI :)
      
      [ci skip]
      
      See merge request !4334
      34861c5f
    • Connor Shea's avatar
      Move JS graph specs to match assets tree. · 586d1a9b
      Connor Shea authored
      586d1a9b
    • Connor Shea's avatar
      Fix teaspoon · 81f46054
      Connor Shea authored
      81f46054
    • Connor Shea's avatar
      Fix indentation · 3eb683b6
      Connor Shea authored
      3eb683b6
    • Connor Shea's avatar
      Attempt to fix the user page. · 026b7abc
      Connor Shea authored
      026b7abc
    • Connor Shea's avatar
      0256e683
    • Robert Speicher's avatar
      Merge branch 'fix-dropdown-cutoff' into 'master' · 933a6e0d
      Robert Speicher authored
      Adds a `title` attribute to the branch dropdown for when the name has been cut off.
      
      Fixes #18056 
      
      See merge request !4395
      933a6e0d
    • Robert Speicher's avatar
      Merge branch 'upgrade-devise-two-factor' into 'master' · 7d33fba7
      Robert Speicher authored
      Upgrade devise, devise-two-factor, and attr_encrypted
      
      Devise 4 includes support for Rails 5, working towards #14286. devise-async doesn't support Devise 4.0 and in 4.1 the bug that was blocking using Devise's built-in ActiveJob integration was fixed. So devise-async is removed. devise-two-factor 3.0.0 is required for Devise 4 support.
      
      attr_encrypted and encryptor are optional but recommended upgrades for devise-two-factor 3.0.0. The mode and algorithm will need to be changed in order to update to attr_encrypted 4.x in the future.
      
      See merge request !4216
      7d33fba7
  2. 01 Jun, 2016 29 commits
    • Stan Hu's avatar
      Merge branch 'downcase-registry-repository' into 'master' · 3b53016f
      Stan Hu authored
      Use downcased path to container repository as this is expected path by Docker
      
      ## What does this MR do?
      
      Docker Engine requires path to be lowercase. This makes all container registry paths to be show and used downcased instead of mixed case.
      
      Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/17959
      
      
      See merge request !4420
      3b53016f
    • Jacob Schatz's avatar
      Merge branch '17187-hover-row' into 'master' · 07fb3815
      Jacob Schatz authored
      Implement new row hover style
      
      ## What does this MR do?
      Implement new row hover style
      
      ## What are the relevant issue numbers?
      Closes #17187 
      
      ## Screenshots (if relevant)
      ![Screen_Shot_2016-06-01_at_1.40.04_PM](/uploads/62db01f405db50531fb648e46dc9598a/Screen_Shot_2016-06-01_at_1.40.04_PM.png)   
      
      cc @dzaporozhets 
      
      See merge request !4422
      07fb3815
    • Kamil Trzcinski's avatar
      77cb8ec4
    • Kamil Trzcinski's avatar
    • Annabel Dunstone's avatar
      Implement new row hover style · d7c76d09
      Annabel Dunstone authored
      d7c76d09
    • Drew Blessing's avatar
      Explicitly mention advisory file locking · c1923aef
      Drew Blessing authored
      c1923aef
    • Drew Blessing's avatar
      Add gdb detach instructions · 91496a09
      Drew Blessing authored
      91496a09
    • Jacob Schatz's avatar
      Merge branch '17676-merge-request-widget-alignment' into 'master' · 0f96a6c2
      Jacob Schatz authored
      MR widget mobile styling
      
      ## What does this MR do?
      
      Corrects MR widget styling for mobile viewing.
      
      ## Are there points in the code the reviewer needs to double check?
      
      I have tested all MR states.
      
      ## Why was this MR needed?
      
      UX.
      
      ## What are the relevant issue numbers?
      
      Closes #17676.
      
      ## Screenshots (if relevant)
      
      ![Screen_Shot_2016-05-31_at_20.04.02](/uploads/b1bc67b1015b2e771ae1b1364b225618/Screen_Shot_2016-05-31_at_20.04.02.png)![Screen_Shot_2016-05-31_at_20.03.50](/uploads/86a552c0d27d6f53cdf1b94631b26449/Screen_Shot_2016-05-31_at_20.03.50.png)![Screen_Shot_2016-05-31_at_20.03.34](/uploads/a5b9e7b634e974095922df9194d493b5/Screen_Shot_2016-05-31_at_20.03.34.png)![Screen_Shot_2016-05-31_at_20.01.35](/uploads/2fc5d49e3c1f2ff2cb324e9527179066/Screen_Shot_2016-05-31_at_20.01.35.png)![Screen_Shot_2016-05-31_at_20.00.25](/uploads/23af913891ff941dc5231872b3c6f177/Screen_Shot_2016-05-31_at_20.00.25.png)![Screen_Shot_2016-05-31_at_19.59.53](/uploads/8abb72609409ba7ca02ad6b35787c15c/Screen_Shot_2016-05-31_at_19.59.53.png)![Screen_Shot_2016-05-31_at_19.58.31](/uploads/fabf1c6c135f5e736614e568fc4cfb53/Screen_Shot_2016-05-31_at_19.58.31.png)![Screen_Shot_2016-05-31_at_19.58.51](/uploads/967a578a34cb3a9dd3723c2f22ce89c4/Screen_Shot_2016-05-31_at_19.58.51.png)![Screen_Shot_2016-05-31_at_23.33.19](/uploads/841af7a90bb3fc3c74a2e51b64fc444f/Screen_Shot_2016-05-31_at_23.33.19.png)![Screen_Shot_2016-05-31_at_23.33.23](/uploads/183fb8407c271d9aa3fefea5d8201238/Screen_Shot_2016-05-31_at_23.33.23.png)
      
      See merge request !4386
      0f96a6c2
    • Stan Hu's avatar
      Merge branch 'fix-note-validation-failure' into 'master' · e9bb895b
      Stan Hu authored
      Fix note validation spec failure
      
      Fix failing build due to project of note not matching issue project.
      
      See merge request !4415
      e9bb895b
    • Kamil Trzcinski's avatar
    • Jacob Schatz's avatar
      Merge branch 'add-edge-support' into 'master' · 5a377d20
      Jacob Schatz authored
      Add Edge to supported browsers list.
      
      Add Microsoft's Edge browser to the list of supported browsers. We support the latest stable version.
      
      See merge request !4416
      5a377d20
    • Connor Shea's avatar
      22e7aab8
    • Robert Speicher's avatar
      Merge branch 'dont-advertise-ci-skip' into 'master' · 93e2761c
      Robert Speicher authored
      Don't advertise the use of "ci skip" in the CONTRIBUTING guide
      
      [ci skip]
      
      See merge request !4406
      93e2761c
    • Robert Speicher's avatar
      Merge branch 'style/enable-semicolon-rubocop-cop' into 'master' · 3d3c9c75
      Robert Speicher authored
      Enable Style/Semicolon rubocop style cop
      
      Don't use semicolons to terminate expressions.
      
      See #17478.
      
      See merge request !4351
      3d3c9c75
    • Robert Speicher's avatar
      Merge branch 'rubocop/enable-style-for-cop' into 'master' · 164fc34e
      Robert Speicher authored
      Enable Style/For rubocop cop
      
      > Do not use `for`, unless you know exactly why. Most of the time iterators should be used instead. `for` is implemented in terms of `each` (so you're adding a level of indirection), but with a twist - `for` doesn't introduce a new scope (unlike `each`) and variables defined in its block will be visible outside it.
      
      See #17478
      
      See merge request !4397
      164fc34e
    • Stan Hu's avatar
      Fix note validation spec failures · 30aa6420
      Stan Hu authored
      30aa6420
    • Dmitriy Zaporozhets's avatar
      Merge branch 'dz-one-ci-cd-tab' into 'master' · 4621531a
      Dmitriy Zaporozhets authored
      Merge Builds and Pipelines tab into one
      
      Based on https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4053#note_12064642. Main idea is to keep all CI-related pages under one tab. So we can keep top navigation clean. 
      
      First I tried `CI/CD` naming but it looked weird so I just used Pipelines as tab name. I did not change project settings naming. 
      
      cc @ayufan @markpundsack @jschatz1 
      
      See merge request !4340
      4621531a
    • Douwe Maan's avatar
      Merge branch 'separate-banzai-references' into 'master' · 2d084dd8
      Douwe Maan authored
      Separate reference gathering from rendering
      
      This is a required step to allow batch processing when gathering references. This in turn would allow grabbing (for example) all mentioned users of an issue/merge request using a single query.
      
      cc @rspeicher @DouweM 
      
      See merge request !3969
      2d084dd8
    • Yorick Peterse's avatar
      Refactor Participable · 580d2501
      Yorick Peterse authored
      There are several changes to this module:
      
      1. The use of an explicit stack in Participable#participants
      2. Proc behaviour has been changed
      3. Batch permissions checking
      
      == Explicit Stack
      
      Participable#participants no longer uses recursion to process "self" and
      all child objects, instead it uses an Array and processes objects in
      breadth-first order. This allows us to for example create a single
      Gitlab::ReferenceExtractor instance and pass this to any Procs. Re-using
      a ReferenceExtractor removes the need for running potentially many SQL
      queries every time a Proc is called on a new object.
      
      == Proc Behaviour Changed
      
      Previously a Proc in Participable was expected to return an Array of
      User instances. This has been changed and instead it's now expected that
      a Proc modifies the Gitlab::ReferenceExtractor passed to it. The return
      value of the Proc is ignored.
      
      == Permissions Checking
      
      The method Participable#participants uses
      Ability.users_that_can_read_project to check if the returned users have
      access to the project of "self" _without_ running multiple SQL queries
      for every user.
      580d2501
    • Douwe Maan's avatar
      Merge branch 'fix-cancelable-retryable' into 'master' · ef6fe42e
      Douwe Maan authored
      Fix cancelability and retriablity of pipeline with generic statuses
      
      Currently it's not possible to cancel or retry generic status since this is external thing to GitLab.
      
      This fixes shown actions of pipelines containing only these actions.
      
      
      See merge request !4380
      ef6fe42e
    • Rémy Coutable's avatar
      Merge branch 'fix/error-500-in-pipeline-when-fork' into 'master' · 4581e71c
      Rémy Coutable authored
      Use project that belongs to pipeline in view
      
      ## What does this MR do?
      
      This MR makes project in pipelines view match the one that pipeline has been created for.
      
      Closes #17943 
      
      See merge request !4376
      4581e71c
    • Rémy Coutable's avatar
      Don't advertise the use of "ci skip" in the CONTRIBUTING guide · ea80259b
      Rémy Coutable authored
      Now that we autoscale runner, we don't need to optimize for their usage
      anymore. Abusing of "ci skip", for instance for view changes can lead to
      broken specs, so let's just not use it!
      Signed-off-by: default avatarRémy Coutable <remy@rymai.me>
      ea80259b
    • Douwe Maan's avatar
    • Kamil Trzcinski's avatar
      Fix rubocop offenses · 57992f3d
      Kamil Trzcinski authored
      57992f3d
    • Douwe Maan's avatar
      Merge branch 'emails-on-push-use-sidekiq-mailers' into 'master' · 9d88a3a1
      Douwe Maan authored
      Make EmailsOnPushWorker use Sidekiq mailers queue
      
      A customer was having issues with EmailsOnPushWorker clogging the Sidekiq queues and merge requests not being handled quickly. While researching whether it would possible to spin up a separate Sidekiq task just to handle merge requests and other key functions, I found that this worker was using the default Sidekiq queue. Moving to the `mailers` queue makes it possible to de-prioritize this worker and give more weight to the others.
      
      See merge request !4390
      9d88a3a1
    • Douwe Maan's avatar
      Merge branch 'prefer-to_reference' into 'master' · 173d83ca
      Douwe Maan authored
      Use `Snippet#to_reference` directly
      
      ## What does this MR do?
      
      Instead of hard coding snippet reference, we could just use the existing method for that.
      
      ## Why was this MR needed?
      
      To address https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4338#note_12166325 from @DouweM
      
      /cc @jschatz1 
      
      See merge request !4379
      173d83ca
    • Douwe Maan's avatar
      Merge branch 'rs-remember-me-2fa' into 'master' · 3416bc53
      Douwe Maan authored
      Pass the "Remember me" value to the 2FA token form
      
      Prior, if a user had 2FA enabled and checked the "Remember me" field,
      the setting was ignored because the OTP input was on a new form and the
      value was never passed.
      
      Closes #18000
      
      See merge request !4369
      3416bc53
    • Douwe Maan's avatar
    • Douwe Maan's avatar
      Merge branch 'fix-404-labels-in-todos' into 'master' · c0f19cc9
      Douwe Maan authored
      Fix 404 page when viewing TODOs that contain milestones or labels in different projects
      
      A user viewing the TODOs page will see a 404 if there are mentioned milestones or labels in multiple different projects. This is likely a caching bug and only occurs
      when Markdown rendering occurs across multiple projects, which is why it's so tricky to reproduce. This is what I think is happening:
          
      1. LabelReferenceFilter#references_in encounters label ~X for ProjectA and finds the label in the DB as id = 1.
      2. LabelReferenceFilter.references_in yields [1, 'X', nil, ...]
      3. Since project_ref is nil, AbstractReferenceFilter#project_from_ref_cache caches nil => ProjectA.
      4. LabelReferenceFilter#references_in encounters label ~Y for ProjectB and finds the label in the DB as id = 2.
      5. LabelReferenceFilter.references_in yields [2, 'Y', nil, ...]
      6. AbstractReferenceFilter#project_from_ref_cache lookups nil and returns ProjectA. It was supposed to be ProjectB.
      7. A is the wrong project, so the label lookup fails.
         
      This MR expands the `project_ref` to the right value as soon as we have it to avoid this caching bug.
          
      Closes #17898
      
      
      See merge request !4312
      c0f19cc9