From 2f448044fd3b9b227b01db9a1e5ea9c7f5ca4e6c Mon Sep 17 00:00:00 2001 From: Clement Verna Date: Oct 26 2016 19:27:42 +0000 Subject: [PATCH 1/2] Add desactivate fedmsg hook to README --- diff --git a/README.md b/README.md index ec1a8b1..7773af3 100644 --- a/README.md +++ b/README.md @@ -9,9 +9,10 @@ CLI tool for importing issues etc. from different sources like github to pagure --- 0. Clone the issue tracker for issues from pagure. Use: ```pgimport clone ssh://git@pagure.io/tickets/foobar.git``` 1. Activate the pagure tickets hook from project settings. This is necessary step to also get pagure database updated for tickets repository changes. -2. Execute ```pgimport```. See Usage section -3. Just answer what is asked. Check below instructions for particular source -4. The script will make commits in your cloned repo: push the changes back to pagure. Use : ```pgimport push foobar.git``` +2. Deactivate the pagure Fedmsg hook from project settings. This will avoid the issues import to spam the fedmsg bus. The Hook can be reactivated once the import has completed. +3. Execute ```pgimport```. See Usage section +4. Just answer what is asked. Check below instructions for particular source +5. The script will make commits in your cloned repo: push the changes back to pagure. Use : ```pgimport push foobar.git``` ## Usage From c5a7cf60d2d751ea3da856f90ad88b48ac15206b Mon Sep 17 00:00:00 2001 From: Clement Verna Date: Oct 26 2016 19:32:47 +0000 Subject: [PATCH 2/2] Nicer way to remove trailling / --- diff --git a/pagure_importer/commands/fedorahosted.py b/pagure_importer/commands/fedorahosted.py index a43050d..811161c 100644 --- a/pagure_importer/commands/fedorahosted.py +++ b/pagure_importer/commands/fedorahosted.py @@ -17,8 +17,7 @@ from pagure_importer.utils.fas import FASclient @click.option('--offset', default=0, help='Number of issue in pagure before import') def fedorahosted(project_url, tags, private, username, password, offset): - if project_url.endswith('/'): - project_url = project_url[:-1] + project_url = project_url.rstrip('/') fasclient = FASclient(username, password, 'https://admin.fedoraproject.org/accounts') project_url = project_url + '/login/jsonrpc'