Project

General

Profile

Bug #18764

Crash on missing 'ceph_version' in daemon metadata

Added by John Spray 4 months ago. Updated 2 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
-
Start date:
02/01/2017
Due date:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Release:
Needs Doc:
No

Description

The get_servers() implementation uses a std::map::at call to get the version: when you run against some older daemons the field isn't there and the manager will crash out when a python module calls get_servers().

Associated revisions

Revision fe9d54ac
Added by Sage Weil 2 months ago

Merge pull request #14129 from SUSE/wip-18764

mgr: fix crash on missing 'ceph_version' in daemon metadata (fixes #18764)

Reviewed-by: Sage Weil <>

History

#1 Updated by John Spray 3 months ago

n.b. it's PyModules::dump_server that actually does the naughty .at() call.

#2 Updated by John Spray 3 months ago

  • Priority changed from Normal to High

#3 Updated by Tim Serong 2 months ago

  • Assignee set to Tim Serong

I'll take this.

#4 Updated by Tim Serong 2 months ago

  • Status changed from New to In Progress

#5 Updated by Tim Serong 2 months ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF