packagedb-cli-commits February 2015

packagedb-cli-commits@lists.stg.fedorahosted.org

1 participants
15 discussions

[packagedb-cli] Created tag 2.8.1
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

The signed tag '2.8.1' was created. Tagger: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Feb 6 11:07:10 2015 +0100 Release 2.8.1 Changes since the last tag '2.8': Pierre-Yves Chibon (2): Actually add the entry_point for pkgdb-admin Release 2.8.1

1 0

[packagedb-cli] Release 2.8.1
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 650880cce8c35b1080ca828af872f742935703e6 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Feb 6 11:06:37 2015 +0100 Release 2.8.1 README | 2 +- setup.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- diff --git a/README b/README index b7bf3f2..903c1cd 100644 --- a/README +++ b/README @@ -5,7 +5,7 @@ Fedora PackageDB-cli :Author: Pierre-Yves Chibon :Contact: pingou(a)fedoraproject.org :Date: Wed Feb 06, 2015 -:Version: 2.8 +:Version: 2.8.1 The `PackageDB-cli` is a commande line interface to the PackageDB of Fedora. diff --git a/setup.py b/setup.py index 709bed6..240e27b 100644 --- a/setup.py +++ b/setup.py @@ -8,7 +8,7 @@ from setuptools import setup setup( name='packagedb-cli', description='A command line tool to access the Fedora Package Database.', - version='2.8', + version='2.8.1', license='GPLv2+', download_url='https://fedorahosted.org/releases/p/a/packagedb-cli/', url='https://fedorahosted.org/packagedb-cli/',

1 0

[packagedb-cli] Actually add the entry_point for pkgdb-admin
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 2ba10deb49041caff5e4794f9802c81f181bca7e Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Feb 6 11:06:12 2015 +0100 Actually add the entry_point for pkgdb-admin setup.py | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) --- diff --git a/setup.py b/setup.py index 7e19929..709bed6 100644 --- a/setup.py +++ b/setup.py @@ -18,6 +18,7 @@ setup( entry_points={ 'console_scripts': [ "pkgdb-cli=pkgdb2client.cli:main", + "pkgdb-admin=pkgdb2client.admin:main", ] }, install_requires=[

1 0

[packagedb-cli: 6/6] Release 2.8
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 28a4cdac4907bcde590a0ba2e601fa564e652982 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Feb 6 10:49:31 2015 +0100 Release 2.8 README | 4 ++-- setup.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) --- diff --git a/README b/README index 7c83f01..b7bf3f2 100644 --- a/README +++ b/README @@ -4,8 +4,8 @@ Fedora PackageDB-cli :Author: Pierre-Yves Chibon :Contact: pingou(a)fedoraproject.org -:Date: Wed Jan 21, 2015 -:Version: 2.7 +:Date: Wed Feb 06, 2015 +:Version: 2.8 The `PackageDB-cli` is a commande line interface to the PackageDB of Fedora. diff --git a/setup.py b/setup.py index 911dd99..7e19929 100644 --- a/setup.py +++ b/setup.py @@ -8,7 +8,7 @@ from setuptools import setup setup( name='packagedb-cli', description='A command line tool to access the Fedora Package Database.', - version='2.7', + version='2.8', license='GPLv2+', download_url='https://fedorahosted.org/releases/p/a/packagedb-cli/', url='https://fedorahosted.org/packagedb-cli/',

1 0

[packagedb-cli: 5/6] Merge pull request #31 from fedora-infra/spec_urls
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 90fc80b196185b9fbc34d9498c148e18aa9c3b01 Merge: c04d455 1e17958 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Mon Feb 2 19:27:04 2015 +0100 Merge pull request #31 from fedora-infra/spec_urls Spec urls pkgdb2client/admin.py | 38 ++++++++++++++++++++++++++++---------- pkgdb2client/cli.py | 36 ++++++++++++++++++++++++++++-------- 2 files changed, 56 insertions(+), 18 deletions(-) ---

1 0

[packagedb-cli] Created tag 2.8
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

The signed tag '2.8' was created. Tagger: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Feb 6 10:50:14 2015 +0100 Release 2.8 Changes since the last tag '2.7': Pierre-Yves Chibon (74): Add the first work on a pkgdb-admin script Fix formatting for the logs Add the `process` action to `pkgdb-admin` to actually process an admin action Handles `request.unretire` in pkgdb-admin list Handle situation where a new action is added to pkgdb but unknown to pkgdb-admin Update the README file to reflect the new structure of the project Re-order the imports Drop the bugzilla integration in the admin controller Create the pkgdb2client.utils module Move the bugzilla query from the CLI module to the utils module Instantiate the bugzilla client only once for the whole module Drop the call for FAS since we do not use it anywhere Add method to login against bugzilla Add utility method to retrieve a specified bug off bugzilla Add utility method to find out all the users that commented on a given bug Add a utility method to check if a specified email belongs to a packager Start working on the check pre-package creation Check if the summary of the bug fits the expectation for package reviews Be consistent on how we build our strings, use .format() everywhere Finally figure out a way to retrieve a FAS user by his/her email address If all checks cleared, return a message saying so Adjust the docstring of the update action Implement interactive processing of the new package requests Convert new package requests into new branch requests if the package exists in pkgdb Add checks to run before creating a new branch for a package Make the utility method is_packager accept either a username or an email as input Adjust the message returned Authenticate against FAS to check if the user is a packager or not Implement the interactive process for new branch Mark if the message is positive (+) or might require more attention from the user (!) Create the action2msg method converting an action's JSON into a human-readable string Display the human-readable description of the action before processing it Raise an exception with the error message only when there is one Full path imports and allow to use single letter code to pass (p) or deny (d) an action Retrieve the username and use it when creating the FAS client Rework the get_bug method to ask for login only if needed Adjust __get_fas_user_by_email Inform the user if an email address does not correspond to a FAS user Adjust the call via FASCLIENT Drop log as it's undefined and catch more AuthError from FAS Get the shortcuts to approve (a), deny (d) and pass (p) to work When checking a branch creation, skip some tests if it's a new package Let's be sure the collection name can start with 'el' or 'epel' which implies lower case When creating a package check the branch for which we create it as well Add a method to retrieve (and cache daily) the RHEL JSON Upon an EPEL branch requests, checks if the package is in RHEL or not pep8 clean up in the admin module pep8 clean up of the utils module Small pep8 clean of the cli module Adjust variable name to make pylint happier Fix logging - thanks pylint Adjust the variable name for the pkgdb client constant and fix import Mention for which package and branch all checks cleared when checking a branch Report to the admin when the package requesting an EPEL branch is not part of RHEL Move the check_branch_creation at the end of the check_package_creation Give a little space before asking what to do Allow the admins to process multiple requests in one call Move handling package requests into its own function Move logic to handle new branch requests into its own function Fix the return of the data and thus display of the information messages Add some space at the end of an action When processing new package requests, add support for the d and p shortcuts Fix call to update the status so that it uses the proper action identifier Make the default be 'pass' if the user entered an invalid action that went through Require the admin to provide a message when denying a request Adjust the check for RHEL packages to use the new JSON format In english there is no 'e' at the end of branch Merge pull request #30 from fedora-infra/pkgdb_admin Expand pkgdb-admin to allow specifying where to query/run Adjust pkgdb-cli to allow specifying which pkgdb instance to query Fix variable name PKGDBCLIENT != pkgdbclient Use logging instead of direct prints to inform the user about the change of target Merge pull request #31 from fedora-infra/spec_urls Release 2.8

1 0

[packagedb-cli: 4/6] Use logging instead of direct prints to inform the user about the change of target
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 1e17958bed3719f42c572a337a935b28faa80f81 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Mon Feb 2 17:56:05 2015 +0100 Use logging instead of direct prints to inform the user about the change of target pkgdb2client/admin.py | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) --- diff --git a/pkgdb2client/admin.py b/pkgdb2client/admin.py index e953fb6..2ec9f2c 100644 --- a/pkgdb2client/admin.py +++ b/pkgdb2client/admin.py @@ -400,7 +400,7 @@ def main(): global PKGDBCLIENT if arg.pkgdburl: - print "Querying pkgdb at: %s" % arg.pkgdburl + LOG.info("Querying pkgdb at: %s", arg.pkgdburl) PKGDBCLIENT = PkgDB( arg.pkgdburl, login_callback=pkgdb2client.ask_password) @@ -410,12 +410,12 @@ def main(): if arg.bzurl: if not arg.bzurl.endswith('xmlrpc.cgi'): arg.bzurl = '%s/xmlrpc.cgi' % arg.bzurl - print "Querying bugzilla at: %s" % arg.pkgdburl + LOG.info("Querying bugzilla at: %s", arg.pkgdburl) utils.BZCLIENT.url = arg.bzurl utils.BZCLIENT._sslverify = not arg.insecure if arg.fasurl: - print "Querying FAS at: %s" % arg.pkgdburl + LOG.info("Querying FAS at: %s", arg.pkgdburl) utils.FASCLIENT.base_url = arg.fasurl utils.FASCLIENT.insecure = arg.insecure

1 0

[packagedb-cli: 3/6] Fix variable name PKGDBCLIENT != pkgdbclient
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit ddfb946c835db9e1fcb5b309b406bc06a4616dfe Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Jan 30 17:58:32 2015 +0100 Fix variable name PKGDBCLIENT != pkgdbclient pkgdb2client/cli.py | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index ecf0917..b860eca 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -784,14 +784,14 @@ def main(): elif arg.verbose: LOG.setLevel(logging.INFO) - global PKGDBCLIENT + global pkgdbclient if arg.pkgdburl: print "Querying pkgdb at: %s" % arg.pkgdburl - PKGDBCLIENT = PkgDB( + pkgdbclient = PkgDB( arg.pkgdburl, login_callback=pkgdb2client.ask_password) - PKGDBCLIENT.insecure = arg.insecure + pkgdbclient.insecure = arg.insecure if arg.bzurl: if not arg.bzurl.endswith('xmlrpc.cgi'):

1 0

[packagedb-cli: 2/6] Adjust pkgdb-cli to allow specifying which pkgdb instance to query
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 80a9edcb6f11d996ba4c388e0fd47edd333f1397 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Jan 30 16:30:17 2015 +0100 Adjust pkgdb-cli to allow specifying which pkgdb instance to query This commit allows specifying which pkgdb, FAS and bugzilla instances to query when running. This way one can run/test pkgdb-cli against a local pkgdb or FAS instance. In addition this commit introduce the --insecure argument allowing to ignore checking the validity of the SSL certificate when running the queries. pkgdb2client/cli.py | 38 +++++++++++++++++++++++++++++--------- 1 files changed, 29 insertions(+), 9 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index 4794a68..ecf0917 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -190,10 +190,17 @@ def setup_parser(): help="Gives more info about what's going on") parser.add_argument('--debug', action='store_true', help="Outputs bunches of debugging info") - parser.add_argument('--test', action='store_true', - help="Uses a test instance instead of the real pkgdb.") parser.add_argument('--version', action='version', version='pkgdb-cli %s' % (__version__)) + parser.add_argument('--insecure', action='store_true', default=False, + help="Tells pkgdb-cli to ignore invalid SSL " + "certificates") + parser.add_argument('--pkgdburl', + help="Base url of the pkgdb instance to query.") + parser.add_argument('--fasurl', + help="Base url of the FAS instance to query.") + parser.add_argument('--bzurl', + help="Base url of the bugzilla instance to query.") subparsers = parser.add_subparsers(title='actions') @@ -777,13 +784,26 @@ def main(): elif arg.verbose: LOG.setLevel(logging.INFO) - if arg.test: - global pkgdbclient - print "Testing environment" - pkgdbclient = PkgDB( - 'https://admin.stg.fedoraproject.org/pkgdb', - login_callback=pkgdb2client.ask_password, - insecure=True) + global PKGDBCLIENT + if arg.pkgdburl: + print "Querying pkgdb at: %s" % arg.pkgdburl + PKGDBCLIENT = PkgDB( + arg.pkgdburl, + login_callback=pkgdb2client.ask_password) + + PKGDBCLIENT.insecure = arg.insecure + + if arg.bzurl: + if not arg.bzurl.endswith('xmlrpc.cgi'): + arg.bzurl = '%s/xmlrpc.cgi' % arg.bzurl + print "Querying bugzilla at: %s" % arg.pkgdburl + utils.BZCLIENT.url = arg.bzurl + utils.BZCLIENT._sslverify = not arg.insecure + + if arg.fasurl: + print "Querying FAS at: %s" % arg.pkgdburl + utils.FASCLIENT.base_url = arg.fasurl + utils.FASCLIENT.insecure = arg.insecure return_code = 0

1 0

[packagedb-cli: 1/6] Expand pkgdb-admin to allow specifying where to query/run
by Pierre-YvesChibon 06 Feb '15

06 Feb '15

commit 9ba3c305df5aca97ba9faecb9f996a60d411fe1d Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Jan 30 16:26:45 2015 +0100 Expand pkgdb-admin to allow specifying where to query/run This commit hads to pkgdb-admin the possibility to specify the pkgdb, FAS and bugzilla instances to query while running. This way one can run pkgdb-admin against its own pkgdb instance. In addition this commit introduce the --insecure argument allowing to ignore invalid or self-signed SSL certificates that python-requests would normally complain about pkgdb2client/admin.py | 38 ++++++++++++++++++++++++++++---------- 1 files changed, 28 insertions(+), 10 deletions(-) --- diff --git a/pkgdb2client/admin.py b/pkgdb2client/admin.py index 233ab7d..e953fb6 100644 --- a/pkgdb2client/admin.py +++ b/pkgdb2client/admin.py @@ -55,10 +55,17 @@ def setup_parser(): help="Gives more info about what's going on") parser.add_argument('--debug', action='store_true', help="Outputs bunches of debugging info") - parser.add_argument('--test', action='store_true', - help="Uses a test instance instead of the real pkgdb.") parser.add_argument('--version', action='version', version='pkgdb-admin %s' % (__version__)) + parser.add_argument('--insecure', action='store_true', default=False, + help="Tells pkgdb-cli to ignore invalid SSL " + "certificates") + parser.add_argument('--pkgdburl', + help="Base url of the pkgdb instance to query.") + parser.add_argument('--fasurl', + help="Base url of the FAS instance to query.") + parser.add_argument('--bzurl', + help="Base url of the bugzilla instance to query.") subparsers = parser.add_subparsers(title='actions') @@ -391,15 +398,26 @@ def main(): elif arg.verbose: LOG.setLevel(logging.INFO) - if arg.test: - global PKGDBCLIENT - print "Testing environment" + global PKGDBCLIENT + if arg.pkgdburl: + print "Querying pkgdb at: %s" % arg.pkgdburl PKGDBCLIENT = PkgDB( - #'https://admin.stg.fedoraproject.org/pkgdb', - #'http://209.132.184.188/', - 'http://127.0.0.1:5000', - login_callback=pkgdb2client.ask_password, - insecure=True) + arg.pkgdburl, + login_callback=pkgdb2client.ask_password) + + PKGDBCLIENT.insecure = arg.insecure + + if arg.bzurl: + if not arg.bzurl.endswith('xmlrpc.cgi'): + arg.bzurl = '%s/xmlrpc.cgi' % arg.bzurl + print "Querying bugzilla at: %s" % arg.pkgdburl + utils.BZCLIENT.url = arg.bzurl + utils.BZCLIENT._sslverify = not arg.insecure + + if arg.fasurl: + print "Querying FAS at: %s" % arg.pkgdburl + utils.FASCLIENT.base_url = arg.fasurl + utils.FASCLIENT.insecure = arg.insecure return_code = 0

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

packagedb-cli-commits February 2015