PR#151: cli: add 'update-discussion' command - fedora-qa/blockerbugs

fedora-qa / blockerbugs

#151 cli: add 'update-discussion' command

Merged 4 years ago by kparal. Opened 4 years ago by kparal.

feature/update-discussion into develop

cli: add 'update-discussion' command

Kamil Páral • 4 years ago

9a40aea

blockerbugs/cli.py

file modified

+12

		`@@ -10,6 +10,7 @@`
		`from blockerbugs.util.update_sync import UpdateSync`
		`from blockerbugs.util.bz_interface import BlockerBugs`
		`import blockerbugs.util.discussion_sync as discussion_sync`
		`+ import blockerbugs.util.pagure_bot as pagure_bot`
		`from blockerbugs.models.bug import Bug`
		`from alembic.config import Config as al_Config`
		`from alembic import command as al_command`
		`@@ -219,6 +220,10 @@`

		`discussion_sync.recreate_discussion(bugid)`

		`+ def update_discussion(args):`
		`+ ticket_id = int(args.ticket_id)`
		`+ pagure_bot.webhook_handler(ticket_id)`
		`+`

		`def main():`
		`parser = ArgumentParser()`
		`@@ -296,6 +301,13 @@`
		`help='Bug ID for which to re-create dicsussion')`
		`recreate_discussion_parser.set_defaults(func=recreate_discussion)`

		`+ update_discussion_parser = subparsers.add_parser('update-discussion',`
		`+ help='Update discussion for a given ticket')`
		`+ update_discussion_parser.add_argument('ticket_id',`
		`+ metavar='<Ticket ID>',`
		`+ help='Ticket ID for which to update dicsussion')`
		`+ update_discussion_parser.set_defaults(func=update_discussion)`
		`+`
		`args = parser.parse_args()`

		`if not hasattr(args, 'func'):`

blockerbugs/util/pagure_bot.py

file modified

+1 -1

		`@@ -274,6 +274,6 @@`
		`comment = summary(trackers_without_summary)`
		`pagure_interface.post_comment(issue_id, comment)`

		`- app.logger.debug('Updating issue cummary')`
		`+ app.logger.debug('Updating issue summary')`
		`vote_summary = summary(trackers, non_voting_users, last_comment_id, SUMMARY_HEADER)`
		`pagure_interface.update_issue(issue_id, vote_summary)`

kparal commented 4 years ago

This allows to manually process a particular discussion ticket in Pagure, recount
the votes and update the description. It is probably mostly useful for testing,
because it allows you to test blocker discussions functionality without setting
up your own Pagure instead and configuring the webhook.

Metadata Update from @kparal:
- Request assigned

4 years ago