Bug 1553012
Summary: | Duplicated node-labels in node-config.yaml while enabling cri-o | ||
---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | Gan Huang <ghuang> |
Component: | Installer | Assignee: | Steve Milner <smilner> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Gan Huang <ghuang> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 3.9.0 | CC: | aos-bugs, ghuang, jokerman, mgugino, mmccomas, sdodson, smilner, smunilla, wmeng |
Target Milestone: | --- | ||
Target Release: | 3.9.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | openshift-ansible-3.9.4-1.git.0.a49cc04.el7 | Doc Type: | Bug Fix |
Doc Text: |
Cause:
The nodes configuration file had hardcoded labels in the cri-o section.
Consequence:
Double labels could occur if labels were set elsewhere in the installer.
Fix:
Remove the hardcoded labels as they are not needed.
Result:
The possibility of having double labels is no longer present.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2018-06-18 18:19:21 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
Gan Huang
2018-03-08 02:29:17 UTC
It probably now needs the "run once" added to it. Oops, this is node-config, not docker-gc. It's not run once on the gc. Which form of cri-o installed was used? This seems to be caused by hard-coding node-labels in kubelet args when using crio: https://github.com/openshift/openshift-ansible/commit/55f6b3879d770d756963564a5894c09806a31003 Are these strictly necessary? I don't think they are, they don't look specific to crio and probably can be removed. Good catch Michael! To my knowledge the node-labels that are hard coded in the template shouldn't be required. I'll remove them and push the PR up. Release-3.9 backport https://github.com/openshift/openshift-ansible/pull/7462 Verified in openshift-ansible-3.9.4-1.git.0.a49cc04.el7.noarch.rpm |