Opening new session to complete task activities

[galvana]

Closes LA-108

Description Of Changes

This PR is to commit the changes done as part of testing in #5547

Code Changes

• Opens a new session to finalize the task activities for the run_access_node, run_erasure_node, and run_consent_node

Steps to Confirm

1. No new functionality, this just makes it so that long running DSR tasks can complete their finalizing activities, even after running a long query or request.

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
• Followup issues:
  • Followup issues created (include link)
  • No followup issues
• Database migrations:
  • Ensure that your downrev is up to date with the latest revision on main
  • Ensure that your downgrade() migration is correct and works
    • If a downgrade migration is not possible for this change, please call this out in the PR description!
  • No migrations
• Documentation:
  • Documentation complete, PR opened in fidesdocs
  • Documentation issue created in fidesdocs
  • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
  • No documentation updates required

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 15, 2025 6:49pm

[cypress]

fides    Run #11779

Run Properties:   Passed #11779  •  db4a5d13e9 ℹ️: Merge d699ac728c0d34bb01e82f6bd145a2dd9439484b into a4f4aec1a4b40ec8dbfee71d89dc...

Project	fides
Branch Review	refs/pull/5667/merge
Run status	Passed #11779
Run duration	00m 52s
Commit	db4a5d13e9 ℹ️: Merge d699ac728c0d34bb01e82f6bd145a2dd9439484b into a4f4aec1a4b40ec8dbfee71d89dc...
Committer	Adrian Galvan
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	4
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

[codecov]

Codecov Report

Attention: Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.

Project coverage is 87.14%. Comparing base (a4f4aec) to head (d699ac7).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
src/fides/api/task/execute_request_tasks.py	94.44%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5667   +/-   ##
=======================================
  Coverage   87.14%   87.14%           
=======================================
  Files         388      388           
  Lines       24014    24022    +8     
  Branches     2593     2593           
=======================================
+ Hits        20928    20935    +7     
- Misses       2525     2526    +1     
  Partials      561      561           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[eastandwestwind]

Reviewed during screenshare CR session. Looks good!

[cypress]

fides    Run #11780

Run Properties:   Failed #11780  •  39dbc4847c: Opening new session to complete task activities (#5667)

Project	fides
Branch Review	main
Run status	Failed #11780
Run duration	01m 07s
Commit	39dbc4847c: Opening new session to complete task activities (#5667)
Committer	Adrian Galvan
View all properties for this run ↗︎

---

Test results
Failures	1
Flaky	0
Pending	0
Skipped	0
Passing	3
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

---

Tests for review