action #2424
closedosc plugin: prototype of a mechanism to notify involved users (without repetition)
0%
Description
The plugin should be able to notify involved users about failed builds and failed tests. The first approach will be writing a comment in the staging project mentioning all the submitters. The plugin will need to store some information about the already notified events, to prevent it to notify twice about the same test result of buildfail if it's invoked several timed (which will happen).
Updated by ancorgs over 10 years ago
Of course, #2428 needs to be done before being able to finish this task.
Updated by ancorgs over 10 years ago
This page contains a good example of how the comments could look like (last comment at the time of writing this)
https://build.opensuse.org/project/show/openSUSE:Factory:Staging:A
Updated by aplanas over 10 years ago
I see the comments, where is the basic API and rules for the plugin.
Updated by ancorgs over 10 years ago
In OBS you have the following routes for creating, reading and deleting comments:
- get 'comments/request/REQUEST_ID'
- post 'comments/request/REQUEST_ID'
- get 'comments/package/PROJECT_NAME/PACKAGE_NAME'
- post 'comments/package/PROJECT_NAME/PACKAGE_NAME'
- get 'comments/project/PROJECT_NAME'
- post 'comments/project/PROJECT_NAME'
- delete 'comment/COMMENT_ID'
Updated by ancorgs over 10 years ago
About the rules, the initial proposal (that probably needs refining but is a valid starting point) is in section 4.1 of
http://etherpad.cloud.suse.de/p/obs_and_openqa
Updated by coolo over 10 years ago
- Priority changed from Normal to Urgent
- Target version deleted (
Staging sprint 5)
Updated by aplanas over 10 years ago
[Only to not forget this thought.]
Makes sense to add this methods in https://build.opensuse.org/apidocs?
Updated by aplanas over 10 years ago
I tried to test the best way I can, but is not enough. For example, I didn't receive any notification from OBS for any comment with @aplanas text.
Also I commented the remove of the comments in the acceptance command to avoid lost of data until there are more test in the code.
Updated by aplanas over 10 years ago
- Status changed from In Progress to Resolved
I will work fixing / adjusting comments when the need appears.