From 152eeb1d1401945fc5e57298ba2c4185572431bf Mon Sep 17 00:00:00 2001 From: Cyrill Troxler Date: Tue, 14 May 2019 21:53:44 +0200 Subject: [PATCH] Update README and deployment, split up dockerfile --- README.md | 108 +++++++++++++++++++--------- cmd/s3driver/Dockerfile | 65 +++-------------- cmd/s3driver/Dockerfile.full | 68 ++++++++++++++++++ deploy/kubernetes/attacher.yaml | 2 +- deploy/kubernetes/csi-s3.yaml | 9 +-- deploy/kubernetes/pod.yaml | 1 + deploy/kubernetes/provisioner.yaml | 9 +-- deploy/kubernetes/secret.yaml | 6 +- deploy/kubernetes/storageclass.yaml | 10 ++- pkg/s3/s3-driver.go | 2 +- 10 files changed, 168 insertions(+), 112 deletions(-) create mode 100644 cmd/s3driver/Dockerfile.full diff --git a/README.md b/README.md index 1783ac9..4d57413 100644 --- a/README.md +++ b/README.md @@ -1,17 +1,23 @@ # CSI for S3 + This is a Container Storage Interface ([CSI](https://github.com/container-storage-interface/spec/blob/master/spec.md)) for S3 (or S3 compatible) storage. This can dynamically allocate buckets and mount them via a fuse mount into any container. -# Status +## Status + This is still very experimental and should not be used in any production environment. Unexpected data loss could occur depending on what mounter and S3 storage backend is being used. -# Kubernetes installation -## Requirements -* Kubernetes 1.10+ +## Kubernetes installation + +### Requirements + +* Kubernetes 1.13+ * Kubernetes has to allow privileged containers * Docker daemon must allow shared mounts (systemd flag `MountFlags=shared`) -## 1. Create a secret with your S3 credentials +### 1. Create a secret with your S3 credentials + The endpoint is optional if you are using something else than AWS S3. Also the region can be empty if you are using some other S3 compatible storage. + ```yaml apiVersion: v1 kind: Secret @@ -20,7 +26,7 @@ metadata: stringData: accessKeyID: secretAccessKey: - # For AWS set it to "https://s3.amazonaws.com" + # For AWS set it to "https://s3..amazonaws.com" endpoint: # If not on S3, set it to "" region: @@ -29,47 +35,60 @@ stringData: encryptionKey: ``` -## 2. Deploy the driver +### 2. Deploy the driver + ```bash cd deploy/kubernetes -$ kubectl create -f provisioner.yaml -$ kubectl create -f attacher.yaml -$ kubectl create -f csi-s3.yaml +kubectl create -f provisioner.yaml +kubectl create -f attacher.yaml +kubectl create -f csi-s3.yaml ``` -## 3. Create the storage class +### 3. Create the storage class + ```bash -$ kubectl create -f storageclass.yaml +kubectl create -f storageclass.yaml ``` -## 4. Test the S3 driver +### 4. Test the S3 driver + * Create a pvc using the new storage class: + ```bash -$ kubectl create -f pvc.yaml +kubectl create -f pvc.yaml ``` + * Check if the PVC has been bound: + ```bash $ kubectl get pvc csi-s3-pvc NAME STATUS VOLUME CAPACITY ACCESS MODES STORAGECLASS AGE csi-s3-pvc Bound pvc-c5d4634f-8507-11e8-9f33-0e243832354b 5Gi RWX csi-s3 9s ``` + * Create a test pod which mounts your volume: + ```bash -$ kubectl create -f poc.yaml +kubectl create -f poc.yaml ``` + If the pod can start, everything should be working. * Test the mount + ```bash $ kubectl exec -ti csi-s3-test-nginx bash $ mount | grep fuse s3fs on /var/lib/www/html type fuse.s3fs (rw,nosuid,nodev,relatime,user_id=0,group_id=0,allow_other) $ touch /var/lib/www/html/hello_world ``` + If something does not work as expected, check the troubleshooting section below. -# Additional configuration -## Mounter +## Additional configuration + +### Mounter + As S3 is not a real file system there are some limitations to consider here. Depending on what mounter you are using, you will have different levels of POSIX compability. Also depending on what S3 storage backend you are using there are not always [consistency guarantees](https://github.com/gaul/are-we-consistent-yet#observed-consistency). The driver can be configured to use one of these mounters to mount buckets: @@ -84,22 +103,26 @@ The mounter can be set as a parameter in the storage class. You can also create All mounters have different strengths and weaknesses depending on your use case. Here are some characteristics which should help you choose a mounter: -### rclone +#### rclone + * Almost full POSIX compatibility (depends on caching mode) * Files can be viewed normally with any S3 client -### s3fs +#### s3fs + * Large subset of POSIX * Files can be viewed normally with any S3 client * Does not support appends or random writes -### goofys +#### goofys + * Weak POSIX compatibility * Performance first * Files can be viewed normally with any S3 client * Does not support appends or random writes -### s3ql +#### s3ql (not recommended*) + * (Almost) full POSIX compatibility * Uses its own object format * Files are not readable with other S3 clients @@ -107,7 +130,8 @@ All mounters have different strengths and weaknesses depending on your use case. * Supports compression before upload * Supports encryption before upload -### s3backer +#### s3backer (not recommended*) + * Represents a block device stored on S3 * Allows to use a real filesystem * Files are not readable with other S3 clients @@ -115,33 +139,47 @@ All mounters have different strengths and weaknesses depending on your use case. * Supports compression before upload (Not yet implemented in this driver) * Supports encryption before upload (Not yet implemented in this driver) +*s3ql and s3backer are not recommended at this point because volume corruption can occur pretty quickly in case of an unexpected shutdown of a Kubernetes node or CSI pod. + Fore more detailed limitations consult the documentation of the different projects. -# Troubleshooting -## Issues while creating PVC +## Troubleshooting + +### Issues while creating PVC + * Check the logs of the provisioner: -``` -$ kubectl logs -l app=csi-provisioner-s3 -c csi-s3 + +```bash +kubectl logs -l app=csi-provisioner-s3 -c csi-s3 ``` -## Issues creating containers +### Issues creating containers + * Ensure feature gate `MountPropagation` is not set to `false` * Check the logs of the s3-driver: -``` -$ kubectl logs -l app=csi-s3 -c csi-s3 + +```bash +kubectl logs -l app=csi-s3 -c csi-s3 ``` -# Development +## Development + This project can be built like any other go application. + ```bash -$ go get -u github.com/ctrox/csi-s3 +go get -u github.com/ctrox/csi-s3 ``` -## Build + +### Build executable + ```bash -$ make build +make build ``` -## Tests + +### Tests + Currently the driver is tested by the [CSI Sanity Tester](https://github.com/kubernetes-csi/csi-test/tree/master/pkg/sanity). As end-to-end tests require S3 storage and a mounter like s3fs, this is best done in a docker container. A Dockerfile and the test script are in the `test` directory. The easiest way to run the tests is to just use the make command: + ```bash -$ make test +make test ``` diff --git a/cmd/s3driver/Dockerfile b/cmd/s3driver/Dockerfile index f6c94f9..3579a68 100644 --- a/cmd/s3driver/Dockerfile +++ b/cmd/s3driver/Dockerfile @@ -1,68 +1,19 @@ -FROM python:3.6 as s3ql-deps +FROM debian:stretch LABEL maintainers="Cyrill Troxler " -LABEL description="s3ql dependencies" +LABEL description="csi-s3 slim image" RUN apt-get update && \ - apt-get install -y \ - python3 python3-setuptools \ - python3-dev python3-pip pkg-config cython \ - libfuse-dev libattr1-dev && \ - rm -rf /var/lib/apt/lists/* - -RUN pip3 install llfuse apsw defusedxml dugong requests pycrypto - -FROM debian:stretch as s3backer -ARG S3BACKER_VERSION=1.5.0 - -RUN apt-get update && apt-get install -y \ - build-essential \ - autoconf \ - libcurl4-openssl-dev \ - libfuse-dev \ - libexpat1-dev \ - libssl-dev \ - zlib1g-dev \ - psmisc \ - pkg-config \ - git && \ + apt-get install -y \ + s3fs wget unzip && \ rm -rf /var/lib/apt/lists/* -# Compile & install s3backer -RUN git clone https://github.com/archiecobbs/s3backer.git /src/s3backer -WORKDIR /src/s3backer -RUN git checkout tags/${S3BACKER_VERSION} - -RUN ./autogen.sh && \ - ./configure && \ - make && \ - make install - -FROM python:3.6-slim -LABEL maintainers="Cyrill Troxler " -LABEL description="csi-s3 production image" - -RUN apt-get update && \ - apt-get install -y \ - libfuse2 gcc sqlite3 libsqlite3-dev \ - s3fs psmisc procps libcurl3 xfsprogs wget unzip && \ - rm -rf /var/lib/apt/lists/* - -ARG S3QL_VERSION=2.29 -ENV S3QL_URL=https://github.com/s3ql/s3ql/releases/download/release-${S3QL_VERSION}/s3ql-${S3QL_VERSION}.tar.bz2 - -COPY --from=s3ql-deps /root/.cache /root/.cache -COPY --from=s3ql-deps /usr/local/lib/python3.6/site-packages /usr/local/lib/python3.6/site-packages -RUN pip install ${S3QL_URL} && rm -rf /root/.cache - -COPY --from=s3backer /usr/bin/s3backer /usr/bin/s3backer - # install rclone ARG RCLONE_VERSION=v1.46 RUN cd /tmp \ - && wget -q https://downloads.rclone.org/${RCLONE_VERSION}/rclone-${RCLONE_VERSION}-linux-amd64.zip \ - && unzip /tmp/rclone-${RCLONE_VERSION}-linux-amd64.zip \ - && mv /tmp/rclone-*-linux-amd64/rclone /usr/bin \ - && rm -r /tmp/rclone* + && wget -q https://downloads.rclone.org/${RCLONE_VERSION}/rclone-${RCLONE_VERSION}-linux-amd64.zip \ + && unzip /tmp/rclone-${RCLONE_VERSION}-linux-amd64.zip \ + && mv /tmp/rclone-*-linux-amd64/rclone /usr/bin \ + && rm -r /tmp/rclone* COPY ./_output/s3driver /s3driver ENTRYPOINT ["/s3driver"] diff --git a/cmd/s3driver/Dockerfile.full b/cmd/s3driver/Dockerfile.full new file mode 100644 index 0000000..ef266c5 --- /dev/null +++ b/cmd/s3driver/Dockerfile.full @@ -0,0 +1,68 @@ +FROM python:3.6 as s3ql-deps +LABEL maintainers="Cyrill Troxler " +LABEL description="s3ql dependencies" + +RUN apt-get update && \ + apt-get install -y \ + python3 python3-setuptools \ + python3-dev python3-pip pkg-config cython \ + libfuse-dev libattr1-dev && \ + rm -rf /var/lib/apt/lists/* + +RUN pip3 install llfuse apsw defusedxml dugong requests pycrypto + +FROM debian:stretch as s3backer +ARG S3BACKER_VERSION=1.5.0 + +RUN apt-get update && apt-get install -y \ + build-essential \ + autoconf \ + libcurl4-openssl-dev \ + libfuse-dev \ + libexpat1-dev \ + libssl-dev \ + zlib1g-dev \ + psmisc \ + pkg-config \ + git && \ + rm -rf /var/lib/apt/lists/* + +# Compile & install s3backer +RUN git clone https://github.com/archiecobbs/s3backer.git /src/s3backer +WORKDIR /src/s3backer +RUN git checkout tags/${S3BACKER_VERSION} + +RUN ./autogen.sh && \ + ./configure && \ + make && \ + make install + +FROM python:3.6-slim +LABEL maintainers="Cyrill Troxler " +LABEL description="csi-s3 full image" + +RUN apt-get update && \ + apt-get install -y \ + libfuse2 gcc sqlite3 libsqlite3-dev \ + s3fs psmisc procps libcurl3 xfsprogs wget unzip && \ + rm -rf /var/lib/apt/lists/* + +ARG S3QL_VERSION=2.29 +ENV S3QL_URL=https://github.com/s3ql/s3ql/releases/download/release-${S3QL_VERSION}/s3ql-${S3QL_VERSION}.tar.bz2 + +COPY --from=s3ql-deps /root/.cache /root/.cache +COPY --from=s3ql-deps /usr/local/lib/python3.6/site-packages /usr/local/lib/python3.6/site-packages +RUN pip install ${S3QL_URL} && rm -rf /root/.cache + +COPY --from=s3backer /usr/bin/s3backer /usr/bin/s3backer + +# install rclone +ARG RCLONE_VERSION=v1.46 +RUN cd /tmp \ + && wget -q https://downloads.rclone.org/${RCLONE_VERSION}/rclone-${RCLONE_VERSION}-linux-amd64.zip \ + && unzip /tmp/rclone-${RCLONE_VERSION}-linux-amd64.zip \ + && mv /tmp/rclone-*-linux-amd64/rclone /usr/bin \ + && rm -r /tmp/rclone* + +COPY ./_output/s3driver /s3driver +ENTRYPOINT ["/s3driver"] diff --git a/deploy/kubernetes/attacher.yaml b/deploy/kubernetes/attacher.yaml index c08998b..5414597 100644 --- a/deploy/kubernetes/attacher.yaml +++ b/deploy/kubernetes/attacher.yaml @@ -67,7 +67,7 @@ spec: serviceAccount: csi-attacher-sa containers: - name: csi-attacher - image: quay.io/k8scsi/csi-attacher:v1.0.1 + image: quay.io/k8scsi/csi-attacher:v1.1.0 args: - "--v=4" - "--csi-address=$(ADDRESS)" diff --git a/deploy/kubernetes/csi-s3.yaml b/deploy/kubernetes/csi-s3.yaml index a84a2d8..69bbc5c 100644 --- a/deploy/kubernetes/csi-s3.yaml +++ b/deploy/kubernetes/csi-s3.yaml @@ -55,7 +55,7 @@ spec: hostNetwork: true containers: - name: driver-registrar - image: quay.io/k8scsi/csi-node-driver-registrar:v1.0.1 + image: quay.io/k8scsi/csi-node-driver-registrar:v1.1.0 args: - "--kubelet-registration-path=$(DRIVER_REG_SOCK_PATH)" - "--v=4" @@ -80,15 +80,10 @@ spec: capabilities: add: ["SYS_ADMIN"] allowPrivilegeEscalation: true - image: ctrox/csi-s3:1.0.1-alpha + image: ctrox/csi-s3:v1.1.0 args: - "--endpoint=$(CSI_ENDPOINT)" - "--nodeid=$(NODE_ID)" - - "--access-key-id=$(ACCESS_KEY_ID)" - - "--secret-access-key=$(SECRET_ACCESS_KEY)" - - "--s3-endpoint=$(S3_ENDPOINT)" - - "--region=$(REGION)" - - "--encryption-key=$(ENCRYPTION_KEY)" - "--v=4" env: - name: CSI_ENDPOINT diff --git a/deploy/kubernetes/pod.yaml b/deploy/kubernetes/pod.yaml index 23e4d4b..1e19c95 100644 --- a/deploy/kubernetes/pod.yaml +++ b/deploy/kubernetes/pod.yaml @@ -2,6 +2,7 @@ apiVersion: v1 kind: Pod metadata: name: csi-s3-test-nginx + namespace: default spec: containers: - name: csi-s3-test-nginx diff --git a/deploy/kubernetes/provisioner.yaml b/deploy/kubernetes/provisioner.yaml index 90d5ebe..fd7cc07 100644 --- a/deploy/kubernetes/provisioner.yaml +++ b/deploy/kubernetes/provisioner.yaml @@ -66,7 +66,7 @@ spec: serviceAccount: csi-provisioner-sa containers: - name: csi-provisioner - image: quay.io/k8scsi/csi-provisioner:v1.0.1 + image: quay.io/k8scsi/csi-provisioner:v1.1.0 args: - "--provisioner=ch.ctrox.csi.s3-driver" - "--csi-address=$(ADDRESS)" @@ -79,15 +79,10 @@ spec: - name: socket-dir mountPath: /var/lib/kubelet/plugins/ch.ctrox.csi.s3-driver - name: csi-s3 - image: ctrox/csi-s3:1.0.1-alpha + image: ctrox/csi-s3:v1.1.0 args: - "--endpoint=$(CSI_ENDPOINT)" - "--nodeid=$(NODE_ID)" - - "--access-key-id=$(ACCESS_KEY_ID)" - - "--secret-access-key=$(SECRET_ACCESS_KEY)" - - "--s3-endpoint=$(S3_ENDPOINT)" - - "--region=$(REGION)" - - "--encryption-key=$(ENCRYPTION_KEY)" - "--v=4" env: - name: CSI_ENDPOINT diff --git a/deploy/kubernetes/secret.yaml b/deploy/kubernetes/secret.yaml index 74564d4..033a872 100644 --- a/deploy/kubernetes/secret.yaml +++ b/deploy/kubernetes/secret.yaml @@ -5,10 +5,10 @@ metadata: stringData: accessKeyID: secretAccessKey: - # For AWS set it to "https://s3.amazonaws.com" - endpoint: + # For AWS set it to "https://s3..amazonaws.com" + endpoint: https://s3.eu-central-1.amazonaws.com # If not on S3, set it to "" region: # Currently only for s3ql # If not using s3ql, set it to "" - encryptionKey: + encryptionKey: "" diff --git a/deploy/kubernetes/storageclass.yaml b/deploy/kubernetes/storageclass.yaml index 10ace23..eca9192 100644 --- a/deploy/kubernetes/storageclass.yaml +++ b/deploy/kubernetes/storageclass.yaml @@ -7,4 +7,12 @@ provisioner: ch.ctrox.csi.s3-driver parameters: # specify which mounter to use # can be set to s3backer, s3ql, s3fs or goofys - mounter: s3backer + mounter: rclone + csi.storage.k8s.io/provisioner-secret-name: csi-s3-secret + csi.storage.k8s.io/provisioner-secret-namespace: kube-system + csi.storage.k8s.io/controller-publish-secret-name: csi-s3-secret + csi.storage.k8s.io/controller-publish-secret-namespace: kube-system + csi.storage.k8s.io/node-stage-secret-name: csi-s3-secret + csi.storage.k8s.io/node-stage-secret-namespace: kube-system + csi.storage.k8s.io/node-publish-secret-name: csi-s3-secret + csi.storage.k8s.io/node-publish-secret-namespace: kube-system \ No newline at end of file diff --git a/pkg/s3/s3-driver.go b/pkg/s3/s3-driver.go index 5431829..83f9bbb 100644 --- a/pkg/s3/s3-driver.go +++ b/pkg/s3/s3-driver.go @@ -40,7 +40,7 @@ type s3Volume struct { } var ( - vendorVersion = "1.0.1-alpha" + vendorVersion = "1.1.0" driverName = "ch.ctrox.csi.s3-driver" )