mirror of https://github.com/kubernetes/kops.git
Merge pull request #16136 from borg-land/sns-char-limit
limit jobname value to 80
This commit is contained in:
commit
6f7fcb0a9a
|
@ -274,6 +274,9 @@ func defaultClusterName(cloudProvider string) (string, error) {
|
||||||
suffix = "k8s.local"
|
suffix = "k8s.local"
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if len(jobName) > 79 { // SNS has char limit of 80
|
||||||
|
jobName = jobName[:79]
|
||||||
|
}
|
||||||
if jobType == "presubmit" {
|
if jobType == "presubmit" {
|
||||||
return fmt.Sprintf("e2e-pr%s.%s.%s", pullNumber, jobName, suffix), nil
|
return fmt.Sprintf("e2e-pr%s.%s.%s", pullNumber, jobName, suffix), nil
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue