On Tue, Oct 4, 2016 at 2:28 PM, Willem Jan Withagen <wjw@xxxxxxxxxxx> wrote: > On 4-10-2016 14:16, John Spray wrote: >> On Tue, Oct 4, 2016 at 1:03 PM, Willem Jan Withagen <wjw@xxxxxxxxxxx> wrote: >>> On 4-10-2016 08:07, John Spray wrote: >>>> As Dan says, `ceph tell` is the new one, the one that sends commands >>>> directly from the CLI to the named service (not via the mon). >>>> However, this is a bit of a placeholder, as we'll eventually be >>>> switching to some commands (especially PG stats ones) to silently go >>>> to the mgr instead of the mon. There are various ways of >>>> accomplishing that, the simplest one is to have the CLI fetch both mgr >>>> and mon command descriptions, and preferentially send commands to >>>> matching entries from the mgr. >>> >>> ATM I had to disable building MGR, because Clang/Python/FreeBSD throw a >>> fit when having to compile on of the python files... >>> Something to do with function-signatures not matching. >>> I'll look into it after I've completed most of my other blocking >>> problems. Unless you'd really want to have the compiler errors now, I'll >>> fireup a build with MGR on. >> >> Ah... yeah, I do seem to recall some places the compiler warned about >> (perfectly safe) const char* casts. I'll look into cleaning that up, >> assuming it's what's making clang unhappy. > > Clang is a lot more picky at times... But usually I get away with a > warning, this time it throws errors. > Can you try this please? https://github.com/ceph/ceph/pull/11307 John >>> Just hope that this switch is not enforced before that... >>> And I'd then expect the WITH_MGR flag to go away? >> >> Yeah, I think it would be reasonable to remove it some time before >> Luminous, or whenever we move essential user-facing stuff (like pg >> dump commands) into mgr so that it ceases to be optional. > > Right, hope that buys me enough time. :) > > --WjW > -- 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
