Project

General

Profile

Feature #4130

ceph-deploy: be polite about pushing ceph.conf

Added by Sage Weil about 11 years ago. Updated about 11 years ago.

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

0%

Source:
Development
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

If it is not present, push it. If it is the same, do nothing. If it is different, print a nice warning for the user, and provide an option/command (pushconf? --push-conf? both?) to replace the remote copy.

History

#1 Updated by Sage Weil about 11 years ago

  • Description updated (diff)

#2 Updated by Sage Weil about 11 years ago

  • Category set to ceph-deploy

#3 Updated by Sage Weil about 11 years ago

  • Target version set to v0.60

#4 Updated by Anonymous about 11 years ago

  • Assignee set to Anonymous

#5 Updated by Sage Weil about 11 years ago

  • Status changed from New to Fix Under Review

see wip-conf in ceph-deploy.git

#6 Updated by Anonymous about 11 years ago

Looks good. The only thing that I had a question about is that if someone is configuring multiple services on one node, they will need to use the override flag ?

#7 Updated by Sage Weil about 11 years ago

Because teh second pass/item would want to write the file? Nope. It only throws an error if the conf file exists AND has different content..

#8 Updated by Sage Weil about 11 years ago

  • Status changed from Fix Under Review to Resolved

Also available in: Atom PDF