Bug 1994481

Summary: Review Request: golang-github-openshift-online-ocm-sdk - SDK for the Red Hat OpenShift Cluster Manager
Product: [Fedora] Fedora Reporter: Sanne Raymaekers <sraymaek>
Component: Package ReviewAssignee: Davide Cavalca <davide>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: davide, package-review, sanne.raymaekers
Target Milestone: ---Flags: davide: 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: 2021-08-24 07:56:49 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:

Comment 1 Sanne Raymaekers 2021-08-23 14:35:26 UTC
koji build succeeded https://koji.fedoraproject.org/koji/taskinfo?taskID=74396433

Comment 2 Davide Cavalca 2021-08-23 16:19:21 UTC
Taking this review

Comment 3 Davide Cavalca 2021-08-23 16:34:04 UTC
go2pm package, simplifying review considerably:
- license is correct and acceptable
- dependencies are all packaged
- no notable rpmlint warnings
- package builds and installs without issues

APPROVED

Comment 4 Sanne Raymaekers 2021-08-23 17:22:45 UTC
Thank you for the review!

Comment 5 Sanne Raymaekers 2021-08-23 19:23:58 UTC
*** Bug 1996827 has been marked as a duplicate of this bug. ***

Comment 6 Sanne Raymaekers 2021-08-24 07:56:49 UTC
Closing this, reopened here under correct reporter email https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=1996995 ; if you have time to re-ack it (it's the exact same package) please do. My fault.

Comment 7 Davide Cavalca 2021-08-24 15:39:57 UTC
Looks like Tom Gundersen already took care of it, so you should be good to go.

Comment 8 Sanne Raymaekers 2021-08-24 15:45:32 UTC
Indeed, thank you nonetheless!