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-5413 Remove unused agents related java code #5494

Merged
merged 4 commits into from
Jun 30, 2017

Conversation

sleshchenko
Copy link
Member

@sleshchenko sleshchenko commented Jun 29, 2017

What does this PR do?

Removes agents launchers
Moves che-core-api-agent and che-core-api-agent to wsmaster module
Renames Agent entity to Installer (code changes only, for renaming in json files and database will be created issue)
Refactors Installer API

What issues does this PR fix or reference?

#5413

Note that changes separated in different commits and it's much easier to review commits one by one.

@sleshchenko sleshchenko self-assigned this Jun 29, 2017
@garagatyi garagatyi added the target/branch Indicates that a PR will be merged into a branch other than master. label Jun 29, 2017
@garagatyi
Copy link

Thank you for splitting PR into purpose-based commits!

@codenvy-ci
Copy link

Build # 2943 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2943/ to view the results.

@codenvy-ci
Copy link

@sleshchenko sleshchenko merged commit 8e6e081 into eclipse-che:spi Jun 30, 2017
@sleshchenko sleshchenko deleted the CHE-5413 branch June 30, 2017 08:40
@skabashnyuk skabashnyuk mentioned this pull request Oct 15, 2017
@skabashnyuk skabashnyuk added this to the 6.0.0-M1 milestone Oct 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target/branch Indicates that a PR will be merged into a branch other than master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants