mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [mesos] wavesoft opened a new pull request #351: Ensure that long CNI labels are short enough
Date Fri, 06 Mar 2020 13:56:10 GMT
wavesoft opened a new pull request #351: Ensure that long CNI labels are short enough
URL: https://github.com/apache/mesos/pull/351
 
 
   Apparently this is an implicit limitation on the maximum length of network labels, enforced
by the [k8s apimachinery library](https://github.com/kubernetes/kubernetes/blob/132d2af/staging/src/k8s.io/apimachinery/pkg/util/validation/validation.go#L154)
that various CNI plugins use for their API abstraction. 
   
   The maximum allowed length is 63 characters. Therefore, in order for us to be able to maximize
the amount of information preserved, I am proposing the following solution for long labels:
   
   ```
   MD5(<label>)[0:7] + "..." + <label>[sizeof(<label>)-53:]
   ```
   
   For example:
   ```
   /teams/backend-services/database-service/deployments/development/tag-1.12-dev
   ``` 
   Becomes:
   ```
   df1d656...database-service/deployments/development/tag-1.12-dev
   ```

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message