Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1528255

Summary:	[Docs][RFE][Metrics User] Document postgresql Plugin based on available documentation
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Emma Heftman <eheftman>
Component:	Documentation	Assignee:	Emma Heftman <eheftman>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Avital Pinnick <apinnick>
Severity:	unspecified	Docs Contact:
Priority:	high
Version:	4.2.0	CC:	eheftman, lbopf, lsurette, rbalakri, rhev-docs, sradco, srevivo, ykaul
Target Milestone:	ovirt-4.2.1	Keywords:	FutureFeature, Triaged
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	docs-accepted
Fixed In Version:		Doc Type:	Enhancement
Doc Text:		Story Points:	---
Clone Of:	1528254
Clones:	1528258 (view as bug list)		Environment:
Last Closed:	2018-01-14 12:41:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Docs	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1464013

Description Emma Heftman 2017-12-21 11:11:07 UTC

https://github.com/signalfx/integrations/tree/master/collectd-postgresql/docs

collectd.postgresql.pg_numbackends	
collectd.postgresql.pg_n_tup_g	
collectd.postgresql.pg_n_tup_c	
collectd.postgresql.pg_xact	
collectd.postgresql.pg_db_size	
collectd.postgresql.pg_blks

Comment 1 Lucy Bopf 2017-12-22 00:24:52 UTC

Accepting into Beta 2 program, and assigning to Emma for review.

Comment 2 Emma Heftman 2018-01-01 08:06:42 UTC

Hi Shirly

collectd.postgresql.pg_n_tup_c = derive
collectd.postgresql.pg_xact = derive

Can you please check this is correct. The signalfx docs state that this is a gauge operation but the Google Sheet states that they are derive.
Thanks
Emma

Comment 3 Emma Heftman 2018-01-01 08:18:01 UTC

(In reply to Emma Heftman from comment #2)
> Hi Shirly
> 
> collectd.postgresql.pg_n_tup_c = derive
> collectd.postgresql.pg_xact = derive
> 
> Can you please check this is correct. The signalfx docs state that this is a
> gauge operation but the Google Sheet states that they are derive.
> Thanks
> Emma

And also collectd.postgresql.pg_blks=derive. Please confirm.

Comment 4 Emma Heftman 2018-01-01 08:37:35 UTC

(In reply to Emma Heftman from comment #3)
> (In reply to Emma Heftman from comment #2)
> > Hi Shirly
> > 
> > collectd.postgresql.pg_n_tup_c = derive
> > collectd.postgresql.pg_xact = derive
> > 
> > Can you please check this is correct. The signalfx docs state that this is a
> > gauge operation but the Google Sheet states that they are derive.
> > Thanks
> > Emma
> 
> And also collectd.postgresql.pg_blks=derive. Please confirm.

In addition, on the same issue, do we use tdx_hit? Your table includes tdx_read but doesn't say tdx_hit.

Comment 6 Shirly Radco 2018-01-01 10:33:32 UTC

(In reply to Emma Heftman from comment #3)
> (In reply to Emma Heftman from comment #2)
> > Hi Shirly
> > 
> > collectd.postgresql.pg_n_tup_c = derive
> > collectd.postgresql.pg_xact = derive
> > 
> > Can you please check this is correct. The signalfx docs state that this is a
> > gauge operation but the Google Sheet states that they are derive.
> > Thanks
> > Emma
> 
> And also collectd.postgresql.pg_blks=derive. Please confirm.

Yes. Please fix according to the signalfx docs.

Comment 7 Shirly Radco 2018-01-01 10:34:45 UTC

(In reply to Emma Heftman from comment #4)
> (In reply to Emma Heftman from comment #3)
> > (In reply to Emma Heftman from comment #2)
> > > Hi Shirly
> > > 
> > > collectd.postgresql.pg_n_tup_c = derive
> > > collectd.postgresql.pg_xact = derive
> > > 
> > > Can you please check this is correct. The signalfx docs state that this is a
> > > gauge operation but the Google Sheet states that they are derive.
> > > Thanks
> > > Emma
> > 
> > And also collectd.postgresql.pg_blks=derive. Please confirm.
> 
> In addition, on the same issue, do we use tdx_hit? Your table includes
> tdx_read but doesn't say tdx_hit.

Where do you see  tdx_read and tdx_hit?

Comment 9 Emma Heftman 2018-01-02 11:05:09 UTC

After discussing this plugin with Shirly, it appears that there are actually many more plugins that need to be documented. Shirly will open a bug for herself, and I will update this bug with the full list.

(In reply to Shirly Radco from comment #7)
> (In reply to Emma Heftman from comment #4)
> > (In reply to Emma Heftman from comment #3)
> > > (In reply to Emma Heftman from comment #2)
> > > > Hi Shirly
> > > > 
> > > > collectd.postgresql.pg_n_tup_c = derive
> > > > collectd.postgresql.pg_xact = derive
> > > > 
> > > > Can you please check this is correct. The signalfx docs state that this is a
> > > > gauge operation but the Google Sheet states that they are derive.
> > > > Thanks
> > > > Emma
> > > 
> > > And also collectd.postgresql.pg_blks=derive. Please confirm.
> > 
> > In addition, on the same issue, do we use tdx_hit? Your table includes
> > tdx_read but doesn't say tdx_hit.
> 
> Where do you see  tdx_read and tdx_hit?

https://github.com/signalfx/integrations/blob/master/collectd-postgresql/docs/pg_blks.tidx_hit.md