Re: [Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-09-16 Thread Petr Vobornik 

On 09/15/2015 03:01 PM, Abhijeet Kasurde wrote:

Ping

On 09/10/2015 11:52 AM, Abhijeet Kasurde wrote:

Hi List,

Please find the following patch with review comments.

Thanks,
Abhijeet Kasurde

On 09/03/2015 06:10 PM, Tomas Babej wrote:


On 09/03/2015 08:16 AM, Abhijeet Kasurde wrote:

Ping

On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:

Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde

ACK, the patch needs a minor rebase on master due to python3
refactoring.




Pushed to master: 1b70521e6b9a16fc0d06b798b8cc1f8b943c476a
Pushed to ipa-4-2: 72e87e8c33d3aa7d777e3a097bdefc95a52e014e
--
Petr Vobornik

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-09-15 Thread Abhijeet Kasurde 

Ping

On 09/10/2015 11:52 AM, Abhijeet Kasurde wrote:

Hi List,

Please find the following patch with review comments.

Thanks,
Abhijeet Kasurde

On 09/03/2015 06:10 PM, Tomas Babej wrote:


On 09/03/2015 08:16 AM, Abhijeet Kasurde wrote:

Ping

On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:

Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde
ACK, the patch needs a minor rebase on master due to python3 
refactoring.






-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-09-10 Thread Abhijeet Kasurde 

Hi List,

Please find the following patch with review comments.

Thanks,
Abhijeet Kasurde

On 09/03/2015 06:10 PM, Tomas Babej wrote:


On 09/03/2015 08:16 AM, Abhijeet Kasurde wrote:

Ping

On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:

Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde

ACK, the patch needs a minor rebase on master due to python3 refactoring.


From 41536f492fb7302d4c2e68b195cdffd94fc97cc7 Mon Sep 17 00:00:00 2001
From: Abhijeet Kasurde 
Date: Thu, 10 Sep 2015 11:46:01 +0530
Subject: [PATCH] Updated number of legacy permission in ipatests

Since IPA 4.2 has an additional permission
"Request Certificate ignoring CA ACLs", the number of legacy
permission in testcase is updated from 8 to 9.

https://fedorahosted.org/freeipa/ticket/5264

Signed off-by: Abhijeet Kasurde 
---
 ipatests/test_xmlrpc/test_permission_plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipatests/test_xmlrpc/test_permission_plugin.py b/ipatests/test_xmlrpc/test_permission_plugin.py
index e791b598149fb9b5291687b101428107415dfc85..88401190a01d8d8420f135ccd8e204172d2ecca2 100644
--- a/ipatests/test_xmlrpc/test_permission_plugin.py
+++ b/ipatests/test_xmlrpc/test_permission_plugin.py
@@ -2849,7 +2849,7 @@ def check_legacy_results(results):
 legacy_permissions = [p for p in results
   if not p.get('ipapermissiontype')]
 print(legacy_permissions)
-assert len(legacy_permissions) == 8, len(legacy_permissions)
+assert len(legacy_permissions) == 9, len(legacy_permissions)
 return True
 
 class test_permission_legacy(Declarative):
-- 
2.4.3

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-09-03 Thread Abhijeet Kasurde 

Ping

On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:

Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde


--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-09-03 Thread Tomas Babej 


On 09/03/2015 08:16 AM, Abhijeet Kasurde wrote:
> Ping
> 
> On 08/27/2015 10:37 AM, Abhijeet Kasurde wrote:
>> Hi All,
>>
>> This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264
>>
>> Thanks,
>> Abhijeet Kasurde
> 

ACK, the patch needs a minor rebase on master due to python3 refactoring.

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [PATCH] Updated no of legacy permission in ipatests

2015-08-26 Thread Abhijeet Kasurde 

Hi All,

This patch fixes bug - https://fedorahosted.org/freeipa/ticket/5264

Thanks,
Abhijeet Kasurde
From 4492ddd0e07c5c82e8fbe2d7ae88e5fb0bce5be0 Mon Sep 17 00:00:00 2001
From: Abhijeet Kasurde akasu...@redhat.com
Date: Thu, 27 Aug 2015 10:31:35 +0530
Subject: [PATCH] Updated number of legacy permission in ipatests

Since IPA 4.2 has an additional permission
Request Certificate ignoring CA ACLs, the number of legacy
permission in testcase is updated from 8 to 9.

https://fedorahosted.org/freeipa/ticket/5264

Signed-off-by: Abhijeet Kasurde akasu...@redhat.com
---
 ipatests/test_xmlrpc/test_permission_plugin.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ipatests/test_xmlrpc/test_permission_plugin.py b/ipatests/test_xmlrpc/test_permission_plugin.py
index c899c428edfcd19c2e7f538cdc38b693e11c8715..fd69ff25dcded412c78c38ccffd0d39413ea6943 100644
--- a/ipatests/test_xmlrpc/test_permission_plugin.py
+++ b/ipatests/test_xmlrpc/test_permission_plugin.py
@@ -2849,7 +2849,7 @@ def check_legacy_results(results):
 legacy_permissions = [p for p in results
   if not p.get('ipapermissiontype')]
 print legacy_permissions
-assert len(legacy_permissions) == 8, len(legacy_permissions)
+assert len(legacy_permissions) == 9, len(legacy_permissions)
 return True
 
 class test_permission_legacy(Declarative):
-- 
2.4.3

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code