You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/05/09 09:32:22 UTC

[GitHub] [apisix-helm-chart] tao12345666333 opened a new issue #70: proposal: restructure the apisix-helm-chart repo

tao12345666333 opened a new issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70


   Currently we provide three available charts:
   
   * apisix
   * apisix-dashboard
   * apisix-ingress-controller
   
   But both apisix-dashboard and apisix-ingress depend on apisix to work properly (and apisix is the core).
   
   Now in our documents, for example, [in the installation steps of apisix-ingress](http://apisix.apache.org/docs/ingress-controller/deployments/ack), apisix needs to be deployed first, and then apisix-ingress needs to be deployed, which causes a certain amount of trouble for users.
   
   
   With reference to many other open source projects, I suggest to adjust the Chart structure.
   
   The goal is: users can use one helm command to complete the deployment (for example, one command to deploy apisix and apisix-ingress, apisix-dashboard is optional)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] gxthrj commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
gxthrj commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835818226


   Do we only need to provide one APISIX charts? `Ingress` and `dashboard` use different templates to manage  in this charts.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tao12345666333 commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835790721


   > We may also support upgrading one of the components without upgrading/installing others.
   
   sure.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tokers commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tokers commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835786909


   That's cool, +1
   
   By the way, currently, apisix-ingress-controller and apisix-dashboard were set as dependencies of apisix, but the repositories are wrong, which results in apisix-ingress-controller and apisix-dashboard will be installed unconditionly.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tokers edited a comment on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tokers edited a comment on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835786997


   We may also support upgrading one of the components without upgrading/installing others.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tao12345666333 commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835764455


   cc @tokers @gxthrj 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tao12345666333 commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835790656


   > By the way, currently, apisix-ingress-controller and apisix-dashboard were set as dependencies of apisix, but the repositories are wrong, which results in apisix-ingress-controller and apisix-dashboard will be installed unconditionly.
   
   let me fix it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tao12345666333 commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-836076776


   please assign to me


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tao12345666333 commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835831896


   @gxthrj We can only provide one APISIX chart, but I want to retain the three (all of our projects can **only using APISIX chart in docs/guides**)
   
   The reason for retaining the other two is that we support the deployment of APISIX in non-k8s environments, or pre-deployment of APISIX. (of course, this can also be done by providing a switch in the chart of APISIX)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tokers commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tokers commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835786997


   We may also support upgrading one of the components.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tokers commented on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tokers commented on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-836402321


   @tao12345666333 Assigned.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [apisix-helm-chart] tokers edited a comment on issue #70: proposal: restructure the apisix-helm-chart repo

Posted by GitBox <gi...@apache.org>.
tokers edited a comment on issue #70:
URL: https://github.com/apache/apisix-helm-chart/issues/70#issuecomment-835786909


   That's cool, +1
   
   By the way, currently, apisix-ingress-controller and apisix-dashboard were set as dependencies of apisix, but the repositories are wrong, which results in apisix-ingress-controller and apisix-dashboard will be installed unconditionally.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org