Skip to content
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

Swagger: backup and restore Scylla API #4100

Merged
merged 3 commits into from
Nov 12, 2024

Conversation

Michal-Leszczynski
Copy link
Collaborator

This PR adds swagger definitions for Storage Service Backup and Restore Scylla API that will be used to replace currently used rclone endpoints.
This PR also includes swagger definitions for Task Manager Scylla API that can be used for managing running Backup and Restore Scylla jobs.

@Michal-Leszczynski
Copy link
Collaborator Author

@karol-kokoszka This PR is ready for review!

The failure is a know flake (#3848).

@Michal-Leszczynski
Copy link
Collaborator Author

@karol-kokoszka This PR is ready for review!

@karol-kokoszka
Copy link
Collaborator

@Michal-Leszczynski is the scylla_v1.json taken from scylla repo ? Or the new endpoints at least ?
Can you share the origin of this change ? Document or swagger from scylla.

@Michal-Leszczynski
Copy link
Collaborator Author

Yes - all swagger definitions from scylla_v1 are taken from Scylla repo (from many files), but they have to be adjusted to the swagger 2.0 format. I can add comments with per-file link to our swagger definitions.

@karol-kokoszka
Copy link
Collaborator

I can add comments with per-file link to our swagger definitions.

Please do.

@Michal-Leszczynski
Copy link
Collaborator Author

@karol-kokoszka I decided to add a separate README file to the swagger module, as JSON format does not allow for comments.

Copy link
Collaborator

@VAveryanov8 VAveryanov8 left a comment

Choose a reason for hiding this comment

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

As far as I can tell - looks good to me :)

@Michal-Leszczynski Michal-Leszczynski merged commit 4fc93b5 into master Nov 12, 2024
51 checks passed
@Michal-Leszczynski Michal-Leszczynski deleted the ml/backup-restore-scylla-api branch November 12, 2024 13:17
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.

3 participants