On Fri, May 8, 2015 at 4:55 PM, Joao Eduardo Luis <joao@xxxxxxx> wrote: > All, > > While working on #11545 (mon: have mon-specific commands under 'ceph mon > ...') I crashed into a slightly tough brick wall. > > The purpose of #11545 is to move certain commands, such as 'ceph scrub', > 'ceph compact' and 'ceph sync force' to the 'mon' module of the ceph-tool. > > These commands have long stood in this format because 'mon'-module > commands have been traditionally considered as being somehow related > with monmaps and/or the MonmapMonitor. However, from a user > perspective, if they relate to the monitor itself (and not cluster-wide) > they should reside under the 'mon'-module. > > As such, I decided they should be moved to 'ceph mon scrub', 'ceph mon > compact' and 'ceph mon sync force'. > > Adding these commands and doing the correct mapping is not hard at all. > However, backward compatibility must be maintained, and simply dropping > the old style commands doesn't seem reasonable at all. > > Keeping the old style commands alongside with the new commands is > trivial enough to not pose a problem, but they must go away at some > point. After everyone get used to the new commands, and as soon as the > vast majority of deployments support the new commands, the old style > commands will simply be clutter. > > And while these commands are not widely used, and while most people > certainly have not ever needed to use them, this sort of thing can at > some point be required for any other (most commonly used) command. > > As I have not been able to find any mentions to guidelines to > deprecating commands, I thus propose the following: > > A command being DEPRECATED must be: > > - clearly marked as DEPRECATED in usage; > - kept around for at least 2 major releases; > - kept compatible for the duration of the deprecation period. > > Once two major releases go by, the command will then enter the OBSOLETE > period. This would be one major release, during which the command would > no longer work although still acknowledged. A simple message down the > lines of 'This command is now obsolete; please check the docs' would > suffice to inform the user. > > The command would no longer exist in the next major release. > > This approach gives a lifespan of roughly 3 releases (at current rate, > roughly 1.5 years) before being completely dropped. This should give > enough time to people to realize what has happened and adjust any > scripts they may have. > > E.g., a command being deprecated in Infernallis would be completely > dropped in the L-release, spanning its existence to at least one > long-term stable (i.e., jewel) and being dropped as soon as the first > dev cycle for the L-release begins. Well, this is an interesting dilemma. "As a user", I think I'd want it to be deprecated and warning me for one release I run — I'll see it when I upgrade and then know I need to fix it — and then it can be obsoleted in the next release I run. It's that "release I run" bit that's tricky though, right? I presume you made it two big releases because that should capture at least one release supported by downstream providers? But the release marking it as obsoleted will often not be captured by downstreams. So it seems like either we should do two releases in each state, or else we should as a community do one release in each state and if the downstreams want to keep command mappings around for longer they can do that. In general it's not like those are going to be tricky patches to apply... Also, I think this set of commands is sufficiently special-purpose that you could probably just make the swap and have the top-level ones spit out an error referring to the move; we don't necessarily need to make decisions on this right now. -Greg -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
