Fix job_task_id not found if db doesnt have the field 🐛 🚑️

[wvangeit]

What do these changes do?

A new field was added to the function jobs to store the celery task id. However, this is not present in existing entities, which generates 500 errors in production atm.
I'm creating this small PR to be applied as a hotfix to handle that case.
By using get() in the dictionary the value will default to None.

How to test

Test code on deployment that has old function job entities

Dev-ops

No changes
To be hotfixed in production.

[wvangeit]

@mergify queue

[mergify]

queue

🛑 Configuration not compatible with a branch protection setting

The branch protection setting Require branches to be up to date before merging is not compatible with max_parallel_checks>1, queue_conditions != merge_conditions and must be unset.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.80%. Comparing base (7a4e794) to head (351ac02).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #8347      +/-   ##
==========================================
- Coverage   86.76%   84.80%   -1.97%     
==========================================
  Files        1771      760    -1011     
  Lines       68674    34985   -33689     
  Branches     1312      175    -1137     
==========================================
- Hits        59585    29669   -29916     
+ Misses       8694     5259    -3435     
+ Partials      395       57     -338     

Flag	Coverage Δ
integrationtests	64.03% <ø> (+0.84%)	⬆️
unittests	87.23% <ø> (+1.10%)	⬆️

Components	Coverage Δ
pkg_aws_library	∅ <ø> (∅)
pkg_celery_library	∅ <ø> (∅)
pkg_dask_task_models_library	∅ <ø> (∅)
pkg_models_library	∅ <ø> (∅)
pkg_notifications_library	∅ <ø> (∅)
pkg_postgres_database	∅ <ø> (∅)
pkg_service_integration	∅ <ø> (∅)
pkg_service_library	∅ <ø> (∅)
pkg_settings_library	∅ <ø> (∅)
pkg_simcore_sdk	76.95% <ø> (-8.08%)	⬇️
agent	∅ <ø> (∅)
api_server	∅ <ø> (∅)
autoscaling	∅ <ø> (∅)
catalog	∅ <ø> (∅)
clusters_keeper	∅ <ø> (∅)
dask_sidecar	∅ <ø> (∅)
datcore_adapter	∅ <ø> (∅)
director	∅ <ø> (∅)
director_v2	78.16% <ø> (-7.19%)	⬇️
dynamic_scheduler	∅ <ø> (∅)
dynamic_sidecar	81.87% <ø> (-8.59%)	⬇️
efs_guardian	∅ <ø> (∅)
invitations	∅ <ø> (∅)
payments	∅ <ø> (∅)
resource_usage_tracker	∅ <ø> (∅)
storage	∅ <ø> (∅)
webclient	∅ <ø> (∅)
webserver	87.99% <ø> (+0.77%)	⬆️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a4e794...351ac02. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pcrespov]

thx

[mergify]

🧪 CI Insights

Here's what we observed from your CI run for 351ac02.

🟢 All jobs passed!

But CI Insights is watching 👀

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud