packagedb-cli-commits November 2014

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

1 participants
115 discussions

Start a nNew thread

[packagedb-cli] Deleted branch orphan_user
by Pierre-YvesChibon 07 Nov '14

07 Nov '14

The branch 'orphan_user' was deleted.

1 0

[packagedb-cli/orphan_user: 3/3] Fix unorphan action, it requires the new POC to be specified.
by Pierre-YvesChibon 07 Nov '14

07 Nov '14

commit 4a7347e39138c9e761a333c797c81f87eedd2d2c Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Nov 7 18:00:41 2014 +0100 Fix unorphan action, it requires the new POC to be specified. The new poc can be specified either with --user or with --poc, but one of them should be there. If it is not, raise an exception and inform the user about it. pkgdb2client/cli.py | 13 ++++++++++--- 1 files changed, 10 insertions(+), 3 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index dd15a4c..d33cb80 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -297,9 +297,7 @@ def setup_parser(): parser_unorphan.add_argument( '--poc', default=None, help="FAS username of the new point of contact of the package " - "This allows to give your package or an orphaned " - "package to someone else. " - "(default: current FAS user)") + "Can be skipped if --user is specified, otherwise is mandatory.") parser_unorphan.set_defaults(func=do_unorphan) ## Request @@ -598,6 +596,11 @@ def do_unorphan(args): pkgdbclient.username = args.username username = args.poc or args.username or pkgdbclient.username + if username is None: + raise argparse.ArgumentError( + args.poc, + 'You must specify either --user or --poc with the username of ' + 'the new point of contact.') LOG.info("new poc : {0}".format(username)) output = pkgdbclient.update_package_poc(pkgs, branches, username) @@ -763,6 +766,10 @@ def main(): LOG.debug('ActionError') print '{0}'.format(err.message) return_code = 7 + except argparse.ArgumentError, err: + LOG.debug('ArgparseError') + print '{0}'.format(err.message) + return_code = 9 except AppError, err: LOG.debug('AppError') print '{0}: {1}'.format(err.name, err.message)

1 0

[packagedb-cli/orphan_user: 2/3] If an user or a poc are specified, restrict the orphaning to this user
by Pierre-YvesChibon 07 Nov '14

07 Nov '14

commit 513ce60d8eab20c388b8de42c3c323bfbf9cefc0 Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Nov 7 17:50:56 2014 +0100 If an user or a poc are specified, restrict the orphaning to this user pkgdb2client/cli.py | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) --- diff --git a/pkgdb2client/cli.py b/pkgdb2client/cli.py index 6e5662f..dd15a4c 100644 --- a/pkgdb2client/cli.py +++ b/pkgdb2client/cli.py @@ -546,6 +546,7 @@ def do_orphan(args): LOG.info("branch : {0}".format(args.branch)) LOG.info("retire : {0}".format(args.retire)) + former_poc = args.poc or args.username if args.package == 'all': pkgs = _get_user_packages(args.poc or args.username) else: @@ -571,7 +572,8 @@ def do_orphan(args): return output = pkgdbclient.retire_packages(pkgs, branches) else: - output = pkgdbclient.orphan_packages(pkgs, branches) + output = pkgdbclient.orphan_packages( + pkgs, branches, former_poc=former_poc) for msg in output.get('messages', []): print msg

1 0

[packagedb-cli/orphan_user: 1/3] Add support for the former_poc argument in the pkgdb2client API
by Pierre-YvesChibon 07 Nov '14

07 Nov '14

commit a230cdc42dce008328473e5fb3edfc86dc55a41c Author: Pierre-Yves Chibon <pingou(a)pingoured.fr> Date: Fri Nov 7 17:41:26 2014 +0100 Add support for the former_poc argument in the pkgdb2client API pkgdb2client/__init__.py | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) --- diff --git a/pkgdb2client/__init__.py b/pkgdb2client/__init__.py index ad6307b..8b09a59 100644 --- a/pkgdb2client/__init__.py +++ b/pkgdb2client/__init__.py @@ -786,7 +786,7 @@ class PkgDB(object): return output - def orphan_packages(self, pkgnames, branches): + def orphan_packages(self, pkgnames, branches, former_poc=None): ''' Orphans the provided list of packages on the provided list of branches. @@ -795,6 +795,10 @@ class PkgDB(object): :arg branches: One or more branch names for the collections in which to orphan the packages :type branches: str or list + :kwarg former_poc: Use this argument to ensure you are only + orphaning the branches where the specified user is POC, even + if the branches you specified are broader than reality. + :type former_poc: str :return: the json object returned by the API :rtype: dict :raise PkgDBAuthException: if this method is called while the @@ -808,6 +812,9 @@ class PkgDB(object): 'branches': branches, } + if former_poc: + args['former_poc'] = former_poc + return self.handle_api_call('/package/orphan/', data=args) def retire_packages(self, pkgnames, branches):

1 0

[packagedb-cli] (3 commits) Created branch orphan_user
by Pierre-YvesChibon 07 Nov '14

07 Nov '14

The branch 'orphan_user' was created. Summary of new commits: a230cdc... Add support for the former_poc argument in the pkgdb2client 513ce60... If an user or a poc are specified, restrict the orphaning t 4a7347e... Fix unorphan action, it requires the new POC to be specifie

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

packagedb-cli-commits November 2014