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

Introduce codenvy/che-test and codenvy/che-file docker images #1986

Merged
merged 1 commit into from
Jul 29, 2016

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Jul 28, 2016

What does this PR do?

It provides two docker images
codenvy/che-test
docker run --rm -v /var/run/docker.sock:/var/run/docker.sock codenvy/che-test <post-check> [URL of Che/Codenvy] [login] [password]
Which create a workspace, start it and display log

codenvy/che-file
still in wip with a run syntax close to
docker run -v /var/run/docker.sock:/var/run/docker.sock -v "$PWD":"$PWD" --rm codenvy/che-file $PWD <init|up>

Tests written?

No

@codenvy-ci
Copy link

@TylerJewell
Copy link

@benoitf - I was able to build the che-test image and it worked great. What should we do to get an initial commit merged in? By having an initial commit, we can then initiate CI build systems to get some images hosted for furtther testing. I'll finish, #1970 once these images are hosted.

// Project: https://github.com/jakearchibald/ES6-Promise
// Definitions by: François de Campredon <https://github.com/fdecampredon/>, vvakame <https://github.com/vvakame>
// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the origin of this file?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deleted check-in, will be part of only a compile time type dependencies

@benoitf benoitf changed the title [WIP] Adds codenvy/che-test and codenvy/che-file docker images Introduce codenvy/che-test and codenvy/che-file docker images Jul 29, 2016
@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jul 29, 2016
@benoitf benoitf self-assigned this Jul 29, 2016
COPY lib/typescript/bin /bin
ADD che-file/src/che.properties /lib/che.properties

ENTRYPOINT ["/bin/che", "che-file"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New line

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

return this.ip;
}

}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

newline

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@TylerJewell
Copy link

lgtm - congratulations!

@benoitf benoitf mentioned this pull request Jul 29, 2016
20 tasks
@benoitf
Copy link
Contributor Author

benoitf commented Jul 29, 2016

if review is OK I could then merge this PR and we can keep adding changes on them as ppl will be able to test it

@TylerJewell
Copy link

ok to merge

First one is codenvy/che-test allowing to check a local or remote Eclipse Che instance by creating a workspace, starting it and displaying console output
The other one is codenvy/che-file allowing to boot a local Eclipse Che by using a local directory. It will mount in Che this local folder as being a project.

Change-Id: Ie794b41f89608f86e589f1aeea06d63f0d9f2edc
Signed-off-by: Florent BENOIT <fbenoit@codenvy.com>
@benoitf benoitf merged commit bc9d96d into master Jul 29, 2016
@benoitf benoitf deleted the che-files branch July 29, 2016 16:23
@codenvy-ci
Copy link

Build # 1347 - FAILED

Please check console output at http://ci.codenvy-dev.com/jenkins/job/che-pullrequests-build/1347/ to view the results.

@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jan 23, 2017
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