PR#6: Try and drop volume change notifications. - koji-fedmsg-plugin

koji-fedmsg-plugin

#6 Try and drop volume change notifications.

Merged 4 years ago by pingou. Opened 4 years ago by pingou.

pingou/koji-fedmsg-plugin volume_notifs into master

Try and drop volume change notifications.

Pierre-Yves Chibon • 4 years ago

2a3e4c3

fedmsg-koji-plugin.py

file modified

		`@@ -203,6 +203,10 @@`
		`body.get('sigkey') == ''):`
		`return`

		`+ # Also, do not want to send a message on volume_id changes`
		`+ if topic == 'build.state.change' and body.get('attribute') == 'volume_id':`
		`+ return`
		`+`
		`# Last thing to do before publishing: scrub some problematic fields`
		`# These fields are floating points which get json-encoded differently on`
		`# rhel and fedora.`

pingou commented 4 years ago

This is a backport of the patch made by Kevin Fenzi on the Fedora
infrastructure ansible repo.
Cf the commit 53103de0bd4510904d2532adb7a392fe087c80a2 in that repo

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr