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...

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

Flag notifications