Revert "feat(crons): Add thresholds to monitor config (#54516)"

This reverts commit d21b29c.

Co-authored-by: rjo100 <7078270+rjo100@users.noreply.github.com>

src/sentry/monitors/models.py

@@ -62,8 +62,6 @@
  "schedule_type": {"type": "integer"},
  "schedule": {"type": ["string", "array"]},
  "alert_rule_id": {"type": ["integer", "null"]},
- "failure_issue_threshold": {"type": ["integer", "null"]},
- "recovery_threshold": {"type": ["integer", "null"]},
  },
  # TODO(davidenwang): Old monitors may not have timezone or schedule_type, these should be added here once we've cleaned up old data
  "required": ["checkin_margin", "max_runtime", "schedule"],

src/sentry/monitors/validators.py

@@ -108,22 +108,6 @@ class ConfigValidator(serializers.Serializer):
  help_text="tz database style timezone string",
  )
 
- failure_issue_threshold = EmptyIntegerField(
- required=False,
- allow_null=True,
- default=None,
- help_text="How many consecutive missed or failed check-ins in a row before creating a new issue.",
- min_value=1,
- )
-
- recovery_threshold = EmptyIntegerField(
- required=False,
- allow_null=True,
- default=None,
- help_text="How many successful check-ins in a row before resolving an issue.",
- min_value=1,
- )
-
  def bind(self, *args, **kwargs):
  super().bind(*args, **kwargs)
  # Inherit instance data when used as a nested serializer

tests/sentry/monitors/endpoints/test_organization_monitor_index.py

@@ -183,8 +183,6 @@ def test_simple(self, mock_record):
  "schedule": "0 0 * * *",
  "checkin_margin": None,
  "max_runtime": None,
- "failure_issue_threshold": None,
- "recovery_threshold": None,
  }
 
  self.project.refresh_from_db()