Update README.md

review comments for README update w/ ascii diagram

Author: jayunit100

README.md

@@ -1,6 +1,6 @@
 # CSI Proxy
 
-CSI Proxy (which might be more aptly named "csi-node-proxy") is a binary that exposes a set of gRPC APIs around local storage operations for nodes in Windows. A container, such as CSI node plugins, can mount the named pipes depending on operations it wants to exercise on the host and
+CSI Proxy (which might be more aptly named "csi-node-proxy") is a binary that exposes a set of gRPC APIs (over named pipes) around local storage operations for nodes in Windows. A container, such as CSI node plugins, can mount the named pipes depending on operations it wants to exercise on the host and
 invoke the APIs.  This allows a storage plugin to run as if were a CSI plugin on linux which have the ability to perform "privileged" actions on the windows host OS.
 
 ```
@@ -11,11 +11,12 @@ invoke the APIs.  This allows a storage plugin to run as if were a CSI plugin on
 |                 |driver    +---------------->                  |
 |                 |registrar |                |                  |
 | NodeStageVolume ----^------+                ----------+--------+
-| NodeUnstageVol      |                                 |  Disk APIs
-| NodePublishVol      |GetNodeInfo                      |  Volume APIs
-| NodeResizeVol       |NodeRegistrationStatus    /pipe  |  SMB APIs
-| NodeGetInfo         |                                 |  FileSystem/System APIs
-| NodeGetVolStats     | csi-plugin-reg.sock   +---------v---------+
+| NodeUnstageVol      |                                   |  Disk APIs
+| NodePublishVol      |GetNodeInfo                        |  Volume APIs
+| NodeResizeVol       |NodeRegistrationStatus  \\.\pipe\  |  SMB APIs
+| NodeGetInfo         |                                   |  File System operations
+|                     |                                   |  Windows System APIs
+| NodeGetVolStats     | csi-plugin-reg.sock   +-----------v---------+
 |                     |                       |                   |
 |                 + --+--------+              |    csi-proxy.exe  |
 |                 | kubelet.exe|              +-------------------+
@@ -31,9 +32,9 @@ invoke the APIs.  This allows a storage plugin to run as if were a CSI plugin on
 ```
 In the above diagram, there are 3 communication channels being utilized for CSI proxy:
 
-- `csi.sock` : This lives at /var/lib/kubelet/plugins/csi-plugin/csi.sock, and is used for the host OS to communicate with the CSI node plugin
-- `csi-plugin-reg.sock` : This lives here, /var/lib/kubelet/plugins_registry/csi-plugin-reg.sock, and is used for the kubelet itself to get metadata as needed from the csi plugin  
-- `/pipe` : /pipe/csi-proxy-api-group-v-... : This pipe lives at \pipe\csi-proxy-api-group-v-..., upon receipt of specific requests from the Node plugin, it carries out actions on the node plugins behalf.  Since it doesn't execute arbitrary commands on the host, its more secure then simply being a black-box proxy to run host commands.
+- `csi.sock` : This lives at c:/var/lib/kubelet/plugins/csi-plugin/csi.sock, and is used for the host OS to communicate with the CSI node plugin
+- `csi-plugin-reg.sock` : This lives here, c:/var/lib/kubelet/plugins_registry/csi-plugin-reg.sock, and is used for the kubelet itself to get metadata as needed from the csi plugin .
+- `\\.\pipe\csi-proxy` : This will live in a pipe, i.e. at \\.\pipe/csi-proxy-api-group-v-... : upon receipt of specific requests from the Node plugin, it carries out actions on the node plugins behalf.  Since it doesn't execute arbitrary commands on the host, its more secure then simply being a black-box proxy to run host commands.
 
 Each named pipe will support a specific version of an API (e.g. v1alpha1, v2beta1)
 that targets a specific area of storage (e.g. disk, volume, file, SMB, iSCSI).
@@ -43,7 +44,6 @@ that targets a specific area of storage (e.g. disk, volume, file, SMB, iSCSI).
 CSI drivers are recommended to be deployed as containers. Node plugin containers need to run with privileges to perform storage related operations. However, Windows does not support privileged containers currently. With CSIProxy, the node plugins can now be deployed as unprivileged pods that use the proxy to perform privileged storage operations on the node. Kubernetes administrators will need to install and maintain csi-proxy.exe on all Windows nodes in a manner similar to kubelet.exe.
 
 ## Compatibility
-
 Recommended K8s Version: 1.18
 
 ## Feature status