fix(_rotate_kms_key): remove SkipPerIssues since the ticket is resolved

As per discussion, this clause can be removed as the ticket is resolved
and doesn't have sct skip label (what could have made it being skipped).
At the same, it led to unexpected failures when self.params appeared to
be dict type instance instead of SCTConfiguration as expected.

refs:
- #9134

(cherry picked from commit b9708e8)

sdcm/cluster.py

@@ -4524,9 +4524,6 @@ def _rotate_kms_key(kms_key_alias_name, kms_key_rotation_interval, db_cluster):
                     AwsKmsEvent(
                         message=f"Failed to rotate AWS KMS key for the '{kms_key_alias_name}' alias",
                         traceback=traceback.format_exc()).publish()
-                if SkipPerIssues("https://github.com/scylladb/scylla-enterprise/issues/3896", self.params):
-                    self.log.warning("KMS encryption check is skipped due to the 'scylla-enterprise/issues/3896'")
-                    continue
                 try:
                     nemesis_class = self.nemesis[0] if self.nemesis else getattr(
                         import_module('sdcm.nemesis'), "Nemesis")