Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Commit

Permalink
Merge pull request #928 from vakwetu/order-cert-mounts
Browse files Browse the repository at this point in the history
Make TLS cert mount order consisitent
  • Loading branch information
openshift-merge-bot[bot] authored Jun 7, 2024
2 parents d5be378 + bcc9fc5 commit 49b2cd8
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion pkg/deployment/deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,14 @@ func (d *Deployer) addCertMounts(
}

if service.Spec.TLSCerts != nil {
for certKey := range service.Spec.TLSCerts {
// sort cert list to ensure mount list is consistent
certKeyList := make([]string, 0, len(service.Spec.TLSCerts))
for ckey := range service.Spec.TLSCerts {
certKeyList = append(certKeyList, ckey)
}
sort.Strings(certKeyList)

for _, certKey := range certKeyList {
log.Info("Mounting TLS cert for service", "service", svc)
volMounts := storage.VolMounts{}

Expand Down

0 comments on commit 49b2cd8

Please sign in to comment.