#50109 Issue 49618 - Set nsslapd-cachememsize to custom value
Closed by spichugi. Opened by bsmejkal.
bsmejkal/389-ds-base ds49618  into  master

Download 50109.patch

Description:
Added function for getting available memory.
Added test case to set nsslapd-cachememsize to custom value above 3805132804 bytes. Test is skipped if available memory is lower than we want to set.

https://pagure.io/389-ds-base/issue/49618

Reviewed by: vashirov (Thanks!)

We avoid using DSLdapObject directly. It is a private class by design.
You can use LDBMConfig(DSLdapObject) here (config.py)

And for particular backends you can use Backends(DSLdapObjects) object and get the one you need like this

backends = Backends(topo.standalone)
userroot_ldbm = backends.get("userroot")

Besides that - looks really good!
Nice job!

rebased onto 068e248e538dffcb144be321bb0273085d7516a8

@spichugi
Thanks! Could you please review the last change? :)

LGTM! Ack
Please, rebase with the latest master and we can merge

Agreed, this looks good. Was this test addressing a specific problem though? I think the auto tuning tests that @spichugi wrote a long time ago would have already checked this case ....

This issue was reported by customer, when autotuning is off. The tests that you have mentioned were checking correctness with autotuning on.

rebased onto f574332fc826da4f22b10187bbae29ec3213bb6d

Pull-Request has been merged by vashirov

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3168

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata