Bug 1396742 – [ceph-ansible] : rolling_update.yml - description/comment about 'serial' config variable is incomplete/misleading

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1396742 - [ceph-ansible] : rolling_update.yml - description/comment about 'serial' config variable is incomplete/misleading

Summary:	[ceph-ansible] : rolling_update.yml - description/comment about 'serial' conf...

Status:	CLOSED ERRATA

Aliases:	None

Product:	Red Hat Storage Console
Classification:	Red Hat
Component:	ceph-ansible (Show other bugs)
Sub Component:
Version:	2
Hardware:	x86_64 Linux

Priority	unspecified Severity medium
Target Milestone:	---
Target Release:	2
Assigned To:	leseb
QA Contact:	vidushi
Docs Contact:	Bara Ancincova

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:	1383917
	Show dependency tree / graph

Reported:	2016-11-19 10:15 EST by Rachana Patel
Modified:	2017-06-19 09:16 EDT (History)
CC List:	13 users (show)

See Also:
Fixed In Version:	ceph-ansible-2.1.9-1.el7scon
Doc Type:	Known Issue
Doc Text:	.The serial parameter must be set to 1 The `rolling_update.yml` Ansible playbook contains a comment about changing the value for the `serial` parameter to adjust the number of servers to be updated. However, upgrading many nodes in parallel can cause disruption to I/O operations. To avoid this problem, ensure that `serial` is set to `1`.
Story Points:	---
Clone Of:
Environment:
Last Closed:	2017-06-19 09:16:11 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2017:1496	normal	SHIPPED_LIVE	ceph-installer, ceph-ansible, and ceph-iscsi-ansible update	2017-06-19 13:14:02 EDT

Groups: None (edit)

Description Rachana Patel 2016-11-19 10:15:14 EST

Description of problem:
=======================
rolling_update.yml says

"# Change the value of 'serial:' to adjust the number of server to be updated."


AFAIK,
'serial' value indicates the number of nodes where the update happens together. So if we make 'serial' as 3  for MON, the update runs on 3 MONs at once and if we dont change it, it will update MON one after other.

Issue I have
1) 'serial' name itself is confusing, on setting that variable update happen in 'parallel' not in 'serial'

2) Description is misleading


"# Change the value of 'serial:' to adjust the number of server to be updated."


this might be interpreted as 'total no of server needs to be updated in cluster for that role'
e.g. how many MONs one wants update in cluster.

Adding completer description like, no of servers to be updated "togather" or "simultaneously" might make it less confusing

Additional info:

Comment 2 seb 2016-11-21 05:30:57 EST

Rachana, what do you think of this? https://github.com/ceph/ceph-ansible/pull/1109

Comment 5 seb 2016-11-21 08:33:08 EST

Thanks Harish, I added your suggestion.

Comment 6 Uday Boppana 2016-11-21 10:05:42 EST

PM is OK with adding this as a release note in 2.1 as noted by Harish in comment 3

Comment 9 seb 2016-11-21 11:03:00 EST

lgtm Bara thanks

Comment 11 seb 2016-11-30 10:06:22 EST

still lgtm :)

Comment 12 Ken Dreyer (Red Hat) 2017-03-03 12:23:31 EST

We think this is fixed in the latest builds currently undergoing testing
(ceph-ansible-2.1.9-1.el7scon as of this writing.) Rachana would you please retest with the latest builds?

Comment 17 errata-xmlrpc 2017-06-19 09:16:11 EDT

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:1496

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