Use - as the prefix so we don't conflict with namespaces

The decision was made around:
https://gitlab.com/gitlab-org/gitlab-ce/issues/26407#note_30624641

config/routes/project.rb

@@ -181,7 +181,7 @@ constraints(ProjectUrlConstrainer.new) do
         end
       end
 
-      resources :jobs, only: [:index, :show], constraints: { id: /\d+/ } do
+      resources :jobs, path: '-/jobs', only: [:index, :show], constraints: { id: /\d+/ } do
         collection do
           post :cancel_all
 

db/post_migrate/20170517101044_rename_projects_named_jobs.rb

@@ -1,14 +0,0 @@
-class RenameProjectsNamedJobs < ActiveRecord::Migration
-  include Gitlab::Database::RenameReservedPathsMigration::V1
-
-  DOWNTIME = false
-
-  disable_ddl_transaction!
-
-  def up
-    rename_wildcard_paths('jobs')
-  end
-
-  def down
-  end
-end

lib/gitlab/path_regex.rb

@@ -80,6 +80,7 @@ module Gitlab
     # By rejecting `badges` the router can _count_ on the fact that `badges` will
     # be preceded by the `namespace/project`.
     PROJECT_WILDCARD_ROUTES = %w[
+      -
       badges
       blame
       blob

spec/controllers/projects/environments_controller_spec.rb

@@ -177,7 +177,7 @@ describe Projects::EnvironmentsController do
         expect(response).to have_http_status(200)
         expect(json_response).to eq(
           { 'redirect_url' =>
-              "http://test.host/#{project.path_with_namespace}/jobs/#{action.id}" })
+              namespace_project_job_url(project.namespace, project, action) })
       end
     end
 
@@ -191,7 +191,7 @@ describe Projects::EnvironmentsController do
         expect(response).to have_http_status(200)
         expect(json_response).to eq(
           { 'redirect_url' =>
-              "http://test.host/#{project.path_with_namespace}/environments/#{environment.id}" })
+              namespace_project_environment_url(project.namespace, project, environment) })
       end
     end
   end

spec/features/projects/artifacts/browse_spec.rb

@@ -15,7 +15,7 @@ feature 'Browse artifact', :js, feature: true do
     end
 
     before do
-      visit browse_url.sub('jobs', 'builds')
+      visit browse_url.sub('/-/jobs', '/builds')
     end
 
     it "redirects to new URL" do

spec/features/projects/artifacts/download_spec.rb

@@ -39,7 +39,7 @@ feature 'Download artifact', :js, feature: true do
 
   context 'when visiting old URL' do
     before do
-      visit download_url.sub('jobs', 'builds')
+      visit download_url.sub('/-/jobs', '/builds')
     end
 
     context 'via job id' do

spec/features/projects/artifacts/file_spec.rb

@@ -67,7 +67,7 @@ feature 'Artifact file', :js, feature: true do
     end
 
     before do
-      visit file_url.sub('jobs', 'builds')
+      visit file_url.sub('/-/jobs', '/builds')
     end
 
     it "redirects to new URL" do

spec/features/projects/artifacts/raw_spec.rb

@@ -15,7 +15,7 @@ feature 'Raw artifact', :js, feature: true do
     end
 
     before do
-      visit raw_url.sub('jobs', 'builds')
+      visit raw_url.sub('/-/jobs', '/builds')
     end
 
     it "redirects to new URL" do

spec/features/projects/jobs_spec.rb

@@ -85,7 +85,7 @@ feature 'Jobs', :feature do
       end
 
       before do
-        visit jobs_url.sub('jobs', 'builds')
+        visit jobs_url.sub('/-/jobs', '/builds')
       end
 
       it "redirects to new URL" do
@@ -203,7 +203,7 @@ feature 'Jobs', :feature do
       end
 
       before do
-        visit job_url.sub('jobs', 'builds')
+        visit job_url.sub('/-/jobs', '/builds')
       end
 
       it "redirects to new URL" do
@@ -473,7 +473,7 @@ feature 'Jobs', :feature do
       end
 
       before do
-        visit raw_job_url.sub('jobs', 'builds')
+        visit raw_job_url.sub('/-/jobs', '/builds')
       end
 
       it "redirects to new URL" do

spec/javascripts/build_spec.js

@@ -8,7 +8,7 @@ import '~/breakpoints';
 import 'vendor/jquery.nicescroll';
 
 describe('Build', () => {
-  const BUILD_URL = `${gl.TEST_HOST}/frontend-fixtures/builds-project/jobs/1`;
+  const BUILD_URL = `${gl.TEST_HOST}/frontend-fixtures/builds-project/-/jobs/1`;
 
   preloadFixtures('builds/build-with-artifacts.html.raw');
 

spec/javascripts/lib/utils/common_utils_spec.js

@@ -356,7 +356,7 @@ import '~/lib/utils/common_utils';
 
     describe('gl.utils.setCiStatusFavicon', () => {
       it('should set page favicon to CI status favicon based on provided status', () => {
-        const BUILD_URL = `${gl.TEST_HOST}/frontend-fixtures/builds-project/jobs/1/status.json`;
+        const BUILD_URL = `${gl.TEST_HOST}/frontend-fixtures/builds-project/-/jobs/1/status.json`;
         const FAVICON_PATH = '//icon_status_success';
         const spySetFavicon = spyOn(gl.utils, 'setFavicon').and.stub();
         const spyResetFavicon = spyOn(gl.utils, 'resetFavicon').and.stub();