-
Notifications
You must be signed in to change notification settings - Fork 104
Open
Labels
kind/enhancementEnhancement, improvement, extensionEnhancement, improvement, extensionkind/epicLarge multi-story topicLarge multi-story topiclifecycle/staleNobody worked on this for 6 months (will further age)Nobody worked on this for 6 months (will further age)
Description
How to categorize this issue?
/kind enhancement
What would you like to be added
Do a cleanup of old code which were added to handle some backward compatibility of features/edgeCases which are no longer valid/applicable.
- [Enhancement] Drop support of passing credentials to etcd-backup-restore through a JSON file #741
- Handling of this failBelowRevision should now be removed from backup-restore, I don't remember when I used this last time.
etcd-backup-restore/pkg/server/httpAPI.go
Line 227 in 43a44d3
failBelowRevisionStr := req.URL.Query().Get("failbelowrevision") - Tests of
Compact
sub-command handling ofv1
directory should also be removed.
Motivation (Why is this needed?)
To remove unnecessary handling of old cases and old features.
Acceptance Criteria
- Shouldn't break the current working of backup-restore.
Resources (optional)
- Contact
- Link
Metadata
Metadata
Assignees
Labels
kind/enhancementEnhancement, improvement, extensionEnhancement, improvement, extensionkind/epicLarge multi-story topicLarge multi-story topiclifecycle/staleNobody worked on this for 6 months (will further age)Nobody worked on this for 6 months (will further age)