Bug 1822686 - Update to python3 and remove python-thrift dependency from python-elasticsearch
Summary: Update to python3 and remove python-thrift dependency from python-elasticsearch
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Logging
Version: 4.4
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
: 4.5.0
Assignee: Jeff Cantrill
QA Contact: Anping Li
URL:
Whiteboard:
Depends On:
Blocks: 1795897 1795899 1795900 1795901 1795902
TreeView+ depends on / blocked
 
Reported: 2020-04-09 15:43 UTC by Jeff Cantrill
Modified: 2020-07-13 17:27 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of: 1795897
Environment:
Last Closed: 2020-07-13 17:26:42 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift origin-aggregated-logging pull 1877 0 None closed Bug 1822686: Require python2-pip in vendoring and removing python-thrift 2020-07-15 17:29:15 UTC
Github openshift origin-aggregated-logging pull 1886 0 None closed Bug 1822686: Update to python3 and remove python-thrift dependency from python-elasticsearch 2020-07-15 17:29:15 UTC
Github openshift origin-aggregated-logging pull 1888 0 None closed Bug 1822686: Dockerfile should install python3 first 2020-07-15 17:29:15 UTC
Red Hat Product Errata RHBA-2020:2409 0 None None None 2020-07-13 17:27:10 UTC

Description Jeff Cantrill 2020-04-09 15:43:25 UTC
+++ This bug was initially created as a clone of Bug #1795897 +++

The python-elasticsearch RPM has a dependency on the python-thrift RPM,
however the Changelog and source code indicates this is no longer needed
(removed in version 1.8.0, 'thrift' does not appear in source).

According to changelog, thrift was removed from elasticsearch-py in 1.8.0 (that was 2015 and Elasticsearch < 2.0.0):
https://github.com/elastic/elasticsearch-py/blob/07fef33accc28e057c9edf2cd730c72d3a835b21/Changelog.rst#180-2015-10-14

The only Logging component using python-elasticsearch seems to be Curator (https://github.com/openshift/origin-aggregated-logging/tree/master/curator).

We should be able to remove the python-thrift dep from python-elasticsearch, and drop the thrift SRPM from OCP 4.4 (and future releases).

Comment 3 Anping Li 2020-04-21 14:22:09 UTC
Move to verified as the description.

Comment 4 Jeff Cantrill 2020-04-22 23:58:10 UTC
Moved back to POST because was unable to build via ART downstream

Comment 9 Anping Li 2020-05-15 14:21:16 UTC
Verified in openshift-ose-logging-curator5:v4.5.0.

Comment 10 errata-xmlrpc 2020-07-13 17:26:42 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:2409


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