2017-01-05 18:39:29 -06:00
|
|
|
#!/usr/bin/python
|
|
|
|
|
|
|
|
import argparse
|
2017-03-09 23:54:58 -06:00
|
|
|
from datetime import datetime
|
|
|
|
import dateutil.parser
|
2017-01-05 18:39:29 -06:00
|
|
|
import sys
|
2017-07-19 00:49:38 -05:00
|
|
|
from lxml import etree as ET
|
2017-01-05 18:39:29 -06:00
|
|
|
|
|
|
|
import osc.conf
|
2017-03-10 00:20:37 -06:00
|
|
|
from osc.core import HTTPError
|
2017-03-09 23:28:48 -06:00
|
|
|
from osc.core import get_request_list
|
2017-03-09 23:30:02 -06:00
|
|
|
from osc.core import get_review_list
|
2017-03-09 23:36:24 -06:00
|
|
|
from osc.core import http_GET
|
|
|
|
from osc.core import makeurl
|
2017-07-19 00:11:33 -05:00
|
|
|
from osc.core import search
|
2017-03-10 00:20:37 -06:00
|
|
|
from osc.core import show_package_meta
|
|
|
|
from osc.core import show_project_meta
|
|
|
|
from osclib.comments import CommentAPI
|
2017-01-05 18:39:29 -06:00
|
|
|
from osclib.conf import Config
|
|
|
|
from osclib.stagingapi import StagingAPI
|
|
|
|
|
|
|
|
|
2017-03-10 00:20:37 -06:00
|
|
|
BOT_NAME = 'devel-project'
|
|
|
|
REMINDER = 'review reminder'
|
|
|
|
|
2017-03-09 23:24:32 -06:00
|
|
|
def staging_api(args):
|
|
|
|
Config(args.project)
|
2017-07-27 23:30:53 -05:00
|
|
|
return StagingAPI(osc.conf.config['apiurl'], args.project)
|
2017-03-09 23:24:32 -06:00
|
|
|
|
2017-01-05 18:39:29 -06:00
|
|
|
def devel_projects_get(apiurl, project):
|
|
|
|
"""
|
|
|
|
Returns a sorted list of devel projects for a given project.
|
|
|
|
|
|
|
|
Loads all packages for a given project, checks them for a devel link and
|
|
|
|
keeps a list of unique devel projects.
|
|
|
|
"""
|
|
|
|
devel_projects = {}
|
|
|
|
|
2017-07-19 00:11:33 -05:00
|
|
|
root = search(apiurl, **{'package': "@project='{}'".format(project)})['package']
|
|
|
|
for devel in root.findall('package/devel[@project]'):
|
|
|
|
devel_projects[devel.attrib['project']] = True
|
2017-01-05 18:39:29 -06:00
|
|
|
|
2017-04-18 17:35:43 -05:00
|
|
|
# Ensure self does not end up in list.
|
2017-10-19 00:19:48 -05:00
|
|
|
if project in devel_projects:
|
|
|
|
del devel_projects[project]
|
2017-04-18 17:35:43 -05:00
|
|
|
|
2017-01-05 18:39:29 -06:00
|
|
|
return sorted(devel_projects)
|
|
|
|
|
2017-03-09 18:15:31 -06:00
|
|
|
def list(args):
|
2017-01-05 18:39:29 -06:00
|
|
|
devel_projects = devel_projects_get(osc.conf.config['apiurl'], args.project)
|
|
|
|
if len(devel_projects) == 0:
|
|
|
|
print('no devel projects found')
|
|
|
|
else:
|
|
|
|
out = '\n'.join(devel_projects)
|
|
|
|
print(out)
|
|
|
|
|
|
|
|
if args.write:
|
2017-07-27 23:30:53 -05:00
|
|
|
api = staging_api(args)
|
|
|
|
api.dashboard_content_ensure('devel_projects', out, 'devel_projects write')
|
2017-01-05 18:39:29 -06:00
|
|
|
|
2017-03-09 23:24:32 -06:00
|
|
|
def devel_projects_load(args):
|
2017-07-27 23:30:53 -05:00
|
|
|
api = staging_api(args)
|
|
|
|
devel_projects = api.dashboard_content_load('devel_projects')
|
2017-03-09 23:24:32 -06:00
|
|
|
|
|
|
|
if devel_projects:
|
|
|
|
return devel_projects.splitlines()
|
|
|
|
|
|
|
|
raise Exception('no devel projects found')
|
|
|
|
|
2017-07-19 00:49:38 -05:00
|
|
|
def maintainer(args):
|
|
|
|
if args.group is None:
|
|
|
|
# Default is appended to rather than overridden (upstream bug).
|
|
|
|
args.group = ['factory-maintainers', 'factory-staging']
|
|
|
|
desired = set(args.group)
|
|
|
|
|
|
|
|
apiurl = osc.conf.config['apiurl']
|
|
|
|
devel_projects = devel_projects_load(args)
|
|
|
|
for devel_project in devel_projects:
|
|
|
|
meta = ET.fromstring(''.join(show_project_meta(apiurl, devel_project)))
|
|
|
|
groups = meta.xpath('group[@role="maintainer"]/@groupid')
|
|
|
|
intersection = set(groups).intersection(desired)
|
|
|
|
if len(intersection) != len(desired):
|
|
|
|
print('{} missing {}'.format(devel_project, ', '.join(desired - intersection)))
|
|
|
|
|
2017-03-09 23:54:58 -06:00
|
|
|
def request_age(request):
|
|
|
|
date = dateutil.parser.parse(request.statehistory[0].when)
|
|
|
|
delta = datetime.utcnow() - date
|
|
|
|
return delta.days
|
|
|
|
|
2017-03-09 23:28:48 -06:00
|
|
|
def requests(args):
|
|
|
|
apiurl = osc.conf.config['apiurl']
|
|
|
|
devel_projects = devel_projects_load(args)
|
|
|
|
|
|
|
|
for devel_project in devel_projects:
|
|
|
|
requests = get_request_list(apiurl, devel_project,
|
|
|
|
req_state=('new', 'review'),
|
|
|
|
req_type='submit',
|
|
|
|
# Seems to work backwards, as it includes only.
|
2017-10-18 14:46:17 -05:00
|
|
|
exclude_target_projects=[devel_project],
|
|
|
|
withfullhistory=True)
|
2017-03-09 23:28:48 -06:00
|
|
|
for request in requests:
|
|
|
|
action = request.actions[0]
|
2017-03-09 23:54:58 -06:00
|
|
|
age = request_age(request)
|
|
|
|
if age < args.min_age:
|
|
|
|
continue
|
|
|
|
|
2017-03-09 23:28:48 -06:00
|
|
|
print(' '.join((
|
|
|
|
request.reqid,
|
|
|
|
'/'.join((action.tgt_project, action.tgt_package)),
|
|
|
|
'/'.join((action.src_project, action.src_package)),
|
2017-03-09 23:54:58 -06:00
|
|
|
'({} days old)'.format(age),
|
2017-03-09 23:28:48 -06:00
|
|
|
)))
|
|
|
|
|
2017-03-10 00:20:37 -06:00
|
|
|
if args.remind:
|
|
|
|
remind_comment(apiurl, args.repeat_age, request.reqid, action.tgt_project, action.tgt_package)
|
|
|
|
|
2017-03-09 23:30:02 -06:00
|
|
|
def reviews(args):
|
|
|
|
apiurl = osc.conf.config['apiurl']
|
|
|
|
devel_projects = devel_projects_load(args)
|
|
|
|
|
|
|
|
for devel_project in devel_projects:
|
|
|
|
requests = get_review_list(apiurl, byproject=devel_project)
|
|
|
|
for request in requests:
|
|
|
|
action = request.actions[0]
|
|
|
|
if action.type != 'submit':
|
|
|
|
continue
|
|
|
|
|
2017-03-09 23:54:58 -06:00
|
|
|
age = request_age(request)
|
|
|
|
if age < args.min_age:
|
|
|
|
continue
|
|
|
|
|
2017-03-09 23:30:02 -06:00
|
|
|
for review in request.reviews:
|
|
|
|
if review.by_project == devel_project:
|
|
|
|
break
|
|
|
|
|
|
|
|
print(' '.join((
|
|
|
|
request.reqid,
|
|
|
|
'/'.join((review.by_project, review.by_package)) if review.by_package else review.by_project,
|
|
|
|
'/'.join((action.tgt_project, action.tgt_package)),
|
2017-03-09 23:54:58 -06:00
|
|
|
'({} days old)'.format(age),
|
2017-03-09 23:30:02 -06:00
|
|
|
)))
|
|
|
|
|
2017-03-10 00:20:37 -06:00
|
|
|
if args.remind:
|
|
|
|
remind_comment(apiurl, args.repeat_age, request.reqid, review.by_project, review.by_package)
|
|
|
|
|
|
|
|
def maintainers_get(apiurl, project, package=None):
|
|
|
|
if package:
|
|
|
|
try:
|
|
|
|
meta = show_package_meta(apiurl, project, package)
|
|
|
|
except HTTPError as e:
|
|
|
|
if e.code == 404:
|
|
|
|
# Fallback to project in the case of new package.
|
|
|
|
meta = show_project_meta(apiurl, project)
|
|
|
|
else:
|
|
|
|
meta = show_project_meta(apiurl, project)
|
|
|
|
meta = ET.fromstring(''.join(meta))
|
|
|
|
|
|
|
|
userids = []
|
|
|
|
for person in meta.findall('person[@role="maintainer"]'):
|
|
|
|
userids.append(person.get('userid'))
|
|
|
|
|
2017-03-10 13:23:21 -06:00
|
|
|
if len(userids) == 0 and package is not None:
|
|
|
|
# Fallback to project if package has no maintainers.
|
|
|
|
return maintainers_get(apiurl, project)
|
|
|
|
|
2017-03-10 00:20:37 -06:00
|
|
|
return userids
|
|
|
|
|
|
|
|
def remind_comment(apiurl, repeat_age, request_id, project, package=None):
|
|
|
|
comment_api = CommentAPI(apiurl)
|
|
|
|
comments = comment_api.get_comments(request_id=request_id)
|
|
|
|
comment, _ = comment_api.comment_find(comments, BOT_NAME)
|
|
|
|
|
|
|
|
if comment:
|
|
|
|
delta = datetime.utcnow() - comment['when']
|
|
|
|
if delta.days < repeat_age:
|
|
|
|
print(' skipping due to previous reminder from {} days ago'.format(delta.days))
|
|
|
|
return
|
|
|
|
|
|
|
|
# Repeat notification so remove old comment.
|
2017-04-11 15:34:29 -05:00
|
|
|
try:
|
|
|
|
comment_api.delete(comment['id'])
|
2017-10-16 22:47:58 +02:00
|
|
|
except HTTPError as e:
|
2017-04-11 15:34:29 -05:00
|
|
|
if e.code == 403:
|
|
|
|
# Gracefully skip when previous reminder was by another user.
|
|
|
|
print(' unable to remove previous reminder')
|
|
|
|
return
|
|
|
|
raise e
|
2017-03-10 00:20:37 -06:00
|
|
|
|
|
|
|
userids = sorted(maintainers_get(apiurl, project, package))
|
2017-03-10 13:24:00 -06:00
|
|
|
if len(userids):
|
|
|
|
users = ['@' + userid for userid in userids]
|
|
|
|
message = '{}: {}'.format(', '.join(users), REMINDER)
|
|
|
|
else:
|
|
|
|
message = REMINDER
|
2017-03-10 00:20:37 -06:00
|
|
|
print(' ' + message)
|
|
|
|
message = comment_api.add_marker(message, BOT_NAME)
|
|
|
|
comment_api.add_comment(request_id=request_id, comment=message)
|
|
|
|
|
2017-03-09 23:54:58 -06:00
|
|
|
def common_args_add(parser):
|
|
|
|
parser.add_argument('--min-age', type=int, default=0, metavar='DAYS', help='min age of requests')
|
2017-03-10 00:20:37 -06:00
|
|
|
parser.add_argument('--repeat-age', type=int, default=7, metavar='DAYS', help='age after which a new reminder will be sent')
|
|
|
|
parser.add_argument('--remind', action='store_true', help='remind maintainers to review')
|
2017-03-09 23:54:58 -06:00
|
|
|
|
2017-01-05 18:39:29 -06:00
|
|
|
|
|
|
|
if __name__ == '__main__':
|
2017-03-09 18:15:31 -06:00
|
|
|
parser = argparse.ArgumentParser(description='Operate on devel projects for a given project.')
|
|
|
|
subparsers = parser.add_subparsers(title='subcommands')
|
|
|
|
|
2017-01-05 18:39:29 -06:00
|
|
|
parser.add_argument('-A', '--apiurl', metavar='URL', help='API URL')
|
|
|
|
parser.add_argument('-d', '--debug', action='store_true', help='print info useful for debuging')
|
2017-03-09 18:15:31 -06:00
|
|
|
parser.add_argument('-p', '--project', default='openSUSE:Factory', metavar='PROJECT', help='project from which to source devel projects')
|
2017-01-05 18:39:29 -06:00
|
|
|
|
2017-03-09 18:15:31 -06:00
|
|
|
parser_list = subparsers.add_parser('list', help='List devel projects.')
|
|
|
|
parser_list.set_defaults(func=list)
|
|
|
|
parser_list.add_argument('-w', '--write', action='store_true', help='write to dashboard container package')
|
|
|
|
|
2017-07-19 00:49:38 -05:00
|
|
|
parser_maintainer = subparsers.add_parser('maintainer', help='Check for relevant groups as maintainer.')
|
|
|
|
parser_maintainer.set_defaults(func=maintainer)
|
|
|
|
parser_maintainer.add_argument('-g', '--group', action='append', help='group for which to check')
|
|
|
|
|
2017-03-09 23:28:48 -06:00
|
|
|
parser_requests = subparsers.add_parser('requests', help='List open requests.')
|
|
|
|
parser_requests.set_defaults(func=requests)
|
2017-03-09 23:54:58 -06:00
|
|
|
common_args_add(parser_requests)
|
2017-03-09 23:28:48 -06:00
|
|
|
|
2017-03-09 23:30:02 -06:00
|
|
|
parser_reviews = subparsers.add_parser('reviews', help='List open reviews.')
|
|
|
|
parser_reviews.set_defaults(func=reviews)
|
2017-03-09 23:54:58 -06:00
|
|
|
common_args_add(parser_reviews)
|
2017-03-09 23:30:02 -06:00
|
|
|
|
2017-03-09 18:15:31 -06:00
|
|
|
args = parser.parse_args()
|
|
|
|
osc.conf.get_config(override_apiurl=args.apiurl)
|
|
|
|
osc.conf.config['debug'] = args.debug
|
|
|
|
sys.exit(args.func(args))
|