feat: helmcharts to deploy Eclipse Che #1184
Conversation
tolusha
requested review from
AndrienkoAleksandr,
flacatus,
mmorhun and
nickboldt
as code owners
helmcharts/Chart.yaml
Outdated
| # | ||
|
|
||
| apiVersion: v1 | ||
| description: A Helm chart for deploying Eclipse Che to Kubernetes |
There was a problem hiding this comment.
| description: A Helm chart for deploying Eclipse Che to Kubernetes | |
| description: A Helm chart for deploying Eclipse Che on a Kubernetes |
helmcharts/README.md
Outdated
|
|
||
| ### Helm Chart Install | ||
|
|
||
| Create a Namespace and install the helm chart for Eclipse Che. |
There was a problem hiding this comment.
| Create a Namespace and install the helm chart for Eclipse Che. | |
| Create a Namespace and install the helm chart for Eclipse Che Operator. |
helmcharts/README.md
Outdated
|
|
||
| ### Helm Versions | ||
|
|
||
| Helm charts are only tested with Helm version 3.7.1 |
There was a problem hiding this comment.
I think we should try and detect minimal version and put something like 3.x.y+ here.
helmcharts/README.md
Outdated
| - [Charts](#charts) | ||
| - [Kubernetes Versions](#kubernetes-versions) | ||
| - [Helm Versions](#helm-versions) | ||
| - [Helm Chart Install](#helm-chart-install) |
There was a problem hiding this comment.
Helm Chart Installation or Install the Helm Chart
helmcharts/README.md
Outdated
|
|
||
| - [Charts](#charts) | ||
| - [Kubernetes Versions](#kubernetes-versions) | ||
| - [Helm Versions](#helm-versions) |
helmcharts/README.md
Outdated
| # Eclipse Che Helm Charts | ||
|
|
||
| - [Charts](#charts) | ||
| - [Kubernetes Versions](#kubernetes-versions) |
There was a problem hiding this comment.
Required Kubernetes version?
| @@ -0,0 +1,33 @@ | |||
| # Eclipse Che Helm Charts | |||
|
|
|||
| - [Charts](#charts) | |||
There was a problem hiding this comment.
To me none of the headlines was clear. Maybe we should have:
Title
- Prerequisites (K8s & Helm versions)
- Installation
Signed-off-by: Anatolii Bazko <[email protected]>
Signed-off-by: Anatolii Bazko <[email protected]>
Signed-off-by: Anatolii Bazko <[email protected]>
Signed-off-by: Anatolii Bazko <[email protected]>
|
@mmorhun |
|
/retest |
|
@tolusha: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: AndrienkoAleksandr, mmorhun, tolusha The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Anatolii Bazko [email protected]
What does this PR do?
Introduce Helm charts to deploy Eclipse Che
Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
eclipse-che/che#20552
How to test this PR?
See helmcharts/README.md
PR Checklist
As the author of this Pull Request I made sure that:
What issues does this PR fix or referenceandHow to test this PRcompletedReviewers
Reviewers, please comment how you tested the PR when approving it.