Merge pull request #311 from brianburnszd/stop-sending-locked

Remove locked attribute

lib/kennel/models/monitor.rb

@@ -83,7 +83,7 @@ def build_json
             include_tags: true,
             escalation_message: Utils.presence(escalation_message.strip),
             evaluation_delay: evaluation_delay,
-            locked: false, # setting this to true prevents any edit and breaks updates when using replace workflow
+            locked: false, # deprecated: setting this to true will likely fail
             renotify_interval: renotify_interval || 0,
             variables: variables
           }
@@ -153,6 +153,9 @@ def build_json
           options[:notification_preset_name] = notification_preset_name
         end
 
+        # locked is deprecated, will fail if used
+        options.delete :locked
+
         data
       end
 
@@ -231,6 +234,9 @@ def self.normalize(expected, actual)
           options.delete(:escalation_message)
           expected_options.delete(:escalation_message)
         end
+        # locked is deprecated: ignored when diffing
+        options.delete(:locked)
+        expected_options.delete(:locked)
       end
 
       private

test/kennel/models/monitor_test.rb

@@ -41,7 +41,6 @@ def monitor(options = {})
         include_tags: true,
         escalation_message: nil,
         evaluation_delay: nil,
-        locked: false,
         renotify_interval: 0,
         thresholds: { critical: 123.0 },
         variables: nil