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

Do snapshots freeze fs before snapshotting ? #1532

Closed
facchettos opened this issue Sep 28, 2020 · 8 comments · Fixed by #1536
Closed

Do snapshots freeze fs before snapshotting ? #1532

facchettos opened this issue Sep 28, 2020 · 8 comments · Fixed by #1536
Labels
question Further information is requested

Comments

@facchettos
Copy link
Contributor

facchettos commented Sep 28, 2020

What new functionality do you want?

Just some doc update, I could not find anywhere in the docs, whether or not when you create a snapshot the fs/block is frozen or not before the snapshot is taken. In the alpha description of volume snapshots (kubernetes), they talked about having to get the fs/bd into a frozen state before doing the snapshot but it disappeared in beta. I wonder if cephcsi does that or if someone that create snapshots should first make sure that the volumes are not in use before doing the snapshot ?

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Sep 28, 2020

Yes, it's the user's responsibility to make sure PVC is not in use before taking snapshots or creating the clone. Hope in feature, kubernetes will add a hook to do options to freeze the IO when we are taking snapshots or clone

@Madhu-1 Madhu-1 added the question Further information is requested label Sep 28, 2020
@facchettos
Copy link
Contributor Author

facchettos commented Sep 28, 2020

Thanks !
Do you think it is worth it to update https://github.com/ceph/ceph-csi/blob/master/docs/cephfs-snap-clone.md#snapshot and possibly for rbd too or not ?

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Sep 28, 2020

Yes please, @facchettos is it possible to send PR to update the doc?

@facchettos
Copy link
Contributor Author

I'm on it =)

@Madhu-1
Copy link
Collaborator

Madhu-1 commented Sep 28, 2020

@humblec
Copy link
Collaborator

humblec commented Sep 28, 2020

@facchettos true , volume should not be in use while snapshot is taken, this has been mentioned as a prerequisties in kube. However there is no enforcement happening from the controller . Also refer kubernetes-csi/external-provisioner#437

@facchettos
Copy link
Contributor Author

facchettos commented Sep 28, 2020 via email

@humblec
Copy link
Collaborator

humblec commented Sep 28, 2020

Right, so the bigger point in the end is that ceph’s snapshots recommend the volume not to be in use while taking the snapshot ? ( https://docs.ceph.com/en/latest/rbd/rbd-snapshot/ )

On Mon, 28 Sep 2020 at 17:57, Humble Devassy Chirammal < @.***> wrote: @facchettos https://github.com/facchettos true , volume should not be in use while snapshot is taken, this has been mentioned as a prerequisties in kube. However there is no enforcement happening from the controller . Also refer kubernetes-csi/external-provisioner#437 <kubernetes-csi/external-provisioner#437> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1532 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFNKIPXSPUD3Z7HB2WEBLULSICW55ANCNFSM4R4SDPQQ .

correct, also ceph as a storage provider offers crash consistent snapshots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants