Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pybind/rados: add missing "length" requires for aio_execute() #12439

Merged
merged 1 commit into from Dec 12, 2016

Conversation

tchaikov
Copy link
Contributor

Signed-off-by: Kefu Chai kchai@redhat.com

Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov
Copy link
Contributor Author

2016-12-10T02:47:57.131 INFO:tasks.workunit.client.0.mira092.stderr:======================================================================
2016-12-10T02:47:57.131 INFO:tasks.workunit.client.0.mira092.stderr:ERROR: test_rados.TestIoctx.test_aio_execute
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:----------------------------------------------------------------------
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:Traceback (most recent call last):
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:  File "/usr/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:    self.test(*self.arg)
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:  File "/home/ubuntu/cephtest/mnt.0/client.0/tmp/test_rados.py", line 840, in test_aio_execute
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:    comp = self.ioctx.aio_execute("foo", "hello", "say_hello", b"", 32, cb, cb)
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:  File "rados.pyx", line 450, in rados.requires.wrapper.validate_func (/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/x86_64/AVAILABLE_ARCH/x86_64/AVAILABLE_DIST/centos7/DIST/centos7/MACHINE_SIZE/huge/release/11.0.2-2392-g10834f2/rpm/el7/BUILD/ceph-11.0.2-2392-g10834f2/build/src/pybind/rados/pyrex/rados.c:4203)
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:  File "rados.pyx", line 433, in rados.requires.check_type (/home/jenkins-build/build/workspace/ceph-dev-new-build/ARCH/x86_64/AVAILABLE_ARCH/x86_64/AVAILABLE_DIST/centos7/DIST/centos7/MACHINE_SIZE/huge/release/11.0.2-2392-g10834f2/rpm/el7/BUILD/ceph-11.0.2-2392-g10834f2/build/src/pybind/rados/pyrex/rados.c:3565)
2016-12-10T02:47:57.132 INFO:tasks.workunit.client.0.mira092.stderr:TypeError: oncomplete must be Callable or None

see http://pulpito.ceph.com/kchai-2016-12-10_02:13:38-rados-wip-kefu-testing---basic-mira/621255/

Copy link
Member

@jdurgin jdurgin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@tchaikov tchaikov merged commit 1b533f8 into ceph:master Dec 12, 2016
@tchaikov tchaikov added this to the kraken milestone Dec 12, 2016
@tchaikov tchaikov deleted the wip-pybind-requires branch December 12, 2016 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants