Skip to content

Legal compliance fix

Rucha Deshpande requested to merge legal-compliance-fix into master

All Submissions:

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

What is the current behavior?

The bug and related investigation is explained in the issue linked. Issue: #70 (closed)

What is the new behavior?

  • The compliance API will now not loop over all the partitions for compliance checks. Instead the compliance trigger application loops over all the partitions and send the correct partition-id for the compliance API to retrieve the tenant countries list (Legal_COO.json) from cloud storage

Does this introduce a breaking change?

  • [YES]

All CSPs need to make sure that the compliance trigger is calling the compliance API (BASE_URL/api/legal/v1/jobs/updateLegalTagStatus) for all the partitions.

Any relevant logs, error output, etc?

(If it’s long, please paste to and insert the link here.)

Other information

Edited by Rucha Deshpande

Merge request reports