The CI (specifically flake8) fails complaining about this line: we should remove it.

I'd just call it event_id instead of freshmaker_event_id since freshmaker seems redundant.

I believe you miss an additional change in freshmaker/handlers/koji/rebuild_images_on_rpm_advisory_change.py
When instantiating the LightBlue class you should pass in there the id of the event. It should be available in there accessing self.event.
I believe this should be the only place.

@gnaponie , PTAL :)

Flake8 complains about:
./tests/handlers/koji/test_rebuild_images_on_rpm_advisory_change.py:592:42: E231 missing whitespace after ','

I guess it's this line. (pagure doesn't show the real line so I'm not 100% sure, but it should be this one...)

Could you remove the WIP state if it's ready?

Mmm now that I see this test... don't hate me :D but maybe it would be more appropriate to add it to tests/test_lightblue.py since in the end we didn't add any test for _find_images_to_rebuild.

@gnaponie , take a look one more time. I'll remove WIP state after it :)

This if statement is not necessary.

This if statement is not necessary.

Fixed

Looks good to me +1

Commit 05770276 fixes this pull-request