PR#1252: greenwave: Disable decision update messages - fedora-infra/ansible

fedora-infra / ansible

#1252 greenwave: Disable decision update messages

Merged a year ago by kevin. Opened a year ago by lholecek.

fedora-infra/ lholecek/ansible greenwave-disable-decision-updates into main

greenwave: Disable decision update messages

Lukas Holecek • a year ago

f8cc826

roles/openshift-apps/greenwave/templates/deploymentconfig.yml

file modified

+1 -1

		`@@ -65,7 +65,7 @@`
		`app: greenwave`
		`service: fedmsg-consumers`
		`spec:`
		`- replicas: 1`
		`+ replicas: 0`
		`selector:`
		`service: fedmsg-consumers`
		`template:`

lholecek commented a year ago

These are no longer used.

lholecek commented a year ago

@adamwill Please help review.

zuul commented a year ago

Build succeeded.

fi-ansible-lint-diff : SUCCESS in 2m 44s
fi-yamllint-diff : SUCCESS in 1m 56s

adamwill commented a year ago

edit: wait, I think I was misremembering what the consumers on the greenwave side do, actually. Assuming this simply has the effect of disabling the resultsdb and waiverdb consumers in the Fedora greenwave deployment, and all they actually do is publish decision change messages, then yes, this should be fine, although I guess we might want to rip things out more definitively than just leaving it in but 'disabling' it like this.

Edited a year ago by adamwill

adamwill commented a year ago

edit: sigh, for the love of Pete, I was reading the diff wrong. I should really switch back to regular coffee, huh.

Edited a year ago by adamwill

adamwill commented a year ago

For now @kevin scaled this to 0 manually, we'll see if anything unexpectedly breaks. If not, I guess we can just yank all the consumer-related code from greenwave upstream and also from here?