Skip to content

temprorially disable publishing legaltag status change events

Kelly Zhou requested to merge update-cron-job into master

All Submissions:

  • [YES/NO] I have added an explanation of what changes in this merge do and why we should include it?
  • [YES/NO] I have updated the documentation accordingly.
  • [YES/NO/NA] I have added tests to cover my changes.
  • [YES/NO/NA] All new and existing tests passed.
  • [YES/NO/NA] My code follows the code style of this project.
  • [YES/NO/NA] I ran lint checks locally prior to submission.

What is the current behavior?

Issue: #23 (closed)

We're temporarily disabling publishing legaltag status change events due to the issue in Storage service: osdu/platform/system/storage#117

We will re-enable it back once storage issue is fixed.

What is the new behavior?

  • legaltag cron job will only update the status of legaltags based on their validity, but not publishing any status update events out to Storage service.

Does this introduce a breaking change?

  • [YES/NO]

No as Storage is having issues deleting large number of records upon legaltag expiration

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

N/A

Other information

N/A

Merge request reports