att-comdev / openstack-helm

PROJECT HAS MOVED TO OPENSTACK
https://github.com/openstack/openstack-helm
69 stars 41 forks source link

Standardize control-plane node label value specification #207

Open intlabs opened 7 years ago

intlabs commented 7 years ago

Is this a bug report or feature request?: Mixed; depends on perspective

Kubernetes Version (output of kubectl version): N/A

Helm Client and Tiller Versions (output of helm version): N/A

Development or Deployment Environment?: N/A

Release Tag or Master: Master

Expected Behavior: As a developer and operator, it would be desirable to have standard control-plane node selector keys in the values.yaml. Currently, Nova uses a different set of keys for this functionality.

What Actually Happened: While editing charts, we cannot use the same standardized manifest for jobs as a result of some inconsistency with the way these values are specified.

How to Reproduce the Issue (as minimally as possible): N/A

Any Additional Comments: I suspect the values used in Nova are perfectly acceptable, and we should consider rolling this out to be the standard labeling for all charts. See the following for an example of how these currently differ:

renmak commented 7 years ago

@v1k0d3n @alanmeadows I can take on this.

So just to clarify, nova uses proper label key and value control_node_selector_key: @openstack-control-plane control_node_selector_value: enabled

This change should be rolled out to other charts. This is basically find and replace occurrence of node_selector_key: openstack-control-plane node_selector_value: enabled

Please confirm. Thanks

intlabs commented 7 years ago

@renmak That naming convention works for me. Note that Neutron will also need some work to bring it in line as well.