On 11/29/2012 07:01 PM, Andrey Korolyov wrote: > On Thu, Nov 29, 2012 at 8:34 PM, Sage Weil <sage@xxxxxxxxxxx> wrote: >> On Thu, 29 Nov 2012, Andrey Korolyov wrote: >>> $ ceph osd down - >>> osd.0 is already down >>> $ ceph osd down --- >>> osd.0 is already down >>> >>> the same for ``+'', ``/'', ``%'' and so - I think that for osd subsys >>> ceph cli should explicitly work only with positive integers plus zero, >>> refusing all other input. >> >> which branch is this? this parsing is cleaned u pin the latest >> next/master. >> >> > > It was produced by 0.54-tag. I have built > dd3a24a647d0b0f1153cf1b102ed1f51d51be2f2 today and problem has > gone(except parsing ``-0'' as 0 and 00000/0000001 as 0 and 1 > correspondingly). A fix for the signed parameter has been pushed to next. However, after consideration, when it comes to the '0+\d' parameters, that kind of input was considered valid; Greg put it best on IRC, and I quote: <gregaf> joao: not sure we want to prevent "01" from parsing as 1, I suspect some people with large clusters will find that useful so they can conflate the name and ID while keeping everything three digits Hope this makes sense to you. -Joao -- 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
