Bug 2072688

Summary: Drop requirement on golang/x/crypto [16.1]
Product: Red Hat OpenStack Reporter: Lon Hohberger <lhh>
Component: distributionAssignee: Lon Hohberger <lhh>
Status: CLOSED WONTFIX QA Contact: Lon Hohberger <lhh>
Severity: low Docs Contact:
Priority: high    
Version: 17.0 (Wallaby)CC: jjoyce, jschluet, mburns, shrjoshi
Target Milestone: z10Keywords: Triaged
Target Release: 16.1 (Train on RHEL 8.2)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: golang-github-Sirupsen-logrus-1.1.1-6.el8ost golang-github-vbatts-tar-split-0.11.1-7.1.el8ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 2072119 Environment:
Last Closed: 2022-12-21 17:45:54 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: 2072119    
Bug Blocks:    

Description Lon Hohberger 2022-04-06 18:46:33 UTC 
+++ This bug was initially created as a clone of Bug #2072119 +++

Description of problem:

Golang is static binaries, and cryptography has been a source of problems.

We ship a binary package called "tar-split", needed by openstack-tripleo-common. This requires logrus (which is a structured logging facility). Unfortunately, logrus imports golang/x/crypto in order to determine whether or not something is a terminal.

Rather than do that, let's use a syscall directly, so we do not end up with the maintenance burden of golang/x/crypto.

golang-googlecode-go-crypto
    golang-github-Sirupsen-logrus | # let me see if I can patch it out here
        golang-github-vbatts-tar-split
            openstack-tripleo-common


Version-Release number of selected component (if applicable): golang-github-Sirupsen-logrus-1.1.1-5.el9ost

--- Additional comment from Lon Hohberger on 2022-04-05 16:12:28 UTC ---

I've been able to patch out golang/x/crypto and unit tests all pass for Sirupsen-logrus.
Comment 3 Lon Hohberger 2022-11-02 13:36:56 UTC 
These are not in the present compose; moving to z10