[MODCONSKC-56] Remove self endpoints invocations

[Saba-Zedginidze-EPAM]

Purpose

[MODCONSKC-56] Remove self endpoints invocations

Approach

• Remove clients for self endpoint invocations
• Add TenantManager alongside TenantService and divide functionality between the two. This was beneficial to avoid complex circular dependencies and improve testing
• Refactor SyncPrimaryAffiliationService to use its own methods instead of self invocation. Includes reference to self in order to utilize @transactional annotation and not bypass the proxy
• Support retry logic for adding tenant by checking if consortium configuration or user tenant exist to prevent creating it
• Update unit tests

[zburke]

There isn't enough info in this PR to review your changes.

• You must summarize the changes in the title
• You must provide context for the changes in the description -- not just the "what", which we can get from git diff, but the "why". If you're fixing a bug, describe the bug and describe why this fix is the correct fix; do not force me to re-triage the problem. If you don't want to use the existing template, fine, but remove it and provide your own explanation.
• You must update NEWS.md
• You should correctly label the Jira ticket, MODCONSKC-56, not Modconskc 56
• It's polite to put a link back to Jira in the description to make it easy for reviewers to check if the work that was done matches the work that was requested

These are general guidelines you should follow on every PR.

[Saba-Zedginidze-EPAM]

This is a draft PR so far, will include necessary details soon

PR updated as requested; thank you!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud