hrak opened a new pull request, #46:
URL: https://github.com/apache/cloudstack-kubernetes-provider/pull/46

   This PR changes the way the CloudStack Cloud Controller Manager handles 
kubelet providerID into a more standardized way that is more common around 
several other CCM's like the Openstack or vSphere one. The changes were needed 
to make node labels work again.
   
   The providerID is configured by either setting the kubelet command line flag 
`--provider-id` (deprecated) or using the `providerID` setting in [kubelet 
config 
yaml](https://kubernetes.io/docs/reference/config-api/kubelet-config.v1beta1/). 
The format of the value is `<providername>://region/instance-id` so in case of 
CloudStack for a platform without region f.e. : 
`cloudstack:///4e7689bc-99ea-43d8-8c37-5ff511c01665`.
   
   It also implements the two interface methods `InstanceShutdownByProviderID` 
and `InstanceShutdown`.
   
   And it fixes the way node labels are sanitized. The old regex approach would 
in some cases strip off allowed characters, for example zone name 
`Development-Internal` would turn into `DevelopmentInternal`, although the `-` 
is allowed in label values. The new approach converts all chars that are not 
allowed in a label value to underscores:
   
   `Development-Internal` -> `Development-Internal`
   `Small Instance (4 GB / 2 CPU)` -> `Small_Instance__4_GB___2_CPU`
   
   


-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to