Description of problem: Duplicated node-labels in node-config.yaml while enabling cri-o: # grep node-labels -A 3 /etc/origin/node/node-config.yaml node-labels: - router=enabled - role=node - registry=enabled -- node-labels: - router=true - registry=true runtime-request-timeout: Version-Release number of the following components: openshift-ansible-3.9.3-1.git.0.e166207.el7.noarch.rpm How reproducible: always Steps to Reproduce: 1. Trigger installation with cir-o enabled # cat inventory <--snip--> openshift_use_crio=true openshift_crio_systemcontainer_image_override=registry.reg-aws.openshift.com:443/openshift3/cri-o:v3.9 [nodes] host-8-241-100.host.centralci.eng.rdu2.redhat.com openshift_node_labels="{'role': 'node','registry': 'enabled','router': 'enabled'}" <--snip--> Actual results: # grep node-labels -A 3 /etc/origin/node/node-config.yaml node-labels: - router=enabled - role=node - registry=enabled -- node-labels: - router=true - registry=true runtime-request-timeout: Expected results: Additional info: Please attach logs from ansible-playbook with the -vvv flag
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.
PR: https://github.com/openshift/openshift-ansible/pull/7453
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