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

Some commands that have worked through bash shell, can work wrong (regression) #2811

Closed
musienko-maxim opened this issue Oct 18, 2016 · 3 comments
Assignees
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code.
Milestone

Comments

@musienko-maxim
Copy link
Contributor

musienko-maxim commented Oct 18, 2016

Reproduction Steps:

After setup of sh as default shell in the Codenvy - some commands that was saved early will be wrong. For example some workspace used command like: echo -e 'some text for check' > ~/check.txt.

Expected behavior:
After execution the command - should be created the check.txt file with next content: some text for check

Observed behavior:
The content in the file will be like: -e some text for check (-e flag will be added into file)

This problem may do many unexpected results for commands, that were saved in the workspaces created earlier. The same problem can be in the factories, which use bash commands. Many users may be puzzled.

Che version: 5.0.0-M6-SNAPSHOT
OS and version: Ubuntu 14.04 LTS
Docker version: 1.12.2
Che install: Docker container

@musienko-maxim musienko-maxim added the kind/bug Outline of a bug - must adhere to the bug report template. label Oct 18, 2016
@bmicklea bmicklea added the severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code. label Oct 18, 2016
@bmicklea bmicklea added this to the 5.0.0-M6 milestone Oct 18, 2016
@JamesDrummond
Copy link
Contributor

Add current-sprint label.

@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. sprint/current and removed sprint/current status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Oct 19, 2016
@benoitf
Copy link
Contributor

benoitf commented Oct 20, 2016

@musienko-maxim please check with next nightly

@musienko-maxim
Copy link
Contributor Author

The test (LoginBySshTest) that covers this feature - passed. I think issue can be closed.

@benoitf benoitf closed this as completed Oct 24, 2016
@bmicklea bmicklea mentioned this issue Oct 28, 2016
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/blocker Causes system to crash and be non-recoverable or prevents Che developers from working on Che code.
Projects
None yet
Development

No branches or pull requests

4 participants