Bug 1861716

Summary:	add option for amqp1 SendQueueLimit
Product:	Red Hat OpenStack	Reporter:	Matthias Runge <mrunge>
Component:	puppet-collectd	Assignee:	Matthias Runge <mrunge>
Status:	CLOSED ERRATA	QA Contact:	Leonid Natapov <lnatapov>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	13.0 (Queens)	CC:	jbadiapa, jjoyce, jschluet, lars, lnatapov, mmagr, mrunge, rmccabe, slinaber, tvignaud
Target Milestone:	z13	Keywords:	Triaged, ZStream
Target Release:	13.0 (Queens)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	puppet-collectd-10.1.0-5.el7ost	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:	1861715	Environment:
Last Closed:	2020-10-28 18:24:04 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1861715
Bug Blocks:	1817124

Description Matthias Runge 2020-07-29 11:47:38 UTC

+++ This bug was initially created as a clone of Bug #1861715 +++

Description of problem:
https://github.com/collectd/collectd/commit/e7dd149f6f8279d844d172663023a841aa032a93

added the SendQueueLimit option for the amqp1 plugin, which helps with collectd memory usage.

https://github.com/voxpupuli/puppet-collectd/pull/950
adds the support

This bz is to track the option to downstream

Comment 6 Matthias Runge 2020-08-20 13:08:57 UTC

How to test:
In an environment file add::

ExtraConfig:
    collectd::plugin::amqp1::send_queue_limit: 40

and observe if this parameter SendQueueLimit is added to /etc/collectd.d/10-amqp1.conf

You should also be able to shut off your stf (or stop the route) and you should not see collectd memory usage going up.

Comment 16 Leonid Natapov 2020-10-20 10:43:13 UTC

# Generated by Puppet
<LoadPlugin amqp1>
  Globals false
  Interval 5
</LoadPlugin>

<Plugin amqp1>
  <Transport "metrics">
    Host "172.17.3.126"
    Port "5666"
    User "guest"
    Password "guest"
    Address "collectd"
    RetryDelay 1
    SendQueueLimit 40
    <Instance "notify">
      Format "JSON"
      Notify true
      PreSettle false
    </Instance>
    <Instance "telemetry">
      Format "JSON"
      PreSettle false
    </Instance>
  </Transport>
</Plugin>

collectd memory remains stable

Comment 21 errata-xmlrpc 2020-10-28 18:24:04 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Red Hat OpenStack Platform 13.0 director bug fix advisory), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:4388