Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Standardize control-plane node label value specification #207

Open
intlabs opened this issue Feb 15, 2017 · 2 comments
Open

Standardize control-plane node label value specification #207

intlabs opened this issue Feb 15, 2017 · 2 comments
Assignees
Labels
Milestone

Comments

@intlabs
Copy link
Contributor

intlabs commented Feb 15, 2017

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
Copy link
Contributor

renmak commented Mar 15, 2017

@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
Copy link
Contributor Author

intlabs commented Mar 15, 2017

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

renmak added a commit to renmak/openstack-helm that referenced this issue Mar 15, 2017
…stack-control-plane in following:docs, glance, rabbitmq, postgresql, memcached, mariadb, maas, keystone, horizon, heat, etcd, cinder and neutron
renmak added a commit to renmak/openstack-helm that referenced this issue Mar 15, 2017
…stack-control-plane in following:docs, glance, rabbitmq, postgresql, memcached, mariadb, maas, keystone, horizon, heat, etcd, cinder and neutron
renmak added a commit to renmak/openstack-helm that referenced this issue Mar 23, 2017
renmak added a commit to renmak/openstack-helm that referenced this issue Mar 29, 2017
…nd Replicas for consistency in values.yaml.
renmak added a commit to renmak/openstack-helm that referenced this issue Mar 29, 2017
…nd Replicas for consistency in values.yaml.
renmak added a commit to renmak/openstack-helm that referenced this issue Apr 6, 2017
…nd Replicas for consistency in values.yaml.
wilkers-steve added a commit that referenced this issue Apr 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants