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

docs: add notes about deprecating VolumeSnapshot functions #2929

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

Conversation

hairyhum
Copy link
Contributor

@hairyhum hairyhum commented Jun 7, 2024

Change Overview

VolumeSnapshot functions are using provider specific code to manage snapshots.
Since CSI has better support for more providers, users should switch to CSI snapshotting functions instead.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test
  • 🏗️ Build

Test Plan

Follow docs generation steps in https://github.com/kanisterio/kanister/blob/master/BUILD.md#documentation

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

@julio-lopez julio-lopez removed their request for review June 7, 2024 19:55
VolumeSnapshot functions are using provider specific code to manage snapshots.
Since CSI has better support for more providers, users should switch to CSI
snapshotting functions instead.
@hairyhum hairyhum force-pushed the docs-deprecate-volume-snapshot branch from 6119d88 to dbd2bfc Compare June 7, 2024 21:48
@hairyhum hairyhum requested a review from pavannd1 August 21, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant