require tenant to be set on models

app/models/ext_management_system.rb

@@ -57,6 +57,7 @@ def self.supported_types_and_descriptions_hash
             :presence   => true,
             :uniqueness => {:scope => [:tenant_id], :case_sensitive => false},
             :if         => :hostname_required?
+  validates :tenant_id, :presence => true
 
   # TODO: Remove all callers of address
   alias_attribute :address, :hostname

app/models/miq_group.rb

@@ -18,7 +18,7 @@ class MiqGroup < ActiveRecord::Base
 
   delegate :self_service?, :limited_self_service?, :to => :miq_user_role, :allow_nil => true
 
-  validates_presence_of   :description, :guid
+  validates_presence_of   :description, :guid, :tenant_id
   validates_uniqueness_of :description, :guid
 
   before_destroy do |g|

app/models/miq_request.rb

@@ -21,6 +21,7 @@ class MiqRequest < ActiveRecord::Base
 
   validates_inclusion_of :approval_state, :in => %w(pending_approval approved denied), :message => "should be 'pending_approval', 'approved' or 'denied'"
   validates_inclusion_of :status,         :in => %w(Ok Warn Error Timeout Denied)
+  validates :tenant_id, :presence => true
 
   validate :validate_class, :validate_request_type
 

app/models/miq_request_task.rb

@@ -19,6 +19,7 @@ class MiqRequestTask < ActiveRecord::Base
   delegate :request_class, :task_description, :to => :class
 
   validates_inclusion_of :status, :in => %w( Ok Warn Error Timeout )
+  validates :tenant_id, :presence => true
 
   include MiqRequestMixin
   include TenancyMixin

app/models/provider.rb

@@ -9,6 +9,7 @@ class Provider < ActiveRecord::Base
   belongs_to :tenant
   belongs_to :zone
   has_many :managers, :class_name => "ExtManagementSystem"
+  validates :tenant_id, :presence => true
 
   default_value_for :verify_ssl, OpenSSL::SSL::VERIFY_PEER
   validates :verify_ssl, :inclusion => {:in => [OpenSSL::SSL::VERIFY_NONE, OpenSSL::SSL::VERIFY_PEER]}

app/models/service.rb

@@ -39,6 +39,7 @@ class Service < ActiveRecord::Base
   virtual_column :has_parent,                               :type => :boolean
 
   validates_presence_of :name
+  validates :tenant_id, :presence => true
 
   def add_resource(rsc, options = {})
     raise MiqException::Error, "Vm <#{rsc.name}> is already connected to a service." if rsc.kind_of?(Vm) && !rsc.service.nil?

app/models/service_template.rb

@@ -23,6 +23,8 @@ class ServiceTemplate < ActiveRecord::Base
 
   has_many   :dialogs, -> { uniq }, :through => :resource_actions
 
+  validates :tenant_id, :presence => true
+
   virtual_has_many :custom_buttons
   virtual_column   :type_display,                 :type => :string
   virtual_column   :template_valid,               :type => :boolean

app/models/service_template_catalog.rb

@@ -1,8 +1,8 @@
 class ServiceTemplateCatalog < ActiveRecord::Base
   include ReportableMixin
   include TenancyMixin
-  validates_presence_of     :name
-  validates :name, :uniqueness => {:scope => :tenant_id}
+  validates :name, :uniqueness => {:scope => :tenant_id}, :presence => true
+  validates :tenant_id, :presence => true
 
   belongs_to :tenant
   has_many  :service_templates, :dependent => :nullify

app/models/tenant_quota.rb

@@ -35,6 +35,7 @@ class TenantQuota < ActiveRecord::Base
   validates :unit, :value, :presence => true
   validates :value, :numericality => {:greater_than => 0}
   validates :warn_value, :numericality => {:greater_than => 0}, :if => "warn_value.present?"
+  validates :tenant_id, :presence => true
 
   scope :cpu_allocated,       -> { where(:name => :cpu_allocated) }
   scope :mem_allocated,       -> { where(:name => :mem_allocated) }

app/models/vm_or_template.rb

@@ -179,6 +179,7 @@ class VmOrTemplate < ActiveRecord::Base
   virtual_has_one   :service,              :class_name => 'Service'
 
   before_validation :set_tenant_from_group
+  validates :tenant_id, :presence => true
 
   alias_method :datastores, :storages    # Used by web-services to return datastores as the property name
 

lib/miq_automation_engine/models/miq_ae_namespace.rb

@@ -7,7 +7,7 @@ class MiqAeNamespace < ActiveRecord::Base
   has_many   :ae_namespaces, :class_name => "MiqAeNamespace",  :foreign_key => :parent_id,    :dependent => :destroy
   has_many   :ae_classes, -> { includes([:ae_methods, :ae_fields, :ae_instances]) },    :class_name => "MiqAeClass",      :foreign_key => :namespace_id, :dependent => :destroy
 
-  validates_presence_of   :name
+  validates_presence_of   :name, :tenant_id
   validates_format_of     :name, :with => /\A[A-Za-z0-9_\.\-\$]+\z/i
   validates_uniqueness_of :name, :scope => :parent_id