-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Uploaders windows support #8569
Uploaders windows support #8569
Conversation
Signed-off-by: Lyndon-Li <lyonghui@vmware.com>
Signed-off-by: Lyndon-Li <lyonghui@vmware.com>
583ea25
to
c025bfd
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8569 +/- ##
=======================================
Coverage 59.17% 59.18%
=======================================
Files 370 370
Lines 39568 39591 +23
=======================================
+ Hits 23416 23433 +17
- Misses 14673 14677 +4
- Partials 1479 1481 +2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Lyndon-Li <lyonghui@vmware.com>
c025bfd
to
cb22dfc
Compare
@@ -434,6 +434,11 @@ func GetPVCAttachingNodeOS(pvc *corev1api.PersistentVolumeClaim, nodeClient core | |||
var nodeOS string | |||
var scFsType string | |||
|
|||
if pvc.Spec.VolumeMode != nil && *pvc.Spec.VolumeMode == corev1api.PersistentVolumeBlock { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we only check the volume mode when we cannot determine the OS via the annotation just like checking the file system type if scFsType == "ntfs"
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At present, Windows nodes doesn't support blockMode volumes. Or in another word, pods running in Windows nodes won't be with blockMode volumes.
Therefore, until Kubernetes supports it in a future release and Velero also supports blockMode on Windows nodes, we should always use linux nodes for blockMode volumes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What will happen when Kubernetes supports the Window block mode volumes in future? Although the PVC is attached to a Windows node, the "Linux" is returned as the OS, and the backup pod will run on the Linux node to handle the Windows block mode volumes, is this the expected behavior?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blockMode volumes are technically be backupable from either linux node or Windows node.
Running blockMode volumes backups on Windows nodes require further efforts.
Therefore, once Windows block mode volumes are supported, on the one hand, we can run all the backups on linux nodes with no efforts; one the other hand, as the ultimate solution, we will support Windows blockMode volumes backup.
fs uploader and block uploader support Windows nodes: