Bug 1822214

Summary: Review Request: golang-gopkg-olivere-elastic-5 - Elasticsearch 5 client for Go
Product: [Fedora] Fedora Reporter: Fabian Affolter <mail>
Component: Package ReviewAssignee: Robert-André Mauchin 🐧 <eclipseo>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: eclipseo, package-review
Target Milestone: ---Flags: eclipseo: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-08-22 19:30:11 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: 1821922    
Bug Blocks:    

Description Fabian Affolter 2020-04-08 13:59:09 UTC
Spec URL: https://fab.fedorapeople.org/packages/SRPMS/golang-github-olivere-elastic-5.spec
SRPM URL: https://fab.fedorapeople.org/packages/SRPMS/golang-github-olivere-elastic-5-5.0.84-1.fc31.src.rpm

Project URL: https://github.com/olivere/elastic

Description:
Elasticsearch 5 client for Go.

Koji scratch build:
fails due to missing dependency

rpmlint output:
$ rpmlint golang-github-olivere-elastic-5-5.0.84-1.fc31.src.rpm 
golang-github-olivere-elastic-5.src: W: spelling-error Summary(en_US) Elasticsearch -> Elastic search, Elastic-search, Elasticated
golang-github-olivere-elastic-5.src: W: spelling-error %description -l en_US Elasticsearch -> Elastic search, Elastic-search, Elasticated
golang-github-olivere-elastic-5.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

$ rpmlint golang-github-olivere-elastic-5-devel-5.0.84-1.fc31.noarch.rpm 
golang-github-olivere-elastic-5-devel.noarch: W: spelling-error Summary(en_US) Elasticsearch -> Elastic search, Elastic-search, Elasticated
golang-github-olivere-elastic-5-devel.noarch: W: spelling-error %description -l en_US Elasticsearch -> Elastic search, Elastic-search, Elasticated
golang-github-olivere-elastic-5-devel.noarch: W: hidden-file-or-dir /usr/share/gocode/src/github.com/olivere/elastic.v5/.goipath
1 packages and 0 specfiles checked; 0 errors, 3 warnings.

Fedora Account System Username: fab

Comment 1 Robert-André Mauchin 🐧 2020-06-21 13:56:22 UTC
SPEC and bug name should be named golang-gopkg-olivere-elastic-5. This is the name derived from the go import path.

 - License ok
 - Latest version packaged
 - Builds in mock
 - No rpmlint errors
 - Conforms to Packaging Guidelines

Package approved. Please fix name of the SPEC and bug before requesting the repo.

Comment 2 Gwyn Ciesla 2020-07-03 13:32:10 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-gopkg-olivere-elastic-5

Comment 3 Fabian Affolter 2020-08-22 19:30:11 UTC
Thanks for the review.