Project

General

Profile

Bug #48408

pybind: test_readlink() fails due to missing terminating NULL char

Added by Tuan Hoang over 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
octopus,nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Affected Nautilus and Octopus.

...
test_cephfs.test_readlink ... FAIL

======================================================================
FAIL: test_cephfs.test_readlink
----------------------------------------------------------------------
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/nose/case.py", line 197, in runTest
self.test(*self.arg)
File "/home/tmhoang/test_cephfs.py", line 99, in test_readlink
assert_equal(d, b"/file-1")
AssertionError: b'/file-1\xe8' != b'/file-1'


Related issues

Copied to Ceph - Backport #48518: nautilus: pybind: test_readlink() fails due to missing terminating NULL char Resolved
Copied to Ceph - Backport #48519: octopus: pybind: test_readlink() fails due to missing terminating NULL char Resolved

History

#1 Updated by Patrick Donnelly over 3 years ago

  • Status changed from New to Fix Under Review
  • Assignee set to Tuan Hoang
  • Target version changed from v15.2.7 to v16.0.0
  • Source set to Community (dev)
  • Backport set to octopus,nautilus
  • Pull request ID set to 38367
  • Affected Versions deleted (v15.2.7)

#2 Updated by Patrick Donnelly over 3 years ago

  • Status changed from Fix Under Review to Pending Backport

#3 Updated by Backport Bot over 3 years ago

  • Copied to Backport #48518: nautilus: pybind: test_readlink() fails due to missing terminating NULL char added

#4 Updated by Backport Bot over 3 years ago

  • Copied to Backport #48519: octopus: pybind: test_readlink() fails due to missing terminating NULL char added

#5 Updated by Nathan Cutler over 3 years ago

Nice work, @Backport Bot!

#6 Updated by Tuan Hoang about 3 years ago

Do I have to backport this or it could be done by ceph team or automatically by Backport Bot ?

#7 Updated by Nathan Cutler about 3 years ago

Tuan Hoang wrote:

Do I have to backport this or it could be done by ceph team or automatically by Backport Bot ?

Hi. I went ahead and staged the backports. If you're interested in helping with backports, we have some documentation describing the procedures that need to be followed and tools that make the process much less painful than when attempting to follow it manually:

https://github.com/ceph/ceph/blob/master/SubmittingPatches-backports.rst

#8 Updated by Nathan Cutler about 3 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Also available in: Atom PDF