Skip to content

Conversation

@tolusha
Copy link
Contributor

@tolusha tolusha commented Mar 17, 2020

Signed-off-by: Anatoliy Bazko abazko@redhat.com

What does this PR do?

Removes deploy.sh ocp.sh scripts as deprecated.

What issues does this PR fix or reference?

#13888

Signed-off-by: Anatoliy Bazko <abazko@redhat.com>
@tolusha tolusha requested a review from skabashnyuk March 17, 2020 09:03
@che-bot che-bot 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 Mar 17, 2020
@che-bot
Copy link
Contributor

che-bot commented Mar 17, 2020

✅ E2E Happy path tests succeed 🎉

See Details

Tested with Eclipse Che Multiuser User on K8S (minikube v1.1.1)

@tolusha tolusha merged commit ca5fc35 into master Mar 17, 2020
@tolusha tolusha deleted the ab/removedeploysh branch March 17, 2020 09:49
@che-bot che-bot added this to the 7.11.0 milestone Mar 17, 2020
@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 Mar 17, 2020
@ibuziuk
Copy link
Member

ibuziuk commented Apr 15, 2020

@tolusha could you clarify why the script was removed, but all the templates are still there?

@tolusha
Copy link
Contributor Author

tolusha commented Apr 15, 2020

@ibuziuk
I was not sure about templates.
I assumed that they are used somewhere besides deploy.sh script

@ibuziuk
Copy link
Member

ibuziuk commented Apr 15, 2020

AFAIK no cc: @skabashnyuk

@skabashnyuk
Copy link
Contributor

yeh. this part https://github.com/eclipse/che/tree/master/deploy/openshift/templates/monitoring is still used. We are going to replace it with operator logic soon.

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants