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

CHE-10590: Create image to build machine exec for terminal-extension and for task api extension. #10812

Closed
wants to merge 5 commits into from

Conversation

AndrienkoAleksandr
Copy link
Contributor

What does this PR do?

Create image to build machine exec for terminal-extension and for task api extension.

What issues does this PR fix or reference?

#10590

Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
Signed-off-by: Oleksandr Andriienko <oandriie@redhat.com>
@benoitf benoitf added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. labels Aug 16, 2018
@AndrienkoAleksandr AndrienkoAleksandr requested a review from a user August 17, 2018 07:27
@sunix
Copy link
Contributor

sunix commented Aug 17, 2018

nice, few suggestions for README file:

  • explain how to configure a workspace with it
  • explain with a sample of code how a client would use it.
  • add few architecture note, maybe a sequence diagram, requirements, and protocol used, etc ...

ENV MACHINE_EXEC_HASH b57b40eb2088271dc24f538f224f8c1ac474a777
ENV MACHINE_EXEC github.com/eclipse/che-theia-terminal-plugin/machine-exec-server

RUN yum -y update && yum install -y git && git --version && \
Copy link
Contributor

Choose a reason for hiding this comment

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

" git --version "
Do we really need it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch, I will fix it.

@AndrienkoAleksandr
Copy link
Contributor Author

Current pull request will be moved to the another repo (it will called che-machine-exec) #10817

@AndrienkoAleksandr
Copy link
Contributor Author

AndrienkoAleksandr commented Aug 21, 2018

Close pull request on favor eclipse-che/che-machine-exec#1 . Repository https://github.com/eclipse/che-machine-exec contains own CI to build docker image.

@skabashnyuk skabashnyuk deleted the CHE-10590 branch February 11, 2019 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants