Skip to content

Conversation

DinglyCoder
Copy link
Contributor

Extended backup APIs to support uploading instance and custom volume backups to an S3 bucket, deleting the local backup afterwards.

@DinglyCoder DinglyCoder requested a review from stgraber as a code owner May 2, 2025 22:05
@github-actions github-actions bot added Documentation Documentation needs updating API Changes to the REST API labels May 2, 2025
@parthgupta2005
Copy link

@stgraber Apologies for the multiple comments. We were hoping you could review the pull request when you have a moment, so we can address any issues in our implementation if needed. Thank you very much for your time!

@stgraber
Copy link
Member

stgraber commented May 6, 2025

@parthgupta2005 no worries. I'm going through the PRs mostly in notification order from Github. There's quite a lot to look at but hopefully I can clear my backlog tomorrow or Thursday.

@stgraber stgraber added this to the incus-6.14 milestone May 29, 2025
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
@stgraber stgraber force-pushed the allow-s3-backups branch from c5b50a7 to 0e6d4c0 Compare June 3, 2025 22:00
@stgraber
Copy link
Member

stgraber commented Jun 3, 2025

Sorry for the delay. I ended up doing a bit of tweaking on this one to make it easier to use and more flexible in the future.

@stgraber stgraber force-pushed the allow-s3-backups branch from 0e6d4c0 to 5ba6360 Compare June 3, 2025 22:01
DinglyCoder and others added 4 commits June 3, 2025 18:08
Signed-off-by: Akshat Kumar <akaman.boss@gmail.com>
Signed-off-by: Akshat Kumar <akaman.boss@gmail.com>
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Signed-off-by: Akshat Kumar <akaman.boss@gmail.com>
@stgraber stgraber force-pushed the allow-s3-backups branch from 5ba6360 to 165a112 Compare June 3, 2025 22:08
@stgraber stgraber enabled auto-merge June 3, 2025 22:09
@stgraber stgraber merged commit 4ee508b into lxc:main Jun 3, 2025
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Changes to the REST API Documentation Documentation needs updating
Development

Successfully merging this pull request may close these issues.

3 participants