Bug 1460740

Summary:	rubygem-fluent-plugin-grok-parser: Fluentd plugin to support Logstash-inspired Grok format
Product:	[Community] RDO	Reporter:	Juan Badia Payno <jbadiapa>
Component:	Package Review	Assignee:	Matthias Runge <mrunge>
Status:	CLOSED CURRENTRELEASE	QA Contact:	hguemar
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	unspecified	CC:	jbadiapa, mrunge
Target Milestone:	---	Flags:	mrunge: rdo-review+
Target Release:	trunk
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-02-05 11:04:06 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Juan Badia Payno 2017-06-12 14:50:46 UTC

SPEC: https://raw.githubusercontent.com/jbadiapa/packages/master/rubygem-fluent-plugin-grok-parser/rubygem-fluent-plugin-grok-parser.spec

SRPM: https://github.com/jbadiapa/packages/raw/master/rubygem-fluent-plugin-grok-parser/rubygem-fluent-plugin-grok-parser-0.3.1-1.el7.centos.src.rpm


Used this configuration for mock: 
https://github.com/jbadiapa/packages/blob/master/helperScripts/mock-jbp-centos-7-base-x86_64.cfg

The output of the fedora-review can be found:
https://github.com/jbadiapa/packages/blob/master/rubygem-fluent-plugin-grok-parser/review.txt

Comment 1 Matthias Runge 2017-06-15 07:41:54 UTC

Reading the review output: 

Note: Directories without known owners:
     /usr/share/gems/specifications, /usr/share/gems/gems/fluent-plugin-
     grok-parser-0.3.1, /usr/share/gems/gems, /usr/share/gems/doc,
     /usr/share/gems

please add 
Requires: rubygems

Otherwise looks ok to me. You can go ahead and request the package to be added to review.rdoproject.org, like this https://review.rdoproject.org/r/7053 or this https://review.rdoproject.org/r/7054

Comment 2 Juan Badia Payno 2017-06-16 13:39:26 UTC

Updated the spec & the srpm files.

Created the review.rdoproject.org request (https://review.rdoproject.org/r/7109)

Comment 3 Matthias Runge 2018-02-05 11:04:06 UTC

It has been merged into opstools. closing this bug.