nucleuscloud / neosync

Open source data anonymization and synthetic data orchestration for developers. Create high fidelity synthetic data and sync it across your environments.
https://www.neosync.dev
Other
3.41k stars 125 forks source link

NEOS-1625: updates temporal schedule errors to be unique #2971

Closed nickzelei closed 1 day ago

nickzelei commented 1 day ago

We were previously logging each error from temporal schedule with the same message. This PR updates that to be unique for better tracking.

Also when describing job statuses, we were layering in the wrong error message, that has been fixed. - It has also been updated to a warn as this error is common when job runs have been archived.

linear[bot] commented 1 day ago

NEOS-1625 Investigate err log bug in GetJobRunStatuses

vercel[bot] commented 1 day ago

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

1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **neosync-docs** | ⬜️ Ignored ([Inspect](https://vercel.com/neosync/neosync-docs/AhCq4c3wcH7cwZvioCiXWFTagYNs)) | | | Nov 21, 2024 5:34pm |
codecov[bot] commented 1 day ago

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 35.01%. Comparing base (f686279) to head (b580e25). Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
backend/services/mgmt/v1alpha1/job-service/jobs.go 0.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2971 +/- ## ======================================= Coverage 35.00% 35.01% ======================================= Files 330 330 Lines 38428 38425 -3 ======================================= + Hits 13452 13454 +2 + Misses 23411 23408 -3 + Partials 1565 1563 -2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features: