Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

In lib389 - MonitorLDBM class does not work properly when we turn off nsslapd-ndn-cache-enabled #2939

Open
389-ds-bot opened this issue Sep 13, 2020 · 5 comments
Labels
lib389 Involves lib389 librabry
Milestone

Comments

@389-ds-bot
Copy link

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/49880


Issue Description

With the help of src/lib389/lib389/monitor.py
I was trying to assert ::
log.info("Check the backend monitor output for Normalized DN cache statistics while nsslapd-ndn-cache-enabled is on")
topo.standalone.config.set('nsslapd-ndn-cache-enabled', 'on')
monitor_backend = MonitorBackend(topo.standalone)
assert monitor_backend.present('normalizeddncachehits')

so it should be able to check normalizeddncachehits is present in monitor_backend when nsslapd-ndn-cache-enabled is on, but it does not work and gives error 🚬
E TypeError: search_ext() argument 1 must be str, not None

Full logs - http://pastebin.test.redhat.com/626448

@389-ds-bot 389-ds-bot added the lib389 Involves lib389 librabry label Sep 13, 2020
@389-ds-bot 389-ds-bot added this to the 1.4.3 milestone Sep 13, 2020
@389-ds-bot
Copy link
Author

Comment from amsharma (@amsharma3) at 2018-08-02 16:16:37

Also there is a weird check for the version in src/lib389/lib389/monitor.py in init for MonitorLDBM class. so even if we have the right version, but we turn off nsslapd-ndn-cache-enabled, the code will fail

@389-ds-bot
Copy link
Author

Comment from amsharma (@amsharma3) at 2018-08-02 16:16:40

Metadata Update from @amsharma3:

  • Custom field component adjusted to None
  • Custom field origin adjusted to None
  • Custom field reviewstatus adjusted to None
  • Custom field type adjusted to None
  • Custom field version adjusted to None

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2018-08-09 17:56:09

Metadata Update from @mreynolds389:

  • Issue set to the milestone: 1.4.0

@389-ds-bot
Copy link
Author

Comment from firstyear (@Firstyear) at 2019-02-04 02:41:37

I think the issue is we allow disabling of ndn cache. We should remove this parameter instead.

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2020-02-12 16:59:15

Metadata Update from @mreynolds389:

  • Issue set to the milestone: 1.4.3 (was: 1.4.0)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lib389 Involves lib389 librabry
Projects
None yet
Development

No branches or pull requests

2 participants