Bug 1795901 - Remove python-thrift dependency from python-elasticsearch
Summary: Remove python-thrift dependency from python-elasticsearch
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Logging
Version: 4.1.z
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 4.1.z
Assignee: Jeff Cantrill
QA Contact: Anping Li
URL:
Whiteboard:
Depends On: 1795897 1795899 1795900 1822686
Blocks: 1795902
TreeView+ depends on / blocked
 
Reported: 2020-01-29 07:30 UTC by Sam Fowler
Modified: 2020-04-28 18:44 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1795900
: 1795902 (view as bug list)
Environment:
Last Closed: 2020-04-28 18:44:16 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)

Description Sam Fowler 2020-01-29 07:30:26 UTC 
+++ This bug was initially created as a clone of Bug #1795900 +++

+++ This bug was initially created as a clone of Bug #1795899 +++

+++ 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).
Note You need to log in before you can comment on or make changes to this bug.