Re: [PR] Amazon Bedrock - Fix system test [airflow]

2024-04-10 Thread via GitHub


ferruzzi merged PR #38887:
URL: https://github.com/apache/airflow/pull/38887


-- 
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



[PR] Amazon Bedrock - Fix system test [airflow]

2024-04-09 Thread via GitHub


ferruzzi opened a new pull request, #38887:
URL: https://github.com/apache/airflow/pull/38887

   The branching control logic in the system test doesn't work as I had 
expected on our internal CI stack, so I've modified the test to use a proper 
Branching Operator.  For testing purposes, I swapped the actual bedrock 
operators out for EmptyOperators just to reduce the wait time on feedback, but 
below are screenshots of both the skip and no-skip loops working as expected in 
Breeze
   
   SKIP = False
   
![image](https://github.com/apache/airflow/assets/1920178/470bdc5c-89a4-4b57-84d5-970209b99c84)
   
   SKIP=True
   
![image](https://github.com/apache/airflow/assets/1920178/17a4b714-0e01-465d-a356-4a1c2075f412)
   
   
   
   
   
   
   
   
   
   ---
   **^ Add meaningful description above**
   Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information.
   In case of fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in a 
newsfragment file, named `{pr_number}.significant.rst` or 
`{issue_number}.significant.rst`, in 
[newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
   


-- 
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