PR#2031: monitor-dashboard: fix deploymentconfig selector - fedora-infra/ansible

fedora-infra / ansible

#2031 monitor-dashboard: fix deploymentconfig selector

Merged 18 days ago by zlopez. Opened a month ago by neil.

fedora-infra/ neil/ansible commops/28 into main

monitor-dashboard: fix deploymentconfig selector

Neil Hanlon • 18 days ago

ea9afb1

roles/openshift-apps/monitor-dashboard/templates/deploymentconfig.yml

file modified

+3 -4

		`@@ -8,9 +8,8 @@`
		`spec:`
		`replicas: 1`
		`selector:`
		`- matchLabels:`
		`- app: monitor-dashboard`
		`- deploymentconfig: monitor-dashboard`
		`+ app: monitor-dashboard`
		`+ deploymentconfig: monitor-dashboard`
		`template:`
		`metadata:`
		`labels:`
		`@@ -89,4 +88,4 @@`
		`# name: grafana-notifiers`
		`# - configMap:`
		`# name: grafana-dashboards`
		`- # name: grafana-dashboards`
		`\ No newline at end of file`
		`+ # name: grafana-dashboards`

neil commented a month ago

DeploymentConfig objects don't expect a selector type for selector,
rather it expects a simple mapping of labels to values, which will be
substituted into the resultant Deployment object as matchLabel:

Fixes: https://gitlab.com/fedora/commops/home/-/issues/28

Signed-off-by: Neil Hanlon neil@shrug.pw