Project

General

Profile

Actions

Bug #30630

closed

clarification/removal needed: docs recommend upgrading with authentication disabled

Added by Joao Eduardo Luis over 5 years ago. Updated over 3 years ago.

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

0%

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

Description

This [1] is a bit that was initially included in 2013, and has been moved around since then. It states

Important

When upgrading, we recommend expressly disabling authentication first, then perform the upgrade. Once the upgrade is complete, re-enable authentication.

I find it weird that we are recommending disabling authentication during an upgrade.

IIRC, I haven't seen anyone explicitly disabling authentication during an upgrade for years, and it would be nice to do one of two things:

1. remove this notice; or,
2. explain in which circumstances this would be advised.

Definitely drop the "recommend" keyword though. I don't think we should be recommending any of that :)

[1] http://docs.ceph.com/docs/master/rados/configuration/common/#authentication

Actions #1

Updated by Joao Eduardo Luis over 5 years ago

  • Description updated (diff)
Actions #2

Updated by Joao Eduardo Luis over 5 years ago

  • Assignee set to Joao Eduardo Luis
Actions #3

Updated by Joao Eduardo Luis over 3 years ago

  • Status changed from New to Need More Info
  • Priority changed from Normal to Urgent
Actions #4

Updated by Joao Eduardo Luis over 3 years ago

  • Assignee changed from Joao Eduardo Luis to Zac Dover
  • Priority changed from Urgent to Normal
Actions #5

Updated by Joao Eduardo Luis over 3 years ago

  • Status changed from Need More Info to New
Actions #6

Updated by Zac Dover over 3 years ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 36478
Actions #7

Updated by Zac Dover over 3 years ago

  • Status changed from Fix Under Review to Resolved

This has been merged.

I jumped the gun again and made a poorly-formatted merge request, but Kefu was there to rap my knuckles and strengthen my resolve to get it right.

Actions

Also available in: Atom PDF