Project

General

Profile

Actions

Bug #39406

closed

ceph_volume_client: d_name needs to be converted to string before using

Added by Rishabh Dave about 5 years ago. Updated almost 4 years ago.

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

0%

Source:
Development
Tags:
Backport:
nautilus,mimic,luminous
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
VolumeClient
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The d_name in DirEntry is obtained by ceph_volume_client.py as bytes array while it expects it to be a string. Make the conversion before using it. Originally this commit converted byte to string which was recently reversed by this commit.


Related issues 4 (0 open4 closed)

Has duplicate CephFS - Bug #39722: pybind: ceph_volume_client py3 errorDuplicateRishabh Dave

Actions
Copied to CephFS - Backport #40323: luminous: ceph_volume_client: d_name needs to be converted to string before usingRejectedActions
Copied to CephFS - Backport #40324: nautilus: ceph_volume_client: d_name needs to be converted to string before usingResolvedPatrick DonnellyActions
Copied to CephFS - Backport #40325: mimic: ceph_volume_client: d_name needs to be converted to string before usingRejectedNathan CutlerActions
Actions #1

Updated by Patrick Donnelly about 5 years ago

  • Target version set to v15.0.0
  • Start date deleted (04/23/2019)
  • Source set to Development
  • Backport set to nautilus
  • Component(FS) VolumeClient added
Actions #2

Updated by Patrick Donnelly about 5 years ago

Also seen in QA:

2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:Traceback (most recent call last):
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:  File "<string>", line 9, in <module>
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:  File "/usr/lib/python3/dist-packages/ceph_volume_client.py", line 506, in connect
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:    self._connect(premount_evict)
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:  File "/usr/lib/python3/dist-packages/ceph_volume_client.py", line 484, in _connect
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:    self.recover()
2019-04-17T22:25:04.761 INFO:teuthology.orchestra.run.smithi170.stderr:  File "/usr/lib/python3/dist-packages/ceph_volume_client.py", line 309, in recover
2019-04-17T22:25:04.762 INFO:teuthology.orchestra.run.smithi170.stderr:    match = re.search(regex, d.d_name)
2019-04-17T22:25:04.762 INFO:teuthology.orchestra.run.smithi170.stderr:  File "/usr/lib/python3.6/re.py", line 182, in search
2019-04-17T22:25:04.762 INFO:teuthology.orchestra.run.smithi170.stderr:    return _compile(pattern, flags).search(string)
2019-04-17T22:25:04.762 INFO:teuthology.orchestra.run.smithi170.stderr:TypeError: cannot use a string pattern on a bytes-like object

From: /ceph/teuthology-archive/pdonnell-2019-04-17_06:02:40-fs-wip-pdonnell-testing-20190417.032809-distro-basic-smithi/3857057/teuthology.log

Actions #3

Updated by Patrick Donnelly about 5 years ago

  • Backport deleted (nautilus)
Actions #4

Updated by Patrick Donnelly almost 5 years ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 27718
Actions #5

Updated by Patrick Donnelly almost 5 years ago

  • Has duplicate Bug #39722: pybind: ceph_volume_client py3 error added
Actions #6

Updated by Patrick Donnelly almost 5 years ago

  • Pull request ID changed from 27718 to 28196
Actions #7

Updated by Patrick Donnelly almost 5 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport set to nautilus,mimic,luminous
Actions #8

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40323: luminous: ceph_volume_client: d_name needs to be converted to string before using added
Actions #9

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40324: nautilus: ceph_volume_client: d_name needs to be converted to string before using added
Actions #10

Updated by Nathan Cutler almost 5 years ago

  • Copied to Backport #40325: mimic: ceph_volume_client: d_name needs to be converted to string before using added
Actions #11

Updated by Nathan Cutler almost 4 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".

Actions

Also available in: Atom PDF