apisix-helm-chart icon indicating copy to clipboard operation
apisix-helm-chart copied to clipboard

proposal: restructure the apisix-helm-chart repo

Open tao12345666333 opened this issue 3 years ago • 9 comments

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, 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)

tao12345666333 avatar May 09 '21 09:05 tao12345666333

cc @tokers @gxthrj

tao12345666333 avatar May 09 '21 09:05 tao12345666333

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.

tokers avatar May 09 '21 11:05 tokers

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

tokers avatar May 09 '21 11:05 tokers

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.

tao12345666333 avatar May 09 '21 11:05 tao12345666333

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

sure.

tao12345666333 avatar May 09 '21 11:05 tao12345666333

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

gxthrj avatar May 09 '21 14:05 gxthrj

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

tao12345666333 avatar May 09 '21 15:05 tao12345666333

please assign to me

tao12345666333 avatar May 10 '21 02:05 tao12345666333

@tao12345666333 Assigned.

tokers avatar May 10 '21 08:05 tokers

I think we've reached this goal

tao12345666333 avatar Sep 01 '22 13:09 tao12345666333