Bug 1476713
Summary: | nodeSelector lost in logging-es deploymentConfigs after update | ||
---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | Ruben Romero Montes <rromerom> |
Component: | Logging | Assignee: | Jeff Cantrill <jcantril> |
Status: | CLOSED NOTABUG | QA Contact: | Xia Zhao <xiazhao> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 3.5.1 | CC: | aos-bugs, pweil, rmeggins |
Target Milestone: | --- | Keywords: | Reopened |
Target Release: | 3.7.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-08-06 07:16:47 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Ruben Romero Montes
2017-07-31 09:23:55 UTC
Update your inventory file to set the node selector to user for the ES deploymentconfig: https://github.com/openshift/openshift-ansible/blob/release-1.5/roles/openshift_logging/defaults/main.yml#L92 @Jeff as far as I know, this is valid when the nodeSelector is the same for all the generated DeploymentConfigs. In this case we are trying to define a different nodeSelector for each of the DeploymentConfigs. The purpose is to have each pod deployed on specific nodes and if no nodeSelector is provided this should not be changed. The install/upgrade process functions more as a replace than a patch. It does not currently provide the ability to retain customized data. This is an enhancement that we'd need to take a look at and is likely to not be necessary when daemonsets are in play. The intention is not to retain data via a fact but for ansible to make the state of logging match the intention defined in the inventory file. This means you as the deployer should be setting the node selector in the inventory. This is not a bug as a user can define the node selector for each individual logging component [1]. It can be provided as a straight hash value like: openshift_logging_es_nodeselector={"node":"infra","region":"west"} or comma delimited list, hash is prefered: openshift_logging_es_nodeselector=node=infra,region=west [1 ]https://github.com/openshift/openshift-ansible/tree/release-1.5/roles/openshift_logging As stated by @Jeff previously and during our discussion on IRC the possibility of providing a "per-dedploymentConfig" nodeSelector is not really necessary and the reason I provided can be just overcome by defining a group of nodes with the hostPath configured, independently of which DC targets which node. From our side this BZ can be closed. Thanks for your feedback |