blakeblackshear / blakeshome-charts

Repo for helm charts
41 stars 52 forks source link

Add possibility to setup ingress Class Name #64

Closed Mrkazik99 closed 3 months ago

Mrkazik99 commented 3 months ago

Hello @blakeblackshear please review and merge my PR Thank you <3

billimek commented 3 months ago

Can you please also bump the chart version in the Chart.yaml file? Maybe to 7.4.0 given that there is a new feature.

Also, adding something, even a placeholder, to values.yaml will expose the fact that there is a new option (ingress.ingressClassName)

Mrkazik99 commented 3 months ago

Hi @billimek Done