webroar 0.6.1 → 0.7.0

data/src/admin_panel/app/{models → mailers}/mailer.rb

@@ -19,7 +19,7 @@
 #++
 
 unless defined?ActionMailer
-  gem 'actionmailer', '2.3.9'
+  gem 'actionmailer', '3.0.9'
   require 'action_mailer'
   require 'uri'
 end
@@ -68,10 +68,7 @@ class Mailer < ActionMailer::Base
 
   # This method is to deliver the email to recipients
   def send_email(subject,body,from,recipients)
-    @subject = subject
-    @from = from
-    @recipients = recipients
-    @body = body
+    mail(:to => recipients,:from => from,:subject => subject,:body => body)
   end
 
   # This method is used to send exception notification to the recipients
@@ -89,7 +86,7 @@ class Mailer < ActionMailer::Base
         body << "#{key.to_s.upcase} : #{value}\n"
       end
       begin
-        deliver_send_email(subject,body,from,recipients)
+        send_email(subject,body,from,recipients).deliver
       rescue Exception => e
         e
       end
@@ -137,7 +134,7 @@ class Mailer < ActionMailer::Base
     body = "#{report_bug[:description]}\n\n"
     body << "Name : #{report_bug[:name]}\nEmail Address : #{report_bug[:email]}\n"
     begin
-      deliver_send_email(subject,body,from,"support@webroar.in")
+      send_email(subject,body,from,"support@webroar.in").deliver
       "<span style='color:green;'>Thank You for reporting the issue, We'll contact you shortly.</span>"
     rescue Exception => exception
       return self.parse_exception(exception),''
@@ -152,7 +149,7 @@ class Mailer < ActionMailer::Base
     body = "#{feedback[:message]}\n\n"
     body << "User Name : #{feedback[:name]}\nEmail Address : #{feedback[:email]}\n"
     begin
-      deliver_send_email(subject,body,from,"support@webroar.in")
+      send_email(subject,body,from,"support@webroar.in").deliver
       "<span style='color:green;'>Thank you for your valuable feedback!</span>"
     rescue Exception => exception
       return self.parse_exception(exception),''

data/src/admin_panel/app/models/app.rb

@@ -32,7 +32,7 @@ class App < ActiveRecord::Base
     #This method is used to get the application data.
     #It needs name of the application as an input.
     def get_application_data(name)
-      app = find(:first, :conditions => ['name = ?', name])
+      app = where(:name => name).first 
       return app
     end
 

data/src/admin_panel/app/models/app_exception.rb

@@ -27,24 +27,25 @@ class AppException < ActiveRecord::Base
   class << self
     #Gives the array of the five (open|closed|ignored) exceptions starting from the value of varriable 'start' for an application.
     def get_all(exception_status, app_id, page = 1, per_page = 5)
-      paginate(:conditions => ['app_id = ?  and exception_status = ?', app_id, exception_status],
-#               :include => [:latest_detail], # found bug with eager loading, its fetching all the associated records              
-               :page => page, :per_page => per_page)
+      where(:app_id => app_id, 
+            :exception_status => exception_status
+            ).paginate(:page => page, 
+            :per_page => per_page)
     end
     
     # Take App.id as argument and returns count of distinct open exceptions for an Application
     def count_open(app_id)
-      count(:exception_message, :conditions => {:app_id => app_id, :exception_status => OPEN_EXCEPTION})
+      where(:app_id => app_id, :exception_status => OPEN_EXCEPTION).count
     end
     
     # Take App.id as argument and returns count of distinct closed exceptions for an Application
     def count_closed(app_id)
-      count(:exception_message, :conditions => {:app_id => app_id, :exception_status => CLOSED_EXCEPTION})
+      where(:app_id => app_id, :exception_status => CLOSED_EXCEPTION).count
     end
     
     # Take App.id as argument and returns count of distinct ignored exceptions for an Application
     def count_ignored(app_id)
-      count(:exception_message, :conditions => {:app_id => app_id, :exception_status => IGNORED_EXCEPTION})
+      where(:app_id => app_id, :exception_status => IGNORED_EXCEPTION).count
     end
     
     # Update all exceptions status for given exceptions id array
@@ -99,7 +100,7 @@ class AppException < ActiveRecord::Base
 
     # TODO Change function to fetch all exception records if we require all exception fields instead of id only  
     def get_exceptions_by_class(app_id,exception_class)
-      find(:all,:select => :id,:conditions=>["app_id = ? and exception_class = ?",app_id, exception_class])
+      where(:app_id => app_id, :exception_class => exception_class).select(:id)
     end
 
     # This method is used to change status of exceptions by searching the exception with exception_class
@@ -112,10 +113,13 @@ class AppException < ActiveRecord::Base
 
     # This function is used to fetch the first exception record for the given exception_message pattern ,
     # exception_class, exception_details.controller , exception_details.method and app_id
-    def get_exception_for_analyzer(exception_hash,app_id)
-      AppException.find(:first,:conditions => ["exception_message like ? and exception_class = ? and controller = ? and method = ? and app_id = ?" ,
-        exception_hash[:exception_message].split(EXCEPTION_MESSAGE_SPLITER)[0] + "%",exception_hash[:exception_class],
-        exception_hash[:controller],exception_hash[:method],app_id])
+    def get_exception_for_analyzer(exception_hash, app_id)
+      where("exception_message like ? and exception_class = ? and controller = ? and method = ? and app_id = ?",
+                          exception_hash[:exception_message].split(EXCEPTION_MESSAGE_SPLITER)[0] + '%',
+                          exception_hash[:exception_class],
+                          exception_hash[:controller],
+                          exception_hash[:method],
+                          app_id).first
     end
   end
 end

data/src/admin_panel/app/models/app_time_sample.rb

@@ -34,23 +34,43 @@ class AppTimeSample < ActiveRecord::Base
       interval = ((end_time-start_time) / 60).to_i
       final_data = Array.new(interval)
       wall_time = Array.new(interval)
-      application_samples = find(:all, :select => 'wall_time, sum(db_time) as db_time,sum(total_time_in_request) as total_time_in_request, sum(number_of_requests) as number_of_requests , avg(avg_response_time) as avg_avg_response_time, avg(peak_requests_served) as avg_peak_requests_served', :conditions => ['app_id = ? and wall_time >= ? and wall_time < ?', app_id, start_time, end_time], :group => 'wall_time')
-     application_samples.each do |application_sample| 
-          current_time = Time.local(application_sample.wall_time.year, application_sample.wall_time.month, application_sample.wall_time.day, application_sample.wall_time.hour, application_sample.wall_time.min, '0')
+      application_samples = select('wall_time, 
+                                    sum(db_time) as db_time,
+                                    sum(total_time_in_request) as total_time_in_request, 
+                                    sum(number_of_requests) as number_of_requests , 
+                                    avg(avg_response_time) as avg_avg_response_time, 
+                                    avg(peak_requests_served) as avg_peak_requests_served'
+                                    ).where('app_id = ? and 
+                                    wall_time >= ? and 
+                                    wall_time < ?', 
+                                    app_id,
+                                    start_time.strftime('%Y-%m-%d %H:%M:%S'),
+                                    end_time.strftime('%Y-%m-%d %H:%M:%S')
+                                    ).group(:wall_time)
+
+      application_samples.each do |application_sample| 
+         current_time = Time.local(application_sample.wall_time.year, 
+                                   application_sample.wall_time.month, 
+                                   application_sample.wall_time.day, 
+                                   application_sample.wall_time.hour, 
+                                   application_sample.wall_time.min, 
+                                   '0')
          db_time_data = 0
          total_time_in_request = 0
          total_requests = 0
          total_data = 0
          index = (current_time-start_time) / 60
          total_time_in_request = application_sample.total_time_in_request.to_f
-         if type == "db"
+         case type
+         when 'db'
             db_time_data = application_sample.db_time.to_f
             total_data = (db_time_data.to_f * 100 / total_time_in_request.to_f).to_f
-         elsif type == "averageresponsetime"
+         when 'averageresponsetime'
            total_data = application_sample.avg_avg_response_time.to_f
-         elsif type == "throughput"
+         when 'throughput'
             total_data = application_sample.avg_peak_requests_served.to_f
          end
+         
          if max < total_data
            max = total_data.to_i
          end
@@ -97,7 +117,15 @@ class AppTimeSample < ActiveRecord::Base
       end
       #      puts 'creating'
       with_exception_handling("Application sample creation for application #{app_id}, wall_time #{wall_time}") do
-        create({:app_id => app_id, :total_time_in_request => sample[0], :db_time => sample[1], :rendering_time => sample[2], :number_of_requests => sample[3], :wall_time => wall_time, :sampling_rate => sampling_rate, :avg_response_time => art, :peak_requests_served => prs})
+        create({:app_id => app_id, 
+                :total_time_in_request => sample[0], 
+                :db_time => sample[1], 
+                :rendering_time => sample[2], 
+                :number_of_requests => sample[3], 
+                :wall_time => wall_time, 
+                :sampling_rate => sampling_rate, 
+                :avg_response_time => art, 
+                :peak_requests_served => prs})
       end      
     end
     # application_samples contains at the most one sample and that is for current sampling period.
@@ -118,18 +146,20 @@ class AppTimeSample < ActiveRecord::Base
         sample[4][wall_time.sec] = (sample[4][wall_time.sec] || 0 ) + 1
         message_analyzer.application_samples[app_id][1] = sample
       elsif wall_time < message_analyzer.application_samples[app_id].first
-        db_sample = find(:first, :conditions => ["app_id = ? and wall_time >= ? and wall_time <= ?",app_id, wall_time, wall_time + message_analyzer.sampling_rate])
+        db_sample = where('app_id = ? and 
+                          wall_time >= ? and wall_time <= ?',
+                          app_id, 
+                          wall_time, wall_time + message_analyzer.sampling_rate
+                          ).first
         if db_sample
           db_sample.total_time_in_request += total_spent_time
           db_sample.db_time += db_time
           db_sample.rendering_time += rendering_time
           db_sample.number_of_requests += 1
-          #db_sample.avg_response_time += (total_spent_time / resource_analyzer.worker_count(app_id))
           db_sample.save! rescue nil
         else
           wall_time += message_analyzer.sampling_rate
           create_sample(app_id, [total_spent_time, db_time, rendering_time, 1, [1]], message_analyzer.sampling_rate, wall_time)
-          #create({:app_id => app_id, :total_time_in_request => total_spent_time, :db_time => db_time, :rendering_time => rendering_time, :number_of_requests => 1, :wall_time => wall_time, :sampling_rate => message_analyzer.sampling_rate, :avg_response_time => 1000, :peak_requests_served => 1 })
         end
       elsif wall_time > message_analyzer.application_samples[app_id].first +  message_analyzer.sampling_rate
         o_wall_time = message_analyzer.application_samples[app_id].first +  message_analyzer.sampling_rate

data/src/admin_panel/app/models/application_specification.rb

@@ -17,8 +17,7 @@
 # You should have received a copy of the GNU General Public License
 # along with WebROaR.  If not, see <http://www.gnu.org/licenses/>.
 #++
-
-require File.join(RAILS_ROOT, "lib", "helper.rb")
+require File.join(File.expand_path('../../../' ,__FILE__), 'lib', 'helper')
 
 class ApplicationSpecification < PseudoModel
   column :app_id,       :string
@@ -31,17 +30,39 @@ class ApplicationSpecification < PseudoModel
   column :type1,        :string
   column :analytics,    :string
   column :environment,  :string
-  column :min_worker, :number
-  column :max_worker, :number
-  validates_presence_of :name, :resolver, :path, :run_as_user, :type1, :environment
-  validates_numericality_of :min_worker, :max_worker, :greater_than_or_equal_to => 1, :less_than_or_equal_to => ALLOWED_MAX_WORKERS
-  validates_format_of :name, :with => /^[A-Za-z0-9_\-]*$/i, :message => " must consist of A-Z, a-z, 0-9 , _ , -  and /."
+  column :min_worker,   :number
+  column :max_worker,   :number
+  
+  validates :name,         :presence => true,
+                          :format => { :with => /^[A-Za-z0-9_\-]*$/i, :message => " must consist of A-Z, a-z, 0-9 , _ , -  and /."},
+                          :length => { :maximum => 30 },
+                          :allow_nil => true
+  validates :resolver,     :presence => true
+  validates :path,         :presence => true
+  validates :run_as_user,  :presence => true,
+                          :length => { :maximum => 30 },
+                          :allow_nil => true
+  validates :type1,        :presence => true
+  validates :environment,  :presence => true,
+                          :length => { :maximum => 30 },
+                          :allow_nil => true
+  validates :min_worker,   :numericality => true, 
+                          :inclusion => { :in => 0..ALLOWED_MAX_WORKERS }
+  validates :max_worker,   :numericality => true, 
+                          :inclusion => { :in => 0..ALLOWED_MAX_WORKERS }
+  
+  validate :ensure_validations
+  
+  #validates_presence_of :name, :resolver, :path, :run_as_user, :type1, :environment
+  #validates_numericality_of :min_worker, :max_worker, :greater_than_or_equal_to => 1, :less_than_or_equal_to => ALLOWED_MAX_WORKERS
+  #validates_format_of :name, :with => /^[A-Za-z0-9_\-]*$/i, :message => " must consist of A-Z, a-z, 0-9 , _ , -  and /."
   #  validates_format_of :path, :with => /^\/.*$/i, :message => "entered is not the complete path for your web application root directory."
-  validates_length_of :name, :maximum => 30, :allow_nil => true
+  #validates_length_of :name, :maximum => 30, :allow_nil => true
   #validates_length_of :path, :maximum => 256
-  validates_length_of :run_as_user, :maximum => 30, :allow_nil => true
-  validates_length_of :environment, :maximum => 30, :allow_nil => true
+  #validates_length_of :run_as_user, :maximum => 30, :allow_nil => true
+  #validates_length_of :environment, :maximum => 30, :allow_nil => true
   #  validates_format_of :baseuri, :with => /^\/[A-Za-z0-9_\-\/]*$/i, :message => "must start with '/' and contains characters A-Z, a-z, 0-9 , _ , -  and /."
+  
   def write
 
     info = YAML::load_file(CONFIG_FILE_PATH) rescue nil
@@ -69,7 +90,7 @@ class ApplicationSpecification < PseudoModel
       info.delete('Application Specification') if info['Application Specification'].length == 0
     end
 
-    app = App.find(:first, :conditions=>["name = ?", app_name])
+    app = App.where(:name => app_name).first
     app.destroy if app
     SignalHelper.send
 
@@ -82,9 +103,17 @@ class ApplicationSpecification < PseudoModel
 
   #Converting ApplicationSpecification obeject into a Hash.
   def obj_to_hash
-    app = Hash["name" => name, "path" => path, "run_as_user" => run_as_user.downcase, "type" => type1.downcase, "analytics" => analytics.to_s.downcase, "environment" => environment.downcase, "min_worker" => min_worker.to_i, "max_worker" => max_worker.to_i]
-    app["baseuri"] = baseuri.strip if baseuri
-    app["host_names"] = host_names if host_names
+    app = {}
+		app['name'] = name
+		app['path'] = path
+		app['run_as_user'] = run_as_user.downcase unless run_as_user.nil?
+		app['type'] = type1.downcase unless type1.nil?
+		app['analytics'] = analytics.downcase unless analytics.nil?
+		app['environment'] = environment.downcase unless environment.nil?
+		app['min_worker'] = min_worker.to_i
+		app['max_worker'] = max_worker.to_i
+		app['baseuri'] = baseuri.strip if baseuri
+		app['host_names'] = host_names if host_names
     return app
   end
 
@@ -99,26 +128,27 @@ class ApplicationSpecification < PseudoModel
   end
 
   #this method is used to validate the various fields of the apps model.
-  def validate
-    #    errors.add_to_base MIN_WORKERS_VALIDATION if min_worker.to_i > 20
+  #def validate
+  def ensure_validations
+    #    errors.add(:base, MIN_WORKERS_VALIDATION) if min_worker.to_i > 20
     if max_worker.to_i < min_worker.to_i
-      errors.add_to_base MAX_WORKERS_VALIDATION_1
+      errors.add(:base, MAX_WORKERS_VALIDATION_1)
       #    elsif max_worker.to_i > 20
       #      errors.add_to_base MAX_WORKERS_VALIDATION_2
     end
     #errors.add_to_base APPLICATION_PATH_EXISTANCE_VALIDATION if !File.directory?(path)
-    errors.add_to_base ANALYTICS_VALIDATION if type1=="Rails" and !(analytics.downcase == "enabled" or analytics.downcase == "disabled")
+    errors.add(:base, ANALYTICS_VALIDATION) if type1 and type1=="Rails" and analytics and !(analytics.downcase == "enabled" or analytics.downcase == "disabled")
     #errors.add_to_base ENVIRONMENT_VALIDATION if !(environment.downcase == "production" or environment.downcase == "development" or environment.downcase == "test")
-    errors.add_to_base TYPE_VALIDATION if !(type1.downcase == RAILS or type1.downcase == RACK)
+    errors.add(:base, TYPE_VALIDATION) if type1 and !(type1.downcase == RAILS or type1.downcase == RACK)
     if path and type1
       if type1.downcase == RAILS
         unless File.exists?(File.join(path, 'config', 'environment.rb'))
-          errors.add_to_base "The entered application path is not a valid Rails application path."
+          errors.add(:base, "The entered application path is not a valid Rails application path.")
         end
       end
       if type1.downcase == RACK
         unless File.exists?(File.join(path, 'config.ru'))
-          errors.add_to_base "The entered application path is not a valid Rack application path."
+          errors.add(:base, "The entered application path is not a valid Rack application path.")
         end
       end
     end
@@ -132,7 +162,7 @@ class ApplicationSpecification < PseudoModel
 
       if (tokens.size == 1 and tokens[0].start_with?('/'))
         write_attribute(:baseuri, resolver)
-        errors.add_to_base "BaseURI must start with '/' and contains characters A-Z, a-z, 0-9 , _ , -  and /." if !(baseuri.strip =~ /^\/[A-Za-z0-9_\-\/]*$/i)
+        errors.add(:base, "BaseURI must start with '/' and contains characters A-Z, a-z, 0-9 , _ , -  and /.") if !(baseuri.strip =~ /^\/[A-Za-z0-9_\-\/]*$/i)
       else
         write_attribute(:host_names, resolver)
         host_name_flag = 1
@@ -145,38 +175,38 @@ class ApplicationSpecification < PseudoModel
           len = token.size
           # Whole name can not exceed total length of 253
           if len > 253
-            errors.add_to_base HOSTNAME_LENGTH_EXCEEDS
+            errors.add(:base , HOSTNAME_LENGTH_EXCEEDS)
             next
           end
           # Subdivision can go down to maximum 127 level.
           labels = token.split(/\./)
           if labels.size > 127
-            errors.add_to_base SUBDIVISION_EXCEEDS_127
+            errors.add(:base, SUBDIVISION_EXCEEDS_127)
             next
           end
           # Start with '/' indicates BaseURI
           if token.start_with?('/')
-            errors.add_to_base BASEURI_AND_HOSTNAMES_EXIST
+            errors.add(:base, BASEURI_AND_HOSTNAMES_EXIST)
             break
           end
           # If wildcard '*' presents, Hostname should prefix with '~'
           if token.include?("*") and !(token =~ /^~/)
-            errors.add_to_base "#{token} - #{START_WTIH_TILD}"
+            errors.add(:base, "#{token} - #{START_WTIH_TILD}")
             next
           end
           # Wildcard '*' can come either at start or at end, not inbetween
           pos = token.index('*', 1)
           if pos and pos != 1 and pos != len-1
-            errors.add_to_base "#{token} - #{WILDCARD_AT_START_OR_END}"
+            errors.add(:base, "#{token} - #{WILDCARD_AT_START_OR_END}")
             next
           end
           pos = token.index('*', 2)
           if pos and pos != len - 1
-            errors.add_to_base "#{token} - #{WILDCARD_AT_START_OR_END}"
+            errors.add(:base, "#{token} - #{WILDCARD_AT_START_OR_END}")
             next
           end
           if token =~ /[.][.]/
-            errors.add_to_base "#{token} - #{CONSECUTIVE_DOTS}"
+            errors.add(:base, "#{token} - #{CONSECUTIVE_DOTS}")
             next
           end
           # Check for Letters, Digits and Hyphen
@@ -187,14 +217,14 @@ class ApplicationSpecification < PseudoModel
           labels = labels[1..labels.size - 2]
           labels.each do |label|
             if label.size > 63
-              errors.add_to_base LABEL_LENGTH_EXCEEDS
+              errors.add(:base, LABEL_LENGTH_EXCEEDS)
               next
             end
             label.each_char do |c|
               if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '-'
                 next
               else
-                errors.add_to_base HOSTNAME_LDH
+                errors.add(:base, HOSTNAME_LDH)
                 err_flag = 1
                 break;
               end
@@ -210,7 +240,7 @@ class ApplicationSpecification < PseudoModel
             c = char_array[0]
             if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '~'
             else
-              errors.add_to_base HOSTNAME_LDH
+              errors.add(:base, HOSTNAME_LDH)
               err_flag = 1
             end
           end
@@ -219,7 +249,7 @@ class ApplicationSpecification < PseudoModel
             c = char_array[1]
             if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '*'
             else
-              errors.add_to_base HOSTNAME_LDH
+              errors.add(:base, HOSTNAME_LDH)
               err_flag = 1
             end
           end
@@ -230,7 +260,7 @@ class ApplicationSpecification < PseudoModel
               if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '-'
                 next
               else
-                errors.add_to_base HOSTNAME_LDH
+                errors.add(:base, HOSTNAME_LDH)
                 err_flag = 1
                 break;
               end
@@ -245,7 +275,7 @@ class ApplicationSpecification < PseudoModel
             c = char_array[char_array.size - 1]
             if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '*'
             else
-              errors.add_to_base HOSTNAME_LDH
+              errors.add(:base, HOSTNAME_LDH)
               err_flag = 1
             end
           end
@@ -255,7 +285,7 @@ class ApplicationSpecification < PseudoModel
             char_array.each do |c|
               if (c >= 'a' and c <= 'z') or (c >= 'A' and c <= 'Z') or (c >= '0' and c <= '9') or c == '*'
               else
-                errors.add_to_base HOSTNAME_LDH
+                errors.add(:base, HOSTNAME_LDH)
                 err_flag = 1
               end
             end
@@ -267,7 +297,7 @@ class ApplicationSpecification < PseudoModel
       end
 
       #BaseURI can not be /admin-panel
-      errors.add_to_base BASEURI_AS_ADMIN_PANEL_BASEURI_VALIDATION if baseuri and baseuri.strip == ADMIN_PANEL_BASE_URI
+      errors.add(:base, BASEURI_AS_ADMIN_PANEL_BASEURI_VALIDATION) if baseuri and baseuri.strip == ADMIN_PANEL_BASE_URI
       all_host_names = []
       curr_host_names_size = curr_host_names.size
 
@@ -278,8 +308,8 @@ class ApplicationSpecification < PseudoModel
       if info and info['Application Specification']
         i = 0
         info['Application Specification'].each do |app_info|
-          errors.add_to_base "#{name} - #{APPLICATION_NAME_REPEATED}" if app_info['name'] == name and tmp_id != i
-          errors.add_to_base BASEURI_EXISTANCE_VALIDATION if baseuri and baseuri.strip and app_info['baseuri'] == baseuri.strip and tmp_id != i
+          errors.add(:base, "#{name} - #{APPLICATION_NAME_REPEATED}") if app_info['name'] == name and tmp_id != i
+          errors.add(:base, BASEURI_EXISTANCE_VALIDATION) if baseuri and baseuri.strip and app_info['baseuri'] == baseuri.strip and tmp_id != i
           all_host_names << app_info['host_names'].split(/ /) if app_info['host_names'] and curr_host_names_size > 0 and tmp_id != i
           i += 1
         end
@@ -287,12 +317,12 @@ class ApplicationSpecification < PseudoModel
       all_host_names.flatten!
       # checking uniqueness of Hostnames
       if curr_host_names_size > 0
-        errors.add_to_base HOSTNAME_REPEATED  if curr_host_names_size != curr_host_names.uniq.size
+        errors.add(:base, HOSTNAME_REPEATED)  if curr_host_names_size != curr_host_names.uniq.size
         if all_host_names.size > 0
           curr_host_names.each do |c_host_name|
             all_host_names.each do |host_name|
               if c_host_name.eql?(host_name)
-                errors.add_to_base "#{c_host_name}-#{HOSTNAME_REPEATED }"
+                errors.add(:base, "#{c_host_name}-#{HOSTNAME_REPEATED }")
                 break
               end
             end