Bug 1458224 - Improve check PGS when OSD daemons are restarted
Summary: Improve check PGS when OSD daemons are restarted
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: Ceph-Ansible
Version: 3.0
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: rc
: 3.0
Assignee: Sébastien Han
QA Contact: Manohar Murthy
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-06-02 10:38 UTC by Guillaume Abrioux
Modified: 2017-12-05 23:33 UTC (History)
10 users (show)

Fixed In Version: RHEL: ceph-ansible-2.3.0-0.1.rc5.el7cp Ubuntu: ceph-ansible_3.0.0~rc3-2redhat1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-12-05 23:33:43 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github ceph ceph-ansible pull 1580 0 None None None 2017-06-02 20:03:33 UTC
Github ceph ceph-ansible pull 1584 0 None None None 2017-06-12 21:59:12 UTC
Red Hat Product Errata RHBA-2017:3387 0 normal SHIPPED_LIVE Red Hat Ceph Storage 3.0 bug fix and enhancement update 2017-12-06 03:03:45 UTC

Description Guillaume Abrioux 2017-06-02 10:38:44 UTC
Description of problem:

The check of PGs is not really efficient because the current check might be satisfied as long as 1 PG is active+clean.


How reproducible:

The check occurs when the handler are triggered after any modification has been brought to ceph.conf.


Steps to Reproduce:
1. Deploy a new cluster
2. Make a change 
3. run the installer again so the daemons are restarted (OSD in this case)

Actual results:

When the OSD daemons are restarted, the check will be satisfied as long as the pattern 'active+clean' is found in the ceph status output.

Expected results:

The check is satisfied only if the total number of PGs are in 'active+clean' state.


fix: https://github.com/ceph/ceph-ansible/pull/1580
backport: https://github.com/ceph/ceph-ansible/pull/1581

Comment 3 seb 2017-06-12 10:04:35 UTC
We have this new approach new: https://github.com/ceph/ceph-ansible/pull/1584

Comment 5 Ken Dreyer (Red Hat) 2017-07-06 11:21:48 UTC
The PRs are merged upstream to master, so this will be fixed in the upcoming ceph-ansible v2.3.0.

Comment 10 errata-xmlrpc 2017-12-05 23:33:43 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:3387


Note You need to log in before you can comment on or make changes to this bug.