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

Add ServerMaintenance API type #208

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

afritzler
Copy link
Member

@afritzler afritzler commented Dec 17, 2024

Proposed Changes

Fixes #76

@afritzler afritzler marked this pull request as draft December 17, 2024 12:38
@github-actions github-actions bot added documentation Improvements or additions to documentation enhancement New feature or request size/M labels Dec 17, 2024
@afritzler afritzler force-pushed the enh/servermaintenance branch 4 times, most recently from ba9b7a6 to d6883cf Compare December 17, 2024 12:55
@afritzler afritzler changed the title Add ServerMaintenance CRD Add ServerMaintenance API type Dec 17, 2024
@afritzler afritzler force-pushed the enh/servermaintenance branch 2 times, most recently from 6a20315 to 68b6e15 Compare December 18, 2024 14:42
@afritzler afritzler force-pushed the enh/servermaintenance branch from 68b6e15 to e50974c Compare December 18, 2024 14:44
@stefanhipfel
Copy link
Contributor

stefanhipfel commented Dec 30, 2024

would it make sense to add a time slot property when to perform maintenances, or should this be done outside of the ServerMaintenance operation?
e.g. I want to allow maintenances only during weekends or at a specific timestamp:
Customer gets notified (Email or dashboard) that there are maintenances pending and needs to define a time when those should be applied.

Currently this could be done setting the OwnerApproval label at the right moment, but this is a manual step or needs another service which automates that step. I think having that implemented in the ServerMaintenance operator makes for a great feature, which is something we need at ccloud.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Declarative Server Maintenance
2 participants