Re: [Freeipa-devel] [PATCH] Use Exception class instead of BaseException

2015-06-02 Thread Niranjan
Niranjan wrote: Greetings, I would like to present patch for replacing StandardError exception with Exception class in ipapython/adminutil.py. Also replacing BaseException class with Exception class. Though the use of StandardError is many places. I would like to start with ipapython

Re: [Freeipa-devel] [PATCH] Use Exception class instead of BaseException

2015-06-03 Thread Niranjan
Petr Viktorin wrote: On 06/01/2015 06:33 AM, Niranjan wrote: Greetings, I would like to present patch for replacing StandardError exception with Exception class in ipapython/adminutil.py. Also replacing BaseException class with Exception class. Though the use of StandardError

[Freeipa-devel] [PATCH] Use Exception class instead of StandardError

2015-06-09 Thread Niranjan
Niranjan wrote: Greetings, Please find the modified patch for ipapython/adminutil.py. I have run few tests manually like running ipa-server-install as non-root user or provide --quiet and --verbose to see if it raises ScriptError properly. Also i checked by running ipa-server-install

Re: [Freeipa-devel] [PATCH] Use Exception class instead of StandardError

2015-06-15 Thread Niranjan
Niranjan wrote: Niranjan wrote: Greetings, Please find the modified patch for ipapython/adminutil.py. I have run few tests manually like running ipa-server-install as non-root user or provide --quiet and --verbose to see if it raises ScriptError properly. Also i checked by running

[Freeipa-devel] [PATCH] Use Exception class instead of BaseException

2015-05-31 Thread Niranjan
first patch. Please let me know if my approach on this is correct. Regards Niranjan From 018312f76952ea86c8c6e2396657e0531d2d61ba Mon Sep 17 00:00:00 2001 From: Niranjan Mallapadi mrniran...@redhat.com Date: Mon, 1 Jun 2015 09:41:05 +0530 Subject: [PATCH] Use Exception class instead

Re: [Freeipa-devel] [patch]-pytest-multihost-Return File Attributes to sftp.put

2015-08-13 Thread Niranjan
Tomas Babej wrote: On 08/13/2015 01:55 PM, Niranjan wrote: Greetings, This patch is regarding pytest-multihost plugin. Including a patch to return FileAttributes for sftp.put function used in the function. Current put_file function in transport.py in ParamikoTransport Class

Re: [Freeipa-devel] [patch]-pytest-multihost-Return File Attributes to sftp.put

2015-08-13 Thread Niranjan
Martin Basti wrote: On 08/13/2015 01:55 PM, Niranjan wrote: Greetings, This patch is regarding pytest-multihost plugin. Including a patch to return FileAttributes for sftp.put function used in the function. Current put_file function in transport.py in ParamikoTransport Class doesn't

[Freeipa-devel] [patch]-pytest-multihost-Return File Attributes to sftp.put

2015-08-13 Thread Niranjan
if the operation was sucessfull or not. Returning FileAttributes to put_file function helps in checking if operation was indeed succesful. Requesting feedback on the patch attached. Regards Niranjan From 5268b93efec39fc0c04f5bd022497143622e87b4 Mon Sep 17 00:00:00 2001 From: Niranjan MR mrniran

Re: [Freeipa-devel] [PATCH] Use Exception class instead of StandardError

2015-06-29 Thread Niranjan
Martin Basti wrote: On 10/06/15 00:59, Niranjan wrote: Niranjan wrote: Greetings, Please find the modified patch for ipapython/adminutil.py. I have run few tests manually like running ipa-server-install as non-root user or provide --quiet and --verbose to see if it raises ScriptError

[Freeipa-devel] Query regarding pytest-multihost plugin

2015-08-12 Thread Niranjan
.adomain.test, (currently this is the behaviour that i am seeing, it ignore external_hostname) 1.https://pypi.python.org/pypi/pytest-multihost Regards Niranjan pgpTLKyYxSCDu.pgp Description: PGP signature -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com

[Freeipa-devel] [PATCH] enable pem=True in export_pem_cert function

2015-10-26 Thread Niranjan
Greetings, export_pem_cert function from ipapython/certdb should export the certificate in pem format but instead exports the cert in der format as it doesn't enable pem=True. This patch specifies pem=True for export_pem_cert function Regards Niranjan From

Re: [Freeipa-devel] [PATCH] enable pem=True in export_pem_cert function

2015-10-27 Thread Niranjan
Tomas Babej wrote: > On 10/26/2015 08:59 PM, Niranjan wrote: > > Greetings, > > > > export_pem_cert function from ipapython/certdb should export the > > certificate > > in pem format but instead exports the cert in der format as it doesn't > > enable pem=

Re: [Freeipa-devel] [PATCH] enable pem=True in export_pem_cert function

2015-11-10 Thread Niranjan
Niranjan wrote: > Tomas Babej wrote: > > On 10/26/2015 08:59 PM, Niranjan wrote: > > > Greetings, > > > > > > export_pem_cert function from ipapython/certdb should export the > > > certificate > > > in pem format but instead exports the c

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-02-29 Thread Niranjan
Niranjan wrote: > Greetings, > > Request to review the patch for > https://fedorahosted.org/python-pytest-multihost/ticket/5 > > Regards > Niranjan > > From 28cd560f58b96817306a6eae6976e26166a9351a Mon Sep 17 00:00:00 2001 > From: Niranjan MR <mrniran...@fedor

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-02-29 Thread Niranjan
Greetings, Attached wrong patch in my last mail. Request to review the patch attached to this email Regards Niranjan From 0bed8ae52972b782158394179b868a9ff028b5e6 Mon Sep 17 00:00:00 2001 From: Niranjan MR <mrniran...@fedoraproject.org> Date: Fri, 26 Feb 2016 15:30:25 +0530 Subject:

[Freeipa-devel] Request to add support to specify username/password for each host

2016-02-26 Thread Niranjan
Greetings, Request to review the patch for https://fedorahosted.org/python-pytest-multihost/ticket/5 Regards Niranjan From 28cd560f58b96817306a6eae6976e26166a9351a Mon Sep 17 00:00:00 2001 From: Niranjan MR <mrniran...@fedoraproject.org> Date: Fri, 26 Feb 2016 15:30:25 +0530 Subject:

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-07 Thread Niranjan
Petr Viktorin wrote: > On 04/06/2016 10:55 AM, Niranjan wrote: > > Greetings, > > > > For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have > > proposed > > a patch, I think this patch is more of a workaround , than a solution. I > > w

[Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-06 Thread Niranjan
with openssh/bash to be installed. Request to review the patch. Regards Niranjan From 5944e86f881516209fcb030ec3da7ad1b58ab4cc Mon Sep 17 00:00:00 2001 From: Niranjan MR <mrniran...@fedoraproject.org> Date: Wed, 6 Apr 2016 14:08:13 +0530 Subject: [PATCH] Add new parameter "type" f

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-13 Thread Niranjan
Niranjan wrote: > Petr Viktorin wrote: > > On 04/06/2016 10:55 AM, Niranjan wrote: > > > Greetings, > > > > > > For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have > > > proposed > > > a patch, I think this patch is

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-08 Thread Niranjan
Petr Viktorin wrote: > On 04/07/2016 10:12 PM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/06/2016 10:55 AM, Niranjan wrote: > >>> Greetings, > >>> > >>> For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have &g

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-12 Thread Niranjan
Petr Viktorin wrote: > On 04/06/2016 10:55 AM, Niranjan wrote: > > Greetings, > > > > For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have > > proposed > > a patch, I think this patch is more of a workaround , than a solution. I > > w

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-03-04 Thread Niranjan
Niranjan wrote: > Petr Viktorin wrote: > > On 03/01/2016 05:18 PM, Petr Viktorin wrote: > > > On 02/29/2016 05:57 PM, Niranjan wrote: > > >> Greetings, > > >> > > >> Attached wrong patch in my last mail. Request to review the patch &

Re: [Freeipa-devel] [python-pytest-multihost] Request to add support to specify username/password for each host

2016-03-03 Thread Niranjan
Petr Viktorin wrote: > On 03/01/2016 05:18 PM, Petr Viktorin wrote: > > On 02/29/2016 05:57 PM, Niranjan wrote: > >> Greetings, > >> > >> Attached wrong patch in my last mail. Request to review the patch > >> attached to this email > >> > &

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-22 Thread Niranjan
Petr Viktorin wrote: > On 04/21/2016 03:04 PM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/21/2016 07:25 AM, Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/20/2016 06:11 AM, Niranjan wrote: > >>>>> Petr

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-19 Thread Niranjan
Petr Viktorin wrote: > On 04/18/2016 12:39 PM, Niranjan wrote: > > Niranjan wrote: > >> Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/06/2016 10:55 AM, Niranjan wrote: > >>>>> Greetings, > >>>>&g

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-18 Thread Niranjan
Niranjan wrote: > Niranjan wrote: > > Petr Viktorin wrote: > > > On 04/06/2016 10:55 AM, Niranjan wrote: > > > > Greetings, > > > > > > > > For https://fedorahosted.org/python-pytest-multihost/ticket/6 , i have > > > > proposed

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-18 Thread Niranjan
Petr Viktorin wrote: > On 04/18/2016 12:39 PM, Niranjan wrote: > > Niranjan wrote: > >> Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/06/2016 10:55 AM, Niranjan wrote: > >>>>> Greetings, > >>>>&g

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-19 Thread Niranjan
Petr Viktorin wrote: > On 04/18/2016 12:39 PM, Niranjan wrote: > > Niranjan wrote: > >> Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/06/2016 10:55 AM, Niranjan wrote: > >>>>> Greetings, > >>>>&g

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-21 Thread Niranjan
Petr Viktorin wrote: > On 04/21/2016 07:25 AM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/20/2016 06:11 AM, Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/18/2016 12:39 PM, Niranjan wrote: > >>>>> Niranja

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-21 Thread Niranjan
Petr Viktorin wrote: > On 04/21/2016 07:25 AM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/20/2016 06:11 AM, Niranjan wrote: > >>> Petr Viktorin wrote: > >>>> On 04/18/2016 12:39 PM, Niranjan wrote: > >>>>> Niranja

Re: [Freeipa-devel] [python-pytest-multihost]-Ticket-6 run_command produces exit code 1 on windows

2016-04-20 Thread Niranjan
Petr Viktorin wrote: > On 04/20/2016 06:11 AM, Niranjan wrote: > > Petr Viktorin wrote: > >> On 04/18/2016 12:39 PM, Niranjan wrote: > >>> Niranjan wrote: > >>>> Niranjan wrote: > >>>>> Petr Viktorin wrote: >

[Freeipa-devel] [patch]pytest-multihost: Add external_ip parameter to specify external_ip when using openstack

2016-11-17 Thread Niranjan
Greetings, When using pytest multihost to connect with hosts provisioned in openstack, it's required to have ability for the test to use floating ip[external ip]. This patch adds another attribute external_ip parameter under hosts . Regards Niranjan From