Skip to content
This repository has been archived by the owner on Oct 21, 2020. It is now read-only.

fix docker file error #215

Merged
merged 1 commit into from
Jul 4, 2017
Merged

fix docker file error #215

merged 1 commit into from
Jul 4, 2017

Conversation

wjin
Copy link
Contributor

@wjin wjin commented Jul 4, 2017

the deployment docker image got error:

chmod: invalid mode: 'x+o'

Signed-off-by: Wei Jin wjin.cn@gmail.com

the deployment docker image got error:

chmod: invalid mode: 'x+o'

Signed-off-by: Wei Jin <wjin.cn@gmail.com>
@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.


Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Jul 4, 2017
@wjin
Copy link
Contributor Author

wjin commented Jul 4, 2017

please review and update the docker image if possible, thanks.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Jul 4, 2017
@wongma7 wongma7 added area/ceph/cephfs lgtm Indicates that a PR is ready to be merged. labels Jul 4, 2017
@wongma7
Copy link
Contributor

wongma7 commented Jul 4, 2017

I think the deployment.yaml is still broken btw, because CMD is just chmod. I will let @rootfs look at it when he gets back

@wongma7 wongma7 merged commit 334cb3c into kubernetes-retired:master Jul 4, 2017
@wjin
Copy link
Contributor Author

wjin commented Jul 5, 2017

Yes, you are right.
For docker run, it will be ok as we append cephfs_provisioner command in the end.
For deployment, maybe he added local-start.sh to Dockerfile when building the base image: "quay.io/external_storage/cephfs-provisioner:latest".

yangruiray pushed a commit to yangruiray/external-storage that referenced this pull request Jul 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/ceph/cephfs cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants