add support for notification plugin in standard CI

Description

None

Activity

Show:

Eyal Edri December 22, 2016 at 12:26 PM

support added, enabling integration with getbadgets will be tracked on another ticket.

Eyal Edri December 21, 2016 at 3:19 PM

I'm not sure I understand how you suggest to support it from automation dir, its a jenkins plugin so it has to be supported in YAML.
Are you talking on providing the URL link from automation scripts and not in YAML? doesn't sound logical to me because its not developers related or code related,
the project maintainers are not exposed to the webhook URL of the game which infra team manage, nor they will have any relation to future reporting done from events on CI Jobs.

Barak Korren December 21, 2016 at 3:05 PM

Maybe assign the ticket to yourself if you're handling it....

Barak Korren December 21, 2016 at 3:01 PM

n/m saw it, so you're doing everything from YAML, so this is the kind of implementation I would call specific, a generic implementation would specify soemthing to put in the automation dir so nobody needs to know YAML...

Barak Korren December 21, 2016 at 2:58 PM

which patch? I'm not seeing any patch linked here...

Eyal Edri December 21, 2016 at 2:52 PM

Not sure I understand, the current patch is as generic as it gets without any relation to getbadgets or gamification,
If its merged nothing will happen on CI (already verified) because the URL for the webhook is empty by default.

Only if you provide a URL as param from a project then its activated, for e.g for jenkins project:
https://gerrit.ovirt.org/#/c/68903/1/jobs/confs/projects/jenkins/jenkins_standard.yaml

The alternative is to create a new template for it and change multiple projects to use it if we want, which I don't see a reason for, if by default the plugin is not active.

Barak Korren December 21, 2016 at 1:51 PM

The big question is if we want to have it generic, which will require extensions to the CI standard, and some thinking and perhaps some non-trivial Jenkins tricks, or, we just want to quickly support GetBadges.

Eyal Edri December 21, 2016 at 12:33 PM

Forgot to add the link to the game: https://ovirt-ovirt-engine.getbadges.io/activity

Eyal Edri December 21, 2016 at 12:31 PM

Of course it doesn't only have to serve GetBadgets, we can use it in the future to capture events from Jenkins from other systems,
one example will be a reporting system that can aggregate results into a cental datawarehouse which we can then create reports from.

Eyal Edri December 21, 2016 at 12:30 PM

any project which will want to use it need to pass the URL as parameter

Fixed

Details

Assignee

Reporter

Priority

Created December 21, 2016 at 12:15 PM
Updated January 3, 2017 at 7:33 AM
Resolved December 22, 2016 at 12:26 PM