Make if logic simpler

Author: indy-independence

src/cnaas_nms/devicehandler/sync_devices.py

@@ -947,13 +947,12 @@ def sync_devices(
             remove_sync_events(hostname)
             dev.last_seen = datetime.datetime.utcnow()
         if not dry_run and get_confirm_mode(confirm_mode_override) != 2:
-            if failed_hosts:
-                if get_confirm_mode(confirm_mode_override) == 1:
-                    logger.error(
-                        "One or more devices failed to commit configuration, they will roll back configuration"
-                        " in {}s: {}".format(api_settings.COMMIT_CONFIRMED_TIMEOUT, ", ".join(failed_hosts))
-                    )
-                    time.sleep(api_settings.COMMIT_CONFIRMED_TIMEOUT)
+            if failed_hosts and get_confirm_mode(confirm_mode_override) == 1:
+                logger.error(
+                    "One or more devices failed to commit configuration, they will roll back configuration"
+                    " in {}s: {}".format(api_settings.COMMIT_CONFIRMED_TIMEOUT, ", ".join(failed_hosts))
+                )
+                time.sleep(api_settings.COMMIT_CONFIRMED_TIMEOUT)
             logger.info("Releasing lock for devices from syncto job: {}".format(job_id))
             Joblock.release_lock(session, job_id=job_id)