goharbor / harbor-helm

The helm chart to deploy Harbor
Apache License 2.0
1.19k stars 760 forks source link

Add entry for `ingress.className` in README #1832

Closed saip92 closed 2 days ago

saip92 commented 1 month ago

The ingress.className value is supported and used by the chart but is missing from the README. This PR adds it to the configuration table.

Vad1mo commented 1 month ago

@saip92, the DCO is missing

saip92 commented 1 month ago

@saip92, the DCO is missing

Sorry about that. Fixed it.

wy65701436 commented 1 month ago

please sync the latest changes to make the ci pass, thanks.

MinerYang commented 1 month ago

Hi @saip92 , Could you help to rebase the branch? Thanks a lot

saip92 commented 1 week ago

@wy65701436 @MinerYang Apologies! I've been caught up recently and couldn't get to this sooner. Rebased and pushed.