change docstring; update return val

Author: cyphronix

aws_sra_examples/solutions/genai/bedrock_org/lambda/src/sra_cloudwatch.py

@@ -247,21 +247,21 @@ def create_oam_sink(self, sink_name: str) -> str:
                 self.LOGGER.error(f"{self.UNEXPECTED} error: {e}")
                 raise ValueError(f"Unexpected error executing Lambda function. {e}") from None
 
-    def delete_oam_sink(self, sink_arn: str) -> None:
-        """Delete the Observability Access Manager sink for SRA in the organization.
-
-        Args:
-            sink_arn (str): ARN of the sink
-
-        Returns:
-            None
-        """
-        try:
-            self.CWOAM_CLIENT.delete_sink(Identifier=sink_arn)
-            self.LOGGER.info(f"Observability access manager sink {sink_arn} deleted")
-        except ClientError as e:
-            self.LOGGER.info(self.UNEXPECTED)
-            raise ValueError(f"Unexpected error executing Lambda function. {e}") from None
+    # def delete_oam_sink(self, sink_arn: str) -> None:
+    #     """Delete the Observability Access Manager sink for SRA in the organization.
+
+    #     Args:
+    #         sink_arn (str): ARN of the sink
+
+    #     Returns:
+    #         None
+    #     """
+    #     try:
+    #         self.CWOAM_CLIENT.delete_sink(Identifier=sink_arn)
+    #         self.LOGGER.info(f"Observability access manager sink {sink_arn} deleted")
+    #     except ClientError as e:
+    #         self.LOGGER.info(self.UNEXPECTED)
+    #         raise ValueError(f"Unexpected error executing Lambda function. {e}") from None
 
     def find_oam_sink_policy(self, sink_arn: str) -> tuple[bool, dict]:
         """Check if the Observability Access Manager sink policy for SRA in the organization exists.

aws_sra_examples/solutions/genai/bedrock_org/lambda/src/sra_config.py

@@ -111,7 +111,7 @@ def find_config_rule(self, rule_name):
         except ClientError as e:
             if e.response["Error"]["Code"] == "NoSuchConfigRuleException":
                 self.LOGGER.info(f"No such config rule: {rule_name}")
-                return False, None
+                return False, {}
             else:
                 self.LOGGER.info(f"Unexpected error: {e}")
                 raise e