Project

General

Profile

Actions

Bug #37356

closed

ceph-volume lvm batch broken on py3 environments

Added by Jan Fajerski over 5 years ago. Updated about 5 years ago.

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

0%

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

Description

[2018-11-21 08:03:15,643][ceph_volume][ERROR ] exception caught by decorator
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/ceph_volume/decorators.py", line 59, in newfunc
    return f(*a, **kw)
  File "/usr/lib/python3.6/site-packages/ceph_volume/main.py", line 156, in main
    terminal.dispatch(self.mapper, subcommand_args)
  File "/usr/lib/python3.6/site-packages/ceph_volume/terminal.py", line 182, in dispatch
    instance.main()
  File "/usr/lib/python3.6/site-packages/ceph_volume/devices/lvm/main.py", line 40, in main
    terminal.dispatch(self.mapper, self.argv)
  File "/usr/lib/python3.6/site-packages/ceph_volume/terminal.py", line 182, in dispatch
    instance.main()
  File "/usr/lib/python3.6/site-packages/ceph_volume/decorators.py", line 16, in is_root
    return func(*a, **kw)
  File "/usr/lib/python3.6/site-packages/ceph_volume/devices/lvm/batch.py", line 155, in main
    description=self.print_help(),
  File "/usr/lib/python3.6/site-packages/ceph_volume/devices/lvm/batch.py", line 127, in print_help
    detected_devices=self.get_devices(),
  File "/usr/lib/python3.6/site-packages/ceph_volume/devices/lvm/batch.py", line 119, in get_devices
    for device, detail in all_devices.items():
RuntimeError: dictionary changed size during iteration

In py3 one shouldn't alter a dict one is currently iterating over. Best avoided in py2 as well.

Actions #2

Updated by Alfredo Deza over 5 years ago

  • Description updated (diff)
Actions #3

Updated by Jan Fajerski over 5 years ago

  • Description updated (diff)
Actions #4

Updated by Jan Fajerski over 5 years ago

  • Status changed from New to Resolved
Actions #5

Updated by Alfredo Deza over 5 years ago

  • Status changed from Resolved to In Progress

setting back to in progress because it needs to be backported still

Actions #7

Updated by Jan Fajerski about 5 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF