Skip to content

Commit

Permalink
Merge pull request #124 from topolvm/fix-cronjob-name
Browse files Browse the repository at this point in the history
fix: use at most 15 letters for node name in cronjob name
  • Loading branch information
ushitora-anqou committed May 14, 2024
2 parents 7c722db + 4dee32a commit 902d05c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions internal/controller/pie/pieprobe_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -514,11 +514,11 @@ func getCronJobName(kind int, nodeNamePtr *string, pieProbe *piev1alpha1.PieProb
if len(pieProbeName) > 10 {
pieProbeName = pieProbeName[:10]
}
if len(nodeName) > 11 {
nodeName = nodeName[:11]
if len(nodeName) > 15 {
nodeName = nodeName[:15]
}
if len(storageClass) > 14 {
storageClass = storageClass[:14]
if len(storageClass) > 12 {
storageClass = storageClass[:12]
}

if kind == ProvisionProbe {
Expand Down

0 comments on commit 902d05c

Please sign in to comment.