CAAPIM / apim-charts

Helm Charts for Layer7 API Management components.
MIT License
11 stars 48 forks source link

[charts/gateway]: adding new repo to the CI config #147

Closed SatishKoney-BRCM closed 2 years ago

SatishKoney-BRCM commented 2 years ago

Description of the change

since we are using new repo for working around the bitnami issue, https://github.com/bitnami/charts/issues/10539 we need to update the bitnami repos for GW an add the same for CI config

Benefits

CI config will now have all the required repos.

Drawbacks

none

Applicable issues

Additional information

Checklist

Gazza7205 commented 2 years ago

Wouldn't it be better to just use the latest versions of the subCharts?

SatishKoney-BRCM commented 2 years ago

Wouldn't it be better to just use the latest versions of the subCharts?

that would also mean the new app versions right? since all the testing for 5.1.1 was done with the old app versions, not changing the actual versions.

Gazza7205 commented 2 years ago

Wouldn't it be better to just use the latest versions of the subCharts?

that would also mean the new app versions right? since all the testing for 5.1.1 was done with the old app versions, not changing the actual versions.

Yes, you could use the same image with the updated Chart versions. Kubernetes 1.25 will likely require this anyway so it's going to have to change again likely in the next release.