Skip to content
This repository has been archived by the owner on Mar 28, 2020. It is now read-only.

backup operator: enabled cluster wide operation. #2072

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

kapouille
Copy link
Contributor

Added cluster wide support to backup operator.

@etcd-bot
Copy link
Collaborator

etcd-bot commented Apr 3, 2019

Can one of the admins verify this patch?

2 similar comments
@etcd-bot
Copy link
Collaborator

etcd-bot commented Apr 3, 2019

Can one of the admins verify this patch?

@etcd-bot
Copy link
Collaborator

etcd-bot commented Apr 3, 2019

Can one of the admins verify this patch?

@hexfusion
Copy link
Member

@etcd-bot ok to test

@hexfusion
Copy link
Member

@kapouille please review gofmt error [1] in the unit test, please.

[1] https://jenkins-etcd-public.prod.coreos.systems/job/etcd-operator-unit-pr/1848/console

@hexfusion hexfusion added this to the v0.9.5 milestone Apr 3, 2019
@kapouille kapouille changed the title backup operator: enabled cluster wide operation. WIP: backup operator: enabled cluster wide operation. Apr 4, 2019
@kapouille
Copy link
Contributor Author

Apologies, this needs more work (not only the gofmt issue).

@kapouille
Copy link
Contributor Author

This should now be better. Had to fix an azure specific issue when handling very large backups.

@kapouille kapouille changed the title WIP: backup operator: enabled cluster wide operation. backup operator: enabled cluster wide operation. Apr 4, 2019
@hasbro17 hasbro17 self-requested a review May 6, 2019 05:42
@hasbro17
Copy link
Contributor

@kapouille Can you please rebase this PR as well. If you still have the time to work on this.

Added cluster wide support to backup operator.
Very large files were previously loaded in-memory (causing fatal crashes).
@kapouille
Copy link
Contributor Author

This should be fine now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants