Bug 2072688 - Drop requirement on golang/x/crypto [16.1]
Summary: Drop requirement on golang/x/crypto [16.1]
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: distribution
Version: 17.0 (Wallaby)
Hardware: Unspecified
OS: Unspecified
high
low
Target Milestone: z10
: 16.1 (Train on RHEL 8.2)
Assignee: Lon Hohberger
QA Contact: Lon Hohberger
URL:
Whiteboard:
Depends On: 2072119
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-04-06 18:46 UTC by Lon Hohberger
Modified: 2022-12-21 17:45 UTC (History)
4 users (show)

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:
Clone Of: 2072119
Environment:
Last Closed: 2022-12-21 17:45:54 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker OSP-15180 0 None None None 2022-05-11 17:29:29 UTC

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


Note You need to log in before you can comment on or make changes to this bug.