lvm-localpv icon indicating copy to clipboard operation
lvm-localpv copied to clipboard

chore(snapshot): adding the design doc for mountable snapshots

Open wowditi opened this issue 2 years ago • 5 comments

Pull Request template

Please, go through these steps before you submit a PR.

Why is this PR required? What issue does it fix?:

What this PR does?: It adds a design for a new feature

Does this PR require any upgrade changes?: No

If the changes in this PR are manually verified, list down the scenarios covered::

Any additional information for your reviewer? : In the drawbacks section I posed the question of whether we should maybe use annotation to indicate that we are mounting a snapshot instead of creating a new PV as a clone of the snapshot. I'd like a second opinion on whether this is a good idea or what a good alternative could be (for example, we could create a new api-resource although I don't think that would be such a good idea). Also a PVC requires the Storage to be set, however, it is a little bit useless in this case especially when mounting snapshots. Any suggestions on handling that better would be appreciated.

Checklist:

  • [ ] Fixes #
  • [x] PR Title follows the convention of <type>(<scope>): <subject>
  • [ ] Has the change log section been updated?
  • [ ] Commit has unit tests
  • [ ] Commit has integration tests
  • [ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • [ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

wowditi avatar Apr 06 '22 09:04 wowditi

@wowditi Can you please provide an update on the plans for taking this forward? Are you willing to work on this?

Abhinandan-Purkait avatar Sep 13 '23 06:09 Abhinandan-Purkait

Hi @wowditi I run Product Mgmt for the OpenEBS project. You've put a lot of amazingly impressive & detailed work into your PR and it received a bunch of good reviews from our ENG team. Overall very positive. LVM-localPV is now getting a major focus in the OpenEBS product as it's being unified into the core platform as a central component, rather than being managed as a separate external Data-Engine. Additionally, LVM-LocalPV now has ~50,000 Daily Active Users and +150,000 product installations. So it's now a very well validated part of the platform. Hence we're are unifying it into the main product.

We'd really like to work on developing your PR and integrate the design into the new unified product.

Are you still interested and willing to support your PR?

orville-wright avatar Mar 29 '24 11:03 orville-wright

Hi, Sorry I forgot about this since my company started using the ZFS variant since it already had the features we need. I do not have a lot of spare time currently, but I could probably help since it wasn't that much work from what I remember.

About the design document, the implementation details are indeed very technical currently and I can make those a bit more high level. After using the ZFS localpv I also noticed that this is basically the same behaviour as what they refer to as cloning, should I rewrite the document to call it cloning and remove the drawback?

PS: I should probably mention that I mainly code in python and not Golang, so it might not be the most beautiful code.

wowditi avatar Apr 02 '24 17:04 wowditi