potiuk commented on code in PR #45704:
URL: https://github.com/apache/airflow/pull/45704#discussion_r1918196249


##########
providers/tests/system/amazon/aws/example_bedrock_retrieve_and_generate.py:
##########
@@ -442,9 +442,8 @@ def delete_opensearch_policies(collection_name: str):
     test_context = sys_test_context_task()
     env_id = test_context["ENV_ID"]
 
-    aoss_client = OpenSearchServerlessHook(aws_conn_id=None)
-    bedrock_agent_client = BedrockAgentHook(aws_conn_id=None)
-
+    aoss_client = OpenSearchServerlessHook()
+    bedrock_agent_client = BedrockAgentHook()

Review Comment:
   yes. that's deliberate change. this is an example dag so unless it is called 
during system tests, this test is not actually executed. I will leave it to 
@ferruzzi, @vincbeck and @o-nikolas to see a) if they already used it in system 
tests, b) whether that change anything for them - I guess they don't have 
conn_id "aws_default" defined in their system test environment, so it will 
anyway fall back to the default authentication mechanism.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to