[sos-devel] [PATCH v2] sosreport: Accept commas, period and alphas in the case id

Aruna Balakrishnaiah aruna at linux.vnet.ibm.com
Fri Sep 5 04:36:58 UTC 2014


On Wednesday 20 August 2014 02:16 PM, Bryn M. Reeves wrote:
> On Wed, Aug 20, 2014 at 01:43:22PM +0530, Aruna Balakrishnaiah wrote:
>>      If case_id is not the appropriate name, please suggest me
>> the name I could use.
> Thanks for taking the time to rework the patch and add the documentation
> changes.
>
> I'm actually happy with that name, although as I said on GitHub, I think it
> may make sense to support multiple synonyms for this attribute - they all
> ultimately mean the same thing and it would mean that users of a particular
> vendor's products can use the term that the vendor itself uses (or at least
> something close). Allowing synonyms is a lot simpler to implement than
> creating a mechanism to allow the policy to control the names of command
> line options (which was the other option I'd considered).
>
> If there seems to be agreement from the other main developers I'll aim to
> get this merged today.

Bryn,

Any updates on this?

Regards,
Aruna

>
> Regards,
> Bryn.
>
>>> -        parser.add_option("--ticket-number", action="store",
>>> -                             dest="ticket_number",
>>> -                             help="specify ticket number")
>>> +        parser.add_option("--case-id", action="store",
>>> +                             dest="case_id",
>>> +                             help="specify case id")
>>>           parser.add_option("--name", action="store",
>>>                                dest="customer_name",
>>>                                help="specify report name")




More information about the sos-devel mailing list