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

Fix CSI issue and upgrade to kubernetes 1.14.0 #272

Merged
merged 2 commits into from
May 8, 2019

Conversation

mavimo
Copy link
Collaborator

@mavimo mavimo commented Apr 7, 2019

This PR will update to kubernetes 1.14.0 and fix issue on CSI.

Actually tested with a new cluster, than create a secret with hetzner token:

cat >secret.yaml <<EOF
---
kind: Secret
apiVersion: v1
metadata:
  namespace: kube-system
  name: hcloud-csi
data:
  token: {ADD_TOKEN_BASE64_ENCODED_HERE}
EOF

and deploy all the required stuff:

kubectl apply -f https://raw.githubusercontent.com/kubernetes/csi-api/release-1.13/pkg/crd/manifests/csidriver.yaml
kubectl apply -f https://raw.githubusercontent.com/kubernetes/csi-api/release-1.13/pkg/crd/manifests/csinodeinfo.yaml
kubectl apply -f https://gist.githubusercontent.com/mavimo/399d9f783a59653bac71c8ddbf886757/raw/bc9ff2bcc6c38fea27f2586ce5da0cfa2a6e004e/hetzner-csi.yaml

at the end create a new pod using the PV, eg:

cat >test-csi.yaml <<EOF
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: csi-pvc
spec:
  accessModes:
  - ReadWriteOnce
  resources:
    requests:
      storage: 10Gi
  storageClassName: hcloud-volumes
---
kind: Pod
apiVersion: v1
metadata:
  name: my-csi-app
spec:
  containers:
    - name: my-frontend
      image: busybox
      volumeMounts:
      - mountPath: "/data"
        name: my-csi-volume
      command: [ "sleep", "1000000" ]
  volumes:
    - name: my-csi-volume
      persistentVolumeClaim:
        claimName: csi-pvc
EOF;

and deploy it:

kubectl apply -f test-csi.yaml

that's work without issue on logs (I inspected with journalctl -u kubelet on master and worker nodes).

@mavimo
Copy link
Collaborator Author

mavimo commented Apr 7, 2019

refs #259

@md2k @suchwerk can you check and let me know if it work also on your case or you still see any issue on it? Thx!

@suchwerk
Copy link

suchwerk commented Apr 7, 2019

Cant test it right now, but looks pretty good.

@BitPhinix
Copy link

Tested it, works like a charm

@mavimo mavimo requested a review from xetys April 15, 2019 13:52
@mavimo mavimo self-assigned this Apr 15, 2019
pkg/clustermanager/configs.go Show resolved Hide resolved
@xetys xetys added this to the 0.5 milestone May 8, 2019
@xetys xetys merged commit 7cfc664 into xetys:master May 8, 2019
@xetys xetys mentioned this pull request May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants