[389-devel] Please review: Ticket 49049 - Fix error_log path in defaults.inf for 1.2.11

2016-11-21 Thread Viktor Ashirov 
https://fedorahosted.org/389/ticket/49049

https://fedorahosted.org/389/attachment/ticket/49049/0001-Ticket-49049-Fix-error_log-path-in-defaults.inf-for-.patch


signature.asc
Description: PGP signature
___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Please review: Ticket 49048 - Fix rpm build failure

2016-11-21 Thread Viktor Ashirov 
https://fedorahosted.org/389/ticket/49048

https://fedorahosted.org/389/attachment/ticket/49048/0001-Ticket-49048-Fix-rpm-build-failure.patch


signature.asc
Description: PGP signature
___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Jenkins build is back to normal : 389-DS-NIGHTLY #140

2016-11-21 Thread Jenkins 
See 

___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Build failed in Jenkins: 389-DS-NIGHTLY #139

2016-11-21 Thread Jenkins 
See 


--
[...truncated 3866 lines...]
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[-True-oper_attr_list1]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[-True-oper_attr_list1-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[-True-oper_attr_list1-objectClass]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-False-oper_attr_list2]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-False-oper_attr_list2-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-False-oper_attr_list2-objectClass]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-True-oper_attr_list3]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-True-oper_attr_list3-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[ou=people,dc=example,dc=com-True-oper_attr_list3-objectClass]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-False-oper_attr_list4]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-False-oper_attr_list4-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-False-oper_attr_list4-objectClass]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-True-oper_attr_list5]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-True-oper_attr_list5-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[uid=all_attrs_test,ou=people,dc=example,dc=com-True-oper_attr_list5-objectClass]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[cn=config-False-oper_attr_list6]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[cn=config-False-oper_attr_list6-*]
 PASSED
suites/filter/rfc3673_all_oper_attrs_test.py::test_search_basic[cn=config-False-oper_attr_list6-objectClass]
 PASSED
suites/get_effective_rights/ger_test.py::test_ger_init PASSED
suites/get_effective_rights/ger_test.py::test_ger_ PASSED
suites/gssapi_repl/gssapi_repl_test.py::test_gssapi_repl PASSED
suites/ldapi/ldapi_test.py::test_ldapi_init PASSED
suites/ldapi/ldapi_test.py::test_ldapi_ PASSED
suites/linkedattrs_plugin/linked_attrs_test.py::test_linked_attrs_init PASSED
suites/linkedattrs_plugin/linked_attrs_test.py::test_linked_attrs_ PASSED
suites/mapping_tree/mapping_tree_test.py::test_mapping_tree_init PASSED
suites/mapping_tree/mapping_tree_test.py::test_mapping_tree_ PASSED
suites/memberof_plugin/memberof_test.py::test_memberof_auto_add_oc PASSED
suites/memory_leaks/range_search_test.py::test_range_search_init PASSED
suites/memory_leaks/range_search_test.py::test_range_search PASSED
suites/monitor/monitor_test.py::test_monitor_init PASSED
suites/monitor/monitor_test.py::test_monitor_ PASSED
suites/paged_results/paged_results_test.py::test_search_success[6-5] PASSED
suites/paged_results/paged_results_test.py::test_search_success[5-5] PASSED
suites/paged_results/paged_results_test.py::test_search_success[5-25] PASSED
suites/paged_results/paged_results_test.py::test_search_limits_fail[50-200-cn=config,cn=ldbm
 
database,cn=plugins,cn=config-nsslapd-idlistscanlimit-100-UNWILLING_TO_PERFORM] 
PASSED
suites/paged_results/paged_results_test.py::test_search_limits_fail[5-15-cn=config-nsslapd-timelimit-20-UNAVAILABLE_CRITICAL_EXTENSION]
 PASSED
suites/paged_results/paged_results_test.py::test_search_limits_fail[21-50-cn=config-nsslapd-sizelimit-20-SIZELIMIT_EXCEEDED]
 PASSED
suites/paged_results/paged_results_test.py::test_search_limits_fail[21-50-cn=config-nsslapd-pagedsizelimit-5-SIZELIMIT_EXCEEDED]
 PASSED
suites/paged_results/paged_results_test.py::test_search_limits_fail[5-50-cn=config,cn=ldbm
 database,cn=plugins,cn=config-nsslapd-lookthroughlimit-20-ADMINLIMIT_EXCEEDED] 
PASSED
suites/paged_results/paged_results_test.py::test_search_sort_success PASSED
suites/paged_results/paged_results_test.py::test_search_abandon PASSED
suites/paged_results/paged_results_test.py::test_search_with_timelimit PASSED
suites/paged_results/paged_results_test.py::test_search_dns_ip_aci[dns = 
"localhost.localdomain"] PASSED
suites/paged_results/paged_results_test.py::test_search_dns_ip_aci[ip = "::1" 
or ip = "127.0.0.1"] PASSED
suites/paged_results/paged_results_test.py::test_search_multiple_paging PASSED
suites/paged_results/paged_results_test.py::test_search_invalid_cookie[1000] 
PASSED

[389-devel] Please review: 49042 Fix nightly tests

2016-11-21 Thread William Brown 
https://fedorahosted.org/389/ticket/49042

https://fedorahosted.org/389/attachment/ticket/49042/0001-Ticket-49042-Test-failure-that-expects-old-default.patch


-- 
Sincerely,

William Brown
Software Engineer
Red Hat, Brisbane


signature.asc
Description: This is a digitally signed message part
___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Re: Build failed in Jenkins: 389-DS-NIGHTLY #138

2016-11-21 Thread William Brown 
Hi,

> topology =  0x7fee22ec8e90>
> attr = '\''nsslapd-dbcachesize'\'', expected_value = '\''1000'\'', 
> required = False
> 
> def _check_configured_value(topology, attr=DBLOCK_ATTR_CONFIG, 
> expected_value=None, required=False):
> entries = topology.standalone.search_s(ldbm_config, ldap.SCOPE_BASE, 
> '\''cn=config'\'')
> if required:
> assert(entries[0].hasValue(attr))
> elif entries[0].hasValue(attr):
> >   assert(entries[0].getValue(attr) == expected_value)
> E   assert '\''33554432'\'' == '\''1000'\''
> E - 33554432
> E + 1000
> 

I'm fixing this test now: It's a bit silly, but I broke it because I
increased this default value to make the server perform better ...

Patch shortly. 

-- 
Sincerely,

William Brown
Software Engineer
Red Hat, Brisbane


signature.asc
Description: This is a digitally signed message part
___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Re: Extending the pblock

2016-11-21 Thread William Brown 
On Mon, 2016-11-21 at 12:53 -0500, Mark Reynolds wrote:
> 
> On 11/20/2016 08:17 PM, William Brown wrote:
> > Hi,
> >
> > I have to add some new getters to pblock.c, but I think we should talk
> > about how to add these.
> >
> > Right now, we have a nearly ~2000 line case switch statement in
> > slapi_pblock_get(pb, TYPE, *void). No matter how we cut it, this is
> > pretty insane.
> :-)  Yeah its a bit much at the moment.
> >  We have huge amounts of defines in slapi-plugin.h for
> > this, and we can't expose those to other langs (python, rust), because
> > they are in a C header file.
> >
> > As well, case switch statements that large, at some point, it's going to
> > be inefficient to access. 
> >
> > I think that we should try to break this up, it's just a bit much at
> > this point.
> >
> > So for new additions I would like to use:
> >
> > slapi_pblock_get_(pblock, out) {
> > *out = pb->item
> > }
> 
> This works for me, but if I may, I propose that we tweak the naming
> style to:
> 
> slapi_pb_get_(Slapi_PBlock *pb)
> slapi_pb_set_(Slapi_PBlock *pb, ...)
> 
> This way we differentiate the new functions, and two, it helps keep the
> function name from getting too long.

Okay, that could be the way to go.

I need to get something from the connection, so maybe I'll wire up a
patch as a demo of what I want to achieve. I certainly don't want to
extend the pblock, if anything I want to get us to break it down and
compact it. 

-- 
Sincerely,

William Brown
Software Engineer
Red Hat, Brisbane


signature.asc
Description: This is a digitally signed message part
___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org

[389-devel] Re: Extending the pblock

2016-11-21 Thread Ludwig Krispenz 


On 11/21/2016 02:17 AM, William Brown wrote:

Hi,

I have to add some new getters to pblock.c, but I think we should talk
about how to add these.

Right now, we have a nearly ~2000 line case switch statement in
slapi_pblock_get(pb, TYPE, *void). No matter how we cut it, this is
pretty insane. We have huge amounts of defines in slapi-plugin.h for
this, and we can't expose those to other langs (python, rust), because
they are in a C header file.

As well, case switch statements that large, at some point, it's going to
be inefficient to access.

I think that we should try to break this up, it's just a bit much at
this point.

So for new additions I would like to use:

slapi_pblock_get_(pblock, out) {
 *out = pb->item
}

This is better, as the function is exported in the .so, it's accessible
to other languages, it's simpler to implement, we stop needing more
macro definitions.

It also has performance benefits, as these functions can be inlined by
GCC, the function lookup is likely quicker than the case switch, it's
also easier to set breakpoints on for debugging.

So for now, I want to make new additions to pblock.c in this form.

One day in the future we can tackle breaking pblock.c down from the
current case switch style, but it is not this day.

What do we think? Are we happy for me to use this style for new
additions?
I'm happy with this style of access, but I am not sure if I'm happy with 
new additions to the pblock :-)




___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org


--
Red Hat GmbH, http://www.de.redhat.com/, Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Michael Cunningham, Michael O'Neill, Eric 
Shander

___
389-devel mailing list -- 389-devel@lists.fedoraproject.org
To unsubscribe send an email to 389-devel-le...@lists.fedoraproject.org