Remove access_level index from runner. Add protected on ci_pipelines. Add...

Remove access_level index from runner. Add protected on ci_pipelines. Add protected index on ci_builds.

app/services/ci/create_pipeline_service.rb

@@ -12,7 +12,8 @@ module Ci
         tag: tag?,
         trigger_requests: Array(trigger_request),
         user: current_user,
-        pipeline_schedule: schedule
+        pipeline_schedule: schedule,
+        protected: project.protected_for?(ref)
       )
 
       result = validate(current_user,

db/migrate/20170816143940_add_protected_to_ci_pipelines.rb

+class AddProtectedToCiPipelines < ActiveRecord::Migration
+  DOWNTIME = false
+
+  def change
+    add_column :ci_pipelines, :protected, :boolean
+  end
+end

db/migrate/20170816133939_add_index_on_ci_runners_access_level.rb → db/migrate/20170816153940_add_index_on_ci_builds_protected.rb

-class AddIndexOnCiRunnersAccessLevel < ActiveRecord::Migration
+class AddIndexOnCiBuildsProtected < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
   DOWNTIME = false
@@ -6,10 +6,10 @@ class AddIndexOnCiRunnersAccessLevel < ActiveRecord::Migration
   disable_ddl_transaction!
 
   def up
-    add_concurrent_index :ci_runners, :access_level
+    add_concurrent_index :ci_builds, :protected
   end
 
   def down
-    remove_concurrent_index :ci_runners, :access_level if index_exists?(:ci_runners, :access_level)
+    remove_concurrent_index :ci_builds, :protected if index_exists?(:ci_builds, :protected)
   end
 end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20170824162758) do
+ActiveRecord::Schema.define(version: 20170830125940) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -255,6 +255,7 @@ ActiveRecord::Schema.define(version: 20170824162758) do
   add_index "ci_builds", ["commit_id", "type", "name", "ref"], name: "index_ci_builds_on_commit_id_and_type_and_name_and_ref", using: :btree
   add_index "ci_builds", ["commit_id", "type", "ref"], name: "index_ci_builds_on_commit_id_and_type_and_ref", using: :btree
   add_index "ci_builds", ["project_id"], name: "index_ci_builds_on_project_id", using: :btree
+  add_index "ci_builds", ["protected"], name: "index_ci_builds_on_protected", using: :btree
   add_index "ci_builds", ["runner_id"], name: "index_ci_builds_on_runner_id", using: :btree
   add_index "ci_builds", ["stage_id"], name: "index_ci_builds_on_stage_id", using: :btree
   add_index "ci_builds", ["status", "type", "runner_id"], name: "index_ci_builds_on_status_and_type_and_runner_id", using: :btree
@@ -337,6 +338,7 @@ ActiveRecord::Schema.define(version: 20170824162758) do
     t.integer "auto_canceled_by_id"
     t.integer "pipeline_schedule_id"
     t.integer "source"
+    t.boolean "protected"
   end
 
   add_index "ci_pipelines", ["auto_canceled_by_id"], name: "index_ci_pipelines_on_auto_canceled_by_id", using: :btree
@@ -375,7 +377,6 @@ ActiveRecord::Schema.define(version: 20170824162758) do
     t.integer "access_level", default: 0, null: false
   end
 
-  add_index "ci_runners", ["access_level"], name: "index_ci_runners_on_access_level", using: :btree
   add_index "ci_runners", ["contacted_at"], name: "index_ci_runners_on_contacted_at", using: :btree
   add_index "ci_runners", ["is_shared"], name: "index_ci_runners_on_is_shared", using: :btree
   add_index "ci_runners", ["locked"], name: "index_ci_runners_on_locked", using: :btree

spec/services/ci/create_pipeline_service_spec.rb

@@ -398,6 +398,7 @@ describe Ci::CreatePipelineService do
         it 'creates a pipeline' do
           expect(execute_service).to be_persisted
           expect(Ci::Pipeline.count).to eq(1)
+          expect(Ci::Pipeline.last).to be_protected
         end
       end
 
@@ -473,6 +474,7 @@ describe Ci::CreatePipelineService do
           expect(execute_service(trigger_request: trigger_request))
             .to be_persisted
           expect(Ci::Pipeline.count).to eq(1)
+          expect(Ci::Pipeline.last).not_to be_protected
         end
       end
     end