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

Update bug template to specific the exact command to deploy Eclipse Che #16774

Merged
merged 4 commits into from
Apr 29, 2020

Conversation

tolusha
Copy link
Contributor

@tolusha tolusha commented Apr 27, 2020

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

What does this PR do?

Update bug template to specific the exact command to deploy Eclipse Che

Signed-off-by: Anatoliy Bazko <abazko@redhat.com>
@tolusha tolusha changed the title Update but template to specific the exact command to deploy Eclipse Che Update bug template to specific the exact command to deploy Eclipse Che Apr 27, 2020
@che-bot che-bot added kind/enhancement A feature request - must adhere to the feature request template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. labels Apr 27, 2020
@che-bot
Copy link
Contributor

che-bot commented Apr 27, 2020

❌ E2E Happy path tests failed ❗

See Details

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

⚠️ https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

ℹ️ Use comment "crw-ci-test" to rerun happy path E2E test.

@tolusha tolusha mentioned this pull request Apr 27, 2020
50 tasks
@tolusha tolusha requested a review from benoitf April 27, 2020 14:05
@@ -52,9 +52,9 @@ assignees: ''
<!-- If applicable, add screenshots to help explain your problem. -->

### Installation method
- [ ] chectl <!--(please specify the options of chectl tool) -->
- [ ] che-operator <!--(version of che-operator) -->
Copy link
Contributor

Choose a reason for hiding this comment

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

are you removing che-operator as an option?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

che-operator
because it is actually chectl with installer operator and it is better to know the full chectl command than a checkbox with che-operator

btw. added checkbox for olm way to install Che

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, that is what I meant: installation via olm was missing.

Signed-off-by: Anatoliy Bazko <abazko@redhat.com>
Signed-off-by: Anatoliy Bazko <abazko@redhat.com>
@che-bot
Copy link
Contributor

che-bot commented Apr 28, 2020

✅ E2E Happy path tests succeed 🎉

See Details

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

Co-Authored-By: Mario Loriedo <mario.loriedo@gmail.com>
@che-bot
Copy link
Contributor

che-bot commented Apr 28, 2020

✅ E2E Happy path tests succeed 🎉

See Details

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

@tolusha tolusha merged commit 417eb03 into master Apr 29, 2020
@tolusha tolusha deleted the ab/templates branch April 29, 2020 05:30
@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 Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants