Use 0 as default version to distinguish faulty reports

RedHatInsights · May 29, 2024 · d16849a · d16849a
1 parent 6baef78
commit d16849a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/ccx_messaging/publishers/rule_processing_publisher.py b/ccx_messaging/publishers/rule_processing_publisher.py
@@ -106,7 +106,7 @@ def publish(self, input_msg, report):
 
         try:
             msg_timestamp = input_msg["timestamp"]
-            msg_version = report.pop("version", 1)
+            msg_version = report.pop("version", 0)
             output_msg = {
                 "OrgID": org_id,
                 "AccountNumber": account_number,

diff --git a/test/publishers/rule_processing_publisher_test.py b/test/publishers/rule_processing_publisher_test.py
@@ -147,7 +147,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },
@@ -174,7 +174,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },
@@ -201,7 +201,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },
@@ -227,7 +227,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },
@@ -254,7 +254,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2023-08-14T09:31:46Z"},
         },
@@ -281,7 +281,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2023-08-14T09:31:46Z"},
         },
@@ -308,7 +308,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },
@@ -335,7 +335,7 @@ def test_publish_bad_argument(wrong_input_msg):
             "ClusterName": "uuid",
             "Report": {"reports": []},
             "LastChecked": "a timestamp",
-            "Version": 1,
+            "Version": 0,
             "RequestId": "a request id",
             "Metadata": {"gathering_time": "2012-01-14T00:00:00Z"},
         },