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.

Sign up for GitHub

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

Jump to bottom

CLI: sync command does not work #1564

Closed
slemeur opened this issue Jan 19, 2017 · 1 comment · Fixed by #1569
Closed

CLI: sync command does not work #1564

slemeur opened this issue Jan 19, 2017 · 1 comment · Fixed by #1569
Assignees
@benoitf
Labels
kind/bug severity/P1

Comments

@slemeur
Copy link
Contributor

slemeur commented Jan 19, 2017

Anytime trying the sync command, it gives the following output:
ERROR: You passed an unknown command.

I used the following command:
docker run -it -v /var/run/docker.sock:/var/run/docker.sock -v /Users/stevanlemeur/dev/codenvy:/data -v /Users/stevanlemeur/Desktop/local-sync:/sync codenvy/cli sync wksp-vh1c --url https://codenvy.io/ --user slemeur@codenvy.com --password ******

By running the same command, but using eclispe/che CLI, it worked properly:
docker run -it -v /var/run/docker.sock:/var/run/docker.sock -v /Users/stevanlemeur/dev/codenvy:/data -v /Users/stevanlemeur/Desktop/local-sync:/sync eclipse/che sync wksp-vh1c --url https://codenvy.io/ --user slemeur@codenvy.com --password codenvy555

I think it's related to this, where we have not updated mountby sync here:
https://github.com/codenvy/codenvy/blob/master/dockerfiles/cli/scripts/cli.sh#L25

Codenvy version: 5.1.0
@benoitf benoitf self-assigned this Jan 19, 2017
benoitf added a commit that referenced this issue Jan 20, 2017
@benoitf
Fix #1564 : Invalid name for sync command
5f03253 
It was still "mount" while command has been renamed to "sync"

Change-Id: I5c0cafcf358db2a753a8b0a675ae58b4a2f69358
Signed-off-by: Florent BENOIT <fbenoit@codenvy.com>
@benoitf benoitf mentioned this issue Jan 20, 2017
Merged
benoitf added a commit that referenced this issue Jan 23, 2017
@benoitf
Fix #1564 : Invalid name for sync command (#1569)
1617458 
It was still "mount" while command has been renamed to "sync"

Change-Id: I5c0cafcf358db2a753a8b0a675ae58b4a2f69358
Signed-off-by: Florent BENOIT <fbenoit@codenvy.com>
@benoitf
Copy link
Contributor

benoitf commented Jan 23, 2017

Fixed by #1569

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@benoitf benoitf

Labels
kind/bug severity/P1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[cli] Fix #1564 : Invalid name for sync command codenvy/codenvy
2 participants
@benoitf @slemeur