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 deploy.sh #16376

Merged
merged 1 commit into from
Mar 17, 2020
Merged

Remove deploy.sh #16376

merged 1 commit into from
Mar 17, 2020

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>
@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.

None yet

5 participants