Skip to content

Refactor api-server to use controller-service-repository architecture and rpc communication with backend #7459

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
1 task done
bisgaard-itis opened this issue Mar 31, 2025 · 3 comments
Closed
1 task done

Refactor api-server to use controller-service-repository architecture and rpc communication with backend #7459

bisgaard-itis opened this issue Mar 31, 2025 · 3 comments
Assignees
@pcrespov @bisgaard-itis
Labels
t:maintenance Some planned maintenance work
Milestone
The Awakening

Comments

@bisgaard-itis
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Planned changes

This is a sub issue from ITISFoundation/osparc-issues#675
@bisgaard-itis bisgaard-itis added the t:maintenance Some planned maintenance work label Mar 31, 2025
@bisgaard-itis bisgaard-itis added this to the The Awakening milestone Mar 31, 2025
@pcrespov pcrespov mentioned this issue Apr 2, 2025
Merged
@pcrespov
Copy link
Member

pcrespov commented Apr 2, 2025 •
edited by bisgaard-itis
Loading

  • three layer design:
    • controllers api/routes --> api/rest
    • service
      • hide rpc/http clients below BaseService class
    • repository (only auth for now but prepare to deprecate)
    • clients
      • rabbit
      • postgress
    • models
      • domain
      • schemas
  • Guarantee atomic operations during aggregations
    • unit of work
  • Gateway (manage request load)
  • Privileges (e.g give priority to paying customers)

@pcrespov pcrespov mentioned this issue Apr 3, 2025
Merged
1 task
@bisgaard-itis bisgaard-itis mentioned this issue Apr 14, 2025
Closed
1 task
@pcrespov pcrespov mentioned this issue Apr 28, 2025
Merged
@pcrespov
Copy link
Member

image

@bisgaard-itis
Copy link
Contributor Author

Closing this as all subissues have been resolved.

@pcrespov pcrespov mentioned this issue Apr 29, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pcrespov pcrespov

@bisgaard-itis bisgaard-itis

Labels
t:maintenance Some planned maintenance work
Projects
None yet
Milestone
The Awakening
Development

No branches or pull requests
2 participants
@pcrespov @bisgaard-itis