kanisterio/kanister

Add CreateCSISnapshotAll, RestoreCSISnapshotAll and DeleteCSISnapshotAll functions

Open

#2,463 opened on Nov 7, 2023

View on GitHub
 (8 comments) (0 reactions) (0 assignees)Go (178 forks)auto 404
acceptedenhancementgood first issuehelp wanted

Repository metrics

Stars
 (883 stars)
PR merge metrics
 (PR metrics pending)

Description

For datastores with multiple replicas, having the "All" alternatives for CreateCSISnapshot, RestoreCSISnapshot and DeleteCSISnapshot is useful in order to:

  1. avoid phase duplication for each of the replicas
  2. address use cases where the number of replicas for the datastore can change in Day 2. Without the "All" alternative, a blueprint update is required with each scale
  3. have a single blueprint regardless of the setup of the datastore in various environments

Contributor guide

Research direction
Study the existing CreateCSISnapshot, RestoreCSISnapshot, and DeleteCSISnapshot functions in the Kanister codebase. Understand how they handle single replicas and then implement new functions that iterate over all replicas for a given datastore. Consider how to discover replicas and handle scaling changes.
Tech stack
go
Domain
backendinfrastructure
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
3-5 days
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GoKubernetesCSI
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40