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

chart: add chart files for helm-install #254

Merged
merged 4 commits into from
Sep 27, 2021

Conversation

xmh19936688
Copy link
Contributor

@xmh19936688 xmh19936688 commented Sep 13, 2021

Copy link

@megaeasex megaeasex left a comment

Choose a reason for hiding this comment

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

[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 254 Deploy Test Success

@zhao-kun
Copy link
Collaborator

zhao-kun commented Sep 13, 2021

This helm chart is dedicated to the K8s ingress controller of the Easegress. As Easegress can be used in different scenarios, I think you'd better explicitly explain the intention in the chart (including use more clear names or add some comments, etc...) for avoiding confusing people.

@localvar
Copy link
Collaborator

localvar commented Sep 14, 2021

Please rename the chart folder as the name is too general and may be used for another purpose later.

Copy link

@megaeasex megaeasex left a comment

Choose a reason for hiding this comment

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

[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 254 Deploy Test Success

Copy link

@megaeasex megaeasex left a comment

Choose a reason for hiding this comment

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

[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 254 Deploy Test Success

@@ -0,0 +1,4 @@
apiVersion: v2
name: easegress-controller
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
name: easegress-controller
name: easegress-ingress-controller

Comment on lines 1 to 5
# easegress ingress-controller chart

Use for easegress ingress controller installation by helm on kubernetes.

## usage
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# easegress ingress-controller chart
Use for easegress ingress controller installation by helm on kubernetes.
## usage
# Easegress ingress controller chart
Helm charts for install Easegress ingress controller on Kubernetes.
## Usage

Copy link

@megaeasex megaeasex left a comment

Choose a reason for hiding this comment

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

[TASK:easegress-pr-test SUCCESS]megaease/easegress Pull Request 254 Deploy Test Success

@benja-wu benja-wu added the dependencies Pull requests that update a dependency file label Sep 27, 2021
@benja-wu benja-wu added this to the v1.4.0 milestone Sep 27, 2021
Copy link
Contributor

@benja-wu benja-wu left a comment

Choose a reason for hiding this comment

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

LGTM

@benja-wu benja-wu merged commit 34e88fa into easegress-io:main Sep 27, 2021
@localvar localvar modified the milestones: v1.4.0, v1.3.0 Oct 13, 2021
@xmh19936688 xmh19936688 deleted the main-chart branch September 16, 2022 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants