Skip to content

Commit

Permalink
Merge pull request #1228 from threefoldtech/development_max_name_for_…
Browse files Browse the repository at this point in the history
…wowkrloads

fix max length of workloads to be 36
  • Loading branch information
rawdaGastan authored Oct 20, 2024
2 parents d0ee0d5 + 6d05165 commit ce96358
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions grid-client/workloads/deployment.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,8 +117,9 @@ func validateName(name string) error {
return fmt.Errorf("name cannot be empty")
}

if len(name) > 50 {
return fmt.Errorf("name cannot exceed 50 characters")
// this because max virtio fs tag length is 36 and it is used by cloud-hypervisor
if len(name) > 36 {
return fmt.Errorf("name cannot exceed 36 characters")
}

if !nameMatch.MatchString(name) {
Expand Down

0 comments on commit ce96358

Please sign in to comment.