feat (VOMS2CS): only send a mail if something changed

chaen · chaen · commit 4ca2f605619d · 2023-08-29T14:35:19.000+02:00
diff --git a/src/DIRAC/ConfigurationSystem/Agent/VOMS2CSAgent.py b/src/DIRAC/ConfigurationSystem/Agent/VOMS2CSAgent.py
@@ -182,7 +182,7 @@ def execute(self):
                 if self.dryRun:
                     self.log.info("Dry Run: mail won't be sent")
                     self.log.info(mailMsg)
-                else:
+                elif voChanged:
                     NotificationClient().sendMail(
                         self.am_getOption("MailTo", voAdminMail), "VOMS2CSAgent run log", mailMsg, self.mailFrom
                     )

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ def execute(self):`
`182`	`182`	`if self.dryRun:`
`183`	`183`	`self.log.info("Dry Run: mail won't be sent")`
`184`	`184`	`self.log.info(mailMsg)`
`185`		`- else:`
	`185`	`+ elif voChanged:`
`186`	`186`	`NotificationClient().sendMail(`
`187`	`187`	`self.am_getOption("MailTo", voAdminMail), "VOMS2CSAgent run log", mailMsg, self.mailFrom`
`188`	`188`	`)`