Skip to content

Merge OSS and cloud - 1 #1665

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.

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

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

Merge OSS and cloud - 1 #1665

wants to merge 37 commits into from

Conversation

mschuwalow
Copy link
Contributor

@mschuwalow mschuwalow commented May 28, 2025

resolves #1618
resolves #1619

Mostly mechanical changes here. More interesting refactors will be done in smaller followup prs

  • Removed all oss services and replaced them with their cloud counterparts
  • Merge the grpc apis
  • Removed the oss client. Only a single client exists now
  • Removed oss only types (but left the polymorphism intact for now. This I will do in the next pr)
  • Adjust the test-framework to use cloud services and the "cloud service"
  • Fix a number of small bugs that I encountered while getting the tests working

@mschuwalow mschuwalow force-pushed the cloud-oss-merge-1 branch from bd529a6 to 34174c5 Compare May 28, 2025 16:12
@mschuwalow mschuwalow force-pushed the cloud-oss-merge-1 branch from 34174c5 to 4d3f479 Compare June 2, 2025 18:09
@mschuwalow mschuwalow marked this pull request as ready for review June 2, 2025 23:37
harshtech123 pushed a commit to harshtech123/golem-openapi that referenced this pull request Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge component service oss+cloud Merge worker service cloud+oss
1 participant