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

kraken: cephfs: src/test/pybind/test_cephfs.py fails #16114

Merged
merged 2 commits into from Jul 19, 2017

Conversation

smithfarm
Copy link
Contributor

(O_WRONLY | O_RDWR) is invaild open flags

Fixes: http://tracker.ceph.com/issues/19890
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit 2c25c99)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "cephfs.pyx", line 672, in cephfs.LibCephFS.open (/home/zhyan/Ceph/ceph-2/build/src/pybind/cephfs/pyrex/cephfs.c:10160)
  File "cephfs.pyx", line 155, in cephfs.OSError.__init__ (/home/zhyan/Ceph/ceph-2/build/src/pybind/cephfs/pyrex/cephfs.c:1889)
TypeError: __init__() takes exactly 3 positional arguments (2 given)

Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit e6493f6)
@smithfarm smithfarm self-assigned this Jul 4, 2017
@smithfarm smithfarm added this to the kraken milestone Jul 4, 2017
@smithfarm smithfarm added bug-fix core cephfs Ceph File System tests and removed core labels Jul 4, 2017
@smithfarm smithfarm changed the title kraken: src/test/pybind/test_cephfs.py fails kraken: cephfs: src/test/pybind/test_cephfs.py fails Jul 4, 2017
@smithfarm
Copy link
Contributor Author

Jenkins re-test this please

@smithfarm
Copy link
Contributor Author

@jcsp This PR passed an fs run at http://tracker.ceph.com/issues/19009#note-57 (there were some failures that required this PR to fix - applied and passed on rerun). Please review/approve/merge the PR if you agree it can be merged.

@smithfarm
Copy link
Contributor Author

@batrick @ukernel This PR passed another fs suite at http://tracker.ceph.com/issues/19009#note-67

Please review.

@smithfarm smithfarm merged commit 0a9badb into ceph:kraken Jul 19, 2017
@smithfarm smithfarm deleted the wip-20500-kraken branch July 19, 2017 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants