Skip to content

Skip database isolation case for task mapping taskinstance tests (#41954) - #41954

Merged
potiuk merged 3 commits into
apache:v2-10-testfrom
astronomer:DBIsolationTask
Sep 3, 2024
Merged

Skip database isolation case for task mapping taskinstance tests (#41954)#41954
potiuk merged 3 commits into
apache:v2-10-testfrom
astronomer:DBIsolationTask

Conversation

@utkarsharma2

@utkarsharma2utkarsharma2 commented Sep 2, 2024

Copy link
Copy Markdown
Contributor

Cherry-picked commits - 0f35c53, 0f35c53

The test has been updated for DB isolation but the retrieval of
task was not intuitive and it could lead to flaky tests possibly
(cherry picked from commit f25adf1)
@utkarsharma2utkarsharma2 changed the title Db isolation taskSkip database isolation case for task mapping taskinstance testsSep 2, 2024
@utkarsharma2utkarsharma2 changed the title Skip database isolation case for task mapping taskinstance testsSkip database isolation case for task mapping taskinstance tests (#41954)Sep 2, 2024
@ephraimbuddy

Copy link
Copy Markdown
Contributor

Conflict

@potiuk
potiuk merged commit 7f49cfc into apache:v2-10-testSep 3, 2024
ephraimbuddy pushed a commit that referenced this pull request Sep 13, 2024
) (#41954)
* Simpler task retrieval for taskinstance test (#41389)
The test has been updated for DB isolation but the retrieval of
task was not intuitive and it could lead to flaky tests possibly
(cherry picked from commit f25adf1)
* Skip database isolation case for task mapping taskinstance tests (#41471)
Related: #41067
(cherry picked from commit 7718bd7)
---------
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
@utkarsharma2utkarsharma2 added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Sep 17, 2024
@utkarsharma2utkarsharma2 added this to the Airflow 2.10.2 milestone Sep 17, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skipChanges that should be skipped from the changelog (CI, tests, etc..)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@utkarsharma2@ephraimbuddy@potiuk@uranusjr@hussein-awala