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

[Feature]: ScheduledBackup resource should be able to backup using plugin method #4422

Open
2 tasks done
dougkirkley opened this issue May 2, 2024 · 0 comments · May be fixed by #4424
Open
2 tasks done

[Feature]: ScheduledBackup resource should be able to backup using plugin method #4422

dougkirkley opened this issue May 2, 2024 · 0 comments · May be fixed by #4424
Assignees
Labels
triage Pending triage

Comments

@dougkirkley
Copy link

Is there an existing issue already for this feature request/idea?

  • I have searched for an existing issue, and could not find anything. I believe this is a new feature request to be evaluated.

What problem is this feature going to solve? Why should it be added?

It will allow the use of backup plugins in the ScheduledBackup resource

Describe the solution you'd like

The pluginConfiguration field is already in the CRD, so it only needs to add "plugin" to the enum for method and the logic to handle the plugin type similar to the Backup controller.

Describe alternatives you've considered

The only alternative I can think of is having a separate cron mechanism to call backups with the required configuration

Additional context

No response

Backport?

Yes

Are you willing to actively contribute to this feature?

Yes

Code of Conduct

  • I agree to follow this project's Code of Conduct
@dougkirkley dougkirkley added the triage Pending triage label May 2, 2024
@dougkirkley dougkirkley linked a pull request May 2, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage Pending triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants