Skip to content

Commit

Permalink
Merge pull request #16136 from borg-land/sns-char-limit
Browse files Browse the repository at this point in the history
limit jobname value to 80
  • Loading branch information
k8s-ci-robot authored Nov 30, 2023
2 parents e880d5e + f768520 commit 6f7fcb0
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tests/e2e/kubetest2-kops/deployer/common.go
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,9 @@ func defaultClusterName(cloudProvider string) (string, error) {
suffix = "k8s.local"
}

if len(jobName) > 79 { // SNS has char limit of 80
jobName = jobName[:79]
}
if jobType == "presubmit" {
return fmt.Sprintf("e2e-pr%s.%s.%s", pullNumber, jobName, suffix), nil
}
Expand Down

0 comments on commit 6f7fcb0

Please sign in to comment.