-
Notifications
You must be signed in to change notification settings - Fork 128
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
Glob pattern based resource selection #28
base: master
Are you sure you want to change the base?
Conversation
Thanks for the PR! Looks like a whole bunch of good improvements but i'll need to test them before merging. Have you tested this yourself with I'm hoping to strike a balance between a nice quick and dirty (but understandable) setup and something more comprehensive like github.com/heptio/ark |
I didn't tested git-crypt because the changes are unrelated to git. If you already have an environment for that I'd be glad if you try it. Actually the entire development was against an OpenShift cluster and I had to change a few more things because of that, but the subject of this PR is the resource name collection only. What I want to say is that I didn't setup a minikube, role, binding, cronjob, git-crypt keys, secret and git repo for this. So again, try it please, I tried only the kubectl commands, they worked fine. Git worked as well except git-crypt. Pretty much the whole script is running now in my OS cluster fine and what I did is replacing 'oc' with 'kubectl' (then try the commands if they behave like oc). If you try and it fails somehow just let me know and I'll setup the env. About heptio/ark it's up to you, but honestly I don't really see the point. I need backup for disaster recovery which should occur never or rare and for historical purposes; but not for recover frequently and easily. And copying objects to different environment I would use the original helm package. |
Sorry for letting this linger. I'm gonna make time to fully review this the coming week (starting tomorrow). |
No problem, take your time :) Actually the whole backup (querying at the first place) is slower than I expected first despite the cache variables. I think it can be speed up running one separate bash process for each resource type and collect the results. In my experiments querying 10 types parallel almost takes the same time as one type. |
…vent killing long run backups
Hey @janosroden and @pieterlange , any news on this getting merged upstream? I really like this way of declaring resources to be backed-up, most of us kubernetes admins and users would really appreciate being able to declare a bunch of them (like I really like the project as a whole, simple quick and dirty is exactly what many users are looking for, the huge projects backed-up by companies are not my sauce for basic tasks. Just wanted to know what's up here before typing my CRD's out, storing them as EDIT: after checking the commits, I see that resources cannot be "glob pattern selected". I think that would be great for operators and other complex meshed resources. |
I found kube-backup project as an excellent start to backup my OpenShift cluster and I want to merge back this improvement.
Unfortunately this is a breaking change but I think it worth it.
Changes: