Remove obsolete redhat-eventlet.patch
See I62ce43a330d7ae94eda4c7498782a655e63747fa for the gorey details on why this exists. As of this fix: https://github.com/eventlet/eventlet/pull/34 which was released in eventlet 0.13, we no longer need the patch. This has now been removed from oslo-incubator, so this is really just syncing that removal. Change-Id: I84267f3c6726cb2e750f615e107c48b12c6ed353
This commit is contained in:
parent
3f24fee218
commit
ae0263e849
@ -1,16 +0,0 @@
|
|||||||
--- .nova-venv/lib/python2.6/site-packages/eventlet/green/subprocess.py.orig
|
|
||||||
2011-05-25
|
|
||||||
23:31:34.597271402 +0000
|
|
||||||
+++ .nova-venv/lib/python2.6/site-packages/eventlet/green/subprocess.py
|
|
||||||
2011-05-25
|
|
||||||
23:33:24.055602468 +0000
|
|
||||||
@@ -32,7 +32,7 @@
|
|
||||||
setattr(self, attr, wrapped_pipe)
|
|
||||||
__init__.__doc__ = subprocess_orig.Popen.__init__.__doc__
|
|
||||||
|
|
||||||
- def wait(self, check_interval=0.01):
|
|
||||||
+ def wait(self, check_interval=0.01, timeout=None):
|
|
||||||
# Instead of a blocking OS call, this version of wait() uses logic
|
|
||||||
# borrowed from the eventlet 0.2 processes.Process.wait() method.
|
|
||||||
try:
|
|
||||||
|
|
@ -15,10 +15,8 @@ module=lockutils
|
|||||||
module=log
|
module=log
|
||||||
module=network_utils
|
module=network_utils
|
||||||
module=notifier
|
module=notifier
|
||||||
module=patch_tox_venv
|
|
||||||
module=policy
|
module=policy
|
||||||
module=processutils
|
module=processutils
|
||||||
module=redhat-eventlet.patch
|
|
||||||
module=rootwrap
|
module=rootwrap
|
||||||
module=rpc
|
module=rpc
|
||||||
module=scheduler
|
module=scheduler
|
||||||
|
@ -3,7 +3,7 @@ amqplib>=0.6.1
|
|||||||
anyjson>=0.2.4
|
anyjson>=0.2.4
|
||||||
argparse
|
argparse
|
||||||
babel>=0.9.6
|
babel>=0.9.6
|
||||||
eventlet>=0.9.12
|
eventlet>=0.13.0
|
||||||
greenlet>=0.3.1
|
greenlet>=0.3.1
|
||||||
iso8601>=0.1.4
|
iso8601>=0.1.4
|
||||||
kombu>=1.0.4
|
kombu>=1.0.4
|
||||||
|
@ -65,7 +65,6 @@ def main(argv):
|
|||||||
install.check_dependencies()
|
install.check_dependencies()
|
||||||
install.create_virtualenv(no_site_packages=options.no_site_packages)
|
install.create_virtualenv(no_site_packages=options.no_site_packages)
|
||||||
install.install_dependencies()
|
install.install_dependencies()
|
||||||
install.post_process()
|
|
||||||
print_help()
|
print_help()
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
|
@ -126,9 +126,6 @@ class InstallVenv(object):
|
|||||||
self.pip_install('-r', self.pip_requires)
|
self.pip_install('-r', self.pip_requires)
|
||||||
self.pip_install('-r', self.test_requires)
|
self.pip_install('-r', self.test_requires)
|
||||||
|
|
||||||
def post_process(self):
|
|
||||||
self.get_distro().post_process()
|
|
||||||
|
|
||||||
def parse_args(self, argv):
|
def parse_args(self, argv):
|
||||||
"""Parses command-line arguments."""
|
"""Parses command-line arguments."""
|
||||||
parser = argparse.ArgumentParser()
|
parser = argparse.ArgumentParser()
|
||||||
@ -161,14 +158,6 @@ class Distro(InstallVenv):
|
|||||||
' requires virtualenv, please install it using your'
|
' requires virtualenv, please install it using your'
|
||||||
' favorite package management tool' % self.project)
|
' favorite package management tool' % self.project)
|
||||||
|
|
||||||
def post_process(self):
|
|
||||||
"""Any distribution-specific post-processing gets done here.
|
|
||||||
|
|
||||||
In particular, this is useful for applying patches to code inside
|
|
||||||
the venv.
|
|
||||||
"""
|
|
||||||
pass
|
|
||||||
|
|
||||||
|
|
||||||
class Fedora(Distro):
|
class Fedora(Distro):
|
||||||
"""This covers all Fedora-based distributions.
|
"""This covers all Fedora-based distributions.
|
||||||
@ -184,10 +173,6 @@ class Fedora(Distro):
|
|||||||
print "Attempting to install '%s' via yum" % pkg
|
print "Attempting to install '%s' via yum" % pkg
|
||||||
self.run_command(['sudo', 'yum', 'install', '-y', pkg], **kwargs)
|
self.run_command(['sudo', 'yum', 'install', '-y', pkg], **kwargs)
|
||||||
|
|
||||||
def apply_patch(self, originalfile, patchfile):
|
|
||||||
self.run_command(['patch', '-N', originalfile, patchfile],
|
|
||||||
check_exit_code=False)
|
|
||||||
|
|
||||||
def install_virtualenv(self):
|
def install_virtualenv(self):
|
||||||
if self.check_cmd('virtualenv'):
|
if self.check_cmd('virtualenv'):
|
||||||
return
|
return
|
||||||
@ -196,25 +181,3 @@ class Fedora(Distro):
|
|||||||
self.yum_install('python-virtualenv', check_exit_code=False)
|
self.yum_install('python-virtualenv', check_exit_code=False)
|
||||||
|
|
||||||
super(Fedora, self).install_virtualenv()
|
super(Fedora, self).install_virtualenv()
|
||||||
|
|
||||||
def post_process(self):
|
|
||||||
"""Workaround for a bug in eventlet.
|
|
||||||
|
|
||||||
This currently affects RHEL6.1, but the fix can safely be
|
|
||||||
applied to all RHEL and Fedora distributions.
|
|
||||||
|
|
||||||
This can be removed when the fix is applied upstream.
|
|
||||||
|
|
||||||
Nova: https://bugs.launchpad.net/nova/+bug/884915
|
|
||||||
Upstream: https://bitbucket.org/which_linden/eventlet/issue/89
|
|
||||||
"""
|
|
||||||
|
|
||||||
# Install "patch" program if it's not there
|
|
||||||
if not self.check_pkg('patch'):
|
|
||||||
self.yum_install('patch')
|
|
||||||
|
|
||||||
# Apply the eventlet patch
|
|
||||||
self.apply_patch(os.path.join(self.venv, 'lib', self.py_version,
|
|
||||||
'site-packages',
|
|
||||||
'eventlet/green/subprocess.py'),
|
|
||||||
'contrib/redhat-eventlet.patch')
|
|
||||||
|
@ -1,38 +0,0 @@
|
|||||||
# vim: tabstop=4 shiftwidth=4 softtabstop=4
|
|
||||||
|
|
||||||
# Copyright 2013 Red Hat, Inc.
|
|
||||||
#
|
|
||||||
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
||||||
# not use this file except in compliance with the License. You may obtain
|
|
||||||
# a copy of the License at
|
|
||||||
#
|
|
||||||
# http://www.apache.org/licenses/LICENSE-2.0
|
|
||||||
#
|
|
||||||
# Unless required by applicable law or agreed to in writing, software
|
|
||||||
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
||||||
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
||||||
# License for the specific language governing permissions and limitations
|
|
||||||
# under the License.
|
|
||||||
|
|
||||||
import os
|
|
||||||
import sys
|
|
||||||
|
|
||||||
import install_venv_common as install_venv
|
|
||||||
|
|
||||||
|
|
||||||
def main(argv):
|
|
||||||
root = os.path.dirname(os.path.dirname(os.path.realpath(__file__)))
|
|
||||||
|
|
||||||
venv = os.environ['VIRTUAL_ENV']
|
|
||||||
|
|
||||||
pip_requires = os.path.join(root, 'requirements.txt')
|
|
||||||
test_requires = os.path.join(root, 'test-requirements.txt')
|
|
||||||
py_version = "python%s.%s" % (sys.version_info[0], sys.version_info[1])
|
|
||||||
project = 'manila'
|
|
||||||
install = install_venv.InstallVenv(root, venv, pip_requires, test_requires,
|
|
||||||
py_version, project)
|
|
||||||
#NOTE(dprince): For Tox we only run post_process (which patches files, etc)
|
|
||||||
install.post_process()
|
|
||||||
|
|
||||||
if __name__ == '__main__':
|
|
||||||
main(sys.argv)
|
|
1
tox.ini
1
tox.ini
@ -16,7 +16,6 @@ setenv = VIRTUAL_ENV={envdir}
|
|||||||
deps = -r{toxinidir}/requirements.txt
|
deps = -r{toxinidir}/requirements.txt
|
||||||
-r{toxinidir}/test-requirements.txt
|
-r{toxinidir}/test-requirements.txt
|
||||||
commands =
|
commands =
|
||||||
python tools/patch_tox_venv.py
|
|
||||||
nosetests {posargs}
|
nosetests {posargs}
|
||||||
|
|
||||||
[testenv:pep8]
|
[testenv:pep8]
|
||||||
|
Loading…
Reference in New Issue
Block a user