Skip to content
Snippets Groups Projects
Commit 72d4e3b4 authored by Krasimir Angelov's avatar Krasimir Angelov :two:
Browse files

Remove IgnorableColumns module for models that have no ignored columns

Ignore rules were removed with
!71161.
parent 3aa9b3d5
No related branches found
No related tags found
No related merge requests found
Pipeline #393229509 passed
Pipeline: GitLab

#393232132

    ......@@ -3,7 +3,6 @@
    module Ci
    class BuildNeed < Ci::ApplicationRecord
    include BulkInsertSafe
    include IgnorableColumns
    belongs_to :build, class_name: "Ci::Processable", foreign_key: :build_id, inverse_of: :needs
    ......@@ -13,12 +12,5 @@ class BuildNeed < Ci::ApplicationRecord
    scope :scoped_build, -> { where('ci_builds.id=ci_build_needs.build_id') }
    scope :artifacts, -> { where(artifacts: true) }
    # TODO: Remove once build_id_convert_to_bigint is not an "ignored" column anymore (see .ignore_columns above)
    # There is a database-side trigger to populate this column. This is unexpected in the context
    # of cloning an instance, e.g. when retrying the job. Hence we exclude the ignored column explicitly here.
    def attributes
    super.except('build_id_convert_to_bigint')
    end
    end
    end
    ......@@ -4,8 +4,6 @@ module Ci
    # The purpose of this class is to store Build related runner session.
    # Data will be removed after transitioning from running to any state.
    class BuildRunnerSession < Ci::ApplicationRecord
    include IgnorableColumns
    TERMINAL_SUBPROTOCOL = 'terminal.gitlab.com'
    DEFAULT_SERVICE_NAME = 'build'
    DEFAULT_PORT_NAME = 'default_port'
    ......
    ......@@ -7,7 +7,6 @@ class BuildTraceChunk < Ci::ApplicationRecord
    include ::Checksummable
    include ::Gitlab::ExclusiveLeaseHelpers
    include ::Gitlab::OptimisticLocking
    include IgnorableColumns
    belongs_to :build, class_name: "Ci::Build", foreign_key: :build_id
    ......
    ......@@ -6,7 +6,6 @@ class Stage < Ci::ApplicationRecord
    include Ci::HasStatus
    include Gitlab::OptimisticLocking
    include Presentable
    include IgnorableColumns
    enum status: Ci::HasStatus::STATUSES_ENUM
    ......
    ......@@ -8,7 +8,6 @@ class Deployment < ApplicationRecord
    include Importable
    include Gitlab::Utils::StrongMemoize
    include FastDestroyAll
    include IgnorableColumns
    StatusUpdateError = Class.new(StandardError)
    StatusSyncError = Class.new(StandardError)
    ......
    ......@@ -5,7 +5,6 @@ class JobArtifactDeletedEvent < ApplicationRecord
    include Geo::Model
    include Geo::Eventable
    include BulkInsertSafe
    include IgnorableColumns
    belongs_to :job_artifact, class_name: 'Ci::JobArtifact'
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Finish editing this message first!
    Please register or to comment