Merge pull request #1114 from ammeon/fix-release-name-limit

fix(tiller): truncate release name returned from moniker
pull/1116/head
Matt Butcher 9 years ago committed by GitHub
commit c08a29d724

@ -333,8 +333,7 @@ func (s *releaseServer) uniqName(start string, reuse bool) (string, error) {
namer := moniker.New() namer := moniker.New()
name := namer.NameSep("-") name := namer.NameSep("-")
if len(name) > releaseNameMaxLen { if len(name) > releaseNameMaxLen {
log.Printf("info: Candidate name %q exceeds maximum length %d. Skipping.", name, releaseNameMaxLen) name = name[:releaseNameMaxLen]
continue
} }
if _, err := s.env.Releases.Get(name); err == driver.ErrReleaseNotFound { if _, err := s.env.Releases.Get(name); err == driver.ErrReleaseNotFound {
return name, nil return name, nil

Loading…
Cancel
Save