Skip to content

Conversation

@ochosi
Copy link
Contributor

@ochosi ochosi commented Jan 22, 2026

Remove the clone endpoints and related functionality:

  • Remove /composes/{composeId}/clone, /composes/{composeId}/clones, and /clones/{id} API endpoints from OpenAPI spec and regenerate api.go
  • Remove CloneCompose, GetCloneStatus, GetComposeClones handlers
  • Remove clone-related DB interface methods (InsertClone, GetClonesForCompose, GetClone) and SQL queries
  • Remove CloneCompose and CloneStatus methods from composer client
  • Remove clone-related tests from handler_test.go and main_test.go
  • Add migration to drop the clones table

The clone feature allowed copying AWS AMIs to other regions, but this functionality is being removed in favor of direct uploads to the desired region.

This is the backend change relating to osbuild/image-builder-frontend#4018

@ochosi
Copy link
Contributor Author

ochosi commented Jan 22, 2026

The frontend PR first needs to be merged.

Copy link
Collaborator

@lzap lzap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lovely. I want more.

@ochosi ochosi removed the blocked label Jan 27, 2026
@ochosi ochosi marked this pull request as ready for review January 27, 2026 18:21
Remove the clone endpoints and related functionality:
- Remove /composes/{composeId}/clone, /composes/{composeId}/clones, and
  /clones/{id} API endpoints from OpenAPI spec and regenerate api.go
- Remove CloneCompose, GetCloneStatus, GetComposeClones handlers
- Remove clone-related DB interface methods (InsertClone, GetClonesForCompose,
  GetClone) and SQL queries
- Remove CloneCompose and CloneStatus methods from composer client
- Remove clone-related tests from handler_test.go and main_test.go
- Add migration to drop the clones table

The clone feature allowed copying AWS AMIs to other regions, but this
functionality is being removed in favor of direct uploads to the desired region.
@ochosi
Copy link
Contributor Author

ochosi commented Jan 27, 2026

We may want to let the frontend changes sit a week or so before we drop this from the backend

Copy link
Collaborator

@lzap lzap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spot instance was killed.

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.

2 participants