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

Remove outdated Che6 concept. Devfile should be used instead #14383

Merged
merged 1 commit into from
Sep 10, 2019

Conversation

skabashnyuk
Copy link
Contributor

What does this PR do?

Removes Chefile

What issues does this PR fix or reference?

Avoid user confusion about how chefile should be used in Che7

Release Notes

n/a

Docs PR

n/a

@che-bot che-bot added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Aug 29, 2019
@che-bot
Copy link
Contributor

che-bot commented Aug 29, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

  • build details
  • "che-server" docker image: maxura/che-server:14383

@che-bot
Copy link
Contributor

che-bot commented Aug 29, 2019

E2E tests of Eclipse Che Multiuser on OCP has been successful:

@skabashnyuk
Copy link
Contributor Author

@nickboldt @l0rd @rhopp @vparfonov please upvote

Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

let's remove it from master 👍

@skabashnyuk skabashnyuk merged commit c00828c into master Sep 10, 2019
@skabashnyuk skabashnyuk deleted the cleanupchefile branch September 10, 2019 07:43
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 10, 2019
@che-bot che-bot added this to the 7.2.0 milestone Sep 10, 2019
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.

3 participants