Re: [389-devel] Please review lib389 ticket 47568: Rename DSAdmin class (2nd)

2013-10-25 Thread Roberto Polli
that should test just Replica. I would use harness and the standard python-ldap methods in setup/teardown, so that we can change the Backend and Replica class without at least breaking the tests. Let me know + Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T

[389-devel] lib389: cleanup __init__

2013-10-31 Thread Roberto Polli
. If we want to add a cache layer we can do it afterward. And with some cache pattern. Let me know + Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma

Re: [389-devel] lib389: cleanup __init__

2013-10-31 Thread Roberto Polli
caching - as they should be quite unmutable, and I was talking about __initPart2() which is called by __init__. Are all the __initPart2() attributes essential? Peace, R: -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680

Re: [389-devel] lib389: cleanup __init__

2013-10-31 Thread Roberto Polli
. We could put that in Config. Peace, R: -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere

Re: [389-devel] Please review lib389 ticket 47584: CI tests: add backup/restore of an instance

2013-11-06 Thread Roberto Polli
/02/python-logging-performance/ - http://stackoverflow.com/questions/4148790/lazy-logger-message-string-evaluation Peace, R: -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040

Re: [389-devel] Please review lib389 ticket 47584 (take #2): CI tests: add backup/restore of an instance

2013-11-06 Thread Roberto Polli
-of-an-insta.patch -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere confidenziale per i destinatari

Re: [389-devel] Please review bug 47586: CI tests: test case for 47490

2013-11-15 Thread Roberto Polli
people/projects using lib389 will be full of our testing code. Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i

[389-devel] Running lib389 tests

2014-01-23 Thread Roberto Polli
): # TCP stuff goes here Can you please tell me: 1- how to run tests 2- which tests should work 3- which is the test environment. Thx+Peace, R. On Thursday 09 January 2014 10:29:10 thierry bordaz wrote: On 01/08/2014 12:54 PM, Roberto Polli wrote: Hi Thierry, before all sorry

Re: [389-devel] 389-ds-base: /bin/sh scripts should use . instead of source

2014-01-24 Thread Roberto Polli
On Friday 24 January 2014 10:02:02 Rob Crittenden wrote: Roberto Polli wrote: On Friday 24 January 2014 07:28:51 Rich Megginson wrote: https://fedorahosted.org/389/ticket/47511 This is just the tracking ticket for the work. Ok. Hope won't take too much (for now I'm fine as I've fixed my

Re: [389-devel] Please review lib389 ticket 47568: Rename DSAdmin class (2nd)

2013-10-25 Thread Roberto Polli
dependencies distracting me. Let me know + Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di

Re: [389-devel] Proof of concept: mocking DS in lib389

2013-10-28 Thread Roberto Polli
this, but not with Roberto. Saw the code, it just imports all files in the folder, right? It's a nice trick, even if it makes the design a bit complex. Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza

Re: [389-devel] Proof of concept: mocking DS in lib389

2013-10-29 Thread Roberto Polli
with some code in our hands, we will be able to understand each other on this 'mocking' issue and come to conclusions more quickly. Let me know what you think. Thank you, Jan -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F

Re: [389-devel] Please review lib389 ticket 47575: add test case for ticket47560

2013-10-30 Thread Roberto Polli
all ticket test cases that are applicable on standalone topology. https://fedorahosted.org/389/attachment/ticket/47575/0001-Ticket-47575-CI-te st-add-test-case-for-ticket47560.patch regards thierry -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T

Re: [389-devel] Please review lib389 ticket 47584: CI tests: add backup/restore of an instance

2013-11-06 Thread Roberto Polli
() Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere confidenziale per i destinatari

Re: [389-devel] Please review ticket 47590: add/split functions around replication

2013-11-14 Thread Roberto Polli
wrote: In order to implement the first CI test with replication instances, I reorganised lib389 functions related to replication setup. https://fedorahosted.org/389/attachment/ticket/47590/0001-Ticket-47590-CI-te sts-add-split-functions-around-rep.patch -- Roberto Polli Community Manager Babel

Re: [389-devel] Please review ticket 47590: add/split functions around replication

2013-11-15 Thread Roberto Polli
with replication instances, I reorganised lib389 functions related to replication setup. https://fedorahosted.org/389/attachment/ticket/47590/0001-Ticket-47590-CI -te sts-add-split-functions-around-rep.patch -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T

Re: [389-devel] Please review ticket 47590 (take #2): add/split functions around replication

2013-11-18 Thread Roberto Polli
(). The replica is inferred from the context! 5- If you move the agreement stuff outside Replica, I would just name the class Agreement and set self.agreement = Agreement(self) or self.replica_agreement -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M

Re: [389-devel] Please review CI tests: fix for test case 47490

2013-12-02 Thread Roberto Polli
Sorry T, but I'm still not fine. Hope to give you some feedback in week. Peace, R. -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo

Re: [389-devel] Please review (tests) 47628: port test cases to new DirSrv interface

2014-01-03 Thread Roberto Polli
be easily implemented in a subclass. Let me know + Peace (and sorry again for my absence), R. https://fedorahosted.org/389/attachment/ticket/47628/0001-Ticket-47628-port- testcases-to-new-DirSrv-interface.patch -- Roberto Polli Community Manager Babel S.r.l. - http://www.babel.it - una

Re: [389-devel] 389-ds-base: /bin/sh scripts should use . instead of source

2014-01-24 Thread Roberto Polli
On Friday 24 January 2014 07:28:51 Rich Megginson wrote: https://fedorahosted.org/389/ticket/47511 sorry for the double posting :( I just cloned the master repository and it's not fixed. Peace, R. -- Roberto Polli Community Manager Babel - a business unit of Par-Tec S.p.A. - http

Re: [389-devel] Running lib389 tests

2014-01-24 Thread Roberto Polli
? in the library code? ) Peace, R. -- Roberto Polli Community Manager Babel - a business unit of Par-Tec S.p.A. - http://www.babel.it T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680 P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma) CONFIDENZIALE: Questo messaggio ed i suoi allegati sono