[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