Make the OpenLDAP Input plugin gather all operations metrics

[mjs510]

Use Case

I would like to be able to easily report totals for all LDAP operations that have taken place. This information is available in OpenLDAP's cn=monitor backend but the Telegraf OpenLDAP Input plugin does not gather the required metrics.

Expected behavior

When using the OpenLDAP input plugin, I would expect the monitorOpCompleted and monitorOpInitiated values to be gathered from cn=Operations,cn=monitor and converted into metrics.

Actual behavior

The information on total ops completed and total ops inititiated is not available in the gathered metrics. This information is present in cn=Operations,cn=monitor via its monitorOpCompleted and monitorOpInitiated attributes. The OpenLdap Input plugin gathers per-operation information from the children of cn=Operations,cn=monitor but does not collect the required attributes from the object itself.

Additional info

cn=Operations,cn=monitor has objectClass monitorContainer so is currently excluded from the query performed by the Input plugin.

Some other monitorContainer objects also provide information directly via their main entry. For example, cn=Overlays,cn=monitor reports the available overlays in its monitoredInfo attribute.

[powersj]

The OpenLdap Input plugin gathers per-operation information from the children of cn=Operations,cn=monitor but does not collect the required attributes from the object itself.

You seem to understand the layout in OpenLDAP. Is this a second search that has to occur? Do you understand enough to make the change to collect these values?

[mjs510]

These could probably be gathered with a second search or by modifying the current search to include the monitorContainer objectClass and then making the necessary changes to the existing logic to handle this.

I will create a PR for this if I'm able but in the meantime I just want to make sure this issue is captured as clearly as possible.

[srebhan]

@mjs510 can you please test the binary in PR #14092 available once CI finished all tests successfully! You need to use the ldap input plugin (with dialect = "openldap") instead of the openldap one.
Let me know if this fixes your issue!

[mjs510]

@mjs510 can you please test the binary in PR #14092 available once CI finished all tests successfully! You need to use the ldap input plugin (with dialect = "openldap") instead of the openldap one. Let me know if this fixes your issue!

@srebhan Many thanks but I'm not able to do this at present.