Joal has uploaded a new change for review.
https://gerrit.wikimedia.org/r/248991
Change subject: Correct two bugs in refinery oozie jobs
......................................................................
Correct two bugs in refinery oozie jobs
send_error_email workflow file was incorrectly named
Incorrect test to send email in pageview whitelist checking
Change a wrong aemail address to the correct one
Bug: T116649
Change-Id: I21949b9ea92bf462701b3ea58bf88cbd72301af7
---
M oozie/pageview/hourly/workflow.xml
R oozie/util/send_error_email/workflow.xml
2 files changed, 3 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/analytics/refinery
refs/changes/91/248991/1
diff --git a/oozie/pageview/hourly/workflow.xml
b/oozie/pageview/hourly/workflow.xml
index c2e9310..e101372 100644
--- a/oozie/pageview/hourly/workflow.xml
+++ b/oozie/pageview/hourly/workflow.xml
@@ -332,11 +332,11 @@
<error to="send_error_email"/>
</action>
- <!-- Check unexpected_values partition is empty using file size = 0 -->
+ <!-- Check unexpected_values partition is empty using file size > 0 -->
<decision name="empty_result_check">
<switch>
<case to="send_whitelist_email">
- ${fs:dirSize(unexpected_values_dataset_directory) eq 0}
+ ${fs:dirSize(unexpected_values_dataset_directory) gt 0}
</case>
<default to="end"/>
</switch>
diff --git a/oozie/util/send_error_email/worflow.xml
b/oozie/util/send_error_email/workflow.xml
similarity index 94%
rename from oozie/util/send_error_email/worflow.xml
rename to oozie/util/send_error_email/workflow.xml
index 72d03ff..46c1296 100644
--- a/oozie/util/send_error_email/worflow.xml
+++ b/oozie/util/send_error_email/workflow.xml
@@ -29,7 +29,7 @@
<property>
<name>to</name>
-
<value>[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]</value>
+
<value>[email protected],[email protected],[email protected],[email protected],[email protected],[email protected]</value>
<description>
The comma-separated list of email recipients
</description>
--
To view, visit https://gerrit.wikimedia.org/r/248991
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I21949b9ea92bf462701b3ea58bf88cbd72301af7
Gerrit-PatchSet: 1
Gerrit-Project: analytics/refinery
Gerrit-Branch: master
Gerrit-Owner: Joal <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
