[c-nsp] "reload" command doesn't check command line parameters

[Pete Lumbis]

The two outputs do have different warnings:

reload reason:
===========================
Router#reload
Proceed with reload? [confirm]
===========================

===========================
Router#reload in 5
Reload scheduled in 5 minutes by console
Reload reason: Reload Command
Proceed with reload? [confirm]
===========================



On Mon, Oct 7, 2013 at 11:46 AM, Octavio Alvarez
<alvarezp at alvarezp.ods.org>wrote:

> On 10/07/2013 05:30 AM, Pete Lumbis wrote:
> > If we fix the behavior what does the fix look like? Do we not allow any
> > reason that starts with "i"(in) "c" (cancel) or "a"(at)? But then what if
> > you want a reload reason of "reload installing new software"? Should this
> > be blocked?
>
> Create "reload reason blahblah" and deprecate "reload blahblah". Issue a
> warning each time "reload blahblah" happens.
>
> Also have different confirmation messages. "Reload in 10" could have
> "Proceed with reload in 10?" while the other could be "Proceed with
> immediate reload?"
>
>